Skip to content

Feat/consys custom formats - #362

Open
tipatterson-dev wants to merge 13 commits into
opensensorhub:masterfrom
Botts-Innovative-Research:feat/consys-custom-formats
Open

tipatterson-dev wants to merge 13 commits into
opensensorhub:masterfrom
Botts-Innovative-Research:feat/consys-custom-formats

Conversation

@tipatterson-dev

Copy link
Copy Markdown
Member

reopening with a different branch: #354

Handlers holding the read-only federated store skipped the commit.

Fixes #18
An emptied series stayed registered, so a new series could get its ID and two
datastreams shared records. Applies to the obs and command stores.

Refs opensensorhub#340
…dProperty

These params replaced the parent filter. Observations under
/samplingFeatures/{id} are now filtered by FOI.

Refs #21, opensensorhub#340
…time

Query results aren't sorted, so the first one isn't always the latest.

Refs opensensorhub#343
Only for series where result time equals phenomenon time.

Refs opensensorhub#331
… schemas

Extend the CustomObsFormat SPI with command-side hooks (getCommandBinding,
getCommandSchemaBinding) and an isAutoSelectable flag, and thread the
configured customFormats map through the obs, command, command-status,
datastream-schema and command-stream-schema handlers so a registered format
(e.g. application/swe+proto) can serve its own data and schema views.
… POST

Dispatch datastream and control-stream creation to a registered custom
format's binding when the embedded schema names it. DataStreamBindingJson
gains a custom-format branch ahead of the application/swe+ prefix match;
CommandStreamBindingJson buffers the schema object and dispatches on
commandFormat, staying backward-compatible (a missing commandFormat still
uses the default JSON binding); CommandStreamHandler threads the
customFormats map through to the binding.
@tipatterson-dev

Copy link
Copy Markdown
Member Author

@alexrobin sorry for dropping the ball on this and confusing things by removing and reopening. This version should address the issues you brought up in PR 354 #354

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.

1 participant