Resolve "Implement basic structure for the hexagon experiment"

Closes #94 (closed)

• New Hexagon class for hexagon simulations
• A few associated unit tests

added hexagon label

added 1 commit

• 3f5d0fe9 - First attempt to simulate hexagon

Compare with previous version

added 1 commit

• 3761b4d5 - 1st version of hexagon

Compare with previous version

added 13 commits

• 3761b4d5...0d4aec35 - 11 commits from branch master
• a9cbc8c2 - First attempt to simulate hexagon
• 51d19318 - 1st version of hexagon

Compare with previous version

added 1 commit

• f4d99601 - small modifications for pylint

Compare with previous version

added 1 commit

• 99c8a3e6 - write() and _write_attr()

Compare with previous version

added 1 commit

• 35cb31a4 - minor modifications for pylint

Compare with previous version

added 1 commit

• cb1bbb5a - minor modifications for pylint

Compare with previous version

added 1 commit

• 8faf2a2e - minor modifications for pylint

Compare with previous version

requested review from @j2b.bayle

That looks really good. Can you just add your name and email at the top of the file, in the author list?

added 1 commit

• a885efe2 - modified an author list

Compare with previous version

added 9 commits

• 519024fe - First attempt to simulate hexagon
• ce8bbac5 - write() and _write_attr()
• 197602a8 - Fix Pylint warnings
• c35263b4 - Add Kohei as author
• 6f5758b4 - Fix missing documentation
• 79962513 - Minor changes
• 017c4839 - Add logging to simulation
• 60552a43 - Refactor writing functions
• b018c59d - Fix unit tests

Compare with previous version

I've added a couple of fixes and changes (mostly cosmetic, and tests).

I've changed the write function so that it's closer to what we have in Instrument (or what it soon will be). Can you check that it's okay with you?

marked this merge request as ready

changed the description

resolved all threads

merged

mentioned in commit ed968c75