• If this is your first visit, you will have to register before you can post. To view messages, please scroll below and select the forum that you would like to visits. Questions? Be sure to check out the Forum FAQ.

Announcement

Collapse
No announcement yet.

Partner 728x90

Collapse

NinjaScript generated code not updated if comments are added

Collapse
X
  • Filter
  • Time
  • Show
Clear All
new posts

    NinjaScript generated code not updated if comments are added

    hi

    I noticed that the NinjaScript generated code is not updated when you add a new parameter to your indicator and put a comment after the definition and before the get/set line.
    The below example will not update the NinjaScript generated code upon saving. The indicator will still run perfectly fine until you reference it from a strategy or another indicator.
    Then you have a mismatch in the # of parameters and it took me quite some time to figure out what was going on.
    You may want to address this in code or mention in the documentation.

    [NinjaScriptProperty]
    [Display(Name = "Period", Order = 10, GroupName = "Parameters")]
    public int Period // some comment put here
    { get; set; }



    #2
    Hello DukuGrinder,

    Thank you for reporting this.

    I was able to reproduce.

    I will submit a feature request to allow for comments after NinjaScriptProperty public object declarations.

    I will also request this be mentioned in the help guide.
    Chelsea B.NinjaTrader Customer Service

    Comment


      #3
      it is not a feature request, it is a bug..

      Comment


        #4
        Hello DukuGrinder,

        NinjaTrader has a NinjaScript wrapper that parses the indicator file before compiling to create the NinjaScript generated code. This wrapper comes with limitations like we see here. We will be making some documentation updates in the future to outline these limitations.

        If there is any other behavior that you are seeing with the NinjaScript Generated code that you would like to report so we can have it documented, we will be happy to review.
        JimNinjaTrader Customer Service

        Comment

        Latest Posts

        Collapse

        Topics Statistics Last Post
        Started by xsgex, Yesterday, 09:46 PM
        0 responses
        2 views
        0 likes
        Last Post xsgex
        by xsgex
         
        Started by Steve L, Yesterday, 09:36 PM
        0 responses
        1 view
        0 likes
        Last Post Steve L
        by Steve L
         
        Started by krugman25, Yesterday, 09:34 PM
        0 responses
        4 views
        0 likes
        Last Post krugman25  
        Started by indy73, Yesterday, 04:42 PM
        0 responses
        15 views
        0 likes
        Last Post indy73
        by indy73
         
        Started by Italianforex, Yesterday, 06:20 AM
        2 responses
        13 views
        0 likes
        Last Post gplatis
        by gplatis
         
        Working...
        X