psychonautwiki / bifrost

The PsychonautWiki API.
MIT License
48 stars 10 forks source link

Add 'systematic_name', queries fallback on 'common_name' #60

Open Im-0xea opened 1 month ago

Im-0xea commented 1 month ago

Hyia!

I recently added the 'systematic_name' property on the wiki so that the API should be able to provide the preferred IUPAC name once appropriate tagging is done, which I would gladly be responsible for.

besides that I did some minor fixes and made queries fall back to common_names if no page exists, so that with slighly more processing broad searches can cover alot more ground.

Let me know what you think!

Lots of love, Xea

Im-0xea commented 1 month ago

20240516_03h45m15s_grim

19h commented 1 month ago

This is a breaking change and very likely breaks our api consumers, which isn't exactly good. What's the problem with chemical class et al.?

Im-0xea commented 1 month ago

@19h I think the formating of the diff made you missunderstand my changes

there is no problem with "chemical class", I have not touched "chemical class", I removed a duplicate function that I found, and added a new property for systematic IUPAC names, that shouldn't be a breaking change

otherwise I added a fallback to the query function to cover more ground, I hope that doesn't disrupt anything, if so I would just revert that commit

19h commented 1 month ago

CleanShot 2024-05-28 at 19 48 54@2x

This looks a bit more than just formatting right?

Im-0xea commented 1 month ago

@19h please look at the full file, I overrode a duplicate function for formating "common_name", if you look just a bit above the snippet you just send me you will see that the "chemical_class" function is still there

alevkov commented 1 month ago

just rewrite in rust