Skip to content

Conversation

@trickyck
Copy link
Contributor

@trickyck trickyck commented May 20, 2022

Describe the change

  • Add about
  • Add build-openms-from-source
  • Add contributors-quickstart-guide
  • Add developer-faq
  • Add reporting-bugs-and-issues
  • Add users-quickstart-guide.md

PR checklist

  • I have added description of the change I'm proposing in the OpenMS Documentation.
    I'm adding all the files listed in the title.
  • I have read and followed OpenMS Documentation Contributing guidelines.
  • I have attached a screenshot of the relevant area after this change.
    There are too many changes to include screenshots.
  • CHANGELOG.md is updated.
  • I have added my name in CONTRIBUTING.md.


Try to restart travis, it sometimes hangs and since it builds on shared infrastructure, the next build may work better. This needs to be done by a OpenMS core developer.

Since we use extensive caching, the build may take much longer when many files are touched and may never complete in that case (running into the travis time limit). In that case we can rebuild the cache using the following approach:
Copy link
Contributor

Choose a reason for hiding this comment

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

Since we use extensive caching

Since extensive caching is used,

Copy link
Contributor

Choose a reason for hiding this comment

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

In that case we can rebuild

In that case, rebuild


Since we use extensive caching, the build may take much longer when many files are touched and may never complete in that case (running into the travis time limit). In that case we can rebuild the cache using the following approach:
```bash
$ git cherry-pick 89c5cd7f2d9d343b3d63fc6bab18e08dcd969c05
Copy link
Contributor

Choose a reason for hiding this comment

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

89c5cd7f2d9d343b3d63fc6bab18e08dcd969c05

`<git-sha>`.


# Now wait for the build to complete

$ git revert 89c5cd7f2d9d343b3d63fc6bab18e08dcd969c05
Copy link
Contributor

Choose a reason for hiding this comment

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

$ git push origin develop
```

## Working in Integrated Development Environments (IDEs)
Copy link
Contributor

Choose a reason for hiding this comment

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

Sentence case.


### Why are there no `source/TEST` and `source/APPLICATIONS/TOPP|UTILS` folder?

All source files added to an IDE are associated with their targets. You can find the source files for each test within its own subproject. The same is true for the `TOPP` and `UTILS` classes.
Copy link
Contributor

Choose a reason for hiding this comment

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

delete:

You can


### I'm getting the error "Error C2471: cannot update program database"

