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

crab fixed #29

Closed
wants to merge 17 commits into from
Closed

crab fixed #29

wants to merge 17 commits into from

Conversation

YujiLee301
Copy link
Collaborator

Hi Ram,

I have taken efforts and made everything works in crabjobs based on our master branch.

Please change your storage location of T2 site in crab_cfg.py and run command:
crab submit --config=crab_cfg.py

Looking for your review, thanks in advance.

Best,
Yuji

Copy link
Owner

@ram1123 ram1123 left a comment

Choose a reason for hiding this comment

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

Delete directory crab_projects_sendPythonFolderFalse. This is created when we submit the job.

Copy link
Owner

Choose a reason for hiding this comment

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

Delete this file.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I tested without this file, and the submission can not be successful.

Copy link
Owner

@ram1123 ram1123 Jan 19, 2024

Choose a reason for hiding this comment

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

hmmm, strange. This job report is created when we run it locally, by switching the job report boolean.

This report helps us to set the crab job parameters like the number of jobs and how many events are in each job.

How does the crab job depend on this file?

crab_script.py Outdated Show resolved Hide resolved
@ram1123 ram1123 linked an issue Jan 14, 2024 that may be closed by this pull request
@ram1123 ram1123 changed the base branch from master to HZZ_Analysis January 19, 2024 14:01
@ram1123
Copy link
Owner

ram1123 commented Jan 19, 2024

Hi @YujiLee301 ,

It looks like you do not stat from the "HZZ_Analysis" branch for the crab job update.

Remember that our master branch is "HZZ_Analysis". So, for any update start from the "HZZ_Analysis" branch as a base branch then rename it (say crabjobs).

As you didn't start from the "HZZ_Analysis" branch there is a merge conflict as I updated the base branch of the job. Please check and fix it.

@ram1123
Copy link
Owner

ram1123 commented Jan 23, 2024

This update moved to PR #31

@ram1123 ram1123 closed this Jan 23, 2024
@ram1123 ram1123 deleted the crabjobs branch October 3, 2024 11:51
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.

No working crab config exists
2 participants