Skip to content
This repository has been archived by the owner on Dec 3, 2021. It is now read-only.

Contribution from ESRF #24

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

rafaelcelestre
Copy link

@rafaelcelestre rafaelcelestre commented Mar 11, 2019

As part of the ViNYL project.

adding srwlib to init.py
fix srwpy packaging + remove versioneer
Rework srwpy to vinyl_srw
bug fix on imports for srwlib
adding windows build
adding dependencies verification before build


Description form @srio (see mrakitin#1):

Now the package is installable:

pip install vinyl_srw

Right now the imports should be:
from vinyl_srw.srwlib import *

Change the calls and directory vinyl_srw to just srwpy to get the imports:

from srwpy.srwlib import *

This is the standard way to import in python. All modules must have the same root. It is not good to have different modules at the main level, like the present situation
from srwlib import *
from uti_math import *

tschoonj and others added 2 commits April 2, 2019 16:19
Fixes:

compiling .pyc files...
  File "lib/python3.6/site-packages/vinyl_srw/SRWLIB_ExampleViewDataFile.py", line 10
    from __future__ import print_function #Python 2.7 compatibility
                                                                  ^
SyntaxError: from __future__ imports must occur at the beginning of the file
SRWLIB_ExampleViewDataFile.py: fix SyntaxError
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants