Skip to content

Create 03bis-LoadingParameters.adoc#1

Merged
prudhomm merged 5 commits into
feelpp:masterfrom
thomas-saigre:patch-1
Aug 3, 2020
Merged

Create 03bis-LoadingParameters.adoc#1
prudhomm merged 5 commits into
feelpp:masterfrom
thomas-saigre:patch-1

Conversation

@thomas-saigre

Copy link
Copy Markdown
Contributor

To see if I have well understood how it works, I tried to write this page. Maybe there are some imprecise...

To see if I have well understood how it works, I tried to write this page. Maybe there are some imprecise...
@prudhomm
prudhomm self-requested a review May 10, 2020 11:35

@prudhomm prudhomm left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

thanks for the doc.
The issue with url referring to a line is that if the line/the code is changed then the reference is out of date
probably a page on command line options is necessary soptions, doptions, ... as well as the functions.a-z...

// -*- mode: adoc -*-

Loading parameters
------------------

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

do not use ------ to underline the title
but use

= Loading parameters

to indicate a H1 header

------------------
=== Load a function

We define the Feel++ expression that we are going to integrate using the

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

to avoid issues with ++ use {feelpp} instead of Feel++

=== Load a function

We define the Feel++ expression that we are going to integrate using the
soption function that retrieves the command line option string `functions.g`.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

soption

@prudhomm

prudhomm commented Aug 3, 2020

Copy link
Copy Markdown
Member

thank you for the contribution

@prudhomm
prudhomm merged commit eb25c70 into feelpp:master Aug 3, 2020
@prudhomm prudhomm added this to the v1 milestone May 24, 2023
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