Skip to content

REWRITE#77

Draft
not-nef wants to merge 31 commits into
mainfrom
rewrite
Draft

REWRITE#77
not-nef wants to merge 31 commits into
mainfrom
rewrite

Conversation

@not-nef

@not-nef not-nef commented Aug 30, 2023

Copy link
Copy Markdown
Member

complete rewrite from the ground up with oop and good this time

help much appreciated

why is the file diff so ducked up

@not-nef not-nef self-assigned this Aug 30, 2023
@sumeshir26

Copy link
Copy Markdown
Member

Whats the point of the StartupWindow?

@not-nef

not-nef commented Aug 31, 2023

Copy link
Copy Markdown
Member Author

It is the Startup Window where you can select create new file, open file, etc. its just not complete yet.

@not-nef

not-nef commented Aug 31, 2023

Copy link
Copy Markdown
Member Author

It is the Startup Window where you can select create new file, open file, etc. its just not complete yet.

Also i will add things like recently opened files there later

@not-nef

not-nef commented Aug 31, 2023

Copy link
Copy Markdown
Member Author

you guys were so right oop is so cool

@not-nef

not-nef commented Sep 1, 2023

Copy link
Copy Markdown
Member Author

It is the Startup Window where you can select create new file, open file, etc. its just not complete yet.

Also i will add things like recently opened files there later

nevermind i got rid of it

@im-coder-lg

Copy link
Copy Markdown
Member

@not-nef need your insight on this review! #77 (comment)

@not-nef

not-nef commented Sep 1, 2023

Copy link
Copy Markdown
Member Author

idk what you want me to look at, that hyperlink does nothing for me

@im-coder-lg

Copy link
Copy Markdown
Member

@not-nef I meant this in that link:

You can find more information in the [wiki](https://github.com/not-nef/onyx/wiki)

Aren't we supposed to change that?

@not-nef

not-nef commented Sep 3, 2023

Copy link
Copy Markdown
Member Author

yeah i will change that in this pr, a lot needs to be redone in this repo

@not-nef

not-nef commented Sep 3, 2023

Copy link
Copy Markdown
Member Author

most docs files are out of date

@not-nef

not-nef commented Sep 7, 2023

Copy link
Copy Markdown
Member Author

linux is really starting to grow on me but i cant switch because i wont be able to play most of my video games anymore :(

@not-nef

not-nef commented Sep 7, 2023

Copy link
Copy Markdown
Member Author

you know what, im just gonna make a seperate windows partition for my games and switch to linux.

@not-nef

not-nef commented Sep 7, 2023

Copy link
Copy Markdown
Member Author

my new motherboard should be arriving today so i am very excited.

@im-coder-lg

Copy link
Copy Markdown
Member

We gotta update the branch.

@im-coder-lg

Copy link
Copy Markdown
Member

@not-nef you added TkFileBrowser to the code. It's not there in requirements.txt yet, maybe?

@not-nef

not-nef commented Sep 30, 2023

Copy link
Copy Markdown
Member Author

yes, because i will most likely add more packages and will update requirements.txt before merging the branch

@im-coder-lg

Copy link
Copy Markdown
Member

Okay, so what about a Gitpod config? I need to add it just for configuring the image to load with VNC preview enabled. It uses Docker, so there would be an issue with that.

@im-coder-lg

Copy link
Copy Markdown
Member

We wouldn't need it much, yes, but if anybody were to test on the fly, it might help. I'm not coding anything on my local computer since Zorin and Python seem to be at odds right now.

@not-nef

not-nef commented Sep 30, 2023

Copy link
Copy Markdown
Member Author

so the app is lagging like crazy, but the tkfilebrowser instance inside of it is running smooth as ever. something is clearly not right

@im-coder-lg

Copy link
Copy Markdown
Member

So, it isn't working on Gitpod. Asciinema(basically fancy terminal video) clip:
asciicast

@not-nef

not-nef commented Sep 30, 2023

Copy link
Copy Markdown
Member Author

manually set TZ=UTC in your environment

source

@im-coder-lg

Copy link
Copy Markdown
Member

Will do so later, thanks!

@im-coder-lg

im-coder-lg commented Oct 1, 2023

Copy link
Copy Markdown
Member

It seems to be rapid for me, apart from the lag you get from noVNC. @not-nef try this, exactly as I say.
Make a new folder, clone PR #77 there, and just initialise a virtualenv there. I think we must start this from the start to test it. Install the modules, and install tkfilebrowser separately. Then, try running the app from src from that folder. Maybe that could tell us if it is a module or your system, coz on Gitpod, it's dandy, as they say.

@im-coder-lg

Copy link
Copy Markdown
Member

Save functions aren't working though, maybe a bug. How do we save it in the first place?

@not-nef

not-nef commented Oct 2, 2023

Copy link
Copy Markdown
Member Author

It seems to be rapid for me, apart from the lag you get from noVNC. @not-nef try this, exactly as I say.

Make a new folder, clone PR #77 there, and just initialise a virtualenv there. I think we must start this from the start to test it. Install the modules, and install tkfilebrowser separately. Then, try running the app from src from that folder. Maybe that could tell us if it is a module or your system, coz on Gitpod, it's dandy, as they say.

You think its the filebrowser module?

@not-nef

not-nef commented Oct 2, 2023

Copy link
Copy Markdown
Member Author

Save functions aren't working though, maybe a bug. How do we save it in the first place?

wdym how do we save it? where the function is?

@im-coder-lg

Copy link
Copy Markdown
Member

Save functions aren't working though, maybe a bug. How do we save it in the first place?

wdym how do we save it? where the function is?

Usually, we have a save button, or at least a shortcut. Neither exists in that, as well as I can't go through folders yet.

You think its the filebrowser module?

Well, that gives a save issue for me, but I think it could be some module that just keeps reloading or something, but is building up the data, piling it up. Maybe that could be the reason behind the lag? I don't know.

@not-nef

not-nef commented Oct 2, 2023

Copy link
Copy Markdown
Member Author

Save functions aren't working though, maybe a bug. How do we save it in the first place?

wdym how do we save it? where the function is?

Usually, we have a save button, or at least a shortcut. Neither exists in that, as well as I can't go through folders yet.

It is in the menu at the top of the screen

You think its the filebrowser module?

Well, that gives a save issue for me, but I think it could be some module that just keeps reloading or something, but is building up the data, piling it up. Maybe that could be the reason behind the lag? I don't know.

i need to further investigate

@im-coder-lg

Copy link
Copy Markdown
Member

There isn't a save button in the Save As UI. Like the button we have on Windows for saving. It could also be due to Xfce.

@not-nef

not-nef commented Oct 8, 2023

Copy link
Copy Markdown
Member Author

finally found some time to code again. im definitely not bugging. i got the main branch to sort of run and it is blazing fast as usual. that means theres got to be something wrong with my code. time to dig

@not-nef

not-nef commented Oct 8, 2023

Copy link
Copy Markdown
Member Author

i am 99% sure its an issue with sv_ttk because if i comment out the line that sets the theme, everything is fast as usual

@im-coder-lg

Copy link
Copy Markdown
Member

I think you might have kept something weird with the theme part. Check that out, it might give some insight, since that could easily cause lag.

@not-nef

not-nef commented Oct 8, 2023

Copy link
Copy Markdown
Member Author

i dont know what you mean

@im-coder-lg

Copy link
Copy Markdown
Member

I didn't explain. Maybe you set up an auto-detection for the system theme, but it kinda loops over and over again, making new processes I guess. Could be something similar.:shrug:

@not-nef

not-nef commented Oct 8, 2023

Copy link
Copy Markdown
Member Author

i didnt set up theme detection yet

@im-coder-lg

Copy link
Copy Markdown
Member

I thought you did. My bad, I got confused. Maybe the theme code is sort of just running on a loop?

@im-coder-lg

Copy link
Copy Markdown
Member

I know it's selective, but maybe it is running that continuously.

@rdbende

rdbende commented Oct 21, 2023

Copy link
Copy Markdown

i am 99% sure its an issue with sv_ttk because if i comment out the line that sets the theme, everything is fast as usual

sv_ttk is slow on Windows

@rdbende

rdbende commented Oct 21, 2023

Copy link
Copy Markdown

@not-nef

not-nef commented Oct 21, 2023

Copy link
Copy Markdown
Member Author

but why is the version before the rewrite so fast?

@rdbende

rdbende commented Oct 21, 2023

Copy link
Copy Markdown

Yeah, that's strange. Are you using a type of widget in the new version that you weren't before?

@not-nef

not-nef commented Oct 21, 2023

Copy link
Copy Markdown
Member Author

not that i could think of, but everything is oop now

@rdbende

rdbende commented Oct 21, 2023

Copy link
Copy Markdown

but everything is oop now

That shouldn't cause slowness

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