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

remove rfml.sigmf-ext.md from canonical extensions #233

Merged
merged 1 commit into from
Sep 26, 2022

Conversation

jacobagilbert
Copy link
Member

@jacobagilbert jacobagilbert commented Mar 19, 2022

This extension has no examples and it is not clear how to use it. Removal would NOT affect the validity of any SigMF using it, but I have been asked about what SigMF using this should look like several times over the past 6 months, and I have no useful answer.

Happy to change this PR to "include examples and documentation for rfml.sigmf-ext.md if that is possible! The 'label heirarchy' seems to be the unique feature this extension offers over what is already in the core spec.

@jacobagilbert jacobagilbert changed the title removing abandoned extension removing insufficiently documented RFML extension Mar 19, 2022
@jacobagilbert jacobagilbert changed the title removing insufficiently documented RFML extension remove insufficiently documented RFML extension Mar 19, 2022
@jacobagilbert jacobagilbert changed the title remove insufficiently documented RFML extension remove rfml.sigmf-ext.md from canonical extensions Mar 20, 2022
@jacobagilbert jacobagilbert self-assigned this Mar 23, 2022
@bhilburn
Copy link
Contributor

This extension was designed and used by NSWC Crane to create the datasets for DARPA's RFMLS program. Unfortunately, the government's lawyers determined those datasets could never be released, and no publicly releasable datasets were made. I'm not aware of anyone else using this particular extension as a result.

I am also in favor of removal, and if it needs to come back, more details and examples would make it more broadly useful.

@jacobagilbert
Copy link
Member Author

Ok, thanks for the context. I'll talk w/ Crane and see if I can get some info. If a good faith effort here fails I'll press along with deprecation on the grounds of this being proprietary.

@Teque5
Copy link
Collaborator

Teque5 commented Aug 14, 2022

I think this PR is ready.

@jacobagilbert jacobagilbert merged commit 6f14253 into sigmf-v1.x Sep 26, 2022
@777arc 777arc deleted the remove_rfml_ext branch November 21, 2022 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants