Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add specification for LB data file for phases skipped or identical #2130

Closed
lifflander opened this issue Apr 10, 2023 · 1 comment · Fixed by #2137
Closed

Add specification for LB data file for phases skipped or identical #2130

lifflander opened this issue Apr 10, 2023 · 1 comment · Fixed by #2137
Assignees

Comments

@lifflander
Copy link
Collaborator

lifflander commented Apr 10, 2023

What Needs to be Done?

We want to specify for every phase in the data file whether data is skipped or identical. This is related to issue #1935 which will allow phase output to be skipped for some outputs. The identical to previous is for data input where the mapping is the same as a previous phase and thus does not need to be output again.

{
	"metadata": {
		"rank": 0,
		"shared_node": {
			"id": 0,
			"num_nodes": 1,
			"rank": 0,
			"size": 4,
			"available_memory": 32768
		},
		"type": "LBDatafile"
                "phases": {
                  "count": 100,
                  "skipped": {
                    "list": [10, 13, 15],
                    "range": [[20, 25], [30, 35]]
                  }
                  "identical_to_previous": {
                    "list": [50, 53, 58],
                    "range": [[60, 70]]
                  }
                }
	},
	"phases": [

	]
}

*Edited to count

@lifflander
Copy link
Collaborator Author

@thearusable Let's actually rename the "length" field to "count" as suggested by @nlslatt

nlslatt added a commit that referenced this issue May 16, 2023
…ses-2

#2130: Add spec for skipped and identical phases
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants