sssom-py icon indicating copy to clipboard operation
sssom-py copied to clipboard

Add the ontoportal mapping model json converter

Open syphax-bouazzouni opened this issue 3 years ago • 1 comments

About

related https://github.com/agroportal/project-management/issues/265

How to use

 python -m sssom convert  basic.tsv -o basic-ontoportal.json -O ontoportal_json

syphax-bouazzouni avatar Jul 04 '22 17:07 syphax-bouazzouni

@syphax-bouazzouni thank you for your contribution! This is great and eyeballing what you have done so far, it looks good. I will provide you with a more detailed review later, but in the meantime, we are in general very keen on the parsers, not as much the writers - the idea being that we can use SSSOM as a universal exchange format, and sssom-py enabling tools to access resources in different formats. Would you be able to provide us with parser and extension of the sssom parse method as well?

matentzn avatar Jul 05 '22 07:07 matentzn

@syphax-bouazzouni , I just received a review request from you. I seemed to have asked a few questions in August (above) and have not received a response. Could you please kindly address them.

hrshdhgd avatar Mar 17 '23 14:03 hrshdhgd

hello @hrshdhgd, I think I responded to them, can you recheck please

syphax-bouazzouni avatar Mar 17 '23 14:03 syphax-bouazzouni

hello @hrshdhgd, I think I responded to them, can you recheck please

Really sorry I forgot to submit my reviews (https://github.com/community/community/discussions/10369)

syphax-bouazzouni avatar Mar 20 '23 08:03 syphax-bouazzouni

hello @hrshdhgd, I think I responded to them, can you recheck please

Really sorry I forgot to submit my reviews (community/community#10369)

Not a problem! Thanks for the PR and the clarification.

hrshdhgd avatar Mar 20 '23 13:03 hrshdhgd

I see that QC check fails but it is an easy fix:

Run

  • tox -e lint
  • tox -e flake8

in your command line and that should fix some errors and show errors that need addressing. This basically formats your code and applies flake8 rules.

hrshdhgd avatar Mar 20 '23 15:03 hrshdhgd

I see that QC check fails but it is an easy fix:

Run

  • tox -e lint
  • tox -e flake8

in your command line and that should fix some errors and show errors that need addressing. This basically formats your code and applies flake8 rules.

Hi, I did the linting and fixed one of my failing tests. All tests passed now (the 63 ones)

syphax-bouazzouni avatar Mar 21 '23 10:03 syphax-bouazzouni