Skip to content

Per-adsorbate filtering of data sets#48

Merged
vict0rsch merged 13 commits intomainfrom
adsorbates-filter
Nov 13, 2023
Merged

Per-adsorbate filtering of data sets#48
vict0rsch merged 13 commits intomainfrom
adsorbates-filter

Conversation

@vict0rsch
Copy link
Collaborator

@vict0rsch vict0rsch commented Oct 31, 2023 •
edited
Loading

Create new option to filter data sets (train and vals) per adsorbate by modifying adsorbates as a list or a string:

# is2re.yaml
[...]
adsorbates: '*O,*OH2'
# is2re.yaml
[...]
adsorbates: '*O'
# is2re.yaml
[...]
adsorbates: ['*O', '*OH2']
# is2re.yaml
[...]
adsorbates:
  - '*O'
  - '*OH2'

vict0rsch and others added 4 commits June 21, 2023 12:26
AlexDuvalinho
AlexDuvalinho approved these changes Nov 9, 2023
View reviewed changes
Copy link
Collaborator

@AlexDuvalinho AlexDuvalinho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good now

@vict0rsch
Copy link
Collaborator Author

vict0rsch commented Nov 13, 2023

I resolved conflicts in gnn_dev.py by keeping only the incoming changes (i.e. this branch)

@vict0rsch vict0rsch merged commit 2126dca into main Nov 13, 2023
vict0rsch added a commit that referenced this pull request Nov 29, 2023
@vict0rsch
import changes from #48
b4ba938
AlexDuvalinho added a commit that referenced this pull request Jan 6, 2024
@AlexDuvalinho
Merge pull request #53 from RolnickLab/disconnected-per-ads-datasets
7420ab5 
Disconnected per adsorbate dataset: import changes from #48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@AlexDuvalinho AlexDuvalinho AlexDuvalinho approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@vict0rsch @AlexDuvalinho