Nested modules

Replaces/Closes #800
This PR closes #800 with extra additions to the PR to assure a good landing of nested modules.
We might want to observe the issues listed at #858 and add extra commits
cc @gibson042

@gibson042 I made two commits to include arguments on the contained modules callback. I appreciate if you review and criticize.
cc @jzaefferer

I made two commits to include arguments on the contained modules callback. I appreciate if you review and criticize.

I'm not sure about these particular parameters. As I've noted elsewhere, I'd like module and test callback signatures to look as similar as possible. But further than that, I'd like module callbacks to be straightforward. It seems to me that data passed in to them—including as this—should be entirely sufficient to define submodules/tests/etc., but hopefully not exceed that.
In my opinion, it's better to pass nothing at all now than it is to guess wrong about the signature. Since this iteration still requires access to the global QUnit for some operations (e.g., submodules), I'd actually prefer an argument-free executeNow() for this PR, saving context/arguments for later as discussed. For example, a possible future that would be precluded by passing test now:
QUnit.module("outer", function( outerModule ) {
	// environment (prototype for test/submodule context)
	this.outer = true;
	this.outerBefore = 0;

	// test before/after
	outerModule.test.beforeEach(function() {
		this.outerBefore++;
	});

	// tests
	outerModule.test("foo", function( assert ) {
		assert.equal(this.outerBefore, 1, "beforeEach updated environment");
	});
	outerModule.test("bar", function( assert ) {
		assert.equal(this.outerBefore, 1, "sibling tests don't share environments");
	});

	// submodules
	outerModule.module("inner", function( innerModule ) {
		// environment (inherits from parent)
		this.inner = true;

		function assertEnvironment( assert, before ) {
			assert.ok(this.outer, "outer module initialized environment");
			assert.ok("outerBefore" in this, "outer module beforeEach updated environment");
			assert.equal(this.outerBefore, 1, "submodules don't share environments");
			assert.ok(this.inner, "inner module initialized environment");
			if ( before ) {
				return;
			}
			assert.ok(this.innerBefore, "inner module beforeEach updated environment");
			assert.ok(this.tested, "test updated environment");
		}

		// test before/after
		innerModule.test.beforeEach(function( assert ) {
			this.innerBefore = true;
			assertEnvironment( assert, true );
		});
		innerModule.test.afterEach(function( assert ) {
			assertEnvironment( assert );
		});

		// tests
		innerModule.test("bar", function( assert ) {
			this.tested = true;
			assertEnvironment( assert );
		});
	});
});

