search

nfdi4plants / arc-export

Automatic building of a Docker container for exporting ARCs to the ARC.json
MIT License
1 stars 0 forks source link

Update ARCtrl and some additional dependencies #4

Closed kMutagene closed 1 month ago

kMutagene commented 1 month ago

@hlweil what do you think how robust such an update will be? this tool is used on every commit on every arc on the datahub.

HLWeil commented 1 month ago

As the current ARCtrl version is used in ARCitect, I would expect the read-in to be fairly robust.

But there might be Issues with very old ARCs https://github.com/nfdi4plants/ARCCommander/issues/240

kMutagene commented 1 month ago

i am mostly concerned with the json output. Ideally, we can use this update occasion to switch the output to RO crate.

So will

All ARCs -> Read -> RO Crate Json

Work? i mean we can and will still put empty json there on error, just asking how ready things are here

HLWeil commented 1 month ago

We could also allow both ISA and RO-Crate json via CLI args.

For RO-Crate I would first implement https://github.com/nfdi4plants/ARCtrl/issues/344, and then use that function here.