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

AK_LORIS_indepth #10

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

Conversation

knoll-alex
Copy link
Contributor

Here is an updated reveal presentation for LORIS

Copy link

@christinerogers christinerogers left a comment

Choose a reason for hiding this comment

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

note to selves: how to preview commit (prior to my review):
https://htmlpreview.github.io/?https://github.com/samirdas/samirdas.github.io/blob/bbd2e2ae1b5782bd8595cf95ac40a413d34bd4a8/LORIS_indepth.html#

Changes requested in line.
Generally, do apply the changes made in other presentations here.
In addition, I stopped adding comments on the last few lines.
one key slide in that last batch to update -- the one with Angelina Paolozza's quote :

  • fix typo - should be NeuroDevNet
  • put ", PhD student, " after her last name
  • There's also a last line missing from her quote: "We analyzed and published our results two years earlier than it would have taken without LORIS."

@@ -216,63 +94,14 @@ <h4 align="center"><font color="f27c30">
</font></h4>
<h3 align="center"><a href="https://demo.loris.ca" target="_blank"><span class="fragment">TRY ME!</span></a></h3>

Choose a reason for hiding this comment

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

See my comments on the MNI Ecosystem slide deck for similar/identical slides
e.g. #9 (comment)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

<!-- What does LORIS do? Column 2 -->

<section>
<section>
<h2>What does LORIS do?</h2>
<img src="images/loris_purpose.jpg" style="border:0" width="80%">

Choose a reason for hiding this comment

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

replace this slide with hexagon tiles from google slides [link again here ]

  • create a second slide with Loris map and circles (N subjects, X visits...)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

thanks, done


<section>
<img src="images/LORIS_features.png">
<p>Scroll down to see more key features of LORIS</p>

Choose a reason for hiding this comment

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

Reveal doesn't actually work with mouse scroll - you have to hit the arrow keys. Can we find a better word than "scroll" to clarify this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

good idea - we don't want to use click either in case someone is viewing on a touchscreen. Maybe "See more key LORIS features below"

<section>

<section>
<img src="images/LORIS_features.png">

Choose a reason for hiding this comment

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

try to update this image with the 4th row shown the google slides [link here]

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

<!-- More Cool Features Column 6-->

<section>
<section data-background="images/cat_cool.jpg">

Choose a reason for hiding this comment

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

cute :)

@@ -670,8 +402,213 @@ <h2>LORIS - Github <font color="yellow"> - Open Source</font></h2>
<h2>LORIS - Github<font color="yellow"> Usage</font></h2>

Choose a reason for hiding this comment

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

yellow -> orange if you can

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done - changed all yellow in the code to loris orange

@@ -670,8 +402,213 @@ <h2>LORIS - Github <font color="yellow"> - Open Source</font></h2>
<h2>LORIS - Github<font color="yellow"> Usage</font></h2>
<img src="images/github_usage2017.png">

Choose a reason for hiding this comment

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

could you find/update this from our github? would be great to show through 2019

Copy link
Contributor Author

Choose a reason for hiding this comment

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

good call, done

</tr></table>
</section>

<!-- OMEGA - small pilot, interest has spiked -->

Choose a reason for hiding this comment

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

remove...

Replace with Open MNI iEEG atlas

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

<!-- Requirements Column 9 -->

<section>
<section>
<h2>LORIS - <font color="yellow">Requirements</font></h2>
<p class="fragment grow">Dedicated developer! (software is free of charge)</p>
<p class="fragment roll-in">LINUX based server</p>

Choose a reason for hiding this comment

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

3 lines below line 617 -- remove "beer for Samir"

Choose a reason for hiding this comment

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

Rename "Requirements for HBHL" to
"Scaling LORIS - Institutions and Consortia "

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@@ -697,7 +634,8 @@ <h2>LORIS - <font color="blue"> Links</font></h2>
<p>Source code: <a href="https://github.com/aces/Loris" target="_blank"> https://github.com/aces/Loris</a></p>

Choose a reason for hiding this comment

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

line above: put the actual loris-dev email address on a new line (it wraps awkwardly)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Cool - I added a break to each line to keep it consistent

Copy link

@christinerogers christinerogers left a comment

Choose a reason for hiding this comment

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

I did another brief pass - the updates look pretty good!

  • i caught some sizing issues, and i also flagged a few further things (so, requesting changes)

<h2>What does LORIS do?</h2>
<img src="images/loris_purpose.jpg" style="border:0" width="80%">
<img src="images/LORIS_hexagons.png">

Choose a reason for hiding this comment

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

@knoll-alex could you enlarge the image so that it's at least 150% of current size? the words inside the hexagons need to be bigger to be readable

Copy link
Contributor Author

Choose a reason for hiding this comment

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

150% was too big - it doesn't fit on the page. I tried to make it as big as possible while still fitting on the page. let me know if it's ok?

Choose a reason for hiding this comment

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

  • @christinerogers ask Derek to make a bigger and transparent version (w/o orange squiggle


<section>
<h3>LORIS Around the World!</h3>
<img src="images/LORIS_map.png">

Choose a reason for hiding this comment

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

likewise (^) bigger please

Copy link
Contributor Author

Choose a reason for hiding this comment

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

same

<span class="fragment"><h3 align="center"><a href="https://demo.loris.ca" target="_blank">Try the LORIS Demo</a></h3></span>
</td>
<td width="40%" style="vertical-align:middle">
<img src="images/loris_arch.png" style="padding:0; margin:0;">

Choose a reason for hiding this comment

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

image is too small... is it because the text is now given enough space to avoid wrap?
can the left/right buffer on the slide be diminished instead?
it seems either the text or the image generally has to suffer with the wide margins we have currently

Copy link
Contributor Author

Choose a reason for hiding this comment

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

tried to change this! played around with the width of each column and the image is bigger now, although the text doesn't look as good...


<!-- LORIS core -->

<section data-background="images/Rogers_poster.PNG" data-background-size="70%">

Choose a reason for hiding this comment

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

<h2>Login</h2>
<a href="https://demo.loris.ca" target="_blank"><img src="images/LORIS_login.png"></a>
<p>High-Level Walkthrough of LORIS</p>
<p><small>...follow along!</small></p>

Choose a reason for hiding this comment

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

print the URL -:

Follow along at :
demo.loris.ca

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

<section>
<h2>Login</h2>
<a href="https://demo.loris.ca" target="_blank"><img src="images/LORIS_login.png"></a>
<p>High-Level Walkthrough of LORIS</p>

Choose a reason for hiding this comment

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

needs to be a header font

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done - I tried to avoid it sometimes since it defaults all letters to caps. not sure how to change that!


<section>
<h2><font color="12a3d8">LORIS-CBRAIN INTEGRATION</font></h2>
<p>Benefits <font color="f27c30">vs</font> Challenges</p>

Choose a reason for hiding this comment

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

take out "Benefits vs challenges" - makes sense in a talk but not in a slide deck

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

Copy link

@christinerogers christinerogers left a comment

Choose a reason for hiding this comment

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

No files look like they're at risk of being deleted. if content is ok, mergable.

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