Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Python and C++ code refactoring #9

Open
wants to merge 41 commits into
base: master
Choose a base branch
from
Open

Python and C++ code refactoring #9

wants to merge 41 commits into from

Conversation

carloslihu
Copy link

  • Python code is now formatted with black and isort, and has been refactored according to PEP 8 style guides.
  • Python code partially commented with google docstring format.
  • C++ code partially commented with doxygen docstring format.
  • Scott's and Normal Reference Rule's bandwidth calculation have been reordered and commented.
  • ArcOperatorSet::update_incoming_arcs_scores formulas have been reordered and commented.

davenza and others added 30 commits September 10, 2024 21:07
@davenza
Release v0.5.1.
d95189a
@davenza
Updates cibuildwheel to support manylinux.
0e38801
@carloslihu
Python code refactored with "black"
cdb619c
@carloslihu
python imports sorted
d25d935
@carloslihu
Merge remote-tracking branch 'upstream/master' into chore/refactoring
8a3bcb6
@carloslihu
extra files removed
10265b3
@carloslihu
Pytests improved and partially commented
a61d2d2
@carloslihu
util_test imports updated
4cebd86
@carloslihu
dataset/ commented
8dc996a
@carloslihu
util commented
00acbc4
@carloslihu
partial commenting
c346b66
@carloslihu
more comments
dd0a4af
@carloslihu
scotts and normal reference bandwidth recoded
297966a
@carloslihu
operators operations reordered
67b83ce
@carloslihu
pytest warning fixed
fd653a4
@carloslihu
extra comments cleaned
3daec54
@carloslihu
0.5.2 changelog updated
e2f80f7
@carloslihu
set unix compilers
42d2e18
@carloslihu
ifdef warnings corrected
54684ad
@carloslihu
typo
045360a
@carloslihu
DiscreteFactor_test fix
3387401
@carloslihu
code formatted
44c0e8d
@carloslihu
pyproject.toml formatted
52613df
@cli-titanium
warning fix
c8c66a8
@cli-titanium
eigen warnings fixed
1d28196
@cli-titanium
util_test upgraded and renamed
c84a04d
@cli-titanium
Tests adapted: Variable naming to uppercase
febdea3
@cli-titanium
util_test -> data
1213cdb
@cli-titanium
pytest -s
0a31a17
@cli-titanium
data.py reviewed
1d01b69
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@carloslihu @davenza @cli-titanium @jcasajus-titanium @JuanFPR-UPM