Skip to content

Conversation

@garcs2
Copy link

@garcs2 garcs2 commented Oct 8, 2024

…. windows compatible


Merge Request Description

What issue does this change request address?
This change request resolves pathing issues encountered with Windows users to sonvalidxml and accert.sch.

What are the changes due to this change request?
This changes line 116 and 117 in Main.py to utilizes os library for pathing, when previously pathing was hardcoded as a string. This resolves pathing issue that was encountered with Windows machine.


For Change Request Review

  • 1. Is the merge request reference an issue? If the issue is closed, the issue close checklist shall be done.
    This does not reference an issue and was found while attempting to run example cases.
  • 2. Is the merge request reference a task? If the task is fulfilled, the task close checklist shall be done.
  • 3. If any changes to the database, there must be an accompanying change to the all the linked tables.

Copy link
Collaborator

@JiaZhou-PU JiaZhou-PU left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants