Strange behavior - worker and db.changes

When testing the db.changes with the worker adapter :
I encounter a strange behavior with my chrome desktop and the chrome Android in regard to the paste.pouchdb.com.
When using the test script on this site (available here) : http://paste.pouchdb.com/paste/1df1i5
I can see in the console logs that the db.changes listener is working.

When I use this code on my local chrome client or on my android chrome client in a html file.
The db.changes listener is not working.
Do you know what is going on, and why I encounter this issue ? ?
Regards,
Ron

I created a testWorker3.html file and opened it on Firefox and Chrome.
When Testing it on my Ionic App, I also have the same problem.

The script is working when using a webServer for the execution.
I'm retesting on Ionic.
Currently on my app, the db.changes doesn't work at all.

This is the result of the script on the Ionic APP (I replaced the current index.html with the script  http://paste.pouchdb.com/paste/1df1i5)
I'm don't have the db.changes results

I'm sorry, can you please provide some simple steps to reproduce? I don't understand what the problem is, or how to reproduce it.
E.g. fill out this form:

Steps to reproduce
Expected result
Actual result
Platform, browser, etc.

I'm on it

I'm getting the localDB object through a AngularJS Service.
it is maybe the origin of the pb.
When I declaring the db.changes straight after the creation it is working fine.

I found the problem.
I wasn't able to see error inside the web worker.
I implemented an external variable in the filter property of the db.changes.
and the web worker wasn't able to get it, and I did not have any errors :)


Could not find a declaration file for module 'vue-good-table'

Trying to get Vue Good Table to work with https://github.com/web2033/vite-vue3-tailwind-starter.
Getting error:

Could not find a declaration file for module 'vue-good-table'. 'c:/Users/vv3/node_modules/vue-good-table/dist/vue-good-table.cjs.js' implicitly has an 'any' type.
Try npm i --save-dev @types/vue-good-table if it exists or add a new declaration (.d.ts) file containing `declare module 'vue-good-table';

When trying to import into component as of: https://xaksis.github.io/vue-good-table/guide/
Installed VGT today (2022-02-07) with npm install --save vue-good-table
What am I doing wrong?

Ouch, guess it's my bad. I'm trying to get this to work on Vue 3. Guessing this topic can closed...


Integrate event logging into app

Add an example interaction that can be logged as a user event, ex: switching between languages

Choosing to use Google Analytics as VAC has familiarity with product

Closed in #42


Implement dry-run mode

Proposed changes:

Support for ComponentReconciler to run in dry-run mode
Only renders manifests and returns them / or error

/retest

/retest


python310Packages.restrictedpython: 5.2 -> 6.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.restrictedpython is: "Restricted execution environment for Python to run untrusted code"
meta.homepage for python310Packages.restrictedpython is: "https://github.com/zopefoundation/RestrictedPython"
meta.changelog for python310Packages.restrictedpython is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 6.0 with grep in /nix/store/r7kpjlx7nw050q3q1h610d6k8l05alsp-python3.10-restrictedpython-6.0
found 6.0 in filename of file in /nix/store/r7kpjlx7nw050q3q1h610d6k8l05alsp-python3.10-restrictedpython-6.0

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

4 package rebuild(s)

First fifty rebuilds by attrpath
home-assistant-component-tests.python_script
portmod
python310Packages.restrictedpython
python39Packages.restrictedpython

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/r7kpjlx7nw050q3q1h610d6k8l05alsp-python3.10-restrictedpython-6.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.restrictedpython https://github.com/r-ryantm/nixpkgs/archive/eb45eb27cbcd15589d7e9d2408b786812fa60543.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/r7kpjlx7nw050q3q1h610d6k8l05alsp-python3.10-restrictedpython-6.0
ls -la /nix/store/r7kpjlx7nw050q3q1h610d6k8l05alsp-python3.10-restrictedpython-6.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

4 packages built:

home-assistant-component-tests.python_script
portmod
python310Packages.restrictedpython
python39Packages.restrictedpython

Maintainer pings
cc @uningan for testing.

I finally get some time to reinstall my home-assistant instance and seems everything using restrictedpython is working well.


Every time build is run, update /configuration/package.js from package.json.

Best done during build-webpack.

Update the package.js, and also add an additional build number.


⚠ ClamAV has degraded performance

In 4b04d59, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 54 ms

Resolved: ClamAV performance has improved in 0d7a131.


Unexpected *args and **kwargs behaviour

Hi, thanks for sharing this project. I'm trying it out and have stumbled upon a few issues related to *args and **kwargs:
Calling JS method with *args passess null context to apply
        function Example() {};
        Example.prototype.foo = function () { console.log(this, arguments); };

        var example = new Example();
        example.foo(1, 2, 3);
        // Works as expected:
        // Object {  } Arguments { , 5 more… }

args = [1, 2, 3]
example = __new__(window.Example())
example.foo(*args)
# is transcribed to example.foo.call(null, args), "foo" context is lost:
# Window Arguments { , 5 more… }
example.foo(1, 2, 3)
# Object {  } Arguments { , 5 more… }

Unexpected **kwargs behaviour:
def some_function(foo, bar=None):
    print(foo, bar)

some_function('foo', bar='bar')
# prints: foo, null

def other_function(foo, bar='default', qaz='default'):
    print(foo, bar, qaz)

other_function('foo', qaz='qaz')
# prints: foo, default, default

I've tested this behaviour on .207 and master

Hi Marcin,
I accidentally closed this one somehow, sorry...
This issue requires a fix and some explanation.
Will get back to you.
Thanks for the feedback!!

Hi Marcin
Since keyword args are expensive in JS, you have to switch them on, preferably locally, at function definition (rather than call) time.
You also can use command line switch -k at compilation time, but this is disadvised, see docs.
__pragma__ ('kwargs')

def some_function(foo, bar=None):
    print(foo, bar)

some_function('foo', bar='bar')
# prints: foo, null

def other_function(foo, bar='default', qaz='default'):
    print(foo, bar, qaz)

other_function('foo', qaz='qaz')
# prints: foo, default, default

__pragma__ ('nokwargs')

As far as *args is concerned: Using a pure JS method with *args wasn't possible in JS 5, since it didn't have the spread operator. But it is possible in JS 6, so I've added that functionality. If you compile for JS 6 (-e 6 switch) it should work from commit ca3e038.
I've added a test for this to the transcrypt_only manual_tests, and committed that as well.
Kind regards
Jacques
PS Let me know if this all works for you.
PS2 You don't need es6 to run es6, the minifier will currently convert to es5, even with the es6 switch on. What this means it that you can use the -e 6 switch on any browser that can run es5. You still can do sourcelevel debuggin on the minified code using sourcemaps.

Hi Jacques, thanks for the rapid response.
As of **kwargs, it seems that I haven't done my homework reading the docs thoroughly :)
Compiling for es6 spread operator seems like a good idea and will definitely do the job for me.
Thanks again for helping me out.

This is another case of the transcrypt compiler not giving an error on code it doesn't support and instead creates broken code. I'm fighting this problem with kwargs right now.
I'm starting to feel that the design of transcrypt has to be fundamentally broken, because this type of thing has happened to me a LOT now. It's fine to not support stuff, but silently failing in hard to debug ways isn't fine.

Many things  in Transcrypt are under command line switches or pragma's. Kwargs is one such thing. The reason that these things are optional is that the code they generate with current JS facilities is bulky. With JS supporting more and more of those Python-like things, the need for switches will diminish.
But then there indeed is the question of reporting. The problem here is that support of kwargs requires inclusion of a mechanism at definition time, whereas the problem becomes manifest at call time, so at different, unpredictable locations. So an error message could only be generated at runtime requiring some kind of run time mechanism, which  would add overhead.
One way to tackle this is start from the opposite direction: Compile in the 'luxury' mode (with all Python compat switches on) and then switch off those facilities you don't use.
All in all there's a compromise here.
As for the design being "fundamentally broken", that would mean design decisions were taken that lead us the wrong way. Compared to other Py to JS compilers I don't think that's the case.  Transcrypt is somewhere in the spectrum between fast on one hand and complete on the other. Starting out with a small sane subset Transcrypt now support most constructs, but indeed some optional. Keeping it lean has been an important objective. An implementation like PyPyJs is very complete (by design) but also unbearabley (and as I fear irrepairably, unless browser manufacturers cooperate) slow. And things like Brython are somewhere inbetween. So which one to use depends on the job. And of course there's Typescript.
People using Trancrypt a long time in practice generally are quite shappy with it. But then again they know the limitations. I  think the error reporting can certainly be improved, it didn't have much priority until now. I've created a special label for this, so that these cases can be considered together.

Yea, ok, "fundamentally broken" was bad phrasing on my part. I take it back. But it seems like unsupported stuff almost never results in an error message or a crash, but instead in invalid generated code. That's maybe a flaw in how you take a lot of small decisions.
For example, the bug what @= became None= suggests something like:
ops = {
   '*': '*',
   '/': '/',
}
js_op = ops.get(python_op)

...and then not checking for None. Crashing is much better than producing invalid output silently.

It's certainly important to have proper error reports.
While improvements are certainly possible (and will happen), the code may be more systematic than you anticipate (and looks a bit like you propose):

        self.operators = {
            # Highest precendence

            ast.Not: ('!', 16),
            ast.Invert: ('~', 16),
            ast.UAdd: ('+', 16),
            ast.USub: ('-', 16),

            ast.Pow: (None, 15),        # Dealt with separately

            ast.Mult: ('*', 14),
            ast.MatMult: (None, 14),    # Dealt with separately
            ast.Div: ('/', 14),
            ast.FloorDiv: (None, 14),   # Dealt with separately
            ast.Mod: ('%', 14),         # Used only for JavaScript %, which differs from Python %

            ast.Add: ('+', 13),
            ast.Sub: ('-', 13),

            ast.LShift: ('<<', 12),
            ast.RShift: ('>>', 12),

            ast.Lt: ('<', 11),
            ast.LtE: ('<=', 11),
            ast.Gt: ('>', 11),
            ast.GtE: ('>=', 11),
            ast.In: (None, 11),         # Dealt with separately
            ast.NotIn: (None, 11),      # Dealt with separately

            ast.Eq: ('==', 10),
            ast.NotEq: ('!=', 10),
            ast.Is: ('===', 10),        # Not really, but closest for now
            ast.IsNot: ('!==', 10),     # Not really, but closest for now

            ast.BitAnd: ('&', 9),
            ast.BitOr: ('|', 8),
            ast.BitXor: ('^', 7),
            ast.And: ('&&', 6),
            ast.Or: ('||', 5)

            # Lowest precedence
        }

   def visit_BinOp (self, node):
        if type (node.op) == ast.FloorDiv:
            self.emit ('Math.floor (')
            self.visitSubExpr (node, node.left)
            self.emit (' / ')
            self.visitSubExpr (node, node.right)
            self.emit (')')
        elif (
            type (node.op) in (ast.Pow, ast.MatMult) or
            (type (node.op) == ast.Mod and (self.allowOperatorOverloading or not self.allowJavaScriptMod)) or
            (type (node.op) in (
                ast.Mult, ast.Div, ast.Add, ast.Sub,
                ast.LShift, ast.RShift, ast.BitOr, ast.BitXor, ast.BitAnd
            ) and self.allowOperatorOverloading)
        ):
            self.emit ('{} ('.format (self.filterId (
                # Non-overloaded
                ('__pow__' if self.allowOperatorOverloading else 'Math.pow') if type (node.op) == ast.Pow else
                '__matmul__' if type (node.op) == ast.MatMult else
                ('__jsmod__' if self.allowJavaScriptMod else '__mod__') if type (node.op) == ast.Mod else

                # Overloaded arithmetic
                '__mul__' if type (node.op) == ast.Mult else
                '__div__' if type (node.op) == ast.Div else
                '__add__' if type (node.op) == ast.Add else
                '__sub__' if type (node.op) == ast.Sub else

                # Overloaded bitwise
                '__lshift__' if type (node.op) == ast.LShift else
                '__rshift__' if type (node.op) == ast.RShift else
                '__or__' if type (node.op) == ast.BitOr else
                '__xor__' if type (node.op) == ast.BitXor else
                '__and__' if type (node.op) == ast.BitAnd else

                'Never here'
            )))
            self.visit (node.left)
            self.emit (', ')
            self.visit (node.right)
            self.emit (')')
        else:
            self.visitSubExpr (node, node.left)
            self.emit (' {} '.format (self.operators [type (node.op)][0]))
            self.visitSubExpr (node, node.right)

However optional operator overloading and things like array slices for numpy greatly complicate matters. I try to find a balance between adding features and regularly reorganizing the code.

I like transcrypt a lot but would like to chime in on suggesting crashing, raising an exception, or at least emitting a warning for unsupported behavior (e.g., *args or **kwargs). I don't have a problem with transcrypt making design decisions not to support things for efficiency reasons, but it's painful to stumble upon these.
Thanks for your great work.


Document how to test packages automatically

We have documentation on how to write NixOS (module) tests.
https://github.com/tfc/nixpkgs/blob/40396a7e12419fc772063d2a3fa87ec7d14a46d9/nixos/doc/manual/development/writing-nixos-tests.xml
But how to write tests for packages?
I asked on IRC and got some helpful informations:
<davidak> are the nixos tests also intended for applications or only for nixos modules? there is for example chromium and fish, but mostly modules
<davidak> i want to add tests for my cli application packages
<gchristensen> they have a very high cost to run
<clever> davidak: for cli apps, its usually simpler to just put it into the derivation, as a checkPhase
<clever> i think the only reason things like chromium are in a nixos test, is because it needs X11 to test
<clever> so chromium itself isnt a module, but it relies on a module to start
...
<davidak> clever, this is a nice simple example of a test every program should have https://github.com/NixOS/nixpkgs/blob/076860e0340a5e4a909b9a710e186508b14d1c90/pkgs/development/tools/scalafix/default.nix#L34

<davidak> clever, eny idea how to test programs that connect to the intenet like `tldr --update`
<clever> davidak: the internet is disabled during all testing
<davidak> clever, so the answer ist "we can't test such cases"? that would be bad :/
gchristensen> we can't actually
<clever> davidak: what exactly is it fetching from the internet? could you fake it with an nginx server and /etc/hosts?
<davidak> i think it clones a git repo or download some archive
* mananamenos has quit (Ping timeout: 240 seconds)
<davidak> with that method it would need a nixos module test, but i think that's too costly for such a small program just a few people use

So my goal with this issue is to get this information documented.

explain the difference between NixOS module tests and package tests
explain when to use which
link to good examples for both
create one example for a module test with nginx to test a package that needs internet access to download stuff

We maybe have to decide on a policy when a test should be added and when not (e.g. it's not important enough to use costly module tests)
Related issues:

https://github.com/NixOS/nixpkgs/issues/34987
https://github.com/NixOS/nixpkgs/issues/27604
https://github.com/NixOS/nixpkgs/issues/46632
https://github.com/NixOS/nixpkgs/pull/36842

There is also passthru.tests to link to NixOS module test derivations from a package.
https://nixos.org/nixpkgs/manual/#sec-standard-meta-attributes
It should be called meta.tests initially, but it's not possible due to technical limitations.
https://github.com/NixOS/nix/issues/2532#issuecomment-487411058
https://github.com/NixOS/nixpkgs/pull/44439

@jtojnar commented in https://github.com/NixOS/nixpkgs/issues/27604#issuecomment-552021623

@davidak IMHO checkPhase is not a good place since it cannot be run independently of the package build. Tweaking the test would mean rebuilding the whole package, and so would disabling flaky test.
Me and @worldofpeace are adding new nixos tests to passthru.tests (example). r-ryantm runs derivations listed in that attribute (ryantm/nixpkgs-update#118) and ofborg might do that as well (NixOS/ofborg#410).
I think using runCommand in there might also work nicely.

So this looks like the way to go for now!

i still want to do it

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/how-many-people-are-paid-to-work-on-nix-nixpkgs/8307/51

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/contributor-retention/12412/45


Create Keyword and CLI option to "launch" a Playwright "BrowserServer"

Three new features:

New Keyword to launch a Browser server with options: Launch Browser Server with all arguments as New Browser plus port and wsPath. The latter two can be set, if the websocket path and port shall not be generated dynamically.
Keyword to close the browser server again. Can be either ALL or the websocket string.
A CLI option to launch-browser-server so that a server can be launched directly from terminal.

will be implemented by #3263


Change IGLWrap_jll package URL

@plut are you OK with taking over your package?

I'm somehow blocked here. If the URL is not changed I cannot upgrade the version of IGLWrap_jll and I cannot make a new release of a package that requires it. The only solution I see is that I create a different package _jll for the same C++ library. But I do not think this is desired.

Ok, we haven't heard from @plut for a few weeks and this is preventing us from developing this package on Yggdrasil.
I'm taking the executive decision to take over this JLL package so that we can continue working on it collaboratively on Yggdrasil.  Of course @plut will be welcome to contribute there.

@giordano Thanks very much! Can you  please re-trigger the registration of https://github.com/JuliaPackaging/Yggdrasil/pull/6943, or is this done differently.


add CellFinder to open-neuroscience.com

Hello!
We are reaching out because we would love to have your project listed on Open Neuroscience, and also share information about this project:
Open Neuroscience is a community run project, where we are curating and highlighting open source projects related to neurosciences!
Briefly, we have a website where short descritptions about projects are listed, with links to the projects themselves, their authors, together with images and other links.
Once a new entry is made, we make a quick check for spam, and publish it.
Once published, we make people aware of the new entry by Twitter and a Facebook group.
To add information about their project, developers only need to fill out this form
In the form, people can add subfields and tags to their entries, so that projects are filterable and searchable on the website!
The reason why we have the form system is that it makes it open for everyone to contribute to the website and allows developers themselves to describe their projects!
Also, there are so many amazing projects coming out in Neurosciences that it would be impossible for us to keep track and log them all!
Please get in touch if you have any questions or would like to collaborate!

Thanks for reaching out, I've been meaning to do this for a while. I've submitted cellfinder.

great, thanks!


ISY994 sensor improvements

Description:
Today, using Insteon sensors via the ISY994 platform is a pretty big pain. Issues:

The user has to name the sensors very specifically inside ISY994 based on the Hass config option.
The user has to specify the device_class manually using customize:
If the hardware sensor is configured to use unique nodes for on and off (which allows more powerful scene configuration in the Insteon ecosystem) then the sensor entities will not function whatsoever inside Hass, requiring the user to set up a "virtual entity" using the Programs feature in the ISY994 platform. While functional, these virtual entities update slower than native devices and require a lot of user configuration inside the ISY994 software.
More recent Insteon hardware report a daily heartbeat for the purpose of determining when a battery is dead, and there is no way to see this inside Hass due to it being an entity that never changes status.

This overhaul eliminates all of those pain points, and make all Insteon ISY994 sensors work out of the box with the following improvements:

The multiple ISY994 "nodes" used by a sensor no longer each get unique entities in Hass, but instead all attach to a single binary_sensor.
The binary_sensor logic is now smart enough to handle BOTH ways that hardware can report changes: Either by the primary node changing status, OR by the primary and negative nodes reporting ON events.
All Insteon ISY994 sensor classes are now automatically determined by looking at the ISY994 "type" attribute, which indicates the exact hardware in use.
The timestamp of the last heartbeat is now stored as a sensor attribute, allowing for automations to trigger on a missed heartbeat. On boot-up, the attribute will be set to STATE_UNKNOWN if and only if a heartbeat node is actually present for the device. This means that older sensors without the heartbeat feature will not get erroneously identified as missing heartbeats (if the automation is written correctly - I will provide an example in the updated docs)!

Pull request in home-assistant.github.io with documentation (if applicable): DOCUMENTATION PENDING
Breaking Change Info
ISY994 sensor devices previously showed up as switches by default, and now they will show up as binary_sensors. Any automations set up to trigger based on those old switch versions should be updated to the binary_sensor versions.
Checklist:
If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] New dependencies have been added to the REQUIREMENTS variable ([example][ex-requir]).
[x] New dependencies are only imported inside functions that use them ([example][ex-import]).
[x] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.

To summarize the question that needs @balloob's input:
Some battery-powered Insteon devices send a heartbeat signal every 24 hours as a mechanism to tell if a battery has died. Currently I have this implemented as:

A heartbeat attribute that gets updated to the current time (datetime.now().isoformat()) whenever the heartbeat signal is received.
If we know the device is heartbeat-capable but haven't seen one yet (on a Hass reboot) then the attribute gets set to STATE_UNKNOWN (allowing an automation to trigger if it stays UNKNOWN for >24 hours, for example)
If we know the device is not heartbeat-capable, the attribute does not exist in the first place

@MartinHjelmare pointed out that timestamps in attributes is frowned-upon, but I'm not immediately seeing another approach here that keeps the edge case handling. What do you think? Is there any heartbeat precedent in other components?

I set it to WIP. You can remove it after you implement that binary and remove the timestamp from attributes :+1:

I merged in some changes from PR #10664 which this is branched off of

@MartinHjelmare when you are done reviewing this PR, I'd love if you could toss the same message on to #10664, which this PR is branched off of (so all of the code in that PR is in the code you've been reviewing here)

If that's true what's the point of #10664?

Originally, #10664 was meant to be a small PR adding an important feature that things like this PR and others would need. But it took quite a while to get the new PyISY version all done and published, so in the meantime I worked on this PR as well.
I was trying to stick to the "smaller PRs" request, but this one depended on that other functionality so there was no choice but to branch from it. In a perfect world that other one would have been merged weeks ago :)
I suppose since you've been reviewing all of the code here, I could just close that other PR and roll both new features in to this one. I'll go with whatever is preferable on that front.

Yes, please close the other PR.

Done. Let me know when we're looking good here and I'll get the documentation written up! I wanted to wait until the implementation was locked-in :)

Re-adding WIP tag. I have discovered some issues with new devices I purchased for testing this change, and possibly with the latest ISY firmware 😭

Okay, should be ready for a final review again. I bought a motion sensor to validate these changes, and it turns out they work totally differently. I added a simple check to only apply the new logic in this PR to leak and door sensors.
I also improved initial state support so that doors with an unknown state on bootup will report as unknown until the sensor reports for the first time.

Excellent; I'll write up the documentation now!

Add a link to a docs PR and I think we can merge this.

Sorry for the one additional commit! In writing the documentation I realized that I never added the small bit of logic necessary for pre-5.x firmware to work, which was always my intention.

Description updated with a link to the documentation PR, and also added event support description here, which became part of this PR.


fix(npm): properly package all quickstart files

[skip ci]
#50

Coverage remained the same at 28.143% when pulling fe1933c0f10c8cf2e7f38c226cf9af48a0bdff83 on dpetev/npm into 7f7d8efa1ba4e557653679f43cac6404117c6672 on master.


One or more parameter values were invalid: Condition parameter type does not match schema type

firstly, thanks for this package, it seems a lot more promising than aws-lambda-graphql which we're currently using. I've followed all of the setup steps and I've got our API deploying w/ subscriptionless, but I'm getting the following error when calling instance.publish:
One or more parameter values were invalid: Condition parameter type does not match schema type

I think this is a dynamodb error...has anyone encountered this?

also (i don't know if this is related) it seems like no connections are being stored in dynamo, checking the logs for handleGatewayEvent is showing a series of CONNECT and DISCONNECT events, the latter with disconnectReason: 'Connection closed abnormally'. any advice would be hugely appreciated!

I got this working eventually, turns out it was issues with AWS privileges and not using graphql-ws correctly on the client! thanks so much again for the package


python3Packages.pyupgrade: 2.26.0 -> 2.27.0

Motivation for this change
Update to the latest upstream release 2.27.0
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 139341 at 1e4ccb74 run on x86_64-linux 1

2 packages built successfully:

python38Packages.pyupgrade
pyupgrade (python39Packages.pyupgrade)

Result of nixpkgs-review pr 139341 run on x86_64-linux 1

2 packages built:

python38Packages.pyupgrade
pyupgrade (python39Packages.pyupgrade)


Add SPNEGO proxy authentication

Added support for Proxy-Authenticate and Proxy-Authorization headers,
which are used for proxy negotiation.  See RFC 2617.

So our usage of this is within our authentication service, where we implement the ServerNegotiator interface for handling the SPNEGO process.
As a consumer of this, I'm not a huge fan of adding an additional method to the interface; I like small interfaces, and this requires changes in our services in order to prevent compilation errors.
Is there any way we can maintain the original interface while inspecting the original incoming http.Header? Hiding the details of the proxy/non-proxy negotiation within the one exposed method?

I don't think we can rely on the incoming HTTP header to indicate if it's a proxy connection or not.  The server is the first to respond with an authentication challenge, so it needs to initialize the conversation with Proxy-Authenticate.
We can keep the same interface if we use Go's polymorphic composition behavior that's not quite inheritance.  See kgraney/gssapi@72f9541 for a proposal.  This has the unfortunate side-effect of making KerberizedProxyServer initialization look somewhat ugly.
spnego.KerberizedProxyServer{spnego.KerberizedServer{lib}}

I can't think of a better way to handle this while letting the initialization of KerberizedServer remain the same.

Makes sense about the header.
I prefer the second proposal on your fork, personally. Even the initialization, while verbose, is intuitive to me; you're putting a proxy in front of a server, basically.

why not just add useProxyAuthentication bool to KerberizedServer? And then use the flag in Negotiate? Seems like would be the simplest, and the most idiomatic

@levb, do you mean like this?
type KerberizedServer struct {
    *gssapi.Lib
    useProxyAuthentication bool
}

Won't this change the interface for existing users who initialize this as KerberizedServer{lib}?

Sorry, meant exported of course, Capitalized.
    *gssapi.Lib
    UseProxyAuthentication bool
}

Prior initializations of KerberizedServer{lib} will no longer compile, and would need to be changed to KerberizedServer{Lib:lib} which IMO is acceptable. @alextoombs agree?

That's fine with me. Agree on the exported nature.

I'm happy with that.  The latest commit here has the change.
I think Travis is broken with an infrastructure bug?

@kgraney Sorry to be a pest, but may I ask you to a) maybe get rid of the type challengeType and replace it with 3 local variables? and b) squash when you are done?
Otherwise, LGTM once the travis issues are worked out. @alextoombs do you know what's up with that? And MANY THANKS for contributing!

Done, and thanks for doing all the work to get this library working with go!

For the Travis issue: I think we need this: https://github.com/apcera/gssapi/pull/47

@kgraney: I'm very sorry for the delay, but if you can rebase on top of master, this issue should be fixed.
The code looks great; thanks for contributing! As soon as the builds come back green, I'll merge.

Cool, thanks!  It looks like the tests pass now.

LGTM


chore(package.json): bump up dependencies version

Description:
This PR updates some of outdated dependencies to latest as well as type definitions.
Related issue (if exists):

Coverage decreased (-0.2%) to 96.94% when pulling 23c3e019f9deac01889d2320ee26248dedaf6f07 on kwonoj:bump-dependencies into e91d113e49acc9b0e12e2524040fa84cf3fa2c78 on ReactiveX:master.

Coverage decreased (-0.2%) to 96.94% when pulling 010a3d625c6fc03da1e52ab0a78ed7ce1856b69d on kwonoj:bump-dependencies into 2a40fb637dba1a62765cd5342ec902c320224af3 on ReactiveX:master.

needs rebase, unclear to me why coverage would be reduced--it shouldn't crawl into dependencies  and count them against us should it?

unclear to me why coverage would be reduced--it shouldn't crawl into dependencies and count them against us should it?

: me too. Only culprit seems nyc which does generating coverage report, may changed mechanism to collect coverage internally. No concrete explanation though.

@kwonoj maybe try pushing to this PR without updating nyc. Would let us know.

@jayphelps let me do that quickly.

https://github.com/ReactiveX/rxjs/pull/1963

@jayphelps seems nyc's reason, (seems) changed coverage generation bit.

Coverage decreased (-0.2%) to 97.001% when pulling 9db2e09829505c9d9be40314485889276a53b169 on kwonoj:bump-dependencies into 98e21ed79419e72752a37c5da19f028c44917939 on ReactiveX:master.

LGTM

Coverage decreased (-0.2%) to 97.001% when pulling 9db2e09829505c9d9be40314485889276a53b169 on kwonoj:bump-dependencies into 98e21ed79419e72752a37c5da19f028c44917939 on ReactiveX:master.


Update project to using AndroidX libraries.

Duplicate of https://github.com/first087/Android-License-Fragment/pull/8 but for the master branch.

Hello, there.
AndroidX went into RC1 this week 🎊! As said in the changelogs, it is production ready.
To keep with backwards compatibility, we could move the master and support-v4 branches to other branches. non-androidx and non-androidx-support-v4 for examples. Then make AndroidX the default.
I would be happy to help however I can. Have an awesome day.

Hello, there.
AndroidX went into RC1 this week 🎊! As said in the changelogs, it is production ready.
To keep with backwards compatibility, we could move the master and support-v4 branches to other branches. non-androidx and non-androidx-support-v4 for examples. Then make AndroidX the default.
I would be happy to help however I can. Have an awesome day.

Thank for contribute. Please update AndroidX Library to stable version.


problem using TextDescriptorsDriftMetric

Hi,
I'm applying to some metrics for my dataset which has text and numerical features. TextDescriptorsDriftMetric is applied to the column content but I get an error saying "ValueError: No objects to concatenate" when save_hmtl is done.
  report = Report(
      metrics=[
          ColumnDriftMetric(column_name="content"),
          ColumnDriftMetric(column_name="predictions"),
          DatasetDriftMetric(),
          DatasetMissingValuesMetric(),
          TextDescriptorsDriftMetric(column_name="content"),
      ]
  )
  report.run(
      reference_data=ref_df,
      current_data=cur_df,
      column_mapping=column_mapping,
  )

  report.save_html("monitor/aggregated.html")

I checked the reference and current datasets and they're equal and not empty. And column_mapping is defined as below. What could be the reason for this error?
from evidently import ColumnMapping

column_mapping = ColumnMapping()
column_mapping = ColumnMapping(
    prediction="predictions",
    text_features=["content", "summary"],
    target=None,
)

Error output:
Traceback (most recent call last):
  File "<string>", line 15, in <module>
  File "/Users/tugcegunes/Library/Caches/pypoetry/virtualenvs/pov004-automatic-news-filter-uqh4bnaG-py3.9/lib/python3.9/site-packages/evidently/suite/base_suite.py", line 184, in save_html
    dashboard_id, dashboard_info, graphs = self._build_dashboard_info()
  File "/Users/tugcegunes/Library/Caches/pypoetry/virtualenvs/pov004-automatic-news-filter-uqh4bnaG-py3.9/lib/python3.9/site-packages/evidently/report/report.py", line 171, in _build_dashboard_info
    html_info = renderer.render_html(test)
  File "/Users/tugcegunes/Library/Caches/pypoetry/virtualenvs/pov004-automatic-news-filter-uqh4bnaG-py3.9/lib/python3.9/site-packages/evidently/metrics/data_drift/text_descriptors_drift_metric.py", line 210, in render_html
    results = obj.get_result()
  File "/Users/tugcegunes/Library/Caches/pypoetry/virtualenvs/pov004-automatic-news-filter-uqh4bnaG-py3.9/lib/python3.9/site-packages/evidently/base_metric.py", line 184, in get_result
    raise result.exception
  File "/Users/tugcegunes/Library/Caches/pypoetry/virtualenvs/pov004-automatic-news-filter-uqh4bnaG-py3.9/lib/python3.9/site-packages/evidently/suite/base_suite.py", line 393, in run_calculate
    calculations[calculation] = calculation.calculate(data)
  File "/Users/tugcegunes/Library/Caches/pypoetry/virtualenvs/pov004-automatic-news-filter-uqh4bnaG-py3.9/lib/python3.9/site-packages/evidently/metrics/data_drift/text_descriptors_drift_metric.py", line 105, in calculate
    curr_text_df = pd.concat(
  File "/Users/tugcegunes/Library/Caches/pypoetry/virtualenvs/pov004-automatic-news-filter-uqh4bnaG-py3.9/lib/python3.9/site-packages/pandas/util/_decorators.py", line 331, in wrapper
    return func(*args, **kwargs)
  File "/Users/tugcegunes/Library/Caches/pypoetry/virtualenvs/pov004-automatic-news-filter-uqh4bnaG-py3.9/lib/python3.9/site-packages/pandas/core/reshape/concat.py", line 368, in concat
    op = _Concatenator(
  File "/Users/tugcegunes/Library/Caches/pypoetry/virtualenvs/pov004-automatic-news-filter-uqh4bnaG-py3.9/lib/python3.9/site-packages/pandas/core/reshape/concat.py", line 425, in __init__
    raise ValueError("No objects to concatenate")
ValueError: No objects to concatenate

Hi @tugcegns,
Thanks for reporting the bug!
Could you share:

Which evidently version are you using?
Are you able to show the report in the notebook? Or the error appears as well?
Does you text data contain missing values in either of the two text feature columns?

Hi,
Thanks for quick response!
I'm using evidently 0.3.3 and I've checked the data but no missing values.
If I run it in jupyter notebook, I'm getting the same error.
This is error log:
File ~/opt/anaconda3/envs/llm_env/lib/python3.8/site-packages/evidently/suite/base_suite.py:393, in Suite.run_calculate(self, data)
    391 logging.debug(f"Executing {type(calculation)}...")
    392 try:
--> 393     calculations[calculation] = calculation.calculate(data)
    394 except BaseException as ex:
    395     calculations[calculation] = ErrorResult(ex)

File ~/opt/anaconda3/envs/llm_env/lib/python3.8/site-packages/evidently/metrics/data_drift/text_descriptors_drift_metric.py:105, in TextDescriptorsDriftMetric.calculate(self, data)
    103 else:
    104     agg_data = True
--> 105 curr_text_df = pd.concat(
    106     [data.get_current_column(x.feature_name()) for x in list(self.generated_text_features.values())],
    107     axis=1,
    108 )
    109 curr_text_df.columns = list(self.generated_text_features.keys())
    111 ref_text_df = pd.concat(
    112     [data.get_reference_column(x.feature_name()) for x in list(self.generated_text_features.values())],
    113     axis=1,
    114 )

File ~/opt/anaconda3/envs/llm_env/lib/python3.8/site-packages/pandas/core/reshape/concat.py:372, in concat(objs, axis, join, ignore_index, keys, levels, names, verify_integrity, sort, copy)
    369 elif copy and using_copy_on_write():
    370     copy = False
--> 372 op = _Concatenator(
    373     objs,
    374     axis=axis,
    375     ignore_index=ignore_index,
    376     join=join,
    377     keys=keys,
    378     levels=levels,
    379     names=names,
    380     verify_integrity=verify_integrity,
    381     copy=copy,
    382     sort=sort,
    383 )
    385 return op.get_result()

File ~/opt/anaconda3/envs/llm_env/lib/python3.8/site-packages/pandas/core/reshape/concat.py:429, in _Concatenator.__init__(self, objs, axis, join, keys, levels, names, ignore_index, verify_integrity, copy, sort)
    426     objs = list(objs)
    428 if len(objs) == 0:
--> 429     raise ValueError("No objects to concatenate")
    431 if keys is None:
    432     objs = list(com.not_none(*objs))

ValueError: No objects to concatenate

Hi @tugcegns,
We ran on all our test datasets and unfortunately cannot reproduce the error.

Is there a chance you can share a small example dataset for which the error occurs? It can be a very small and obfuscated sample, or a totally toy example as long as you get the same error.

Also, are you able to generate a ColumnSummary metric for the text columns in your dataset?

summary_overview = Report(metrics=[
   ColumnSummaryMetric(column_name="summary"),
])

summary_overview.run(reference_data=ref_df, current_data=cur_df, column_mapping=column_mapping)
summary_overview

If yes, could you share a screenshot to understand the shape and parameters of the data?

Hello @elenasamuylova ,
I prepared the example dataset with text features, here you can find it: https://github.com/tugcegns/evidently-monitoring/blob/main/example_dataset.csv
For the ColumnSummaryMetric, it throws an error which is ValueError: unknown feature type text
Could you let me know if anything is wrong with my dataset values? Thanks!

Hi @tugcegns,
Thanks for sharing! I reproduced your code on your example dataset in Colab. Here it is: https://colab.research.google.com/drive/1H4SMRHzWC_GFfqGAnhMTQGUtpeQ6Rlmc?usp=sharing
It appears to work correctly.
To double-check, did you import NLTK (required for some descriptors) in your code?
import nltk
nltk.download('words')
nltk.download('wordnet')
nltk.download('omw-1.4')
nltk.download('vader_lexicon')

If yes, and the error still persists, you share any other details about your environment? Are you running the code specifically in Jupyter notebook?

Glad it is solved @tugcegns!


Unfoldable

UNFOLDABLE interface with List and Array implementation.
I used Js.Array.concat for the array implementation, but if you're really keen on performance, I'll switch it to use an explicit loop to build the array. Actually, the Belt library already uses optimized stuff like that for their concat, but not sure how you feel about using Belt in this library.

Thanks for the PR! I don't mind Belt -- I'm not a purist by any means, I use it and |. all of the time (it's grown on me). It's your call though, I'm ok with merging it as is

I switched to Belt.Array.concat since they use an optimized concat.


Update LuisRecognizer description after sample reorganization

Tracking Issue:
Update LuisRecognizer description after sample reorganization efforts.
The LuisRecognizer currently points to the 12.nlp-with-luis sample which may be renumbered in the upcoming sample work. If it is renumbered, we will need to change the link in the doc string:
/**
 * Recognize intents in a user utterance using a configured LUIS model.
 *
 * @remarks
 * This class is used to recognize intents and extract entities from incoming messages.
 * See this class in action [in this sample application](https://github.com/Microsoft/BotBuilder-Samples/tree/master/samples/javascript_nodejs/12.nlp-with-luis).
 *
 * This component can be used within your bots logic by calling [recognize()](#recognize).
 */
export class LuisRecognizer {

Link to code in master

@Kaiqb, please let us know if you need help on this and we can reassign to an SE

@CoHealer - now that luis sample is renamed and renumbered this needs to be updated.


Delete Option Functionality Streamlined

When deleting an option, the user is shown a list of the affected resources and links to their edit pages before confirming they want to delete that option.

fixed for comments!! @anniemeng

changed @anniemeng


Make compatible with ghc 8.8 and latest dependencies

Hi. I wasn't sure what to do about MonadFail but this at least builds.
When this is OK I'll do 8.10 as well.

This might be correct now.

Ok now it's working. I added the old manifest as stack-8.6.5.yaml

Ok, good I hope

Could there be a new hackage release with these new bounds?

@locallycompact v<IP_ADDRESS> uploaded to hackage

Thanks very much. Is it OK if I add these to stackage nightly?

sure if you want to check off the stackage requirements!


optimization: 安装 Agent 被忽略主机展示完整业务名称 (closed #1102)

Codecov Report

Merging #1109 (d94ef31) into V2.2.23-rc (3873344) will increase coverage by 0.01%.
The diff coverage is n/a.

@@              Coverage Diff               @@
##           V2.2.23-rc    #1109      +/-   ##
==============================================
+ Coverage       73.21%   73.22%   +0.01%
==============================================
  Files             322      322
  Lines           22978    22978
==============================================
+ Hits            16823    16826       +3
+ Misses           6155     6152       -3

Impacted Files
Coverage Δ

apps/node_man/handlers/validator.py
95.74% <ø> (ø)

apps/node_man/handlers/job.py
84.84% <0.00%> (-0.31%)
:arrow_down:

...nents/collections/agent_new/choose_access_point.py
95.60% <0.00%> (+0.48%)
:arrow_up:

apps/backend/subscription/steps/plugin.py
55.86% <0.00%> (+0.65%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Error relating to "kubeclt apply failed" presented for around 5 minutes during cluster provisioning

Version: master-head(00ff15976)
Steps:

Create a DO cluster in master-head
Provisioning error is displayed as below for around 5 minutes.

It recovers after 5 minutes

This happened to me today
Rancher version master-head (08/12/2020) d19f179c4
I was using DO driver to provision a cluster. Usually after deploying Rancher and provisioning a cluster shortly after I get this error occasionally

Tested with master-head(master-76c26fb0f9)
Deployed few node driver and custom clusters.
The error as mentioned above was not observed during provisioning.


Kerberos Authentication

Describe the feature you'd like
Option to authenticate using Kerberos, a widely used network authentication protocol.
Describe the benefits this would bring to existing BookStack users
This will enhance security and streamline the authentication process for users who are already using Kerberos in their environment.
Can the goal of this request already be achieved via other means?
Using the AD FS with SAML
Have you searched for an existing open/closed issue?

[X] I have searched for existing issues and none cover my fundamental request

How long have you been using BookStack?
Under 3 months
Additional context
No response

Thanks for the suggestion @PiRomant,
From my view, I wouldn't be keen to support additional auth systems without there being a clear and significant demand/requirement from the user-base, and since I've had very little request for this in the last 8 years, I don't think it's evident that such a demand/requirement is there.


distribution of prediction points

Thank you for your work!
May I ask if there was any uneven distribution of prediction points during your testing on the PCN dataset?
Looking forward to your reply!

Another question I would like to ask you is, for all datasets（PCN、EPN3D、scannet...）, does the network output 2048 data points?
Looking forward to your reply!

Thank you for your work! May I ask if there was any uneven distribution of prediction points during your testing on the PCN dataset? Looking forward to your reply!

Thank you for your interesting question. The point distribution focus mostly on observed regions, for unobserved regions. The points are very sparse. This is something we should improve.

Another question I would like to ask you is, for all datasets（PCN、EPN3D、scannet...）, does the network output 2048 data points? Looking forward to your reply!

Yes, for all dataset, we set the number of output points as 2048


Allow map to return nullable

What is the reason behind this? It make map to look almost like flatMap

It's based off your Some.java and it's consistent with that as Func1 doesn't claim the return value is non-null. It's for the cases where a property is Nullable (and can't be changed to a Option). E.g. -
// Nullable name
data class Cat(val name : String?)

fun myCat() : Option<Cat> { /** .. */ }

fun printCatName() {
  myCat().map { it.name }.ifSome { println("Name is: $it ") }
}

// Flatmap would require that Cat class had `Option<String>` for name
// In that case it could be  myCat().flatMap { it.name }.ifSome { println("Name is: $it ") }
}


Filters sort and uniq

First of all, this is a really interesting package, thanks for the effort.
In my scripts I often use the shell commands sort and uniq so I feel these two would be a good additions for this package.

Yes, I'd like that too! Could you provide a short example program of the kind that you'd like to write in script that uses sort and uniq the way you want to use them? That gives us a concrete design proposal to talk about.

The use case is normally to process some kind of log file where I try to find out the frequency of some thing. So in bash this normally looks like this:
cat some.log | grep "the log lines of interest" | awk '{ print $5 }' | sort | uniq -c | sort -rn | head

The awk part is used to select a particular column (see #6).
With uniq I normally use the -c to count the frequency.
sort is used in two ways, first to sort alphabetically and then to sort by frequency (emited by uniq) in reverse order.
In most cases I am only interested in the top X, therefore I use head.

Great! Can you turn this into a rough example of the Go program you'd like to write using script?

One challenge is, that the unix tools like grep, tr, sort, uniq, etc. have a lot of options and flags, that control the exact behaviour. In order to create an API, that is easy to use and flexible to extend, maybe functional options could be something worth to consider (but this is maybe a topic for another issue).
I think, something like this would be great (already combined with the functional options, which are placed in distinct packages for better readability):
script.Stdin().Match("the log lines of interest").Col(" ", 5).Sort().Uniq(uniq.WithCount()).Sort(sort.Reverse(), sort.Numeric()).Stdout()

I'm completely in sympathy with your ideas, but one of the design principles for script is that the API should be very simple. The most common things should need zero config. Something like this would be more in keeping with the spirit of the package:
script.Stdin().Match("the log lines of interest").Column(5).Uniq().Sort().CountLines().Stdout()

What do you think?

@bitfield In my opinion, the sample you have put together does not the same as my command, but there is also one simplification, that I like.
The simplification
For the unix command uniq to work, the imput needs to be sorted. Therefore my command had the sequence Sort().Uniq().Sort(). My understanding of your sample is, that the Uniq(), that would be implemented in the script package would not have this precondition (because the sorting is maybe done with a map), so there is no need to sort the imput first.
The differences
Column(5)
I can life with the fact, that the default behaviour of Column() would be to split by whitespace. Of course, there are a lot of cases, where the delimiter is something different (like , or ;). So my question is, how would you address these cases. ColumnByComma()?
Uniq
The default behaviour of the unix command uniq is, that it just returns all unique lines of the input. With the flag -c, the number of occurences are counted and returned together with the lines. This looks like this:
  5 foobar
12 barbaz
  4 foobaz

So if you want to have a "simple" API, you would need something like UniqWithCounts().
Sort
The default behaviour of the unix command sort is to sort alphabetically. If you want to sort an imput by numeric value, you need to pass the flag -n, which then looks for a number starting from the left side of the line and then sorts by these numbers. The flag -r reverses the order, such that the lines with the most hits are provided first in the output (because these lines are of the most interest).
So this would become something like SortByNumberReversed()
CountLines
This does not work, because I am not interested in the number of lines after sort and uniq.
Alternative Proposal
script.Stdin().Match("the log lines of interest").Column(5).UniqWithCounts().SortByNumberReversed().Stdout()

So this boils down to the question, what exactly is considered an "simple" API. I feel like removing the posibility for options/flags leads to a very broad (but still not very flexible) API. Alone the case for Sort would currently need:

Sort(): sort alphabetically
SortReversed(): sort alphabetically reversed
SortByNumber(): sort by leading numbers/integers
SortByNumberReversed(): sort by leading numbers/integers reversed

You can imagine your self, where this leads with more options added. The unix command sort provides some more options e.g. ignore case (but I think the case is preserved in the output), sort by human numbers (like 1K, 1G, etc.) and by month. I can also imagin, that sort by date could be interesting.
Bottom line
I personally prefer the API with the functional options. But this is your package, so you have to give the guidelines.

Very thoughtful contribution, thank you!
You make an excellent point: that if you try to consider every possible way someone might want to use your functionality, you end up with a very complicated API (one way or the other). Either you have to provide lots of variants of no-config methods, or methods with lots of config.
This is exactly why I'm being such a hardass about use cases. I've rejected a lot of my own feature proposals on the basis that they're just something I thought would be cool, without having an actual, real requirement for them in a production program.
It sounds like the use case you're proposing is something roughly like this:

Find the top 10 most common 'column 5' values from a set of input lines

(That's quite complex! What specific real-world thing is it you're trying to do here? It's always helpful to know the concrete details.)
Now, we don't necessarily have to do this in a one-liner. That's the nice thing about a Go library; we can just use Go for anything the library doesn't have built in. But if we were to do this entirely in a script pipe, we would need at least the following:

Cut by columns
Count frequencies of identical lines
Sort lines numerically
Get first N lines

It's worth noting that I'm not particularly interested in following the API of specific Unix commands, except where it's obvious enough that it doesn't matter if you're familiar with the Unix equivalent.)
So here's a counter-counter-proposal, for five specific features:

Column(col) selects a specified column number, delimited by whitespace
CountFreq() outputs the unique lines in its input, prepended by a frequency count and whitespace
SortNumeric() sorts its input lines by numeric order
Reverse() reverses the input lines
First(n) outputs only the first N lines of input

Here's how your use case might be implemented using these:
script.Stdin().Match("foo").Column(5).CountFreq().SortNumeric().Reverse().First(10).Stdout()

By the way, one thing I often do which is similar to this is to analyse webserver logs to find the IP address generating the most requests. For example, I cut the IP address out of access.log lines, and sort them by frequency to get the top 10 requesters.

PS: maybe CountFreq should default to sorting its results with the highest frequency first? Does any other default make sense? If not, this seems like a sensible idea.

My real world example I had in mind, when you asked me about a specific use case was pretty similar to the use case you outline in your above comment. In my case it was the log of an ssh daemon and I was interested in the source IP addresses from where the most failed login attempts have been made.
This use case I obviously solved with a bash one-liner. If I use a particular bash one-liner often, I put it in a script. Maybe later, some additional logic is added to process the logs in different ways. In the end I have a bash script, that would be better implemented in something like Go. So this is the way it normally takes in my case. Therefore I would prefere an API, that is similar to the unix commands I already use, because the translation process from the existing bash one-liner or even script to a Go program with this package would become easier.
That being said, your counter-counter proposal covers my use case pretty good. Based on your last comment, it could be simplified to:
script.Stdin().Match("foo").Column(5).CountFreq().First(10).Stdout()

where CountFreq() would count the frequency of identical lines and return the result in descending order of the frequencies, each unique line prefixed with the frequency and a whitespace (the command uniq takes care of proper indention. because we return the lines with the most frequent line first, this should be an easy thing to do here as well).

I would prefere an API, that is similar to the unix commands I already use, because the translation process from the existing bash one-liner or even script to a Go program with this package would become easier.

Yes, understood. By and large the Unix API is clear, straightforward, and unambiguous, and where that's so, I'm happy to adopt it. Some things or options are called by weird names, for historical reasons, and where there's an alternative that's clearer to people who don't write shell scripts, I'll use that instead. Clarity first, Unix familiarity second.
For example, Match() is called Match(), not Grep() (although that was naturally the first thing I thought of).

Based on your last comment, it could be simplified to:

script.Stdin().Match("foo").Column(5).CountFreq().First(10).Stdout()

This looks very good to me. Are you interested in working on implementation of any of these? I might start putting a PR together with a log-line counting example, and the code to make it work.

Hi, for a side note
The name CountFreq feels a little strange to me, I think what it does includes two part

Uniq ( uniq lines, so no same line show again )
Print frequency ( should be an option UniqCount or PrefixCount?  maybe turn on by default )

So I think CountFreq use the name Uniq with option feels more intuitive.

I think CountFreq use the name Uniq with option feels more intuitive.

I understand why you might feel that way, but here's another way of looking at it:
The primary thing that the method does is to count frequency. As a side effect of this it reduces the input to unique lines. But if you just wanted unique lines, you wouldn't use this method, because it prepends frequency counts.
I can quite see that one might want a method which produces unique lines, and that method should be called Unique(), certainly. So I don't want to lose that name by allocating it to something whose main purpose is not to produce unique lines.

Looks good to me now .

@breml have a look at #11 and see what you think. All the components for our visitor-counting example are now in place.

@breml thanks for all the help on this one! Do you have everything you want now in v0.9.0?

LGTM, thanks.


cross post editing

When two different posts are edited at the same time, if one person types in one post it (may) show up in the other post.

Screenshots: https://imgur.com/a/6qCYfou

Thats.... Very interesting lol

no clue, i dont know how it happened, it just happened at some point

afaik it always happens

And if you refresh it's gone?

@NULLx76 and me just retried it

yeah refreshing removes it

But it shouldn't even show up of course

Okay then I have a hunch as to where something is wrong

good

Closed by https://github.com/RobbinBaauw/CSHub/commit/b80d5b7b2460343cee044c1657d78abef7f73df4


Sometimes not loading css?

I have some kind of problem. People are reporting me that page sometimes loads without styles when it should be with.
Also im using apache2 + mod_pagespeed and nginx as proxy but static files are served by apache2(im using plesk panel so i cant compile nginx + pagespeed beacause plesk use their own)
Also i have multiple domains, maybe i have to configure them somehow special ?
My config:
http://pastebin.com/BSxB60z4 - prioritze critical css is disabled
how it sometimes looks like : http://imageshack.com/a/img673/6971/tWacV8.jpg
how it supposed to look : http://imageshack.com/a/img538/9783/6b83an.png

can you check with the chrome/firefox page inspector (f12) to see if some files are not getting loaded and what errors there are? Is it only happening on that admin page, or is it happening in another public place on the site where it would be possible to reproduce.

It just error 404 - it happens when it try first time to parse css, its pretty hard to reproduce beacause its totally random :/

Oh sorry, its not 404 -  its just fail with no any response. So my guess sometimes its just failing something. Are there are any logo for pagespeed.

Okay - it looks like pagespeed InPlaceResourceOptimization on; causing a problem, when i commented it it works fine.

Could you try this again with our new release, <IP_ADDRESS>?  We've a couple bugs that could have caused situations like this.

Well i installed it and still there are some bugs but i THINK I got whats causing a problem i have two styles first have only fonts and second have style of website, when combine_css is on its sometimes put fonts into combined style, and ALL STYLES from second file are just GONE.

Sending me your css files would be helpful.  Could you make a zip with your original non-PageSpeed html, your css files, your PageSpeed-rewritten html, and your PageSpeed rewritten css files, and send it to<EMAIL_ADDRESS>

Alternatively, you could send me a link to your site.


openxr: Clean Vulkan virtual swapchain on destroy

Whether the OpenXR swapchain is explicitly or implicitly destroyed.

CI gfxreconstruct build queued with queue ID 298145.

CI gfxreconstruct build # 5323 running.

CI gfxreconstruct build # 5323 passed.


update DISABLE_SPEEDY to use webpack environment variable

When the Webpack config variable SC_DISABLE_SPEEDY is set to false it will not be used currently, this fixes that to allow this to be disabled even in development.

Not sure why the build is failing, could someone give me some pointers here?  This is the log
https://travis-ci.org/styled-components/styled-components/jobs/519311928
W: GPG error: http://dl.google.com/linux/chrome/deb stable Release: The following signatures were invalid: KEYEXPIRED<PHONE_NUMBER>  KEYEXPIRED<PHONE_NUMBER>  KEYEXPIRED<PHONE_NUMBER>  KEYEXPIRED<PHONE_NUMBER>
W: The repository 'http://dl.google.com/linux/chrome/deb stable Release' is not signed.
W: GPG error: https://packagecloud.io/github/git-lfs/ubuntu trusty InRelease: The following signatures couldn't be verified because the public key is not available: NO_PUBKEY 6B05F25D762E3157
W: The repository 'https://packagecloud.io/github/git-lfs/ubuntu trusty InRelease' is not signed.
W: There is no public key available for the following key IDs:
6B05F25D762E3157
W: http://ppa.launchpad.net/couchdb/stable/ubuntu/dists/trusty/Release.gpg: Signature by key 15866BAFD9BCC4F3C1E0DFC7D69548E1C17EAB57 uses weak digest algorithm (SHA1)
The command "sudo -E apt-get -yq --no-install-suggests --no-install-recommends $(travis_apt_get_options) install libnss3 google-chrome-beta" failed and exited with 100 during .

@devrelm @probablyup would you mind having a look?

I have the same problem. Please review this PR


Integrate healthcheck into dashboard + improve coverage

This integrates proper healthcheck for the dashboard, thus enabling monitoring for the dashboard application. This also restructures the dashboard codebase to comply with Abacus coding strategies, thereby improving code coverage.
Fixes #27
Fixes #23

Coverage increased (+0.6%) to 87.836% when pulling 5a38f3acac996e4855fed35d9427fbdfbd822009 on abh1kg:abacus-healthcheck into ac69bf1f80e7df85f75bbf2a0e54c59a9cb8c138 on cloudfoundry-incubator:master.

Coverage increased (+1.5%) to 88.751% when pulling 26e2152cfe14b1fe7069fd3662bd7356a3838154 on abh1kg:abacus-healthcheck into ac69bf1f80e7df85f75bbf2a0e54c59a9cb8c138 on cloudfoundry-incubator:master.


No conversion actions in the displayed lightbulb

While trying to convert my classes to hooks, I cannot find options of the extension displayed in the light bulb
Please find code which i am trying to convert: https://gist.github.com/RamyEl-basyouni/804880469617e68557fbcba2142419db
And the the gif indicating the steps of producing issue:
https://giphy.com/gifs/RhSeDbDzInlaE3uHGC/fullscreen
Info:
OS: 10.15.4
VSCode Version: 1.46.1
Glean Version: 5.0.1

Hi Guys,
Any updates regarding this?

I can't see this option too

Same, no updates?

Same, no updates?

same here, not working, any context menu on lightbulb or right click.

+1

+1


Supported?

is this no longer supported?

Although not actively developed, this is still supported. If you have any questions please feel free to ask.

@licson0729 : I have tried this on local and it is working fine but throwing 503 request timeout on heroku/GCP.

@licson0729 : I have tried this on local and it is working fine but throwing 503 request timeout on heroku/GCP.

PaaS platform like Heroku may have request buffering and/or script execution time limits that prevented long HTTP requests to finish which is a


Support for Apple Silicon M2?

Is your feature request related to a problem? Please describe.
There is no support on Macbooks M2 Apple Silicon CPUs.
Describe the solution you'd like
An ISO file tailored for Apple Silicon machines.

Additional context
By having helloSystem available on modern Macs, it would be very useful for using high level security software.

https://wiki.freebsd.org/AppleSilicon

Thanks for your suggestion.
Depends on

https://github.com/helloSystem/hello/issues/190

Once there are not many red "X" anymore on https://wiki.freebsd.org/AppleSilicon, we can consider it. It'll be probably a few years. Let's reopen then.


Fix border flickering on navigation

Closes #2257
@xraystyle1980 white border flickering was caused by transition and unsetting border/border color. Please check if behavior looks good now

Closes #2257
@xraystyle1980 white border flickering was caused by transition and unsetting border/border color. Please check if behavior looks good now

Looks good 👍


dynatrace_aws_credentials without metrics configured resulting in error of having metrics configured.

Describe the bug
As a result of https://github.com/dynatrace-oss/terraform-provider-dynatrace/issues/364 we have switched to just using the provider to manage the integration and not the configured services/metrics. We first ran an apply with no metrics configured to remove them. After this we noticed that some accounts didn't have all services removed, some still have auto-scaling left. After that we changed the value of supporting_services_managed_in_dynatrace to true and when attempting to run that with no metrics configured at all we got an error saying that we can't configure the auto-scaling service using this endpoint.

We were running this against 57 different AWS accounts and we noticed that it was seemingly random which ones ended up with auto-scaling configured which makes it hard to reproduce.
To Reproduce
Steps to reproduce the behavior:

First configure an AWS account using the dynatrace_aws_credentials resource. On this account, configure 71+ supporting services and their metrics. Use the dynatrace_aws_credentials resource for the metrics/services too.
Remove the metrics/services and apply the new configuration.
Change the value of supporting_services_managed_in_dynatrace to true and apply again.

Expected behavior
With no metrics/services configured the provider should not be attempting to configure any.
Desktop (please complete the following information):
OS: Windows 11
Version: 1.46.1

Hello @FinUD,
Sorry to hear that you're still facing issues with AWS Services. Root cause for your continuing troubles are our attempts to keep the resource dynatrace_aws_credentials 100% backwards compatible for possible users who are still maintaining their services via this resources instead of dynatrace_aws_service.
We've now made the decision that we will remove the ability to maintain services using dynatrace_aws_credentials. The block supporting_services_to_monitor will get deprecated. The attribute supporting_services_managed_in_dynatrace won't have an effect. The attribute remove_defaults will remain, in order to enforce removing any default services upon create (won't have an effect for updates).
It's the only way we can avoid issues like the ones you've run into in the past.
We're planning to push out a release next week - it will include these changes.

Hey @Dynatrace-Reinhard-Pilz, thanks for the update as always. Does this mean if you don't specify any dynatrace_aws_service resource blocks and only dynatrace_aws_credentials that it will remove any manually configured/API configured resources when you apply?

Our plan is to change the resource like this:
The resource dynatrace_aws_credentials will ignore the services completely. Any services you're adding or removing via WebUI will be remain the same. Also, any services you're adding via resource dynatrace_aws_service will get ignored. The resource dynatrace_aws_credentials will simply not address them.
Just with one exception:
When Terraform CREATES an instance of dynatrace_aws_credentials, the target Dynatrace Environment usually creates a default set of services for these credentials.
If you set the attribute remove_defaults to true, you instruct Terraform to get rid of these services right after creation. Once that has happened remove_default won't have any further effect. Any subsequent updates for that instance of dynatrace_aws_credentials will ignore the services and will not attempt to modify the services.


Add Codecov Coverage

Current coverage is 55.00%

Branch #27 has no coverage reports uploaded yet.

No diff could be generated. No reports for master found.

Powered by Codecov. Updated on successful CI builds.


http://...special/index more important than it should be

http://aquarius.tw.rpi.edu/datafaqs-view/special/index
lists classes, but the URL doesn't indicate that at all.
redirect to the "plain old" URL for the class lister?

From version 20120121 installation provides an 'index.html' page that include links to:

special/classes (previously special/index)
special/namedGraphs

Check the new version, if it is OK please, close the ticket

http://aquarius.tw.rpi.edu/datafaqs/DataFAQs-lodspeakr/index looks much more indexy -- and doens't have the "special" which is a bonus.


Implemented tags function for grpc stats handler

Coverage decreased (-0.8%) to 97.194% when pulling 5ad6df1d7e7a205b6f4ddd89f11351666da0b28a on request-stats into 2bf92247da940a5b550d88733a9fcd4001e56ab3 on master.

Coverage decreased (-0.8%) to 97.194% when pulling 5ad6df1d7e7a205b6f4ddd89f11351666da0b28a on request-stats into 2bf92247da940a5b550d88733a9fcd4001e56ab3 on master.

There is a small BC break here: I have removed the rpc begin stat as it had little-to-no use and it was impossible to apply the tags from the tags func to it. If the reviewer decides it should be brought back, there is no problem with that, it will, however, lack the customized tags.


Fix maxFeePerGas bug in example app

Description
Fixes maxFeePerGas bug in example app by increasing from 0.0007 gwei to 5 gwei (5e9 wei).
The bug was:
fee-currency-2d720879575e6073.js:1 EstimateGasExecutionError: The fee cap (`maxFeePerGas` = 0.0007 gwei) cannot be lower than the block base fee.

Estimate Gas Arguments:
  from:                  0xcf941f3e6a147Eb62c0f978CADCeD28e30F0D446
  to:                    0x22579CA45eE22E2E16dDF72D955D6cf4c767B0eF
  value:                 0.1 A-CELO
  maxFeePerGas:          0.0007 gwei
  maxPriorityFeePerGas:  0.0007 gwei

Details: err: max fee per gas less than block base fee: address 0xcf941f3e6a147Eb62c0f978CADCeD28e30F0D446, maxFeePerGas: 700000 baseFee:<PHONE_NUMBER> (supplied gas 5010499)
Version<EMAIL_ADDRESS>    at _app-934dd97cff7957aa.js:1:661115
    at b (_app-934dd97cff7957aa.js:1:661142)
    at async fee-currency-2d720879575e6073.js:1:3812

Other changes
N/A
Tested
Ran app locally with yarn run dev
Related issues
N/A
Backwards compatibility
Yes, should work as before.
Documentation
N/A

(for future reference)
The release workflow (that failed) is used to publish the NPM package.
Since this PR only modified the example app, no NPM publishing was needed.
The Vercel deployment worked as expected.
Fixing https://github.com/celo-org/rainbowkit-celo/issues/106 was important to ensure future NPM releases work as expected, but didn't unblcoked the Vercel example app fix, since that was never blocked in the first place.


graylog: improve JRE handling

Motivation for this change
Since the upstream graylogctl script will prefer finding its java
executable based on JAVA_HOME, we now set this instead of PATH in
order to allow it to find the JRE. By setting it conditionally on it
not already being set, we allow selecting a different JRE at runtime.
Things done

[x] Tested nixos/tests/graylog.nix
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 117809 at 52a82e38 run on x86_64-linux 1

1 package built successfully:

graylog

2 suggestions:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/tools/misc/graylog/default.nix:20:3:
   |
20 |   installPhase = ''
   |   ^

warning: unclear-gpl
gpl3 is a deprecated license, please check if project uses gpl3Plus or gpl3Only and change meta.license accordingly.
Near pkgs/tools/misc/graylog/default.nix:29:5:
   |
29 |     license     = licenses.gpl3;
   |     ^

@lheckemann tests fail in ofborg

And it's not transient:
machine # [   21.094467] graylogctl[999]: which: no java in (/nix/store/b86fbamwswj596c1md5f0sh0a5wsbz8x-which-2.21/bin:/nix/store/yyw2db5czblg7k86py43x0628w157p8x-procps-3.3.16/bin:/nix/store/15gmaah0dihis48xpr4nlm50giwlv403-libressl-3.2.5-nc/bin:/nix/store/9fkdn82nzxgkxyax3m9qdh2knj7apaa5-coreutils-8.32/bin:/nix/store/7nx8fqzpi5nsbi07md1qb1gbf9fscf9i-findutils-4.7.0/bin:/nix/store/ia1dn8zyi07phxkx02c0wwqc7v7c54nl-gnugrep-3.6/bin:/nix/store/ij7k48dhdrda13gcp3lbpi507b2awnmr-gnused-4.8/bin:/nix/store/caz5k18ar764161b0amywc002cp39lgx-systemd-247.3/bin:/nix/store/b86fbamwswj596c1md5f0sh0a5wsbz8x-which-2.21/sbin:/nix/store/yyw2db5czblg7k86py43x0628w157p8x-procps-3.3.16/sbin:/nix/store/15gmaah0dihis48xpr4nlm50giwlv403-libressl-3.2.5-nc/sbin:/nix/store/9fkdn82nzxgkxyax3m9qdh2knj7apaa5-coreutils-8.32/sbin:/nix/store/7nx8fqzpi5nsbi07md1qb1gbf9fscf9i-findutils-4.7.0/sbin:/nix/store/ia1dn8zyi07phxkx02c0wwqc7v7c54nl-gnugrep-3.6/sbin:/nix/store/ij7k48dhdrda13gcp3lbpi507b2awnmr-gnused-4.8/sbin:/nix/store/caz5k18ar764161b0amywc002cp39lgx-systemd-247.3/sbin)
machine: waiting for TCP port 9000
machine # [   21.532366] graylogctl[995]: Running graylog-server ...
machine # [   21.540196] graylogctl[995]: Unrecognized VM option 'UseConcMarkSweepGC'
machine # [   21.541386] graylogctl[995]: Error: Could not create the Java Virtual Machine.
machine # [   21.542246] graylogctl[995]: Error: A fatal exception has occurred. Program will exit.
machine # [   21.545431] systemd[1]: graylog.service: Main process exited, code=exited, status=1/FAILURE
machine # [   21.546462] systemd[1]: graylog.service: Failed with result 'exit-code'.
machine # [   21.547379] systemd[1]: graylog.service: Consumed 222ms CPU time, no IO, received 1.0K IP traffic, sent 1.4K IP traffic.


npm install hangs

Hi all...
I've been able to download the code, when I try to the step "npm install" it kicks off and runs, but then it hangs on this... I've blown away the entire project, did a fresh git, rebooted my machine... But, it always hangs right here.  Unfortunately, I am VERY new to npm.  Just watched my first course on it in LinkedIn about 2 hours ago... So... I am lost.
[            ......] \ extract:@schematics/angular: sill extract<EMAIL_ADDRESS>extracted to G:\Web Workspace\Training\LinkedIn\Angular Essential Training\angular-essential-training\node_modules\.staging

Well.   False alarm. I guess it just took a LOOOONG time.  After a long phone call, i came back to my computer and it was finished.


compile flutter failed

ninja: Entering directory `out/android_debug_x86'
[3533/3574] ACTION //flutter/lib/snapshot:generate_snapshot_bin(//build/toolchain/android:clang_x86)
FAILED: gen/flutter/lib/snapshot/vm_isolate_snapshot.bin gen/flutter/lib/snapshot/vm_snapshot_instructions.bin gen/flutter/lib/snapshot/isolate_snapshot.bin gen/flutter/lib/snapshot/isolate_snapshot_instructions.bin
python ../../build/gn_run_binary.py clang_x86/gen_snapshot --snapshot_kind=core --enable_mirrors=false --vm_snapshot_data=/home/wuchuanping/flutter/src/out/android_debug_x86/gen/flutter/lib/snapshot/vm_isolate_snapshot.bin --vm_snapshot_instructions=/home/wuchuanping/flutter/src/out/android_debug_x86/gen/flutter/lib/snapshot/vm_snapshot_instructions.bin --isolate_snapshot_data=/home/wuchuanping/flutter/src/out/android_debug_x86/gen/flutter/lib/snapshot/isolate_snapshot.bin --isolate_snapshot_instructions=/home/wuchuanping/flutter/src/out/android_debug_x86/gen/flutter/lib/snapshot/isolate_snapshot_instructions.bin /home/wuchuanping/flutter/src/out/android_debug_x86/flutter_patched_sdk/platform_strong.dill
Traceback (most recent call last):
File "../../build/gn_run_binary.py", line 22, in
sys.exit(subprocess.call(args))
File "/usr/lib/python2.7/subprocess.py", line 523, in call
return Popen(*popenargs, **kwargs).wait()
File "/usr/lib/python2.7/subprocess.py", line 711, in init
errread, errwrite)
File "/usr/lib/python2.7/subprocess.py", line 1343, in _execute_child
raise child_exception
OSError: [Errno 8] Exec format error
ninja: build stopped: subcommand failed.
compile the flutter engine failed with ubuntu 16.04 in window10, ubuntu install from the MS Store.  anyone can help me?

./flutter/tools/gn --android --unoptimized
ninja -C out/android_debug_unopt

Any resolution for this??

no solutions   Any idear
| |
吴川平
<EMAIL_ADDRESS>|
签名由 网易邮箱大师 定制
On 06/22/2019 19:09, Krishna Keshan wrote:
Any resolution for this??
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


ConPTY should less aggressively reprint the screen

Tasks in VS Code listens to terminal output to detect diagnostics (errors/warnings) and when watch tasks start and end. It also writes to the terminal itself some status messages such as:
> Executing task: echo %FOO% <

Because of the way ConPTY reprints the screen frequently, this causes problems with this though as it will replace the terminal-side messages like the "Executing task" message and will also report diagnostics multiple times (which I think we de-dupe but could potentially cause issues).
It would be better if ConPTY kept track of when only new output was printed at the cursor and only reprinted when absolutely necessary like on a resize. This would bring it much closer in line to how winpty and nix ptys work.
VS Code issue: https://github.com/microsoft/vscode/issues/74063

This should also improve performance as it would avoid clearing out all the rows which happens a lot right now.

Do we have a particular repro case where the screen is repainted when we don't expect it to be?
For the record, conpty already only repaints the regions it believes have changed in the course of a frame. Most of the code is in this directory - with invalidate.cpp and paint.cpp probably being the most interesting. The Renderer in the sibling directory base acts as the controller for painting only the regions the VtEngine (which powers conpty) thinks are invalid.
There was some bug in either RS5 or in-between RS5 and 19H1 where we'd incorrectly repaint the screen whenever the active attributes were changed (which is pretty constantly). I believe that's fixed now, so this shouldn't be as bad as before, but it could always be better. Concrete scenarios would certainly make this easier to drill into.
In the linked scenario, it almost seems like the task is spawning a new conpty instance to run the task, and then writing that output to the parent, is that correct? If that's the case then maybe the child task needs to inherit the cursor position of the parent terminal (using dwFlags= PSEUDOCONSOLE_INHERIT_CURSOR

@zadjii-msft the flow in https://github.com/microsoft/vscode/issues/74063 is roughly:

Instantiate xterm.js
Write "> Executing task: echo %FOO% <" to xterm.js
Launch the process via conpty and hook it up to xterm.js
Conpty reprints the screen right after process launch

@alexr00 what's your Windows 10 build number?

Yea that sounds exactly like the kind of situation where INHERIT_CURSOR is needed.
Since conpty has to maintain the state of a console buffer, by default it assumes that the buffer is empty. However, with INHERIT_CURSOR, conpty asks the terminal on startup where the cursor currently is. When the terminal replies, conpty uses that as the initial cursor position, and won't render any text above that point. It'll use that y-coordinate as a "virtual top" which is inaccessible. As the text scrolls, we'll move that virtualtop up, until the entire buffer is addressable again.
I believe WSL and ssh.exe both use this flag, to make console sessions more continuous. Otherwise, as you've noted, we'll start by clearing the screen, which is only really desirable if you know you're starting from a fresh terminal window.

What's the problem you're trying to fix by not assuming a cursor position of 1,1 when conpty starts?

We are assuming a cursor position of 1,1 when ConPTY starts. PSEUDOCONSOLE_INHERIT_CURSOR overrides that behaviour.
The reason we by default assume a cursor position of 1,1 is that Win32 console applications are more likely than Unix VT applications to try to read back the screen buffer contents. They can figure out where the cursor is without asking for a cursor report, and they can request that regions of console screen buffer memory be copied back into their processes. They can also write from their own buffers directly into the console buffer at an absolute (not viewport-relative like VT!) location.
To support these APIs from a pseudoconsole, we had to do two things.
First and foremost, we can't lie to a Win32 console application about where the cursor is, or where any output has been drawn. That would force us to pile up hacks to adjust the location of every buffer- or cursor position-related call. So, we assume the cursor is at 1,1 unless we have explicit knowledge from the hosting terminal as to where the cursor should be.
Because there is no "API" for requesting the cursor position apart from requesting DSR and waiting for a response, INHERIT_CURSOR requires that the connecting terminal provide DSR before it'll continue. Again, this is required for API consistency.
The second thing we do to ensure that the win32 console and its VT representation are in sync (that is: a console application reading a buffer out of the console won't see blank space where the terminal says there's text, and vice-versa), we clear the screen. This only applies when we home the cursor to 1,1 on startup.

Thanks it sounds like we should just default to PSEUDOCONSOLE_INHERIT_CURSOR and the repainting issue sounds like a bug that's already fixed.


Added support for setting tolerance for doubles in mock

Added overloaded methods for withParameter and withDoubleParameter which takes a tolerance as well as the expected value for doubles

Coverage increased (+0.0003%) to 99.879% when pulling 0128134e0068621ba4608e26e7eca13892325567 on oyvindlr:support-double-tolerance-for-mock into 8f9a0130e41cb3d5100e5f05fab7a3d35465c2f9 on cpputest:master.

Thanks. Looks good. Just one minor issue that could be improved.


💥  NICE-21 node@18; semantic-release upgrades

This is going to encapsulate a few updates in one.

[x] Support Node 18+, it's a requirement for semantic-release, will bump accordingly across ahead of Node16 EOL
[x] Drop @jeromefitz/design-system from pnpm workspace for the short-term; moving back to tailwind, for now hold on any development
[x] Package upgrades to prepare for Semantic Release ESM upgrades
[x]<EMAIL_ADDRESS>-- with design-system drop, this now works, heh. Is a known error that we will have to side-step at some point until it is fixed intsup (#1021)
[ ] Code Updates and Verification of Semantic release upgrades

BREAKING CHANGE: Only support Node18+

This whole thing is going to have me chuck semantic-release altogether. I get this is difficult getting people to move in-sync. But this is absurd the amount of hoops.


prepare release 1.6.2

Codecov Report

Merging #472 into 1.6 will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              1.6     #472   +/-   ##
=======================================
  Coverage   34.58%   34.58%
=======================================
  Files          20       20
  Lines        1923     1923
  Branches      226      226
=======================================
  Hits          665      665
  Misses       1208     1208
  Partials       50       50

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 3676bec...a186bda. Read the comment docs.


docs(migration): change _ to - in file and directory

add aliases to front matter
also took care of {% raw %} in code blocks in using-dinghy.md

I tested this one on the preview build, and it's not working: https://deploy-preview-29--docs-hugo.netlify.app/docs/admin-guides/advanced-configuration/
advanced-configuration to advanced_configuration

OH wait n/m dumb user error on my part. The old url wouldn't have /docs/ in it. Duh.


LazyMapsAPILoader example

Can someone post an example on how to use LazyMapsAPILoader?

What do you want to achieve?


Fix assignment of references

Fixes #269 and allows to fix #234

Assignments should work fine now.

Thanks! Could you please put in some comments describing the new code?

OK.

Done. If there are unclear moments, let us review them.

Thanks. I have merged your PR. I made the makeReference function non-static as I didn't really see a reason for it to be static in the first place.
I also renamed some variables and added some more comments for clarity. I also moved curly braces to their own line for consistency.

I made the makeReference function non-static as I didn't really see a reason for it to be static in the first place

Well, it was supposed to be used like this:
Php::Value variable = 1;
Php::Value reference = Php::Value::makeReference(variable);

That is, it acts just like a constructor, it creates a new Value which is set as a reference to another Value.

I must say your implementation of makeReference() looks better.


⚠ Bofrost.de has degraded performance

In d9aefd3, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1126 ms

Resolved: Bofrost.de performance has improved in 4a3499c after 6 minutes.


permission button doesn't work for locationWhileInUse

I'm not sure why, all other Permissions work just fine, but not for locatioWhileInUse
I have used this key in my info.list
NSLocationAlwaysAndWhenInUseUsageDescription
I tried all the other three, but same problem
in podFile
platform :ios, '9.0'
target 'weekend' do
use_frameworks!

pod 'Sparrow/Modules/RequestPermission', :git => 'https://github.com/IvanVorobei/Sparrow.git'
xcode 9.3 swift 4.3 iPhone 7 Plus, iOS 11.3
thanks

I have tried these too, the problem with them is as soon as I start the app it ask for location permission. any idea why ?

because you are request it
create empty project and test


Indexing for episode number

When episode number in title is enabled, only the following cases would the number show

Episode list under player when user is playing in the context of show
Title under the player where episode is being played in the context of show

Background:
Its confusing for users when the following case have indexing that is according to its episode number

When a specific episode is added as a feature banner, indexing is also not necessary

Should I make Episode.number nullable and just set it null if it's not supposed to be visible @andreasgangso ?

@fredrikved don't you already have dynamic titles, so this is just about adding another condition?
also @mulundapm aren't we doing some similar "title modification" with production dates?

But determining when the title has a number and when it shouldn't isn't very easy from my side when it's page-based.

Me and fredrik talked and concluded on adding an extra checkbox on admin for collections: show episode number in title.
So you now have 2 places to configure this:

Show level: Episodes shown in the context of a show (e.g. on episode page)
Collection level: Episodes viewed in the context of a collection (e.g in a section or episode page navigated via section).

It's a tradeoff which creates a little bit more admin work, but it simplifies the backend logic a lot and gives a bit more flexibility (you can have episode numbers even though the show default is to not have that).

It was a thing, still am but the definition is not defined yet since we have various cases. Can work on defining it this week.

Season and collection yes.

@fredrikved "to be tested" 👍


fixed delete view

Please test it before merging, to make sure we both are on the same page, and this is what is the intended behavior. @ZihengSun
Thanks!

It seems the issue is still there, please double check

I have made a few changes. Please let me know when you are free.
Thanks!

I think you should make changes to the delete button in I first. The delete button in Weaver toolbar was fine (it is alright to see a clean workspace after erase the processes).

So the delete button outside does not clear the workspace. It just sets nodes to empty list in UI and NOT in database, so when you open it again everything comes back. So I made the changes in DB as well to makes nodes and edges to empty array


backscrub: init at 0.3.0 (WIP)

Motivation for this change
A virtual background program for linux
Things done
Building fails with
nix log /nix/store/k4yl5pz54qk1bq2k1xpn0w029pa78f8n-backscrub-0.3.0.drv
@nix { "action": "setPhase", "phase": "unpackPhase" }
unpacking sources
unpacking source archive /nix/store/q4cc18jiamzz50i98anhl90hwsyrb4h3-source
source root is source
@nix { "action": "setPhase", "phase": "patchPhase" }
patching sources
/build/source/tensorflow /build/source
patching file tensorflow/lite/tools/make/Makefile
Hunk #1 succeeded at 139 with fuzz 1 (offset -1 lines).
patching file tensorflow/lite/tools/make/Makefile
Hunk #1 succeeded at 138 with fuzz 2 (offset -1 lines).
patching file tensorflow/lite/tools/make/Makefile
/build/source
@nix { "action": "setPhase", "phase": "configurePhase" }
configuring
fixing cmake files...
cmake flags: -DCMAKE_FIND_USE_SYSTEM_PACKAGE_REGISTRY=OFF -DCMAKE_FIND_USE_PACKAGE_REGISTRY=OFF -DCMAKE_EXPORT_NO_PACKAGE_REGISTRY=ON -DCMAKE_BUILD_TYPE=Release -DCMAKE_SKIP_BUILD_RPATH=ON -DBUILD_TESTING=OFF -DCMAKE_INSTALL_LOCALEDIR=/nix/store/lv5ds11qy0himqxy83i2pdvrm1cfdmlx-backscrub-0.3.0/share/locale -DCMAKE_INSTALL_LIBEXECDIR=/nix/store/lv5ds11qy0himqxy83i2pdvrm1cfdmlx-backscrub-0.3.0/libexec -DCMAKE_INSTALL_LIBDIR=/nix/store/lv5ds11qy0himqxy83i2pdvrm1cfdmlx-backscrub-0.3.0/lib -DCMAKE_INSTALL_DOCDIR=/nix/store/lv5ds11qy0himqxy83i2pdvrm1cfdmlx-backscrub-0.3.0/share/doc/backscrub -DCMAKE_INSTALL_INFODIR=/nix/store/lv5ds11qy0himqxy83i2pdvrm1cfdmlx-backscrub-0.3.0/share/info -DCMAKE_INSTALL_MANDIR=/nix/store/lv5ds11qy0himqxy83i2pdvrm1cfdmlx-backscrub-0.3.0/share/man -DCMAKE_INSTALL_OLDINCLUDEDIR=/nix/store/lv5ds11qy0himqxy83i2pdvrm1cfdmlx-backscrub-0.3.0/include -DCMAKE_INSTALL_INCLUDEDIR=/nix/store/lv5ds11qy0himqxy83i2pdvrm1cfdmlx-backscrub-0.3.0/include -DCMAKE_INSTALL_SBINDIR=/nix/store/lv5ds11qy0himqxy83i2pdvrm1cfdmlx-backscrub-0.3.0/sbin -DCMAKE_INSTALL_BINDIR=/nix/store/lv5ds11qy0himqxy83i2pdvrm1cfdmlx-backscrub-0.3.0/bin -DCMAKE_INSTALL_NAME_DIR=/nix/store/lv5ds11qy0himqxy83i2pdvrm1cfdmlx-backscrub-0.3.0/lib -DCMAKE_POLICY_DEFAULT_CMP0025=NEW -DCMAKE_OSX_SYSROOT= -DCMAKE_FIND_FRAMEWORK=LAST -DCMAKE_STRIP=/nix/store/hy3lz2vfv9qq2v5jz9nzlx6mmiaq79rj-binutils-2.35.1/bin/strip -DCMAKE_RANLIB=/nix/store/hy3lz2vfv9qq2v5jz9nzlx6mmiaq79rj-binutils-2.35.1/bin/ranlib -DCMAKE_AR=/nix/store/hy3lz2vfv9qq2v5jz9nzlx6mmiaq79rj-binutils-2.35.1/bin/ar -DCMAKE_C_COMPILER=gcc -DCMAKE_CXX_COMPILER=g++ -DCMAKE_INSTALL_PREFIX=/build/source/tensorflow/tensorflow/lite/tools/make/downloads
-- The CXX compiler identification is GNU 10.3.0
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /nix/store/qvv5y4fx4x879rbsbs4g27mypl9wxbb9-gcc-wrapper-10.3.0/bin/g++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Could NOT find Git (missing: GIT_EXECUTABLE)
Version: v0.2.0-no-git
-- Found OpenCV: / (found version "4.5.2") found components: core imgproc imgcodecs video videoio highgui
-- The C compiler identification is GNU 10.3.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /nix/store/qvv5y4fx4x879rbsbs4g27mypl9wxbb9-gcc-wrapper-10.3.0/bin/gcc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
CMake Error at /nix/store/p1lk3bb1wzyn5v3nw8nyxrc44iy60vwk-cmake-3.19.7/share/cmake-3.19/Modules/ExternalProject.cmake:2542 (message):
  error: could not find git for clone of abseil-cpp-populate
Call Stack (most recent call first):
  /nix/store/p1lk3bb1wzyn5v3nw8nyxrc44iy60vwk-cmake-3.19.7/share/cmake-3.19/Modules/ExternalProject.cmake:3430 (_ep_add_download_command)
  CMakeLists.txt:13 (ExternalProject_Add)

-- Configuring incomplete, errors occurred!
See also "/build/source/build/_deps/abseil-cpp-subbuild/CMakeFiles/CMakeOutput.log".

CMake Error at /nix/store/p1lk3bb1wzyn5v3nw8nyxrc44iy60vwk-cmake-3.19.7/share/cmake-3.19/Modules/FetchContent.cmake:977 (message):
  CMake step for abseil-cpp failed: 1
Call Stack (most recent call first):
  /nix/store/p1lk3bb1wzyn5v3nw8nyxrc44iy60vwk-cmake-3.19.7/share/cmake-3.19/Modules/FetchContent.cmake:1118:EVAL:2 (__FetchContent_directPopulate)
  /nix/store/p1lk3bb1wzyn5v3nw8nyxrc44iy60vwk-cmake-3.19.7/share/cmake-3.19/Modules/FetchContent.cmake:1118 (cmake_language)
  tensorflow/tensorflow/lite/tools/cmake/modules/OverridableFetchContent.cmake:531 (FetchContent_Populate)
  tensorflow/tensorflow/lite/tools/cmake/modules/abseil-cpp.cmake:34 (OverridableFetchContent_Populate)
  tensorflow/tensorflow/lite/tools/cmake/modules/absl-config.cmake:18 (include)
  tensorflow/tensorflow/lite/CMakeLists.txt:128 (find_package)

-- Configuring incomplete, errors occurred!
See also "/build/source/build/CMakeFiles/CMakeOutput.log".

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/backscrub-depends-on-tensorflow-lite-sources-cmake-does-not-help/15809/1

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-in-distress/3604/47