Skip to content

Conversation

@dsrishyla
Copy link
Contributor

No description provided.

@scott-huberty
Copy link
Owner

scott-huberty commented Oct 17, 2023

Thanks @dsrishyla ! Let's keep this PR open until the content is there.

A tip on the image and video. I don't think we need an image since it is just a flash of light ;). I can add a video of PLR to my Vimeo account (and while I'm at it, maybe I can share that folder with you can give you admin rights to it).

FYI I think you need to set your username and email in git so that you are properly credited for your Pr's (you were not credited for your last PR).

In your terminal, while in the Q1K-neuroimaging directory run each of the lines below:

git config user.name "Your Name"
git config user.email "your email"

Putting your actual name and email in.

@scott-huberty scott-huberty self-requested a review October 17, 2023 16:13
Copy link
Owner

@scott-huberty scott-huberty left a comment

Choose a reason for hiding this comment

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

See comments.

.. vimeo::
:align:

Conditions
Copy link
Owner

Choose a reason for hiding this comment

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

Since this task has no conditions, we probably don't need this. You can just change this section name to "Overview" and describe the task.

-----------


Animations
Copy link
Owner

Choose a reason for hiding this comment

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

We probably don't need this section. Can just describe the task in the Overview section.

PLR (PLR)
================

.. _PLR-image:
Copy link
Owner

Choose a reason for hiding this comment

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

Probably don't need this image directive or the figure directive below it.

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.

2 participants