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

OperatorID: Make OPERATOR ID messages optional #69

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

muramura
Copy link
Contributor

The Operator ID message is optional.

  1. if the number of text characters is 0, the message is not sent.
  2. Operator ID information can be set in the configuration parameter.

TEXT LEN > 0
Screenshot_20220922-150811868

TEXT LEN < 1
Screenshot_20220922-151853446

@tridge
Copy link
Contributor

tridge commented Jan 10, 2023

same Q as for the Self-ID PR. Should we add the complexity of not sending or just send blank like we do now?
the spec is vague about which jurisdictions require this message

@friissoren
Copy link
Contributor

the spec is vague about which jurisdictions require this message

The operator ID is only mandatory in the EU. The rules and additional documents for the US and Japan specify the operator ID as optional. Whether any other jurisdiction will make it mandatory can only be speculated upon at this point.

@BluemarkInnovations
Copy link
Contributor

In my opinion you should not broadcast packets with empty data. This saves energy and reduces co-existence issues.

@tridge
Copy link
Contributor

tridge commented Oct 16, 2023

@muramura @BluemarkInnovations I think this is handled in #101 ?

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