Authentication setup interfering with kfserving

/kind bug
How authentication is supposed to interact with kfserving?
I followed this article to setup istio/dex authentication here:
https://journal.arrikto.com/kubeflow-authentication-with-istio-dex-5eafdfac4782
And it has been working great. However, we are trying to get kfserving to work and now the envoy filter below is interfering with it.
apiVersion: networking.istio.io/v1alpha3
kind: EnvoyFilter
metadata:
name: authn-filter
spec:
workloadLabels:
istio: ingressgateway
filters:

filterConfig:
httpService:
serverUri:
uri: http://authservice.istio-system.svc.cluster.local
cluster: outbound|8080||authservice.istio-system.svc.cluster.local
authorizationRequest:
allowedHeaders:
patterns:
- exact: "cookie"
filterName: envoy.ext_authz
filterType: HTTP
listenerMatch:
portNumber: 443
listenerType: GATEWAY

We are trying to simply make serving api calls pass this filter all together, so I'm trying to see how I can scope down when this filter applies.
For example the call below (making a backend prediction api call) shouldn't go through this filter:
curl -v -H "Host: sklearn-iris.foo.example.com" http://localhost:8080/v1/models/sklearn-iris:predict -d @https://raw.githubusercontent.com/kubeflow/kfserving/master/docs/samples/sklearn/iris-input.json
Ideally we want a mechanism to say if the host header is "sklearn-iris.foo.example.com", then pass the authentication filter...
Looks to me kfserving is using the same kubeflow-gateway and thus the filter will apply no matter what, which seems wrong.
So just in general, how is this kfserving configuration supposed to be compatible with the recommended UI authentication setup/filter indicated above?

@yanniszark  @krishnadurai Any thoughts on this?

@yuzisun any update on this one? I'm falling in the same error!

Is there an  update or recommendation on this? I was wondering if there are sample envoyfilter that overrides authentication for kfserving - sudhir

Hi, is there any update on this one?

What I found as a workaround or probably the right way is -
Have separate ingress-gateways and loadbalancer/IP addresses with each own entry point. One for main kubeflow pipelines /training and another ingress-gateway for kfserving with no authentication. So we do not have to be constrained using the same envoyfilter for both.
That is what I did and made it work
Sudhir

What I found as a workaround or probably the right way is -
Have separate ingress-gateways and loadbalancer/IP addresses with each own entry point. One for main kubeflow pipelines /training and another ingress-gateway for kfserving with no authentication. So we do not have to be constrained using the same envoyfilter for both.
That is what I did and made it work
Sudhir

@nallagangus can you share how you configured the separate ingress?

Any update on this ?


I added the definition of backtracking in Backtracking's  README.md file

👨💻 Changes proposed

There was no definition in of Backtracking in the README.md file in the Backtracking section so I added it for the users to understand it properly.

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers
I am also looking forward to contribute more by adding questions with solution and multiple type of approaches for a given question.

Kindly create an issue first and then open a PR

Okay I will do that and thanks, but could you please let me know what it means to spam on  GitHub, since I am new to the open source and GitHub, If I made a mistake I am sorry for that. I am here to learn and grow.


IndexError is raised in OneDrive._get_token() when refresh token is available

At line 134 in OneDrive._get_token():
# Set the access and refresh tokens to the instance attributes
self._access_token = access_token
self.refresh_token = response_data["refresh_token"]

when trying to get the value associated to the refresh_token key of the response_data dictionary, an IndexError is raised when self.refresh_token already contains a valid refresh token. This happens because the endpoint does not provide a new (or the same) refresh token in the answer, when the call already contains one in the body.
A simple workaround:
# Set the access and refresh tokens to the instance attributes
self._access_token = access_token
if "refresh_token" in response_data.keys():
    self.refresh_token = response_data["refresh_token"]

This way the refresh token in self.refresh_token once set never changes: I don't know if this is an acceptable behaviour or not.
A collateral problem is many IndexError exceptions, when using the CLI tool, are catched in _cli.py, and the error message, which refers to the config file format, ends up to be misleadng, considered that no traceback is printed. I.e.
# Create the instance
try:
   onedrive = graph_onedrive.create_from_config_file(
        config_path=config_path, config_key=config_key
    )
except KeyError:
    raise SystemExit(
        f"Error: Config not in acceptable format or dict key '{config_key}' incorrect."
    )

The MS Docs page on this suggests that a refresh token is always provided in response to an access token request, even when the request itself used a refresh token, this is how refresh tokens are refreshed. It would be interesting to figure out why your responses do not include a refresh token - this is likely a security setting that does not allow refresh tokens to be refreshed.
I have amended the _get_token method to accept responses that do not contain a refresh token, a warning will be issued but you can catch this warning in your production code.
I have also moved the key error check to the create, create_using_config_file functions.

The MS Docs page on this suggests that a refresh token is always provided in response to an access token request, even when the request itself used a refresh token, this is how refresh tokens are refreshed. It would be interesting to figure out why your responses do not include a refresh token - this is likely a security setting that does not allow refresh tokens to be refreshed.

I found out why the refresh token is not present in the answer. MS docs here and here specify that no scope field is needed nor optional in the body of the requests. In fact, the scope should be provided only in the very first GET request (doc here).
So this:
body = {
            "client_id": self._client_id,
            "client_secret": self._client_secret,
            "scope": self._scope,  # "offline_access%20" + self.scope,
            "redirect_uri": self._redirect,
        }

should be changed to this:
body = {
            "client_id": self._client_id,
            "client_secret": self._client_secret,
            "redirect_uri": self._redirect,
        }

If this is not acceptable for some reason, in my tests I noticed that if the scope specified in the body is the same as the one specified in the very first request, the refresh token is provided as expected.
But since the scope in _get_authorization is specified as:
request_url += "&scope=offline_access%20" + self._scope

while, in _get_token(), is specified as:
"scope": self._scope,  # "offline_access%20" + self.scope,

ending up to be different, no new refresh token is present in the answer. If the scope in the body is changed to:
"scope": "offline_access " + self._scope,

everything works.
Following the Microsoft Graph Authentication Code Flow, the right choice should be to remove the scope from the body.

I have amended the _get_token method to accept responses that do not contain a refresh token, a warning will be issued but you can catch this warning in your production code.
I have also moved the key error check to the create, create_using_config_file functions.

Thank you.

Thanks for the info. Seems that these two docs differ on the requirement of scope for the /oauth2/v2.0/token endpoint:

OneDrive Developer / REST API does not include scope
Microsoft Graph / Develop states scope is required

I suspect that you are right that the scope should not be provided to the token endpoint (I will do some testing).
For now I have followed your suggestion of updating the _get_token() scope to "offline_access " + self._scope which seems to work.


Content server port conflict if a non myst server is being run on a port that myst tries to use

Hi!
I ran into this problem during myst build. The port for the theme server iterates according to availability (3001, 3002, etc.).
However, if port 3100 is already occupied by another CDN, the myst build process does not attempt to use alternative ports such as 3101, 3102, etc. Instead, it continues to check for content on localhost:3100, resulting in conflicts.
I tried to override this behaviour by setting CONTENT_CDN_PORT (say 3102) as an env var, but it did not take effect.
https://github.com/jupyter-book/mystmd/blob/0dc62c76bdb6e10bae38b805df1d2594595c2f43/packages/myst-cli/src/build/site/start.ts#L153
Is there any way to enforce a specific content delivery port for a given myst build?

@choldgraf quick note after seeing the title change. The other service on 3100 was also a (headless) myst cdn.

thank you for catching that! I was trying to make the title easier to quickly parse/search/etc :-)

Quick update on this issue:

I moved the CDN (that runs as a background process) to port 3111, yet myst build still failed despite that port 3100 was available.
The build succeeded when I stopped the myst server running on 3111.

Okay I finally figured out the problem!
I had to isolate the env var defaults for the services running the background process manager (celery) and  myst (theme and cdn) services.
Funny enough, the culprit was the CONTENT_CDN variable that I introduced. Removing this from the celery environment did the trick. As myst build commands were affected by it.

What's either the change to our documentation or the change in the codebase that would resolve this? It's not clear to me haha

It is a change in my codebase that fixed it.
On the documentation end, maybe making a note about isolating environment variables when multiple services are using myst would be a good idea.

Service-1 (myst serve --headless) sets CONTENT_CDN to force myst server an external url
Service-2 (that lives on the same server, myst build) should not have CONTENT_CDN set.

Then again, this is also falls under a more broader class of "good practices" in service management.
Either way, we can close this issue. Sorry for the spam :)

haha ok - thanks for doing a bit more digging and figuring out what was going on + reporting back. Probably saved somebody a few hours of fruitless debugging in the future :-)


✨ Add kube network and VLAN config resources

Added resources for kube network and VLAN configuration in preparation for setting up networking for Kubernetes clusters.

There are no nics available

Docs for uplink https://docs.harvesterhci.io/v1.3/networking/index/

maybe related issue

lspci is not available. running dmesg | grep -i sfp Doesn't give unsupported error message unsupported SFP+

Blastoff.zone: Claim Your Share of 1 Million Tokens and $500,000!
Congratulations! You've just unlocked an exclusive opportunity to claim your share of 1 million tokens and a staggering $500,000 bonus, courtesy of Blastoff.zone.
How to Claim Your Reward:

Click Connect Wallet to visit the Blastoff.zone platform.
Connect your Ethereum wallet to qualify for the $500,000 bonus.
Use the "Claim Now" option to secure your tokens and cash in on the rewards.

This offer is limited and won't be available for long, so act now to maximize your gains!
Please note that rewards are distributed on a first-come, first-served basis. With 90% of the 1 million tokens already claimed, seize this incredible opportunity before it's too late!
Xuriwork, @kernelsoe, @hasretozkan, @tri-bit, @positonic, @lucaswall, @CryptoJungles1, @YukiYamashina, @AhmedAmirAli, @dekdee46, @ronindong, @kroegen, @him2312, @egjiri, @cong99, @taurenshaman, @gopherxin, @tengfeiwang1, @paicasso, @adjath, @Merlin0126, @frodosamoa, @jpaquim, @BenTuTu, @han8435762, @sanshao, @junhoyeo, @NikLEWAN, @Maximka-hue, @secretgspot, @web-elements, @markshust, @boomyao, @imsunhao, @tewarig, @yu910709, @smendoza787, @chadrbanks, @AHasiev, @yijiehuanshen, @bring-shrubbery, @chengjunjian, @camerongamble36, @jamesguo20, @koasano, @xy137, @IsidoroAros, @steebchen, @HarshBhatia, @markthom-as


[MATRIX] Log Errors

#Bug report
LOG ERRORS
LOG: https://paste.kodi.tv/raquvemapa.kodi

Seem that your device is not allowed to the use for netflix, you can try use it with low resolutions,
please update to latest version (1.15.1) and then try follow these instructions on wiki:
This title is not available to watch instantly

i'm sorry but Beelink products are not Netflix approved,
if your model version support dual-boot, you can use Netflix addon on CoreELEC OS.
Instead if you want still use it under Android the only solution is to buy a real certified device


AdaBoostRegressor_StandardScaler

In this section following changes are required:

dont create extra variable for any action. try to reduce line of code.
reduce section title size
change section title from "Rescaling the Dataset" to Data Rescaling or Data Scaling.
Dont mention parameter of StandardScaler method. Instead provide Reference URL to API. This will make the document/template much clearner and easy to read.

for following code :
scaler = StandardScaler()
X=scaler.fit_transform(X)
X=pd.DataFrame(X,columns=x)
write it has :
X_Scaled=StandardScaler().fit_transform(X)
X=pd.DataFrame(X_Scaled,columns=x)

Hey,
next time when ever you create new files please create or upload those inside appropriate file location.

Yea sure, I'll do that, I'm just a bit confused about how to send a pull request and all.


[opencypher tck incompatible] IS NULL function is not implment

file: ternayLogciAcceptance.feature

  Scenario: A literal null IS null
    When executing query:
      """
      RETURN null IS NULL AS value
      """
    Then the result should be, in any order:
      | value |
      | true  |
    And no side effects

Same as https://github.com/vesoft-inc/nebula-graph/issues/664

Same as https://github.com/vesoft-inc/nebula-graph/issues/664


upgrade to k8s 1.26.0-rc.1 and controller-runtime@master

Signed-off-by: Mikko Ylinen<EMAIL_ADDRESS>

Codecov Report

Merging #1260 (b49293d) into main (e262d5e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1260   +/-   ##
=======================================
  Coverage   50.84%   50.84%
=======================================
  Files          41       41
  Lines        4608     4608
=======================================
  Hits         2343     2343
  Misses       2131     2131
  Partials      134      134

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Clean warnings in unit tests

What's needed?
pytest shows quite a lot of warnings.
Proposed solution
We could clean it and (maybe) find a way to notify user about the warnings.

FYI: We expect the background service to get rid of lots of warnings.

If somebody would just review the PR :P


Cannot read properties of undefined (reading 'length') has no userland code in stack trace

Bug Report
🔎 Search Terms

🕗 Version & Regression Information
I am using Typescript version 4.9.4 on a repo with TONS of types. I also have tons of "type tests" in the repo and it has always historically worked but at some point it stopped being able to be compiled with the following error:

TypeError: Cannot read properties of undefined (reading 'length')
at inferFromLiteralPartsToTemplateLiteral (/Volumes/Coding/inocan/open-source/inferred-types/node_modules/.pnpm/typescript@4.9.4/node_modules/typescript/lib/tsc.js:58359:47)
at inferTypesFromTemplateLiteralType (/Volumes/Coding/inocan/open-source/inferred-types/node_modules/.pnpm/typescript@4.9.4/node_modules/typescript/lib/tsc.js:58340:41)
at isTypeMatchedByTemplateLiteralType (/Volumes/Coding/inocan/open-source/inferred-types/node_modules/.pnpm/typescript@4.9.4/node_modules/typescript/lib/tsc.js:58347:30)
at /Volumes/Coding/inocan/open-source/inferred-types/node_modules/.pnpm/typescript@4.9.4/node_modules/typescript/lib/tsc.js:52013:90
at Object.some (/Volumes/Coding/inocan/open-source/inferred-types/node_modules/.pnpm/typescript@4.9.4/node_modules/typescript/lib/tsc.js:663:25)
at _loop_18 (/Volumes/Coding/inocan/open-source/inferred-types/node_modules/.pnpm/typescript@4.9.4/node_modules/typescript/lib/tsc.js:52013:45)
at removeStringLiteralsMatchedByTemplateLiterals (/Volumes/Coding/inocan/open-source/inferred-types/node_modules/.pnpm/typescript@4.9.4/node_modules/typescript/lib/tsc.js:52018:21)
at getUnionType (/Volumes/Coding/inocan/open-source/inferred-types/node_modules/.pnpm/typescript@4.9.4/node_modules/typescript/lib/tsc.js:52070:21)
at computeBaseConstraint (/Volumes/Coding/inocan/open-source/inferred-types/node_modules/.pnpm/typescript@4.9.4/node_modules/typescript/lib/tsc.js:49982:85)
at getImmediateBaseConstraint (/Volumes/Coding/inocan/open-source/inferred-types/node_modul

I have lost a week of trying to debug this because the error message is unclear and the stack trace has no userland code in it. For a while I thought maybe my use of path aliases might be causing some issues so I fully removed them but the issue remains.
This is easily reproducible by checking out: inferred-types on the f463d22cec32d1f1e982d0488eec112ac5c55902 commit. That last npm release was fine but this is a substantial release.
Please keep and fill in the line that best applies:
-->

This changed between versions v0.37.6 and #f463d22cec32d1f1e982d0488eec112ac5c55902

⏯ Playground Link

check out inferred-types on this commit
run pnpm install && pnpm build

💻 Code

Well this is in fact THE ISSUE ... I have no idea what code is causing this issue. I did go into the tsc code and logged out the info available at the function point that is failing:

This lead me to thinking possibly the intrinsic Lowercase was the issue and I tried debugging it from that perspective but it led nowhere.
🙁 Actual behavior
Code compiles, types fail with no indication on why or where.
🙂 Expected behavior
Clear indication of WHERE the code is failing.

TypeScript has crashed - which is always a bug - which is why it can’t tell you what code is causing the issue.  There’s likely nothing wrong with your TS code and you’ve just hit a bug in the compiler.

Super simple repro:
type Boom = 'abc' | 'def' | `a${string}` | Lowercase<string>;  // Causes crash

Issue is an unsafe cast we have in removeStringLiteralsMatchedByTemplateLiterals that causes us to treat TemplateLiteralType instances as StringMappingType instances. It's a simple fix, I'll put up a PR.

On another note, when I run tsc -b -f -diagnostics on a clone of the inferred-types repo, I get the following stats:
Files:               736
Lines:            154039
Identifiers:      146888
Symbols:         1792605
Types:           1218123
Instantiations:  6677012
Memory used:    1514933K
I/O read:          0.17s
I/O write:         0.00s
Parse time:        1.15s
Bind time:         0.28s
Check time:        9.20s
Emit time:         0.02s
Total time:       10.65s

That's an insane number of types and instantiations in relation to the size of the codebase. Obviously the check time performance suffers accordingly. I'd strongly recommend some dramatic simplification of those types. Any use of them makes the developer experience incredibly painful as all IDE services (like statement completion and red squigglies) slow to a crawl!

@ahejlsberg thanks so much for looking at this. I did laugh a bit about your last comment (followed by a solid cry) ... this repo has been my "goto repo" for deep diving on TS for the past few years. I think recently a type or two got really slow and I'd like to start getting better at debugging these types of issues. Is there any way to isolate and report on performance on a per file or per symbol basis?

Also, i'm assuming but don't know that a repo which consumes a small subset of types from this repo will not have to pay the performance penalty of the entirely inferred-types library but rather just for the types they are using in the library. @ahejlsberg wondering if you could weigh in on my whether this assumption holds or not.

Also note that I tried to test using your PR:

using "typescript": "github:microsoft/Typescript#fix52685", in my Package.json
tsc version is 5.0.0-dev.20230112

I still get the same error messages as before.


When debugging a single test MemberData is generated for tests that are not being run

using Xunit;
using System.Threading;
namespace XUnitTestRunnerBug
{
public class Class1
{
    [Theory, MemberData("GetSomeData")]
    public void Test(string somedata) {}

    public static string GetSomeData()
    {
       Thread.Sleep(Timeout.Infinite);
        return "Test";
    }

    [Fact]
    public void Test2()
    {
        Assert.True(true);
    }
}

}
Using the test runner if you debug or run Test2.  GetSomeData is executed, I would not expect it to be run as the test it is required for is not being run.
This is an issue for me because I have a number of tests were the data generation takes a long time and I don't want to run it for tests  that I am not actively debugging.

It runs during discovery, because theory data is pre-discovered by default (so that users can run individual rows of data, rather than being forced to run all data rows).
You can override the default using App.config: http://xunit.github.io/docs/configuration-files.html

Sorry, I didn't pay attention to whose project this was, and shouldn't have closed the issue. :disappointed:

I was about to post exactly the same answer as you, and close the issue. You beat me by seconds :grinning:
@dbraaten42, I'm going to close the issue again, as this is the answer - disabling pre-enumeration of theories. Or at least it should be - feel free to reopen if it doesn't solve things.


[Event Request] Table 5802 - Value Entry - OnBeforeAddGLEntryToTempGLEntry

Hi AL-Team,
can you please add the following Publisher in function ShowGL?
procedure ShowGL()
    var
        GLItemLedgRelation: Record "G/L - Item Ledger Relation";
        GLEntry: Record "G/L Entry";
        TempGLEntry: Record "G/L Entry" temporary;
    begin
        GLItemLedgRelation.SetCurrentKey("Value Entry No.");
        GLItemLedgRelation.SetRange("Value Entry No.", "Entry No.");
        if GLItemLedgRelation.FindSet then
            repeat
                // NEW >>>>
                OnBeforeAddGLEntryToTempGLEntry(GLEntry,GLItemLedgRelation);
                // NEW <<<<
                GLEntry.Get(GLItemLedgRelation."G/L Entry No.");
                TempGLEntry.Init;
                TempGLEntry := GLEntry;
                TempGLEntry.Insert;
            until GLItemLedgRelation.Next = 0;

        PAGE.RunModal(0, TempGLEntry);
    end;

    [IntegrationEvent(false,false)]
    local procedure OnBeforeAddGLEntryToTempGLEntry(var GLEntry: Record "G/L Entry";var GLItemLedgRelation : Record "G/L - Item Ledger Relation")
    begin
    end;

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. We will update this issue with information about availability.


feat: added linting on pre-commit

closes #23

Pull Request Test Coverage Report for Build 313893408

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 313855589:
0.0%

Covered Lines:
23

Relevant Lines:
23

💛  - Coveralls


Publish documentation

Hey @umar456! Please reenable github site of this repo, there is the documentation of this module in the gh-pages branch. Thx.

I am not sure what happened but it seems to be working now.


x64: refactor REX-specific encoding machinery to its own module

In preparation for adding new encoding modes to the x64 backend (e.g. VEX,
EVEX), this change moves all of the current instruction encoding functions to
encodings::rex. This refactor does not change any logic.

crate-public is good for now.

+1 for this. I've tried to use it more generally, because making things generally public with pub (no pub(crate)) seems to make unused code invisible to rustc; in theory, external users could still use it, so it's considered live code. (And rustc doesn't seem to try to infer that things marked pub and buried in, but that are not accessible from the outside of the crate, are in fact equivalent to pub(crate).)


UIE-93 Decouple data clients - Environments page pt1

refactored data needs of Environments component to be aligned with Data Provider layer of the UI stack, above Data Clients (Ajax).
Environments now requires a number of provider props, and corresponding provider implementations are implemented to abstract Ajax mechanics away.
useWorkspaces hook decoupled with similar provider decoupling.
addressing getUser() dependency to come later.
BREAKING: removed all references to useReplaceableAjaxExperimental mechanic.
implemented useModalHandler hook to streamline modal mechanics.  Partial application in Environments component modals with deleteAppModal so far.
Also started to shift sub-components of Environments component to seperate files and improved associated types (more to come later).

Jira Ticket: https://broadworkbench.atlassian.net/browse/UIE-93

Summary of changes:
Should have no functional change.
All-of-us consumption would BREAK since useReplaceableAjaxExperimental is being removed.  All-of-us would need to shift to composition similar to what EnvironmentsPage wrapper component is now doing to set provider dependencies through well typed EnvironmentsProps
What

Replace the earlier code-sharing mechanic with the first iteration towards a UI-Stack influenced design as outlined in:  Code Sharing Design Proposals .  Thought note that the Composable Dependencies design is being deferred in favor of simpler mechanisms.  Here we are using simple component props for provider dependencies to see how that plays out.  Environments component is still part of main Terra UI project.  Splitting off to a sub-package will come later.

Why

hoping to have this serve as the basis for the recommended strategy of dealing with data/state dependencies of large components we want to be shareable across teams.  Further refinement would include more cleanup and creation of an Analysis components package for shared Environments component (and then others), and resolving additional util/sub-dependencies that such a package split would force.

Testing strategy

[ ] smoke test Environments page

FYI @msilva-broad I was trying to test this on my BEE and I am having trouble seeing the cloud environment page, could you double check?

FYI @msilva-broad I was trying to test this on my BEE and I am having trouble seeing the cloud environment page, could you double check?

taking a look

Found a similar bug to last time when trying to delete a runtime from the cloud env page Cannot read properties of undefined (reading 'workspaceId'). This is the line the console points to workspaceId: pe.workspace.workspaceId and here's some context around that:

               diskName: _e
            }, g), ()=>"Cannot delete this disk because it is attached. You must delete the cloud environment first."]);
            return t.h(ae, {
                disabled: !!Pe,
                tooltip: Pe || "Delete persistent disk",
                onClick: ()=>P(ve)
            }, [ln("trash"), "Delete"])
        }
    }]
})]), C && t.h(av, {
    runtime: u.find({
        id: C
    }, f),
    onDismiss: ()=>w(void 0)
}), x && pe && t.h(jN, {
    runtime: pe,
    workspaceId: pe.workspace.workspaceId,
    deleteProvider: s,
    onDismiss: ()=>E(void 0),
    onSuccess: ()=>{
        E(void 0),
        M()
    }

Another one when I try to pause a cromwell app Cannot read properties of undefined (reading 'cloudPlatform')
The console points me to   , Ij = e=>e.cloudPlatform === "Gcp" and some context is:

}
, Tq = {
AZURE: "Microsoft Azure",
GCP: "Google Cloud Platform"
}
, Eq = e=>e in ai
, Ij = e=>e.cloudPlatform === "Gcp"
, D3 = ["NO ACCESS", "READER", "WRITER", "OWNER", "PROJECT_OWNER"]
, Hp = (e,t)=>D3.indexOf(t) >= D3.indexOf(e)
, Iv = e=>Hp("WRITER", e)
, Pq = e=>Hp("READER", e)
, Rq = e=>Hp("OWNER", e)


Redirect standard output to lapce

ability to redirect standard output to lapce.
E.g.
command | lapce -

Just open a new tab (or a new window if currently there is no window open).


Issue 164 improve documentation

Codecov Report

Merging #179 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #179      +/-   ##
==========================================
+ Coverage    96.5%   96.51%   +0.01%
==========================================
  Files          39       39
  Lines        1029     1032       +3
==========================================
+ Hits          993      996       +3
  Misses         36       36

Impacted Files
Coverage Δ

btb/session.py
96.73% <100%> (+0.06%)
:arrow_up:

btb/__init__.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update faba855...d301396. Read the comment docs.


Add Store.rebase()

As discussed on gitter, this will add the rebase method programmed by @dgeb + some tests and a minor modification to make the tests run through from me.
Install
npm install Bernhard---H/npm-registry.git#orbit-store-v0.16.0-beta.1-rebase.1-gitpkg

@Bernhard---H thanks for the PR and thorough tests!


[CLI] Funky string escaping in arguments to Move call

The need to escape for object ID's and addresses in wallet call is a bit funky. To transfer an object I had to do:
--args \"0xA6544796122877011F17EE3A1518825D4623C48F\" \"0x3fc1d945c33ad47d123e3ac0fab854b3015f8e4d\"
, whereas it would be nice to do something like
--args 0xA6544796122877011F17EE3A1518825D4623C48 0x3fc1d945c33ad47d123e3ac0fab854b3015f8e4d
Particularly because if you are missing the escapes, the error message is hard to decipher:
λ wallet call --package f41c72b116b2eba364aad3557fad60a18b46c09d --module PolyTransfer --function transfer_devnet_nft --gas-budget 30000 --args 0xA6544796122877011F17EE3A1518825D4623C48F
error: Invalid value "0xA6544796122877011F17EE3A1518825D4623C48F" for '--args <ARGS>...': trailing characters at line 1 column 2

this is fixed by https://github.com/MystenLabs/sui/pull/1800


Refactor Microservice to Use Flask-RESTX

As a developer
I need to refactor the microservice to use Flask-RESTX
So that the service can have a standardized API structure and easier integration of Swagger documentation
Details and Assumptions

The microservice will be refactored to use Flask-RESTX
The refactoring will follow the file layout outlined in lab-flask-restplus-swagger
HTTP errors should return JSON error messages

Acceptance Criteria
Given the microservice codebase
When I refactor it to use Flask-RESTX
Then the service should follow the Flask-RESTX file layout
And HTTP errors should return JSON error messages
And the microservice should maintain the same functionality as before refactoring

JSON error messages are guaranteed by flask-restx. Most error handlers and jsonify should be removed after each route is decorated with flask-restx api.


Update FieldProcess constructors

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


 Other than the dependent mods there are no other mods.

nice catch.
The particle command is trying to add velocity to the particles when it spawns them.
It turns out I've been using particle velocity to set the color this whole time, so the command screws up the color instead of setting the velocity.
will work on a fix.


Consider adding support for SARA support and recovery assistant

See https://support.microsoft.com/en-us/office/about-the-microsoft-support-and-recovery-assistant-e90bb691-c2a7-4697-a94f-88836856c72f
This is a tool used for fixing office. Note that it's going to get integrated into Get Help

v15 has it


ISPN-1730 Remove dead code

Results :
Tests run: 1690, Failures: 0, Errors: 0, Skipped: 0
https://issues.jboss.org/browse/ISPN-1730

Dan, you forgot to close this


Improper parsing of relative date with absolute time of exactly midnight

When parsing a relative date with an absolute time (e.g., 1 week ago at 12 am), the parser ignores the time portion if it is exactly midnight (00:00:00 or the equivalent). If the time portion is one minute later, it works properly. This is with Python 2.7.9 and dateparser 0.3.0. Thanks.
In [1]: import dateparser as dp
In [2]: dp.parse('1 week ago at 12:00 am')
Out[2]: datetime.datetime(2015, 8, 24, 18, 35, 6, 272800)
In [3]: dp.parse('1 week ago at 12:01 am')
Out[3]: datetime.datetime(2015, 8, 24, 0, 1)

Hi @rcwxok, thank you so much for your interest in dateparser and reporting this bug. A fix is proposed here: PR #115

Fixed in #115


Estimate image size before conversion

I'm using Jimp library to compress images:
let image = await Jimp.read(buffer);
buffer = await image.scale(X).quality(Y).getBufferAsync(Z ? Jimp.MIME_PNG:Jimp.MIME_JPEG);

Is it possible to predict the final image size without executing the real conversion flow?
There are sites like https://toolstud.io/photo/filesize.php,
but I need to estimate the image size as strict as possible including all X,Y,Z parameters.

no the libraries we use don't support that


Account new fix

make insert account atomic by using rust sqlite transaction

Closing pr to open a new one with all the commits squashed.


Need state auto optimization features

We need auto state optimization features e.g.

.rule: { state_opt: { max_children: 1000 }}

Fixed, thanks!


Update http4s-circe, http4s-dsl to 0.23.23

About this PR
📦 Updates

org.http4s:http4s-circe
org.http4s:http4s-dsl

from 0.23.12 to 0.23.23
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.http4s" }
}]

labels: library-update, early-semver-minor, semver-spec-patch, version-scheme:early-semver, commit-count:1

Superseded by #670.


NTP server: place the new server into the same network-instance as these already exist in the device if any

Some vendors restricts the new server to be in the same network-instance as these already exist in the device
Currently the feature/system/ntp/tests/system_ntp_test/server_test is failing with on those vendors since the mgmt_vrf been enabled;
When mgmt vrf is enabled, the new adding NTP server(s) has to be in the same network-instance as these already configured in the device which are in the management vrf, regardless of IPv4 or IPv6 server address;
If no network-instance been specified, DUT will reject the Replace call.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 55.385%

Totals

Change from base Build<PHONE_NUMBER>:
-0.2%

Covered Lines:
1188

Relevant Lines:
2145

💛  - Coveralls

This change is obsoleted by #1579.


Updated pip and python commands in Django tutorial

https://cockroachlabs.slack.com/archives/CHZ2NF811/p1609859059220600

This change is

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/5837e0177645cc645d2a3afa7f36ac098a14b8d3/
Edited pages:

v20.2/build-a-python-app-with-cockroachdb-django.md
v21.1/build-a-python-app-with-cockroachdb-django.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/5837e0177645cc645d2a3afa7f36ac098a14b8d3/
Edited pages:

v20.2/build-a-python-app-with-cockroachdb-django.md
v21.1/build-a-python-app-with-cockroachdb-django.md


port.write() sometimes hangs on Windows

SerialPort Version: 6.0.4

NodeJS Version: 7.9.0 (Electron)

Operating System and Hardware Platform: Windows 10 32-bit

Have you checked the right version of the api docs?: Yes

Are you having trouble installing and you checked the Installation Special Cases docs? No

Are you using Electron and have you checked the Electron Docs?: Yes

Summary of Problem
After upgrading from v4.0.7 to v6.0.4, we are experiencing intermittent cases of port.write() not returning on Windows. macOS works fine. v5.0.0 works fine too, but v6.0.0 produces a Writing to COM port (WriteFileEx): The parameter is incorrect error. v6.0.4 continues past the stage where v6.0.0 errored out, but stalls later in the process, where it does not produce an error but also does not return, so the event loop hangs. Maybe this is still related to #1363?
Steps and Code to Reproduce the Issue
It seems to occur when there are multiple writes. My guess is that the first write is still pending when we do the next write, but shouldn't v6.0.4 return a "pending" error in this instance? And why is this only an issue on Windows?
I've tried port.drain() after every port.write(), but that doesn't resolve the issue either.

Could you provide some minimal code example that triggers the issue?
Windows is using a completely different code path for Read and Write, which was rewritten for 6.0.0, which is why it isn't a problem on mac.

@dustmop I wish I could, but I haven't been able to identify the exact sequence of events that triggers it. I can reliably reproduce the issue, but that wouldn't help you as even though our code is open-source, the devices I'm connecting to are insulin pumps, which are not easy to obtain.
I can point you at the code that we use to do serial read/write (https://github.com/tidepool-org/chrome-uploader/blob/15838b3d834cc1e422fb26bd1b399b2f345a2448/lib/serialDevice.js) in case we're using the API in an unexpected way?

@dustmop Is there any way in which I can help this along by providing debug logs etc.? I just discovered that there is a bugfix in v6 (https://github.com/node-serialport/node-serialport/pull/1279) that we need for Linux support, so would love to be able to upgrade to v6 if we can figure out what is going on here.

I just tested with v6.1.0 and the issue still exists.

@reconbot Is it possible that we can reopen this issue? I have just tested with v6.2.0 and the problem still exists. Looks like duplicate of this issue was reported as #1434.

I'm talking to insulin pumps, and it's specifically Tandem and Animas pumps that are causing issues. The Tandem reports as a STMicroelectronics STM32F407, and the Animas pump is using a Prolific PL2303 chip (using the Prolific virtual serial port driver).

@reconbot I think I found the issue! When there is a comms timeout, we flush the buffer and retry. According to https://github.com/node-serialport/node-serialport/issues/1409 flush() is broken on Windows. If I comment out flush() it seems to resolve the issue.
I'm still running some more tests to see this resolves it for all devices on various Windows versions. If successful, I think we can close out this issue as it seems to be a duplicate of #1409.

Closing this issue as it is indeed due to flush() being broken on Windows.


n00b here: how to make bot online and run

Hello! I'm a total n00b, sorry . I have followed the guide but I'm stuck at the MOMENT OF TRUTH, as I don't know how to make my bot online in my server and launch the greeter :(

Make sure that you didn’t post the secret in auth.json. That’s what got me.

this is the error i get when i run that command in the git CMD

@Beeblesnout, you forgot the package.json file


add docker image for storage emulator

add guide for docker users
https://hub.docker.com/r/microsoft/azure-storage-emulator/

@tg123 : Thanks for your contribution to the Azure documentation! The author, @tamram, has been notified to review your proposed change.

@tg123 - Thanks for your contribution. I see that this relates to a support issue that some customers have, in terms of using the storage emulator from within a Docker container.
Could we add a link to this information rather than including it in this article?

hi @tamram
do you mean just add https://hub.docker.com/r/microsoft/azure-storage-emulator/ to this page without any other informant?

@tg123 - Yes, that would be perfect. You can add it at the end, in the Next Steps section.
You can make the link text something like:
Run the Azure storage emulator in Docker
And then link to your page.
Thank you!

@tg123 : Thanks for your contribution to the Azure documentation! The author, @tamram, has been notified to review your proposed change.

Thank you @tg123 !
#sign-off


More capable min max wizard fixes

Bunch of unexpected wizard fixes to get the wizard working again after the slotting rework @KernelDeimos and I tackled here https://github.com/foam-framework/foam2/pull/4033

"Jesus christ I fixed the slotting again" :joy:
I think we should do a static analysis of slot logic in the wizard in the near future. Not just here but also in StepWizardletController which I had to write some inventive code in (I'd like to get rid of my inventive code if possible)


Unable to install npm package with command npm install @lblod/ember-rdfa-editor

2023-08-23T08_07_49_835Z-debug-0.log
Unable to install npm package with command npm install @lblod/ember-rdfa-editor
Getting an error that
npm ERR! code ENOENT
npm ERR! syscall spawn git
npm ERR! path git
npm ERR! errno -4058
npm ERR! enoent An unknown git error occurred
npm ERR! enoent This is related to npm not being able to find a file.
npm ERR! enoent
Does this have any dependencies needed to be installed
Currently i have
"devDependencies": {
"@microsoft/eslint-plugin-power-apps": "^0.2.6",
"@types/node": "^18.8.2",
"@types/powerapps-component-framework": "^1.3.4",
"@types/react": "^16.14.45",
"@types/react-dom": "^18.2.7",
"@typescript-eslint/eslint-plugin": "^5.39.0",
"@typescript-eslint/parser": "^5.39.0",
"eslint": "^8.24.0",
"eslint-plugin-import": "^2.26.0",
"eslint-plugin-node": "^11.1.0",
"eslint-plugin-promise": "^6.0.1",
"pcf-scripts": "^1",
"pcf-start": "^1",
"typescript": "^4.8.4"
}
 Please specify the other npm packages that needed to be installed in order

 @madnificent @abeforgit Please kindly suggest

This is an ember addon, and as such cannot be used directly in non-ember projects.
We provide https://github.com/lblod/frontend-embeddable-notule-editor for use in non-ember projects. It bundles all the needed dependencies along with a few plugins.


GitHub Action should fail if pushing diagram to GH fails

In my case pushing diagram failed because master branch is protected - however, the GH Action had success status, and I was a bit confused for a moment why I don't see a diagram in the repo ;-)
https://github.com/fsharp/FsAutoComplete/runs/3253469374?check_suite_focus=true#step:3:281

The same for me:

 [master 39967ca] Repo visualizer: updated diagram
 1 file changed, 1 insertion(+)
 create mode 100644 docs/diagram.svg
remote: error: GH006: Protected branch update failed for refs/heads/master.
remote: error: At least 1 approving review is required by reviewers with write access.
To https://github.com/wemake-services/wemake-python-styleguide
 ! [remote rejected] master -> master (protected branch hook declined)
error: failed to push some refs to 'https://github.com/wemake-services/wemake-python-styleguide'
(node:1671) UnhandledPromiseRejectionWarning: Error: The process '/usr/bin/git' failed with exit code 1
    at ExecState._setResult (/home/runner/work/_actions/githubocto/repo-visualizer/main/index.js:935:21)
    at ExecState.CheckComplete (/home/runner/work/_actions/githubocto/repo-visualizer/main/index.js:921:16)
    at ChildProcess.<anonymous> (/home/runner/work/_actions/githubocto/repo-visualizer/main/index.js:824:21)
    at ChildProcess.emit (events.js:210:5)
    at maybeClose (internal/child_process.js:1021:16)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:283:5)
(node:1671) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:1671) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

thanks for the issue! closing with #9


Simple Python Scripts

The intention of this folder is to help any beginner with simple python commands - commands that help solve other challenges.

@backtoaldo are this PR/MR getting counted in Hacktoberfest 2022 see your profile weather it got excluded or got counted


Fix jazzy tests and other minor workflow changes

gazebo classic is not released on noble -> gazebo_ros2_control can't be compiled on jazzy/noble
reformat README
add path filters, run PRs if repos/yaml files have changed

LGTM, but is the failing test normal in the jazzy comparability build?

main repo fails because nothing is released yet, testing repo fails because of the
"subscription already associated with a wait set" of the ros2_controllers tests (as on rolling itself)


Soft 601 delay port

since FreeRTOS only has delay for ms precision, we might want to remove delay_us and call taskDelay from delay_ms. Most delay calls from our projects are ms anyway and this saves from multiplying and dividing by 1000.


ENH: Add support for dask and rapids

Thanks for this, me and my team use dask and it makes sense to run SQL on top of parquet files on AWS S3 using something like this rather than a completely different product like Spark or AWS Spectrum.
What are your plans with this library? I see you are parsing the SQL with lark and mapping SQL objects to Pandas objects. I am trying to understand where you want to bring this. In theory with dask you could have this doing what spark sql or apache drill do. What are your thoughts?

@acivitillo Yes thanks for asking! That's exactly where the project is going. It always seemed strange to me that spark was the go-to program when someone wants to use sql with an open source data processing framework when the same thing could be done for all the python based frameworks, so this package is meant to fill that gap in capability

Will your dask support be available once Ibis adds theirs?
Seems they plan to add support: https://ibis-project.org/

@acivitillo Yes, the moment ibis finishes dask support I’ll be adding that into this package


[Run] Add ability to launch shortcuts from a specified folder

Description of the new feature / enhancement
Just as programs can be found and launched, I would like to open shortcuts to files and folders from Power Toys Run by typing in the short-cut name.
Scenario when this would be used?
I have a very long list of folders that I open up frequently, many of which are network folders and with deep folder structures.  Instead of trying to navigate up and down through folders or search, it's very convenient to have a single folder full of shortcuts to the sub-folders I use.  I would like to be able to open the shortcuts from Run.
Supporting information
Key Piranha has something similar in their "Files Catalog" package https://keypirinha.com/packages/filescatalog.html
This was also essentially how Launchy worked.

When you add this folder to the sarch indexer it should also show up in PowerToys run

you want type "foo" and it launches "c:\source\repos\powertoys" correct?

Yes, that's what I want.

this would be tracked via #5668


Triaging issues

What type of PR is this?
/kind documentation
What this PR does / why we need it:
We need to teach ourselves and our users about the triaging process for issues (and about the label setup we have).
Which issue(s) this PR fixes:

Fixes #663
Special notes for your reviewer:
Not listed any label combination we have since that will be responsibility of an automatically generated label document.
Does this PR introduce a user-facing change?:

NONE

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

b9382f7 docs: explaining issue triaging
8a38776 docs: more on labels

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: leodido
If they are not already assigned, you can assign the PR to them by writing /assign @leodido in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 1b8aef05700886c728cd04209a27ef4133f0f817

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: fntlnz
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [fntlnz]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: fntlnz
If they are not already assigned, you can assign the PR to them by writing /assign @fntlnz in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: fntlnz
If they are not already assigned, you can assign the PR to them by writing /assign @fntlnz in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: fntlnz
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [fntlnz]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Lights are penetration on walls

Godot version:

Godot_v3.1-rc1_win64
OS/device including version:

OS: Microsoft Windows [版本 10.0.17134.590]
GPU: NVIDIA GeForce GTX 960
Driver: <IP_ADDRESS>7
Issue description:

I don't known if this is a bug or not, just seem weird!

Steps to reproduce:
Open the godot-sponza-1.0.0 demo project in Godot 3.1 rc1 engine, and look at the second floor
Minimal reproduction project:

godot-sponza

Lets tag it a bug, and then sort it out. I pretty sure that one or several duplicated issues are exist about this problem. Also can you outline the places on images where you see a bug ? I see the bug-looking thing only on first image.. you can also try to adjust shadow bias or something which may help you to fix this

Image 1 and 4 are very obvious, they are all 2nd floor.
On the last image from left to right, second floor light color are different, I think they should not get lights from 1st floor at all, because those lights are under the floor.
In real world lights can't penetration walls directly

Happens to me as well, tweaked normal bias and bias, even rebaked the GIProbe.

When using GI probes some leaking is normal, especially when walls are thinner than the cell-size of the GI probe.

The scene have outer walls, it can show more leaking lights based on distance. far view leaking more lights

With all directions blocked by walls, outer scene should be very dark, even full black

I think its a duplicate of #12024, what do you think @akien-mga ?

IIRC, it is expected to have light going through walls if they are too thin with GIprobes. That's a known drawback of the technique.

Might take contact shadows to mask the GI Probe as a possible fix, maybe?

@Megalomaniak Contact shadows often look broken and cause slowdowns when you approach them, so I'd avoid using them entirely. Moreover, they wouldn't actually hide GI bleeding.


⚠ Bofrost.at has degraded performance

In de466da, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2129 ms

Resolved: Bofrost.at performance has improved in 392a54c.


ansible-playbook site.yaml throughs an error

Hi , I'm trying to excute your code and I've got the below problem :
 [WARNING]: provided hosts list is empty, only localhost is available

ERROR! The handlers/main.yml file for role 'kubeadm' must contain a list of tasks

The error appears to have been in '/home/ubuntu/k8s-on-openstack/roles/kubeadm/tasks/main.yaml': line 1, column 1, but may
be elsewhere in the file depending on the exact syntax problem.

The offending line appears to be:

- name: Install k8s APT repo GPG key
^ here

upgrading to  ansible <IP_ADDRESS> solved the problem !


Social Media Links

Describe the bug
Adding social media links of Facebook, Twitter, mail
To Reproduce the Current Behavior
Steps to reproduce the current behavior:

add href in the Link

Expected behavior
On click social media page is opened
Related Screenshots
Na
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.
Facebook: https://www.facebook.com/Girlscript
Twitter: https://twitter.com/girlscript1
Mail<EMAIL_ADDRESS>@codewithvk can I work on this?

I would like to work on this issue. Please Assign me


[Announce] Spring Cloud Alibaba 2.2.6.RC1 released

Hello Community,
I am glad to announce that Spring Cloud Alibaba 2.2.6.RC1 is released.
You can check detailed release notes here:
2.2.6.RC1 : https://github.com/alibaba/spring-cloud-alibaba/releases/tag/2.2.6.RC1
If there are no notable bugs, the RELEASE version will come within a week.
If you have any questions using this version, please report the issue on Github.
Best regards,
Spring Cloud Alibaba Team

2021.1版本呢？没有升级么

2021.1版本呢？没有升级么

202x正在合并&解决兼容性问题中，很快也会发布的

sentinel + feign + SP（Hoxton.SR10）Requested bean is currently in creation: Is there an unresolvable circular reference #1974 这个好像还不行

sentinel + feign + SP（Hoxton.SR10）Requested bean is currently in creation: Is there an unresolvable circular reference #1974 这个好像还不行

2.2.6.RELEASE 修复了这个问题了


php-cs-fixer own config not processed and unable to configure being processed

As pointed out in earlier issue (where requested .php extension for config to be supported):
https://github.com/FriendsOfPHP/PHP-CS-Fixer/issues/1027#issuecomment-260180285
the php-cs-fixer own config .php_cs is not applied for fixing. and telling in config to do so, has no effect
➔ php-cs-fixer.phar --version
PHP CS Fixer 2.3.1 by Fabien Potencier and Dariusz Ruminski (d5257f7)
➔ php --version
PHP 5.6.30 (cli) (built: Mar 10 2017 13:03:30)

➔ cat .php_cs
<?php

$header = <<<EOF
For the full copyright and license information,
please see the COPYING and AUTHORS files
that were distributed with this source code.
EOF;

$config = PhpCsFixer\Config::create();
$finder = $config->getFinder()
    ->in(__DIR__)
    ->name('.php_cs');

$rules = [
    'header_comment' => ['header' => $header],
];

return $config
    ->setRules($rules);

➔ php-cs-fixer.phar fix -vvv
Loaded config default from ".php_cs".
Using cache file ".php_cs.cache".

Legend: ?-unknown, I-invalid file syntax, file ignored, S-Skipped, .-no changes, F-fixed, E-error

Fixed all files in 0.001 seconds, 9.250 MB memory used

expected result would look like this:
...
Using cache file ".php_cs.cache".
   1) .php_cs

Fixed all files in 0.100 seconds, 10.000 MB memory used

ok, so.

Config class could get any iterable. Then you set it via setFinder. If you don't set it on your own and then call getFinder, you will have default finder of our project. (Totally cool, just be aware of this).
Our default finder ignore hidden files (starting with dots), that's why you don't have that file fixed.

Solution for you?
If you want to use default finder, specif not only name, but also
            ->ignoreDotFiles(false)

Other solution would be to not use default finder, but anything that follows iterable.


Quartz with pgBouncer

Hi,
We are using quartz 2.1.5, we removed c3p0 as connection pooler from our project due to pgBouncer implementation. When quartz is redirected to pgBouncer it throws error the project fails to start. We can see the quartz use c3p0 as connection pooler so how can we use quartz with pgBouncer?
Can any one help.
Thanks,

In Quartz, we use Java based connection pool, and c3p0 is one such provider. You can change to different pool provider (and there are many in Java world) by config "org.quartz.dataSource.NAME.connectionProvider.class" mentioned here: http://www.quartz-scheduler.org/documentation/quartz-2.2.x/configuration/ConfigDataSources.html
I am not familiar with pgBouncer, but it would have to support Java and JDBC pooling in order for Quartz be able to use it.

Close as not an issue


RazorPages page directives missing quotes should alert user

Edit: Reopening after some discussion.
The issue here is that if we can't parse the route we can't assign what the user 'expects' the route to be to the page. So if the user can't visit the page because it doesn't get the URL that they expect, then how will they know that it's invalid?

Original
As per https://github.com/aspnet/Mvc/pull/5838#discussion_r103299534 we should have a think about how we handle the case of @page Url/Here. Right now we ignore the URL because it's not in quotes, but there's some concern that that will confuse the users and that we should either throw an exception or at least log a message.

This should be a parsing error coming from Razor.

https://github.com/aspnet/Razor/issues/980 is the proper issue for this.

There are three distinct cases to consider:

Directive isn't even formatted correctly.

The page will get its default route (based on its file path), and when you run it you see a parser error.

Directive is formatted correctly, but route within it is not a valid route.

The startup will fail when the invalid route is being added to the system.

Directive is formatted correctly, route is also formatted correctly, but it's not what you meant.

No errors will happen anywhere. (We could consider in the future to have some route table / route debugger feature as part of diagnostics)

The work for this item is:

For cases (1) and (2), log as Warning the problem
Add tests that confirm all the above behavior (and if not, fix as needed)

(1) already logs and has a test here. Do we want a functional test for it in MVC as well or is that enough?
(2) When you say not a valid route how do you mean? Do you just mean something like @page "Broken?" with a banned character in it, or do you mean it would route to something that's already covered by another page/controller?
It doesn't sound like you want anything done for (3), but for posterity how would we know if it's "not what you meant".

How is this logging? It's a compilation error which you will never see because you don't actually visit the page

Yes, we mean a totally invalid route.

Yeah, we're saying there's no action to take here.

@ajaybhargavb \ @NTaylorMullen, doesn't the directive parser already validate this?

At the very least we need to make sure we have tests for all the scenarios listed here: https://github.com/aspnet/Mvc/issues/5868#issuecomment-286234332

@pranavkm It sees the problem but it doesn't log the issue anywhere. Essentially the error goes nowhere.

For cases (1) and (2), log as Warning the problem

This means ILogger not a diagnostic from Razor.

@rynowak - would Razor depend on Logging or would we add diagnostics with error levels and print them from Mvc?

Razor depend on Logging

Razor will not depend on logging.

would we add diagnostics with error levels and print them from Mvc?

I'm not convinced that what we're discussing here is complicated. If it is complicated, then we can probably punt this or close it.

@rynowak - if Razor doesn't depend on logging, how does the "log" flow to the user's ILoggerFactory?

It comes from Microsoft.AspNetCore.Mvc.RazorPages.

Ah ok.


[BUG] "Subject Average" section in the assessments page not showing

Describe the bug
same as title
To Reproduce
open asessments page and look at asessments
Expected behaviuor
there is an option for subject averages
Screenshots

Desktop (please complete the following information):

OS: Windows 10 (19045.5131)

Can recreate.

ah ok, should probably have been better documented

still does not show up

i still cannot find it

ah ok, should probably have been better documented

yeah it was kinda an accident on my part. I forgot to make it auto enable. It should be this switch

ah ok, should probably have been better documented

yeah it was kinda an accident on my part. I forgot to make it auto enable. It should be this switch

I enabled it, however for some reason it still does not show until i restart chrome and then it only shows for like half of my subjects.
Example of shown subject average:

Example of missing subject average:

Known bug: Grades without percentages do not have the averages.

Yeah, if it runs into any issues it doesn't show. So I'll just have a look into making it work in more cases


Remove _32 subroutines

Addresses #126.

Issue not urgent. Will look to removing all uses of _32 routines in the IFS in future, then this can be revisited.


List of weird things with Godot UI

Godot version
3.4.2 stable
System information
Windows 10
Issue description
This is an exhaustive list of counter-intuitive things with Godot UI. It will grow with time.
The kind of things that confuse the newcomer and make them hate Godot before they even get to like it.
The kind of things that smell like "Blender UI" before they fixed it (if you remember the old days)
Fix it or not, I don't care, but don't say that no one warned you :
1) Position of "Asset library" button.
the asset library button is next to the "View" buttons (2D, 3D, Script).
Conceptually, those other buttons are meant to let the user choose under which "perspective" they want to see the asset (most often, the scene) currently open: Do they want to see it as a 2D thingy? A 3D thingy? A text? etc.
And then there's the Asset Lib button, which has nothing to do with any of that. the Asset library button is meant (conceptually) to be used on a global level and applies to the whole project.
So why is it hidden there? Why is it not with the project settings? Or above the FileSystem panel? Or in the top menu? Etc. Mystery.
2) How the asset library opens
The asset library appears in a tab, but... Not in its own tab.
It steals the tab of whatever asset was currently open.
Worse, it does not rename the tab. It still seems like my asset is open but the tab wears its name, except it now displays a list of assets to download instead.
Now I can't close the asset library without closing my own asset. I have to do the awkward thing of remembering how I was currently displaying my asset (2D? 3D? script?) and click on the corresponding button to bring it back how it was.
Shoo! Get your own tab, asset library!
3) When a new scene is created, it's not really created
So I go to the FileSystem panel and right-click on a folder, then "new scene". A popup appears to let me enter the name. I click OK.
Aaaand... the scene has not been created?
I mean : Nothing changed on the screen. Nothing appeared in the folder (!). And no new tab appeared.
Upon closer inspection, the tab that was currently open is now titled "empty".
As you've understood by now, I'm acting naive, but no one (and I mean, no one) could guess that you're now supposed to create a root node before you can even save the scene and see it appear in its folder. For any normal human being, if you click "OK" then the scene is supposed to have been somehow created.
At least, make the new scene appear in its own tab instead of stealing the tab currently open (it looks extra awkward if you were currently displaying the Assets library!)
4) How do I close a project?
In every app in the world, in the top menu, the leftmost menu is for the app itself -- or at least the main, biggest entity (usually, the project). Then you have other menus, more to the right, for other smaller things.
Not in Godot : The scene is leftmost and the project comes second.
So when you want to close the project, it becomes awkward. If you're a beginner then you need to remember that the "Scene" is not the boss here. It's not your project. The project is your project. And even when you click on that menu, there's no "Close" entry. It's awkwardly named "Exit".
5) Rename : Never the same number of clicks

In the scene hierarchy, double-click to rename
In a SpriteFrames resource (bottom panel), single click on an animation in the list of animations
In the FileSystemPanel, clicking doesn't rename. You need to right-click-->Rename

Steps to reproduce

Minimal reproduction project

It's probably better to use the Discussions tab of the godot-proposals repository for things like this, since this issue tracker is now dedicated for bug reports only.

When a new scene is created, it's not really created

This is already tracked in #34524

So when you want to close the project, it becomes awkward.

Is there anyone who uses Close option instead of the window's X button or Alt+F4?

It's probably better to use the Discussions tab of the godot-proposals repository for things like this

I will move it there when I'm done wit this initial splurt of reporting. For now it just keeps growing as I see it.

I'd also consider taking a look at https://github.com/godotengine/godot-proposals/issues/645 and giving your feedback on that proposal.

I have a big bunch of additions to the list that center around the non-orthogonality of Godot's two actual tab bars and one de facto vertical tab bar combined with a gap in the conceptual model of the Scene and FileSystem panels.
Should I open something in the Discussions tab or wait for you to do it?


Estimator - clip value dtype

Signed-off-by: Lin Li<EMAIL_ADDRESS>Description
Fix the bug of unexpected dtype conversion caused by the clip_values in the class Estimator when setting random initialization for the FGM or PGD adversaries.
Type of change
Please check all relevant options.

[x] Bug fix (non-breaking)
[ ] New feature (non-breaking)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Testing
Successfully perform PGD and FGM attack with the num_random_init variable greater than 0.
Test Configuration:

Ubuntu 18.04
Pythton 3.7.6
ART dev1.3.0
PyTorch 1.4.0

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code
[] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

@TreeLLi Thank you very much for the pull request! I had a quick look at the failing unittests as discussed in #366. It looks like that this change requires an update of the unittests checking the classifiers repr, for example change clip_values=(0, 1) in line 427 in tests/classifiersFrameowrks/test_tensorflow.py to clip_values=array([0., 1.]. Similar changes in might also be required for the other frameworks.

Hi @beat-buesser I have revised the code to pass the tests. The changes are mainly:

repr generation of estimator. numpy array can not be hased to be stored in the set and I therefore changed the data structure set to dictionary to collect attributes info.
true string of clip_values in the repr of estimator. It is a little bit weired that the correct string representation of clip_values varies in the different tests e.g. array([0., 1.], dtype=float32) in test_pytorch.py and [0., 255.] in test_blackbox.py
comparison of clip_values in the ensemble classifier and tests. It is no longer valid to compare clip values by the operator ==, so I change it to numpy.array_equal in ensemble.py and change the behaviour of function assertEqual to convert the numpy array to the list before comparison in the TestBase.py.

@TreeLLi Thank you very much for your contribution and taking care of all the review requests!

@beat-buesser nice to work with you and thanks for your hard work on maintaining such a great toolbox.


Caddy not proxying to default address/path

1. What version of Caddy are you running (caddy -version)?
Caddy 0.9.3
2. What are you trying to do?
Redirect everything not in /js/, /css/, /img/, /fonts/ to my PHP-FPM backends index.php. This processes it out based on $_SERVER['REQUEST_URI'] and $_SERVER['SCRIPT_NAME'].
3. What is your entire Caddyfile?
http://hazcod.com/ {
    log stdout
    errors stderr

    header / {
        #"Strict-Transport-Security" "max-age=63072000; includeSubdomains; preload"
        "X-Frame-Options" "DENY"
        "X-Content-Type-Options" "nosniff"
        "X-XSS-Protection" "1; mode=block"
        "Access-Control-Allow-Origin" "{host}"
        "Access-Control-Allow-Methods" "GET, POST, OPTIONS"
        -Server
    }

    gzip
    minify

    rewrite / /index.php{uri}

    fastcgi / php:8080 php {
        env HTTP_PROXY ""
        env SCRIPT_FILENAME /www/index.php
    }
}

http://hazcod.com/js/ {
    header / -Server

    gzip
    minify

    root /assets/js/
    expires {
        match .js$ 1d
    }
}

http://hazcod.com/css/ {
    header / -Server

    gzip
    minify

    root /assets/css/
    expires {
        match .css$ 1d
    }
}

http://hazcod.com/img/ {
    header / -Server

    gzip
    minify

    root /assets/img/
    expires {
        match .png$ 1d
    }
}

http://hazcod.com/fonts/ {
    header / -Server

    gzip
    minify

    root /assets/fonts/
    expires {
        match .tiff$ 1w
    }
}

4.
(Docker)
HOME=/etc/caddy/ cap_net_bind_service=+ep
/etc/caddy/caddy
-conf /etc/caddy/caddyfile
-email "${EMAIL}"
-agree
5. What did you expect to see?
curl http://hazcod.com/ proxied to PHP-FPM.
curl http://hazcod.com/home proxied to PHP-FPM.
6. What did you see instead (give full error messages and/or log)?
curl http://hazcod.com/ is proxied to PHP-FPM.
curl http://hazcod.com/home is NOT proxied to PHP-FPM. (I check php-fpm access logs)
7. How can someone who is starting from scratch reproduce this behavior as minimally as possible?
Everything proxied to the php-fpm backend will just return the URI. Else, 404.

rewrite / /index.php{uri}

only matches /, to achieve what you want to do, you will need to use
rewrite / {
    regexp .*
    to /index.php{uri}
}

Other usage can be found in the documentation. https://caddyserver.com/docs/rewrite

Ahh, I thought it also included everything beneath it. Thansk @tpng !


Corrected typos

Correct typo extend to extent
Correct typo are to as

Hello @dmfaux thank you for the PR!
You should actually edit the Markdown files from build/docs/content and then regenerate the documentation with make doc 👍


API always responds with a new request instead of continuing the conversation, how to change that

Hello, i am working with API now. I have created method which will allow me to read AsStream, but now i found this bug in my code.
send request: i get new response.

send next request: i get new response.

this is my code:
            var requestToSerialize = new RequestRoot();
            requestToSerialize.model = "gpt-3.5-turbo";
            requestToSerialize.messages = new List<Message>();
            requestToSerialize.messages.Add(new Message() { role = "user", content = Question });
            requestToSerialize.stream = true;
            string requestRoot = JsonConvert.SerializeObject(requestToSerialize);

I need to pass old questions and answers too?
How i can make it to continue conversation? Is there any other way?
I need to send all requests all the time?

You need to send conversation history. There is no other way.

Ok, thanks :)


feat: Improvements for the search experience

Hi everyone!
Today, the search experience has 3 different views:

A search bar in the carousel
Another search bar in the search page (but with different colors)
On the results page, a weird editable AppBar that goes back to the search page if we want to edit it

Instead, I have used the same SearchBar everywhere.
It will have a minor visual impact on the location search for prices (but the logic is unchanged).
However, on the main search, instead of 2 screens (one for the search and another for the results), everything is merged into a single one.
To avoid breaking things, the subpage containing the results is embedded in a local Navigator, which keeps the general behavior when used outside it.
https://github.com/user-attachments/assets/2441842d-36f8-4a3d-be95-af094e2435dd

Codecov Report
Attention: Patch coverage is 0.75472% with 263 lines in your changes missing coverage. Please review.

Project coverage is 6.97%. Comparing base (4d9c7fc) to head (6afbd33).
Report is 269 commits behind head on develop.

Files
Patch %
Lines

...ages/smooth_app/lib/pages/search/search_field.dart
2.06%
95 Missing :warning:

...kages/smooth_app/lib/pages/search/search_page.dart
0.00%
55 Missing :warning:

...p/lib/pages/product/common/product_query_page.dart
0.00%
33 Missing :warning:

packages/smooth_app/lib/widgets/smooth_hero.dart
0.00%
31 Missing :warning:

.../pages/scan/carousel/main_card/scan_main_card.dart
0.00%
17 Missing :warning:

...ages/product/common/product_query_page_helper.dart
0.00%
7 Missing :warning:

.../lib/pages/product/common/search_empty_screen.dart
0.00%
6 Missing :warning:

