Get the commit messsage data using the new read function and merge either nothing, the
title or message and title into the commit.data of the proj.conf instance.

See #180

Signed-off-by: Niklas Schneider <[email protected]>

Add the new attribute "commit.messages" to the project configuration class with options "none",
"title" and "message" to make it possible to specify what exactly of the commit message data
is to be merged to the commit data.

See #180

Signed-off-by: Niklas Schneider <[email protected]>

Signed-off-by: Niklas Schneider <[email protected]>

Add two tests for testing the merge functionality for both full commit messages and
titles only. Fix bug that merges message body instead of title when selecting
option "title"

See #180

Signed-off-by: Niklas Schneider <[email protected]>

Signed-off-by: Niklas Schneider <[email protected]>

Also exchange the merge attribute when merging data frames of commit
messages from commit.id to hash.

Signed-off-by: Niklas Schneider <[email protected]>

Signed-off-by: Niklas Schneider <[email protected]>

As commit.id was the first column of the data frame anyway,
merging has not changed the order. But when using the hash column
it is taken as the first colum of the resulting data frame.
Change the order of the columns in order to not break anything
that relies on the order.

See #180

Signed-off-by: Niklas Schneider <[email protected]>

Follow the review suggestions of @clhunsen.

See #180

Signed-off-by: Niklas Schneider <[email protected]>

Following the review of #193

Signed-off-by: Niklas Schneider <[email protected]>

Remove some empty lines and indent some lines.
Also remove commit.message.data.unprocessed variable and use the
commit.message.data variable from the beginning. Add column names
beforehand in order to enable access without indices.

See #193

Signed-off-by: Niklas Schneider <[email protected]>

Create private function update.commit.message.data in util-data.R
which handles the merge and change the location where it is
called in set.commits.

See #193

Signed-off-by: Niklas Schneider <[email protected]>

Fix an error where the value of a variable that is defined
in an if block is returned outside that if block.

See #193

Signed-off-by: Niklas Schneider <[email protected]>

Replaced a loop with a conversion from a list of vectors
in a data frame and access its columns directly

See #193

Signed-off-by: Niklas Schneider <[email protected]>

Move functions concerning reading commit messages and the constants used
by them to a new section in util.read.
Replace subset with proper indexing and minor comment fixes.

See #193

Signed-off-by: Niklas Schneider <[email protected]>

Also adapt 'update.commit.messages' to better match the implementation
of similar methods.
Add 'set.commit.messages' in order to be able to set the commit messages
to NULL.

See #193.

Signed-off-by: Niklas Schneider <[email protected]>

Introduce new function 'format.commit.ids' in along with new section in
util-read.R. Also put format "<commit-%s>" into a constant.

See #193

Signed-off-by: Niklas Schneider <[email protected]>

Take advice by @clhunsen to replace if else cascade for rearranging columns with
better merge call.
Also modify test-data tests regarding commit messages: Row names are no longer
ignored.

See #193

Signed-off-by: Niklas Schneider <[email protected]>

Change order in 'README.md', 'util-conf.R' and 'util-data.R'
Also fix table of contents in the readme.

See #193

Signed-off-by: Niklas Schneider <[email protected]>

See #193

Signed-off-by: Niklas Schneider <[email protected]>

Add the package in 'install.R' and a description in the 'README.md'.
Also rearrange the parameter descriptions of 'ProjectConf' to be sorted
alphabetically.

See #193

Signed-off-by: Niklas Schneider <[email protected]>

Use the new data.table package to replace do.call with data.table::rbindlist
which is faster in processing data.frames.

See #193

Signed-off-by: Niklas Schneider <[email protected]>

Signed-off-by: Niklas Schneider <[email protected]>

See #193

Signed-off-by: Niklas Schneider <[email protected]>

Remove hardcoded string formatting and replace it in tests for creating
expected data using the new function 'format.commit.ids'.

See #193

Signed-off-by: Niklas Schneider <[email protected]>

Follow @clhunsen's advice to create commit message data with an
lapply to avoid having a for-loop and an additional lapply call
afterwards

See #193

Signed-off-by: Niklas Schneider <[email protected]>

Add check for the ProjectConf attribute 'commit.messages' before
calling 'update.commit.messages'.
Also fix a few errors in comments as well as one if condition where
the wrong attribute was checked.

See #193

Signed-off-by: Niklas Schneider <[email protected]>

Add the getter call to the 'additional.data' list.

See #193

Signed-off-by: Niklas Schneider <[email protected]>

See #193

Signed-off-by: Niklas Schneider <[email protected]>

Add (empty) commit message data to all data split tests in 'tests-split.R'.
Also sor the additional data sources alphabetically in the tests.

See #193

Signed-off-by: Niklas Schneider <[email protected]>

Add (empty) commit message data to all data split tests in
'test-split-sliding-window.R'.
Also sort the additional data sources alphabetically in the tests.

See #193

Signed-off-by: Niklas Schneider <[email protected]>

See #193

Signed-off-by: Niklas Schneider <[email protected]>

These functions act like cleanup.pasta, they remove rows from the data
that are not part of a range data object anymore.

See #193

Signed-off-by: Niklas Schneider <[email protected]>

Fix a copy-paste-error where a varibale name was not changed from a
prior code snippet.

See #193

Signed-off-by: Niklas Schneider <[email protected]>

See #193

Signed-off-by: Niklas Schneider <[email protected]>

Warn user when updating commit messages or synchronicity that they should
call the corresponding cleanup method.
Remove calls to unlist in those cleanup functions.
Fix copy-pasted comments.

See #193

Signed-off-by: Niklas Schneider <[email protected]>

Fix the regex that removes spaces at the end of a commit message.
Change the assignment of the 'commit.message.data' data frame such that
no new data frame is instantiated anymore.

See #193

Signed-off-by: Niklas Schneider <[email protected]>

Our CI pipeline does currently not work for "R-3.3" (but for all the other
R versions in the pipeline). The reason for that is the following:
As of 2021-01-26, there is a new version of package `memoise` (2.0.0),
which is imported by `RSQLite`, which in turn is imported by `sqldf`.
The new version of `memoise` imports package `cachem`, which in turn
imports package `fastmap`. Using all these packages is not a problem per
se, they are compatible with R 3.3.3. However, the docker container
`r-base:3.3.3`, which we use in our pipeline "R-3.3", uses g++ version
6.3.0-9, which contains a bug. Due to the bug in the compiler, `fastmap`
cannot be compiled and installed in tje `r-base:3.3.3` docker container,
letting the CI pipeline fail.

To circumvent this problem but still keep "R-3.3" in our CI pipeline, we
now use docker image `r-base:3.3.2` instead, as there is another g++
version installed in this docker image. With that, the CI pipeline should
succeed for "R-3.3" again.

Signed-off-by: Thomas Bock <[email protected]>

Add commit message merge functionality

Reviewed-by: Claus Hunsen <[email protected]>
Reviewed-by: Thomas Bock <[email protected]>
Reviewed-by: Christian Hechtl <[email protected]>

Make more clear what the parameters do by referring to the igraph documentation
and add default value TRUE to parameters 'directed' and 'unconnected'.

See #181

Signed-off-by: Niklas Schneider <[email protected]>

See #195

Signed-off-by: Niklas Schneider <[email protected]>

Add function to replace manual checks for smallworldness similar to the one
for scale freeness.
Adapt existing comments to code style conventions and remove the ones
mentioning manual checks for smallworldness.

See #195

Signed-off-by: Niklas Schneider <[email protected]>

Add a try clause to the 'erdos.renyi.game' function call to prevent an
unreadable error message; print a user warning and abort the operation
instead.

See #195

Signed-off-by: Niklas Schneider <[email protected]>

