Announcement

Collapse
No announcement yet.

Partner 728x90

Collapse

Vendor Licensing export to csv generates extra/empty record

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

    Vendor Licensing export to csv generates extra/empty record

    From Vendor Licensing, when you export to csv, the resulting file generates an extra/empty record because of a trailing comma at the end of each line.

    It generates:
    Name,Machine ID,Product,Type,Start,End,

    The expected data should not have a comma at the end like:
    Name,Machine ID,Product,Type,Start,End

    Each line also has a trailing comma.

    This seems very inconsistent with any csv I have worked with, and a simple import of the file into python for example generates the following (see attachment). The 6th field should not exist.
    You can see this from a simple csv reader like:

    import csv
    with open(r"D:\temp\NinjaTrader Grid 2022-03-05 02-01 PM.csv") as fp:
    reader = csv.reader(fp, delimiter=",", quotechar='"')
    data_read = [row for row in reader]
    print(data_read)

    Click image for larger version

Name:	Screenshot 2022-03-05 201128.png
Views:	189
Size:	36.4 KB
ID:	1192557

    #2
    Yep.

    And, in other news, the world keeps spinning, despite
    too many commas in that crap csv file.

    Flex your programming skills.
    Write a filter program to strip that trailing comma.
    Run filter on crap csv file to create a good csv file.

    Problem solved, right?
    But, yes, I agree with you.
    Crap csv file is arguably wrong.

    Faster to just accept crap csv file.
    Create workaround to convert to good csv file.
    Gotta pick your battles wisely.

    Just my 2˘.

    Comment


      #3
      Sorry, sounded harsh, was going for humor.

      But, I mean, you're a programmer.
      Write a filter, this is an easy, trivial program.

      Also, consider that, if NT 'fixed' that csv output,
      they might receive an equal number of bug reports
      complaining about that missing comma ... because,
      ya know, it's been there all this time, so plenty of
      people might have processes where that crap
      trailing comma is already accounted for.

      My point is:
      Given your skillset, this is a case where it is much
      easier if you adjust to the world rather than ask
      world to adjust to you (even if the world is wrong).

      Comment


        #4
        Well bltdavid there are many options that Ninja could do to fix the issue and not break backwards compatibility. They could have a setting (in the ui, registry, config file, etc) that choses current or new behavior such as a number like "version format" or add a 3rd option to the export dialog for current csv format, new csv format, and excel.

        Comment


          #5
          Hello korygill,

          Thanks for your post.

          This is currently the expected behavior when exporting information from the Vendor Licensing window. I have created an internal feature request ticket to track interest in changing this feature and added your vote

          The internal tracking number for your feature request is SFT-5472. Please reference this internal tracking number when contacting Platform Support if you ever have questions regarding this feature request.

          When a feature request is implemented, you'll find a description of the new feature in the release notes:Let us know if we may assist further.
          Brandon H.NinjaTrader Customer Service

          Comment

          Latest Posts

          Collapse

          Topics Statistics Last Post
          Started by tkaboris, Today, 05:13 PM
          0 responses
          2 views
          0 likes
          Last Post tkaboris  
          Started by GussJ, 03-04-2020, 03:11 PM
          16 responses
          3,281 views
          0 likes
          Last Post Leafcutter  
          Started by WHICKED, Today, 12:45 PM
          2 responses
          19 views
          0 likes
          Last Post WHICKED
          by WHICKED
           
          Started by Tim-c, Today, 02:10 PM
          1 response
          10 views
          0 likes
          Last Post NinjaTrader_ChelseaB  
          Started by Taddypole, Today, 02:47 PM
          0 responses
          5 views
          0 likes
          Last Post Taddypole  
          Working...
          X