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

cleanup the xsl based on group stuff done in #900 #919

Merged
merged 1 commit into from
Feb 9, 2024

Conversation

troy2914
Copy link
Contributor

fixes #917

can be tested like this
java -cp ../pc2v9/dist/pc2.jar edu.csus.ecs.pc2.core.util.XSLTransformer index.xsl results_D1.xml index_D1.html
java -cp ../pc2v9/dist/pc2.jar edu.csus.ecs.pc2.core.util.XSLTransformer group.xsl results_Hawii - D1.xml group_Hawaii - D1.html

with the results file from #909 (possibly modified to remove the site (non-D1/D2 groups).

@troy2914 troy2914 added this to the 9.10.0 milestone Jan 28, 2024
@troy2914 troy2914 added One Approval A very low risk and low LOE PR/Change NEXT Contest Consider fixing for next contet labels Jan 28, 2024
@troy2914 troy2914 force-pushed the i917_samps_post_multigroups branch from 65e1875 to 842ff88 Compare January 29, 2024 15:31
Copy link
Collaborator

@johnbrvc johnbrvc left a comment

Choose a reason for hiding this comment

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

Since these files are in the samps folder, they have no direct impact for the average user. These are specific to those folks looking to do group specific rankings and will require changes by anyone who wishes to do something similar.

@johnbrvc johnbrvc merged commit ae6e0f0 into pc2ccs:develop Feb 9, 2024
3 checks passed
@troy2914 troy2914 deleted the i917_samps_post_multigroups branch March 3, 2024 20:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NEXT Contest Consider fixing for next contet One Approval A very low risk and low LOE PR/Change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Standings index XSL files need to be updated for multiple groups
2 participants