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

Data_Engine: Add clean list method #1993

Closed
IsakNaslundBh opened this issue Sep 15, 2020 · 0 comments · Fixed by #1994
Closed

Data_Engine: Add clean list method #1993

IsakNaslundBh opened this issue Sep 15, 2020 · 0 comments · Fixed by #1994
Assignees
@IsakNaslundBh
Labels
severity:critical No workaround exists. Essential to continue type:bug Error or unexpected behaviour type:compliance Non-conforming to code guidelines
Milestone
BHoM 3.3 β Release

Comments

@IsakNaslundBh
Copy link
Contributor

Description:

Method for cleaning list (removing nulls) was correctly removed from the Excel_Engine in https://github.com/BHoM/Excel_Toolkit/pull/243 .

This method was however not replaced in the Data_Engine during this refactoring. Having the ability to clean up lists of objects in excel is critical and hard to achieve in Excel without a method of this sort, with no simple workaround, and this is functionality that has now gone missing from previous beta.

This is for those reasons proposed to be pushed as a critical issue to be included in the beta release, even if feature of nature, as critical for usability of the Excel_Toolkit for several workflows.
@IsakNaslundBh IsakNaslundBh added severity:critical No workaround exists. Essential to continue type:feature New capability or enhancement labels Sep 15, 2020
@IsakNaslundBh IsakNaslundBh added this to the BHoM 3.3 β Release milestone Sep 15, 2020
@IsakNaslundBh IsakNaslundBh self-assigned this Sep 15, 2020
@IsakNaslundBh IsakNaslundBh added type:bug Error or unexpected behaviour type:compliance Non-conforming to code guidelines and removed type:feature New capability or enhancement labels Sep 15, 2020
@IsakNaslundBh IsakNaslundBh mentioned this issue Sep 15, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@IsakNaslundBh IsakNaslundBh

Labels
severity:critical No workaround exists. Essential to continue type:bug Error or unexpected behaviour type:compliance Non-conforming to code guidelines
Projects
None yet
Milestone
BHoM 3.3 β Release
Development

Successfully merging a pull request may close this issue.

Data_Engine: Adding CleanList method BHoM/BHoM_Engine
1 participant
@IsakNaslundBh