You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In order to address a few of the issues in Chisel, Marc and I might take a stab at overhauling a very convoluted part of ConceptQL -- the vocabulary operators. We currently have 3 different ways to add vocabulary operators: hard-coded (i.e. we've written explicit code for an operator), read in from a CSV, or read in and generated dynamically from Lexicon. All of these ways are supported by 4 main vocabulary classes that have been jerry-rigged to work together.
The code to support this is pretty awful and is why we're seeing weird issues with warnings (#184), missing columns (#145), duplicate operators (#201), and other issues you haven't discovered yet (handling vocab operators that appear in one Jigsaw instance but not another, etc).
Marc and I believe we can eliminate almost all of the hard-coded operators in favor of dynamically generating virtually all vocabulary operators. We might be able to eliminate the CSV as well. We'll clean up the vocabulary code and hopefully reduce the classes to 1 main class and maybe a couple small subclasses. This will be a good opportunity to possibly address the slowness you're seeing in Chisel with certain "select all" algorithms as well. We can also probably improve the operator descriptions in the JAM as well.
This is probably a multi-day project, but most of the bugs we want to address in Chisel are a direct result of the code I'm hoping to replace.
The text was updated successfully, but these errors were encountered:
This is completed to my satisfaction. @marc-outins, anything we missed in this ticket? Looks like #237 still needs to be addressed but that looks like it's in its own ticket.
In order to address a few of the issues in Chisel, Marc and I might take a stab at overhauling a very convoluted part of ConceptQL -- the vocabulary operators. We currently have 3 different ways to add vocabulary operators: hard-coded (i.e. we've written explicit code for an operator), read in from a CSV, or read in and generated dynamically from Lexicon. All of these ways are supported by 4 main vocabulary classes that have been jerry-rigged to work together.
The code to support this is pretty awful and is why we're seeing weird issues with warnings (#184), missing columns (#145), duplicate operators (#201), and other issues you haven't discovered yet (handling vocab operators that appear in one Jigsaw instance but not another, etc).
Marc and I believe we can eliminate almost all of the hard-coded operators in favor of dynamically generating virtually all vocabulary operators. We might be able to eliminate the CSV as well. We'll clean up the vocabulary code and hopefully reduce the classes to 1 main class and maybe a couple small subclasses. This will be a good opportunity to possibly address the slowness you're seeing in Chisel with certain "select all" algorithms as well. We can also probably improve the operator descriptions in the JAM as well.
This is probably a multi-day project, but most of the bugs we want to address in Chisel are a direct result of the code I'm hoping to replace.
The text was updated successfully, but these errors were encountered: