-
Notifications
You must be signed in to change notification settings - Fork 25
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
Errors with scope code "dataset" #590
Comments
@hmaier-fws @jwaspin Per conversation, "geographicDataset" was never a code. It has always been "dataset". ISO -2 code definition defined it as a geographic dataset. ISO -1 later removed the word "geographic" from the definition to read as a generic dataset. mdEditor has handling for displaying the "dataset" code as "geographicDataset" in the codelist due to a previous use case considering dataset to mean geographic dataset. This handling can be removed which will present the dataset code in a consistent manner as generic dataset. Geographic Dataset is not a current coding in Scope Code. No need for conversion handling in mdTranslator. |
@dwalt I removed the bug label since this is actually functioning as designed. This would be a change to the existing behavior. I also think this needs a discussion with the stakeholders to confirm that this should be the desired behavior. Without informing users of the change, they will likely incorrectly categorize their data. (that is the reason the display label of the "dataset" code was changed initially). Some background for discussion:
|
@hmaier-fws I agree this should be discussed by stakeholders. I would add that mdToolkit has otherwise used ISO codes verbatim and extended codelists as agreed upon by the community. This on-the-fly recasting of this code was likely done before profiles existed (are there any more recastings?). I would suggest that geographicDataset become an extended code, or part of a profile code "recasting" functionality. Of the solutions, I would favor implementation of filtered codelists in profiles, then use a combination of filtered codelists in profiles and extended codes to meet this need. In the short term, an extended code would meet the requirement without confusing new users with a recast code in the drop down list. |
@dwalt , @jwaspin
I also notice that this change has already been merged into the iso_scope.yml file on the develop branch. We can probably keep the code list as it is since the underlying values stored in the JSON do not change. But we'll probably need to retain the current behavior of displaying "geographicDataset" in the picklist until we figure out how to deal with this. |
@hmaier-fws I thought we had reached agreement on this issue. The simple solution without getting into profiles is to extend the codelist for Geographic Dataset. Then mdEditor can offer a straight codelist drop down. Probably what we are missing is to trap for previous use of the Dataset code by FWS and convert it to a new extended Geographic Dataset code, preserving the categorization. |
Issue
mdEditor Version (find version under settings): V0.11.0-Dev 2
mdEditor Web Address (enter the URL): dev.mdeditor.org
Issue Description:
Scope code in Data Quality appears to be written incorrectly in mdJSON and misinterpreted in mdtranslator
Steps to reproduce (Bug reports only):
Observed Results (Bug reports only):
Appears that geographicDataset is no longer a valid code. Appears that there is conversion handling to convert geographicDataset to dataset in mdEditor, however this same conversion is not being applied to -1 and -2 writers.
Translation to -1 or -2 is writing the wrong code.
Expected Results (Bug reports only):
mdcodes: dataset, Information applies to a dataset
mdEditor drop down list: dataset not geographicDataset
mdTranslator -1, -2: scope code of dataset
The text was updated successfully, but these errors were encountered: