Skip to content
This repository has been archived by the owner on Mar 17, 2024. It is now read-only.

Feature/stackn 176 #169

Closed

Conversation

adamhagevall
Copy link

Status

  • Ready
  • Draft
  • Hold

Description

Changed x-axis unit in performance charts to show time scale (day) and the charts will now show indicators for version releases. Also, the user will see the exact parameter values that have been logged for each training session when hovering over the corresponding data point. Finally, stackn train CLI command will take model version as an argument, where default is the latest version. This is to make it easier to fall back to earlier versions of the model (will probably need some additional features).

Types of changes

What types of changes does your code introduce to STACKn?

  • Hotfix (fixing a critical bug in production)
  • Bugfix
  • New feature
  • Documentation update

Checklist

If you're unsure about any of the items below, don't hesitate to ask. We're here to help!
This is simply a reminder of what we are going to look for before merging your code.

  • This pull request is against develop branch (not applicable for hotfixes)
  • I have included a link to the issue on GitHub or JIRA (if any)
  • I have included migration files (if there are changes to the model classes)
  • I have read the CONTRIBUTING doc
  • I have included tests to complement my changes
  • I have updated the related documentation (if necessary)
  • I have updated the release notes (docs/releasenotes.md)
  • I have added a reviewer for this pull request
  • I have added myself as an author for this pull request

Further comments

Anything else you think we should know before merging your code!

Adam Hagevall added 2 commits December 8, 2020 20:30
…arts and added model version to model logs, i.e. model version will be specified when training a model
@adamhagevall adamhagevall changed the base branch from master to develop December 14, 2020 09:31
Copy link
Member

@morganekmefjord morganekmefjord left a comment

Choose a reason for hiding this comment

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

Exclude these

@stefanhellander
Copy link
Contributor

Already merged together with another PR. Closing this one.

niklastheman added a commit that referenced this pull request Jul 5, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants