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.

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

RNGDet implementation #11156

Open
wants to merge 58 commits into
base: master
Choose a base branch
from
Open

RNGDet implementation #11156

wants to merge 58 commits into from

Conversation

mjyun01
Copy link
Contributor

@mjyun01 mjyun01 commented Feb 7, 2024

Description

Code implementation for RNGDet: Road Network Graph Detection by Transformer in Aerial Images
Check rngdet/requirements.txt for Check the requirements.txt for the required configuration settings.

Type of change

  • A new research paper code implementation

Tests

To train and evaluate new RNGDet network with cityscale dataset, follow descriptions in Readme.md.
Due to the randomness existing in both sampling and training, the final performance of the proposed models might be slightly different from the number reported in the original paper. Also, different threshold can make different results.

Test Configuration:

backbone = resnet50, num_quires = 10
extract_candidate_threshold = 0.65, candidate_filter_threshold = 25, alignment_distance = 10, logit_threshold = 0.75
More than 10 epoch training with default threshold settings makes reasonable result images.

Checklist

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@laxmareddyp laxmareddyp requested review from Zarjagen and removed request for saberkun and rachellj218 February 7, 2024 17:57
@laxmareddyp laxmareddyp added the models:official models that come under official repository label Feb 7, 2024
@Zarjagen Zarjagen self-assigned this Feb 8, 2024
@Zarjagen Zarjagen added the ready to pull ready to pull (create internal pr review and merge automatically) label Feb 8, 2024
Zarjagen
Zarjagen previously approved these changes Feb 8, 2024
@Zarjagen Zarjagen removed the ready to pull ready to pull (create internal pr review and merge automatically) label Feb 8, 2024
@sineeli sineeli requested a review from Zarjagen February 8, 2024 17:00
Zarjagen
Zarjagen previously approved these changes Feb 10, 2024
@laxmareddyp laxmareddyp added the ready to pull ready to pull (create internal pr review and merge automatically) label Feb 12, 2024
@laxmareddyp laxmareddyp removed the ready to pull ready to pull (create internal pr review and merge automatically) label Feb 12, 2024
@laxmareddyp laxmareddyp added the ready to pull ready to pull (create internal pr review and merge automatically) label Feb 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
models:official models that come under official repository ready to pull ready to pull (create internal pr review and merge automatically)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants