App hangs on resize (linux, wayland, ubuntu)

Hi! I just got the editor. Big fan of the original onivim!
I was trying the editor on my linux laptop with SDL_VIDEODRIVER=wayland and the app kept hanging. I did some testing and it looks like the problem might be when the window manager resizes the window.
I managed to get the app running fine by adding for_window [title="Welcome - <user> - Onivim 2"] floating enable to my sway config file. That line tells the wm to allow the window to be a floating window by default. This also avoids a resize of the window on startup. Then, if I try to resize the window, the app just stops responding.
No crash or anything just stops. Logs stop showing, it stops reacting to the "close window" key combination and even when run with --no-fork it will still keep "running" after I CTRL-C it on the terminal (the terminal programs closes, but the window app stays "alive" and frozen).
Edit: I just tried on Ubuntu 20.04 with wayland and I see the same behaviour.
Edit 2: I'm attaching the logfile. As you can see at the end it just stops with a END: Render frame. After that it stops responding and no extra logs are printed
onivim2.2.log
Edit 3: Might be related to https://github.com/onivim/oni2/issues/890 but, in my case, unfocus+refocus cycle does not work.

Hi @Galbar ,

Hi! I just got the editor. Big fan of the original onivim!

Thanks for supporting the project! Sorry you're hitting this on first run.
I wasn't able to reproduce on my Manjaro + weston setup - but I'll try on Sway+Ubuntu to see if I can hit it too.
One other idea - would you mind trying the stable build as well? If that works, it would indicate there is some regression between then and now in Onivim 2.
Based on the log - it may be hanging when it is actually presenting / swapping buffers, which would potentially suggest an OpenGL / GPU issue. Is there any interesting configuration for your GPU? (ie, overclocking)?
Looking through Mesa - I saw this issue that may be related: https://gitlab.freedesktop.org/mesa/mesa/-/issues/3748
And these:

https://gitlab.freedesktop.org/drm/intel/-/issues/562
https://gitlab.freedesktop.org/mesa/mesa/-/issues/2899

The last mesa issue looks interesting, and has some actionable ideas for troubleshooting.
Looking through the syslog for GPU HANG might confirm if it is indeed the same issue (or dmesg for 'preemption')
If it is the case that we're hitting the same thing... there is a workaround posted in one of the threads:

It is a timeout defined by the kernel driver. There is no need to compile any component, you can play with it by changing value in /sys/class/drm/card0/engine/rcs0/preempt_timeout_ms and then running the executable, default value is 640.
(sudo su and then echo "5000" > /sys/class/drm/card0/engine/rcs0/preempt_timeout_ms)

With the following caveat:

It is sort of a double-edged sword. Problem is that a buggy (or evil) application could make your desktop very unresponsive when timeout is big enough.

(found at the end of: https://gitlab.freedesktop.org/mesa/mesa/-/issues/2899)
But we should confirm that this is in fact the same issue before tweaking that value - one way is looking at demsg logs after reproducing the hang - a sample from those threads is something like this:
[  157.384432] i915 0000:00:02.0: Resetting rcs0 for preemption time out
[  157.384451] i915 0000:00:02.0: UE4Editor[2751] context reset due to GPU hang
[  157.407763] i915 0000:00:02.0: GPU HANG: ecode 9:1:859ffffb, in UE4Editor [2751]
[  157.407766] GPU hangs can indicate a bug anywhere in the entire gfx stack, including userspace.
[  157.407767] Please file a _new_ bug report at https://gitlab.freedesktop.org/drm/intel/issues/new.
[  157.407769] Please see https://gitlab.freedesktop.org/drm/intel/-/wikis/How-to-file-i915-bugs for details.
[  157.407770] drm/i915 developers can then reassign to the right component if it's not a kernel issue.
[  157.407771] The GPU crash dump is required to analyze GPU hangs, so please always attach it.
[  157.407773] GPU crash dump saved to /sys/class/drm/card0/error

Hi @bryphe,
I tried the stable build and it works!
I'm running it on a Dell XPS 13 9370. No overclocking or anything. It is normal Ubuntu 20.04 with an extra wm entry for sway.
I could reproduce the issue in normal Ubuntu UI with wayland too.
I could not find any GPU HANG (or any GPU related messages for that matter) by running any of the following while reproducing the issue:

grep "GPU HANG" /var/log/
journalctl -f
dmesg -w

I'm not very well versed in the world of mesa drivers or graphic drivers in general in linux. In case it is of any help, this are all the mesa-related packages I have installed and their versions:
libegl-mesa0/focal-updates,now 20.2.6-0ubuntu<IP_ADDRESS> amd64 [installed,automatic]
libegl1-mesa-dev/focal-updates,now 20.2.6-0ubuntu<IP_ADDRESS> amd64 [installed,automatic]
libgbm1/focal-updates,now 20.2.6-0ubuntu<IP_ADDRESS> amd64 [installed,automatic]
libgl1-mesa-dev/focal-updates,now 20.2.6-0ubuntu<IP_ADDRESS> amd64 [installed,automatic]
libgl1-mesa-dri/focal-updates,now 20.2.6-0ubuntu<IP_ADDRESS> amd64 [installed,automatic]
libglapi-mesa/focal-updates,now 20.2.6-0ubuntu<IP_ADDRESS> amd64 [installed,automatic]
libgles2-mesa-dev/focal-updates,now 20.2.6-0ubuntu<IP_ADDRESS> amd64 [installed,automatic]
libglu1-mesa/focal,now 9.0.1-1build1 amd64 [installed,automatic]
libglu1-mesa-dev/focal,now 9.0.1-1build1 amd64 [installed,automatic]
libglx-mesa0/focal-updates,now 20.2.6-0ubuntu<IP_ADDRESS> amd64 [installed,automatic]
libwxgtk3.0-gtk3-0v5/focal,now 3.0.4+dfsg-15build1 amd64 [installed,automatic]
mesa-utils/focal,now 8.4.0-1build1 amd64 [installed,automatic]
mesa-va-drivers/focal-updates,now 20.2.6-0ubuntu<IP_ADDRESS> amd64 [installed,automatic]
mesa-vdpau-drivers/focal-updates,now 20.2.6-0ubuntu<IP_ADDRESS> amd64 [installed,automatic]
mesa-vulkan-drivers/focal-updates,now 20.2.6-0ubuntu<IP_ADDRESS> amd64 [installed,automatic]
xserver-xorg-video-nouveau/focal,now 1:1.0.16-1 amd64 [installed,automatic]

(I just realized I have nouveau installed, although I have no nvidia card in the laptop :confused: :question: )

Hi @Galbar ,

I tried the stable build and it works!

Thanks for testing this out! That's a helpful data point - based on this, I don't think it's an OpenGL / hardware issue or related to that set of bugs I suspected in Mesa. Must be a regression somewhere in Onivim or Revery.
My suspicion is potentially this PR: https://github.com/revery-ui/revery/pull/1077 , based on the description that it's hanging on resize.
Still working to see if I can reproduce this to track down the regression -

Was able to reproduce on Manjaro + Sway - working on a fix here: https://github.com/revery-ui/revery/pull/1083


Support For Logical Operators

PR Description

Added Support for pointer, print, logical operator and some basic loops . Loops need to be tested with various examples
Related Issue

shader Sample

Checklist

[ ] Have you added the necessary tests?
[ ] Only modified the files mentioned in the related issue(s)?
[ ] Are all tests passing?


OSOE-156: Code styling

OSOE-156

What an adventure!

It was a huge waste of time fighting with the peculiarities of how splatting works.

You might have been able to test that locally - not the docker command, just the argument splatting.
Anyway, you are smarter now! 🎉
It's probably worth sharing this hard-won knowledge in our wiki on some PS specific page.

I did what I can locally, but most of the stuff was related to Docker (under Ubuntu). I'll add this to the wiki.

You don't have to test it with docker tho.

That's what I thought, but hash table-based splatting doesn't work with anything non-PS. Array-based splitting does, but I could get that right in one go, after spending half an hour figuring out why hash table-based splatting won't work the same as with a PS script.

You can see from my screenshot that splatting a hash table prepends the keys with a single dash. So I wouldn't expect it to work with docker that expects longform arguments to have the GNU style double dashes.

What you linked actually results in docker run ---name sql2019 --env 'ACCEPT_EULA=Y SA_PASSWORD=Password1!' --publish '1433:1433' --detach 'mcr.microsoft.com/mssql/server:2019-latest'. Note the triple dash. Like I said, splatting a table prepends one dash to the key. It's a real misstep from Microsoft because if it prepended double dash then it would be more compatible.

It appears that you've given up right before removing the final double dash from your hash table.


Handle multiple RST_STREAM frames

When a client receives two RST_STREAM frames, a NYI exception is raised. This is not ideal, as servers are allowed to send multiple RST_STREAM frames.
From RFC7540§5.4.2:

A RST_STREAM is the last frame that an endpoint can send on a stream. The peer that sends the RST_STREAM frame MUST be prepared to receive any frames that were sent or enqueued for sending by the remote peer. These frames can be ignored, except where they modify connection state (such as the state maintained for header compression Section 4.3 or flow control).
Normally, an endpoint SHOULD NOT send more than one RST_STREAM frame for any stream.  However, an endpoint MAY send additional RST_STREAM frames if it receives frames on a closed stream after more than a round-trip time.  This behavior is permitted to deal with misbehaving implementations.

The first RST_STREAM frame should close the stream. In the closed state, extra RST_STREAM frames should be ignored (unless they arrive after a "significant" amount of time).
From RFC7540§5.1:

Endpoints MUST ignore WINDOW_UPDATE or RST_STREAM frames received in this state, though endpoints MAY choose to treat frames that arrive a significant time after sending END_STREAM as a connection error (Section 5.4.1) of type PROTOCOL_ERROR.

Here is a relevant issue:
https://github.com/dotnet/aspnetcore/issues/32442
I have added a test for this problem, and will add a case to solve it soon.

Thanks for this, I’ll look over it soon. Note that we might need to do the same for the server implementation (up to you whether you add it in this PR or another one)

Looking at the match statements in server_connection.ml's process_rst_stream_frame, I would expect the server to not run into this problem. I'll try to write a test to check.

I have fixed the client and added a server test (that might need improvement), which passes.

I added a new test verifying we also behave correctly for 2 RST_STREAM frames with a No_error payload.

I have fixed the client and added a server test (that might need improvement), which passes.

Looking over the code in server_connection, I feel like what might be happening is that we aren't ever calling the stream-level error handler. That's why your test passes.
A follow-up PR could unify the behavior with the client implementation, if it makes sense.

Sounds good! That would also be useful for solving an issue I'll likely raise tomorrow where it seems that the server continues to send RESET_STREAM and DATA frames after sending a RESET_STREAM frame due to a malformed content-length header.

I added a new test verifying we also behave correctly for 2 RST_STREAM frames with a No_error payload.

Also, this makes me think that the error code might not be relevant, as far as the above parts of the protocol mention. Would it make sense to have the match statement in client_connection look like the following instead:
(match respd.state, error_code with
| Idle, _ -> ...
| Closed _, _ -> ()
| Active _, Error_code.NoError -> ...
| _ -> ...

I applied your suggestion, thanks!


Remove add site/user from Role admin

/admin/accounts/role/<id>/change/ allow to create sites and users which makes no sense there and should be removed (using hardcoded permissions in the admin class).

Will be part of RDMO 2.2.0.


Fixes "Release notes appear on every window"

This issue was once already fixed in https://github.com/microsoft/vscode/issues/77583 with https://github.com/microsoft/vscode/commit/01c2f555539d9eeada4e4ef456f6bfbac11b0f26
Regressed with @bpasero https://github.com/microsoft/vscode/commit/f0b99530c857c74bd0cfba6d0f835afc7a56a40f#diff-10b67ccbd65ac0e04fe13cd734b466f4L128

Don't really know why document.hasFocus() would ever lie, but it appears it can!
Fixes #90632

🥂


Move Linux's POLLRDHUP into linux_like and fix its type.

This was originally posted as #2390, but since it was a breaking change,
that PR instead just added deprecated warnings. There haven't been any
concerns for a while, so this is now a PR to do the change actually, per
the breaking-change-policy.
This fixes two errors in #2247.

It moves the definitions of POLLRDHUP out of linux_like/linux
and into linux_like, so that they're available on Android as well.

It changes the type from c_int to c_short to match the other
POLL* flags.

@bors r+

:pushpin: Commit 1f657fc1bc136a8187fd1986f6508e510d3d4f26 has been approved by JohnTitor

:hourglass: Testing commit 1f657fc1bc136a8187fd1986f6508e510d3d4f26 with merge abbfad26832a90e187c56cbbfd79049938ad960d...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: JohnTitor
Pushing abbfad26832a90e187c56cbbfd79049938ad960d to master...


View articles from a logged in user only

Hello!
I'm using your Mean Stack Relational as a basis for a new Contact Management App.
I did the whole setup and connected everything to a local SQLite database.
All works just fine my only question is how can I get it to work in such a way that a logged in user gets to see only his own articles (contacts in my case)? Right now a logged in user sees all the articles even those from another user. A logged in user is not allowed to delete an article from another user so it should be possible to list only the articles (contacts) of the user's articles or contacts.
Please let me know so I can work on this.
With kind regards,
Alessandro Giacoppo

Hello Alessandro,
It seems like what you are asking for is authorization. I have demonstrated authentication but not authorization which is highly custom on each project. This explains the difference between authorization and authentication
What you will need to do is decide how you want to structure your different authenticated roles. For example do you want a user to see only his own contacts? If you want this then you could add a foreign key to each contact and match it to the user that the contact belongs to and modify the query to perform a join between the contacts and the users when queried.
You may want to authenticate specific groups of users. For example you might want to have several admins that can see all contacts and users that can see only their users. If this is the case then you would need to hook the application up to an Access control list
Since authorization is not the primary focus of this project I will close this issue but I hope it points you in the right direction.


Add tree-building exercise

@ErikSchierboom I'll let you fix the build for this one before I review it

@robkeim Fixed!

Thanks @ErikSchierboom just one small change needed and I'll merge this PR.


[Bug][Numpy] get_constant is not supported in numpy

Description
import mxnet as mx
import numpy as np
from mxnet.gluon import HybridBlock
mx.npx.set_np()

class Foo(HybridBlock):
    def __init__(self, prefix=None, params=None):
        super(Foo, self).__init__(prefix=prefix, params=params)
        self.weight = self.params.get_constant('const', np.ones((10, 10)))

    def hybrid_forward(self, F, x, weight):
        return x, weight.astype(np.float32)

foo = Foo()
foo.hybridize()
foo.initialize()

Error Message
TypeError: Operator `_copyto` registered in backend is known as `_copyto` in Python. This is a legacy operator which can only write to legacy ndarrays, while received an MXNet numpy ndarray. Please call `as_nd_ndarray()` upon the numpy ndarray to convert it to a legacy ndarray, and then feed the converted array to this operator.

Fixed in https://github.com/apache/incubator-mxnet/pull/16637.


GIRF draws

Hi, Junior.
I am working with that model with occasionally binding constraints that I told you about, and I have some doubts about the RISE's irf function. I tried to get the syntax of the dsge/irf function, but it is an "internal function".

Is there a command that returns the confidence interval of the girfs generated by the irf function, or, at least, that returns all the simulations used to calculate the girf?

I believe that the parameter "irf_regime_specific" becomes useless once we set "irf_type" to "girf". Is that right?

What is exactly the difference between the parameters "simul_honor_constraints" and "simul_honor_constraints_through_switch", most specifically in the context of occasionally-binding-constraint models?

The parameter "irf_girf_regime_uncertainty" assures all girf simulations will have the same regime sequence of the first simulation drawn. Is there any way of guaranteeing that in that first simulation the model remains at all periods in a specific regime, say regime 1? By doing that, I believe I would get the girf conditional to the model ex-post never switching, even though agents behaved all the time as it could happen. Does it make sense to you?

Best and thanks for all the help,
Eduardo

I trust by now you have answers to all these questions.


How to check throughput (tokens/s) in LLamaFactory

How to check throughput (tokens/s) in LLamaFactory?
How can I use the command?
Thanks all

put --include_num_input_tokens_seen in your training args

But, When inference with models after merge, quantized 4bit, awq, how to use? Thanks @injet-zhou

put --include_num_input_tokens_seen in your training args

But, When inference with models after merge, quantized 4bit, awq, how to use? Thanks @injet-zhou

put --include_num_input_tokens_seen in your training args

Throughput is currently disabled when using llama-factory for inference; you can just use vLLM instead. @phanxuanphucnd

Throughput

oke. thanks bro

But, When inference with models after merge, quantized 4bit, awq, how to use? Thanks @injet-zhou

oke. thanks bro


Backport "Move CI management from lampepfl/dotty-ci" to LTS

Backports #19697 to the LTS branch.
PR submitted by the release tooling.
[skip ci]

No regressions detected in the community build up to lts-20295.
Reference


Vertical Alignment

The vertical alignment of the main puzzle board with the wing boards is off

Closing this issue as application is being rewritten altogether


How are you handling updates on the store?

Please, how are you replicating the updates on the store to your local pouch? Sorry if the question is off-topic, but I read you created this to work in a project you were considering pouch-redux-middleware.  Since this will solve the pouch->state sync I'm wondering if you came up with something for state->pouch syncing.

You've got a couple options:

Manually put each document that is added to the live query and delete if it is removed. You can specify the _rev field and use force: true in the options to make sure the saved doc keeps the same rev as the original.

Use filtered replication with the new selector option. This will replicate all documents that match the query, but won't remove them when they no longer match. If you want documents removed, the above is probably better.

This is a good idea though, and I'd definitely welcome a pull request to implement selective replication with correct removal of docs that no longer match.

Hey @colinskow, sorry, I don't understand your answer. If something is added to the live query, I understand it's already on Pouch, right?
My main concern here is how to replicate something I do on my store to my local pouch instance. Let's say I add a new item to the store. When should I add it to pouch? Right now I'm adding an object to pouch first and just after receiving that in the live query I'm adding this object to the store. This was working fine, but now I started to think about some business rules and interactions of objects in the state and this is not working anymore.
I'm thinking about adding those objects to a queue on the state and persisting them on pouch by a middleware after the store is updated. But this would probably require me to add some extra state fields to my objects....


Breaking: Microformats.parse(url) explodes

require 'microformats'

doc = Microformats.parse("http://tantek.com")
puts JSON.pretty_generate(doc.to_h)

This worked before. Something changed / broke.
TypeError: no implicit conversion of nil into String

method value=	in parser.rb at line 35
method block in parse	in parser.rb at line 35
method traverse	in node.rb at line 585
method block in traverse	in node.rb at line 584
method block in each	in node_set.rb at line 187
method upto	in node_set.rb at line 186
method each	in node_set.rb at line 186
method traverse	in node.rb at line 584
method block in traverse	in node.rb at line 584
method block in each	in node_set.rb at line 187
method upto	in node_set.rb at line 186
method each	in node_set.rb at line 186
method traverse	in node.rb at line 584
method block in traverse	in node.rb at line 584
method block in each	in node_set.rb at line 187
method upto	in node_set.rb at line 186
method each	in node_set.rb at line 186
method traverse	in node.rb at line 584
method block in traverse	in node.rb at line 584
method block in each	in node_set.rb at line 187
method upto	in node_set.rb at line 186
method each	in node_set.rb at line 186
method traverse	in node.rb at line 584
method block in traverse	in node.rb at line 584
method block in each	in node_set.rb at line 187
method upto	in node_set.rb at line 186
method each	in node_set.rb at line 186
method traverse	in node.rb at line 584
method parse	in parser.rb at line 28
method parse	in microformats.rb at line 19
method <main>	in mf.rb at line 3

This appears to be fixed now, if you are still having trouble please re-open


Demo

Add elastic search plugin
Add printer
Add 'has' on Properties
Add more documentation
Add starting of webserver of flink (this might need change)

Pull Request Test Coverage Report for Build 413

15 of 36 (41.67%) changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-3.7%) to 95.958%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/main/scala/org/codefeedr/pipeline/Pipeline.scala
4
5
80.0%

src/main/scala/org/codefeedr/pipeline/buffer/KafkaBuffer.scala
6
7
85.71%

src/main/scala/org/codefeedr/plugins/Printer.scala
0
1
0.0%

src/main/scala/org/codefeedr/plugins/elasticsearch/ElasticSearchOutputStage.scala
0
18
0.0%

Totals

Change from base Build 406:
-3.7%

Covered Lines:
546

Relevant Lines:
569

💛  - Coveralls


RangeError: Maximum call stack size exceeded - when creating subsets

Hi,
When creating a very large subset I get the following error:
IFCLoader.js?8ae8:478 Uncaught RangeError: Maximum call stack size exceeded
at SubsetCreator.insertIndicesAtGroup (IFCLoader.js?8ae8:478)
This is at this line;
this.tempIndex.splice(position, 0, ...(indicesByGroup.slice(offset, offset + batchSize)));
My initial thought was the variable spread operator is causing the issue - but it looks like some attempted fix has already been in place. My only thought is that the batch size must be reduced further - but really not sure what this is happening.

Solved in 0.0.98. Let us know if this happens again!


Geneva master resize

Which issue this PR addresses:
Net new creation of ARO cluster master resize geneva action
What this PR does / why we need it:
This work will allow us to leverage geneva actions to resize the compute assigned to ARO master nodes.
Test plan for issue:
Functionality can be tested through both direct calls to the RP as well as the pre-approved Geneva environment mock setup
Is there any documentation that needs to be updated for this PR?
Documentation and SOP will be provided once process (and PR) has been solidified.

This is blocked on https://github.com/Azure/ARO-RP/pull/1839

/azp run ci

/azp run e2e


Error initializing module

Hi! I'm receiving some errors when initializing the module using terraform init:
There are some problems with the configuration, described below.

The Terraform configuration must be valid before initialization so that
Terraform can determine which modules and providers need to be installed.

Error: Attribute redefined

  on .terraform/modules/eks_workers/main.tf line 135, in data "aws_ami" "eks_worker":
 135:   most_recent = true

The argument "most_recent" was already set at
.terraform/modules/eks_workers/main.tf:127,3-14. Each argument may be set only
once.

Removing the second most_recent does indeed fix the error.

Please merge it to master @aknysh
Still have this error, on TF 0.12
Olegs-MacBook-Pro:terraform2 mukolaich$ AWS_PROFILE=elite terraform12 init
Initializing modules...
Downloading git::https://github.com/cloudposse/terraform-aws-eks-cluster.git?ref=master for eks_cluster...
- eks_cluster in .terraform/modules/eks_cluster
Downloading git::https://github.com/cloudposse/terraform-terraform-label.git?ref=tags/0.2.1 for eks_cluster.label...
- eks_cluster.label in .terraform/modules/eks_cluster.label
Downloading git::https://github.com/cloudposse/terraform-aws-eks-workers.git?ref=master for eks_workers...
- eks_workers in .terraform/modules/eks_workers
Downloading git::https://github.com/cloudposse/terraform-aws-ec2-autoscale-group.git?ref=tags/0.1.3 for eks_workers.autoscale_group...
- eks_workers.autoscale_group in .terraform/modules/eks_workers.autoscale_group
Downloading git::https://github.com/cloudposse/terraform-terraform-label.git?ref=tags/0.1.6 for eks_workers.autoscale_group.label...
- eks_workers.autoscale_group.label in .terraform/modules/eks_workers.autoscale_group.label
Downloading git::https://github.com/cloudposse/terraform-terraform-label.git?ref=tags/0.2.1 for eks_workers.label...
- eks_workers.label in .terraform/modules/eks_workers.label
There are some problems with the configuration, described below.

The Terraform configuration must be valid before initialization so that
Terraform can determine which modules and providers need to be installed.

Error: Attribute redefined

  on .terraform/modules/eks_workers/main.tf line 135, in data "aws_ami" "eks_worker":
 135:   most_recent = true

The argument "most_recent" was already set at
.terraform/modules/eks_workers/main.tf:127,3-14. Each argument may be set only
once.

addressed in https://github.com/cloudposse/terraform-aws-eks-workers/releases/tag/0.5.1
thanks @fliphess
(the module is not converted to TF 0.12 yet, we are working on it)


Textinput: Simplified the swipe feature logic. Fixed a bug that was preventing to show the select all / paste bubble

On certain devices (this is due to how frequently touches are dispatched), a bug was preventing to show the "selectall/paste bubble" when the field is empty.
This PR tries to simplify the swipe feature logic introduced in #7610, in order to fix the bug that was preventing to open the bubble.

Maintainer merge checklist

[ ] Title is descriptive/clear for inclusion in release notes.
[ ] Applied a Component: xxx label.
[ ] Applied the api-deprecation or api-break label.
[ ] Applied the release-highlight label to be highlighted in release notes.
[ ] Added to the milestone version it was merged into.
[ ] Unittests are included in PR.
[ ] Properly documented, including versionadded, versionchanged as needed.

Hi @misl6!
The simplification of logic sounds good to me! I haven't investigated the origin yet, but I've noticed that some issues have been introduced, shown in the video below:

The scroll action is lost, possibly after the scroll_timeout is exceeded, even if the scroll_distance is reached.
Selection is lost when scrolling text.

https://user-images.githubusercontent.com/73297572/154849399-a7405623-f7aa-49e5-b747-c0c28f6d0c1a.mp4

Hi @misl6!
The simplification of logic sounds good to me! I haven't investigated the origin yet, but I've noticed that some issues have been introduced, shown in the video below:

The scroll action is lost, possibly after the scroll_timeout is exceeded, even if the scroll_distance is reached.
Selection is lost when scrolling text.

ti.mp4

Thank you for testing it!

Selection is lost when scrolling text: Needs to be fixed. I wouldn't have noticed without your help.
The scroll action is lost, possibly after the scroll_timeout is exceeded, even if the scroll_distance is reached.: This follows the same logic of the Scroll behavior. The scroll is considered valid only if the scroll_distance is reached before the scroll timeout.

Sorry, I wasn't clear. Item correction:

The scroll action is lost, possibly after the scroll_timeout is exceeded, even if the scroll_distance is reached before the scroll_timeout

Got it.
I pushed a fix, can you confirm that everything looks fixed on your side?


Multiple formatting and code fixes

@BaileyLawson : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Improve testing of events

What this PR does / why we need it:
Until now we have been using events as a global thing. Meaning if we had to check VMI1-caused event we were checking if any VMI caused the particular event. This is not the best solution as it is error-prone. Events from previous tests can cause a match. At the same time, Kubernetes events contain associations with a particular object.
This PR is making use of the above fact, making it easier to test events in the future. It also removes redundant code and makes 2 Serial tests run in parallel.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:
NONE

/cc @acardace @stu-gott @enp0s3

/retest

/retest

/test pull-kubevirt-e2e-k8s-1.25-sig-performance

/remove-lifecycle stale

/hold cancel

/hold

/unhold
@fossedihelm PTAl if CI is green

/lgtm
I will check the CI, but it should work! :) Thanks


subset Table objects

This implements getindex, view, and Tables.subset, so it is possible to select a subset of rows of a GBIF2.Table and get a Table back.
(I also fixed a little typo in the show method)

Documentation fails on LoadError: ArgumentError: Package LibGit2 does not have LibGit2_jll in its dependencies, which I've never seen before. I think it might have something to do with the change to julia v1.11?

Yeah looks like not our problem. But I thought it was fixed. Maybe we have some old deps?

Oh there is a Manifest.toml in the docs we need to delete

That worked!


adding unkey

Thanks. Do you want to add Unkey to more sections like Tailwind and Next.js?

Hey. Yes, that sounds good.

Should be added to contents too:
https://github.com/elie222/learn-from-open-source#table-of-contents

@elie222 I am not sure about that. I think it will be a better fit to be added to just Projects for now.

Thanks for the PR!


Refacture cli to operation

Purpose
Heavily reduces argument counts in some functions used in lib. Fixes some Clippy warnings.
Code changes
Moved fields of Dnum variant add and modify into on a own struct .

@dwgnr Merge conflicts resolved.


Not support AMD Athlon silver 3050e/3020e

This Soc is used  on the Windows Game handheld Anbernic Win600.
Ryzen controller works with this devices.
I think this may can be supported.

supported here: https://github.com/project-sbc/Power-Control-Panel-v2

Hi yes, I’m working on v2 of the software and it will support any RyzenAdj cpus
Get Outlook for iOShttps://aka.ms/o0ukef

From: plum @.>
Sent: Thursday, July 14, 2022 6:11:59 AM
To: project-sbc/Power-Control-Panel @.>
Cc: Subscribed @.***>
Subject: [project-sbc/Power-Control-Panel] Not support AMD Athlong silver 3050e/3020e (Issue #35)
This Soc is used on the Windows Game handheld Anbernic Win600.
Ryzen controller works with this devices.
I think this may can be supported.
—
Reply to this email directly, view it on GitHubhttps://github.com/project-sbc/Power-Control-Panel/issues/35, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASZ4DJMX5RIBUYWJEKQGHXDVT7RW7ANCNFSM53RWPKAA.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Thanks for your nice working.
I sold out my Win600 and getting steamdeck. So I can't test if it works.
But I think it shoud be work nicely.


Save to playlist, watch later, etc 3 dots link & notifications not working in Firefox

Script version
3.9
Describe the bug
The three dots link for save to watch later, etc no longer works. Same with the notifications icon at the top
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
attached
Desktop (please complete the following information):
Firefox 122.0.1
Windows 11

I'm on Chrome Windows 10 and this happens to me too.

Having the same issue.


Remove fake tabs from install guides

What kind of changes does this PR include?

New or updated content

Description

Supersedes #2326

The “fake” tab pattern in the install guides has been tripping us up a bit. These are two separate pages, so let’s treat them that way properly! This also gives us some leeway to stress preferences a bit and encourage use of the create astro guide which is generally the better DX.

I'm noticing we have no pagination at the bottom of the manual installation page.

Yeah, only pages in the left sidebar know what their before/after pages are, so anything not included there won’t have pagination links.

Yay looks good!
Glad this could be solved nicely


Change ReadMe

[Api Client] Update README.md with usage

@Heleo2705 Please do the needful changes. Thanks in adv


Full webcam images being stored

We should be cropping the webcam images to only the portion visible so as not to store any information the user cannot see.

Moved from LLK/scratchjr-private#183
/cc @markroth8

Look at this for rework


Broadcast stops on network change

Description
It seems like broadcast stops on network change without throwing an error or close callback.
Steps to reproduce
Steps to reproduce the behavior:

Setup

const port = 3333;
const server = dgram.createSocket({
  reusePort: true,
  debug: true,
  type: 'udp4',
});

server.bind(port);

server.setBroadcast(true);

setInterval(() => {
server.send(...) with port 3333 and address <IP_ADDRESS>
}, 5000)

Observe responses in server.on("message") listener
Change wifi network or close wifi to use network data
Go back to the app
No more responses in message listener, "close" and "error" listeners was not called
server.send callback is not called

Relevant information

OS
ios/android

react-native
71

react-native-udp
4.1.7

Hey there, it looks like there has been no activity on this issue recently. Has the issue been fixed, or does it still require the community attention? This issue may be closed if no further activity occurs.

Nothing changed


fix: Fix 401 Unauthorized Error on App Image Click in App Details

Description

This PR addresses the issue where clicking on app images in the "App Details" page results in a 401 Unauthorized error.
Bug Fix: Resolved the 401 Unauthorized error that occurred when users clicked on app images in the "App Details" page.

Why

This issue was causing unauthorized errors, disrupting the user experience. Fixing this will ensure that users can interact with app images without encountering access issues.

Issue
#291

Checklist
Please delete options that are not relevant.

[x] I have followed the contributing guidelines
[ ] I have performed IP checks for added or updated 3rd party libraries
[x] I have created and linked IP issues or requested their creation by a committer
[x] I have performed a self-review of my own code
[x] I have successfully tested my changes locally
[ ] I have added tests that prove my changes work
[x] I have checked that new and existing tests pass locally with my changes
[ ] I have commented my code, particularly in hard-to-understand areas

Hi, @evegufy Can you add an available reviewer to this PR? I have created this PR 2 weeks ago.


Bugfix - Linux parse problem

Fixes #5
If the ip address is not active in the ARP table, there are only 3 columns on the second row. Switch based on the number of columns to select the correct item.

👍


fleetd: use a sane default set of etcd endpoints

fixes #1301

@mischief LGTM - please cut a v0.11.1 release with this after you merge


dsl expression based conditional matcher / filter

Please describe your feature request:
Similar to httpx, DSL based matcher/filter can be applied to the output of the katana.
   -mdc, -match-condition string      match response with dsl expression condition
   -fdc, -filter-condition string      filter response with dsl expression condition

Describe the use case of this feature:
$ katana -u https://tesla.com -mdc 'status_code == 200'

$ katana -u https://tesla.com -mdc 'contains(endpoint, "XXX") && status_code != 403'

$ katana -u https://tesla.com -mdc 'contains(body, "XXX") || contains(headers, "YYYY")'

https://github.com/iamargus95 is working on this


Capitalized extensions are handled incorrectly

Capitalized extensions, as in IMG.JPEG are not detected correctly using this method of generating $LF_ICONS, though these extensions are handled properly by exa. A potential solution would be to enter both capitlized and uncapitalized versions of every extension into $LF_ICONS.

Closed by cafdbe6.


Can we get correct List Id from context or somewhere else when a list is added as a WebPart on a custom page? Is #4238 still an issue in latest SPFx versions?

What type of issue is this?
Question
What SharePoint development model, framework, SDK or API is this about?
💥 SharePoint Framework
Target SharePoint environment
SharePoint Online
What browser(s) / client(s) have you tested

[ ] 💥 Internet Explorer
[X] 💥 Microsoft Edge
[X] 💥 Google Chrome
[ ] 💥 FireFox
[ ] 💥 Safari
[ ] mobile (iOS/iPadOS)
[ ] mobile (Android)
[ ] not applicable
[ ] other (enter in the "Additional environment details" area below)

Additional environment details

SPFx version: 1.15
Node.js version: 16.17.0

Issue description
I'm having trouble getting the correct list id form context in a List View Command Set Extension when the list is added as a WebPart on a custom page. When researching, I came across this issue #4238 in which @andrewconnell says it may not be possible. Is this possible in latest SPFx versions? OR is this still an issue? Are there any work arounds to get the list id?
@andrewconnell Greatly appreciate if you could respond to this.

The issue you referenced is from 2019... that's pretty old. The SPFx API has improved.
To your question, see:
https://www.voitanos.io/blog/sharepoint-framework-v1-14-whats-in-latest-update-of-spfx/#new-api-for-list-view-command-set-extensions---better-insight-into-list-view-events--state

@andrewconnell This helped. Thank you.


Added the CimSession to the schema

When running a Get-DSCConfiguration it crashes because the HashTable returns a value (CimSession) not matching the schema.mof.

This change is

Hi @Minkukel, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

I agree the review comments that this change should not be merged. I suggest that this PR be closed and the resource should be refactored to not use the CimSession object past from Get-TargetResource in the Set-TargetResource function. This is being discussed in issue #53.
@kwirkykat do you agree that we can close this PR?

I'm closing this PR as per previous comment,


use gcdsocket when close the socket ,receive curStateID is 3  and never resume,but the http is normal.

use gcdsocket when close the socket ,receive curStateID is 3  and never resume,but the http is normal.

I guess you use vpn or something on you device. Such agents do not support ICMP which support the ping action. See this: https://github.com/dustturtle/RealReachability/issues/16


change pylon config for log-manager redirection

Remove yarn log-redirection config.
update config for log-manager

Coverage remained the same at 34.391% when pulling 9b10d0181ebd44e9c2f214807b455589f66d0afb on binyli/log-pylon into d434a3c096be72bc49073f176e403d303dd8df9c on master.


Build and validate documentation on build

@jenmalloy @k-wall This PR is based on #1527, so only the last commit requires review for this PR. (The reason is that #1527 has some fixes that this PR would fail if not applied).

@jenmalloy Please have another look


CCD-3710 AppInsights Health Sampling

Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x] No

Closing the PR. Master has the latest changes.


Recognizes scoped package dependencies in mix extension - resolves #1577

Previous regex pattern replaces all characters after '@' symbol.
So package names like @fortawesome/fontawesome would be converted to an empty string.
Resulting in failure to detect the installed dependencies.

Great, thanks!


IPython kernel cpu usage increases with time

From<EMAIL_ADDRESS>on 2013-06-19T13:13:13Z
Spyder Version:  2.2.0
Python Version:  2.7.5
Qt Version:      4.8.4, PyQt4 (API v2) 4.10.1 on Darwin
What steps will reproduce the problem?

Start Spyder (from terminal or standalone application)
Open an IPython console (I have this set to open automatically at startup)
Wait... What is the expected output? What do you see instead? Without running any code whatsoever, I expect the cpu resources to remain the same. Instead, I see ever increasing consumption of cpu resources. Up to 110% (dual core) after 5 days. Although as much as 10% is observed after only one day. This only seems to occur for IPython within Spyder; IPython started from the terminal does not have this issue.

Please provide any additional information below
. Screen shot of Activity monitor is attached.
Using IPython 0.13.2, via MacPorts
Using Spyder 2.2.0, via MacPorts
Using Spyder 2.2.0, standalone application
Preferences: IPython Console: Graphics: Activate Support = True, Backend = Automatic.
Preferences: IPython Console: Startup: Open an IPython console at startup = True
Attachment: Screen Shot 2013-06-17 at 8.25.31 AM.png
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=1450

From<EMAIL_ADDRESS>on 2013-09-18T06:18:55Z
Similar experience here.
i cannot comment on the slow increase in CPU usage, but i experience general high CPU usage for an idling Spyder session on Mac OS 10.8.4 . Roughly every 10 sec, CPU spikes up to 50% on a 2.4 GHz Core 2 Duo iMac.
Also, the UI is generally sluggish with a 2 sec. waiting period for each switch of focus between editor, IPython console or Object Inspector.
this is without anything running, but a few matplotlib windows open in the background.

From ccordoba12 on 2013-06-29T09:29:43Z
Thanks for reporting, I've noticed this behavior as well, it's really annoying.
I'll try to look into it for our next release.
Status: Accepted
Labels: -Priority-Medium Priority-Critical MS-v2.3 Cat-IPythonConsole

From<EMAIL_ADDRESS>on 2013-09-18T06:20:56Z
p.s., Python 2.7.5 64bits, Qt 4.7.4, PySide 1.1.2 on Darwin

From<EMAIL_ADDRESS>on 2013-09-18T06:30:28Z
ok, i have narrowed down the culprit:
none of the slowness occurs when the variable explorer is hidden. unfortunately, disabling the variable auto-update did not help to resolve the sluggishness on focus change.
should i open a new bug?

From ccordoba12 on 2013-09-18T08:38:26Z
Thanks a lot for narrowing down the problem. I was suspecting the cause was something like that. Unfortunately that means the bug is rooted in our core architecture, which will make it much harder to solve.
Does this happen without evaluating a single line of code in the console? Or after you've evaluated expressions with big numpy arrays on it?

From<EMAIL_ADDRESS>on 2013-09-18T11:14:43Z
When I performed the original experiment to file the issue, no code was run after console startups. No scripts were run or data loaded into memory. Just starting the applications, that is all. For the Spyder applications, I have a default window configuration which was used. This configuration does have the Variable Explorer available in one pane, but not visible (see attached pic).
Attachment: Screen Shot 2013-09-18 at 11.04.52 AM.png

From ccordoba12 on 2013-09-19T21:18:05Z
What are your IPython versions, please?

From ccordoba12 on 2013-09-19T21:40:22Z
Thanks! That's what I'm seeing too, i.e. the problem seems to appear only on 1.0+.
Could you checkout 0.13.2 in your repo, install it and tell me if you see the problem there too?

From<EMAIL_ADDRESS>on 2013-09-19T21:32:27Z
Ipython commit bc6128a (i.o.w. current HEAD of master).  I confirmed the issue in Spyder again this morning.

From ccordoba12 on 2013-09-20T11:15:09Z
I think I found the culprit: it's the auto-refresh feature of our Variable Explorer. Please help me to test it by doing this:

Select The Variable Explorer tab.
2, Deactivate auto-refresh by pressing the button I'm showing in the attached image. Then CPU usage should between 0 to 2%.

This solved the issue completely for me.
Labels: -MS-v2.3 MS-v2.2.5
Attachment: autorefresh button.png

From ccordoba12 on 2013-09-21T13:59:52Z
This issue was updated by revision fd58c2493c27 .

When auto_refresh_button was active, our IPython kernels consumed more and
more CPU during time in a dramatic way! After just three hours of being idle,
they were using about 30% of CPU.
This has an important side effect: now we are forced to refresh the Variable
Explorer after each cell execution (but that's probably what most people
intuitively expect).

Status: Started

From ccordoba12 on 2013-09-26T11:25:55Z
Ok, thanks a lot for the confirmation. The revision mentioned in comment #11 not only unchecked but also disabled the auto-refresh button for all our IPython consoles (so that users can't check it even if they want to).
Sorry for taking so much time to fix it, this problem was really serious!
Status: Verified

From<EMAIL_ADDRESS>on 2013-09-23T12:33:28Z
I can also confirm that disabling the auto refresh button in the variable explorer appears to remedy the symptoms. Spyder = 1.7%; iPython kernal = 0.3% for both terminal started and standalone apps after ~2.5 days of idle runtime.
The auto-refresh button seems to be engaged upon startup, regardless of the previous state. I don't use the Variable Explorer, so I will try removing it from my window layout as a workaround for now.


Update 01-installation.md

fix #3728

@popod please sign Google's CLA :)

Also, please fix the commitlint problem :)

No more relevant..


Can't update users email address or password using Thinktecture IdMgr + MembershipReboot

When I edit a user in IdManager I can change their name, and age and other "custom" properties but when I try to change their email address or password my custom Membership Reboot repository update method is called once to save the change and then again to revert the change.
At the moment there is no stack trace other than "external code" around my update method so I'm going to build from source to try and get more info.
There is only 1 HTTP post coming from the IdMgr application so it's not a client side issue. The HTTP call returns 204 and looking through the code this is a success response.

This is the first stack trace...

Gaist.Authentication.dll!Gaist.Authentication.MR.MembershipRebootRepository.Update(Gaist.Authentication.Mongo.Models.CustomUser item) Line 71	C#
[External Code]
IdentityManager.MembershipReboot.dll!IdentityManager.MembershipReboot.MembershipRebootIdentityManagerService<Gaist.Authentication.Mongo.Models.CustomUser,Gaist.Authentication.Mongo.Models.CustomGroup>.SetUserPropertyAsync(string subject, string type, string value) Line 511	C#
IdentityManager.dll!IdentityManager.Api.Models.Controllers.UserController.SetPropertyAsync(string subject, string type) Line 205	C#
[External Code]
IdentityManager.dll!IdentityManager.Configuration.Hosting.KatanaDependencyResolver.SendAsync(System.Net.Http.HttpRequestMessage request, System.Threading.CancellationToken cancellationToken) Line 35	C#
[External Code]
IdentityManager.dll!IdentityManager.Configuration.Hosting.AutofacContainerMiddleware.Invoke(System.Collections.Generic.IDictionary<string,object> env) Line 51	C#
[External Code]
IdentityManager.dll!Owin.IdentityManagerAppBuilderExtensions.UseIdentityManager(Microsoft.Owin.IOwinContext ctx, System.Func<System.Threading.Tasks.Task> next) Line 55	C#
[Resuming Async Method]
[External Code]

As you can see the UserController.SetPropertyAsync controller gets called in both but I'm not sure why.
I'm using IdMgr and IdSvr in the same project, is it possible that there are two containers  and the same controller is registered with both?

Looks like this is an issue with Mongo and how IdMgr is making assumptions about EF behavior. In short, we load the user account, set the property, and save changes in the DB. For certain properties, like username and email, changing those properties are done via APIs on the user account service. Internally those also load the account object, set the property, and save changes in the DB.
So in IdMgr, it's loading one copy of the user account, calling the SetPassword API, which loads a 2nd copy, updates the 2nd copy, then IdMgr is calling save changes on the first loaded copy, which for mongo is reverting your values.
EF when you load an entity multiple times returns the same reference and thus the problem doesn't appear with EF.

So I'm not sure if this is something I should try to address in IdMgr or not -- there are many assumptions up and down this stack about the repository layer returning the same instance on each call to GetByID. If the mongo layer did caching based on ID this I suspect this would be an ok workaround.
Thoughts?

I really think you should consider this but it sounds like it'll be a lot of design changes. I ran into the same problem when using a different AspNet.Identity provider for the user/role stores (because I really didn't want to use EF at the time...I've since reconsidered). A common project shown to people to easily work/mod as backend for AspNet.Identity with MySQL is https://aspnet.codeplex.com/SourceControl/latest#Samples/Identity/AspNet.Identity.MySQL/. Which will totally break in IdMgr for the same reasons you specify. Your implementations of IIdentityManagerService expect the UserManager/RoleManager's (the additional AspNet.Identity layer between you and the actual user/role stores) to always give back the same object reference. This would entirely be dependent on what is backing the UserManager, which could be EF or could be plain old AdoNet. So someone rolling their own simple user/role stores for AspNet.Identity with Ado.Net (or even Mongo) would probably just return a new object (exactly what I did), and won't even know your implementations actually has some other hidden expectations. And this is from the AspNet.Identity side of things, I'm surprise the same problem is apparent on the MembershipReboot side. But the Microsoft implementations of UserManager/RoleManager have no promises that they will return the same reference at all so you are kind of breaking them.
So the problem in my mind extends to any other implementation for any other DB driver that doesn't have very EF'esque features. Which sort of limits IdMgr/IdSrv. If nothing else you should make very clear notes in the wiki/docs/comments the expectations of the repository layer.
I would have reported this a long time ago, but then I considered what would even need to be done to "fix" it and the likelihood of that receiving a positive response seemed low. As it wouldn't be simple (at least I can't think of a simple way to do it), and would only muddle up the very nice design you have between the upper level manager services and repository layer (even though they have a deep dark secret). Solution is basically to just accept EF and use nothing else.

@brockallen I have the same problem here cause I have an Oracle database behind the UserManager.

Are you using IdentityServer.EntityFramework? With your oracle specific EF driver?
You might be running into another large problem in that IdentityServer.EntityFramework is totally broken for any db that isn't MSSQL currently. I've made necessary fix (really just one thing) for MySQL, which might be enough for Oracle as well. But no sure on that front.
https://github.com/Nicholi/IdentityServer3.EntityFramework.MySql

@Nicholi Hi. Actually I'm not using EF... I'm using Dapper instead.

Ahh ok, probably related to what I mentioned in my older post then. I wouldn't expect a "fix" anytime soon (though its more of a large design change), and just go the EF route. But then you'll run into the other issues for anything non-MSSQL.

I'm writing a InMemoryCache as a workaround. I just can't use EF here. :( Btw thanks for the help o/

@Nicholi Apparently after implementing caching the issue is resolved.
https://github.com/saitolabs/AspNet-IdSrv-Identity-Oracle-Dapper/blob/master/src/AspNet.Identity.Oracle/UserStore.cs#L52

Yeah that should do it, as long as the repository layer returns the same reference object. As that was the main issue here.

@brockallen ,
I use aspnet.identity with an odbc connection (don't ask, ...)  and also have the issue when updating email or phone.
Anyway I send a pull request on IdentityManager/IdentityManager.AspNetIdentity which fixes the issue for me.


⚠ Fortnite has degraded performance

In 66463f9, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1524 ms

Resolved: Fortnite performance has improved in 8c511d6.


Scala steward library upgrades combined

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 86.66667% with 4 lines in your changes missing coverage. Please review.

Project coverage is 79.48%. Comparing base (25f07ea) to head (364acae).
Report is 4 commits behind head on master.

Files
Patch %
Lines

...com/ing/baker/runtime/inmemory/InMemoryBaker.scala
50.00%
3 Missing :warning:

...main/scala/com/ing/baker/il/RecipeVisualizer.scala
83.33%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1711      +/-   ##
==========================================
- Coverage   79.69%   79.48%   -0.22%
==========================================
  Files         143      144       +1
  Lines        4842     5021     +179
  Branches      162      663     +501
==========================================
+ Hits         3859     3991     +132
- Misses        983     1030      +47

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


Refine icontool.py

Description

Exactly what it says on the title. Grouped stuff into functions, and organized a lot of stuff to make it easier to read. Also used subcommands.
So from:
python3 icontool.py -s /path/to/icon.svg -c com.app.app/com.app.app.appActivity -n "App Name"

To:
python3 icontool.py add /path/to/icon.svg com.app.app/com.app.app.appActivity "App Name"

I'll work on documentation, and optimizing the sorting/adding to xml code maybe tommorow.
cc @thismoon, since i can't assign you to this

Type of change

:x: Bug fix (non-breaking change which fixes an issue)
:x: Icon addition (non-breaking change that adds/modifies Lawnicons's icons)
✅ General change (non-breaking change that doesn't fit the above categories like copyediting)

Finished the docs and other misc improvements.


juju: 2.9.11 -> 2.9.17

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/juju/juju/releases.
meta.description for juju is: "Open source modelling tool for operating software in the cloud"
meta.homepage for juju is: "https://juju.is"
meta.changelog for juju is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
Warning: no invocation of /nix/store/m6c3jrw6pssh0namds2fb8zbwx40s7yy-juju-2.9.17/bin/juju-metadata had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/m6c3jrw6pssh0namds2fb8zbwx40s7yy-juju-2.9.17/bin/filetoconst had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/m6c3jrw6pssh0namds2fb8zbwx40s7yy-juju-2.9.17/bin/charmhub had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/m6c3jrw6pssh0namds2fb8zbwx40s7yy-juju-2.9.17/bin/actions had a zero exit code or showed the expected version
/nix/store/m6c3jrw6pssh0namds2fb8zbwx40s7yy-juju-2.9.17/bin/juju-list-blobstore passed the binary check.
/nix/store/m6c3jrw6pssh0namds2fb8zbwx40s7yy-juju-2.9.17/bin/juju-inspect passed the binary check.
/nix/store/m6c3jrw6pssh0namds2fb8zbwx40s7yy-juju-2.9.17/bin/leadershipclaimer passed the binary check.
/nix/store/m6c3jrw6pssh0namds2fb8zbwx40s7yy-juju-2.9.17/bin/mgo-run-txn passed the binary check.
/nix/store/m6c3jrw6pssh0namds2fb8zbwx40s7yy-juju-2.9.17/bin/ssh-keycheck passed the binary check.
/nix/store/m6c3jrw6pssh0namds2fb8zbwx40s7yy-juju-2.9.17/bin/containeragent passed the binary check.
/nix/store/m6c3jrw6pssh0namds2fb8zbwx40s7yy-juju-2.9.17/bin/juju-force-upgrade passed the binary check.
Warning: no invocation of /nix/store/m6c3jrw6pssh0namds2fb8zbwx40s7yy-juju-2.9.17/bin/juju-wait-for had a zero exit code or showed the expected version
/nix/store/m6c3jrw6pssh0namds2fb8zbwx40s7yy-juju-2.9.17/bin/juju-bridge passed the binary check.
Warning: no invocation of /nix/store/m6c3jrw6pssh0namds2fb8zbwx40s7yy-juju-2.9.17/bin/jujuc had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/m6c3jrw6pssh0namds2fb8zbwx40s7yy-juju-2.9.17/bin/winuserdata had a zero exit code or showed the expected version
/nix/store/m6c3jrw6pssh0namds2fb8zbwx40s7yy-juju-2.9.17/bin/generate-password passed the binary check.
Warning: no invocation of /nix/store/m6c3jrw6pssh0namds2fb8zbwx40s7yy-juju-2.9.17/bin/schemadocs had a zero exit code or showed the expected version
/nix/store/m6c3jrw6pssh0namds2fb8zbwx40s7yy-juju-2.9.17/bin/juju-blobstore-cleanup passed the binary check.
/nix/store/m6c3jrw6pssh0namds2fb8zbwx40s7yy-juju-2.9.17/bin/jujud passed the binary check.
/nix/store/m6c3jrw6pssh0namds2fb8zbwx40s7yy-juju-2.9.17/bin/schemagen passed the binary check.
Warning: no invocation of /nix/store/m6c3jrw6pssh0namds2fb8zbwx40s7yy-juju-2.9.17/bin/juju had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/m6c3jrw6pssh0namds2fb8zbwx40s7yy-juju-2.9.17/bin/certgen had a zero exit code or showed the expected version
5 of 22 passed binary check by having a zero exit code.
11 of 22 passed binary check by having the new version present in output.
found 2.9.17 with grep in /nix/store/m6c3jrw6pssh0namds2fb8zbwx40s7yy-juju-2.9.17
found 2.9.17 in filename of file in /nix/store/m6c3jrw6pssh0namds2fb8zbwx40s7yy-juju-2.9.17

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

2 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
0 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
0 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
juju

Instructions to test this update (click to expand)

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

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/m6c3jrw6pssh0namds2fb8zbwx40s7yy-juju-2.9.17
ls -la /nix/store/m6c3jrw6pssh0namds2fb8zbwx40s7yy-juju-2.9.17/bin

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

1 package built:

juju

Maintainer pings
cc @citadelcore @kalbasit for testing.

Upstream is ahead


Strange error while running the script

C:\Program Files (x86)\PRTG Network Monitor\custom sensors\EXEXML\Veeam.ps1 :
Cannot convert the "System.Object[]" value of type "System.Object[]" to type
"System.Guid".
At line:1 char:1

&'C:\Program Files (x86)\PRTG Network Monitor\custom
sensors\EXEXML\Veeam.ps1' ' ...

    + CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorExcep
   tion
    + FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorExceptio
   n,Veeam.ps1

C:\Program Files (x86)\PRTG Network Monitor\custom sensors\EXEXML\Veeam.ps1 :
at


python3Packages.tensorflow: 2.4.1 -> 2.4.2

Motivation for this change
Numerous CVEs listed at https://github.com/tensorflow/tensorflow/releases/tag/v2.4.2
(Attempting a rebuild is tomorrow night's task)
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

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

[ ] Fits CONTRIBUTING.md.

gym was broken but fixed in https://github.com/NixOS/nixpkgs/pull/131176

Should I bump tensorflow-bin in this PR too?

Hold on I think I also need to update fetchAttrs...

Oh, I think I see the issue here - fetchAttrs produces a different result on macos and linux. :shrug:

Wasn't expecting that to work, just to remind me to do it manually...


Cow: Blacklist sextoysindubai.com

Cow requests the blacklist of the website sextoysindubai\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

sextoysindubai\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Rejected by Jesse in Charcoal HQ. Reason: 'Not enough TPs for blacklisting yet.'


Access denied when trying to create objects using PowerShell

Hi,
I'm fairly new to this, so pardon the general question.  I've been able to query information on SPO using PnP.  However, whenever I try to create new content (e.g. create site collection, import term group, add site column) I always get error messages.
Connect-SPOnline -url $siteCollectionURL -Credential (Get-Credential -Message "Please provide global admin credentials for $tenantSPRoot" -UserName $globalAdminUser)
New-SPOSite -Url $tenantSPRoot -Title $tenantTitle -Template "STS#0" -Owner $globalAdminUser -StorageQuota $storageQuota
New-SPOSite : No connection available. Use Connect-SPOService before running this CmdLet.
Import-SPOTermGroupFromXml -Path "C:\ts1.xml"
Import-SPOTermGroupFromXml : Access denied. You do not have permission to perform this action or access this resource.
Add-SPOTaxonomyField -DisplayName "Practice" -InternalName "Practice" -TermSetPath "Vorys|Practice" -Group "Vorys" -MultiValue -Required
Add-SPOTaxonomyField : Access denied. You do not have permission to perform this action or access this resource.
I'm running the code using my global tenant admin and am site collection admin on the site collection I'm working in.
Any thoughts?

You are actually mixing two different modules. New-SPOSite is a cmdlet which is part of the Microsoft provided SharePoint Online cmdlets, while Connect-SPOnline is part of the PnP cmdlets.
Notice that New-SPOSite tells you to use Connect-SPOService, not Connect-SPOnline.
The PnP cmdlet equivalent of New-SPOSite is called New-SPOTenantSite and it has almost the same parameters.
With regards to Import-SPOTermGroupFromXml, navigate to the tenant administration site and make sure you're listed in the Managed Metadata Service as administrator.

Hi Erwin,
Thank for pointing out the mix-up on the modules.  As for the Import-SPOTermGroupFromXml, I am a term store admin.  Should my context be to the admin site or the site collection I'm working with?  Export-SPOTermGroup worked fine.

You context should be towards the site collection.

Seems that issue has been resolved. If there's similar questions around the usage model, would suggest to use the PnP Yammer group at http://aka.ms/OfficeDevPnPYammer.


Squarelink working, tests passing

Works fine, it just raises an exception when users hit cancel on the squarelink window.
Unhandled Rejection ([Squarelink Error]): Request rejected by user

@crisgarner Hey, just fixed the issue, let me know if it's good to go!


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-29-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 4591

Questions ?
Please refer to the Backport tool documentation


Interface's default parameters are ignored

Expected
1

Actual
undefined

Steps To Reproduce
Bridge Live
Dot Net Fiddle
public interface IInterface
{
    int function (int v = 1);
}

public class Class : IInterface
{
    public Class ()
    {

    }

    public int function (int v)
    {
        Console.WriteLine(v);
        return v;
    }
}

public class App
{
    [Ready]
    public static void Main()
    {
        IInterface value = new Class();
        value.function();
    }
}

Thanks for the excellent sample. We really appreciate all the simplified samples you are providing. They are very helpful.
We are reviewing this defect to determine what is going wrong. We expect to fix for the 1.14 release.


Can we use ContractCalls enums in bindings in SimulationContract?

I noticed that the bindings have an enum for all the calls (and events) that can be made to the contract. This could be helpful for encoding/decoding. E.g.,
    pub enum LiquidExchangeCalls {
        Admin(AdminCall),
        ArbiterTokenX(ArbiterTokenXCall),
        ArbiterTokenY(ArbiterTokenYCall),
        Price(PriceCall),
        SetPrice(SetPriceCall),
        Swap(SwapCall),
    }

Food for thought.

I think this could be a nice quality of life thing, however not sure what the priority should be.

I iceboxed it.

Closed in favor of middleware


Testnet 9.2

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 16.61721% with 562 lines in your changes missing coverage. Please review.

Project coverage is 35.89%. Comparing base (eab3d59) to head (90e2cb8).

Files with missing lines
Patch %
Lines

src/app.rs
0.33%
299 Missing :warning:

src/app/migrations.rs
0.00%
99 Missing :warning:

src/bitcoin/mod.rs
4.16%
69 Missing :warning:

src/bitcoin/checkpoint.rs
64.59%
57 Missing :warning:

src/bitcoin/header_queue.rs
0.00%
22 Missing :warning:

src/bin/nomic.rs
0.00%
9 Missing :warning:

src/bitcoin/threshold_sig.rs
14.28%
6 Missing :warning:

src/bitcoin/relayer.rs
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #307      +/-   ##
===========================================
- Coverage    37.85%   35.89%   -1.96%
===========================================
  Files           23       23
  Lines         9470     9995     +525
===========================================
+ Hits          3585     3588       +3
- Misses        5885     6407     +522

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


BotAssistExplorer volatility filter

Filter pairs with volatility percentage above configured value out of the new list of pairs. Also allow changing the base (USDT to BUSD for example) if desired.

Thanks! You should have gotten an invite too.


Is Contents property an array of CartItem or a single CartItem

The Contents property is documented as a single CartItem but the description indicates plural cart items.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: fa8201d6-841a-a14d-1d0f-d9b65da7ebf5
Version Independent ID: 43e585f3-9e46-b308-e60c-f4cf529f6060
Content: player_started_purchase - PlayFab
Content Source: playfab-docs/api-references/events/player-started-purchase.md
Product: playfab
GitHub Login: @joannaleecy
Microsoft Alias: joanlee

Thanks for highlighting this @artmasa. I will clarify this with the team and post an update.


Support for app flavors in flutter tooling

Fixes https://github.com/flutter/flutter/issues/11547.
flutter build, flutter run, and flutter drive now understand app "flavors" or "variants". This means that you can build different flavors of your app from the same source code, but with different configuration as defined by platform-specific build settings. Flavors are orthogonal to the existing debug/profile/release build modes:
flutter run --flavor free            # Launch the free app in debug mode
flutter run --flavor paid            # Launch the paid app in debug mode
flutter run --profile --flavor free  # Launch the free app in profile mode
flutter run --profile --flavor paid  # Launch the paid app in profile mode
flutter run --release --flavor free  # Launch the free app in release mode
flutter run --release --flavor paid  # Launch the paid app in release mode

Flavors at the Flutter CLI level map to product flavors in Android projects (with Flutter build modes mapping to Android build types) and to schemes in iOS projects (with Flutter build mode/flavor pairs mapping to Xcode build configurations).
The Android Gradle build tasks corresponding to the above would be
assembleFreeDebug
assemblePaidDebug
assembleFreeProfile
assemblePaidProfile
assembleFreeRelease
assemblePaidRelease

Android makes the flavor available to Java code as BuildConfig.FLAVOR. It can be communicated to the Dart side using platform channels (a convenient plugin for handling this could be built).
For Xcode, one can define build configurations according to a custom naming scheme, e.g.
Debug-Free
Debug-Paid
Release-Free
Release-Paid

---and then map each build configuration to one of two custom schemes named Free and Paid. Each build configuration might include a user-defined Build Setting entry, such as FLAVOR with values free or paid. This value can be exposed to ObjC by adding a Flavor: $(FLAVOR) key/value pair to Info.plist and then accessed using [[NSBundle mainBundle].infoDictionary valueForKey:@"Flavor"]. Flutter tooling does not care about how you set this up, it merely builds the Xcode project specifying the relevant build configuration and scheme.
A sample project using this setup is included for integration testing in dev/integration_tests/flavors. More documentation will be provided in a separate PR.

Test run went fine; this is very nice!

FYI @tvolkert @devoncarew


Preventing spam requests

Spam is very likely to occur very early, and I’d suggest we integrate a form of human validation from the beginning for all anonymous users (so, all users as we are not having logged in users from the beginning). That can be one of:

reCaptcha
hCaptcha (I’m using this in a couple of places as I prefer not to use google)
email or phone ping (get the user to click a verify link sent via SMS or email, depending on the data she has added to the form).

The third option is the more complex, but, it also plays into our value proposition of facilitating communication between the city and citizens, and, when the requester is anonymous, that is the only way we know the details of the requester are valid (so we meet two needs - verify humanness and verify the human is contactable).

(see https://github.com/govflow/govflow/discussions/58 for a focused discussion on email spam submissions)


fix link

fix broken link

@xginn8 guess I should have read the contribution guidelines first :-)
Should be fixed now in any case!

@balena-ci retest

I think the CI is confused by your component: fix link almost-footer. you can move that to the first line or just drop it entirely and i suspect CI will pass.

@xginn8 yep that seemed to sort it. thanks!


fix: Modifies static host to send Content-Type: text/html

Resolves #463
Disclaimer: these are my first lines of Rust ever.  It appears to work.
Thanks for having a straightforward CONTRIBUTING.md file.  I ran ./scripts/e2e.sh successfully.  At first, cargo insta failed, but I figured out what it was and installed it.  A link to cargo-insta might help other first-timers.

Looks pretty well.

A link to cargo-insta might help other first-timers.

An error complaining missing command will guide you to search and find a way to install it, but it should be better to provide a link to help install. lets improve it right now.


Shotgun + Boombox = ?

Well, not sure if it's intended, but it kept constantly boom and moving, so.. Maow (´・ω・｀)

shotgun has glitches with multiple other items in vanilla game


Update lifetime-elision-in-impl.md

An extra lifetime parameter was present; the Rust 2015 version has SomeTrait only taking one lifetime parameter.

Thanks!


Add accessors

[x] Closes #143
[x] Tests added
[x] Passes pre-commit run --all-files
[x] New functions/methods are listed in api.rst
[x] Changes are summarized in docs/source/whats-new.rst

            @register_datatree_accessor("demo")
>           class Foo:
E           Failed: DID NOT WARN. No warnings of type (<class 'Warning'>,) were emitted. The list of emitted warnings is: [].

datatree/tests/test_extensions.py:39: Failed

This error occurs both locally and in CI, which is weird, because I just copied the code across from xarray.core.extensions and xarray.tests.test_extensions.py :confused:

that's not weird, actually: the xarray test checks both Dataset and DataArray accessors at once, and thus it can delete the Dataset accessor and later check that redefining the DataArray accessor does correctly emit a warning (which DataTree can't, of course, because we're testing just a single class).
To fix this test, you can just move the del check below the warning check.

that's not weird, actually

Good point, well made!


Add the ability to create a folder

Make the New Folder button on the /flashcards page functional, by making it actually add a folder. This involves writing the python backend to facilitate this, adding any animations, including loading icons, and making the page work on mobile. You should follow the UI designs.
If you're interested at all, please reply, and I'll send through more detailed information, including the UI designs

I'll begin working on this

Created the API endpoint, but I need to add unit tests and see what happens when you mess up the folder path, like adding "///dsdj//" and things

I broke the unit tests, so they need to be fixed. Also, the pipeline doesn't fail if the unit tests fail - this needs to be resolved

This is now finished


Gh 19434 emoji gen file cli

Summary
Updated emoji_gen file to accept cli param for path to mattermost-server instead of checking for environment var
Ticket Link
Fixes https://github.com/mattermost/mattermost-server/issues/19434
Related Pull Requests
GH-19434 emoji_gen file cli
Release Note
Path for emoji_data file is now set throw cli: npm run make-emojis -- --server-env ../mattermost-server

Hello @scottaudet,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

The file Makefile is in the blocklist for external contributors. Hence, these changes are not tested by the CI pipeline active until the build is re-triggered by a core committer or the PR is merged. Please be careful when reviewing it.
/cc @mattermost/core-security @mattermost/core-build-engineers

The file Makefile is in the blocklist for external contributors. Hence, these changes are not tested by the CI pipeline active until the build is re-triggered by a core committer or the PR is merged. Please be careful when reviewing it.
/cc @mattermost/core-security @mattermost/core-build-engineers

The file Makefile is in the blocklist for external contributors. Hence, these changes are not tested by the CI pipeline active until the build is re-triggered by a core committer or the PR is merged. Please be careful when reviewing it.
/cc @mattermost/core-security @mattermost/core-build-engineers

The file Makefile is in the blocklist for external contributors. Hence, these changes are not tested by the CI pipeline active until the build is re-triggered by a core committer or the PR is merged. Please be careful when reviewing it.
/cc @mattermost/core-security @mattermost/core-build-engineers

The file Makefile is in the blocklist for external contributors. Hence, these changes are not tested by the CI pipeline active until the build is re-triggered by a core committer or the PR is merged. Please be careful when reviewing it.
/cc @mattermost/core-security @mattermost/core-build-engineers

The file Makefile is in the blocklist for external contributors. Hence, these changes are not tested by the CI pipeline active until the build is re-triggered by a core committer or the PR is merged. Please be careful when reviewing it.
/cc @mattermost/core-security @mattermost/core-build-engineers

please re request review once you are ready :)

The file Makefile is in the blocklist for external contributors. Hence, these changes are not tested by the CI pipeline active until the build is re-triggered by a core committer or the PR is merged. Please be careful when reviewing it.
/cc @mattermost/core-security @mattermost/core-build-engineers

All changes have been made - thanks for the feedback @M-ZubairAhmed!

We're moving the contents of https://github.com/mattermost/docs/pull/5400 to the User Forum. Once this PR is merged, please ensure Docs is notified so that we can update that User Forum post with these enhancements (which simplify the user's effort). Thanks!

@cwarnermm yes will definitely inform the Docs once this gets merged. Will also update the google doc with the updated information

Adding security review since it contains changes to Makefile

/update-branch

The file Makefile is in the blocklist for external contributors. Hence, these changes are not tested by the CI pipeline active until the build is re-triggered by a core committer or the PR is merged. Please be careful when reviewing it.
/cc @mattermost/core-security @mattermost/core-build-engineers

/update-branch

The file Makefile is in the blocklist for external contributors. Hence, these changes are not tested by the CI pipeline active until the build is re-triggered by a core committer or the PR is merged. Please be careful when reviewing it.
/cc @mattermost/core-security @mattermost/core-build-engineers

Updating this again to see if that fixes the upload-s3 CI step. I'm not sure why that would be failing at the moment, but it's likely not related to any changes made in this PR.

i am going to try to remove the changes to makefile and see if it ci upload passes.

/update-branch


⚠ Bofrost.fr has degraded performance

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

HTTP code: 200
Response time: 1179 ms

Resolved: Bofrost.fr performance has improved in 61269ca.


Rest API

Hi I am getting this error when trying to use conn.request.
Error: TypeError [ERR_INVALID_ARG_TYPE]: The first argument must be one of type string or Buffer. Received type object
How my code looks like:
    var _request = {
        url: '/services/data/v45.0/actions/custom/flow/Test1',
        method: 'POST',
        body: {"inputs": [{}]},
        headers: {
            "Content-Type": "application/json"
        }
    };
    var test = conn.request(_request, function(err, resp) {
        if (err) {
            console.log(err)
        } else console.log(resp);
    });
}

if you are using request() you will need to serialize the body yourself. You can use requestPost() as an alternative where JSForce will serialize the body for you.
I have submitted a PR for the docs to add instructions and example for working with request().
https://github.com/jsforce/jsforce-website/pull/10
Here is what I have submitted to the docs - hopefully you find is useful:
Request
Make REST api calls to APIs that are not explicitly supported by JSForce.
Setting the URL
The Endpoint URL can be in one of the following formats:

Absolute URL: https://na1.salesforce.com/services/data/v32.0/sobjects/Account/describe.
Relative path from root: /services/data/v32.0/sobjects/Account/describe.
Relative path from version root: /sobjects/Account/describe.

This is only supported if you have explicitly set a default version.

Making Requests
You can use Connection#request() to make api requests.
For GET requests, you can pass in a string URL.
/* @interactive */
conn.request('/services/data/v47.0/ui-api/object-info').then(response => {
  console.log(response);
});

If you prefer to use callbacks instead of promises, pass a callback as the second parameter.
conn.request('/services/data/v47.0/ui-api/object-info', function(err, response) {
  console.log(response);
});

For other HTTP methods, you can pass an object to the request method. Ensure that you serialize the body of the request.
/* @interactive */
// Bulk API 2.0 - Query
const requestBody = {
  operation: 'query',
  query: 'SELECT Id, Name FROM Account LIMIT 1000',
};

conn
  .request({
    method: 'POST',
    url: '/services/data/v47.0/jobs/query',
    body: JSON.stringify(requestBody),
    headers: {
      'content-type': 'application/json',
    },
  })
  .then(response => {
    console.log(response);
  });

Request Helper Methods
In addition to Connection#request(), JSForce provides the following helper methods that can also be used:

Connection#requestGet()
Connection#requestPatch()
Connection#requestPost()
Connection#requestPut()
Connection#requestDelete()

For requestPatch, requestPost and requestPut, these will be serialized automatically and the content-type will be set to application/json.
/* @interactive */
const requestBody = {
  operation: 'query',
  query: 'SELECT Id, Name FROM Account LIMIT 1000',
};

conn.requestPost('/services/data/v47.0/jobs/query', requestBody).then(response => {
  console.log(response);
});

Request HTTP Options
All request methods allow setting HTTP options to be passed to the HTTP request.

Name
Type
Description

responseType
string
overrides the content-type from the response to change how the response is parsed. Valid values are text/xml, application/xml, application/json, text/csv. If you do not want JSForce to auto-parse the response, set this to any other value, e.x. text.

noContentResponse
any
Alternative response when no content returned in response (= HTTP 204)

transport
Transport
Transport for http api - you should not need to set this option.

If you would like to opt-out of parsing, you can set the responseType to text. This is useful if you want the raw response from Salesforce instead of having the results automatically parsed.
// Get raw CSV data instead of allowing JSForce to parse the CSV to JSON
requestGet('/services/data/v47.0/jobs/query/7502J00000LYZC4QAP/results', { responseType: 'text' }).then(response => {
  console.log(response);
});

Hi,
I trying to make a GET call from a node server to 'services/data/v42.0/sobjects/EventLogFile/0AT3h00000PE85uGAD/LogFile' via :
conn.request('/services/data/v42.0/sobjects/EventLogFile/0AT3h00000PE85uGAD/LogFile')
But I get a TypeError saying
TypeError: conn.request is not a function. Any idea what I am doing wrong?

ref: https://jsforce.github.io/document/#request


Consider first-class type (i.e. 'NumberRange' or 'Interval') vs. a 'range' method

In https://github.com/tc39/proposal-slice-notation/issues/19#issuecomment-415995994 I suggested adding something similar to the C# Index and Range types which could also address these use cases with a more flexible API than an overloaded Number.range method.
An example of these APIs can be found here:

Index - https://github.com/esfx/esfx/blob/c50876e542bc73df907b5e3ff4ef95ad656cb3b3/packages/interval/src/index.ts#L23
Interval - https://github.com/esfx/esfx/blob/c50876e542bc73df907b5e3ff4ef95ad656cb3b3/packages/interval/src/index.ts#L107

The advantage of this (or a similar) API is that it provides a convenient place to attach functionality like .includes(value), as well as a place to provide different ways to generate an Interval without relying on possibly confusing overloaded behavior like Number.range(10).

I'm trying to avoid adding a global name on the globalThis but if everyone is okay with that, I'm also okay to change the API design.
On the other hand, the slice notation can behave the following way
// Literal `Range` syntax:
let range = 1:3;
// -> range = Number.range(1, 3);

And I will add @@slice, @@geti on the return value. (Currently, it is immutable so @@seti won't be involved)

I'm opposed to a syntax that would require a new just to create a range; it's visual noise that distracts from what should be a lightweight operation. If the function can be called without a new, then I have no opinion on whether it's an object or just a plain iterator.

There's no reason Interval(1, 3) couldn't be synonymous with new Interval(1, 3). If we had value types, I imagine something like Interval would be a value type (in which case the new would be unnecessary/would result in boxing like new Boolean vs Boolean).

Value type for range? Do you mean the record and tuple or a separate new value type? (I don't think it should be a value type)

If so, I would prefer NumberSequence or NumberStream. So you can make functions like:

NumberSeq.range(from, to, step = 1), NumberStream.range(to);
NumberSeq.count() // output [0, 1, 2, 3, ...]
NumberSeq.iterate(initial, iterateFunc)  // output [initial, iterateFunc(initial), iterateFunc(iterateFunc(initial)), ...]`
NumberSeq.repeat(seed) // output [...seed, ...seed, ...seed, ...], or [seed, seed, seed, ...]
...

But not need to limit to NumberRange.
We can also make the class work for non-numeric types, and rename it as Sequence or Stream. This may be too far from what this spec want to do. But API like this make more sense to me.

I like the idea of being value type. It allow us use === (for example, ^1 === ^1 could return true if Index is value type)

I also like the idea of range.includes(), but it doesn't require range to be value type, so it could be a separate issue. I believe #17 already cover that.

I think we still need separate classes (or value types) for different range (interval), aka. NumberRange, BigIntRange, IndexRange. Note they could be Number.Range, BigInt.Range, Index.Range so we don't need adding too many names to globalThis.

Consider possible range literals, (1n:3n) should be BigInt.Range(1n, 3n), (1:^1) should be Index.Range(Index(1), Index(1, {fromEnd:true}), but what about (1:3), whether it create Number.Range or Index.Range?

Value types are pretty far out. If we had them now, it could make sense. I think this proposal fills an urgent need, and we shouldn't wait on value types for them. The Temporal proposal is making a similar tradeoff.

What would happen on property access with this new built in type?
const r = Range(1, 3)
[1, 2, 3, 4, 5][r]
// [2, 3] or undefined?

(Note your code have ASI problem)
I don't think we should overwrite the behavior of [ ]. (I think this belong to the slice notation proposal)

(I think this belong to the slice notation proposal)

Why?

Maybe things like [r.from:r.to] or there notation. I'm not preferring to overload the [expr]

I'm not preferring to overload the [expr]

I agree.
I'm not sure we'd want different behavior for slice notation depending on where it's used. Not a fan of this:
const r = 1: 3; // creates a Range object
[1, 2, 3, 4, 5][r]; // undefined
[1, 2, 3, 4, 5][1: 3]; // [2, 3]

I'd prefer this:
const f = 1:3; // SyntaxError
const r = Range(1, 3);
[1, 2, 3, 4, 5][r]; // undefined
[1, 2, 4, 5, 5][r.from: r.to]; // [2, 3]
[1, 2, 3, 4, 5][1: 3]; // [2, 3]

late to this issue, but wanted to re-up the notion of a .includes method. I'm opposed to the idea of reusing a range, but I think that's not necessarily a problem; for example, Rust's iterator semantics allow for a .includes method, which simply consumes the iterator. I don't see why we couldn't do that here, as a way to allow this convenience method without cracking open the reusability discussion.

I'm not sure what you mean by ".includes" - you mean a method on the iterator itself?

late to this issue, but wanted to re-up the notion of a .includes method.
curious to hear thoughts on .includes specifically.

What would be the semantics of an includes method? Would it consume the iterator? Might it only compare a searchValue with the range endpoints, or should it take the step value in to account and return true only for a searchValue that would be yielded? Would it, like Array#includes, use SameValueZero equality?
proposal-iterator-helpers has a spec for %Iterator.prototype%.find. I think it would be least surprising if an iterator's includes method — wherever specified — were functionally equivalent to
proto.includes = function (searchValue) {
    return this.find(value => SameValueZero(value, searchValue)) !== undefined;
};

If the semantics are likely to be different to those that might be specified elsewhere, it may be preferable to consider using a different name for the suggested method.

I'm not sure what you mean by ".includes" - you mean a method on the iterator itself?

yep, like Number.range(0, 10).includes(5)

What would be the semantics of an includes method? Would it consume the iterator?

I think yes, it would have to consume the iterator.

Might it only compare a searchValue with the range endpoints, or should it take the step value in to account and return true only for a searchValue that would be yielded?

I could see this going either way. should Number.range(0, 10).includes(5.5) return true? I'm not sure on which side I fall there.

Would it, like Array#includes, use SameValueZero equality?

I think so, yes.

I actually wasn't aware of the iterator helpers proposal - %Iterator.prototype%.some does seem like a reasonable possibility here, in which case there's nothing to be added to this proposal. but would the other semantic - i.e. simply checking that the search value lies within the bounds of the iterator, with no regard for the step size - be useful? if so, perhaps that would be useful here (and perhaps under a different name).

but would the other semantic - i.e. simply checking that the search value lies within the bounds of the iterator, with no regard for the step size - be useful? if so, perhaps that would be useful here (and perhaps under a different name).

I think it would be more useful to instead have something like
Math.inInterval = function (number, inclusive, exclusive) {
    if (inclusive < exclusive) {
        return inclusive <= number && number < exclusive;
    }
    return exclusive < number && number <= inclusive;
};

This, or something similar, could be specified in https://github.com/rwaldron/proposal-math-extensions

This, or something similar, could be specified in https://github.com/rwaldron/proposal-math-extensions

Yes, I agree, but the includes can be also specified in the iterator helper proposal.
An example in another language: https://doc.rust-lang.org/std/iter/trait.Iterator.html#method.find

Consider slice notation seems have trouble to advance, I think we could reconsider the ideas of range[i] or range.item(i) (random access like array, at least python range has that). Note current semantic allow us have O(1) performance and don't rely on iterator. Similarly, includes(x) could also be O(1) and don't rely on iterator.

What would happen on property access with this new built in type?

Sometimes it is very handy to get range of array, but overloading [] is too much, maybe it would be nice for Interval/range to be callable, like:
const rows = [ ['a','b','c','d'], ['d','e','f','g'], ['q','w','e','r']];
(1..2)(rows) == [['d','e','f','g'], ['q','w','e','r']];
rows.map(1..2) == [ ['b','c'], ['e','f'], ['w','e']];

I think a range should be in Number, because we are talking about use a range of numbers, create a new class of Interval could be too much just to a range.

looks like a first-class value is very far from us... closing this for now but still welcome discussions


[iphone] entering invoice amount on receivePage doesn't allow to close keyboard

tested 1.4.0

workaround:

enter amount
scroll page to show entire QR code

keyboard won't disappear, but QR is correct


Wondering whether CogVLM2 supports SFT for multi-image QA in a sample

Feature request  / 功能建议
Hi, CogVLM2 team.
Thank you for your brilliant work and this neat and easy-to-follow codebase.
This morning, I've read through this repo quickly, and I have some related question to ask. Now, my challenge is to simultaneously input 6 images in a single QA turn, and ask some questions which need to retrieve image information from six images (e.g. What are important objects in these six images and tell me why).
To achieve this target, it seems that I only need to prepare the dataset, modify the following code lines (https://github.com/THUDM/CogVLM2/blob/2af46662b2cad1ba0acf743a42e3b61437e2c6df/finetune_demo/peft_lora.py#L71-L93) , feed the images as a python list and  select suitable max_input_tokens. Since currently I don't have enough knowledge about your vision encoders of CogVLM2, so I'm asking the above questions.
Could you please tell me my understanding is right? In addition, if the max_input_tokens is fixed to 8192, we can simultaneously feed 6 images? Thank you again for your great work and look forwards to your reply.
Best regards,
Xuefen
Motivation / 动机
Support multi-image SFT
Your contribution / 您的贡献
None

Any feedbacks plz？

This model can only process one image. If multiple images are used, subsequent incoming images will overwrite the previous images. This is a limitation of the model structure. The max_input_tokens parameter is for the language

As a hack, you can try "merging" several images into 1 image, but you'd probably have to finetune to model a bit.


Refactor keygen FE code

Problem

FE code around generating and setting networking keys is repeated.
(minor UI fix) No apps show up in dock on boot.

Solution

Refactor the identical logic.
Show App Store if no favorite apps.

Testing

[ ] Register a new node:

[x] indirect with KNS
[ ] indirect with ENS
[x] direct KNS
[ ] direct ENS

[x] Reset the password.
[x] Reset the networking info. Likewise... etc.
[ ] Claim an invite (are we still doing these?) and confirm you can start up a node.
[x] Confirm App Store icon shows up on fresh node homepage dock.

Notes
Some paths remain untested; I don't have an ENS name yet, and haven't tried an invite.

#367

np
On Thu, May 30, 2024 at 10:00 PM doria @.***> wrote:

@.**** approved this pull request.
@0x70b1a5 https://github.com/0x70b1a5 can you please resolve the merge
conflict, then this is good to go
—
Reply to this email directly, view it on GitHub
https://github.com/kinode-dao/kinode/pull/368#pullrequestreview-2089758525,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACTWJB3AALDDVMJU2U2YW5LZE7KVNAVCNFSM6AAAAABINTUBIKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDAOBZG42TQNJSGU
.
You are receiving this because you were mentioned.Message ID:
@.***>


Update OffsetPagination.php

In OffsetPagination, no way of setting private member variables $labelPrevious, $labelNext, $labelFirst, $labelLast (Issue #177)

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
✔

Thank you!


Improving overall UI of the whole website!

I reviewed the project and its UI, which I found to be quite impressive and functioning smoothly.
However, I believe there is significant room for improvement in the UI. I propose a gradual upgrade of the entire website, focusing on one component and page at a time.
Here's the proposed timeline for the UI upgrades:

Home
About
Login
Create Post
Blog Post
And subsequently, other pages, one by one...

By following this systematic approach, we can ensure a comprehensive enhancement of the UI throughout the website.

Hi, @sobhanbera you may work on this and please refer to the Contribution guidelines on the readme once.
All the best! Please mention issue no while raising the pr.

Thank you for that! I will definitely consider your suggestions.

I have done all the required changes. The PR could be merged!
PR: #231


[Snyk] Security upgrade python from 3.7-alpine to 3.13.0b1-alpine

This PR was automatically created by Snyk using the credentials of a real user.Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.
Changes included in this PR

examples/docker/Dockerfile

We recommend upgrading to python:3.13.0b1-alpine, as this image has only 0 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.
Some of the most important vulnerabilities in your base image include:

Severity
Priority Score / 1000
Issue
Exploit Maturity

161
Resource Exhaustion SNYK-ALPINE318-EXPAT-6241039
No Known Exploit

161
CVE-2023-5363 SNYK-ALPINE318-OPENSSL-6032386
No Known Exploit

161
CVE-2023-5363 SNYK-ALPINE318-OPENSSL-6032386
No Known Exploit

167
Out-of-bounds Write SNYK-ALPINE318-OPENSSL-6152404
No Known Exploit

167
Out-of-bounds Write SNYK-ALPINE318-OPENSSL-6152404
No Known Exploit

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Resource Exhaustion

Beta version