Bespoke Layout Performance

[WilliamsTravis]

Bug Description
This function is not quick enough as caching does not appear to working properly. Check the caching or only pull one point at a time from the data set.

[alexandermorgan]

You could rely on the reset_index method's default of returning a copy to avoid making your own copy and save a fair amount of work here:

Current:

    sc_developable_df = sc_df[sc_df['capacity'] > 0].copy()
    # reset the index because otherwise the unpacker will get messed up
    sc_developable_df.reset_index(drop=True, inplace=True)

Alternative:
sc_developable_df = sc_df[sc_df['capacity'] > 0].reset_index(drop=True)

There are a few other similarly simple speed ups available in this repo by avoiding making unnecessary iterables, using translate tables instead of multiple calls to str.replace, etc. I'm happy to make a pr if you're looking for this sort of thing. This is a really cool project!

[WilliamsTravis]

Hello, yes, that all sounds excellent, glad to see someone outside our little group using reView. I've been quite busy with other projects for a while and I would definitely appreciate a pull request for this piece!