Skip to content

Fix typos in InformathAPI.hs - #14

Open
amelieled wants to merge 1 commit into
GrammaticalFramework:mainfrom
amelieled:patch-3
Open

Fix typos in InformathAPI.hs#14
amelieled wants to merge 1 commit into
GrammaticalFramework:mainfrom
amelieled:patch-3

Conversation

@amelieled

Copy link
Copy Markdown
Contributor

Need to update the HTML doc?

Need to update the HTML doc?
Comment thread src/InformathAPI.hs
and a ConstantTable (file .dkgf).
Both of these can be customized and passed as values of flags.
The following functions read them directly, but need hardly be called explicitly.
The following functions read them directly, but need to be called explicitly.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

hardly is a soft negation and cannot be omitted

Comment thread src/InformathAPI.hs
}

-- | When just converting form Dk to another formalism, no GF is needed.
-- | When just converting from Dedukti to another formalism, (GF is not needed).

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I think the original comment is clearer,

Comment thread src/InformathAPI.hs
]

-- | The scores for each tree an string, in JSON.
-- | The scores for each tree in string, in JSON.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

should be "and"

Comment thread src/InformathAPI.hs
gftree2nat env lang tree = linearize (grammar env) lang tree

-- | To unlex in a latex-like style, overridded by flag -no-unlex.
-- | To unlex in a LaTeX-like style, overrided by flag -no-unlex.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

overridden

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.

2 participants