Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

number of threads -> 79 #48

Closed
jincho1 opened this issue Aug 7, 2021 · 5 comments
Closed

number of threads -> 79 #48

jincho1 opened this issue Aug 7, 2021 · 5 comments
Labels
bug Something isn't working
Milestone

Comments

@jincho1
Copy link

jincho1 commented Aug 7, 2021

When I run the same formula on my windows machine and in our cluster, Linux, I got 3 for the number of threads. "C2OH4 3 .017 3". I did not get an output with 79. So I need more information for the issue you faced to be able to understand why you got such an output. Can you please open a new issue for that and give us more information about how you run the MAYGEN-1.5.jar file? Did you download the jar file or build the maven project yourself ? But in any case, the jar was built and released based on the latest src code we have in the repository.

I downloaded MAYGEN-1.5.jar:
wget https://github.com/MehmetAzizYirik/MAYGEN/releases/download/V1.5/MAYGEN-1.5.jar

and ran like this:

java -jar MAYGEN-1.5.jar -f C2OH4 -v -t
MAYGEN is generating isomers of C2OH4...
The number of structures is: 3
Time: .015 seconds
C2OH4 3 .015 79

I ran the program in Linux release 7.9.2009

I also noticed that there is a problem with the sdf the program generates when I used a more complex formula

java -jar MAYGEN-1.5.jar -f C10H9N3 -v -t -d temp -m 10

I copy and pasted the problem area. It looks like each thread is writing to the file at the same time.

Molecule 2
MAYGEN 20210615
22 22 0 0 0 0 0 0 0999 V2000

Molecule 3
MAYGEN 20210615
0.0000 0.0000 0.0000 C 0 0 0 0 0 0 0 0 0 0 0 0
22 22 0 0 0 0 0 0 0999 V2000
0.0000 0.0000 0.0000 C 0 0 0 0 0 0 0 0 0 0 0 0
0.0000 0.0000 0.0000 C 0 0 0 0 0 0 0 0 0 0 0 0
0.0000 0.0000 0.0000 C 0 0 0 0 0 0 0 0 0 0 0 0
0.0000 0.0000 0.0000 C 0 0 0 0 0 0 0 0 0 0 0 0
0.0000 0.0000 0.0000 C 0 0 0 0 0 0 0 0 0 0 0 0
0.0000 0.0000 0.0000 C 0 0 0 0 0 0 0 0 0 0 0 0
0.0000 0.0000 0.0000 C 0 0 0 0 0 0 0 0 0 0 0 0
0.0000 0.0000 0.0000 C 0 0 0 0 0 0 0 0 0 0 0 0
0.0000 0.0000 0.0000 C 0 0 0 0 0 0 0 0 0 0 0 0
0.0000 0.0000 0.0000 C 0 0 0 0 0 0 0 0 0 0 0 0
0.0000 0.0000 0.0000 C 0 0 0 0 0 0 0 0 0 0 0 0
0.0000 0.0000 0.0000 C 0 0 0 0 0 0 0 0 0 0 0 0
0.0000 0.0000 0.0000 C 0 0 0 0 0 0 0 0 0 0 0 0
0.0000 0.0000 0.0000 C 0 0 0 0 0 0 0 0 0 0 0 0

Molecule 6
MAYGEN 20210615
0.0000 0.0000 0.0000 C 0 0 0 0 0 0 0 0 0 0 0 0
22 22 0 0 0 0 0 0 0999 V2000
0.0000 0.0000 0.0000 C 0 0 0 0 0 0 0 0 0 0 0 0
0.0000 0.0000 0.0000 C 0 0 0 0 0 0 0 0 0 0 0 0
0.0000 0.0000 0.0000 C 0 0 0 0 0 0 0 0 0 0 0 0
0.0000 0.0000 0.0000 C 0 0 0 0 0 0 0 0 0 0 0 0
0.0000 0.0000 0.0000 N 0 0 0 0 0 0 0 0 0 0 0 0
0.0000 0.0000 0.0000 C 0 0 0 0 0 0 0 0 0 0 0 0
0.0000 0.0000 0.0000 N 0 0 0 0 0 0 0 0 0 0 0 0

@MehmetAzizYirik
Copy link
Owner

@jincho1 Thanks a lot Jin for the explanation. We will look at why you got that 79 in your output, although we dont get such an output when we test.

We will look at the SDF issue in the multithread mode. Please, for now, run MAYGEN in single-thread mode, so without -m option in default.

@javadev Valentyn, please take a look at the SDF issue as soon as possible.

@javadev
Copy link
Contributor

javadev commented Aug 7, 2021

We may collect the molecule block and write it once not line by line. It may help.

@javadev
Copy link
Contributor

javadev commented Aug 8, 2021

@MehmetAzizYirik I prepared a fix #49

@javadev
Copy link
Contributor

javadev commented Aug 11, 2021

@jincho1 the new issue was created for the wrong number of threads in sequence mode #50

@MehmetAzizYirik
Copy link
Owner

@jincho1 we already solved the issue regarding the number of threads and it is merged. Also the SDF generation issue is solved and merged to the main branch. It will be included in the future release.

@MehmetAzizYirik MehmetAzizYirik added the bug Something isn't working label Aug 11, 2021
@MehmetAzizYirik MehmetAzizYirik added this to the V1.6 milestone Aug 11, 2021
@javadev javadev mentioned this issue Feb 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants