Skip to content
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

Three things #2

Closed
milkdropper opened this issue Sep 15, 2022 · 7 comments
Closed

Three things #2

milkdropper opened this issue Sep 15, 2022 · 7 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@milkdropper
Copy link

I thought I submitted this issue already but can't see it now. If I duplicated anything, sorry.

I have three things:

  1. It's great you are moving this project forward! It seems the original maintainer has gone silent.
  2. Could you implement this PR for multiple monitor support? Implement support for multiple monitors mvsoft74/BeatDrop#4
  3. Can you give an option to toggle unlimited FPS? I am on a 244hz monitor and getting 200+FPS. This looks really nice for some presets, but others act like they are going way too fast. This is because many presets are based on the FPS counter which is expected to be 60 and were designed originally on 60.

Thanks!

@milkdrop2077
Copy link
Collaborator

milkdrop2077 commented Sep 18, 2022

hey milkdropper,
I can't test the unlimited FPS feature because my screen only support 60fps, do you have the same problem with my version BeatDrop2077? I could implement an option for that. Do you have this problem with the milkdrop version (MilkDrop v2.33.2+) of wacup? it's limited to 144fps, and wacup has some extra code like UpdateMaxFps in the file config.cpp

@milkdrop2077
Copy link
Collaborator

I thought I submitted this issue already but can't see it now. If I duplicated anything, sorry.

I have three things:

1. It's great you are moving this project forward! It seems the original maintainer has gone silent.

2. Could you implement this PR for multiple monitor support? [Implement support for multiple monitors mvsoft74/BeatDrop#4](https://github.com/mvsoft74/BeatDrop/pull/4)

3. Can you give an option to toggle unlimited FPS? I am on a 244hz monitor and getting 200+FPS. This looks really nice for some presets, but others act like they are going way too fast. This is because many presets are based on the FPS counter which is expected to be 60 and were designed originally on 60.

Thanks!

Also I've implemented your ALT+SHIFT stretching code into my version of BeatDrop2077, it seems to work well but I don't have multiple monitor to test it so please let me know if it works like it should be!

@OfficialIncubo
Copy link
Owner

My laptop's monitor runs 144 FPS on it because it has 144hz.
I should do an action, such as multiple monitor stretch and always on top feature while I come back home.

@OfficialIncubo OfficialIncubo added the enhancement New feature or request label Sep 19, 2022
@OfficialIncubo
Copy link
Owner

Done! Look at these commits:

@OfficialIncubo
Copy link
Owner

OfficialIncubo commented Sep 26, 2022

Ok, @milkdropper, for toggling the Unlimited FPS on and off, what hotkey should I specify?

When off, the framerate should change to 0 to the max_fps_w and max_fps_fs read from milk2.ini.

  • You should give me a code suggestion by opening a Pull Request.

@OfficialIncubo OfficialIncubo added the good first issue Good for newcomers label Sep 26, 2022
@OfficialIncubo
Copy link
Owner

Alright. @milkdrop2077 now implemented this feature and this code will be released in the future or later (F3 hotkey).

I will close this issue right now.

@OfficialIncubo
Copy link
Owner

OfficialIncubo commented Nov 5, 2022

Can you give an option to toggle unlimited FPS? I am on a 244hz monitor and getting 200+FPS. This looks really nice for some presets, but others act like they are going way too fast. This is because many presets are based on the FPS counter which is expected to be 60 and were designed originally on 60.

Hey, @milkdropper! I have implemented my own version of toggle 30/60/90... fps! You can check my commit now: mvsoft74@75af8fd

So, It's very easy for me to implement it. What do you think?
...and the 1.2.3 version is now RELEASED!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants