foxglove/mcap

rust: default next channel id is inconsistent with other languages

Closed this issue · 3 comments

Description

let next_channel_id = self.channels.len() as u16;

void McapWriter::addChannel(Channel& channel) {
channel.id = uint16_t(channels_.size() + 1);
channels_.push_back(channel);
}

channel_id = len(self.__channels) + 1

Hi there,
In the future, please follow the issue template when filing issues.

Could you describe why this is causing a problem for you? The MCAP spec does not dictate what the default channel id should be.

Closing for no response. If there's more detail you can provide we are happy to revisit when you do.