Skip to content

Beta diversity tweaks#114

Open
Glycocalex wants to merge 3 commits intoBojarLab:devfrom
Glycocalex:dev
Open

Beta diversity tweaks#114
Glycocalex wants to merge 3 commits intoBojarLab:devfrom
Glycocalex:dev

Conversation

@Glycocalex
Copy link
Copy Markdown
Contributor

⚠️ IMPORTANT: Branch Strategy

This repository follows a strict branch strategy:

  • master branch is ONLY for PyPI release mirroring
  • All development PRs MUST target the dev branch
  • If your PR targets master, it will be flagged and you'll be asked to retarget to dev

Description of Changes

beta diversity output changed to include distance matrix
...

Changelog

  • I have added my changes to CHANGELOG.md under the appropriate module/submodule section

@Glycocalex Glycocalex requested a review from Bribak April 7, 2026 15:00
@codacy-production
Copy link
Copy Markdown

codacy-production bot commented Apr 7, 2026

Up to standards ✅

🟢 Issues 1 high · 1 medium

Results:
2 new issues

Category Results
UnusedCode 1 medium
ErrorProne 1 high

View in Codacy

🟢 Metrics 0 complexity · 0 duplication

Metric Results
Complexity 0 (≤ 20 complexity)
Duplication 0 (≤ 15 duplication)

View in Codacy

TIP This summary will be updated as you push new changes. Give us feedback

@Bribak
Copy link
Copy Markdown
Contributor

Bribak commented Apr 8, 2026

Thank you! Will be a nice improvement

Just a few to-dos here:

  • no need to import DataFrame, since we typically just use pd.DataFrame
  • no need to import List, since we use the nice typing introduced by Python 3.10
  • you don't need to wrap the outputs into a list, just having both outputs in the same return line will output a tuple, which is better
  • You'll need to update the test_get_biodiversity unit tests in test_core_functions to deal with this changed output format, so that the tests will run through successfully before I merge:-)

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.

2 participants