Fix docs for do_handshake/2

do_handshake/2 can return without generated packets. In such case we get an empty binary. It's important to mention about it in docs as sending empty packets can fail. Sender before sending generated packets should check if any packets have been generated.
Packets might not be generated when host working in server mode calls do_handshake/2 before receiving any remote packets.
An alternative would be to return from function with packets or without packets but this can introduce naming inconsistency e.g. we would have {:ok, packets} | {:need_packets} but {:ok, packets} in fact means we generated our packets and we are waiting for the remote ones.

@mickel8 if returning empty packets usually implies substantially different action than non-empty packets, then we should return a different value, for example just :ok, what do you think?

@mat-hek it could be better but I can't find appropriate naming. We can't return {:ok, packets} | :ok as it will generate two functions with the same name and different arity in C.


Fix Cannot access a disposed object crash in LruCache.SizeOf method

Fix crash reported here.

Thank you for your collaboration on this 👍


deselecting a Polygon2D does not disable the point UI / red outline

Godot version:
3.2.3.stable.mono
OS/device including version:
Windows 10
Issue description:
When creating and editing a Polygon2D, the edit handles & the red outline are visible.
however, when deselecting the polygon2D by clicking outisde of the object, the handles remain visible.
only when selecting another object, the handles disappear.
this makes the iteration of creating / editing multiple of these shapes slower, as is takes longer to see the final result (without outlines)
especiialy, since common drawing apps always hide the handles when deselecting an object, this behaviour is very unconventional and irritates people coming from other software.
Steps to reproduce:

create polygon2D object

start creating new path with the green icon

click the path, and close it

the object is selected

click outside of the path (or an empty spot in the scene tab) to deselect

-> outline and handles are still visible

select another object

-> handles will be hidden

note
one could guess that this would be affected by the "poly editor"/ "show previous outline" setting, but it has no impact:

This is a duplicate of another issue (related to all editor gizmo plugins in general, in both 2D and 3D) but I can't find it right now.

Duplicate of #35989


Feature request: Exclude domain or host header

The script lists all bindings in IIS. Since a cert issued to "mydomain.com" works as well for "www.mydomain.com" I would like to exclude "www.mydomain.com" from automatic fetch/renewal.
I would also like to exclude domains that have bindings but aren't live on the Internet and don't need certificates. Example:  "staging.mynewdomain.com." It has a binding and my hosts file points to it so it's recognized using a host header, but it's not live on the Internet.
Thank you.

This is already possible, either from the command line (--excludebindings) or through the UI when you create a certificate "with advanced options".


Fail to make the tool chain

[@localhost grumpy]$ make
build/src/grumpy/lib/itertools/module.go:5: can't find import: "grumpy/lib/sys"
make: *** [build/pkg/linux_amd64/grumpy/lib/itertools.a] Error 1

[z@localhost gium]$ cd ~/work/grumpy/
[z@localhost grumpy]$ ls -la build/src/grumpy/lib/sys
total 4
drwxrwxr-x.  2 z z   21 Jan  4 19:03 .
drwxrwxr-x. 41 z z 4096 Jan  4 19:03 ..
-rw-rw-r--.  1   z z  0 Jan  4 19:03 module.d

anything wrong @trotterdylan

Okay so the .go file didn't get generated. It may be because the dependencies are incorrect. Here's what I see in the deps file for itertools:
$ cat build/src/grumpy/lib/itertools/module.d
build/pkg/darwin_amd64/grumpy/lib/itertools.a: build/pkg/darwin_amd64/grumpy/lib/_collections.a
build/pkg/darwin_amd64/grumpy/lib/itertools.a: build/pkg/darwin_amd64/grumpy/lib/sys.a
What do you see in build/src/grumpy/lib/itertools/module.d?

It is empty file

What's your system Python? I suspect things will not work unless it's 2.7. Mine is:
$ python --version
Python 2.7.10

[z@localhost z]$ python -V
Python 2.7.5

Any python module I have to install? @trotterdylan

No, shouldn't need any Python modules besides what ships with the interpreter. It's possible there are differences in the modulefinder library between 2.7.5 and 2.7.10.
I'll see if I can reproduce the issue with 2.7.5, but I probably won't have a chance to get to it for a couple days. In the mean time, you could try installing 2.7.10+ to see if that helps.

Same issue here.
 $ cat build/src/grumpy/lib/itertools/module.d build/pkg/_/grumpy/lib/itertools.a: build/pkg/_/grumpy/lib/_collections.a build/pkg/_/grumpy/lib/itertools.a: build/pkg/_/grumpy/lib/sys.a
python --version
Python 2.7.13 :: Anaconda custom (x86_64)
Also tried with non anaconda python:
Python 2.7.11
Same error

@llazzaro Hmm it looks like a different issue because build/src/grumpy/lib/itertools/module.d is empty for yyh-graphsql.  What's the result of:
$ go env

Got this result.
$  go env
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/leonardolazzaro/go"
GORACE=""
GOROOT="/usr/local/go"
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/1f/pdmszzls4xs6px9gmf627ng40000gn/T/go-build055261285=/tmp/go-build -gno-record-gcc-switches -fno-common"
CXX="clang++"
CGO_ENABLED="1"

@llazzaro Would you mind pasting the output of this command run from the root of the repo:
$ python -m modulefinder -p $PWD/lib:$PWD/third_party/stdlib lib/itertools.py

Sure, here is the output
 $ python -m modulefinder -p $PWD/lib:$PWD/third_party/stdlib lib/itertools.py

  Name                      File
  ----                      ----
m __main__                  lib/itertools.py
m _collections              /Users/leonardolazzaro/workspace/grumpy/lib/_collections.py
m sys

Okay, that looks correct. Maybe the issue is on the awk part of things. Try:
$ python -m modulefinder -p $PWD/lib:$PWD/third_party/stdlib lib/itertools.py | awk '{if (($1 == "m" || $1 == "P") && $2 != "__main__" && $2 != "itertools") {gsub(/\./, "/", $2); print "build/pkg/darwin_amd64/grumpy/lib/itertools.a: build/pkg/darwin_amd64/grumpy/lib/" $2 ".a"}}'

Huh, that all looks right. That output should appear verbatim in build/src/grumpy/lib/itertools/module.d which specifies sys.a as a dependency of itertools.a so sys will be built first.
Something's broken in the dependency chain there. You could probably manually get around by build sys.a manually: make build/pkg/darwin_amd64/grumpy/lib/sys.a and then make build/pkg/darwin_amd64/grumpy/lib/itertools.a.

Sorry I haven't got anything more satisfying for you at the moment. I'll take a look at this again tomorrow.

I bumped into the same issue. Had to manually build a dozen of modules, thanks to suggestion by @trotterdylan. A Dockerfile would be really handy, can provide one once you resolve the dependency issue.

You could run make clean, to clean up build folder.

Any idea of this... ?
build/src/grumpy/lib/os/path_test/module.go:4: can't find import: "grumpy/lib/os

@hiemal you could config GOPATH link

@mskims Thanks but this is not the issue.  This is an issue of grumpy not Go

As suggested by S-YOU, it's possible that a make clean will help here.
Anybody who's having this problem that's willing, please delete the build/ subdir with rm -rf build and then paste the output of the following:
python -V
go version
make -v

Finally dump the output of make -d. I recommend putting it in pastebin or somewhere because it will be very large.

make clean works. ty

Thanks for letting me know. Do you happen to remember what steps lead you to encounter the issue in the first place? Did you have to reinstall Python or something?

@trotterdylan
$ rm -rf build
$ python -V
Python 2.7.12 :: Continuum Analytics, Inc.
$ go version
go version go1.7.4 darwin/amd64
$ make -v
GNU Make 3.81
Copyright (C) 2006  Free Software Foundation, Inc.
This is free software; see the source for copying conditions.
There is NO warranty; not even for MERCHANTABILITY or FITNESS FOR A
PARTICULAR PURPOSE.

This program built for i386-apple-darwin11.3.0
mike at gnash in ~/dev/grumpy on git:master * conda:2x
$ make -d > make-d.log
line 723: binary op not implemented: Pow
make: *** [build/src/grumpy/lib/collections/module.go] Error 2

I'd share the output of make -d but pastebin told me "You have exceeded the maximum paste size of 512 kilobytes per paste."

@trotterdylan my issue was that python 3 is my default interpreter, I did a make and then switched to python 2.
This was solved doing a make clean

Great. Thanks for the update. I'm going to add some checks to the Makefile to verify the correct Python and Go versions.

I'm going to close this in favor of https://github.com/google/grumpy/issues/21 which has a nicer explanation of the solution.

Still getting this in the form:
build/src/grumpy/lib/itertools/module.go:4: can't find import: "grumpy/lib/_collections"
make: *** [build/pkg/darwin_amd64/grumpy/lib/itertools.a] Error 1

This thread is very long with many possible routes, so I'm not sure how to proceed. The solution suggested by the OP of #21 seems obsolete since the makefile has changed.

Have you tried "make clean; make"?
On Fri, Feb 3, 2017 at 11:28 PM Chris Redford<EMAIL_ADDRESS>wrote:

Still getting this in the form:
build/src/grumpy/lib/itertools/module.go:4: can't find import: "grumpy/lib/_collections"
make: *** [build/pkg/darwin_amd64/grumpy/lib/itertools.a] Error 1
This thread is very long with many possible routes, so I'm not sure how to
proceed. The solution suggested by the OP of #21
https://github.com/google/grumpy/issues/21 seems obsolete since the
makefile has changed.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/google/grumpy/issues/24#issuecomment-277425776, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAHLe2cbAhCu4_wl5kCPXnWOBEKiy5LZks5rZCiLgaJpZM4LbNeR
.

@trotterdylan Several times, yes. No fix, unfortunately.
make after a make clean causes a huge succession of repeated lines like this:
IOError: [Errno 2] No such file or directory: 'grumpyTest/grumpy/third_party/stdlib:/Users/credford/Dropbox/Years/__2017/__Scripts/Professional/8'
Traceback (most recent call last):
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/runpy.py", line 162, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/runpy.py", line 72, in _run_code
    exec code in run_globals
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/modulefinder.py", line 671, in <module>
    mf = test()
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/modulefinder.py", line 663, in test
    mf.load_file(arg)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/modulefinder.py", line 119, in load_file
    with open(pathname, READ_MODE) as fp:
IOError: [Errno 2] No such file or directory: 'grumpyTest/grumpy/third_party/stdlib:/Users/credford/Dropbox/Years/__2017/__Scripts/Professional/8'
build/src/grumpy/lib/itertools/module.go:4: can't find import: "grumpy/lib/_collections"
make: *** [build/pkg/darwin_amd64/grumpy/lib/itertools.a] Error 1

The CWD contains a ':' AND is in Dropbox.  Could either of those be a problem?  Did you try make install?  Do you use homebrew?

@cclauss
Good spotting on the path. There wasn't a : in the path. Rather, there was a space. The code being run by the makefile was mutating the path and appended the end to the beginning.
The makefile was mutating the path from (with a space after the 8)
/Users/credford/Dropbox/Years/__2017/__Scripts/Professional/8 grumpyTest/grumpy/third_party/stdlib
to
grumpyTest/grumpy/third_party/stdlib:/Users/credford/Dropbox/Years/__2017/__Scripts/Professional/8
Changing the name of the offending folder from 8 grumpyTest to 8-grumpyTest relieved the error. You may want to make your makefile more robust for spaces in paths, though.
I've been running code, including many Go and Python programs, in Dropbox for a year with no issues. I have several git repositories within it and make several commits a day.
It looks like the make has been successful, so thank you. However, I would much rather manage this through homebrew, which I do use, yes. Is there a recipe? If so, could you tell me what it is?

The homebrew formula is here but it is not yet accepted by the homebrew crew until grumpy has a release.  However the formula works if you put it in place by hand and then do brew install grumpy --HEAD.

I was sort of aware that spaces in filenames would be problematic. Thanks for diagnosing this. @drgrib would you mind filing another issue specifically about that with some of the information gathered here?

@trotterdylan Sure thing.


Proposal: Add api to modify docker daemon labels

Hi guys,
We use swarm as orchestration tools. And we add labels on docker daemon to let swarm to do some specific scheduling, such as schedule redis container to the docker daemon which has ssd label.
But in some case, we need to change the docker daemon labels, such as we want add MapR-ability label to some daemon in the midnight, so the map reduce container could be scheduled to this daemon.
There's no API to change the daemon labels on the fly, and we can't afford to restart daemon. So we want add API to do that, such as:
Usage: docker labels [COMMAND] [OPTION]

Manage docker daemon labels:

list - show the docker daemon labels.
add - add new labels to docker daemon, docker labels add --label=[k=v]
remove - remove any exist labels from docker daemon, docker labels remove --label=[k]

Pretty good, I like this idea.

Would be great :) :+1:

@hacpai Let's do it.

@feiyang21687 Okay, I finished it.
https://github.com/docker/docker/pull/16583

ping @cpuguy83

As I mentioned in #16583, this feature is too narrow. We've had discussions about having a more general command to change configuration options in the daemon that could be in a better scope than this.
Thanks for opening the issue, it helps us notice those little details we need to nail down better. I'm going to close this issue since we won't implement it.

I would also appreciate a manner to add / remove labels from running docker daemons without having to restart all containers on the host where the daemon is running.

You can set labels in the daemon.json and reload the daemon, without restarting containers; see https://docs.docker.com/engine/reference/commandline/dockerd/#configuration-reloading


Update AutoRest C# version to 3.0.0-beta.20230714.2

Update AutoRest C# version to 3.0.0-beta.20230714.2

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.DataFactory


Errors and long queuing

I needed to create the variant lists for ll the CZI and groups and I set up 28 lists in rapid succession.  The last 8 seem to be stuck 5 in the queue and 3 are giving back errors.
The 5 stuck in the queue are:
https://genie.broadinstitute.org/variant-lists/d3d8d871-842d-47d5-8602-c88fbb55b183/
https://genie.broadinstitute.org/variant-lists/9f811c70-3f18-4557-adc3-dd6f4c2d34af/
https://genie.broadinstitute.org/variant-lists/82551a9f-3872-43a9-ad85-4dde00e82c83/
https://genie.broadinstitute.org/variant-lists/deba0245-af4f-46da-a58b-f0311eb9ce68/
https://genie.broadinstitute.org/variant-lists/466a984a-b7e4-4a0b-9cf6-7955faa80cf5/
The 3 with errors are:
https://genie.broadinstitute.org/variant-lists/8d727ca5-e593-4f5d-940b-e0166dac6808/
https://genie.broadinstitute.org/variant-lists/058e3b81-6722-4f36-9390-b615d74403d0/
https://genie.broadinstitute.org/variant-lists/9d91a313-fc8f-4fad-93c1-786f326e3ec7/
The error text returned is:
Error details
Traceback (most recent call last):
  File "/app/worker/src/worker/tasks.py", line 416, in process_variant_list
    _process_variant_list(variant_list)
  File "/app/worker/src/worker/tasks.py", line 353, in _process_variant_list
    gnomad = hl.read_table(
  File "<decorator-gen-1362>", line 2, in read_table
  File "/usr/local/lib/python3.9/site-packages/hail/typecheck/check.py", line 577, in wrapper
    return __original_func(*args_, **kwargs_)
  File "/usr/local/lib/python3.9/site-packages/hail/methods/impex.py", line 2576, in read_table
    ht = Table(ir.TableRead(tr, False))
  File "/usr/local/lib/python3.9/site-packages/hail/table.py", line 343, in __init__
    self._type = self._tir.typ
  File "/usr/local/lib/python3.9/site-packages/hail/ir/base_ir.py", line 339, in typ
    self._compute_type()
  File "/usr/local/lib/python3.9/site-packages/hail/ir/table_ir.py", line 248, in _compute_type
    self._type = Env.backend().table_type(self)
  File "/usr/local/lib/python3.9/site-packages/hail/backend/spark_backend.py", line 288, in table_type
    jir = self._to_java_table_ir(tir)
  File "/usr/local/lib/python3.9/site-packages/hail/backend/spark_backend.py", line 275, in _to_java_table_ir
    return self._to_java_ir(ir, self._parse_table_ir)
  File "/usr/local/lib/python3.9/site-packages/hail/backend/spark_backend.py", line 268, in _to_java_ir
    ir._jir = parse(r(ir), ir_map=r.jirs)
  File "/usr/local/lib/python3.9/site-packages/hail/backend/spark_backend.py", line 243, in _parse_table_ir
    return self._jbackend.parse_table_ir(code, ref_map, ir_map)
  File "/usr/local/lib/python3.9/site-packages/py4j/java_gateway.py", line 1304, in __call__
    return_value = get_return_value(
  File "/usr/local/lib/python3.9/site-packages/hail/backend/py4j_backend.py", line 29, in deco
    raise FatalError('%s\n\nJava stack trace:\n%s\n'
hail.utils.java.FatalError: OutOfMemoryError: Java heap space

Java stack trace:
java.lang.OutOfMemoryError: Java heap space
	at

Hail version: 0.2.85-9b98676b6ad8
Error summary: OutOfMemoryError: Java heap space

Closing this issue, as this has seemed to not be a problem. Will keep an eye on performance and re-open this issue / create a new issue if needed.


Only document platform-relevant dependencies

The recent change in #1939 ended up accidentally pulling in irrelevant
platform-specific dependencies into a normal cargo doc, causing those builds
to fail. This change tweaks the logic for calculating documentation dependencies
to account for filtering based on the relevant platform.

r? @wycats
(rust_highfive has picked a reviewer for you, use r? to override)

cc @impl

r? @huonw

@bors r+

:pushpin: Commit a3d056d has been approved by huonw

:hourglass: Testing commit a3d056d with merge 71f2a9f...

:sunny: Test successful - cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64


The default max tokens of 215 is too small, answers are often cut off.I will modify it to 512 to address this issue.

What problem does this PR solve?
Briefly describe what this PR aims to solve. Include background context that will help reviewers understand the purpose of the PR.
Type of change

[ ] Bug Fix (non-breaking change which fixes an issue)
[ ] New Feature (non-breaking change which adds functionality)
[ ] Documentation Update
[ ] Refactoring
[ ] Performance Improvement
[ ] Other (please describe):

I can relate and I think this makes sense! @KevinHuSh


Final loss value

Hi @yanwei-li, first thanks for this great work ! I use this code for segmentation training on CITYSCAPES. I'm using the Dynamic-Raw 16 layers with ImageNet and after  60K steps the loss is already between 0.1 and 0.06, and seems to hardly decrease. So is it normal or is it decreasing too fast ? What should be the final loss after 186K iters ?
Thanks :)

Hi, thanks for your interest in our work! It's normal that the loss is decreased quickly. In my experiments, the loss is stable at 0.11@60K and 0.06@186K with ImageNet Pretraining.

Thanks for your answer !


Support preheat status on Cafe oven (dual oven, in my case)

It looks like this is being used in the Advantium setup but not the general oven info. I've run a test tests to confirm that pre-heat mode on my oven (both upper and lower) is simply reported as Bake both during and after preheat.
I pulled down gehome and ran the websocket example and it pulled down some logs, summarized here:
Setting ErdCode.UPPER_OVEN_COOK_MODE to Bake (350°)
received event: appliance_state_change, processing callbacks...
received event: appliance_update_received, processing callbacks...
Setting ErdCode.LOWER_OVEN_RAW_TEMPERATURE to 127
received event: appliance_state_change, processing callbacks...
received event: appliance_update_received, processing callbacks...
Setting ErdCode.UPPER_OVEN_RAW_TEMPERATURE to 274
received event: appliance_state_change, processing callbacks...
received event: appliance_update_received, processing callbacks...
Appliance state change detected in GeAppliance(REDACTED) (ErdApplianceType.OVEN). Updated keys: ErdCode.UPPER_OVEN_RAW_TEMPERATURE
Setting ErdCode.UPPER_OVEN_COOK_MODE to Bake (350°)
received event: appliance_state_change, processing callbacks...
received event: appliance_update_received, processing callbacks...
Appliance state change detected in GeAppliance(REDACTED) (ErdApplianceType.OVEN). Updated keys: ErdCode.UPPER_OVEN_COOK_MODE
Setting ErdCode.UPPER_OVEN_DISPLAY_TEMPERATURE to 274
received event: appliance_state_change, processing callbacks...
received event: appliance_update_received, processing callbacks...
Appliance state change detected in GeAppliance(REDACTED) (ErdApplianceType.OVEN). Updated keys: ErdCode.UPPER_OVEN_DISPLAY_TEMPERATURE
Setting ErdCode.UPPER_OVEN_CURRENT_STATE to Preheat
received event: appliance_state_change, processing callbacks...

This suggests that gehome (or at least the latest version in git) supports preheat, but whatever I have with ha_gehome=v0.6.7 does not. I don't see any new git changes since v0.6.7 so I suspect it's missing something.
I'm happy to submit this as a pull request, but I don't even know where to begin to figure out what is/isn't making its way into the ha plugin.

I think I'm a little confused on this, seems like turning it on to "Bake" is the solution?  If you're just looking for something to say "Preheat", I think all that is needed is to add a new sensor for the UPPER_OVEN_CURRENT_STATE code (and presumably the lower equivalent).

When I turn the oven on to any mode (e.g. Bake) it goes into a preheat mode. You can see this in the logs above. This state is not reflected in Home Assistant, which just shows the oven in Bake even while it is preheating.
The goal for this would be to have a sensor that detects when preheating is done. The oven has an incredibly silent beep, and my wife is often not near her phone (which gets a push notification from the official app), so I'd like to set up an automation to perform some other alert.
I'm also seeing some issues like ErdCode.LOWER_OVEN_COOK_TIME_REMAINING (another way-too-quiet alert from the oven when that reaches zero) coming through properly on the web socket but not making it into Home Assistant, so that may also be a related missed signal.

Added the oven state sensor referenced in your original debug message.  LOWER_OVEN_COOK_TIME_REMAINING is already present in HA, so if you're having an issue with that, please open a new issue and provide some debug logs of what you're seeing there.

Thanks for fixing the missing LOWER_OVEN_COOK_TIME_REMAINING but that is a separate issue. This issue was about the lack of Preheat status. See the log message in the original post referencing Setting ErdCode.UPPER_OVEN_CURRENT_STATE to Preheat
I've enabled debug logging in Home Assistant and there is no reference to UPPER_OVEN_CURRENT_STATE (or the lower oven equivalent) when I turn the oven on/off.
I'm unable to reopen this issue but would rather not file a separate duplicate unless you request it.

Never mind. This is me getting confused again about terminology. I'm used to master / stable or dev / main for brach naming conventions. I see now that the change is present on the dev branch and the v0.6.8-dev6 tag … and the preheat status is now showing in home assistant:

I have a Cafe dual oven as well and have installed the latest version, but I do not see Preheat as the current state, I still see whatever the current mode is (ie. Frozen Pizza). This happens on both ovens

I do not see Preheat as the current state

This was working for me on v0.6.8-dev6 and I just updated to v0.6.8 and see that it still works with regular cooking. However, the special modes (that "baked goods" mode is amazing for 🍪) don't show a separate preheat state (in HA or the logs) even though the official app can. Maybe we need to file a new bug? Probably run some tests with the websocket example program to see what codes are being sent and missed.

You are correct; regular cooking shows Preheat but the other specialty modes show the mode title. I'll take a look at that websocket example program to see if I can figure it out.

I'll take a look at that websocket example program to see if I can figure it out.

let me know if you need a beta tester


[POAE7-2911] Cider Parquet Reader to Arrow

What changes were proposed in this pull request?
new Class CiderParquetReader::Reader, which reads local parquet file to ArrowSchema&ArrowArray directly.

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?
CiderParquetReaderTest.cpp

Which label does this PR belong to?

@xieqi Please take a review, thanks.


⚠ Bofrost.fr has degraded performance

In 9583ce2, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 903 ms

Resolved: Bofrost.fr performance has improved in b8a6a62 after 8 minutes.


Error with brush.rect

Hey so i get this error 'sometimes' :
TypeError: Cannot read properties of undefined (reading 'CENTER')
Error at line 1 in t.rect() in p5.brush.min
am simply using
brush.rect(x,y,currWidth,currHeight, CENTER)

Hey so i get this error 'sometimes' : TypeError: Cannot read properties of undefined (reading 'CENTER')
Error at line 1 in t.rect() in p5.brush.min
am simply using brush.rect(x,y,currWidth,currHeight, CENTER)

Hi @kala0606,
Could you try using "center", rather than CENTER ?
It might be a problem when using instance mode.
If you are using instance mode, you need to do p.CENTER, probably. Anyway, "center" should work.

Hi @acamposuribe Thanks a lot for your revert.
So i think it might be something else.
I am able to reproduce the error in the happy grid example https://editor.p5js.org/acamposuribe/sketches/oqlWTatWG when i replace brush.field("truncated") with brush.noField() in the LATEST updated p5brush.js
NOTE: works fine in the older version which is uploaded on the editor there

Hi @kala0606
Apologies for the late reply. So, the problem was with the loading of the library.
It is solved by doing brush.load() after canvas creation.
In any casy, I've edited the code to prevent this from happening. Thx for the feedback


Icons token

Some tokens have a unique icon others don’t. Not sure if this will be changed at a later stage?

we have the generic design for all the tokens (design's not matching what's implemented)
the generic token should be a mid step to finally having each of the tokens done more carefully (with unique colors etc.)
@boolkaco , im assigning you back to this ticket, you can find the design on figma (UI kit page) with the note on the system design.
please ping me if something has to be clarified


Add drone-based sandbox system

Description of change
Add a sandbox system that runs a drone sim from our open-core.
Guide to reproduce test results.
Build and push the new system images and experiences and make sure they work in the sanbox.
Checklist:

[x] I have self-reviewed this change.
[x] I have tested this change.
[ ] This change is covered by tests that are already landed, or in this PR.

I'd be happy to land this as-is, though it might benefit from a readme to help navigate

I made a ticket to track adding a README, but I'm gonna go ahead and land since I don't think I'll have time to work on it imminently.


It'll cause scroll jerky on Android devices

When I enabled this SDK in my project and run on an Android device, I found it was jerky in any scrollable view when I scrolled slowly. Everything is perfect when I remove this plugin.
SDK Version
The UXCam Flutter SDK version is 2.2.2
How I use it
FlutterUxcam.optIntoSchematicRecordings();
FlutterUxcam.startWithKey('xxxxxxxxx');
Here is the flutter version:
[✓] Flutter (Channel stable, 3.3.10, on macOS 13.0 , locale en-CN)
• Flutter version 3.3.10 on channel stable at /Users/kim/fvm/versions/3.3.10
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 135454af32 (5 weeks ago), 2022-12-15 07:36:55 -0800
• Engine revision 3316dd8728
• Dart version 2.18.6
• DevTools version 2.15.0
[✓] Connected device
• moto g51 5G (mobile) • android-arm64  • Android 12 (API 31)
Here are two videos, the first one integrates with the UXCam SDK, and the second one without integrates  the SDK
https://user-images.githubusercontent.com/1001783/213317697-9851b83b-a68b-4ba5-8f09-cf70e2774d82.mp4
https://user-images.githubusercontent.com/1001783/213317727-ad437356-bbf7-4720-9561-3b8b0512e351.mp4

Hello @kimlet thanks for bringing this up, the information provided will be extremely helpful.
As a side note, it would be ideal if you could change the startup method to the updated one shown here.
We will keep you posted on any updates. Thanks!

Hi @gabrieluxcam
There are nothing changes when I change the startup method to
FlutterUxcam.optIntoSchematicRecordings();
FlutterUxConfig config = FlutterUxConfig(       userAppKey: "xxxxxxxxxxx",       enableAutomaticScreenNameTagging: true,     );
FlutterUxcam.startWithConfiguration(config);

I have the same problem, once I enabled this SDK in my project I noticed stutter in scrolling and a huge impact on app performance, and it is more noticeable on android devices.

I am also seeing this behavior. I think I'll need to switch to a different screen recording library.

This isn't just an Android issue. Is someone looking into this?

Hi everyone! First of all I just wanted to apologize for the delay in this regard. But would it be possible to use the latest version of the Flutter SDK which is 2.4.6? This should be fixed already in this version.

Hello @DFDIEGOH
This issue still persists on the latest version. Scroll stutters still appears when scrolling quickly.

the issue still for now


Support add custom EVM tokens

[x] Support adding custom ERC-20, ERC-721 on EVM networks
[x] Fix this bug upon importing custom tokens

Version test : http://<IP_ADDRESS>:8900/s/tkH4isbmSTJWe4f
Validation passed in case :

Import token ERC-20

Displays the correct balance of the account with the custom token balance

ON/OFF custom token
Remove custom token

Support adding custom ERC-721 on EVM networks -> Handle on issue #113


1781 drop down selection

I'm going to close this PR since it did not branch of off develop and has some leaked commits in it.  @javtran please reopen with a fresh branch.


fix(es/parser): don't allow type annotation after assign

Fixes #1209.
I was not sure if I was supposed to return a syntax error or just emit it.
Also TS does not have specific error for this and shows very unhelpful message, so went with this new error.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Support glyphicons/font-awesome natively in package

Currently users need to explicitly specify URLs for icons/shapes. The package should provide the ability to specify icon names, a la https://shiny.rstudio.com/reference/shiny/latest/icon.html

Let me know if you run into any trouble here, and I'll gladly assist.


docs(gateway): fix absolute path

Signed-off-by: slonka<EMAIL_ADDRESS>fix absolute path that would lead to new versions linking to old versions of the docs
Did you sign your commit? Instructions yes
Have you read Contributing guidelines? yes

Yeah, that looks right. Should this also be fixed in docs/docs/1.8.x/quickstart/kubernetes.md?

Yes, but this in master right now.


Add automation testing for Openshift 3.9

A previous PR has been opened that covers automation for GKE, Openshift 3.11 and 3.10 (https://github.com/cyberark/secrets-provider-for-k8s/pull/46) for the Secrets Provider for K8s. This is a separate card to support 3.9. At current, tests run manually and this effort is to build automaticity into our pipeline for 3.9
Open questions to consider:
Do we want to support 3.9? Are our customers on 3.9 still? This is a decision that needs to be had between the team and PM
Why do we get a different response from 3.11/3.10 and 3.9. Maybe there is something in our scripts that is not supported in 3.9 - deleting deployments, pods, etc. See authn-client and demo

Closing as we won't add automation for ocp 3.9 at this stage


Unable to login with cookie

Tried all the cookie values below one by one, but none worked. Can you please help me? I was able to log in about half a year ago.

Closed as not planned by @erbanku on Tuesday, March 12, 2024

@erbanku You use it in the wrong way. Capture a page request, then copy the Cookie header.

@erbanku You use it in the wrong way. Capture a page request, then copy the Cookie header value.

Fixed, thank you.🙃


doc-fix:add IP white list document guide to avoid disabling the firewall of the host

I add IP white list document guide to avoid disabling the firewall of the host to resolve and avoid https://github.com/secretflow/secretflow/issues/527

Hi, can you update the CN po files ?


[TEP-0100] Add new updatePipelineRunStatusFromChildRefs function

Changes
And of course, this is part of https://github.com/tektoncd/community/blob/main/teps/0100-embedded-taskruns-and-runs-status-in-pipelineruns.md,
building on a pile of other PRs (#4705, #4734, #4753, #4757).
This adds a new function to pkg/reconciler/pipelinerun/pipelinerun.go,
specifically for updating pr.Status.ChildReferences during reconciliation. It's
analogous to the existing updatePipelineRunStatusFromTaskRuns and
updatePipelineRunStatusFromRuns functions. This PR doesn't actually call the
new function - behavior is exactly the same. But it adds the new function,
along with other functions it depends on. In the final step of the
implementation, these other functions will also be used in ...FromTaskRuns
and/or ...FromRuns.
I also reworked pkg/reconciler/pipelinerun/pipelinerun_updatestatus_test.go
to improve its test fixtures, so that they're easier to reuse and instantiated
via YAML parsing as much as possible.
/kind tep
/kind feature
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[ ] Docs included if any changes are user facing
[x] Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Release notes block below has been filled in
(if there are no user facing changes, use release note "NONE")

Release Notes
NONE

/assign @jerop
/assign @lbernick
/assign @vdemeester

/retest
GitHub's API has been flaking out sporadically lately...

/retest


Fixes to image sizes, etc.

Can you also make the image resolution physically smaller and ensure they are imageOptim'ed

Will do.


FC2 Feedback: results to be in alphabetical order (Christian name), as they used to be, not by d.o.b.

Need to draw attention to how we currently allow this.

Click the column heading to sort into different orders?

As noted above the option is there


"go get" seems to fail upfront

Hello there, finally happy top find someone like-minded that put the effort to actually write a program that was in my mind for a long time! :)
I am not an expert in go, and was giving a shot at your program but am having an issue while trying to install:
me@host:~$ go get github.com/m-manu/rsync-sidekick
package embed: unrecognized import path "embed" (import path does not begin with hostname)
package io/fs: unrecognized import path "io/fs" (import path does not begin with hostname)
After this the script seems to keep going on, and after a few seconds it returns without any message, but I cannot invoke nor find any rsync-sidekick executable.
The test system is quite "old" but regularly updated, and I found the go version is in the same shape (many version behind):
me@host:~$ go version
go version go1.10.4 linux/amd64
Any hint as to make your program run on this platform?
Thanks in advance 😉

You're on Go 1.10. For this tool to work with go get, you need Go 1.16

I suspected this. Is there an alternative way for installing then?
Sorry for asking basic questions, it's just I never worked with this tools (yet)

If you can't upgrade to Go 1.16, you've two options:

Use Docker (see README)
Temporarily upgrade to Go 1.16. Do go get (this creates executable). Then go back to old version that you need it to be.


Port upstream snort3 app changes

Porting upstream snort3 app changes from https://gitlab.com/synack3/shuffle_dev/-/tree/master. This PR:
Dockerfile:

Updates snort to <IP_ADDRESS>
Updates DAQ to 3.0.2
Adds flex, flex-dev apk packages (snort <IP_ADDRESS> requirements)
Shifts SAFEC URL to leverage runtime arg
Implements Docker security recommendations: --no-cache on pip
Adds snort3 user and group to container
Sets up /app for new snort3 user
Migrate CMD to Docker best practice array format

api.yaml:

Whitespace updates based on automated yamllint scan results

app.py:

Formatting updates per black
Force snort3 execution as snort3 user/group
Change download directory to /app to support new user

:eyes:

Just wanted to release this blogpost before I got back to this: https://medium.com/shuffle-automation/the-shuffle-automation-and-detection-framework-open-source-soar-e7005aa8f5c3?source=your_stories_page-------------------------------------
This is amazing, and has definitely helped shape how I think about us doing detection and automation, meaning more of our focus will go in this direction for usecases.
Would you be open to sharing how you'd ideally use Shuffle with Snort and other detection tools sometime?


77 grant data not match grants gov

Ticket
https://github.com/usdigitalresponse/usdr-gost/issues/77
Description
https://github.com/usdigitalresponse/usdr-gost/issues/77#issuecomment-1315986138
Screenshots / Demo Video
n/a
Testing
Manually spot-checked the dates in my local. Grants within the 120 day range (my local only) were fixed, others outside that range (e.g., older) were not.
Checklist

[x] Provided ticket and description
[x] Your PR title contains the ISSUE ticket number e.g "86: ..."
[x] Provided screenshots/demo
[x] Provided testing information
[x] Provided adequate test coverage for all new code
[x] Run automated tests (docker compose exec app yarn test)
[x] Added PR reviewers
[ ] Ensure at least 1 review before merging

Added tests for each function. Let me know if this is what you are looking
for.
On Thu, Nov 17, 2022 at 4:44 PM Tyler Hendrickson @.***>
wrote:

@.**** requested changes on this pull request.
Looks good. If you don't mind, I'd like to see a few tests for
convertDateToEST() and formatElapsedMs() functions.
—
Reply to this email directly, view it on GitHub
https://github.com/usdigitalresponse/usdr-gost/pull/608#pullrequestreview-1185305663,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA55MVTXWZQJRYVQE6ZK2U3WI3GNVANCNFSM6AAAAAASBOU34E
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
-CK
https://github.com/chrisxkeith

Mind holding off on merging this PR until after we get a chance to
discuss on Tuesday?

Happy to! Look forward to the conversation.
On Fri, Nov 18, 2022 at 2:59 PM Tyler Hendrickson @.***>
wrote:

@.**** commented on this pull request.
@chrisxkeith https://github.com/chrisxkeith Tests look great, thanks
for adding those.
I was chatting with @as1729 https://github.com/as1729 this afternoon
and we think some of the details around the bug discussed in #77
https://github.com/usdigitalresponse/usdr-gost/issues/77 are a bit
different from some of the behavior we're seeing in the staging
environment, which might impact the way we want to approach a fix in this
PR. It might be better to chat at the next GOSTbusters meeting on Tuesday
so that we can get aligned on the details before we move forward with
merging this.
Mind holding off on merging this PR until after we get a chance to discuss
on Tuesday?
—
Reply to this email directly, view it on GitHub
https://github.com/usdigitalresponse/usdr-gost/pull/608#pullrequestreview-1186915808,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA55MVQ45P3SZW7HNZZR2CTWJAC4FANCNFSM6AAAAAASBOU34E
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
-CK
https://github.com/chrisxkeith

Realized this will only work during a certain timeframe. E.g., since the scraper is run every hour (on the half-hour), there will be times when the GMT and EST days match, and times when they don't. May work if we reduce the scraper runs to once-a-day at an appropriate time.


Fix for Inconsistent Naming in Responsive Layout Classes

Pairs up the hide-* with the corresponding col-*-x classes.
Example:
Two sibling elements, one has class="hide-sm col-3", and the other has
"col-sm-12 col-9"
--> Screen size <= 600px: First element is hidden, second element spans
the entire width
--> Screen size > 600px: First element takes one quarter, second element
three quarters of the screen width.

The diff is actually quite confusing to look at, so here's the gist:

for screen size > @size-xs, the col-sm-* classes will now apply (instead of col-xs)
for screen size > @size-sm, the col-md-* classes will now apply (instead of col-sm)
... and so on
col-xs-* have been moved to apply globally. Technically they could be removed, as they are not identical to col-*, but I put them there for backwards compatibility.
No idea why the resulting dist has changed regarding -webkit, I did not make any changes regarding that, so the sources must have been slightly out of sync to what is in the dist folder in your master.


get details page

related #43
fix #43

The item type/colour etc also have to bold - check this in figma.


supportedOrientations

PLS add supportedOrientations to Modal to support Orientation Changes.

Hey @davidbiller, thanks for issue, good point. I will try to do it this weekend or earlier.

any update ? , when orientation changed to 'right' or 'left' up and down slide change the image :/

same issue, any luck on this?


ExternalTexture in gltf is not working properly

I am trying to set video texture to gltf model.
I created a custom material for test, but it's working awkwardly. The gltf model just turns red as shown below.

I wonder if i am doing anything wrong or it's the issue of filament.
Below is mat file i used.
material {
    name : ubershader_lit_opaque,
    requires : [ uv0, uv1, color ],
    shadingModel : lit,
    blending : opaque,
    depthWrite : true,
    doubleSided : true,
    flipUV : false,
    specularAmbientOcclusion: simple,

    parameters : [
        // Base Color
        { type : samplerExternal, name : videoTexture }
    ],
}

fragment {
    void material(inout MaterialInputs material) {
        prepareMaterial(material);
        vec2 uv = getUV0();
        material.baseColor = texture(materialParams_videoTexture, uv).rgba;
    }
}

and android code
    rm.setMaterialInstanceAt(renderable, 0, materialInstance)
    val mat = rm.getMaterialInstanceAt(renderable, 0)

    val sampler = TextureSampler()
    val surfaceTexture = SurfaceTexture(0)
    surfaceTexture.detachFromGLContext()
    val surface = Surface(surfaceTexture)
    val stream = Stream.Builder().stream(surfaceTexture).build(EngineInstance.getEngine().filamentEngine)

    val mediaPlayer = MediaPlayer.create(context, R.raw.arachi_jump)
    mediaPlayer.setSurface(surface)
    mediaPlayer.setLooping(true)
    mediaPlayer.start()

    val engine = EngineInstance.getEngine()
    val textureSampler =
        com.google.android.filament.Texture.Sampler.SAMPLER_EXTERNAL
    val textureInternalFormat =
        com.google.android.filament.Texture.InternalFormat.RGB8
    val videoTexture = com.google.android.filament.Texture.Builder()
        .sampler(textureSampler)
        .format(textureInternalFormat)
        .build(engine.filamentEngine)

    videoTexture.setExternalStream(engine.filamentEngine, stream)

    mat.setParameter("videoTexture", videoTexture, sampler)

I made sure video texture is actually applied to gltf model, but somehow works like this. What am I missing?

This bug sounds similar to #2659, which is now fixed.  If you don't have the fix you can work around by ensuring that the glTF file has a placeholder texture for baseColor.

@prideout
Thanks in advance for your prompt response!
Our gltf already has baseColorTexture in it. I don't think getUV returns 0,0 in our model.
After I visuailized UV coordinates as you suggested, its like below.

Interesting, thanks.  Hmm.  Do you see anything interesting in Logcat when filtering for Filament?

@prideout
Not that I know of.
It's weird because if i try the same thing with Galaxy Fold(Android 9). gltf model doesn't turn red but black.
Is there any other possible thing i could be missing now?

The difference you see between various devices might be due to uninitialized texture memory, if the texture isn't getting set.
If you can construct a repro case and attach a zip to the issue, then we can take a closer look.

@prideout
Thanks for giving me great insight!
It was a sampler problem. I solved the issue by setting TextureSampler's filter and wrap mode.
I really appreciate your help.


fix: double-slash issue in health check paths and constraints for Zeebe Gateway

Which problem does the PR fix?
Related to: https://github.com/camunda/camunda-platform-helm/issues/2282
The main issue occurs when both zeebeGateway.contextPath and zeebeGateway.ingress.rest.path are set in ways that lead to a double-slash (//) in the health check paths (e.g., /actuator/health/readiness). This happens when:
zeebeGateway.contextPath is set to / and zeebeGateway.ingress.rest.path starts with a /.
Similarly, the constraints.tpl file enforces strict matching between zeebeGateway.contextPath and zeebeGateway.ingress.rest.path, causing errors when paths differ only in trailing slashes.
In our default values the context path is not set and the constraint will trigger when zeebeGateway.ingress.rest.enabled: true is set .
zeebeGateway:
  ingress:
    rest:
      path: "/"
  contextPath: ""

When you set the context path to match the path it will pass the constraint but will cause the double-slash error in the readiness probe (e.g., //actuator/health/readiness) this leads for the Zeebe Gateway pods not to come up in the ready state.
zeebeGateway:
  ingress:
    rest:
      path: "/"
  contextPath: "/"

What's in this PR?
Health Probes Fix:

Updated Helm template logic for readinessProbe to handle cases where both zeebeGateway.contextPath and zeebeGateway.ingress.rest.path are /, ensuring the resulting path is correctly formed as /.
Used trimPrefix and trimSuffix Helm functions to remove redundant slashes and prevent double slashes (//) in paths.

Constraints Fix:

Modified the constraints.tpl to trim trailing slashes when comparing zeebeGateway.contextPath and zeebeGateway.ingress.rest.path, allowing for flexible configurations without triggering unnecessary errors.

Scenarios Addressed:

zeebeGateway.contextPath is / and zeebeGateway.ingress.rest.path is /: Results in /, not //.
zeebeGateway.contextPath is empty "" and zeebeGateway.ingress.rest.path is /: Results in /, ensuring the path is valid.
zeebeGateway.contextPath is /zeebe and zeebeGateway.ingress.rest.path is /zeebe: Results in /zeebe/actuator/health/readiness, as expected.

Checklist
Please make sure to follow our Contributing Guide.

Before opening the PR:

[x] In the repo's root dir, run make go.update-golden-only.
[x] There is no other open pull request for the same update/change.
[ ] Tests for charts are added (if needed).
[ ] In-repo documentation are updated (if needed).

After opening the PR:

[x] Did you sign our CLA (Contributor License Agreement)? It will show once you open the PR.
[ ] Did all checks/tests pass in the PR?

_helpers.tpl Modifications:
created a named helper template camundaPlatform.zeebeGatewayReadinessProbePath in _helpers.tpl to generate the correct readiness probe path based on different configurations. The key modifications ensure:

If contextPath is empty, the helper uses ingress.rest.path as the base path.
If contextPath is /, it correctly handles the readiness probe path starting with /.
If contextPath has trailing slashes (like /test/), it trims them to prevent double slashes in the final path.
It ensures no extra slashes are appended to the final path, and there is always exactly one slash between the context path and the probe path.

Scenarios Tested in Unit Tests
1. TestReadinessProbeWithContextPathAndIngressEnabled

Scenario: contextPath and ingress.rest.path are both set to /test.
Expected Result: The final readiness probe path should be /test/actuator/health/readiness.
Purpose: This test ensures that when both contextPath and ingress.rest.path are non-root paths (i.e., /test), the readiness probe path is correctly appended with the probe path, with no extra or missing slashes.

2. TestReadinessProbeWithRootContextPathAndIngressEnabled

Scenario: contextPath and ingress.rest.path are both set to / (the root path).
Expected Result: The final readiness probe path should be /actuator/health/readiness.
Purpose: This test verifies that when both contextPath and ingress.rest.path are set to /, the readiness probe path starts with a single / and is followed directly by the readiness path without double slashes.

3. TestReadinessProbeWithEmptyContextPathAndIngressEnabled

Scenario: contextPath is empty (""), and ingress.rest.path is set to /.
Expected Result: The final readiness probe path should be /actuator/health/readiness.
Purpose: This test checks that when contextPath is empty, the readiness probe path falls back to using ingress.rest.path correctly and forms the proper probe path without extra slashes.

4. TestReadinessProbeWithTrailingSlashInContextPath

Scenario: contextPath and ingress.rest.path are both set to /test/ (with trailing slashes).
Expected Result: The final readiness probe path should be /test/actuator/health/readiness.
Purpose: This test ensures that trailing slashes in both contextPath and ingress.rest.path are properly handled, trimming the extra slashes and ensuring that only one slash separates the contextPath from the readiness probe path.

Key Outcomes:

The tests cover various combinations of contextPath and ingress.rest.path configurations, including empty, root, non-root, and trailing-slash cases.
Each test ensures the correctness of the readiness probe path, verifying that no extra or missing slashes exist in the final path.
These tests provide confidence that the Helm chart will generate valid readiness probe paths across multiple common configurations of the Zeebe Gateway, ensuring robust and flexible handling of contextPath and ingress.rest.path.


avoid binary issue with barbican payload endpoint

Trying to skip the binary issue here for the barbican payload endpoint while leaving everything else functioning normally.

deployed this commit in qade1 and dint observe the sentry event.
Do we need to fix the tox error ?

The tox error is unrelated, and does not require being fixed. I will however spend some time on that now.
The only concern from this point is that all openstack services are aware this is being upgraded, they will see it in their diffs for deploy, and they should be mindful if there's an issue to contact immediately for us to sort out.


Add XamlQRCode.GetUnitsPerModule() method

Summary
This is a helper method allowing client to obtain an exact measure of module size without a need to repeat the calculation by itself.
When the client wants to draw a border around QR code or some other adornments which need to use the same size and/or spacing as the module size this commit simplifies its calculation.

[ ] Introduces XamlQRCode.GetUnitsPerModule() helper method

Thanks for your support!


Add option to set default route or mock app.vue

I want to setup test environment without rendering my index page, please add config option for setting globally default route, for example '/test' instead of '/' or option to mock app.vue component.
So far I figured out workaround, in nuxt.config.ts I am doing mocking of index page manually with hooks
hooks: {
    'pages:extend'(pages: NuxtPage[]) {
      if (String(process.env?.TEST) === 'true') {
        const indexPage = pages.find((page) => page.name === 'index')
        const testPage = pages.find((page) => page.name === 'test')
        if (indexPage != null && testPage != null) {
          indexPage.file = testPage.file
        }
      }
    },
  },

Note this hook is not working when placed in vitest.config.ts overrides because function is not serializable.

This should now be implemented - we no longer render app.vue by default since https://github.com/nuxt/test-utils/pull/260.


browsers() is taking error

Getting following error when upgraded from 2.6.0.
06 03 2018 08:27:44.640:ERROR [config]: Invalid config file!
TypeError: Cannot read property 'testing' of null
at Object.browsers (C:\workspaces\Dev1\clinical-278\node_modules\availity-workflow-settings\index.js:144:30)
on method
browsers() {
return this.configuration.testing.browsers;
},
Can fix by adding to config but documentation states will default to Chrome, if not set.  So I believe the testing section should be defaulted if not present.

Closing since we no longer use browsers for tests.


[Stepper] Add component

Add Stepper component, hiding complexity of inner Step, StepLabel and other nested components.
See for reference: https://material-ui.com/components/steppers/

:tada: This issue has been resolved in version 9.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Hide label of afInputField when using boolean datatype

I´m using an afInputField with a Boolean datatype.
I don´t know why, but using Boolean always shows a label and I can´t get rid of it.
 But I´ve figured out that with this solution you can change the label, if the string is not empty. Wtf
new SimpleSchema({
        field1: {
            type: Boolean,
            optional: true,
            label: "new label"
        }
    });

Is there any working solution for simply hiding that field label?

the workaround solution is to use a blank like label: ' ' OR hide via css


Remove "target" column that is not used

Remove "target" column that is confusing when analyzing results and it is not used in the rest of the query.
OR, alternatively, mv-expand TargetResources, and then extend their properties with "TargetResources.property" instead of "TargetResources[0].property".
The change should depend on the length of TargetResources, in my tenant I have only seen it of size 1.
Fixes #
Unuseful column in results.
Proposed Changes
Remove "target column", OR, alternatively, apply the query to every element in array "TargetResources"

@petebryan - can you have a look at this one.  You did the original and based on others for this, I think this is a typo of not using the 'target' field later in the query.

Looks like a good change. Target appears to be an un-necessary extend.


Add Zwave refresh services

Description:
Add 2 Zwave refresh services:

refresh_node that refreshes the whole node by node_id
refresh_entity that refreshes a single entity by refreshing the values the entity depends on.

Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>
If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass

Is that threadsafe?

Instead to add entities to hass.data can you use helpers/dispatcher.py and fire a signal.

@pvizeli Those service calls are not safe like the other calls - access data out of zwave thread.
I'll look into helpers/dispatcher.py

@pvizeli Switched to dispatcher


feat(schema): Add Returns.Title chained decorator

Closes: #1083
Informations

Type
Breaking change

Feature
No

Hello @dvarnai
Can you review the PR please. I want to be sure that the story is correctly implemented :)
Thanks for your help  ;)

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

v6.8.0
GitHub release

Your semantic-release bot :package::rocket:


⚠ Empornium has degraded performance

In 71f7501, Empornium ($EMP) experienced degraded performance:

HTTP code: 500
Response time: 1274 ms

Resolved: Empornium performance has improved in 19055c4 after 422 days, 11 hours, 4 minutes.


[FLINK-26351][k8s/web ui] After scaling a flink task running on k8s, …

…the flink web ui graph always shows the parallelism of the first deployment.

What is the purpose of the change
(After scaling a flink task running on k8s, the flink web ui graph always shows the parallelism of the first deployment.)
Brief change log

When the flink k8s task builds the ExecutionGraph, it is return this graph instead of clone graph.

Verifying this change
Please make sure both new and modified tests in this PR follows the conventions defined in our code quality guide: https://flink.apache.org/contributing/code-style-and-quality-common.html#testing
This change is already covered by existing tests, such as (please describe tests).
AdaptiveSchedulerTest
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: ( no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

hello

@rmetzger Would you like to review this pr? Thanks for your time.

@rmetzger Would you like to review this pr? Thanks for your time.


CustomPainter performance issue

I have a simple app with CustomPainter which draws a scrolling chart.
The code is drawing around 700 lines every frame using simplest Paint possible (no antialias, stroke 1, style=PaintingStyle.stroke). The code in general is very fast and mostly it draws fast enough to run at 60 FPS.
But.. sometimes it jitters. In the profiler I can see that GPU frame takes around 12-13 ms. When the jitter happens - somehow GPU frame take 30-50ms, and in the Dart DevTools Timeline in the GPU section I can see that it's SkCanvas::Flush method that takes so much time.
How is that possible if nothing unusual happened during the drawing of the frames?
One more thing - on iOS calling canvas.drawLine() 700 times is faster, than calling canvas.drawPath() with 700 lines in it once.

I just found out - antialiased lines are drawn much faster than non-antialiased.

@fpv999
Cloned your repo and tested on Android emulator Pixel 3XL using latest stable version (1.22.4). I had to change valueOnChart to 300 to see the lines rendering on screen. I also checked Flutter Performance while lines were being rendered and saw Frames per second ranging from 20 to 26 mps.

The lines being rendered were okay-ish but didn't notice any jitters.
49558_1.mov.zip
Can you confirm if the current performance using latest stable is the expected behavior or if this issue still need to be kept open ?

flutter doctor -v
[✓] Flutter (Channel stable, 1.22.4, on Mac OS X 10.15.4 19E2269 darwin-x64,
    locale en-IN)
    • Flutter version 1.22.4 at /Users/dhs/documents/Fluttersdk/flutter
    • Framework revision 1aafb3a8b9 (4 weeks ago), 2020-11-13 09:59:28 -0800
    • Engine revision 2c956a31c0
    • Dart version 2.10.4

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
    • Android SDK at /Users/dhs/Library/Android/sdk
    • Platform android-30, build-tools 30.0.0
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.0.1, Build version 12A7300
    • CocoaPods version 1.9.3

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 46.0.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.51.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.16.0

[✓] Connected device (1 available)
    • sdk gphone x86 arm (mobile) • emulator-5554 • android-x86 • Android 11
      (API 30) (emulator)

• No issues found!

Yes, it's okay-ish on your video too, but being okay-ish is sometimes not enough. It's just not smooth.
But if you set isAntiAlias to true, it will work much better.


Adding additional datatypes support for Parquet extension

Hi Team,
As described on the below readme,
https://github.com/sathishs/druid/tree/parquet-extension/extensions-contrib/parquet-extensions
We have included the support for parsing additional datatypes to the current parquet-extension. Please do review the changes and let us know your review comments
Thanks,
Sathish

@sathishsri88 I've finished my initial review, can you take a look at the comments when you get a chance?

Hi @clintropolis,
Thanks for the changes,on the high level looks good,
let me go through all of our test cases and send you a PR with the additional test cases and fixes if required
Thanks,
Sathish


Plugins not overwritten when passed as options

Environment

Operating System: Darwin
Node Version:     v20.11.0
Nuxt Version:     3.11.2
CLI Version:      3.11.1
Nitro Version:    2.9.6
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      devtools, modules, pinia, typescript
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
https://github.com/lucasecdb/nuxt-app-pinia-repro
Install dependencies with pnpm install then run pnpm test
Describe the bug
If you configure a vue plugin using Nuxt plugins, like the following:
// plugins/pinia.ts
import { createPinia } from 'pinia';

export default defineNuxtPlugin((nuxtApp) => {
  nuxtApp.vueApp.use(createPinia())
})

Given the following component and Pinia store:
<template>
  <p>{{ date }}</p>
</template>

<script setup>
const store = useMyStore();

const date = computed(() => store.date);
</script>

// stores/my-store.ts
export const useMyStore = defineStore('my-store', () => {
  return {
    date: ref(new Date()),
  };
});

And then try to overwrite that plugin during testing:
// __tests__/my-component.test.ts
import { createPinia } from 'pinia';

it('should pass', async () => {
  await renderSuspended(MyComponent, {
    global: { plugins: [createPinia()] },
  });

  // ...
});

The instance returned by useMyStore will always use the Pinia store supplied by plugins/pinia, and not from the global.plugins parameter provided to renderSuspended.
Additional context
No response
Logs
No response

The issue occurs because on @vue/test-utils the plugins from the global properties are applied before the provides:

Plugins are applied on line 276: https://github.com/vuejs/test-utils/blob/1cdd7ae5a973e8a2de8eef8c22159bc0cc97f911/src/createInstance.ts#L269-L278
Provides values are copied to the vue instance on line 303: https://github.com/vuejs/test-utils/blob/1cdd7ae5a973e8a2de8eef8c22159bc0cc97f911/src/createInstance.ts#L299-L305

And since renderSuspended copies over the provided values from the global instance down to the render call, the plugin will always be overwritten back by the vueApp provided value:
https://github.com/nuxt/test-utils/blob/08ee35155579889e6c41f8abe9340d0e5f803111/src/runtime-utils/render.ts#L136

This is now resolved in the latest @vue/test-utils: https://github.com/vuejs/test-utils/releases/tag/v2.4.6.


code-insiders --prof-startup failes with exception

Elementary Loki (Ubuntu 16.04)
Version: 1.26.0-insider
Commit: c86cab2
Date: 2018-08-02T05:18:18.450Z

start code-insiders --prof-startup
dialog opens ' Successfully created profiles', click on 'Creas Issues and Restart'
nothing happens. Dev tools show

TypeError: Path must be a string. Received undefined
    at assertPath (path.js:28:11)
    at Object.join (path.js:1251:7)
    at file:///usr/share/code-insiders/resources/app/out/vs/workbench/workbench.main.js:4820:889
    at Object.g [as _notify] (file:///usr/share/code-insiders/resources/app/out/vs/workbench/workbench.main.js:151:729)
    at Object.enter (file:///usr/share/code-insiders/resources/app/out/vs/workbench/workbench.main.js:155:61)
    at n.Class.derive._oncancel._run (file:///usr/share/code-insiders/resources/app/out/vs/workbench/workbench.main.js:156:902)
    at n.Class.derive._oncancel._completed (file:///usr/share/code-insiders/resources/app/out/vs/workbench/workbench.main.js:156:343)
    at e.doRequest.n.TPromise.t.handlers.(anonymous function) (file:///usr/share/code-insiders/resources/app/out/vs/workbench/workbench.main.js:3

After a while the console from with code-insiders was started show:
[ [ { Error: connect ECONNREFUSED <IP_ADDRESS>:9223
    at Object._errnoException (util.js:1024:11)
    at _exceptionWithHostPort (util.js:1046:20)
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1182:14)
      code: 'ECONNREFUSED',
      errno: 'ECONNREFUSED',
      syscall: 'connect',
      address: '<IP_ADDRESS>',
      port: 9223 },
    { Error: connect ECONNREFUSED <IP_ADDRESS>:9223
    at Object._errnoException (util.js:1024:11)
    at _exceptionWithHostPort (util.js:1046:20)
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1182:14)
      code: 'ECONNREFUSED',
      errno: 'ECONNREFUSED',
      syscall: 'connect',
      address: '<IP_ADDRESS>',
      port: 9223 },
    { Error: connect ECONNREFUSED <IP_ADDRESS>:9223

In the debugger I see that the home folder /user/aeschli is searched for files with a given prefix, but no file or folder has been created there.

Now I suddenly get profile files. The dialog shows the file names and an issue can be created.

Hm, maybe it connected to the wrong instance? When running with --prof-startup it should take a little while before starting and showing a window. I will tweak this to try with less predictive/busy ports...

I have improved this and made things more stable. Now, when running with --prof-startup when it's not the only window an error is printed, tho a little delayed


Couple of tweaks

Hi there, love your AWS wrangling tool. Added a couple of tweaks :+1:

Hi Alex,
Thanks for the PR.
Although reviewing some of your changes, I think this diverges from the roots of what BAM is as a tool. I do appreciate some of your changes and suggestions, but I believe BAM will become the tool of the future and will not be replaced with any old tool (like "am" for instance).


remove lerna from developer tooling repo

instead use yarn workspaces  (not to be confused with the yarn workspace (singular) command

Relates to this:

https://github.com/celo-org/celo-labs/issues/1033

fixed in https://github.com/celo-org/developer-tooling/pull/58


Internal CLR error. (0x80131506)

Steps to reproduce
open powershell

Expected behavior

Actual behavior
Internal CLR error. (0x80131506)
at Microsoft.PowerShell.TaskbarJumpList.CreateElevatedEntry(System.String)
at Microsoft.PowerShell.ConsoleHost+<>c.b__4_0()
at System.Threading.Tasks.Task.InnerInvoke()
at System.Threading.Tasks.Task+<>c.<.cctor>b__274_0(System.Object)
at System.Threading.ExecutionContext.RunFromThreadPoolDispatchLoop(System.Threading.Thread, System.Threading.ExecutionContext, System.Threading.ContextCallback, System.Object)
at System.Threading.Tasks.Task.ExecuteWithThreadLocal(System.Threading.Tasks.Task ByRef, System.Threading.Thread)
at System.Threading.Tasks.Task.ExecuteEntryUnsafe(System.Threading.Thread)
at System.Threading.Tasks.Task.ExecuteFromThreadPool(System.Threading.Thread)
at System.Threading.ThreadPoolWorkQueue.Dispatch()
at System.Threading._ThreadPoolWaitCallback.PerformWaitCallback()
Environment data

Windows 10 20H1 18908

@yao00jun Please add PSVertionTable

I can reproduce this same exact error and stack trace intermittently by right-clicking pwsh.exe (7.0.0-preview.1) and running it as an Administrator.
I'm using the zip distribution: PowerShell-7.0.0-preview.1-win-x64.zip
My version table when I AM able to open powershell is:
Name                           Value
----                           -----
PSVersion                      7.0.0-preview.1
PSEdition                      Core
GitCommitId                    7.0.0-preview.1
OS                             Microsoft Windows 10.0.18908
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

@bergmeister Perhaps we should skip this step if PowerShell is installed side-by-side. Thoughts?

When you pin the task bar item to the taskbar, then that 'run as admin' context menu is specific to the location of the binary (after it is run once, it will even persist), therefore the feature still makes sense for side-by-side. I have always had the daily build and the RTM installation side by side on my machine without any problem. By looking at the OS version, the user is running a Windows Insider build of  20H1, therefore I imagine this to be an issue of this preview version of Windows that should be reported to the Windows Insider team.

@SteveL-MSFT Can you assign next milestone?

Dupe of https://github.com/PowerShell/PowerShell/issues/9295

The fix for this issue has been back-ported to the recent release of 6.2.2, please update and provide feedback if it fixed it. Users of the preview will have to wait for 7.0.0-preview.2
https://github.com/PowerShell/PowerShell/releases/tag/v6.2.2
cc @yao00jun @jared-is-ray


License clarification

Hello,
So you have had a BSD 3-Clause License file in the repo for 4 years:
https://github.com/executablebooks/MyST-NB/blob/master/LICENSE
During the rewrite 2 years ago, you specified in your pyproject.toml, that your project was MIT licensed:
https://github.com/executablebooks/MyST-NB/blob/9943ec214c35844c4535d0184f7840574fc7ab03/pyproject.toml#L17
Could you clarify which one applies?
Thanks.

That's a great spot, thank you! The license has not changed, so the pyproject.toml content is a mistake. Thankfully, it only applies to the Trove classifiers, and not the wider metadata.
Let's fix that, and make a new release.


Add Punk TSO

Canary Punks are proud to finally launch our data provider after almost a year of launching and developing our NFT project on the Songbird Network!
20% of our DP earnings will be given to our NFT holders on both Flare & Songbird, distributing rewards back to the community.
Our NFT holders can benefit from not only TSO rewards, but 100% of all sale royalties, staking rewards and proceeds from our future DEX and NFT marketplace income.
We are very much looking forward to helping decentralize the network, as well as give back to the community, all in one!
Feel free to checkout our socials to stay up to date with our project;
https://canarypunks.xyz/
https://twitter.com/Canary_Punks
discord.gg/8cdPB9M3e8

Logo uploaded with correct address, old one removed. Hopefully ready for merge now!

Inactive on both networks. The merge will be reverted.


Lacks separation between "force" and "assume yes" option

See that in apt-get and other package managers the semantics of user query and forced install are separated:

"-y" / yes option --- assume all confirmations as "yes" for batch processing
"-f" / force option --- force reinstalling the package for whatever reasons (e.g. for fixing a partially broken installation)

Currently, install command lacks this separation. It can choose to force the installation, but its true meaning is yes.

Closed with #54


Freeze linters

What this PR does / why we need it:
Freezes gometalinter using a docker image.
Removes separate install-prerequisites step for buildifier and gometalinter.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Vendorization script is adjusted to remove double-symlinks since they don't work well with docker mounts. Generated files are placed into vendor/ whenever possible for simpler cleanup.
Release note:

cc @ZackButcher @sebastienvas if this looks fine to you, I can do a similar PR in mixer.

This refers to a directory inside the docker image.
On Wed, Sep 27, 2017, 10:25 PM Laurent Demailly<EMAIL_ADDRESS>wrote:

@ldemailly commented on this pull request.
In bin/check.sh
https://github.com/istio/pilot/pull/1363#discussion_r141528544:

@@ -26,10 +36,9 @@ gometalinter --concurrency=${NUM_CPU} --enable-gc --deadline=300s --disable-all
--enable=varcheck
--enable=vet
--enable=vetshadow\

./...

/go/src/istio.io/pilot/...

that won't work on people's machine will it ? (my directory is ~/go no /go)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/istio/pilot/pull/1363#pullrequestreview-65754509, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AJGIxpmktze4v6fKm7pCP1i0Lge-ZO5rks5smy3hgaJpZM4PlMjR
.

It's built once. I didn't do automation since linters shouldn't be changing
that often realistically.
On Wed, Sep 27, 2017, 10:34 PM Laurent Demailly<EMAIL_ADDRESS>wrote:

@ldemailly commented on this pull request.
In bin/check.sh
https://github.com/istio/pilot/pull/1363#discussion_r141529247:

@@ -26,10 +36,9 @@ gometalinter --concurrency=${NUM_CPU} --enable-gc --deadline=300s --disable-all
--enable=varcheck
--enable=vet
--enable=vetshadow\

./...

/go/src/istio.io/pilot/...

oh right, sorry, I didn't see the docker run on line 11
is that image built each time ? (cached maybe but still) ?
can we put the linters in some prow image we already have ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/istio/pilot/pull/1363#discussion_r141529247, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AJGIxtlYYpKFXjzudtLtGDOGjv4ICkGVks5smy_pgaJpZM4PlMjR
.

/lgtm


Make reviewing of new-docs easier by marking them as binary.

The new-docs contains the generated output for the documentation website. However, this makes reading/reviewing PRs a bit more difficult, as a lot of it is "noise". Therefore, could the new-docs be marked as binary in the .gitattributes so that GitHub hides the diff?
If you still would want to review the new-docs output, a preview deployment GitHub action might be a good solution.

I agree there’s a lot of noise (more than we expected!) due to these new-docs. It would be nice to address this in some way, at least until the old docs are fully removed.
However, marking these as binary seems like the wrong solution — they’re not binary files. We could just .gitignore them (going against our earlier decision, @jackfranklin), and perhaps do a preview deployment as you suggest.

The main thing I wanted to achieve with having them in the repo was to ensure that as people worked they didn't introduce any bugs/issues with the documentation. e.g. for each change, I wanted to force that the new-docs were able to be generated successfully and the diff looked reasonable.
What about if we:

gitignore the new-docs folder
Ensure on CI for each PR that the docs get generated successfully
Try to add linting or something to ensure all new code is documented via the TSDoc comments
Add a GitHub preview deploy to each PR

And that way we'd still ensure the docs are in a good state but avoid the diff noise on each PR.
WDYT?

Since new-docs are machine-generated and we don't want git to merge any generated output (instead the docs should be regenerated), marking them as binary appears to be appropriate per the git documentation: https://git-scm.com/book/en/v2/Customizing-Git-Git-Attributes#_binary_files
That said, adding them to the .gitignore with the other steps seems like a good solution as well 👍


WIP: Support '/open', '/close', and new '/completion' and '/completionItem/resolve' OmniSharp end points

This is a work in progress. It's related to https://github.com/OmniSharp/omnisharp-roslyn/pull/884
TODO:

[x] Handle commit characters properly
[ ] Handle multi-line text edits

I don't believe there is anything left in this PR that needs to be implemented.


Get entries in big endian format

Hi,
As the current implementation creates a reversed linked list (little endian notation), I thought it'd be a good idea to have a method which formats the result of the getEntriesPaginated method and returns everything in the big endian notation.
Example:
Considering the following linked list: 1 2 3 4 5 6 7 8, by calling the getEntriesPaginatedInOrder with start equal to SENTINEL and pageSize to 4, the result will be 5 6 7 8, compared to the default getEntriesPaginated call which will result in 8 7 6 5.

@zeroknots, @kopy-kat would appreciate to get your feedback on this!

is this needed onchain? couldnt you just call getEntriesPaginated and then reverse this offchain?

There may be cases where the order of items is important (e.g. rewarding people on a first-come, first-served basis),  therefore manually reverting the list (on-chain) would be an extra step compared to having this natively in the library.

yeah idm this should be a 1 liner with solady libsort but we can add it into the sentinel list too - in that case I think it would also make sense to use libsort since its much more gas efficient - btw note that consumer contracts like smart accounts or modules anyways need to expose this function so any existing contracts wont get this feature out of the box


Dockerfile improvements

Before the fix
Changing any file lead to repeatedly creating new layers for RUN apt update -y  and apt install python3-pip -y which is slow and takes up much space
After the fix
Apply best practices from https://docs.docker.com/develop/develop-images/dockerfile_best-practices/#run and https://docs.docker.com/develop/develop-images/dockerfile_best-practices/#build-cache
RUN apt update -y  and apt install python3-pip -y is cached. The docker build becomes faster, fewer layers are created. Users of the Dockerfile and docker-compose need to download less from Dockerhub if anything in this project is changed

Could be even improve with further PR to run RUN pip3 install --no-cache-dir gunicorn /httpbin only if setup.py or Pipfile or Pipfile.lock changed


Simplify fundeps in From/ToPlutusType

https://github.com/Plutonomicon/cardano-transaction-lib/blob/fb2ae073af1f61b928ab2a3f290954851e5080fd/src/Plutus/ToPlutusType.purs#L41-L43
Should be able to be turned into
class ToPlutusType f t pt | t -> pt f where
  ...

As pt is already determined by t
I'd argue we don't even need f, which removing would simplify the Identity cases

Closing in favor of https://github.com/Plutonomicon/cardano-transaction-lib/issues/452


wait for the response from getChannelTimezone

Summary
Issue:
when the user tries to post a @all/@channel message the alert popup show a message to the user alerting that the user is about to post a message to an X number of users. if the experimental Timezones is enabled, the message also shows the number of timezones together with the number of users.
the issue is the timezone number was wrong when you at mention the channel, it was getting an old value from another channel. if the user dismisses the pop-up and posts a second message the information show correctly, however, if the user changes the channel and post an at channel again the number is wrong.
this PR fix this issue, waiting for the response from the function before post shows the pop-up.
thanks for @saturninoabril for the guidance in how to use properly the async.
Ticket Link
https://mattermost.atlassian.net/browse/MM-13848
Checklist

[x] Ran make check-style to check for style errors (required for all pull requests)
[x] Ran make test to ensure unit and component tests passed
[x] Added or updated unit tests (required for all new features)

@cpanato Is this ready for review?

Yes @hanzei

after @enahum point out one item, I realize the state variable was completely wrong was channelMembersCount which is not correct since this variable holds the timezone count.

@enahum agree but this can be a follow up

@cpanato yes that is why I marked the PR as ready to merge.. feel free to merge it at any time. Isn't this one to be cherry-picked for 5.9?

@enahum checking with @amyblais

cherry picked f8a51cd90b997b0d0cde7a9fdb95eff162683dfc into release 5.9 branch


Need direct download links

Needed download links for edge browser and for adk 2004 , addson pe 2004
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: d651de3c-b8eb-e8d9-8138-1eadb758bf2e
Version Independent ID: a19a8b65-0f4a-efd5-7f83-56bd07cc8c3b
Content: What's new in Windows 10, version 2004 - What's new in Windows
Content Source: windows/whats-new/whats-new-windows-10-version-2004.md
Product: w10
Technology: windows
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

I would prefer not to have direct download links on this page as these links can change, and it's a blind link. It's better to link to the summary page that is updated if the links change.


Doesn't work on Colab

Getting this error when trying to run it: tcmalloc: large alloc<PHONE_NUMBER>8 bytes
Have followed all instructions.
python blink/main_dense.py -i is the command it fails on.

@aced125 Hi, the model requires a large amount of memory to run. The issue seems to be complaining that there's isn't enough memory available.

Yeah, this model requires a large RAM to run, according to this: #2, does anyone finds a workaround.


Generate api refs with typedoc

API refs now on standalone site (typedoc, served via the public folder of a next.js app lol)
Update code block generation to link to new site

And fix CI


istio mc does not support dynamically update secrets

Create a istio mc control panel.
Create a istio remote cluster.
Create a secret for remote cluster and label the secrets as follows:

kubectl create secret generic ${CLUSTER_NAME} --from-file ${CLUSTER_NAME} -n istio-system
kubectl label secret ${CLUSTER_NAME} istio/multiCluster=true -n istio-system

The cluster was not added to cluster registry.

The root cause is that the secretecontroller do not support update yet https://github.com/istio/istio/blob/master/pilot/pkg/config/clusterregistry/secretcontroller.go#L90-L96
/cc @sdake @sbezverk

code is merged


README.* should serve as index even when it has frontmatter

It's confusing for users that when they add frontmatter to a readme (e.g. to specify a layout) it stops generating index.html, even if they don't have any other files producing an index.html.
Would it make sense to adjust this logic so that README's always work like index files unless there already is another index file, or they explicitly set a different permalink?

I think that makes sense. It'd be a matter of adding || site.pages.find { |page| page.relative_path =~ README_REGEX } to https://github.com/benbalter/jekyll-readme-index/blob/master/lib/jekyll-readme-index/generator.rb#L25.

:+1: I could use this also.

If anyone is interested in submitting a pull request for this, I'd be more than glad to review it. There's some pseudo code above which may help.

Hello,
Is there any support for README.adoc files?


feat(shim): Removing support for BITBOX shim

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Use the callback passed to Session.options.onSend everywhere

The callback passed to this function in Session.sendBatch is supposed to reset session.sendingBatch to false, making it available for further messages. However, in two cases, this callback was not called:

In ConsoleConnector.send it was simply ignored
In UniversalBot.send it was not called if an error occured

See #1023 for an example where it caused problems.

Nice catch


create Icons for IOS and BROWSER(favicon) does not work

From @mburger81 on March 15, 2017 13:12
Ionic version:  (check one with "x")
[ ] 1.x
[x] 2.x
I'm submitting a ...  (check one with "x")
[x] bug report
[x] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
After creating my  icons using ionic resource --splash and ionic resource --icon the cli generates the pictures for splash and icons. As described in this very old bug [https://github.com/driftyco/ionic-cli/issues/1855], where I also add a comment, the pictures are only created for android and not for ios.
The other thing is, what's about the favicon for the browser? Have a look in the config.xml after creating pictures with cli, I see this new entry
<icon src="resources/android/icon/drawable-xhdpi-icon.png"/>
I think this should be the pictures which will be used instead of the favicon, but this does not work.
Expected behavior:

Please close the CLI bug for creating also the IOS pictures.
If the pictures from android resource configured as icon in config.xml should be the favicon for browser, it does not work. If this should be the picture for PWA icons, I think the created pictures for browser and PWA could and should be the same.
If I misunderstand anything, this should also be a feature request :) Please create also favicon on creating resource

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
Cordova CLI: 6.5.0
Ionic Framework Version: 2.2.0
Ionic CLI Version: 2.2.1
Ionic App Lib Version: 2.2.0
Ionic App Scripts Version: 1.1.4
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Linux 4.8
Node Version: v6.10.0
Xcode version: Not installed

Copied from original issue: driftyco/ionic#10787

The issues for ios creation has been resolved in CLI v3 beta. The favicon for web is something that we plan on adding in the future but it will likely be closer tied to our pwa/web plugins instead of cordova. Let me know if you have any other questions. Thanks!

Ok, I think favicon for web is something nice to have and it does not cost a lot.
btw when do release the next cli3 beta which close this bug https://github.com/driftyco/ionic-cli/issues/1973 we are still waiting for the bug fix to test your new cli :)
Tomorrow was three days ago :) JOKE

Today it will be released. 😄  I think the favicon is something we all want just trying to identify the right way to integrate it.


remote_config.go fetchConfig failed - AccessDenied error

Using v5.6.0 and seeing this error during initialization:
gobrake: 2022/11/29 22:22:38 remote_config.go:71: fetchConfig failed for https://notifier-configs.airbrake.io/2020-06-18/config/310441/config.json. Reason: <?xml version="1.0" encoding="UTF-8"?>
2022-11-29T22:22:38.337000+00:00 2022/11/29/[$LATEST]219418aad2fa41a5adc5368cd5b2ef31 <Error>AccessDenied<Message>Access Denied</Message><RequestId>9SDF86VD99EY93HY</RequestId><HostId>UwlWU4vbP+Bb4wbC9lsQQ+gHrwsAqr/vMJ/Xd9TTlX7PtJEieB57E9k4iLcJsM9l3C7zEnBuRoc=</HostId></Error>

Is that URL supposed to be publicly readable?

Hey @coopernurse. Can you please email us at<EMAIL_ADDRESS>in order to avoid leaking sensitive information?

Why is this issue closed?  From my read of remote_config.go this request has nothing to do with the customer's credentials.
Either this URL is incorrect: https://notifier-configs.airbrake.io/2020-06-18/config/310441/config.json
Or the S3 perms on it are wrong (need  to allow public read).


systemd upgrade failure

Version
10.0.22621.521
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>-microsoft-standard-WSL2
Distro Version
Ubuntu 22.04.1
Other Software
No response
Repro Steps
run apt update and check if new version of systemd is available
run apt upgrade systemd
Expected Behavior
successful package upgrade
Actual Behavior
Output on Ubuntu:
Unpacking systemd (249.11-0ubuntu3.6) over (249.11-0ubuntu3.4) ...
dpkg: error processing archive /tmp/apt-dpkg-install-eZFUq7/3-systemd_249.11-0ubuntu3.6_amd64.deb (--unpack):
 unable to make backup link of './usr/lib/tmpfiles.d/tmp.conf' before installing new version: Invalid cross-device link
dpkg-deb: error: paste subprocess was killed by signal (Broken pipe)
dpkg: considering deconfiguration of systemd, which would be broken by installation of udev ...
dpkg: yes, will deconfigure systemd (broken by udev)

Output on Debian:
Unpacking systemd (247.3-7+deb11u1) over (247.3-7) ...
dpkg: error processing archive /var/cache/apt/archives/systemd_247.3-7+deb11u1_amd64.deb (--unpack):
 unable to make backup link of './usr/lib/tmpfiles.d/tmp.conf' before installing new version: Invalid cross-device link
dpkg-deb: error: paste subprocess was killed by signal (Broken pipe)
Errors were encountered while processing:
 /var/cache/apt/archives/systemd_247.3-7+deb11u1_amd64.deb
E: Sub-process /usr/bin/dpkg returned an error code (1)

Diagnostic Logs
No response

Quick workaround is to start w/out systemd, upgrade, then re-enable systemd = true in /etc/wsl.conf. Tested on Ubuntu.

Duplicate of https://github.com/microsoft/WSL/issues/8847

/dupe #8847
Should be resolved with 0.68.2


Fix path resolution for local paths mixed with absolute URLs

I think this solves the same issue as https://github.com/tangrams/tangram-es/pull/1658 but with a more limited code change.

Yeah considering this only (in practice at least) affects desktop versions, I think this should suffice as a stop-gap solution.
Thanks

Honestly I mostly did this to just make rebasing my other branch easier - so thanks :)


HTTP: Rename OSSL_HTTP_REQ_CTX_i2d()

Rename the function OSSL_HTTP_REQ_CTX_set1_req(),
which easier to understand and actually more to the point.
This PR has been carved out from #14630 for alleviating reviews,
making this PR trivial.

This pull request is ready to merge

Pushed - thanks @t8m


No database shows in the attached PostgreSQL server if there is another attached PostgreSQL server with a database

OS: All
Build Version: 20200908.6
Repro steps:

Create several Databases for PostgreSQL server.
Attach PostgreSQL server with a database (add the database name next to the connection string).
Attach the above PostgreSQL server without database (remove the database name from the connection string).
Expand and check the attached PostgreSQL server without database.

Expect:
It shows all databases in the attached PostgreSQL server.

Actual:
No database shows in the attached PostgreSQL server.

More Info:
This issue does not reproduce if detach the PostgreSQL server with a database.

Related #1692


Brainfuck error

ciphey -t '++++++++++[>+>+++>+++++++>++++++++++<<<<-]>>>------------------....'
Traceback (most recent call last):
File "/usr/local/bin/ciphey", line 8, in
sys.exit(main())
File "/usr/lib/python3/dist-packages/click/core.py", line 829, in call
return self.main(*args, **kwargs)
File "/usr/lib/python3/dist-packages/click/core.py", line 782, in main
rv = self.invoke(ctx)
File "/usr/lib/python3/dist-packages/click/core.py", line 1066, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/usr/lib/python3/dist-packages/click/core.py", line 610, in invoke
return callback(*args, **kwargs)
File "/usr/local/lib/python3.9/dist-packages/ciphey/ciphey.py", line 263, in main
result = decrypt(config, kwargs["text"])
File "/usr/local/lib/python3.9/dist-packages/ciphey/ciphey.py", line 32, in decrypt
res: Optional[iface.SearchResult] = config.objs["searcher"].search(ctext)
File "/usr/local/lib/python3.9/dist-packages/ciphey/basemods/Searchers/ausearch.py", line 281, in search
self.recursive_expand(node)
File "/usr/local/lib/python3.9/dist-packages/ciphey/basemods/Searchers/ausearch.py", line 221, in recursive_expand
self.expand_decodings(node)

This isn't really a bug. Your code ++++++++++[>+>+++>+++++++>++++++++++<<<<-]>>>------------------.... decodes to  4444 which isn't English or anything that Ciphey's checkers would catch. This is how Ciphey checks if something is successfully decoded/decrypted or not: https://github.com/Ciphey/Ciphey/wiki/Checkers,-explained
If it's not English, JSON, or anything in pyWhat (URLs, API keys, email addresses, phone numbers, etc.) then Ciphey won't catch it.
If you want to test Ciphey run this: ciphey -t "++++++++++[>+>+++>+++++++>++++++++++<<<<-]>>>++++++++++++++.>++++.+.++++++++++.<<++.>>----------.++++++++++.<<.>+++++++++++++.<.>>+.<++++.>-.+."