Skip to content

✨ Add a function to create a download link for artifacts that can be used by agents when referencing the artifact in markdown/html - #3853

Open
sheetalgiri wants to merge 6 commits into
mainfrom
create_download_link_artifact
Open

✨ Add a function to create a download link for artifacts that can be used by agents when referencing the artifact in markdown/html#3853
sheetalgiri wants to merge 6 commits into
mainfrom
create_download_link_artifact

Conversation

@sheetalgiri

@sheetalgiri sheetalgiri commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Why

Solution

@sheetalgiri
sheetalgiri force-pushed the create_download_link_artifact branch from 77cce97 to 0535c13 Compare August 12, 2026 15:50
@falexwolf

Copy link
Copy Markdown
Member

Nice idea! But rather than adding this here, I think we should extend UPath.to_url(): https://github.com/laminlabs/lamindb-setup/blob/307c46e1e64b73def27c9fd3d137a850ab1eaa87/lamindb_setup/core/upath.py#L851-L871

to_url() currently only works for public S3 buckets, since it misses the full authorization layer. It'd be nice if this was overloaded with a Lamin URL from our own storage service if the storage location is not public (the path object is authed).

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