Enable parallel build inside Visual Studio

Hi @vasily-kirichenko, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

@vasily-kirichenko, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;

Epic! ;-)

@forki I think you are right :) Huge thanks @vladima

awesome @vasily-kirichenko !!

Wow, very cool.
@vladima - any further details that you recall from earlier experiments you mentioned?  Were there known issues that resulted in this feature being off by default? I am curious why it is turned off if it is indeed fully baked and tested, or if there are known scenarios that need some attention?

@latkin as I remember Brian has added this key shortly before he left so I think that this feature was never thoughtfully tested.

As so, maybe it's better to add a proper setting for this (I mean in the VS UI), at least for first F# 4 release?

For those wanting to try this out you can enable it by following the instructions here: https://gist.github.com/vasily-kirichenko/84f7ebc284d92742493b
@vasily-kirichenko I'm speaking for @latkin and @KevinRansom, but my understanding is that the feature window has closed for Visual Studio 2015 RTM (they are now in "ask mode", meaning explicit approval is required for bug fixes), but that this would be considered for the first update.
I also found some internal emails about this prototype which @latkin now has, pointing out that there is a minor thread affinity interaction with the VS type provider security warning dialog, but that was the only known issue.

Another big plus for getting rid of the dialog ;-)
On Jun 9, 2015 11:11, "Don Syme"<EMAIL_ADDRESS>wrote:

For those wanting to try this out you can enable it by following the
instructions here:
https://gist.github.com/vasily-kirichenko/84f7ebc284d92742493b
@vasily-kirichenko https://github.com/vasily-kirichenko I'm speaking
for @latkin https://github.com/latkin and @KevinRansom
https://github.com/KevinRansom, but my understanding is that the
feature window has closed for Visual Studio 2015 RTM (they are now in "ask
mode", meaning explicit approval is required for bug fixes), but that this
would be considered for the first update.
I also found some internal emails about this prototype which @latkin
https://github.com/latkin now has, pointing out that there is a minor
thread affinity interaction with the VS type provider security warning
dialog, but that was the only known issue.
—
Reply to this email directly or view it on GitHub
https://github.com/Microsoft/visualfsharp/pull/487#issuecomment-110284517
.

@dsyme OK, I think we can live with the regex setting for a while (however, VS removes it on regular basis, so it's a bit annoying to constantly check if the flag is still there).

Just an FYI: modifying anything in the _Config will go away when Visual Studio rebuilds its cache (typically, when running devenv /setup, maybe in other scenarios).
Also, the manual registry tweak only works in the "main" VS hive, not any other custom ones.
A better "workaround" would be to create a enable_fsharp_parallel_builds.pkgdef file, like this:
[$RootKey$\Projects\{f2a71f9b-5d33-465a-a702-920d77279786}]
"IsMultiThreadedBuildEnabled"="1"

and place it under $RootFolder$\Common7\IDE\Extensions
Then, close VS, run devenv /updateconfiguration once from a Developer command prompt, and that's it - VS will add this automatically every time the configuration is updated.
This is a hack, but a more consistent one than manual registry manipulation.

@hmemcpy an hack worthy of a wiki page, for similar issues

Applied to OOB branch

Great!!


Fix Project Euler: Problem 70 Typo

Describe your problem and how to reproduce it:
Following this post on the forum: https://www.freecodecamp.org/forum/t/project-euler-problem-70-totient-permutation-error-in-description/377577
The instructions should have 1 < n < 107, be 1 < n < 10**7.
Add a Link to the page with the problem:
https://www.freecodecamp.org/learn/coding-interview-prep/project-euler/problem-70-totient-permutation
First-timers welcome.

Hey, I'm a newbie to open-source. Would like to contribute. Is the issue still open and can I get to fix it?

Hello, @daksh-goyal . Yes, this issue is still open. Absolutely, go ahead and create a PR. Be sure to read the documentation for contributing, if you have not already.

@Sky020 I have just created a pull request fixing the issue. Please guide me on what I could do to contribute further. Thanks :)

@daksh-goyal , thank you, for getting that out. The main thing you can do is have a look at helping on issues with the help wanted label.


vcs export --exact-with-tags argument not working, reverts to --exact

Hi,
Avid fan of this package, cheers for all the work on it.
I've been attempting to use the vcs export --exact-with-tags flag but appear to have discovered that --exact with anything after it is considered as --exact, including --exact-with-tags, appearing to suggest an issue with argument parsing. For example, vcs export --exact, vcs export --exact-xyz and vcs --export-with-tags all produce the exact same output (git hashes only) using Ubuntu 20.04.4 and python3-vcstool 0.3.0-1.
Curious if anyone else has experienced this. I've had a go at debugging but haven't yet figured out how to modify my system install and get it to enact the changes I've made.
Cheers,
Joseph

Expected behavior:
$ vcs export --exact-with-tags
repositories:
  some_pkg:
    type: git
    url: git@some_server.com:some_group/some_pkg.git
    version: v0.0.999

Actual behavior:
$ vcs export --exact-with-tags
repositories:
  some_pkg:
    type: git
    url: git@some_server.com:some_group/some_pkg.git
    version: 7301a67082f3031de1a42bf342b26fdc6e110b67


icpx: error: unable to execute command: posix_spawn failed: Operation not permitted

When I pull docker image from docker hub and try to build a simple HelloWorld.cpp, I got an error message like title:
icpx: error: unable to execute command: posix_spawn failed: Operation not permitted
What happened?

Searching online, I see this error is sometimes connected to WSL usage. Is that the case in this instance?  If not, what environment are you running the container in, Docker? Podman? etc.


Padding Issues on List Items

There's an issue with the padding on the list of names that causes the whole screen to move when mousing over the name.  Would be a great issue for someone looking to practice/brush up on their CSS skills 🔥

I'll have a look :)

@mh84 Alright, much thanks 👍


LuckPerms doesn't recognize the game client language?

Description
Now I need to re-set the language on the client side every time I enter the server so that LuckPerms can correctly recognize the corresponding language.
Otherwise, English is displayed by default.
My previous server version was 1.20.1 and the language recognition was working fine.
As soon as I entered the game, "luckperms" were able to recognize the language correctly.
The language of the game client is Chinese.
Reproduction Steps
The game client sets Chinese, enters the server, and uses the luckperms command to view the language.
Expected Behaviour
Once inside the server, luckperms should recognize the language correctly
Server Details
purpur1.20.4 #2171        spigot 1.20.4
LuckPerms Version
5.4.121
Logs and Configs
Brand new and default
Extra Details
LuckPerms-Velocity-5.4.121 language recognition is normal

It doesn't matter

确实 这鸟bug挺烦 我的建议是直接用旧版本

Can you see if this is fixed by 9805f8b2c41c77afbda83bcd1f14cc1fdb7b6c67 (v5.4.131 or newer - https://luckperms.net/download)

Can you see if this is fixed by 9805f8b (v5.4.131 or newer - https://luckperms.net/download)
That's great! This problem has been solved!


422 mistake

i'm getting 422 error when trying to fetch from my node server
do you know what might be the reason?

im assuming it might be something to do with cross-domain access as on your demo site it works fine
is there a way to play around gpt-j api (from my server) without running my own instance?

Hi @maximignatev!
The 422 error seems related to an ill-formed call to the API (https://stackoverflow.com/questions/59929028/python-fastapi-error-422-with-post-request).
How are you calling the API? I provide two alternatives that are working for me, from an external machine:

Bash:

curl -X 'POST' \
  'http://api.vicgalle.net:5000/generate?context=In%20a%20shocking%20finding%2C%20scientists%20discovered%20a%20herd%20of%20unicorns%20living%20in%20a%20remote%2C%20previously%20unexplored%20valley%2C%20in%20the%20Andes%20Mountains.%20Even%20more%20surprising%20to%20the%20researchers%20was%20the%20fact%20that%20the%20unicorns%20spoke%20perfect%20English.&token_max_length=512&temperature=1&top_p=0.9' \
  -H 'accept: application/json' \
  -d ''

Python:

import requests

context = "In a shocking finding, scientist discovered a herd of unicorns living in a remote, previously unexplored valley, in the Andes Mountains. Even more surprising to the researchers was the fact that the unicorns spoke perfect English."

payload = {
    "context": context,
    "token_max_length": 512,
    "temperature": 1.0,
    "top_p": 0.9,
}
query = requests.post("http://api.vicgalle.net:5000/generate", params=payload)
response = query.json()

print(response)

Let me know if those work for you!

Hey. So I been trying to use the api from the node server (next.js api routes)

i'm not sure what was exact reason but i wasn't able to execute the request successfully when using axios library for data fetching but it's working fine now with the default fetch
thanks for reaching out for me

that's an offtopic but do you know if i can use advanced parameters like start/stop_sequence and provide examples on how i want model to operate when using your api?
i want to achieve something similar to this: https://github.com/TheProtaganist/gpt-j/blob/main/advanced_usage_template.py
or would you recommend running my own instance and tweaking parameters there?
do you know if i can host it somewhere for free (or with a free trial period at least)?
that must be too many questions 😅 can't stop being curious..

Regarding implementing more options with the API, I'm waiting til the model is merged withing huggingface's library (see #1).
But yeah, I'm planing on implementing more features in the near future!
If you want to deploy your own server, you can get TPUs for free by filling this form: https://sites.research.google/trc/. They should answer in a few hours

@vicgalle thats awesome thanks!!!


Added the few questions in FAQ section #1237

Hii @anuragverma108 !
I have added some more questions in the FAQ's section. It would help the user of our platform for better understanding. Like what to do in the payment failure etc and stuffs like that. It would bring more trust in the user and they would be more likely to donate in our project of saveGuarding wild animal.
Accept this PR under the hacktoberfest and hacktoberfest-accepted label.
Thankyou .

@brijgopal03 Thanks for your contribution! Please include before/after images or videos from next time to help us easily review visual changes.


[FMV] use long long type for marchid and mimpid

PR #74 introduced __riscv_cpu_model data struct for Function Multi-Versioning.
However, unlike fixed 32-bit mvendorid CSR, marchid and mimpid are MXLEN-sized CSR. The Linux kernel will also return these in u64 type, which takes a long typed value from MXLEN-sized SBI ecall. I think it's better to turn these fields to unsigned long long instead of  unsigned. It's still possible to make this change since these codes were only implemented in LLVM, but they now have no users.

cc @kito-cheng @BeMg

Will defer the merge until either LLVM or GCC merge for this change.

Will defer the merge until either LLVM or GCC merge for this change.

Now LLVM merged for this change.


[Event Request] Codeunit 12 "Gen. Jnl.-Post Line" - OnApplyCustLedgEntryOnAfterSetApplyingDate

Describe the request
Hello, can you please add a new Event OnApplyCustLedgEntryOnAfterSetApplyingDate in Codeunit 12 "Gen. Jnl.-Post Line"?
    procedure ApplyCustLedgEntry(var NewCVLedgEntryBuf: Record "CV Ledger Entry Buffer"; var DtldCVLedgEntryBuf: Record "Detailed CV Ledg. Entry Buffer"; GenJnlLine: Record "Gen. Journal Line"; Cust: Record Customer)
.
.
.
        OnApplyCustLedgEntryOnBeforePrepareTempCustLedgEntry(GenJnlLine, NewCVLedgEntryBuf, DtldCVLedgEntryBuf, NextEntryNo);
        if not PrepareTempCustLedgEntry(GenJnlLine, NewCVLedgEntryBuf, TempOldCustLedgEntry, Cust, ApplyingDate) then
            exit;

        GenJnlLine."Posting Date" := ApplyingDate;
        OnApplyCustLedgEntryOnAfterSetApplyingDate(ApplyingDate);
        // Apply the new entry (Payment) to the old entries (Invoices) one at a time
        repeat
            TempOldCustLedgEntry.CalcFields(
              Amount, "Amount (LCY)", "Remaining Amount", "Remaining Amt. (LCY)",
              "Original Amount", "Original Amt. (LCY)");
            TempOldCustLedgEntry.CopyFilter(Positive, OldCVLedgEntryBuf.Positive);
            OnApplyCustLedgEntryOnBeforeCopyFromCustLedgEntry(GenJnlLine, OldCVLedgEntryBuf, TempOldCustLedgEntry, NewCVLedgEntryBuf);
            OldCVLedgEntryBuf.CopyFromCustLedgEntry(TempOldCustLedgEntry);

            PostApply(
              GenJnlLine, DtldCVLedgEntryBuf, OldCVLedgEntryBuf, NewCVLedgEntryBuf, NewCVLedgEntryBuf2,
              Cust."Block Payment Tolerance", AllApplied, AppliedAmount, PmtTolAmtToBeApplied);
.
.
.

    [IntegrationEvent(false, false)]
    local procedure OnApplyCustLedgEntryOnAfterSetApplyingDate(ApplyingDate: Date);
    begin
    end;

Additional context
We???d like to globally save the ApplyingDate because we use it later on.
Internal work item: AB#542067

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 22883.


[QUESTION] Is build-essential really a needed dependency of this cookbook?

Cookbook version
2.2.1
Question
Is build-essential really a needed dependency of this cookbook?

build-essential is potentially used for compilation of the apps so yes you'll need gcc and the like for that.

but it does not appear to be used in the code .... i could only find references to it in the documentation, metadata.rb and test-kitchen configuration.  it seems like all of the packages that are needed are installed via attributes.


Return usernames too

Use case: https://twitter.com/swcarlosrj/status/1273887551099752448
I would like to have the @ usernames of everyone somehow, people could copy&paste on a tweet and they'd be able to follow their connections!
Congrats on such great work 🎊

Fixed in https://github.com/duiker101/twitter-interaction-circles/commit/174668805ceef3afd71e56c7762b7e176976b527
You can now run
yarn build --text

and the output with all the usernames will be in a users.txt file in the root of the project. Ready for use! 🥳

Very much appreciated @duiker101 🎊 Have a fantastic weekend


Unknown new bug, wrong modifier highlights appearing

https://discord.com/channels/170995199584108546/697845559435853865/1222739281529802754

When I target, it seems to label that the enemies has stuff that I have

That and the origin seems to be off-- and as a result, posts the DC's the enemies has

I could not reproduce this myself :/
I tried having Dorako enabled and targeting a third token while making a save against the DC of a second token, but no luck.

It looks like after updating everything, it seems to be no longer occuring.

original reporter (in DM)


[SPARK-34817][SQL] Read parquet unsigned types that stored as int32 physical type in parquet

What changes were proposed in this pull request?

Unsigned types may be used to produce smaller in-memory representations of the data. These types used by frameworks(e.g. hive, pig) using parquet. And parquet will map them to its base types.
https://github.com/apache/parquet-format/blob/master/src/main/thrift/parquet.thrift
  /**
   * An unsigned integer value.
   *
   * The number describes the maximum number of meaningful data bits in
   * the stored value. 8, 16 and 32 bit values are stored using the
   * INT32 physical type.  64 bit values are stored using the INT64
   * physical type.
   *
   */
  UINT_8 = 11;
  UINT_16 = 12;
  UINT_32 = 13;
  UINT_64 = 14;

UInt8-[0:255]
UInt16-[0:65535]
UInt32-[0:4294967295]
UInt64-[0:18446744073709551615]

In this PR, we support read UINT_8 as ShortType, UINT_16 as IntegerType, UINT_32 as LongType to fit their range. Support for UINT_64 will be in another PR.
Why are the changes needed?

better parquet support
Does this PR introduce any user-facing change?

yes, we can read unit[8/16/32] from parquet files
How was this patch tested?

new tests

Test build #136327 has started for PR 31921 at commit 0c8b6d4.

Test build #136327 has finished for PR 31921 at commit 0c8b6d4.

This patch fails Java style tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/136327/

Test build #136330 has started for PR 31921 at commit 8ff3267.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40909/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40909/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/40909/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40913/

cc @HyukjinKwon @cloud-fan @dongjoon-hyun thanks for reviewing

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40913/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/40913/

Test build #136330 has finished for PR 31921 at commit 8ff3267.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/136330/

Test build #136451 has started for PR 31921 at commit 0da5d07.

Test build #136451 has finished for PR 31921 at commit 0da5d07.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/136451/

Test build #136452 has started for PR 31921 at commit efe9c4a.

Test build #136452 has finished for PR 31921 at commit efe9c4a.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/136452/

Test build #136455 has started for PR 31921 at commit 3642f91.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41035/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/136455/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41036/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41036/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/41036/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41039/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41035/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/41035/

@cloud-fan @liancheng @HyukjinKwon @maropu please take another look

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41039/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/41039/

Test build #136472 has started for PR 31921 at commit d9afc79.

Test build #136472 has finished for PR 31921 at commit d9afc79.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/136472/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41056/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41056/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/41056/

Test build #136490 has started for PR 31921 at commit 02cee4f.

Test build #136497 has started for PR 31921 at commit 71496bd.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41076/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41076/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/41076/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41081/

Test build #136490 has finished for PR 31921 at commit 02cee4f.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/136490/

thanks, merging to master!

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41081/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/41081/

Test build #136497 has finished for PR 31921 at commit 71496bd.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/136497/


Box not found in cxx::bridge scope when using alloc

cxx::bridge appears to be using Box instead of a fully elaborated name when Box is used in a function declaration, which prevents it from being found. There doesn't appear to be a way to import another path for Box in a cxx::bridge module (use items are not allowed within cxx bridge).
error[E0412]: cannot find type `Box` in this scope
  --> src/lib.rs:15:21
   |
15 |         fn foo() -> Box<Bar>;
   |                     ^^^ not found in this scope
   |
help: consider importing one of these items
   |
14 |         type use alloc::prelude::v1::Box;
   |              ++++++++++++++++++++++++++++
14 |         type use crate::Box;
   |              +++++++++++++++

# Cargo.toml
[package]
name = "cxx_test"
version = "0.1.0"
edition = "2021"

# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

[dependencies]
cxx = {version="1.0.71", default-features=false, features=["alloc"]}

// src/lib.rs
#![no_std]
extern crate alloc;
use alloc::boxed::Box;

struct Bar {}

fn foo() -> Box<Bar> {
    Box::new(Bar {})
}

#[cxx::bridge]
mod ffi {
    extern "Rust" {
        type Bar;
        fn foo() -> Box<Bar>;
    }
}

Hi,
I am pretty sure I am hitting this same issue, but with Vec and String instead of Box.
Did you find a way to make it work?

I published a fix in cxx 1.0.80. Please report any other cases you hit with this kind of error, this stuff is tricky to get right.


Doc: FastStore Onboarding

What's the purpose of this pull request?
This PR adds the new documentation for the FastStore Onboarding app. It also adds a new section for the (coming soon) Setting Up the Project doc.
How it works?
The new FastStore Onboarding documentation consists of the following:

App's overview
Step-by-step on how to start the project
Next Step: links to the (coming soon) Setting Up the Project documentation

How to test it?
This is a documentation improvement only.
Starters Deploy Preview
Preview

I'll merge for now and keep updating this Getting Started track. Also because we need to add other tracks (e.g., components). Next steps:

Fix Onboarding images feat. Renatinha
Update Setting Up the Project
Add content to Understanding the Project Structure and Next Steps.


Add KERNEL_FULL_VERSION as a default build argument

Fixes (#400)
Signed-off-by: Hersh Pathak<EMAIL_ADDRESS>

/ok-to-test
/test all

A general question: we now have KERNEL_VERSION and KERNEL_FULL_VERSION as build arguments, and they both have the same value. Why not just keep one of them?

Why not just keep one of them?

We don't want to remove KERNEL_VERSION as it would break existing Dockerfiles. Maybe in 2.0, although I don't see the urgency of doing that.

Why not just keep one of them?

We don't want to remove KERNEL_VERSION as it would break existing Dockerfiles. Maybe in 2.0, although I don't see the urgency of doing that.

So why not just keep KERNEL_VERSION? The existing Dockerfiles are using it, and the new Dockerfiles can start using it, no?

So why not just keep KERNEL_VERSION?

Right now the containerImage field replaces KERNEL_FULL_VERSION and we provide the KERNEL_VERSION build argument. This is very confusing for users. See #400

Can we add a tech debt for removing KERNEL_VERSION build arg in 2.0 milestone?

@hershpa please rebase this PR on top of main.


Align tables with 'style' attribute instead with 'align'

This is how Parsedown parses aligned tables:
| A1 | B1 | C1 |
| :- | - | - |
| a1 | b1 | c1 |

a1
Since web and CSS are slowly moving toward HTML5, you should use 'style' attribute:
a1

@kminchev
Could you do some research and link to or quote official documents on the subject?

This is what I had in mind when I started this issue: http://www.w3schools.com/tags/tag_td.asp

Appreciated!
PS: thoughts on my comment about header row above?

By the way, I think the header row should always be centered. So, the colon should affect only  elements.

I'm not sure about this. It doesn't seem to be a part of the spec and it might be a bit controversial.

Controversial how? I can't imagine someone right-aligning their header row. And tables aren't part of the spec to begin with :P
Here's something you might want to consider: http://www.w3schools.com/tags/tag_th.asp
Now, I know it says regular cells are left-aligned by default, but that's not always the case. Usually columns with numbers are right-aligned, but I can't think of a reason why header should be anything but centered.
I'll be happy to see other people's opinions about this.

@kminchev
Parsedown already uses th tags for table headers.

Aligning content with specified user input "inserting colon" is one thing, if user wants to align header rows with colon, it's up to them, but enforcing any kind of style, like aligning headers to center is just plain wrong. I don't see any harm allowing specific alignment of header rows.
If you want your headers to be centered only, just make a css rule with !important.

@kminchev That's just my way of thinking, but table headers in HTML are left-aligned by default. Therefore they should be left-aligned by default in GFM.

@hkdobrev, according to the link I pasted above, th are centered by default and td are lefted.
@naNuke, I know that, I was just thinking out loud if this approach is good enough to make it hardcoded. Anyway, thanks all for your inputs!

@kminchev if th are aligned to center by default, why should parsedown do it then.

@naNuke, in case the user placed a colon on the left.

@kminchev in that case the only thing the colon does is changing the alignment to left so it may be used to achieve exactly that. The other columns should not change as  is left aligned anyway.
Btw, w3schools is not a good source for things like this, here is HTML spec:
http://www.w3.org/TR/html5/tabular-data.html#the-th-element
http://www.w3.org/TR/html401/struct/tables.html#h-11.2.6
Both do not explicitly say to align center but note that it is commonly displayed as such.

I know this is already resolved and Parsedown uses style tags, but I was bit with a Content-Security-Policy today, because I had blocked inline styles. I'm now using my own markdown class that extends Parsedown, and overrides blockTable and blockTableContinue.
GitHub still appears to use HTML align attributes, and I wish Parsedown had stayed with align attributes, but that ship has sailed :)


Secrets docs, PodSpec API broken link

This is a Bug Report
Problem:
Secret page contains the following broken link:

See the PodSpec API for more information about the imagePullSecrets field.

Additional info:
The current link is https://kubernetes.io/docs/reference/generated/kubernetes-api/v1.29/#podspec-v1-core.
The older versions i.e. /v1.28/#podspec-v1-core seem to work fine.
Page to Update:
https://kubernetes.io/docs/concepts/configuration/secret/#using-imagepullsecrets-1

/sig docs

Let's Work on this Collaboratively!!

/kind bug
/language en
This largely duplicates other issue reports that PR #44345 would fix.

Fixed in PR https://github.com/kubernetes/website/pull/44345
/close


Extension for Quaternions.jl

This allows for construction, conversion, and promotion between Quaternion and AbstractCliffordNumber{VGA(3)} instances: Quaternion{T} is treated as effectively identical to EvenCliffordNumber{VGA(3),T,4}.
Promotion preserves CliffordNumbers.jl semantics, so any promotion involving both Quaternion and AbstractCliffordNumber{VGA(3)} will return an AbstractCliffordNumber{VGA(3)}.
The Quaternions.jl slerp implementation is defined for AbstractCliffordNumber arguments, but in the future CliffordNumbers.jl may provide its own slerp implementation for arbitrary algebras.

wow this took forever


No grid lines on X axis

I'm doing some coding to have log-log plots, so I need to specify the ticks values manually. So far, everything is going great on y-axis, but on x-axis I'm having this issue with disappearing grid lines. See this fiddle as an example...only 1 line is appearing, and it's not related to any point of the data.
http://jsfiddle.net/asnj1k4n/2/
How can I make the x grids appear again?

I tried to fix it...maybe its the fact that you were using quotes when declaring the attributes.
http://jsfiddle.net/wkgzkztu/
Did I do something very different from you?

I've tried removing the quotes(I've dumped the object directly from the chrome console) but the grid lines are still invisible...also, in your fiddle, the lines don't match the actual grid value.

I know. That is as close as I could get...I guess then there is an issue...


NetFxWorker sample project cleanup

Updating NetFxWorker sample project to use stable version of packages.

Also, added a local.settings.json to the project. Without this file, core tools shows a warning when running the app, which might be confusing for users who are trying our sample.

/check-enforcer override

/check-enforcer evaluate


Create aid package - supplier detail needs to show the CIF (medicine price) value as well

[ ] Create aid package - supplier detail needs to show the CIF (medicine price) value as well.

@nshamit Would it be possible to get a screenshot of where we would want to see this information?

It should be displayed here:
Also, we should display the unit price, as well.


Drop support of Ruby 3.0

Ruby 3.0 will be EoL at 2024-03-31.

I will merge (rebase) this PR after 2024-04-01.


Update Travis for macOS

Depends On: #124
I took a stab at updating Travis to run against the latest version of LLVM on macOS. This will likely require some iteration on Travis until I work the kinks out.

I think I finally have this working 😓
Note that I intentionally excluded LLVM 3.8 from testing on macOS because<EMAIL_ADDRESS>package is broken: its llvm-config.cmake file includes an unexpanded CMake template variable:
set(LLVM_DEFINITIONS @LLVM_DEFINITIONS@)

@jslee02 Can you do another review pass?

its llvm-config.cmake file includes an unexpanded CMake template variable

Yeah, I saw that too.
👍 This PR looks good to me! I'm happy to see Chimera is built on macOS with different versions of LLVM toolchains.


Answers to Learn Java Exercises

Answers to Exercises - Learn Java

@OrlaithBegley Hey, any updates to that PR?


Added counter for limit queries

uncover -q 'nginx' -o test.txt -limit 15 | wc -l

  __  ______  _________ _   _____  _____
 / / / / __ \/ ___/ __ \ | / / _ \/ ___/
/ /_/ / / / / /__/ /_/ / |/ /  __/ /
\__,_/_/ /_/\___/\____/|___/\___/_/ v1.0.2

                projectdiscovery.io

      15

Suggested potential issues and points to clarify:

limit is mainly aimed at placing an upper limit to the number of iterations from a search engine (to avoid consuming all available requests)
To limit the number of results on the console, I think we either need to introduce a new flag, or users can pipe the output to head -n xx (cc @ehsandeep @parthmalhotra )

@Mzack9999 we can use the same option to control the lower limit as well, basically replicating the shodan behavior
shodan search --fields ip_str wordpress --limit 10

<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>

@ehsandeep, I think output filtering should be handled on the command line. Otherwise, it's worth refactoring the scan logic, as many engine tokens would be wasted if the amount of results collected is much lower than the one requested as output. (for example, if the limit is 10, it's probably worth iterating only on one engine instead of querying 10 in parallel and discarding 90 valid results)


Help for pull request

I added a new feature to EKF2.
The problem is that it requires both modifications to PX4/Firmware, but also to lib/ecl/EKF.
How is the correct protocol for pull request in both main firmware and submodules?
I have my fork and I can pull request for PX4/Firmware, but how to include the submodule? Do I need separate fork with separate pull request? What if the pull request is accepted in one and not in the other (the code won't compile)?

Hi @tuloski
Open the pull request on https://github.com/PX4/ecl, if it requires a corresponding PX4/Firmware change open a simultaneous PR with the submodule temporarily changed to your own fork. Make it clear in both PRs that they're dependent and we'll review and test them together. Once accepted we merge PX4/ecl, then immediately update the PX4/Firmware PR to point to the new ecl master and merge.
This is the worst case when changes are required on both sides. In many cases the changes can actually go in separately.

Ok perfect I'll do that.
But before I need to fix https://github.com/PX4/Firmware/issues/11072 because idk if it is my issue in the code or a problem with QGC.


Add support for Azure, OpenAI, Palm, Anthropic, Cohere Models - using litellm

I'm the maintainer of litellm https://github.com/BerriAI/litellm - a simple & light package to call OpenAI, Azure, Cohere, Anthropic API Endpoints.
This PR allows users to use models from the above providers in Chatbot.py
Attached ss of this working

cc @sfc-gh-cfrasca @dataprofessor @snehankekre

updated based on feedback from @dataprofessor😊


Elasticsearch service controller

New ACK Service Controller
Support for Amazon Elasticsearch Service
List of API resources

ElasticsearchDomain
ElasticsearchDomainConfig

API Actions that may be problematic
These API actions may be problematic due to misalignment with the Kubernetes RESTful resource semantics:

UpgradeElasticsearchDomain

/lifecycle frozen


review supported browsers

We should review if the minimal supported browser versions are still valid:
Internet Explorer: version 9 or newer
Firefox: version 22.0 or newer

Maybe we can increment the versions for some newer releases.

The webapp seems not to work with IE11. The login page works, but the product list does not load after it.

The result is a blank page. Using:

Version 11.0.65 (on Windows 10) doesn't help either, getting the same errors:

@whisperity I created a patch for this which will solve this problem: #1708

Thank you. However, a test should be done with these changes "merged" into a local deploy to ensure the rest of the code works. Due to this break, in IE/Edge, the page does not appear and I was unable to check the rest of CodeChecker features. If possible, it should be a single patch that fixes the current problems with compatibility, and all of them.


Removed aboslute path import and unnecesarry circular import

Once I upgrade gulp as per #22, I noticed an import failure. This fixes that problem and makes the app actually work on my machine.
Thanks for the awesome project.

This addressed #26 for me.

Thank you for the contribution, @jimmyhmiller. I would appreciated if you or anybody else helped me to develop this project.


Fix/config-default-exclude

This PR fixes https://github.com/vitest-dev/vitest/issues/2172.

[x] Added to the docs

Seems like the macos CI machine needs to upgrade its specs since eve≠the linux one is almost 2x faster. And it's probably why the test times out.


renv 1.0.2 Error in if (n == 1) word else paste(word, "s", sep = ""): app[web.1]: the condition has length > 1

Hi, first time deploying to Heroku using renv (v 1.0.2).
The build is successful but the app shows this error:

heroku[web.1]:Starting process with command R --file=/app/run.R --gui-none --no-save
app[web.1]:
app[web.1]: R version 4.2.1 (2022-06-23) -- "Funny-Looking Kid"
app[web.1]: Copyright (C) 2022 The R Foundation for Statistical Computing
app[web.1]: Platform: x86_64-pc-linux-gnu (64-bit)
app[web.1]:
app[web.1]: - Project '~/' loaded. [renv 1.0.2]
app[web.1]: Error in if (n == 1) word else paste(word, "s", sep = "") :
app[web.1]: the condition has length > 1
app[web.1]: Calls: source ... sprintf ->  -> nplural -> paste -> plural
app[web.1]: Execution halted
heroku[web.1]: Process exited with status 1
heroku[web.1]: State changed from starting to crashed

I wonder if it's related to the Renv 1.0.2 version since there is a recent post  about the same error message.
I also wonder if the fact I'm using R  4.2.1 on my desktop is causing the problem.
Any guidance on what to do would be greatly appreciated.

Hi @DaboinCJ
Apologies for the delay in responding.
Unfortunately, I haven't used recent versions of renv with the buildpack, so I haven't encountered this issue.
Reading the post you linked and follow up comments, the issue seems related to the existence of the renv cache directory.
I'll attempt to replicate the issue and revert back.

It appears that renv is attempting to create or use the ~/.cache/R/renv directory for cached packages. This path doesn't exist within the Heroku slug so when the .Rprofile is executed it fails.
Setting the RENV_PATHS_CACHE environment variable allows overriding the default path.
Run the following heroku CLI command to set the variable (or use the Heroku Dashboard):
heroku config:set RENV_PATHS_CACHE=/app/renv/.cache

I'll investigate setting this variable by default as part of the buildpack logic, so that this step isn't needed.

Due to the constraints of Heroku, all project files together with the R runtime, configurations and cache directories are all located within the /app directory (which is also the home directory).

As a result, when R starts and .Rprofile sources the renv/activate.R script, renv fails while searching through all the R files found within these directories.
Adding a .renvignore file to the project root directory to exclude these directories resolves the issue.
# .renvignore
.cache
R

Hi @DaboinCJ
Were you able to run your application with the changes suggested?

Hi @virtualstaticvoid, thanks for digging deep into this, I appreciate it.
I made the tool work with an init file instead, so I haven't tried that solution yet. I'll try next week.
I bypassed the problem by using the init file again and installing the exact version I needed for a couple of problematic libraries from packagemanager.posit.co.


OggDude Importer error: Invalid Compendium Pack Data: errors.type: may not be a blank string

Foundry v10 Build 291
Star Wars FFG 1.72
OggDude <IP_ADDRESS>
Reproductions sequence

Zipping data folder of OggDude's data folder
Using OggDude's Import function in Foundry
Error message: Invalid Compendium Pack Data: errors.type: may not be a blank string

Any suggestions greatly appreciated

SWFFG 1.72 is not intended for Foundry V10. If you need to use V10, SWFFG 1.8 Alpha5 is the most stable. Otherwise V9 is recommended.

SWFFG 1.72 is not intended for Foundry V10. If you need to use V10, SWFFG 1.8 Alpha5 is the most stable. Otherwise V9 is recommended.

Many thanks with SWFFG 1.8 it works.


Add argument to not override a style of the same element

It's probably necessary to compare the classes to make it possible to remove a style again.
{
  "title": "Green",
  "args": ["em", "class", "BasicText—green", "toggle"]
},
{
  "title": "Condensed (All Caps)",
  "args": ["em", "class", "BasicText--condensed", "toggle"]
},

Here's the corresponding documentation in Redactor: https://imperavi.com/redactor/docs/settings/formatting/#setting-formattingAdd

Version 2.0.5 now passes the args straight to the inline.format function (→ see API documentation), the plugin used a workaround to apply attributes in the past, as inline.format didn’t work reliably to do so.
Unfortunatelly, passing in "toggle" (or "add", "remove") as the forth param didn’t do what I expected. But I’m gonna close this issue, as it’s up to Redactor to fix this and we can’t really do anything about it.


Optionally ignore imbalanced AZs on MapNode step

When restoring a cluster, the new nodes AZ distribution may not match the one stored in the backup manifest. In this case, the MapNode fails even though the new nodes may actually be better distributed now.
The change allows mapping new nodes to their backup index in the presence of imperfect match between node AZs and backup AZs. A new exception is added to identify the imbalance error and catch it.
The option to the step defaults to false so as to retain backwards compatibility with the previous behaviour.
Ideally, a better solution would be to consider an AZ imbalance metric relative to some target replication factor, instead of naively remapping the backup indices. However, let's say this is good enough for the moment.

This is not a good solution, and the problem shouldn't persist the both backup and new node AZ distributions are balanced.


Replace PBKDF2 with Argon2id

Hello,
Thank you for your interest and positive feedback. I appreciate your kind words.
Yes, I plan to continue working on the proposal and make improvements as necessary. I will definitely keep you informed about any progress and plans to reopen this.

I am still interested, thank you

@zorggomat give me your e-mail so we can exchange ideas


imports are broken?

DEMO on a TS project
DEMO on a JS project

Seems the specified Capi version in both of those is 0.0.1. Should be 0.1.0-gamma.1.

Seems the specified Capi version in both of those is 0.0.1. Should be 0.1.0-gamma.1.

Then perhaps the docs should be updated? Because npm i capi installs version 0.0.1

Perhaps in stackblitz it does. Doesn't on my machine.

Also installing version 0.1.0-gamma.1 produces the following error:
A FAILURE: PACKAGE NOT FOUND
Unable to find package @capi/westend-dev in the NPM registry.
Please try installing a different package name

Seems to be an issue with how they are runnning browser code... A node project is able to install capi
https://stackblitz.com/edit/stackblitz-starters-2u3qzv?file=index.js

Seems to be an issue with how they are runnning browser code... A node project is able to install capi
https://stackblitz.com/edit/stackblitz-starters-2u3qzv?file=index.js

to install it, yes. To run it: no 🙁. Try importing anything from CAPI and running it (it uses node 16, so...)

Looks like their browser setup doesn't correctly handle URL packages (which we use for codegen), and their node setup uses an old version of node. I don't see much we can do here 🤷
We'll eventually build a Capi playground (#1153) which should address some of these usecases.

Looks like their browser setup doesn't correctly handle URL packages (which we use for codegen), and their node setup uses an old version of node. I don't see much we can do here 🤷
We'll eventually build a Capi playground (#1153) which should address some of these usecases.

Hi @tjjfvi,
I wanted to put forward a suggestion regarding the issue at hand. It would be beneficial to leave the issue open until a viable alternative or solution is identified. In my opinion, keeping it open would be advantageous because the ability to quickly demonstrate issues or features in the browser holds immense value. My concern is that a CAPI playground might not adequately address the needs of the majority of use cases. For example, it might not be suitable for demonstrating node project setups, parcel project setups, vitejs setups, and more.
With that in mind, I kindly request that you reconsider reopening this issue and collaborate on devising an action plan for bundling CAPI in a manner that truly serves the community's needs. While a CAPI playground is a nice addition, it is crucial to address this urgent matter as I previously explained. The CAPI playground falls short in meeting the requirements of the majority of use cases, and as far as I am aware, there hasn't been significant interest expressed by anyone other than CAPI team members.
Thank you for considering this suggestion, and I look forward to working together to find the best solution.

I'm not sure there's anything we can do to fix this, but I'll still reopen it to track the status. (Perhaps stackblitz will soon update their version of Node and unblock this.)

I'm not sure there's anything we can do to fix this, but I'll still reopen it to track the status. (Perhaps stackblitz will soon update their version of Node and unblock this.)

Could you please also look into why it doesn't work on codesandbox? 🙏
Also, the issue with stackblitz goes beyond node, as it can not be used on regular TS/JS projects, either. So, IMO that can (and IMO should) be fix as it has to do with the way that the library is being package.

What is the error you are getting on codesandbox? Is it a TLA error?

What is the error you are getting on codesandbox? Is it a TLA error?

Try creating a TS sandbox from scratch, then install capi and then try to reproduce the code reported in this issue (which I wanted to illustrate with a codesandbox/stackblitz, but wasn't able to 😞)... The result is this.
There are at least 2 problems in there:

Cannot find module 'capi' or its corresponding type declarations (and yes, moduleResolution is set to Node16)
SyntaxError: await is only valid in async functions and the top level bodies of modules


AST: introduce the opposite options for ABIBreakingToAdd and others

Adding ABIBreakingToAdd and other options for decl attribute kind isn't
sufficient because future attributes may forget to add the ABI/API impact bits.
This patch introduces the opposite options of these breaking bits (ABIStableToAdd, etc)
, and adds several static assertions to ensure one of the opposite ABI/API impact
flags is explicitly specified.

@swift-ci please smoke test

@swift-ci please smoke test


Show track history

I often listen to a station and don't pay enough attention to what's playing to be able to vote a song up/down unless it's something really new to me and I like it or it doesn't fit the station at all.  Would it be possible to implement a track history of the current session similar to the seeds and likes/dislikes, or the station sidebar?

This already exists but is clearly not discoverable enough.  You can click the icon in the toolbar which will either be a broadcasting antenna (stations) or a clock with counterclockwise arrow (history), or use View → History or ⇧⌘H.

Ah thank you...and yeah, that wasn't exactly intuitive, but does precisely what I expected.  Thanks!

There should be a tooltip on that icon — you can also right-click the toolbar and choose "Icon & Text" to see labels on each of the buttons if you want.

I swear that tooltip wasn't there before...tried and tried to make it appear, no luck.    Witchcraft.  I'll go with that.


We need to extract a small sample from CantusDB to try to upload that to Virtuoso directly

This issue follows https://github.com/DDMAL/linkedmusic-datalake/issues/104. And it's closely related with https://github.com/DDMAL/linkedmusic-datalake/issues/67
Over time, we have become uncertain that regarding CSV2RDF (LinkedMusic), whether using python (plus RDFLib package) is the best approach. As a result we begin trying an "RDF-specific" method using R2RML (RDB to RDF Mapping Language, compliant with W3C standards) in Virtuoso to achieve RDB2RDF & reconciliation.
The CantusDB as a whole is too large. I just need a very small sample with complete schema information. Ideally, this sample would include a few entries for each entity and each relation, sufficient for experimental purposes.

Adding this here as a tool that might be useful for this task:
https://github.com/mla/pg_sample


Model datatype in intake metadata

@dougiesquire The model datatype in intake metadata is an array but experiments I have seen so far contains only one entry "ACCESS-OM2" in that array. Is that intentional or are there component models missing?

Hi @utkarshgupta95. Yes that's correct. An experiment might contain multiple models, but atm most in the access-nri intake catalog contain only one. I think the CMIP entries are the only exception at the moment - these contain many models.

Thanks for explanation


Multiline comments not stripped when having trailing spaces

try stripping the below comments
//this will be removed
var v="sss"; //this will not be removed
/*
* This comment is not removed.
*/
        /**
        *This will be removed.
        */

sdasd
/*
** will not remove
*/
the plugin is so unpredictable, could you please debug and check ?

It is currently only stripping /** style comments for multiline, not /* style
This will be fixed by tonight, adding support for /* */ style comments, just writing some tests. This repo has had some major entropy due to my negligence but I finally have time to get it back up to a really solid place.

resolved


Error putting S3 policy: MalformedPolicy: Invalid principal in policy

I guess this is a blocker.
This module wouldn't work unless AWS fixes this issue or you start using canonical ID instead of CloudFront Origin Access Identity ID.
Recently AWS made a few changes to their origin access identity policy format. The S3 bucket policy is now rejecting principal with spaces in it.
"Principal": {
"AWS": "arn:aws:iam::cloudfront:user/CloudFront Origin Access Identity EO3XXXXXXXX"
}
They now replaced it with
"Principal": {
"AWS": "arn:aws:iam::cloudfront:user/CloudFront_Origin_Access_Identity_EO3XXXXXXXX"
}
This module is still generating policy with spaces in its principal and hence the issue.
Can we change it to use canonical ID instead?

I am having this issue too and I thought I was going crazy!!!

`# module.computercompany_cdn_dev.aws_s3_bucket_policy.default will be created

resource "aws_s3_bucket_policy" "default" {

bucket = "dev-computercompanyccc-net-origin"
id     = (known after apply)
policy = jsonencode(
{
+ Statement = [
+ {
+ Action    = "s3:GetObject"
+ Effect    = "Allow"
+ Principal = {
+ AWS = "arn:aws:iam::cloudfront:user/CloudFront Origin Access Identity "
}
+ Resource  = "arn:aws:s3:::dev-computercompanyccc-net-origin/*"
+ Sid       = ""
},
+ {
+ Action    = "s3:ListBucket"
+ Effect    = "Allow"
+ Principal = {
+ AWS = "arn:aws:iam::cloudfront:user/CloudFront Origin Access Identity "
}
+ Resource  = "arn:aws:s3:::dev-computercompanyccc-net-origin"
+ Sid       = ""
},
]
+ Version   = "2012-10-17"
}
)
}`

I think its a problem with the labeler.
this is in the tfstate
"json": "{\n  \"Version\": \"2012-10-17\",\n  \"Statement\": [\n    {\n      \"Sid\": \"\",\n      \"Effect\": \"Allow\",\n      \"Action\": \"s3:GetObject\",\n      \"Resource\": \"arn:aws:s3:::${bucket_name}${origin_path}*\",\n      \"Principal\": {\n        \"AWS\": \"arn:aws:iam::cloudfront:user/CloudFront Origin Access Identity xxxxxxxxxx\"\n      }\n    },\n    {\n      \"Sid\": \"\",\n      \"Effect\": \"Allow\",\n      \"Action\": \"s3:ListBucket\",\n      \"Resource\": \"arn:aws:s3:::${bucket_name}\",\n      \"Principal\": {\n        \"AWS\": \"arn:aws:iam::cloudfront:user/CloudFront Origin Access Identity xxxxxxxxxxxx\"\n      }\n    }\n  ]\n}",

Its related to this https://github.com/terraform-providers/terraform-provider-aws/issues/10158

Created a PR that fixes this https://github.com/cloudposse/terraform-aws-cloudfront-s3-cdn/pull/51

This issue may not be relevant any more according to the last comment in PR https://github.com/cloudposse/terraform-aws-cloudfront-s3-cdn/pull/51

I'm having this issue trying this code. Still checking what could be wrong. Any help will be appreciated. Thanks in advance.
Error
│ Error: Error putting S3 policy: MalformedPolicy: Invalid principal in policy
│       status code: 400, request id: D5EJ0K88W45CZHPF, host id: Ii5V1c632IO4fRi8qSboduNHvDpHie4pPhMTUcyVMGV1345Zx/kng4zgd5FN+u47amjVbN6vCYc=
│
│   with module.cdn.aws_s3_bucket_policy.default[0],
│   on .terraform/modules/cdn/main.tf line 243, in resource "aws_s3_bucket_policy" "default":
│  243: resource "aws_s3_bucket_policy" "default" {
│

Code
module "cdn" {
  source = "cloudposse/cloudfront-s3-cdn/aws"
  # Cloud Posse recommends pinning every module to a specific version
  # version = "x.x.x"

  namespace         = "eg"
  stage             = "prod"
  name              = "app"

  deployment_principal_arns = {
    "arn:aws:iam::*:role/principal2" = [""]
  }
}


A little bug in melonnpan_predict.R

I came across the same problem as #14 . I checked the code in melonnpan_predict.R and noticed a strange:
 # Load weight matrix
  if (is.null(weight.matrix)){
    weight.matrix<-melonnpan::melonnpan.trained.model
  } else{
    train.weight<-as.data.frame(t(readTable(weight.matrix)))
  }

The variable train.weight will not actually be set if weight.matrix is not provided.

Hi @zhifeng-essen - I think you are not using the most updated version of the code in which this issue is fixed. Let me know if you run into any issue (feel free to send me an email for faster reply) when you run it after reinstalling the package. Thanks!


Can I keep part of the binder fixed and fill in the rest?

I want to keep part of the designed binder chain fixed and only diffuse over a few loops in the residue. Is it possible to do that yet?

Not currently


Create an i8n Style Guide

Create an i8n Style Guide
Ref: https://github.com/cncf/foundation/blob/master/translation.md

This should be worked through with the LF China team


Replace lru_cache decorators by freely configurable alternative

The LRU caches are currently used by the functions _levenshtein_dist() and lemmatize().
Problem description:

So far their maximum sizes cannot be set by the user.
The LRU decorator also stands in the way of further function use, e.g. in a class or in another application.
There is no function directly available to the users which allows for clearing the caches (apart from function_name.cache_clear())

See also these comments for reference:

https://github.com/adbar/simplemma/pull/38#issuecomment-1401780261
https://github.com/adbar/simplemma/pull/38#issuecomment-1402420705
https://github.com/adbar/simplemma/pull/42#discussion_r1085853608

A possible solution, wrap the function in another instead of using a decorator:
from functools import lru_cache
cached_levenshtein_dist = lru_cache(maxsize=65536)(_levenshtein_dist)

Advantages: removes cache-linked roadblock on function _levenshtein_dist, only one line of code
Drawbacks: no way to set cache size (a settings.cfg mecanism is imaginable)

How is the function different than the decorator?
This doesn't solve the problem of having _levenshtein_dist in other internal methods and doesn't solve the problem of controlling the cache size.

The cache for the Levenshtein distance wasn't necessary, it's hardly used during dictionary construction and during lemmatization it's a cache within a cache, so I deleted it, one problem less.
@juanjoDiaz Now there's only the cache around _lemmatize() left. It's marginally better since it's clearly isolated. The main point is that you can now choose to wrap your own class around _lemmatize() or _return_lemma() without being hindered by this cache.

@juanjoDiaz Do you still have improvements in mind concerning this issue or is it now solved?


Performance

I've been missing a proper label3D in Godot since forever, hence this plugin is a godsend.
My question is how performant is it? Should I use it sparingly? Contrary to my current implementation of sprite sheet animated sprite 3D where i slap lots of nodes of those changing frame numbers according to which character i need.
Thanks!

well to me the greatest advantage of 3d text is the fact it becomes a part of the 3D world, hence allowing for diegetic UI. For instance UI around the main character showing their HP, name, DEF and other stats.
Or another example would be 3D cards, that would exist in 3d space to enhance immersion.
(I've so far used a combination AnimatedSprite3Ds, Control nodes and viewports but each came with a limitation that Lbael3D would solve)
I'm curious now, do you think it would be worth opening a proposal for an official Label3D? As in would the core team be able to implement it in a manner which doesn't rely on viewports?

Well, perhaps the fonts could have their curves loaded in directly rather than by relying on drawing their bitmaps. It would also be more performant. However seeing as how this wouldn't really be used too often, and how it can be implemented with Godot already has, there is a thin chance that this would be part of core.


Line 92 of StatusBarFeature flagged by static analysis

Line 92 flagged. SA believes that menu.triggered is a function and should not have a field connect.
It seems like the intention was to do
menu.connect('triggered', self.toggle_option_visible)
or something similar, but I am only passing through this part of the code, and I am not sure what this is about
Originally posted by @samiebee43 in https://github.com/WasatchPhotonics/ENLIGHTEN/pull/216#discussion_r1251296958

Dunno, it's definitely a signal, and I can see that the callback (toggle_option_visible) is working (checking / unchecking items from the "..." menu on the StatusBar causes the respective fields to dis/appear).

It's possible that PySide2 or Qt5 is using a getattr as part of its bindings to Qt.
I'll consider this a 'reimplement', where the thing is working, but we'd be in better form if the analyzer can understand it.


added animation on scroll

added animation on scroll close issue #20

@tushargupta1504 bro none of my contribution is listed on gssoc website why?

@tushargupta1504 bro PRs ka point to label nhi kiye 🙂


HuggingFace error

My test environment is as below.
Ubuntu 24.04
Nvidia GPU (Coda toolkit and openexr is installed)
Python 3.9.21
Use pipenv instead of conda.
Somehow when I run the initial command as below.
 python inpaint.py --dataset model --output_dir output

It failed with the below message.
Traceback (most recent call last):
  File "/home/tempest/Projects/DiffusionLight/inpaint.py", line 15, in <module>
    from relighting.inpainter import BallInpainter
  File "/home/tempest/Projects/DiffusionLight/relighting/inpainter.py", line 2, in <module>
    from diffusers import ControlNetModel, AutoencoderKL
  File "/home/tempest/Projects/DiffusionLight/.venv/lib/python3.9/site-packages/diffusers/__init__.py", line 5, in <module>
    from .utils import (
  File "/home/tempest/Projects/DiffusionLight/.venv/lib/python3.9/site-packages/diffusers/utils/__init__.py", line 36, in <module>
    from .dynamic_modules_utils import get_class_from_dynamic_module
  File "/home/tempest/Projects/DiffusionLight/.venv/lib/python3.9/site-packages/diffusers/utils/dynamic_modules_utils.py", line 28, in <module>
    from huggingface_hub import HfFolder, cached_download, hf_hub_download, model_info
ImportError: cannot import name 'cached_download' from 'huggingface_hub' (/home/tempest/Projects/DiffusionLight/.venv/lib/python3.9/site-packages/huggingface_hub/__init__.py)

Is there anything I missed?

DiffusionLight is not compatible with newer version of huggingface-hub please downgrade it. I think 0.19.4 should be good
pip install huggingface-hub==0.19.4

@pureexe Thanks. It works.


Update TT-NN to 0.53.1-rc16

This PR updates TT-NN wheel to the latest pre-release version.

Reported an issue to the team. Looks like a regression


Bot radio chat spam.

Hello,  I have used some of the builds suggested in this close issues section along with the latest build 5832.
On build 5395 when changing command yb_communication_type "0" in config it does not work the bots keep on chattering non stop.  If I trigger in the console during gameplay it will disable it but in the next map it's right back to yb_communication_type "2" and they chatter once again.
On build 5832 yb_communication_type does not work at all and they still chatter
I saw somewhere in a closed issue of another recommended build recommended to a user and it's exhibiting the same behavior.  Any way to get this annoying thing to stop spamming radio chatter?
BOT123 (RADIO): Hold This Position.
BOT123 (RADIO): Hold This Position.
BOT123 (RADIO): Hold This Position.

I have tried the latest build instead as recommended.  It's the same thing.  The config has no reference to the commands you listed above.  It's still doing the same thing.

Hello.
Prince right, in latest builds cvar named yb_radio_mode.
Latest build: https://yapb.ru/files/binaries/5832/release/
On Sat, Feb 1, 2020, 03:31 iclassdon<EMAIL_ADDRESS>wrote:

I have tried the latest build instead as recommended. It's the same thing.
The config has no reference to the commands you listed above. It's still
doing the same thing.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jeefo/yapb/issues/118?email_source=notifications&email_token=AANS4YTFDC2FT42F2ZEV2FDRAS7H3A5CNFSM4KNHPUS2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKQN3BY#issuecomment-580967815,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANS4YWHRUPRHX4N4ULSL6LRAS7H3ANCNFSM4KNHPUSQ
.

@iclassdon Try that in-game, the repository only have the binaries listed there, so the .cfg doesn't get updated, or in my case it didn't, also i just checked the command it's "yb_radio_mode" now.

Thanks that worked.


Shoot specification: require clientID and issuerURL when setting oidcConfig

How to categorize this PR?

/area usability
/kind bug
What this PR does / why we need it:
This PR fixes a bug in the Shoot spec which allowed configuring the spec.kubernetes.kubeAPIServer.oidcConfig field with setting which would cause the kube-apiserver to be stuck in Error state.
All setting in oidcConfig are set as --oidc-* flags in the kube-apiserver ref (except for spec.kubernetes.kubeAPIServer.oidcConfig.clientAuthentication which does not have any use #1433). However, setting any --oidc-* flag without setting oidc-issuer-url and oidc-client-id would lead to an error.
This PR improves the validation of spec.kubernetes.kubeAPIServer.oidcConfig by validating that issuerURL and clientID are also set when oidcConfig is set.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

A bug has been fixed which was allowing users to set `Shoot` oidc configurations for the `kube-apiserver` without setting the `issuerURL` and `clientID` fields in `spec.kubernetes.kubeAPIServer.oidcConfig`, which would lead to the `kube-apiserver` stuck in a `Error` state.

Did you think about backwards-compatibility of this change (what if there are Shoots that are already misconfigured)?

Shoots that are currently misconfigured are stuck in creation with their apiserver pods failing with
"command failed" err="oidc-issuer-url and oidc-client-id must be specified together when any oidc-* flags are set"

When gardener is updated to a version containing the validation changes from this PR the shoot creation would still be stuck but in an earlier step.
Both failed states can be fixed by editing the shoot spec to comply with the new validation.

OK. What about Shoots that are already in Delete Processing phase with this misconfiguration when Gardener gets upgraded to a new version containing this PR?

This makes sense. I'm just wondering whether gardenlet will be able to perform the deletion (including removal of finalizer, etc.) if some human manually fixed the KAPI. Did you try this scenario?

Gardenlet failed to complete the shoot deletion. The shoot gets stuck in Delete Succeeded state with gardenlet not being able to remove the Shoot's finalizer since the shoot spec is invalid.
Perhaps we can leave the current implementation with the known bug that oidc can be misconfigured. Keeping in mind that in a feature release we will deprecate/ remove the oidc configurations in favour of structured authentication.

@AleksandarSavchev your call :)

Since the the oidc settings will be deprecated/ removed in a future release and this change can bring problems to existing shoot clusters stuck in deletion I have decided to close this PR.
/close

Hi @rfranzke,
When going over this closed PR with @AleksandarSavchev , we concluded that the validation can be performed for new Shoots only to allow updating deleting Shoots with already wrong oidcConfig. It does not make sense to allow Shoot creation with wrong oidcConfg when we know that the Shoot creation itself will fail with the given oidcConfig (the kube-apiserver will fail to start).
Let us know if you have objections/comments.


Compiling C++ inference with -O1 produces wrong results

Please go to Stack Overflow for help and support:
https://stackoverflow.com/questions/tagged/tensorflow
If you open a GitHub issue, here is our policy:

It must be a bug, a feature request, or a significant problem with documentation (for small docs fixes please send a PR instead).
The form below must be filled out.
It shouldn't be a TensorBoard issue. Those go here.

Here's why we have that policy: TensorFlow developers respond to issues. We want to focus on work that benefits the whole community, e.g., fixing bugs and adding features. Support only helps individuals. GitHub also notifies thousands of people when issues are filed. We want them to see you communicating an interesting problem, rather than being redirected to Stack Overflow.

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
Not in the tensorflow, but I have a code that utilizes C++ API

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Linux VirtualBox 4.13.0-41-generic #46~16.04.1-Ubuntu SMP Thu May 3 10:06:43 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux - running in VirtualBox on a Windows Host

TensorFlow installed from (source or binary):
Source

TensorFlow version (use command below):
tf.VERSION = 1.8.0-rc1
tf.GIT_VERSION = v1.8.0-rc1-1239-gd0f5bc1
tf.COMPILER_VERSION = v1.8.0-rc1-1239-gd0f5bc1
Sanity check: array([1], dtype=int32)

Python version:
Python 2.7.12

Bazel version (if compiling from source):
Build label: 0.13.0
Build target: bazel-out/k8-opt/bin/src/main/java/com/google/devtools/build/lib/bazel/BazelServer_deploy.jar
Build time: Mon Oct 18 21:33:40 +50297<PHONE_NUMBER>620)
Build timestamp:<PHONE_NUMBER>620
Build timestamp as int:<PHONE_NUMBER>620

GCC/Compiler version (if compiling from source):
c++ (Ubuntu 5.4.0-6ubuntu1~16.04.9) 5.4.0 20160609

CUDA/cuDNN version:
No

GPU model and memory:
No

Exact command to reproduce:
Please see below
You can collect some of this information using our environment capture script:

https://github.com/tensorflow/tensorflow/tree/master/tools/tf_env_collect.sh
You can obtain the TensorFlow version with
python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
Describe the problem
Describe the problem clearly here. Be sure to convey here why it's a bug in TensorFlow or a feature request.
I'm using a model I built in Python to run inference in C++ and compare the results to the ones I got when running it in Python. I save the exact batches I fed in Python and feed them in C++. When I use the default build settings I get very close results. However, I've found that matrix calclulations I performs with the results are incredibly slow (3 minutes for matrix division of 2 1000 by 5 matrices - I will submit a separate issue for that). So I tried to compile with -O1.
The result returned by running the inference very different now from the expected one.
Please clarify what is going on here.
So without optimization I get:
LoadModel passed!
Iteration 1
outputs[0]: Tensor<type: float shape: [1,5] values: [0.928591609 0.129222199 1.01102]...> <== Inferred by C++
output_tensor Tensor<type: float shape: [1,1,5] values: [[0.928591728 0.129222214 1.01102006]]...> <== Inferred by Python
With -O1:
LoadModel passed!
Iteration 1
outputs[0]: Tensor<type: float shape: [1,5] values: [0.907290399 0.124970555 1]...> <== Inferred by C++
output_tensor Tensor<type: float shape: [1,1,5] values: [[0.928591728 0.129222214 1.01102006]]...><== Inferred by Python
diff is bigger than expected: 0.0445271
Source code / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached. Try to provide a reproducible test case that is the bare minimum necessary to generate the problem.
optimization_problem_tensorflow.zip

@mkravchik , can you include the exact command you used to build TensorFlow in each configuration?

I meant that I compile the attached inference_cc.cc file by adding -O1 to the CMakeLists.txt like in the attached file the original file is in the zip attached above.
CMakeLists.txt

@asimshankar - I hear you may have some insight here?


Added initial report junit generator

Adding this to be able to have this integrated into GitLab pipelines reporter.
Initial idea. Let me know if there is something that we can improve regarding the reporter handling.

junit reporter is added in the v2.0.0 version


the text goes out of text cell boundary

copy a long text
open a new notebook and add a text cell
paste the text
expected: horizontal scroll within the cell boundary

actual:

Moving to November for now to ensure this does not get missed. Will wait to see how investigation looks to determine if this is a regression, and if it is, what the fix looks like.

Was able to repro on stable. So this is not a regression. Moving to December.

Also, the workaround is once a user exits the cell will autoformat and word-wrap the text in the cell. Then the cell will work as expected.


Bump to 1.48.7

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


VSCode recommends the mssql extension for everything

Issue Type: Bug
I haven't used MS SQL Server in 15 years but every time I open a SQL file for my Postgres / MySQL / Hive projects VSCode tries to get me to install that extension.
VS Code version: Code 1.43.2 (0ba0ca52957102ca3527cf479571617f0de6ed50, 2020-03-24T07:34:57.037Z)
OS version: Darwin x64 19.4.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4650U CPU @ 1.70GHz (4 x 1700)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
11, 5, 4

Memory (System)
8.00GB (0.05GB free)

Process Argv
.

 It's hard to strike a balance for which extension to recommend when opening a .sql file, as we cannot tell if it is SQL server or MySQL. We balance it out by having an "important recommendation" for the mssql extension and "passive recommendations" for vscode-database and SQL Tools extensions. You can click on the gear of the notification and click "don't show again".
Discussion for this decision in a previous issue:
https://github.com/microsoft/vscode/issues/41368#issuecomment-516196125

Could “Don't show again” be more clear about whether it means “Don't suggest mssql for .sql files”, “don't suggest anything for .sql files”, or “don't suggest extensions for anything”?

Two recent tweets about this too:
https://twitter.com/jbinsen/status/1301114423688470528
https://twitter.com/JLarky/status/1301242179092795392
@chrisdias thoughts on removing the "important" mssql recommendation and just leave the passive recommendation in?

fixed with 5177f0d777468ce08a31670ff6fe55b70f9b4a8a, mssql should not be an important recommendation, but instead will just show up in the recommended list (type @recommended in Extension view).