Skip to content

Conversation

@mehmetbercan
Copy link

No description provided.

@aerispaha
Copy link
Member

Thanks for this contribution, @mehmetbercan! The reason the checks are failing is a because the new column headers introduced by the "Node Inflow Summary" section of the RPT file conflict with column headers already brought into the nodes dataframe by default. This results in this error a failed test_model_to_networkx unit test. Here's the main error message:

ValueError: columns overlap but no suffix specified: Index(['MaxLatInflow', 'MaxTotalInflow', 'LatInflowV', 'TotalInflowV', 'FlowBalErrorPerc']

I think we need to consider a couple things to resolve this:

  1. we should be consistent with what columns are called
  2. we should potentially let users define what columns should be return when calling model.nodes(), similar to Optionally specify which attributes to return in model.conduits() ? #36
  3. an easy fix would be to specific a left/right suffix to Pandas when attempting to join dataframes with non-unique columns names

@aerispaha aerispaha added this to the v0.4.0 milestone Apr 22, 2020
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