dvtm: 0.15 -> master

Motivation for this change
While the project is still seeing commits coming in, there has not been a release in a while.
Things done

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

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

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

dvtm releases are uncommon because the common use-case is to compile from source and apply patches as configuration.

No. Would you prefer I inlined the patch?

The latest release doesn't work with Darwin (except with the patch I provided last time), and has dirty buffer issues in some cases. I could rejig this and submit a larger patch based of the previous release if that would be more palatable? I just thought that this was more honest...

I think Darwin support is usually a good enough reason to add an extra version (if there is no stable source URL for that extra patch yet, I guess inlining is a better solution), or to add a small-ish amount of small-ish patches.
As for complete replacement, I want to know @vrthra opinion as the maintainer.
It looks like by the previous upstream standards, there should have been a couple of releases since the last one, so maybe after merging PR-86 you could ask about that…

@GrahamcOfBorg build dvtm

Success on x86_64-darwin (full log)
Attempted: dvtm
Partial log (click to expand)
installing terminfo description
2 entries written to /nix/store/2zwzzs2h16yr8h75kqdsvx5gwpgxnfnm-dvtm-master/share/terminfo
post-installation fixup
gzipping man pages under /nix/store/2zwzzs2h16yr8h75kqdsvx5gwpgxnfnm-dvtm-master/share/man/
strip is /nix/store/g5r4apl0za012ffs6ladinwa5w0m1l3k-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/2zwzzs2h16yr8h75kqdsvx5gwpgxnfnm-dvtm-master/bin
patching script interpreter paths in /nix/store/2zwzzs2h16yr8h75kqdsvx5gwpgxnfnm-dvtm-master
/nix/store/2zwzzs2h16yr8h75kqdsvx5gwpgxnfnm-dvtm-master/bin/dvtm-pager: interpreter directive changed from "/bin/sh" to "/nix/store/n9hba031gjky8hpjgx9fnlaxhidyzxbz-bash-4.4-p23/bin/sh"
/nix/store/2zwzzs2h16yr8h75kqdsvx5gwpgxnfnm-dvtm-master/bin/dvtm-status: interpreter directive changed from "/bin/sh" to "/nix/store/n9hba031gjky8hpjgx9fnlaxhidyzxbz-bash-4.4-p23/bin/sh"
/nix/store/2zwzzs2h16yr8h75kqdsvx5gwpgxnfnm-dvtm-master

Success on aarch64-linux (full log)
Attempted: dvtm
Partial log (click to expand)
shrinking /nix/store/xj4aqx08nchgakgqwv6pkfn7azx78dvi-dvtm-master/bin/dvtm-editor
shrinking /nix/store/xj4aqx08nchgakgqwv6pkfn7azx78dvi-dvtm-master/bin/dvtm
gzipping man pages under /nix/store/xj4aqx08nchgakgqwv6pkfn7azx78dvi-dvtm-master/share/man/
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/xj4aqx08nchgakgqwv6pkfn7azx78dvi-dvtm-master/bin
patching script interpreter paths in /nix/store/xj4aqx08nchgakgqwv6pkfn7azx78dvi-dvtm-master
/nix/store/xj4aqx08nchgakgqwv6pkfn7azx78dvi-dvtm-master/bin/dvtm-pager: interpreter directive changed from "/bin/sh" to "/nix/store/dsyc1z7ck08ga7l0b1jcxx35wj69qcii-bash-4.4-p23/bin/sh"
/nix/store/xj4aqx08nchgakgqwv6pkfn7azx78dvi-dvtm-master/bin/dvtm-status: interpreter directive changed from "/bin/sh" to "/nix/store/dsyc1z7ck08ga7l0b1jcxx35wj69qcii-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/xj4aqx08nchgakgqwv6pkfn7azx78dvi-dvtm-master...
/nix/store/xj4aqx08nchgakgqwv6pkfn7azx78dvi-dvtm-master

Success on x86_64-linux (full log)
Attempted: dvtm
Partial log (click to expand)
shrinking /nix/store/sv2fy18478wf56vzvyq6c20cjqpx5k1j-dvtm-master/bin/dvtm-editor
shrinking /nix/store/sv2fy18478wf56vzvyq6c20cjqpx5k1j-dvtm-master/bin/dvtm
gzipping man pages under /nix/store/sv2fy18478wf56vzvyq6c20cjqpx5k1j-dvtm-master/share/man/
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/sv2fy18478wf56vzvyq6c20cjqpx5k1j-dvtm-master/bin
patching script interpreter paths in /nix/store/sv2fy18478wf56vzvyq6c20cjqpx5k1j-dvtm-master
/nix/store/sv2fy18478wf56vzvyq6c20cjqpx5k1j-dvtm-master/bin/dvtm-pager: interpreter directive changed from "/bin/sh" to "/nix/store/r47p5pzx52m3n34vdgqpk5rvqgm0m24m-bash-4.4-p23/bin/sh"
/nix/store/sv2fy18478wf56vzvyq6c20cjqpx5k1j-dvtm-master/bin/dvtm-status: interpreter directive changed from "/bin/sh" to "/nix/store/r47p5pzx52m3n34vdgqpk5rvqgm0m24m-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/sv2fy18478wf56vzvyq6c20cjqpx5k1j-dvtm-master...
/nix/store/sv2fy18478wf56vzvyq6c20cjqpx5k1j-dvtm-master

This should be named dvtm-<date> rather than dvtm-master: https://nixos.org/nixpkgs/manual/#sec-package-naming

Thanks!

Using fetchFromGitHub now.

Just noticed another issue — since the name is changing from dvtm to dvtm-unstable, I think this update won't be caught by nix-env -u. It might make sense to split the unstable dvtm into its own toplevel attr.

Ok, so, I'm just going to leave dvtm-0.15 in there if that is the consensus. I've added dvtm-unstable which can update faster than the release cycle. IF a new release is ever going to happen, people who use nixpkgs.dvtm will get it, but they won't get this update. In the meantime, people on Darwin should use dvtm-unstable.

The commit message should be "dvtm-unstable: init at 2018-03-31"

Yes, sorry, got my repository standards mixed up.


Updating directions for testing out kompose

Made some corrections to the documentation, as the output was not what I experienced, and also added directions for cleaning up after testing the example.
This is the first time I've contributed to open source, so if I need to change something, please let me know.

/assign @kbhawkey

@kbhawkey sorry I didn't mean to tag you in this yet, label is missing for lgtm; misunderstood the approval and didn't wait for the tag to be applied.

/lgtm

/assign @kbhawkey


encode LTable to JSON Array

Would it be possible, and within the scope of this library, to add a function that imported numeric keys in a table to an array instead of an object with the label of the number?

I don't think I fully understand what you're requesting. Could you provide some code with expected output and actual output? Thanks.

In lua, arrays are basically just maps with numeric keys. the code
a = {}    -- new array
for i=1, 1000 do
    a[i] = i
end

creates a table with matching indexes and values. currently, this library converts this to JSON by creating an object who's name is the string representation of the integer value
{
  "1":1,
  "2":2,
  "3":3
   ...
}

I am wondering if it would be possible to have a special case included in this library where integer indexes are put into arrays.
[
  1,
  2,
  3
  ...
]

Because of the nature of lua tables, it would either have to be limited to special cases where the indexes are ONLY integers
[
    1,
    2,
    3
   ...
]

or a reserved name would have to be chosen(or be an argument) to be able to work with other keys/indexes.
{
  "Name": "integers",
  "array_values": [
    1,
    2,
    3
   ...
  ]
}

If it was reserved, I suspect that it would be handled the same way that a[1] and a["1"] are different in the table, but have the same value in the JSON. (I don't know how the library currently handles this, but I suspect that it throws an error. )

The library already does do this.  See the test I just pushed: d0a7d8b4c69aea3341d0131a64b59a6df9759be2

Ok, I ran a test myself. I figured out that the issue was that I started at 0, which then results in
{"0":0,"1":1,"2":2,"3":3,"4":4,"5":5}
Understandable albeit a bit confusing considering that goalng is 0 indexed vs lua's 1 indexing. Something for the documentation.


Initial commit of aws_cloudsearch_domain resource

@tyler-ball I have addressed most of your comments and left comments for the two I didn't address.
I went down the road of making more complete integration tests, but unless cloudsearch gets a lot faster, I'm not sure it is realistic.  In my testing it can take a VERY long time for many of the options that you set to actually show up in the API calls that are supposed to return their value.  I think we could get around this by writing a complete model object for this CloudSearchDomain's that is able to understand all of the various states of "partial update" that the domain might be in.  Unfortunately, I think that is a bit out of scope for us right now since we haven't started using cloudsearch yet and thus don't necessarily understand what users of this will expect.  I'd rather not pin us into an API until we have some experience with this resource.

Some minor comments - once those are addressed I am :+1: and will merge this

Actually, before we merge this it needs to be added to ref_full.rb and ref_destroy.rb.  These are good examples for users of what the resource looks like when all its attributes are populated.


[Server] - Fix export bug

Details
When the user attempts to export their bookmarks none of the links are present on the resulting html file.
Tasks

[ ] Investigate ExportBookmark.java
[ ] Create a couple of bookmarks or use the test user's account jsmith/test
[ ] Hit the export button in the browser or use curl/Postman to hit /bookmark/export endpoint.
[ ] Apply changes to fix bug
[ ] Add unit tests

Hello, I would like to work on this if possible


WIP Add scanning for periodic advertising chains with short regular PDU intervals

This is a very limited implementation that is able to synchronize and receive periodic advertising trains where PDUs in a train are:

send in short intervals e.g. 500us
intervals between PDUs are always constant.

@ppryga can this be closed now that @andrzej-kaczmarek has https://github.com/zephyrproject-rtos/zephyr/pull/36588 open?

@cvinayak yes, of course.


[JBEAP-12701] IndexOutOfBoundsException during Jaxrs app deployment

Issue: https://issues.jboss.org/browse/JBEAP-12701

I'm closing this as it's not actually solving the problem afaiu. Feel free to re-open if needed, though.


Installation problems

Installation fails with this error:
Command "/usr/bin/python -c "import setuptools, tokenize;file='/tmp/pip-build-jK6NYC/evdev/setup.py';exec(compile(getattr(tokenize, 'open', open)(file).read().replace('\r\n', '\n'), file, 'exec'))" install --record /tmp/pip-o3VBQV-record/install-record.txt --single-version-externally-managed --compile" failed with error code 1 in /tmp/pip-build-jK6NYC/evdev
It seems like it might be a compiler error but i don't know how to proceed

Which Python version? Which OS? What command did you run?

os is ubuntu 14.04,
i do not know how to check what python version pip uses
On Fri, Jul 31, 2015 at 9:42 AM, Lauri Niskanen<EMAIL_ADDRESS>wrote:

Which Python version? Which OS? What command did you run?
—
Reply to this email directly or view it on GitHub
https://github.com/chrippa/ds4drv/issues/55#issuecomment-126695154.

Does this still occur?

I have this same issue trying to sudo pip install ds4drv, running opensuse tumbleweed 20170703.
pip is version 9.0.1 on python 3.6.

I'm closing this. For anybody else who might have this issue in the future, try installing python-devel package (or equivalent on your distro).


Pairing Based on Language?

I'd love to see an addition to pairing bot where you can restrict matches to specific programming languages. I was going to look into making a pull request for this, but unfortunately I don't know any Go.

Hi Kelly! Thanks for the suggestion!
When I first made pairing bot, I didn't want to include this feature
because I thought the added complexity might decrease engagement. Also, it
significantly increases the complexity of the matching algorithm, and
that's not the sort of problem that I find very fun.
That being said, I'm not opposed to adding it if someone else comes up with
a matching algorithm for it! A good algorithm for this would need to have a
non-exclusive preference for matching people based on language (since
otherwise the same two people may get matched every single day if they are
the only two people who've listed that language preference).I think this
means that pairing bot would need to start remembering historical matches,
which she doesn't currently do. So it's a big change.
On Wed, Apr 28, 2021 at 11:04 AM Kelly Foulk @.***>
wrote:

I'd love to see an addition to pairing bot where you can restrict matches
to specific programming languages. I was going to look into making a pull
request for this, but unfortunately I don't know any Go.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/thwidge/pairing-bot/issues/11, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJILHU2GZJJQWZMPXUY6GVTTLAPWZANCNFSM43XIRYOQ
.


pulling changes from 10.0 to 10.1

Before issuing a pull request, please see the contributing page.

Updates to 10.1


Expose jit args to accommodate serialization

This would expose the original args passed to jax.jit as jitted_func._jit_args to accommodate serialization of jit-compiled functions along the lines of:
class FunctionApproximator:
  def __init__(self):
    def func(a, b, c, d):
      return a + b + c + d
    self.func = jax.jit(func, static_argnums=(1, 2), donate_argnums=3)

  def __getstate__(self):
    jit_args = self.func._jit_args
    return (jit_args.fun, jit_args.static_argnums, jit_args.donate_argnums)

  def __setstate__(self, state):
    f, s, d = state
    self.func = jax.jit(f, static_argnums=s, donate_argnums=d)

closes #5043

@hawkinsp I just rebased on master. By the way, if you're not keen on merging this, don't worry about it. Just let me know. I can handle it downstream.

Hi all, returning to this after a while, I think it might be better to handle this downstream instead.
In my case, I added an alternative jit util that supports serialization here.
Thanks for looking into this. Will close this PR now.

Hi all, returning to this after a while, I think it might be better to handle this downstream instead.
In my case, I added an alternative jit util that supports serialization here.
Thanks for looking into this. Will close this PR now.


Migrate typings away from DefinitelyTyped

The RDFJS typings should be maintained and published by the RDFJS organization. Some of the issues with hosting the typings on DefinitelyTyped are:

issues and PRs on DT are not linked to this organization and therefore do not undergo the same review process, nor are RDFJS org members notified. Issues and PRs should be tracked, discussed, reviewed, etc., on RDFJS GH org.
DT maintainers are responsible for reviewing PRs and thus control releases; RDJFS should have authoritative control over publishing and maintaining releases.
DT is useful for adding types to untyped JS projects, but (as far as I know) was never intended for defining interfaces of standardized specifications.

Several libraries are currently using the de facto @types/rdf-js package for importing RDFJS interface types. Open question is whether or not DT/TypeScript will allow upgrading this package so that it simply acts as a proxy for new @rdfjs/types package by importing and exporting it.

For reference, DT or DefinitelyTyped is this repository: https://github.com/DefinitelyTyped/DefinitelyTyped

+1 for taking control

Open question is whether or not DT/TypeScript will allow upgrading this package so that it simply acts as a proxy for new @rdfjs/types package by importing and exporting it.

If not, we deprecate.

I think the safest option would be to make this repo a clone of DT where we accept (and discuss) changes to the RDF/JS typings, and push to the upstream DT when we reach a consensus.
While I agree that @rdfjs/types would be a much cleaner solution, the convention of using DT (and @types/* packages) may lead to unforeseen problems in the future.
At the moment I don't know of any specific problems that would occur, but I can for example imagine that there could be linting tools that have special behaviour when they encounter @types/* packages, and would therefore misbehave on @rdfjs/types.

On a closer look maybe it would not be as troublesome, at least for package consumers. Right now the types package pretends to describe rdf-js. A package which indeed do exist on NPM
I think that the path of least resistance is to revive that package and remove @types/rdf-js from DT. Any existing import {} from 'rdf-js' should just work because typescript tries to resolve *.d.ts files in the same way it resolves other modules, if I understand this correctly.
The issue I found is on the end of any @types/ packages which depend on the current @types/rdf-js. According to the readme of DT any package can actually have package.json dependencies to "real" packages with types. The only nuisance is that any such package has to be explicitly added to  allowedPackageJsonDependencies.txt in DefinitelyTyped-tools.

Hey all,
I think reviving the rdf-js package would definitely mean most people don't have to find and replace their existing imports. We could add a depracation notices to @types/rdf-js, and existing libraries would get a notice but still work.
However this could lead to incompatibility if we make updates to the new types and a package is using the old ones. Thus I think "ripping the band-aid off" might be thee best option. That could entail either

@tpluscode 's idea of removing the DT package
Moving the new types to a new import location like @rdfjs/types, which would force clients to update code. Then at least they'd recognize if we've updated the interfaces and would be able to update code.

Regardless, I think its great that we're moving to our own repo so we can keep the types consistent with the spec and implement more consistent versioning.
I personally have several times experience TS errors when using packages that depend on @types/rdf-js version 3 vs version 4, so I'm very hopeful this will improve interoperability.


make params a dictionary in Python output

This should be equivalent because Python dicts keep insertion order since CPython 3.6+
I still need to update all the Python tests.

Dicts don't allow duplicate keys though. I believe there is a test that checks for this.

I thought that too, but curlconverter converts those to a list for one param name, not multiple entries in the list of params:
curl 'https://example.com/?firstparam=1&secondparam=2&firstparam=1'

is converted to
# [...]
params = (
    ('firstparam', ['1', '1']),
    ('secondparam', '2'),
)
# [...]

I wonder if that's what the comment "It seems impossible to parse and reproduce query strings 100% accurately" is about... it might be best if this situation was detected, and in that case you output a tuple of tuples in the correct order (might need to do the URL parsing with a different library), but a dictionary otherwise.

Ah, you've discovered a pretty big inconsistency.
Curlconverter respects duplicate keys for POST but not GET.
curl -X POST 'https://example.com/' --data firstparam=1&secondparam=2&firstparam=1
I think the move here is to add support for duplicate keys in GET requests, to match the behavior of curl itself.

I think the difference here is whether they're in the --data or in the URL.
It does respect duplicate keys in GET request though (or I don't understand what you're saying). Having a list for one entry or multiple entries works the same in requests:
>>> resp = requests.get("https://example.com/", params=(('first', ['1', '1']),))
>>> resp.request.url
'https://example.com/?first=1&first=1'
>>> resp = requests.get("https://example.com/", params=(('first', '1'), ('first', '1')))
>>> resp.request.url
'https://example.com/?first=1&first=1'

and curlconverter doesn't respect the order for POST requests either, your link is converted to
data = [
  ('firstparam', '1'),
  ('firstparam', '1'),
  ('secondparam', '2'),
]

not
data = [
  ('firstparam', '1'),
  ('secondparam', '2'),
  ('firstparam', '1'),
]

I'm going to close this PR in favor of adding "Copy as Python requests" to Chrome directly https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/2680281 (still a WIP)
Out of curiosity, I went looking for the source code of where this happens in Chrome, it's this function
https://github.com/ChromeDevTools/devtools-frontend/blob/400dcba55573eee9e36168985616f0c88aa58bec/front_end/network/NetworkLogView.js#L2241-L2370


Contains method for strings does not add N before string literal in SQL query

I use accent insensitive queries to DB and this is important for me to match .Net strings as unicode strings in MS SQL Server.
In fact .Net String type is Unicode string and for CONTAINS SQL clause it should always use N prefix.

@azhoshkin what version are you using? AFAIR we did the work necessary to fix this for RC2.

@divega I'm still on RC1. Sorry, search only in open issues. Looks like mine is duplicate of https://github.com/aspnet/EntityFramework/issues/4686
Thanks! Closing this.


[Spec Deviation] Error type should have a stackTrace built in method

Description:
As per the spec (v0.990:2019-01-16)
The following  built-in methods are provided on the error basic type:
reason: returns the error’s reason string
detail: returns the error’s detail record as a frozen mapping
stackTrace: returns an object representing the stack trace of the error (Needs to be implemented)
Example:
function testErrorTypeDescriptorsStackTraceBroken() {
    error error1 = error("Error One");
    io:println(error1.stackTrace());
}

This is fixed in JBallerina 1.0.0-Alpha.
https://github.com/ballerina-platform/ballerina-lang/blob/6881b7afb01a2b98e2dac09681ef44ea8d8cb989/langlib/lang.error/src/main/ballerina/src/lang.error/error.bal#L49


remove body for GET/HEAD request in http binding

fixes: https://github.com/aws/aws-sdk-js-v3/issues/1633
This change doesn't affect RPC protocols because they use
POST all the time
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Seems like Smithy does support GET/HEAD request with a body. But it will warn the users to make sure they know what they do: code.
Since the problem this PR to solve only exists in browsers. We should allow these requests with a body on modelling level. But in browser, the GET/HEAD requests' body will be omitted.


Whiterun exterior troops spawns

So someone on nexus mentioned the fact that they failed the battle of Whiterun 3 times because
The Stormcloak troops started spawning on the lever at the lever to the lower bridge once the defend the drawbridge objective was reached
their question was
Is the stormcloaks supposed to have teleporting soldiers/giants during battle of Whiterun?
now
I have a question relating to that, since the only way to the drawbridge is via the ramp behind the archer towers,
Does the stormcloak path finding take them there or do they just spawn in up there?

If their pathfinding is trying to take them there but it's blocked because something is in the way, I think it's normal for them to teleport instead.

them to teleport

The issue they are asking about is actually one I have encountered before which is why I play mostly on god mode, I think after a certain amount of time one spawns in to trigger the next stage.

What I was asking is does the path finding of the Stormcloaks take them up the ramp and to the upper part?

I think it does when I'm attacking. No idea about when the player defends Whiterun.

I think it does when I'm attacking. No idea about when the player defends Whiterun.

As someone how has extensive experience in battle cheese, I can assure you this happens on defence,
I’ll be minding my own business,  killing Ulfrics boys then suddenly out of nowhere the drawbridge drops
(I intentionally lose the battle of Whiterun a lot to trigger the defence scenario)

What I was asking is does the path finding of the Stormcloaks take them up the ramp and to the upper part?

Yes path finding takes troops up the ramp. Most troops just fight but 2 of them are programmed to go after the objective.

If their pathfinding is trying to take them there but it's blocked because something is in the way, I think it's normal for them to teleport instead.

This is also true. There are invisible markers at each of the siege phases (there are 5 from attack scene with field co to the final objective being between before the troops run in). As troops are killed, depending on the phase, the troop would be reset at one of these invisible markers. I will check to see if there are any of these markers right next to the bridge. This happened in vanilla but none of the troops were programmed to pull the lever so I guess no-one ever noticed.

So I verified that the respawn markers all spawn at around the front gate where the barricade is. But they will pathfind all the way to the second gate lever. Best strategy is to field the choke point and make sure nothing gets passed :)


chore: Remove python packages entirely

We previously used docuploader and releasetool; these are both now reimplemented in .NET.

This PR should only be merged after we've successfully done a package release with the new tools. But it's good to get some review before then.

A release has succeeded, so we're good to go.


Windows Release builds failing due to missing deps (Patch File Fix)

Hi! 👋
Firstly, thanks for your work on this project! 🙂
Today I used patch-package to patch<EMAIL_ADDRESS>for the project I'm working on.
when building for release on react native windows this library fails due to missing dependencies that are listed for Debug but not in Release. this patch just adds the dependencies for release as well enabling a successful build. 😄 hope this helps!!
Here is the diff that solved my problem:
diff --git a/node_modules/react-native-svg/windows/RNSVG/RNSVG.vcxproj b/node_modules/react-native-svg/windows/RNSVG/RNSVG.vcxproj
index 11ad6f5..bd96537 100644
--- a/node_modules/react-native-svg/windows/RNSVG/RNSVG.vcxproj
+++ b/node_modules/react-native-svg/windows/RNSVG/RNSVG.vcxproj
@@ -120,6 +120,9 @@
     <ClCompile>
       <PreprocessorDefinitions>NDEBUG;%(PreprocessorDefinitions)</PreprocessorDefinitions>
     </ClCompile>
+    <Link>
+      <AdditionalDependencies>dxguid.lib;WindowsApp.lib;%(AdditionalDependencies)</AdditionalDependencies>
+    </Link>
   </ItemDefinitionGroup>
   <ItemGroup>
     <ClInclude Include="BrushView.h" />

This issue body was partially generated by patch-package.

bump, this works and is needed for windows support

Hello @mat2718,
As I see the issue was resolved, can we close it?

Hello @mat2718,
As I see it was already implemented.
Thank you.


Carbonmark data token prices card

Description

Carbonmark data token prices card
Refix translation on server side component
Simplifies ChartCard API for simple cards
Fixes price formatting
Intrduce a light barel file for lib/utils (see: https://github.com/KlimaDAO/klimadao/issues/1502)

Related Ticket
Resolves #1458
Changes

Before
After

Checklist

[ ] I have run npm run build-all without errors
[ ] I have formatted JS and TS files with npm run format-all

I couldn't run the branch it its current state
[1] Module not found: Package path ./utils/getLocales is not exported from package /Users/selim/KlimaProjects/klimadao/node_modules/@klimadao/lib (see exports field in<EMAIL_ADDRESS>[1] > 1 | import { getLocales } from "@klimadao/lib/utils/getLocales";
[1]   2 | import { IS_PRODUCTION } from "lib/constants";
[1]   3 | import createMiddleware from "next-intl/middleware";
[1]   4 | import { NextRequest } from "next/server";

Fixed in 4ebb124e


Fix use of deprecated add() method on HealthSupport in quickstart and bookstore test

Use addLiveness() instead of add() (which has been deprecated) in quickstart se and bookstore test app. Also fix README to use correct graalvm version.

How about the health docs ?

How about the health docs ?

Done


Update vendor.json for picking up dcos-go changes

This PR is for picking up the default path updated in dcos-go for Windows powershell change

Can one of the admins verify this patch?


Live

Test PR

Docs Build status updates of commit af1d0a4:
:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Test PR


Allow specifying since and until times for exports

This changes the exporting mechanism to allow the time range to be configured. This is to meet the need of being able to do ad-hoc ranges, such as from Friday morning until Sunday evening, or between 9am and 11am on a particular day.
To accommodate this a few changes have been made:

No longer using the terminology "daily report", since that naming is not always applicable.
The filenames generated are now more verbose, in the format of 2020-05-01-1000-to-2020-05-01-1100 to give clarity about their contents.
Allows SINCE_TIME and UNTIL_TIME environment variables to specify when an export runs to. These can be relative times, for example SINCE_TIME=10:00 UNTIL_TIME=10:00, to specify the times from the previous day and the current day or they can be absolute, for example SINCE_TIME="2020-05-01 10:00" UNTIL_TIME="2020-05-03 11:00" to specify an exact range.
Now prints the time range out at the start of the export to offer an opportunity for confirmation of a time range.

Thanks @benthorner, comments should all be resolved.


Spelling mistake in readme.md

hello guys, I don't know if i should report this as an issue or not, I was checking out the repository and found a spelling mistake.
I'm attaching the screenshot here. it's written suscribed which i feel like should be subscribed

thanks for the report :), it has been fixed.


Explicitly set global read permissions in build workflow

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 90.10%. Comparing base (2f675c8) to head (e4a9dc9).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #127      +/-   ##
==========================================
- Coverage   90.45%   90.10%   -0.35%
==========================================
  Files          11       11
  Lines         576      576
==========================================
- Hits          521      519       -2
- Misses         55       57       +2

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


Push 'Confirm Address' button when user wins

When the user wins a sweep they are given an url which takes them to a page showing the item they have won and a button saying "Confirm Address" with the address listed below.  Once you are certain amazon has the right address I would think the preferred choice would be to have the software auto click the "Confirm Address" button to ensure the prize is locked in.  Sure you might get a few things you didn't really want but it would prevent you from losing the win of a really nice cell phone because the power went out at your house and you lost the terminal window.

Well I can confirm the sending on emails works great other than getting stuck in gmail's spam folder till I marked it as not spam.   Additionally I am attaching both a screenshot and a zip of the html and elements necessary to display the page to assist in this issue.
Amazon Giveaways.zip

Awesome, thank you! Added this to my todo list.
For my notes, looks like that button is:
<input name="ShipMyPrize" class="a-button-input" type="submit" value="ShipMyPrize" aria-labelledby="lu_co_ship_box-announce">

Any update on this? Lost a few things because I wasn’t paying attention and the database didn’t show W.

Added in v4.3.0

How do you update?

I think npm install -g giveaway-grabber@latest

Is it possible to add a setting to turn off the "auto confirm address" feature? I'd love to switch addresses for different items.


Submit a Resource Default emails

The email copy can be found here: https://docs.google.com/document/d/1RBJyC8F0yejv6qdx3qtXwDxEK1_4P4JI_ufHrRCivbU/edit

The best hookup point for this is the submit.save method.
https://github.com/AASHE/hub/blob/master/hub/apps/submit/forms.py#L33-L41

@monika-aashe - In the emails that begin "Dear ," what would you like to do when the user hasn't provided a first name?  (There are lots of existing Drupal users that haven't provided a first name.)  Substitute "Hello!"?

I'm fine with updating from "Dear" to "Hello" across the board, but maybe a
comma instead of an exclamation point so we're not implying that we're
excited to let you know that your resource has been declined.
So in other words:

Hello Jane,
Hello,

On Wed, Dec 16, 2015 at 12:26 PM, Robert Erb<EMAIL_ADDRESS>wrote:

@monika-aashe https://github.com/monika-aashe - In the emails that
begin "Dear 'first-name'," what would you like to do when the user hasn't
provided a first name? (There are lots of existing Drupal users that
haven't provided a first name.) Substitute "Hello!"?
—
Reply to this email directly or view it on GitHub
https://github.com/AASHE/hub/issues/47#issuecomment-165183385.

--
Monika Urbanski
Data & Content Manager
Association for the Advancement of Sustainability in Higher Education
www.aashe.org |<EMAIL_ADDRESS>|<PHONE_NUMBER> ext. 26
--
Connect with us:   Website http://www.aashe.org/ | Facebook
https://www.facebook.com/aasheorg | Twitter
https://twitter.com/AASHENews | LinkedIn
http://www.linkedin.com/company/aashe
http://www.aashe.org
Email Confidentiality Notice
http://www.aashe.org/email_confidentiality_notice

@rerb - do you know the status on this? I haven't ever gotten any emails to<EMAIL_ADDRESS>so my guess is this hasn't been implemented yet or we're waiting until it's on the hub.aashe.org server.

@monika-aashe this was implemented back in December.  You're not getting any messages because the email environmental variables haven't been set on the server.  I'll enter a new issue for that.

Related to #204 and #219


Rename mod file to fix remote install error

bors try

PR ready for review.

Please squash.

Please squash.

Done.

bors r+


docs: add request-lifecycle to the documentation + new homepage

Description
Add a request-lifecycle page with the needed information to understand what Serinus is doing, for the hooks see #18. Also add new homepage for the Serinus Documentation.
Fixes #15
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Add a Gitter chat badge to README.md

Microsoft/malmo-challenge now has a Chat Room on Gitter
@katja-hofmann has just created a chat room. You can visit it here: https://gitter.im/malmo-challenge/Lobby.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot


boehmgc: 8.0.6 -> 8.2.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/boehm-gc/versions.
meta.description for boehmgc is: "The Boehm-Demers-Weiser conservative garbage collector for C and C++"
meta.homepage for boehmgc is: "https://hboehm.info/gc/"
meta.changelog for boehmgc is: "https://github.com/ivmai/bdwgc/blob/v8.2.0/ChangeLog"
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 8.2.0 with grep in /nix/store/yss81vc4nnbfc9bdjm9nq6q27xbbmkxl-boehm-gc-8.2.0
found 8.2.0 in filename of file in /nix/store/yss81vc4nnbfc9bdjm9nq6q27xbbmkxl-boehm-gc-8.2.0

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

24851 total rebuild path(s)

24851 package rebuild(s)

First fifty rebuilds by attrpath

ArchiSteamFarm
AusweisApp2
CHOWTapeModel
ChowCentaur
ChowKick
ChowPhaser
CuboCore.coreaction
CuboCore.corearchiver
CuboCore.corefm
CuboCore.coregarage
CuboCore.corehunt
CuboCore.coreimage
CuboCore.coreinfo
CuboCore.corekeyboard
CuboCore.corepad
CuboCore.corepaint
CuboCore.corepdf
CuboCore.corepins
CuboCore.corerenamer
CuboCore.coreshot
CuboCore.corestats
CuboCore.corestuff
CuboCore.coreterminal
CuboCore.coretime
CuboCore.coretoppings
CuboCore.coreuniverse
CuboCore.libcprime
CuboCore.libcsys
DisnixWebService
EBTKS
EmptyEpsilon
Fabric
HentaiAtHome
LibreArp
LibreArp-lv2
MACS2
MIDIVisualizer
MMA
OSCAR
OVMF
OVMFFull
PageEdit
R
SDL
SDL2
SDL2_gfx
SDL2_image
SDL2_mixer
SDL2_net

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/yss81vc4nnbfc9bdjm9nq6q27xbbmkxl-boehm-gc-8.2.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 boehmgc https://github.com/r-ryantm/nixpkgs/archive/b40af597bc19c38ccdbf0006ac771683e19133c9.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/yss81vc4nnbfc9bdjm9nq6q27xbbmkxl-boehm-gc-8.2.0
ls -la /nix/store/yss81vc4nnbfc9bdjm9nq6q27xbbmkxl-boehm-gc-8.2.0/bin

Pre-merge build results
NixPkgs review skipped

Maintainer pings
cc @AndersonTorres for testing.

Closing in favor of #188454


AWS ELB with https

Using your "simplest" manifest under https://github.com/yuvipanda/jupyterhub-simplest-k8s and enabling SSL at the AWS load balancer causes the "loading kernel" step after successfully launching a pod to fail. This seems to be a proxy routing issue but my nginx-fu is not up to par. Would you have any ideas?
kubectl logs hub-proxy-deployment-1347898349-beane | grep 400

<IP_ADDRESS> - - [05/Oct/2016:19:20:17 +0000] "GET /user/s/api/kernels/d74f27c7-2bbb-40fa-b44d-9602713c318f/channels?session_id=232532B3186A427FA17CC644623763BC HTTP/1.1" 400 34 "-" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/53.0.2785.116 Safari/537.36"

Do you see the pod running successfully? What's in the pod's logs? Also can you try the persistent-hubdb configuration instead? And how did you setup k8s?

I figured it out -- AWS ELB was set to HTTPS and not SSL with TCP instead. This solved the problem. Thanks!


DOCS Expand Method Creation docs, shift BasicMath components into them

The front-end side of the MFA Method creation documentation is now in decent shape, with the BasicMath components as examples, and I've split the backend Method Availability docs into a separate file (which should be expanded to cover the rest of the backend Method creation docs at some point).

@ScopeyNZ I've added a note to the onCompleteVerification prop docs stressing the importance of server-side validation - does it read well?


Static styles

Description
When the DEBUG setting is set to True strange things happen if the styles are statically imported. Now to avoid this it has been incrusted into the base.html file, but this is not the best solution. This error needs to be addressed ASAP.

The application will be released in DEBUG mode. End of thread.


open in google colab ?

<a target="_blank" href="https://colab.research.google.com/github/<github_link>.ipynb">

can we please add a colab link, It would be helpful for people who dont have high spec system to run this locally.
Thanks

Hey Praveeno, I would suggest you try using webrtc to use your local microphone on google colab, and then run it.


Remove VERSION from documentation landing page template as it's only …

…replaced on releases, but docs are sometimes published without a release which results in broken links.

Although, I have merged the pull-request. But, I am planning to remove this landing page and instead use the drop-down on the main documentation page.


[UI] Notification Center: Doesn't always fully open on click

Current Behavior
While the Notification Center is to peek out on hover of the Notification Center button (usually a bell, but sometimes can exclamation mark icon), it should always fully open when are use her clicks on the button. In the attached video, you can see that despite being clicked on the sidebar, never fully opens.
Desired Behavior
Sidebar open and remain open on click.
Screenshots/Logs
https://user-images.githubusercontent.com/7570704/212572486-8a4d9ac8-825e-427a-89be-ef7198d9592b.mov
Environment

Browser: Chrome Safari Firefox
Host OS: Mac Linux Windows
Meshery Server Version: edge

Contributor Guides and Handbook

🎨 Wireframes and designs for Meshery UI in Figma
🖥 Contributing to Meshery UI
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

@leecalcote  Can I work on this?

@deepakdinesh1123, yes, that would be great. 👍

@leecalcote i would like to work on this issue, can you assign me this.

@ajaysinghpanwar2002 thanks for volunteering. I encourage you to review @deepakdinesh1123's PR - https://github.com/meshery/meshery/pull/6905


The ScrollViewer scrolling randomly breaks down

Describe the bug

So, this is sort of a weird issue to open, if you will, as I don't really have much detail to share on this. The problem is that the native ScrollViewer control will randomly break down and stop scrolling entirely. The weird thing is that if you scroll on the mouse wheel, you'll still be able to see the scroll bar move up and down as if you were actually scrolling, but the content itself will not move at all.
Also, if you actually use the mouse to manually click somewhere on the scroll bar, the content then does scroll instantaneously to that offset. It just won't move at all while using the mouse wheel though.
NOTE: when this happens, all the ScrollViewer in the app are affected, it's not a single one that breaks down. In one case I had 3 ListViews side by side in different panels, and all three of them started showing this behavior.
NOTE 2: this can sometimes recover on its own. One time it happened, I minimized the app for a while and when I brought it back up again the ScrollViewers started working fine again, without the need to close the app and open it again.
NOTE 3: this happens both with a local Debug build as well as with a Release build (.NET Native) downloaded from the Store.
This is happening in two different apps for me, and I know of other devs that had the same issue too. I can also confirm that this first happened quite a few builds ago, I think I've first noticed this back on 14393 (although I can't be sure it wasn't already there before that one too).
I know you're planning on including an entirely new ScrollViewer with WinUI 3.0, but since you said you'd still be updating the current one with bug fixes I thought I'd mention this anyway.
Now, I don't really know how to reproduce this. If you have any ideas, please let me know what I can do to help out. I just know that for me, it'll randomly break down, with no exceptions being thrown or anything like that, it'd just stop working. Note that this happens very rarely, but it still happens from time to time (it was not just a one time issue that never happened again).
Steps to reproduce the bug

Steps to reproduce the behavior:

..?

Expected behavior

The ScrollViewer should scroll as usual
Screenshots

I have a video that shows the issue, but since it's from an app I'm working on that I don't want to announce yet, I can't link it here. I'd be happy to share it via DM to one of you guys from the WinUI team, if that helps.
Anyway, that's just a ListView with some content, and you can see the scrollbar moving up and down while the rest of the content remains perfectly still.
It also shows that while the issue is present, you can still click on the scrollbar with the mouse to make the content jump to a different offset, that still works.
Version Info

This happens with the native ScrollViewer, both with and without the WinUI package installed.
I've noticed this on W10 Desktop (x64, both Home and Pro if that matters)
NuGet package version:

Windows 10 version
Saw the problem?

Insider Build (xxxxx)
Yes

October 2018 Update (17763)
Yes

April 2018 Update (17134)
Yes

Fall Creators Update (16299)
Yes

Creators Update (15063)
Yes

Anniversary Update (14393)
Yes

Device form factor
Saw the problem?

Desktop
Yes

Mobile

Xbox

Surface Hub

IoT

Additional context

Thanks for reporting this, it's definitely something we've heard reports of before but we've never had a reliable repro. Unfortunately I don't think this area has particularly good tracing so we probably need a live repro of this to debug.
Is this something that happens often enough that we could work with you to set up a remote debug session? Or maybe you could get the repro set up in a VM and then save the VM state and share that out?

Hey @jevansaks - sure, I figured something like this would indeed be hard to track, but I figured having an open issue about this could help anyway, if nothing else so that other devs would be able to chime in and possibly share more useful info on this if they can.
So, as I said it's completely random for me. Sometimes it'll happen twice in less than an hour, sometimes it could be a week or more before it happens again 😕
That said though, it does seem to be particularly frequent with this new app I'm working on for some reason, so that might be useful to debug this. So to answer your questions:

I don't think a remote debug session would help, since as I said, sometimes it can still be days between occurrences of this bug.
Sure, I could try to setup a VM and run the app for a while. Since I've had reports of this happening with builds from the Store as well, I guess I could just get the app from there instead of having to setup VS and everything. Also, again, I'm not sure if this would help since the issue doesn't happen that frequently, but I could share a private download link for you if you'd like to try the app and see if the issue happens on your side (if you're lucky).

Another question: what if this happens when I'm just debugging the app from VS, with my primary OS and not from a VM? Is there some sort of snapshot I could take at that point from VS, or you'd have to have a full VM snapshot in order for it to provide useful info?

I hear you, these types of issues are tricky. We're working on adding better diagnostics to the more complex controls (like ScrollViewer) going forward so that we would be able to debug issues in the field better.

Is there some sort of snapshot I could take at that point from VS,

I'm not aware of a way to snapshot a VS debugging session like that -- but if you were using remote debugging of the app running in a VM then when it repros you could detach the visual studio debugger and then save the VM at that point. I also understand if you don't want to go to these lengths to get us a repro. :)
If you can share a link to the app in the store or just an appx here, we could try reproing with that locally.

Oh, the improved diagnostics sounds like a good idea for the future! 😄
No it's fine, I'm happy to help out where/if I can!
I just don't have experience working with remote debugging sessions, so if all you need is just a VM snapshot I guess I could just grab one of those official W10 VM images from MS, install the app and play with it until I eventually (hopefully) get the issue, then make a snapshot I can share with you guys.
Would that work?
As for either a download link or an appx build to share, as I said I'd like not to just share either of them here. If that's ok for you, I could either email those to you, or DM them on Twitter (just followed you there). I realize it's not idea and I'm sorry if this is annoying, I'd just like to keep the app under wraps for now 😅

I could just grab one of those official W10 VM images from MS

Yup, that's what I was thinking -- set it up in a local Hyper-V session and it should be pretty easy to manage the state.
And no worries about not wanting to share the app openly. DM me on Twitter with a link to the app. Thanks!

I reported this issue mid 2018.
It could be reproduced by Microsoft and the root cause was found.
Micah Lewis emailed me on July 18, 2018:
Thought you’d like to know… A dev from the other team has identified the root cause for the scrolling issue.  Next step is identifying the right fix and hopefully getting it in to be part of the next major release.
Some nitty-gritty details in case you’re interested…
When switching between chat/calendar windows, viewports are getting destroyed and recreated. If it so happens that the viewport was destroyed when it’s in an inertia (or even running state too), dmanip doesn’t get a chance to  clean it up from the _activeViewportMap. Hence the orphaned viewport entry lingers in _activeViewportMap and thus compositor bit won’t get set to dirty due to the reason below.
I took some traces of the app in repro state. From what I’m seeing so far, it looks like the _activeViewportMap map has two viewports in it. One of them doesn’t actually seem to be active (it’s state is READY). I’m not sure which viewport this corresponds to. The viewport that is actually active is infact getting its viewport update callback. However, since the code in OnViewportUpdated() sets the dirty bit only when all viewports in the _activeViewportMap is updated, it never ends up setting the compositor’s dirty flag (hence no updation).
It looks like some kind of race must have caused the inactive viewport to accidentally remain the _activeViewportMap structure.
And on October 30, 2018:
Hi Wim, it looks like the fix was checked into a lower level branch two weeks ago.  Depending on how quickly those changes are integrated from that team’s branch up to MAIN (which might take a few more weeks) the fix should be available on a future Insider’s build and part of the next major release in 2019.

The power of the hive mind! Thanks @wbokkers for closing the loop on this, that's incredible. And awesome to hear it's fixed now.
Based on that it sounds like it should no longer repro on insider builds and the upcoming Windows 1903 release.

The issue has still be seen on Insider builds by @Sergio0694. So maybe Micah Lewis (@micahl ?) can confirm that the fix will be in 1903?

@Sergio0694 which insider build # have you hit this?
@jevansaks the issue Wim hit was tracked by internal issue # 17855072.  It says the fix should be in builds as of 18282.1000.rsmaster.181109-1528.

Hi, thank you guys for chiming in!
I'm running 18362.86 right now, so I can confirm the fix still isn't working on this build.

@wbokkers have you had a chance to try your app on a recent build and verify that the issue no longer repros for you?

@micahl I can't reproduce it on 18362.86. I can take some time before the issue shows up, but I scrolled for about 15 minutes.
I can't reproduce it on 17763.475 either. But that's probably because I'm on a new PC and not all devices have this problem afaik. Our app is being tested for quite some time now and I get a lot of feedback, but never about this issue. So, I assumed that this issue had already been solved in some cumulative update.

@Sergio0694, it's possible this is a different issue than the one @wbokkers hit.  As Jevan mentioned earlier we've seen some issues like this with the existing ScrollViewer control and they can be elusive to track down.  If you can get a VM with a repro using as recent version of the OS as you can find then let @jevansaks know.

@micahl is there a plan to ship the fix of this issue as an update for Windows Versions prior to 1903? Our Company is currently mostly on 1809 and the responsible staff has no intention to update to 1903 in the near future they might skip this and update next year to 1909. But our users can't wait that long for a fix for this issue.

@DerGary, no promises, but I have started an thread internally to raise the issue.

@micahl thank you very much, that is very kind of you. In the meantime (maybe I should have asked that first) is there a known workaround that I can build inside the app to prevent this issue from happening in the first place?

@wbokkers, you did something to avoid hitting this issue, right?  What was it?  @DerGary, unfortunately I'm not aware of a guaranteed workaround.

@michahl, no I couldn't avoid hitting the issue, as far as I can remember. I can't reproduce the issue now, and there are no user complaints about it either.

@DerGary, what is your company?  I am preparing to bring the fix to Windows 10 1809 per your request, some information about your company and the product would help to make the business case.  If you have a support agreement with Microsoft, we could even send you a private update for you to verify ahead of time. Thanks!

I'm working for the s.Oliver Bernd Freier GmbH & Co. KG. We are employing about 7000 people worldwide. We have 2 UWP Apps that are used each by around 300 employees. We have usage of around 150-180 unique users per day per app. The Users depend on these apps because they have no other way of doing there work currently. And yes we do have a support agreement with Microsoft.

Pay attention this person just signed up on gh, it looks like a phishing attempt. And mentions "1809" an old release of Windows.

@adrientetar Reading up on the latest comments, I'm not sure this is a "phishing attempt". @DerGary is explicitly asking for a 1809 backporting of the fix and @micahl did say a couple days ago he was starting a thread internally about backporting it. Since github is also a relatively new way of interacting with customers for certain parts of MS, the sign-up date also doesn't have to necessarily be a suspicious sign.
Perhaps someone can clarify? @jevansaks @Ambigulous @micahl

I was also sceptic about the signup date. But I did not provide any confidential information and I also won't do that in a public forum. But thanks for your concerns @adrientetar and @Felix-Dev. I would also be happy if @micahl would clarify that account.

I was giving @ambigulous the benefit of the doubt as well but their account isn't associated with the Microsoft org on GitHub so now I'm suspicious too.  @ambigulous can you email me so we can get this sorted out?

@DerGary, i've reached out to the folks with whom I've been communicating and will update once I know.  fyi, we do have an internal issue tracking the request to backport to 1809 that is going through review.

I am properly linked now.  Sorry for causing doubt, but it is good to see the high level of caution  :)  Yes 1809 is an old version of Windows but it is still supported.  That is the work I am involved in, providing security/functionality/reliability updates to in-market versions of Windows, via Windows Updates.

Thanks @Ambigulous!!

@Ambigulous are you sure you are properly linked? Github does not show me that you are a member of the microsoft organization?

One thing with GitHub is organization memberships aren't public by default, you have to go to that organization page and hit the lock icon next to your alias in the members list.

It takes a little while for that card in the activity feed to show up, but I confirmed on the internal site for linking accounts that @Ambigulous is a Microsoft employee helping out with servicing this fix. :)

Yep, we're all good.  Thanks everyone!

@Ambigulous was the information I provided sufficient for your business case?

@DerGary, yes it helped a lot, thank you!
We have already made the fix to Windows 10 1809, and is being tested. It should become available in the September 17th "Preview of Monthly Rollup" optional update.

@Ambigulous can you provide the KB number of this update? I'm unable to find it.

Thanks for the ping! Looks like it's ready to go but got delayed a couple days. My sources tell me "KB4516077 will be live around 10:00 AM tomorrow".


Comment in beggining of SQL statement returns empty recordSet. In clickhouse-client works OK.

When we provide Sql statement with comment in clickhouse-client it works. - no matter if comment is at the  begining or at the end of sql statement.
docker run --rm yandex/clickhouse-client -h ch8.local -q "select * from system.numbers limit 3 /*comment*/"
0
1
2
uporabnik@lju1135:~$ ^C
uporabnik@lju1135:~$ docker run --rm yandex/clickhouse-client -h ch8.local -q "/*comment*/select * from system.numbers limit 3"
0
1
2

When running clickhouse-jdbc and we use comment at the beggining it FAILS
  @Test
    public void clickhouseJdbcFailsBecauseOfCommentInStart() throws Exception {

        Connection connection = DriverManager.getConnection("jdbc:clickhouse://ch8:8123/test4ssp?dataTransferTimeout=600&connection_timeout=5000");
        String sqlStatement = "/*comment*/ select * from system.numbers limit 3";

        try (Statement statement2 = connection.createStatement()) {
            try (ResultSet resultSet2 = statement2.executeQuery(sqlStatement)) {
                if (resultSet2 == null) {
                    Assert.assertTrue(false);
                    throw new Exception("resultset was null");
                } else {
                    while (resultSet2.next()) {
                        System.out.println(resultSet2.getString(1));
                    }
                    Assert.assertTrue(true);
                }
            }
        }
    }

and if we supply comment at the end of statement if works
 @Test
    public void clickhouseJdbcPassOkBecauseOfNoCommentInStart() throws Exception {

        Connection connection = DriverManager.getConnection("jdbc:clickhouse://ch8/test4ssp?dataTransferTimeout=600&connection_timeout=5000");
        String sqlStatement = "select * from system.numbers limit 3 /*comment*/";

        try (Statement statement2 = connection.createStatement()) {
            try (ResultSet resultSet2 = statement2.executeQuery(sqlStatement)) {
                if (resultSet2 == null) {
                    Assert.assertTrue(false);
                    throw new Exception("resultset was null");
                } else {
                    while (resultSet2.next()) {
                        System.out.println(resultSet2.getString(1));
                    }
                    Assert.assertTrue(true);
                }
            }
        }
    }

Why is this a problem?  We normally add some small comments above sql statements in order to describe statement.

@josepowera can you please try with a newer driver version? The issue should (hopefully) be fixed by a recent change.

looks ok.  tested with 0.1.46


BrowserSession() NotImplementedError on Windows

On Windows 10, Python 3.10.1:
import hrequests
page = hrequests.BrowserSession()
Results in the follow exception:
Task exception was never retrieved
future: <Task finished name='Task-7' coro=<Connection.run() done, defined at <redacted>\hrequests\venv\lib\site-packages\playwright\_impl\_connection.py:264> exception=NotImplementedError()>
Traceback (most recent call last):
  File "<redacted>\hrequests\venv\lib\site-packages\playwright\_impl\_connection.py", line 271, in run
    await self._transport.connect()
  File "<redacted>\hrequests\venv\lib\site-packages\playwright\_impl\_transport.py", line 127, in connect
    raise exc
  File "<redacted>\hrequests\venv\lib\site-packages\playwright\_impl\_transport.py", line 116, in connect
    self._proc = await asyncio.create_subprocess_exec(
  File "C:\Users\<redacted>\AppData\Local\Programs\Python\Python310\lib\asyncio\subprocess.py", line 218, in create_subprocess_exec
    transport, protocol = await loop.subprocess_exec(
  File "C:\Users\<redacted>\AppData\Local\Programs\Python\Python310\lib\asyncio\base_events.py", line 1652, in subprocess_exec
    transport = await self._make_subprocess_transport(
  File "C:\Users\<redacted>\AppData\Local\Programs\Python\Python310\lib\asyncio\base_events.py", line 493, in _make_subprocess_transport
    raise NotImplementedError
NotImplementedError
Exception in thread Thread-5 (spawn_main):
Traceback (most recent call last):
  File "C:\Users\<redacted>\AppData\Local\Programs\Python\Python310\lib\threading.py", line 1009, in _bootstrap_inner
    self.run()
  File "C:\Users\<redacted>\AppData\Local\Programs\Python\Python310\lib\threading.py", line 946, in run
    self._target(*self._args, **self._kwargs)
  File "<redacted>\hrequests\venv\lib\site-packages\hrequests\browser.py", line 115, in spawn_main
    asyncio.new_event_loop().run_until_complete(self.main())
  File "C:\Users\<redacted>\AppData\Local\Programs\Python\Python310\lib\asyncio\base_events.py", line 641, in run_until_complete
    return future.result()
  File "<redacted>\hrequests\venv\lib\site-packages\hrequests\browser.py", line 119, in main
    self.client = await hrequests.PlaywrightMock(
  File "<redacted>\hrequests\venv\lib\site-packages\async_class.py", line 173, in __await__
    yield from self.create_task(
  File "<redacted>\hrequests\venv\lib\site-packages\hrequests\playwright_mock\playwright_mock.py", line 19, in __ainit__
    self.playwright = await async_playwright().start()
  File "<redacted>\hrequests\venv\lib\site-packages\playwright\async_api\_context_manager.py", line 52, in start
    return await self.__aenter__()
  File "<redacted>\hrequests\venv\lib\site-packages\playwright\async_api\_context_manager.py", line 47, in __aenter__
    playwright = AsyncPlaywright(next(iter(done)).result())
  File "<redacted>\hrequests\venv\lib\site-packages\playwright\_impl\_connection.py", line 271, in run
    await self._transport.connect()
  File "<redacted>\hrequests\venv\lib\site-packages\playwright\_impl\_transport.py", line 127, in connect
    raise exc
  File "<redacted>\hrequests\venv\lib\site-packages\playwright\_impl\_transport.py", line 116, in connect
    self._proc = await asyncio.create_subprocess_exec(
  File "C:\Users\<redacted>\AppData\Local\Programs\Python\Python310\lib\asyncio\subprocess.py", line 218, in create_subprocess_exec
    transport, protocol = await loop.subprocess_exec(
  File "C:\Users\<redacted>\AppData\Local\Programs\Python\Python310\lib\asyncio\base_events.py", line 1652, in subprocess_exec
    transport = await self._make_subprocess_transport(
  File "C:\Users\<redacted>\AppData\Local\Programs\Python\Python310\lib\asyncio\base_events.py", line 493, in _make_subprocess_transport
    raise NotImplementedError
NotImplementedError

Hello,
This seems to be an issue with Playwright's async API.
By any chance, are you running this in a Jupyter notebook? I was able to reproduce this error in there. Unfortunately, it doesn't look like Jupyter support can be fixed. (See here)

Ok. Yes I was running this from a notebook.


Change setenv to set -gx

Change to make swiftenv work with fish 2.6.
See:

https://github.com/pyenv/pyenv-virtualenv/issues/216
https://github.com/pyenv/pyenv-virtualenv/pull/218

Thanks @herrbischoff


Document adding one-off custom CSS through the css prop

I had to dive into the documentation of styled-system's system-components package to spot a little note about the css prop. I think that this behavior should be documented more explicitly on the "Extending" page of the docs.

This has been done in https://github.com/jxnblk/rebass/commit/717f0968b69d9ba9bc339cb737269373e243303d. Thanks!


cannot push to children after creating empty array

i called children.push after creating an empty array for children and then get the follow message.
unless children is defined and then i push then it will work.
using latest version 1.2.3
core.es5.js?de3d:1020 ERROR TypeError: Cannot read property 'push' of undefined

what if user wanted to add children later? there should be a boolean value or something in the config that allows for this. this is for dynamic population.

I am having this same issue. Is there a solution?

I was able to work around the issue by initializing the children array with a dummy value {text: '', value: ''} then splicing it out before pushing in the desired children

Hello, was this problem resolved? What @pcvogler suggested works but is more of a workaround than an actual solution.

exact same problem here. @leovo2708

same problem here, any updates please !


Wrap file for jsoncpp

Hey guys! I created a wrap file for jsoncpp lib, which already supports Meson.
Following the instructions, I also created a repository: https://github.com/koponomarenko/jsoncpp . How can I transfer it to mesonbuild?

[*] project() has version string
[*] project() has license string
[*] if new project, master has tagged commit as only commit
[*] if new branch, it is branched from master
[*] contains a readme.txt
[*] contains an upstream.wrap file
[*] download link points to authoritative upstream location
[ ] wrap repository contains only build system files
[ ] merge request is pointed to correct target branch (not master)
[*] wrap works
[*] repo does not have useless top level directory (i.e. libfoobar-1.0.0)

https://github.com/mesonbuild/jsoncpp is up, please create a PR targeting branch 1.8.4.


Removing all columns from matrix DDS - column remove events mismatched with DDS data state

Describe the bug

We use fluid internally in an SDK and do not expose it to our clients.
Performing this call on the matrix DDS:
matrix.removeCols(0, matrix.colCount);

will result in arbitrary number of "column removal" events from the framework with no indication of when the events are done firing.
This makes it difficult to track and to communicate a "data structure empty" event when using fluid internally and it not exposed to clients.
Also, when querying the matrix DDS on "column removed" events, it seems that the DDS is already empty and contains no data, and thus the event information and the state of the DDS is then mismatched. This adds to the complexity in knowing when the last "column removed" event has been sent and to reliably send out a "data structure empty" event. It also provides no way of acting on the events when state of the DDS is to be used in business logic due to the mismatch.
Expected behavior

Either:

match DDS state with "column removed" event, but this would not be async, or
for our specific use-case we need to clear the whole matrix DDS, thus a matrix.clear() would work and then a single event that signifies the all the columns that were removed.

Behind the scenes, SharedMatrix uses MergeTree to track the mapping between logical row/cols and their current locations in storage.  Modifying the rows/cols manipulates the segments in the MergeTree, which generates MergeTree 'delta' events.  The SharedMatrix maps these events to the current logical rows/cols when raising the SharedMatrix callbacks.
What the customer is observing is that removing a consecutive range of items in the MergeTree can generate effect multiple MergeTree segments, each of which is generating a separate notification as we map removed segments to their logical row/col positions.  (See 'permutationVector.ts:onDelta()')
The segment ranges are already sorted by descending position to help avoid the need to deal with shifting indices when reacting to notifications.  Consider also coalescing adjacent segments prior to invoking callbacks.

Any further positioning on this issue? Will there be a fix for this forthcoming?


Create 876-Middle-of-the-Linked-List.cs

File(s) Modified: 876-Middle-of-the-Linked-List.cs
Language(s) Used: csharp
Submission URL: https://leetcode.com/submissions/detail/789755830/

Thanks, @sulaimanhayek!


fix(api): add missing build step and use faster OS

fix: include built curriculum in image
perf: use debian instead of alpine

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

I don't know what the problem with alpine was, but I noticed that it would take roughly 90ms to handle /update-my-keyboard-shortcuts while the debian-based image was closer to 50ms.
For completeness, I checked the api-server and it was around the 50ms mark, too.

Thanks Oliver, it must have taken a while to debug this.

To be honest, I got lucky. It was the third thing I tried :)


Nuget error attempting to query latest version of SSH.NET

Using lpless on script that uses SSH.NET from nuget, I get an error:
Packages directory: N:\LINQPad Queries\packages
Packages target: .NETFramework,Version=v4.5.2
Querying latest version of SSH.NET
Unable to load the service index for source https://api.nuget.org/v3/index.json.
Could this be because I need to configure a Proxy?

Could this be because I need to configure a Proxy?

Does your environment generally need a proxy configured for Internet access? Does this error occur no matter which package you try to reference? That is, it's not specific to referencing SSH.NET?

Closing due to no follow-up.


Jcenter shut down causes problems on the module

Today we have started to see this issue :
':react-native-haptic-feedback'.
> Could not resolve all artifacts for configuration ':react-native-haptic-feedback:classpath'.
   > Could not resolve com.android.tools.build:builder:3.0.0.
     Required by:
         project :react-native-haptic-feedback > com.android.tools.build:gradle:3.0.0 > com.android.tools.build:gradle-core:3.0.0
      > Could not resolve com.android.tools.build:builder:3.0.0.
         > Could not get resource 'https://jcenter.bintray.com/com/android/tools/build/builder/3.0.0/builder-3.0.0.pom'.
            > Could not HEAD 'https://jcenter.bintray.com/com/android/tools/build/builder/3.0.0/builder-3.0.0.pom'.
               > Read timed out
   > Could not resolve org.eclipse.jdt.core.compiler:ecj:4.6.1.
     Required by:
         project :react-native-haptic-feedback > com.android.tools.build:gradle:3.0.0 > com.android.tools.build:gradle-core:3.0.0 > com.android.tools.lint:lint:26.0.0
      > Skipped due to earlier error
   > Could not resolve com.google.code.gson:gson:2.3.
     Required by:
         project :react-native-haptic-feedback > com.android.tools.build:gradle:3.0.0 > com.android.tools.build:gradle-core:3.0.0 > com.android.tools.lint:lint:26.0.0 > com.android.tools.build:manifest-merger:26.0.0
         project :react-native-haptic-feedback > com.android.tools.build:gradle:3.0.0 > com.android.tools.build:gradle-core:3.0.0 > com.android.tools.lint:lint:26.0.0 > com.android.tools.build:manifest-merger:26.0.0 > com.android.tools:sdklib:26.0.0
      > Skipped due to earlier error
   > Could not resolve com.google.errorprone:error_prone_annotations:2.0.18.
     Required by:
         project :react-native-haptic-feedback > com.android.tools.build:gradle:3.0.0 > com.android.tools.build:gradle-core:3.0.0 > com.android.tools.build:gradle-api:3.0.0 > com.google.guava:guava:22.0
      > Skipped due to earlier error
   > Could not resolve org.jetbrains.kotlin:kotlin-reflect:1.1.3-2.
     Required by:
         project :react-native-haptic-feedback > com.android.tools.build:gradle:3.0.0 > com.android.tools.build:gradle-core:3.0.0 > com.android.tools.lint:lint:26.0.0 > com.android.tools.lint:lint-checks:26.0.0 > com.android.tools.lint:lint-api:26.0.0
      > Skipped due to earlier error

Is there any workaround on this?
Thank you in advance!

I stumbled upon this issue as well lately. It seems like this line is causing the error.
Replacing jcenter() with mavenCentral() worked for me locally.

Currently the only way is either using this or waiting for jcenter to come online again. However, jcenter will be shut down by February 2022 anyways so it might be a good idea to replace it already.

Thank you very much! I saw that you have already opened a PR. I have the same feeling with you to replace it already. Are you going to cut a new release to include the mavenCentral()?

I guess it's on the Organisation to merge this pull request. For now I've patched my app like this:
`diff --git a/node_modules/react-native-haptic-feedback/android/build.gradle b/node_modules/react-native-haptic-feedback/android/build.gradle
index c326a26..198fb2a 100644
--- a/node_modules/react-native-haptic-feedback/android/build.gradle
+++ b/node_modules/react-native-haptic-feedback/android/build.gradle
@@ -4,7 +4,7 @@ def DEFAULT_TARGET_SDK_VERSION              = 27
buildscript {
repositories {

   jcenter()

   mavenCentral()
   google()

}

`
using https://www.npmjs.com/package/patch-package


update old release branch from new one <IP_ADDRESS> 🚧

for reference only. No need to merge.
Branch <IP_ADDRESS> is to replace this old 0.19.0.dev8, and is ahead of it

old release ".dev8" branch deleted


Improve console messages on add and update config files

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
❌

Fixed issues
#13 #19

Example:
Config files has been added:
 - config/packages/test/k/config/params.php
 - config/packages/test/k/config/common.php

Config files has been changed:
 - config/packages/test/a/config/params.php
 - config/packages/test/a/config/web.php
Please review files above and change it according with .dist files.

Message about new config files output also on first install.

👍


Game design overview

Use comments below to suggest ideas and stuff
For more specific rubbish, open a new issue

Database use:

High scores
Store settings

@Squibblet space

Text based adventure semester 2 project.docx

<EMAIL_ADDRESS>


[FEATURE REQUEST] Backup/Restore - Firebird 3 -SKIP(_DATA) | Firebird 4 -INCLUDE(_DATA)/-SKIP(_DATA)

Hello, it would be nice if these options for backup/restore could also be integrated into Flamerobin.
Since Firebird 3
https://firebirdsql.org/file/documentation/release_notes/html/en/3_0/bk02ch11s03.html#d0e19573
Since Firebird 4
https://firebirdsql.org/file/documentation/release_notes/html/en/4_0/rlsnotes40.html#rnfb40-util-gbak-include-data

Hi @nLeonardt95
FlameRobin already have options to Backup/Restore databases.
Please let us know if it is enough for you.
BACKUP

RESTORE

Yes, I know that, but I mean the Skip_data and include_data option.

Hi @nLeonardt95 ,
This is part of the FB3 / FB4 updates, please be patient.
./jo


[ONNX] Add ReduceSum opset13 support (non-dynamic)

Please review @AndrewZhaoLuo

@AndrewZhaoLuo All tests passed. Please merge.

                # Get axis and unpack scalar
                constant_axis = int(inputs[1].data.numpy()[0])

https://github.com/apache/tvm/blame/b3a5e18f6f096c2828512ae56f2e40261350c8b3/python/tvm/relay/frontend/onnx.py#L3164
It seems this line does not check inputs[1].data.numpy() has more than one element. for example of ReduceSum:


[Incident] uwhackweeks hub not spawning due to AWS quota limits

Summary
https://2i2c.freshdesk.com/a/tickets/109 reports that users can't spin up new nodes, even the smallest. This is due to the AWS account running out of quota.
Impact on users
The hub is unusable for new users.
Important information

Hub URL: https://uwhackweeks.2i2c.cloud/
Support ticket ref: https://2i2c.freshdesk.com/a/tickets/109

Tasks and updates

[x] Discuss and address incident, leaving comments below with updates
[x] Incident has been dealt with or is over
[ ] Copy/paste the after-action report below and fill in relevant sections
[ ] Incident title is discoverable and accurate
[ ] All actionable items in report have linked GitHub Issues

After-action report template
# After-action report

These sections should be filled out once we've resolved the incident and know what happened.
They should focus on the knowledge we've gained and any improvements we should take.

## Timeline

_A short list of dates / times and major updates, with links to relevant comments in the issue for more context._

All times in {{ most convenient timezone}}.

- {{ yyyy-mm-dd }} - [Summary of first update](link to comment)
- {{ yyyy-mm-dd }} - [Summary of another update](link to comment)
- {{ yyyy-mm-dd }} - [Summary of final update](link to comment)

## What went wrong

_Things that could have gone better. Ideally these should result in concrete
action items that have GitHub issues created for them and linked to under
Action items._

- Thing one
- Thing two

## Where we got lucky

_These are good things that happened to us but not because we had planned for them._

- Thing one
- Thing two

## Follow-up actions

_Every action item should have a GitHub issue (even a small skeleton of one) attached to it, so these do not get forgotten. These issues don't have to be in `infrastructure/`, they can be in other repositories._

### Process improvements

1. {{ summary }} [link to github issue]
2. {{ summary }} [link to github issue]

### Documentation improvements

1. {{ summary }} [link to github issue]
2. {{ summary }} [link to github issue]

### Technical improvements

1. {{ summary }} [link to github issue]
2. {{ summary }} [link to github issue]

https://us-east-1.console.aws.amazon.com/support/home?region=us-east-1#/case/?displayId=9786816411&language=en is the case where I've asked for a bump in the quota.

It sounds like the next step here is just to wait for AWS to respond. But for a long-term question, is this something we'll need to do for every new AWS account? It seems like quotas are account- and region-specific so I guess we should be prepared to run into this issue the first time a new account gets a bump in usage?

https://us-east-1.console.aws.amazon.com/support/home?region=us-east-1#/case/?displayId=9786816411&language=en is the case where I've asked for a bump in the quota.

Not sure if the region in that URL matters, but just wanted to flag that we're running in us-west-2.

is this something we'll need to do for every new AWS account?

I definitely had to do this for the Pangeo AWS account a number of years back, but it appears AWS has since moved from per-instance-type limits to vCPU total limits:
https://aws.amazon.com/ec2/faqs/#EC2_On-Demand_Instance_limits

On-Demand Instance Limit Name
Default vCPU Limit

Running On-Demand Standard (A, C, D, H, I, M, R, T, Z) instances
1152 vCPUs

Which clearly fine for events where everyone requests a machine with 2vCPU? I'm just surprised we got to this limit so quickly... unless there is some other service quota at play

Not sure if the region in that URL matters, but just wanted to flag that we're running in us-west-2.

Good catch! It looks like the support is hosted in us-east-1 regardless. I explicitly specified we want us-west-2 in the body of the ticket:

Apparently the limit we had was... 32!? 1152 would have been plenty fine.

They bumped up the limit to 1024 and we're all good now!

I think that this one is now resolved - I am not sure if there's anything we can do as a follow-up improvement on this. Maybe we could document that a quota bump will be commonly-needed for AWS quickly? But I suspect that we would have ran into this problem either way. @yuvipanda is there anything you had in mind?

@choldgraf I think we should document 'check quotas' as a step before any event, and also after creating a new AWS account. We can close this after that. Not sure where that documentation should go though.

@yuvipanda how about this:

https://github.com/2i2c-org/infrastructure/pull/1149

Closing as it seems resolved with related docs updated, and uwhackweeks is removed!

@choldgraf you removed "type: hub incident" and it made me think it didn't merit being documented in https://github.com/2i2c-org/incident-reports/tree/main/reports, but I figure it should be.
Action point

[ ] Add an entry about this in https://github.com/2i2c-org/incident-reports/tree/main/reports.


add nds caching for gaedatastore Get, Put operations

@alainbrown Thanks for trying this caching library with Cayley. Do you have any benchmarks to see how it affects performance?
Also, please remove docker from glide.lock. We intentionally removed it to save time for CI builds.

Thanks @dennwc I'll go ahead and made the changes you mentioned to the glide.lock file.
As far as benchmarks, do you have any suggestions for capturing that? Or maybe an example of something folks might find interesting here?
For my purpose, it is mainly for cost saving as google datastore charges on a per operation basis but the memcached layer operations are free. Each query on my graph touch hundreds of nodes, but since a lot of them are the same nodes and quads again and again, we get a huge cost savings.

@alainbrown Results of integration tests will be useful. Just want to make sure nothing gone wrong with a new library.


Cloud Integration Appearing even though it is not configured

**Home Assistant release with the issue: ** 0.100.2
**Last working Home Assistant release (if known): ** Not sure, just noticed user
**Operating environment (Hass.io/Docker/Windows/etc.): ** Docker
Integration:
cloud:
Description of problem:
I do not have default_config or cloud: in my configuration.yaml file but yet cloud integration is activated and added a system administrator for Home Assistant Cloud User to my system.  I cannot delete this user.   I'm a little worried having a user that I cannot delete in the environment.  I also see the Cloud free trial options showing up under the configuration and am not sure if this should be hidden when not calling cloud: integration.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
[https://github.com/mcaminiti/homeassistant/blob/master/configuration.yaml](https://github.com/mcaminiti/homeassistant/blob/master/configuration.yaml
)```

**Traceback (if applicable):**

**Additional information:**

It seems like this is related to the ios beta app.
After adding mobile_app to my config, i see this too in integrations and users.

Yes, mobile_app has cloud as dependency:
https://github.com/home-assistant/home-assistant/blob/dev/homeassistant/components/mobile_app/manifest.json#L10

So the mobile app requires me to authenticate and save the token for my user. Why would it require a hard coded user that I cannot delete then? I’m concerned about who has access tot his admin account forced into HA without my ability to remove it.

I am as well not sure of this required user for mobile_app and would like to see some clarity place on it.

@robbiet480,
I just ran into this as well after updating to the new HA companion app at it was launched on the App store. Why is cloud required for mobile_app?

I've opened a PR to no longer create a cloud user if cloud is not in use #29150

Will this allow the deletion of the user if not using cloud as well? Currently I cannot delete the user.

It will be deleted when you log out of cloud and not created for new users.

If I’m not logged in to cloud and it already exists though how will I go about deleting it?


chore(gitignore): fix coverage files not being ignored properly

:tada: This PR is included in version 1.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Panel sizes are not restored when terminal is closed.

When I close the terminal after running vim-test, panels are horizontally equalized.
This is annoying when you have set the size manually.
The horizontal size should be stored before opening the terminal and restored after its close.

Ah, I've noticed that some time before, but completely forgot about it. This seems to be the product of botright new command, after you close the opened window things don't go back to normal. An alternative is to use belowright new (or probably just new), which has behaviour that it opens a terminal right below the current buffer, with the same width as the current buffer (as opposed to botright which opens the window on the bottom in full-width).
Can you try typing in the following and let me know if this is what you want:
:belowright new | call termopen('your test command') | startinsert

I think that's a better default behaviour, since it seems to bring windows back to original.

Indeed, it brings windows back to original.
I liked botright too, is it possible to add an option to configure which behaviour to use ?
But most important: is it a bug from vim that should be reported upstream?

I believe so. So, the issue is that when you close a window opened with botright, all windows that are in vertical line of where your cursor is currently will properly get resized back, while other windows won't. So if you're inside the terminal window opened by botright, Vim considers that you're in the "left lane", and after closing it you will be put on the nearest left-most buffer (Test.vim overrides that to put you back in the alternate buffer instead).
It's strange that I couldn't find people complaining about this behaviour on the Internet. I expected that I will find people having problem with it on Neoterm, but no luck.
However, I realized that FZF is doing the same thing, opening the Neovim terminal on the very bottom full-width, and closing the terminal keeps the windows intact. The difference that I noticed is that when FZF opens the Terminal window, no other windows move.
While writing this I figured that if you give botright an explicit window height, e.g. :botright20 new, it will have the desired behaviour. Now I just need to figure out what should be the default value. The author of FZF managed to have it as a percentage, and he did some calculations.
Btw, have you tried out Neoterm with Test.vim (by setting let test#strategy = 'neoterm')? It is very configurable, and among other things lets you set this buffer height, which then has desired behaviour (Neoterm also uses botright). In https://github.com/janko-m/vim-test/issues/121 I basically discuss whether the neovim strategy should be deprecated over Neoterm, because I don't want to start introducing strategy-specific configuration in Test.vim.

After testing FZF again, I managed to reproduce the same resizing bug, botright<n> still messes up window placement, so this behaviour is definitely is something that should be reported to the Vim project.
I don't know what should now be the course of action for test.vim, though.

Closing as a duplicate of https://github.com/janko-m/vim-test/issues/146.


S01E06 the power of F/OSS mark and brand

Why we must care about brand (logo ,mark) ~
Why open license cat not protect brand ?
what can we do ?

recording video and audio:
https://zoom.us/rec/play/hmlUMh0vU7HljDPV_P4Pc5Ici3uKDWYDHa-LBiqnTg4b_KLtehmY4gBHb0khujEzSM4Fu6IqCwa4QRct.Wq1VS0L1kCsr8Hsa


default value for enum in proto2 not correct

In the protobuf 2 language spec reads the following:
" For enums, the default value is the first value listed in the enum's type definition."
if I do something like:
enum MyPayload {
test1 = 5;
test2 = 7;
}
message MyTest {
required MyPayload payload = 1;
}
the resulting initializer end up looking like:
#define MyTest_init_default                      {(MyPayload)0}
which is incorrect
The workaround would be to explicitly define a default like:
message MyTest {
required MyPayload payload = 1 [default = test2];
}
which does seem to generate the init correctly.
I realize in protobuf3 language spec this statement is also present but they have an additional requirement that all enums must start at zero. in which case it would be ok except you would also have to remove the "required" from the message definition.
But never the less the protobuf2 init for enums is not correct.

Indeed, fortunately it should be quite simple fix in the generator.

looking forward to a fix

It will probably be a while until I'll have time to get to it.

I was using this as my test file (defaultTest.proto):
syntax = "proto2";
enum MyPayload {
test1 = 5;
test2 = 7;
}
message MyTest1 {
required MyPayload payload = 1;
}
message MyTest2 {
required MyPayload payload = 1 [default = test2];
}
I looked into this and I see if I change the following in the get_initializer function at approx. line 452:
from:
elif self.pbtype in ('ENUM', 'UENUM'):
inner_init = '(%s)0' % self.ctype
to:
elif self.pbtype in ('ENUM', 'UENUM'):
inner_init = '_%s_MIN' % self.ctype
This will result in the correct xxx_init_default and xxx_init_zero strings in the resulting defaultTest.pb.h file:
/* Enum definitions */
typedef enum _MyPayload {
MyPayload_test1 = 5,
MyPayload_test2 = 7
} MyPayload;
#define _MyPayload_MIN MyPayload_test1
#define _MyPayload_MAX MyPayload_test2
#define _MyPayload_ARRAYSIZE ((MyPayload)(MyPayload_test2+1))
...
/* Initializer values for message structs */
#define MyTest1_init_default                     {_MyPayload_MIN}
#define MyTest2_init_default                     {MyPayload_test2}
#define MyTest1_init_zero                        {_MyPayload_MIN}
#define MyTest2_init_zero                        {_MyPayload_MIN}
My guess this should be ok for proto2 and proto3 as in the proto3 case it will also resolve to zero if enum start at zero.
Does this look like a valid fix?

Hm, not sure what should happen with _zero initializer for an enum that does not have zero value. But I guess the fix makes sense.

@dvescovi1 Committed, thanks :)

Thanks for the check.
My guess the "xxx_init_zero" name is now slightly misleading as it seems "xxx_init" might make it more understandable.
In my test case I would use it like:
MyTest1 msg = MyTest1_init_default;
or
MyTest1 msg = MyTest1_init;    // instead of ... = MyTest1_init_zero
...minor detail, I can live with ether ;)


Bot stucks on date from the Past in periodic task start date

Describe the bug
Bot stucks when I try to set date from the past in periodic task and does not respond in any way until you start the dialogue again using the /start command.
Reproduction

Start creating periodic task.
Send date from the past.
Try to click buttons, or send messages - it will not respond.
Send /start command - and it works.

Screenshots / Additional context
No response

I cannot reproduce the bug. Is there still a problem? @dantetemplar

I cannot reproduce the bug. Is there still a problem? @dantetemplar

I guess there is no problem now.


Introduction of thanks job

Fixed bug: The lack of automation around the generation of thanks.json
What got changed here :

package.json

Introduction of kaku-thanks
Introduction of npm run thanks (Please execute npm i before)

TODO BEFORE MERGING:

@EragonJ  or others

As I don't want to break the master branch can you guys generate/push the package-lock.json according to my changes? Indeed as my node -v == v8.5.0 and npm -v == 5.4.2, I don't want to break the dependencies and others around master ...

Note
It (npm run thanks) should work like a charm if you are a maintainer of the Transifex repository but if there's an issue after tests let me know :smile_cat:

@funilrys it seems that for MacOSX,  there is no /bin/env?
sh: ./node_modules/kaku-thanks/thanks.py: /bin/env: bad interpreter: No such file or directory

some other errors:
SyntaxError: Non-ASCII character '\xe2' in file ./node_modules/kaku-thanks/thanks.py on line 92, but no encoding declared; see http://python.org/dev/peps/pep-0263/ for details

  File "./node_modules/kaku-thanks/thanks.py", line 6, in <module>
    from fasternix_stratalorn import get as translators
ImportError: No module named fasternix_stratalorn

TBH Kaku is node.js based, all related dependencies are installed under the folder called node_moduels to make sure after installing, Kaku can easily be used and developed under the environment with node.js installed.
Is it possible to be a node script ?

Any idea is welcome BTW !

Is it possible to be a node script ?

@EragonJ I'm ready to rewrite my 3 python repository to nodejs but I simply can't find a fully implemented curl library, that's why I do prefer python or bash ...

Kaku is node.js based, all related dependencies are installed under the folder called node_moduels

thanks for letting me know ... Actually, the command is located under node_modules did you read the changes?

Is there any purpose that you need curl to do communication? Some lib with a simple interface to let you interact with API endpoints should be enough, no?
And why you close this PR, I am just asking for ideas instead of asking you to remove everything xD

I didn't remove everything :joy:

Yeah transifex communication is done with curl and github API too :joy_cat:

@funilrys right now the only problem is that this script is not fit for MacOSX users like me, did you check the error message above ?

@EragonJ I'm working on it actually I just forgot the link to execute-me-first :smile_cat:

It seems that for MacOSX, there is no /bin/env

Mac is so weird --"

@EragonJ https://github.com/funilrys/kaku-thanks/commit/02174a2fd56a6545aff5bca46a92c0343b48190d should fix your issue with bad interpreters :)
Please execute rm -rf ./node_modules/kaku-thanks && npm run install && npm run thanks to test :+1:

@funilrys
running install
running build
running build_py
running install_lib
creating /Library/Python/2.7/site-packages/obstructing_trio
error: could not create '/Library/Python/2.7/site-packages/obstructing_trio': Permission denied
./node_modules/kaku-thanks/execute_me_first: line 4: cd: ./node_modules/kaku-thanks/Fasternix-Stratalorn: No such file or directory
  File "./node_modules/kaku-thanks/thanks.py", line 92
SyntaxError: Non-ASCII character '\xe2' in file ./node_modules/kaku-thanks/thanks.py on line 92, but no encoding declared; see http://python.org/dev/peps/pep-0263/ for details

looks like there must be some utf8 issues in the script ?


Fix LSPs using on_new_config

This PR includes a few things:

Allow custom nix_pkg override via the config (when you want to quickly set up a new LSP without directly forking lazy-lsp).
Create a common_config to configure fields that are similar on each server (I was tired of forgetting to put capacities and on_attach every time I added a custom config for a server).
Fix LSPs that depends on the on_new_config to mutate the command line (this was the case of omnisharp, for example)
Fix omnisharp's dependencies / cmd
Add prisma to the list of supported LSPs.

PS: I could not run the script to update stats, I got the error  module 'serpent' not found:No LuaRocks module found for serpe nt

Thanks for the PR!
Just to give some context, my philosophy is that things should work out of the box as much as possible. As Justin Keyes would say additional configuration options add entropy.

* Allow custom `nix_pkg` override via the config (when you want to quickly set up a new LSP without directly forking lazy-lsp).

By not having this option it incentivizes people to report issues so we can fix it and improve the plugin for everyone. For example the lua server that you give as an example got fixed a few days ago: https://github.com/dundalek/lazy-lsp.nvim/issues/5

* Create a `common_config` to configure fields that are similar on each server (I was tired of forgetting to put `capacities` and `on_attach` every time I added a custom config for a server).

Coming from perspective to minimize necessary options again, do you think it would work for the use case if we relied on the existing default_config and merge custom configs on top of it?

* Fix LSPs that depends on the `on_new_config` to mutate the command line (this was the case of omnisharp, for example)
* Fix omnisharp's dependencies / cmd

Nice, I somehow missed that on_new_config existed.

* Add prisma to the list of supported LSPs.

Did you verify that this works? I would expect the package to show up in search. This might be a case that requires packaging it first.

PS: I could not run the script to update stats, I got the error module 'serpent' not found:No LuaRocks module found for serpe nt

Good catch, fixed in https://github.com/dundalek/lazy-lsp.nvim/commit/c0abcbc8b8a8b62eac821d02058aae885a93b72e

Next steps:

I think we can add the on_new_config and omnisharp changes, would you be up for cherry-picking those and open a new clean PR?
Also merging in default_config instead of introducing common_config would make a good change. That could be another PR, I like small focused PRs which are easier to review.

I have reservations about the other changes.

When I first started to use the plugin, I thought that the default_config was always merged with custom configs, so I am happy with this idea. I created a new config value instead because that is a breaking change.
I did test prisma and omnisharp (on unstable) and it works.
I totally understand the doubt of the new config option and agree with the idea of creating PRs to add new LSPs. I only created it because I know that I'll sometime open a file in an obscure language and want an LSP to go to work fast and won't want to fork/PR right away. I figured that it could be nice to have since it is not something hard to maintain but if you don't think that's a good idea I will remove it.
Thank you for the quick review, I will start the two PRs later in the day.

Superseded by #8 and #9


Browser contextmenu not showing anymore

The PR #7544 stops openeing the browser contextmenu. This is because we call for every contextmenu event preventDefault():
https://github.com/Leaflet/Leaflet/blob/a9d7c4fe7d56a109eeba7d5c05b6dad93feb13e7/src/map/Map.js#L1427-L1429
But according to the docs it should only prevented if there are listeners on the target:

Fired when the user pushes the right mouse button on the map, prevents default browser context menu from showing if there are listeners on this event. Also fired on mobile when the user holds a single touch for a second (also called long press).

https://leafletjs.com/reference.html#map-contextmenu
I think we should to keep the behavior "only preventing if there are listeners on the target"

But there is such code:
https://github.com/Leaflet/Leaflet/blob/a9d7c4fe7d56a109eeba7d5c05b6dad93feb13e7/src/map/Map.js#L1425
So it should work according docs.

Plunkr with newest master: https://plnkr.co/edit/odU7aOGfa59wufzT

Maybe Leaflet internal listeners

tapHold handler?

Ok, this is the issue source:
https://github.com/Leaflet/Leaflet/blob/a9d7c4fe7d56a109eeba7d5c05b6dad93feb13e7/src/map/Map.js#L1368-L1370