Skip to content
This repository has been archived by the owner on Nov 18, 2023. It is now read-only.

Don't Plot Preexisting Graph Elements #107

Closed
wants to merge 20 commits into from

Conversation

jmsfltchr
Copy link
Contributor

@jmsfltchr jmsfltchr commented Dec 3, 2019

What is the goal of this PR?

Update graph plots in line with the loss function being used (where preexisting graph elements are not penalised)

What are the changes implemented in this PR?

  • Preexisting graph elements are only drawn for the ground truth plot
  • Use consistent colours and change only the opacity to indicate certainty
  • Simplify the plotting code

@jmsfltchr jmsfltchr changed the title Fix plotting preexisting Don't Plot Preexisting Graph Elements Dec 3, 2019
@jmsfltchr jmsfltchr self-assigned this Dec 3, 2019
@jmsfltchr jmsfltchr added this to the 0.2.1 milestone Dec 3, 2019
@jmsfltchr jmsfltchr requested a review from lolski December 3, 2019 19:27
@jmsfltchr
Copy link
Contributor Author

Replaced with #108

@jmsfltchr jmsfltchr closed this Dec 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants