Minor fixes around annotations

[nvasilakis]

No description provided.

[github-actions]

OS:ubuntu-18.04
Thu Jul 29 16:33:48 UTC 2021
intro: 2/2 tests passed.
interface: 6/6 tests passed.
compiler: 48/54 tests passed.
evaluation/tests/results/ann-agg_2_distr_.time
evaluation/tests/results/ann-agg_8_distr_.time
evaluation/tests/results/bigrams_2_distr_.time
evaluation/tests/results/bigrams_8_distr_.time
evaluation/tests/results/shortest_scripts_2_distr_.time
evaluation/tests/results/shortest_scripts_8_distr_.time

[github-actions]

OS:ubuntu-20.04
Thu Jul 29 16:53:02 UTC 2021
intro: 2/2 tests passed.
interface: 6/6 tests passed.
compiler: 48/54 tests passed.
evaluation/tests/results/ann-agg_2_distr_.time
evaluation/tests/results/ann-agg_8_distr_.time
evaluation/tests/results/bigrams_2_distr_.time
evaluation/tests/results/bigrams_8_distr_.time
evaluation/tests/results/shortest_scripts_2_distr_.time
evaluation/tests/results/shortest_scripts_8_distr_.time

[angelhof]

I think that we should not merge this before we investigate what are the new failing test cases. I might have some time to take a look. If we want to merge these (and other changes) faster (without making sure that correctness tests pass) we can use the future branch.

[nvasilakis]

@angelhof Agreed! BTW we think we know why tests are failing, and we're working to fix them.

[angelhof]

This is not yet supported by PaSh. We can open an issue or add a test with a long version of one of these flags and use that in a PR to implement that.

[angelhof]

Similarly, this is not implemented at the moment (and is very crucial for all the commands that read from stdin if they have no arguments). In its current form, the annotation leads to test failures because uses of cut that read from stdin are not supported (that is why previously cut's annotation only reads from stdin).

[angelhof]

This is probably the reason why tests fail.

[nvasilakis]

OK there are probably multiple reasons why the tests are failing:P