Skip to content

Lists of fitting results#300

Merged
helenst merged 18 commits intomasterfrom
276-list-of-fitting-results
Oct 20, 2020
Merged

Lists of fitting results#300
helenst merged 18 commits intomasterfrom
276-list-of-fitting-results

Conversation

@helenst
Copy link
Copy Markdown
Contributor

@helenst helenst commented Oct 12, 2020

Fixes #276

@helenst helenst force-pushed the 276-list-of-fitting-results branch from bd5e9e7 to 5a27060 Compare October 12, 2020 11:33
@helenst helenst force-pushed the 276-list-of-fitting-results branch from 5a27060 to fa02254 Compare October 12, 2020 11:53
@helenst helenst force-pushed the 276-list-of-fitting-results branch from fe24fec to aed1056 Compare October 13, 2020 09:40
@helenst helenst marked this pull request as ready for review October 13, 2020 09:55
@helenst helenst requested a review from jonc125 October 13, 2020 09:55
@helenst helenst changed the title 276 list of fitting results Lists of fitting results Oct 13, 2020
Comment thread weblab/templates/datasets/compare_fittings.html Outdated
@jonc125 jonc125 mentioned this pull request Oct 14, 2020
Copy link
Copy Markdown
Contributor

@jonc125 jonc125 left a comment

Choose a reason for hiding this comment

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

Looking good. I tweaked some aspects of the display; figured it was easier just to make the changes than put them as review comments. There are just a few minor things to finish off I think.

Comment thread weblab/templates/entities/compare_fittings.html Outdated
Comment thread weblab/entities/views.py
Comment thread weblab/datasets/tests/test_views.py
Comment thread weblab/entities/tests/test_views.py
@helenst helenst force-pushed the 276-list-of-fitting-results branch from 957a4e6 to ef824cb Compare October 15, 2020 15:53
@helenst helenst requested a review from jonc125 October 16, 2020 10:25
Copy link
Copy Markdown
Contributor

@jonc125 jonc125 left a comment

Choose a reason for hiding this comment

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

Just a very minor suggestion!

I was confused at first that the visibility form still wasn't showing for me (hence the extra test added) but killing & restarting my dev server resolved it 🤷‍♂️

Comment thread weblab/datasets/tests/test_views.py Outdated
@helenst helenst merged commit 9446fdb into master Oct 20, 2020
@helenst helenst deleted the 276-list-of-fitting-results branch October 20, 2020 10:07
MauriceHendrix pushed a commit that referenced this pull request Jul 22, 2021
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.

Simple list views of fitting results

2 participants