Skip to content

Conversation

@jl-wynen
Copy link
Member

No description provided.

@jl-wynen jl-wynen changed the title Link pooch to scicat datasets Show links to SciCat datasets Jan 30, 2025
def simulated_vanadium_sample() -> str:
"""Path to a GEANT4 CSV file for a vanadium sample.
SciCat:
Copy link
Member

Choose a reason for hiding this comment

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

Missing scicat identifier?

def simulated_diamond_sample() -> str:
"""Path to a GEANT4 CSV file for a diamond sample.
SciCat: 20.500.12269/d17b7169-b963-4b23-9455-d7332b992184
Copy link
Member

Choose a reason for hiding this comment

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

Can we just put a single url that points to the right place?
At the moment, it's not obvious how to even reach the file from that number.

You have to append this to the url below, but then replace the / in the middle with %2F.
I can't find the right combination of search fields from the scicat home page to reach the dataset (test search, keyword, or even the big search at the top of the page)

Copy link
Member Author

Choose a reason for hiding this comment

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

You have to append this to the url below, but then replace the / in the middle with %2F.

This is why I didn't want to show a url but the PID. But I can put both.

I can't find the right combination of search fields from the scicat home page to reach the dataset (test search, keyword, or even the big search at the top of the page)

I can't seem to find one either. Searching by PID works, but everything else doesn't. But that seems to be a bug in scicat. E.g., searching by owner group gives me all datasets, not just the ones for that group.

Copy link
Member

Choose a reason for hiding this comment

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

I feel the search on Scicat has been broken for a long time...

@jl-wynen jl-wynen merged commit 94eca18 into main Jan 30, 2025
4 checks passed
@jl-wynen jl-wynen deleted the scicat-links branch January 30, 2025 12:56
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