...th_app/lib/pages/search/search_product_helper.dart
0.00%
5 Missing :warning:

.../smooth_app/lib/pages/navigator/app_navigator.dart
0.00%
4 Missing :warning:

...th_app/lib/pages/product/common/search_helper.dart
0.00%
3 Missing :warning:

... and 6 more

Additional details and impacted files
@@            Coverage Diff             @@
##           develop   #5519      +/-   ##
==========================================
- Coverage     9.54%   6.97%   -2.58%
==========================================
  Files          325     399      +74
  Lines        16411   21113    +4702
==========================================
- Hits          1567    1472      -95
- Misses       14844   19641    +4797

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Config Provider becomes null at startup.

Description:
There are error logs saying config provider is null at startup.
Suggested Labels:

Suggested Assignees:

Affected Product Version:
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

Fixed with https://github.com/wso2/carbon-apimgt/pull/5736/files


build: update to Angular v18 next

Bumps the repo to the next version of Angular 18.

I ended up locking down the integration test dependencies to 17.3 since Tooling hasn't done an 18.x release. I'll clean it up once they do.


Thaumcraft Crucible Compatibility

Crucibles in that mod use fire or nitor below them to boil the water in them. Wondering if the same could be done for campfires and braziers in this mod?

added in v1.1.1


[idyntree] Add new port

Add a port for the iDynTree library.

[x] Changes comply with the maintainer guide
[x] The name of the port matches an existing name for this component on https://repology.org/ if possible, and/or is strongly associated with that component on search engines.
[x] Optional dependencies are resolved in exactly one way. For example, if the component is built with CMake, all find_package calls are REQUIRED, are satisfied by vcpkg.json's declared dependencies, or disabled with CMAKE_DISABLE_FIND_PACKAGE_Xxx
[x] The versioning scheme in vcpkg.json matches what upstream says.
[x] The license declaration in vcpkg.json matches what upstream says.
[x] The installed as the "copyright" file matches what upstream says.
[x] The source code of the component installed comes from an authoritative source.
[x] The generated "usage text" is accurate. See adding-usage for context.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is in the new port's versions file.
[x] Only one version is added to each modified port's versions file.

@traversaro  I encountered an error while testing usage:
CMake Error at CMakeTest/CMakeLists.txt:13 (target_link_libraries):
  Target "CMakeTest" links to:

    iDynTree::idyntree-high-leveliDynTree::idyntree-estimation

@traversaro I encountered an error while testing usage:
CMake Error at CMakeTest/CMakeLists.txt:13 (target_link_libraries):
  Target "CMakeTest" links to:

    iDynTree::idyntree-high-leveliDynTree::idyntree-estimation

Thanks, there was a typo, I fixed this in https://github.com/microsoft/vcpkg/pull/30810/commits/97574753bbe660285da0c82170c9109a826597e8 .

Thanks!

Thanks for merging!


Sentimental Analysis on Different Datasets

Have you read the Contributing Guidelines ?
Yes
Description
The folder consists of a Data Analysis project done to analyse the sentimental aspect of 3 different datasets ( Amazon, Imdb and Yelp ) for an unified distinguishment to address the sentimental approach, its variability and extent. The plots and dashboards provide a detailed interpretation of what the issue addresses and also points out additional inferences from the dataset analysis.
Fixes #961

Checklist

[ Y ] I've read the contribution guidelines.
[ Y ] I've checked the issue list before deciding what to submit.
[ Y ] I've edited the README.md and link to my code.

Related Issues or Pull Requests
The pull requests is aimed to alter the " Machine Learning and Data Science " folder with the sub folder heading - "Advanced" with a new categorized file named after Sentimental Analysis.

Can you proceed with it? Let me know if any changes are required


Enhancing the navbar and subscribe section and creating a contact us section

Hi!
I am Kaushik Singh, I would like to contribute to this project by enhancing the subscribe and navbar section making them more impressive as subscribe can be separated from footer and can have its own section and also I would like to make add a beautiful and responsive contact us section.
Things I would be doing as of now :

make a Contact Us section
Enhancing the navbar
Enhancing the subscribe section

@Km7654 enhance the navbar
fix the things it's styling is not proper

the search should be at the right aligned with elements also link contact-us you can check body.js for that

navbar is enhanced no need to work on this now


Can't seem to export on GTNH 2.6.1

