stanford-crfm / ecosystem-graphs

257 stars 35 forks source link

[Don't need to merge; superseded by #87] Fix all errors so website loads #86

Closed rishibommasani closed 1 year ago

rishibommasani commented 1 year ago

@tholiao This should fully fix all issues (the website loads locally for me).

  1. In the previous commit/PR I addressed linting. You can just run yamllint (which is described in precommit.sh, and also will be run as an automatic check for stuff that gets merged) to identify error.
  2. Beyond linting errors, there are several things that break (which is why I want you to look at this PR). These are:

(i) You need to enter something for each field, even if empty string using '', otherwise it breaks the type checking. (ii) dependencies is a list, so when empty, its needs to be [] not ''. (iii) For Galactica, the field names got screwed up. I imagine you entered that one manually or something, but just noting since obviously that breaks (e.g. it sayed created_data and training_emission {missing the s}).

As an aside, let's avoid the "large" in LLM and just say LM, since we neither use that descriptor for any other modality nor do we use it in a terrible useful way if stuff can be large that is 100M/1B params and also 500B+ params.