Skip to content

Conversation

@pgetta
Copy link
Collaborator

@pgetta pgetta commented Mar 12, 2025

No description provided.

@pgetta pgetta enabled auto-merge (squash) March 12, 2025 11:52
Copy link
Collaborator

@GianlucaFicarelli GianlucaFicarelli left a comment

Choose a reason for hiding this comment

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

For more context, in what cases is the name of the job going to be used?

pgetta and others added 2 commits March 13, 2025 09:34
Co-authored-by: Gianluca Ficarelli <26835404+GianlucaFicarelli@users.noreply.github.com>
@pgetta
Copy link
Collaborator Author

pgetta commented Mar 13, 2025

For more context, in what cases is the name of the job going to be used?

In the UI we want to show something to the user that would allow them to identify the cost source (in addition to type and subtype).

In theory, if we store somewhere the execution itself (which is not the case for many services), we can map jobId from accounting to those entities and have the name from there. Not sure what's best, please let me know your thoughts.

The current idea about the name value:

  • for single cell model it's going to be the name of the model itself (because user can re-run simulation multiple times without saving anything - maybe this is actually something to address).
  • for bigger simulations - user will define the name.
  • for ML - there is a title of the chat that is automatically generated.

@pgetta pgetta merged commit bb1dd7b into main Mar 18, 2025
1 check passed
@pgetta pgetta deleted the feat/extend-job-report-with-name branch March 18, 2025 15:03
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