#41

Looks solid to me, arguably you don’t need to backticks on the number 2 in the meadowphysics.md file as it’s not “code”. But it’s neither here nor there.

Good thinking to mention Ansible!

Do you want to have a go at opening a PR? Or would you like me to add the files for you?

#42

Yea i think I’m up for it!

I just did the quick github “Hello World” tutorial. If I followed correctly, do I make a branch of the teletype-docs, add my files to the docs folder, commit the change, and then open a pull request?

Did i miss something big, or is there a better way to do all that?

#43

That’s it. Make sure it’s from your own fork though.

The workflow is:

Fork my repo on GitHub, then…

git clone <your fork>
git branch meadowphysics-docs
git checkout meadowphysics-docs

Make your changes

git add <changed files> # multiple times if necessary
git commit              # warning this will open VIM... :wq
git push -u origin

Go on the GitHub website and open a PR

If you’re using the desktop client, hopefully the command line steps should correspond to what you do in the GUI.

#44

I think I follow, and it seemed pretty straightforward to do all that inside the github webpage

I opened a pull request in my branch in my fork. Let me know if I need to do anything else!

#45

Ah, you’ve opened a pull request to yourself…

Can you try editing the PR, and changing the base fork to “samdoshi:docs”, that way you’ll open one up to me.

If not, I can grab the commit myself (and you’ll still show up as the author).

1 Like
#46

Merged :heart_eyes:

2 Likes
#47

I can jump on metronomes next

1 Like
#48

Cool, another short one. But one that probably needs a bit of introduction, you could just crib from the current docs: http://monome.org/docs/modular/teletype/

  • M (default value 1000)
  • M.ACT (default value 1)
  • M.RESET

M and M.ACT have set versions.

The lowest value for M is 10. But it might be worth warning about too much i2c in M scripts at 10ms

@tehn what’s the lowest recommended lowest value for M before things start getting a bit crazy?

#49

Just a heads up, I’m thinking about introducing a M! operator for setting the metro to unsupported speeds.

You may wish to take on a different section for now. If you’ve already written up the metro bits, I can tweak them with info about the new op. (And sorry about that!)

#50

Submitted a pull request for WW ops. I can take the ES ops next if you like?

#51

Perfect. Thanks for the PR, I’ll get it merged in later today.

Earthsea ops:

  • ES.PRESET
  • ES.MODE
  • ES.CLOCK
  • ES.RESET
  • ES.PATTERN
  • ES.TRANS
  • ES.STOP
  • ES.TRIPLE
  • ES.MAGIC
#52

Hmm, no II op for ES.EDGE. That seems like something worth adding to the ES code.

#53

OK, it’s about 4 lines of code to add that. Can someone let me know if that’s something to include. Otherwise the ES ops are documented.

#54

Would it be possible to hang back on that? I’d really like to get 2.0 released, and I feel like I’m fighting on too many fronts already.

(I take it will also require changes to the Earthsea firmware too?)

#55

No problem. The pull request only has the existing commands. It’s a change to the ii.h and Earthsea main.c, so yes, would require an Earthsea firmware update.

#56

Yeah, we’ve got the added complication with the Trilogy modules that they need a very small amount of modernisation to build against the latest libavr32 after the changes that have been made there. At the moment we’re just concentrating on keeping Ansible up to date.

I don’t think the changes will take much more than 30 mins for each module (including testing). But until I’ve got my Teletype back into my main rack I can’t do it.

I’ve merged your PR, thank you! :heart:

When I build 2.0b10 later today, I’ll include the latest PDF in it too.

#57

are you proposing simply a sort of TT-control of the EDGE output? or toggling the various edge modes?

#58

The latter, the ability to set the current edge mode from TT.

1 Like
#59

makes sense. feel free to create git issues for this on both repos

#60

No problem, I worked on those last night, so I’ll set up a pull request this morning for what I came up with and you can tweak away

1 Like