Skip to content

set empty array for datasets in case it's missing #1162

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.

Sign up for GitHub

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

Jump to bottom
Merged
denisiuriet merged 1 commit into from
Jan 19, 2026
Merged

set empty array for datasets in case it's missing #1162

denisiuriet merged 1 commit into from
Jan 19, 2026
+6 −2

Conversation

@denisiuriet
Copy link
Contributor

@denisiuriet denisiuriet commented Jan 19, 2026 •
edited
Loading

Fixes #1091 .

Changes proposed in this PR:

  • Allow initializeCompute without datasets

@denisiuriet denisiuriet self-assigned this Jan 19, 2026
@denisiuriet denisiuriet marked this pull request as ready for review January 19, 2026 13:31
@denisiuriet denisiuriet merged commit dfa34d6 into main Jan 19, 2026
11 checks passed
@denisiuriet denisiuriet deleted the set-dataset-initialize-compute branch January 19, 2026 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alexcos20 alexcos20 alexcos20 approved these changes

@ndrpp ndrpp ndrpp approved these changes

@bogdanfazakas bogdanfazakas Awaiting requested review from bogdanfazakas bogdanfazakas is a code owner

@giurgiur99 giurgiur99 Awaiting requested review from giurgiur99 giurgiur99 is a code owner

@andreip136 andreip136 Awaiting requested review from andreip136 andreip136 is a code owner

Assignees

@denisiuriet denisiuriet

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Allow initializeCompute to run without datasets

4 participants

@denisiuriet @alexcos20 @ndrpp