unused overloaded doGatherShapeN() function.

[JustinRayAngus]

The following function does not appear to be used anywhere in the code. Should it be removed?

[Relevant code in FieldGather.H](

warpx/Source/Particles/Gather/FieldGather.H

Lines 1504 to 1571 in 343dfd6

	/**
	* \brief Field gather for particles
	*
	* \tparam depos_order deposition order
	* \tparam lower_in_v lower shape order in parallel direction (Galerkin)
	* \param getPosition A functor for returning the particle position.
	* \param getExternalEB A functor for assigning the external E and B fields.
	* \param Exp,Eyp,Ezp Pointer to array of electric field on particles.
	* \param Bxp,Byp,Bzp Pointer to array of magnetic field on particles.
	* \param exfab,eyfab,ezfab Array4 of the electric field, either full array or tile.
	* \param bxfab,byfab,bzfab Array4 of the magnetic field, either full array or tile.
	* \param np_to_gather Number of particles for which field is gathered.
	* \param dinv 3D cell size inverse
	* \param xyzmin The lower bounds of the domain
	* \param lo Index lower bounds of domain.
	* \param n_rz_azimuthal_modes Number of azimuthal modes when using RZ geometry
	*/
	template <int depos_order, int lower_in_v>
	void doGatherShapeN(const GetParticlePosition<PIdx>& getPosition,
	const GetExternalEBField& getExternalEB,
	amrex::ParticleReal * const Exp, amrex::ParticleReal * const Eyp,
	amrex::ParticleReal * const Ezp, amrex::ParticleReal * const Bxp,
	amrex::ParticleReal * const Byp, amrex::ParticleReal * const Bzp,
	amrex::FArrayBox const * const exfab,
	amrex::FArrayBox const * const eyfab,
	amrex::FArrayBox const * const ezfab,
	amrex::FArrayBox const * const bxfab,
	amrex::FArrayBox const * const byfab,
	amrex::FArrayBox const * const bzfab,
	const long np_to_gather,
	const amrex::XDim3 & dinv,
	const amrex::XDim3 & xyzmin,
	const amrex::Dim3 lo,
	const int n_rz_azimuthal_modes)
	{
	amrex::Array4<const amrex::Real> const& ex_arr = exfab->array();
	amrex::Array4<const amrex::Real> const& ey_arr = eyfab->array();
	amrex::Array4<const amrex::Real> const& ez_arr = ezfab->array();
	amrex::Array4<const amrex::Real> const& bx_arr = bxfab->array();
	amrex::Array4<const amrex::Real> const& by_arr = byfab->array();
	amrex::Array4<const amrex::Real> const& bz_arr = bzfab->array();
	amrex::IndexType const ex_type = exfab->box().ixType();
	amrex::IndexType const ey_type = eyfab->box().ixType();
	amrex::IndexType const ez_type = ezfab->box().ixType();
	amrex::IndexType const bx_type = bxfab->box().ixType();
	amrex::IndexType const by_type = byfab->box().ixType();
	amrex::IndexType const bz_type = bzfab->box().ixType();
	// Loop over particles and gather fields from
	// {e,b}{x,y,z}_arr to {E,B}{xyz}p.
	amrex::ParallelFor(
	np_to_gather,
	[=] AMREX_GPU_DEVICE (long ip) {
	amrex::ParticleReal xp, yp, zp;
	getPosition(ip, xp, yp, zp);
	getExternalEB(ip, Exp[ip], Eyp[ip], Ezp[ip], Bxp[ip], Byp[ip], Bzp[ip]);
	doGatherShapeN<depos_order, lower_in_v>(
	xp, yp, zp, Exp[ip], Eyp[ip], Ezp[ip], Bxp[ip], Byp[ip], Bzp[ip],
	ex_arr, ey_arr, ez_arr, bx_arr, by_arr, bz_arr,
	ex_type, ey_type, ez_type, bx_type, by_type, bz_type,
	dinv, xyzmin, lo, n_rz_azimuthal_modes);
	}
	);
	}

[RemiLehe]

Thanks for reporting this!
Yes, feel free to remove that function.