This is a bug in Visual Studio and there is a [bug fix] (http://code.msdn.microsoft.com/KB946040) Only apply it if you encounter the error. The bug fix might have unwanted side effects!
Copy link
Contributor

Choose a reason for hiding this comment

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

delete space after [bug fix], it is rendering incorrectly.
Screenshot 2022-05-21 at 8 17 36 PM


### Visual Studio can't read the clang-format file.

Depending on the Visual Studio version you're using you might get an error like `Error while formating with ClangFormat`. This is because Visual Studio is using an outdated version of clang-format. Unfortunately there is no easy way to update this using Visual Studio itself.
Copy link
Contributor

Choose a reason for hiding this comment

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

you're using you might get an error like

it might error with

### Visual Studio can't read the clang-format file.

Depending on the Visual Studio version you're using you might get an error like `Error while formating with ClangFormat`. This is because Visual Studio is using an outdated version of clang-format. Unfortunately there is no easy way to update this using Visual Studio itself.
There is a plugin provided by LLVM designed to fix this problem, but the plugin doesn't work with every Visual Studio version. However, you can update clang-format by hand using the pre-build clang-format binary. Both the binary and a link to the plugin can be found [here](https://llvm.org/builds/).
Copy link
Contributor

Choose a reason for hiding this comment

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

you can update clang-format by hand using the pre-build clang-format binary. Both the binary and a link to the plugin can be found

In that case, update clang-format manually using the pre-build clang-format binary.


Depending on the Visual Studio version you're using you might get an error like `Error while formating with ClangFormat`. This is because Visual Studio is using an outdated version of clang-format. Unfortunately there is no easy way to update this using Visual Studio itself.
There is a plugin provided by LLVM designed to fix this problem, but the plugin doesn't work with every Visual Studio version. However, you can update clang-format by hand using the pre-build clang-format binary. Both the binary and a link to the plugin can be found [here](https://llvm.org/builds/).
To update clang-format by hand just download the binary and exchange it with the clang-format binary in your Visual Studio folder.
Copy link
Contributor

Choose a reason for hiding this comment

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

delete:

just


### The indexer gets stuck at some file which ``#includes seqan``

It seems that SeqAn code is just too confusing for older eclipse C++ indexers. You should upgrade to eclipse galileo (CDT 6.0.x). Also, increase the available memory limit in eclipse.ini, e.g. -Xmx1024m for one gig.
Copy link
Contributor

Choose a reason for hiding this comment

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

You should

It is recommended to

Copy link
Contributor

Choose a reason for hiding this comment

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

`eclipse.ini`, e.g. `-Xmx1024m`


### The parser is confused after OPENMS_DLLAPI and does not recognize standard C++ headers

Go to ``Project -> Properties -> C/C++ Include Paths and Preprocessor Symbols -> Add Preprocessor symbol -> "OPENMS_DLLAPI="``. This tells eclipse that the macro is defined empty. In the same dialog you can also add an external include path to e.g. ``/usr/include/c++/4.3.3/``, etc. The issue with C++ headers was fixed in the latest galileo release.
Copy link
Contributor

Choose a reason for hiding this comment

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

#25

Copy link
Contributor

Choose a reason for hiding this comment

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

delete:

you can also


### How do I run a single test?

You can can execute an OpenMS class test using the CTest regular expressions:
Copy link
Contributor

Choose a reason for hiding this comment

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

delete:

You can can


ctest -V -R "^<class>_test"

# To build a class test, you simply call the respective make target in ./source/TEST:
Copy link
Contributor

Choose a reason for hiding this comment

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

delete:

you simply


make <class>_test
```
To run a TOPP test, you can use:
Copy link
Contributor

Choose a reason for hiding this comment

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

delete:

you can

```
### How do I debug uncaught exceptions?

You can dump a core if an uncaught exception occurs, by setting the environment variable `OPENMS_DUMP_CORE`.
Copy link
Contributor

Choose a reason for hiding this comment

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

am i seeing a repeat?


### (Linux) Why is no core dumped, although a fatal error occured?

Try the `ulimit -c` unlimited command. It sets the maximum size of a core to unlimited.
Copy link
Contributor

Choose a reason for hiding this comment

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

(repeated?)


### How can I wrap my new class with PyOpenMS?

You will have to create a new file `src/pyOpenMS/pxds/CLASS_NAME.pxd` that is explained in the [pyOpenMS documentation](https://pyopenms.readthedocs.io/en/latest/wrap_classes.html?highlight=Wrapping%20classes#wrapping-workflow-and-wrapping-new-classes).
Copy link
Contributor

Choose a reason for hiding this comment

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

delete:

You will have to


### How can I wrap my new method with PyOpenMS?

You will have to add an entry to `src/pyOpenMS/pxds/CLASS_NAME.pxd` with the signature of your new method(s).
Copy link
Contributor

Choose a reason for hiding this comment

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

delete:

You will have to


### Where can I find the definition of the main page?

You can find a definition of the main page [here](https://github.com/OpenMS/OpenMS/edit/develop/doc/doxygen/public/Main.doxygen).
Copy link
Contributor

Choose a reason for hiding this comment

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

repeated?

Pull Request Checklist
======================

Before you open a pull request, make sure you check the following:
Copy link
Contributor

Choose a reason for hiding this comment

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

Before you open a pull request, make sure you check the following:

Before opening a pull request, please check the following:

* **Is the code documented?**

Document all new classes, including their methods and parameters.
It is also recommended that you also document non-public members and methods.
Copy link
Contributor

Choose a reason for hiding this comment

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

that you also

to

Make sure to:
* **Rebase before you open a pull request.**

To include all recent changes, rebase your branch on `develop` before you open a pull request.
Copy link
Contributor

Choose a reason for hiding this comment

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

you open a pull request.

opening a pull request.


A list of known issues in the current OpenMS release can be found [here](https://abibuilder.informatik.uni-tuebingen.de/archive/openms/Documentation/nightly/html/known_dev_bugs.html). Please check if your OpenMS version matches the current version and if the bug has already been reported.

In order to report a new bug, please use either our [GitHub issues system](write-and-label-github-issues.md) or [contact us](contact-us.md).
Copy link
Contributor

Choose a reason for hiding this comment

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

In order to report a new bug, please use either our GitHub issues system or contact us.

In order to report a new bug, please create a [GitHub issue] following our bug reporting guidelines, you can also contact us.


1. Go to the [OpenMS codebase](https://github.com/OpenMS/OpenMS).
2. Click **Issues** in the menu at the top of the screen.
3. Click **New issue**.
Copy link
Contributor

Choose a reason for hiding this comment

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

To label an issue:
1. On the right of the screen, select the cog icon under **Labels**.
2. Choose a label from the list. Normally, an issue can have one or more of the following labels:
* **defect**
Copy link
Contributor

Choose a reason for hiding this comment

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

i think one more space on left will indent these bullet points properly inside 2..

Screenshot 2022-05-21 at 8 36 24 PM

@tapaswenipathak tapaswenipathak self-requested a review May 23, 2022 11:20
@tapaswenipathak tapaswenipathak self-assigned this May 23, 2022
@tapaswenipathak tapaswenipathak mentioned this pull request May 26, 2022
13 tasks
@tapaswenipathak
Copy link
Contributor

=> #40.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants