Skip to content

FIX: Create Maxwell project datasets with import_dataset3d#7538

Open
jvela018 wants to merge 15 commits intomainfrom
fix/3d_dataset_maxwell
Open

FIX: Create Maxwell project datasets with import_dataset3d#7538
jvela018 wants to merge 15 commits intomainfrom
fix/3d_dataset_maxwell

Conversation

@jvela018
Copy link
Copy Markdown
Contributor

Description

The method import_dataset3d sets the is_project_database to False. By doing this, Maxwell (nor Icepak) can't import datasets as project datasets

Issue linked

#7537

Checklist

  • I have tested my changes locally.
  • I have added necessary documentation or updated existing documentation.
  • I have followed the coding style guidelines of this project.
  • I have added appropriate tests (unit, integration, system).
  • I have reviewed my changes before submitting this pull request.
  • I have linked the issue or issues that are solved by the PR if any.
  • I have agreed with the Contributor License Agreement (CLA).

@jvela018 jvela018 self-assigned this Apr 13, 2026
@jvela018 jvela018 requested a review from a team as a code owner April 13, 2026 08:56
@jvela018 jvela018 linked an issue Apr 13, 2026 that may be closed by this pull request
2 tasks
@github-actions github-actions bot added the bug Something isn't working label Apr 13, 2026
@Samuelopez-ansys
Copy link
Copy Markdown
Member

@jvela018 Project datasets must always have $ symbol right?

@Samuelopez-ansys
Copy link
Copy Markdown
Member

Maybe what is wrong is allowing the user to specify it in the argument.

image

@jvela018
Copy link
Copy Markdown
Contributor Author

@Samuelopez-ansys, I added the case when user defines is_project_dataset, handling the character "$" at the beginning of the name.

Comment thread src/ansys/aedt/core/application/design.py Outdated
Comment thread src/ansys/aedt/core/application/design.py
@jvela018 jvela018 marked this pull request as draft April 14, 2026 14:13
@jvela018 jvela018 marked this pull request as ready for review April 15, 2026 07:15
@jvela018 jvela018 dismissed Samuelopez-ansys’s stale review April 15, 2026 07:47

Performed the changes requested

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

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

import_dataset3d defaults to non-project database in Maxwell and Icepak

3 participants