search

FluxML / fluxml.github.io

Flux Website
https://fluxml.ai
MIT License
20 stars 45 forks source link

Issue with example Code #106

Closed nilsmartel closed 2 years ago

nilsmartel commented 2 years ago

I think I found a bug in your docs, though I can't find the right document, where it occurred.

Here params(model) should be ps (where ps = params([W, b])) if I'm not mistaken. (Still learning, might be wrong. Though params(model) yields Params([]), which seems wrong enough.

FAx6E1-XEAIa0Gr

Also, on the same page is this: FAx6XcOX0AMpNA-

in this section I think is the right moment to talk about VSCode as the preferred Julia IDE. I myself was quite confused, what I am supposed to use.

ToucheSir commented 2 years ago

WRT params vs Params, the full example at the bottom (i.e. the thing you can actually run) is correct but the snippet up top is not. This example is getting revamped in https://github.com/FluxML/fluxml.github.io/pull/103 and should be much nicer soon.

As for the section about IDEs, that was addressed in https://github.com/FluxML/fluxml.github.io/pull/103. Perhaps you have a stale cache?

nilsmartel commented 2 years ago

It took me a while to open this issue, I did not notice, that the website was updated in the meantime