Skip to content

Jenke/updates#235

Merged
JenkeScheen merged 17 commits into
mainfrom
jenke/updates
Jan 27, 2026
Merged

Jenke/updates#235
JenkeScheen merged 17 commits into
mainfrom
jenke/updates

Conversation

@JenkeScheen

@JenkeScheen JenkeScheen commented Jan 26, 2026

Copy link
Copy Markdown
Contributor

Adds:

@JenkeScheen

Copy link
Copy Markdown
Contributor Author

Ready for a quick review @jchodera

@JenkeScheen JenkeScheen requested a review from jchodera January 26, 2026 15:58
@jchodera

Copy link
Copy Markdown
Collaborator

I can confirm that this renders locally, and love the new ChEMBL datasets!

For grant reporting purposes, we need the actual deposition and/or release dates of the ChEMBL depositions---the year isn't sufficient because it isn't clear if this falls within the reporting period. Is it possible to get at least one of deposition_date and/or release_date in each entry, with standard YYYY-MM-DD date?

@JenkeScheen

Copy link
Copy Markdown
Contributor Author

Luckily that was present in the ChEMBL export from Nick so I added in that change. I'll merge this later today

@JenkeScheen JenkeScheen merged commit 0302d56 into main Jan 27, 2026
1 check passed
@JenkeScheen JenkeScheen deleted the jenke/updates branch January 27, 2026 14:45
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