fix(datagrid): trackBy items by provided trackBy option

When we have two for loops in a single <clr-datagrid> but one of them has a trackBy in simple ngFor and another one is inside *clrDgItems ClrDatagridItemsTrackBy selector only selects the trackBy of the simple ngFor because this is what we told Angular to search for and Angular never went searching for clrDgItemsTrackBy directive which represents the trackBy in clrDgItems.
Signed-off-by: Bogdan Bogdanov<EMAIL_ADDRESS>PR Checklist

[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
[ ] If applicable, have a visual design approval

PR Type

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] clarity.design website / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?
Issue Number: #3936
What is the new behavior?
The correct trackBy function is applied when using *clrDgItems
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

It would be nice to have some tests with the specific reported setup, with two loops with trackby, and make sure there is no interference both ways.

I also think that some tests could be helpful here ..

I'm not sure this won't break the ngFor support for trackby. Please make sure we have a way to test both clrDgItems and ngFor use cases and that we pick up the right trackBy.

Also we should first make this PR against master and then work on backports.

The issue and its example were quite misleading so it took me more time to figure out what's happening.
The ClrDatagridItemsTrackBy actually never goes to clrDgItems to take the provided trackBy. It will get only ngFor and the last declared trackBy will be assigned to Items. it works Many(Direcvites) -> One(grid items) and the last one wins.
Items directive needs to get the trackBy function in order the Selection directive to use it to track/select the correct items.
I think this is the only thing we need to fix here from our side related to trackBy in Datagrid. Anything else that provides trackBy inside the datagrid I think is a sign of poor implementation and needs a redesign.
Another solution that comes into my mind rn will be to remove the ClrDatagridItemsTrackBy directive and provide an @Input in ClrDatagrid component with trackBy function for the items. That way we will get rid of Many -> One relation and let the users use as many ngFor and trackBy as they want in the datagrid.

This looks like a better solution here, but adding @hippee-lee as a reviewer to double check this since he made the change to the iterable proxy setup (granted this was a while ago).
We used to do this, but it was refactored and seems to mean we don't capture trackby properly. I think this fix is good, but wanted to check on side effects. https://github.com/vmware/clarity/blame/f6a4874497655fbf6859f8a1417dc92d1e48f984/src/clr-angular/data/datagrid/datagrid-items.ts#L51

@bbogdanov after some investigation, I don't think this PR is solving #3936.
The trackyBy function on columns is being mistaken for the row items' trackby. So the culprit is in the assignment we make in datagrid-items.tracky.ts where we first instantiate and overload Angular’s default [ngForTrackBy]. That means any trackyBy function whether it’s on row items or some other datagrid component element will be used for row items trackBy. So we have to do that assignment of this._items.trackBy = value only after we make sure that the trackyBy is being used on row items. But that’s really hard… I just don’t see any way to check and make sure ngForTrackBy is being used on rows at this moment.
The workaround to the issue is not to use ngForTrackBy anywhere except on row items AFAIS. Here, our first mistake was to overload a default Angular directive and assumed it would be used only on row items. As Matt suggested above, we could add another @input in clr-dg-row for its trackBy function.

@Shijir Actually this will solve the issue because clrDgItems will apply the trackBy because it is the last instance. Anyway I am not actually trying to solve the entire situation with ngForTrackBy directive, because there is no point in doing that and it is something we would like to avoid supporting. What we will do is to encourage everyone to avoid using trackBy on anything else than clr-dg-row or in clrDgItems because there is no benefit at all to have a trackBy on anything else in the datagrid.
P.S. Matt is saying actually the opposite of what you thought he is saying.

@Shijir From your last comment, I understand that you would like to remove the ClrDatagridItemsTrackBy  directive. I would like to do that too but it is a breaking change. I suggest we at least give the users a major release period of time to change their approaches from using ngFor to clrDgItems and then remove the directive.
About the input on clr-dg-row I think it getting back to the same approach we have with the directive. We will face the same issue. The last clr-dg-row with a trackBy will win. (If I understand you correctly)

Not overloading the directive is the right solution for the long term, but this will require us to discard the use of ngFor, which will be a big breaking change. It will be great solution, too, as using clrDgItems vs ngFor is a common source of confusion. Nevertheless, we have this as tech debt and I don't think it's the right time for this at this point of the product lifecycle. I hope we won't have the same problem in Core.
Also, having trackby in ngFor used for columns is useless - how many items do you have there, why would you want to update them dynamically and preserve state, what state? Hhaving it in action buttons - the same.
In internal ngFors is also a sign of bad architecture and design, though we may see it occasionally. It would be great to be able to avoid it on the Clarity side, but I don't think it's critical and worth the heavy lifting effort at this moment.
In the same time, having clrDgItems silently drop the trackby is a trap that got several devs into and required extensive research and debugging. It's an outstanding problem and its fix is not breaking change.
So, my suggestion is to split the overload scenario from the current case and file new issue for the overload, so we can plan it for some major update. I confirm my approval review for the currently presented commit.

Not overloading the directive is the right solution for the long term, but this will require us to discard the use of ngFor

I think we should try to remove the overloaded instance of ngForTrackBy in one of the next major releases. But I don't quite understand how that will force us to discard the use of ngFor.

Also, having trackby in ngFor used for columns is useless

Yes, I agree! Because of that, I thought, we should just recommend not to use ngForTrackyBy on anywhere except exactly
on rows until we solve the root cause of the issue once and for all.

my suggestion is to split the overload scenario from the current case and file new issue (or discussion) for the overload, so we can plan it for some major update

The dilemma I am facing here is that this PR solves just one symptom of the issue. But at the same time as we haven't addressed the root cause, we will see other reports of similar or related symptoms. Actually, in the comment section of the issue, we've already received them.
I agree that it's probably better to separate the overload issue from the case scenario this PR is solving. So I will just remove my block on this PR. But we also need to file another issue to track for the overloaded ngForTrackBy.

#5044


feat(pie-textarea): DSW-2141 add assistive text and status properties

Describe your changes (can list changeset entries if preferable)
[Added] - assistiveText property, status property with the possible values default, success and error
[Added] - storybook controls for the assistiveText and status properties
Author Checklist (complete before requesting a review)

[x] I have performed a self-review of my code
[x] I have added thorough tests where applicable (unit / component / visual)
[x] I have reviewed the PIE Storybook PR preview
[x] I have reviewed visual test updates properly before approving
[x] If changes will affect consumers of the package, I have created a changeset entry.

Reviewer checklists (complete before approving)
Reviewer 1 - Maryia

[x] I have reviewed the PIE Storybook PR preview
[x] If there are visual test updates, I have reviewed them

Reviewer 2

[ ] I have reviewed the PIE Storybook PR preview
[ ] If there are visual test updates, I have reviewed them

The designs show the textarea with a red border when it is in an error state. Please can you include that as part of this PR, or in a follow-up?

Details

Updated

The designs show the textarea with a red border when it is in an error state. Please can you include that as part of this PR, or in a follow-up?

Updated


Add CanBeBuilt Toolbuilder-test for RunningPollview

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.7%) to 89.756%

Totals

Change from base Build<PHONE_NUMBER>:
1.7%

Covered Lines:
1805

Relevant Lines:
2011

💛  - Coveralls


Fix HTML-encoded characters

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


Tracking (GA) clicks on Slack and Cilium links

Introduce two pages with redirects to Slack and Cilium in order to track the clicks in GA Behavour Flow section. Buttons clicks can also be tracked using GA Events, but in this case Behaviour Flow graph will not be updated:

decided not to implement


UNO R4 Minima: CAN transmit/receive doesn't work

I'm using an UNO R4 Minima together with a NXP TJA1051 CAN high speed transceiver at 500 kBaud.
It's not possible to receive or transmit any CAN frames.
The TJA1051 I've cross-checked in my exiting CAN setup on an ESP32 and there it's properly working (there is also a Raspberry Pi2 with CAN HAT plus USB2CAN-FD connected to my Linux PC on which SavvyCAN and a socketCAN application is running).
By using the Arduino example "CANWrite" incl. changed baud rate from 250 to 500 kBaud, the following error is shown in the terminal:
CAN.write(...) failed with error code -60003

One addition which isn't directly changing the upper issue: If I check the HW used in the CAN tutorial for the Minima, I'm starting to wonder about the used transceiver breakout board. At least the available variants of SN65HVD230 ICs on DigiKey are all only supporting 3.3V and none 5V as it would be required for the Minima. Looks little like the tutorial page is copied over from Portenta C33 which is 3.3V IO level and would work with the SN65HVD230.

Hi @srolf :coffee: :wave:
I've been successfully using this breakout board with the Uno R4 Minima and WiFi. It's PHY driver is a MCP2516.
The error code you see (FSP_ERR_CAN_TRANSMIT_NOT_READY = -60003) indicates that the previous frame has not been sent. This can have a couple of reasons:

No termination on CAN bus.
Wrong pins connected to CAN_TX/RX
Calling write too often, i.e. before the previous frame has been sent.

CAN-Pins on Minima:
#define PIN_CAN0_TX       (4)
#define PIN_CAN0_RX       (5)

I've also just re-checked it w/ my hardware, CAN on Minima works fine :+1:
Another stumbling block could have been to change the bitrate to 500k at CANWrite, but not at CANRead.

Thank you @aentinger for the confirmation, it was motivating me to continue the analysis.
There are now two observation:
The most silly one I did by the PIN assignment: Used 3,4 instead of 4,5... stupid me started counting not with "0"... even if I was looking at it 20 times and also tried to prevent to have interchanged TXD/RXD. Now I can send frames even at 500 kBaud.
But there is a second one: Is the default frame format "extended" instead of "standard" frame format (29 vs 11 bit identifier)? If I use 0x123 as ID in the code, SavvyCAN is showing it as 0x00000123 and marks it as "Extended".
I'm not sure, but by ~/.arduino15/packages/arduino/hardware/renesas_uno/1.0.1/cores/arduino/api/CanMsg.h and
CanMsg(uint32_t const can_id, uint8_t const can_data_len, uint8_t const * can_data_ptr)
it's not looking like to be possible to and a parameter for extended or standard frame format.

The most silly one I did by the PIN assignment: Used 3,4 instead of 4,5... stupid me started counting not with "0"... even if I was looking at it 20 times and also tried to prevent to have interchanged TXD/RXD. Now I can send frames even at 500 kBaud.

Well I'm glad it was nothing more serious :wink: .
Re standard/extended: you're right. Currently only extended IDs are supported. That being said the 32-bit ID variable has room for three additional bits which I would like to model very much like linux/can.h. So if the MSBit is set to 1, you've received/are sending a frame with extended ID (which is currently ignored), if its 0 you've received/are sending a standard ID.
Personally I'd like to add support for standard IDs and you are not the first one to ask. Alas I need internal support in order to start development or an external user provided PR that I can help getting merged. So if you want this feature at the very least create an issue with a feature request or - if you're feeling up to it - get started on a PR that I can help you along :wink: .

That being said the 32-bit ID variable has room for three additional bits which I would like to model very much like linux/can.h.

I see, thank you.
Now I also understood why only EFF and no SFF frames were received: The current driver applies 0x1FFFFFFFU as filter.

Good, I'll be closing this issue then.
Please consider adding a feature request via another issue or start a PR on supporting standard CAN IDs :wink:


Control Multiple Plugs with one node

Hi and first of all:
Great work. Thank you very much.
Is it possible to inject the ip of the plug via payload such that I don't have to create a node for each of my smart plugs?
I tried configuring the node with "{{msg.ip}}" in the IP property but it results in this error:
"There was an Error: Error: Invalid URI "http:///%7B%7Bmsg.ip%7D%7D/config""

@SC-Syndikat no, this isn't expected. You must create a node for each smart plug.


Shell commands after install

Could be great to have the option to specify shell commands to be executed after all other actions, normal shell execution, links and clean.
That way we can execute additional shell commands after we have all the files deployed. For example we could change permissions of some folders not staged on repository.. Compile some package source, and so on.

Dotbot supports this already. You're allowed to use the shell directive more than once.
For example:
- shell:
  - touch ~/.testfile
- link:
    ~/.vimrc: vimrc
- shell:
  - rm ~/.testfile

Thank you @anishathalye! I didn't know this feature :)
Maybe should be good to point this in the Readme


Making sure thumbnail can be set and behaves as expected

Adding a setter to thumbnail to behave as expected.

Tests failed because of: "AssetRequest.cs(43,21): error CS0103: The name 'TryAdd' does not exist in the current context" in 2019 and 2020. Is that something added in a later C# version?

If it is the case we can solve it by checking first if it contains the value, and then setting it instead of using the TryAdd.
Will be a bit more code, but performance and behaviour should be the same.
I'll take a look!


UNSTABLE_EmptyState Integration

Description

Additional context

Issue reference

Changes unknown
when pulling 652e92287576663dd7614c35b027d74977139c79 on integration/unstable-emptystate
into ** on main**.

Changes unknown
when pulling 5c4fbf6e01e5e95e9e9a65e4a9e7880a29671918 on integration/unstable-emptystate
into ** on main**.

Changes unknown
when pulling 5c4fbf6e01e5e95e9e9a65e4a9e7880a29671918 on integration/unstable-emptystate
into ** on main**.

coverage: 78.751% (+0.8%) from 77.957%
when pulling 2b11515bd63114c810a07bef982ff9a7feb1742d on integration/unstable-emptystate
into 5ef605b8f1889fed0631b9af5558968d8bc38f29 on main.

coverage: 75.315% (-2.6%) from 77.957%
when pulling 2b11515bd63114c810a07bef982ff9a7feb1742d on integration/unstable-emptystate
into 5ef605b8f1889fed0631b9af5558968d8bc38f29 on main.


Vehicle Type generator

For a variety of car types.
http://sumo.dlr.de/wiki/Tools/Misc#createVehTypeDistributions.py

Use this to create a random flow of vehicles.

And this random trips generator.


JBDS-4066 DevSuite Installer Account step styles are inconsistent with Target Folder step

Codecov Report

Merging #568 into master will increase coverage by 0.39%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #568      +/-   ##
==========================================
+ Coverage   83.59%   83.99%   +0.39%
==========================================
  Files          25       25
  Lines        2377     2505     +128
  Branches      215      215
==========================================
+ Hits         1987     2104     +117
- Misses        390      401      +11

Impacted Files
Coverage Δ

browser/services/platform.js
87.9% <ø> (-1.58%)
:x:

browser/directives/breadcrumb.js
10% <ø> (-1.12%)
:x:

browser/services/request.js
20% <ø> (-0.84%)
:x:

browser/model/installable-item.js
97.51% <ø> (-0.39%)
:x:

browser/directives/pathValidator.js
82.97% <ø> (-0.36%)
:x:

browser/model/cdk.js
67.77% <ø> (-0.04%)
:x:

browser/pages/start/controller.js
44.79% <ø> (-0.04%)
:x:

browser/model/helpers/hash.js
100% <ø> (ø)
:white_check_mark:

browser/model/helpers/installer.js
100% <ø> (ø)
:white_check_mark:

browser/model/cygwin.js
100% <ø> (ø)
:white_check_mark:

... and 13 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 84d811a...2837070. Read the comment docs.

Could you try to align the entire form so the input fields have a proper length?


Add noncentral t distribution

Could you add noncentral t distribution? The noncentral t distribution is required in statistical power calculation.

I'll take a shot at the CDF, here's my reference.

Computing discrete mixtures of continuous distributions: noncentral chisquare, noncentral t and the distribution of the square of the sample multiple correlation coefficient, D. Benton, K. Krishnamoorthy, Computational Statistics & Data Analysis 43 (2003)
pdf link from second author

The density function could be more difficult, but Wikipedia mentions an evaluation from the CDF.
-- forgot to submit this comment before I worked on it.


Support Jammy stacks

Support Jammy Stacks
See https://github.com/paketo-buildpacks/python/issues/490 for context.

Blocked on #232

Closing in favor of #236


seeing issue in weave ipam after upgrading kernel packages

hi,
we are using weave 2.8.1 .
We upgraded linux packages in our ubuntu vms ("Ubuntu 20.04.6 LTS) from From 5.4.0-187-generic  to linux-image-5.4.0-190-generic and then we see the below error.
any clues on where we can look into to triage and fix the below error ?
INFO: 2024/07/31 12:54:57.657959 [allocator]: Allocate request for bf3f246f18b9b8a415251c0e46cddc24891adae104431fbcd436b8e461e2e7d2 cancelled
WARN: 2024/07/31 12:54:58.437172 Error intercepting response: for "net:default": invalid CIDR address: cancelled
INFO: 2024/07/31 12:55:14.840112 Creating container with WEAVE_CIDR ""
INFO: 2024/07/31 12:55:16.531743 Attaching container 2c75b6758a19433d08ed2b4c04662f78a52b2040ea3ab86e36f3d86d258437ed with WEAVE_CIDR "" to weave network
INFO: 2024/07/31 12:56:15.663149 [allocator]: Allocate request for 2c75b6758a19433d08ed2b4c04662f78a52b2040ea3ab86e36f3d86d258437ed cancelled
WARN: 2024/07/31 12:56:16.492068 Error intercepting response: for "net:default": invalid CIDR address: cancelled
INFO: 2024/07/31 12:56:23.960781 Attaching container 2c75b6758a19433d08ed2b4c04662f78a52b2040ea3ab86e36f3d86d258437ed with WEAVE_CIDR "" to weave network
Could you please share any clues on where to start looking ?

Because of this issue , all weave connections are in "sleeve" mode only and not moving to "fastdp" mode.

After enabling debugs and launching weave in debug I see these messages
INFO: 2024/08/01 05:35:26.962459 [allocator fa:e5:26:56:81:20] No valid persisted data
INFO: 2024/08/01 05:35:27.001192 [allocator fa:e5:26:56:81:20] Initialising via deferred consensus
INFO: 2024/08/01 05:35:27.025688 Listening for DNS queries on <IP_ADDRESS>:53
INFO: 2024/08/01 05:35:27.095244 Sniffing traffic on datapath (via ODP)
INFO: 2024/08/01 05:35:27.096718 ->[<IP_ADDRESS>:6783] attempting connection
INFO: 2024/08/01 05:35:27.099495 ->[<IP_ADDRESS>:6783|26:86:8d:31:03:52(site1-dra-master0)]: connection ready; using protocol version 2
INFO: 2024/08/01 05:35:27.100313 overlay_switch ->[26:86:8d:31:03:52(site1-dra-master0)] using fastdp
INFO: 2024/08/01 05:35:27.100382 ->[<IP_ADDRESS>:6783|26:86:8d:31:03:52(site1-dra-master0)]: connection added (new peer)
INFO: 2024/08/01 05:35:27.100443 Setting up IPsec between fa:e5:26:56:81:20(site1-dra-worker2) and 26:86:8d:31:03:52(site1-dra-master0)
INFO: 2024/08/01 05:35:27.101400 ipsec: InitSALocal: <IP_ADDRESS> -> <IP_ADDRESS> :6784 0x2643b41f
INFO: 2024/08/01 05:35:27.113669 Listening for HTTP control messages on <IP_ADDRESS>:6784
INFO: 2024/08/01 05:35:27.115548 Listening for status+metrics requests on <IP_ADDRESS>:6782
INFO: 2024/08/01 05:35:27.115646 Waiting for Weave API Server...
DEBU: 2024/08/01 05:35:27.116411 [http] GET /status
INFO: 2024/08/01 05:35:27.117770 Finished waiting for Weave API Server
INFO: 2024/08/01 05:35:27.117958 Listening on /run/docker/plugins/weave.sock for global scope
INFO: 2024/08/01 05:35:27.118149 Listening on /run/docker/plugins/weavemesh.sock for local scope
INFO: 2024/08/01 05:35:27.118194 Creating default "weave" network
DEBU: 2024/08/01 05:35:27.119917 fastdp ->[<IP_ADDRESS>:6784|26:86:8d:31:03:52(site1-dra-master0)]: confirmed
INFO: 2024/08/01 05:35:27.121005 [ipam] RequestPool weavelocal <IP_ADDRESS>/12  map[]
DEBU: 2024/08/01 05:35:27.121034 sleeve ->[<IP_ADDRESS>:6783|26:86:8d:31:03:52(site1-dra-master0)]: Confirm
DEBU: 2024/08/01 05:35:27.121075 [ipam] RequestPool result weave-<IP_ADDRESS>/12-<IP_ADDRESS>/12 <IP_ADDRESS>/12 map[com.docker.network.gateway:<IP_ADDRESS>/12]
DEBU: 2024/08/01 05:35:27.122025 [net] CreateNetwork &{NetworkID:9f2064805be996dee030575ef36f308491522361ff9be7d1bc6e5e1e75747718 Options:map[com.docker.network.enable_ipv6:false com.docker.network.generic:map[works.weave.multicast:true]] IPv4Data:[{AddressSpace:weavelocal Pool:<IP_ADDRESS>/12 Gateway:<IP_ADDRESS>/12 AuxAddresses:map[]}] IPv6Data:[]}
INFO: 2024/08/01 05:35:27.122072 [net] CreateNetwork 9f2064805be996dee030575ef36f308491522361ff9be7d1bc6e5e1e75747718
DEBU: 2024/08/01 05:35:27.123209 sleeve ->[<IP_ADDRESS>:6783|26:86:8d:31:03:52(site1-dra-master0)]: confirmed
DEBU: 2024/08/01 05:35:27.123251 sleeve ->[<IP_ADDRESS>:6783|26:86:8d:31:03:52(site1-dra-master0)]: sendHeartbeat
DEBU: 2024/08/01 05:35:27.127112 sleeve ->[<IP_ADDRESS>:6783|26:86:8d:31:03:52(site1-dra-master0)]: handleHeartbeat
DEBU: 2024/08/01 05:35:27.198492 [http] GET /status
DEBU: 2024/08/01 05:35:27.200507 InvalidateRoutes
DEBU: 2024/08/01 05:35:27.201523 [allocator fa:e5:26:56:81:20]: Allocator.OnGossip: 954 bytes
DEBU: 2024/08/01 05:35:27.288139 [ring fa:e5:26:56:81:20]: ReportFree token=<IP_ADDRESS> peer=fa:e5:26:56:81:20 version=30
INFO: 2024/08/01 05:35:27.288898 ->[<IP_ADDRESS>:6783] attempting connection
INFO: 2024/08/01 05:35:27.289098 ->[<IP_ADDRESS>:6783] attempting connection
INFO: 2024/08/01 05:35:27.289194 ->[<IP_ADDRESS>:6783] attempting connection
INFO: 2024/08/01 05:35:27.289343 ->[<IP_ADDRESS>:6783] attempting connection
INFO: 2024/08/01 05:35:27.289505 ->[<IP_ADDRESS>:6783] attempting connection
INFO: 2024/08/01 05:35:27.291024 ->[<IP_ADDRESS>:6783|ee:24:a9:e7:50:0e(site1-dra-control0)]: connection ready; using protocol version 2
INFO: 2024/08/01 05:35:27.291195 overlay_switch ->[ee:24:a9:e7:50:0e(site1-dra-control0)] using fastdp
INFO: 2024/08/01 05:35:27.291341 ->[<IP_ADDRESS>:6783|ee:24:a9:e7:50:0e(site1-dra-control0)]: connection added (new peer)
INFO: 2024/08/01 05:35:27.291400 Setting up IPsec between fa:e5:26:56:81:20(site1-dra-worker2) and ee:24:a9:e7:50:0e(site1-dra-control0)
INFO: 2024/08/01 05:35:27.291574 ipsec: InitSALocal: <IP_ADDRESS> -> <IP_ADDRESS> :6784 0x80fa2dc8
INFO: 2024/08/01 05:35:27.292008 ->[<IP_ADDRESS>:6783|76:a6:72:91:2a:47(site1-dra-control1)]: connection ready; using protocol version 2
INFO: 2024/08/01 05:35:27.292173 overlay_switch ->[76:a6:72:91:2a:47(site1-dra-control1)] using fastdp
INFO: 2024/08/01 05:35:27.292235 ->[<IP_ADDRESS>:6783|76:a6:72:91:2a:47(site1-dra-control1)]: connection added (new peer)
INFO: 2024/08/01 05:35:27.292271 Setting up IPsec between fa:e5:26:56:81:20(site1-dra-worker2) and 76:a6:72:91:2a:47(site1-dra-control1)
INFO: 2024/08/01 05:35:27.294236 ->[<IP_ADDRESS>:6783|aa:e0:01:9b:27:4e(site1-dra-director1)]: connection ready; using protocol version 2
INFO: 2024/08/01 05:35:27.294350 ->[<IP_ADDRESS>:6783|e6:53:1c:64:cf:41(site1-dra-worker1)]: connection ready; using protocol version 2
INFO: 2024/08/01 05:35:27.294464 overlay_switch ->[aa:e0:01:9b:27:4e(site1-dra-director1)] using fastdp
INFO: 2024/08/01 05:35:27.294579 ->[<IP_ADDRESS>:6783|aa:e0:01:9b:27:4e(site1-dra-director1)]: connection added (new peer)
INFO: 2024/08/01 05:35:27.294694 Setting up IPsec between fa:e5:26:56:81:20(site1-dra-worker2) and aa:e0:01:9b:27:4e(site1-dra-director1)
INFO: 2024/08/01 05:35:27.294732 ->[<IP_ADDRESS>:6783|56:61:58:e3:9d:0d(site1-dra-director2)]: connection ready; using protocol version 2
INFO: 2024/08/01 05:35:27.294947 overlay_switch ->[56:61:58:e3:9d:0d(site1-dra-director2)] using fastdp
INFO: 2024/08/01 05:35:27.295030 ->[<IP_ADDRESS>:6783|56:61:58:e3:9d:0d(site1-dra-director2)]: connection added (new peer)
INFO: 2024/08/01 05:35:27.295110 Setting up IPsec between fa:e5:26:56:81:20(site1-dra-worker2) and 56:61:58:e3:9d:0d(site1-dra-director2)
INFO: 2024/08/01 05:35:27.295919 overlay_switch ->[e6:53:1c:64:cf:41(site1-dra-worker1)] using fastdp
INFO: 2024/08/01 05:35:27.295951 ->[<IP_ADDRESS>:6783|e6:53:1c:64:cf:41(site1-dra-worker1)]: connection added (new peer)
INFO: 2024/08/01 05:35:27.296060 Setting up IPsec between fa:e5:26:56:81:20(site1-dra-worker2) and e6:53:1c:64:cf:41(site1-dra-worker1)
DEBU: 2024/08/01 05:35:27.315805 fastdp ->[<IP_ADDRESS>:6784|ee:24:a9:e7:50:0e(site1-dra-control0)]: confirmed
DEBU: 2024/08/01 05:35:27.316100 sleeve ->[<IP_ADDRESS>:6783|ee:24:a9:e7:50:0e(site1-dra-control0)]: Confirm
DEBU: 2024/08/01 05:35:27.316691 [allocator fa:e5:26:56:81:20]: Allocator.OnGossip: 954 bytes
DEBU: 2024/08/01 05:35:27.316742 sleeve ->[<IP_ADDRESS>:6783|ee:24:a9:e7:50:0e(site1-dra-control0)]: confirmed
DEBU: 2024/08/01 05:35:27.316805 sleeve ->[<IP_ADDRESS>:6783|ee:24:a9:e7:50:0e(site1-dra-control0)]: sendHeartbeat
INFO: 2024/08/01 05:35:27.316930 ipsec: InitSALocal: <IP_ADDRESS> -> <IP_ADDRESS> :6784 0xd5bcdb45
DEBU: 2024/08/01 05:35:27.318068 sleeve ->[<IP_ADDRESS>:6783|ee:24:a9:e7:50:0e(site1-dra-control0)]: handleHeartbeat
DEBU: 2024/08/01 05:35:27.332535 fastdp ->[<IP_ADDRESS>:6784|76:a6:72:91:2a:47(site1-dra-control1)]: confirmed
INFO: 2024/08/01 05:35:27.332787 ipsec: InitSALocal: <IP_ADDRESS> -> <IP_ADDRESS> :6784 0x2f5468b7
DEBU: 2024/08/01 05:35:27.333015 sleeve ->[<IP_ADDRESS>:6783|76:a6:72:91:2a:47(site1-dra-control1)]: Confirm
DEBU: 2024/08/01 05:35:27.333265 sleeve ->[<IP_ADDRESS>:6783|76:a6:72:91:2a:47(site1-dra-control1)]: confirmed
DEBU: 2024/08/01 05:35:27.333296 sleeve ->[<IP_ADDRESS>:6783|76:a6:72:91:2a:47(site1-dra-control1)]: sendHeartbeat
DEBU: 2024/08/01 05:35:27.345639 fastdp ->[<IP_ADDRESS>:6784|aa:e0:01:9b:27:4e(site1-dra-director1)]: confirmed
DEBU: 2024/08/01 05:35:27.345774 sleeve ->[<IP_ADDRESS>:6783|aa:e0:01:9b:27:4e(site1-dra-director1)]: Confirm
INFO: 2024/08/01 05:35:27.345892 ipsec: InitSALocal: <IP_ADDRESS> -> <IP_ADDRESS> :6784 0xa504600c
DEBU: 2024/08/01 05:35:27.345987 sleeve ->[<IP_ADDRESS>:6783|aa:e0:01:9b:27:4e(site1-dra-director1)]: confirmed

Any clues / pointers  ?

Sorry, day job is keeping me busy. I'll take a look as soon as i can.

hi can you help us proceed forward in debugging this ?

Okay, I am back. Looking at your second set of logs, it looks like fastdp connections had restarted at some point. Can you look at recent logs, and search specifically for the following messages:
got Heartbeat from peer, marking fastdp forwarder as healthy
or
missed Heartbeat from peer, marking fastdp forwarder as un-healthy
See which one came later, and from what IP. Date/time and IP will be shown earlier in the same log line.
If "marking fastdp forwarder as healthy" came later, you are good. If not, we will have to break and re-establish connections between peers.  But first, tell me what you see in the logs.

Also, please upgrade your weave version! I started maintaining this fork because I had problems with 2.8.1.

Please upgrade, as soon as possible. Meanwhile, the logs seem to suggest that the fallback to sleeve is because of MTU. The default MTU setting for weave net is 1376, and the underlay should allow for 84-87 bytes of overhear. The logs seem to say that the effective MTU is 1414, so lowering weave net's MTU should solve the problem. Try setting the WEAVE_MTU environment variable to 1324, wherever you are launching weave.
While you are at it, set the environment variable CHECKPOINT_DISABLE to 1, also. Later versions do this automatically. This won't affect your problem, but you will stop seeing Error checking version: errors in your logs.

I tested with setting WEAVE_MTU  to 1324 .
export  WEAVE_MTU=1324
weave launch --log-level=debug --trusted-subnets <IP_ADDRESS>/12 -H unix:///var/run/weave/weave.sock -H tcp://<IP_ADDRESS>:12375 --ipalloc-init --password=cisco123  consensus=3 <IP_ADDRESS>
But it didnt help and connections came up in sleeve mode only.
Will continue testing  and update.
DEBU: 2024/08/12 18:02:56.049577 fastdp: broadcast{0 true} {8e:bc:56:4f:93:e9 ff:ff:ff:ff:ff:ff}
DEBU: 2024/08/12 18:02:56.329842 sleeve ->[<IP_ADDRESS>:6783|e6:7f:3a:29:67:f0(dra1-sys04-control-0)]: handleHeartbeat
DEBU: 2024/08/12 18:02:56.361739 fastdp ->[<IP_ADDRESS>:6784|e6:7f:3a:29:67:f0(dra1-sys04-control-0)]: missed Heartbeat from peer, marking fastdp forwarder as un-healthy
DEBU: 2024/08/12 18:02:56.361901 overlay_switch ->[e6:7f:3a:29:67:f0(dra1-sys04-control-0)] Removing fastdp from the list of forwarders
INFO: 2024/08/12 18:02:56.362032 overlay_switch ->[e6:7f:3a:29:67:f0(dra1-sys04-control-0)] using sleeve
DEBU: 2024/08/12 18:02:56.452147 sleeve ->[<IP_ADDRESS>:6783|e6:7f:3a:29:67:f0(dra1-sys04-control-0)]: sendHeartbeat
DEBU: 2024/08/12 18:02:56.487989 fastdp ->[<IP_ADDRESS>:6784|e6:7f:3a:29:67:f0(dra1-sys04-control-0)]: sending Heartbeat to peer

tested it with changing MTU , but didnt help .
but when we upgraded to the next  version we didnt see the connections moving to sleeve .

Thank you. Closing this issue now.


Swift 6

Going to wait to figure out the CI issues.. Seems like Linux doesn't want to install Swift 6.0 and Windows is having some MainActor errors that work fine on macOS.


Add docstrings to every file

A pleasure to review. Everything looks great, and you know your way around using git. ⭐


Input fields with measurement unit on Web Components

Originally submitted by Sami Raboun https://buildings-dev.mepdevelopment.com/rules-app/modus-ux-review

Not taking up until the https://github.com/trimble-oss/modus/issues/525 is worked on


GPU memory rises continuously

When I finetune the ALBEF on VQA task, I find that the GPU memory rises continuously.
It ends up with 1.5 times as much GPU memory as it started with.
Is this situation normal? How could I solve this problem.....THANKS

Can I know who much GPU memory it occupies? A memory rise at the beginning of training should be normal.


Create the basic infrastructure for the Rust wrapper for Namada SDK

Subtask of https://github.com/anoma/namada-interface/issues/177
In the current state Namada SDK exposes a Rust interface. There is a need for a layer of Rust with https://rustwasm.github.io/wasm-bindgen/ macros for creating the TypeScript glue code that is being used by the web client.
This https://github.com/anoma/using-namada-sdk-poc should pretty much implement this task but is lacking the actual types from Namada SDK.
Summary of what this should do:

Use types from Namada SDK
Defined something like NamadaWebClient and perform calls from Namada SDK passing that NamadaWebClient in.
Exposing all the calls from Namada SDK as closely as feasible and decorating those types/methods with wasm-bindgen to get the glue code created

Outcomes:
"Namada SDK - Web", something that is as close to Namada SDK but contains TypeScript types and works in browsers.

Closing as done


Allow to enable frame copy on a video track

Description
This PR aims to give access to copy frame feature of Broadcaster in high level functions such as track.NewEncodedReader().
Today, track.New*Reader() functions forces to disable frames copy from Broadcaster. This is not convenient when you want to use high level API with multiple readers, leading to video artefacts on some codec/frame format combination.
This API is not very elegant but doesn't introduce any complex or breaking changes.

Will this PR be merged ? If not what is holding it from being merged ?


[Backport] add synchronization to SupervisorManager

Backport of #3077 to 0.9.1.

oops wrong base


resnet 101 backbone

Thank you for your repo. How can I train on VG with a resnet 101 backbone like in bottom-up-attention?

Just change 50 to 101 in this config and run it. https://github.com/facebookresearch/grid-feats-vqa/blob/master/configs/R-50-updn.yaml


Improve Opsgenie integration

The current Opsgenie service is limited to sending message, and description.  It would be great to be able to send additional optional fields especially priority, and tags, as well as dynamically setting the responders at the point the alert is generated.
I would be willing to contribute a PR for this functionality if there is interest in it.

We need this 🥺


User Manifest Dashboard

🚀 Feature Request
Add a new page that shows the manifests (focusing on the relevant and recent ones) for all sites that a user has access to on one page.
There should be a new link in the left layout nav titled "Manifests" with an icon to the left.

I'm leaving this vague for now, anyone looking at contributing for this PR just ping me here and I'll describe further. The scope of this is larger than just a new react component.

closed with #340


Cyrillic fonts is not supported

Hi, how can i convert RTF to HTML with cyrillic fonts?

Have you tried? Cyrillic text are supported by the lib. Any example?


V2.6.4 zip4j in Android 11 os and Linux Kernel 5.4  Have permission problem

V2.6.4 zip4j in Android 11 os and Linux Kernel 5.4 , file cannot be read or written after decompression
Comment code FileUtil. applyPosixFileAttributes file permission is ok

Hope the zip4j author can solve this problem

zipxx folder is Comment code FileUtil. applyPosixFileAttributes
zip folder is v2.6.4 zip4j source code

zipi4j restores the permissions that are defined in the zip headers, and those permissions in file headers are taken from the file permissions when a zip file is created. If you want some other permissions than what zip4j restores you have to manually set them to the values you need.

Closing this issue as there is no further feedback.


Up to Java 17

Pull Request
Description

due to incompatible dependencies, up the java version to 17

since we're releasing breaking version 2.0.0 this makes sense to do now too
Closes

closes #141

Thanks for the update and for the release!


Some questions about SMIRK

Hello George!
     Thank you for your work and open-source contributions; it's indeed very enlightening.  I have a few simple questions about your work:
      1) Besides the common encoding of 3DMM params such as shape, expression, global pose+jaw pose, and caw, an additional 'eyelip' parameter is used in smirk. I'm not sure if this corresponds to the eye_pose parameter in the generic FLAME model. Has the FLAME model used in your open-source code been modified as shown in Figures 1 and 2, or does this parameter serve an additional purpose? Can it be used for decoding with the generic FLAME model?
      2) For the same input image (as shown in Figure 3), SMIRK's reconstruction shows a better eye closure effect compared to DECA and Emoca, as shown in Figures 4 and 5. Since SMIRK does not provide code for generating meshes, I used the decoded FLAME vertices from SMIRK combined with the generic FLAME faces to generate an OBJ file. Although SMIRK's reconstruction is closer to the input, a noticeable issue is the overlapping of eyelids and eyeballs, which is not an isolated case (as seen in the zoom in of image). What could be causing this problem, and is there a solution?

fig1: the 3DMM params in smirk

fig2: the 3DMM params in deca and emoca

fig3: the input image

fig4: the smirk result

fig5: the emoca result

Hey @GrizzCMX, inline answers follow :)

the eyelid parameters independently control the left and right eyelid closure. We got these from the metrical tracker of MICA: https://github.com/Zielon/metrical-tracker/tree/master/flame/blendshapes. Inside FLAME.py you can see how these extra two blendshapes are used in the model. These are essentially extra expression blendshapes because I think original space could not model independent eyelid closure.

This seems indeed like an erroneous reconstruction - the process you mentioned for getting the .obj file seems right. This may be attributed to the fact that we have noticed that the eyelid_parameters do have some weird interplay with the other parameters. We are going to look into it and try to reproduce it. In the meantime maybe you can try some very practical stuff such as maybe reducing just a bit the Z coordinate of the eyeball and let us know how it goes.

Btw, our FLAME.py is very similar and practically the same with the FLAME.py from DECA and EMOCA (just renaming and splitting of some variables) with the addition of the eyelids taken from the metrical tracker above.

Thank you for your response. I also think this issue might be caused by the 'eyelip' parameter not being completely disentangled from other 3DMM parameters. While manually adjusting the position of the eyeballs might improve this phenomenon. However, I have conducted experiments on extensive datasets (such as the MEAD dataset) and found that this issue is not incidental, so manual adjustments are quite limited. I really wish your team consider researching improvements to make this work more applicable to a broader range of downstream tasks (Because its effect is indeed impressive!). Thank you again for your work and open-source contributions.

How do you manually reposition the the eyes?


Uninstall_KOBman

Getting error like below while installing & uninstalling KOBman

Uninstall command is been working now, Please find below


allowMissingProperty for old models that weren't setup with versionin…

Not sure if you have a cleaner way of doing this, but this seems to work ok.  Basically we need a way to support non-existent modelVersion data for deserialization because we often are dealing with models who were not setup with versioning from day 1 which essentially means we can treat them like version 0 in most cases.  This gives us that flexibility so we can deal with modelVersion being null in the converters.

After thinking about this more, would just having a defaultDeserializeToVersion property work?  In the case that the property is missing, it could default to convert to a particular version?

Ya. I like that actually.

I went ahead and made your suggested changed here: https://github.com/jonpeterson/jackson-module-model-versioning/pull/4


Merge pull request #1 from phildawes/master

update

It looks like something went wront here.

Nice Pull Request


Make outline work

The hierachy in the outline in Eclipse LSP4E is based on nested ranges.
Since in yang.lsp we return the significiant range (the range of the identifier), they are not nested.
We should check if reveal within monaco (Theia & VsCode) works well with full ranges, too.
See also https://bugs.eclipse.org/bugs/show_bug.cgi?id=520770

I've tried with VSCode:
I had to install the (Code Outline)[https://marketplace.visualstudio.com/items?itemName=patrys.vscode-code-outline] extension to see something a code outline.
Extending the regions works in general, but of course the entire region of the element is selected when the node is clicked. Given that the children are also sorted in alphabetic order, the selection feels pretty quirky when we step through the children. Worst of all, the caret is placed at the end of the region. The code lens interrupts the selection visually in an ugly way.
Most of these problems seem to be caused by the Code Outline extension. I have installed TypeScript Hero and that one behaves in the same bad way: Full region select, caret placed at the end of the region. Pretty unusable. But it TS Hero contributes its own additional code outline that does not select, places the cursor at the beginning at the region and does nothing when selecting child nodes (in fact it shows less children than the other outline, e.g. nothing within functions).
Given the other outline issues, I am inclined to implement our own outline.

We should go for the nested ranges. Pushed a fix to master. Maybe we should change that in xtext-ide?


fix: server.watch.ignored option not work

Description

Additional context

Related Issues:close #12607
What is the purpose of this pull request?

[x] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[ ] Read the Contributing Guidelines.
[ ] Read the Pull Request Guidelines and follow the PR Title Convention.
[ ] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[ ] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

I don't think this is the right fix as sun0day pointed out.


Auth without Azure AD app

Is this possible? Docs says 'either or' but in my case I only want to use my creds without creating special Azure AD App. I tried and it errors about app permissions and wants app.

Yes it works with just username and password without registering an app in Azure AD. Did you go here and authorize the app?

I see, thanks


[A2-771] AuthorizedProjects handles deny

:nut_and_bolt: Description
This PR refines the rego behind ProjectsAuthorized to ensure a deny overrides an allow on the same project, even with statements containing All-Projects
:+1: Definition of Done

[x] when request includes projects and statement denies all projects, ProjectsAuthorized returns no projects
[x] when request includes no projects and statement denies all projects, ProjectsAuthorized returns no projects
[x] when request includes no projects and one statement allows all projects and another statement denies one project, authz-service returns a list of all projects minus the denied projects

:athletic_shoe: Demo Script / Repro Steps

rebuild services  and make some projects, teams, and roles

rebuild components/authz-service

export TOK=`chef-automate iam token create ADMIN --admin`
curl -kH "api-token: $TOK" -d '{"name":"project 1", "id": "p1"}' https://a2-dev.test/apis/iam/v2beta/projects | jq
curl -kH "api-token: $TOK" -d '{"name":"project 2", "id": "p2"}' https://a2-dev.test/apis/iam/v2beta/projects | jq

curl -kH "api-token: $TOK" -d '{"name":"role 1", "id": "r1", "actions": ["do:the:thing"], "projects": ["p1"]}' https://a2-dev.test/apis/iam/v2beta/roles | jq
curl -kH "api-token: $TOK" -d '{"name":"role 2", "id": "r2", "actions": ["do:the:thing"], "projects": ["p2"]}' https://a2-dev.test/apis/iam/v2beta/roles | jq

curl -kH "api-token: $TOK" -d '{"name":"team 2", "id": "t2", "projects": ["p2"]}' https://a2-dev.test/apis/iam/v2beta/teams | jq

upgrade to 2.1 and make a test token

chef-automate iam upgrade-to-v2 --beta2.1
export TEST=`chef-automate iam token create test`

make a policy for that token

# save as policy.json
{
  "name": "testing deny",
  "id": "test",
  "members": [ "token:test" ],
  "statements": [
    {
      "effect": "DENY",
      "actions": [ "iam:teams:list" ],
      "projects": [ "*" ]
    },
   {
      "effect": "ALLOW",
      "actions": [ "iam:teams:list" ],
      "projects": [ "p2" ]
    },
  {
      "effect": "ALLOW",
      "actions": [ "iam:roles:list" ],
      "projects": [ "*" ]
    },
   {
      "effect": "DENY",
      "actions": [ "iam:roles:list" ],
      "projects": [ "p2" ]
    }
 ]
}

$ curl -kH "api-token: $TOK" -d @policy.json https://a2-dev.test/apis/iam/v2beta/policies | jq

run test cases

a. When a user wants a list of teams belonging to p2, but a statement denies listing teams for all projects, ProjectsAuthorized returns no projects.
$ curl -kH "api-token: $TEST" -H "projects: p2" https://a2-dev.test/apis/iam/v2beta/teams | jq
{
  "error": "unauthorized: subjects [\"token:test\"] cannot perform action \"iam:teams:list\" on resource \"iam:teams\"",
  "message": "unauthorized: subjects [\"token:test\"] cannot perform action \"iam:teams:list\" on resource \"iam:teams\"",
  "code": 7,
  "details": []
}

$ sl | grep authz-service
[2019-04-23T18:39:03+00:00] authz-service.default(O): time="2019-04-23T18:39:03Z" level=info msg="Projects Authorized Query" action="iam:teams:list" projects="[p2]" resource="iam:teams" result="[]" subject="[token:test]"

b. When a user wants a list of teams they can see and doesn't apply a project filter, but a statement denies listing teams for all projects, ProjectsAuthorized returns no projects
$ curl -kH "api-token: $TEST" https://a2-dev.test/apis/iam/v2beta/teams | jq
{
  "error": "unauthorized: subjects [\"token:test\"] cannot perform action \"iam:teams:list\" on resource \"iam:teams\"",
  "message": "unauthorized: subjects [\"token:test\"] cannot perform action \"iam:teams:list\" on resource \"iam:teams\"",
  "code": 7,
  "details": []
}

$ sl | grep authz-service
[2019-04-23T18:36:54+00:00] authz-service.default(O): time="2019-04-23T18:36:54Z" level=info msg="Projects Authorized Query" action="iam:teams:list" projects="[]" resource="iam:teams" result="[]" subject="[token:test]"

c. when request includes no projects and one statement allows all projects and another statement denies one project, authz-service returns a list of all projects minus the denied projects
$ curl -kH "api-token: $TEST" https://a2-dev.test/apis/iam/v2beta/roles | jq
# returns chef-managed roles and role 1, no role 2
{
  "roles": [
    {
      "name": "Editor",
      "id": "editor",
      "type": "CHEF_MANAGED",
      "actions": [
        ...
      ],
      "projects": []
    },
    {
      "name": "Owner",
      "id": "owner",
      "type": "CHEF_MANAGED",
      "actions": [
        "*"
      ],
      "projects": []
    },
    {
      "name": "IAM Members Viewer",
      "id": "iam-members-viewer",
      "type": "CHEF_MANAGED",
      "actions": [
        ...
      ],
      "projects": []
    },
    {
      "name": "role 1",
      "id": "r1",
      "type": "CUSTOM",
      "actions": [
        "do:the:thing"
      ],
      "projects": [
        "p1"
      ]
    },
    {
      "name": "Ingest",
      "id": "ingest",
      "type": "CHEF_MANAGED",
      "actions": [
        ...
      ],
      "projects": []
    },
    {
      "name": "Project Admin",
      "id": "project-admin",
      "type": "CHEF_MANAGED",
      "actions": [
        ...
      ],
      "projects": []
    },
    {
      "name": "Viewer",
      "id": "viewer",
      "type": "CHEF_MANAGED",
      "actions": [
        ...
      ],
      "projects": []
    }
  ]
}

$ sl | grep authz-service
[2019-04-23T22:52:19+00:00] authz-service.default(O): time="2019-04-23T22:52:19Z" level=info msg="Projects Authorized Query" action="iam:roles:list" projects="[]" resource="iam:roles" result="[p1 (unassigned)]" subject="[token:test]"

:chains: Related Resources
:white_check_mark: Checklist

[ ] Necessary tests added/updated?
[ ] Necessary docs added/updated?
[ ] Code actually executed?
[ ] Vetting performed (unit tests, lint, etc.)?

Oh, almost forgot. Probably should consider adding some tests to conformance_v2_test.go to cover the new functionality.


Support React 16

PR for #1411

Thanks! This is a great start. I think the ^16.0.0-rc version should be compatible with ^16.0.0 but this won't hurt, and could make it even more clear to folks that we support with React 16.
For issue #1411 I would like react and react-dom updated in the devDependencies, which led to some errors last time I checked. Probably will also need to update to the latest Enzyme, and fix any breaking changes there. I'll update the issue description with those details.


enhancement print backend ip when brpc light pool is full

Proposed changes
Issue Number: close #xxx

run buildall


run tests via github actions

I suppose use github actions for run tests, autocheck PR before merge via tests

fixed #3b68757dfa900c325ea78da7e9b431e413ae43b7


Network errors on OEIS sequences lead to bad sequences being cached

As far as I can tell, sequence A002487 is returning this.seq.first as +Infinity which is probably not good.
https://oeis.org/A002487

As with #409, this sequence is seeming fine here. Changing the title of the bug to reflect that it is likely a caching error.

Ok, I cleared all browser history for all time, and then this sequence worked.  But A003462 doesn't.  Even with totally cleared browser history.  Am I going nuts?

Also A000225 and A007583 fail.
But these new-to-me ones work: A115067, A000326, A003463
This is after repeatedly clearing all browser history.

Did you pull my latest PR? It supposedly has a fix, which is to say I made a change I think will help.Let  me know if it does or does not.

Tried -- same result.

OK, on new browser with cleared cache and my latest PR, when you get a sequence mess up like this, open a new tab and try the same sequence and let me know if the same sequence fails twice in a row in different tabs. Thanks!

Yes, still happening in multiple tabs.

FWIW, it's not working at the old scope at numberscope.colorado.edu either.

Is it working for you?!

For example, go to numberscope.colorado.edu, add OEIS sequence A000225 and then try Differences -- it reports too few terms.  Which is not the case on the OEIS.

Ah, I see that https://numberscope.colorado.edu/api/get_oeis_name_and_values/A000225 is not working. So this one is a backend problem and needs to be diagnosed as such.

Same thing with A003462

Of the sequences flagged above, the ones that have "crawling too fast" errors logged are:

A007583
A000225
A003462

These are precisely the ones that still have problems as of today. Calling get_oeis_names_and_values on them returns a "value fetching in progress" error.

This needs https://github.com/numberscope/backscope/issues/154 to be resolved. Hence assigning to milestone beta, so as not to block alpha on backscope. But we could move it up if the team prefers and if @Vectornaut will have availability to work on the backscope part.


Replace gomock import with go.uber.org/mock

How to categorize this PR?

/area dev-productivity
/kind cleanup
What this PR does / why we need it:
Since as of Jun 27th 2023 the https://github.com/golang/mock/ project as been archived and the project continues to be maintained here: https://github.com/uber-go/mock this PR introduces a replacement of the golang/mock package with the new one.
This PR might be potentially breaking as extension projects are importing and using mockclient.NewMockReader in their tests.
Release note:

Replace `gomock` import with `go.uber.org/mock`

/retest


Game forgets tracker

Getting a bug where, if you play and close the game then come back to it, the tracker resets but also some of the actual drops seem to change too
To reproduce, make a world, turn on loot tables and villager trades, play the game to unlock some of the tracker info. Close world and close game. Re-open game and world, check tracker, info should be missing

Adding onto this, it also forgets it when you die (and I have just confirmed that it definitely does completely change loot tables if you exit game and reload game

This should have been fixed in the latest release, are you using 2.5.1?

Hi, yes I am using 2.5.1

I can't replicate this in 2.5.1, so I'm going to assume that it is a mod conflict

Should be fixed in 2.5.2; close if so

Okay I will update when 2.5.2 is out and see if the drops change - if they still do I will assume it's a mod conflict and I will figure out which one it is and let you know

its out already https://modrinth.com/mod/trulyrandom/version/2.5.2


Document Microsoft Graph OAuth Config

See the comment here: https://github.com/influxdata/chronograf/issues/4823#issuecomment-440040140
we should add a section into the generic config chronograf docs for Microsoft Graph as Generic OAuth: https://docs.influxdata.com/chronograf/v1.7/administration/managing-security/#configuring-generic-authentication

Hey @russorat, I need more specifics and context with this one. What exactly is it that needs to be added?

I think you should add the information from the comment to the example section of that page, just below the other one: https://docs.influxdata.com/chronograf/v1.7/administration/managing-security/#examples

Hello guys, for two type of User: Azure AD and Invited User from live or hotmail:
I Use "unique_name" rather than "upn" in GENERIC_API_KEY
I hope this tips help you!
Regards,
Marhbere


Multiple improvements

This PR includes code refactoring, database table name configuration and PHPUnit tests.
Further work on tests is required due to incomplete coverage.

Closing this, I’m not looking for contributors
Not only this, your changes are breaking changes, refactor migrations should be appended not edited, otherwise it requires rollback and unpredictable effects to existing environments that use the package (3 of which are my own).
Feel free to fork and release your own version if you are keen to access these changes via packagist.


log level of the driver should be configurable

Allow to define the log level of the driver via the agent.yml file.

Implemented.


why my Transitions between Activities not work?

I do it it like this in my BasicActivity onCreate():
getWindow().requestFeature(Window.FEATURE_CONTENT_TRANSITIONS);
setupWindowAnimations();
super.setContentView(R.layout.layout_base);
setContentView(getContentViewID());
My setupWindowAnimations() function like this:
protected void setupWindowAnimations() {
L.i("setupWindowAnimations");
Slide slide = (Slide) TransitionInflater.from(this).inflateTransition(R.transition.activity_slide_out);
Fade fade = (Fade) TransitionInflater.from(this).inflateTransition(R.transition.activity_fade_in);
getWindow().setEnterTransition(slide);
getWindow().setExitTransition(fade);
}
when I call startActivity(A,B), the animations not worked.Can u help me? Thanks a lot!

@chenchongyu try this to start your activity:
Intent intent = new Intent(this, YourActivity.class);
ActivityOptionsCompat options = ActivityOptionsCompat.makeSceneTransitionAnimation(this);
startActivity(intent, options.toBundle());

Thanks for reply! I have another question.May I use this animation below sdk 21 ?

@chenchongyu I don't know. But maybe this would be useful stackoverflow

It is usually not recommended to try to fake those transitions on lower API levels. But it's up to you :)

@guliash Thank you! This worked for me.

override fun onClick(view: View?) {
if (view != null) {
val intent: Intent = Intent(view.context, DisksActivity::class.java)
val bundle: Bundle = Bundle()
bundle.putString(view.context.getString(R.string.EXTRA_DISK_TITLE), itemView.tag.toString())
intent.putExtras(bundle)
//TODO try shared element feature (i have to translate my comments)
val options = ActivityOptionsCompat.makeSceneTransitionAnimation( view.context as Activity)
view.context.startActivity(intent,options.toBundle())
}
}

I had the same issue that @chenchongyu had had with transitions, but in 2018 and in kotlin.
@lgvalle right: fake animations for lowers API levels are ugly.


[Feature]:  Multi-agent example

AgentScope is an open-source project. To involve a broader community, we recommend asking your questions in English.
Can anyone give me a multi-agent system using agentscore with Hierarchical Agent Group strategy?
or else tell me the multi-agent related functions in this library

Sorry, what do you mean by "hierarchical agent group strategy"?
To build multi agents conversation, agentscope.msghub is recommended.


fix: i2c - async write_read in parallel

Attempt at parallelising the write_read I2C async fn implementation.

[ ] There might be a need for a join-style async fn wait_for_completion that will wait for both operations to finish, instead of select-ing one or the other

Must

[ ] The code compiles without errors or warnings.
[ ] All examples work.
[ ] cargo fmt was run.
[ ] Your changes were added to the CHANGELOG.md in the proper section.
[ ] You updated existing examples or added examples (if applicable).
[ ] Added examples are checked in CI

Nice to have

[x] You add a description of your work to this PR.
[ ] You added proper docs for your newly added features and code.

Thanks for the PR, I'll do my best to get to this next week!

What's the status here? Can we move this along somehow?

I would suggest someone to pick up this branch/issue and continue it.
I have switched to a different platform for the time being and don't think I'm going back to the issue any time soon.
It still needs a few fixes I suppose and to be tested

@jessebraham where/how was this fixed?
Can you provide a link?

It wasn't fixed yet, you said you were not going to work on it anymore so I closed the PR. I was planning on looking into it this or next week.


Bump IREE to latest commit on main.

We're approaching our 3.1.0 release across all IREE packages, so bumping to build/test closer to HEAD.
PR https://github.com/nod-ai/shark-ai/pull/721 updates to the latest nightly, but we can also update to the latest commit, which includes multiple changes to https://github.com/iree-org/iree/tree/main/runtime/src/iree/hal/drivers/hip that we should test soon.
Commit range relative to https://github.com/nod-ai/shark-ai/pull/721: https://github.com/iree-org/iree/compare/iree-3.1.0rc20241220...0a0483e997bea993cdf26e62ef5c68a043011e1b

From these logs: https://github.com/nod-ai/shark-ai/actions/runs/12439009356/job/34732288777?pr=722#step:7:253, these errors:
ERROR tests/apps/sd/e2e_test.py::test_sd_server - RuntimeError: Timeout waiting for server start
ERROR tests/apps/sd/e2e_test.py::test_sd_server_bs4_dense - RuntimeError: Timeout waiting for server start
ERROR tests/apps/sd/e2e_test.py::test_sd_server_bs8_percall - RuntimeError: Timeout waiting for server start
ERROR tests/apps/sd/e2e_test.py::test_sd_server_bs4_dense_fpd2 - RuntimeError: Timeout waiting for server start
ERROR tests/apps/sd/e2e_test.py::test_sd_server_bs8_dense_fpd8 - RuntimeError: Timeout waiting for server start

seem to be new regressions, but I would expect more helpful error output, ideally from a unit test.

FYI @AWoloszyn @MaheshRavishankar . The only new IREE commits here are from both of you. Maybe the test failures make more sense to one of you 🤷


Cart encodeUrl Bug

PHP Version
8.0.11
Shopware Version
<IP_ADDRESS>
Actual behaviour
Calling the cart will lead to an hard error after adding the plugin. Since Shopware stores the serialized object in the DB it also stores the image path. By doing so the encodeUrl will crash at the second explode call.
[2022-05-13T09:55:55.491135+00:00] request.CRITICAL: Uncaught PHP Exception TypeError: "explode(): Argument #2 ($string) must be of type string, null given" at /home/www-data/XXXXX/vendor/store.shopware.com/froshplatformthumbnailprocessor/src/Storefront/Framework/Twig/Extension/UrlEncodingTwigFilter.php line 35 {"exception":"[object] (TypeError(code: 0): explode(): Argument #2 ($string) must be of type string, null given at /home/www-data/enjoy365/vendor/store.shopware.com/froshplatformthumbnailprocessor/src/Storefront/Framework/Twig/Extension/UrlEncodingTwigFilter.php:35)"} []
Expected behaviour
Migrate the URL somehow or add info to the Readme that the cart table has to be emptied.
Steps to Reproduce?
Use a shop without the plugin. Add things to the cart. Install the plugin and call the cart afterwards

could you please tell me what url goes into there? it sounds like it is not a valid url/path 🤔

Just catch that error lol and throw an exception 😅

@tinect I think it was a CDN Url, but valid, smth like:
https://reitsport.gorillacdn.ch/media/e8/01/6a/1630940608/3be214dfe80645b6b4af989b85d5eaf6.jpg
Can't reproduce it, but we had multiple customers that could not login (due to cart recreation error). I agree on the try / catch

This sounds really creepy to me, while someone adds an invalid path for media.
Something local stuff on your devs? :-)
But you are welcome to create a valid PR.

Nope, was live and the urls where checked and valid. Anyways, too lazy to find a week old backup. If it happens again I will create a PR


Men 3679: Document all the tools available in the Mender-docs repository

Do not forget to clean up history before merging. I assumed you would do it, so did not comment :)

Do not forget to clean up history before merging. I assumed you would do it, so did not comment :)

What's wrong with the history here?

All of these are just fixes to the original commits that are non-existing in master.
Add a Readme for the 'didyoumean' script …
Review fixups …
Rename from Readme-X.md -> README-X.markdown …
Remove the $ prefix …
Change bash highlighting to console for console examples

Above I consider "review history", but I think that is what the PR is for on github, and there is no need to have this information in the git history.
Cleaner would be:
Add a Readme for the passive.sh script …
Add a Readme for the weasel.sh checker …
Add a Readme for the checklinks script …

Also, https://github.com/mendersoftware/mender/blob/master/CONTRIBUTING.md#example :)
As this is what this PR did.

All of these are just fixes to the original commits that are non-existing in master.
Add a Readme for the 'didyoumean' script …
Review fixups …
Rename from Readme-X.md -> README-X.markdown …
Remove the $ prefix …
Change bash highlighting to console for console examples

Above I consider "review history", but I think that is what the PR is for on github, and there is no need to have this information in the git history.
Cleaner would be:
Add a Readme for the passive.sh script …
Add a Readme for the weasel.sh checker …
Add a Readme for the checklinks script …

Also, https://github.com/mendersoftware/mender/blob/master/CONTRIBUTING.md#example :)

Touche!
Noted.


Organization suggest: elasticsearch really needed?

Search in elasticsearch now done as follows:

fields id, name_dut, name_eng, other_id.ugent_id, other_id.ugent_memorialis_id and  other_id.biblio_id are copied to a "catch all field" all
client splits input string on whitespace, and executes a wildcard search on each part: BI becomes all:BI*

So search is rather simpel, so could "easily" be converted to a sql statement:
SELECT * FROM organization WHERE id ILIKE 'BI%' OR name_dut ILIKE 'BI%' OR name_eng ILIKE 'BI%'

but I am excluding the jsonb queries, which may become very difficult, and which probably have no LIKE available (WHERE other_id @> '[{"id":"BI"}]' is too specific)

Ranking results:
select id, full_name, ts_rank_cd(ts, tsq) as tsr from person, to_tsquery('nico:* & fra:*') tsq where ts @@ tsq order by tsr asc;

cf. https://leandronsp.com/a-powerful-full-text-search-in-postgresql-in-less-than-20-lines
But that cannot be done in entgo


Incorrect type definition in json-schema.d.ts

What version of Ajv are you using? Does the issue happen if you use the latest version?
Checked on "ajv": "^8.6.0"
Your typescript code

In library file json-schame.d.ts
declare type StrictNullChecksWrapper<Name extends string, Type> = undefined extends null ? `strictNullChecks must be true in tsconfig to use ${Name}` : Type;

Typescript compiler error messages
Used "tsc": "^2.0.3"
yarn run v1.22.10
$ yarn run build:compile && yarn run build:copyfiles
$ tsc
node_modules/ajv/dist/types/json-schema.d.ts:1:92 - error TS1005: '?' expected.

1 declare type StrictNullChecksWrapper<Name extends string, Type> = undefined extends null ? `strictNullChecks must be true in tsconfig to use ${Name}` : Type;
                                                                                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

node_modules/ajv/dist/types/json-schema.d.ts:1:151 - error TS1005: ';' expected.

1 declare type StrictNullChecksWrapper<Name extends string, Type> = undefined extends null ? `strictNullChecks must be true in tsconfig to use ${Name}` : Type;
                                                                                ~

Found 2 errors.

error Command failed with exit code 2.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
error Command failed with exit code 2.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Strange, but ts-node runs that code as expected.

Describe the change that should be made to address the issue?
Fix types  to use with tsc.
Are you going to resolve the issue?

How was this fixed?

any progress？

it's not something that was fixed, likely incorrect settings (strictNullChecks must be true) or old tsc version (must be >4.2.3) or some other installation issue.

any news about this issue ?

How should this be fixed ?

I have tried using     "strictNullChecks": true in tsconfig.json , but still got the error.

I am going to close comments on this issue - it is not really defined what this issue was, and the issue may be that you simply have an incorrect schema or that JSONSchemaType has limitations and cannot be used in your case.
Please submit a new issue with a full code sample from a clear repository once you've checked the following:

strict mode is enabled in tsconfig.json (or at least strictNullChecks)
TS version is >= 4.2.3
you are importing the latest Ajv version (sometimes calling npm install ajv may install the previous version because some other dependency uses it - use npm install ajv@8 and try clean install: rm -rf node_modules && npm I
check that your schema is indeed valid - that required properties are listed in "required", that optional properties have nullable: true, that "type" is set in all schemas - in many cases what was reported as JSONSchemaType issue was a user error where their schema was incorrect - it correctly shown compilation error.
Check other open issues related to JSONSchemaType - there are some known limitations.

A simple temporary workaround to use Ajv without using JSONSchemaType - it is optional.


Generator of tests for private requests

coverage: 27.355%. remained the same
when pulling fc7c7aeeaccfd2a7157d464229b897d19e3b01e6 on yzh-pelle:private_tests_generator
into c5296c7508c9315ac1bc0dbf7bce688dd16cfe97 on sammchardy:master.


Could not fetch specs from http://gems.hashicorp.com/

Vagrant 1.7.4
Fedora 22
Virtual Box: 5.0.2
$ sudo vagrant plugin install vagrant-vbguest
Installing the 'vagrant-vbguest' plugin. This can take a few minutes...
Bundler, the underlying system Vagrant uses to install plugins,
reported an error. The error is shown below. These errors are usually
caused by misconfigured plugin installations or transient network
issues. The error from Bundler is:
Could not fetch specs from http://gems.hashicorp.com/
Warning: this Gemfile contains multiple primary sources. Using source more than once without a block is a security risk, and may result in installing unexpected gems. To resolve this warning, use a block to indicate which gems should come from the secondary source. To upgrade this warning to an error, run bundle config disable_multisource true.
curl http://gems.hashicorp.com/

Resolves ok.

Sorry @lukehinds, this is not a vagrant-vbguest issue. The messages states:

These errors are usually caused by misconfigured plugin installations or transient network issues

which is, dispute the successful "curl", most likely the probably the reason.

Related:
http://stackoverflow.com/questions/33109186/how-to-install-the-proxy-plugin-behind-a-proxy-for-vagrant


Tasks panel update

It would be nice when you start flashdevelop or open/switch a project, it will refresh the tasks panel. It would be nicer if it keeps up-to-date, now I manually have to refresh.

There is a setting to make it update automatically, it's on manual by default. Atm it updates only the opened files or when user hits refresh.

Ah, is that the "complete" mode setting?
May I ask why it isn't enabled by default?

Sorry, i forgot to respond. The setting is not enabled by default as it constantly monitors the files and cause file reads even if you don't use it at all. This might cause issues to some for example when working under virtualization or with a slow HDD.

Thanks! :smile:


[BUG][opensearch] Opensearch helm chart failing to install due to kyverno securitycontext restriction

Describe the bug
I am trying to install opensearch helm chart in a k8s cluster that has kyverno cluster policies that enforces having securityContexts in place. There are couple of initContainers which securityContext is missing and this is not allowing me to install the chart.
1 error occurred:\n\t* admission webhook \"validate.kyverno.svc-fail\" denied the request: \n\nresource StatefulSet/opensearch-test/groot-opensearch-cluster-master was blocked due to the following policies \n\ndisallow-privilege-escalation:\n  autogen-privilege-escalation: 'validation error: Privilege escalation is disallowed.\n    The fields spec.containers[*].securityContext.allowPrivilegeEscalation, spec.initContainers[*].securityContext.allowPrivilegeEscalation,\n    and spec.ephemeralContainers[*].securityContext.allowPrivilegeEscalation must\n    be set to false.          . rule autogen-privilege-escalation failed at path\n    /spec/template/spec/initContainers/0/securityContext/'\n\n\n\n"
To Reproduce
Kyverno should be installed on a k8s cluster
Chart Name
opensearch and opensearch-dashboards

[Triage]
Hey @kndoni is Kyverno blocking the installation? Can you try without the Kyverno once and see, I assume there must be some allowlist using Kyverno. If not the there has to be a setting to use allowPrivilegeEscalation to false.
Thank you
@peterzhuamazon


petl.fromdicts([OrderedDict(),...]) should use header ordering from ordered dicts

This would be nice to have and seems like a reasonable expectation for fromdicts() behavior.

Looking at this now.


Missing required parameters for Route

Laravel Version: 6.9.0
PHP Version: 7.2.25
Database Driver & Version: Mysql

Description:
After upgrading Laravel 5.8 to 6.9 I got errors regarding url generation from named routes. The error is 'Missing required parameters for Route: XXXX'.
Steps To Reproduce:
This was warking just fine:
$url = route('online.invoice', ['subdomain' => $user->subdomain, 'uuid' => $invoice->urlcode]);

Now, only way to make this work is to remove the keys like this
$url = route('online.invoice', [$user->subdomain, $invoice->urlcode]);

Did you read the upgrade guide: https://laravel.com/docs/6.x/upgrade
Especially the part on URL generation?

Laravel Version: 6.9.0
PHP Version: 7.2.25
Database Driver & Version: Mysql

Description:
After upgrading Laravel 5.8 to 6.9 I got errors regarding url generation from named routes. The error is 'Missing required parameters for Route: XXXX'.
Steps To Reproduce:
This was warking just fine:
$url = route('online.invoice', ['subdomain' => $user->subdomain, 'uuid' => $invoice->urlcode]);

Now, only way to make this work is to remove the keys like this
$url = route('online.invoice', [$user->subdomain, $invoice->urlcode]);

Thank you i have upgraded laravel from 5 to 9 and now this method is working perfectly fine...


Regression: sites with next export should cause the plugin to no-op

Describe the bug
Because sites using next export don't need special treatment (they generate static assets that follow the Jamstack architecture pattern), this plugin doesn't need to do anything when that option is set. However, we do want to plugin available in case folks switch from using export to using Next features that need the plugin.
To support that, we need the plugin to no-op when next export is set. This worked before 3.0.0, which introduced a lot of changes around next config handling to support the breaking changes in<EMAIL_ADDRESS>so we have a regression somewhere.

just kidding! it still works. we were seeing this due to using an outdated version of the plugin 😅


gh-actions: ci: use ubuntu-18.04

Signed-off-by: Alessandro Comodi<EMAIL_ADDRESS>

Don't have a problem with this.  Are you planning to do a follow up and update this to ubuntu-20.04?

Are you planning to do a follow up and update this to ubuntu-20.04?

Yes. The main reason for this PR was to fall back to 18.04 as the ubuntu-latest is now giving problems with GH CI on master (the latest version must have changed recently):
Package g++-6 is not available, but is referred to by another package.
This may mean that the package is missing, has been obsoleted, or
is only available from another source

E: Package 'g++-6' has no installation candidate


BUNDLER_HOME env var

Since 1.14.x, Bundler tries to create a fallback home directory underneath Dir.tmpdir in case the user home read from Bundler.rubygems.user_home (doing File.expand_path "~") is not writable. This is causing problems in certain scenarios such as #5371 and #5363.
How about adding an environment variable $BUNDLE_HOME:

When set to a path, the path is used instead of Bundler.rubygems.user_home. Good for deployments to readwrite FS where the regular user home is not writable due to security restrictions.
When set to "", no fallback home directory is created and Bundler does not attempt to read the global config. Good for deployments to writeonly FS (e.g. Docker).

As I posted in https://github.com/bundler/bundler/issues/5371, bundler should swallow the error when attempting to read the global config anyways


CSP error when attempting fetch of Data URIs in the web

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

VS Code Version: 1.96.0-insider
OS Version: Windows_NT x64 10.0.26120

Description:

Refused to connect to 'data:' because it violates the following Content Security Policy directive: "connect-src 'self' https: wss: http://localhost:* http://<IP_ADDRESS>:* ws://localhost:* ws://<IP_ADDRESS>:*".

Steps to Reproduce:

Clone feat-oniguruma-streaming-compilation
Run npm install
Run git submodule update --init --recursive
Run npm test

Tests will fail at harness-matlab-web with TypeError: failed to fetch.

The issue appears in the latest Insiders and in 1.95.1 on github.dev.

An easier repro is to install the github.vscode-github-actions extension to  github.dev.
The extension throws an error trying to fetch the WASM octet stream in libsodium-wrappers.


Set DYLD_LIBRARY_PATH so dyn libs can be loaded

Running rustfmt when inside the environment created by devenv on a Mac fails for me with the following:
$ rustfmt
dyld[3574]: Library not loaded: @rpath/librustc_driver-b3a4a775d2ec4bdd.dylib
  Referenced from: <no uuid> /nix/store/hf22ddbp9pqlgskriy9b16zmjwlcp3rc-rustfmt-1.68.2/bin/rustfmt
  Reason: tried: '/nix/store/1icn9va0m09gi8l64061f49gdwjq2pbd-apple-framework-CoreFoundation-11.0.0/Library/Frameworks/librustc_driver-b3a4a775d2ec4bdd.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/nix/store/1icn9va0m09gi8l64061f49gdwjq2pbd-apple-framework-CoreFoundation-11.0.0/Library/Frameworks/librustc_driver-b3a4a775d2ec4bdd.dylib' (no such file), '/nix/store/1icn9va0m09gi8l64061f49gdwjq2pbd-apple-framework-CoreFoundation-11.0.0/Library/Frameworks/librustc_driver-b3a4a775d2ec4bdd.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/nix/store/1icn9va0m09gi8l64061f49gdwjq2pbd-apple-framework-CoreFoundation-11.0.0/Library/Frameworks/librustc_driver-b3a4a775d2ec4bdd.dylib' (no such file), '/usr/local/lib/librustc_driver-b3a4a775d2ec4bdd.dylib' (no such file), '/usr/lib/librustc_driver-b3a4a775d2ec4bdd.dylib' (no such file, not in dyld cache)
[1]    3574 abort      rustfmt

Setting DYLD_LIBRARY_PATH as suggested in https://github.com/rust-lang/rustfmt/issues/1707#issuecomment-309975978 seems to fix the issue, although I'm not sure if my proposal could have other consequences. I have checked the issues but I haven't seen any mentioning this problem.
Perhaps setting LD_LIBRARY_PATH is also needed for Linux systems, but I have not tested it.

@DavSanchez AFAIK setting DYLD_LIBRARY_PATH should not be necessary.  It appears that the rustfmt in nixpkgs is currently broken for darwin.  I tried it on an older revision of nixpkgs that I have pinned and it ran without error:
$ nix shell 'github:NixOS/nixpkgs?rev=f5ffd5787786dde3a8bf648c7a1b5f78c4e01abb#rustfmt'

$ rustfmt --help
Format Rust code

usage: rustfmt [options] <file>...

Ahh I hadn't thought about rustfmt being actually broken for unstable. Just checked from an earlier revision as well and was able to run it without issues. I guess this can be closed then. Thanks a lot for your help!


olleH dlroW

I think your Example from the start page produces   olleH dlroW and not dlroW olleH. Only the message is reversed (single OnNext argument) and not the whole message list is inverted. Or did I missed something in the code? Cause you didn't add a special threading scheduler for rx pat the order of messages should be delivered as added with OnNext calls. Didn't tried, just reading your code sample... ;-)

Hi :)
Sorry, apparently I've spent too much time playing with more recent versions of the dotnet SDK, so I can't build / run this right now. It's been a while, so I don't remember what the expected behaviour is, but at first glance it does look like you're right!
I'll try to get it building with a more recent version of the SDK and build a test-harness to try it out.

(either way, thanks for taking the time to report it!)

Yep you're right about the order - don't know how I missed that.
I'll update the README. That ToSubject extension method doesn't actually exist BTW, I hadn't got around to building it when I forgot about this project.
Maybe this is something worth working on over the holiday break :)

I've close this issue, but only because I'm tracking the work in #2.
Thanks again for the catch.


MAINT: special: remove dependence of xsf::numpy on sf_error

Reference issue

What does this implement/fix?

This PR makes it so scipy/special/xsf/numpy.h no longer depends on scipy/special/sf_error.cc, which is outside of the xsf codebase. This is the last structural barrier in the way of beginning the migration of the scipy special C++ headers into the xsf library, as described here: https://github.com/scipy/xsf/issues/1.
Additional information

@izaid, the change here is pretty straightforward, let me know if everything looks good to you. @mdhaber, would you be willing to merge with @izaid's approval?

Thanks @rgommers!

Following the following guidance from the SciPy Core Developer's guide:
In case review doesn’t happen quickly enough and it is important that the PR is merged quickly, the submitter of the PR should send a message to the forum saying they intend to merge that PR without review at time X for reason Y unless someone reviews it before then.
I made a request to self-merge on discourse (https://discuss.scientific-python.org/t/request-to-self-merge-gh-21802/1520), on Monday, stating I would self-merge with only @izaid's approval if no one merged before today. Thanks @rgommers for reviewing this; I updated the stale comment you noticed and plan to follow up with a PR to fix the inconsistent naming that you noticed, but perhaps after the xsf migration takes place.


Deploy containers to azure host and expose to the internet

As a system owner, I would like the code base to be deployed to the azure cluster and visible from the internet.

Deployed at http://kubernetes-master.cloudapp.net/


Revised cultural fit description

New wording by @superroz, to give a more precise definition and make it consistent with the way we describe 'Technical competence'.

Looks good.  Would be nice to fix the version number, but other than that  :+1:

You'll need to add a small story to the backlog to pull this updated content into the buyer app.


Explicit peer discovery

This creates a new module PeerListener that is responsible for finding peers and then integrates it into tunnel manager.
The end result is a pretty large refactor and a significant replacement of shelling out with pure rust code. This already seems faster and more stable than the previous version.
What it does still need is some tests for both tunnel manager and peer listener and some more IRL testing. This should be backwards compatible.

So I'm 80% sure that the optimal route failure just a parsing problem @drozdziak1. Backwards compat takes the same topology and puts a rubber stamp on it. if you could investigate I would appreciate it.

@drozdziak1 one last review then merge, hopefully.

@drozdziak1 been testing this on real routers and exits + integration tests locally all day, it's rock solid as far as I can tell. We also really need it in to send out testing builds by Friday (aka Thursday night)


HAL LINK for Sets on Event Pages

Problem Description:
the front end wishes to add missing data to set/collection pages by creating a HAL link that will add information of a  Set to the Event.
Expected Behavior/Solution:
Create a HAL link which returns the following query:
TBD- dependent on when peter creates the HAL link
Requirements:
Once HAL link is created, Set information will appear under Event. (Please see mock of dependency issue)
Needed for promotion:
If an item on the list is not needed, it should be crossed off but not removed.

[ ] ~Wireframe/Mockup - Mike~
[ ] ~Committee discussions - Sarah~
[ ] ~Feasibility/Team discussion - Sarah~
[ ] ~Backend requirements - TBD~
[ ] ~Frontend requirements- TBD~
[ ] ~Questions~

UAT/LUX Examples:
https://lux-front-tst.collections.yale.edu/view/set/664c4aeb-532f-46a2-9db8-4d0b51c73a64 should render information related to https://lux-front-tst.collections.yale.edu/view/activity/ef9bfa8b-1d19-4b44-a004-a5ab75cdf6d7
Dependencies/Blocks:
Dependency:  https://github.com/project-lux/lux-frontend/issues/88
Related Github Issues:

https://github.com/project-lux/lux-middletier/issues/40 (new version of issue)

Related links:
~Wireframe/Mockup:~
~Place wireframe/mockup for the proposed solution at end of ticket.~

@kamerynB Would u be able to help me form this? It is the HAL link issue for 88 https://github.com/project-lux/lux-frontend/issues/88

I have formed this the best I could. If any information is missing please let me know or feel free to make changes. Thank you.

@ar2674 Thank you for adding this. I must have missed it.

closing as dupe of new tix: #40


Passing Rust struct ownership into JS

Currently you can't pass ownership of a rust struct from rust to JS through an import, like this:
#[wasm_bindgen]
struct Foo();

#[wasm_bindgen]
extern {
    fn take_struct(foo: Foo);
}

but you can return it from an exported function:
#[wasm_bindgen]
fn create_foo() -> Foo {
    Foo()
}

Is this a conscious style choice or is passing in structs by value just not implemented yet?
In both cases js takes ownership of the object and becomes responsible for freeing it if it's dropped. But I can see that being clearer when the object is a return from a rust export, than when receiving an object as an argument from rust.

Oh oops I think that's an oversight! I think this definitely makes sense to support

Added in https://github.com/alexcrichton/wasm-bindgen/pull/70


Dependency upgrades

Upgrade build dependencies to the latest minor version (the only runtime dependency of the UI components is marked)

Fixes a bunch of warnings regarding deprecated sub-dependencies

Removes underscore since the computed property wasn't in use
Explicitly moves marked to a peer (external) dependency - it's already marked as external and not included in the bundle
Major version upgrade of Rollup - unaffected by breaking changes

Currently unable to get this to run/install locally, working through what the issue may be

👍 to @joepavitt's changes


Gopass mv moves fuzzy names even on exact match and doesn't rename directories

Steps To Reproduce
$ GOPASS_HOMEDIR=~/test gop generate comm/test 12
$ GOPASS_HOMEDIR=~/test gop generate comm/test2 12
$ GOPASS_HOMEDIR=~/test gop generate communication/t1 12
$ GOPASS_HOMEDIR=~/test gop ls
gopass
 ├── communication/
 │          └── t1
 └── comm/
        ├── test
        └── test2

$ GOPASS_HOMEDIR=~/test gop mv comm email
$ GOPASS_HOMEDIR=~/test gop ls
gopass
└── email/
    └── comm/
        ├── test
        ├── test2
        └── unication/
            └── t1

Expected behavior
Should get instead:
$ GOPASS_HOMEDIR=~/test gop ls
gopass
 ├── communication/
 │          └── t1
 └── email/
        ├── test
        └── test2

Notice also that the folder comm wasn't renamed but actually moved into a new folder instead.
This is not similar to how Unix's mvworks, which is what we aim for with gopass mv IIRC.

I think the Steps to Reproduce is slightly off. email/ should be comm/ in the first ls output.
But then I think you are right: This looks like a bug :/

Right, I decided to add the first ls after having run the rest. Edited.


Update NNNN-async-function-isolation.md

I wasn't quite able to get this code to compile, but I think this is what you were going for?

Oops sorry I did this wrong. Will leave this in the swift-evolution PR instead!


FatWallet closed their operations

https://www.fatwallet.com/fatwallet-goodbye
What other referral-sites are there besides Slickdeals?

@borzaka I think it is time to remove those domains from the list


[gardening] Move class local to a .cpp file into anonymous namespace instead of the swift namespace.

Anonymous namespaces are how one communicates in c++ that a class is only
available in the local file. It allows for the optimizer then to make more
aggressive assumptions around the class's invariants since it is file local.

Just NFC cleanup.

@swift-ci smoke test and merge

Its forward declared in a header... sigh.


setup episode needs to be updated

especially https://enccs.github.io/julia-for-hpda/setup/#installing-packages and https://enccs.github.io/julia-for-hpda/setup/#configuring-mpi and https://enccs.github.io/julia-for-hpda/setup/#using-eurohpc-systems

Have updated the package list. Will refine it a bit later.
Removed the rest. Including MPI and EUROHPC sections.


fix(all): rename remaining 'serde-impls' features gates to 'serde'

This renames the remaining 'serde-impls' features I forgot to change in aac3e00 to 'serde'

Thanks!


Stored UCANs must be referenced by CID

https://github.com/ucan-wg/ucan-as-bearer-token#22-ucan-by-cid-table

Implemented on next branch.


[Bug] Github version of ulx and ulib breaks this addon possibly

It will say "this (whatever) has been restricted), however you can still spawn it.

This means that another addon is interfering with URM. Any addon that uses the "PlayerSpawn" hooks badly will override URM.

I first tested the version of ulib and ulx from these two links:
http://ulyssesmod.net/archive/ulx/ulx-v3_62.zip
http://ulyssesmod.net/archive/ULib/ULib-v2_52.zip
The restrictions worked fine with these versions.
I then tested with the github version of ulx and ulib from these two links:
https://github.com/Nayruden/Ulysses.git/trunk/ulx
https://github.com/Nayruden/Ulysses.git/trunk/ulib
The restrictions no longer worked.

It appears you are right. I cannot however locate the problem, until I do, I suggest you use the version from their website.

This has been fixed.


GROOVY-4990 Make File.write() create any missing directories. Add test.

This PR will:

Fix issue where java.io.FileNotFoundException is thrown when parent directories don't exists
Add test

https://issues.apache.org/jira/browse/GROOVY-4990

Hi, thanks for your PR. While automatically creating the parent directories is in some ways super nice, it might also be surprising for folks familiar with current Java behavior. I think I prefer the mkParentDirs suggestion mentioned in the Jira issue, though I'd probably call it createParentDirectories these days to match the newer nio styling. Then it could also be used for setBytes and so on, not just write. What do you think about that suggestion and if you like the idea, would you feel up for altering the PR?
Suggested methods to add:
File ResourceGroovyMethods#createParentDirectories(File self)
Path NioExtensions#createParentDirectories(Path self, FileAttribute<?>... attrs)

Usage:
file.createParentDirectories().write(...)
path.createParentDirectories().bytes = ...

What do you think? Or if you want me to progress, just let me know.

@paulk-asert We implented this (easy) feature because participating an open source project is a mandatory part of our university's course Software Engineering Methodologies. So to be honest I [we] have no further opinion about this.

I merged and tweaked your test and modified the DGM methods as per the earlier suggestion. Thanks.


Addresses an issue when using multiple BrickCollectionViews as Cells …

…within a BrickCollectionView and then scrolling fast.
The collectionView which is performing the batchUpdates on the AsynchronousResizableCell gets deallocated inflight, and causes a crash.  This changes it from using having the cell hold a closure, to using a delegate callback to the collectionView which then performs the batch update.

Codecov Report

Merging #125 into swift3 will increase coverage by 0.07%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           swift3     #125      +/-   ##
==========================================
+ Coverage   93.06%   93.14%   +0.07%
==========================================
  Files          38       38
  Lines        3043     3048       +5
==========================================
+ Hits         2832     2839       +7
+ Misses        211      209       -2

Impacted Files
Coverage Δ

Source/Cells/BrickCell.swift
97.8% <ø> (ø)
:arrow_up:

Source/ViewControllers/BrickCollectionView.swift
95.56% <100%> (+0.69%)
:arrow_up:

Source/Bricks/Image/ImageBrick.swift
77.98% <100%> (ø)
:arrow_up:

Source/Bricks/Collection/CollectionBrick.swift
86.73% <100%> (+0.13%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 27445bb...9ed61bb. Read the comment docs.