Fix default values from vectors to their respective first element because
of calls to 'match.arg'.
Also correct a comment stating a wrong return value.

See #195

Signed-off-by: Niklas Schneider <[email protected]>

If there are no vertices in the network, print a warning and return NA
istead of letting igraph run into an error.

See #195

Signed-off-by: Niklas Schneider <[email protected]>

Add a function to simplify getting a data frame containing author names
and their respective centrality values.

See #195

Signed-off-by: Niklas Schneider <[email protected]>

Create a new helper function to be used by 'metrix.centrality' in order
to compute the authors from the data.sources of the networks's relations
automatically.

See #195

Signed-off-by: Niklas Schneider <[email protected]>

Sort entries of resulting data frame and check whether the new default value
(that is, computing the data.source directly from the network relations)
should be used or not.

See #195

Signed-off-by: Niklas Schneider <[email protected]>

Change order of functions in 'util-networks.R'.
The function introduced in previous commits to get the authors
from relations now only computes the data.sources and leaves the
author computation to its caller.

See #195

Signed-off-by: Niklas Schneider <[email protected]>

Check for a character(0) relation and set NA as the resulting data source.

See #195

Signed-off-by: Niklas Schneider <[email protected]>

See #195

Signed-off-by: Niklas Schneider <[email protected]>

Add two tests in 'test-networks.R':
- The first one checks the data sources of a multi-network with two relations
- The second one checks the data source of an author-network
Fix a bug where the function would return a named vector, now it only
returns a value vector.

See #195

Signed-off-by: Niklas Schneider <[email protected]>

See #195

Signed-off-by: Niklas Schneider <[email protected]>

See #195

Signed-off-by: Niklas Schneider <[email protected]>

Fix minor documentation issues.
Remove check for NA in 'metrics.is.smallworld' as it is unnecessary.
Change error handling in 'metrics.smallworldness': Now check beforehand
if the network is simplified and stop the execution if this is not the case.

See #195

Signed-off-by: Niklas Schneider <[email protected]>

Change 'author' to 'vertex', rename parameter and rename the function
to 'metrics.vertex.centralities'.
Fix minor documentation issues.

See #195

Signed-off-by: Niklas Schneider <[email protected]>

Signed-off-by: Niklas Schneider <[email protected]>

See #195

Signed-off-by: Niklas Schneider <[email protected]>

See #195

Signed-off-by: Niklas Schneider <[email protected]>

Signed-off-by: Niklas Schneider <[email protected]>

As the default restrict parameter for 'metrics.centrality' depends
on the network's vertices, add checks and correpsonding calls to
'get.artifact' if a network contains artifact vertices.
Vectorize 'get.artifact' in 'util-data.R' in order to be able to
process and return artifacts from different data sources. The function
now takes a vector of data sources (or still a single one) and
returns all correpsonding artifacts.

See #195

Signed-off-by: Niklas Schneider <[email protected]>

Signed-off-by: Niklas Schneider <[email protected]>

Add the case that there are both authors and artifact vertices in the
network. In this case compute the lists for both and set the restrict
parameter to the combination of these two.

See #195

Signed-off-by: Niklas Schneider <[email protected]>

Clarify 'unconnected' parameter in 'metrics.avg.pathlength'.
Remove debug print statements in 'util-data.R'.

See #195

Signed-off-by: Niklas Schneider <[email protected]>

Now that there can be multiple values in the argument, the
'match.arg.or.default' function shows a different behaviour. Instead of
returning only the first element of the default vector when no argument
is passed, it returns the whole vector.
Add manual checks for a missing argument and set it to the first element
in this case. Inspired from the implementation of 'match.arg.or.default'

See #195

Signed-off-by: Niklas Schneider <[email protected]>

Replace nasty if-statement similar to 'match.arg.or.default' with
simple trick achieving the same effect.

See #195

Signed-off-by: Niklas Schneider <[email protected]>

See #195

Signed-off-by: Niklas Schneider <[email protected]>

Signed-off-by: Niklas Schneider <[email protected]>

Replace several s- and lapply calls with a single one.

See #195

Signed-off-by: Niklas Schneider <[email protected]>

Fixes in the metrics module

Reviewed-by: Christian Hechtl <[email protected]>
Reviewed-by: Thomas Bock <[email protected]>

Add a new directory 'logo' which should contain all the logo variants.

Signed-off-by: Thomas Bock <[email protected]>

Use our README.md file for automatically generating a GH-pages website
for coronet.

To achieve that, a _config.yml is necessary in which we specify how the
website should be generated.

Also links in the README.md should be relative links, otherwise the
links will only work in the README rendered on the GitHub repo, but not
on the generated GH-pages website.

In addition, also fix some outdated links, add missing section to table
of contents, etc.

Add favicon to README.md, CONTRIBUTING.md, and NEWS.md.

Signed-off-by: Thomas Bock <[email protected]>

Add several logo variants, both in svg and png format:

1. Coronet logo without font

2. Coronet logo with font

3. Coronet favicon

4. Coronet logo without font, white background

5. Coronet logo with font, white background

6. Coronet logo for dark mode without font

7. Coronet logo for dark mode with font

Signed-off-by: Thomas Bock <[email protected]>

Add svg files which contain all logo variants using different tiers
(normal, dark mode, white background, black background, etc.),
one without font and one with font.

Signed-off-by: Thomas Bock <[email protected]>

As Google now uses BigCamelCase naming convention instead of '.' as separator
in names, just linking to Google's guide is not useful any more. So, in
addition, add a link to Google's previous xml file containing the old naming
conventions (which unfortunatley cannot be rendered on GitHub any more).

Also add one sentence that pull requests should not only be opened against
'dev' branch but also base on 'dev' branch.

Signed-off-by: Thomas Bock <[email protected]>

Signed-off-by: Thomas Bock <[email protected]>

Logo & Website

Special thanks to Christian Kaltenecker <[email protected]>

Reviewed-by: Claus Hunsen <[email protected]>
Reviewed-by: Christian Hechtl <[email protected]>

…thor for issue #188

Signed-off-by: Johannes Hostert <[email protected]>

…#188

Signed-off-by: Johannes Hostert <[email protected]>

Tests for PRs are missing since there is no test data including PRs yet

Signed-off-by: Johannes Hostert <[email protected]>

…n all varieties one can imagine (and more)

Signed-off-by: Johannes Hostert <[email protected]>

Signed-off-by: Johannes Hostert <[email protected]>

Signed-off-by: Johannes Hostert <[email protected]>

Signed-off-by: Johannes Hostert <[email protected]>

…t.issues to make it similar to get.commits[.filtered]

Close #185.

Signed-off-by: Johannes Hostert <[email protected]>

Signed-off-by: Johannes Hostert <[email protected]>

Signed-off-by: Johannes Hostert <[email protected]>

Signed-off-by: Johannes Hostert <[email protected]>

Signed-off-by: Johannes Hostert <[email protected]>

Signed-off-by: Johannes Hostert <[email protected]>

Signed-off-by: Johannes Hostert <[email protected]>

Signed-off-by: Johannes Hostert <[email protected]>

* Use match.arg in get.author.issue.count and related
* Enhance documentation of preprocess.issue.data to clarify it using the unfiltered issue data
* Add get.issues.filtered.uncached similar to get.commits.filtered.uncached
* Enhance issue count unit tests:
  - use "issue" relation instead of "cochange"
  - add test covering the "all" case, i.e. sums of issues.only and prs.only

Signed-off-by: Johannes Hostert <[email protected]>

Signed-off-by: Johannes Hostert <[email protected]>

Signed-off-by: Johannes Hostert <[email protected]>

Signed-off-by: Johannes Hostert <[email protected]>

Signed-off-by: Johannes Hostert <[email protected]>