Skip to content

Add small mcstas image files for testing.#51

Merged
YooSunYoung merged 1 commit into
mainfrom
test-file
Nov 8, 2024
Merged

Add small mcstas image files for testing.#51
YooSunYoung merged 1 commit into
mainfrom
test-file

Conversation

@YooSunYoung

Copy link
Copy Markdown
Member

Related to scipp/ess#49

Added two files for open-beam(background) and sample.

They are subset of actual files from nextcloud

@YooSunYoung YooSunYoung merged commit 5fa0945 into main Nov 8, 2024
@YooSunYoung YooSunYoung deleted the test-file branch November 8, 2024 14:13
Comment thread src/ess/imaging/data.py
'small_mcstas_sample_images.h5': 'md5:3c42570951cabec7caedc76d90d03fa3',
'small_ymir_images.hdf': 'md5:cf83695d5da29e686c10a31b402b8bdb',
'README.md': 'md5:0f375972d4008de6060b065ac13ba17f',
'README.md': 'md5:9e1beeb325f127d691a8d7882db3255d',

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

looks like there is no README anymore? https://public.esss.dk/groups/scipp/ess/imaging/1/

The tests are failing.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@MridulS
image

It is there. It just doesn't show up in the web-thingy.

I'll see why it's failing though. It worked last time I made this change.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Which tests are failing btw?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Ahhh, the failure should go away after a release. The failure is at latest release which doesn't take into account the new url (with _version) and trying to find README files at the old location. Ignore me :)

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.

2 participants