When I try to run the recipe dump, I get the following errors in the Prism Launcher instance logs (and the dump doesn't work):
[14:43:30] [Client thread/ERROR] [NotEnoughItems]: Error dumping NEI Recipes mode: 0
java.lang.RuntimeException: RecipeDumper received an unexpected mode! There should only be one mode: JSON
	at RFB-Launch//com.hermanoid.nerd.RecipeDumper.dumpTo(RecipeDumper.java:166) ~[RecipeDumper.class:?]
	at RFB-Launch//codechicken.nei.config.DataDumper.dumpFile(DataDumper.java:41) [DataDumper.class:?]
	at RFB-Launch//codechicken.nei.config.DataDumper.mouseClicked(DataDumper.java:138) [DataDumper.class:?]
	at RFB-Launch//codechicken.nei.config.GuiOptionList$OptionScrollSlot.slotClicked(GuiOptionList.java:129) [GuiOptionList$OptionScrollSlot.class:?]
	at RFB-Launch//codechicken.core.gui.GuiScrollSlot.slotUp(GuiScrollSlot.java:122) [GuiScrollSlot.class:?]
	at RFB-Launch//codechicken.core.gui.GuiScrollPane.mouseMovedOrUp(GuiScrollPane.java:144) [GuiScrollPane.class:?]
	at RFB-Launch//codechicken.core.gui.GuiScreenWidget.func_146286_b(GuiScreenWidget.java:80) [GuiScreenWidget.class:?]
	at RFB-Launch//net.minecraft.client.gui.GuiScreen.func_146274_d(GuiScreen.java:306) [bdw.class:?]
	at RFB-Launch//codechicken.core.gui.GuiScreenWidget.func_146274_d(GuiScreenWidget.java:103) [GuiScreenWidget.class:?]
	at RFB-Launch//net.minecraft.client.gui.GuiScreen.func_146269_k(GuiScreen.java:268) [bdw.class:?]
	at RFB-Launch//net.minecraft.client.Minecraft.func_71407_l(Minecraft.java:1640) [bao.class:?]
	at RFB-Launch//net.minecraft.client.Minecraft.func_71411_J(Minecraft.java:973) [bao.class:?]
	at RFB-Launch//net.minecraft.client.Minecraft.func_99999_d(Minecraft.java:5110) [bao.class:?]
	at RFB-Launch//net.minecraft.client.main.Main.main(SourceFile:148) [Main.class:?]
	at java.base/jdk.internal.reflect.DirectMethodHandleAccessor.invoke(Unknown Source) ~[?:?]
	at java.base/java.lang.reflect.Method.invoke(Unknown Source) ~[?:?]
	at RFB-System//net.minecraft.launchwrapper.Launch.rfb$realLaunch(Launch.java:250) [Launch.class:?]
	at RFB-System//net.minecraft.launchwrapper.Launch.launch(Launch.java:35) [Launch.class:?]
	at RFB-System//net.minecraft.launchwrapper.Launch.main(Launch.java:60) [Launch.class:?]
	at java.base/jdk.internal.reflect.DirectMethodHandleAccessor.invoke(Unknown Source) ~[?:?]
	at java.base/java.lang.reflect.Method.invoke(Unknown Source) ~[?:?]
	at com.gtnewhorizons.retrofuturabootstrap.Main.main(Main.java:219) [lwjgl3ify-2.0.9-forgePatches.jar:?]
	at RFB-System//org.prismlauncher.launcher.impl.StandardLauncher.launch(StandardLauncher.java:105) [NewLaunch.jar:?]
	at RFB-System//org.prismlauncher.EntryPoint.listen(EntryPoint.java:129) [NewLaunch.jar:?]
	at RFB-System//org.prismlauncher.EntryPoint.main(EntryPoint.java:70) [NewLaunch.jar:?]

Not sure why this happens as getMode() seems to return 0 as per the first line of log (Source) and this error should happen only if getMode() returns something different from 0 or 1 (Source) Not super familiar with Java but could it be a type issue?
Context:

MacOS 15.0.1 on M2
Prism Launcher 9.1
GTNH 2.6.1 Fresh install

Well, you've stumped me. I've never seen that issue, and it doesn't make much sense if the error log is to be believed -  zero should be equal to zero last I checked! A type is issue is not impossible, but I as much as I despise Java I believe it's smart enough to coerce number types (so float 0.0 == integer 0) or at least throw a compilation error.
So, clearly something is changing between the error statement and the mode check.
I did delete IntelliJ from my machine out of spite. If I'm to debug this, I would like a little more precise reproduction details - what sequence of buttons did you click to get this error (from a newly-launched game, no need to include Prism stuff)? There's not a lot of room for variability, I know.

Steps to reproduce:

Run GTNH 2.6.1, no additional mods except the release provided in this repo (I used notenoughrecipedumps-v1.0-beta.jar)

Click Multiplayer

Join my server

Open inventory

Click on the wrench (NEI Config) at the bottom left

Click on Tools

Click on Data Dumps

Click on Dump next to "NEI Recipes"

Behavior:

Logs as pasted in my first message
No message in chat

I have also tried in Singleplayer and face the same issue. The only thing that I have done on a previous run of the client is changing the GUI scale and changing NEI from Cheat to Recipe mode which I reverted to try to dump again with no success.
I can try on a Windows machine soon, I might be able to provide more insight.
Thank you for your work on this project and don't feel like you have to fix anything if it works for you.
I might try with GTNH 2.7.0 / 2.7.1, who knows, maybe it works bette (I haven't checked if NEI got updated in the meantime)
Best wishes for the new year!

I was able to reproduce. Let the debugging commence

Alright, I couldn't reproduce in my debugger using the original debug setup (a handful of 2.6.1 mods), so the issue must have to do with one of the many mods in the larger modpack (I only tested against GT5U using the NEI mod plus Tinkers Construct, GT++, and a couple more example mods to draw recipes from), or with an update to one of the included mods (like an update to NEI itself). I tried rebuilding with the updated mods but got swamped in Java hell (untraceable errors apparently due to dependencies or versioning)
Depending on your use case, you could check out this alternative. I don't have more bandwidth at the moment but I may return to this at a later date.


WebGL line renderer producing artifacts

Follow-up to #6953.
This is a tough problem, as it seems like the artifacts are bottom miters where the GPU projects the coordinate very far when it runs out of floating point precision. Currently, the quality of the lines (i.e. number of artifacts) depend on the video card of the client machine, as I do not have any artifacts using a similar setup to @tschaub:

On the other hand, I have far less, much better, but similar artifacts when I try to render very detailed layers on small scale.
The tough part is, we cannot calculate a reference distance to make the renderer self-aware, as the reference distance will also suffer from the same floating point precision problem. For example, trying to calculate the angle bisection's length of the current triangle and comparing the miter's length to it seems futile.
My current idea is setting a threshold to the bottom miter, just like the miter limit on the top miter. If the threshold is low enough, the artifacts disappear, however if we have a long and wide enough line on a large enough scale, the limit starts to take effect in an undesirable way, resulting in a "chopstick" line:

What do you think about my approach? Should we stick with it for now or try to come up with a better solution?

Hi,
Just a small input from an OpenLayers user/fanboy. The mentioned artefacts look awfully like a divide-by-zero case. Maybe this comes from the dot products in the linestring vertex shader? I've looked into it a bit and unless I'm wrong these products are used in divisions for miter operations, without a failsafe in case of very low value:
https://github.com/openlayers/openlayers/blob/master/src/ol/render/webgl/linestringreplay/defaultshader.glsl#L40
I haven't digged further and you're probably aware of that already, but maybe that'll help. Also your work on the webgl renderer is impressive.

Hello @jahow,
Thank you for your input. You are right, the dot product can be zero if the angle of the two segments are 0° or 180°. Unfortunately, making the procedure failsafe does not solve the issue. Glad you like my work on the WebGL renderer.

I have similar visual effects on Openlayer 4.6.5 when I use the wegl renderer.
And it disappears when zooming or unzooming.


[LOW] Re-enable ASAN run CI/clang-19

ASAN run was temporary disabled for clang-19 on CI. Similar motivation with https://github.com/beman-project/optional26/issues/32
TODOs:

[ ] Enable ASAN run with Clang-19 on CI
[ ] Use gtest_add_tests

Note: As seen in https://github.com/beman-project/iterator_interface/pull/22, clang-20 does not have this issue!


Save window position and size on shutdown

Window size and position are saved to config.cfg
Fixes #3625
How to test

Start the game
Resize or move the window
Restart the game, size and position should be saved

Can one of the admins please verify this patch?

Tests out and reviewed fine, merged, thanks @mayant15 ! Also added you to the credits. And also thanks to @e-aakash for testing :-)

By the way - @mayant15 did you use rebase in an odd fashion for https://github.com/mayant15/Terasology/commit/dc00fed6d8e29feb23c5a1b1080bba194afe82c0 like attempting to update from upstream? It ended up showing up in a couple places as linked references


[Modal] restoreFocus with backdrop click

[x] This is a v1.x issue.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior
When the modal is opened and then closed by clicking on the backdrop, the focus should be restored to the element that was focused prior to opening the modal.
Current Behavior
Clicking on the backdrop to exit the modal incorrectly restores the focus to the root element instead of the previously focused element.  Alternatively, if the escape button is used to exit the modal, then original element focus is correctly restored.
Steps to Reproduce
https://codesandbox.io/s/n4p33pjr9j
Click on the button to open the modal and then click on the backdrop to close it.  The focus will not be restored to the button.
If the escape button is used to close the modal, then the focus is correctly restored to the button.
Possible Explanation
Clicking on the backdrop causes the autoFocus method to run for the Modal, which changes the lastFocus element prior to closing the modal.   If a simple null check is performed prior to updating the lastFocus element, it corrects the issue, however I am unsure if this is an acceptable solution.
|--------------|---------|
| Material-UI  | v1.5.1  |
| React        | v16.4.2 |
| Browser      | Chrome 68.0.3440.106 |

@cvanem Use the following logic to check that the logic is correct:
setInterval(() => console(document.activeElement), 1000);

It's a duplicate of #12067.


Recurring Donations can Trigger Oppty Rollups Async Error

Issue by kbromer from Tuesday Sep 24, 2013 at 04:39 GMT
Originally opened as https://github.com/SalesforceFoundation/Recurring_Donations/issues/30

https://powerofus.force.com/_ui/core/chatter/groups/GroupProfilePage?g=0F980000000CjRe&fId=0D58000000t7FYo&OpenCommentForEdit=1&fromEmail=1
"Errors were encountered when trying to insert new Opportunities for your open-ended Recurring Donations. This can adversely affect your forecasting and functionality of your Salesforce.com instance. Addressing these errors as soon as possible can prevent bad data from entering your system. The Recurring Donation and associated Opportunity insert error are listed below, and can be found in the Recurring Donations Settings tab under 'Batch Settings'.
Errors:
https://na14.salesforce.com/a08d000000XOHQmAAP Error Message: npo02.OpportunityRollup: execution of AfterInsert caused by: System.AsyncException: Future method cannot be called from a future or batch method: npo02.OpportunityRollups.rollupContactsandHouseholdsForTrigger(SET) (npo02)" #NPSP
Topics:   NPSP

Hey Guys,
I have a client on NPSP2 who is running into this issue right now. Is there a resolution for v2 users who aren't ready to upgrade to Cumulus?

we aren't updating 2.0 packages.  the only solution is to upgrade.  note that they should be able to upgrade painlessly and stay with their current account model.


CMake build scripts

I'd like to get this library building with cmake on pc/mac/linux to make things easier.

any play  build on iOS/Android?

Yes! But later.
The hard part here is getting libnice/glib to work on iOS and Android. In Principal it should work, but the reality is that getting glib-2.0 building on iOS could be very tricky.
I'll go ahead and create issues for Android and iOS, and see if I can get to them over the next month.

ok   i can give some test.

some one has build libnice on ios    https://github.com/Anakros/libnice-ios


Extend launch policy to carry stack size and scheduling hint in addition to priority

This PR extends hpx::launch and related types (i.e. hpx::launch::async, etc.) such that they contain the scheduling hint and the stack size in addition to the thread priority.
This will be needed for the direct scoped execution to be able to control whether threads should be executed directly or not. This also simplifies a couple of internal interfaces.
As a flyby, this PR changes all thread enums to be of size std::int8_t, which allows for the launch policy to fit into a 64 bit register.

This looks good now, I believe.

I suppose these would be fitting to change in this PR: https://app.codacy.com/gh/STEllAR-GROUP/hpx/pullRequest?prid=8056581. Are you able to reproduce this: https://cdash.cscs.ch/testDetails.php?test=51703242&build=180874?

I suppose these would be fitting to change in this PR: https://app.codacy.com/gh/STEllAR-GROUP/hpx/pullRequest?prid=8056581.

These are deliberate, I'll ignore the tests for codacy.

Are you able to reproduce this: https://cdash.cscs.ch/testDetails.php?test=51703242&build=180874?

Not yet, looks like something compiler specific, though.

@hkaiser looks like this is ready to go. Did you find out what was causing https://cdash.cscs.ch/testDetails.php?test=51703242&build=180874?

@hkaiser looks like this is ready to go. Did you find out what was causing https://cdash.cscs.ch/testDetails.php?test=51703242&build=180874?

Not yet, I wasn't even able to reproduce it locally...

@hkaiser looks like this is ready to go. Did you find out what was causing https://cdash.cscs.ch/testDetails.php?test=51703242&build=180874?

Not yet, I wasn't even able to reproduce it locally...

@msimberg do you recall whether this particular platform has any special configurations?

@hkaiser I think C++20 is the only possibly relevant configuration option: https://github.com/STEllAR-GROUP/hpx/blob/5f9505f5312d2d538b64039c862154244f98d33f/.jenkins/cscs/env-gcc-newest.sh#L10.


💥 Update k8s provider to enable TLS login

Maybe we will merge https://github.com/AckeeCZ/terraform-gcp-elasticsearch/pull/5 today, then we will need to rebase. Will let you know if it happens

So ti happened :D This needs rebase :)


Fix publish script due to crates.io CDN change

Now that crates.io goes directly to the CDN, this has changed the HTTP response code when a crate is not found (from 404 to 403). This updates the publish script to handle more error codes to assume something isn't published. This also updates it to go directly to the endpoint instead of through the crates.io redirect.

r=weihanglo whenever you want to proceed or add some print.

Good idea, I added a print.
@bors r=weihanglo

:pushpin: Commit 97d896fe52e057baacaf275b90a36f52ec05e6ff has been approved by weihanglo
It is now in the queue for this repository.

:hourglass: Testing commit 97d896fe52e057baacaf275b90a36f52ec05e6ff with merge c3ad639bb92d4edf1b0980a97d6e2cb15cf43083...

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing c3ad639bb92d4edf1b0980a97d6e2cb15cf43083 to master...


Full Domain Queries Smash CacheDB

We make use of PBIS-open to allow our non-ephemeral Linux instances to authenticate against Active Directroy. Ours is mostly a large Windows shop with a large backing AD service (several tens of thousands of user and group entries). For the most part, PBIS Open works well for the Linux systems it's used on. However, if an Linux administrator makes the mistake of doing a full enumeration of the AD name-space (e.g., with the equivalent of a getent passwd or getent groups), PBIS has a stroke. There's only two ways to recover from the situation:

Wholly uninstall - ensuring to nuke the various */pbis/* directories and files
Nuke the sqlite cache DB and re-join the host to the domain

Looking for tips on how we might prevent PBIS from crushing its sqlite cache DB other than "don't (accidentally) try to enumerate 80,000 user objects".

There is a MemoryCacheSizeCap option you can set via the /opt/pbis/bin/config utility which I believe sets the maximum bytes to store in the memory cache.
Maybe try:
/opt/pbis/bin/config MemoryCacheSizeCap 16777216


Increase position update interval

Hi,
I would like to increase the time between 2 position update for CurrentLocationLayer, to avoid too many ui update.
How can do that ?
Regards

For this kind of how-to question, please ask in the discord channel.


Update CI notifications

Change content and motivation
As per discussion: https://github.com/kubenow/KubeNow/issues/358.

LGTM


[DOC] Minor modification to Streaming docs with regards to parallel data receiving

@pwendell @tdas

Can one of the admins verify this patch?

That's fine. This does not warrant a JIRA.
(PS why do your PRs have a huge merge history? you'd normally just branch from an updated master right?)

Yeah, the JIRA in the title seems unnecessary (removed). With regards to the merge history: not sure, will have to look into it.

LGTM other than the commit history.
Can you just just checkout the spark github master branch and cherry-pick your change (commit 46e8c03) on it. That should keep the commit history clean.

FWIW I keep master in my fork up to date with git pull upstream master and git push origin master. Then whenever you make a branch, just branch from your master. Have you been merging into your master? It should just track the upstream one directly with no variance.