Skip to content

Made library work when internet is not connected #21

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

Closed
wants to merge 1 commit into from

Conversation

evaherrada
Copy link
Collaborator

In my dorm, there isn't really any way to connect my MatrixPortal to the internet (VT uses an enterprise network). I still wanted to make use of this library, so I added a new parameter to the matrixportal constructor that lets you specify when you don't want to connect to internet. It would be really great if someone could make sure this still works when you're connecting to internet, since that's not something I can test

@evaherrada evaherrada requested a review from a team September 26, 2020 21:17
@ladyada ladyada requested review from makermelissa and removed request for a team September 26, 2020 22:10
@dglaude
Copy link

dglaude commented Sep 27, 2020

I am very much interested into this kind of option.
Many Matrix Portal learn guide could work on an M4+Matrix FeatherWing (maybe adding RTC for clock related guide).
Anything where request is not mandatory (or could be replaced by RTC).
I adapted my copy of this library for that purpose (and for the wiring of my 64x64 matrix).

@makermelissa
Copy link
Collaborator

Hi, thanks for this PR. I'm trying to understand the main functions you would like out of the top level matrixportal library (which was intended to tie the network and graphics stuff together) that don't use network? Since this PR goes against the intention of this class, I would like to understand your needs better (both @dherrada and @dglaude).

Most likely the functions you require should probably be moved down to the graphics layer, which is the correct approach and then you could just use that. It would result in less memory usage as well.

@makermelissa
Copy link
Collaborator

Also, if it's helpful, here's a diagram of the layout of the library:
MatrixPortal

@PaintYourDragon
Copy link

I’ve been using just the Matrix side of MatrixPortal (see Creature Eyes guide) as-is, doesn’t require invoking Network.

@evaherrada
Copy link
Collaborator Author

@PaintYourDragon Thanks, I'll take a look at that.

@evaherrada
Copy link
Collaborator Author

@makermelissa Ah, ok. That makes sense.

@evaherrada evaherrada closed this Sep 30, 2020
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.

4 participants