-
Notifications
You must be signed in to change notification settings - Fork 23
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
Job name has an ampersand in it #303
Comments
qtop_py/plugins/pbs.py
|
added in latest experimental branch |
hi @stetner : I think @sfranky just implies that you should try about it https://github.com/qtop/qtop/tree/develop your feedback?! |
or, perhaps, this one is being referred (not sure which one, because I see no relevant PR on it yet): |
Looks good in that version! Thanks |
Sigh - users..... Now I have a name with a hash in it..... Aug_#_100 Can we add # to the regex Thanks |
I've got a $... is there a better way? What characters are known to not be valid? |
I've been wondering about that myself lately, maybe we should drop the
restriction...
…On Aug 30, 2017 11:38, "Edd Edmondson" ***@***.***> wrote:
I've got a $... is there a better way? What characters are known to not be
valid?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#303 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ABdZk1ajn0mHFIih2T2h7wzhYtN8-scCks5sdS2dgaJpZM4OwEj7>
.
|
@sfranky : i'd say don't be far too liberal yet: we'll tackle this right when we'll go for internationalization ... |
Just hit another one... added a '-' to the user search to match a username like a.smith-brown
|
Hit another one - probably a muser error but PBSPro seems to accept it: 1507877.delta compile_sim_${h i.xxxxxx 0 H normal So looks like $ and { can be in there too :p |
v0.9.20161222
not properly parsed by regex expression. Unexpected character.
The text was updated successfully, but these errors were encountered: