-
Notifications
You must be signed in to change notification settings - Fork 10
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
Add support for recent versions of MySQL.jl #74
Comments
Happy to accept pull requests. Unfortunately this package does not have an active maintainer. |
But if people want to make pull requests, I can merge them. |
This is a cross-post from https://julialang.zulipchat.com/#narrow/stream/232791-sciences/topic/PubChemCrawler/near/216013362. If I were to create a package to search PubChem,
|
I think JuliaHealth would be a great place to host it!
This question is trickier. The problem is that BioMedQuery.jl currently doesn't have a maintainer. Additionally, I'm not sure if the package is currently functional. @timholy If you want, you are welcome to take over as maintainer of BioMedQuery.jl. But that might entail some work, e.g. to get CI passing again. Alternatively, it will probably be easier to develop the PubChem package as a separate package, e.g. something like JuliaHealth/PubChemAPI.jl or something like that. I would probably recommend the latter approach. Using HTTP.jl and JSON3.jl, it's pretty quick to get a working REST client. |
Sounds good. I'll do the standalone but plan to host it here at JuliaHealth. Thanks for the input! |
Greetings!
I tried using BioMedQuery.jl for the first time today. Installation went smoothly, however, when I attempt to load the package, I run into an error during the pre-compilation step:
Looking at the code for
src/DBUtils/mysql_db_utils.jl
, I noticed that the functiondb_query()
is attempting to callMySQL.query()
, which doesn't appear to be in the current API for MySQL.jl.I'm guessing that the MySQL.jl API has changed in some recent version, and that BioMedQuery.jl is still referring to the old interface? It's also possible that I am just misunderstanding something..
As a side note, I also noticed that
init_mysql_database()
function that gets automatically called during the initial attempt to load BioMedQuery.jl attempts to look for the "test" database. Since it is common to remove thetest
db after installation, it might be good to avoid this, if possible.Thanks!
System Info
Package versions:
The text was updated successfully, but these errors were encountered: