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

Benchmark #5

Open
LAAP opened this issue Mar 19, 2020 · 3 comments
Open

Benchmark #5

LAAP opened this issue Mar 19, 2020 · 3 comments
Assignees

Comments

@LAAP
Copy link

LAAP commented Mar 19, 2020

Are we going to have a Benchmark like in Volpe? Which one? Barcelona, Volpe, a synthetic one based in North American cities...?

@doorleyr
Copy link
Collaborator

There is now a front-end issue which includes handling of multiple scenarios on the indicators plots (eg. current vs benchmark or baseline).

We need to decide how to handle this information in the module. In the Corktown project, the reference scenario will be the baseline i.e. calculations based on data for the existing site with all geogrid cells set to None. (in another project, the reference scenario could be a different district).

Maybe each indicator needs to have a function: calculate_baseline()
which outputs the baseline indicator values for that district (either by reading and analyzing baseline data or by reading data produced manually etc.). Then we would pass a parameter to the Handler indicating whether the baseline values should be included with each indicator.
@crisjf what do you think?

@crisjf
Copy link
Collaborator

crisjf commented Apr 23, 2020

That sounds good. Let's not pass the parameter though, let the Handler take care of it by checking if the user has defined the calculate_baseline function.

Given how the Handler behaves, the calculate_baseline function can return a dictionary with multiple scenarios. Something like:

def return_baseline(self):
    barcelona = 1
    paris = 2
    return {'baseline': paris, 'benchmark': barcelona}

@crisjf crisjf self-assigned this Apr 23, 2020
@crisjf
Copy link
Collaborator

crisjf commented Apr 23, 2020

commit 784f3b1 adds placeholder return_baseline function to Indicator class. The Handler class still does not know what to do with the return_baseline function.

crisjf added a commit that referenced this issue May 22, 2020
…get employment by metro area by NAICS code; will be useful for benchmark indicators (we can choose to calculate every indicator that uses NAICS for diferent cities #5) and the rankings in #4
crisjf added a commit that referenced this issue May 22, 2020
…owledge and skills indicator for all 3-digit NAICS categories. Useful for building scenarios #5
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

No branches or pull requests

3 participants