Skip to content
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

Update platform_model with seexplorers detailed list #277

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

vturpin
Copy link
Member

@vturpin vturpin commented Feb 14, 2025

add a more detailed list of seaexplorer

Proponents:
Moderator: @OceanGlidersCommunity/format-mantainers

Type of PR

  • Typo without possible change of interpretation of the related text.
  • Fix of some error, inconsistency, unforeseen limitation.
  • Style that only affects visually the compiled document
  • Addition that does not require change in the current structure.
  • Enhancement that require changes to improve the format.

Related Issues

Dates when it got review approvals

Release checklist

  • Approved by at least two members of the committee?
  • There were modifications after the review approvals? If so, please
    ask reviewers to update their review.
  • Proponents and moderador should explicitly agree that it is ready to
    to merge.
  • The moderador is the one in charge to actually merge or close this PR
    according to the final decision.

For maintainers

  • Update the moderator with a volunteer from the committee. It would be
    best to have one single moderator to guide and help this PR to move
    forward. It is OK to update the moderador pass it to another one.
  • Confirm that the associated branch was deleted after the merging.
  • Wrap-up and close the related issues.

Comments

add a more detailed list of seaexplorer
@callumrollo
Copy link
Member

Hi Victor, where are you getting the SeaExplorer models from? Particularly the Shallow X2 and Shallow X3? To my knowledge, Alseamar have not completed manufacture of a market ready shallow glider yet. It seems odd to already list two seperate models of it...

On a seperate note, is there a reason to maintain our own vocabulary of gliders rather than use e.g. the NVSB76 sensor models vocab? https://vocab.nerc.ac.uk/collection/B76/current/

@vturpin
Copy link
Member Author

vturpin commented Feb 14, 2025

Alseamar ask us to add those models. So I reported it here. Not to forget.

No there is no need for our own vocab I think NVSB76 is fine.

in this list I think we are storing our need, we need to make sure this is available in the B76.

@callumrollo
Copy link
Member

Thanks for the explanation! It would be good if alseamar could provide some more information on the models on their website that we can link to. At the moment all of the SeaExplorer models point to the same web page. Perhaps something to pass onto your contact?

For now I'm happy to list these as requested by alseamar until we can add them to B76

@emmerbodc
Copy link
Collaborator

Hello @vturpin, can you or someone in Alseamar create a ticket here to request those models adding to the B76 vocab please
https://github.com/nvs-vocabs/B76/issues

There is a template to follow here if you click on new issue - new term

@vturpin
Copy link
Member Author

vturpin commented Mar 4, 2025

@fmargirier Could you request the following terms to the control vocabulary B76 that OG1.0 is using as a reference.
I think we need to request new terms for all the seaexplorer glider models.

This is:

  • SeaExplorer X2,
  • SeaExplorer X3,
  • SeaExplorer X2 Shallow,
  • SeaExplorer X3 Shallow.

As you are the most expert here, feel free to remove, add or rename model as you wish. The description is also important and I am sure you have the proper descritpion internally

Here is the link where you can request new terms. I think you need to create an issue for any new terms.

https://github.com/nvs-vocabs/B76/issues

Let me know if you face any difficulty. I am happy to help and do it if needed.

@vturpin
Copy link
Member Author

vturpin commented Mar 18, 2025

@emmerbodc
Copy link
Collaborator

Great, @vturpin can you update those URLs to this file changed and then i think this can be closed.

vturpin added 2 commits April 10, 2025 11:03
Update the seaexplorer uri recently edited in NVS
Remove SeaExplorer X2 shallow that does not exist
change url to uri for seaexplorer
@vturpin
Copy link
Member Author

vturpin commented Apr 10, 2025

Done
I also remove the seaexplorer X2 shallow that does not exist.
I also have a question on the generic glider model.
Like allowing "SeaExplorer" instead of "SeaExplorer X2" for instance. This could happen if the operator does not know the model exactly.
This is particularly relevent for SeaGlider that have change licenced so many time...

@fmargirier
Copy link

What about specifying that the model is unknown with (undefined) or (model unknown) for these cases ?
Thanks for the rest !

@vturpin
Copy link
Member Author

vturpin commented Apr 10, 2025

Well I haven't seen any operator not knowing about the generic model of the glider they operates.
Seaglider
Seaexplorer
Slocum
Spray
are already generic name.

I would suggest to avoid 'unknown" but authorize the generic names like "slocum", "seaexplorer" and "seaglider"
The thing is, because this B76 is large, I don't know if we can request such generic platform models.

@fmargirier
Copy link

i meant spray (unkown) but you are right it is redundant and just adding weight instead of clarifying things.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants