Skip to content

Conversation

fm3
Copy link
Member

@fm3 fm3 commented Sep 3, 2025

Based on and thus blocked by #8844

Steps to test:

  • refresh local database schema
  • start up, log in, should see l4_sample_remote in the dashboard dataset list
  • should be usable.

Issues

  • contributes to Include remote datasets in initial testing data #8813 (we can easily add a few more once we have them publicly statically hosted somewhere. Ideal would be something with plenty of attachments and something with a different data type, like n5 or neuroglancerPrecomputed)

  • Added changelog entry (create a $PR_NUMBER.md file in unreleased_changes or use ./tools/create-changelog-entry.py)
  • Considered common edge cases

fm3 and others added 20 commits September 15, 2025 13:55
full datasource validation now also runs when reading
datasource-properties.jsons from disk. The result is shown in the
dashboard on “show error”.

The same validation also runs in the reserveManualUpload route

This also fixes a regression from #8844 where you could no longer hit
reload on unusable datasets.

It has, however, been slightly relaxed (no more path checks as those
have been moved elsewhere in #8844 )

### Steps to test:
- Edit datasource on disk, making it invalid (e.g. introduce duplicate
layer names or mags)

### Issues:
- fixes https://scm.slack.com/archives/CMBMU5684/p1747643139269819

------
- [x] Added changelog entry (create a `$PR_NUMBER.md` file in
`unreleased_changes` or use `./tools/create-changelog-entry.py`)
- [x] Considered [common edge
cases](../blob/master/.github/common_edge_cases.md)
- [x] Needs datastore update after deployment
Base automatically changed from reserve-manual to master September 22, 2025 10:54
@fm3 fm3 requested review from MichaelBuessemeyer and removed request for frcroth September 22, 2025 11:19
Copy link
Contributor

@MichaelBuessemeyer MichaelBuessemeyer left a comment

Choose a reason for hiding this comment

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

Nice, works splendid 👍

@fm3 fm3 enabled auto-merge (squash) September 23, 2025 07:31
@fm3 fm3 merged commit 86ba40d into master Sep 23, 2025
5 checks passed
@fm3 fm3 deleted the initial-dataset branch September 23, 2025 07:39
@coderabbitai coderabbitai bot mentioned this pull request Oct 6, 2025
18 tasks
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.

2 participants