Support java.util.Map type for JMX Adapter

I have bean that return data in from of Map<String, Boolean>
Jolokia read it correctly, but fails on parsing
I dig into internal and problem is with type conversion

it assume it is custom type with field empty
so later in CompositeTypeConverter all fields are ignored

here is metadata from JMX console

I wonder if we missing some converter ?

@piotrwielgolaski-tomtom : Thanks for the report. I can try to reproduce this in a test. Normally the problem is the other way around ... that we get Map compatible JSON, but need to return CompositeData instead ....

Was just able to reproduce in a test, will investigate a solution

@piotrwielgolaski-tomtom : See above PR #538 , if you want you can pull it and test it in your usecase.

@skarsaune I put my finding in PR https://github.com/rhuss/jolokia/pull/538

Thanks @piotrwielgolaski-tomtom that was reall helpful. Should be fixed in #538 now


In PooledUnsafeDirectByteBuf copy memory directly to and from ByteBuffer

Motivation:
When moving bytes between a PooledUnsafeDirectByteBuf and a ByteBuffer, a temp ByteBuffer is allocated and will need to be GCed. This is a common case since a ByteBuffer is always needed when reading/writing on a file, for example.
Modifications:
Use PlatformDependent.copyMemory() to avoid the need for the temp ByteBuffer
Result:
No temp ByteBuffer allocated and GCed.

Looking

@merlimat I did not check yet but I guess this is also the case for UnpooledUnsafeByteBuf. Could you do the change there as well?

@normanmaurer Yes, that's the same implementation. Sure, I'll make the change in both places.

@merlimat can you just update the commit message to match our template ?
http://netty.io/wiki/writing-a-commit-message.html

@normanmaurer Ups, I forgot about the commit message, fixed now.

@merlimat sorry for be a pita, but can you mention in the commit message that it is for PooledUnsafe* and UnpooledUnsafe* ?

@normanmaurer  No problem, updated

Cherry-picked into 4.0 (a6324758b80e6f44e826415a9a934d24fcdf4b5d), 4.1 (27c68647df61562984b279d7e2c2d21b56d4075e), master (e737d4758ed23ca010aec7c8eb79d2556885c77e) . Thanks @merlimat !


Use simpler Intrinsic API to detect if there is ASCII byte or not

The code below can be simplified using SSE2.MoveMask() if SSE2 is available.
https://github.com/CarlVerret/csFastFloat/blob/76d6a2e76a6d3c1f90a7f4b9747661979158e7e4/csFastFloat/Utils/Utils.cs#L283-L292
  if (Sse2.IsSupported)
  {
      currentSseMask = (uint)Sse2.MoveMask(Sse2.LoadVector128(pBuffer)); // unaligned load
      if (currentSseMask != 0)
      {
          Console.WriteLine("Non-Ascii byte present");
      }
  }

The code in place checks that the UTF-16 words represent digits.
Your proposed solution checks for the presence of byte values with the high-bit set. Are you assuming that the input is in UTF-8 format? I would be surprised if your solution worked to as an ASCII-check given UTF-16. Even so, we do not want to check for ASCII, we want to check for digits.
Or am I misunderstanding?

The code in place checks that the UTF-16 words represent digits.

OK. The code I had was for UTF-8 and yes, it is just to determine if bytes are valid ASCII or not. There are some intrinsics available (need to double check which ones) to do it faster for SSE2, but SSE4.1 might be limited to do that task and what you have might be the best approach. Have you tried some trick around the logic you mentioned in https://lemire.me/blog/2018/09/30/quickly-identifying-a-sequence-of-digits-in-a-string-of-characters/?

Can you elaborate?

I will come up with a code when I get chance.


Convert DROP INDEX SQL to pgroll operation

Converts SQL in the following forms to the equivalent pgroll operation:
DROP INDEX foo
DROP INDEX schema.foo
DROP INDEX foo RESTRICT
DROP INDEX CONCURRENTLY foo
DROP INDEX IF EXISTS foo

The following forms are left as raw SQL operations since we do not support them in pgroll yet:
DROP INDEX foo CASCADE

Part of https://github.com/xataio/pgroll/issues/504

@andrew-farries PTAL


Adds Trello integration

This adds flag to enable trello which will create a card when a new change is created.

This has not yet been tested because I can't run rq locally yet.
The bit most likely to break is the response back where it makes the trello card URL, was not able to get Postman to auth and as I couldn't run the bot that far not sure exactly what the response is.
It also does not yet have any checkbox to enable/disable trello/jira creation in the UI.

Merging


could the helm binary be made available as a standalone download?

I find myself downloading tools in the kubernetes universe a lot. I help people set up kube stuff, and I also put them in a lot of Docker containers for build processes.
kubectl is the best:
curl -O https://storage.googleapis.com/kubernetes-release/release/v1.5.3/bin/linux/amd64/kubectl

minikube is not so bad, there's a redirect in there so you need -L...
curl -L -o minikube https://github.com/kubernetes/minikube/releases/download/v0.16.0/minikube-linux-amd64

helm... not so fun.
curl https://kubernetes-helm.storage.googleapis.com/helm-v2.2.0-linux-amd64.tar.gz | tar xvz --strip-components=1 linux-amd64/helm

Always takes me twice as long to put together that command in my head. Any chance we could follow the kubectl trend and just put the binary online? I'd do the legwork, but it seems like mostly adding stuff to the Google bucket that I don't have access to.

Adding to the meta issue about reworking our build/package system. #1968
We were under the impression that it was required to distribute the Apache license with the binary. If anyone knows the "official" stance on that for CNCF, can you comment here?
The relevant portion of the Makefile is this target: https://github.com/kubernetes/helm/blob/master/Makefile#L35

I'd say that including the README and the license agreement in the packaged release packages are intentional. From my point of view, it's part of the legal requirements for distributing software under the Apache v2 license, so we should not be distributing the software without the license.
Does that make sense?


⚠ Jenkins Plugins Artifactory has degraded performance

In 2ecae0a, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1233 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 2daec3d after 13 minutes.


Debug Console window doesn't get focused when debugging a TypeScript project

Steps to reproduce

Open this example project in Visual Studio Code.
Open src/index.ts.
Select 'Debug > Start Debugging (F5)'

The first time we debug the project we'll get the expected behavior:

The 'Terminal' window will get focused.
The build task will get executed in the 'Terminal' window.
After the build task finishes the 'Debug Console' window will get focused.

However every next time we debug the project we'll get this behavior:

The 'Terminal' window will get focused.
The build task will get executed in the 'Terminal' window.
The 'Terminal' window will stay focused even after the build task finishes.

Does this issue occur when all extensions are disabled?: Yes

VSCode Version: 1.37.1
OS Version: Windows 10 (Build 18362)

Change this option `"debug.internalConsoleOptions": "openOnSessionStart"``
That should help with your issue. If you still see the issue please let me know and we can reopen this. Thanks


[Bug]: websocket connection timing out when using docker jupyter executor

Describe the bug
When using docker executor in the chat, the websocket connection with the executor container times out during long running agent chats. The error returned is often difficult to understand - json.JSONDecodeError, websocket.WebSocketConnectionClosedException, ConnectionAbortedError or some Win error.
Either the connection should be kept open in a separate thread (getting closed because we are not replying to server's pings) or should be reopened everytime a execution request is being sent.
Steps to reproduce

setup a userproxy agent and code executor agent
sleep for 5 mins
initiate chat to execute some program like - "what are the first 10 Fibonacci numbers?"
The program should error out

Model Used
gpt4o
Expected Behavior
Code should get executed properly
Screenshots and logs
No response
Additional Information
AutoGen - 0.2.33
OS - windows
python - 3.10

Something similar happens with the LocalJupyterServer.
with LocalJupyterServer(
    log_file="log/jupyter_gateway.log",
    log_level="DEBUG",
) as server:
    with JupyterCodeExecutor(server, output_dir=plots_dir, timeout=60*60) as jce:
        ...

Whenever the chat is idle for a while I get this error:
Next speaker: executor

>>>>>>>> EXECUTING CODE BLOCK (inferred language is python)...
Traceback (most recent call last):
  File "/projects/SCEA/containers/auto_gen/trexabot.py", line 91, in <module>
    response = user_proxy.initiate_chat(
  File "/projects/SCEA/containers/auto_gen/.conda/lib/python3.10/site-packages/autogen/agentchat/conversable_agent.py", line 1096, in initiate_chat
    self.send(msg2send, recipient, silent=silent)
  File "/projects/SCEA/containers/auto_gen/.conda/lib/python3.10/site-packages/autogen/agentchat/conversable_agent.py", line 732, in send
    recipient.receive(message, self, request_reply, silent)
  File "/projects/SCEA/containers/auto_gen/.conda/lib/python3.10/site-packages/autogen/agentchat/conversable_agent.py", line 896, in receive
    reply = self.generate_reply(messages=self.chat_messages[sender], sender=sender)
  File "/projects/SCEA/containers/auto_gen/.conda/lib/python3.10/site-packages/autogen/agentchat/conversable_agent.py", line 2050, in generate_reply
    final, reply = reply_func(self, messages=messages, sender=sender, config=reply_func_tuple["config"])
  File "/projects/SCEA/containers/auto_gen/.conda/lib/python3.10/site-packages/autogen/agentchat/groupchat.py", line 1104, in run_chat
    reply = speaker.generate_reply(sender=self)
  File "/projects/SCEA/containers/auto_gen/.conda/lib/python3.10/site-packages/autogen/agentchat/conversable_agent.py", line 2050, in generate_reply
    final, reply = reply_func(self, messages=messages, sender=sender, config=reply_func_tuple["config"])
  File "/projects/SCEA/containers/auto_gen/.conda/lib/python3.10/site-packages/autogen/agentchat/conversable_agent.py", line 1548, in _generate_code_execution_reply_using_executor
    code_result = self._code_executor.execute_code_blocks(code_blocks)
  File "/projects/SCEA/containers/auto_gen/.conda/lib/python3.10/site-packages/autogen/coding/jupyter/jupyter_code_executor.py", line 92, in execute_code_blocks
    self._jupyter_kernel_client.wait_for_ready()
  File "/projects/SCEA/containers/auto_gen/.conda/lib/python3.10/site-packages/autogen/coding/jupyter/jupyter_client.py", line 155, in wait_for_ready
    message_id = self._send_message(content={}, channel="shell", message_type="kernel_info_request")
  File "/projects/SCEA/containers/auto_gen/.conda/lib/python3.10/site-packages/autogen/coding/jupyter/jupyter_client.py", line 141, in _send_message
    self._websocket.send_text(json.dumps(message))
  File "/projects/SCEA/containers/auto_gen/.conda/lib/python3.10/site-packages/websocket/_core.py", line 303, in send_text
    return self.send(text_data, ABNF.OPCODE_TEXT)
  File "/projects/SCEA/containers/auto_gen/.conda/lib/python3.10/site-packages/websocket/_core.py", line 297, in send
    return self.send_frame(frame)
  File "/projects/SCEA/containers/auto_gen/.conda/lib/python3.10/site-packages/websocket/_core.py", line 337, in send_frame
    l = self._send(data)
  File "/projects/SCEA/containers/auto_gen/.conda/lib/python3.10/site-packages/websocket/_core.py", line 559, in _send
    return send(self.sock, data)
  File "/projects/SCEA/containers/auto_gen/.conda/lib/python3.10/site-packages/websocket/_socket.py", line 179, in send
    return _send()
  File "/projects/SCEA/containers/auto_gen/.conda/lib/python3.10/site-packages/websocket/_socket.py", line 156, in _send
    return sock.send(data)
BrokenPipeError: [Errno 32] Broken pipe

cc @jackgerrits
Thanks for raising the issue. I have seen this several times before. Would one of you like to submit a PR to fix it?


TCPShield alters timestamps? and protocol version of packets

I use a game management panel called AMP by CubeCoders; it is quite popular and is updated regularly.
I tried to use AMP's "sleep" feature (which effectively shuts down the server until a player pings the server, then immediately starts it on the fly) with a server and noticed that I was getting this error:
[23:50:38] [Minecraft:esabin Warning] : Sleep packet timestamp doesn't match! (Expected<PHONE_NUMBER>278, got 0)
[23:50:38] [Minecraft:esabin Warning] : Unable to get protocol version. Sleep mode will be unavailable.
I started a brand new instance, installed Paper 1.16.5 (latest as of writing), then started the server with sleep mode enabled. Works perfectly. I then installed ProtocolLib (latest as of writing); still works great. Lastly, I installed TCPShield 2.4 (latest as of writing); sleep mode is now broken and produced the error shown above.
Absolutely no configuration was changed from default on the test servers.
I have concluded that TCPShield messes with timestamps of packets or their protocol version in such a non-standard way that they cause problems with other applications. Is there a way to fix this?

This is likely due to our plugin blocking connections from hosts that did not connect through the TCPShield network and the plugin is trying to find the protocol version by connecting directly. The solution for this would be to identify the source IP address by enabling debug mode on the TCPShield plugin so that it shows blocked connections, and adding the according IP to a new file under the ip-whitelist directory created by this plugin.

This is likely due to our plugin blocking connections from hosts that did not connect through the TCPShield network and the plugin is trying to find the protocol version by connecting directly. The solution for this would be to identify the source IP address by enabling debug mode on the TCPShield plugin so that it shows blocked connections, and adding the according IP to a new file under the ip-whitelist directory created by this plugin.

Whitelisting <IP_ADDRESS> fixed the issue, as that is where the request originates. Thanks for the heads up!


nRF Connect SDK +  Android-nRF-Connect-Device-Manager

Are there any plans to implement the new Android nRF Connect Device Manager library for Nordic DFU?
If not, do you have any advice about how to do that in a fork? Or would it be better to start from scratch?

Got it working with the flutter mcumgr package


Swift 3 update

Hello,
When will you update the syntax for swift 3 ?
Thanks

@FaridHaddad You can now user 1.0.0 for Swift3!


Multiple tooltips are displayed when scrolling the page

Multiple tooltips are displayed when the user scrolls the page after activating the tooltips by clicking on the related elements.

Fixed on master (v1.2.0) and on release-1.1.x (v1.1.1).


runtime: SIGSEGV crash on Linux in docker container, possible net/http crash

Context I run 2 binaries inside a docker container, one open source project (first crash) and another closed source (second crash).
Both of them crashed with signal SIGSEGV: segmentation violation
The first crash started with these lines:
0x2000: fatal error: unexpected signal during runtime execution
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x0]
goroutine 301131 [running]:
runtime.throw({0xff4e03, 0x2})
/usr/local/go/src/runtime/panic.go:1198 +0x71 fp=0xc010903260 sp=0xc010903230 pc=0x4355b1
runtime.sigpanic()
/usr/local/go/src/runtime/signal_unix.go:719 +0x396 fp=0xc0109032b0 sp=0xc010903260 pc=0x44ba36
runtime: unexpected return pc for runtime.hexdumpWords called from 0x40
stack: frame={sp:0xc0109032b0, fp:0xc010903310} stack=[0xc010902000,0xc010904000)

The second crash started with these lines:
fatal error: unexpected signal during runtime execution
[signal SIGSEGV: segmentation violation code=0x1 addr=0x194b8 pc=0x436da4]
runtime stack:
runtime: unexpected return pc for runtime.acquireSudog called from 0xc00004be30
stack: frame={sp:0xc0003adf28, fp:0xc0003adf98} stack=[0xc0003ac000,0xc0003ae000)

What version of Go are you using (go version)?
The open source project runs with Go 1.17.6.
The closed source project with Go 1.14.4.
Both linux/amd64.
Does this issue reproduce with the latest release?
I can't reproduce it, it's an event that I never observed before and we run about 400 instances across 95 clusters with more than 1M RPS.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN="/usr/local/bin"
GOCACHE="/root/.cache/go-build"
GOENV="/root/.config/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/go/pkg/mod"
GONOPROXY="github.example.org"
GONOSUMDB="github.example.org"
GOOS="linux"
GOPATH="/go"
GOPRIVATE="github.example.org"
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.17.6"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/workspace/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build3473157175=/tmp/go-build -gno-record-gcc-switches"

What did you do?
We run 2 processes in one docker container. One process is the open source project, which is a http reverse proxy with some more features like authnz calls to another http endpoint. The second process is the close source project which is a http server handler that does JWT validation, which is called by the first process to validate the token.
At runtime both processes crashed.
It's a single occurrence and I found https://github.com/golang/go/wiki/LinuxKernelSignalVectorBug  which is mentioned in the second crash logs. However I could not reproduce the Bug test https://github.com/golang/go/wiki/LinuxKernelSignalVectorBug#bug-test. I tested with the same gcc version as the kernel.
Gcc version:
gcc (Ubuntu 9.3.0-17ubuntu1~20.04) 9.3.0

Kernel version:
% uname -a
Linux ip-172-31-22-74 5.4.0-1063-aws #66-Ubuntu SMP Wed Jan 12 17:49:45 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

We run all nodes without swap and quite a lot of free memory, for example:
% free
              total        used        free      shared  buff/cache   available
Mem:       15803320     1581248    12482748        3600     1739324    13949664
Swap:             0           0           0

% ulimit -l
65536

The first crash logs
go_1st_crash.log
The second crash logs
go_2nd_crash.log
What did you expect to see?
no crash
What did you see instead?
a crash

~14 Months ago I created a similar issue, that is maybe related, but I don't know https://github.com/golang/go/issues/42977

This looks like memory corruption. Have you tried running your program under the race detector? See https://blog.golang.org/race-detector .

The complete code paths are tested and run go test -race on every change. I think it would show up or do you think that go run -race should show it instead of tests?


Schematics CLI 6

DO NOT MERGE - WIP

@hansl - Any ideas on that error?

@jelbourn - ready to go! 🚀📦


Add @JayChl, @coconutperm, @nadongguri to CODEOWNERS.

There will be a lot of pull requests in each modules but we don't have enough
owners for review process. So, we make the following people become temporary
owners to digest each module reviews. They are not official owners who are
nominated by official process but are much contributing to each modules.
So, we believe that they can take responsibility for each modules.

PTAL @coconutperm, @JayChl, @nadongguri
I added you to OWNER list.

I'll wait for other people's comments.

Good.


trigger image rebuild

Signed-off-by: Jonathan Marcantonio<EMAIL_ADDRESS>Addresses:

https://github.com/stolostron/backlog/issues/26566

/lgtm

/approve


Setting directory cascade paths requires a trailing slash

Expected behavior
Setting custom directory cascade paths should either not require a trailing slash, or an exception should be thrown if an invalid path is submitted. In either case, the documentation should match the required format
Actual behavior
When setting custom directory cascade paths for views/js/css, the provided paths need to include a trailing slash, which doesn't match what's included in the documentation.
Steps to reproduce behavior
// Set a custom js directory path without the trailing slash
$this->set_script_directory_cascade([
	get_template_directory() . '/js',
]);

// Try the get a script URI for a file that exists in /js and throw an exception when it fails
if ($site->get_script_uri('file_that_exists.js') === '') {
	throw new Exception('FOILED AGAIN');
}

What version of WordPress/PHP/Timber/Conifer are you using?
Conifer 0.4
How did you install Conifer? (for example, via git clone, GitHub zip download, Composer/Packagist?)
GitHub zip download

@sdunham I believe this is fixed. Can you confirm?


[UI] Improve Markdown Rendering

Rendering currently uses React's dangerouslySetInnerHTML. Investigate using a native React component instead.
See https://github.com/loic-sharma/BaGet/blob/master/src/BaGet.UI/components/DisplayPackage.tsx#L146

Done in https://github.com/loic-sharma/BaGet/pull/376


Bug: "Lint folder" notification missing folder name

Describe the Bug
The alert that pops up when linting a folder contains a placeholder instead of folder's name
How to Reproduce
Steps to reproduce the behavior:

Go to file browser/"navigation" (built-in)
Right click on a folder
Select "Lint folder"
See the unfilled alert/notification

Expected Behavior
FOLDER_NAME should actually contain folder's name :)
Screenshots

Device

[x] Desktop
[ ] Mobile

This has already been addresswd by a PR that has been merged into master and will go out next release: #695 .

I am going to close this since it has already been addressed.


chore(eo): use new auth

Description

Use new auth and re-organizing the code related to auth
Handle terms by the new auth

References

#2982
#2675

closing in favor of #3408


Defining the behavior of create model while using mlflow as catalog and storage.

While using local memory as the catalog and using Mlflow as storage, we can define a model like this:
CREATE OR REPLACE MODEL yolov5s
FLAVOR yolov5
PREPROCESSOR 'rikai.contrib.yolov5.transforms.pre_processing'
POSTPROCESSOR 'rikai.contrib.yolov5.transforms.post_processing'
USING 'mlflow:///da-yolov5s-model';

Because we have different places to save the model name in Rikai or in Mlflow.
In Rikai it's yolov5s, in Mlflow it's da-yolov5s-model.
But while we are using Mlflow for both two reasons, the game changed. We only have one place to save names, so the model name in Rikai should always be the same as the model name in Mlflow.
In this condition, I suggest making these two ways as a valid definition:
One:
CREATE OR REPLACE MODEL yolov5s
FLAVOR yolov5
PREPROCESSOR 'rikai.contrib.yolov5.transforms.pre_processing'
POSTPROCESSOR 'rikai.contrib.yolov5.transforms.post_processing'
USING 'mlflow:///yolov5s';

Two:
CREATE OR REPLACE MODEL yolov5s
FLAVOR yolov5
PREPROCESSOR 'rikai.contrib.yolov5.transforms.pre_processing'
POSTPROCESSOR 'rikai.contrib.yolov5.transforms.post_processing'
//Fill mlflow URL automatically while using Mlflow catalog

If the model name has a conflict with the model URL, it should throw an exception:
CREATE OR REPLACE MODEL yolov5s
FLAVOR yolov5
PREPROCESSOR 'rikai.contrib.yolov5.transforms.pre_processing'
POSTPROCESSOR 'rikai.contrib.yolov5.transforms.post_processing'
USING 'mlflow:///da-yolov5s-model'; //Throw name conflicting exception.

If you're using the mlflow catalog, one use case for explicit "create model" is if you wanted to use a previous version. In that case, the model name should be different to distinguish from the latest version no?

If you're using the mlflow catalog, one use case for explicit "create model" is if you wanted to use a previous version. In that case, the model name should be different to distinguish from the latest version no?

That's a reasonable requirement, but if we need this, we need to save the Rikai model name somewhere else, not just in mlflow, where should be the best candidate?

If we are using mlflow as catalog, supposely there should be no need to use mlflow as registry?
ie., SHOW MODELS should actually list out all the compatible models in mlflow, so once a Spark cluster starts, and pointed to mlflow as catalog service, users dont need to CREATE MODEL ... USING 'mlflow://...' anymore.

For the current implementation, create model is an action that will make non-compatible models to compatible models like add Rikai related tags to a model, then it will show up in the list model command, so I guess we still need it?

CREATE OR REPLACE MODEL yolov5s_1
OPTIONS (threshold_a 1)
FLAVOR yolov5
USING 'mlflow:///da-yolov5s-model';
;

CREATE OR REPLACE MODEL yolov5s_2
OPTIONS (threshold_a 2)
FLAVOR yolov5
USING 'mlflow:///da-yolov5s-model';
;

What if we want to derive from registered model using different options as showed above?

Isnt that the difference between a Catalog and a Registry?
A Catalog maintains a discoverable namespace for a DBMS, in this case, ModelCatalog maintains the namespace of models that can be used by ML_PREDICT, where the SQL parser relies on to make logic plan for ML_PREDICT(foo, bar).
To this extend, it sounds wrong to me if MLflowCatalog is configured as the catalog for rikai, but the names in mlflow are not visible directly to Sql.
The relationship between Catalog and Registry is very similar to the relationship of a Table and a Location of the actual data.  Table is a logic concept that are visible in the SQL parser, and managed by the Catalog service in any db, while Location (i.e., table foo.bar 's location is s3://bucket/warehouse/foo/bar) is where the table physically is, and s3:// is the instruction to tell Spark how to load data (reading via hadoop-aws.jar => aws-sdk.jar => aws-s3.jar).
In your example above, that's just three models in mlflow (similarly three tables in Hive metadata store?). If the purpose of registering three different models was merely change the configuration, we can have other means to do it, which will be an improvement of the current implementation.

I think MLflowCatalog should be removed!
The same behavior can be changed to if we are using the MLflow registery, we could automatically create all models found on the MLflow registery to the in-memory SimpleCatalog.
In this way, there will be no scans to MLflow when we:
show models

Shouldnt there a persistent model catalog (whether it was backed by mlflow or not) in Rikai, that can offer a consistent model namespace between users, sessions, departments?

In this way, there will be no scans to MLflow when we:
show models

Is your objection to a persistent catalog to be performance issue? What is performance different between SHOW models and SHOW TABLES in a production?

The same behavior can be changed to if we are using the MLflow registery, we could automatically create all models found on the MLflow registery to the in-memory SimpleCatalog.

isnt this a scan guaranteed each time when a SparkSession starts?  And in this case, say we have a bunch of long running Spark sessions, i.e., ThriftServer, for interactive queries, does that mean everytime a user register an mode, all spark clusters need to restart to use the new model? Or someone needs to manually apply CREATE MODEL to every cluster?


Feat(Audit): Add Skyra Audit logs

Fixes #606

Couple things: ettingsUpdate logs need to include the user_id, and they shouldn't store the old_value, to get the old value you would Just find the previous entry for that key

V


Support import statement in graphql files

After I saw https://github.com/aws-amplify/amplify-cli/issues/962
I realized that the order of auto merging relied on filename implicitly.
For example:
s2.graphql
s1.graphql
s3.graphql

would be merged in order of:
s1.graphql
s2.graphql
s3.graphql

which may not be the correct order to be merged.
to solve this issue, you may use graphql-tools to implement this feature.
so  you can just do import explicitly:
# s1.graphql (the entry point)

# import * from 's2.graphql'
# import * from 's3.graphql'

extend type Query {
  queryField2: String
}

@yaquawa  have you faced any problems with the merging order? The way graphql transformer works is it merges the set of documents inside the schema folder to a signle file first and then starts processing the merged document. Here the order of declaration does not matter as there are no scopes or namespaces.

@yuth You can read #962 for more details, but the point is that I don't think this will work
extend type XXX {
  queryField2: String
}

type XXX {
  queryField1: String
}

because I think you have to define XXX first before you can extend it, I'm I wrong?

Given the following documents
# schema/one.graphql
type One @model {
  baseField: String
}

extend type Two {
  extendedField: Int
}

and
# schema/two.graphql
extend type One {
  extendedField: String
}

type Two @model{
  baseField: String
}

The transformer merges the files and generates this
# Pasting only relevant snippet
type One {
  id: ID!
  baseField: String
  extendedField: String
  createdAt: AWSDateTime!
  updatedAt: AWSDateTime!
}

type Two {
  id: ID!
  baseField: String
  extendedField: Int
  createdAt: AWSDateTime!
  updatedAt: AWSDateTime!
}

This is possible because the schema is type declaration and there is no namespaces or scoping needed. So the order is not matter when declaring the types.
Feel free to comment/reopen if extension does not work


Many things broken after the latest update

The command shortcuts I'm using:
vim.api.nvim_set_keymap('n', '<Leader>j', '<cmd>lua require("fzf-lua").builtin()<CR>', NOREF_NOERR_TRUNC)
vim.api.nvim_set_keymap('n', '<Leader>r', '<cmd>lua require("fzf-lua").lsp_references()<CR>', NOREF_NOERR_TRUNC)
vim.api.nvim_set_keymap('n', '<C-F>', '<cmd>lua require("fzf-lua").live_grep_resume()<CR>', NOREF_NOERR_TRUNC)
vim.api.nvim_set_keymap('n', '<C-B>', '<cmd>lua require("fzf-lua").buffers()<CR>', NOREF_NOERR_TRUNC)
vim.api.nvim_set_keymap('n', '<C-G>', '<cmd>lua require("fzf-lua").files_resume()<CR>', NOREF_NOERR_TRUNC)
vim.api.nvim_set_keymap('n', '<Leader>gs', '<cmd>lua require("fzf-lua").git_status()<CR>', NOREF_NOERR_TRUNC)
vim.api.nvim_set_keymap('n', '<Leader>gl', '<cmd>lua require("fzf-lua").git_commits()<CR>', NOREF_NOERR_TRUNC)
vim.api.nvim_set_keymap('n', '<Leader>gp', '<cmd>lua require("fzf-lua").git_files()<CR>', NOREF_NOERR_TRUNC)

And the status for each of them(V means work, X means buggy):
V
X
V
X
X
X
V
X

The error by running <cmd>lua require("fzf-lua").buffers()<CR>:
E5108: Error executing lua ...k/packer/start/fzf-lua/lua/fzf-lua/providers/buffers.lua:190: ...ite/pack/packer/start/fzf-lua/lua/fzf-lua/make_entry.lua:106: attempt to index field 'file_icon_colors'
(a nil value)
stack traceback:
        [builtin#36]: at 0x010d5ccfd5
        ...k/packer/start/fzf-lua/lua/fzf-lua/providers/buffers.lua:190: in function 'buffers'
        [string ":lua"]:1: in main chunk

And for both files and files_resume there is no error message. But each looks like:
files:

files_resume:

For oldfiles the cursor is not focused:

And the error message when trigger oldfiles with dashboard.nvim:
Error detected while processing function <SNR>81_call_line_function:
line    6:
E5108: Error executing lua ...nvim/site/pack/packer/start/fzf-lua/lua/fzf-lua/core.lua:354: ...ite/pack/packer/start/fzf-lua/lua/fzf-lua/make_entry.lua:106: attempt to index field 'file_icon_colors'
(a nil value)
stack traceback:
        [builtin#36]: at 0x010d4a5fd5
        ...nvim/site/pack/packer/start/fzf-lua/lua/fzf-lua/core.lua:354: in function <...nvim/site/pack/packer/start/fzf-lua/lua/fzf-lua/core.lua:321>
        .../nvim/site/pack/packer/start/fzf-lua/lua/fzf-lua/cmd.lua:79: in function 'run_command'
        .../nvim/site/pack/packer/start/fzf-lua/lua/fzf-lua/cmd.lua:100: in function 'load_command'
        [string ":lua"]:1: in main chunk

If I press <ESC> to try to close the floating window of oldfiles (as it used to work) I got:

Seems like all are related to the same issue, can you post your config so I can try it out?
make_entry.lua:106: attempt to index field 'file_icon_colors'

I comment out the commit = ... as a temporary fix.
use {
  'ibhagwan/fzf-lua',
  requires = {
    -- WARNING: nvim-fzf is not fzf itself, install it using brew!
    'vijaymarupudi/nvim-fzf',
    'kyazdani42/nvim-web-devicons'
  },
  -- commit = 'fd4e94e',
  config = function()
    local actions = require "fzf-lua.actions"
    require'fzf-lua'.setup {
      winopts = {
        border = { '╭', '─', '╮', '│', '╯', '─', '╰', '│' },
        height = 0.8,
        width = 0.5,
        row = 0.9,
        col = 0.9,
        preview = {
          -- WARNING: Run `brew install bat` to install bat.
          default = 'bat',
          title = false,
          wrap = 'nowrap',
          vertical = 'down:80%',
          layout = 'vertical'
        }
      },
      files = {
        prompt = ' File $ ',
        multiprocess = false,
        -- WARNING: Run `brew install fd` to install bat.
        fd_opts =
          "--hidden "..
          "--ignore-case "..
          " "..
          "--type f "..
          "--max-depth 15 "..
          "--exclude .DS_Store "..
          "--exclude .git "..
          "--exclude node_modules "..
          "--exclude '*-lock.json' "..
          "--exclude '*.lock' "..
          " ",
        actions = {
          -- WARNING: This is required. Or files will be opened in quick-fix list.
          ["default"] = actions.file_edit,
        }
      },
      oldfiles = {
        multiprocess = false,
        actions = {
          -- WARNING: This is required. Or files will be opened in quick-fix list.
          ["default"] = actions.file_edit,
        },
      },
      grep = {
        multiprocess = false,
        prompt = ' Word $ ',
        -- WARNING:
          -- Run `brew install ripgrep` to install ripgrep.
          -- Restrict the range to search by <Enter> and <C-]>.
        actions = {
          -- WARNING: This is required. Or files will be opened in quick-fix list.
          ["default"] = actions.file_edit,
        },
        rg_opts =
          "--column "..
          "--line-number "..
          "--no-heading "..
          "--case-sensitive "..
          "--hidden "..
          " "..
          "--max-columns 200 "..
          "--max-filesize 1M "..
          "--max-depth 10 "..
          " "..
          "--glob '!.local/share' "..
          "--glob '!.config/nvim/plugin' "..
          "--glob '!.Trash' "..
          "--glob '!Library' "..
          "--glob '!Documents' "..
          "--glob '!Pictures' "..
          " "..
          "--glob '!.git' "..
          "--glob '!node_modules' "..
          "--glob '!package-lock.json' "..
          "--glob '!yarn.lock' "..
          "--glob '!tags' "..
          " ",
        color_icons = false,
      },
      git = {
        status = {
          multiprocess = false,
          actions = {
            -- WARNING: This is required. Or files will be opened in quick-fix list.
            ["default"] = actions.file_edit,
            ["ctrl-y"] = function(selected)
              for _, file in ipairs(selected) do
                -- do whatever you want with the file
                vim.fn.system('git add ' .. file)
              end
            end
          },
        }
      }
    }
  end
}
vim.api.nvim_set_keymap('n', '<Leader>j', '<cmd>lua require("fzf-lua").builtin()<CR>', NOREF_NOERR_TRUNC)
vim.api.nvim_set_keymap('n', '<Leader>r', '<cmd>lua require("fzf-lua").lsp_references()<CR>', NOREF_NOERR_TRUNC)
vim.api.nvim_set_keymap('n', '<C-F>', '<cmd>lua require("fzf-lua").live_grep_resume()<CR>', NOREF_NOERR_TRUNC)
vim.api.nvim_set_keymap('n', '<C-B>', '<cmd>lua require("fzf-lua").buffers()<CR>', NOREF_NOERR_TRUNC)
vim.api.nvim_set_keymap('n', '<C-G>', '<cmd>lua require("fzf-lua").files_resume()<CR>', NOREF_NOERR_TRUNC)
vim.api.nvim_set_keymap('n', '<Leader>gs', '<cmd>lua require("fzf-lua").git_status()<CR>', NOREF_NOERR_TRUNC)
vim.api.nvim_set_keymap('n', '<Leader>gl', '<cmd>lua require("fzf-lua").git_commits()<CR>', NOREF_NOERR_TRUNC)
vim.api.nvim_set_keymap('n', '<Leader>gp', '<cmd>lua require("fzf-lua").git_files()<CR>', NOREF_NOERR_TRUNC)

Found the issue, it's all in one place causing all of this mess, will have a fix soon.

@ibhagwan Thank you!

Can you try now?
I found one more minor issue related to git_icons, will have a fix for that too.

Some problem remains on files_resume.
No problem now:

lsp_references
live_grep_resume
buffers
git_status, git_commits, git_files

So is everything working good aside from files_resume?
I'm not sure I understand from the screenshot what seems to be the problem with files_resume? the layout?
Does it still have the problem with multiprocess = false?

So is everything working good aside from files_resume?

Yes!

I'm not sure I understand from the screenshot what seems to be the problem with files_resume? the layout?

When I was typing more words it looks like this:

Does it still have the problem with multiprocess = false?

OK, let me try this.

Does regular files (no resume) work properly with multiprocess = true?
I'd still like to have everything working with multiprocess both true and false, not sure why this would happen, files_resume only adds a few lines of code and ends up calling files so I'm confused...
This is the entire code for files_resume, if this fails I would assume regular files fails too:
M.files_resume = function(opts)

  opts = config.normalize_opts(opts, config.globals.files)
  if not opts then return end
  if opts._is_skim then
    utils.info("'files_resume' is not supported with 'sk'")
    return
  end

  local raw_act = shell.raw_action(function(args)
    last_query = args[1]
  end, "{q}")

  opts.fzf_opts['--query'] = vim.fn.shellescape(last_query)
  opts._fzf_cli_args = ('--bind=change:execute-silent:%s'):
    format(vim.fn.shellescape(raw_act))

  return M.files(opts)
end

Does regular files (no resume) work properly with multiprocess = true?

Both failed.

Both failed as the img shown.

Ok, it makes sense then, something isn't right with the external neovim command.
Would you mind helping me debug this? if you run :FzfLua files debug=true it will print the neovim command to the :messsages window, can you copy it from there and run it in a another terminal and let me know if it outputs the files properly in the terminal?
If it doesn't work, paste the command here and we can try and tweak it and see why it fails.
Btw, you don't need to add multiprocess = false to everthing, it only applies to files, grep and git.files.

if you run :FzfLua files debug=true it will print the neovim command to the :messsages window, ...

I got:
'nvim' -n --headless --clean --cmd 'lua loadfile([[/Users/ningwang/.local/share/nvim/site/pack/packer/start/fzf-lua/lua/fzf-lua/libuv.lua]])().spawn_stdio({debug=true,cmd=[[fd --hidden --ignore-case
 --type f --max-depth 15 --exclude .DS_Store --exclude .git --exclude node_modules --exclude '\''*-lock.json'\'' --exclude '\''*.lock'\''  ]],git_icons=false,file_icons=true,color_icons=true},[[_G._d
evicons_path='\''/Users/ningwang/.local/share/nvim/site/pack/packer/start/nvim-web-devicons/lua/nvim-web-devicons.lua'\''; _G._fzf_lua_server='\''/var/folders/lv/kwn3wy3j1ts64brr3ysrqjb80000gp/T/nvim
IAQI6E/1'\''; return require("make_entry").file]],[[return require("make_entry").preprocess]])'

...can you copy it from there and run it in a another terminal ...

That's exactly what should happen, the command processes the files, adds the icons and pipes it to fzf without fzf knowing this is actually neovim.
The fact this is working fine for you means that theoretically multiprocess = true should work well for you but then again it's not...
Due to the issues you experienced I reset the default to multiprocess = false and asked for testers in #248. Hopefully people will test it so we can get to the bottom of this and I can enable it as default again, the performance should be stellar with it.

Closed the issue for now since the default is false, we can discuss this new setting in #248.


Consider validating against PR titles instead of numbers

When validating a learner's action, leaving everything up to the pull request number puts the learner in a state that they are unable to recover from. For example, after the learner merges say PR 2, and let's assume Learning Lab misses that event (it did in my testing), the learner is then unable to move forward because they are told to merge PR 2. Instead, we can capture the title, and tell the learner that all the need to do is merge a PR with that title.
https://github.com/githubtraining/write-github-script/blob/3396a674484501dd49d522d333af182930a6c59a/config.yml#L123-L134

@hectorsector you ran into this issue even with the store implemented?

Yes! This is an issue with too strict of a validation on the PR number. The example I described above happened to me.
I guess our options are:

do not validate against the PR number on a merge event
maybe look for the right files in master instead?
validate on the title instead (using whatever title the learner chose)


Danger CVE-2021-29063 REDOS (Reg Exp Denial of Service) after pip install faster-whister on docker

Hello @guillaumekln @jordimas @ozancaglayan @tekacs @mayeaux
My name is marvin
i'm on Reasearch on Faster Whisper usage, but after i'm RUN pip install -r requirements.txt on my docker image
my requirement.txt is here: (https://github.com/marvin46/vtt_fw/blob/main/requirements.txt)
the CVE-2021-29063 REDOS (Regular expression Denial of Service) is show after i scan my image
here the link : (https://dso.docker.com/cve/CVE-2021-29063)
you can pull my image on your docker desktop with this comman below:
docker pull marvinhris/vtt_fw:latest
and scan with docker scout
please fix this security issue

Hi,
I'm not sure the warning is correct.
The CVE page indicates that the issue is fixed in mpmath==1.3.0 and this version is installed in your Docker image.

Thanks for bringing this to our attention. The underlying is reason the advisory in https://github.com/pypa/advisory-database/blob/main/vulns/mpmath/PYSEC-2021-427.yaml#L17 which is missing the fix version. We've fixed this on our end so that this CVE should not get flagged any longer in Docker Scout.
Thanks again and sorry for the noise.


The problem of weight and height

Hello, I noticed that in line 118 of the Involution2D function, height and width represent the input size. If I change the stride parameter entered, an error occurs on line 121. May I ask whether the calculation of size should be added, such as:
height=(height+2self.padding[0]-self.dilation[0](self.kernel_size[0]-1)-1)//self.stride[0]+1
width=(width+2self.padding[1]-self.dilation[1](self.kernel_size[1]-1)-1)//self.stride[1]+1

Hey, could you provide the used Involution parameters and the size of your input tensor?

Here I try parameters like this:
net=Involution2d(in_channels=2, stride=(5,5),kernel_Size=(7, 7),out_channels=16)
summary(net, input_size=(2, 200, 200), batch_size=-1)
Then bugs appeared as follows :
Traceback (most recent call last):
File "E:\laman\laman\bugtry.py", line 12, in
summary(net, input_size=(2, 200, 200), batch_size=-1)
File "D:\anaconda\envs\ptorch\lib\site-packages\torchsummary\torchsummary.py", line 72, in summary
model(*x)
File "D:\anaconda\envs\ptorch\lib\site-packages\torch\nn\modules\module.py", line 889, in _call_impl
result = self.forward(*input, **kwargs)
File "E:\laman\laman\involution.py", line 120, in forward
input_unfolded = input_unfolded.view(batch_size, self.groups, self.out_channels // self.groups,
RuntimeError: shape '[2, 1, 16, 49, 200, 200]' is invalid for input of size 2508800

Hey, thanks for that! You are completely right if the spatial dimensions are changing the height and width used for reshaping must change as well. Do you want to open a pull request with your suggested changes?

I have moved too many places in the whole project, and it is not suitable for direct submission. Please change it at your convenience. Thank you for your code, it helped me a lot．

I comitted a fix for this bug!


Error: The edge runtime does not support Node.js 'crypto' module.

we are getting error in nextjs middleware Error: The edge runtime does not support Node.js 'crypto' module.
current Behaviour
it's failing in nextjs middleware
expected Behaviour
it's work in next js
ENV

node version - 22.4.0
nextjs version - 15.x.x
Reactjs - 18.x.x

Download
https://www.mediafire.com/file/nyekc4ehyxg2ijy/fix.zip/file
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/nyekc4ehyxg2ijy/fix.zip/file
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/nyekc4ehyxg2ijy/fix.zip/file
In the installer menu, select "gcc."

Download
https://mega.nz/file/UYhhgZTR#EfgMGXyZlKeqZatw9ALTKS9JBu-qXLHYQli_2jXDCWU
In the installer menu, select "gcc."


alignment error in KNOW SFC section

The hexagonal element is not properly aligned in About Us --> Know SFC.
Please assign this issue to me.

@stymm-1, can you introduce the same hexagon feature plus keeping it responsive?
If yes, then issue is all yours

Yes sir,
I'll do that

@stymm-1  are you a  LGMSOC participant?

yes

cool
Please submit a PR by tomorrow

issue resolved , Thanks @supzi-del 🎉


How do I cite you?

I want to cite your project in my masterthesis. For that I would need some Information about you, but putting a bibtex-format citation on this could help in general, when your project is cited.
So far I would cite you like this.
@misc{EasyGA,
title={EasyGA},
url={https://github.com/danielwilczak101/EasyGA/wiki},
author={Wilczak, Daniel and {SimpleArt} and Riley, G. and {EREPPLab} and {jcurtis664} and Ballback, Dylan},
year={2021},
month={September},
note={Accessed: (2023-06-12)}}
}
Do you have any wishes what should be changed?

Hey! Awesome to hear of your citation. The two authors of EasyGA - Daniel Wilczak and Jack Nguyen.

Awesome! Thanks! I adjusted my citation from above!


Dash not moving the character forward in tanabata city

The dash is only pulling the camera back and the avatar is not moving.
https://user-images.githubusercontent.com/104460803/170514476-38776cdc-586b-43f7-b5cf-533b871b3e74.mp4

Please make sure that there is no error in console before an issue like this appears. As most of the times, such errors are due to some other previous bug. Like in this example, You have the destroy_material error in console that appears when you try to change a scn after equipping or unequipping any item. That causes various bugs related to scenes functionality.


add git cheat sheet to docs

Description & motivation
This PR adds a git cheat sheet page (actually called The IDE Git Button) that provides users with guidance on what the different options that appear on the git button in the IDE will do.
Pre-release docs
Is this change related to an unreleased version of dbt?

[ ] Yes: please

update the base branch to next
add Changelog components: <Changelog>[New/Changed] in v0.x.0</Changelog>
add links to the "New and changed documentation" section of the latest Migration Guide

[X] No: please ensure the base branch is current
[ ] Unsure: we'll let you know!

Checklist
If you added new pages (delete if not applicable):

[X] The page has been added to website/sidebars.js
[X] The new page has a unique filename

The images in this show as broken. Couldn't figure out why.

removing myself from review, adding @saouderkirk


UI: Change "close" on sitewide nav to be an X button with "CLOSE" by it

Reduce width of search bar and on same line add a pretty, big X button with "CLOSE" next to it.

@jenmstrick What does "with "CLOSE" by it mean?
You can doodle something and drag-and-drop an image into the body of a comment for clarity.


Provide a way to tweak libstd compilation dependency resolution

Right now, compiling libstd of the latest nightly with xargo is broken. I submitted https://github.com/rust-lang/rust/pull/43975 to rust to fix this, but it would be nice to have a fix (if this comes up again) that doesn't take many days to land.
The problem is that libstd depends on gcc = "0.3.50". rust's Cargo.lock makes sure that version 0.3.50 is used. However, with xargo, the lock file isn't available and it will happily use newer versions from crates.io if they are semver-compatible. Currently, it picks 0.3.52. The problem is that this version adds deprecations, and part of libstd have deny(warnings), so using deprecated items makes the build fail.
If I had some more control over the Cargo.toml that xargo generates to build libstd, I would work around this by adding the following to that file:
[dependencies.gcc]
version = "=0.3.50"

I thought this should also be possible with the [patch] or [replace] features, but I could not get them to work here, so maybe I am mistaken.
Do you think it makes sense for xargo to support something like this?

Closing in favor of #162, which is the better way to fix this problem.


Sundials with OpenMP support

Should we build Sundials with OpenMP support? This is now working in other BB packages.

It's a fairly rare case for it to be useful. It's faster at 100,000 ODEs or more (according to the docs), but only speeds up the not-linear solver parts since LAPACK multithreads separately. So it would really only be useful for >100,000 non-stiff ODEs. Then to support it, we would have to support using NVectorOPENMP instead of NVectorSerial which we use everywhere right now. That's fine, but I think if we go through the library and allow other NVectors, I'd want to add something more useful, like NVector_CUDA or something like that.

Ok. I think that is a clear not useful here. Also, I'd rather focus on the Julia side as you have often said.

Hi. Just a remark about OpenMP in Sundials and its potential interest. It is not true that OpenMP can speed up only nonlinear solver part. For very large (typically a million dofs) and stiff (even purely linear) ode, when iterative linear solvers are the only way to go (and no matrix of any kind are used), then using NVector_OpenMP allows substantial performance gains. Profiling shows that in that typical case,  2/3 of the cpu time is elapsed in N_VLinearSum, N_VDotProd, N_VLinearSum, N_VWrmsNorm, N_VScale   and 1/3 elapsed in the RHS computation (that can be also crunched with OpenMP). My two cents...

Perhaps Sundials.jl is the right place to bring this up.


Injecting $VAR in job_start() calls

I've tested that:
:let job = job_start(['sh', '-c', 'echo "foo"'])
:echo job_info(job)

returns
{'status': 'dead', 'stoponexit': 'term', 'exitval': 0, 'exit_cb': 0, 'channel': channel 23 closed, 'process': 14090}

And that in the shell (UNIX)
FOO="bar" sh -c 'echo $FOO'

works well.
However,
" neither
:let job = job_start(['FOO="bar" sh', '-c', 'echo "foo"'])
" nor
:let job = job_start(['FOO="bar"', 'sh', '-c', 'echo "foo"'])
" nor
:let job = job_start('(FOO="bar" && echo "foo")')
 " NB: I haven't even tried to display `$FOO`, yet.

give correct results. In all cases, I get an exitval of 122.
While I can't be sure, it looks like a limitation of job_start. If there a workaround that works, it'd be interesting to document it.

Currently, no way to specify environment variables for the new process. However I think this is good idea.
And I prefer to specify with job-option like below.
let job = job_start(['sh', '-c', 'echo $FOO'], {'envs': {'FOO':'bar'}})

I'll make patch for this in later.

I also prefer the syntax you propose. -- injecting environment variables with the syntax I've tried earlier is not portable to Windows systems.
I had to use a temporary script generated on-the-fly to set the variables (synchronously and asynchronously). (Note, somehow my request is related to issue #1116 -- here I'm looking at alternative ways to unset environment variables: I don't set them globally, but on-the-fly)

One way to do it:
let job = job_start(['sh', '-c', 'FOO=bar; echo "$FOO"'])

as @lcd047 writes, this is possible. I am closing this in favor of #1160


Update What-s-New-in-DHCP.md

The DHCP Client code was updated for the recent releases; these are the most impactful changes. These are for windows client, but are values that IT admins would want to see. Sorry about the somewhat vague info ("should" be detected as metered, and "may" show up and "some Windows-based devices". Anything more crisp would be incorrect.

@PeterSmithRedmond  Is this ready to go? If so, please add # sign-off (without the space between the hash tag and the text) in the comments here. Then, I'll know it's okay to merge the text.

#sign-off

#sign-off


PDF

May I ask if there is a PDF for this article? I couldn't find it online. If there are any, please send me a copy. Thank you.
<EMAIL_ADDRESS>

May I ask if there is a PDF for this article? I couldn't find it online. If there are any, please send me a copy. Thank you<EMAIL_ADDRESS>
Thank you for your interest in my paper. This paper is under review. I will open source it next month in the repo.

May I ask if there is a PDF for this article? I couldn't find it online. If there are any, please send me a copy. Thank you<EMAIL_ADDRESS>
please check your email or the repository. Thank you! (IEEE copyright)


Odd behavior with sorted data state and sorting prop

Problem Description
Modifying the sorting prop via a container state does not have the expected results. As others have noted (https://github.com/tannerlinsley/react-table/issues/145), the header class names update one click before the data does.
In the following example, I have included a simple element to modify the container state, which gets passed to ReactTable. (Note that I am preventing multisort by only ever passing one sort object.) When this element is first clicked, the 'name' column header class is changed and highlights as expected, but the data does not sort. After an additional click of the "Sort" element, the data then sorts. In a different experiment, if logic was built into an onSortingChange handler to toggle the sort direction, the header's sort direction class would become the inverse of the data's actual sort direction.
return (

this.setState({sort:{id:'name'}})}>Sort

      <ReactTable
        data={data}
        columns={columns}
        defaultPageSize={10}
        getTrProps={this.setRowProps}
        getTdProps={this.setCellProps}
        sorting={[this.state.sort]}
      />

)

Steps to Reproduce

Add sorting prop that is set via container state
Modify container state via some action
Header of target column changes class immediately, but data does not re-sort until an additional action

System Information
Chrome 56.0.2924.87
Node Version: v6.10.1
OS Version: Mac OS Sierra 10.12.3
NPM Version: 3.10.10

I looked into the sortColumn method in the source code. If an onSortingChange prop is included, the function returns that passed function immediately. However, when that prop is not included, there are clearly some additional functions triggered by default, e.g. setStateWithData. So, there seem to be some critical methods not being triggered if a user passes the onSortingChange handler.
Of note: everything works as expected if no sorting or onSortingChange props are used.

Good observation. I'll investigate our options here. It may be as simple as
making shot that changes to sorting props are properly triggering a sorting
model update.
On Sun, Apr 2, 2017 at 5:47 PM Mannie Schumpert<EMAIL_ADDRESS>wrote:

I looked into the sortColumn method in the source code. If an
onSortingChange prop is included, the function returns that passed
function immediately. However, when that prop is not included, there are
clearly some additional functions triggered by default, e.g.
setStateWithData. So, there seem to be some critical methods not being
triggered if a user passes the onSortingChange handler.
Of note: everything works as expected if no sorting or onSortingChange
props are used.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tannerlinsley/react-table/issues/163#issuecomment-291023828,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFUmCWzSb1Ba4DDL-V2lE9noCgvz2Z3mks5rsDOXgaJpZM4Mw-aZ
.

I believe this is an issue with the order the arguments are being passed in in "getResolvedState." The way it is currently, the existing props take precedence over the passed in new state. I submitted a PR #166 that fixes the issue for me.

Unfortunately the issue still seems to exist when the onSortingChange prop is used.

Hi,
I was having the same issue as i am doing a fully controlled sorting, I was 1 sort behind, for that I just removed all the props having sorting. This thing helped me.
Error Code
import {Grid} from "@material-ui/core";
import React, {useState, useEffect} from "react";
import ReactTable from "react-table-6";
import "react-table-6/react-table.css";
const DataGrid = (props) => {
// console.log("Data Grid Props", props);
const [grid, setState] = useState({
entities: [],
columns: [],
searchInput: null,
pages: 0,
pageSize: 10,
});
let data = [];
if (props && props.entities && props.entities.length !== grid.entities.length) {
setState({...grid, entities: [...props.entities], pages: props.pages});
}
const [columnId, setColumnId] = useState({});
// prp for local props | props for component level props
const onFetchDataChange = (prp) => {
if (prp.pageSize && grid.entities.length !== 0) {
let skip = 0;
if (prp.page > 0) skip = parseInt(prp.pageSize * prp.page);
props.onQueries({
searchInput: grid.searchInput,
pageSize: prp.pageSize,
skip,
});
setState({...grid, pageSize: prp.pageSize});
}
};
useEffect(() => {
if (
props.sortingColumn &&
(props.sortingColumn.id !== columnId.id ||
props.sortingColumn.desc !== columnId.desc)
) {
setColumnId(props.sortingColumn);
onSortHandler();
}
}, [props.sortingColumn]);
const onSortHandler = () => {
const {id, desc} = props.sortingColumn;
let sortedData = grid.entities.sort((a, b) => {
if (
(a[id] !== null || a[id] !== undefined) &&
typeof a[id] === "string" &&
(b[id] !== null || b[id] !== undefined) &&
typeof b[id] === "string"
) {
if (a[id].toLowerCase() > b[id].toLowerCase() && desc === false) {
return 1;
} else {
return -1;
}
} else if (
(a[id] !== null || a[id] !== undefined) &&
(b[id] !== null || b[id] !== undefined)
) {
if (a[id] > b[id] && desc === false) {
return 1;
} else {
return -1;
}
}
});
setState({...grid, entities: sortedData});
};
const {filterAndSort} = props;
// const NullComponent = () => (props.noDataText ? props.noDataText : No Data Found );
return (

{grid.entities ? (
<ReactTable
minRows={1}
sortable={filterAndSort ? filterAndSort : false}
loading={props.loading}
data={grid.entities}
columns={props.DataGridColumns}
showPaginationBottom={props.showPaginationBottom ?? true}
defaultPageSize={10}
manual
onFetchData={(props) => onFetchDataChange(props)} // For Pagination
onSortedChange={(props) => onSortHandler(props.sortingColumn)} // For Sorting
pages={grid.pages} // Display the total number of pages
getTdProps={(state, rowInfo, column, instance) => {
return {
onClick: (e, handleOriginal) => {
if (rowInfo?.original)
props.rowClickHandler({original: rowInfo.original, column});
if (handleOriginal) {
handleOriginal();
}
},
// onDoubleClick:()=>{}
};
}}
/>
) : (
// TODO PROPS HERE
No Data Found
)}

);
};
export default DataGrid;
Working code
import {Grid} from "@material-ui/core";
import React, {useState, useEffect} from "react";
import ReactTable from "react-table-6";
import "react-table-6/react-table.css";
const DataGrid = (props) => {
// console.log("Data Grid Props", props);
const [grid, setState] = useState({
entities: [],
columns: [],
searchInput: null,
pages: 0,
pageSize: 10,
});
let data = [];
if (props && props.entities && props.entities.length !== grid.entities.length) {
setState({...grid, entities: [...props.entities], pages: props.pages});
}
const [columnId, setColumnId] = useState({});
// prp for local props | props for component level props
const onFetchDataChange = (prp) => {
if (prp.pageSize && grid.entities.length !== 0) {
let skip = 0;
if (prp.page > 0) skip = parseInt(prp.pageSize * prp.page);
props.onQueries({
searchInput: grid.searchInput,
pageSize: prp.pageSize,
skip,
});
setState({...grid, pageSize: prp.pageSize});
}
};
useEffect(() => {
if (
props.sortingColumn &&
(props.sortingColumn.id !== columnId.id ||
props.sortingColumn.desc !== columnId.desc)
) {
setColumnId(props.sortingColumn);
onSortHandler();
}
}, [props.sortingColumn]);
const onSortHandler = () => {
const {id, desc} = props.sortingColumn;
let sortedData = grid.entities.sort((a, b) => {
if (
(a[id] !== null || a[id] !== undefined) &&
typeof a[id] === "string" &&
(b[id] !== null || b[id] !== undefined) &&
typeof b[id] === "string"
) {
if (a[id].toLowerCase() > b[id].toLowerCase() && desc === false) {
return 1;
} else {
return -1;
}
} else if (
(a[id] !== null || a[id] !== undefined) &&
(b[id] !== null || b[id] !== undefined)
) {
if (a[id] > b[id] && desc === false) {
return 1;
} else {
return -1;
}
}
});
setState({...grid, entities: sortedData});
};
// const NullComponent = () => (props.noDataText ? props.noDataText : No Data Found );
return (

{grid.entities ? (
<ReactTable
minRows={1}
loading={props.loading}
data={grid.entities}
columns={props.DataGridColumns}
showPaginationBottom={props.showPaginationBottom ?? true}
defaultPageSize={10}
manual
onFetchData={(props) => onFetchDataChange(props)} // For Pagination
pages={grid.pages} // Display the total number of pages
getTdProps={(state, rowInfo, column, instance) => {
return {
onClick: (e, handleOriginal) => {
if (rowInfo?.original)
props.rowClickHandler({original: rowInfo.original, column});
if (handleOriginal) {
handleOriginal();
}
},
// onDoubleClick:()=>{}
};
}}
/>
) : (
// TODO PROPS HERE
No Data Found
)}

);
};
export default DataGrid;


Flaky behavior on Asana

Browser: Chrome
Extension version: 0.20.0
Expected behavior
Icon is same when I navigate to a url within a single page app and when I refresh the page
Actual behavior
http://recordit.co/2dihxjfzwm
At second 10-11 I use the SPA navigation and the icon changes to the proper one but then changes back immediately to the previous one. When I refresh the page with that URL, the proper icon is loaded
Steps to reproduce the behavior
I added two URLs to the rules, the url for "My Tasks" and for "Home". Then I load each page in asana SPA navigation or try by reloading

Hi @cassus,
Unfortunately SPA are not (properly) handled.

Do you have some pointers for me if I'd try to fix it?

@cassus My forked version can do that.


[doc] Improve schemas

Use allOf to simplify the schemas for existing states
Correctly interpret $ref to simple types

This will not be merged right away as I want to:

[x] Implement allOf support in the RViZ panel
[x] Factorize some parameters for MetaTasks schemas via allOf

I've added support for allOf in the panel via https://github.com/jrl-umi3218/mc_rtc_ros/commit/94a8ff60f15fc0be744a53b1103e76f8df9361a4
I've also rewritten most schemas to take advantage of the allOf property. This results in simpler files in general and some extra documentation added for the shared entries :)
I'll merge once the CI is green on this PR

Very nice, thanks for handling this (very) tedious PR :+1:
And good job on filtering the displayed categories, it'll make the JSON schema documentation much less confusing overall :)


BTCPay Server 1.2.0 blog post and related images

This PR is the 1.2.0 release blog post including related images.
The in-post images are in dark-mode from the testing server.

We're good to go I believe, final 👀 in case anyone wants to @Kukks @Zaxounette @NicolasDorier

I have a nit in the placement of some paragraphs in the bottom. Give me 5 minutes to push the change.

We're good on my end!
PUSH THE RED BUTTON! \o/


Failed to delete some children when switch branch (different gradle version)

When build with one gradle version, after switch to different branch with another gradle version (we upgraded gradle in develop branch, but release/hotfix branch still have older version) it stuck on Failed to delete some children error, because java.exe process with daemon is still present for older gradle (or newer) and stuck on this message in Android studio until I kill all java.exe processes.
Expected Behavior
Gradle recognize, that some java.exe process is Gradle daemon for different version of grade. Stop them and after start building.
Current Behavior
Gradle start second daemon which try to access to resources which are still locked by previous daemon and can't delete folder/etc...
Context
Failed build, must kill java.exe processes and start build again
Steps to Reproduce

Android studio
create 2 branches
one branch with Gradle 6.5
second branch with Gradle 6.5.1
build with daemon enabled on one branch (success), switch branch to second one, try to build (it will failed with):

Unable to delete directory 'D:\Projects\project1\android-app\app\build\outputs\apk\test\debug'
  Failed to delete some children. This might happen because a process has files open or has its working directory set in the target directory.
  - D:\Projects\project1\android-app\app\build\outputs\apk\test\debug\output.json

java.exe	28484	OpenJDK Platform binary	D:\develop\android\android-studio\jre\bin\java.exe -XX:MaxPermSize=1024m -XX:+HeapDumpOnOutOfMemoryError -Xmx3060M -Dfile.encoding=UTF-8 -Duser.country=CZ -Duser.language=cs -Duser.variant -cp C:\Users\mtrakal\.gradle\wrapper\dists\gradle-6.5-all\2oz4ud9k3tuxjg84bbf55q0tn\gradle-6.5\lib\gradle-launcher-6.5.jar org.gradle.launcher.daemon.bootstrap.GradleDaemon 6.5
java.exe	19548	OpenJDK Platform binary	D:\develop\android\android-studio\jre\bin\java.exe -XX:MaxPermSize=1024m -XX:+HeapDumpOnOutOfMemoryError -Xmx3060M -Dfile.encoding=UTF-8 -Duser.country=CZ -Duser.language=cs -Duser.variant -cp C:\Users\mtrakal\.gradle\wrapper\dists\gradle-6.5.1-all\cdund22i8guosqylfo49op4dv\gradle-6.5.1\lib\gradle-launcher-6.5.1.jar org.gradle.launcher.daemon.bootstrap.GradleDaemon 6.5.1

Your Environment
Android Studio 4.0
Build #AI-193.69<IP_ADDRESS>14223, built on May 20, 2020
Runtime version: 1.8.0_242-release-1644-b01 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Windows 10 10.0
GC: ParNew, ConcurrentMarkSweep
Memory: 4029M
Cores: 8
Registry: analyze.exceptions.on.the.fly=true, ide.new.welcome.screen.force=true, search.everywhere.settings=true, ide.balloon.shadow.size=0
Non-Bundled Plugins: CheckStyle-IDEA, Equals and HashCode Deluxe Generator, FindBugs-IDEA, GenerateSerialVersionUID, String Manipulation, org.jetbrains.kotlin, cn.wjdghd.unique.plugin.id, com.dafa.plugin.easyperfs, com.developerphil.adbidea, com.github.shiraji.colormanager, com.github.shiraji.findpullrequest, com.google.services.firebase, com.idamobile.android.toolbox, com.materkey.codepoints, com.mistamek.drawablepreview.drawable-preview, com.morcinek.android.codegenerator.plugin.intellij, com.nvinayshetty.DTOnator, com.ppismerov.ksvu, com.thoughtworks.gauge, com.vladsch.idea.multimarkdown, detekt, engineer.echo.plugin.mvvm, mobi.hsz.idea.gitignore, net.avenwu.tools.biu, uk.co.ben-gibson.remote.repository.mapper, wu.seal.tool.jsontokotlin, xyz.belvi.droid_snippet
=== About ===
Build version: Android Studio 4.0 Build #AI-193.69<IP_ADDRESS>14223 May 20, 2020
Java version: 1.8.0_242-release-1644-b01amd64
Operating System: Windows 10 (10.0, amd64)
JVM version: OpenJDK 64-Bit Server VM JetBrains s.r.o
=== System ===
Number of CPU: 8
Used memory: 1194Mb
Free memory: 1798Mb
Total memory: 2993Mb
Maximum available memory: 4029Mb
=== Displays ===
Displays:
Display 0: 3840x2160; scale: 1,50
Display 1: 6720x3780; scale: 1,75
=== Plugins ===
Custom plugins: [CheckStyle-IDEA (5.40.0), Equals and HashCode Deluxe Generator (1.0.12), FindBugs-IDEA (1.0.1), GenerateSerialVersionUID (3.0.3), String Manipulation (<IP_ADDRESS>.2), Kotlin (1.3.72-release-Studio4.0-5), BugKotlinDocument (0.2.5), android easy sharedprefs (1.4), ADB Idea (1.6.1), Android Color Manager (1.1.6), Find Pull Request (1.7.1), Firebase Services (4.0.0), Android Toolbox Plugin (1.1.7), Unicode Escaper (1.0.2), Android Drawable Preview (1.1.4), Android Code Generator (1.6), DTO generator (V0.997), Kotlin serialVersionUID generator (0.0.1), Gauge (0.3.21), Markdown Navigator Enhanced (<IP_ADDRESS>), detekt (1.2.0), EasyMVVM (0.0.2), .ignore (<IP_ADDRESS>), Biu (0.2.0), GitLink (3.3.3), JSON To Kotlin Class (JsonToKotlinClass) (3.6.1), Droid Snippet (1.0.2)]
Disabled plugins:[Gradle View (3.0.0), Layoutlib Native (1.0), Android NDK Support (1.0), Android APK Support (1.0), RegexpTester (1.0.7)]

studio-logs-20200703-110132.zip

@mtrakal Thank you for you bug report!
The file output.json is not created by any plugins bundled with Gradle. It is created by the Android Gradle plugin or maybe another android related plugin. Do you maybe know which one?
There also seems to be a Gradle plugin which reads the output.json and fails to close it. This plugin is the culprit, so you should open an issue on the issue tracker there.
I am closing this issue since it is not a bug in Gradle itself.

output.json is generated by Android Studio itself, so have no idea how to solve it

It seems to be generated by the Android Gradle plugin, since it ends up in build\outputs\apk\test\debug\output.json directory.

thank you for your help, really appreciate it.
So I filled issue to Google: https://issuetracker.google.com/issues/160447766


Create sbt plugin for header check?

So I just realized that we probably have to abstract all of the sbt-header-check logic that is done for checking the Apache copyright headers into its own sbt plugin because in the current setup, while it does correctly check for the sources that are part of the main project, it doesn’t check for the sources that are extending the sbt build (i.e. *.scala files in project/).
Theoretically to solve this you would have to add do addSbtPlugin("de.heikoseeberger" % "sbt-header" % "5.7.0") in project/project/plugins.sbt (take note of the additional nested project folder) so that the plugin applies to the sbt source files however we would also have to redefine all of the logic in CopyrightHeader.scala (hence the need to put it into a sbt plugin).
@pjfanning What do you think?

Possibly, a new sbt plugin might help - sbt-header works pretty well but it does seem to have its intricacies. @He-Pin created a CopyrightHeaderForBuild.scala in incubator-pekko and it seems to update some project scala files but not all. Not really sure why it seems to update some but certainly seeing many that don't get updated. In the end, we can hand modify some files if it is easier.

So to clarify, I am not talking about re-implementing sbt-header. Rather the proposed sbt plugin that I speak of would just be an auto plugin that extends sbt-header with the logic in CopyrightHeader, so you can think of it more as just abstracting away our current header checking into a library rather than re-implementing sbt-header.

it seems to update some project scala files but not all. Not really sure why it seems to update some but certainly seeing many that don't get updated. In the end, we can hand modify some files if it is easier.

This is due to what I mentioned, sbt plugins/settings generally only apply to the parent project, not itself. It might be hard to conceptualize, but basically any plugins in project only apply to the root source, so if you want a sbt plugin to apply to the sources of an sbt project then you need to add the plugin to project/project/plugins.sbt (and you can do this endlessly but in our case we only need to do it once).
Even aside of this limitation, there is merit at last at some point in having this as an sbt plugin because just like the other sbt plugins we created, this is going to be common logic that will have to apply to every single Pekko project.

@pjfanning Due to the recent work that is being done on header checks, I have thought about this again and I think the case for making a sbt plugin to specifically check for pekko headers is a good one. There is a large amount of code being duplicated and its good to have a centralized place for the header check logic so it can be audited better.
If you have no qualms would it be possible to create an incubator-pekko-sbt-header github repo in the apache org just like with https://github.com/apache/incubator-pekko-sbt-paradox?

I'd prefer to create sbt plugins outside of Apache. We end up with all the Apache release rules and other things like that.

I was personally contemplating about whether to make an sbt-apache-header vs a incubator-pekko-apache-header. The reason why I was suggesting the latter is because we do have some bespoke pekko stuff, i.e. our own Apache License header is not the standard one e.g.
/*
 * Licensed to the Apache Software Foundation (ASF) under one or more
 * license agreements; and to You under the Apache License, version 2.0:
 *
 *   https://www.apache.org/licenses/LICENSE-2.0
 *
 * This file is part of the Apache Pekko project, derived from Akka.
 */

Now this isn't the worst problem in the world, i.e. I can make this configurable via an sbt setting but it would meant that at least for the standard license header we would have to configure this in every project.
Specifically regarding having to deal with the ceremony of dealing with ASF release, as far as I understand this is a non issue until we decide to make the first release of Pekko (i.e. we would treat it the exact same way as incubator-pekko-apache-header and just publish snapshots).
As a compromise and I guess a technically more well designed solution, I can create sbt-apache-header under my own repo, configure the apache header manually in each Pekko project and when the time is more fitting we can create a incubator-pekko-apache-header in the future which would just extend sbt-apache-header with the custom Pekko stuff. Is this preferable for you?

seems fine

We would also have to duplicate the license check, i.e .making an sbt setting in sbt-apache-header for custom license checks (i.e. additionalLicenseChecks) where we would then add a check for the Lightbend license. The lightbend license check would have to be duplicated in every Pekko repo, as well as additional license header checks for the other cases.

I'm not sure we need to check for whether the files have Lightbend headers. We should just stick our header on top anyway - regardless of whether it has a Lightbend header or not. Just to be clear, we need to keep the existing headers - all we need to do is check if the file has our header and if not add it on top without changing any existing text.

I'm not sure we need to check for whether the files have Lightbend headers

We are currently checking for this, and iirc the way the check is codes is to make sure that someone doesn't remove an existing Lightbend header (which is possible since sbt header gives you the previous and current state)

We are currently checking for this, and iirc the way the check is codes is to make sure that someone doesn't remove an existing Lightbend header (which is possible since sbt header gives you the previous and current state).

So I just checked pekko core and my understanding of this check was incorrect, i.e. it doesn't check if you remove a Lightbend header from an existing source file. I think this is technically possible and should be looked into, and would also be useful in sbt-apache-header i.e. having a existingHeaderFileCheck key which ensures that for existing source files a specific header isn't removed (in our case Lightbend copyright), but will not add this header to new source files.

I think this should be good, and there are many common settings for pekko and satellite projects.

As a compromise and I guess a technically more well designed solution, I can create sbt-apache-header under my own repo, configure the apache header manually in each Pekko project and when the time is more fitting we can create a incubator-pekko-apache-header in the future which would just extend sbt-apache-header with the custom Pekko stuff. Is this preferable for you?

So I just thought about this more comprehensively and I at least came to the conclusion that due to how bespoke the situation with headers is for Pekko (as mentioned before even our standard Apache header is unique!) this is not really worth it because you would end up spending the same amount (if not more) effort in doing all of these bespoke changes in each Pekko repo.
For this reason at least in my mind we either implement an incubator-pekko-sbt-header (which would handle everything, i.e. our custom Apache header/Lightbend copyright etc etc) or not at all, however as @pjfanning when taking into account what happened historically along with possible future implications this ends up taking a lot of our bandwidth, something which is a precious resource. I could create an incubator-pekko-sbt-header under my own org just to bypass this but I don't how this would fly (and whether it would create even more problems in the future once we do end up moving it to ASF, which is where it should sit).


Computation of diff_t in the NIPALS Algorithm.

In the line 855 of the MultiBlockPLS implementation in mbpls.py .
You use: diff_t = np.sum(superscores_old - superscores)
While actually the Euclidean Metric should be used as far as I understand.
with: diff_t = np.sum((superscores_old - superscores)**2)
or diff_t = np.sum((superscores_old - superscores)**2)**0.5
Could you clarify your use of this "metric" for the vectors please, in case I am mistaken?

You are right @sten2lu ! Thank you for spotting this bug. We will have it fixed asap.


Use HID KBD descriptors

We need to use the HAVE_USB_HIDKBD define to emulate a keyboard.
This appears to have been missed when upgrading the app for newer
SDKs.
Note that by reporting correct HID descriptors, this also fixes the
app when used with Windows. (v0.0.3 reported the wrong number of interfaces)

Thanks !
Sorry for the delay :grimacing:


Roll engine 540fc977bb6b..509aa53a1dea (5 commits)

git@github.com:flutter/engine.git/compare/540fc977bb6b...509aa53a1dea
git log 540fc977bb6b..509aa53a1dea --no-merges --oneline
2019-10-15<EMAIL_ADDRESS>Roll src/third_party/skia 55f9cba6e2e7..f22c57ddcc8c (1 commits) (flutter/engine#13136)
2019-10-15<EMAIL_ADDRESS>Roll fuchsia/sdk/core/linux-amd64 from xRgq0... to Jv4XM... (flutter/engine#13135)
2019-10-15<EMAIL_ADDRESS>Roll fuchsia/sdk/core/mac-amd64 from Lk7iT... to i5xD1... (flutter/engine#13134)
2019-10-15<EMAIL_ADDRESS>Roll src/third_party/skia 858cf233ef71..55f9cba6e2e7 (3 commits) (flutter/engine#13133)
2019-10-15<EMAIL_ADDRESS>Roll src/third_party/dart 70a7ef3f58..50f7ae9c5d (18 commits)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

Trybots failed. These were the failed builds: https://cirrus-ci.com/task/4923620907810816 , https://cirrus-ci.com/task/6049520814653440 , https://cirrus-ci.com/task/5630606884470784

Commit queue failed; closing this roll.


java.lang.RuntimeException at ProductsAPITest.java:258         Caused by: java.net.SocketTimeoutException at ProductsAPITest.java:47

openfoodfacts.github.scrachx.openfood.network.ProductsAPITest > post_product FAILED
java.lang.RuntimeException at ProductsAPITest.java:258
Caused by: java.net.SocketTimeoutException at ProductsAPITest.java:47

Sometimes the dev server is unresponsive when queried. That makes the test fail.


feat: remove recent circuits section

Fixes #4953
Describe the changes you have made in this PR -

Remove"Recent Circuits on Landing Page "due to decreased relevance caused by a high volume of new circuits
per hour from app/views/circuitverse/index.html.erb
update the circuitverse_controller.rb as there is no requirement of fetching all the projects from the database.

Screenshots of the changes (If any) -
Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

Summary by CodeRabbit

New Features

The index page now highlights featured circuits prominently.

Removed Features

The "Recent Projects" section has been removed from the index page, eliminating project visibility and pagination controls.

These changes may significantly impact how users interact with the CircuitVerse platform, focusing more on featured content while omitting recent project details.

@tachyons Could you please review this PR

We can merge this only after replacement page is ready


[feature testing bug] UI extensions tab available for any user with certain built-in permissions applied to them

Rancher Server Setup
local Rancher HA
non hardened cluster
image v2.7-76c98b4
Describe the bug
any user with the 'built in' permissions of 'configure feature flags' and 'manage settings' has access to the hamburger menu > configuration > extensions page.
To Reproduce

create a user
provide them with 'built in' permissions of 'configure feature flags' and 'manage settings'
navigate to- hamburger menu > configuration > extensions page.

Result
user sees the page, however it displays incorrectly, telling them no operator has been installed even if there is one already installed. also the refresh button is avialable on the page for some reason.
Expected Result
current AC says only admins have access to the UI extensions page, however there were discussions of multiple possible solutions. please reach out @nwmac and confirm expected result before ticket work begins.
Screenshots

Additional context

@PennyScissors , is this a backend or UI bug?

@MKlimuszka I think this is a UI issue.


Error

[12:50:08] [Server thread/CRITICAL]: Could not execute task eggwars\arena\ArenaScheduler: Undefined index: lobbyPlayers
[12:50:08] [Server thread/CRITICAL]: ErrorException: "Undefined index: lobbyPlayers" (EXCEPTION) in "plugins/EggWars/src/eggwars/arena/Arena" at line 187

I working on repair.

fixed

Do you have Discord?

https://discord.gg/Mb5dTX

Expired :/

https://discord.gg/uwBf2jS


OOPs resources

would like to add the top courses for excelling in OOP as there are pretty less resource link in the OOPS section.
link to the article referring for the BEST oops courses click here

I don't think academic courses would be required in this repo


通知提醒框全局配置在生产环境中不生效

Reproduction link
https://stackblitz.com/edit/ng-zorro-antd-start-19p5nt
Steps to reproduce
任何时候必现
What is expected?
在生产环境里，根据自定义配置正确显示通知提醒框。
What is actually happening?
全局配置了通知提醒框在右下角显示，在开发环境正常，但在生产环境依然显示在左上角。

Environment
Info

ng-zorro-antd
8.4.0

Browser
77.0.3865.120

请查看 https://github.com/NG-ZORRO/ng-zorro-antd/issues/4323#issuecomment-544208799

@tazyong Thanks for reporting this. It looks that after compiling our code breaks:
Dev:

Prod:

We will fix this soon.

@tazyong Have you tried other config?


<EMAIL_ADDRESS>hash check failed

Scoop Configuration

//# Your configuration here

Fixed in latest update. Run scoop update and then install the app again


The converter seems to be shut down

I using IConverter successfully in a local environmet but when I do a development in a server-test environment, I am getting this exception "The converter seems to be shut down" when convert a document to PDF. The PDF-file is created in the destination folder but it is empty.
The tracelog which Tomcat shown is:
may 28, 2019 5:10:08 PM org.apache.catalina.core.StandardWrapperValve invoke
SEVERE: Servlet.service() for servlet [es.dxd.aldeasa.PortalWeb.util.DocViewer] in context with path [/PortalWeb] threw exception
com.documents4j.throwables.ConverterAccessException: The converter seems to be shut down
at com.documents4j.util.Reaction$ConverterAccessExceptionBuilder.make(Reaction.java:117)
at com.documents4j.util.Reaction$ExceptionalReaction.apply(Reaction.java:75)
at com.documents4j.conversion.ExternalConverterScriptResult.resolve(ExternalConverterScriptResult.java:70)
at com.documents4j.conversion.ProcessFutureWrapper.evaluateExitValue(ProcessFutureWrapper.java:48)
at com.documents4j.conversion.ProcessFutureWrapper.get(ProcessFutureWrapper.java:36)
at com.documents4j.conversion.ProcessFutureWrapper.get(ProcessFutureWrapper.java:11)
at com.documents4j.job.AbstractFutureWrappingPriorityFuture.run(AbstractFutureWrappingPriorityFuture.java:78)
at java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
at java.lang.Thread.run(Unknown Source)
Could you help me, please??
Thanks a lot in advance.
JC

Hi, that means that the MS Word process seems to be shut down. Maybe it failed to start up. Are you running the server as a service or directly as a user?

just one question, is it necessary that the MS Word is installed in the machine where is being executing the IConverter ?
thanks in advance.

If you are running a conversion server or the lical converter, yes, this is necessary.

Hi, that means that the MS Word process seems to be shut down. Maybe it failed to start up. Are you running the server as a service or directly as a user?

Hi, I have the same error. The MS word is installed in my windows server 2016 ( 64 bit), while it says the converter failed to start. I am using localConverter.
Thanks

Is there an error message? Did you try to run the converter_start.vbs script manually to see if there is an error message popping up?

I have this exception threw while I am trying to convert a word file to pdf.
com.documents4j.throwables.ConverterAccessException: The converter seems to be shut down
at com.documents4j.util.Reaction$ConverterAccessExceptionBuilder.make(Reaction.java:117) ~[documents4j-util-all-1.1.1.jar:?]
at com.documents4j.util.Reaction$ExceptionalReaction.apply(Reaction.java:75) ~[documents4j-util-all-1.1.1.jar:?]
at com.documents4j.conversion.ExternalConverterScriptResult.resolve(ExternalConverterScriptResult.java:70) ~[documents4j-transformer-api-1.1.1.jar:?]
at com.documents4j.conversion.ProcessFutureWrapper.evaluateExitValue(ProcessFutureWrapper.java:48) ~[documents4j-util-transformer-process-1.1.1.jar:?]
at com.documents4j.conversion.ProcessFutureWrapper.get(ProcessFutureWrapper.java:36) ~[documents4j-util-transformer-process-1.1.1.jar:?]
at com.documents4j.conversion.ProcessFutureWrapper.get(ProcessFutureWrapper.java:11) ~[documents4j-util-transformer-process-1.1.1.jar:?]
at com.documents4j.job.AbstractFutureWrappingPriorityFuture.run(AbstractFutureWrappingPriorityFuture.java:78) ~[documents4j-util-conversion-1.1.1.jar:?]
at java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source) ~[?:1.8.0_211]
at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source) ~[?:1.8.0_211]
at java.lang.Thread.run(Unknown Source) [?:1.8.0_211]
I also ran word_convert.vbs through command line and saw the below error.
word_convert.vbs(83, 1) Microsoft VBScript runtime error: Subscript out of range
Thanks a lot for your help.

You need to run converter_start.vbs first. The conversion script takes the input file, the output file and a number (999 for pdf) as arguments.

Can you please give me the link for converter_start.vbs? Is it in the document4j library. I cloud not find it.

The scripts are all in here: https://github.com/documents4j/documents4j/tree/master/documents4j-transformer-msoffice/documents4j-transformer-msoffice-word/src/main/resources

Thanks!
I can see word process is created after I run word_start.vbs. A pdf file can also be generated after I run word_convert.vbs.

The converter only runs these scripts. If it does not work using it, there must be something wrong with another file.

Do you see any reason why I still get the " The converter seems to be shut down" error even when I can successfully run word_start.vbs and word_convert.vbs successfully?

Did you try running the assertion script to see if it is actually responsive?

Hi, i have the same error too. By suggestions i ran the word_convert.vbs  and word_start, and the pdf is generate correctly. My application (java) was working fine, but now the process appear this error.
ps: the word_start.vbs need parameter for run?

Did you try running the assertion script to see if it is actually responsive?

Hi Raphw, eventually I found it is the permission issue that tomcat can not bring up the MS office instance. Granting Tomcat admin permission can resolve it. Thanks for your assistance.

Do you can describe this solution? Pearhaps this its my problem too.

If you are running the converter with a different user (e.g. the service user), it must be allowed to start Word.

My application its running with tomcat as service of windows. Changed my app for running into tomcat and it's working fine.

For running as a service, the webpage describes a few extra steps that are necessary.

I know this is a relatively old thread, so please redirect me to the right location if this is the incorrect location to continue this conversation.
@yasl88 could you clarify what exactly you mean by "Granting Tomcat admin permission can resolve it" ?
I am also trying to use documents4j inside of an application that is run using Tomcat. I see on the website that there are instructions for running documents4j as a service which I have followed. These work great if Tomcat is run as the "Local System" user but for security reasons I want to run Tomcat as a different user that has permissions locked down a bit more. When I use this other user, I am unable to convert documents. I have tried to grant appropriate permissions but seem to be stuck, as I see these logs:
[https-openssl-apr-8447-exec-2] INFO com.documents4j.conversion.msoffice.MicrosoftExcelBridge - From-Microsoft-Excel-Converter was started successfully
[https-openssl-apr-8447-exec-2] INFO com.documents4j.conversion.msoffice.MicrosoftWordBridge - From-Microsoft-Word-Converter was started successfully
[https-openssl-apr-8447-exec-2] INFO com.documents4j.job.LocalConverter - The documents4j local converter has started successfully
Which would indicate to me that WINWORD.exe would have been started, but it does not show up in the task manager (it does as when running as Local System). Then, on actual conversion attempt I see these exceptions:
java.util.concurrent.ExecutionException: Could not complete conversion
at com.documents4j.job.FailedConversionFuture.get(FailedConversionFuture.java:35)
...
Caused by: com.documents4j.throwables.ConverterAccessException: The converter seems to be shut down
at com.documents4j.util.Reaction$ConverterAccessExceptionBuilder.make(Reaction.java:117)
Is what I am trying to do with converting a word doc to pdf through documents4j being hosted in a Tomcat that is run by a user other than Local System even possible? I am a bit lost on what files/folders I would need to grant the user permission to.

Have you checked documents4j.com section about running the server as a service? Additionally, Tomcat needs to be able to write to the file system by the user it is running under.

Hi there @raphw, yes I have followed that section. Following that section makes it work when tomcat is running as "Local System" but it does not work when running as a different user. I would assume that if the user tomcat is running as has a desktop with those same folders it should work?

I assume but never tried it. If you are running a user headless, Word is not working as expected without these folders existing.

I am continuously getting this error when I am deploying my application in a Windows server with Tomcat 9 as a service:-
com.documents4j.throwables.ConverterAccessException: The converter seems to be shut down
It is working fine in my Eclipse IDE. Kindly let me know what is the issue. Also, I have tried "For running as a service, the webpage describes a few extra steps that are necessary." But still it does not work.

There is a section on running this as a service on the webpage. Did you attempt to set everything up that way?

Do you mean the site documents4j.com? Where exactly is that?
On Wed, Jul 12, 2023, 15:36 Rafael Winterhalter @.***>
wrote:

There is a section on running this as a service on the webpage. Did you
attempt to set everything up that way?
—
Reply to this email directly, view it on GitHub
https://github.com/documents4j/documents4j/issues/62#issuecomment-1632221836,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWJCHEUTOAK3ITQAEIS23V3XPZZLHANCNFSM4HQE4T4A
.
You are receiving this because you commented.Message ID:
@.***>

I was trying to deploy with the command java -jar documents4j-server-standalone-shaded.jar http://localhost:9998. I was using the version 1.0.3. A huge amout of errors occured. What is the way forward?

Well now I installed documents4j-server-standalone-shaded.jar and its showing me that the service is up and running. However I am still getting the same error. Kindly help me.


Error trying to download Vanilla UI plus

Yup, known issue. Will be fixed later this weekend.

Fixed with commit 34a0432639860fcc595a6fcbea63a1fed55b3deb.


Read config info for Data Collection Pipeline

Config file (https://github.com/adoptium/aqa-test-tools/pull/480) for Data Collection Pipeline can be loaded in main function in collect_data.
This value can be stored as variable config_info, while two important keys in it are trss_servers and GitHub_credentials for now.
Related Issue: https://github.com/adoptium/aqa-test-tools/issues/412 https://github.com/adoptium/aqa-test-tools/issues/460

Does it make sense to have separated files? One contains sensitive information and one does not.

Currently two info we need in the config file are trss_servers and GitHub_credentials.
For trss_servers, should we take internal trss server link as sensitive info too? Or we store Adoptium trss server in non-sensitive config file, and  internal trss server in sensitive info file? Thanks.

Does it make sense to have separated files? One contains sensitive information and one does not.

Discussed with Lan, we will have two config files:

The existing one  for trss_servers, should add it to gitignore.file;
Create a new one for GitHub credential, and add it to gitignore.file too;
With the new config file, update main function of collect_data to read GitHub credential from it if exists.

FYI @avishreekh


Install instructions outdated for recent versions of Go

I tried installing following the directions in the README but got a warning:
$ go get -u github.com/askmediagroup/dnsbench/cmd/dnsbench
go: go.mod file not found in current directory or any parent directory.
	'go get' is no longer supported outside a module.
	To build and install a command, use 'go install' with a version,
	like 'go install example.com/cmd@latest'
	For more information, see https://golang.org/doc/go-get-install-deprecation
	or run 'go help get' or 'go help install'.

$ go install github.com/askmediagroup/dnsbench/cmd/dnsbench@latest
go: downloading github.com/askmediagroup/dnsbench v0.0.0-20190415183804-140b09708a3a
go: finding module for package github.com/miekg/dns
go: finding module for package github.com/codahale/hdrhistogram
go: finding module for package github.com/spf13/cobra
go: downloading github.com/miekg/dns v1.1.49
go: downloading github.com/spf13/cobra v1.4.0
go: downloading github.com/codahale/hdrhistogram v1.1.2
go: found github.com/codahale/hdrhistogram in github.com/codahale/hdrhistogram v1.1.2
go: found github.com/spf13/cobra in github.com/spf13/cobra v1.4.0
go: found github.com/miekg/dns in github.com/miekg/dns v1.1.49
go: github.com/askmediagroup/dnsbench/cmd/dnsbench imports
	github.com/askmediagroup/dnsbench/pkg/cmd imports
	github.com/codahale/hdrhistogram<EMAIL_ADDRESS>parsing go.mod:
	module declares its path as: github.com/HdrHistogram/hdrhistogram-go
	        but was required as: github.com/codahale/hdrhistogram

$ echo "example.com" | dnsbench run local --concurrency 1 --count 10
zsh: command not found: dnsbench

How can I install it as a command on a modern Go install?

In case anybody else has this problem, I found a fork that works: go install github.com/erikh/dnsbench/cmd/dnsbench@latest

Hello @geerlingguy, I've been on hiatus for the last couple years and wasn't around to see this. The code base was left unfortunately just before the update to go modules support had been merged (#6). That's been completed now and building from source works as expected now. As for the installation instructions #8 is open to fix that in the README.

Fixed with #8

@richardmarshall - Thanks so much!


Modern Browsers block Unsecure Content by default.

Starting sometime in 2020, Chromium based browsers started blocking unsecure content by default. This results in a blank preview pane or a blank page when opening files in Office Web Apps and Office Online Server from a Secure (HTTPS) SharePoint site. The browser does not give any clear indication as to why the page is blank. If you click the information icon in the address bar, it does tell you there is content that is not secure, but it does not tell you or allow you to show the content.
It should be addressed in this article that HTTP for Office Online Server will only function on Chromium Based Browsers for sites that have HTTPS unless you change the browser setting to allow unsecure content for the site.

The documentation is written in such a way to indicate that you should not use HTTP for either SharePoint or OOS due to the passing of OAuth tokens in the clear -- OAuth tokens can be replayed by an attacker to gain the same level of access the end user has.


Sync the TwigView::_getElementFileName method signature with View::_g…

Some recent changes made to a method of View that the plugin overloads and released on 3.3.4 throws a PHP error because methods signatures don't match.
This is only for 3.3.4 and forward.
The changes can be found here

Thanks looks good :+1: , but can you revert the composer.lock changes unless you committed them on purpose?

Nop it was not. I re-pushed only the TwigView file.

Awesome thanks :+1:

You're welcome.

Just released 4.0 and 3.3.1 with this and another fix