Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Lab meeting blog 23 Jan & updated about page #98

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

Conversation

KirstieJane
Copy link
Member

Please review and submit a PR to this branch to add an image (please!) or any adjustments you'd like to make to the text ✨

Copy link
Contributor

@sgibson91 sgibson91 left a comment

Choose a reason for hiding this comment

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

LGTM!

In her spare time, she enjoys music and cross-stitch.

**More information:**
* CV: *delete this row if you don't want to include this information*
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
* CV: *delete this row if you don't want to include this information*


**More information:**
* CV: *delete this row if you don't want to include this information*
* GitHub: *delete this row if you don't want to include this information*
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
* GitHub: *delete this row if you don't want to include this information*
* GitHub: [@sgibson91](https://github.com/sgibson91)

**More information:**
* CV: *delete this row if you don't want to include this information*
* GitHub: *delete this row if you don't want to include this information*
* Twitter: *delete this row if you don't want to include this information*
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
* Twitter: *delete this row if you don't want to include this information*
* Twitter: [@drsarahlgibson](https://twitter.com/drsarahlgibson)

* CV: *delete this row if you don't want to include this information*
* GitHub: *delete this row if you don't want to include this information*
* Twitter: *delete this row if you don't want to include this information*
* ORCID: *delete this row if you don't want to include this information*
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
* ORCID: *delete this row if you don't want to include this information*
* ORCID: [0000-0003-0356-2765](https://orcid.org/0000-0003-0356-2765)

* GitHub: *delete this row if you don't want to include this information*
* Twitter: *delete this row if you don't want to include this information*
* ORCID: *delete this row if you don't want to include this information*
* Email: *delete this row if you don't want to include this information*
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
* Email: *delete this row if you don't want to include this information*

Copy link
Contributor

@sparkler0323 sparkler0323 left a comment

Choose a reason for hiding this comment

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

Looks great~


<figure>
<img src="https://www.frontiersin.org/files/Articles/385392/fnins-12-00662-HTML/image_m/fnins-12-00662-g004.jpg" alt="Two scatter plots">
<figcaption>Alpha waves characteristics analyzed by Matlab 2013 versus Matlab 2014.
Copy link
Contributor

Choose a reason for hiding this comment

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

Do you know which version of the EEGLab toolbox was used in each of these plots? I'm unsure from the text alone whether it's the version of MATLAB causing these differences or if it's also down to the version of the custom toolbox used.

(Although this can backfire when it comes to the degree requirements - at least at the moment!)

**Sarah** asked how to encourage people to use a service?
Hub23 (the Turing's private BinderHub) doesn't currently have any users which is a shame because she's put a lot of work into making it really easy for me to maintain (e.g. [writing bots](https://github.com/HelmUpgradeBot) to keep the helm chart up-to-date and the container registry in a manageable state).
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Hub23 (the Turing's private BinderHub) doesn't currently have any users which is a shame because she's put a lot of work into making it really easy for me to maintain (e.g. [writing bots](https://github.com/HelmUpgradeBot) to keep the helm chart up-to-date and the container registry in a manageable state).
Hub23 (the Turing's private BinderHub) doesn't currently have any users which is a shame because she's put a lot of work into making it really easy to maintain (e.g. [writing bots](https://github.com/HelmUpgradeBot) to keep the helm chart up-to-date and the container registry in a manageable state).

@sgibson91
Copy link
Contributor

Properly read your request and opened a PR #99, my review comments are addressed by that 👾

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.

4 participants