QUnit.module("outer", function( outerModule ) {
    // environment (prototype for test/submodule context)
    this.outer = true;
    this.outerBefore = 0;

    // test before/after
    outerModule.test.beforeEach(function() {
        this.outerBefore++;
    });

    // tests
    outerModule.test("foo", function( assert ) {
        assert.equal(this.outerBefore, 1, "beforeEach updated environment");
    });
    outerModule.test("bar", function( assert ) {
        assert.equal(this.outerBefore, 1, "sibling tests don't share environments");
    });

    ...

These this.outerBefore tests seems very confusing to me. I can understand the point here but I'm uncomfortable with it so far.

I'm not sure about these particular parameters.

And reviewing my code I'll indeed remove the first test argument.
And following your own example, I would stick with the first argument being an object with the hooks methods and it can later being extended. This way:
QUnit.module("outer", function( outerModule ) {
    ...
    outerModule.beforeEach(function() {
        ...
    });

    outerModule.afterEach(function() {
        ...
    });

but also, later in further additions:
QUnit.module("outer", function( t ) {
    t.beforeEach(function() {
        ...
    });
    t.test( "the test method", function( assert ) {
        ...
    });
});

and using ES6:
QUnit.module("outer", ( { beforeEach } ) => {
    beforeEach(function() {
        ...
    });
    QUnit.test( "the test method", function( assert ) {
        ...
    });
});

The reason to start with beforeEach and afterEach methods is to avoid calling modules with three arguments, where the hooks object is already big enough:
QUnit.module( "outer", {
        beforeEach: function( assert ) {
            ...
        },
        afterEach: function( assert ) {
            ...
        }
    },
    function( t ) {
        QUnit.test( "the test method", function( assert ) {
            ...
        });
    }
);

@gibson042: both 29f5b84 and 53778e3 has the shared this based on your example.
The this object is only propagated from the outer to the inner scopes without duplicating the test hooks.
I also removed the test argument as you can see on the fixup commits.
We may then extend the module callback argument to include test later. I want to at least bring the shared this and the hooks when landing the nested modules.

:+1: The single callback argument is a decent (if limited) module stand-in and has good room for growth, and you are absolutely right about this:

The reason to start with beforeEach and afterEach methods is to avoid calling modules with three arguments, where the hooks object is already big enough

Let's hope that module and test follow suit, so that we can eventually get to a point where module callbacks need nothing other than what's passed directly to them, just like current test callbacks.

I think this is good to go; module environment inheritance can be addressed with a subsequent issue & PR.

Opened #869 to follow up with this.


Battery usage

I know this is significantly Electron's fault, but Reacto uses dramatically more power than other editors. It's likely pretty difficult to optimize and test, but I want to bring it up as a longer-term thing.
I pulled up (production build, not dev) Reacto + BBEdit next to each other, on the same project with the same file, in macOS Split View, running on my 2013 MacBook Pro with Retina Display 13". (Reacto is unusable at smaller window widths due to the sidebar, I know you've added the ability to hide that since this build was released :)

Here's Activity Monitor's Energy tab while the two apps are in the foreground. Atom is also Electron-based (same project open there, but in the background), yet uses 1/5 the power.

A moment later, with the apps open but in the background. BBEdit is able to shut down most of what it's running and use almost no power, but Reacto continues using resources.

Finally, here's a moment after both BBEdit and Reacto are minimized in the Dock, doing nothing.

I will look into it as soon as possible. Totally agree with you. I'm still trying to stay organized and not do anything at the same time.
It should be better anytime soon, thanks for the precise report @lachlanjc! I'll update you as soon as there's progress made. :)

@eveningkid Fantastic. Just wanted to bring this to your attention. I think the most important fixes are for when Reacto is in the background/minimized, but there are certainly savings to be made in the active editor as well. Thanks so much for being so responsive and working so hard on this project!

Alright, sorry it's been a while.
Good news though:

I found what was the major problem causing Reacto to use way too much CPU compared to others! It was simply the file watcher that was using polling although it shouldn't.
I will soon release a new version that will finally be much better! Right now I am working on a lot of things that will for sure bring a much better experience. :)
I'll close the issue. Please give it a try when the new release is out, but it should be working perfectly after I publish changes. ✨

Fabulous! Your sustained work on everything is truly impressive. Thank you 😊


commander error messages

A common pain point (especially for new users) is the lack of useful error messages from commander when state transitions fail (arming or modes).
Maybe it's time for the event interface? http://discuss.px4.io/t/event-interface/2318
This has been mentioned in quite a few other places, but it's sufficiently annoying that I'm creating a new issue.

let's keep this issue alive.

@Antiheavy it will now stay openG actually, just marked stale.


Use originally provided extrinsic payload to verify signature

I currently have an issue with a custom SCALE Encode/Decode implementation. I'm compiling the implementation to JS/WASM.
My transaction that uses the JS encoded representation is failing with 1010: Invalid transaction: Bad signature. I believe it is because of the following psuedocode:
js_encoded = js_encode(obj);
rust_decoded = rust_decode(js_encoded);
rust_encoded = rust_encode(rust_decoded);

js_encoded != rust_encoded

Basically, this means that my implementation can parse the JS encoding, but the Rust encoding doesn't end up matching. I'm still investigating why.
Either way, the Substrate node is not accepting my transaction because, I assume, it is:

Decoding the original extrinsic payload (probably to check for validity)
Encoding the in-memory object
Checking the signature matches a newly generated payload using the Rust encoding

I know my implementation of Encode should work the same in Rust and JS, but Substrate should still use the original transaction payload to validate signatures.

Even if we would do this it would fail later. The encoding needs to match, there is no way around this.


feat(convertShapeToPath): convert rect with rounded corners using arcs

Convert rect shape with a defined rx and/or ry to path when the convertArcs parameter is true.
E.g.
<rect x="40" y="50" width="30" height="20" rx="2" ry="5"/>

will become
<path d="M68 50A2 5 0 0 1 70 55L70 65A2 5 0 0 1 68 70L42 70A2 5 0 0 1 40 65L40 55A2 5 0 0 1 42 50z"/>

When rx and ry are invalid or 0 the conversion will be performed without arcs as there are no rounded corners in that case.

why would you want to do this? this seems like the opposite of optimization

why would you want to do this? this seems like the opposite of optimization

That's true for all conversions that are enabled by the convertArcs flag.
The need for this is use-case dependant.
If the flag is enabled it should do all the arc conversions imho. We want to reduce all shapes to a single path. You can also see the need for this in PR  #1767.

This is basically the opposite of #1748, there are definitely cases where one is closer than the other.


Drop .NET Framework 4.5.2 and 4.6 which will reach End of Support on April 26, 2022

Describe the feature
Would it make sense to stop supporting .Net 4.5.2 and 4.6 right now? After all, the end date is April 2022.

Additional context
https://devblogs.microsoft.com/dotnet/net-framework-4-5-2-4-6-4-6-1-will-reach-end-of-support-on-april-26-2022/#closing

Hi Jan,
even Windows 7 can run .Net 4.8 and .Net 5.0 (tested it recently), So I think dropping the older versions should be no problem in most cases.
Have a great Sunday
Tim


Unleashed javaslang.collection.Iterator. Refactored Collections.

Fixes #431 Implement Scala like Iterator with lazy map/flatMap capabilities
Fixes #291 Remove internal abstract classes from public API
Fixes #256 Re-organize collection.Tree
Towards #433 Make complex hashCodes lazy  // but Deserialization of transient Lazy does not work!
Towards #113 Make interface Tree extend Traversable

// it will at least extend Iterable and FilterMona, maybe TraversableOnce but not Traversable

@ruslansennov I'm on vacation with my family for one week, starting tomorrow. Notebook has to stay home o_O... I will try to go online at most once a day. We will see.

@ruslansennov I refactored the collections a little bit. Please have a look (HashSet, HashMap). There is now javaslang.collection.Iterator...

Have a nice trip! It's a good idea to leave the laptop at home

Yes...

I made HAMT and Interface. That leaks the inner classes to the public.
Pro: this enables pattern matching by subtype, 'introspection' and Proxying of the HAMT.
Con: the public API shows more classes
I'm not sure. We can switch back, when in doubt. The RedBlackTree is also designed as Interface with inner static classes. I like interfaces in general. They are more flexible, and it seems a good design practice to me.

Con: the public API shows more classes

This was the reason I chose abstract class instead of interface
But there is not problem, I believe it's just a matter of taste

Ok. Abstract classes were always some intermediate types to me, not intended to be directly used other than extending them for own types. We will go with interfaces for now. I hope that Java 9 gives us some extra syntax regarding interfaces to hide private methods and maybe private inner classes. We will see.


Request to add DNS query

I would love to see the ability to submit a simple DNS query to verify name resolution is working.

I would love to see the ability to submit a simple DNS query to verify name resolution is working.

What kind of DNS query have you been thinking about?
Sounds like you want a normal DNS Lookup for a domain or is there something more specific you need?

I would love to see the ability to submit a simple DNS query to verify name resolution is working.

What kind of DNS query have you been thinking about?
Sounds like you want a normal DNS Lookup for a domain or is there something more specific you need?

A record would work fine for me. So basically just need IP of DNS server, and the query input. Success/fail based on getting back a response vs timeout.

I think we can close this issue, since it'll be implemented in Version 1.5.0.


Use Ptr type if it implements Unmarshaler

Sorry for stealing this @tchajed but your solution is better than mine!

Wrong branch


Unload asset pack

The loader plugin can currently load all assets from an asset-pack via scene.load.pack(...), but it lacks the functionality for unloading an asset-pack.
Since the pack function must already have the code to read and iterate through assets in an asset-pack, maybe something like scele.load.unloadAssetPack(...) can be easy to implement, as it would just have to call destroy instead of load for each asset in the asset-pack.
Motivation:
Building multi-scene games for mobile with lots of assets can be difficult to do without this, as it requires the developer to track every asset it loads and manually and individually unload it when needed. This is almost like managing an asset-pack manually and it kind of defeats the purpose of the builtin asset-pack.
Pretty much all game engines support something along the lines of unload unused assets or unload asset bundle or unload scene (which also unloads the assets).

Please try https://github.com/phaserjs/phaser/pull/6793

Needs CHANGELOG entry.

Needs CHANGELOG entry.

Thanks @samme. Have added the changelog entry.


Meta-args are messing up their scope

In this example:
module T

open FStar.Tactics

let m (a:Type) (idx:int) = unit -> Tot a

let ret (#a:Type) (#i:int) (x:a) : m a i = fun () -> x

let bind (#a #b : Type) (#i : int)
         (x:m a i) (f:(a -> m b i)) : m b i =
        fun () -> f (x ()) ()

let tau () : Tac unit =
    dump "1";
    let b = nth_binder (-1) in
    exact b

let f () (#[tau ()] idx:int) : m int idx =
  fun () -> 42

let test2 () : m int _ =
  let z = 17 in
  bind (f ()) (fun i ->
  ret 42)

something funny happens. There are two uvars introduced in test2, the first one ?u1 for the _ in the return type and the one for the meta-arg in the call to f (). The second one ?u2 has z in scope, but the first one does not. During unification, they are solved to be equivalent (?u2 is set to ?u1), but remain unsolved. We then run the tactic for ?u2, which in this case is engineered to solve to z, and try to solve ?u2 to that. That fails, since ?u2 := ?u1 which does not have z in scope. Furthermore the error is very bad:
T.fst(37,7-37,13): (Error 54) (*?u28*) _ is not equal to the expected type z

I'm working on a solution that runs the tactic for a meta arg in the smaller of the two contexts on cases like this. In that case the tactic could not provide an ill-scoped solution. Unsure of the impact.
This was reported by @R1kM in Steel.

Similar issue. It runs to completion, but leaves an SMT goals that's ill formed.
module Bug1966

open FStar.Tactics

assume val t : int -> Type
assume val lem : x:int -> y:(t x) -> Lemma (y == y)

let f (x:int) (y : t x) =
  assert (y == y)
      by (dump "1";
          let x = nth_binder (-2) in
          rename_to x "z";
          dump "2";
          apply_lemma (`lem);
          dump "3")

F* may be in an inconsistent state.
Please file a bug report, ideally with a minimized version of the program that triggered the error.


handle exceptions from execution function in OneAtATimeAction

@Mergifyio backport release/2.16.x release/2.17.x


feat: add bot-list package

This plugin is a fork of the @devtomio/plugin-botlist plugin, which was created by jzeuzs which has not been maintained for over a year now and has some issues.

Hi, sorry for the delay, could you please resolve the conflict?


meadowrun-manage-ec2: command not found

Have tried installing in base virtual env using poetry and pip.  Can't seem to be found.  Appreciate any tips. Thanks in advance.
Ubuntu 22.04, python 3.7.11, meadowrun 0.11 // with non-sudo and sudo

Thanks for trying meadowrun! I suspect it's the python 3.7 that is biting you, I recently added it to our regular test matrix and found a few issues in the process - #77 has not been released yet.
Could you try
python -c "import meadowrun.manage; meadowrun.manage.main_ec2()" --help / install / ...

to check? If it gives some typing related import errors that would be a sure indication that is the issue.
Or try with a more recent python version if you can.
We will release an updated version with the 3.7 fixes.

We did have a fix for a python 3.7 issue that I just released (so if you get 0.1.12 you shouldn't run into that), but that was an issue with from typing import Literal issue, which is not what you were running into.
My guess is that you're somehow using a pip that's related to a python version that's not on your path. We would recommend using the advice here: https://stackoverflow.com/questions/2812520/dealing-with-multiple-python-versions-and-pip which is to always call pip via python -m pip ... rather than just pip by itself. As long as the python you use in that command is on your path, then the meadowrun-manage-ec2 command should work. But in general the recommendation would be to use a virtualenv. I just ran this on Ubuntu and it worked:
python3.7 -m venv py37venv
source py37venv/bin/activate
pip install meadowrun
meadowrun-manage-ec2 install

Thanks very much for the fast response.  Believe you are right as I switched to python ^3.10 as poetry suggests and so far so good.  I'll post here again if any more observations.


[BUG] Using the percent symbol in square brackets of the class name causes an error in the "tailwindcss/no-custom-classname" rule

Describe the bug
I add the class text-ffffff/[24%] and get an error:

Expected behavior
There should be no error.
Environment (please complete the following information):

OS: Windows 10
Softwares + version used:

VSCode: 1.63.2
eslint-plugin-tailwindcss: 3.2.0
tailwindcss: 3.0.13

eslint config file or live demo
https://play.tailwindcss.com/AzOrcWBRP4

Please try
npm i<EMAIL_ADDRESS>and provide feedback 😉

@francoismassart Yeah, it works for me.


Fix race condition in setting ReferenceCountedOpenSslEngine.needTask flag

Fixes #12139
What's changed
This PR reorders resetting ReferenceCountedOpenSslEngine.needTask flag within the io.netty.handler.ssl.ReferenceCountedOpenSslEngine.TaskDecorator.run() method. The flag shouldn't be unset before the task execution finishes.
What's not changed
This PR doesn't change the needTask flag handling in AsyncTasks. I'm not sure when and how this type is used.
There could be also a problem if more tasks are allowed to be returned for subsequent getDelegatedTask() call.
    List<Runnable> collectTasks(SSLEngine sslEngine) {
        List<Runnable> tasks = new LinkedList<Runnable>();
        Runnable task;
        while ((task = sslEngine.getDelegatedTask()) != null) {
            tasks.add(task);
        }
        return tasks;
    }

Then the single boolean variable is not enough to track task processing. It would rather deserve a task counter.

It was specifically changes to this in #11739 because there are other issues if the task in question inspects the needTask field.
I don't recall the details of that, but just saying there's more going on and we shouldn't fix one bug by bringing back another.

Let me investigate...

working on this now

closing... This is replaced by https://github.com/netty/netty/pull/12149


MB-5483 MB-5543 MB-5544 Update service to check for configuration vars or error

Description
As part of MB-5483 we want to ensure that the variables for configuring Syncada SFTP connection are passed in. If they are not present we should gracefully handle that. This PR implements MB-5543 and MB-5544 to check for the presence of the configuration options and return an error if they are missing.
Reviewer Notes
Not that I can think of
Setup
Currently this code is only used in a test code.
make server_test

Code Review Verification Steps

[X] Request review from a member of a different team.
[X] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change
Jira subtask for this change
Jira subtask for this change

Messages

:book:
:link: MB-5483, MB-5543, MB-5544

  Generated by :no_entry_sign: dangerJS against b0e229b4e8296f3f02ec80a3d7ac71c3dcbf2615

I got the expected error but only when I switched the payload to sendToSyncada: true. I believe there's just a typo in the description. unless the reverse is expected?

You're right, I've updated the description.


Fixed #351

Codecov Report

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

@@           Coverage Diff           @@
##           master     #354   +/-   ##
=======================================
  Coverage   85.05%   85.05%
=======================================
  Files          58       58
  Lines        1887     1887
=======================================
  Hits         1605     1605
  Misses        282      282

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 1ef5036...2bc1ab1. Read the comment docs.


@ionoc/cloud-angular does not work with Rollup

I have an Ionic app that's setup to use Rollup as bundler and it seems that during the build, rollup cannot find the module:
10:34:02]  ionic-app-scripts 1.0.0
[10:34:02]  build prod started ...
[10:34:02]  clean started ...
[10:34:02]  clean finished in 6 ms
[10:34:02]  copy started ...
[10:34:02]  ngc started ...
[10:34:19]  ngc finished in 16.13 s
[10:34:19]  rollup started ...
[10:34:19]  copy finished in 16.38 s
[10:34:26]  rollup failed: Could not resolve '@ionic/cloud-angular' from
            /Users/rolandjitsu/Projects/infarm/maintenance/src/app/app.module.ts
[10:34:26]  ionic-app-script task: "build"
[10:34:26]  Error: Could not resolve '@ionic/cloud-angular' from
            /Users/rolandjitsu/Projects/infarm/maintenance/src/app/app.module.ts
Error: Could not resolve '@ionic/cloud-angular' from /Users/rolandjitsu/Projects/infarm/maintenance/src/app/app.module.ts
    at Error (native)
    at /Users/rolandjitsu/Projects/infarm/maintenance/node_modules/rollup-plugin-node-resolve/dist/rollup-plugin-node-resolve.cjs.js:78:21
    at /Users/rolandjitsu/Projects/infarm/maintenance/node_modules/browser-resolve/index.js:265:24
    at /Users/rolandjitsu/Projects/infarm/maintenance/node_modules/browser-resolve/node_modules/resolve/lib/async.js:46:14
    at process (/Users/rolandjitsu/Projects/infarm/maintenance/node_modules/browser-resolve/node_modules/resolve/lib/async.js:173:43)
    at ondir (/Users/rolandjitsu/Projects/infarm/maintenance/node_modules/browser-resolve/node_modules/resolve/lib/async.js:188:17)
    at load (/Users/rolandjitsu/Projects/infarm/maintenance/node_modules/browser-resolve/node_modules/resolve/lib/async.js:69:43)
    at onex (/Users/rolandjitsu/Projects/infarm/maintenance/node_modules/browser-resolve/node_modules/resolve/lib/async.js:92:31)
    at /Users/rolandjitsu/Projects/infarm/maintenance/node_modules/browser-resolve/node_modules/resolve/lib/async.js:22:47
    at FSReqWrap.oncomplete (fs.js:123:15)

Webpack seems to work just fine, but the problem is that with Webpack I cannot have env variables passed to the app when the app is build for production (something about the AOT), so I need to use Rollup at the moment.
System Info:
Cordova CLI: 6.4.0
Ionic Framework Version: 2.0.0-rc.4
Ionic CLI Version: 2.1.18
Ionic App Lib Version: 2.1.9
Ionic App Scripts Version: 1.0.0
ios-deploy version: Not installed
ios-sim version: Not installed
OS: macOS Sierra
Node Version: v6.7.0
Xcode version: Xcode 8.2.1 Build version 8C1002

+1

Closing this as there is a PR with a fix.


[Bug]: Logj4j logs url credentials in "navigatToUrl" method

Some urls has the following format<EMAIL_ADDRESS>so when using navigatToUrl method, it logs the full url with the authentication data and for security reason this should be avoided.
Current solution it to change the log4j level to anything rather than info.
I tried to add regex expression, filters or custom PatternLayout to log4j file but it wasn't read .

simple test

result before

result after

@MohabMohie same happen in :

Get current url
Initial URL


Cleanup PATH variable in web container

Is there an existing issue for this?

[X] I have searched the existing issues

Is your feature request related to a problem?
When I run echo $PATH in the web container of a fresh configured ddev project I get the following:
~/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/var/www/html/vendor/bin:/var/www/html/bin:/var/www/html/vendor/bin
You can see there are:

double entries for /var/www/html/vendor/bin
these paths might do not exists on some projects:/var/www/html/vendor/bin
/var/www/html/bin

Describe your solution
On a fresh configured ddev Project the PATH variable in the ddev webcontainer should look like this:
~/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
Describe alternatives
Additional paths could be added depending on:

The ddev project type
a new config key like webimage_extra_paths: []

Additional context
The PATH variable seems to be set in https://github.com/drud/ddev/blob/master/containers/ddev-webserver/ddev-webserver-base-files/etc/bashrc/commandline-addons.bashrc

Why do you think it's important to not have items in the PATH that don't exist? This is a common situation. Does it do you some harm?

This is just my aspiration for keeping things as clean and clear as possible. I found the extra unused and double paths distracting when I found it.

Yeah, but adding the code to make it feel clean to you would be many lines of code, adding significant complexity. There's more than one side to these things! Closing for now, happy to continue the conversation here or elsewhere.


 This can be used for easier local testing.
We need to make the backend.run() method of these to actually run the simulations using BasicAer and give back the result. Ideally I'd like to run Aer with noise simulation, but that would introduce a dependency of Terra to Aer.

You could just do it only if aer is available. Something like:
def run(qobj):
    if qiskit.Aer:
        sim = qiskit.Aer.get_backend('qasm_simulator')
    else:
       sim = qiskit.providers.basicaer.BasicAer.get_backend('qasm_simulator')
    return sim.run(qobj)

Although you'll to adapt all the options etc to match what's needed

I am interested in it. In order to exploit BasicAer, is there a methodology less-computationally expensive than density matrix formalism for modeling noise?
Moreover, I read that the five-qubit fake hardware is FakeTenerife, which is not available anymore. Is it required to create different classes for fake five-qubit backends, e.g. FakeYorktown, FakeVigo and FakeOurense?

@giovanniamedeocirillo  great let me know if you'd like to work on it.

is there a methodology less-computationally expensive than density matrix formalism for modeling noise

Aer injects noise, so you don't have to do anything. You just have to use a basic_device_noise_model and make the backend.run() method of these fake backends use that. See the tutorial here: https://github.com/Qiskit/qiskit-iqx-tutorials/blob/master/qiskit/advanced/aer/2_device_noise_simulation.ipynb

Is it required to create different classes for fake five-qubit backends, e.g. FakeYorktown, FakeVigo and FakeOurense?

All of these are already there in qiskit/test/mock. And they have backend.properties() which means that the above device noise model can be made from them.

@ajavadia I am still interested in working on it, thank you.


Rg51-powerpointpainter

Changes to UK-Hub only:
-automate front page
-shape dimensions adjusted
-grid name changed from ‘core_q5[{core_q5a}].core_q5_grid’ to ‘core_q5’
-net position in charts

Will create a new Rg for this...
Ignore for now.


Replace deprecated functions

vld_no_missing() is deprecated as of lifecycle 0.2.0.
Please use vld_not_any_na() instead.

done


Remane .json to .to_dict and make another method .to_json

Now the .json() method actually returns a duct, to make it more clear, we should rename it to .to_dict and have another method name .to_json

This is incorporated in the new version 0.1.1


Coroutine test broken

@SalomonBrys coroutines typescript tests are not running anymore on my side, did not test since Kotlin upgrade, do you know if it's related?

Fixed, sorry for the ping. Also there is still some kotlin test failing due to bad imports, no time to investigate that currently...


Request to share your successful setup/configuration

I would like to ask all of you to share your successful setup/configuration for Distrod.
Please include:

the baseline of your host as given via systeminfo (freed from personal data)
the reduced/simplified step-by-step you took to get Distrod running for the given options mentioned in the README
include a proof regarding a task that requires systemd is working correctly, for example lxc, lxd, snap - feel free to mention what is not working at the end.

Why i am asking for it?
Because it helps everyone interested to use Distrod and might show up issues with the README, the general process, issues yet not identified although vaguely discovered with the core-routines, and so on.
Thank you.

Successfully installed on a nested virtualization VMware Workstation instance

The baseline via systeminfo (personal data deleted)

Host Name:
OS Name:                   Microsoft Windows 11 Enterprise
OS Version:                10.0.22000 N/A Build 22000
OS Manufacturer:           Microsoft Corporation
OS Configuration:          Standalone Workstation
OS Build Type:             Multiprocessor Free
Registered Owner:
Registered Organization:
Product ID:
Original Install Date:     2021/10/3
System Boot Time:          2021/11/22
System Manufacturer:       VMware, Inc.
System Model:              VMware7,1
System Type:               x64-based PC
Processor(s):              1 Processor(s) Installed.
                           [01]: Intel64 Family 6 Model 63 Stepping 2 GenuineIntel ~3492 Mhz
BIOS Version:              VMware, Inc. VMW71.00V.18452719.B64.2108091906, 2021/8/9
Windows Directory:         C:\WINDOWS
System Directory:          C:\WINDOWS\system32
Boot Device:               \Device\HarddiskVolume1
System Locale:             en-us;English (United States)
Input Locale:              en-us;English (United States)
Time Zone:                 (UTC-08:00) Pacific Time (US & Canada)
Total Physical Memory:     8,191 MB
Available Physical Memory: 2,885 MB
Virtual Memory: Max Size:  16,383 MB
Virtual Memory: Available: 9,642 MB
Virtual Memory: In Use:    6,741 MB
Page File Location(s):     C:\pagefile.sys
Domain:                    WORKGROUP
Logon Server:
Hotfix(s):                 3 Hotfix(s) Installed.
                           [01]: KB5006363
                           [02]: KB5007215
                           [03]: KB5006755
Network Card(s):           3 NIC(s) Installed.
                           [01]: Wintun Userspace Tunnel
                                 Connection Name:
                                 DHCP Enabled:    No
                                 IP address(es)
                                 [01]:
                                 [02]:
                                 [03]:
                           [02]: Intel(R) 82574L Gigabit Network Connection
                                 Connection Name: Ethernet0
                                 DHCP Enabled:    No
                                 IP address(es)
                                 [01]:
                                 [02]:
                           [03]: Wintun Userspace Tunnel
                                 Connection Name:
                                 DHCP Enabled:    No
                                 IP address(es)
                                 [01]:
                                 [02]:
Hyper-V Requirements:      A hypervisor has been detected. Features required for Hyper-V will not be displayed.

Simplified step-by-step of the installation process

.\distrod_wsl_launcher.exe -d Ubuntu
Choose Ubuntu -> Focal to install. After installation, update the system, then run wsl.exe --shutdown to terminate all WSL instances.

The proof regarding a task that requires systemd is working correctly

The reason I use fail2ban as the proof is that if fail2ban is installed on Ubuntu Focal without systemd, it will prevent the WSL instance from successfully executing sudo do-release-upgrade, for during the upgrade process, it is trying to use systemctl to stop its daemon which is never actually running after the installation. Since systemctl cannot be found, forcing do-release-upgrade to roll back. Please note that if by editing /etc/apt/sources.list to change the codename then run sudo apt full-upgrade to upgrade Ubuntu, fail2ban would not cause any problem
sudo apt install fail2ban
Run Restart-Computer in PowerShell. After Windows reboot, launch the WSL instance
sudo systemctl status fail2ban
● fail2ban.service - Fail2Ban Service
     Loaded: loaded (/lib/systemd/system/fail2ban.service; enabled; vendor preset: enabled)
     Active: active (running) since Tue 2021-11-23 01:10:25 PST; 1min 7s ago
       Docs: man:fail2ban(1)
    Process: 98 ExecStartPre=/bin/mkdir -p /run/fail2ban (code=exited, status=0/SUCCESS)
   Main PID: 103 (f2b/server)
      Tasks: 5 (limit: 4709)
     Memory: 17.0M
     CGroup: /system.slice/fail2ban.service
             └─103 /usr/bin/python3 /usr/bin/fail2ban-server -xf start

Nov 23 01:10:24 systemd[1]: Starting Fail2Ban Service...
Nov 23 01:10:25 systemd[1]: Started Fail2Ban Service.
Nov 23 01:10:26 fail2ban-server[103]: Server ready

Change Prompt=lts to Prompt=normal in /etc/update-manager/release-upgrades to see if fail2ban prevents the system from upgrade
sudo do-release-upgrade
After it prompt for restart, run wsl.exe --shutdown
then
sudo systemctl status fail2ban
● fail2ban.service - Fail2Ban Service
     Loaded: loaded (/lib/systemd/system/fail2ban.service; enabled; vendor preset: enabled)
     Active: active (running) since Tue 2021-11-23 01:24:49 PST; 1min 8s ago
       Docs: man:fail2ban(1)
    Process: 80 ExecStartPre=/bin/mkdir -p /run/fail2ban (code=exited, status=0/SUCCESS)
   Main PID: 85 (fail2ban-server)
      Tasks: 5 (limit: 4709)
     Memory: 18.6M
     CGroup: /system.slice/fail2ban.service
             └─85 /usr/bin/python3 /usr/bin/fail2ban-server -xf start

Nov 23 01:24:49 vsystemd[1]: Starting Fail2Ban Service...
Nov 23 01:24:49 systemd[1]: Started Fail2Ban Service.
Nov 23 01:24:49 fail2ban-server[85]: Server ready


perf: reuse gzip reader and avoid rereading buffer for compressed requests

Similar to the other change for uncompressed requests, do not reread
the buffer from the beginning and start where we left off.
The gzip reader is being reused and more data is read lazily.
Depends on https://github.com/elastic/go-docappender/pull/122

Sorry I'm late to the party, but has it been considered to just track the document boundaries in a separate array? By recording the value of buf.Len() ) each time we add a doc. It looks like we could get away with 2 bytes per document to track that, which would avoid all this extra processing.
This seems this could save both on CPU (no need to decompress/recompress) and memory usage (2 bytes per document seems much smaller than having to use copyBuf, since we wouldn't need copyBuf, and wouldn't need to decompress+recompress.
The only potential downside I can see is that we'd need to flush the gzip writer every time we add a document (se we can accurately record buf.Len() - but intuitively it seems the simplification and lowered memory usage seems worth it?
I might be missing something though (apologies if that's the case!)

The only potential downside I can see is that we'd need to flush the gzip writer every time we add a document (so we can accurately record buf.Len() - but intuitively it seems the simplification and lowered memory usage seems worth it?

Compression of multiple documents would suffer though, so we would need to measure it. I think we should optimise for the happy path, and retries can be a little more expensive.
We could record the positions of the uncompressed documents though. We would still need to decompress/recompress, but we wouldn't need to scan for newlines.

Yeah, maybe that wasn't the best idea - what really prompted my reaction is that I am generally wary of optimizing for the happy path at the expense of the slow path when the slow path is more likely during operational events. In this case if ES is sending us 429s it probably means we're under higher load, and making things even more expensive when that happens seems like a slippery slope. In cases like this consistent performance is often preferable to peak performance.


RichEditor does not render in dependable group

Describe the bug
If you add the RichEditor field in a dependable group, you get this once you select the value to show the group.
 If I add the contents of both to resources/views/components/layouts/base.blade.php it does work.

Can you recache your views and recheck please?
php artisan view:cache

@danharrin thanks, recaching the views worked.


IonConfig swipeBackEnabled not working on 4.0.0-beta17

Bug Report
Ionic Info
Ionic:

   ionic (Ionic CLI)             : 4.4.0
   Ionic Framework               : @ionic/angular 4.0.0-beta.17
   @angular-devkit/build-angular : 0.8.4
   @angular-devkit/schematics    : 0.8.4
   @angular/cli                  : 6.1.5
   @ionic/angular-toolkit        : not installed

Cordova:

   cordova (Cordova CLI) : 7.1.0
   Cordova Platforms     : android 7.1.2, browser 5.0.4, ios 4.5.5
   Cordova Plugins       : cordova-plugin-ionic-keyboard 2.1.3, cordova-plugin-ionic-webview 2.1.4, (and 15 other plugins)

System:

   Android SDK Tools : 26.0.1 (/Users/jirik/Library/Android/sdk)
   ios-deploy        : 2.0.0
   ios-sim           : 7.0.0
   NodeJS            : v8.12.0 (/usr/local/bin/node)
   npm               : 2.15.12
   OS                : macOS High Sierra
   Xcode             : Xcode 10.0 Build version 10A255

Describe the Bug
IonConfig swipeBackEnabled:false does not work. Still can swipe to go back.
Same behaviour for Android and iOS.
@NgModule({
  declarations: [AppComponent],
  entryComponents: [],
  exports: [],
  imports: [
    BrowserModule,
    BrowserAnimationsModule,
    IonicModule.forRoot({
      mode: 'ios',
      swipeBackEnabled: false,
      backButtonText: ''
    })
]})

Expected Behavior
swipeBackEnabled configuration should be respected

I was able to disable swipe-to-go-back by adding [swipeGesture]="false" to ion-router-outlet like so:
<ion-router-outlet main [swipeGesture]="false"></ion-router-outlet>
I have not tried configuring it in IonicConfig like the OP.

I was able to disable swipe-to-go-back by adding [swipeGesture]="false" to ion-router-outlet like so:
<ion-router-outlet main [swipeGesture]="false"></ion-router-outlet>
I have not tried configuring it in IonicConfig like the OP.

Thanks, this works too. Anyway, thanks @manucorporat for fixing it globally.


Velocity does not work

Don't know about other use cases - in my case i was applying kinetic to the body, and using it to scroll around a fixed size div.  It scrolled but the deceleration functions were not being called. I don't know if this issue would happen if kinetic was applied to other elements (i assume not?) But again, time is tight, no time to test! :p
This was because of :  [line 410]
if (settings.x && scroller.scrollWidth > 0) { ..decelerate.. }
Where scrollWidth was being returned as a number , I checked its value and it was often 0, sometimes - or + by varying random seeming amounts
Taking out the tests for scroller.scrollWidth > 0 fixed the issue,
I don't have time to fully investigate but maybe an extra test to see if the kinetic object is body, don't check for scrollWidth/Height?
Cheers for the plugin anyway dude ,
Merry Christmas ;)

I have the same issue with scroller.scrollWidth, maybe @davetayls could explain that.
But I found a method to work around.
var $element = $("#dom");
$element.kinetic();
var kinetic = $element.data($.Kinetic.DATA_KEY);
kinetic._getScroller = function() {
	return {
		scrollLeft: function(left) {}, // custom scrollLeft function
		scrollTop: function(top) {}, // custom scrollTop function
		0: {
			scrollWidth: 1,
			scrollHeight: 1
		}
	};
};

Scroll properties are sometimes attached to 'html' and sometimes 'body'
depending on browsers. Have you tried $('html,body').kinetic()?
It was a very long time ago but I remember this coming up before but never
had to tackle it myself. Maybe check through old issues.
Finally you can alter methods on Kinetic via its prototype
$.Kinetic.prototype
Hope that helps
On Sat, 5 Dec 2015 at 21:53, Tsai KD<EMAIL_ADDRESS>wrote:

I have the same issue with scroller.scrollWidth, maybe @davetayls
https://github.com/davetayls could explain that.
But I found a method to work around.
var $element = $("#dom");
$element.kinetic();
var kinetic = $element.data($.Kinetic.DATA_KEY);
kinetic._getScroller = function() {
return {
scrollLeft: function(left) {}, // custom scrollLeft function
scrollTop: function(top) {}, // custom scrollTop function
0: {
scrollWidth: 1,
scrollHeight: 1
}
};
};
—
Reply to this email directly or view it on GitHub
https://github.com/davetayls/jquery.kinetic/issues/92#issuecomment-162250486
.


Change ikhal's color theme through config file #705

Theme should be now  configurable for users in config file
[view]
theme = dark  # light

You can do better


Remove App Icon Template

Old site is now https://applypixels.com/ which is a new pay service

          1 Error

  :no_entry_sign:
  Found 1 link issue

Link issue by awesome_bot

Line
Status
Link

2004
504
https://littlebitesofcocoa.com/

  Generated by :no_entry_sign: danger

Makes sense.....
And why does Little Bits of Cocoa keep going down!?!??!


CAMEL-14117: Removed code block that always disables mtom

Thanks the PR has been merged to the branches.


Undefined property: stdClass::$role.id_role (select -> table.column)

Hi, I read the documentation here about join query.
I have two simple table to join with. But there is an error says that Undefined property: stdClass::$role.id_role
Somehow the library recognize the table name as the column name, including the period symbol.
In the documentation says that the offices.officeCode is fine.
Here is my code :
User table structure  :
id_user*
id_role**
name
Role table structure :
id_role*
role_name
Controller :
$builder = $db->table('user')
->select('name, role.id_role, role.role_name')
->join('role', 'role.id_role = user.id_role');
return DataTable::of($builder)
->addNumbering()
->toJson(true);
Error :
ErrorException
Undefined property: stdClass::$role.id_role
VENDORPATH/hermawan/codeigniter4-datatables/src/DataTableQuery.php at line 101

can you show your javascript code?
what php and codeigniter version do you use?

can you show your javascript code?
what php and codeigniter version do you use?

I'm using php 8.0 and codeigniter 4.1.3
here is my javascript :
$('#table').DataTable({
processing: true,
serverSide: true,
ajax:{
url: 'user/datatable',
method: 'POST'
},
columnDefs: [
{ targets: 0, orderable: false}, //first column is not orderable.
],
sDom: "<'row'<'col-md-4'l><'col-md-4'B><'col-md-4'f>r>t<'row'<'col-md-6'i><'col-md-6'p>>",
buttons: [
'copy',
{
extend: 'excel',
orientation: 'landscape',
pageSize: 'A4',
},
{
extend: 'pdf',
orientation: 'landscape',
pageSize: 'A4',
},
{
extend: 'print',
orientation: 'landscape',
pageSize: 'A4',
}
]
});

try define your datatable option with columns like example. also if using numbering also set parameter addNumbering('number');
in your case maybe like this
columns: [
    {data: 'number'},
    {data: 'id_role'},
    {data: 'role_name'},
]

try define your datatable option with columns like example. also if using numbering also set parameter addNumbering('number');
in your case maybe like this
columns: [
    {data: 'number'},
    {data: 'id_role'},
    {data: 'role_name'},
]

Great, it works.. I think the error from before is come from the backend side not in the frontend side..
Thank you very much


ADD groot normalizer

I've marked this as ready for review now as the normalizer is out. Once #61 is merged, I'll go ahead and add the tests for groot

There are several unanswered comments (I just added a few more)

Oh, could you please let me know where I could find these comments? I can't find them in my mail or on GitHub.

Thanks. I think this context is not clear from the proposed documentation text.

I just updated the readme for db_harmonisation now. Is it ok now?


[FE]: Support additional Rally objects

Description:
(details of what and how should work)
Enahnce the list of supported Rally API objects
Additional Context & Ad Hoc Info:
(Add any other context about the problem here.)
Required

[ ] 1. DefectSuite (ex. DS2287)
[ ] 2. TestFolder (ex. TF11393)

Other (optional)

[ ] 1. Milestone (ex. MI254)
[ ] 2. ObjectiveConversationPost (learn what this is as it seems related to required one)
[ ] 3. PullRequest (now it gets interesting...)
[ ] 4. ConversationPost (any post in discussions)

Properly test out DefectSuite "Use as Template" feature in RVI!


Binary dependency for Windows?

I'd like to use this package on Windows.
Unfortunately I don't have the required environment setup to build the dependencies.
Would it be possible to provide a pre-compiled binary for Windows users?

Unfortunately, I don't really have the bandwidth to figure out how to supply binaries right now. A pull request would be great if you can figure out to do setup a system with binaries.

@GregPlowman Exciting news here. Our infrastructure for binaries has improved a lot and I just got the Windows binaries working. I'll tag a new release right away.

Exciting news indeed!
Thanks so much for this.


Fix the namespace variable

It was missed that the order of operations was resulting in a bool set for namespace, not the namespace desired
Cleaned up some related namespace things in install.yml and in the templates to make it easier to read

Fixes #35

Works great, thanks @bcrochet

/lgtm


Install lightweight client only

Hello! I was reading the installation here: https://sensuapp.org/docs/0.26/platforms/sensu-on-ubuntu-debian.html#sensu-core
It states:

The Sensu Core package installs several processes including sensu-server, sensu-api, and sensu-client.

It's great that yous have provided a .deb file for installation, this saves me time.
I have requirements to keep builds however as small as possible. So for the client Docker images we are to build I have to strip it as much as possible. Meaning that I would like to skip installing the sensu-server and sensu-api for the client images. How does one do this?
If it were me making this product I'd have two three .deb files. sensu-server, sensu-api and sensu-client. Then let me decide which hosts should have what.
So again how do I get the most lightweight install of a client?

@basickarl the difference would be a few KB (just the Ruby lib files), as every component shares the common base and set of runtime dependencies.

Creating separate packages for each component is not worth it at this time. In the future, when Sensu components have less in common and fewer runtime dependencies, you can expect separate packages.


PrismaClient proxy create an infinite loop

Bug description
I use Prisma with nestjs, when my service access the PrismaClient  instance enter an infinite loop. causing the service to hang.
I found through the debugger that the infinite loop occurred in @prisma/client/runtime/libary
https://github.com/prisma/prisma/assets/20939091/06c2ce03-31cc-423c-a10d-fc374437b62a
I'm sure I didn't use it improperly, Don't know what causes this problem
How to reproduce
@Injectable()
export class RegisterBackend {
  constructor(private readonly prisma: PrismaService) {}

  @Loader()
  async loader(@RemixArgs() _remixArgs: LoaderFunctionArgs) {
    console.log(this) // just access
    return {};
  }

  @Action()
  async post(@Body() body: RegisterDto) {
    return {};
  }
}

Expected behavior
No response
Prisma information

// This is your Prisma schema file,
// learn more about it in the docs: https://pris.ly/d/prisma-schema

generator client {
  provider = "prisma-client-js"
}

datasource db {
  provider = "mongodb"
  url      = env("DATABASE_URL")
}

model User {
  id          String      @id @default(auto()) @map("_id") @db.ObjectId
  email       String      @unique
  nickname    String
  gender      String
  birthday    DateTime
  password    String
  validated   Boolean
}

Environment & setup

OS: macOS
Database: MongoDB
bun version: 1.0.4

Prisma Version
"@prisma/client": "^5.5.0",
"prisma": "^5.5.0",

Are you effectively reporting the same problem as https://github.com/prisma/prisma/issues/18292?

Are you effectively reporting the same problem as #18292?

I guess so, not sure if this is a problem with nestjs compatibility, I tried to locate the problem, it seems that the _engine attribute is always undefined, Node addons seem to be unable to load, when I switch to singleton pattern everything is fine.
import { PrismaClient } from '@prisma/client'

const prisma = new PrismaClient()

prisma.$connect().then(() => {
	console.log('prisma connected')
})

export default prisma

Hey @JinYuSha0, could you please provide us with a reproduction? Without it we will not be able to make progress and will end up closing the issue via our internal process. Thanks :pray:

Hey @JinYuSha0  ,
Did you get a chance to have a look at @millsp's previous comment? It'd be helpful for us to move your issue forward.
Without further information, we'll have to close this issue. Thank you 🙏

@millsp @SevInf
I'd like to help you, But I can't provide a code repository. If you want to reproduce the issue, try wrapping prisma in a Nest provider, inject it into any service, and access it.
This problem is still there, I can provide code snippets
// PrismaProvider.ts
import { Provider } from '@nestjs/common';
import { ConfigService } from '@nestjs/config';
import { PrismaClient } from '@prisma/client';

export const PRISMA = Symbol('PrismaSymbol');

export const PrismaProvider: Provider = {
  provide: PRISMA,
  useFactory: (configService: ConfigService) => {
    return new PrismaClient();
  },
  inject: [ConfigService],
};

// CommonModule.ts
import { PrismaProvider } from<EMAIL_ADDRESS>
@Global()
@Module({
  imports: [CacheModule],
  providers: [
    PrismaProvider,
  ],
  exports: [
    PrismaProvider,
  ],
})
export class CommonModule {}

// SomeService.ts
@Injectable()
export class SomeService {
 constructor(
    @Inject(PRISMA) private readonly prisma: typeof Prisma,
  ) {}

  public async someMethod() {
    this.prisma;  // just access
  }
}

https://github.com/prisma/prisma/pull/23228 could have fixed this and will be included in the next release, but of course a confirmation of it now working as intended would be nice. Could you please try to temporarily install prisma@dev and @prisma/client@dev in your project and see if this fixes the problem?

Hey, I have the bug currently and would like to try prisma@dev and @prisma/client@dev but I have this message :

How can I do it ?

I did
npm install prisma@dev @prisma/client@dev
and I'm happy to say it works. Thank you !

@JinYuSha0 could you confirm that the issue is fixed in 5.11 as well?
Thank you.

@jkomyno @SevInf Sorry I haven't followed that project for a while, I'll take a look at it over the weekend

I am going to close the issue for now. If it still happens for you for prisma 5.11 or creater, feel free to open a new one.
Thank you!


Add GCS support

What is this PR for?
Add GCS support
This PR includes

Add GcsHandler
Add UI for gcs mode
Add test

What type of PR is it?
Feature
What is the issue?
https://github.com/rekcurd/dashboard/issues/69
How should this be tested?
Run unittest

Codecov Report

Merging #95 into master will increase coverage by 0.08%.
The diff coverage is 82.75%.

@@            Coverage Diff             @@
##           master      #95      +/-   ##
==========================================
+ Coverage   83.69%   83.77%   +0.08%
==========================================
  Files          42       42
  Lines        2098     2127      +29
==========================================
+ Hits         1756     1782      +26
- Misses        342      345       +3

Impacted Files
Coverage Δ

rekcurd_dashboard/apis/kubernetes_handler.py
82.95% <ø> (ø)
:arrow_up:

rekcurd_dashboard/models/data_server.py
93.18% <100%> (+5.37%)
:arrow_up:

rekcurd_dashboard/apis/api_data_server.py
95.39% <80.76%> (-3.02%)
:arrow_down:

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 aa4f7e5...95c1355. Read the comment docs.


Unable to login as admin

Empty username and password doesn't work. How can I debug it and provide more info?

hi @FieryCod! an empty username is correct, but your password should not be empty. the initial password was printed during the quickstart. you can also set a new password by running "mox setadminpassword" (mox doesn't have to be running).
if you are already sending the non-empty admin password, the next step would be to look at the logging output. the default log level is "debug", which prints access logging for all http requests. the request to the admin endpoint should be something like:
debug: "http request" (cid: 188f4180a72; pkg: http; httpaccess: ; handler: admin; method: get; url: /admin/; host: localhost:1080; duration: "254.544µs"; statuscode: 200; proto: http/1.1; remoteaddr: <IP_ADDRESS>:53364; tlsinfo: plain; useragent: "Mozilla/5.0 (X11; Linux x86_64; rv:109.0) Gecko/20100101 Firefox/114.0"; referrr: ; size: 79061)

the important part is handler: admin. if the handler isn't admin, the request isn't getting to the admin endpoint. it could be that you're trying to access the /admin/ page on the "public" listener. the quickstart only enables it on the "internal" listener by default, with internal ip's. i.e. this in config/mox.conf:
Listeners:
        internal:
                IPs:
                        - <IP_ADDRESS>
                        - [...]
                AdminHTTP:
                        Enabled: true

at startup, mox prints the listeners and handlers that it sets up. there should be a line like this:
l=print m="http listener" pkg=http name=local kinds="admin-http at /admin/" address=<IP_ADDRESS>:1080

so the admin endpoint is at http://<IP_ADDRESS>:1080/admin/.
if this also isn't the problem, it would help if you could send the "http request" logging output. it will probably give me a hint for where to look next. a little bit of additional logging in func checkAdminAuth in http/admin.go would probably show what's going on.
hope this helps!

I also failed. The page shows http 401 - unauthorized - mox account - login with email address and password. After checking the log, I found that it was judged asaccount  handler, not admin.

Ah, interesting, are you saying that you were opening http://localhost/admin/ and ended up at the account handler? And was this a default quickstart config?

I serve the mail server on remote machine, and I test it using ssh forward, so the url is http://localhost:8080/admin

I got it, I lost the / at the end. so that should be http://localhost:8080/admin/, not http://localhost:8080/admin

Good info, thanks! I'll look into adding a redirect from /admin to /admin/ because I think more people will run into this at some point.

I hope the previous commit solved it. Closing the issue, but feel free to reopen if there is more to do.


Latest ldapjs not yet in npm

ldapjs version 1.0.0 has many useful improvements. I need to fetch git project for ldapjs to work properly for my configuration.
Great work with version 1.0.0 !!

The 1.0.0 package was published to npm minutes after the tag was pushed.
$ npm install --silent ldapjs
<EMAIL_ADDRESS>node_modules/ldapjs
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>└──<EMAIL_ADDRESS><EMAIL_ADDRESS>

Great, Thank You. I realized it shortly after I submitted the issue. I had the old version in the package.json file.

⚠ This issue has been locked due to age. If you have encountered a recent
problem that seems to be covered by this issue, please open a new issue.
Please include a minimal reproducible example
when opening a new issue.


Fix regex for IPv6

Only match strings which end with ::1
Fixes #830

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "regex" git@github.com:achanda/libnetwork.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

LGTM

LGTM


Documentation for MediaLibraryService(), MediaSessionService, MediaLibrarySession.Callback, MediaSession.Callback

Where can we find proper Documentation for MediaLibraryService(), MediaSessionService, MediaLibrarySession.Callback, MediaSession.Callback
When I checked the media3 documentation on developer.android.com I could not find any documentation
all I found was the old documentation about MediaBrowserService
https://developer.android.com/guide/topics/media/media-controls
https://developer.android.com/guide/topics/media/media-controls#mediabrowserservice_implementation
and this
https://developer.android.com/guide/topics/media/media3/getting-started/mediasession
after going through the sample app and UAMP media3 example we can see we need to implement  MediaLibrarySession.Callback and need to override various methods like onConnect , onPostConnect ,  onCustomCommand ,  onGetLibraryRoot, onGetItem, onSubscribe, onGetChildren, onAddMediaItems, getMediaItemFromSearchQuery
We need proper documentation for these methods, we have to dig deep into android code to find any info, we only find info about  onConnect , onCustomCommand  however proper documentation will be helpful like how to use onGetLibraryRoot, onGetItem etc
also we need documentation about how to use media3 with various google products like google assistant, google auto, Automative OS, wear OS and how we can test these features.

I am new to using media session and I am learning by comparing old UAMP code and new UAMP-- media3/compose code
earlier we need to override onGetRoot() and onLoadChildren() which was required to be Called to get information about the children of a media item , in MediaBrowserServiceCompat() ,
now there is no need to do this?
what are the corresponding methods we need to override?
Suppose if we dont override these methods still can, ---> other apps mediabrowser/mediacontroller can communicate with our service like say wear OS, google assitant etc ? --> or do we need to override other methods

If you are familiar with the old API then you can look into the migration guide.
There is a section answering your question about overrdiing domain methods of the service.

I came across this for  mediabrowserservicecompat
https://medium.com/androiddevelopers/mediabrowserservicecompat-and-the-modern-media-playback-app-7959a5196d90
can we have similar blog post for  MediaLibraryService


Is it possible to show git branch and repo details when in a git directory for cmd.

I was wondering how it is possible to display git repo and branch information as shown in the illustration below.

source
Description of the new feature/enhancement
The illustration above shows git repo and branch details in a powershell window. Is the same possible in cmd? If yes, how? Else it would be a great feature to have.

This is the ONLY reason I never use MS Terminal....has to show the git branch else it is useless to me.  Every other git bash shell does this so why would I use this one?

This is the ONLY reason I never use MS Terminal....has to show the git branch else it is useless to me. Every other git bash shell does this so why would I use this one?

I notice there is a tendency to confused Terminal with the shell it happens to be tied to a Terminal pane.  Is that happening here?
I have not attempted to run it inside Terminal, but the Git Bash that is installed by along with the GitHub Desktop app on Windows seems to do what you want.

I notice that the GitCMD does not seem to do such a thing, unless there is something in the prompt string that achieves it.  I did not look into that.


More flexible coef greek

This is kind of neat! I included a new raw_tex argument that can be passed for the coefficients. For the intercept, it will still take "alpha" or "beta", but if you have raw_tex = TRUE you can also use raw tex code for it. For example

Unfortunately it looks like the underscores in the variable names are messing some things up here. I feel like we had fixed that at some point but it doesn't seem to be working here...

Oops.. meant to mention this addresses #54.

Also, I don't know why it's showing up as bill\_depth\_mm because it doesn't show up that way when I use preview and the code looks correct to me. Thoughts @andrewheiss? (If you don't have time to work on this now, that's fine)

I'm going to merge this for now and open a new issue about the variable name escaping.

cool! thanks!


A few questions about the project

How can I create a definition package for my module having version 1.0.0?
Can I say that the package definitions for module version> = 1.0.0 <= 1.2.1?
Will the ability to publish a package definition, as simple as in npm?
I understand it correctly, it can automatically download definition files for the dependencies specified in package.json, bower.json?

How can I create a definition package for my module having version 1.0.0?

Just write it. There's a few issues and examples around, but until official documentation is created on specifically this you'll have look around. Assuming you're familiar with .d.ts files, it's just a matter of creating them without declare module "...".

Can I say that the package definitions for module version> = 1.0.0 <= 1.2.1?

Currently, no. Since there's no official enforcement yet, there's no need. Eventually, it'll be good to use typings.json with the version field as a range to achieve this. For now, the registry is stored against the minimum valid version.

Will the ability to publish a package definition, as simple as in npm?

Yes, see https://github.com/typings/typings/issues/27 (it's not yet available, but is certainly going to happen down the line).

I understand it correctly, it can automatically download definition files for the dependencies specified in package.json, bower.json?

Not yet, there's a task to track that: https://github.com/typings/typings/issues/24.

angular2 has several modules Angular2 core, bootstrap, router, http, animate,  platform/browser, platform/server. How to create a definition package, if I use it as follows:

import {bootstrap} from "angular2/bootstrap";
import {Component} from "angular2/core";

I found solution, create index.d.ts and add a
import './animate';
import './bootstrap_static';
import './bootstrap';
import './common';
import './compiler';
import './core';
import './http';
import './instrumentation';
import './router';
import './testing_internal';
import './testing';
import './upgrade';

In the typings.json specify main as index.d.ts

Currently, no. Since there's no official enforcement yet, there's no need. Eventually, it'll be good to use typings.json with the version field as a range to achieve this. For now, the registry is stored against the minimum valid version.

Extremely inconvenient to follow the version using commit. I consider it necessary to require an indication of the version. This will prevent unnecessary manual work. If you need to find a definition for the older version of it turns into a nightmare

@khomyakov42 There are existing issues for that, please consider searching the backlog. I realise it can be tricky to traverse, but it makes it easier for contributors to keep track: https://github.com/typings/typings/issues/108

Extremely inconvenient to follow the version using commit. I consider it necessary to require an indication of the version. This will prevent unnecessary manual work. If you need to find a definition for the older version of it turns into a nightmare

Please attach something actionable, I can only assume you haven't actually look into the project here. There is no basis for "finding an older version turns into a nightmare". Why is it a nightmare? Please look at the registry and you'll see there's already support for multiple versions (E.g. https://api.typings.org/versions/dt/node).


spcomp.exe stack, 100% cpu

Basic informations

OS: [Windows]
VSCode version: Insider
Extension version: Latest

Further Information
When working with methodmaps my computer started taking off.

I don't know the root of the cause, other than methodmaps being when it started with repeated compilers.
To Reproduce

Make a methodmap entry in your .sp code
Make some kind of invalid code (?)

methodmap HelloThere {
 public native bool
}

Expected behaviour
I would expect the previous ongoing compile to be closed — giving an error — limited to 1-2 compiles at a time, or shutting down the extension host as an error.
Error messages
These may or may not have occurred when I forced the processes to stop.
[Extension Host] TypeError: Cannot read properties of null (reading 'text')
	at ib (c:\Users\yruge\.vscode-insiders\extensions\sarrus.sourcepawn-vscode-4.2.1\dist\spIndex.js:440:9234)
	at Sp.walkTree (c:\Users\yruge\.vscode-insiders\extensions\sarrus.sourcepawn-vscode-4.2.1\dist\spIndex.js:442:714)
	at ed (c:\Users\yruge\.vscode-insiders\extensions\sarrus.sourcepawn-vscode-4.2.1\dist\spIndex.js:442:86)
	at hb (c:\Users\yruge\.vscode-insiders\extensions\sarrus.sourcepawn-vscode-4.2.1\dist\spIndex.js:443:90)
	at Timeout._onTimeout (c:\Users\yruge\.vscode-insiders\extensions\sarrus.sourcepawn-vscode-4.2.1\dist\spIndex.js:442:1718)
	at listOnTimeout (node:internal/timers:559:17)
	at process.processTimers (node:internal/timers:502:7) (at console.<anonymous> (c:\Program Files\Microsoft VS Code Insiders\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:90:142292))
log.ts:313   ERR Cannot read properties of undefined (reading 'name'): TypeError: Cannot read properties of undefined (reading 'name')
	at nv (c:\Users\yruge\.vscode-insiders\extensions\sarrus.sourcepawn-vscode-4.2.1\dist\spIndex.js:485:6076)
	at Cd.provideCompletionItems (c:\Users\yruge\.vscode-insiders\extensions\sarrus.sourcepawn-vscode-4.2.1\dist\spIndex.js:485:11403)
	at K.provideCompletionItems (c:\Program Files\Microsoft VS Code Insiders\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:92:51689)
	at c:\Program Files\Microsoft VS Code Insiders\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:92:76815
	at ne._withAdapter (c:\Program Files\Microsoft VS Code Insiders\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:92:66986)
	at ne.$provideCompletionItems (c:\Program Files\Microsoft VS Code Insiders\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:92:76791)
	at i._doInvokeHandler (c:\Program Files\Microsoft VS Code Insiders\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:100:13680)
	at i._invokeHandler (c:\Program Files\Microsoft VS Code Insiders\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:100:13362)
	at i._receiveRequest (c:\Program Files\Microsoft VS Code Insiders\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:100:12081)
	at i._receiveOneMessage (c:\Program Files\Microsoft VS Code Insiders\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:100:10834)
	at c:\Program Files\Microsoft VS Code Insiders\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:100:8941
	at p.invoke (c:\Program Files\Microsoft VS Code Insiders\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:61:145)
	at v.deliver (c:\Program Files\Microsoft VS Code Insiders\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:61:2266)
	at b.fire (c:\Program Files\Microsoft VS Code Insiders\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:61:1844)
	at c.fire (c:\Program Files\Microsoft VS Code Insiders\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:69:19049)
	at c:\Program Files\Microsoft VS Code Insiders\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:116:17106
	at p.invoke (c:\Program Files\Microsoft VS Code Insiders\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:61:145)
	at v.deliver (c:\Program Files\Microsoft VS Code Insiders\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:61:2266)
	at b.fire (c:\Program Files\Microsoft VS Code Insiders\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:61:1844)
	at c.fire (c:\Program Files\Microsoft VS Code Insiders\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:69:19049)
	at MessagePortMain.<anonymous> (c:\Program Files\Microsoft VS Code Insiders\resources\app\out\vs\workbench\api\node\extensionHostProcess.js:116:15146)
	at MessagePortMain.emit (node:events:526:28)
	at MessagePortMain._internalPort.emit (node:electron/js2c/utility_process_init:5:364)
13console.ts:137 [Extension Host] TypeError: Cannot read properties of null (reading 'text')
	at ib (c:\Users\yruge\.vscode-insiders\extensions\sarrus.sourcepawn-vscode-4.2.1\dist\spIndex.js:440:9234)
	at Sp.walkTree (c:\Users\yruge\.vscode-insiders\extensions\sarrus.sourcepawn-vscode-4.2.1\dist\spIndex.js:442:714)
	at ed (c:\Users\yruge\.vscode-insiders\extensions\sarrus.sourcepawn-vscode-4.2.1\dist\spIndex.js:442:86)
	at hb (c:\Users\yruge\.vscode-insiders\extensions\sarrus.sourcepawn-vscode-4.2.1\dist\spIndex.js:443:90)
	at Timeout._onTimeout (c:\Users\yruge\.vscode-insiders\extensions\sarrus.sourcepawn-vscode-4.2.1\dist\spIndex.js:442:1718)
	at listOnTimeout (node:internal/timers:559:17)
	at process.processTimers (node:internal/timers:502:7)

Pretty much the same as #63 I don't think the issue was ever solved. What SourcePawn Compiler version are you currently using for your project though?

Pretty much the same as #63 I don't think the issue was ever solved. What SourcePawn Compiler version are you currently using for your project though?

Ah — yes. I'm using the default one provided by the extension at user/sourcemodAPI. Can't tell the version. I'll try to work a bit on the compiler that comes with `sourcemod-1.11.0´ and see if it makes a difference.
I do relate to https://github.com/Sarrus1/sourcepawn-vscode/issues/63 statement of

I suspect that this comes from the linter, which never closes when the compile fails.

I'll let you know if the issue persists.

Thanks for reporting this issue.
I had someone reach out to me on Discord a while back with the same issue, which was also resolved by updating the compiler.

I'll let you know if the issue persists.

👍

Reoccurs seemingly at random;

Using SourcePawn Compiler <IP_ADDRESS>11 via sourcemod-1.11.0, which I've downloaded and selected via"sourcepawn.SpcompPath": "C:/Users/username/Documents/MEGA/Gits/csgo-plugins/sourcemod/scripting/spcomp64.exe" — from VSCode workspace settings; C:/Users/username/Documents/MEGA/Gits/csgo-plugins/.vscode/settings.json

Fixed in v5 with the use of the LSP.


Add missing ilab model help screen descriptions

Adds missing descriptions on the evaluate and download commands in the ilab model help screen. Also, capitalizes the List help text to be consistent with the others.

Checklist:

[x] Commit Message Formatting: Commit titles and messages follow guidelines in the
conventional commits.
[ ] Changelog updated with breaking and/or notable changes for the next minor release.
[ ] Documentation has been updated, if necessary.
[ ] Unit tests have been added, if necessary.
[ ] Integration tests have been added, if necessary.

New output:
(venv) alina@fedora:~/dev/instruct-lab/cli/instructlab$ ilab model --help
Usage: ilab model [OPTIONS] COMMAND [ARGS]...

  Command Group for Interacting with the Models in InstructLab.

  If this is your first time running ilab, it's best to start with `ilab
  config init` to create the environment.

Options:
  --help  Show this message and exit.

Commands:
  chat      Run a chat using the modified model
  convert   Converts model to GGUF
  download  Downloads model from a specified repository
  evaluate  Evaluates the trained model
  list      Lists models
  serve     Start a local server
  test      Runs basic test to ensure model correctness
  train     Takes synthetic data generated locally with `ilab data...

@alinaryan can you squash? can merge this in after :+1:

@nathan-weinberg Done


Get Web3Provider type from web3 in TS typings

Fixes issue #465

Closed by https://github.com/trufflesuite/ganache-core/commit/2f81da29cd13ad56b6464e0c61198538d12d7e20


Error during setup of component sonoff

HASSIO Core 2021.4.5
Home Assistant OS 5.13
Call Stack:
Log Details (ERROR)
Logger: homeassistant.setup
Source: util/async_.py:139
First occurred: April 18, 2021, 5:49:33 PM (1 occurrences)
Last logged: April 18, 2021, 5:49:33 PM
Error during setup of component sonoff
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/setup.py", line 242, in _async_setup_component
result = await task
File "/config/custom_components/sonoff/init.py", line 53, in async_setup
hass.data[DOMAIN] = Sonoff(hass, config)
File "/config/custom_components/sonoff/init.py", line 100, in init
self.do_login()
File "/config/custom_components/sonoff/init.py", line 175, in do_login
r = requests.post('https://{}-api.coolkit.cc:8080/api/user/login'.format(self._api_region),
File "/usr/local/lib/python3.8/site-packages/requests/api.py", line 119, in post
return request('post', url, data=data, json=json, **kwargs)
File "/usr/local/lib/python3.8/site-packages/requests/api.py", line 61, in request
return session.request(method=method, url=url, **kwargs)
File "/usr/local/lib/python3.8/site-packages/requests/sessions.py", line 542, in request
resp = self.send(prep, **send_kwargs)
File "/usr/local/lib/python3.8/site-packages/requests/sessions.py", line 655, in send
r = adapter.send(request, **kwargs)
File "/usr/local/lib/python3.8/site-packages/requests/adapters.py", line 439, in send
resp = conn.urlopen(
File "/usr/local/lib/python3.8/site-packages/urllib3/connectionpool.py", line 699, in urlopen
httplib_response = self._make_request(
File "/usr/local/lib/python3.8/site-packages/urllib3/connectionpool.py", line 394, in _make_request
conn.request(method, url, **httplib_request_kw)
File "/usr/local/lib/python3.8/site-packages/urllib3/connection.py", line 234, in request
super(HTTPConnection, self).request(method, url, body=body, headers=headers)
File "/usr/local/lib/python3.8/http/client.py", line 1255, in request
self._send_request(method, url, body, headers, encode_chunked)
File "/usr/local/lib/python3.8/http/client.py", line 1266, in send_request
self.putrequest(method, url, **skips)
File "/usr/local/lib/python3.8/site-packages/urllib3/connection.py", line 214, in putrequest
return HTTPConnection.putrequest(self, method, url, *args, **kwargs)
File "/usr/src/homeassistant/homeassistant/util/async.py", line 150, in protected_loop_func
check_loop()
File "/usr/src/homeassistant/homeassistant/util/async.py", line 139, in check_loop
raise RuntimeError(
RuntimeError: I/O must be done in the executor; Use await hass.async_add_executor_job() at custom_components/sonoff/init.py, line 175: r = requests.post('https://{}-api.coolkit.cc:8080/api/user/login'.format(self._api_region),

Happen to me as well when I upgraded Core version.
I reverted back to core-2021.3.4 and it's working again.
It's definitely not the solution, but it'll keep your integrations until a proper solution is available.

I believe this is due to the breaking changes listed in the latest home assistant update. There has been a deprecation warning for a year. The question is whether this repo is abandoned and/or if we can find another maintainer for it.

Same issue here... I have reverted the upgrade...

I tested this version which works for me: https://github.com/devbis/HASS-sonoff-ewelink
PR here: https://github.com/peterbuga/HASS-sonoff-ewelink/pull/248


x/playground: buttons Format and Share look disabled in dark theme

Go version
1.23
Output of go env in your module/workspace:
irrelevant in playground

What did you do?
Visit https://go.dev/play/
What did you see happen?

What did you expect to see?
I expected the two buttons Format, Share to have higher contrast colors, to show they are enabled.
The buttons do work properly:

Format formats the code
Share generates a permalink

It makes sense to have to button Run stand out as more prominent, as it is the most important button on the page, and the other buttons a bit less prominent. In light theme, there is no problem as all the buttons already look enabled. In dark theme, the colors are confusing and one may not use the buttons, thinking the features are not available.

Suggested fix for the buttons Format and Share:

apply the same font color as the Run button

By keeping the same borders (bright border for Run, no border for Format and Share) we'll still have the Run button stand out more prominently.

CC @toothrot, @golang/tools-team.

Thanks for reporting this. From a quick look, this might actually be a bug that affects only the "auto" mode. I can reproduce your screenshot when the theme toggle is set to "auto":

But it looks different, likely as originally intended, when the theme toggle is set to "dark":

So this probably just needs a fix to make the dark version of "auto" behave like the forced "dark" mode.

Good catch!
I can confirm "auto (dark)" is affected, while "dark" is not affected.

--turq-bright (#00769c) is being used instead of --turq-med (#50b7e0)
"turq-bright" may be a misnomer as it is rather dark, and has the same value as "turq-dark"
Regardless, we should make sure that rendering "auto (dark)" is always the same as rendering "dark"

Exact same problem with the "Popular Packages" links on the homepage go.dev :

They're too dark in "auto (dark)", while they look good in "dark".
Some styles are shared by the CSS classes .Playground-button and .WhyGo-reasonLink.


Relation Data tab: Sort is unclear

Is your feature request related to a problem? Please describe.
The header of each column in the Data tab when viewing a relation contains blue vertical arrows which allow to sort rows based on the value of each column. This feature is very useful, but I fairly often get confused using it.
I did not test extensively, but this happened often enough for me to be somewhat confident this originates from 2 issues:

When the user uses a column's arrows to sort by its value, then another column's arrows to sort by its value, the result becomes sorted according to the values of both clicked columns. Several applications behave this way, but only when some modifier key (Shift) is pressed during the second click. That issue should probably be the topic of a different ticket.
When rows are sorted by multiple columns, the UI does not indicate the precise way it sorts, excepts if one looks at the underlying query. The columns used to sort display a single arrow instead of 2, but there is no indication of which one has priority over which.

Issue #2 can be seen in the following screenshot, which shows a sort primarily on date_expires and secondarily on no_seq_masas_naad_adna_ifb_s:

The first column may suggest that some rows are missing, but in fact that's because they do not have NULL as date_expires and therefore show below. The corresponding query is
SELECT * FROM admgeo1.masas_naad_adna_infob_s
ORDER BY date_expires DESC,no_seq_masas_naad_adna_ifb_s DESC

Describe the solution you'd like
I believe the best solution would be to simply add a number to the right of the sorting arrow indicating the order of the column in sort priority. In the above example, there would be a "1" after the arrow for date_expires, and a "2" after the arrow for no_seq_masas_naad_adna_ifb_s. Seeing the number 2 would make it quite clear that the column is not the primary sorting criterion.
It would by the way make it extra clear that several columns are involved if the arrows had different colors when they are used for sorting. For example, the arrows of the 2 columns could be yellow instead of blue.
Additional context
This affects DBeaver 6.3.5.

thanks for suggestion


Fix acl permissions

Things included in this PR:

while creating ACLs in the initContainer in the daemonset, change the default user and group of the otel image to 999 (this is because splunk-otel-collector has uid of 999)

add post delete hook which deletes ACLs set for directories used by agent, gateway and clusterReceiver while those pods run as non-root users

This is a separate PR for some changes made in #668 . Those changes were originally included in #617 however, that PR was reverted.

Please link an issue that this PR resolves. This needs clarification why 2000 -> 999 user change is required. Also, how do we handle upgrades for users that already run agents with 2000?

I tested this migration 2000 -> 999  and pods didn't crash, both logs and metrics were still sent properly and there was no data loss.

@wojtekzyla Thanks for making acl changes for init containers. We are using helm v0.72.0 deployed in AKS env, But Azure defender reported running container runAsUser as root should be avoided. Please correct me if any thing missing here.
Later made changes to ds template runAsUser value to 999 under initi container (
https://github.com/signalfx/splunk-otel-collector-chart/blob/main/helm-charts/splunk-otel-collector/templates/daemonset.yaml#L108  & https://github.com/signalfx/splunk-otel-collector-chart/blob/main/helm-charts/splunk-otel-collector/templates/daemonset.yaml#L165 ).
Post deployment Splunk agents wasn't initialized and here is error:
Error from server (BadRequest): container "otel-collector" in pod "*-splunk-otel-collector-agent-2rnnr" is waiting to start: PodInitializing
Since we are not using fluentd I have made changes to acl (https://github.com/signalfx/splunk-otel-collector-chart/blob/main/helm-charts/splunk-otel-collector/templates/daemonset.yaml#L165) like below to resolve container initialization issues with Splunk Agents pods.
setfacl -n -Rm d:g:{{ $agent.securityContext.runAsGroup | default 999 }}:rx,g:{{ $agent.securityContext.runAsGroup | default 999 }}:rx /var/log/pods;
( discarded d:m::rx,m::rx,)
I believe setacl command was changed part of helm chart v0.68.0.
With Above changes Azure defender stopped reporting running container runAsUser as root should be avoided.
Please do review and assist me with refrain from making template changes or permanent fix for defender error.

Hi @kumachop2 , init-containers must run as root user in order to set the ACLs on the host machine.  Also. this dm:rx,m::rx, that you mentioned was added, because without it those ACLs weren't set correctly while using some container runtimes.

@wojtekzyla Thank you for prompt response. I would like to know the fix/steps to address Azure defender error on  init-containers  running as root user.


Add information on how to connect to Kubernetes

What type of suggestions?
Missing Information
What page is concerned?
No response
What is the issue?
Add information on how to connect to Kubernetes.
This suggestion was written by @johndietz in mkdocs.yml
Code of Conduct

[X] I agree to follow this project's Code of Conduct

@johndietz did you mean with kubectl?


0.1.123 Android: canvasses are redrawn on react-navigation tab switch

On Android, when switching tabs using react-navigation, Skia canvasses seem to be redrawn every time a tab is switched.
This is visually noticeable as it happens.
I don't think these redraws are necessary, as the rest of the react-renderings are not redrawn either.

An example would be extremely useful. Do you know the source of the redraw? Is it just because the component that contains the canvas re-render but since the canvas didn't change, we should try to avoid redrawing? Or this is something else?

Thanks for your quick response!
The components actually do not re-render on tab switch.
This bottom tab navigator switches the view, but keeps the react renderings alive.
I validated this by adding a useEffect console log to my components. Those get triggered just once, at initial render.
When switching tabs the console logs are not fired, but I see the Skia canvasses redraw.

Here's an example of the issue;
https://github.com/laurens-lamberts/react-native-skia-example
You'll notice that when you run this project on Android, switching the bottom tabs will redraw all canvasses.
On iOS, the canvasses will reappear instantly without an addition draw.

Thank you for providing such a simple, easy to reproduce the example. We are looking into this.

Could this be related to RNScreens and lazy loading etc causing React Native to re-create the native Skia view in the background?

We tested this with enableScreens(false) and the problem was no longer visible, so this has to do with the fact that the Skia View will be re-rendered when the view is invalidated by RNScreens. This is something we'll try to address.

We are investigating the possibility to offer an option to control this behaviour. In the meantime, we would like to understand your use case a little bit better, you create a canvas per number of views, may we ask why?

Sounds good!
We render a Skia canvas for some components we use in our react-native app.
For example, in the screenshot here you see the 'polaroid-like' components. The image in the polaroid is drawn using Skia, which allows us to draw those 'blurred badges' on top of the image.
If I understood the implementation guidelines of react-native-skia correctly, each of those individual polaroids should have their own Skia canvas JSX element, right?
This results in our Polaroid component containing the following Skia JSX hierarchy;
Canvas style={{ width: imageSize, height: imageSize }}>
  {!!skiaImage && (
    <>
      <Image
        x={0}
        y={0}
        width={imageSize}
        height={imageSize}
        image={skiaImage}
        fit="cover"
      />
      {hasBadge && skiaBadge}
    </>
  )}
</Canvas>

Hi @wcandillon,
We draw those leafs using the regular react-native Image component.
For now we draw those behind some Skia images, because we did not find an 'onError' callback for the Skia useImage hook (or the image rendering) yet. We needed a way to display a fallback image when Skia failed to render an image (because of broken image / broken image URL).
In the screenshot you share, there is indeed no Skia rendering yet.

The main reason for Android being a "problem" here is due to the difference between Android's TextureViews and iOS Metal rendering (the latter is much faster to initialise for a new view afaik).

Hi @chrfalch,
It looks exactly like what we need.
We will update react-native-skia in our project this week, and give the onError callback and undefined input a try.
Thanks a lot!

Though, the canvasses being redrawn (original topic of this ticket) is probably not related to the fixes you mention, right?

No, that is correct. The same reason will still apply: When the tab navigator switches back and forth the Skia Views will be remounted and repainted unfortunately.
So this is kind of hard to solve when you have a lot of Skia canvases in a view - it will take some time to render each of them, so maybe render them in a single canvas instead could help?

Would there be a way to keep the Skia views mounted and painted when using a tab navigator?
Reason I think this would be feasible is because the react elements also stay in place while switching tabs. Those do not get remounted and repainted. In our use case there is no practical reason to have the Skia views repainted when switching tabs, and I believe this is also pretty costly in terms of app performance.
Painting all Skia drawings in a single Canvas would not be a practicable solution for us, because we use a lot of separate components in our screens. Those components currently have their own canvasses. If we would need to have Skia drawings in components drawn in a central canvas, we would need to radically change our project architecture, and not in a react-preferable way I believe.

Ah, sorry for being unclear. The Canvas / Skia Views are not remounted as React components - but their native views are remounted. So as for how React handles this it is not a remount.
Since you are using react-native-screens (I think) the views behind the different tabs are unloaded and loaded natively when you switch tabs (the React state is kept).
This is unfortunately nothing we control from React Native Skia at the moment - and since the Skia views has a small (but noticeable) overhead when they are loaded, you will experience this janky behaviour when you mount (natively) a lot of views at the same time, especially on Android.

Thanks for your time explaining the difficulty. I understand this is not something that can be (or should be) solved within react-native-skia. Maybe we'll try an alternative to the bottom tabs. Something that keeps the renderings alive, if that won't cause performance issues. Otherwise we'll accept the currently visible Skia renderings.

Thanks for those kind words, and thanks for all your valuable feedback - it really helps. If you stumble upon something interesting, don't hesitate contacting us!

@wcandillon, that sounds great! Probably the visible drawing delay while using tabs with RN screens is negligible when the first-time-to-display is slightly improved. Looking forward to it! Thanks a lot.


Options example

I got super tripped up trying to pass options like allow_other. The only mention I found of the right syntax was https://github.com/zargony/rust-fuse/issues/92. This PR adds some example code in hello.rs that should help other people discover the syntax.

Thanks.
I think there's much room for improvements with mount options ;). Options are currently simply passed to libfuse and unfortunately there's no Rust idiomatic interface for passing options yet.


What is the meaning of 'flat' variable in vit_rollout.py?

The code doesn't use the variable after reevaluating it in line 27 https://github.com/jacobgil/vit-explain/blob/15a81d355a5aa6128ea4e71bbd56c28888d0f33b/vit_rollout.py#L27

@yojayc
This is for discarding the lowest attention weights, flat is generated as a view into the attention_heads_fused, therefore modifying flat in line 27, results in modifying attention_heads_fused, you can learn more about views here.
Not also as the indices that are equal to zero are filtered out, this is done because the attention weights accounting for the CLS token are kept by default.
I hope this helps.


@conda-forge-admin,please add bot automerge

[ ] I read the conda-forge documentation and could not find the solution for my problem there.

Issue:

Environment (conda list):

$ conda list

Details about  conda and system ( conda info ):

$ conda info

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/graph-tool-feedstock#62.


Switch to non-root user

One improvement to the Docker images we might consider is switching to a non-root user and provide ownership of some files (like the Conda install) to that user.
This ideally means changing the user ID at container startup to match the user ID outside the image. However changing the user ID recursively at startup can get kind of slow.
A trick we use in conda-forge's Docker images is to add a new group during build and change ownership of files to that group. While this is still slow, this happens at Docker image build time (as opposed to container startup time). So the user need not wait for this. That way at container startup, we need only add a user to the group owning those files with the user ID provided, which is much faster.
More details in this blogpost
Maybe we could adapt this strategy to our use case?

With the overhaul in #539, the images now use a non-root user of rapids who is a member of the conda group with write permissions to the conda directories.
Are there any other changes you need are needed?

Thanks Ray! 🙏
No let's close. We can follow up on anything else in new issues


Future Apple toolchain fixes

Fix build failures here and in dependencies with future Apple toolchain. See firebase/firebase-ios-sdk#7563

Sent https://github.com/grpc/grpc/pull/25611


Fix harvesting not showing 25% of the beneficiary

Now receipts are reduced to add all the fees harvested per block, not only the first one

Tested on my end.
Explorer -> http://explorer.testnet.symboldev.network/blocks/130681
Wallet


Convert projects in this repo to use ProjectReference

Follow-up to #3598.
The first phase of reducing repos was to merge the source code into a single repo. The second phase is to convert them from isolated folders of code with PackageReference across repo boundaries, to projects which use ProjectReference.
We've already started some of this work. See https://github.com/aspnet/AspNetCore/pull/4013 for example and https://github.com/aspnet/AspNetCore/tree/master/src/DataProtection.
As done in #4013, we implement this using <Reference> items in .csproj and custom Reference resolution targets. This is done so we can automatically pick the right versions of assemblies to used based on conditions such as, is this a test project? is this branch building a patch or a new minor release? what versions were used in the last stable release? etc.
See also https://github.com/aspnet/specs/blob/master/runtime/design-notes/2018-10-03-mondo-source-org.md for the spec on source code organization.

cc @jkotalik who is helping do this for src/IISIntegration/

Once I get this done for IISIntegration, I'll do other repos too. IISIntegration was significantly more difficult to port than other repos (due to native dependencies).

I'm adding a checklist to the original issue to track the groups of subrepos that we need to convert. Let's start by doing these one at a time.

FYI - here is the script I've been using to convert Project/Package refs to just <Reference>
https://gist.github.com/natemcmaster/40556be8f8d821a225f48afbd743bbeb
Usage:
PS> Get-ChildItem *.csproj -Recurse | % { C:\src\scripts\mk_ref.ps1 $_ }

Once https://github.com/aspnet/AspNetCore/pull/4954 is merged and makes its way up to master, I think we should be unblocked to convert Components and Mvc.
FYI @SteveSandersonMS @javiercn - the refactoring on Components is likely to be fairly disruptive. Do you have any big changes that need to go in this week?

Thanks for the heads-up, @natemcmaster! I’d definitely like to merge #5166 before any other disruptive change goes in. Should be ready to merge as soon as it’s reviewed.

@natemcmaster #5166 is merged now, so you should be OK to do changes to Components to use <Reference> etc. When doing so, please be sure to run the E2E tests (Microsoft.AspNetCore.Components.E2ETest) manually to verify that it hasn't broken aspects of how we bundle stuff for both server and client-side execution. We're going to run those E2E tests automatically in CI soon (#5158) but for now it's manual.

@SteveSandersonMS I started looking into this but found code in src/Components/ that I'm unsure about, specifically, the code in src/Components/blazor/. Can you take a look at the details I posted in this comment? https://github.com/aspnet/AspNetCore-Internal/issues/1535#issuecomment-450685185


Bad triangle in first created solid

https://github.com/wootguy/bspguy/pull/37

Does this happen only with the -createnew code? I don't see this when adding a new solid to an existing map.

Yes If you create empty bsp(), and add solid. You lost one triangle, in game also this triangle not visible.