Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Checkin #5

Open
wants to merge 18 commits into
base: new_delphipot
Choose a base branch
from
Open

Checkin #5

wants to merge 18 commits into from

Conversation

enryH
Copy link
Contributor

@enryH enryH commented Jul 31, 2021

@MendesMarcus I updated the delphipot.js script with a switch and some help text.

Could you update the script further from this latest version?

Henry added 13 commits July 29, 2021 11:47
- avoid key-errors due to change of loop units
- changed input (to be committed in a next step)
- plot is still gibberish
dict with
 - key as comb. of pos. and resdidual
 - value: surface potential
combining the atoms pot to residues
- similarity DataFrame. Features are similarites/distances
- normalization can be added now vectorized
- only unique distances/similiarities, no self-correlation
- old data file processing used
- new one has additional checks
- notes after meeting
- start in this repository is provided in
list_pdb_files
- get potentials for further processing (step 1)
  in workflow described in bin/README.md
- produce either residual or surface potentials
- add more "help" to delphipot.js script
- add surface potential creation to snakemake

- idea: best would be to adapt the js.script to write to disk to avoid
  repeated downloads and invocations of the node script...
@enryH enryH requested a review from MendesMarcus July 31, 2021 16:19
Henry added 3 commits July 31, 2021 18:30
- get feedback from Mahita on logic
- think about output format
- create dictionary view of data
- add analysis for number of regions available
- select 180 AGs
- enlarge plot
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants