-
Notifications
You must be signed in to change notification settings - Fork 882
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
project reports cant find package
for a package that is not referenced at all
#1832
Comments
Thank you for reporting this issue. I am adding it to my list for investigation. As an alternative to unblock you, you could move this code to a WebContainer node project (https://stackblitz.com/fork/node) which will work the same as a local node project. You will need to add a bit more code to "serve" the example app though. |
@markwhitfeld unfortunately, suggested workaround is not an option as this is not about single project (even then moving it from browser to node is basically changing it completely) as tensorflow is used by hundreds of modules and all of them are impacted by this core issue - thus high severity impact. |
Ok, thank you for the additional context. If there is anything else that you can provide that may help to debug this, please add the info. I will take a look tomorrow. |
in case you need to understand tensorflow module internal dependencies:
|
If you are able to create a bare minimal reproduction project, with the least amount of code and the smallest number of dependencies, it will do wonders for getting this resolved quickly... Plus it will become part of our internal acceptance test pack.🎉 |
i did that, link is in the original issue, copied here as well: https://stackblitz.com/edit/typescript-blff19 |
The root of this issue lies in a bug in SystemJS. One critical piece of info I need from you though, is about which js file in your package should be used in the browser.
The normal default in StackBlitz is to use the |
thanks - seems my guess about string parsing was good. anyhow, there is no real difference between anyhow, typical behavior for most build environments is that if |
Thanks! Ok, that is great news, because we will be able to add a quicker fix for you that won't depend on that SystemJS PR. |
@vladmandic This issue should be resolved now. Apologies for the radio silence, my PC's SSD crashed last week! |
project reports
cant find package
for a package that is not referenced at allsimple reproduction: https://stackblitz.com/edit/typescript-blff19
project as-is works fine, but uncommenting two lines that import and access
@tensorflow/tfjs-backend-cpu
results in build error:but
@tensorflow/tfjs-backend-cpu
does not reference@tensorflow/tfjs-node
anywhere(and yes,
@tensorflow/tfjs-node
cannot be installed since its a nodejs module while this is a pure browser project)the only mention of
tfjs-node
insidetfjs-backend-cpu
module anywhere is inside string block:is it possible that stackblitz parser misbehaves and does not consider this a string and reports a dependency purely because of this?
also note that using other libraries such as
@tensorflow/tfjs-backend-webgl
that have a hard dependency on@tensorflow/tfjs-backend-cpu
do not present an issue and are working just fineand according to 3rd party user reports, all this used to work until january when some changes went live on stackblitz (according to some unconfirmed notices in old issues, thats when new module parser when live)?
note: this issue is also being tracked by @tensorflow team as a P0 issue - see tensorflow/tfjs#6315
The text was updated successfully, but these errors were encountered: