Fix: Astrocyte grower infinite loop

[jacquemi-bbp]

• append a copy of point into the points list instead of a refence to the point.

Description

Fix 103

Checklist

This pull request is:

• A documentation / typographical error fix
  • Good to go, no issue or tests are needed
• A short code fix
  • Please include: Fixes: #<issue number> in the description if it solves an existing issue
    (which must include a complete example of the issue).
  • Please include tests that fail with the main branch and pass with the provided fix.
• A new feature implementation or update an existing feature
  • Please include: Fixes: #<issue number> in the description if it solves an existing issue
    (which must include a complete example of the feature).
  • Please include tests that cover every lines of the new/updated feature.
  • Please update the documentation to describe the new/updated feature.

[jacquemi-bbp]

How does the versioning of this package works ?
Do I need to change something for the versioning ?

[eleftherioszisis]

How does the versioning of this package works ? Do I need to change something for the versioning ?

No, I don't think so. Versioning should be automatic when a new release is made.

[arnaudon]

write Fix: blabla in commit title, and remove #103, you can add this in description

[arnaudon]

ah, this changes the algo a bit apparently, as a test break, I guess you can update the test, it should not be such a big deal, but @eleftherioszisis should know more

[eleftherioszisis]

I guess you can update the test, it should not be such a big deal, but @eleftherioszisis should know more

It changes the tests, but it's indeed a bug, therefore it's ok to update them.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.86%. Comparing base (d47bf8d) to head (fb82c3f).

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #104   +/-   ##
=======================================
  Coverage   97.86%   97.86%           
=======================================
  Files          39       39           
  Lines        2201     2201           
  Branches      383      383           
=======================================
  Hits         2154     2154           
  Misses         29       29           
  Partials       18       18           

Flag	Coverage Δ
pytest	97.86% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files	Coverage Δ
neurots/astrocyte/section.py	80.24% <100.00%> (ø)

[jacquemi-bbp]

Is anything missing to merge the PR ?