Skip to content
This repository has been archived by the owner on Nov 17, 2021. It is now read-only.

Shift to piston v2 endpoint #53

Open
Shivansh-007 opened this issue May 21, 2021 · 0 comments
Open

Shift to piston v2 endpoint #53

Shivansh-007 opened this issue May 21, 2021 · 0 comments
Labels
level: 1 - intermediate This PR is suitable for intermediate users. priority: 1 - high High Priority status: approved type: feature Relating to the functionality of the application.

Comments

@Shivansh-007
Copy link
Owner

All that has change from v2 (16c585d41d649af8a1be4cd35559d60970f27339) -> v3

  • Runtimes can now have their versions selected
  • Configuring run time and compile timeouts
  • Configuring compile_memory_limit and run_memory_limit
  • Multi-file support (Running folders)
  • Addition of signal and code

An example request now would look like:

{
    "language": "js",
    "version": "15.10.0",
    "files": [
        {
            "name": "my_cool_code.js",
            "content": "console.log(process.argv)"
        }
    ],
    "stdin": "",
    "args": [
        "1",
        "2",
        "3"
    ],
    "compile_timeout": 10000,
    "run_timeout": 3000,
    "compile_memory_limit": -1,
    "run_memory_limit": -1
}

Changes to be done

  • Add flags to configure compile_timeout, run_timeout, compile_memory_limit, run_memory_limit, this would be optional flags and can be used on shell, file, link and the default ask_input. Remember this would be taken as flags.
  • Add support for passing folders, on passing folders, the user would be asked to input the name of their main file which is to be run.
  • Add -v/--version flag which would take the runtime version and we can remove the current --version flag which is used for getting the piston-cli version to probably --info (?).
  • Change the message output i,e. Here is your python output: according to the return code and signal returned by piston v2 api.
  • Any more ideas are appreciated, This is a basic idea, the flag names and implementation can be discussed on this issue or the discord server

If you are interested in doing some parts of the issue, let me know :D

@Shivansh-007 Shivansh-007 added level: 1 - intermediate This PR is suitable for intermediate users. status: approved priority: 1 - high High Priority type: feature Relating to the functionality of the application. labels May 21, 2021
@Shivansh-007 Shivansh-007 added this to the V1.5.0 milestone May 30, 2021
@Shivansh-007 Shivansh-007 removed this from the V1.5.0 milestone Oct 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
level: 1 - intermediate This PR is suitable for intermediate users. priority: 1 - high High Priority status: approved type: feature Relating to the functionality of the application.
Projects
None yet
Development

No branches or pull requests

1 participant