Add isolde rules - #572
Conversation
|
|
||
| Returns the model instance if found, otherwise None. | ||
| """ | ||
| models = _load_models(self.dojson_model.entry_point_models) |
There was a problem hiding this comment.
I think you don't have to load it again, it is loaded somewhere in the app already, if you pass the loaded models in the exception then we could just select by the key
3a16f03 to
9e07d5b
Compare
|
|
||
| __query__ = '693__.a:"CERN ISOLDE" AND (980__:ARTICLE OR 980__:PREPRINT OR 980__:conferencepaper OR 980__:NOTE OR 980__:REPORT) -980__:DELETED -980__.c:MIGRATED -980__:DUMMY' | ||
|
|
||
| __ignore_keys__ = { |
There was a problem hiding this comment.
could we import and concatenate with the keys specified in _config.py?
29832b3 to
8361bb9
Compare
| cli_logger.info( | ||
| f"[apprn] Found approval report number {identifier!r} — storing as apprn" | ||
| ) | ||
| scheme = "apprn" |
There was a problem hiding this comment.
I'm not sure about adding them as apprn in the rule, if we' do this we need to change ep-approval workflow, see: restricted entry, public entry. We need to separate the report numbers somehow, for example faser records both have cern-ep-draft- and cern-ep- rep numbers, will this going to keep cern-ep-draft- as cdsrn?
I think we should raise if there's a ep report number and no ep information to check if there is any other record exists in the system. If we say we'll fix it when we have the error (when we found the internal record or etc) it might be more manual work to fix, wdyt?
There was a problem hiding this comment.
DIscussed IRL: we should add a step to check for all the identified EP numbers and collect the duplicates
36860d6 to
4dbc74b
Compare
| raise IgnoreKey("title") | ||
|
|
||
|
|
||
| _FUNDING_MODEL_MAP = { |
There was a problem hiding this comment.
should it be moved to config.py in transform folder? also the mapping is a bit strange...
| "intnoteatspubl": {"id": "publication-note"}, | ||
| "intnotehie-isoldepubl": {"id": "publication-note"}, | ||
| "hie-isolde-project-notes": {"id": "publication-note"}, | ||
| "intnotebepubl": {"id": "publication-note"}, |
There was a problem hiding this comment.
records from BE shouldn't be in this data set, unless ISOLDE owns them, same with ATS above (to be checked)
88a42c2 to
acbb1dd
Compare
…rds as already approved
acbb1dd to
88ff7ab
Compare
3 participants
closes #478
relevant GL issue: https://gitlab.cern.ch/cds-team/cds-rdm-openshift/-/work_items/24