= Description =

## Please read this page. Lines prefixed with ## such as this one are comments,
## you can remove them (except for those after the "Plan" section.)
## Please follow the instructions given in those comments and the text.
## After creating the bug page, please subscribe to it! We may have questions
## that only you can answer, and if you get email when your bug changes it'll
## be much faster to get it fixed since you can answer the questions!

According to the documentation

http://www.magpar.net/magpar/doc/html/preprocessing.html#gmsh

the "Physical Volumes" in Gmsh .geo files should be numbered consecutively starting from 0.

However, this is an error in the documentation. Their numbering of the "Physical Volumes" has to start with 1. 

Bug reported by Rajmani Gaur - thank you!

== Steps to reproduce ==
## Describe the steps needed to reproduce the bug. If we can't reproduce it, we  probably can't fix it.
 1. See attached file [[attachment:cylinder.geo]]
 1. Mesh it with "gmsh -3 cylinder.geo", which gives cylinder.msh
 1. "gmshtoucd cylinder.msh" gives cylinder.inp
 1. Run simulation with cylinder.inp.

== Example ==
magpar output/error message:
{{{
[...]
Opening file cylinder.inp
Number of grid vertices: 2937
Number of grid elements: 13696
Number of vertex data:   3
Number of element data:  0
Number of element data2: 0
Allocating array for vertices and reading their coordinates
Allocating array for element connectivity and reading it
magpar.exe: io/readinp.c:220: int ReadINP(GridData*, char*, _p_Vec*, int):
Assertion `a1>0' failed.
Aborted
}}}

== Details ==

## magpar output, magpar input files (as attachments)
## Please fill in the details here:

|| '''magpar version''' || 0.8 ||
|| '''OS and version''' || any ||
|| '''Compiler name and version''' || any ||
|| '''Machine/hardware/CPU type''' || any ||
|| '''Library versions''' || any ||

## refer to/explain attached files (attach after completing this bug report):
## [[attachment:allopt.txt]]
## [[attachment:model.inp]]
## [[attachment:model.krn]]
## [[attachment:stdouterr.txt]]

== Workaround/Fix ==
## How to deal with the bug until it is fixed

Make sure "Physical Volumes" in .geo files are numbered consecutively starting from 1.

Here is a fixed version of the Gmsh file: [[attachment:cylinder_fixed.geo]]

= Plan =
## This part is for magpar developers:

 * Priority: medium
 * Assigned to: WernerScholz
 * Status: fixed: documentation fixed in development version leading to 0.9

= Attachments =
<<AttachInfo>>
<<AttachList>>

----
## Category MagparBugConfirmed - if the bug has been confirmed by a 3rd party and bug report is complete
## Category MagparBugFixed - after the bug is fixed/a patch available
CategoryMagparBugFixed