-
Notifications
You must be signed in to change notification settings - Fork 9
General review of rate section #134
Comments
The actual correct info on purging: |
@hkershaw-brown @mttunca who has the knowledge/language to review that section. Who wants to take the lead? |
@hkershaw-brown There were some changes to the rates page, did they address the issues you brought up here or should we keep this issue open? |
@fernandogelin I would keep this issue open. It looks like there are still many problems with this page, e.g:
TL;DR this should go to back to Oscar management for policy review |
@hkershaw-brown thank you! I'll bring this up during the triage meeting tomorrow. |
Good luck @fernandogelin ! |
Hi, I just made a pull request for this. I also originally pushed using git push instead of npm run push, so there is a branch called content-rates that can be deleted. The correct branch is content-rates-correct. Thanks, |
Hi Kuba,
`npm run` is just for committing, to push you just do `git push`, there's
no `npm run push`.
Fernando
…On Fri, Jan 31, 2020 at 10:30 AM kubatarlowski ***@***.***> wrote:
Hi,
I just made a pull request for this. I also originally pushed using git
push instead of npm run push, so there is a branch called content-rates
that can be deleted. The correct branch is content-rates-correct.
Thanks,
Kuba
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/brown-ccv/ccv-website/issues/134?email_source=notifications&email_token=AAXKMBASS35KPMN2JHU3PWLRAQ72NA5CNFSM4JVSUSMKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKPABAY#issuecomment-580780163>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAXKMBHKDWDKKFHM3S47H3TRAQ72NANCNFSM4JVSUSMA>
.
--
<http://ccv.brown.edu/> Fernando Gelin <https://github.com/fernandogelin>
<http://directory.brown.edu/uuid/8144f0fb-0d36-47ed-ae64-ab07c232ebba>
*Senior Research Software Engineer*
Data & Visualization
Center for Computation and Visualization
Brown University | CIS
|
my mistake, I meant npm run commit. I did a regular git commit -m and then pushed that branch under "content-rates" |
@mttunca @singhsaluja were these issues addressed in the latest changes to the rate section? Can this be closed? |
@fernandogelin - we think this can be closed. Closing. |
This still doesn't seem correct. Is there a Max Wall time for jobs of 96 hours? This doesn't match the footnote, e.g. a 104-core job to run for 160 hours. |
Is there a document somewhere which details what the account limits are supposed to be and what the slurm associations are. The website doesn't match what is on Oscar so it really confusing for users. |
General comments on the rates page. I think this page needs a good proof read/rewrite, here are a few things I noticed
The text was updated successfully, but these errors were encountered: