Skip to content

Commit

Permalink
Update README.md.
Browse files Browse the repository at this point in the history
Shorten development section by moving it to the wiki and linking there.
  • Loading branch information
DivvyCr committed May 5, 2020
1 parent e346c87 commit 0fdc233
Showing 1 changed file with 4 additions and 32 deletions.
36 changes: 4 additions & 32 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -190,35 +190,7 @@ Fix:
`chmod +x "carball/rattletrap/rattletrap-6.2.2-linux"`


## Developing
Everyone is welcome to join the carball (and calculated.gg) project! Even if you are a beginner, this can be used as an opportunity to learn more - you just need to be willing to learn and contribute.

### Usage of GitHub
All contributions end up on the carball repository. If you are new to the project you are required to use your own fork for first changes. If you do not have any previous git / github experience that is completely fine - we can help with it.
If we believe that you are comitted to working on the project and have experience in git we may give you write access so that you no longer have to use a fork. Nonetheless, please wait until your contrubtion is ready for a review to make the pull request because that will save resources for our tests and reduce spam.
For testing you should use your own fork, but take note that some carball tests may fail on a fork

### Learning about carball
Currently, there is active creation of the carball wiki on GitHub - it aims to provide all relevant information about carball and development, so if you are a beginner, definitely have a look there. If you can't find information that you were looking for, your next option is the calculated.gg Discord server, where you may send a message to the #help channel.

The carball code is also documented, although sparsely. However, you still may find information there, too.

### Testing
The main requirement is to run PyTest. If you are using an IDE that supports integrated testing (e.g. PyCharm), you should enable PyTest there. The secondary requirement (to compile the proto files) is to run the appropriate `setup` file (setup.bat for Windows, setup.sh for Linux/Mac).

If you've never tested your code before, it is a good idea to learn that skill with PyTest! Have a look at their official documentation, or any other tutorials.

### carball Performance
Carball powers calculated.gg, which analyses tens of thousands of replays per day. Therefore, performance is very important, and it is monitored and controlled using PyTest-Benchmarking, which is implemented via GitHub Actions. However, you may see your contribution's performance locally - look into PyTest-Benchmarking documentation. If your contribution is very inefficient - it will fail automatically.

If you wish to see the current carball analysis performance, it is split into 5 replay categories, and can be accessed below:
* [Short Sample](https://saltierl.github.io/carball/dev/bench/short_sample/)
* A very short soccar replay - for fast benchmarking.
* [Short Dropshot](https://saltierl.github.io/carball/dev/bench/short_dropshot/)
* A very short dropshot replay - to test dropshot performance.
* [Rumble](https://saltierl.github.io/carball/dev/bench/full_rumble/)
* A full game of rumble - to test rumble performance.
* [RLCS](https://saltierl.github.io/carball/dev/bench/oce_rlcs/)
* A full soccar RLCS game.
* [RLCS (Intensive)](https://saltierl.github.io/carball/dev/bench/oce_rlcs_intensive/)
* A full soccar RLCS game, but run with the intense analysis flag.
# Developing
If you are looking to help us develop carball, please have a look at [THIS WIKI PAGE](https://github.com/SaltieRL/carball/wiki/Developing-carball). It's a good introduction to what you might need to know as well as some other key notions.

You are advised to explore [the carball wiki](https://github.com/SaltieRL/carball/wiki), because it has information on some FAQs.

2 comments on commit 0fdc233

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Carball Benchmarks short_dropshot

Benchmark suite Current: 0fdc233 Previous: 7f188c8 Ratio
carball/tests/benchmarking/benchmarking.py::test_short_dropshot 0.5683223439541217 iter/sec (stddev: 0.023685866555870235) 0.6077960552972108 iter/sec (stddev: 0.14301398941268187) 1.07

This comment was automatically generated by workflow using github-action-benchmark.

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Carball Benchmarks short_sample

Benchmark suite Current: 0fdc233 Previous: 7f188c8 Ratio
carball/tests/benchmarking/benchmarking.py::test_short_sample 0.6483753328192665 iter/sec (stddev: 0.038938172370971524) 0.8930344604838997 iter/sec (stddev: 0.010219842775693095) 1.38

This comment was automatically generated by workflow using github-action-benchmark.

Please sign in to comment.