Skip to content

adds project name field#126

Closed
capetillo wants to merge 1 commit into
mainfrom
name-field
Closed

adds project name field#126
capetillo wants to merge 1 commit into
mainfrom
name-field

Conversation

@capetillo

@capetillo capetillo commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Adds optional "name your project" field in the frontend and the backend sends it back in the output. If you have suggestions for better wording, please suggest it

@capetillo
capetillo requested a review from jnation3406 July 9, 2026 20:22

@jnation3406 jnation3406 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why are we naming an operation? It already has a "source" name and its input file set, why do we need a human name too?

@capetillo

Copy link
Copy Markdown
Contributor Author

Why are we naming an operation? It already has a "source" name and its input file set, why do we need a human name too?

we don't always use source name. human name is so that when users download the light curve, they know what that light curve belongs to. because it's not always the basename (e.g. ngc7331)

@jnation3406

Copy link
Copy Markdown
Contributor

Why are we naming an operation? It already has a "source" name and its input file set, why do we need a human name too?

we don't always use source name. human name is so that when users download the light curve, they know what that light curve belongs to. because it's not always the basename (e.g. ngc7331)

I would say in that case we should just prompt them for a filename when they click to save the plot? Or call it something like "lightcurve_of_ra123123xdec123123" if they don't specify a source name. No need to add a name to the operation itself I think.

@capetillo

Copy link
Copy Markdown
Contributor Author

Why are we naming an operation? It already has a "source" name and its input file set, why do we need a human name too?

we don't always use source name. human name is so that when users download the light curve, they know what that light curve belongs to. because it's not always the basename (e.g. ngc7331)

I would say in that case we should just prompt them for a filename when they click to save the plot? Or call it something like "lightcurve_of_ra123123xdec123123" if they don't specify a source name. No need to add a name to the operation itself I think.

@markBowman @sfoale what do you two think?

@capetillo capetillo closed this Jul 13, 2026
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