Mentions légales du service

Skip to content
Snippets Groups Projects

Renaming

Merged Mathieu Faverge requested to merge faverge/chameleon:issue47/renaming into master

This MR is supposed to fix issue #47 by applying the suggestion of renaming discussed and a little more. Here is a summary of the filename changes:

In include directory:

  • morse.h => chameleon.h
  • chameleon/chameleon_config.h.in stays as it is
  • chameleon/morse_zc.h => chameleon/chameleon_zc.h with all mixed-precision routines
  • chameleon/morse_z.h => chameleon/chameleon_z.h with all single precision routines
  • chameleon/morse_constants.h => chameleon/chameleon_constants.h for all Chameleon constants
  • chameleon/morse_fortran.h => chameleon/chameleon_fortran.h for wrappers of CHAMELEON_... routines only
  • chameleon/morse_simulate.h => chameleon/chameleon_simulate.h might be integrated in chameleon.h
  • chameleon/morse_struct.h => chameleon/chameleon_struct.h for CHAMELEON structures (descriptors, context)
  • chameleon/morse_types.h => chameleon/chameleon_types.h
  • chameleon/morse_runtime.h => runtime.h
  • chameleon/morse_struct.h => runtime/runtime_struct.h for RUNTIME structures (context, sequence, options)
  • chameleon/morse_kernels.h => Integrated into chameleon_tasks.h
  • chameleon/morse_tasks.h => chameleon/chameleon_tasks.h
  • chameleon/morse_tasks_z.h => chameleon/chameleon_tasks_z.h
  • chameleon/morse_tasks_zc.h => chameleon/chameleon_tasks_zc.h

Regarding the functions and structures:

  • Everything related to the RUNTIME API (except the tasks) becomes RUNTIME_ something
    • This includes RUNTIME_sequence_t, RUNTIME_request_t and RUNTIME_option_t that were called MORSE_xxxx
    • MORSE_TASK_xxx becomes InsertTask_xxxx to remove the MORSE keyword, and not put the RUNTIME one, but any other suggestion is more than welcome
  • All MORSE_ functions become CHAMELEON_
  • The same for all structure related to Chameleon, except morse_desc_t that become matrix_desc_t in an attempt to be generic.

All, and especially those involved in #47 discussions (@sylvand, @ltaief, @agullo, @fpruvost, @thibault, @furmento) please have a look and comment directly in the code on the web interface to start discussions where it's needed. I would like to close this issue as soon as possible, so we can definitely move towards the 1.0.0. Thanks.

Edited by Mathieu Faverge

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading