Skip to content

Populate instrument session via graph#16

Merged
EmsArnold merged 3 commits intomainfrom
populate_instrument_session_via_graph
Oct 24, 2025
Merged

Populate instrument session via graph#16
EmsArnold merged 3 commits intomainfrom
populate_instrument_session_via_graph

Conversation

@EmsArnold
Copy link
Contributor

Use graph-nightly for both instrument sessions and workflows.

Select instrument session from drop-down list provided by graph-nightly query, give free visit input which allows for any session to be set.

Separate IS dropdown from IS query

Remove repository from workflows
<Typography variant="body1" align="center">
Maintainer: {maintainer}
</Typography>
{repository && (
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Graph-nightly should be up to date now with Workflows main, so repository can go back in (if you want it).

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have added repository back in, but trying to do pnpm relay with the new schema gives:

[INFO] [default] compiling...

thread 'main' panicked at crates/schema/src/in_memory.rs:836:21:
not yet implemented: SchemaExtension not implemented: extend schema @link(url: "https://specs.apollo.dev/federation/v2.5", import: ["@key", "@tag", "@shareable", "@inaccessible", "@override", "@external", "@provides", "@requires", "@composeDirective", "@interfaceObject", "@requiresScopes"])

note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
 ELIFECYCLE  Command failed with exit code 101.

Removing

extend schema
  @link(...)

resolves the issue, but feels like it isn't correct?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, we also remove that last part for the same reason.

@EmsArnold EmsArnold merged commit a385378 into main Oct 24, 2025
3 checks passed
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.

3 participants