{2023.06,zen4}[foss/2023a] R-bundle-Bioconductor 3.18 #613
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
boegel
added
2023.06-software.eessi.io
|
Instance
|
|
Instance
|
|
bot: build repo:eessi.io-2023.06-software arch:x86_64/amd/zen4 |
Updates by the bot instance
|
Updates by the bot instance
|
|
New job on instance
|
bedroge
changed the title
|
failed due to corrupt download, probably another job (like #614) was downloading that exact same file at the same time... Tarball was indeed corrupt, trying to manually unpack it resulted in |
|
bot: build repo:eessi.io-2023.06-software arch:x86_64/amd/zen4 |
Updates by the bot instance
|
Updates by the bot instance
|
|
New job on instance
|
|
Failed again due to another checksum failure: I've removed the corrupt I also don't understand how job Any ideas here @trz42 ? |
|
bot: build repo:eessi.io-2023.06-software arch:x86_64/amd/zen4 |
Updates by the bot instance
|
|
New job on instance
|
|
@bedroge You have some experience here? |
Someone else reported the same issue, but it looks like there's no solution (other than trying a newer version): |
|
@bedroge I guess we could try and pinpoint where the fix was made in between Highway 1.0.4 and 1.0.7, so we can come up with a patch, but I took a quick stab at that, and doesn't seem easy... |
I tried something similar for the issue I ran into, but also gave up at some point. In some versions quite a lot of code was changed, and backporting stuff was really not feasible... |
|
@bedroge So how do you think we should proceed? Ignore the test failure? Disable that particular test? |
|
@boegel Perhaps we can try changing the optimization flags? |
|
Analysed this a bit... The failing test @@ -432,15 +435,61 @@ struct TestTruncateTo {
};
HWY_NOINLINE void TestAllTruncate() {
- ForUnsignedTypes(ForPartialVectors<TestTruncateTo>());
+ ForU163264(ForDemoteVectors<TestTruncateTo>());
}The change from
template <class Func>
void ForUnsignedTypes(const Func& func) {
func(uint8_t());
func(uint16_t());
func(uint32_t());
#if HWY_HAVE_INTEGER64
func(uint64_t());
#endif
}
template <class Func>
void ForU163264(const Func& func) {
func(uint16_t());
func(uint32_t());
#if HWY_HAVE_INTEGER64
func(uint64_t());
#endif
}
|
|
@trz42 Isn't the change from |
|
easybuilders/easybuild-easyconfigs#20942 should fix the issue, however, the patch has been only tested for |
… 2023.06-software.eessi.io_zen4_R-bundle-Bioconductor-3.18-foss-2023a-R-4.3.2
|
Rebuilding after Highway got fixed (and ingested)... bot: build repo:eessi.io-2023.06-software arch:x86_64/amd/zen4 |
Updates by the bot instance
|
Updates by the bot instance
|
|
New job on instance
|
ocaisa
approved these changes
Aug 5, 2024
ocaisa
added
the
bot:deploy
|
Staging PR merged and ingested |
boegel
deleted the
2023.06-software.eessi.io_zen4_R-bundle-Bioconductor-3.18-foss-2023a-R-4.3.2
branch
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.