Issue #2 resolved

Error in "class moptions_et(_enum)" enumeration

Petros Aristidou
created an issue

Hi,

First of all, thanks for this amazing tool!

I believe there is an error in the moptions_et enumeration. In metis.h (5.1.0) there exists the option "METIS_OPTION_NO2HOP" which doesn't exist in metis.py. It's between METIS_OPTION_SEED and METIS_OPTION_MINCONN.

As a result, when I try to activate the CONTIG flag from metis.py, it activates the MINCONN instead (shifted by 1).

That is: (edgecuts, parts) = metis.part_graph(G, k, contig=True, dbglvl=1) Minimize connectivity: Yes Create contigous partitions: No

(edgecuts, parts) = metis.part_graph(G, k, contig=False, dbglvl=1) Minimize connectivity: No Create contiguous partitions: No

Thanks in advance!

Petros

Comments (3)

  1. Petros Aristidou reporter

    Changing the class moptions_et to: class moptions_et(_enum): METIS_OPTION_PTYPE = 0 METIS_OPTION_OBJTYPE = 1 METIS_OPTION_CTYPE = 2 METIS_OPTION_IPTYPE = 3 METIS_OPTION_RTYPE = 4 METIS_OPTION_DBGLVL = 5 METIS_OPTION_NITER = 6 METIS_OPTION_NCUTS = 7 METIS_OPTION_SEED = 8 METIS_OPTION_NO2HOP = 9 METIS_OPTION_MINCONN = 10 METIS_OPTION_CONTIG = 11 METIS_OPTION_COMPRESS = 12 METIS_OPTION_CCORDER = 13 METIS_OPTION_PFACTOR = 14 METIS_OPTION_NSEPS = 15 METIS_OPTION_UFACTOR = 16 METIS_OPTION_NUMBERING = 17 #/ Used for command-line parameter purposes / METIS_OPTION_HELP = 18 METIS_OPTION_TPWGTS = 19 METIS_OPTION_NCOMMON = 20 METIS_OPTION_NOOUTPUT = 21 METIS_OPTION_BALANCE = 22 METIS_OPTION_GTYPE = 23 METIS_OPTION_UBVEC = 24

    Solves the problem.

  2. Log in to comment