Bug List

Member EditCmd::EditCmd (const string typestr, const string arg)

This is kind of ugly. If someone tries to specify 3.02 codes, they'll be interpreted incorrectly.

This is kind of ugly. If someone tries to specify 3.02 codes, they'll be interpreted incorrectly.

Member gnsstk::GLOFNavAlm::dump (std::ostream &s, DumpDetail dl) const override

this shouldn't be here, but I'm putting it here temporarily to make it easier to compare with legacy code output.

Member gnsstk::PNBBDSD2NavDataFactory::processEph (unsigned long pgid, const PackedNavBitsPtr &navIn, NavDataPtrList &navOut)

I can't find in the ICD where it says what the scale factor is for the SOW field. It's stored in 20 bits even though 604800 only requires 19 bits.

Member gnsstk::PNBGLOFNavDataFactory::processEph (unsigned long stringID, const PackedNavBitsPtr &navIn, NavDataPtrList &navOut)

This doesn't seem right, adding a day to seconds of day then subtracting? Shouldn't it just be subtracting 3 hours (10800 seconds from the CommonTime object once constructed?

Member gnsstk::PNBGLOFNavDataFactory::processTime (const PackedNavBitsPtr &navIn, NavDataPtrList &navOut)

Table 4.9 in the ICD suggests there are two sets of scales and number of bits, one for legacy GLONASS and one for GLONASS-M, but it doesn't say if tau_c starts at the same bit position in legacy. Another implementation always uses the -M bit pattern. It's not clear what's correct.

This is a bit sketchy because it relies on the hard-coded integer leap seconds in gnsstk::getLeapSeconds(), but I can't think of a more reliable way to get this data here at the moment. Luckily changes are few and far between.

Member gnsstk::Rinex3ObsHeader::obsTypeList

this is being used but is not actually being filled

Member gnsstk::RinexNavDataFactory::convertToOffset (const Rinex3NavHeader &navIn, NavDataPtrList &navOut)

the time stamp is likely to cause problems for GLONASS which doesn't have a reference time.

Member gnsstk::RinexObsID::RinexObsID (const std::string &strID, double version)

Not convinced this is an appropriate value, but it's what has been used in the past.

Member gnsstk::SP3Header::reallyGetRecord (FFStream &s)

i is uninitialized at this point and it's not clear what was intended in this exception message, so commenting out the use of i for now.

Member gnsstk::StringUtils::doub2for (const double &d, const std::string::size_type length, const std::string::size_type expLen, const bool checkSwitch=true)

The rendered width is not correct if checkSwitch=false

Member gnsstk::YumaNavDataFactory::process (const std::string &filename, NavDataFactoryCallback &cb) override

The use of gotdata is a kludge. It appears that the YumaStream implementation will read invalid data to the end of the file, and we need it to not do that.

Member RinexObsID_T::decodeTest ()

The constructor rather unintelligently returns the same band regardless of the GNSS being decoded. Fixing this will require rewriting the ObsID constructor which I'm putting off for now.

Page

RinEdit does not remove duplicate data from the output.