Skip to content
This repository has been archived by the owner on Aug 20, 2019. It is now read-only.

Error when submitting a job #15

Closed
djw8605 opened this issue Jan 22, 2016 · 14 comments
Closed

Error when submitting a job #15

djw8605 opened this issue Jan 22, 2016 · 14 comments
Assignees
Labels
Milestone

Comments

@djw8605
Copy link
Collaborator

djw8605 commented Jan 22, 2016

I am unable to submit a job, I receive this error:
screen shot 2016-01-22 at 1 00 23 pm

@djw8605 djw8605 added the bug label Jan 22, 2016
@djw8605 djw8605 added this to the V1.0 milestone Jan 22, 2016
@adamfitzgibbon
Copy link
Contributor

This is interesting, I thought it was working when I last committed. Were you submitting a job without any modules? I remember I was getting an undefined error when I tried submitting without modules, but I thought I resolved the issue. Perhaps I unintentionally changed something before committing. In any case, when I come in today I'll take a look and see if I can iron those out.

@djw8605
Copy link
Collaborator Author

djw8605 commented Jan 22, 2016

Here is an image of my submission that failed. It had a module listed.
image

@adamfitzgibbon
Copy link
Contributor

Thanks, hopefully I'll resolve this issue and then send a pull request later today with the visual enhancements you've asked for in other issues.

@djw8605
Copy link
Collaborator Author

djw8605 commented Jan 22, 2016

if possible, try to keep the different improvements in separate pull requests. It makes it easier to discuss and find problems.

@adamfitzgibbon
Copy link
Contributor

So a pull request for each issue?

@djw8605
Copy link
Collaborator Author

djw8605 commented Jan 22, 2016

Yeh, that would be ideal. Hopefully that isn't too hard.

@adamfitzgibbon
Copy link
Contributor

No that's fine for me. I just didn't want to spam you with pull requests if that wasn't what you wanted.

djw8605 added a commit that referenced this issue Jan 22, 2016
Added labels to Gauges

Fixes #12 and #15.
@djw8605
Copy link
Collaborator Author

djw8605 commented Jan 22, 2016

Fixed with merge 2ac31a5

@djw8605 djw8605 closed this as completed Jan 22, 2016
@adamfitzgibbon
Copy link
Contributor

Sorry to mislead you with my edits, but I haven't quite fixed this issue yet.

@djw8605
Copy link
Collaborator Author

djw8605 commented Jan 22, 2016

Oh, it seemed to be fixed. At least, I'm not getting the error again.

@djw8605 djw8605 reopened this Jan 22, 2016
@adamfitzgibbon
Copy link
Contributor

Oh... Interesting. I'm not actually sure why that's happening because I was trying to bind the default file locations before making the gauge edits. It must have been something that I changed in between the pull requests. Leave it open for now maybe and I'll test it next week to make sure it's okay.

@djw8605
Copy link
Collaborator Author

djw8605 commented Jan 22, 2016

specifically, I think it was this commit: c8583e2

@adamfitzgibbon
Copy link
Contributor

Oh I do remember finding that empty callback but I think I was getting sla different error so I didn't realize that was for that issue. Well that was probably the problem then.

@djw8605
Copy link
Collaborator Author

djw8605 commented Jan 22, 2016

Ok, in that case, I'll close this issue.

@djw8605 djw8605 closed this as completed Jan 22, 2016
djw8605 pushed a commit that referenced this issue Feb 15, 2017
A few fixes discovered while testing
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants