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

Could not find files for the given pattern(s) #645

Closed
wdrury-uk opened this issue Oct 30, 2020 · 10 comments
Closed

Could not find files for the given pattern(s) #645

wdrury-uk opened this issue Oct 30, 2020 · 10 comments

Comments

@wdrury-uk
Copy link

Description

To ensure not same issue as #182

  1. Rename .env.example to .env
  2. Config .env file
  3. Listed 8x stores in UK --- hidden in below example by xx
  4. npm run start

[17:34:28] info :: ℹ selected stores: xx, xx, xx, xx, xx
[17:34:28] info :: ℹ selected brands: yyy, yyy , yyy
[17:34:28] info :: ℹ selected series: zzzz
INFO: Could not find files for the given pattern(s).
INFO: Could not find files for the given pattern(s).
INFO: Could not find files for the given pattern(s).
INFO: Could not find files for the given pattern(s).
INFO: Could not find files for the given pattern(s).
INFO: Could not find files for the given pattern(s).
INFO: Could not find files for the given pattern(s).

continue to run but only does stores first 1st, 2nd, 5th, 7th stores

Have checked store names as per list and copy/pasted the names as required.

Possible solution

Tried all solutions under issue #183 and check no file extension of .txt - its not, just .env file.

@jef
Copy link
Owner

jef commented Oct 30, 2020

Has to do with sound notifications. Are you running Windows? Could be relevant: #275

@wdrury-uk
Copy link
Author

Thanks - yes running Windows and also installed the cmdmp3new.zip files into the code folder as detailed in the wiki,

@wdrury-uk
Copy link
Author

UPD: I have just move the .env out of the folder to see if INFO: cant find files error stop. Confirmed no messages and defaults used.

Next check is reset the .env file to the default one and run.

@wdrury-uk
Copy link
Author

Continue to test.

PASS Brands = asus,zotac,evga,msi,gigabyte,palit
PASS UK Stores = amazon-uk,currys,aria,ebuyer,box

FAIL = nvidia (COUNTRY=great_britain) - hangs, does not scan
FAIL = overclockers - hangs, does not scan
FAIL = scan - hangs, does not scan
FAIL = ccl - fails with STATUS CODE ERROR 503
FAIL = novatech - hangs, does not scan
FAIL = very - hangs, does not scan

continuing to verify other changed settings to find INFO root cause.

@wdrury-uk
Copy link
Author

wdrury-uk commented Oct 30, 2020

ROOT Cause identified its the PLAY_SOUND setting on Windows.

PLAY_SOUND=swiftly.mp3

followed the Windows Audio Notification Guide therefore put the file in the folder of the code "nvidia-snatcher" as per wiki guide linked recommends.

Its seems to still play the sound on a success stock find.
So not sure why the INFO: Could not find files for given pattern is happening.

I leave it for @jef to investigate if solvable. I feel more important is the number of UK site scans which hang and provide no results. Happy to assist, but although 30+ year developer, nodejs all new to me as left dev for last 8 year :)

@jef
Copy link
Owner

jef commented Oct 30, 2020

Awesome! Thanks for the details. I'm still a little lost on the Could not find files for given pattern is happening.

What's your PLAY_SOUND= set to? And if you leave it out, does it give you that warning still?


FAIL = nvidia (COUNTRY=great_britain) - hangs, does not scan

Just as a heads up, this API is kind of deprecated. Not sure if/when it will be online again.

@ethanwc
Copy link

ethanwc commented Oct 30, 2020

I also was facing this issue. I added the c files to play the sound which made that portion work. For me the warning is there only when I specify a sound file:

PLAY_SOUND=notification.wav

@wdrury-uk
Copy link
Author

wdrury-uk commented Oct 30, 2020

@jef
The PLAY_SOUND is set as I mention above.

PLAY_SOUND=swiftly.mp3

If I remove the setting to:

PLAY_SOUND=

The warnings not longer appear.

So as a developer I be looking for code which detects a file set to play, then the code its validates if I have a valid mp3 player and exclude any players not for Windows if the execution OS is Windows. As it appears its that validation code for other OS which is causing the warnings (not sure as not dug into the code yet). However, the sound does still play because I have installed the cmdmp3 binaries on Windows.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 5, 2020

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days

@jef
Copy link
Owner

jef commented Dec 9, 2020

Use PLAY_SOUND=./sound.mp3 if sound.mp3 is in the same folder as streetmerchant.

@jef jef closed this as completed Dec 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants