ntedgi / node-efficientnet

tensorflowJS implementation of EfficientNet 🚀
https://www.npmjs.com/package/node-efficientnet
MIT License
257 stars 37 forks source link

Feature/language support scan check #81

Closed TomShlaim closed 2 years ago

TomShlaim commented 2 years ago

@ntedgi It's a dummy PR - I'll delete it soon

ntedgi commented 2 years ago

@ntedgi It's a dummy PR - I'll delete it soon

💪 Can you please open in separate issues the changes you made here I eventually want all of them to happened

TomShlaim commented 2 years ago

@ntedgi Do you mean by this to open separate branches? I'm afraid that my commits aren't well separated.

And just to emphasize, this specific PR is just for testing the nodejs scan problem.

ntedgi commented 2 years ago

@ntedgi Do you mean by this to open separate branches?

I'm afraid that my commits aren't well separated.

And just to emphasize, this specific PR is just for testing the nodejs scan problem.

I mean here

https://github.com/ntedgi/node-efficientnet/issues

For each small-medium change (for example: ‘upgrade angular cli’ ) open an new issue assign it with the correct labels . and then each change should be in separate branch

TomShlaim commented 2 years ago

@ntedgi Np, just one thing - It seems that I don't have permission to add labels

ntedgi commented 2 years ago

On it

ntedgi commented 2 years ago

On it

Now you basically can do anything include push to origin master 🚧☠️