symbols_distances.gnuplot: fix the pngcairo terminal output file name

CallDistanceTransceiver.py: assign a float value to inter_symbol_distance

For consistency always assign a float value to inter_symbol_distance, as
it is going to be used in calculations with other float values.

This also fixes an pylint message:

  R:101,12: Redefinition of inter_symbol_distance type from float to int
  (redefined-variable-type)

CallDistanceTransceiver.py: clarify the use of a transceiver in DummyModem

Put standard modules imports before other imports

This fixes wrong-import-order message from pylint.

CallDistanceTransceiver.py: consider possible delays in transmit_symbol()

The ATH command can take some time, resulting in a positive delay; or
the first time.sleep() may terminate earlier resulting in a negative
delay.

In either case compensate for these delays in the second sleep, in order
to try to diverge as little possible from the total nominal symbol
transmission distance.

Add a program to measure the call setup time from Android to PC

plot: add some more logs from the Android app

CallDistanceTransceiver.py: use a more conservative call_setup_time_max default

MorseDistanceModulator.py: improve a comment

Don't refer to "calls" anymore, MorseDistanceModulator is quite generic
and uses the period/pulse terminology.

Improve the grammar too...

MorseDistanceModulator.py: be more precise in an error message

Move the MorseDistanceModulator class to its own file

TODO: mention some of the known issues in the TODO list

call_distance_modulation.gnuplot: try to improve the look of xtics labels

Increasing the xtics spacing and decreasing the font size makes the plot
look a little less crowded for long transmissions.

CallDistanceTransceiver.py: rename receive_character() to receive_symbol()

The name receive_symbol() is more appropriate because the function
receives Morse symbols, not directly text characters.

CallDistanceTransceiver.py: make log_symbol() a standalone function

There is no need for it to be a method, it does not use "self".

CallDistanceTransceiver.py: destination_number does not have to be a member

The destination_number is only used int he transmit functions, it can
very well be just an argument of the transmit functions.

CallDistanceTransceiver.py: rename test_send_receive to test_transmit_receive

All the other code uses the "transmit" verb instead of "send".

CallDistanceTransceiver.py: reset modem.ring_count before receiving

CallDistanceTransceiver.py: add a transmission test in test_send_receive()

CallDistanceTransceiver.py: make DummyModem look more like Modem

The "response" argument is optional in the get_response() method, this
avoids getting an error when trying to transmit with DummyModem:

  TypeError: get_response() missing 1 required positional argument: 'response'

CallDistanceTransceiver.py: use the modulator member in test_send_receive()

CallDistanceTransceiver.py: factor out a MorseDistanceModulator class

Abstract the modulation into its own class, this simplifies the code of
the CallDistanceTransceiver class.

While at it also start using the difference between a "period" and
a "pulse" in SymbolTime, some more documentation about what they mean
will be added in some future commit.

call_distance_modulation.gnuplot: s/ingoing/incoming/

Don't add trailing spaces when producing logs

Add tools to plot the symbol distances

This helps comparing the effect of different parameters values.

Add measure_call_setup_time_to_android.py

This makes it easy to measure the call time when using the Android app
as a receiver.

The tool parses the "adb logcat" output and gets info from the
SaveMySugar app log.

Add session_stats.py: a tool to perform some call time analysis on logs

plot: add some logs acquired using th Android app as a receiver

README.md: improve wording when explaining how to measure parameters

measure_ring_distance.py: improve output messages

measure_call_setup_time.py: don't measure "uncertainty" anymore

It's not immediately useful to know the uncertainty now that
CallDistanceTransceiver takes minimum and maximum parameters explicitly.

While at it also adjust a print message to be more descriptive.

logs: add logs produced with the new parameters

plot: update logs to the new format

call_distance_modulation.gnuplot: adjust to plot the new logs

CallDistanceTransceiver.py: make test_send_receive() use the new scheme

In  test_send_receive() use the new way of passing parameters to
CallDistanceTransceiver, and while at it, improve the test adding some
randomness.

CallDistanceTransceiver.py: use a min,max scheme instead of avg,uncertainty

Specify parameters using a minimum and a maximum values, like
call_setup_time_min,call_setup_time_max and ring_time_min,ring_time_max.

This makes it more intuitive to measure the stability of the line, and
makes the code more readable too.

CallDistanceTransceiver.py: remove a leftover print() statement

MorseTranslator.py: don't sanitize away word spaces

Don't strip word spaces when sanitizing strings, those spaces may be
part of the original message.

receive.py: fix a print statement and also flush the output

Remove unneeded parentheses in a print statements, and also flush the
output: when using logging and redirection sometimes the output does not
get printed.

CallDistanceTransceiver.py: reformat printing the symbols times

Have the same number of spaces on each line, this makes it easier to
plot the data by fields using the space as a field delimiter.

While at it also split the lines as they would be too long according to
PEP8.

CallDistanceTransceiver.py: remove unneeded empty line

CallDistanceTransceiver.py: fix a print format in a debug message

The times here are not exact integers, so print them as float.

CallDistanceTransceiver.py: fix an error message

The error is about call_distance, not ring_distance.

measure_call_setup_time.py: make the output more readable

Sleep _after_ having printed the info about the call, the output flow is
more understandable this way.

While at it also avoid the "First ring" message which is misleading
besides, it was a leftover from measure_ring_distance.py

Resume command mode after dialing, always

Put a semicolon after the AT command when dialing the destination
number. Without this, some modems won't accept the ATH command and won't
be able to terminate a call before the receiver has answered.

CallDistanceTransceiver.py: resume command mode after dialing

Put a semicolon after the AT command when dialing the destination
number. Without this, some modems won't accept the ATH command and won't
be able to terminate a call before the receiver has answered.

CallDistanceTransceiver.py: don't put the general case in an else

Since the code returns when the particular case of the very first call
occurs, there is no need to put the general case in the else.

README.md: improve wording and make Corrado's name more visible

Modem.py: use the more idiomatic "while True" instead of "while 1"

MorseTranslator.py: remove unneeded code in sanitize_morse()

The Morse string will never contain a '|' because it would have been
already replaced in a previous sanitization step.

Also we don't define anywhere that '|' can act as an alternative word
separator, so it should be replaced with a space just like any other
invalid character.

MorseTranslator.py: fix sanitize_text

Use a regex more adherent to the characters actually translated, also
follow the order in which the characters appear in the signals table.

Initial import