Coder Social home page Coder Social logo

Comments (46)

akumidv avatar akumidv commented on May 31, 2024 3

Best "Net Profit: All" Annealing works but negative values display as absolu

This is another error. The problem is in $ before digits. I'll fix this today or tomorrow. For now you can change the currency in strategy properties for that ones where currency after digits.

image

from tradingview-assistant-chrome-extension.

Steadyeye3 avatar Steadyeye3 commented on May 31, 2024

same here. test runs but csv thats generated after the test has no data

from tradingview-assistant-chrome-extension.

akumidv avatar akumidv commented on May 31, 2024

Thank you for your issue. I fix this error in 1.4 version. It was published on weekend, but for now still on checking in store. Can you check after the extension will be updated?

from tradingview-assistant-chrome-extension.

akumidv avatar akumidv commented on May 31, 2024

Until update you can use fixed version by install extension in developer mode. The instruction in repository README file

from tradingview-assistant-chrome-extension.

Steadyeye3 avatar Steadyeye3 commented on May 31, 2024

yes, still happening in the fixed version, ID: afgffeoiimejhoekaeimhcaadcidlogb

from tradingview-assistant-chrome-extension.

akumidv avatar akumidv commented on May 31, 2024

This ID is id of extension not the version. Version published two hours ago, Can you reload your browser. Or remove extension, add it again. Check please that version on the extensions page is 1.4. Then reload tradingview page, this is mandatory. And try.

from tradingview-assistant-chrome-extension.

DriesVR1232 avatar DriesVR1232 commented on May 31, 2024

I tried and it gives the same result unfortunately. NaN in the popup screen to follow the progression of the test and the message "There is no data for conversion" in the excel file.

from tradingview-assistant-chrome-extension.

agentollie avatar agentollie commented on May 31, 2024

When I download from here version is from 14th May - im gathering 1.3

from tradingview-assistant-chrome-extension.

jonathanhyunsung avatar jonathanhyunsung commented on May 31, 2024

Both versions, 1.3 and 1.4, are producing the same CSV error.
As OP mentions- it was working fine yesterday

from tradingview-assistant-chrome-extension.

4while avatar 4while commented on May 31, 2024

Version - 1.4 / Size 3.7 MB
brings NaN as some guys said before
Used Profit Factor All (annealing or brute) has the same effect
I'm using the new UI with the deep history slider

from tradingview-assistant-chrome-extension.

ogerik avatar ogerik commented on May 31, 2024

Same issue here, on the older version its working but on the versions which is installed yesterday and today (brand new installation) has the same issues.

from tradingview-assistant-chrome-extension.

akumidv avatar akumidv commented on May 31, 2024

Unfortunately, I don't have access to the beta version and I can't test on it. On the regular version, I can't reproduce this error.

from tradingview-assistant-chrome-extension.

4while avatar 4while commented on May 31, 2024

Unfortunately, I don't have access to the beta version and I can't test on it. On the regular version, I can't reproduce this error.

Open a support ticket and ask for beta testing enrollment (for deep history), i don't know who's eligible and who is not, they did it for me in 1 hour.

from tradingview-assistant-chrome-extension.

ogerik avatar ogerik commented on May 31, 2024

You mean the beta version of Chrome? You think this could be the problem?

from tradingview-assistant-chrome-extension.

Steadyeye3 avatar Steadyeye3 commented on May 31, 2024

Same issue on 1.4 update. NaN while optimizing and "There is no data for conversion" is the only thing is the results .csv file

from tradingview-assistant-chrome-extension.

Steadyeye3 avatar Steadyeye3 commented on May 31, 2024

was not using the deep data beta for my testing

from tradingview-assistant-chrome-extension.

jonathanhyunsung avatar jonathanhyunsung commented on May 31, 2024

was not using the deep data beta for my testing

Same. Also, the version installed manually from the README method says 1.3 in the extension manager, the version from the Chrome Store is 1.4, both versions have the error.

from tradingview-assistant-chrome-extension.

4while avatar 4while commented on May 31, 2024

I start to think there are three versions of UI
The old one, the new one without the deep history slider and the ones with the slider. In any case the extension doesn't work for now.

from tradingview-assistant-chrome-extension.

Ayahuasca94 avatar Ayahuasca94 commented on May 31, 2024

I guess there is no solution. we get nan error in both versions.

from tradingview-assistant-chrome-extension.

4while avatar 4while commented on May 31, 2024

I don't think there are "both" aka two versions because if akumidv said he sees no problem but i (with the deep history slider) and a friend of mine (without the deep history slider) can't pass the NaN error then we have more than 2 versions of the UI. However certainly since most of us can't the extension anymore, i believe they're rolling faster their new UI

from tradingview-assistant-chrome-extension.

akumidv avatar akumidv commented on May 31, 2024

Unfortunately, I don't have access to the beta version and I can't test on it. On the regular version, I can't reproduce this error.

Open a support ticket and ask for beta testing enrollment (for deep history), i don't know who's eligible and who is not, they did it for me in 1 hour.

Yes, I did the same, but have got decline

from tradingview-assistant-chrome-extension.

akumidv avatar akumidv commented on May 31, 2024

Colleagues, I have corrected the error. All the fixes are in the repository, you can install and check. I sent the new version 1.5 to the publication in Google Store
The reason for the error is in two dimensions - now the backtesting data is hidden in the TV API, at least I didn't find them quickly. And the html selectors of the backtesting results table were changed too. I.e. both the main and reserve method to get data were broken.
Please try it, if the error repeats - write here please. If you can take a screenshot of the entire screen with the console open (F12, console tab)

from tradingview-assistant-chrome-extension.

DriesVR1232 avatar DriesVR1232 commented on May 31, 2024

Still same issue.
extension problem

Seems like the values are not given to the variables at line 142 and 165.

from tradingview-assistant-chrome-extension.

jonathanhyunsung avatar jonathanhyunsung commented on May 31, 2024

Still same issue. extension problem

Seems like the values are not given to the variables at line 142 and 165.

With appreciation to the Author's hard work -
Getting the same NaN error and CSV file, the F12 console shows the same logs about lines 142 and 165, from the manual installation of the 1.5 version.

from tradingview-assistant-chrome-extension.

akumidv avatar akumidv commented on May 31, 2024

Seems like the values are not given to the variables at line 142 and 165.

Are you sure that you use 1.5 version? Because in new version when the extension do not get table selectors it show an error about this and do not process backtesting optimization. In chrome version are not updated, it demand 2-3 days. You can test only if install extension from repository in developer mode.
I'll check on both TV UI version. On current and on beta.

from tradingview-assistant-chrome-extension.

akumidv avatar akumidv commented on May 31, 2024

image
Example

from tradingview-assistant-chrome-extension.

jonathanhyunsung avatar jonathanhyunsung commented on May 31, 2024

Untitled
Untitled2

Hello,

For 1.5:

  1. Best "Net Profit: All" Annealing works but negative values display as absolute values like in the other thread.
    -> Best "Net Profit: All" Annealing is optimizing based on absolute values, both negative and positive.
    -> After everything runs, the "Best 'Net Profit: All': 7139" in the screenshot shows as "Best 'Net Profit: All': -7139" in the Performance Summary tab, it's a negative value and should be picked up by the extension as a negative value instead of positive.
  2. Best "Net Profit: All %" Annealing produces the NaN error. Using % Annealing was the workaround to Problem (1.) when it was working a few days ago, but it's been making the NaN error since.

from tradingview-assistant-chrome-extension.

DrDenizblc avatar DrDenizblc commented on May 31, 2024

Screenshot_4
I installed the extension from the repository.
I restarted chrome.
Then I had the parameters analyzed and separated the unnecessary parameters.
I restored the parameters to version 1.5.
Then I ran it but I got an error like this NAN
can you guys tell me where i made a mistake?
I appreciate your work...

from tradingview-assistant-chrome-extension.

micant avatar micant commented on May 31, 2024

@akumidv
Tested by installing manually in developer mode - can confirm is working again for v1.5 and Chrome v101.0.4951.64.
Thank you for your hard work on this excellent plugin.

from tradingview-assistant-chrome-extension.

Ayahuasca94 avatar Ayahuasca94 commented on May 31, 2024

@akumidv

Tested by installing manually in developer mode - can confirm is working again for v1.5 and Chrome v101.0.4951.64.

Thank you for your hard work on this excellent plugin.

I say negative results

from tradingview-assistant-chrome-extension.

akumidv avatar akumidv commented on May 31, 2024

I installed the extension from the repository. I restarted chrome. Then I had the parameters analyzed and separated the unnecessary parameters. I restored the parameters to version 1.5. Then I ran it but I got an error like this NAN can you guys tell me where i made a mistake? I appreciate your work...

Thanks. This is an another problem. And as I see it is not from current version of TV assistants, May be from another fork of this extension (The optimizer maybe?). For now It doesn't have function getStrategyTemplate
May be you use both? Then you need for disable another version of extension they can conflict each other.

from tradingview-assistant-chrome-extension.

jonathanhyunsung avatar jonathanhyunsung commented on May 31, 2024

Can any of the above be the cause of the absolute values in "Best Net Profit: All" for Annealing? I've disabled/removed all TV Assistants except TV Assistant 1.5, the screenshots are from the backtest.js of 1.5.

from tradingview-assistant-chrome-extension.

akumidv avatar akumidv commented on May 31, 2024

Are any of the above the cause of the absolute values? I've disabled/removed all TV Assistants except TV Assistant 1.5, the screenshots are from the backtest.js of 1.5.

No, This is another part of code. The parser function is tv.parseReportTable() and it have two method to detect negative values, so I wondered why both failed.

from tradingview-assistant-chrome-extension.

jonathanhyunsung avatar jonathanhyunsung commented on May 31, 2024

Best "Net Profit: All" Annealing works but negative values display as absolu

This is another error. The problem is in $ before digits. I'll fix this today or tomorrow. For now you can change the currency in strategy properties for that ones where currency after digits.

Switching the currency to NZD is making the right optimizations, using Net Profit: All; annealing. Thank you again.

from tradingview-assistant-chrome-extension.

akumidv avatar akumidv commented on May 31, 2024

Best "Net Profit: All" Annealing works but negative values display as absolu

This is another error. The problem is in $ before digits. I'll fix this today or tomorrow. For now you can change the currency in strategy properties for that ones where currency after digits.

Switching the currency to NZD is making the right optimizations, using Net Profit: All; annealing. Thank you again.

I fixed parsing values with currency. So it can correct parse values like -$100. Can you check. It seems working now. Changes in repository as version 1.6.

from tradingview-assistant-chrome-extension.

akumidv avatar akumidv commented on May 31, 2024

Example
image

from tradingview-assistant-chrome-extension.

jonathanhyunsung avatar jonathanhyunsung commented on May 31, 2024

It's working perfectly for Default Currency, Net Profit: All, Annealing.

Thank you, very grateful for your awesome work.

from tradingview-assistant-chrome-extension.

akumidv avatar akumidv commented on May 31, 2024

It's working perfectly for Default Currency, Net Profit: All, Annealing.

Thank you, very grateful for your awesome work.

Update please again. I fixed some problem with names of max drawdown and parse data for %

from tradingview-assistant-chrome-extension.

jonathanhyunsung avatar jonathanhyunsung commented on May 31, 2024

The % was only a workaround/alternative because of the negative value issue in "Net Profit: All" (which was connected to the currencies). The regular "Net Profit: All" works perfectly.

from tradingview-assistant-chrome-extension.

DriesVR1232 avatar DriesVR1232 commented on May 31, 2024

The "Net Profit: All" works fine
extension problem2

the "Net Profit:All%" doesnt. there i still get the NaN error.
extension problem3

from tradingview-assistant-chrome-extension.

akumidv avatar akumidv commented on May 31, 2024

I'll try to check tomorrow again. For testing results it works on new 1.6 version for UI version with deep history too.
image

from tradingview-assistant-chrome-extension.

4while avatar 4while commented on May 31, 2024

I see a different graph of profit - drawdown.
Do they attempt to redraw again the area ?
The xpath to the strategy tab and performance summary link don't look the same

from tradingview-assistant-chrome-extension.

JasonSPH avatar JasonSPH commented on May 31, 2024

The "Net Profit: All" works fine for me too, but the "Sharpe Ratio: All" still not working...

from tradingview-assistant-chrome-extension.

DrDenizblc avatar DrDenizblc commented on May 31, 2024

With version 1.6, I ran a scan for very long hours. Without using % as currency NZD. It saved the best setting out of 5000 scans and did not show it to me. The values it saved are wrong.
I got another error, when I run the extension, after get the best parameters. it didn't update the strategy, losing all the parameters got . I hope you fix it, becaouse is amazing extension. thank you so much for all you work.

from tradingview-assistant-chrome-extension.

4while avatar 4while commented on May 31, 2024

@akumidv : Instead on focusing on XPATHs and DIVs and such, why don't you focus on file manipulation of the strategy details?
You will gain time from future cosmetic upgrades of the javascript code and even if they change the details in the csv, is just a different set in the field array, another comma and done.
Maybe few MBytes extra at the cache or non real time results but what matters is robustness and overall performance.

from tradingview-assistant-chrome-extension.

akumidv avatar akumidv commented on May 31, 2024

@akumidv : Instead on focusing on XPATHs and DIVs and such, why don't you focus on file manipulation of the strategy details? You will gain time from future cosmetic upgrades of the javascript code and even if they change the details in the csv, is just a different set in the field array, another comma and done. Maybe few MBytes extra at the cache or non real time results but what matters is robustness and overall performance.

@4while The extension implements emulation of user actions. I know of no other way to make this work, other than to build a robot that works with the interface. Yes, any change in the interface breaks the work, but usually it won't take long to fix it. As for reliability and performance the most reliable way is to make all calculation by using python. That's exactly what I do for myself and test in the Google/AWS cloud (some simple examples in my current course for local community https://github.com/akumidv/quants-algotrading-khv and there https://github.com/akumidv/startup-khv-ai-study).

A test of 2-5 seconds per iteration is a very slowly to optimally test a large parameter space. It should be at least 100-1000 times faster. Perhaps I will release it as an application in the future.

from tradingview-assistant-chrome-extension.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.