Skip to content

Add python methods for symbol series#29

Merged
EsIstJosh merged 8 commits into
mainfrom
dev
Apr 18, 2025
Merged

Add python methods for symbol series#29
EsIstJosh merged 8 commits into
mainfrom
dev

Conversation

@EsIstJosh
Copy link
Copy Markdown
Owner

@EsIstJosh EsIstJosh commented Apr 18, 2025

No description provided.

@EsIstJosh EsIstJosh merged commit f78e6b1 into main Apr 18, 2025
3 checks passed
github-advanced-security[bot]
github-advanced-security AI found potential problems Apr 18, 2025
View reviewed changes
Comment thread lightweight_charts/js/styles.css Outdated

Check warning

Code scanning / Csslint (reported by Codacy)

Rule doesn't have all its properties in alphabetical order. Warning

Rule doesn't have all its properties in alphabetical order.
github-advanced-security[bot]
github-advanced-security AI found potential problems Apr 18, 2025
View reviewed changes
Comment thread package-lock.json
"version": "6.2.5",
"resolved": "https://registry.npmjs.org/vite/-/vite-6.2.5.tgz",
"integrity": "sha512-j023J/hCAa4pRIUH6J9HemwYfjB5llR2Ps0CWeikOtdR8+pAURAk0DoJC5/mm9kd+UgdnIy7d6HE4EAvlYhPhA==",
"integrity": "sha512-j023J/hCAa4pRIUH6J9HemwYfjB5llR2Ps0CWeikOtdR8+pAURAk0DoJC5/mm9kd+UgdnIy7d6HE4EAvlYhPhA==",=

Check warning

Code scanning / Jacksonlinter (reported by Codacy)

Unexpected character ('=' (code 61)): was expecting double-quote to start field name Warning

Unexpected character ('=' (code 61)): was expecting double-quote to start field name
github-advanced-security[bot]
github-advanced-security AI found potential problems Apr 18, 2025
View reviewed changes
Copy link
Copy Markdown

@github-advanced-security github-advanced-security AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pylintpython3 (reported by Codacy) found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

github-advanced-security[bot]
github-advanced-security AI found potential problems Apr 18, 2025
View reviewed changes
Copy link
Copy Markdown

@github-advanced-security github-advanced-security AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pylint (reported by Codacy) found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

github-advanced-security[bot]
github-advanced-security AI found potential problems Apr 18, 2025
View reviewed changes
Comment thread lightweight_charts/js/styles.css Outdated

Check warning

Code scanning / Stylelint (reported by Codacy)

Expected indentation of 2 spaces (indentation) Warning

Expected indentation of 2 spaces (indentation)
Comment thread lightweight_charts/js/styles.css Outdated

Check warning

Code scanning / Stylelint (reported by Codacy)

Expected indentation of 2 spaces (indentation) Warning

Expected indentation of 2 spaces (indentation)
Comment thread lightweight_charts/js/styles.css Outdated

Check warning

Code scanning / Stylelint (reported by Codacy)

Expected indentation of 2 spaces (indentation) Warning

Expected indentation of 2 spaces (indentation)
Comment thread lightweight_charts/js/styles.css Outdated

Check warning

Code scanning / Stylelint (reported by Codacy)

Expected newline after ";" in a multi-line declaration block (declaration-block-semicolon-newline-after) Warning

Expected newline after ";" in a multi-line declaration block (declaration-block-semicolon-newline-after)
Comment thread lightweight_charts/js/styles.css Outdated

Check warning

Code scanning / Stylelint (reported by Codacy)

Unexpected whitespace at end of line (no-eol-whitespace) Warning

Unexpected whitespace at end of line (no-eol-whitespace)
Comment thread lightweight_charts/js/styles.css Outdated

Check warning

Code scanning / Stylelint (reported by Codacy)

Expected indentation of 2 spaces (indentation) Warning

Expected indentation of 2 spaces (indentation)
Comment thread lightweight_charts/js/styles.css Outdated

Check warning

Code scanning / Stylelint (reported by Codacy)

Expected indentation of 2 spaces (indentation) Warning

Expected indentation of 2 spaces (indentation)
Comment thread lightweight_charts/js/styles.css Outdated

Check warning

Code scanning / Stylelint (reported by Codacy)

Expected indentation of 2 spaces (indentation) Warning

Expected indentation of 2 spaces (indentation)
Comment thread lightweight_charts/js/styles.css Outdated

Check warning

Code scanning / Stylelint (reported by Codacy)

Unexpected empty line before declaration (declaration-empty-line-before) Warning

Unexpected empty line before declaration (declaration-empty-line-before)
Comment thread lightweight_charts/js/styles.css Outdated

Check warning

Code scanning / Stylelint (reported by Codacy)

Expected indentation of 2 spaces (indentation) Warning

Expected indentation of 2 spaces (indentation)
github-advanced-security[bot]
github-advanced-security AI found potential problems Apr 18, 2025
View reviewed changes
Comment thread lightweight_charts/abstract.py
from webview.errors import JavascriptException

import shutil
from typing import Optional

Check warning

Code scanning / Prospector (reported by Codacy)

Reimport 'Optional' (imported line 6) (reimported) Warning

Reimport 'Optional' (imported line 6) (reimported)
Comment thread lightweight_charts/abstract.py
from webview.errors import JavascriptException

import shutil
from typing import Optional

Check warning

Code scanning / Prospector (reported by Codacy)

redefinition of unused 'Optional' from line 6 (F811) Warning

redefinition of unused 'Optional' from line 6 (F811)
Comment thread lightweight_charts/abstract.py
(line) => line.series === {self.id}.series
);

if (legendItem) {{

Check warning

Code scanning / Prospector (reported by Codacy)

Access to a protected member _lines of a client class (protected-access) Warning

Access to a protected member _lines of a client class (protected-access)
Comment thread lightweight_charts/abstract.py
);

if (legendItem) {{
{self._chart.id}.legend.div.removeChild(legendItem.row);

Check warning

Code scanning / Prospector (reported by Codacy)

Access to a protected member _lines of a client class (protected-access) Warning

Access to a protected member _lines of a client class (protected-access)
Comment thread lightweight_charts/abstract.py
:param key: The key under which to store the script.
:param options: A JSON string containing the script options.
"""
"""Save script JSON data to memory and disk."""

Check warning

Code scanning / Prospector (reported by Codacy)

Unexpected keyword argument 'dirs_exist_ok' in function call (unexpected-keyword-arg) Warning

Unexpected keyword argument 'dirs_exist_ok' in function call (unexpected-keyword-arg)
Comment thread run.py
get_bar_data(chart, chart.topbar['symbol'].value, chart.topbar['timeframe'].value)

if __name__ == '__main__':
import pandas as pd

Check warning

Code scanning / Prospector (reported by Codacy)

Reimport 'pandas' (imported line 4) (reimported) Warning

Reimport 'pandas' (imported line 4) (reimported)
Comment thread run.py
get_bar_data(chart, chart.topbar['symbol'].value, chart.topbar['timeframe'].value)

if __name__ == '__main__':
import pandas as pd

Check warning

Code scanning / Prospector (reported by Codacy)

redefinition of unused 'pd' from line 4 (F811) Warning

redefinition of unused 'pd' from line 4 (F811)
Comment thread run.py

if __name__ == '__main__':
import pandas as pd
from time import sleep

Check warning

Code scanning / Prospector (reported by Codacy)

Unused sleep imported from time (unused-import) Warning

Unused sleep imported from time (unused-import)
Comment thread run.py

if __name__ == '__main__':
import pandas as pd
from time import sleep

Check warning

Code scanning / Prospector (reported by Codacy)

'time.sleep' imported but unused (F401) Warning

'time.sleep' imported but unused (F401)
Comment thread run.py
))

sleep(0.2)

Check warning

Code scanning / Prospector (reported by Codacy)

Trailing newlines (trailing-newlines) Warning

Trailing newlines (trailing-newlines)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@EsIstJosh @github-advanced-security