Skip to content

Commit edd3101

Browse files
committed
merge develop
2 parents 5d8e42a + 519ad8f commit edd3101

File tree

150 files changed

+4548
-553
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

150 files changed

+4548
-553
lines changed

.github/workflows/dependencies.txt

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,8 @@ $PYTHON-devel
22
$PYTHON-numpy
33
$PYTHON-pytest
44
$PYTHON-sphinx
5+
$PYTHON-setuptools
6+
$PYTHON-dnf
57
CCfits-devel
68
blas-devel
79
$BOOST-$PYTHON-devel

.github/workflows/main.yml

Lines changed: 7 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -47,9 +47,6 @@ jobs:
4747
matrix:
4848
os-type: [fedora]
4949
os-version: ${{ fromJson(needs.get_fedora_releases.outputs.stable) }}
50-
include:
51-
- os-type: centos
52-
os-version: 7
5350
container: ${{ matrix.os-type }}:${{ matrix.os-version }}
5451
steps:
5552
- name: Checkout
@@ -58,14 +55,14 @@ jobs:
5855
id: package-version
5956
uses: astrorama/actions/[email protected]
6057
- name: Install dependencies
61-
uses: astrorama/actions/setup-dependencies@v3.4
58+
uses: astrorama/actions/setup-dependencies@v3.7
6259
with:
6360
dependency-list: .github/workflows/dependencies.txt
6461
- name: Build
6562
id: build
6663
uses: astrorama/actions/[email protected]
6764
- name: Upload RPM to GitHub
68-
uses: actions/upload-artifact@v2
65+
uses: actions/upload-artifact@v4
6966
with:
7067
name: ${{ matrix.os-type }}${{ matrix.os-version }}
7168
path: ${{ steps.build.outputs.rpm-dir }}/*.rpm
@@ -85,6 +82,7 @@ jobs:
8582
# Run tests #
8683
#############
8784
litmus-test:
85+
if: ${{ always() }}
8886
needs: [ build, get_fedora_releases ]
8987
runs-on: ubuntu-latest
9088
strategy:
@@ -95,21 +93,21 @@ jobs:
9593
container: ${{ matrix.os-type }}:${{ matrix.os-version }}
9694
steps:
9795
- name: Checkout
98-
uses: actions/checkout@v2
96+
uses: actions/checkout@v4
9997
- name: Install dependencies
100-
uses: astrorama/actions/setup-dependencies@v3
98+
uses: astrorama/actions/setup-dependencies@v3.7
10199
with:
102100
dependency-list: .github/workflows/test-dependencies.txt
103101
- name: Download RPM
104-
uses: actions/download-artifact@v2
102+
uses: actions/download-artifact@v4
105103
with:
106104
name: ${{ matrix.os-type }}${{ matrix.os-version }}
107105
- name: Install SourceXtractor++
108106
run: yum install -y SourceXtractorPlusPlus*.x86_64.rpm
109107
- name: Run tests
110108
run: ./.github/workflows/run-litmus.sh
111109
- name: Upload reports and logs
112-
uses: actions/upload-artifact@v2
110+
uses: actions/upload-artifact@v4
113111
with:
114112
name: ${{ matrix.os-type }}${{ matrix.os-version }}-test-results
115113
path: /tmp/sourcex/

.github/workflows/run-litmus.sh

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,8 @@ TARGET_BRANCH=${GITHUB_BASE_REF:-$GITHUB_REF}
66
# In the case of pull requests, this is the origin branch
77
ORIGIN_BRANCH=${GITHUB_HEAD_REF:-$GITHUB_REF}
88

9+
export GIT_CLONE_PROTECTION_ACTIVE=false
10+
911
# Platform-specific configuration
1012
source /etc/os-release
1113

.jenkinsFile

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
#!groovy
2+
@Library('integration-library@release-9') _
3+
pipelineElements(artifactId:"SourceXtractorPlusPlus", groupId:"EL", component:'eden.3.0')

CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,4 +6,4 @@ find_package(ElementsProject)
66
#---------------------------------------------------------------
77

88
# Declare project name and version
9-
elements_project(SourceXtractorPlusPlus 0.20 USE Alexandria 2.28 DESCRIPTION "SourceXtractor++, the next generation SExtractor")
9+
elements_project(SourceXtractorPlusPlus 1.1.0 USE Alexandria 2.32.0 DESCRIPTION "SourceXtractor++, the next generation SExtractor")

README.md

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -17,13 +17,13 @@ Documentation: https://astrorama.github.io/SourceXtractorPlusPlus/
1717
SourceXtractor++ is available on [Anaconda Cloud for Linux and MacOSX](https://anaconda.org/astrorama/sourcextractor)
1818

1919
```bash
20-
conda install -c conda-forge -c astrorama sourcextractor==0.19
20+
conda install -c conda-forge -c astrorama sourcextractor==1.0.3
2121
```
2222

2323
We would recommend, however, to install it into its own environment.
2424

2525
```bash
26-
conda create -n sourcex -c astrorama -c conda-forge sourcextractor==0.19
26+
conda create -n sourcex -c astrorama -c conda-forge sourcextractor==1.0.3
2727
conda activate sourcex
2828
```
2929

@@ -36,15 +36,15 @@ conda activate sourcex-dev
3636

3737
We recommend always using an explicit version number as there is no guarantee that `conda` will install the latest version otherwise.
3838

39-
## Fedora / CentOS
39+
## Fedora
4040

41-
SourceXtractor++ is shipped directly in Fedora and CentOS. It can be installed simply as follows:
41+
SourceXtractor++ is shipped directly in Fedora. It can be installed simply as follows:
4242

4343
```bash
4444
sudo dnf install sourcextractor++
4545
```
4646

47-
[We also build our own RPMs as part of our CI setup](http://repository.astro.unige.ch/euclid/devel/)
47+
[We also build our own RPMs as part of our CI setup](http://repository.astro.unige.ch/euclid/)
4848
They can be installed adding a `.repo` file under `/etc/yum.repos.d` with the following content:
4949

5050
```ini
@@ -81,7 +81,8 @@ dnf install SourceXtractorPlusPlus
8181
## Test Data Set
8282

8383
A test data set covering various data reduction scenarios is available [here](https://cloud.physik.lmu.de/index.php/s/3K4KemBsw5y9yqd).
84-
I contains the input data, all relevant configuration files, the SourceXtractor++ commands and the expected output results for a reference.
84+
It contains the input data, all relevant configuration files, the SourceXtractor++ commands and the expected output results for a reference.
85+
The test data set is described as a [pre-print](http://arxiv.org/abs/2212.02428) for the corresponding conference proceedings.
8586

8687
## Discussion group for SourceXtractor++ users
8788

@@ -96,7 +97,7 @@ join the group (if a google account is at hand) or to ask us to add you as a mem
9697
While we are working on several refereed papers describing SourceXtractor++ and its software architecture, the software package can be referenced with two ADASS conference proceedings:
9798

9899
- [Bertin, E.; Schefer, M. ; Apostolakos, N. ; Álvarez-Ayllón, A.; Dubath, P. ; Kümmel, M.: 2020ASPC..527..461B](https://ui.adsabs.harvard.edu/abs/2020ASPC..527..461B/abstract)
99-
- [Kümmel, M. ; Bertin, E.; Schefer, M. ; Apostolakos, N. ; Álvarez-Ayllón, A.; Dubath, P.: 2020ASPC..527...29K](https://ui.adsabs.harvard.edu/abs/2020ASPC..527...29K/abstract)
100+
- [Kümmel, M.; Álvarez-Ayllón, A.; Bertin, E.; Dubath, P.; Gavazzi, R.; Hartley, W.; Schefer, M.: 2022arXiv221202428K](https://ui.adsabs.harvard.edu/abs/2022arXiv221202428K/abstract)
100101

101102
Resources for the SourceXtractor++ development are in most part provided by funding from the [Euclid satellite project](https://www.euclid-ec.org/) via the various national agencies.
102103
For Euclid Consortium papers it is sufficient to point to the [SourceXtractor++ webpage](https://github.com/astrorama/SourceXtractorPlusPlus) and add

SEBenchmarks/src/program/BenchBackgroundModel.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ class BenchBackgroundModel : public Elements::Program {
8080
if (i != args.end())
8181
return i->second.as<std::string>();
8282
auto input = boost::filesystem::path(m_detection_config.getDetectionImagePath());
83-
auto basename = input.leaf();
83+
auto basename = input.filename();
8484
while (!basename.extension().empty())
8585
basename = basename.stem();
8686
auto algo = args.at("algorithm").as<std::string>();

SEFramework/SEFramework/Aperture/TransformedAperture.h

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@
2525
#define _SEFRAMEWORK_SEFRAMEWORK_APERTURE_TRANSFORMEDAPERTURE_H
2626

2727
#include "Aperture.h"
28+
#include <array>
2829

2930
namespace SourceXtractor {
3031

SEFramework/SEFramework/Image/Image.h

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@
2323
#ifndef _SEFRAMEWORK_IMAGE_IMAGE_H
2424
#define _SEFRAMEWORK_IMAGE_IMAGE_H
2525

26+
#include <cstdint>
2627
#include <memory>
2728

2829
#include "SEUtils/Types.h"

SEFramework/SEFramework/Image/ImageChunk.h

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,9 +24,11 @@
2424
#ifndef _SEFRAMEWORK_IMAGE_IMAGECHUNK_H_
2525
#define _SEFRAMEWORK_IMAGE_IMAGECHUNK_H_
2626

27+
#include "SEFramework/Image/Image.h"
28+
2729
#include <cassert>
30+
#include <vector>
2831

29-
#include "SEFramework/Image/Image.h"
3032

3133
namespace SourceXtractor {
3234

0 commit comments

Comments
 (0)