-
Notifications
You must be signed in to change notification settings - Fork 8
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
add JuliaLang #32
base: main
Are you sure you want to change the base?
add JuliaLang #32
Conversation
First step to handle [Add Julia to OmegaUp](omegaup/omegaup#702)
@@ -14,6 +14,7 @@ use crate::args; | |||
const DEFAULT_EXTRA_MEMORY_SIZE_IN_BYTES: u64 = 16 * 1024 * 1024; | |||
const RUBY_EXTRA_MEMORY_SIZE_IN_BYTES: u64 = 56 * 1024 * 1024; | |||
const GO_EXTRA_MEMORY_SIZE_IN_BYTES: u64 = 512 * 1024 * 1024; | |||
const JULIA_EXTRA_MEMORY_SIZE_IN_BYTES: u64 = 512 * 1024 * 1024; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
how was this measured?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just took the largest I saw.
I'm guessing it will need tweaking.
@@ -686,6 +687,21 @@ impl JailOptions { | |||
seccomp_profile_name = String::from("go"); | |||
execve_args.extend([format!("./{}", args.run_target)]); | |||
} | |||
args::Language::Julia => { | |||
extra_memory_size_in_bytes = JULIA_EXTRA_MEMORY_SIZE_IN_BYTES; | |||
seccomp_profile_name = String::from("jl"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
seems like the seccomp files are missing from this change! can they be added?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure - I don't know what those are though 😅
That part of the build that is failing is due to
I didn't know how to work the invocation arguments for the Julia runner, but if I have a file print(parse(Int, readline()) and an input file called
to run it I would do julia solution.jl < input.txt And it would spit out
|
Bump @lhchavez |
First step to handle Add Julia to OmegaUp