Skip to content
This repository has been archived by the owner on Sep 7, 2020. It is now read-only.

Some minor changes #12

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Open

Some minor changes #12

wants to merge 13 commits into from

Conversation

Moohan
Copy link
Member

@Moohan Moohan commented May 12, 2020

Made some corrections and changes noticed from delivering yesterday and in preparation for doing block 3 and 4 later


# View plot
## View the bar plot
Copy link
Contributor

Choose a reason for hiding this comment

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

View scatter plot?


# View plot
## View the bar plot
Copy link
Contributor

Choose a reason for hiding this comment

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

View scatter plot?

@graemegowans
Copy link
Contributor

Looks good, only suggested one small change!

Copy link

@ciarag01 ciarag01 left a comment

Choose a reason for hiding this comment

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

Hi James,

Sorry for the delay in getting to this. I think these changes are fine and I'm happy for them to be merged once someone else has checked and approved too.

Ciara

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