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.

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

Table.remove_columns(): Specify what happens when removing the last column, so the table gets empty #276

Closed
zzril opened this issue May 5, 2023 · 2 comments · Fixed by #384
Assignees
Labels
documentation 📖 Improvements or additions to documentation enhancement 💡 New feature or request question Further information is requested released Included in a release

Comments

@zzril
Copy link
Contributor

zzril commented May 5, 2023

Is your feature request related to a problem?

When removing the only column in a 1-column-table, the number_of_rows property is set to 0. This behaviour should be documented in the Table class docstring as well as in the following methods: Table.keep_only_columns, Table.remove_columns, Table.remove_columns_with_missing_values and Table.remove_columns_with_non_numerical_values.

Desired solution

  • document the behaviour

Possible alternatives (optional)

No response

Screenshots (optional)

No response

Additional Context (optional)

No response

@zzril zzril added documentation 📖 Improvements or additions to documentation enhancement 💡 New feature or request question Further information is requested labels May 5, 2023
@github-project-automation github-project-automation bot moved this to Backlog in Library May 5, 2023
@lars-reimann
Copy link
Member

lars-reimann commented May 5, 2023

Might be related to #123. In general, an empty table should have row count 0, but some methods might not be able to work properly with an empty table yet.

Example: It should be possible to add any column to an empty table.

@alex-senger alex-senger moved this from Backlog to Todo in Library Jun 23, 2023
@alex-senger alex-senger moved this from Todo to Ready for Review in Library Jun 23, 2023
alex-senger added a commit that referenced this issue Jun 23, 2023
…e last column (#384)

Closes #276 

### Summary of Changes

When removing the only column in a 1-column-table, the number_of_rows
property is set to 0. This behaviour should be documented in the Table
class docstring as well as in the following methods:
Table.keep_only_columns, Table.remove_columns,
Table.remove_columns_with_missing_values and
Table.remove_columns_with_non_numerical_values.

<!-- Please provide a summary of changes in this pull request, ensuring
all changes are explained. -->

---------

Co-authored-by: patrikguempel <128832338+patrikguempel@users.noreply.github.com>
@github-project-automation github-project-automation bot moved this from Ready for Review to ✔️ Done in Library Jun 23, 2023
@lars-reimann
Copy link
Member

🎉 This issue has been resolved in version 0.14.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@lars-reimann lars-reimann added the released Included in a release label Jun 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation 📖 Improvements or additions to documentation enhancement 💡 New feature or request question Further information is requested released Included in a release
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants