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

update the 750 AstraZeneca ELN pbtxt file #122

Merged
merged 18 commits into from
Oct 22, 2021
Merged

update the 750 AstraZeneca ELN pbtxt file #122

merged 18 commits into from
Oct 22, 2021

Conversation

bznan
Copy link
Contributor

@bznan bznan commented Oct 21, 2021

Corrected the ligand amount of the previous dataset.

bznan added 18 commits October 13, 2021 11:34
one ligand smiles missing
The .pbtxt file generated from 750 reactions from AstraZeneca ELN.
One Smiles missing
Solving one missing catalyst smiles string.
A pbtxt file includes 750 reactions generated from AstraZeneca ELN
A pbtxt file includes 750 reactions generated from AZ ELN
The spreadsheet for 750 reactions generated from AZ ELN
This pbtxtx file includes 750 Buchwald_Hartwig reactions generated from AstraZeneca ELN. New features have been updated including reaction type, index, created date, dataset description, experimental Note.
correct ligand amount information of the original AZ_dataset.pbtxt file.
@connorcoley
Copy link
Contributor

It looks like many of the examples still have 1:1 metal:ligand, but not all. Are you confident in these values now? Are there any other additives/variations that might be missing from the spreadsheet?

2 similar comments
@connorcoley
Copy link
Contributor

It looks like many of the examples still have 1:1 metal:ligand, but not all. Are you confident in these values now? Are there any other additives/variations that might be missing from the spreadsheet?

@connorcoley
Copy link
Contributor

It looks like many of the examples still have 1:1 metal:ligand, but not all. Are you confident in these values now? Are there any other additives/variations that might be missing from the spreadsheet?

@connorcoley connorcoley changed the base branch from main to #122 October 22, 2021 18:27
@bznan
Copy link
Contributor Author

bznan commented Oct 22, 2021

Yes, I am confident in these values, because they are updated from one extra amount column I added generated from ELN. And about 20 missing values are added corresponding to the procedure details by hand. Some of the amounts are indeed the same.

@connorcoley connorcoley reopened this Oct 22, 2021
@connorcoley
Copy link
Contributor

Great, thanks! If the tests pass, I'll merge it in

@connorcoley connorcoley merged commit bb8ba65 into open-reaction-database:#122 Oct 22, 2021
connorcoley added a commit that referenced this pull request Oct 23, 2021
* update the 750 AstraZeneca ELN pbtxt file (#122)

* uploading Dataset pbtxt for AstraZeneca dataset

* Delete AZ_dataset.pbtxt

one ligand smiles missing

* AZ dataset

The .pbtxt file generated from 750 reactions from AstraZeneca ELN.

* Delete AZ_dataset.pbtxt

One Smiles missing

* updated AZ dataset pbtxt file

Solving one missing catalyst smiles string.

* Delete AZ_dataset.pbtxt

* upload AstraZeneca dataset pbtxt file

A pbtxt file includes 750 reactions generated from AstraZeneca ELN

* Delete AZ_dataset.pbtxt

* uploading AZ dataset pbtxt file 

A pbtxt file includes 750 reactions generated from AZ ELN

* upload data spreadsheet for AZ ELN

The spreadsheet for 750 reactions generated from AZ ELN

* Reaction Template using web submission

* reaction index modified

* updated AZ_ELN pbtxt file

This pbtxtx file includes 750 Buchwald_Hartwig reactions generated from AstraZeneca ELN. New features have been updated including reaction type, index, created date, dataset description, experimental Note.

* Delete AZ_ELN.pbtxt

* Delete AZ_dataset.xlsx

* Change the time stamp errors

* Delete AZ_dataset.pbtxt

* update AZ_dataset.pbtxt

correct ligand amount information of the original AZ_dataset.pbtxt file.

* Update submission

* Update badges

Co-authored-by: Bozhao Nan <57164369+bznan@users.noreply.github.com>
Co-authored-by: github-actions <github-actions@github.com>
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