Difference between revisions of "Talk:RadiMation Application Note 100"

From RadiWiki
Jump to: navigation, search
 
(6 intermediate revisions by 3 users not shown)
Line 1: Line 1:
−
I do have some suggestions to further improve the application note:
+
MAHE: I have applied the requested changes
−
* The table of contents is shown twice
 
−
* Isn't it better to have a screenshot of the result in the introduction. It will make it clear how the result will be.
 
−
* The 'ScreenElement' macro can be used to indicate button names. I already modified it on a few locations. But not yet on all locations
 
−
* The 'Remove' button and the 'Custom Status window' checkbox are not explained.
 
−
* The section of 'Horizontal' or 'Vertical' doesn't show any text. Maybe it is better to provide a screenshot of the enhanced status window for each situation. Then the difference will be very clear...
 
−
* I Suggest to include a screenshot of an example after the 'List Of measurement parameters'.
 
−
* Then at the end of the document it is very helpful to show a screenshot during a test, where also the enhanced status window is shown
 
−
* In that last section it should also be explained that if the enhanced status window is closed, it can be shown again from the 'Window' menu.
 
  
−
[[User:Joro|Joro]] 10:01, 15 July 2015 (UTC)
 
  
 +
I do have a few remarks for the improvements of this application note:
 +
* The last screenshot: ([[:File:EnhancedWindowWithTest.png]]), shows an EUT window, a test and the Enhanced status window. The EUT Window is not relevant for the purpose of the screenshot, but it does cause that the screenshot is very wide. I think we should improve the screenshot to NOT show the EUT window.
 +
* The values on all the screenshots that show the enhanced status window are very strange. They should be updated to show more logical values (maybe configure it in the KETC??):
 +
** The last screenshot ([[:File:EnhancedWindowWithTest.png]]) shows: -200 dBm signal power (How is that possible??) while the field is expected to be almost 100 V/m. That is practically not possible. Also the frequency is just too low. At 280 kHz no 100 V/m can be generated.
 +
** The horizontal screenshot shows a frequency of 1 Hz.
 +
** The vertical screenshot shows a frequency of 1 Hz
  
−
* The table of contents is shown twice
 
−
'''-> removed the additional table of contents.'''
 
−
* Isn't it better to have a screenshot of the result in the introduction. It will make it clear how the result will be.
 
−
'''-> agreed inserted two screenshots of possible end results.'''
 
−
* The 'ScreenElement' macro can be used to indicate button names. I already modified it on a few locations. But not yet on all locations.
 
−
'''-> Good idea, will update the whole text and use it from now on.'''
 
−
* The 'Remove' button and the 'Custom Status window' checkbox are not explained.
 
−
'''-> Extended the explanation to include this.'''
 
−
* The section of 'Horizontal' or 'Vertical' doesn't show any text. Maybe it is better to provide a screenshot of the enhanced status window for each situation. Then the difference will be very clear...
 
−
'''-> Already had those screenshots ready to be implemented. Updated the text.'''
 
−
* I Suggest to include a screenshot of an example after the 'List Of measurement parameters'.
 
−
'''-> Already had this screenshots ready to be implemented. Updated the text.'''
 
−
* Then at the end of the document it is very helpful to show a screenshot during a test, where also the enhanced status window is shown
 
−
'''- > I agree and will create such screenshot and description.'''
 
−
* In that last section it should also be explained that if the enhanced status window is closed, it can be shown again from the 'Window' menu.
 
−
'''- > I agree, however I was not able to reopen this window after it was closed...'''
 
  
−
[[User:Sast|Sast]] 11:17, 14 August 2015 (UTC)
 
  
 +
== Review status ==
 +
Applied review changes.        [[User:Mahe|Mahe]] ([[User talk:Mahe|talk]]) 09:45, 12 Februari 2019 (CET)
  
−
Ok. These improvements are very good additions.
+
Did review. See remarks above. [[User:Joro|Joro]] ([[User talk:Joro|talk]]) 09:39, 30 January 2019 (CET)
−
I further improved some sentences, and implemented some markup changes.
 
  
−
All is ok for me, but the the last section is very strange.  
+
ROGE: Reviewed. [[User:Roge|Roge]] ([[User talk:Roge|talk]]) 15:45, 26 February 2019 (CET)
−
You explain everything us Frequency, signal power and calculated field. Then at the end suddenly the calculated field is replaced by the Average field.
 
−
I agree that it is described in the text, however it seems very unlogical.
 
−
I suggest to create a new screenshot that shows: Frequency, Signal power and Calculated field.
 
−
Maybe it is even better to show in this last section, a screenshot of the complete RadiMation window, including the test running, and the enhanced status window.
 
−
 
 
−
And I decided that we will not allow to reopen the Enhanced Status Window after it has been closed. This functionality has not been present anymore for a long time (if at all it ever has been working in the past).
 
−
But see the request: #5698: Possibility to re-open the Enhanced Status Window after closing
 
−
 
 
−
[[User:Joro|Joro]] 14:29, 27 August 2015 (UTC)
 

Latest revision as of 14:47, 26 February 2019

MAHE: I have applied the requested changes


I do have a few remarks for the improvements of this application note:

  • The last screenshot: (File:EnhancedWindowWithTest.png), shows an EUT window, a test and the Enhanced status window. The EUT Window is not relevant for the purpose of the screenshot, but it does cause that the screenshot is very wide. I think we should improve the screenshot to NOT show the EUT window.
  • The values on all the screenshots that show the enhanced status window are very strange. They should be updated to show more logical values (maybe configure it in the KETC??):
    • The last screenshot (File:EnhancedWindowWithTest.png) shows: -200 dBm signal power (How is that possible??) while the field is expected to be almost 100 V/m. That is practically not possible. Also the frequency is just too low. At 280 kHz no 100 V/m can be generated.
    • The horizontal screenshot shows a frequency of 1 Hz.
    • The vertical screenshot shows a frequency of 1 Hz


Review status

Applied review changes. Mahe (talk) 09:45, 12 Februari 2019 (CET)

Did review. See remarks above. Joro (talk) 09:39, 30 January 2019 (CET)

ROGE: Reviewed. Roge (talk) 15:45, 26 February 2019 (CET)

Retrieved from "https://wiki.radimation.com/wiki/index.php?title=Talk:RadiMation_Application_Note_100&oldid=18635"