Skip to content

Increase days of no Internet connectivity to 30#1096

Closed
ErichMoraga wants to merge 2 commits intocommaai:masterfrom
ErichMoraga:patch-1
Closed

Increase days of no Internet connectivity to 30#1096
ErichMoraga wants to merge 2 commits intocommaai:masterfrom
ErichMoraga:patch-1

Conversation

@ErichMoraga
Copy link
Copy Markdown
Contributor

@geohot
Copy link
Copy Markdown
Contributor

geohot commented Feb 14, 2020

Instead, we made it so you can reboot to reset this. Fair?

Also, if this is broken at all, we def want to fix it.

@geohot geohot closed this Feb 14, 2020
@ErichMoraga
Copy link
Copy Markdown
Contributor Author

Rebooting is a fair compromise, and should encourage basic users to get their devices online regularly. Next time someone complains about the warning, I'll advise they reboot, and if that doesn't do the trick, we'll investigate accordingly.

@geohot
Copy link
Copy Markdown
Contributor

geohot commented Feb 14, 2020

Sounds good

@ErichMoraga ErichMoraga deleted the patch-1 branch February 14, 2020 21:20
ErichMoraga added a commit to ErichMoraga/openpilot that referenced this pull request May 22, 2020
ErichMoraga added a commit to ErichMoraga/openpilot that referenced this pull request May 23, 2020
ErichMoraga added a commit to ErichMoraga/openpilot that referenced this pull request Jun 2, 2020
ErichMoraga added a commit to ErichMoraga/openpilot that referenced this pull request Jul 21, 2020
ErichMoraga added a commit to ErichMoraga/openpilot that referenced this pull request Aug 21, 2020
sshane added a commit to sshane/openpilot that referenced this pull request Sep 5, 2020
* SnG hack

* Eased up Internet requirement

See...
commaai#1096

* VBATT CHARGING & Turn off charge after 3 hrs

* Prii ZSS w/ Prime-centric tuning, etc.

* Minimized custom fork warning

* Update interface.py

* Update thermald.py

* Update interface.py

* Update interface.py

* Update lane_planner.py

* ZSS Prius INDI Tune

Co-authored-by: ErichMoraga <33645296+ErichMoraga@users.noreply.github.com>
Co-authored-by: ErichMoraga <mailto:ejmoraga@gmail.com|ejmoraga@gmail.com>
Co-authored-by: d412k5t412 <36134180+d412k5t412@users.noreply.github.com>
sshane added a commit to sshane/openpilot that referenced this pull request Sep 6, 2020
* merge ZSS to SA (#197)

* SnG hack

* Eased up Internet requirement

See...
commaai#1096

* VBATT CHARGING & Turn off charge after 3 hrs

* Prii ZSS w/ Prime-centric tuning, etc.

* Minimized custom fork warning

* Update interface.py

* Update thermald.py

* Update interface.py

* Update interface.py

* Update lane_planner.py

* ZSS Prius INDI Tune

Co-authored-by: ErichMoraga <33645296+ErichMoraga@users.noreply.github.com>
Co-authored-by: ErichMoraga <mailto:ejmoraga@gmail.com|ejmoraga@gmail.com>
Co-authored-by: d412k5t412 <36134180+d412k5t412@users.noreply.github.com>

* revert chmod

* revert chmod

* revert comment for standstill_req

* revert interface changes

* revert all carstate changes

* add code to detect ZSS

* fix

* add ZSS code

* use needs_angle_offset

* switch around for diff readability

* diff readability

* diff readability

* test

* this should work

* move code

* ugh, fix

* this should work

* use hex like the other fp checks

* ZSS tuning

* revert to test

* better implementation of getting ZSS

* better implementation of getting ZSS

* revert

* follow naming conventions

* simplify hasZss tuning

* one line

* debug

* rename

* gather more data

* remove debugging

* just for safety

* this is probably better

* nah this is fine

Co-authored-by: ErichMoraga <33645296+ErichMoraga@users.noreply.github.com>
Co-authored-by: ErichMoraga <mailto:ejmoraga@gmail.com|ejmoraga@gmail.com>
Co-authored-by: d412k5t412 <36134180+d412k5t412@users.noreply.github.com>
ErichMoraga added a commit to ErichMoraga/openpilot that referenced this pull request Oct 6, 2020
ErichMoraga added a commit to ErichMoraga/openpilot that referenced this pull request Oct 24, 2020
ErichMoraga added a commit to ErichMoraga/openpilot that referenced this pull request Nov 27, 2020
ErichMoraga added a commit to ErichMoraga/openpilot that referenced this pull request Dec 18, 2020
ErichMoraga added a commit to ErichMoraga/openpilot that referenced this pull request Feb 27, 2021
ErichMoraga added a commit to ErichMoraga/openpilot that referenced this pull request Mar 30, 2021
ErichMoraga added a commit to ErichMoraga/openpilot that referenced this pull request May 20, 2021
ErichMoraga added a commit to ErichMoraga/openpilot that referenced this pull request May 26, 2021
ErichMoraga added a commit to ErichMoraga/openpilot that referenced this pull request Jun 11, 2021
ErichMoraga added a commit to ErichMoraga/openpilot that referenced this pull request Jul 23, 2021
ErichMoraga added a commit to ErichMoraga/openpilot that referenced this pull request Aug 7, 2021
sobomax pushed a commit to sobomax/openpilot that referenced this pull request Aug 10, 2021
ErichMoraga added a commit to ErichMoraga/openpilot that referenced this pull request Aug 27, 2021
ErichMoraga added a commit to ErichMoraga/openpilot that referenced this pull request Sep 11, 2021
ErichMoraga added a commit to ErichMoraga/openpilot that referenced this pull request Oct 28, 2021
I like 1 month, since it's the avg. interval b/w releases.

See...
commaai#1096
commaai#22607
ErichMoraga added a commit to ErichMoraga/openpilot that referenced this pull request Nov 29, 2021
I like 1 month, since it's the avg. interval b/w releases.

See...
commaai#1096
commaai#22607
wtogami pushed a commit to wtogami/openpilot that referenced this pull request Jul 31, 2025
* Tinygrad bump from sync-20250627

* bump tinygrad_repo

* Reformat metadata generator to match driving_models.json

* bump tinygrad

* Revert "bump tinygrad"

This reverts commit f479dfd.

* revert me after SP model compiled

* Model recompiled successfully, initiate "revert me after SP model compiled"

This reverts commit 95706eb.

* The "FillMe" placeholder caused an extra 10 seconds of work

* bump to 22Jul2025

* Update build-all-tinygrad-models.yaml

* Update build-all-tinygrad-models.yaml

* Update build-all-tinygrad-models.yaml

* Update build-all-tinygrad-models.yaml

* Update build-all-tinygrad-models.yaml

* Update build-all-tinygrad-models.yaml

* Update build-all-tinygrad-models.yaml

* Allow more dynamic short names

This should hopefully be future-proof for now.. It's robust enough to return the correct word-digit format (see example on how it generates from given display name below):

'Last Horizon V2 (November 22, 2024)' -> LHV2
'Alabama (November 25, 2024)' -> ALABAMA
'PlayStation (December 03, 2024)' -> PLAYSTAT
'Postal Service (December 09, 2024)' -> PS
'Null Pointer (December 13, 2024)' -> NP
'North America (December 16, 2024)' -> NA
'National Public Radio (December 18, 2024)' -> NPR
'Filet o Fish (March 7, 2025)' -> FOF
'Tomb Raider 2 (April 18, 2025)' -> TR2
'Tomb Raider 3 (April 22, 2025)' -> TR3
'Tomb Raider 4 (April 25, 2025)' -> TR4
'Tomb Raider 5 (April 25, 2025)' -> TR5
'Tomb Raider 6 (April 30, 2025)' -> TR6
'Tomb Raider 7 (May 07, 2025)' -> TR7
'Down to Ride (Revision: May 10, 2025)' -> DTR
'SP Vikander Model (May 16, 2025)' -> SPVM
'VFF Driving (May 15, 2025)' -> VFFD
'Secret Good Openpilot (May 16, 2025)' -> SGO
'Vegetarian Filet o Fish (May 29, 2025)' -> VFOF
'Down To Ride (Revision: May 30, 2025)' -> DTR
'Vegetarian Filet o Fish v2 (June 05, 2025)' -> VFOFV2
'Kerrygold Driving (June 08, 2025)' -> KD
'Tomb Raider 10 (June 16, 2025)' -> TR10
'Organic Kerrygold (June 17, 2025)' -> OK
'Liquid Crystal Driving (June 21, 2025)' -> LCD
'Vegetarian Filet o Fish v3 (June 21, 2025)' -> VFOFV3
'Vibe Model [Custom Model]' -> VMCM
'Tomb Raider 13 (June 27, 2025)' -> TR13
'Aggressive TR (June 28, 2025)' -> ATR
'Tomb Raider 14 (June 30, 2025)' -> TR14
'Cookiemonster Tomb Raider (July 02, 2025)' -> CTR
'Down to Ride (Revision: July 07, 2025)' -> DTR
'Simple Plan Driving (July 07, 2025)' -> SPD
'Down to Ride (Revision: July 08, 2025)' -> DTR
'Tomb Raider 15 (July 09, 2025)' -> TR15
'Tomb Raider 15 rev-2 (July 11, 2025)' -> TR15R2
'Le Tomb Raider 14 (July 14, 2025)' -> LTR14
'Le Tomb Raider 14h (July 17, 2025)' -> LTR14H
'Tomb Raider 16 (July 18, 2025)' -> TR16
'Tomb Raider 16v2 (July 21, 2025)' -> TR16V2

* Update build-all-tinygrad-models.yaml

* Update build-all-tinygrad-models.yaml

* No need to sleep 3 seconds, just send it

* try dynamic

* cleanup

* Update build-single-tinygrad-model.yaml

* bc devtekve said. also, this is repetitive af

* Revert "bc devtekve said. also, this is repetitive af"

This reverts commit 3a0c156.

* maybe we could use a script instead that both build all

That both build all and sunnypilot-build-model reference

* refactor: consolidate model building steps into a single workflow

* tweak

* tweakx2

* tweakx3

* tweakx4

* dunno dunno...

* output dir

* lots of changes

* Revert "lots of changes"

This reverts commit 4aadb0e.

* fail if all fail

* no inputs needed

* make it easier for us

* note failure and exit 0

* Update build-all-tinygrad-models.yaml

* not needed unless we really want it

* Update build-single-tinygrad-model.yaml

* Merge branch 'sync-20250627-tinygrad' of github.com:sunnypilot/sunnypilot into sync-20250627-tinygrad

* retry for failed ?

* always run this step because sometimes one build fails

which causes the matrix to fail, but most builds still have uploaded artifacts.

* strip

* no escape

* Update build-all-tinygrad-models.yaml

* Test case from terminal run

(openpilot) james@Mac sunnypilot % jq -c '[.bundles[] | select(.runner=="tinygrad") | {ref, display_name: (.display_name | gsub(" \\([^)]*\\)"; "")), is_20hz}]' \
  /Users/james/Documents/GitHub/sunnypilot-docs/docs/driving_models_v6.json > matrix.json

mkdir -p output
touch "output/model-Tomb Raider 16v2 (July 21, 2025)-544"
touch "output/model-Space Lab Model (July 24, 2025)-547"
touch "output/model-Space Lab Model v1 (July 24, 2025)-548"

built=(); while IFS= read -r line; do built+=("$line"); done < <(
  ls output | sed -E 's/^model-//' | sed -E 's/-[0-9]+$//' | sed -E 's/ \([^)]*\)//' | awk '{gsub(/^ +| +$/, ""); print}'
)

jq -c --argjson built "$(printf '%s\n' "${built[@]}" | jq -R . | jq -s .)" \
  'map(select(.display_name as $n | ($built | index($n | gsub("^ +| +$"; "")) | not)))' \
  matrix.json > retry_matrix.json

cat retry_matrix.json

[]
(openpilot) james@Mac sunnypilot %

* always

* great success

* add suffix to retry artifact so it doesn't conflict

* retry to get_model too

* and there haha

* unnecessary hyphen

* compare built to missing. include retries

* adjust copy of artifacts.

* Update build-all-tinygrad-models.yaml

* Update model selector versioning and add documentation

* Update retry condition for failed models in build-all-tinygrad-models.yaml

* Update retry condition for failed models in build-all-tinygrad-models.yaml

* Update build-single-tinygrad-model.yaml

* false

* default none because why not

* red diff? i think?

* meh ... not needed i guess

* error error error

* Nayan is watching... always watching mike wazowski

* string all the way

* lots of retries just in case because im scared

* more robust

* ONLY ONE!!!!!!

* delete.... a lot

* fix artifacts

* fix artifacts

* make sure each is unique :)

* skip files like artifact duhhhh

* artifact name dir

* concurrency

* copy here

* Update build-single-tinygrad-model.yaml

* Update build-single-tinygrad-model.yaml

* bump

* bump tinygrad

* max parallel? if not, i have the other remedy ready in build-all

* revert me!

* I resynced tinygrad woo hoo

* setup shouldnt fail

* pull

* big ole diff

* condition

* Update build-all-tinygrad-models.yaml

* not always() never always() never!!!

* not failure instead of great success

* Update build-all-tinygrad-models.yaml

* yay that worked. lets invoke build-single one last time

* these arent used and are just taking up 250MB space

* really frog?

* bump back to 3

* self-hosted, tici

* rename to trigger tests

* 2 and done

---------

Co-authored-by: DevTekVE <devtekve@gmail.com>
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.

2 participants