NYU-DLSP20 icon indicating copy to clipboard operation
NYU-DLSP20 copied to clipboard

Figure Number Issue in 14-1.md

Open marofmar opened this issue 5 years ago • 10 comments

Hi, there is inconsistency in the name of the figure and how it is indicated in the following explanation.

For example, in figure 5 case, it is actively used to describe the content but it is called figure 19 in the explanation. As the figure itself contains the number, it makes sense, but this hinders a smooth understanding of the lecture note.

I think there are two solutions to this:

  1. Remove figure name part in the figure picture itself or
  2. Change figure name in the explanation

<center>
<img src="{{site.baseurl}}/images/week14/14-1/Fig5.png" style="background-color:#DCDCDC;" /><br>
Figure 5.
</center>

A simple example of a factor graph is shown in Figure 19 (top). The energy function is the sum of four factors:






<center>
<img src="{{site.baseurl}}/images/week14/14-1/Fig6.png" style="background-color:#DCDCDC;" /><br>
Figure 6.
</center>

The factor graph shown in Figure 20 is a log domain factor graph for linear structured models ("simple energy-based factor graphs" we are talking about)

marofmar avatar Aug 20 '20 01:08 marofmar

I would also add that figures might need captioning as in all of the other documents. Would you be so kind as to add captioning to the EN version of 14-1?

marcozullich avatar Oct 08 '20 13:10 marcozullich

@marofmar, thanks for pointing this out. Can you convert the bitmap caption in textual caption? In this way we can translate it to other languages as well. Moreover, could you add captions to all the other figures as well?

Thanks!

Atcold avatar Oct 16 '20 18:10 Atcold

Btw, since I hadn't got an answer to my first reply, I've already started putting captions while I'm translating the Italian version, in case @marofmar won't answer.

marcozullich avatar Oct 16 '20 20:10 marcozullich

Hi @marcozullich, sorry that I checked your reply so late, @marcozullich @Atcold noted!! I will add captions to all the other figures in [EN]14-1.md

marofmar avatar Oct 16 '20 22:10 marofmar

@Atcold bitmap caption which is included in the bitmap image is already used in the textual caption to explain the content. I modified a bit for consistency, but it results in messy ordering in the numbering system in figures of 14-1.md.

I think modifying the bitmap image itself, cropping it out the bitmap caption in the image, would be a better option. Anyhow please check that out on the github.io webpage and the PR of mine.

marofmar avatar Oct 26 '20 10:10 marofmar

Yes, we should crop the images, put the caption in written form, and update the references to the figure number to reflect the correct numbering.

Atcold avatar Nov 12 '20 02:11 Atcold

As the PR of mine on Oct 17 has not been merged yet, and won't be, I think cropping the images would be the only work that left! @marcozullich @Atcold

marofmar avatar Nov 27 '20 07:11 marofmar

oh I just checked the note again, and will make a quick fix and PR again, thanks @marcozullich !

marofmar avatar Nov 27 '20 08:11 marofmar

Thanks! Getting to it right now!

Atcold avatar Dec 11 '20 03:12 Atcold

I see now that the fig. numbers should be fine, is it @marofmar ? Now I guess we'd have to put some captions on the images

marcozullich avatar Dec 21 '20 09:12 marcozullich