Skip to content

AOP-tk wrappers for download, chemical identification + matching + normalization, PDF parsing, relationship finding#728

Merged
hechth merged 177 commits into
RECETOX:masterfrom
rdurnik:new_wrappers
Jun 18, 2026
Merged

AOP-tk wrappers for download, chemical identification + matching + normalization, PDF parsing, relationship finding#728
hechth merged 177 commits into
RECETOX:masterfrom
rdurnik:new_wrappers

Conversation

@rdurnik

@rdurnik rdurnik commented Jan 29, 2026

Copy link
Copy Markdown
Contributor

AOP-tk wrappers for download, chemical identification + matching + normalization, PDF parsing, relationship finding

@rdurnik rdurnik requested a review from hechth January 29, 2026 09:55
@rdurnik rdurnik changed the title AOP-tk wrappers for download, chemical identification + matching, PDF parsing, relationship finding AOP-tk wrappers for download, chemical identification + matching + normalization, PDF parsing, relationship finding Jan 29, 2026
Comment thread tools/aoptk/aoptk_chemical_identification.py Outdated
@hechth

hechth commented Jun 17, 2026

Copy link
Copy Markdown
Member

@arash77 this tool uses credentials to save and supply an openai api key and endpoint - how would I go about testing this, do you have any suggestions? I'd be really grateful!

@arash77

arash77 commented Jun 17, 2026

Copy link
Copy Markdown

@arash77 this tool uses credentials to save and supply an openai api key and endpoint - how would I go about testing this, do you have any suggestions? I'd be really grateful!

Unfortunately, there isn't any way to test it in the GitHub environment with real credentials for now, so you will have to either test it locally or simulate a failure.

@hechth

hechth commented Jun 18, 2026

Copy link
Copy Markdown
Member

thanks @arash77 for the quick reply - then we will go with failure for now until we have some better options I think.

@hechth hechth merged commit 197f0f4 into RECETOX:master Jun 18, 2026
13 checks passed
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.

4 participants