Update ctakes-ytex to support MySQLversion 8.0.x database#10
Open
jlpainter wants to merge 2 commits intoapache:mainfrom
Open
Update ctakes-ytex to support MySQLversion 8.0.x database#10jlpainter wants to merge 2 commits intoapache:mainfrom
jlpainter wants to merge 2 commits intoapache:mainfrom
Conversation
… the concept graphs to reference SNOMEDCT as SNOMEDCT_US which is the current name in UMLS. Update the create_tables.sql to support MySQL 8.0.x SQL commands. Update the ytex.properties examples to reflect that we need an xmldb.url to specify when JDBC URLs contain an ampersand and must be escaped in the generation of the conn.xml scripts in the ant build process.
Closed
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
This pull requests makes the following changes:
Further notes for testing:
Once this patch is applied, you can generate new concept graphs using a locally installed version of the UMLS database following the previous directions from the ctakes website. Running the command "ant -f build-setup.xml all" will create the ytex database using a new version of the UMLS, but you should update the scripts/data/umls/MRCONSO.RRF and MRSTY.RRF with your current extract. These files are too large now to commit to github. If you are going to build multiple sources into a single concept graph, insure that all sources are represented in the MRCONSO.RRF file for the umls_aui_fword generation steps which setup the ytex database prior to creating concept graphs.
The concept graph sct-rxnorm database by default as specified in the buildnon-asf.xml file. If you want to generate an alternate concept network, update the cg.name to match one of the templates provided (e.g. sct-umls will build ONLY the Snomed concept graph - you can specify any number of sources as long as they exist in your local UMLS database). See the main method in o.a.ctakes.ytex.kernal.dao.ConceptDaoImpl.java for examples on how to generate a new concept graph from code.