Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

1200 call hets with auto #1201

Merged
merged 6 commits into from
Feb 12, 2015
Merged

1200 call hets with auto #1201

merged 6 commits into from
Feb 12, 2015

Conversation

0robustus1
Copy link
Contributor

Shall solve #1200.

@@ -4,7 +4,7 @@ class ParseCaller < Caller
# arguments: `hets +RTS -K1G -RTS -a none`
MODE_ARGUMENTS = {
fast_run: %w(just-structured),
default: %w(full-signatures full-theories)
default: %w(full-signatures full-theories auto)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Style/TrailingComma: Put a comma after the last item of a multiline hash.

@tillmo
Copy link
Member

tillmo commented Feb 12, 2015

wow, that was quick. While we are changing the options: could you also please add the option --casl-amalg=none? This prevents Hets from a time-consuming analysis of architectural specifications, which we (currently) do not need for Ontohub anyway.

Inside of the hets.yml there is a key server_options which
will allow us to pass arguments to the hets-server instance which
is managed by god or by the rake task (which is in turn called by
invoker in development).

As an additional gimmick: The rake task for hets now uses
the search-paths in the hets.yml.
@0robustus1
Copy link
Contributor Author

Should be done with a6ef163 for hets -X, for cmd-line tasks we were already using -a none which is actually the short version of --casl-amalg=none.

@eugenk
Copy link
Member

eugenk commented Feb 12, 2015

👍, but I prefer --casl-amalg=none over -a none and --apply-automatic-rule over -A. If you change it or not, is in your hands. You get the sign off either way.

0robustus1 added a commit that referenced this pull request Feb 12, 2015
@0robustus1 0robustus1 merged commit 83a5864 into staging Feb 12, 2015
@0robustus1 0robustus1 deleted the 1200-call_hets_with_auto branch February 12, 2015 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants