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

Bait selection added #21

Merged
merged 8 commits into from
Oct 9, 2021
Merged

Bait selection added #21

merged 8 commits into from
Oct 9, 2021

Conversation

DaGuT
Copy link
Contributor

@DaGuT DaGuT commented Oct 2, 2021

Do not merge it yet, I have not tested it.

But with this PR you'll be able to select two new places (position of bait and position of equip button for bait confirmation).

After repairing it'll automatically select bait (so it has the same timer as repair!)

image

@DaGuT
Copy link
Contributor Author

DaGuT commented Oct 2, 2021

#16

@DaGuT
Copy link
Contributor Author

DaGuT commented Oct 2, 2021

feels like it works, I guess you can merge it now

@DaGuT DaGuT mentioned this pull request Oct 2, 2021
@DaGuT DaGuT mentioned this pull request Oct 3, 2021
@Siterizer
Copy link
Owner

@DaGuT sorry for late reeponse but you know, weekend etc. Before i merge this i would like to test IT myself, but you know The queues. I think i will test Ii tommorow.
Btw thanks for your help with contribution 🥰

@DaGuT
Copy link
Contributor Author

DaGuT commented Oct 3, 2021

sure, no problem :)

Enjoy the game. That's what it is meant for

@JKamsker
Copy link

JKamsker commented Oct 4, 2021

Here, have some cookie for that @DaGuT 🍪

@DaGuT
Copy link
Contributor Author

DaGuT commented Oct 5, 2021

There is a bug. When bait is disabled, it still uses it. Gonna fix it this evening after I come home:)

UPD: Yeah, I forgot to add .get() when checking if bait is enabled :(((

@DaGuT
Copy link
Contributor Author

DaGuT commented Oct 5, 2021

fixed with DaGuT@8211b38

@ivan-cavero
Copy link

I have been testing it and it works very well, so you can get another person's opinion. Thank you both very much for your work :)

@davidgao93
Copy link

Can confirm that this works for me as well, for some reason the original version would stop after a few hours and the camera would bug out but this one is fine.

@tallglen
Copy link

tallglen commented Oct 7, 2021

I have been testing it and it works very well, so you can get another person's opinion. Thank you both very much for your work :)

I'll do some testing tonight.

@ivan-cavero
Copy link

I have been testing it and it works very well, so you can get another person's opinion. Thank you both very much for your work :)

I'll do some testing tonight.

Does it work after the update?

@tallglen
Copy link

tallglen commented Oct 7, 2021

I have been testing it and it works very well, so you can get another person's opinion. Thank you both very much for your work :)

I'll do some testing tonight.

Does it work after the update?

Yup worked so another +1 from me

@danmarzola
Copy link

Working Fine

@litlhogn
Copy link

litlhogn commented Oct 8, 2021

I have been been using this commit for 3 days now. I have had no issues with it functioning as written. Obviously, it would be nice if it could eventually determine which bait type to use, but it works fantastic.

@BmanXD
Copy link

BmanXD commented Oct 8, 2021

How do you test this version? I'd like to give it a go

@DaGuT
Copy link
Contributor Author

DaGuT commented Oct 8, 2021

How do you test this version? I'd like to give it a go

https://github.com/Siterizer/new-world-fishing-bot#code-installation-guide

@dvz95
Copy link

dvz95 commented Oct 8, 2021

Sorry, how exactly do I try this code coz that link above was just to the original code’s readme

@ivan-cavero
Copy link

The other day I could use it perfect, and I used it 2 days, but after the patch when I try to use it I get in console "[INFO] Pause fishing" I have tried different things, throw the fishing rod and it works fine until I recognize the fish, but then I get again "[INFO] Pause fishing". Do you know what could it be? I haven't changed anything, and it was working fine before.

@DaGuT
Copy link
Contributor Author

DaGuT commented Oct 8, 2021

The other day I could use it perfect, and I used it 2 days, but after the patch when I try to use it I get in console "[INFO] Pause fishing" I have tried different things, throw the fishing rod and it works fine until I recognize the fish, but then I get again "[INFO] Pause fishing". Do you know what could it be? I haven't changed anything, and it was working fine before.

There is a high chance for:

  1. You settings have changed after update
  2. You weared something that has "pause" color in it.

So, make sure you have proper settings, reduce the box so that it does not include anything orange and red. You can also look at the ground to reduce the chance of anything being in the "spot zone".

So it's not about update, it's about you doing it with different conditions

@danmarzola
Copy link

Sorry, how exactly do I try this code coz that link above was just to the original code’s readme

Just download from @DaGuT fork https://github.com/DaGuT/new-world-fishing-bot

@danmarzola
Copy link

The other day I could use it perfect, and I used it 2 days, but after the patch when I try to use it I get in console "[INFO] Pause fishing" I have tried different things, throw the fishing rod and it works fine until I recognize the fish, but then I get again "[INFO] Pause fishing". Do you know what could it be? I haven't changed anything, and it was working fine before.

There is a high chance for:

  1. You settings have changed after update
  2. You weared something that has "pause" color in it.

So, make sure you have proper settings, reduce the box so that it does not include anything orange and red. You can also look at the ground to reduce the chance of anything being in the "spot zone".

So it's not about update, it's about you doing it with different conditions

I noticed that "torchs" in cities like Mourningdale mess with color detection, just avoid anything orange~yellow in your fishing rectangle

@dvz95
Copy link

dvz95 commented Oct 8, 2021 via email

@dvz95
Copy link

dvz95 commented Oct 8, 2021

Sorry, how exactly do I try this code coz that link above was just to the original code’s readme

Just download from @DaGuT fork https://github.com/DaGuT/new-world-fishing-bot

There is no exe to run.

@danmarzola
Copy link

danmarzola commented Oct 8, 2021

There is no exe to run

________________________________ From: danmarzola @.> Sent: Saturday, October 9, 2021 1:19:07 AM To: Siterizer/new-world-fishing-bot @.> Cc: dvz95 @.>; Comment @.> Subject: Re: [Siterizer/new-world-fishing-bot] Bait selection added (#21) Sorry, how exactly do I try this code coz that link above was just to the original code’s readme Just download from @DaGuThttps://github.com/DaGuT fork https://github.com/DaGuT/new-world-fishing-bot — You are receiving this because you commented. Reply to this email directly, view it on GitHub<#21 (comment)>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQ7RLABG3FB6A3PZ6YPXXVDUF34VXANCNFSM5FGNGQHQ. Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You need create it

Install Python, open console go to git folder and get path, write down something like "cd C:\path\new-world-fishing-bot"
follow instruction in cmd:

Create python virtual enviroment python -m venv path\new-world-fishing-bot
Enter virtual enviroment Scripts\activate just type "cd C:\path\new-world-fishing-bot\Scripts" then activate in console
Install following modules:
pip install pyyaml
pip install pywin32
pip install numpy
pip install opencv-python
pip install Pillow

then you can:

If you would like to create your own .exe file:
Install: pip install pyinstaller
Run following command: pyinstaller --add-data resources;resources bot.py
Your exe file should generate in dist\bot\bot.exe

You should run without generating .exe first to get your settings saved.
Just type "cd C:\path\new-world-fishing-bot" then "python bot.py" to run and to save your settings by just exitting in GUI

@Mpaulo
Copy link

Mpaulo commented Oct 8, 2021

sorry for my little understanding, but does using the bait make fishing a lot better? in the amount of rare and legendary fish? I was seeing the best baits and there is not much information in them and I would like to test this version

@tallglen
Copy link

tallglen commented Oct 8, 2021

sorry for my little understanding, but does using the bait make fishing a lot better? in the amount of rare and legendary fish? I was seeing the best baits and there is not much information in them and I would like to test this version

It's all based on % chance. Adding bait adds some % chance. So it's better but it's not going to make catching things guaranteed.

@tallglen
Copy link

tallglen commented Oct 8, 2021

The other day I could use it perfect, and I used it 2 days, but after the patch when I try to use it I get in console "[INFO] Pause fishing" I have tried different things, throw the fishing rod and it works fine until I recognize the fish, but then I get again "[INFO] Pause fishing". Do you know what could it be? I haven't changed anything, and it was working fine before.

There is a high chance for:

  1. You settings have changed after update
  2. You weared something that has "pause" color in it.

So, make sure you have proper settings, reduce the box so that it does not include anything orange and red. You can also look at the ground to reduce the chance of anything being in the "spot zone".

So it's not about update, it's about you doing it with different conditions

I had a similar issue; starting using the bot with another character that I'm doing a different build on and the appearance is not exactly the same as my other character and the bot isn't working as well. Going to try later without any clothes equipped and point it at the ground to see if it helps.

@BmanXD
Copy link

BmanXD commented Oct 9, 2021

I cant install 3 modules
pip install pywin32
pip install numpy
pip install opencv-python

got a ton of errors

@tallglen
Copy link

tallglen commented Oct 9, 2021

I cant install 3 modules pip install pywin32 pip install numpy pip install opencv-python

got a ton of errors

Did you use a virtualenv?

@BmanXD
Copy link

BmanXD commented Oct 9, 2021

Yes, activated then typed in what was provided for all modules. 2 worked fine and the other 3 didnt

@JKamsker
Copy link

JKamsker commented Oct 9, 2021

With the bait the bot seems to have a problem with the secret fishermans spot when hitting a good spot. The fishes are cought faster than the bot expects.

@DaGuT
Copy link
Contributor Author

DaGuT commented Oct 9, 2021

With the bait the bot seems to have a problem with the secret fishermans spot when hitting a good spot. The fishes are cought faster than the bot expects.

That cant be. Reduce the greenbox size so that your pc can handle cv

New functions are not computantional heavy at all, so they can not be a problem of that.

Moreover, they are checked and called only after repair is used, so there is not even a single cpu operation that is different at any other moment

@Siterizer
Copy link
Owner

@DaGuT Trully awesome work, i love you <3 works in the first try.
Changed gui a little bit (sorry but i am perfectionist xd) b36c2cc
image

@Siterizer Siterizer merged commit 7b1c89e into Siterizer:main Oct 9, 2021
@JKamsker
Copy link

JKamsker commented Oct 9, 2021

@DaGuT I think you got me wrong.
The time the fish appears is far inside the post casting sleep.

@D-main
Copy link

D-main commented Oct 9, 2021

Works great

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.