Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Minor Typos mainly

See original GitHub issue

Subject of the documentation

Minor typos as I read through some docs.

What are you trying to do and which part of the code is hard to understand or incorrect or missing? Please provide a link to the readthedocs page and code if available.

I’m just pretending Im a complete beginner.

If the documentation request is approved, would you be willing to submit a PR? (Help can be provided if you need assistance submitting a PR)



  • “Intensity based loss is modality-independent” sounds wrong or needs further clarification, as for example, SSD won’t work on MR-CT registration. So, choice of loss function is dependent on modality.
  • “measures the deformation to regularize the transformation.” sounds wierd. I’d say "measures the amount of deformation, such that too severe a deformation can be penalised’, as beginners may not understand that much about regularization.
  • If I was a newcomer, I’d probably be lazy, and not read learn2reg. Then, by the time I had finished the first Tutorial section, I am still wondering … “well, what sort of network is DeepReg implementing?”. I’m just reading straight through the docs, and haven’t seen a description of the main network.
  • For adding custom loss functions, it seems odd to modify core functionality in dissimilarity_fn. This is the sort of thing that should be solved with dependency injection, using a Strategy pattern, rather then editing an elif.
  • Looking at build_backbone, again, we are having to edit core DeepReg code. This could be improved by using an Factory pattern.
  • Both these suggestions of a StrategyPattern or Factory pattern are optional. Its more relevant for compiled code which needs to be extensible. Might be overkill for a Python/Research project.
  • The demo page for “Paired CT Lung registration”, doesn’t give any examples or pictures. So, its not really saying any more than the instructions on how to run the command line apps. While there is a link to source code, there is no explanation of it.
  • Same comment for several demos. Needs pictures.

Issue Analytics

  • State:closed
  • Created 3 years ago
  • Reactions:3
  • Comments:8 (8 by maintainers)

github_iconTop GitHub Comments

mathpluscodecommented, Nov 6, 2020

I’ll take a look. Is there anyway we can maintain this issue as a place for people to record typos etc? there are bound to be more mistakes in future…

it’s fine… I added the tag typo so we can just keep creating issues with this tag?

YipengHucommented, Nov 2, 2020
  • Good point on the network. We should at least add a pointer to the API ref where the different architectures, Local/Global U-Nets are described with references.
  • Strategy/factory patterns worth separate tickets? @mathpluscode what do you think?
  • The pictorial addition to the demos have also been mentioned by @ebonmati - do we have a ticket for this? Just have not got around to add them.

@YipengHu jumping on this at the mention of missing tickets, did we open all the tickets from prior DeepReg meeting? If not, shall I open tickets from our previous “deepreg dev ideas” ticket?

tbh, i lost track on this. I believe it is time for specific idea ticket now? feel free to open new ones, i will check again this week.

Read more comments on GitHub >

github_iconTop Results From Across the Web

Is it OK to change mistakes/typos when you are in the minor ...
Yes, this is completely OK. You are not bound to perpetuate typos from earlier versions just because reviewers did not find them.
Read more >
How do you feel about receiving edits for minor typos? - Quora
There is nothing called a "minor" typo, and typo is just a fancy name given to a harmful vice called "making a spelling...
Read more >
Typo (Typographical Error) | What is One Example of a Typo?
The word typo is a common term heard often in writing and research, especially in a classroom setting and in the working world....
Read more >
Common Grammar Mistakes: 11 Goofs You Might Know Too ...
Basically, you use an apostrophe in two cases: For contractions (don't for do not); To show possession (Frank's blog means the blog belongs...
Read more >
TSM Numot on Twitter: "Unfortunate 5-2 Day 2 of the Open ...
Unfortunate 5-2 Day 2 of the Open, still mainly due to my own errors. ... a few small changes but just couldn't stop...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Post

No results found

github_iconTop Related Hashnode Post

No results found