Skip to content

Commit 91467a8

Browse files
Merge pull request #5 from IBM/4-finish-the-documentation
4 finish the documentation
2 parents be0aa1f + 2045daf commit 91467a8

File tree

4 files changed

+68
-35
lines changed

4 files changed

+68
-35
lines changed

CONTRIBUTING.md

+22-11
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,10 @@
22
Our project welcomes external contributions. If you have an itch, please feel
33
free to scratch it.
44

5-
To contribute code or documentation, please submit a **FIXME** [pull request](https://github.com/ibm/javacore-analyse/pulls).
5+
To contribute code or documentation, please submit a [pull request](https://github.com/IBM/javacore-analyser/compare).
66

77
A good way to familiarize yourself with the codebase and contribution process is
8-
to look for and tackle low-hanging fruit in the **FIXME** [issue tracker](https://github.com/ibm/javacore-analyse/issues).
8+
to look for and tackle low-hanging fruit in the [issue tracker](https://github.com/ibm/javacore-analyser/issues).
99
Before embarking on a more ambitious contribution, please quickly [get in touch](#communication) with us.
1010

1111
**Note: We appreciate your effort, and want to avoid a situation where a contribution
@@ -14,15 +14,16 @@ cannot be accepted at all!**
1414

1515
### Proposing new features
1616

17-
If you would like to implement a new feature, please **FIXME** [raise an issue](https://github.com/ibm/javacore-analyse/issues)
17+
If you would like to implement a new feature, please
18+
[raise an issue](https://github.com/IBM/javacore-analyser/issues/new)
1819
before sending a pull request so the feature can be discussed. This is to avoid
1920
you wasting your valuable time working on a feature that the project developers
2021
are not interested in accepting into the code base.
2122

2223
### Fixing bugs
2324

24-
If you would like to fix a bug, please **FIXME** [raise an issue](https://github.com/ibm/javacore-analyse/issues) before sending a
25-
pull request so it can be tracked.
25+
If you would like to fix a bug, please [raise an issue](https://github.com/IBM/javacore-analyser/issues/new) before
26+
sending a pull request so it can be tracked.
2627

2728
### Merge approval
2829

@@ -72,13 +73,23 @@ git commit -s
7273
Please feel free to connect with us on our [Slack channel](https://ibm.enterprise.slack.com/archives/C01KQ4X0ZK6).
7374

7475
## Setup
75-
**FIXME** Please add any special setup instructions for your project to help the developer
76-
become productive quickly.
76+
1. Install Pycharm
77+
2. Navigate to **Project from Version Control...** and follow next steps
78+
79+
To run the tool with sample data perform the following steps:
80+
1. Right click on **javacore_analyzer.py** directory in **Project** view and select **Modify Run Configuration...**.
81+
When the window appears, add the following commandline to **run parameters**
82+
`test/data/javacores /tmp/javacoreanalyser_output`
83+
Change the second parameter to the directory where you want the output report be created.
84+
2. Right click again on **javacore_analyzer.py** and select **Run** or **Debug**.
7785

7886
## Testing
79-
**FIXME** Please provide information that helps the developer test any changes they make
80-
before submitting.
87+
As default the tests in Pycharm are ran in the current selected directory. However we want to run them in main
88+
directory of the tool (**javacore-analyser** directory, not **test** directory).
89+
1. Right click on **test** directory in **Project** view and select **Modify Run Configuration...**.
90+
When the window appears, the **Working Directory** will be set to **test** directory.
91+
Change it to **javacore-analyser** directory
92+
2. Right click again on **test** and select **Run** or **Debug**.
8193

8294
## Coding style guidelines
83-
**FIXME** Optional, but recommended: please share any specific style guidelines you might
84-
have for your project.
95+
We use [PEP 8](https://peps.python.org/pep-0008/) Python style for Python files.

MAINTAINERS.md

+3-3
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
# MAINTAINERS
22

3-
- Krzysztof Kazmierczyk <[email protected]>
4-
- Piotr Aniola <[email protected]>
5-
- Tadeusz Janasiewicz <[email protected]>
3+
* Krzysztof Kazmierczyk <[email protected]>
4+
* Piotr Aniola <[email protected]>
5+
* Tadeusz Janasiewicz <[email protected]>

README.md

+42-20
Original file line numberDiff line numberDiff line change
@@ -7,32 +7,43 @@
77
<!-- Not always needed, but a scope helps the user understand in a short sentance like below, why this repo exists -->
88
## Scope
99

10-
The tool analyzes Javacores and verbose gc logs and provides some reports like cpu/gc usage, blocked threads, some tips regarding the javacores
11-
<!-- A more detailed Usage or detailed explaination of the repository here -->
12-
## Usage
13-
14-
This repository contains some example best practices for open source repositories:
10+
The tool analyzes Javacores and verbose gc logs and provides some reports like cpu/gc usage, blocked threads, some tips regarding the javacores. The tool can process the following data:
11+
* Set of Javacores from the same run. Optionally you can add verbose.gc log file
12+
* Single Javacore
1513

16-
* [LICENSE](LICENSE)
17-
* [README.md](README.md)
18-
* [CONTRIBUTING.md](CONTRIBUTING.md)
19-
* [MAINTAINERS.md](MAINTAINERS.md)
20-
<!-- A Changelog allows you to track major changes and things that happen, https://github.com/github-changelog-generator/github-changelog-generator can help automate the process -->
21-
* [CHANGELOG.md](CHANGELOG.md)
22-
* [REQUIREMENTS.md](REQUIREMENTS.md)
14+
15+
<!-- A more detailed Usage or detailed explaination of the repository here -->
16+
## Installation and usage
17+
18+
### Installation:
19+
The tool requires Python 3.9 or higher plus some packages - see more in [REQUIREMENTS](REQUIREMENTS.md). Despite it is not mandatory, it is recommended in Python to use virtual environment to manage packages.
20+
21+
Steps:
22+
1. Download and install Python. Usually the latest version is supported. Search for supported versions in
23+
[REQUIREMENTS](REQUIREMENTS.md)
24+
2. Download and unpack the tool.
25+
3. Create and activate Virtual Environment according to [Creating virtual environments](https://docs.python.org/3/tutorial/venv.html#creating-virtual-environments).
26+
4. Install all required pip packages. Navigate in command line to unpacked tool and type
27+
`pip install -r requirements.txt`
28+
29+
### Running the tool:
30+
1. Activate your created virtual environment according to activate Virtual Environment according to [Creating virtual environments](https://docs.python.org/3/tutorial/venv.html#creating-virtual-environments)
31+
2. Navigate in command line to unpacked tool and run the following command:
32+
`python javacore_analyzer.py <input-data> <output-dir>`
33+
Where `<input-data>` is one of the following:
34+
* The directory containing javacores and optionally verbose gc
35+
* Archive (7z, zip, tar.gz, tar.bz2) containint the same
36+
* List of the javacores separated by `'` character. Optionally you can add `--separator` option to define your own separator.
37+
You can type the following command to obtain the help:
38+
`python javacore_analyzer.py --help`
2339

24-
<!--
25-
> These are optional
26-
-->
2740

2841
<!-- The following are OPTIONAL, but strongly suggested to have in your repository. -->
2942
<!--
3043
* [dco.yml](.github/dco.yml) - This enables DCO bot for you, please take a look https://github.com/probot/dco for more details.
3144
* [travis.yml](.travis.yml) - This is a example `.travis.yml`, please take a look https://docs.travis-ci.com/user/tutorial/ for more details.
3245
-->
3346

34-
These may be copied into a new or existing project to make it easier for developers not on a project team to collaborate.
35-
3647
<!-- A notes section is useful for anything that isn't covered in the Usage or Scope. Like what we have below. -->
3748
## Notes
3849

@@ -72,8 +83,19 @@ If you would like to see the detailed LICENSE click [here](LICENSE).
7283
```
7384
## Authors
7485

75-
- Krzysztof Kazmierczyk <[email protected]>
76-
- Piotr Aniola <[email protected]>
77-
- Tadeusz Janasiewicz <[email protected]>
86+
* Krzysztof Kazmierczyk <[email protected]>
87+
* Piotr Aniola <[email protected]>
88+
* Tadeusz Janasiewicz <[email protected]>
7889

7990
[issues]: https://github.com/IBM/javacore-analyser/issues/new
91+
92+
## Another pages
93+
94+
Another useful pages:
95+
* [LICENSE](LICENSE)
96+
* [README.md](README.md)
97+
* [CONTRIBUTING.md](CONTRIBUTING.md)
98+
* [MAINTAINERS.md](MAINTAINERS.md)
99+
<!-- A Changelog allows you to track major changes and things that happen, https://github.com/github-changelog-generator/github-changelog-generator can help automate the process -->
100+
* [CHANGELOG.md](CHANGELOG.md)
101+
* [REQUIREMENTS.md](REQUIREMENTS.md)

REQUIREMENTS.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
### System Requirements
22

3-
1. Python 3.8 or newer
3+
1. Python 3.9 or newer
44
2. Modules installed from **requirements.txt** file. You can install them using
55
`pip install -r requirements.txt` command.

0 commit comments

Comments
 (0)