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

Uploading dataset pbtxt for Shields Doyle Paper #51

Closed
wants to merge 1 commit into from
Closed

Uploading dataset pbtxt for Shields Doyle Paper #51

wants to merge 1 commit into from

Conversation

Jesus00-1
Copy link
Contributor

Pbtxt files were created using the interactive web editor. Reactions are from the paper https://doi.org/10.1038/s41586-021-03213-y.
Template reactions pbtxt and csv files are attached.

Note 0% Yields did not show up in the schema when checking for consistency.

This is an enumerated dataset of 1728 reactions from imidazole arylation for figure S42.

Shields_arylation_figS42.zip

This is an enumerated dataset of 256 reactions from imidazole arylation for figure S45 (Extended ligand set for testing model out of sample prediction performance)

experiment_index_extended_dataset.zip

Note 0% Yields did not show up in the schema when checking for consistency.
Imidazole arylation for figure S42.
https://doi.org/10.1038/s41586-021-03213-y
@Jesus00-1
Copy link
Contributor Author

The extended ligand set was not labeled. It is attached with a label here.

experiment_index_extended_dataset_labeled.zip

@skearnes
Copy link
Contributor

skearnes commented Feb 5, 2021

Hi @Jesus00-1, thanks for the PR and the bug report. I've updated the web editor with open-reaction-database/ord-editor#81; would you mind re-adding the zero yield values to the submission? They should be preserved now.

@Jesus00-1
Copy link
Contributor Author

Hi,

They are attached. Let me know if there is anything else I need to do on my end.

Figure S42 and the extended dataset (S45)

experiment_index_S42.zip

experiment_index_extended.zip

Thank you!

@skearnes
Copy link
Contributor

skearnes commented Feb 8, 2021

Oh, sorry, would you mind committing the updated files so they can be merged in this PR?

@skearnes
Copy link
Contributor

skearnes commented Feb 8, 2021

Oh, sorry, would you mind committing the updated files so they can be merged in this PR?

(If it's easier, feel free to a create a new PR with the updated pbtxt.)

@Jesus00-1 Jesus00-1 closed this Feb 8, 2021
This was referenced Feb 8, 2021
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