Skip to content

Commit

Permalink
share new managed lane notebook bug
Browse files Browse the repository at this point in the history
  • Loading branch information
@DavidOry
DavidOry committed Aug 19, 2020
1 parent 2a01239 commit d4173ff
Showing 1 changed file with 63 additions and 206 deletions.
Loading

4 comments on commit d4173ff

@DavidOry
Copy link
Member Author

@DavidOry DavidOry commented on d4173ff Aug 19, 2020 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@i-am-sijia, #104

When I run my Managed Lane notebook now (which I've moved to Lasso), I get an error with self.calculate_county(), which is new. May this be related to the recent changes we've been making with the shapes?

@i-am-sijia
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I cannot replicate the error, gonna try with a clean install of environment.

@i-am-sijia
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Still cannot replicate the error with new installation, @e-lo can you also test?

@DavidOry
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hmm. Strange. It works with an older conda configuration. Can you please test with a fresh install using the updated Lasso readme directions, specifically:

conda config --add channels conda-forge
conda create python=3.7 rtree geopandas folium osmnx -n <my_lasso_environment>

The configuration that works for me as uniformly older packages, which is a bit worrying.

Please sign in to comment.