-
Notifications
You must be signed in to change notification settings - Fork 12
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
feat(openchallenges): add Mariadb Connection and Load the EDAM Concepts #2680
Closed
Closed
Changes from 7 commits
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
11a8e80
add mariadb connection
mdsage1 31a4bef
add the dataframe input
mdsage1 74e2450
Load the data to the table
mdsage1 33cc7de
add mariadb required library
mdsage1 26211c2
update to pip command
mdsage1 0a6bb0b
update project.json
mdsage1 8c652f1
add mariadb packaging
mdsage1 9bcd9e9
remove binaries
mdsage1 a384fcc
add pip command to prepare
mdsage1 9e779fa
update prepare format
mdsage1 69c5f9f
make prepare command plural
mdsage1 cb453b1
add install of maria db connector
mdsage1 631f3d2
update mariadb prep commands
mdsage1 77c7693
update env prep
mdsage1 3b5f6a8
change config for mariaDB
mdsage1 7842cd6
revert env prep
mdsage1 a65676f
fix command so its plural
mdsage1 ce17395
add commands to prepare-python
mdsage1 5f0332e
remove brackets
mdsage1 0da3bbd
change pip wheel command
mdsage1 3268c14
remove install for test
mdsage1 552cf2d
Merge branch 'Sage-Bionetworks:main' into load_to_mariadb
mdsage1 e8391fa
update docker image system dependencies
mdsage1 50cfa9c
remove trailing white spaces
mdsage1 970048f
remove trailing whitespace
mdsage1 c626cac
Merge branch 'Sage-Bionetworks:main' into load_to_mariadb
mdsage1 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The following command fails:
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Running the script with Docker fails:
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
mdsage1 marked this conversation as resolved.
Show resolved
Hide resolved
|
Binary file not shown.
Binary file not shown.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The version should be pinned. You can't simplify change this value, instead you should reinstall the package with
poetry
while specifying the version:poetry add mysql-connector@2.2.9
.Same comments for the packages below.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We discussed the problems with using poetry to install this package back in May. It did not work. I will try again and hopefully there has been a poetry update to improve functionality when working with Maria DB and mysql.