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

Tech magma #102

Open
wants to merge 38 commits into
base: tech-master
Choose a base branch
from
Open

Tech magma #102

wants to merge 38 commits into from

Conversation

sangallidavide
Copy link
Member

Interface to the magma library for diagonalization of the BSE (also on GPU)

Developments by @andrea-ferretti and @blmelp

Pull request opened after suggesion of @andrea-ferretti

nicspalla and others added 14 commits March 22, 2023 10:28
    MODIFIED *  include/driver/version.h dipoles/DIPOLE_overlaps.F

    NEW *

    Bugs:
    - workaround for issue #729 to fix compilation

    Patch sent by:  Davide Sangalli <[email protected]>
Conflicts:
	config/report.in
	include/driver/version.h
	src/modules/.objects
MODIFIED *  configure include/version/version.m4

DELETED *  include/driver/version.h

Changes:
- deleted old version.h file (not needed anymore)

Patch sent by:  Davide Sangalli <[email protected]>
@sangallidavide
Copy link
Member Author

There is also a discussion in this connected pull-request: blmelp#1

Was this solved?

MODIFIED *  config/m4/acx_report.m4 config/report.in configure include/version/version.m4

Bugs:
- Few small fixes in the configure

Additions:
-

Changes:
-

Patch sent by:  Davide Sangalli <[email protected]>
@sangallidavide
Copy link
Member Author

There is also a discussion in this connected pull-request: blmelp#1

Was this solved?

@andrea-ferretti should we proceed with the merge?

I checked the code. I see:

  • the magma can be linked only externally for now
  • the magma are used only in case of serial diagonalization with complex eigenvalues (in practice bse with coupling). Is this correct?

MODIFIED *  include/version/version.m4 modules/mod_linear_algebra.F

Changes:
- [yambo] Added few messages

Patch sent by:  Davide Sangalli <[email protected]>
MODIFIED *  config/m4/acx_report.m4 config/m4/magma.m4 config/mk/global/actions/compile_external_libraries.mk config/mk/global/actions/download_external_libraries.mk configure include/version/version.m4 lib/archive/Makefile.loc lib/archive/package.list

NEW *  lib/magma/Makefile.loc

Changes:
- [yambo] few more steps to prepare internal magma compilation

Patch sent by:  Davide Sangalli <[email protected]>
@andreamarini
Copy link
Member

Dear Davide, MAGMA libs are necessary to use this branch?

If the answer is yes a documentation is needed in order to install them properly.

This is also needed in order to add the lib to the robots libraries.

THX

MODIFIED *  configure include/version/version.m4

DELETED *  config/stamps_and_lists/project_dependencies.stamp

Changes:
- [yambo] Configure re-generated after merge

Patch sent by:  Davide Sangalli <[email protected]>
MODIFIED *  configure include/version/version.m4

DELETED *  config/stamps_and_lists/project_dependencies.stamp

Changes:
- configure re-generated

Patch sent by:  Davide Sangalli <[email protected]>
sangallidavide and others added 11 commits October 15, 2024 00:54
MODIFIED *  config/m4/magma.m4 configure include/version/version.m4 lib/magma/Makefile.loc

NEW *  lib/magma/MGmake.inc_lib

DELETED *  config/stamps_and_lists/project_dependencies.stamp

Changes:
- [magma]  created first version of magma Makefile / make.inc

Patch sent by:  Davide Sangalli <[email protected]>
MODIFIED *  config/m4/acx_report.m4 config/m4/magma.m4 config/mk/global/actions/compile_internal_libraries.mk config/mk/global/libraries.mk config/report.in configure include/version/version.m4 modules/.objects

NEW *  lib/magma_fmodules/.objects

RENAMED *  modules/mod_magma2.F -> lib/magma_fmodules/mod_magma2.F modules/mod_magma2_cfortran.F -> lib/magma_fmodules/mod_magma2_cfortran.F modules/mod_magma2_common.F -> lib/magma_fmodules/mod_magma2_common.F modules/mod_magma2_dfortran.F -> lib/magma_fmodules/mod_magma2_dfortran.F modules/mod_magma2_sfortran.F -> lib/magma_fmodules/mod_magma2_sfortran.F modules/mod_magma2_zfortran.F -> lib/magma_fmodules/mod_magma2_zfortran.F

Changes:
- [magma] fmodules coved inside lib/magma_fmodules folder as discussed
  during last meeting

Patch sent by:  Davide Sangalli <[email protected]>
MODIFIED *  config/m4/magma.m4 config/mk/global/defs.mk.in config/mk/global/libraries.mk config/report.in configure include/version/version.m4 lib/magma_fmodules/.objects

Changes:
- [magma_fmodules] Check for modules refined

Patch sent by:  Davide Sangalli <[email protected]>
MODIFIED *  configure include/version/version.m4 lib/qe_pseudo/DOUBLE_project.dep lib/slatec/DOUBLE_project.dep

Changes:
- fixed .dep files

Patch sent by:  Davide Sangalli <[email protected]>
MODIFIED *  configure include/version/version.m4 pol_function/X_redux.F

Bugs:
 -Fixed a small bug in parallelization with intel

Additions:
-

Changes:
-

Patch sent by:  Claudio Attaccalite <[email protected]>
MODIFIED *  configure include/version/version.m4

Additions:
 -Version updated

Changes:
-

Patch sent by:  Claudio Attaccalite <[email protected]>
MODIFIED *  configure include/version/version.m4 pol_function/X_redux.F

Bugs:
 -Sorry I did a commit in the wrong branch, reverted

Additions:
-

Changes:
-

Patch sent by:  Claudio Attaccalite <[email protected]>
MODIFIED *  config/m4/netcdf_f90.m4 configure include/version/version.m4

Changes:
- [yambo] Tentative fix for issue detected in
  #168 with netcdf detection

Patch sent by:  Davide Sangalli <[email protected]>
@sangallidavide
Copy link
Member Author

Dear Davide, MAGMA libs are necessary to use this branch?

No, they are optional. I'm not the expert here. @andrea-ferretti did most of the coding

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants