Memory leak for async/await with es6 target

When the supplied code is compiled with target es6 it results in a memory leak, however when the target is es2017 this is not the case.
TypeScript Version:  3.7.4
Node Version: 11.14.0

Search Terms:
memory leak
target
async
__awaiter
Code
class AsyncDoer {
    public async doIt() {
        return null;
    }
}

const run = async () => {
    for (let index = 0; index < 10000000; index++) {
        if(index % 100000 === 0) console.log(Math.floor(process.memoryUsage().heapUsed / 1000000), index);

        const doer = new AsyncDoer();
        await doer.doIt();
    }
}

run();

when compiled with es6 target results in:
var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
    function adopt(value) { return value instanceof P ? value : new P(function (resolve) { resolve(value); }); }
    return new (P || (P = Promise))(function (resolve, reject) {
        function fulfilled(value) { try { step(generator.next(value)); } catch (e) { reject(e); } }
        function rejected(value) { try { step(generator["throw"](value)); } catch (e) { reject(e); } }
        function step(result) { result.done ? resolve(result.value) : adopt(result.value).then(fulfilled, rejected); }
        step((generator = generator.apply(thisArg, _arguments || [])).next());
    });
};
class AsyncDoer {
    doIt() {
        return __awaiter(this, void 0, void 0, function* () {
            return null;
        });
    }
}
const run = () => __awaiter(this, void 0, void 0, function* () {
    for (let index = 0; index < 10000000; index++) {
        if (index % 100000 === 0)
            console.log(Math.floor(process.memoryUsage().heapUsed / 1000000), index);
        const doer = new AsyncDoer();
        yield doer.doIt();
    }
});
run();

when compiled with es2017 target, results in:
class AsyncDoer {
    async doIt() {
        return null;
    }
}
const run = async () => {
    for (let index = 0; index < 10000000; index++) {
        if (index % 100000 === 0)
            console.log(Math.floor(process.memoryUsage().heapUsed / 1000000), index);
        const doer = new AsyncDoer();
        await doer.doIt();
    }
};
run();

Expected behavior:
When compiled with the following config:
{
    "compilerOptions": {
        "target": "es6",
    }
}

The js output can be run to completion using the command node --max-old-space-size=100.
Actual behavior:
The program crashes with an out of memory:
<--- Last few GCs --->

[1108:0x3dcd1f0]    16382 ms: Mark-sweep 49.5 (92.7) -> 49.5 (86.7) MB, 17.2 / 0.0 ms  (average mu = 0.744, current mu = 0.001) allocation failure GC in old space requested
[1108:0x3dcd1f0]    16400 ms: Mark-sweep 49.5 (86.7) -> 49.5 (55.7) MB, 18.0 / 0.0 ms  (average mu = 0.590, current mu = 0.001) last resort GC in old space requested
[1108:0x3dcd1f0]    16418 ms: Mark-sweep 49.5 (55.7) -> 49.5 (55.7) MB, 18.7 / 0.0 ms  (average mu = 0.412, current mu = 0.001) last resort GC in old space requested

<--- JS stacktrace --->

==== JS stack trace =========================================

    0: ExitFrame [pc: 0x31fa1cfcfc5d]
Security context: 0x37d53b59adf9 <JSObject>
    1: /* anonymous */ [0x164abf9127b9] [/home/chris/dev/mem_test/memTest.js:~17] [pc=0x31fa1d071475](this=0x2253745902f1 <Object map = 0x30ac8c082521>)
    2: next [0x37d53b596a79](this=0x225374584ac1 <JSGenerator>,0x38cb8e2022a1 <null>)
    3: fulfilled [0x2253745849e9] [/home/chris/dev/mem_test/memTest.js:~4] [pc=0x31fa1d06eaec](this=0x03e469c02331 <JSGlo...

FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - JavaScript heap out of memory
 1: 0x95b8f0 node::Abort() [node]
 2: 0x95c836 node::OnFatalError(char const*, char const*) [node]
 3: 0xb3b77e v8::Utils::ReportOOMFailure(v8::internal::Isolate*, char const*, bool) [node]
 4: 0xb3b9b4 v8::internal::V8::FatalProcessOutOfMemory(v8::internal::Isolate*, char const*, bool) [node]
 5: 0xf3b002  [node]
 6: 0xf4b10f v8::internal::Heap::AllocateRawWithRetryOrFail(int, v8::internal::AllocationSpace, v8::internal::AllocationAlignment) [node]
 7: 0xf14fa6 v8::internal::Factory::AllocateRawArray(int, v8::internal::PretenureFlag) [node]
 8: 0xf1aa2e v8::internal::Factory::CopyWeakArrayListAndGrow(v8::internal::Handle<v8::internal::WeakArrayList>, int, v8::internal::PretenureFlag) [node]
 9: 0x104f38a v8::internal::WeakArrayList::EnsureSpace(v8::internal::Isolate*, v8::internal::Handle<v8::internal::WeakArrayList>, int, v8::internal::PretenureFlag) [node]
10: 0x104f653 v8::internal::PrototypeUsers::Add(v8::internal::Isolate*, v8::internal::Handle<v8::internal::WeakArrayList>, v8::internal::Handle<v8::internal::Map>, int*) [node]
11: 0x1052e24 v8::internal::JSObject::LazyRegisterPrototypeUser(v8::internal::Handle<v8::internal::Map>, v8::internal::Isolate*) [node]
12: 0x10533c7 v8::internal::Map::GetOrCreatePrototypeChainValidityCell(v8::internal::Handle<v8::internal::Map>, v8::internal::Isolate*) [node]
13: 0xfabb34 v8::internal::LoadHandler::LoadFromPrototype(v8::internal::Isolate*, v8::internal::Handle<v8::internal::Map>, v8::internal::Handle<v8::internal::JSReceiver>, v8::internal::Handle<v8::internal::Smi>, v8::internal::MaybeObjectHandle, v8::internal::MaybeObjectHandle) [node]
14: 0xfb4ba4 v8::internal::LoadIC::ComputeHandler(v8::internal::LookupIterator*) [node]
15: 0xfbbfec v8::internal::LoadIC::UpdateCaches(v8::internal::LookupIterator*) [node]
16: 0xfbc6bc v8::internal::LoadIC::Load(v8::internal::Handle<v8::internal::Object>, v8::internal::Handle<v8::internal::Name>) [node]
17: 0xfc0c05 v8::internal::Runtime_LoadIC_Miss(int, v8::internal::Object**, v8::internal::Isolate*) [node]
18: 0x31fa1cfcfc5d
Aborted (core dumped)

Are you sure the memory is leaking, and you're not just running out of memory? As you're not awaiting any call you're creating 10.000.000 async calls simultaneously, and the polyfill requires a lot more memory than the natively supported version.

Apologies, my original code does contain an await - missed that out. Will update example code - and retest to ensure the memory still increases.

This is an annoying problem that i've search for days to find the solution,  tsconfig.json use option {target:'es6'} ,then the generated code use __awaiter which implemented through es6 Generator  instand of 'async & await',so this problem will come out if you use node 12 then use TypeScript to compile a async/await codes to es6,that's destructive. the solution of mine is set target to es7,then the problem is gone.


[5.9] [SourceKit] Record module loading errors when generating interfaces

5.9 cherry-pick of #66676

Explanation: Improves the error message we produce when we fail to load a module for a generated interface.
Scope: Affects error handling logic of generated interfaces.
Issue: rdar://109511099
Risk: Low, only affects the error handling logic of generated interfaces
Testing: Added tests to test suite
Reviewer: Alex Hoppen

@swift-ci please test


Ada Lovelace

Add Ada Lovelace's program for calculating Bernoulli Numbers using Charles Babbage's Analytical Engine. It's open source, critical, and feminist because Ada published her program in 1843, one hundred years before Alan Turing created the computer that broke Enigma. Her algorithm is in Note G of the citation. Her program uses "Variable-cards" and a loop called a "cycle of a cycle of Variable-cards".

Thanks for opening the first pull request!
I'm struggling with this one honestly. There's no denying the momentous impact that Ada Lovelace had on computer science and computing as a whole, but could you elaborate on its relationship to open source specifically? I'm having a hard time putting it in that context.
(Please note that something can absolutely be of critical impact to technology overall but be less of a critical event for open source. I would anticipate that if we can tie this event back to open source more concretely, I would recommend making this a note to indicate that it is more tangential/commentary.)

Well, it's source code, and she openly published it, so that fits the definition of open source, per se.

I originally labeled it critical because she did have impacts beyond the open-source ecosystem, as an example of an accomplished woman 80 years before women got the right to vote and as a brilliant person. I mean, the Analytical Engine only existed on paper, and she wrote a program for it! Computer scientists named a whole computer language after her. The Imitation Game could easily have been made about her.
Anyway, you already know her impact, so I defer to your good judgment.

This has lit up different parts of my brain. Rather than spinning these thoughts internally, hoping to discuss more with you both (all!) here.

Could this be argued to be the first open source program ever released? If not in 2022 legal + technical terms, but in the spirit of what open source software is? Maybe we should break down further what should be included as "open source" - definition or intent? Do we get to accredit the work before the movement was defined or named with its modern meanings?

Other potential modern labels - "public domain", "free software"

What's the criteria we should be using here to measure against what is an open source event? Must it be discretely definable? How broad does the impact have to be and for how long? Good questions for discussion!

@amcasari Glad you liked the discussion.

Yes. Yes. No. Yes.
Possibly.

"Public domain" software is a legal minefield. If you wrote it and someone dies, you could be sued. But sure, it could be a label.
"Free software" should be defined with the GNU four freedoms: (0) to run the program, (1) to study and change the program in source code form, (2) to redistribute exact copies, and (3) to distribute modified versions.

Criteria: Are they instructions for a machine? Can you run, see, modify, and distribute them without being sued? See the 4 freedoms above.

Julia is oversubscribed, so I'm fine with her decision, whatever it is.

While we do deal with legal milestones on this timeline, I don't think that we need to get into the weeds of liability (outside of documented cases that are, themselves, open source events).
I'm thinking there should be a cutoff on timing, but I'll move that discussion to an issue.
I really dislike declining this PR, both as an admirer of Ada Lovelace and as a feminist, but I think this is a bit too far removed from the cultural and technical open source software movement to be included.
I propose closing this PR, but with the understanding that we will reopen it if the scope changes. @wirelessben, does that sound okay to you?

I'm fine with it, @juliaferraioli. Ada's code was the first event I thought of when you tweeted. If others don't feel the same, then I was just a one off. Thanks for your consideration.


Unit test fails "No provider for $uiRouter"

I've read through https://github.com/ui-router/ng1-to-ng2 and my app is working great, ng1 routing still works and now I can use uiSref in my ng2 component. But my unit test fails for the ng2 component that is using uiSref.
Here's the code for the unit test and the app.module
https://gist.github.com/miked1090/d6a2e99f8e3dcaa6f27a9b2b7d5840a4
In the test.component.html there's an uiSref, if I comment it out the test runs fine, otherwise it fails with
"No provider for $uiRouter".
This is also a component that is not registered with the router, it's just a small component that is being used on the page.
Any help would be much appreciated, thanks.
Edit: I made some changes and removed importing AppModule in the spec file and instead declared the component in there. This fixes the issue of "No provider for $uiRouter", but now I get an error saying [uiParams] is not a valid attribute for the 'a' tag.

Fixed it by changing my Testbed configuration to

TestBed.configureTestingModule({
imports: [UIRouterModule.forRoot({ useHash: true })],
declarations: [TestComponent],
providers: [{ provide: TestService, useValue: testService}
]
})


Do not append string to reasons if no feature gates or admission plugins are removed

How to categorize this PR?

/area quality
/kind bug
What this PR does / why we need it:
Even if no feature gates are removed, maintenance reasons contain:
  lastMaintenance:
    description: 'All maintenance operations successful. Worker pool "local": Updated
      machine image "local" from "2.1.0" to "2.2.0". Reason: Automatic update of the
      machine image version is configured (image update strategy: major), Removed
      feature gates from "spec.provider.workers[0].kubernetes.kubelet.featureGates"
      because they are not supported in Kubernetes version "1.29.0": , Removed admission
      plugins from "spec.kubernetes.kubeAPIServer.admissionPlugins" because they are
      not supported in Kubernetes version "1.29.0": ''

This PR fixes this issue by appending to reasons only if something is removed.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

An issue causing the Shoot `status.lastMaintenance.description` to contain "Removed feature gates from" or  "Removed admission plugins from" messages with zero entries is now fixed.

/cherry-pick release-v1.92

/cherry-pick release-v1.91

Thanks for the fix! I have one small confusion. Here else block will be executed in case err != nil. Is this what we need?
https://github.com/gardener/gardener/blob/475167f4c19d820c57519bf1e7751c0e2e5c8dde/pkg/controllermanager/controller/shoot/maintenance/reconciler.go#L882-L888

Thanks for the review, Ideally this should not happen since this is part of existing Shoot spec and has undergone validation. I can add a comment to the code if that helps.

/approve


How to stream from List data type

Checklist

[X] I made sure that the issue I am raising doesn't already exist

Use case / Problem
Hi, I was reading the documentation and I could not find a way stream from List data type
I have a stream data looks like this
[0.00005239215533947572, 0.0000014616052794735879, -0.00008511867054039612, -0.00006905655754962936, 0.000015688463463447988, -0.0000017326583474641666, -0.00006541159382322803, -0.000050078597269020975, 0.0001162810658570379, -0.00012878471170552075, -0.00018615851877257228, -0.0000412149092881009, -0.0001031801657518372, -0.00037546356907114387, -0.0004687689943239093, -0.0002827091084327549, -0.00020072983170393854, -0.00016192495240829885, 0.00006172947905724868, 0.00004658072430174798, -0.00020569296611938626, -0.0003184914239682257]
any idea how to run it??
thanks for the help.
Proposal / Solution
add an example, show how to run List the current examples only support List
Example Code

Example Code
void main() {
}

Affected platforms
Android, iOS, web, Windows, Linux, macOS
Platform details
No response
Audio Files/URLs/Sources
No response
Mockups
No response
Related issues / more information
No response
Working on PR
no way

It's not available for all platforms yet, see #1269 . Feel free to contribute.


ASP.NET Core 2.2 Signature validation failed in JwtBearer Authentication (just in Linux/MacOS)

Describe the bug
After updating to 2.2 RSA signature cant' be verify in Linux and MacOS ( in windows it is ok).
I use this code in the startup to set authentication:
  services.AddAuthentication(options =>
            {
                options.DefaultAuthenticateScheme = JwtBearerDefaults.AuthenticationScheme;
                options.DefaultSignInScheme = JwtBearerDefaults.AuthenticationScheme;
                options.DefaultScheme = JwtBearerDefaults.AuthenticationScheme;
            })
            .AddJwtBearer(config =>
            {
                config.Events = new API.Core.Security.JwtBearerEvents();
                config.RequireHttpsMetadata = !CurrentEnvironment.IsDevelopment();
                config.IncludeErrorDetails = CurrentEnvironment.IsDevelopment();
                config.SaveToken = true;
                config.TokenValidationParameters = new TokenValidationParameters()
                {
                    RequireSignedTokens = true,
                    ValidateIssuerSigningKey = true,
                    ValidateLifetime = true,
                    ValidateAudience = false,
                    ValidateIssuer = false,
                    IssuerSigningKey = new RsaSecurityKey(rsa)
                };

            });

After update in any endpoint with [Authorize] attribute in Linux or MacOS authentication middleware throw can't verify signature error.
in Older version or in windows it works just fine.
To Reproduce
Steps to reproduce the behavior:

Using this version of ASP.NET Core '2.2.101'
Add AddJwtBearer in Authentication
Run in Linux/MacOs
Call endpoint with [Authorize]
See the error

Sounds like
https://github.com/AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet/issues/1067
We had to roll back to 5.2.4 - 5.3.0 seems to have issues.

Agree with @leastprivilege , so closing this issue. Thanks!

Thank you for closing and ignoring the issue.😑😑😐

Sorry for the confusion, we must have mis-understood it. Re-opening so we can evaluate again.


3.0 Deprecation Warning: Deprecate fallback spaCy model

Background
This warning was found in:

rasa.nlu.utils.spacy_utils.py:L140

"f""SpaCy model is not properly configured! ""
 f""Please add a `model` property to `SpacyNLP`. ""
f""Will use '{spacy_model_name}' as a fallback spaCy model. ""
f""This fallback will be deprecated in Rasa 3.0"""

Definition of Done

[ ] resolve warning in 3.0 version
[ ] Removed all uses of _check_model_fallback in favour of using directly model from config. Here I assume we don't have to do any checking on model -- if a Spacy model with that name doesn't exist, then Spacy will raise an error? 🤔 Currently, we do checking but it requires hard-coding all possible Spacy model names, which then leads to issues like Support spaCy 3.1 #9068 which are just about updating that hard-coded list afaics.
[ ] Remove tests for _check_model_fallback
[ ] Removed the references in the docs

I might be able to pick this one up. I implemented the spaCy 3.0 and wouldn't mind wrapping it up for Rasa 3.0.
Just to check, I can make a fix with a PR to master?

@koaning this issue needs to be done before 3.0, while #9068 doesn't necessarily have to go into 3.0 (strictly speaking). Seeing that you assigned yourself on this one, are you actually working on it? To me it looks like an issue that research can easily pick up...
As for a definition of done, the method in question (_check_model_fallback) is found in the codebase and in some tests (but not in docs), so we might say:

Removed all uses of _check_model_fallback in favour of using directly model from config. Here I assume we don't have to do any checking on model -- if a Spacy model with that name doesn't exist, then Spacy will raise an error? 🤔  Currently, we do checking but it requires hard-coding all possible Spacy model names, which then leads to issues like #9068 which are just about updating that hard-coded list afaics.
Remove tests for _check_model_fallback

@samsucik I'll gladly pick it up if it removes a burden from the research team. If you folks have the capacity then I certainly won't mind anybody else picking this up. 👍
I agree that strictly speaking it's no needed to upgrade spaCy. There are some extra languages added to the newer version though, which would be nice to add.

@koaning Thanks for picking this up. This will be blocking the architecture revamp. Once 2.8 is released early next week, if you have not completed this issue, Enable might reach out about taking it over, so it is not blocking them

Clear. I've set it as a TODO for Monday.

The PR is ready and no longer a work in progress. The upgrade to spaCy 3.1 will be a separate PR.


Opportunities "Date Posted (Latest)" sort is backwards

Oldest posts are showing up at the top, and newest at the bottom.
Perhaps when fixing this, add another sort option, so we can have "Date Posted (Oldest)" and "Date Posted (Latest)"

Oops, nice catch. I'll fix the bug in the next PR and leave the issue open for the enhancement.


Target temperature always "null" in v0.27

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
0.27
Python release (python3 --version):
3.4.2
Component/platform:
zwave/climate
Description of problem:
Target temperature on all of my devolo thermostats is always "null" after upgrade to 0.27.
Current temperature is displayed correctly. Changing target temperatures is also working.
Expected:
Target temperature to be displayed correctly.
Problem-relevant configuration.yaml entries and steps to reproduce:

Traceback (if applicable):
16-08-30 00:20:23 homeassistant.components.climate.zwave: Value changed on network home_id: [0xde8e99f2] id: [72057594076774418] parent_id: [2] label: [Heating 1] data: [10.0]
16-08-30 00:20:23 homeassistant.components.climate.zwave: Value changed on network home_id: [0xde8e99f2] id: [72057594076823684] parent_id: [2] label: [Override State] data: [None]
16-08-30 00:20:23 homeassistant.components.climate.zwave: Value changed on network home_id: [0xde8e99f2] id: [72057594076823697] parent_id: [2] label: [Override Setback] data: [127]
16-08-30 00:20:23 homeassistant.components.climate.zwave: Value changed on network home_id: [0xde8e99f2] id: [72057594085982212] parent_id: [2] label: [Protection] data: [Unprotected]
16-08-30 00:20:23 homeassistant.components.climate.zwave: Value changed on network home_id: [0xde8e99f2] id: [72057594086227971] parent_id: [2] label: [Wake-up Interval] data: [60]
16-08-30 00:20:23 homeassistant.components.climate.zwave: Value changed on network home_id: [0xde8e99f2] id: [72057594076479506] parent_id: [2] label: [Temperature] data: [27.010000228881836]
16-08-30 00:20:23 homeassistant.components.climate.zwave: Value changed on network home_id: [0xde8e99f2] id: [72057594076479506] parent_id: [2] label: [Temperature] data: [27.010000228881836]
16-08-30 00:20:23 homeassistant.components.climate.zwave: Value changed on network home_id: [0xde8e99f2] id: [72057594077773825] parent_id: [2] label: [Battery Level] data: [63]
16-08-30 00:20:23 homeassistant.components.climate.zwave: Value changed on network home_id: [0xde8e99f2] id: [72057594077790212] parent_id: [2] label: [Day] data: [Tuesday]
16-08-30 00:20:23 homeassistant.components.climate.zwave: Value changed on network home_id: [0xde8e99f2] id: [72057594077790225] parent_id: [2] label: [Hour] data: [5]
16-08-30 00:20:23 homeassistant.components.climate.zwave: Value changed on network home_id: [0xde8e99f2] id: [72057594077790241] parent_id: [2] label: [Minute] data: [28]

Additional info:
See excerpt of log level debug for climate component above. Target temperature should be the first entry with label "Heating 1"

This is fixed in dev.


New installation: Undefined index: argv - Error in IgnitionServiceProvider - runningInConsole?

If I install a new installation of Laravel via composer or direct download of the latest version from Github, I get an error in the browser window.
ErrorException
Undefined index: argv
Illuminate\Foundation\Bootstrap\HandleExceptions::handleError
C:\projec-name\app\vendor\facade\ignition\src\IgnitionServiceProvider.php:83
The IgnitionServiceProvider thinks, that I run the app from the console.
If I comment out the runningInConsole() condition, the site works again.
I'm really not sure why Laravel thinks, that it runs from console here. I have installed another older Laravel project (Laravel Jetstream) installed on the same server which works without this problem. Also I installed the dev-master version which also don't outhput this error.
\PHP_SAPI returns cgi-fcgi
The problem is this part of the IgnitionServiceProvider.php
        if ($this->app->runningInConsole()) {
            ...
            if (['artisan', 'tinker'] === $_SERVER['argv']) {
                Api::sendReportsInBatches(false);
            }
        }

Laravel Version: 8.x (Latest)
PHP Version: 7.4.1

[2021-02-15 19:27:21] local.ERROR: Undefined index: argv {"exception":"[object] (ErrorException(code: 0): Undefined index: argv at C:\\project-name\\app\\vendor\\facade\\ignition\\src\\IgnitionServiceProvider.php:83)
[stacktrace]
#0 C:\\project-name\\app\\vendor\\facade\\ignition\\src\\IgnitionServiceProvider.php(83): Illuminate\\Foundation\\Bootstrap\\HandleExceptions->handleError(8, 'Undefined index...', 'C:\\\\project-name...', 83, Array)
#1 C:\\project-name\\app\\vendor\\laravel\\framework\\src\\Illuminate\\Container\\BoundMethod.php(36): Facade\\Ignition\\IgnitionServiceProvider->boot()
#2 C:\\project-name\\app\\vendor\\laravel\\framework\\src\\Illuminate\\Container\\Util.php(40): Illuminate\\Container\\BoundMethod::Illuminate\\Container\\{closure}()
#3 C:\\project-name\\app\\vendor\\laravel\\framework\\src\\Illuminate\\Container\\BoundMethod.php(93): Illuminate\\Container\\Util::unwrapIfClosure(Object(Closure))
#4 C:\\project-name\\app\\vendor\\laravel\\framework\\src\\Illuminate\\Container\\BoundMethod.php(37): Illuminate\\Container\\BoundMethod::callBoundMethod(Object(Illuminate\\Foundation\\Application), Array, Object(Closure))
#5 C:\\project-name\\app\\vendor\\laravel\\framework\\src\\Illuminate\\Container\\Container.php(596): Illuminate\\Container\\BoundMethod::call(Object(Illuminate\\Foundation\\Application), Array, Array, NULL)
#6 C:\\project-name\\app\\vendor\\laravel\\framework\\src\\Illuminate\\Foundation\\Application.php(869): Illuminate\\Container\\Container->call(Array)
#7 C:\\project-name\\app\\vendor\\laravel\\framework\\src\\Illuminate\\Foundation\\Application.php(850): Illuminate\\Foundation\\Application->bootProvider(Object(Facade\\Ignition\\IgnitionServiceProvider))
#8 [internal function]: Illuminate\\Foundation\\Application->Illuminate\\Foundation\\{closure}(Object(Facade\\Ignition\\IgnitionServiceProvider), 14)
#9 C:\\project-name\\app\\vendor\\laravel\\framework\\src\\Illuminate\\Foundation\\Application.php(851): array_walk(Array, Object(Closure))
#10 C:\\project-name\\app\\vendor\\laravel\\framework\\src\\Illuminate\\Foundation\\Bootstrap\\BootProviders.php(17): Illuminate\\Foundation\\Application->boot()
#11 C:\\project-name\\app\\vendor\\laravel\\framework\\src\\Illuminate\\Foundation\\Application.php(230): Illuminate\\Foundation\\Bootstrap\\BootProviders->bootstrap(Object(Illuminate\\Foundation\\Application))
#12 C:\\project-name\\app\\vendor\\laravel\\framework\\src\\Illuminate\\Foundation\\Http\\Kernel.php(152): Illuminate\\Foundation\\Application->bootstrapWith(Array)
#13 C:\\project-name\\app\\vendor\\laravel\\framework\\src\\Illuminate\\Foundation\\Http\\Kernel.php(136): Illuminate\\Foundation\\Http\\Kernel->bootstrap()
#14 C:\\project-name\\app\\vendor\\laravel\\framework\\src\\Illuminate\\Foundation\\Http\\Kernel.php(110): Illuminate\\Foundation\\Http\\Kernel->sendRequestThroughRouter(Object(Illuminate\\Http\\Request))
#15 C:\\project-name\\app\\public\\index.php(52): Illuminate\\Foundation\\Http\\Kernel->handle(Object(Illuminate\\Http\\Request))
#16 {main}
"}

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.


Mimic Tree UI of ChatGPT

I reviewed ChatGPT's UI and this PR makes it so that the tree UI now mimics ChatGPT's implementation.
Please review and merge when ready. Thank you @littlemoonstones!

Thanks! However, in mobile view, I found that the button extends beyond the screen. I will try to fix it later.

Thanks! However, in mobile view, I found that the button extends beyond the screen. I will try to fix it later.

Ah great catch, thank you!

Sorry, I'm not sure how to merge my branch (mimic_tree) into your PR. Could you please merge b2476f4e666a81db168cb923d53e6d334278f0d0 into your PR instead? I'm not certain if this is the correct course of action.
Desktop

Mobile

@littlemoonstones Just merged your changes into this PR! Feel free to review and merge if you like it in its current state

OK, thanks!


Added new interface FilterInterfaceWithArguments

Description
Fixing #3216 by implementing a new interface to avoid breaking already implemented Filters.
Checklist:

[X] Securely signed commits
[X] Component(s) with PHPdocs
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[X] Conforms to style guide

All tests pass locally, so I am not sure what I'm seeing...

@tangix Can you pull the latest changes from the dev branch and add Services::reset() to the FiltersWithArgumentsTest::setUp() just after the call to the parent::setUp()?

Done.

@lonnieezell @MGatner
We probably need to update the user guide for this, but besides that - are you good with these changes?

@lonnieezell @MGatner
We probably need to update the user guide for this, but besides that - are you good with these changes?

Also, please consider if there is a better way to do this without break backward compatibility.

So basically all we're doing is creating a new interface that accepts $args, correct? In my mind, this is a bug since it is expected to be working currently. So all we need to do is add the argument to the interface and method. And maybe add some better docs, right?
While I realize this is technically a BC break, it won't break any existing code. And should be working actually since Myth:Auth uses those filters, though I haven't tried it in a while.

@lonnieezell
Yeah, I was torn between changing the current interface (breaking code) or adding a new interface (possibly cluttering down the framework)...

@tangix And I totally respect that you were trying to avoid the BC break here and really appreciate it! I just think this qualifies as a bug so we can fix it the right way.

@tangix And I totally respect that you were trying to avoid the BC break here and really appreciate it! I just think this qualifies as a bug so we can fix it the right way.

I can live with that and I think that would be the correct way forward to avoid another interface and added complexity to CI4.

@lonnieezell
I see 4.0.4 is around the corner but with no fix for this issue. Do you want me to create a new PR based on develop branch, change the FilterInterface (introduce a BC breaking fix) and also check over the docs?

@tangix the goal for 4.0.4 is within the next 24 hours. If you can get this reworked and submitted today I think it would be a great addition given that developers using it will require manual intervention, and a point release is a great time for that.

OK, created #3316 fixing the actual bug instead of adding a new interface.


Fix https://github.com/mozilla/thimble.webmaker.org/issues/1182 - allow javascript: and other common non-http schemes

This allows things like the following to work:
An alert should pop up from this link.
<a<EMAIL_ADDRESS>email should get created.

First reported in https://github.com/mozilla/thimble.webmaker.org/issues/1182

r=me


Remove unused types from pkg/apis/crd/v1alpha1

Some L7 types were not removed when NetworkPolicy CRDs were promoted to v1beta1.

/skip-all


Don't force app orientation

When opening the app it will force portrait orientation, but the OS is designed to be used in any orientation.

Fixed with 1a4bee1.


For consistency, Cursor class objects should be made in Renderer class.

Cursor stands out as it does not have a parent nor a creator function unlike most other classes, like Renderer is parent to Window.
Renderer should be the parent of Cursor the same way Renderer is parent to Monitor or Window.

Done.


swiftlang: build type of swiftlang should agree with the standard libary

@swift-ci please smoke test

Seems reasonable to me.


Bad input in associate gives traceback

tower-cli organization associate --user test_user

I expect a user error stating what went wrong. But I get:
Traceback (most recent call last):
  File "/Users/alancoding/.virtualenvs/cli2/bin/tower-cli", line 4, in <module>
    __import__('pkg_resources').run_script('ansible-tower-cli==2.3.1', 'tower-cli')
  File "/Users/alancoding/.virtualenvs/cli2/lib/python2.7/site-packages/pkg_resources/__init__.py", line 724, in run_script
    self.require(requires)[0].run_script(script_name, ns)
  File "/Users/alancoding/.virtualenvs/cli2/lib/python2.7/site-packages/pkg_resources/__init__.py", line 1650, in run_script
    exec(code, namespace, namespace)
  File "/Users/alancoding/.virtualenvs/cli2/lib/python2.7/site-packages/ansible_tower_cli-2.3.1-py2.7.egg/EGG-INFO/scripts/tower-cli", line 80, in <module>
    cli()
  File "/Users/alancoding/.virtualenvs/cli2/lib/python2.7/site-packages/click/core.py", line 716, in __call__
    return self.main(*args, **kwargs)
  File "/Users/alancoding/.virtualenvs/cli2/lib/python2.7/site-packages/click/core.py", line 696, in main
    rv = self.invoke(ctx)
  File "/Users/alancoding/.virtualenvs/cli2/lib/python2.7/site-packages/click/core.py", line 1060, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/Users/alancoding/.virtualenvs/cli2/lib/python2.7/site-packages/click/core.py", line 1060, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/Users/alancoding/.virtualenvs/cli2/lib/python2.7/site-packages/click/core.py", line 889, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Users/alancoding/.virtualenvs/cli2/lib/python2.7/site-packages/click/core.py", line 534, in invoke
    return callback(*args, **kwargs)
  File "/Users/alancoding/.virtualenvs/cli2/lib/python2.7/site-packages/ansible_tower_cli-2.3.1-py2.7.egg/tower_cli/utils/decorators.py", line 49, in answer
    return method(*inner_a, **inner_kw)
  File "/Users/alancoding/.virtualenvs/cli2/lib/python2.7/site-packages/ansible_tower_cli-2.3.1-py2.7.egg/tower_cli/models/base.py", line 287, in func
    result = method(*args, **kwargs)
  File "/Users/alancoding/.virtualenvs/cli2/lib/python2.7/site-packages/ansible_tower_cli-2.3.1-py2.7.egg/tower_cli/resources/organization.py", line 34, in associate
    return self._assoc('users', organization, user)
  File "/Users/alancoding/.virtualenvs/cli2/lib/python2.7/site-packages/ansible_tower_cli-2.3.1-py2.7.egg/tower_cli/models/base.py", line 731, in _assoc
    url = self.endpoint + '%d/%s/' % (me, url_fragment)
TypeError: %d format: a number is required, not NoneType

Strange. I tested it and could not replicate the issue:
(tower_cli_devel) sitan-OSX:tower-cli sitan$ tower-cli organization associate --user test_user
Error: Could not get user. The requested object could not be found.

Give it a valid user and I bet you'll see it too.


Added interval parameter to Position

created interval parameter and a test for the new parameter

xref: #94

Thanks for this pull request @mwarnick123 ! I'm trying to understand if, after merging this, we should add this to all properties.
Also, the quality check is failing because you need to tox -e reformat the code.

Also, see discussion in #94


LicenseScan Tests failed

https://github.com/dotnet/sdk/pull/41016 is fixed the issues in runtime repo but caused other issues https://dev.azure.com/dnceng/internal/_build/results?buildId=2456048&view=results in arcade, aspnetcore, roslyn-analyzers and source-build-reference-packages repo
arcade repo:
UpdatedLicenses.arcade.json
scancode-results.json
roslyn-analyzers repo:
UpdatedLicenseExclusions.roslyn.txt
scancode-results.json

@ellahathaway - Can you take a look at this? Something weird is going on. Just looking at the arcade result, I see this in UpdatedLicenses.arcade.json:
{
  "files": [
    {
      "path": "eng/xcopy-msbuild/msbuild.nuspec",
      "detected_license_expression": "ms-visual-2015-sdk"
    }
  ]
}

That happens in spite of an exclusion existing for that file: https://github.com/dotnet/dotnet/blob/6a63fb085655a71e23a637cd3d37111a32da1894/test/Microsoft.DotNet.SourceBuild.SmokeTests/assets/LicenseScanTests/LicenseExclusions.txt#L20

Hm, that's very odd. It shouldn't cause an error if the exclusion is listed. I'll investigate.

Oh - it's because the linked run is my test run off of my Azdo branch, not the latest run off of main. I think we can close this issue.


ARROW-8961: [C++] Add utf8proc library to toolchain

@xhochy I do not see utf8proc being built here:
https://ci.ursalabs.org/#/builders/84/builds/9077/steps/4/logs/stdio

@xhochy do you want me to rebase https://github.com/apache/arrow/pull/7449 on this? So we can see if it's all working?

@xhochy do you want me to rebase #7449 on this? So we can see if it's all working?

Yes, that would make sense. Might also show me better what the issue is that @pitrou pointed out.

@xhochy I do not see utf8proc being built here:
https://ci.ursalabs.org/#/builders/84/builds/9077/steps/4/logs/stdio

Was an upper- vs lowercase issue. Now it errors out.

Would also be good to be sure we're not switching to unilib https://github.com/apache/arrow/pull/7449#issuecomment-645023981 before merging this.

Would also be good to be sure we're not switching to unilib #7449 (comment) before merging this.

It will be nearly the same PR. If this one works, we can replace all utf8proc occurences with unilib.

Appears that unilib is a no go. I'm not a fan of one-developer projects anyway

@ursabot build

@maartenbreddels This is passing now, feel free to rebase again.
@pitrou @kou @kszucs Any suggestions for Crossbow jobs I should run here?

@github-actions crossbow run -g cpp -g wheel

@github-actions crossbow submit -g cpp -g wheel

@kszucs Do we have some kind of race condition with github-actions/crossbow. The jobs were never run but the linked branch is actually from a different PR where I summoned the bot nearly at the same time.

@github-actions crossbow submit -g cpp

Valgrind failure looks totally unrelated: https://github.com/ursa-labs/crossbow/runs/788860352?check_suite_focus=true

@github-actions crossbow submit -g wheel

@github-actions crossbow submit test-debian-10-cpp

@github-actions crossbow submit -g linux

@github-actions crossbow submit centos-8-*

+1 for Linux packages.

@github-actions crossbow submit -g linux conda

@github-actions crossbow submit -g conda-*

@github-actions crossbow submit conda-*

@maartenbreddels Feel free to rebase again, this patch is ready for a merge.
@wesm Should we merge this as-is or is delay it until @maartenbreddels branch is ready?

@xhochy I'm fine with merging this -- it's a non-mandatory dependency right (sorry have not reviewed the patch yet and the PR description does not say)?

TODO:

Need the UTF8PROC_STATIC definition with bundled utf8proc
Need to add include path for utf8proc.h

+1

Once the utf8_lower/utf8_upper patch lands I am going to make utf8proc not mandatory. See ARROW-9220.


Add a blacklist option to ignore some games

Would be a great option because some don't require to be saved (online games, for example)

Good idea! I'll look into this.

Thanks ! and tyvm for your work, it is a great app.

An easy implementation of this might be to ignore any games with a specific tag and then let us tag any game with a phrase like "nobackup".

This is now available in v0.3.0 by creating a ludusavi-skip tag.
@N0tJust1n Good call on the tag suggestion :D Much better than what I was originally imagining.

Didn't know this plugin had such an option.
For better UX, please add a tick like this:

[ ] Remember my choice

To dialog window like this:

@jerrygreen Thanks for the suggestion! I've made a separate ticket to track that idea: https://github.com/mtkennerly/ludusavi-playnite/issues/31


GH-44765: [Python][Parquet] ParquetDataset should expose partition_base_dir

Rationale for this change
Currently ParquetDataset object can receive partition as a parameter, but it's not possible to set a partition_base_dir. We need to expose this in the constructor parameters and pass it in the ds.dataset call.
What changes are included in this PR?

We add a partition_base_dir parameter to the constructor
pass it down to ds.dataset

Are these changes tested?
no test introduced - functionality should be covered by the tests of ds.dataset
Are there any user-facing changes?
the interface for the class constructor changed, but no breaking changes. documentation updated accordingly.

GitHub Issue: #44765

@kou Could you check the changes, it's a very minor change - the tests for the functionality should be covered by ds.dataset's tests. we're just exposing it to the constructor of ParquetDataset

@pitrou Do we need a test for this change?

cc @jorisvandenbossche


Upgrade quinn to 0.10

This PR is an attempt at updating to quinn 0.10.
As of the initial push, there are three unit tests that are failing:

quic::endpoint::test::close_incoming: I cannot figure out how to get quinn to reject incoming connections, now that IncomingBiStreams isn't a thing anymore.
quic::endpoint::test::close/quic::endpoint::test::close_and_reuse: time out due to not getting None from a closed server.

Closes #59, #60, #61.

Today I attempted to add two different signals to the incoming connection task: full shutdown vs shutdown incoming. This allowed me to have a loop that called accept() and then immediately drop the Connecting. There are no functions on Connecting to refuse the connection.
Unfortunately, despite the immediate drop, the client still considers it a success and returns Ok.
It feels messy, but I think the only way to get quinn to reject connections is to rebind the server to a new unknown address... but that doesn't actually stop the listening, it just moves it somewhere else. I can't see any way to actually disable incoming connections, and as such, I'm not sure fabruic's existing API can continue to work as-designed.

There are no functions on Connecting to refuse the connection.

FWIW this seems like a reasonable thing for us to have.

As per Discord, I'm merging without waiting for a review. I'm not going to release an update with this code until I've been able to run the BonsaiDb test suite against it -- which requires me to update a lot of dependencies now that I can update to the current rustls.
The only thing I'm not happy with is that I ended up manually implementing futures when I felt like I should have been able to get select/select_biased to work correctly.


#150798547 Fix missing front tests

What does this PR do?
Adds more tests to the hc/front app
Description of Task to be completed?
Have the following tests working:

test team access works for add_channel

test add_channel fails on bad channel kinds

test team access doesn't work for non-team members for add_channel

test team access doesn't works for team members for add_channel

test team access works for add_checks

test team access doesn't work for non-teams for add_checks

test add_pushover validates invalid priority

test add_pushover validates valid priority

How should this be manually tested?
After cloning the repo, cd into it and RUN python manage.py test hc/front/tests
Any background context you want to provide?
These tests have been carried using Python 3
What are the relevant pivotal tracker stories?
#150798547

Coverage increased (+0.2%) to 80.405% when pulling cf29206f70e91db7bc612b07228a5885344d2b3c on bg-front_tests-150798547 into 252752e61bdc671984b697e0242f778a29b8c236 on develop.

Coverage increased (+0.2%) to 80.405% when pulling 5f7b2aef83da5e4ca577c2e42992f40c2df65849 on bg-front_tests-150798547 into 252752e61bdc671984b697e0242f778a29b8c236 on develop.


HDXDSYS-899 fix failing 3W

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

32 of 33 (96.97%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 93.058%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/hapi/pipelines/utilities/mappings.py
9
10
90.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.1%

Covered Lines:
1354

Relevant Lines:
1455

💛  - Coveralls


antigenic site annotation

It would be good to have the antigenic site annotation incorporated with the aadiff command - or as a separate option. Would this be possible by the next VCM meeting? Or - continue using excel lookup tables...

@tkanderson It is quite doable. I think it would be reasonable to include any published sets of antigenic sites in the tool itself and also to allow custom tables. For example, we might have a --caton82 flag for H1 and an --annotation=myfile.tsv annotation for your custom annotations. Each annotation could have its own column.


constructor naming

i just made a method for a submatrix (which i will need for def pos tester, which i need for choleski), there will be one to get the diagonal on so on. So when you write $matrix.submatrix or $marix.diagonal all is fine. But our current diagonal or zero and more i plan to do step a bit out of the scheme because $matrix.zero(...) creates a new which has nothing to do with the old.
thatswhy i propose as a first step to rename zero to new-zero($r,$c) which will be shortcut for new( :@size => ($r,$c), :$defaultvalue => 0).

I have some question about submatrix first, that you can see in the pull request, inline with code
But, it the direction, i agree.
Thanks

since you also prepare some changes i will now restrict myself and only propose here next constructor additions so we can complete that before stepping to accessors. when is that stable code wil shrink and get cleaner (as with my new trace method using diagonal accessor)

since you also prepare some changes i will now restrict myself and only propose here next constructor additions so we can complete that before stepping to accessors. when is that stable code wil shrink and get cleaner (as with my new trace method using diagonal accessor)

what i basically plan is an optional arbument rows and colums but as hash so  it can only hold some indexes but i want to clear first if we want to go 0 based or 1 based.

I can see the point of going on 1 indexing, however, we are still talking about perl, i think 0 makes more sense.
For instance, PDLA with Perl5 is indexing from 0 as well, and from what i have seem, so is numpy in python


Ppp

Codecov Report

Merging #161 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #161   +/-   ##
=========================================
  Coverage     75.68%   75.68%
  Complexity      881      881
=========================================
  Files           125      125
  Lines          2575     2575
  Branches        291      291
=========================================
  Hits           1949     1949
  Misses          520      520
  Partials        106      106

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 c403a5f...ced1600. Read the comment docs.


Unable to create merge-request: mr_create.go:105: gitlab project not found

This likely occurs because the repo name is strange.
I was able to lab fork foo/SomeHorribleRepoName.com
I was able to push.
But then, when I tried lab merge-request I got...
mr_create.go:105: gitlab project not found
Likewise lab mr list gives the same error.
I don't have control over the server as I'm a contractor hitting an internal instance over vpn.  If this is because of the . character in the repo name, I realize the handling responsibility shouldn't traditionally be on lab to do it.
However, from a user perspective, it feels reeeeally off that one can successfully do some stuff, but not others when the main entry point (in terms of user scenario) for working with lab starts with commands that will work (i.e. lab clone ... and lab fork ...).
At the very least, it would be nice to have a check for this problem that occurs when using those "entry point" commands that warns the user if they hit this scenario.

Improving this would be a great UX improvement, given all these issue reports. I stepped into this myself for one project, whereas antoher project works.


Remove CreateOrganization method from scm package.

CreateOrganization in scm package is only used by the mock fake scm client and can be initialized when creating a mock scm instead of keeping a dedicated method.

Closing as already fixed.


Reusable resize container

Closes #2959
Depends on https://github.com/camunda/camunda-modeler-design/issues/90

moves properties panel resize handlers to reusable component (supported positions for now: right, bottom)
use resizable container for bottom panel

https://camunda-modeler-on-demand.s3.eu-central-1.amazonaws.com/2959-resize-container/camunda-modeler-2959-resize-container-linux-x64.tar.gz
https://camunda-modeler-on-demand.s3.eu-central-1.amazonaws.com/2959-resize-container/camunda-modeler-2959-resize-container-mac.dmg
https://camunda-modeler-on-demand.s3.eu-central-1.amazonaws.com/2959-resize-container/camunda-modeler-2959-resize-container-mac.zip
https://camunda-modeler-on-demand.s3.eu-central-1.amazonaws.com/2959-resize-container/camunda-modeler-2959-resize-container-win-ia32.zip
https://camunda-modeler-on-demand.s3.eu-central-1.amazonaws.com/2959-resize-container/camunda-modeler-2959-resize-container-win-x64.zip

I tried to remove the "globe" drag image on resizing. Apparently, that's not an easy task.

What I read in various docs, event.dataTransfer.setDragImage should be the way to go. It seems we already tried to fix this in the past via our dragger util
event.dataTransfer.setDragImage(emptyCanvas(), 0, 0);

Which doesn't work in the current versions; the drag preview also looks different in older versions (although we didn't change the behavior).

Therefore I'd propose to de-scope this issue from this PR and fix it in an orderly manner.

This is ready for review now. I just added @andreasgeier for a design review (if he finds the time).

@pinussilvestrus Thank you for taking this issue.
The "globe" on drag is annoying, we should tackle that soon as well
The click button (the grey icon) is not vertically centered
I had a situation in which the click button does not reappear on closed state - but cannot reproduce
There seem to be 2 border-top on the panel

as is
expected

While we have quite large snapping distance for the properties panel (which makes sense because of the contents), we could reduce the snapping points for the vertical resizing to, say, 200px. So, a user can minimize the view while still. keeping it open.

@pinussilvestrus Thank you for taking this issue.
The "globe" on drag is annoying, we should tackle that soon as well The click button (the grey icon) is not centered I had a situation in which the click button does not reappear on closed state - but cannot reproduce
There seem to be 2 border-top on the panel
as is	expected

I removed the second top border (it was a left over from the bottom panel)
I set the toggle left position to calc(50%)

The minimal height is already set to 200px

ok, so maybe 100 then. I think the min height is too much currently

I set the toggle left position to calc(50%)

Didn't check it yet but calc(50%) is actually not correct. You need to consider the width of the button, too, and subtract 50% of that number, to be precise. Or better, use transform:translateX(50%) on the button.

For reference: min height of bottom panel in VSCode:

I adjusted both the min-height (100px) and the vertical position of the bottom toggle. It should be centered now. Updated builds are in the making. 👍

Do we expect to be able to close the container by a simple click?

As I understand, the toggle is only visible when the container is closed. Therefore you can't use it to close. The test case (should toggle (close) is just testing the API, but not possible via UI.

When the log is open and you collapse the panel, you get to a weird state where errors is shown twice, even though the panel is closed

Weird stuff 😆  I will have a look, probably just a CSS issue

Do we expect to be able to close the container by a simple click?

As I understand, the toggle is only visible when the container is closed. Therefore you can't use it to close. The test case (should toggle (close) is just testing the API, but not possible via UI.

Ah - got it. Thanks

When the log is open and you collapse the panel, you get to a weird state where errors is shown twice, even though the panel is closed:

I fixed this. However, now I see more problems coming up with the Log and the Panel open at the same time:

One way to fix it would be to make the maximum height of the resizable panel depending on the Log. Still resizing the Log when the Panel is on maximum height would cause the same trouble. This is also hacky, as both panels shouldn't know about each other and work independently.
For me, it seems like a better fix would be to re-arrange the Log (cf. https://github.com/camunda/camunda-modeler-design/issues/90). I'm inclined to move this one to Draft until we have a proper solution about the Panel - Log dependency.

This was fixed by https://github.com/camunda/camunda-modeler/pull/3509.


Use a redistributable version of electron.

Download electron from our S3 storage.

Coverage remained the same at 41.949% when pulling fbf92dd58fcbb2e141822f72109071d223c5f3e0 on Mossop:redistributable into 71131e9f815d8b43b77e3a5f23ec736291ee360a on mozilla:master.

Coverage remained the same at 41.949% when pulling fbf92dd58fcbb2e141822f72109071d223c5f3e0 on Mossop:redistributable into 71131e9f815d8b43b77e3a5f23ec736291ee360a on mozilla:master.


template folder updated with new pages

index, add, item updated
navbar, images, annotation js added
category cho pages created

closing this one as a new updated one is already open


Added tests for RegisterDomain, ListDomain and DescribeDomain

What changed?
Tests:

Register domain
List domain
describe domain

Why?
As part of test coverage week

How did you test it?
Unit test

Potential risks
Tests to avoid risk

Release notes
N.A.

Documentation Changes
Not required

Pull Request Test Coverage Report for Build 018eb7d2-563d-48f5-a99a-287c3b9c5e4f
Details

0 of 0   changed or added relevant lines in 0 files are covered.
55 unchanged lines in 11 files lost coverage.
Overall coverage increased (+0.01%) to 67.294%

Files with Coverage Reduction
New Missed Lines
%

service/matching/taskReader.go
2
84.88%

common/task/weighted_round_robin_task_scheduler.go
2
88.56%

common/persistence/sql/sqlplugin/mysql/task.go
2
73.68%

service/history/task/transfer_active_task_executor.go
2
72.83%

service/matching/matcher.go
2
90.72%

common/persistence/sql/sqlplugin/mysql/db.go
2
79.49%

common/persistence/historyManager.go
2
66.67%

service/history/queue/timer_queue_processor_base.go
3
77.82%

service/history/task/fetcher.go
5
85.57%

service/history/task/cross_cluster_task_processor.go
8
80.79%

Totals

Change from base Build 018eb4d2-dd06-4921-a558-dd33d923e1a6:
0.01%

Covered Lines:
98235

Relevant Lines:
145979

💛  - Coveralls


Move 'me' related calls to own endpoint class

I found the mix in the Users endpoint class a little confusing and decided to pull out all me related calls to its own endpoint class.

I don't actively use the library myself, but when going through it the last days it seemed a little unintuitive to have the me endpoint calls mixed in with the users endpoint calls, not knowing on which of both of the endpoints the calls will be made.
What do you think @LeviHassel @roisnir @hayribakici ?

Yeah I agree, separating those endpoints is a good idea


Update max version of bazel supported by tensorflow to 4.1.0

For fix the issue: https://github.com/tensorflow/tensorflow/issues/50966#issuecomment-888748570
I'm a member of Microsoft VCPKG team, we recently upgraded the version of bazel used in vcpkg to 4.1.0, then tensorflow build failed with following error:
You have bazel 4.1.0 installed.
Please downgrade your bazel installation to version 3.99.0 or lower to build TensorFlow! To downgrade: download the installer for the old version (from https://github.com/bazelbuild/bazel/releases) then run the installer.

This issue is due to the fact that tensorflow has set the maximum supported version of bazel to 3.99.0 on line 53 of the https://github.com/tensorflow/tensorflow/blob/master/configure.py file.
For fixing this issue, update the max version of bazel supported by tensorflow to 4.1.0.

@googlebot I signed it!

@googlebot I signed it!

@googlebot I signed it!


How to edit/delete saved card.

Greetings
I am using your package and everything is working fine but I am facing issues with saved card drop-in UI is it possible to edit/delete saved card in drop-in UI.
Like: https://developers.braintreepayments.com/guides/drop-in/customization/javascript/v3
"Delete a saved payment method"

@srikanthonl  could you please add the below code:
this.dropinConfig.vaultManager = true;
Line No : 220
File : ngx-braintree/fesm5/ngx-braintree.js
So that it can work with npm run.

This would be a great addition to have.


Learn to troubleshoot RKE-CI tests (Deploy Epinio on RKE2 + MetalLB)

The purpose of this task is to be able run epinio/acceptance/test/scenario3_test.go either locally or vm in case is needed for troubleshooting.
Somethings to learn in the the way:

How to fully deploy RKE2 locally
How to deploy MetalLB and bind it to RKE2

Applied make tools-install
Installed RKE2 applying:

curl -sfL https://get.rke2.io | sudo INSTALL_RKE2_METHOD=tar sh -
          # Allow non-root user to read KUBECONFIG file
          sudo sh -c "echo RKE2_KUBECONFIG_MODE=0644 > /etc/sysconfig/rke2-server"
          # Disable RKE integrated ingress-nginx controller
          sudo sh -c "mkdir -p /etc/rancher/rke2/config.yaml.d"
          sudo sh -c "echo 'disable: rke2-ingress-nginx' > /etc/rancher/rke2/config.yaml.d/00-epinio.yaml"
          # Start RKE cluster
          sudo systemctl enable --now rke2-server
          # Wait a "little" to let enough time for RKE cluster to be up and running
          sleep 120

Note: I had to manually create /etc/sysconfig/rke2-server"
Result:
 epinio git:(main) kubectl get svc -A
NAMESPACE      NAME                        TYPE        CLUSTER-IP      EXTERNAL-IP   PORT(S)         AGE
cert-manager   cert-manager                ClusterIP   <IP_ADDRESS>     <none>        9402/TCP        43m
cert-manager   cert-manager-webhook        ClusterIP   <IP_ADDRESS>   <none>        443/TCP         43m
default        kubernetes                  ClusterIP   <IP_ADDRESS>       <none>        443/TCP         65m
kube-system    rke2-coredns-rke2-coredns   ClusterIP   <IP_ADDRESS>      <none>        53/UDP,53/TCP   64m
kube-system    rke2-metrics-server         ClusterIP   <IP_ADDRESS>    <none>        443/TCP         64m

When I try to execute locally ➜  epinio git:(main) ginkgo -v --focus Scenario3 acceptance/install/ , I get:
➜  epinio git:(main) ginkgo -v --focus Scenario3 acceptance/install/
Running Suite: Acceptance Suite - /home/mmartin/suse/epinio-repos/epinio/acceptance/install
===========================================================================================
Random Seed:<PHONE_NUMBER>

Will run 1 of 6 specs
------------------------------
[SynchronizedBeforeSuite]
/home/mmartin/suse/epinio-repos/epinio/acceptance/install/suite_test.go:68
STEP: Installing and configuring the prerequisites 07/25/22 10:10:13.714
REGISTRY_USERNAME or REGISTRY_PASSWORD environment variables are empty. Pulling from dockerhub will be subject to rate limiting.
STEP: Compiling Epinio 07/25/22 10:10:13.714
STEP: Creating registry secret 07/25/22 10:10:14.31
STEP: Installing cert-manager 07/25/22 10:10:14.31
STEP: Installing ingress controller 07/25/22 10:10:16.316
REGISTRY_USERNAME or REGISTRY_PASSWORD environment variables are empty. Pulling from dockerhub will be subject to rate limiting.
------------------------------
[SynchronizedBeforeSuite] PASSED [2.603 seconds]
[SynchronizedBeforeSuite]
/home/mmartin/suse/epinio-repos/epinio/acceptance/install/suite_test.go:68

  Begin Captured GinkgoWriter Output >>
    STEP: Installing and configuring the prerequisites 07/25/22 10:10:13.714
    STEP: Compiling Epinio 07/25/22 10:10:13.714
    STEP: Creating registry secret 07/25/22 10:10:14.31
    STEP: Installing cert-manager 07/25/22 10:10:14.31
    STEP: Installing ingress controller 07/25/22 10:10:16.316
  << End Captured GinkgoWriter Output
------------------------------
<Scenario3> RKE, Private CA, Configuration, on External Registry
  Installs with private CA and pushes an app with configuration
  /home/mmartin/suse/epinio-repos/epinio/acceptance/install/scenario3_test.go:71
------------------------------
• [FAILED] [0.065 seconds]
<Scenario3> RKE, Private CA, Configuration, on External Registry [BeforeEach]
/home/mmartin/suse/epinio-repos/epinio/acceptance/install/scenario3_test.go:35
  Installs with private CA and pushes an app with configuration
  /home/mmartin/suse/epinio-repos/epinio/acceptance/install/scenario3_test.go:71

  Expected
      <string>:
  not to be empty
  In [BeforeEach] at: /home/mmartin/suse/epinio-repos/epinio/acceptance/install/scenario3_test.go:44
------------------------------
SSSSS
------------------------------
[SynchronizedAfterSuite]
/home/mmartin/suse/epinio-repos/epinio/acceptance/install/suite_test.go:106
------------------------------
[SynchronizedAfterSuite] PASSED [0.001 seconds]
[SynchronizedAfterSuite]
/home/mmartin/suse/epinio-repos/epinio/acceptance/install/suite_test.go:106
------------------------------

Ran 1 of 6 Specs in 2.669 seconds
FAIL! -- 0 Passed | 1 Failed | 0 Pending | 5 Skipped

The problem seems related with the IP needed to be included in the EPINIO_SYSTEM_DOMAIN.
If I understand correctly to get this, I need the MetalLB IP. The problem is that I am not sure how to get it. I have been reviewing the documentation directly from the official docs here: https://docs.rke2.io/install/ha/ but I am not sure I find the exact answer I am looking for.
@jimmykarily  or @nunix  or anyone else, any idea how exactly to extract the IP needed for?

Thanks

I don't know what the right way to install rke2 or metallb is but I don't see you running any ingress controller (e.g. Traefik). What would metallb give an IP address to?

thanks @jimmykarily .
The thing I was not sure myself either what to expect.
If I understood the rke.yaml script correctly, this part disables the default RKE nginx ingress controller:
   # Disable RKE integrated ingress-nginx controller
          sudo sh -c "mkdir -p /etc/rancher/rke2/config.yaml.d"
          sudo sh -c "echo 'disable: rke2-ingress-nginx' > /etc/rancher/rke2/config.yaml.d/00-epinio.yaml"

When I try to install Traefik manually it stays forever in a <pending> sate.
helm install traefik --namespace traefik "https://helm.traefik.io/traefik/traefik-10.3.4.tgz" --set globalArguments='' --set-string ports.web.redirectTo=websecure --set-string ingressClass.enabled=true --set-string ingressClass.isDefaultClass=true
Error from server (AlreadyExists): namespaces "traefik" already exists
WARNING: Kubernetes configuration file is group-readable. This is insecure. Location: /etc/rancher/rke2/rke2.yaml
WARNING: Kubernetes configuration file is world-readable. This is insecure. Location: /etc/rancher/rke2/rke2.yaml
NAME: traefik
LAST DEPLOYED: Mon Jul 25 13:44:34 2022
NAMESPACE: traefik
STATUS: deployed
REVISION: 1
TEST SUITE: None
➜  ~ k get svc -A
NAMESPACE      NAME                        TYPE           CLUSTER-IP      EXTERNAL-IP   PORT(S)                      AGE
cert-manager   cert-manager                ClusterIP      <IP_ADDRESS>     <none>        9402/TCP                     4h18m
cert-manager   cert-manager-webhook        ClusterIP      <IP_ADDRESS>   <none>        443/TCP                      4h18m
default        kubernetes                  ClusterIP      <IP_ADDRESS>       <none>        443/TCP                      4h40m
kube-system    rke2-coredns-rke2-coredns   ClusterIP      <IP_ADDRESS>      <none>        53/UDP,53/TCP                4h40m
kube-system    rke2-metrics-server         ClusterIP      <IP_ADDRESS>    <none>        443/TCP                      4h40m
traefik        traefik                     LoadBalancer   <IP_ADDRESS>   <pending>     80:31653/TCP,443:30112/TCP   14s

If try to get the IP as the script in rke.yml states it does not work for me:
➜  epinio git:(main) export RANGE_IP="$(scripts/get-free-ip.sh)"
Error: either "dev" is duplicate, or "wlp9s0" is a garbage.
➜  epinio git:(main) echo ${RANGE_IP}

So I am not quite sure how to proceed here

Note: I had to manually create /etc/sysconfig/rke2-server"

Are you running your test on a SUSE based OS? I don't think so if you had to manually create the /etc/sysconfig directory... The file itself is created with the redirection (the > sign) but of course the directory needs to exist first. So if you're not an an SUSE OS please use the correct one ;-)
Also, are you trying to run the test directly on your laptop? The IP get-free-ip.sh script tries to find an available IP on the LAN, but it has to be used either on the self-hosted runner or the script may need to be adapted to your config. Or if you know which IP address is free on your LAN you can set the RANGE_IP accordingly. For example, if you know that IP <IP_ADDRESS> is free then you can set to something like this: RANGE_IP=<IP_ADDRESS>-<IP_ADDRESS> (I'm writing this from memory, I didn't had a look at that test since some months).

Thanks @ldevulder,
Yes I noticed that /etc/sysconfig did not existed and created the directory. That was not an issue.
The problem is that I don't know what IP is free for the RANGE_IP. I can try one random, but I did not think that was the best way.  To me it was not clear that the get-free-ip.sh was only for self-hosted runner.
I will try a couple IP addresses like the above-mentioned and if not I will try in Azure straight.

The problem is that I don't know what IP is free for the RANGE_IP. I can try one random, but I did not think that was the best way.

Some LAN knowledge is needed yes ;-) You can try one random on your lab, this shouldn't hurt I think.

To me it was not clear that the get-free-ip.sh was only for self-hosted runner.

Again, you can use it on your lab I think, but with some adjustment. For example the error message you have (Error: either "dev" is duplicate, or "wlp9s0" is a garbage.) looks like because the interface used is not good (but I didn't checked the code). And again this code ise for SUSE OS, don't know which one you are using, but as /etc/sysconfig wasn't created clearly it's not a SUSE one. I just hope that you aren't trying to run this test directly on MacOS for example (or WSL), as it will clearly not work!

As I wrote in our chat, please check that fping command is installed on you lab, it might be the issue here. And if it's because of this I think that you can add a short test at the beginning of the script to be sure that fping is installed, and if not raise an error to stop the test here, could be helpful ;-)

No, I am not trying to test on MAC, I am on Ubuntu 22.04. As commented earlier, I know that /etc/sysconfig was not created there (I did not know back then that it was because it was meant for SUSE OS) but I did created  manually and the rke2-server was placed there:
  sysconfig pwd
/etc/sysconfig
➜  sysconfig ls
rke2-server

So the key thing here is the RANGE_IP. The range RANGE_IP=<IP_ADDRESS>-<IP_ADDRESS> does not seem to work, but I totally get your point is  difficult to guess.
I will try the fping now. Thanks

Further work locally in Ubuntu did not worked. Created a VM with OpenSUSE Tumbleweed and there it seemed to worked correctly finding and setting the RANGE_IP. The issue now seems pulling the registryUsername. I will keep on my own

I bypassed the registries by hardcoding the credentials. Managed to install Metallb and Traefik.
Now the problem that I have is installing Epinio at the end of the script. The problem seems located at /epinio/acceptance/helpers/epinio/epinio.go
	opts := []string{
		"upgrade",
		"--install",
		"-n",
		"epinio",
		"--create-namespace",
		"epinio",
		"helm-charts/chart/epinio",
		"--wait",
	}

In particular seems an issue with this part "helm-charts/chart/epinio",

@jimmykarily sorry to bother, but  since I saw you worked a bit with that part do you foresee what am I missing there?
Thanks

Previsouly (didn't checked) helm-charts/chart/epinio was in the epinio/epinio git repo, so was clone locally. Did you clone the epinio/epinio repo? Looks like the issue IMO.

Previsouly (didn't checked) helm-charts/chart/epinio was in the epinio/epinio git repo, so was clone locally. Did you clone the epinio/epinio repo? Looks like the issue IMO.

thanks @ldevulder . Yes, I cloned it epinio/epinio straight so the helm-charts/chart/epinio/ was empty. Thanks for the hint

As far as I remember helm-charts is a git submodule, so not cloned by default I think, you havbe to force the cloning (but I don't remember the exact command ;-)).

Charts were cloned later.
Still had issues and had a timeout when installing from helm

Investigating a bit further there is a pod rke2-canal-tn54q (calico-node ) in bad state

Later, checking the nodes one of them, localhost.localdomain seems not healthy:

I disabled later rke2 from my own system outside the vm in case it could interfere. Did not helped
I tried to delete both nodes using kubectl delete nodes --all and I was not able.
What worries more is that I tried to uninstal rke2 from vm and script rke2-uninstall.sh seems not present. So I start to wonder if I may have some missconfigurations.

Indeed the calico pod in bad state indicated an issue. Before everything I had to configure the Networkmanager as specified here, creating a rke2-canal.conf file with the following content
[keyfile]
unmanaged-devices=interface-name:cali*;interface-name:flannel*

This worked and now all the pods are in healthy state and the installation succeeds.
Hit another rock in a further step with the credentials:
  Begin Captured GinkgoWriter Output >>
    STEP: Installing MetalLB 07/27/22 16:27:17.046
    STEP: Checking LoadBalancer IP 07/27/22 16:27:18.868
    STEP: Configuring local-path storage 07/27/22 16:27:18.963
    STEP: Creating private CA issuer 07/27/22 16:27:19.223
    STEP: Installing Epinio 07/27/22 16:27:19.391
    STEP: Connecting to Epinio 07/27/22 16:27:40.163
  << End Captured GinkgoWriter Output

  Timed out after 120.009s.
  Expected
      <string>:
      🚢  Login to your Epinio cluster [https://epinio.<IP_ADDRESS>.omg.howdoi.website]

      ✔  Trusting certificate...

      ❌  error verifying credentials: error while connecting to the Epinio server: cannot parse JSON response: 'Bad Gateway': invalid character 'B' looking for beginning of value

  to contain substring
      <string>: Login successful
  In [It] at: /epinio/acceptance/install/scenario3_test.go:140

I created this document with specific steps on how to deploy and troubleshoot RKE_CI tests: RKE-CI notes on local troubleshooting
Aside from this,  I could not find why in the latest part of the execution, Epinio could not be installed. Actually Epinio got installled but later removed. The cluster is healthy, Traefik is correctly installed with a previous autogenerated IP <IP_ADDRESS>, matching the range within metallb file resources.yml. I paired with @jimmykarily to troubleshoot it but it was not clear what was happening (thanks @jimmykarily  for the help).

mmartin24@mmartin-thinkpad-p15-gen-2i:/epinio> kubectl get svc -A
NAMESPACE      NAME                        TYPE           CLUSTER-IP      EXTERNAL-IP       PORT(S)                      AGE
cert-manager   cert-manager                ClusterIP      <IP_ADDRESS>     <none>            9402/TCP                     17h
cert-manager   cert-manager-webhook        ClusterIP      <IP_ADDRESS>     <none>            443/TCP                      17h
default        kubernetes                  ClusterIP      <IP_ADDRESS>       <none>            443/TCP                      18h
kube-system    rke2-coredns-rke2-coredns   ClusterIP      <IP_ADDRESS>      <none>            53/UDP,53/TCP                18h
kube-system    rke2-metrics-server         ClusterIP      <IP_ADDRESS>    <none>            443/TCP                      18h
metallb        metallb-webhook-service     ClusterIP      <IP_ADDRESS>   <none>            443/TCP                      17h
traefik        traefik                     LoadBalancer   <IP_ADDRESS>     <IP_ADDRESS>   80:32754/TCP,443:32116/TCP   17h

The main purpose of understanding better how this acceptance test works is fulfilled. Unfortunately, I cannot spend more time analyzing what is happening here in order to focus in other issues.
Closing this ticket


mgmt: ec_host_cmd: add config to create a dedicated thread

Add a config to decide if a new dedicated thread for Host Command is created during initialization.
If not, the ec_host_cmd_task has to be called by another thread to handle host commands.

Ohh, I've uploaded an incorrect commit, sorry.


Add KubeOrbit

Pre-submission checklist:
Please check each of these after submitting your pull request:

[x] Are you only including a repo_url if your project is 100% open source? If so, you need to pick the single best GitHub repository for your project, not a GitHub organization.
[x] Is your project closed source or, if it is open source, does your project have at least 300 GitHub stars?
[x] Have you picked the single best (existing) category for your project?
[x] Does it follow the other guidelines from the new entries section?
[x] Have you added your SVG to hosted_logos and referenced it there?
[x] Does your logo clearly state the name of the project/product and follow the other logo guidelines?
[x] Does your project/product name match the text on the logo?
[x] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[x] ~15 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

@ericjee can you update the PR and put it in alphabetical order :)?

@caniszczyk thx for the reminder, updated:)

LGTM


Let's rename project to something like intellij-memory-toolbox-plugin

As we want our plugin to support more features than just gcViewer wrapper, let's find a more appropriate plugin name, something like intellij-csv-validator

Plus it'll be better discoverable via search engines)

Or maybe even intellij-performance-toolbox-plugin as GC and memory leaks could affect overall performance.

Renamed


Using 4.23.1 and failed to compile.

I copied whole folder into Plugins in my Unreal project and tried to compile but got an error message.
ERROR: Could not find definition for module 'ShaderCore', (referenced via Target -> StreetMapRuntime.Build.cs)

Bumping because I have the same error.

Giving this another nudge as I too am having the same issue.

Anyone figure this out? Running into same prob, even when running on 4.22

Same problem on 4.26

But, best to use the release from GameInstitute which is updated for 4.26. Got that working in the end


[FEATURE REQUEST] Cross-platform support

I would love to see another option for chart editors that can also be used on Mac and Linux (especially since wine doesn't play nice with newer .NET versions). Grepping through the code, I didn't see too much windows specific that would have to be replaced outside of compiler options and I guess window creation, but I also have little to no C# (and especially MonoGame) experience, so I can't tell how much work it would be to rewrite the windows specific parts to work cross-platform.

A few folks have mentioned they'd like this feature. It's not an insurmountable problem, but it also wouldn't be quick. Some considerations:

C# and dot net are multiplatform, so I'm not too worried about that.
Monogame supports Linux and macOS but I imagine there will be a lot of bumps in the road getting that set up.
Currently the application uses DirectX. It would need to swapped to something like OpenGL. Monogame should make this not too bad.
Shaders are in HLSL but I think Monogame transpiles them so this may not be an issue.
Would need to do a pass on the third party libraries but that should really just be fmod and Dear ImGui and I don't think either of those would be problematic.
Some assumptions sprinkled throughout the application that it is on Windows. Window resizing and drag+drop handling come to mind.
This is just a personal obstacle but I'd need to set up a development environment in Linux/macOS in order to do any of this work and test anything, and I don't currently have that set up.

Perhaps some day when I have a lot of time to devote to a real meaty task I can begin tackling this but just to be realistic I'm not sure I'll have the bandwidth for it any time soon. Though I am very open to the idea of others contributing to the project to work on this task if they would like. Someone reached out to me on Discord recently about potentially taking on this work but they may not have the time for it. I'm always happy to talk through these issues though and offer my opinions on how to approach the problem.


argp-standalone: init at 1.3

Motivation for this change
This library ships the argp.h header for the argparse lib. Lack of it can result in build issues with certain packages on Mac (notably, the goxel nixpkg fails on macOS builds partly because of lack of argp.h).
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

can't currently run nix path-info -S on macOS - returns error on master.
this library is also available for linux (which is why I wrote the conditional fetchpatch), but I can't test it there. I suspect that these patches will be required to get it compiling on linux.
this PR includes a commit I made adding myself to the maintainers list, hope that's ok.

Useful for musl as well!

Anything else I can do to get this merged? Not sure who to @

Let's see how well it builds (as a formality)
@GrahamcOfBorg build argp-standalone

No attempt on x86_64-linux (full log)
The following builds were skipped because they don't evaluate on x86_64-linux: argp-standalone
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: argp-standalone
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-darwin (full log)
Attempted: argp-standalone
Partial log (click to expand)
make[2]: Entering directory '/private/tmp/nix-build-argp-standalone-1.3.drv-0/argp-standalone-1.3/testsuite'
make[2]: Nothing to be done for 'install-exec-am'.
make[2]: Nothing to be done for 'install-data-am'.
make[2]: Leaving directory '/private/tmp/nix-build-argp-standalone-1.3.drv-0/argp-standalone-1.3/testsuite'
make[1]: Leaving directory '/private/tmp/nix-build-argp-standalone-1.3.drv-0/argp-standalone-1.3/testsuite'
post-installation fixup
strip is /nix/store/g5r4apl0za012ffs6ladinwa5w0m1l3k-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/nmdkpkn051zarc54vzsc88cbwxh3f2gx-argp-standalone-1.3/lib
patching script interpreter paths in /nix/store/nmdkpkn051zarc54vzsc88cbwxh3f2gx-argp-standalone-1.3
/nix/store/nmdkpkn051zarc54vzsc88cbwxh3f2gx-argp-standalone-1.3

@LnL7 any thoughts? (Seems to be a darwin-exclusive.)

@samueldr i admit the expression as written is only for darwin, but argp itself works on linux too. I just don't have access to a linux box yet to test the compilation there.

I didn't try to build something with it but it looks good to me.

Thanks! As I hopefully mentioned previously this is useful for a few
packages when using musl as libc, so it has uses on Linux-musl anyway
:).
Thanks for seeing this through, apologies for not giving it more
attention earlier--it's been in my TO-REVIEW pile for some time now,
never gave up just got busy :3.
On Mon, 12 Nov 2018 15:52:08 -0800, Samuel Dionne-Riel<EMAIL_ADDRESS>wrote:

Merged #48738 into master.
--
You are receiving this because you commented.
Reply to this email directly or view it on GitHub:
https://github.com/NixOS/nixpkgs/pull/48738#event-1961665626Non-text part: text/html


Modify login.js

the document.location.replace should direct to /dashboard instead of /profile.

Done!


Archaius 2.x

WORK IN PROGRESS
This is the initial implementation of Archaius v2 without a dependency on apache configuration. This implementation is far simpler, has no static state and uses Typesafe Config for loading of external resources.
This code is not yet at feature parity with 1.0 but is at a state where we can start discussing the merits of this approach. In addition we will be creating an archaius-legacy subproject with bindings to the older static API.

archaius-pull-requests #119 FAILURE
Looks like there's a problem with this pull request

archaius-pull-requests #120 FAILURE
Looks like there's a problem with this pull request

:+1: looking forward to this.


Bug in function oapply

Julia 1.9, Catlab v0.14.17
When I run the following lines of code, I see an error Feet of spans are not equal: FinSet(3) != FinSet(4775367456).
using Catlab, Catlab.CategoricalAlgebra, Catlab.Programs, Catlab.WiringDiagrams

wd = @relation (x,) begin
    f(x, y)
end

span = Multispan([
    FinFunction([1, 2, 3]),
    FinFunction([1, 2, 3])])

oapply(wd, [span])

It looks like there is something wrong with line 52 in this file. When Ob = FinSetInt, the function call isassigned(junction_feet, j) is always true.

Thanks for the report. I can also reproduce this bug on Catlab master. My guess is that it's actually a bug in ACSets.jl but we'll see.

Nevermind, it's a bug in the oapply method itself. I'm posting a fix.


Go top after click on page

Hello
after click on the page button, how i can move to top page?

If you want to move to the first page, you can just call container.pagination('go', 8) or container.pagination(8)
ref: http://pagination.js.org/docs/index.html#go

I think @Lowell130 meant he wants to get automatically scrolled to the top of the container. So when you click on 2, 3, etc. you start seeing the first results of the new page instead of it keeps you hanging at the bottom of the container (where the pagination is).

afterRender: function() {
$('html').animate({
scrollTop: ($("body").offset().top)
}, 150);
}
@Lowell130 @eaglejohn  : I used the afterRender method to manually trigger scroll to top

afterRender: function() {
    $('html').animate({
          scrollTop: ($("body").offset().top)
     }, 150);
}

@Lowell130  @eaglejohn  : I used the afterRender method to manually trigger scroll to top


[1Password] ...

Extension
https://www.raycast.com/khasbilegt/1password
Description
I want to create a sharable link for an item with a default configuration set.
Who will benefit from this feature?
Anyone who wants to share the item.

Added feature to my current open request. 😉

With the latest update you are now able to share passwords 🎉

@raycastbot close this issue

Thanks a lot for rolling out the requested feature quickly 😄 . I don't see a configuration option available for the expiry of the link. Which isn't that important for me but just want to know the default configuration set.

It uses 1Password default configuration. 😉


bp of shared parameters and experts

The ddp in pytorch can not distinguish experts and other shared parameters. And experts may be updated with shared gradient.
The TutelDistributedOptimizer seems to be an implementation of zero, which does not affect the graident. How does tutel deal with the problem?

Yes, TutelDistributedOptimizer is a replacement of Pytorch DDP in that example (helloworld_ddp_tutel) to make the whole model sychronization transparent.
TutelDistributedOptimizer not only implements ZeRO optimization, but also leverages built-in mask (_tutel_expert) to distinguish whether a parameter is shared or from the creation of tutel.moe.moe_layer.
Note that TutelDistributedOptimizer only treats parameters created by tutel.moe.moe_layer to be expert parameters. If the model never uses tutel.moe.moe_layer, there is no difference with Pytorch DDP (expect TutelDistributedOptimizer includes ZeRO feature).

Thank you for your answer.
I notice that _tutel_expert flag is used to split the parameters. But it seems that gradient of experts with _tutel_expert will also be allreduced by ddp.

Yes, To use TutelDistributedOptimizer which has parameter synchronization included, you should no longer warp the model with DistributedDataParallel.

I notice the code in swin-transformer repo(https://github.com/microsoft/Swin-Transformer/blob/main/main_moe.py), which uses pytorch optimizer and ddp to train these moe models. Maybe there is something wrong.  Thanks a lot.

It is a version that manually distinguish parameter types, which follows helloworld_ddp.py

Does it work by setting skip_allreduce as true in the scan function?

To use tutel moe in Pytorch DDP backend, you need to not only set skip_allreduce as true in the moe scan function, but also recollect parameters with those masks, and tell DDP to skip synchronizing them by  function.https://github.com/microsoft/tutel/blob/main/tutel/examples/helloworld_ddp.py#L92. Otherwise, Pytorch DDP won't know they are expert parameters, so they'll be synchronized unexpectedly.


DeprecationWarning regarding use of distutils Version classes

What happened:
While working on some tests that catch and check for warnings in my library I found that xarray with new versions of Python (I think this is the trigger) causes a ton of DeprecationWarnings on import:
/home/davidh/miniconda3/envs/satpy_py39/lib/python3.9/site-packages/xarray/core/pycompat.py:22: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
  duck_array_version = LooseVersion(duck_array_module.__version__)

What you expected to happen:
No warnings.
Minimal Complete Verifiable Example:
import warnings
warnings.simplefilter("always")

import xarray as xr

Results in:
/home/davidh/miniconda3/envs/satpy_py39/lib/python3.9/site-packages/xarray/core/pycompat.py:22: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
  duck_array_version = LooseVersion(duck_array_module.__version__)
/home/davidh/miniconda3/envs/satpy_py39/lib/python3.9/site-packages/xarray/core/pycompat.py:37: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
  duck_array_version = LooseVersion("0.0.0")
/home/davidh/miniconda3/envs/satpy_py39/lib/python3.9/site-packages/xarray/core/pycompat.py:37: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
  duck_array_version = LooseVersion("0.0.0")
/home/davidh/miniconda3/envs/satpy_py39/lib/python3.9/site-packages/setuptools/_distutils/version.py:351: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
  other = LooseVersion(other)
/home/davidh/miniconda3/envs/satpy_py39/lib/python3.9/site-packages/setuptools/_distutils/version.py:351: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
  other = LooseVersion(other)
/home/davidh/miniconda3/envs/satpy_py39/lib/python3.9/site-packages/xarray/core/npcompat.py:82: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
  if LooseVersion(np.__version__) >= "1.20.0":
/home/davidh/miniconda3/envs/satpy_py39/lib/python3.9/site-packages/setuptools/_distutils/version.py:351: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
  other = LooseVersion(other)
/home/davidh/miniconda3/envs/satpy_py39/lib/python3.9/site-packages/xarray/core/pdcompat.py:45: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
  if LooseVersion(pd.__version__) < "0.25.0":
/home/davidh/miniconda3/envs/satpy_py39/lib/python3.9/site-packages/setuptools/_distutils/version.py:351: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
  other = LooseVersion(other)

Anything else we need to know?:
 Looks like we can update LooseVersion to packaging.version...

It's started to show up in the xarray CI as well now:
https://github.com/pydata/xarray/runs/4600524127?check_suite_focus=true

We currently use from distutils.version import LooseVersion - any opinion if we should replace this by version.parse or by version.Version?
from distutils.version import LooseVersion
from packaging import version

LooseVersion(xr.__version__)
version.parse(xr.__version__)
version.Version(xr.__version__)

# currently:
if LooseVersion(mod.__version__) < LooseVersion(minversion):
    pass

# options:
if version.parse(mod.__version__) < version.parse(minversion):
    pass

if version.Version(mod.__version__) < version.Version(minversion):
    pass

if Version(mod.__version__) < Version(minversion):
    pass

See https://packaging.pypa.io/en/latest/version.html
(I think version.parse either returns a Version or a LegacyVersion. A five minute search suggests this is the preferred method but probably someone has a more informed opinion.)

@mathause I agree that .parse is the preferred method when you are taking a version string from an outside source. If you were using a static/constant string then it would probably be fine to use Version.
I wasn't sure what the best approach would be for xarray given that LooseVersion is a public property on the duck array wrapper, right? I'm not sure if packaging's Version is backwards compatible and whether or not that matters inside xarray.

I agree that .parse is the preferred method when you are taking a version string from an outside source. If you were using a static/constant string then it would probably be fine to use Version.

True we could potentially use version.parse(h5netcdf.core.h5py.__version__) >= version.Version("3.0.0") in #6096.

I wasn't sure what the best approach would be for xarray given that LooseVersion is a public property on the duck array wrapper, right? I'm not sure if packaging's Version is backwards compatible and whether or not that matters inside xarray.

Good point. It's not really backwards compatible. packagings Version cannot be compared to a string (while LooseVersion can). However, DuckArrayModule is probably not meant as a public interface - or were you referring to something else?
https://github.com/pydata/xarray/blob/5e8de55321171f95ed9684c33aa47112bb2519ac/xarray/core/pycompat.py#L11-L13

Yes, sorry, I meant DuckArrayModule.

This also happens in seaborn

assert LooseVersion(tf.version) >= LooseVersion("2.0")
DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
How do resolve it?

@AppaAppa - this is quite an old ticket. If you are running into something similar with Xarray, can you open a new issue and include the relevant information about versions and your environment?

Hi, i am running Mask RCNN file from github. In the model file , it shows
the error
assert LooseVersion(tf.version) >= LooseVersion("2.0")
DeprecationWarning: distutils Version classes are deprecated. Use
packaging.version instead.
How cal i fix it?
On Tue, Sep 5, 2023 at 12:01 PM Joe Hamman @.***> wrote:

@AppaAppa https://github.com/AppaAppa - this is quite an old ticket. If
you are running into something similar with Xarray, can you open a new
issue and include the relevant information about versions and your
environment?
—
Reply to this email directly, view it on GitHub
https://github.com/pydata/xarray/issues/6092#issuecomment-1706020308,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A4AOO4MMU45NAOXAR3M4ZEDXY3BK3ANCNFSM5KQRGBKQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


Bot starts but doesnt create channels

Bot starts when victim executes but doesnt create channels
I am in Spain, good WiFi conection, bot with admin and also, all intents enabled.
Screenshots

Desktop (please complete the following information):

OS: Windows 11,
Python Version: 3.9

update: python version: 3.11.9 (i updated it)

fixed


Incorrect create function code action

Description:
In the first two scenarios we do not give the expected return type and its default value in the created function even though we identify the return type by the compiler. In the 3rd scenario, the number of parameters are doubled and has changed its name and type.

Steps to reproduce:
1)
function test1() {
    fail getFail();
}

function test2() returns int {
    return getReturn();
}

function test3() {
    boolean a = true;
    a = getBool(boolean q, boolean w);
}

Affected Versions:
SL 2201.0.0 RC1

For 1 and 2, in both cases we get CompilationErrorType as the type of the symbol.
For the case 3, the source is having a syntax error. The recovered syntax is assuming that we have a function with four arguments. Hence we will need to check this more deeply on how to handle such sccenarios.

Please consider the following scenario as well
function test4() {
    int a = check getCheck();
}


Awesome project

I found your project because i was looking for alternatives to svelte-grid since I am not sure if it's still maintained.
What are your plans with this library? do you think your api will be compatible with svelte-grid? do you have any timeline?
Thanks in advance! 🙌

Thanks @cuire ! I was looking at your API and I think is way more cleaner.

Love the project and am quite excited to work with it in the future.
Let me know if there's anyway i could support you in the development, maintenance or documentation!

+1 to this, amazing work @cuire  works as expected and the API is way cleaner, it is just what I was looking for, keep the good work!


Handle cookie fields without values (for #5)

Hey! My C is pretty rusty, but here's a stab at it. Cheers. :)

Alternately, we could drop "previous_state" and "vend = -1" entirely and return the field with an empty string. But there are some tests for headers ending with "YYY" that I didn't want to break.

Fixed a bit differently, thanks for the push in the right direction!


About Sensitivity Specificity loss function implementation

Hello, I have a doubt about Sensitivity Specificity loss function's implementation.

So I think that the correct way to translate this Sensitivity into **prediction ** and **ground_truth ** terms for a loss function would be similar to:
true_positives = prediction*ground_truth
false_negatives = (1-prediction)*ground_truth

But in the implementation:
Sensitivity = (square(ground_truth-prediction) * (1-one_hot) ) / (1-one_hot)
How should I interpret its expressions(The same occurs with the Specificity )？

Hi,
Please check their paper for the details


Improve integration with gigantum

postBuild scripts
publish link to public gigantum image (loadable on try.gigantum.com)

see new README https://github.com/digital-science/dimensions-api


Problem definition for Verified Email Domains for GitHub

Background

We need to clarify what problem we are trying to solve with verified email domains.
Proposed user journey

TBC
Approach

Speak to Service Owner to understand what was driving this discussion so that we have a better understanding of what problem we are trying to resolve.
Questions / Assumptions

TBC
Acceptance Criteria

[ ] We have a clear problem statement/user need definition as a basis for this work.

Reference
How to write good user stories

Discussing problem and value with Service Owner

Problem definition confirmed. Jamboard updated. This will result in creation of some new stories.


Force LCM scheduler for LCM safetensors models

The safetensors file does not specify a scheduler, so the diffusers library picks one according to the base model (DDIM for SD 1.5).
As in lcm-lora, we check for 'lcm' in the file name, since most safetensors files follow that convention.
The DDIM scheduler adds a grainy texture on images generated with LCM checkpoints, but depending on the checkpoint it may not be immediately noticeable. Testing with dreamshaper_8LCM.safetensors:

And with the LCM scheduler:

@wbruna Thanks you


license

Hi conclave team,
very interesting proof of concept.
I couldn't find licensing information, could you clarify?

Oh that's a good catch!
We mean for this to be under the MIT/Open Source license as it is an open source, we just forgot to include the actual License file. I will add that right now.