Skip to content

Commit

Permalink
More minor code fixes to end compiler warnings; fixed location of tes…
Browse files Browse the repository at this point in the history
…t-reference file in do_makemultimages_tests.sh
  • Loading branch information
perwin committed Jul 24, 2024
1 parent eb0b371 commit a41a9d7
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 11 deletions.
2 changes: 1 addition & 1 deletion do_makemultimages_tests.sh
Original file line number Diff line number Diff line change
Expand Up @@ -241,7 +241,7 @@ echo -n "[WARNING: NO REFERENCE IMAGE YET!]"
if [ $do_fits_tests = "1" ]
then
echo ""
./python/compare_fits_files.py --min-value=1.0e-7 temptest/multimage_osamp_1.fits tests/${osname}/oversampled_orig.fits
./python/compare_fits_files.py --min-value=1.0e-7 temptest/multimage_osamp_1.fits tests/oversampled_orig.fits
STATUS+=$?
fi

Expand Down
14 changes: 6 additions & 8 deletions function_objects/func_gaussianring3d.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
* [v0.1]: 24 Aug 2012: Created (as modification of func_exp3d.cpp).
*/

// Copyright 2012--2022 by Peter Erwin.
// Copyright 2012--4 by Peter Erwin.
//
// This file is part of Imfit.
//
Expand Down Expand Up @@ -172,7 +172,7 @@ double GaussianRing3D::GetValue( double x, double y )
double y_diff = y - y0;
double xp, yp, x_d0, y_d0, z_d0, totalIntensity;
double integLimit;
double xyParameters[13];
double xyParameters[12];
// int nSubsamples;

// Calculate x,y in component's (projected sky) reference frame: xp,yp
Expand All @@ -197,9 +197,8 @@ double GaussianRing3D::GetValue( double x, double y )
xyParameters[7] = q;
xyParameters[8] = J_0;
xyParameters[9] = a_ring;
xyParameters[10] = sigma;
xyParameters[11] = h_z;
xyParameters[12] = twosigma_squared;
xyParameters[10] = h_z;
xyParameters[11] = twosigma_squared;
F.params = xyParameters;

// integrate out to +/- integLimit, which is multiple of ring radius
Expand Down Expand Up @@ -235,9 +234,8 @@ double LuminosityDensityRing( double s, void *params )
double q = paramsVect[7];
double J_0 = paramsVect[8];
double a_ring = paramsVect[9];
double sigma = paramsVect[10];
double h_z = paramsVect[11];
double twosigma_squared = paramsVect[12];
double h_z = paramsVect[10];
double twosigma_squared = paramsVect[11];

// Determine 3D Cartesian coordinates in bar's native frame of reference
std::tie(x_ring, y_ring, z_ring) = Compute3dObjectCoords(s, x_d0, y_d0, z_d0, sinInc,
Expand Down
4 changes: 2 additions & 2 deletions function_objects/integrator.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -76,10 +76,10 @@ double Integrate_cquad( gsl_function F, double s1, double s2 )
double Integrate_Alt( gsl_function F, double s1, double s2 )
{
double result, error;
size_t n_eval;
// size_t n_eval;
int status;
gsl_integration_workspace * workspace;
gsl_integration_cquad_workspace * workspace_cquad;
// gsl_integration_cquad_workspace * workspace_cquad;
// gsl_integration_romberg_workspace * workspace_romberg;

// allocate and free the workspace object here (referencing it with a local
Expand Down

0 comments on commit a41a9d7

Please sign in to comment.