Skip to content
This repository has been archived by the owner on Sep 2, 2022. It is now read-only.

Windows Filename issue #77

Merged
merged 1 commit into from Aug 3, 2020
Merged

Conversation

cyber-jack
Copy link
Contributor

$ git clone https://github.com/peerchemist/finta
Cloning into 'finta'...
remote: Enumerating objects: 160, done.
remote: Counting objects: 100% (160/160), done.
remote: Compressing objects: 100% (109/109), done.
remote: Total 1227 (delta 84), reused 112 (delta 49), pack-reused 1067
Receiving objects: 100% (1227/1227), 522.49 KiB | 1.59 MiB/s, done.
Resolving deltas: 100% (657/657), done.
error: invalid path 'tests/data/bittrex:btc-usdt.csv'
fatal: unable to checkout working tree
warning: Clone succeeded, but checkout failed.
You can inspect what was checked out with 'git status'
and retry with 'git restore --source=HEAD :/'

Colon (:) is an illegal character in Windows, changed filenames so I can clone the repo.

@cyber-jack cyber-jack mentioned this pull request Aug 3, 2020
@peerchemist
Copy link
Owner

Okay, I did not know ":" is illegal in Windows.

@peerchemist peerchemist merged commit 05a2060 into peerchemist:master Aug 3, 2020
@cyber-jack cyber-jack deleted the windows branch August 3, 2020 13:33
@peerchemist
Copy link
Owner

Welp, it seems this nuked all the tests. I would guess you have corrupted the datasets.

@cyber-jack
Copy link
Contributor Author

I only changed the filenames? Both the filename and the line in code that refers to them.

@peerchemist
Copy link
Owner

Upon further inspection it seems your PSAR commit is what is nuking the tests. I am digging in to see what is going on.
I presume PSAR is writing into ohlc dataset rather than just using it's own internal numbers.

Give it a try maybe you spot what is going on.

@cyber-jack
Copy link
Contributor Author

cyber-jack commented Aug 6, 2020

I closed pull request #76 as I am still working on the PSAR function. Due to the nature of how PSAR is calculated the function will need further tweaking. I didn't have time to tweak it, can we revert the PSAR commit for now? I'll open another PR when finished tweaking the function and tested properly (next few days hopefully).

@peerchemist
Copy link
Owner

Just push patch onto master when done.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants