Closed cjh1 closed 8 years ago
@cjh1 @chetnieter
I've uploaded new screenshots for the steps. Looks pretty complete overall. Wordpress' spell checker is pretty good too, ran that on it and caught a few things. Let's look it over tomorrow again on a hangout or something.
Preview link if one of us is editing: https://blog.kitware.com/?p=15620&preview=true
@TristanWright Thanks for the updates, a few things:
@chetnieter Can you take a look? Particular making sure the steps make sense. You should have permission to edit it.
I just read it over and made a few minor grammar edits. I found that both the AWS profile screenshot and the 'Running the Simulation' screenshot are not clickable. I agree that the screenshots should probably be bigger. I will spend some time this morning testing the process described in the blog to see if we missed anything.
@cjh1
I wonder if we can increase the displayed size of the images.
Click on an image and click the pen icon. In the window that opens afterwards there's a sidebar with "Display options" and you can set either a custom size or a predefined one.
The DEMO=1 looks strange the way its on separate line, can we get it to be formated inline.
I agree, I was looking at it yesterday a bit. I think because it's in monospace it changes it. Alternatively we could italicize it.
What did you do to produce the final image in ParaView, the dataset seem split?
It's the second frame of the output with a scaler filter on it and display in wireframe. It's just a little more interesting that a dot on a plane, can easily change though.
@cjh1 @TristanWright Issues with blog instructions:
The third screenshot doesn't seem to be clickable?
@TristanWright In case you missed it in my comment above the fourth screenshot is also not clickable.
In the status/tool bar on first screenshot "simputttt / pyfr with simput".
I'll update that
The third screenshot doesn't seem to be clickable? @TristanWright In case you missed it in my comment above the fourth screenshot is also not clickable.
Resolved too, you can change the clickability by editing the image and changing the "Link to" option in the Display Options
@chetnieter Thanks for the QA, I am going to edit your comment to convert the issue into check boxes we can check off if that is OK?
Constants is misspelled on pyfr simulation input list (as Contstants).
@chetnieter This is fixed, you need to npm install
Thanks for the QA, I am going to edit your comment to convert the issue into check boxes we can check off if that is OK?
@cjh1 Yes
@chetnieter can you move the remaining to separate issues if you're still seeing them?
The tagging I can take care of. When creating them label them as "release"
can you move the remaining to separate issues?
Yes - should I label them all release?
@chetnieter Yes :smile:
Jinx
Constants is misspelled on pyfr simulation input list (as Contstants). @chetnieter This is fixed, you need to npm install
@TristanWright I am running in DEMO mode so I am guessing that npm install should be run and then the new files in dist should be commited to the repo to fix it for DEMO mode?
I always feel uneasy about how we commit dist/
files from other repositories, running npm install
will add the new pyfr with the spelling error. We can commit it.
@TristanWright I went to re-size the screenshots in the blog and it looks like they are pretty low rez images because they look fuzzy when blown up.
@chetnieter @cjh1 screenshots have been updated
How do I pull the blog?
On Fri, Sep 23, 2016 at 12:12 PM, TristanWright notifications@github.com wrote:
@chetnieter https://github.com/chetnieter @cjh1 https://github.com/cjh1 screenshots have been updated
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/Kitware/HPCCloud/issues/440#issuecomment-249263944, or mute the thread https://github.com/notifications/unsubscribe-auth/ADBhsW37cOC3hmrMCP-N_B3EJpnOpE6tks5qtBaLgaJpZM4JO_kC .
@patrickoleary Was just about to send you the link and ask for your feedback, I will send you the link.
@patrickoleary @cjh1 I added Patrick as an author in case he needs that to access the blog post.
Thanks @chetnieter You bet me to it!