Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

First cut at readvertise_nlsr #75

Merged
merged 9 commits into from
Dec 19, 2024
Merged

First cut at readvertise_nlsr #75

merged 9 commits into from
Dec 19, 2024

Conversation

pulsejet
Copy link
Collaborator

#69

For now it's a oneshot attempt to advertise only.
Works with ndn-dv, not tested with NLSR.

@pulsejet pulsejet requested a review from zjkmxy December 18, 2024 09:32
@zjkmxy
Copy link
Member

zjkmxy commented Dec 18, 2024

FYI: table/measurement is there because originally we decided to let the forwarding strategy rely on standardized measurement table instead of internal metrics. But it is never used.

@pulsejet pulsejet merged commit 6ac41fb into master Dec 19, 2024
12 checks passed
@pulsejet pulsejet deleted the ranlsr branch December 19, 2024 04:51
pulsejet added a commit that referenced this pull request Dec 24, 2024
@pulsejet
Merge pull request #75 from named-data/ranlsr
cd9c03e 
First cut at `readvertise_nlsr`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zjkmxy zjkmxy zjkmxy approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@pulsejet @zjkmxy