breuerfelix/instapy-gui

Allow randomisation of 'amount' parameter in actions which provide it

Opened this issue · 3 comments

Hello @breuerfelix, thanks for this great project :) I've been using it for some time now and got a feature request which is the following.

Some actions, like follow_by_locations have an amount parameter which determines the amount of times to perform said action. However, it doesn't seem possible to randomise this value. I believe randomising this value can greatly benefit users to lower chance of having their accounts blocked and having more "organic" templates.

As for implementation, one of these should work:

  • random parameter where the user inputs a range like 40-100
  • Deprecate the amount parameter in favour of min_amount and max_amount

The second implementation would be more intuitive, I think.

Is this something you would be willing to tackle?

Thank you for your time

hey @sashaafm thanks for this great feedback !

well this would be a great feature indeed but there is one problem with this. i build the interface, functions, parameters, etc by extracting the values from the instapy source code. so the amount input field you see in the UI is there because there is an amount parameter in the instapy function which is an integer. if tim would change that to have a min_amount and max_amount parameter, i run the update script on my server and everything would change automatically :) i did that because instapy has PLENTY of functions and creating these in the gui manually and maintaining these would be a pain in the ass :D
so creating this exception for some functions would require major code changes. also this would destroy everything if the instapy source code changes. so it is also a pain in the ass in terms of maintaining :)

so instead of building this into instapy gui, the best place would be to build this into instapy itself :) then instapy gui would get updated with the appropriate functions and everybody got the randomization.

i know that right now you could just randomize these values inside the quickstart script but that would require programming skills and i did the gui for people not knowing how to program :D so of course this is the little drawback of this. you can't provide the same functionality. BUT i also thought about making a box where you can add custom python code to your template and then use variables like $time in the other functions but but but but... as always so many ideas in my head and since instagram is going hard against instapy i think that this is not worth the time because chances are high that instapy will work less and less good in the near future... also because currently there is not a single active developer on the project.

So thanks for the idea and sorry for the long reply :D

@breuerfelix Isn't it possible to at least, if we detect - in the value, and both of the sides are numbers, to assume we want to randomize it, and in the next run randomize them? It would be low effort, a little work around but should work (rather then creating mr in instapy - which takes forever to respond to even little ones)
I will open a pr to show what I mean later

@ErezAmihud would be cool if you could provide the PR but also, you can give me a heads up on discord about some instapy changes. due to work, i am currently not able to review all of them but if i get a list of PRs that are important and already reviewed, it should be easy to merge them :) i know that tim is also busy right now