Skip to content
This repository has been archived by the owner on Jul 16, 2024. It is now read-only.

Add some notes about restricting the CPU cores used by PV #301

Closed

Conversation

prensing
Copy link
Contributor

@prensing prensing commented Jan 1, 2024

I added a few "notes" in the install docs to point people to the ability to restrict which CPU cores are used. This is relevant on ARM big.LITTLE architectures, including Orange Pi 5.

@mcm001
Copy link
Contributor

mcm001 commented Jan 1, 2024

FYI, I have a PR open to create a orange pi image with this flag set by default in the service file. Lgtm tho

mcm001
mcm001 previously approved these changes Jan 2, 2024
mdurrani808 and others added 3 commits January 3, 2024 15:10

Added:

    networking diagram
    3D Coordiante systems
    Reorganized getting started
    Removed AprilTag example page
    Added note on AruCo vs AprilTag
    Added page on MultiTag, needs docs
    Basic system requirements

Did some basic reorganization as well to make this more approachable.
…n#304)

Instructions for offline update as well as flashing pre-release versions of Photon
* Add mrcal and camera inspection docs

* Update calibration.rst

* Flashbang!

* Update calibration.rst

* Address comments
@mcm001
Copy link
Contributor

mcm001 commented Jan 5, 2024

Needs to be rebased

@prensing
Copy link
Contributor Author

prensing commented Jan 5, 2024

Done. Hopefully correctly (not a git wizard!).

@mcm001
Copy link
Contributor

mcm001 commented Jan 5, 2024

Looks completely FUBAR

@prensing
Copy link
Contributor Author

prensing commented Jan 5, 2024

OK, I did not notice a link had changed. Should have tested my build ;-(. Done, and CI passes now.

@mcm001
Copy link
Contributor

mcm001 commented Jan 5, 2024

I still see 103 files changed????

@prensing
Copy link
Contributor Author

prensing commented Jan 5, 2024

OK, I am mystified. I only touched 2 files. It looks like somehow it dragged into a bunch of other commits from mdurranni808 2 days ago. No idea how that is possible.
Maybe I will just kill this and then resubmit, but I can't right now (need to run).

@prensing
Copy link
Contributor Author

prensing commented Jan 5, 2024

Actually, snuck it in. See PR306. I did not bother with the note in the OPi page, because I don't know about the OPi 4.

@prensing prensing closed this Jan 5, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants