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

Fix the link check for Github sites #61

Merged
merged 8 commits into from
Nov 28, 2022
Merged

Fix the link check for Github sites #61

merged 8 commits into from
Nov 28, 2022

Conversation

shankari
Copy link
Collaborator

By passing in the correct config file for the link checker code github/docs#17358

By passing in the correct config file for the link checker code
github/docs#17358
Without the fix, we get the error

```
Traceback (most recent call last):

  File "/srv/conda/envs/notebook/lib/python3.7/site-packages/IPython/core/interactiveshell.py", line 3552, in run_code
    exec(code_obj, self.user_global_ns, self.user_ns)

  File "/tmp/ipykernel_59/3601587137.py", line 2, in <module>
    import emeval.input.spec_details as eisd

  File "<fstring>", line 1
    (post_body=)
              ^
SyntaxError: invalid syntax
```

This works on my local environment with python 3.8.2

```
Set up the dependencies

import emeval.input.spec_details as eisd
import emeval.input.phone_view as eipv
import emeval.input.eval_view as eiev

import sys
sys.version

'3.8.2 (default, Mar 25 2020, 11:22:43) \n[Clang 4.0.1 (tags/RELEASE_401/final)]'
```

So forcing the environment to 3.8.2 so we don't have to go around and fix all the f-strings
Since version 16 is no longer supported
@shankari
Copy link
Collaborator Author

Current failure:

     "ename": "AssertionError",
     "evalue": "not found: data_file='/home/shankari/bin/data/[email protected]/train_bus_ebike_mtv_ucb/config~evaluation_spec/0_9223372036854775807.json'",

Not sure how this will ever work unless we have the data files checked in, given that cardshark is also down...
Let's check the data file sizes and see if we can check them in...

@shankari
Copy link
Collaborator Author

GitHub warns for greater than 50MB, fails for greater than 100MB.
The biggest files we have are ~ 20M so let's go ahead and check them in, barring coordination for storing them in a better location.

- Remove line related to `bin/data` from the gitignore
- Check in all the data files pending coordination with NREL data storage folks
Check in the related data files
Fix the evaluation view to take the time range for the ground truth retrieval into account
Change the notebook to read from the spec and use the new eval view function
In the data exploration template as well
Follow-on to e7c1749
@shankari shankari merged commit 719bb19 into master Nov 28, 2022
@shankari shankari deleted the fix_github_actions branch November 28, 2022 04:30
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.

1 participant