search

censoredplanet / censoredplanet-analysis

Analysis of the CensoredPlanet data.
Apache License 2.0
14 stars 4 forks source link

update satellite outcome names #247

Closed ohnorobo closed 1 year ago

ohnorobo commented 1 year ago

based on the list here: https://docs.google.com/spreadsheets/d/131vLGGeLF-3_5ssKcGamrFhlo7eo_XQV0CiEzL24_7I/edit?resourcekey=0-6pSwdwt3v73XL5wKfvgzNg#gid=1240852033

This is currently branched off my weird side branch, but should be mergable once that is merged.

Example dashboard using these new strings: https://lookerstudio.google.com/c/u/0/reporting/2afecc26-2865-47e0-9384-76bf9356ba99/page/p_011unrxq4c

(ignore the CI error, it's a mypy update)

ohnorobo commented 1 year ago

Can we change HTTPS and HTTP as well?

That would have to happen in https://github.com/censoredplanet/censoredplanet-analysis/blob/master/pipeline/metadata/hyperquack_outcome.py and would be a seperate PR from this one.

On the upside that change could be merged directly into master, while this one can't be currently.

ohnorobo commented 1 year ago

Closing in favor of https://github.com/censoredplanet/censoredplanet-analysis/pull/257