Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Apr 21 03:19

    brunchboy on main

    Create project. Remove stray Clojure reference. Update POM in preparation for s… and 53 more (compare)

  • Apr 21 03:19

    brunchboy on master

    (compare)

  • Oct 18 2020 18:46

    dependabot[bot] on maven

    (compare)

  • Oct 18 2020 18:46
    dependabot[bot] closed #5
  • Oct 18 2020 18:46
    dependabot[bot] edited #5
  • Oct 18 2020 18:46
    dependabot[bot] commented #5
  • Oct 18 2020 18:46
    dependabot[bot] edited #5
  • Oct 18 2020 18:45
    brunchboy commented #5
  • Oct 18 2020 18:42

    brunchboy on master

    Update change log [skip ci] (compare)

  • Oct 18 2020 18:42

    github-actions[bot] on v0.1.9-SNAPSHOT

    (compare)

  • Oct 18 2020 18:41

    brunchboy on master

    Update junit due to security co… (compare)

  • Oct 13 2020 04:36
    dependabot[bot] labeled #5
  • Oct 13 2020 04:36
    dependabot[bot] opened #5
  • Oct 13 2020 04:36

    dependabot[bot] on maven

    Bump junit from 4.13 to 4.13.1 … (compare)

  • May 25 2020 01:09

    brunchboy on master

    Add missing require [skip ci] Remove unused requires [skip ci] (compare)

  • May 13 2020 03:06

    brunchboy on master

    Only index the latest version o… (compare)

  • May 07 2020 03:23

    brunchboy on master

    Really fix comment, and indenta… (compare)

  • May 07 2020 03:21

    brunchboy on master

    Fix comment [skip ci] (compare)

  • May 07 2020 03:01

    brunchboy on master

    Remove fragments from site navi… (compare)

  • May 07 2020 02:52

    brunchboy on master

    Fix a great many broken links. … (compare)

James Elliott
@brunchboy
Have you been able to see the SysEx bytes?
Wait!
There is clearly an error in the manual. For the hex version of the command they have the value 0b, and for decimal they have 11.
Sorry, for decimal they have 10.
0b is 11.
Let’s try sending 11 rather than 10 and see if it works.
Would you like to edit the file or have me push the change?
Benjamin Gudehus
@hastebrot
I could edit it if you show me the line number :)
James Elliott
@brunchboy
Line 88 of launchpad_mk2.clj.
Change 24 10 led to 24 11 led.
Benjamin Gudehus
@hastebrot
Done. And now I have to reload my file into the REPL?
James Elliott
@brunchboy
Just that function
(the whole file should be harmless, but if you can find the key for just the function, that is safest)
Benjamin Gudehus
@hastebrot
Hmm, so I just call mk/set-pad-color and it will automatically use the changed source code?
James Elliott
@brunchboy
Correct.
REPL-based development is awesome like that.
(If you have reloaded that function.)
There must be a command in Cursive for that.
Benjamin Gudehus
@hastebrot
Sync files in REPL (Ctrl-Shift-M)
Whoa!
Now it is correct.
James Elliott
@brunchboy
Syncing all the files is way overkill, I still am trying to find how you could have just evaluated that one form, but EXCELLENT.
I will let the person at Novation I am working with about this error in the manual!
How strange that it was doing anything at all with the wrong SysEx prefix.
Benjamin Gudehus
@hastebrot
Loading src/afterglow/controllers/launchpad_mk2.clj... done
Cursive intelligently knows which files were changed.
James Elliott
@brunchboy
The reason I want to do just the function, not the whole file, is that sometimes protocols get redefined, which causes incompatibilities with things that were created before reloading the file.
In this case that did not happen, but in other parts of Afterglow, it does cause issues.
Benjamin Gudehus
@hastebrot
Ahh, ok.
James Elliott
@brunchboy
(It won’t be an issue when you are in your own project.)
Benjamin Gudehus
@hastebrot
Sample cues are also correct.
James Elliott
@brunchboy
That’s great.
Benjamin Gudehus
@hastebrot
(mk2/deactivate mk2-watcher)
CompilerException clojure.lang.ExceptionInfo: Invariant violation in `afterglow.controllers.launchpad-mk2:564` [pred-form, val]:
 [(#{:launchpad-mk2 :launchpad-mk2-watcher} (type controller)), :launchpad-pro] {:*?data* nil, :elidable? true, :dt #inst "2016-02-12T02:08:55.758-00:00", :val :launchpad-pro, :ns-str "afterglow.controllers.launchpad-mk2", :val-type clojure.lang.Keyword, :?err nil, :*assert* true, :?data nil, :?line 564, :form-str "(#{:launchpad-mk2 :launchpad-mk2-watcher} (type controller))"}, compiling:(/home/benjamin/Sources/clojure/afterglow/test/afterglow/afterglow_sample.clj:16:1)
James Elliott
@brunchboy
I can’t see anything after the first line.
Benjamin Gudehus
@hastebrot
Added the rest.
Don't know if this is caused by (def mk2-controller @(:controller mk2-watcher)).
James Elliott
@brunchboy
I am giving up on fighting Cursive to try to figure out how you can run Tools :arrow_forward: REPL :arrow_forward: Run sexp before cursor
That is the command I wanted you to be able to use, but I will leave Cursive support to a Cursive user. I would suggest you explore Edit :arrow_forward: Structural Editing and Navigate :arrow_forward: Structural Movement when you have time.
What does (type c) give you?
Oh, I see!
Yes, my mistake.
Line 686 needs to be {:type :launchpad-mk2})]
Benjamin Gudehus
@hastebrot

Run sexp before cursor

You mean "Send form before caret to REPL"?

James Elliott
@brunchboy
Ah, sounds good! The page I was looking at was outdated.
So if you had put your cursor after the closing paren of set-pad-color and issued that command, it would have redefined just that function for us.
Benjamin Gudehus
@hastebrot
Ahh!
James Elliott
@brunchboy
Do you want to make a pull request with these two fixes, or shall I just do them locally?
Benjamin Gudehus
@hastebrot
I first thought something like (reload 'afterglow.controllers....set-led-color)..
James Elliott
@brunchboy
Ah, I don’t think Clojure itself offers any way of reloading just part of a file like that. But Cursive and CIDER do.
I’m afraid the easiest way to recover from the fact that I gave your controller the wrong type tag is to shut down and reload the REPL after that line is fixed.