Skip to content
GitLab
    • Explore Projects Groups Snippets
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • M metaseq
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 95
    • Issues 95
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 41
    • Merge requests 41
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Administrator
  • metaseq
  • Merge requests
  • !527

[tboard] Don't run out of letters

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Administrator requested to merge tboverflow into main 2 years ago
  • Overview 2
  • Commits 1
  • Pipelines 0
  • Changes 1

Created by: stephenroller

Patch Description Our restart logic is too good these days. Make sure tboard doesn't ever run out of suffixes.

Testing steps quick job test created the correct tboard folder.

Compare
  • main (base)

and
  • latest version
    ec86c106
    1 commit, 2 years ago

1 file
+ 1
- 1

    Preferences

    File browser
    Compare changes
metaseq/logging/progress_bar/tensorboard_progress_bar.py
+ 1
- 1
  • View file @ ec86c106

  • Edit in single-file editor

  • Open in Web IDE


@@ -53,7 +53,7 @@ class TensorboardProgressBarWrapper(BaseProgressBar):
if key not in _writers:
# tensorboard doesn't play well when we clobber it with reruns
# find an acceptable suffix
for suffix in [""] + list(string.ascii_uppercase):
for suffix in (f"{i:04d}" for i in range(10000)):
logdir = os.path.join(self.tensorboard_logdir + suffix, key)
if not os.path.exists(logdir):
logger.info(f"Setting tensorboard directory to {logdir}")
0 Assignees
None
Assign to
0 Reviewers
None
Request review from
Labels
1
cla signed
1
cla signed
    Assign labels
  • Manage project labels

Milestone
No milestone
None
None
Time tracking
No estimate or time spent
Lock merge request
Unlocked
1
1 participant
Administrator
Reference: root/metaseq!527
Source branch: tboverflow

Menu

Explore Projects Groups Snippets