THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY.

A new Flyspray task has been opened.  Details are below. 

User who did this - Markus Wolff (m_wolff) 

Attached to Project - DuMuX
Summary - Inconsistent naming - special status of saturation
Task Type - Bug Report
Category - General
Status - New
Assigned To - 
Operating System - Unspecified
Severity - Low
Priority - Normal
Reported Version - SVN
Due in Version - Undecided
Due Date - Undecided
Details - We have already discussed this topic, but it feels like years ago...

I think the naming of variables, types, functions, etc. is pretty much in 
agreement with the naming guidelines formulated in the handbook.

However, whenever saturation is abbreviated by the letter S, it is written as 
capital letter independent of the position. E.g. Sw(), Sw, SwIdx, etc. 

I suggest to change this and write e.g. sw(), sn, snIdx, etc.

It would be even nicer to use a capital phase index like sW(), sW, sWIdx, etc. 
but this would include even more changes as we mostly us small phase indices 
like pnIdx, krw(), krn(), etc. The capillary pressure function on the other 
hand is named pC()...

Anyway, we should get this consistent!

More information can be found at the following URL:
http://www.dumux.org/flyspray/index.php?do=details&task_id=180

You are receiving this message because you have requested it from the Flyspray 
bugtracking system.  If you did not expect this message or don't want to 
receive mails in future, you can change your notification settings at the URL 
shown above.
_______________________________________________
Dumux mailing list
[email protected]
https://listserv.uni-stuttgart.de/mailman/listinfo/dumux

Reply via email to