[v6] Server side rendering support

What version of React Router are you using?
6
Steps to Reproduce
Is there any documentation on server-side rendering with react-router v6? I am currently going through this guide:
https://morioh.com/p/995f1244f33b
and have encountered this error:
TypeError: a.replace is not a function
when removing context from the StaticRouter.
I also can't seem to find any updated docs for server side rendering with react router v6. These are the current docs:
https://reactrouter.com/web/guides/server-rendering
which still use context.
Expected Behavior
To not experience any bug after removing context from StaticRouter
Actual Behavior
ReactDOMServer.renderToString throws an error.

Hey @aneurycasado, if you can provide a minimal reproduction, perhaps in a test repo, I'll be happy to re-open this. All docs for v6 are a work-in-progress, but you can find them here on GitHub for now on the dev branch: https://github.com/ReactTraining/react-router/blob/dev/docs/
The specific doc I think you're looking for is on migrating from 5 to 6: https://github.com/ReactTraining/react-router/blob/dev/docs/advanced-guides/migrating-5-to-6.md
Let me know if you still have any issues. Others may be able to help as well, but without some reproduction for us to spot the error it's very difficult to diagnose and point to any specific solutions.


Use ws url based on current page's url

Instead of using a fixed URL, base the ws url based off of the current http URL. This will make it easier to modify the examples, such as connecting to an existing Wi-Fi network instead of creating one.

Thanks!


Can't still stick the bottom of Alacritty when drag the Alacritty to scale

Can'tstill stick the bottom of Alacritty when dragging the Alacritty to scaling
To Reproduce
Steps to reproduce the behavior:

dragging the Alacritty to scaling
zjstatus can't still stick the bottom of Alacritty

Expected behavior
when dragging the Alacritty to scaling,  zjstatus can't still stick the bottom of Alacritty
Screenshots
as the screenshot shown below:

Desktop (please complete the following information):

OS: Sonoma 14.2.1 (23C71)
Zellij version v0.39.2
Version latest version

OK, Thanks for your response so quickly. So you mean that to layout rendered incorrectly caused this issue? you mentioned that if set  setting the TERM environment variables correctly, the issue will occur, how to set TERM environment variables, could you give me a sample when you are free? Thanks  a lot

Can you please post your layout?
With the TERM var, you could check how it is set with echo $TERM. For alacritty, outside of zellij it should be set to alacritty-direct and in zellij to xterm-256color. At least this is how it is configured on my side for the demo video.
To set these variables, you need to configure them as environment variables in each of your programs.
This is for alacritty for your config file:
[env]
TERM = "alacritty-direct"

This for zellij for the config file:
env {
    TERM "xterm-256color"
}

Feel free to try it. Hopefully it will resolve the issue.

Thanks a lot. I did what you suggested. There is some improvement.

Is the issue fixed right now?

@dj95 Thanks for your great support


ASN.1 grammar parse error (token "id-SNSSAI"): syntax error, , unexpected TOK_identifier, expecting '}' ETSI TS 138 463 V15.3.0

I'm facing the above issue while compiling asn files for E1AP.  Spec details : ETSI TS 138 463 V15.3.0.
Kindly help to solve this problem.

Hi Appullz,
Could you provide the asn1 file?
Which asn1c version do you use?
Could you try with vlm_master branch from mouse07410's fork?

These are the asn1 files that I'm using : E1AP.zip
I initially used v0.9.28 and got the above mentioned error. So again tried with v0.9.29, I'm getting a different problem .
issueasnv29.txt. I guess the above problem is not coming when I use v0.9.29.
Sure I will try and let you know.

1. These are the asn1 files that I'm using : [E1AP.zip](https://github.com/vlm/asn1c/files/3426927/E1AP.zip)

2. I initially used v0.9.28 and got the above mentioned error. So again tried with v0.9.29, I'm getting a different problem .
   [issueasnv29.txt](https://github.com/vlm/asn1c/files/3426995/issueasnv29.txt). I guess the above problem is not coming when I use v0.9.29.

I've compiled it successfully with the latest master 88ed3b5cf012918bc1084b606b0624c45e0d2191 from this repo.
$ asn1c -fcompound-names ./E1AP/ASN1/E1apComDef.asn1 ./E1AP/ASN1/E1apConsDef.asn1 ./E1AP/ASN1/E1apContainerDef.asn1 ./E1AP/ASN1/E1apInfoElem.asn1 ./E1AP/ASN1/E1apPduDef.asn1
$ make -f converter-example.mk
$ ./converter-example -p list

Note that the order of the ASN.1 files is important.

3. Sure I will try and let you know.

As E1AP uses Aligned PER encoding you'll have to use the mouse 's fork as currently this repo does not contains support for Aligned PER.

3GPP TS 38.463 version 15.3.0 Release 15
9.5 Message Transfer Syntax
E1AP shall use the ASN.1 Basic Packed Encoding Rules (BASIC-PER) Aligned Variant as transfer syntax, as specified in ITU-T Recommendation X.691 [7].

Do you have some pcap files or binaries of E1AP messages that you could share?

I started working on E1AP couple of days back. I will post in the same link whenever I get some updates regarding this. Thank you so much for your help.

Hi, I have a query, when I compile E1AP using above mentioned command,  ProtocolIE-Container.h is not generated properly,
It has only one structure showing:
typedef struct ProtocolIE_Container_85P0 {
A_SEQUENCE_OF(struct ProtocolIE_Field) list;
    /* Context for parsing across buffer boundaries */
    asn_struct_ctx_t _asn_ctx;

} ProtocolIE_Container_85P0_t;
I'm using  https://github.com/vlm/asn1c/commit/88ed3b5cf012918bc1084b606b0624c45e0d2191(same as mentioned above ) branch for compiling it.
These are some warnings I'm getting while compiling:
WARNING: Parameterized type maxProtocolIEs expected for maxProtocolIEs at line 122 in ./E1AP-Containers.asn
WARNING: Parameterized type E1AP-PROTOCOL-IES expected for E1AP-PROTOCOL-IES
WARNING: Parameterized type maxProtocolExtensions expected for maxProtocolExtensions at line 159 in ./E1AP-Containers.asn
WARNING: Parameterized type E1AP-PROTOCOL-EXTENSION expected for E1AP-PROTOCOL-EXTENSION at line 163 in ./E1AP-Containers.asn
I don't know if I'm following the wrong way to compile it, Kindly help to solve this issue.

Hi @Appuliz,

when I compile E1AP using above mentioned command, ProtocolIE-Container.h is not generated properly,
It has only one structure showing:
typedef struct ProtocolIE_Container_85P0 {
A_SEQUENCE_OF(struct ProtocolIE_Field) list;
    /* Context for parsing across buffer boundaries */
    asn_struct_ctx_t _asn_ctx;

} ProtocolIE_Container_85P0_t;

Yes, this seems like a bug.

I'm using 88ed3b5 (same as mentioned above ) branch for compiling it.

Have you tried the vlm_master branch from mouse07410's fork?
As I've mentioned you are going to need APER encoding so it doesn't really matter where it works with current master or not.

These are some warnings I'm getting while compiling:
WARNING: Parameterized type maxProtocolIEs expected for maxProtocolIEs at line 122 in ./E1AP-Containers.asn
WARNING: Parameterized type E1AP-PROTOCOL-IES expected for E1AP-PROTOCOL-IES
WARNING: Parameterized type maxProtocolExtensions expected for maxProtocolExtensions at line 159 in ./E1AP-Containers.asn
WARNING: Parameterized type E1AP-PROTOCOL-EXTENSION expected for E1AP-PROTOCOL-EXTENSION at line 163 in ./E1AP-Containers.asn

I don't know what these warnings are. I just ignore them and everything seems to be working.

Hi,
I tried with above mentioned of ASN1C : https://github.com/mouse07410/asn1c
It worked. Thank you so much for your guidelines .


Show/hide columns based on device width

[ ] bug report => search github for a similar issue or PR before submitting
[x] feature request
[ ] support request => Please do not submit support request here
Current behavior
It's not possible to hide elements on different devices, like tablet or mobile.
Expected behavior
It should be possible to show/hide columns depending on widht of the device.
Reproduction of the problem
What is the motivation / use case for changing the behavior?
On mobile view is not enough space for so many columns like on desktop.
Please tell us about your environment:

Table version: 0.7.0

Angular version: 2.4

Browser: [all]

Language: [all]

Thanks for the suggestion, however, this is a implementation detail. This can be accomplished now quite easily, given the number of configurations I don't want to embed this type of logic in our implementation. Take care!

@amcdnl I have been trying to do this with flexShow for example, but it doesn't work on column component. How would you suggest, to achieve this??

Use a tool to get the media query and then adjust as needed.

@amcdnl But there is no identification on specific column, is there a way how to add ID or something? To column header and then to each column row...

@amcdnl Another thing is if you try to hide it just with css, you can't use flexwidth, it will just break the design. So you would need much more hacking. Are you sure, this shouldnt be there by design?

Why not use the prop attribute for identification?

I can't see anything unique in there
<datatable-header-cell draggable="" long-press="" resizeable="" ng-reflect-drag-model="[object Object]" ng-reflect-resize-enabled="true" ng-reflect-sort-type="single" ng-reflect-column="[object Object]" ng-reflect-sort-ascending-icon="icon-down" ng-reflect-sort-descending-icon="icon-up" ng-reflect-header-height="50px" ng-reflect-sorts="" class="datatable-header-cell sortable resizeable" ng-reflect-class-name="datatable-header-cell sortable resizeable" title="Date" style="width: 276.667px; left: auto;">
</datatable-header-cell>

not even in row column
<datatable-body-cell class="datatable-body-cell sort-active" tabindex="-1" ng-reflect-row="[object Object]" ng-reflect-column="[object Object]" ng-reflect-row-height="auto" ng-reflect-class-name="datatable-body-cell" style="width: 276.667px; height: auto;">
  </datatable-body-cell>

@amcdnl How would one use the prop attribute with a media query? Could you give an example please.

https://github.com/swimlane/ngx-datatable/issues/423#issuecomment-377068947


Add 'Contacts' Feature : Add Contacts

Renaming operations in some classes and methods from Person to Contact were necessary for the program to maintain a working state

Codecov Report

Merging #54 (acabe30) into master (3d1dcc4) will not change coverage.
The diff coverage is 83.67%.

@@            Coverage Diff            @@
##             master      #54   +/-   ##
=========================================
  Coverage     68.80%   68.80%
  Complexity      426      426
=========================================
  Files            76       76
  Lines          1359     1359
  Branches        131      131
=========================================
  Hits            935      935
  Misses          386      386
  Partials         38       38

Impacted Files
Coverage Δ
Complexity Δ

...main/java/seedu/dictionote/logic/LogicManager.java
75.00% <ø> (ø)
3.00 <0.00> (ø)

...a/seedu/dictionote/logic/commands/FindCommand.java
100.00% <ø> (ø)
6.00 <0.00> (ø)

...edu/dictionote/logic/parser/FindCommandParser.java
100.00% <ø> (ø)
3.00 <0.00> (ø)

...java/seedu/dictionote/logic/parser/ParserUtil.java
97.56% <ø> (ø)
16.00 <0.00> (ø)

...n/java/seedu/dictionote/model/contact/Address.java
80.00% <ø> (ø)
6.00 <0.00> (?)

...ain/java/seedu/dictionote/model/contact/Email.java
80.00% <ø> (ø)
6.00 <0.00> (?)

...main/java/seedu/dictionote/model/contact/Name.java
80.00% <ø> (ø)
6.00 <0.00> (?)

...ain/java/seedu/dictionote/model/contact/Phone.java
80.00% <ø> (ø)
6.00 <0.00> (?)

.../contact/exceptions/DuplicateContactException.java
100.00% <ø> (ø)
1.00 <0.00> (?)

...va/seedu/dictionote/model/util/SampleDataUtil.java
20.00% <0.00%> (ø)
1.00 <0.00> (ø)

... and 17 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 3d1dcc4...acabe30. Read the comment docs.

I notice some class may need to be renamed:

PersonUtil -> ContactUtil?
PersonListPanel -> ContactListPanel?
PersonCard -> ContactCard?
JsonAdaptedPerson -> JsonAdaptedContact?

otherwise, LGTM

Right, but those were left unchanged because other Contact commands rely on them.
I will rename them when I update the other Contact commands 👍 .

LGTM


WIP: trying to fix more flaky tests..

Signed-off-by: Miquel Sabaté Solà<EMAIL_ADDRESS>

On average I guess the tests are way faster but I'm still not sure about the stability of rebooting expectation and click_when_enabled helper.
I agree on reverting the click when enabled change of but not the rebooting text.

@vitoravelino I think this PR is fine as it is now. It's way faster, and unit tests passed. I'll trigger more rebuilds, but I think it should good to go. We can improve tests in another PR if you want, but the top priority was to fix the flaky tests...

I re-triggered a few times yesterday, the first two had 1 failure and the other 3 passed. I've just re-triggered. It's not so flaky, eventually with 1 or 2 re-triggers pass. If someone feels annoyed we can get back to this.

After I added the wait_until, duration increased a lot. I ran a few times and not sure if it's just a coincidence. :(

@mssola any other suggestion? If not, feel free to squash everything.

This approach is taking us nowhere. We'll start re-visiting how we mock the backend, since maybe it's stuck on some heavy-lifting from that side...

Rebased on top of master to gain all the changes on the CI

Pushed the branch into the upstream repository so it can be picked up by another PR.


feat: Add call depth to vm.stopAndReturnStateDiff() results

Motivation
Closes #6632
Solution
The call depth was already recorded in the cheatcodes::inspector::AccountAccess wrapper. This PR removes that wrapper type in favor of only using cheatcodes::vm::AccountAccess, which now has a depth field.
Credit to ercembu in #6667 for the initial implementation. I decided to pick this up since it's important to us at Sphinx, and that PR appears to be abandoned. I recommend closing that once this is merged.

FYI I've edited the description to use GitHub keywords when referencing issues.

this requires a forge-std change as well, right?

Yeah, I believe so. Is there a contributor guide I can reference for that?

There is no need, this will get included in the next release automatically.


Is there any option to pretty print lists and dicts when they are evaluated as expression?

I'm curious is it possible to integrate pprint…

For the 'normal' python REPL I have overwritten sys.displayhook to do this but ptpython doesn't seem to use that to print.

I'd love to see integration with tommikaikkonen/prettyprinter.

Integrating a 3rd party pretty printer like prettyprinter would need some kind of configurable hook to replace the default logic to print the evaluated value. Looks like the printing happens here, but it's only one part of a bigger method:
https://github.com/prompt-toolkit/ptpython/blob/b1bba26a491324cd65e0ef46c7b818c4b88fd993/ptpython/repl.py#L130-L159
The only thing you could monkeypatch is the whole surrounding _execute method, but that's difficult since it handles much more than just printing the output.


Add TinyGitHub as project (#25)

iOS App Project with github-trending-api

Thanks @k-lpmg

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Java] Question about new driver

What would you like help with?
I'm thinking of implementing an adbc driver for AWS Athena. And our codebase is in java. It looks like there is no driver in java implemented yet. So is it more preferred to implement it in other language like C++ or Go and use it in Java?

For the time being, Java can only use drivers written in Java unfortunately.
(Also, it appears Athena is row-oriented, so I'm not sure you could expect much speedup?)

That said, a driver would be welcome, whether in Java or not (I still would like to add the JNI bridge but just don't have time)

Athena can use parquet as storage format, so it should work with arrow column format? I'll look deeper into it see if that could work. I'm not familiar with JNI, but intested in learning it as well. :)


Hide opcode bytes in listing?

Is there an option/way to disable showing the opcode bytes in the listing view?
I've been able to use a temporary workaround/solution to visually hide them, but it's not great:

Set the Listing Fields -> Bytes Field -> Maximum Lines To Display option to 1 to get rid of the extra newlines in the listing view.
Set the Listing Display -> Bytes color to the same as the background to visually hide the bytes.

Check out this button on the top right of the listing window. You can do all of that over there...

@yakamoz147 Ah, perfect. Thank you!


[CRITICAL] [UX Reliability] Expensify DM showing in focus mode in LHN despite not having any new messages

Problem
In focus mode, only pinned chats, drafts, GBR, unread chats should be showing in LHN. There is now also an Expensify onboarding chat that is not unread, has not new messages, no GBR or RBR in it. Here is the onyx data.
{
    "reportID": "2440351539034289",
    "reportName": "Chat Report",
    "type": "chat",
    "chatType": "system",
    "ownerAccountID": 0,
    "managerID": 0,
    "policyID": "_FAKE_",
    "participants": {
        "778531": {
            "hidden": false
        },
        "11665625": {
            "hidden": false
        }
    },
    "isPinned": false,
    "lastReadTime": "2024-04-26 14:56:02.997",
    "lastReadSequenceNumber": 0,
    "lastVisibleActionCreated": "2024-04-26 14:56:02.997",
    "lastVisibleActionLastModified": "2024-04-26 14:56:02.997",
    "lastMessageText": "",
    "lastActionType": "CREATED",
    "lastActorAccountID": "",
    "notificationPreference": "daily",
    "stateNum": 0,
    "statusNum": 0,
    "oldPolicyName": "",
    "isOwnPolicyExpenseChat": false,
    "lastMessageHtml": "",
    "hasOutstandingChildRequest": false,
    "hasOutstandingChildTask": false,
    "policyName": "",
    "writeCapability": "all",
    "description": "",
    "total": 0,
    "unheldTotal": 0,
    "currency": "USD",
    "isWaitingOnBankAccount": false,
    "nonReimbursableTotal": 0,
    "isCancelledIOU": false,
    "permissions": [
        "read"
    ],
    "policyAvatar": "",
    "welcomeMessage": "",
    "errorFields": {},
    "participantAccountIDs": [
        11665625
    ],
    "visibleChatMemberAccountIDs": [
        11665625
    ]
}

Why is it showing?
Solution
Investigate why the Expensify DM chat is showing in focus mode based on the report data above.

Issue OwnerCurrent Issue Owner: @situchan

Ah got it! Closing!


Versioned bundles support

added support for versioned bundels loading via gulp-html-replace
updated markup task to include gulp-html-replace
configuration is specified as htmlreplace object in config.js

@greypants : any interest in this feature? it will allow you to build versioned bundles like app-1.23.js and include that version in the html page.

Cool! Yeah, I'll look this over when I get a chance. Thanks.

Code looks great, but I don't know that versioned filenames is something I want to include here. Instead, I'd rather append md5 hashes that change anytime the file has changed, so you can cache files forever. I'm in the middle of a big refactor have this handled there. (check out that branch and run NODE_ENV=production gulp)
Good notes additions to .gitignore I'll add those and a bunch more in the 2.0 branch.


[Automated] Update eventing-kafka-broker-eventing nightly

/assign @knative-extensions/eventing-wg-leads
/cc @knative-extensions/eventing-wg-leads
Produced via:
for x in eventing-crds.yaml eventing-core.yaml eventing-tls-networking.yaml; do
  curl https://storage.googleapis.com/knative-nightly/eventing/latest/$x > ${GITHUB_WORKSPACE}/./third_party/eventing-latest/$x
done

Details:
bumping knative.dev/eventing f81f1d3...734c413:
  > 734c413 fix: install script picks up on platform architecture by default (# 7821)
  > ebb311d Remove eventing-kafka from downstream tests as it's deprecated (# 7826)
  > ce5b77f [main] Update community files (# 7828)
  > 1920235 CrossNamespace: updated namespace for subscription (# 7804)
  > 366afed ETs reference subscriptions/triggers on reply (# 7733)
  > 8069d71 [main] Upgrade to latest dependencies (# 7824)
  > 4cec554 [main] Upgrade to latest dependencies (# 7818)
  > 7e1c082 [main] Format Go code (# 7819)
  > e581d36 Update eventtype v1beta3 type to reflect spec changes (# 7708)
  > 93e0c85 CrossNamespace: added brokerRef in trigger spec (# 7815)
  > 4754325 [main] Update community files (# 7817)
  > ff32fbe Update DEVELOPMENT.md (# 7790)
  > 018bf31 [main] Upgrade to latest dependencies (# 7811)
  > 2aa8985 Adding broker namespace for triggers (# 7798)
  > 292e29c feat: add triggers to event lineage graph (# 7746)
  > 74803f8 [main] Upgrade to latest dependencies (# 7800)
  > 63068bd [main] Upgrade to latest dependencies (# 7794)
  > acc6385 [main] Update community files (# 7787)
  > c7db5cb Reduce the scope for the Config validation webhook to only the `knative-eventing` namespace. (# 7792)
  > 0604334 Replace all usages of deprecated wait.PollImmediate with wait.PollUntilContextTimeout (# 7786)
  > b644dad [main] Upgrade to latest dependencies (# 7785)
  > bbb9051 Fusion Hack  (# 7740)
  > e886832 [main] Upgrade to latest dependencies (# 7781)
  > 8a18b16 CrossNamespace: adding feature flag (# 7752)
  > 6f944bf Bump github.com/go-jose/go-jose/v3 from 3.0.1 to 3.0.3 (# 7750)

/tide

/retest-required

/retest-required

/retest-required

/test reconciler-tests

/retest-required

/test reconciler-tests

/retest-required

/retest-required

https://github.com/knative-extensions/eventing-kafka-broker/pull/3845 should help with some of the test failures

/retest-required

/test integration-tests

/retest-required

/retest-required

/test reconciler-tests-namespaced-broker

/test reconciler-tests

Seems upgrade tests are failing, because we're missing something like https://github.com/knative-extensions/eventing-kafka-broker/pull/3845 in 1.13
(at least locally I see in the eventing controller API Warning: unknown field \"spec.subscribers[0].name\")
Installed kafkachannel CRD during the up-/downgrade is from 2792efb540c7de72d5330b3191ee1a4a385d07fa (1.13 branch):
apiVersion: apiextensions.k8s.io/v1
kind: CustomResourceDefinition
metadata:
  ...
  labels:
    app.kubernetes.io/version: 2792efb540c7de72d5330b3191ee1a4a385d07fa
    ...
  name: kafkachannels.messaging.knative.dev

/test reconciler-tests-namespaced-broker

/retest-required

/retest-required

/test reconciler-tests


Option to remove fancy zone bar from top of screen

Description of the new feature / enhancement
Provide option to remove fancy zone bar from top of screen.
Scenario when this would be used?

It's annoying that it shows up.
Trying to snap windows to the top of the screen for full screen is now weird.
Aligning a window with another one for various reasons is now impossible, because the bar gets in the way.

Supporting information

Hi Jay,
Thanks for the quick reply. I assumed it was Fancy Zones because it looks and acts so much like Fancy Zones. Turns out it's a Windows 11 Dev Beta feature they quietly added into the windows snap settings. Hovering over that bar gives you a set of zones to snap to.
Sorry for the confusion,
Charles


Support for popupwindow in few websites

Some website have Login with facebook/twitter option where javascript create a popup window and handles the request. Is it possible to add support for such popup windows?

We figured this out by using WKWebView instead of UIWebView and wrote the whole controller from scratch. I will try to create a controller using WKWebView and send a PR or create a new repo.


Error:CUDA out of memory

This project is so cool and I want to run it on my PC, I downloaded  the repo,configured and the  error(CUDA out of memory) occurred.I change max_m_strokes ，max_divide two  parameters, it works correctly, but the outpu is not meet my demand. I know nothing about the torch framework, can you help me to solve this problem, please ?

Thanks for your valuable feedback. Actually what you concerned about is exactly what I am working on these days. In the near further (hopefully next week), I will be adding a lightweight rendering model so that people with limited GPU memory can run smoothly on it and get high-resolution results. I will let you know when this feature is ready to use.

Hi @WShihan, a few updates have been made that I believe would be useful for you:

We have provided some lightweight renderers where users now can easily generate high-resolution paintings with much more stroke details. With the lightweight renders, the rendering speed also improves a lot (x3 faster). This update also solves the out-of-memory problem when running our demo on a GPU card with limited memory (e.g. 4GB).
Animated results are available at the output_dir (*_animated.mp4)
Please check out more details in the README file.

Thank you so much,

I'm closing this issue. Please feel free to leave your comment here if you have any future questions. Thanks!


Add druid-rocketmq module

This pull request adds support for RocketMQ, a Kafka-like messaging system as data ingesting source.
License Header Added

@lizhanhui Thanks for this contrib! Just a quick unrelated question, we've been trying to fix transient unit test failures in Druid and I noticed there is a failure in your PR. This is PR based from the absolute latest master?

Looks some new commits have been merged into master branch since my last
rebase.
I have rebased my PR.
Be there any issue, do not hesitate to post me back.
On Fri, Jan 8, 2016 at 1:48 AM, Fangjin Yang<EMAIL_ADDRESS>wrote:

@lizhanhui https://github.com/lizhanhui Thanks for this contrib! Just a
quick unrelated question, we've been trying to fix transient unit test
failures in Druid and I noticed there is a failure in your PR. This is PR
based from the absolute latest master?
—
Reply to this email directly or view it on GitHub
https://github.com/druid-io/druid/pull/2226#issuecomment-169754084.


Fixing FindBugs warnings from PR #549

Deleting a bunch of duplicate code from OutputStreamReporter.Builder that also exists in MetricReportReporter.Builder

@ibuenros can you review?

LGTM


dlrs__LookupRollupSummaryScheduleItems__c are being created for "Calculation Mode = Realtime" DLRS's on Account.

Describe the bug
We have 5 DLRS rollups with a Calculation Mode of "Realtime" that are generating dlrs__LookupRollupSummaryScheduleItems__c records for some reason.
The dlrs__LookupRollupSummaryScheduleItems__c records in question were all created today. None of the affected DLRS configurations were last updated more recently than a week ago. All of them have a calculation mode of "Realtime" and a "Parent Object" of Account, but have differing Child Objects. The triggers are all generated and do not have any additions/changes.
To Reproduce
Note: We were unable to replicate this behavior in a full sandbox environment, and are unsure why this occurring on some but not other rollups.

Update a child record of Account to trigger one of the 5 affected DLRS calculations.
Query the dlrs__LookupRollupSummaryScheduleItems__c table and observe that a schedule item was incorrectly created, and is related to one of the 5 affected DLRS configurations.
Note: the Account is still updated in real time.

Expected behavior
No dlrs__LookupRollupSummaryScheduleItems__c should be generated when the Calculation Mode is "Realtime"
Screenshots

This sounds like a duplicate of #1300, please review my notes at the end of that issue as to where the code might create scheduled items as a result of record merging.

@aheber Ah, so it's currently undocumented. I had heard that merging might be an issue but I couldn't find any evidence of that in the official docs, thanks for the confirmation. I double checked and all of the accounts related to scheduled items were merged, so that does indeed appear to be the issue. Thank you!

I've added a ticket to get it added to the docs. Hopefully the next person to run into it will be able to get an easier answer.


Bump framework version

Purpose

$subject

Related PRs:

https://github.com/wso2/carbon-identity-framework/pull/6179
https://github.com/wso2/carbon-identity-framework/pull/6180

PR builder ran for the following PR.

https://github.com/wso2/carbon-identity-framework/pull/6179

For the other PR, the changes are not testable in the PR builder

Refer https://github.com/wso2/carbon-identity-framework/pull/6180#issuecomment-2519424281

Hence not need to run the PR- builder for this PR


Run shell integration on pty host and restore on reconnect

Fixes #142669

High level changes:

Capabilities and ShellIntegrationAddon now live in platform so they can be used by the pty host.
Commands and now serialized by the pty host via ShellIntegrationAddon/CommandDetectionCapability and restored by the client after replay/attach.
PartialCommandDetectionCapability now uses onData instead of onKey so that it works on xterm-headless too, they should be functionally equivalent.

TODO:

[ ] Ensure the command string is serialized -> restored
[ ] Fix getOutput/hasOutput

@meganrogge ready for review, excluding those 2 TODOs remaining.

After reload, I see the decorations are restored, but:

click the button and Re-run command doesn't do anything
the copy output action isn't there
they're not stored in the list for this session (maybe by design?)


Show descriptions for admins

Coverage: 93.601%. Remained the same when pulling d4ce2cb320fb985b90376ab2072a77b79463a4ad on nimmo-name-desc-admin into 922e849b96a2f0c786589e71541380f83ca9efa5 on main.


Regression in synthesis for synchronous memory (ice40)

Steps to reproduce the issue
Attached a .zip that includes the input verilog file (stencil.v), the input .pcf for arachne-pnr, and the output .blif from yosys. Also includes a Makefile with the relevant yosys and arachne-pnr command.
stencil.zip

arachne-pnr version
❯ arachne-pnr -v
arachne-pnr 0.1+207+ 0 (git sha1 1e81432, c++ 9.0.0 -O2)

Checkout and install yosys 4f31cb6d
❯ git checkout 61f681162754fa170494e567f1a1a9ae2d17eb69
...
HEAD is now at 61f68116 Yosys 0.7

NOTE: Because abc has moved to github and removed the makefile from the default clone of their repository, I had to make the following patch to the makefile, otherwise it fails when trying to call make clean
❯ git diff
diff --git a/Makefile b/Makefile
index 0a61fe65..d11a0a65 100644
--- a/Makefile
+++ b/Makefile
@@ -393,7 +393,7 @@ ifneq ($(ABCREV),default)
                test $(ABCPULL) -ne 0 || { echo 'REEBE: NOP abg hc gb qngr naq NOPCHYY frg gb 0 va Znxrsvyr!' | tr 'A-Za-z' 'N-ZA-Mn-za-m'; exit 1; }; \
                echo "Pulling ABC from $(ABCURL):"; set -x; \
                test -d abc || hg clone $(ABCURL) abc; \
-               cd abc && $(MAKE) DEP= clean && hg pull && hg update -r $(ABCREV); \
+               cd abc && hg pull && hg update -r $(ABCREV); \
        fi
 endif
        $(Q) rm -f abc/abc-[0-9a-f]*

❯ make -j
❯ sudo make install

run make with files in attached .zip (icepack command running not required, should just show arachne-pnr not failing. Here is the output .blif file: blif.zip
❯ wget https://github.com/YosysHQ/yosys/files/2215703/stencil.zip
❯ unzip stencil.zip
❯ cd stencil
❯ make
yosys -q -p 'synth_ice40 -top main -blif stencil.blif' stencil.v
Warning: Yosys has only limited support for tri-state logic at the moment. (stencil.v:735)
arachne-pnr -q -d 1k -o stencil.txt -p stencil.pcf stencil.blif
icepack stencil.txt stencil.bin

Checkout and install yosys master
❯ cd ..
❯ git checkout Makefile  # revert Makefile patch
❯ rm -r abc  # so it clones the git version
❯ git checkout master
❯ git show
commit 323f6f6f6006eadcaec180f2cc1556f1f3303be3 (HEAD -> master, origin/master, origin/HEAD)
Merge: 1de07eee 68b5d0c3
Author: Clifford Wolf<EMAIL_ADDRESS>Date:   Fri Jul 20 19:22:59 2018 +0200

    Merge pull request #586 from hzeller/more-sourcepos-logging

    Convert more log_error() to log_file_error() where possible.
❯ make -j
❯ sudo make install

run make with files in attached .zip
wget https://github.com/YosysHQ/yosys/files/2215703/stencil.zip
❯ unzip stencil.zip
❯ cd stencil
❯ make
yosys -q -p 'synth_ice40 -top main -blif stencil.blif' stencil.v
Warning: Yosys has only limited support for tri-state logic at the moment. (stencil.v:735)
arachne-pnr -q -d 1k -o stencil.txt -p stencil.pcf stencil.blif
stencil.blif:1138: fatal error: unknown model `$mem'
make: *** [stencil.bin] Error 1

Here's the blif file that it generates: blif.zip

Expected behavior
An old version of yosys was able to correctly compile the coreir_mem module for the ice40 target to a .blif file that arachne-pnr could place and route.
Actual behavior
The latest version of yosys master produces a $mem cell which arachne-pnr cannot place.
The verilog module of interest is copied below
module coreir_mem #(parameter depth=1, parameter has_init=1, parameter width=1) (
  input clk,
  input [width-1:0] wdata,
  input [$clog2(depth)-1:0] waddr,
  input wen,
  output reg [width-1:0] rdata,
  input [$clog2(depth)-1:0] raddr
);
  reg [width-1:0] data[depth-1:0];
  always @(posedge clk) begin
    if (wen) begin
      data[waddr] <= wdata;
    end
    rdata <= data[raddr];
  end

endmodule //coreir_mem

It seems that when I make a smaller design using the memory, such as
module coreir_mem #(parameter depth=1, parameter has_init=1, parameter width=1) (
  input clk,
  input [width-1:0] wdata,
  input [$clog2(depth)-1:0] waddr,
  input wen,
  output reg [width-1:0] rdata,
  input [$clog2(depth)-1:0] raddr
);
  reg [width-1:0] data[depth-1:0];
  always @(posedge clk) begin
    if (wen) begin
      data[waddr] <= wdata;
    end
    rdata <= data[raddr];
  end

endmodule //coreir_mem

module main (output [4:0] J3, input  CLKIN);
  wire [7:0] mem__rdata;
  coreir_mem #(.depth(16),.has_init(0),.width(8)) mem(
    .clk(CLKIN),
    .raddr(0),
    .rdata(mem__rdata),
    .waddr(0),
    .wdata(mem__rdata),
    .wen(1)
  );
  assign J3 = mem__rdata[4:0];
endmodule

yosys and arachne can compile it. Here is the input verilog and output blif for the above small example (also includes .pcf and makefile) small.zip

As far as I can tell, what is happening here is that the memory map process is failing to correctly identify the write port, thus producing a memory with an asynchronous write port that cannot be mapped in any sensible way.
Clearly the memory is being written in a synchronous way however. Running this script, which is the synthesis process until anything memory-related starts:
read_verilog -lib  +/ice40/cells_sim.v
read_verilog stencil.v
hierarchy -top main
proc
flatten
opt_expr
opt_clean
opt
wreduce
alumacc
share
opt
fsm
opt -fast
write_ilang dump.ilang

Shows the following write port - at this point lacking a clock is OK because memory_dff hasn't been run yet:
  cell $memwr $techmap\inst49.inst0.lbmem_1_0.mem.$memwr$\data$stencil.v:750$74
    parameter \ABITS 4
    parameter \CLK_ENABLE 0
    parameter \CLK_POLARITY 0
    parameter \MEMID "\\inst49.inst0.lbmem_1_0.mem.data"
    parameter \PRIORITY 74
    parameter \WIDTH 8
    connect \ADDR $techmap\inst49.inst0.lbmem_1_0.mem.$memwr$\data$stencil.v:750$68_ADDR
    connect \CLK 1'x
    connect \DATA \inst49.inst0.lb1d_0.reg_1.outReg
    connect \EN 8'11111111
  end

Then running memory_dff, we get:
16. Executing MEMORY_DFF pass (merging $dff cells to $memrd and $memwr).
Checking cell `$techmap\inst49.inst0.lbmem_1_0.mem.$memwr$\data$stencil.v:750$74' in module `\main': no (compatible) $dff for data input found.
Checking cell `$techmap\inst49.inst0.lbmem_1_0.mem.$memrd$\data$stencil.v:752$73' in module `\main': merged data $dff to cell.

This shows there is a problem with the data input.
Looking at the data port, in the ilang dump, we see that although the net connect to the data input is correctly driven by a $dff, it also fans directly out to a number of LUT inputs which means that it can't be folded into the memory write port. I suspect this is an overly eager optimisation pass doing this.
I think this could be solved in two ways:

Preventing this optimisation happening when a net connects to a $memwr input
Duplicating the $dff in $memory_dff
Not sure what you think @cliffordwolf ?

Thanks for taking a look at this and the quick response! Tagging @adamdai, a fellow collaborator, so he  will get notification of the resolution.


Features/mayor/add pagination

Add pagination and refactor the code.
The repository is independent, the model uses the repository and graphql (outside the model), it only use the model, it does not have access directly to the repositories

Invalid branch name


Not working on ARM64

Here's the error log
 => [beammp_server internal] load build definition from Dockerfile                                     0.1s
 => => transferring dockerfile: 846B                                                                   0.0s
 => ERROR [beammp_server internal] load metadata for docker.io/library/archlinux:base                  0.6s
------
 > [beammp_server internal] load metadata for docker.io/library/archlinux:base:
------
failed to solve: archlinux:base: failed to resolve source metadata for docker.io/library/archlinux:base: no match for platform in manifest: not found

Could it be related to the Dockerfile?

It's looking like that to me since from the website I can see that archlinux only supports amd64.
Could you look into updating the Dockerfile to use an OS that supports more platforms?

Currently testing a modification of your script with debian instead of archlinux so that every platform is supported

I was able to make it run on Arm64 and it also reduced the size of your scripts

I didnt add arm support because i dont have arm devices to test run it on, but im curious about the pr (:

I will create a script tonight that will download the latest server version to automate the process a bit more

I decided to use https://github.com/RouHim/beammp-container-image instead so I will close this PR.


Got "No file to preview" - When try to open the file after download

if we provide timeout and open, it works normally

Hi! Can you post your snippet?


Support for ruby 3?

Using ruby 3.0.1, running gem install ethereum.rb is producing
ERROR:  While executing gem ... (NoMethodError) undefined method `request' for nil:NilClass`.

Seems like it might be related to digest-sha3's support for Ruby 3?
https://github.com/phusion/digest-sha3-ruby/blob/master/digest-sha3.gemspec#L12

I cloned the repo and swapped out the included digest-sha3 for this ruby-3 compatible version https://rubygems.org/gems/digest-sha3-patched-ruby-3 and all seems to be working well.  I can communicate with the blockchain and load my smart contracts from my truffle project without any issues.

add
gem 'digest-sha3', git: 'https://github.com/makevoid/digest-sha3-ruby', ref: '887b1051dd8d642ad9b16ea2ba075e3463051d83'

to your gemfile and it should work. YMMV.
This basically pegs your gem to a branch that has updated that ruby requirement to be any >=2 version.

add
gem 'digest-sha3', git: 'https://github.com/makevoid/digest-sha3-ruby', ref: '887b1051dd8d642ad9b16ea2ba075e3463051d83'

to your gemfile and it should work. YMMV.
This basically pegs this gem to a branch that has updated that ruby requirement to be any >=2 version, which then resolved any dependency issues from this gem (and the ruby-eth gem too)

Thanks, I can confirm this works perfectly.  Thanks for the information, much appreciated.

add
gem 'digest-sha3', git: 'https://github.com/makevoid/digest-sha3-ruby', ref: '887b1051dd8d642ad9b16ea2ba075e3463051d83'

to your gemfile and it should work. YMMV.
This basically pegs this gem to a branch that has updated that ruby requirement to be any >=2 version, which then resolved any dependency issues from this gem (and the ruby-eth gem too)

Works perfectly! Thanks for the info!!

I'm going to do Ruby 3 support. Please wait a little bit :)

I'm going to do Ruby 3 support. Please wait a little bit :)

It appears that the eth gem is no longer actively maintained. https://github.com/se3000/ruby-eth/issues/57
What do you think about swallowing the eth gem and add the functionality to ethereum.rb? I'm specifically asking about signing and recovering signatures.


Review: Memory Management

issues caught

At "managing it as smaller fixed-size" paragraph cuts to new line


Bug 2124558: Allow MigrationPolicy creation only to privileged user

📝 Description
Fixes:
https://bugzilla.redhat.com/show_bug.cgi?id=2124558
Allow MigrationPolicy creation only to privileged user by displaying Create MigrationPolicy button only for such users.
Screenshots
Before:
The unprivileged user can click on Create MigrationPolicy button:

After:
No any  Create MigrationPolicy button present for unprivileged user:

/bugzilla refresh

@avivtur @metalice @pcbailey @yzamir @vojtechszocs please review

Warning in advance: don't know precisely how roles are managed in general in the console
Shouldn't we use useAccessReview with verb: 'create' and/or ver: 'patch? Maybe there can be non-admin roles with creation and editing permission?

/retest

The code itself looks good. Have you asked anyone about Ugo's question about useAccessReview vs useIsAdmin? I'm curious about that as well. So far, we've been operating under the assumption that there are only cluster admins and non-admins/project admins. It would be nice to have some formal guidance that this is the case.
Also, shouldn't we provide a message when there are no migration policies telling the user that they don't have permissions to create them?

The code itself looks good. Have you asked anyone about Ugo's question about useAccessReview vs useIsAdmin? I'm curious about that as well. So far, we've been operating under the assumption that there are only cluster admins and non-admins/project admins. It would be nice to have some formal guidance that this is the case.

Not yet. Figuring out who's the best person to ask about this...

Also, shouldn't we provide a message when there are no migration policies telling the user that they don't have permissions to create them?

IMO that's the question for the UX. I'm not sure if there is already some design for this case. I will continue investigating.
Anyway, thanks very much for the review! ❇

Hi @hstastna
I got confused about something something,

Hi @hstastna
I must say that I'm confused, for some reason your "before" screenshots show the list of MigrationPolicies resources for a non-privileged user, when I get this error:

can you explain how that error does not occur for you? (this error is also it is shown in the BZ screenshot)

Hi @hstastna I must say that I'm confused, for some reason your "before" screenshots show the list of MigrationPolicies resources for a non-privileged user, when I get this error:
can you explain how that error does not occur for you? (this error is also it is shown in the BZ screenshot)

There was a bug, so I better moved the PR to WIP. I need to update it, to fix problems and to make sure about some implementation details. Thanks very much, Aviv, for pointing on this!

Hi @hstastna :)
I was fixing a similar bug for the DataSources page, I believe you can follow the same pattern here as well
please have a look: https://github.com/kubevirt-ui/kubevirt-plugin/pull/900

/retest

/lgtm

/approve


arm

Thank you, author. How can I use it on an arm64 device?

Is it possible for the author to consider adding support for it? Thank you!


bpnetlite.attributions.dinucleotide_shuffle errors out when there is less than 4 nucleotides in the sequence

from bpnetlite.attributions import dinucleotide_shuffle
from bpnetlite.io import one_hot_encode
import torch

dinucleotide_shuffle(torch.LongTensor(one_hot_encode('AGGTAGGT')))

gives
---------------------------------------------------------------------------
IndexError                                Traceback (most recent call last)
Cell In[360], line 5
      2 from bpnetlite.io import one_hot_encode
      3 import torch
----> 5 dinucleotide_shuffle(torch.LongTensor(one_hot_encode('AGGTAGGT')))

File ~/.miniconda3/lib/python3.10/site-packages/bpnetlite/attributions.py:191, in dinucleotide_shuffle(sequence, n_shuffles, random_state)
    188 	next_idxs_ = numpy.where(idxs[:-1] == char)[0]
    189 	n = len(next_idxs_)
--> 191 	next_idxs[char][:n] = next_idxs_ + 1
    192 	next_idxs_counts[char] = n
    194 shuffled_sequences = numpy.zeros((n_shuffles, *sequence.shape), dtype=numpy.float32)

IndexError: index 3 is out of bounds for axis 0 with size 3

the three there being "A", "G", and "T".

Arry Potter and the broken dinucshuf()

Looks like there were sort of two issues. The first issue is that my one_hot_encode command was returning a transposed version of what it should be. That has been fixed throughout the repo now. The second is that the dinucleotide shuffled sequence was making the shape of the returned sequence be based on the number of unique characters rather than just using the shape. So, that's been fixed too. v0.5.6 is up and should contain these fixes. Let me know if that works?

Works well, thanks!


Make it possible to write longer separator string

Current state:
Separator string is only 12 bytes long

Wanted state:
It would be great if program could handle more than 12 bytes or 12 ascii chars
Why the change is sensible:
Themes, customization.
Neofetch users are customizing it, like here, using unicode characters. Now fastfetch can handle only 3 unicode chars in separator string a time

The separator is currently always exactly the length of the title (username@host). The type of theming you wan't is currently not possible, but i am working on it.

The separator is currently always exactly the length of the title (username@host).
So I can modify username@host to be exact same length of separator string, it's an idea

I did some small changes, and now i am able to recreate the first example you linked, with following config file:
--separator "  "
--set-keyless HardwareStart=┌───────── \e[1mHardware Information\e[0m ─────────┐
--set-keyless SoftwareStart=├───────── \e[1mSoftware Information\e[0m ─────────┤
--set-keyless InformationEnd=└────────────────────────────────────────┘
--set-keyless Colors=    \e[38m \e[37m \e[36m \e[35m \e[34m \e[33m \e[32m \e[31m
--structure HardwareStart:Host:CPU:GPU:Memory:Resolution:SoftwareStart:OS:Kernel:WM:Shell:Terminal:TerminalFont:Packages:InformationEnd:Colors
--host-key "  "
--cpu-key "  "
--gpu-key "  ﬙"
--memory-key "  塞"
--resolution-key "  "
--os-key "  "
--kernel-key "  "
--wm-key "  "
--shell-key "  "
--terminal-key "  "
--terminal-font-key "  "
--packages-key "  "

Is this sufficient?

It's absolutely awesome, thanks for your job! Best fetch utility I've seen

If you have a cool theme feel free to open a pull request to add it to presets/. This way everybody can access it with a simple --load-config <your-unique-theme-name>.

When trying this i get
Error: unknown option: --set-keyless HardwareStart=┌───────── \e[1mHardware Information\e[0m ─────────┐


Can't get cell at row=..., col=...

Seeing a lot of this in insiders dev tools today

Haven't seen this lately


Create a SAP Fiori App and Deploy it to SAP BTP, ABAP Environment

Tutorials: https://developers.sap.com/tutorials/abap-environment-deploy-cf-production.html
In my opinion, in step 8, the task to assign the newly created Business Catalog (ZTRAVEL_BC_XXX) to an existing Business Role (SAP_BR_DEVELOPER for instance) is missing.
Then, in step 9, app testing should also be done through the Business Catalog Tile from Launchpad App finder.

Hi @phdaucourt ,
the assgingment takes place in this tutorial: https://developers.sap.com/tutorials/abap-environment-abap-flp.html#b73e28a5-4da3-49e0-b982-159411b14f82
Testing also takes places in this tutorial: https://developers.sap.com/tutorials/abap-environment-abap-flp.html#b73e28a5-4da3-49e0-b982-159411b14f82
Here is the complete group: https://developers.sap.com/group.abap-env-restful-managed.html
Thank you.
Best regards,
Merve


[Storage] Fix network-rule commands

Due to the breaking change storage introduced in the latest management SDK, these commands now fail. This updates the references to the new field names.

This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines

[X] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

Command Guidelines

[N/A] Each command and parameter has a meaningful description.
[N/A] Each new command has a test.

(see Authoring Command Modules)

Looks legit.


Support installing with setup.py

Module should be installable via a setup.py file using pip. This also makes testing easier as the module can be installed into the testing environment with pip install -e ..

Completed as of #0912904f6b8bcf729cb98fece7a1dcb7be63fc21.


Use dependencies instead of devDependencies for deep mode

First off, absolutely love this library. I’ve created many private node modules for work and I’ve always used either Babel cli directly or a shell script to perform the same thing Preppy does. This library is a far superior replacement; it’s like the webpack for node modules for me...
I was wondering why deep mode doesn’t use the dependencies instead of devDependencies. According to the package.json spec, dependencies are those that will be installed when installing your package into another package.  This also causes potential issues when you have (for instance) rimraf as a dev dependency, but you’re packing a client side package.
Maybe I’m missing something, but I’d like some more insight into this choice...
🙏

"dependencies" will be relevant when the package is installed by others later. Why I use "devDependencies" is because these are not relevant for the users of the package. Actually it would be helpful to have another list of packages dependencies (requirements of the library for the user), devDependencies (tooling for author), bundleDependencies (libraries to bundle). Unfortunately we do not have that... also: for my packages it worked pretty fine. It's not that Preppy is bundling all devDependencies but only the ones which are references by the code. Deep mode basically produces self-contained libraries with no or minimal dependency lists. Often times these only contain modules with native-code parts (Node-GYP powered, ...)

Closing this as no further feedback came along.


run it on HPC

FAILED
https://github.com/APSIMInitiative/ApsimX/issues/6383#issue-864618622

probably won't be solved in a short time


chore: release v1.12.0

Draft Release Notes:
Hey Lodestar users! We've just released v1.12.0 packed with a bunch of updates! We recommend all users to upgrade when possible and ensure to upgrade both your validator and beacon node clients together. More information below about notable changes:
produceBlockV2 builder/execution race moved from validator to beacon: You will have issues building blocks if you do not upgrade the Lodestar beacon and validator together to v1.12.0. You can continue using the --builder flag on your validator client, but it will default to --builder.selection maxprofit. If you do not want to use the builder on a validator that might still be hooked onto a builder enabled beacon, you may provide the --builder.selection executiononly or skip --builder. DVTs which are based off on only builder based block production can still continue using --builder.selection builderonly.
Ephemery Network: v1.12 supports using the --ephemery flag to configure Lodestar for the ephemery network. To see additional commands, please see the PR here: https://github.com/ChainSafe/lodestar/pull/6054
IDs for JWT claims: We've added --jwt-id to allow for auth + id of Lodestar being sent to eth1 execution clients. This allows for more flexibility than the 1-1 connection between the consensus and execution layer clients. It is now compliant with the execution spec identified here: https://github.com/ethereum/execution-apis/blob/main/src/engine/authentication.md#jwt-claims
Swagger UI on root endpoint of REST API: If you would like to enable a REST swagger API, you can now add --rest.swaggerUI to enable and it will be served under http://beaconnode:9596/documentation
Some additional support for keymanager-API: Added validator voluntary exit endpoint (https://github.com/ethereum/keymanager-APIs/blob/master/apis/voluntary_exit.yaml) and allow configuration per validator pubkey graffiti (https://github.com/ethereum/keymanager-APIs/pull/63)

:tada: This PR is included in v1.12.0 :tada:


Update post & comments features

updated to be backend features, and not frontend

the tests that didn't work have been turned into .txt files, chenge whenever you want


Proposal: standardize terragrunt hcl files to tg.hcl extension (for editor support)

I started working on a dedicated vim syntax plugin for terragrunt, and realized that it is not ideal to assign all hcl files to use the terragrunt syntax highlighting, since not all hcl files are terragrunt config files.
This is a proposal to standardize docs and examples to use the .tg.hcl extension instead of .hcl to indicate terragrunt config files (in addition to the base terragrunt.hcl file) so that we can have more dedicated editor plugins that operate on terragrunt configuration files, without compromising the ability to work with other hcl files that use the hcl extension (e.g., packer uses .pkr.hcl).

What's different about Terragrunt syntax highlighting than normal HCL?

We can highlight terragrunt keywords. See https://github.com/yorinasub17/vim-terragrunt/blob/main/syntax/terragrunt.vim#L29 and https://github.com/yorinasub17/vim-terragrunt/blob/main/syntax/terragrunt.vim#L22 .
This will also help when developing a terragrunt language server and adding autocompletion for the terragrunt structs.

And is file extension the only way these tools match? Or do they do a content scan too?
Seems like many file extensions (e.g., .yml) are used for all sorts of different tools...

File extension is one of the more convenient and efficient ways to do this, since this is what the editors are optimized for. Otherwise, we need to inject some sort of marker into the file (e.g., # vi:syntax=terragrunt would force the file to be parsed as terragrunt), or add some special logic to look for editor specific markers into the plugins.

Seems like many file extensions (e.g., .yml) are used for all sorts of different tools...

I don't think I ever come across language servers for yaml, or tools that autocomplete yaml, which is a weakness of using yaml as a config file as opposed to dedicated language constructs.

To be explicit, I am only proposing this as a convention to use for editor support. Nothing should change in terragrunt, and we should continue to support reading in and using plain .hcl extensions. In fact, all the blocks that read terragrunt config support any file name, so it doesn't even have to be called .hcl.

What about using the file name rather than extension? Match on terragrunt.hcl and terragrunt-*.hcl?
I'm just trying to avoid renaming the Terragrunt config file again. We had .terragrunt, terraform.tfvars, terragrunt.hcl, and now one more...

Oh I think you might be misunderstanding me. This proposal is specifically targetting the hcl files we are importing with read_terragrunt_config. We basically keep the terragrunt.hcl file and the other hcl files will use the .tg.hcl.

Ohhhhhh. Haha, OK, yea, that's fine. Might want to update title/description accordingly :)


root-signing-staging branhc protection changed

root-signing-staging has a setup where sigstore-bot can push to two branches main and publish.
Pushing to both worked May 23rd 10:19 GMT+3. Then on May 24th 15:26 GMT+3 pushing to main worked but pushing to publish failed with "protected branch hook declined".
So something clearly changed in the config. I can see that main and publish have slightly different configuration so this is likely an easy fix, I'm just a bit worried how the setup changed?
CC @kommendorkapten @haydentherapper

I'm just a bit worried how the setup changed?

To be clear: the yaml hasn't changed in months... but possibly the branch protection settings haven't been applied after that last change


Specify the order of schedules

This looks awesome! I haven't tried it out yet. One idea that I built into my tool was te ability to choose the order in which actions run.
I used an interface instead of cron jobs as people don't know how to use crons (I have to google it every time myself). I added a sort order field. Then in the batch finish it ran the next schedule. This was important to stop lock row errors and when running automation in a specific order

This will be easier to do in the next release I'm currently working on. Introducing a unique name on the configuration records that can be referenced via the invocable apex action to Run Mass Actions.
There will also be a new last run date/time field so that you can define Process Builders on the Mass_Action_Configuration__c object, monitor for that date/time to change, and that lets you know that one of the runs has completed, so use process builder to enqueue the next action. You can chain as many as you'd like. I'll write up a more formal documentation and how-to for the wiki once this goes live.

That's great.
We can also get row lock on things firing at the same time for the average user. That's why my version went through each schedule in sequence.
So there is never two batches running in parallel that start at the same time.


Macosx build support

build support for MacOS X using brew installed gcc-4.8 using the following commands:
brew tap homebrew/versions
brew install gcc@48
if blackbird crashes with the following command:
what():  locale::facet::_S_create_c_locale name not valid
try to change LC_ALL to C:
export LC_ALL="C"

What do we do with this PR? Thanks

Maybe have someone else with a MacOS X test it make sure the changes work?

I tested this on my Mac.  I followed the instructions in the readme, but when starting blackbird I ran into the error mentioned in the description of this PR.  After exporting LC_ALL="C" per the fix mentioned above, blackbird was able to start successfully.
I would considering adding to the readme, the section from the description in this PR about blackbird crashing and exporting LC_ALL="C".

@calveto you're right, I forgot to update the README, I'll do it asap


Correct way to coerce struct fields from strings to other types?

I have a struct like this:
defmodule StreamingBuffer do
  defstruct [:estimatedRows, :estimatedBytes, :oldestEntryTime]

  @type t :: %__MODULE__{
    estimatedRows: non_neg_integer,
    estimatedBytes: non_neg_integer,
    oldestEntryTime: non_neg_integer
  }
end

But the API that I'm calling returns each of those fields as strings, not as integers, and Poison decodes the struct with string fields:
x = "{\"estimatedRows\" => \"12\", \"estimatedBytes\" => \"1024\", \"oldestEntryTime\" => \"1\"}"
Poison.decode!(x, as: %StreamingBuffer{})

%StreamingBuffer{estimatedBytes: "1024", estimatedRows: "12",
 oldestEntryTime: "1"}

Is there a good way to tell Poison to decode those fields as integers? Or is the correct approach to write a Decoder implementation for each of my structs that have this issue?
As an aside, it seems a little weird to me that the value passed in to a custom decoder is an instance of the struct, as opposed to a plain map? I'm sure there are reasons why that I'm missing, but I'd love to have a little more insight into how & why the decodes work this way? I haven't done much with protocols in elixir yet.

This does not seem like a missing feature of Poison, in my opinion. Rather, the API you are calling is not following the conventions of JSON.
A simple (but maybe not the cleanest) solution you could do, is to iterate over the constructed fields, converting them to integers if (String.to_integer(field) |> Integer.to_string) == field. A similar thing can of course also be done with floats.

Yeah, at the time I wrote this issue I was less familiar with Poison than I am now, so I was hoping maybe I was missing something in the documentation that might cover how to do this.
API-wise, I think writing a decoder for this type and converting from Strings to the correct type for each field is the right way to go. I can't picture a way to implement this more generically in the Poison API without things getting fairly ugly.


Zend setRequestUrl function exist bug

setRequestUrl function like blow:

 390     public function setRequestUri($requestUri = null)
 391     {
 392         if ($requestUri === null) {
 393             if (isset($_SERVER['HTTP_X_ORIGINAL_URL'])) {
 394                 // IIS with Microsoft Rewrite Module
 395                 $requestUri = $_SERVER['HTTP_X_ORIGINAL_URL'];
 396             } elseif (isset($_SERVER['HTTP_X_REWRITE_URL'])) {
 397                 // IIS with ISAPI_Rewrite
 398                 $requestUri = $_SERVER['HTTP_X_REWRITE_URL'];

but  we allways recevie some mobile system request with empty HTTP_X_REWRITE_URL value. Just like this:

get http://xxxxx/v1/channel/click
Host: xxxxx
x-original-URL:
accept: */*

which case the requestUri is not the truth url (/v1/channel/click).  Please fix it.

This issue has been closed as part of the bug migration program as outlined here - http://framework.zend.com/blog/2016-04-11-issue-closures.html


typo in the source code?

https://github.com/developit/preact/blob/master/src%2Fpreact.js#L202
if (build.parentNode!==parent) {

build should be built ?

Wow , thanks for spotting this! Would be causing a small performance drop in some of the benchmarks, so this is key! I'll get a fix in right away.

@whwei Fix has just been released, version 2.8.3.


2.2.0

Added MANIFEST file - Pilot 2 is now registered with pypi which means it can be pip-installed without referencing github

pip install panda-pilot

Data component upgrade

Refactored and unified ES StagingClients
Automatically prefer LAN protocol (read_lan/write_lan) for stage-in/stage-out file if source/destination RSE is local for given PQ (defined in inputddms=astorages['read_lan'])
Base movers workflow upgraded
Introduced require_input_protocols mode to look up and manually form input replicas for specific copytool (activated for the objectstore mover, ES workflow)
Refactored and simplified the objectstore copytool
Implemented fail-over transfer for ES stage-out

Preparing for containerized middleware commands [minor update]
Added debug messages for potential problem with relying on SC_CLK_TCK

Local problem seen on MPPMU (https://bigpanda.cern.ch/job/4482660563/). Pilot cannot calculate CPU consumption without this value

Added LFN in diagnostics message for checksum errors. Corrected mislabelled checksum types (MD5SUM reported instead of ADLER32). Requested by R. Walker
Cleaned up stage-in/out error messages containing irrelevant Traceback info (should now be concise)
Following an update in the auto-setup script, the pilot is now using RUCIO_LOCAL_SITE_ID instead of the deprecated DQ2_LOCAL_SITE_ID for localsite in Rucio traces
Simplification of pilot arguments: now using resource name from queuedata instead of relying on pilot option -r  (which can now be removed from wrapper)
Instead of a traceback, now reporting the real error returned from rucio download or upload. However, the current version of rucio does not propagate errors well so the message will always be "None of the requested files have been downloaded". D. Cameron is working on fixing this so a future version of Rucio will report the real error
Changed minimum allowed local space from 5 GB to 2 GB (as verified during payload running); the higher limit affected event index jobs run at OU. Requested by H. Severini
Pilot is now always setting ATHENA_CORE_NUMBER (previously only set for event service jobs)
Updated memory leak calculation to be consistent with new prmon field names (changed PSS+Swap to pss+swap)
Added new error code 1352, “Failed to stat proc file for CPU consumption calculation” which is set when the pilot cannot access /prod/pid/stat. Requested by P. Svirin
Corrected the local/remoteSite sent with the traces - previously if the pilot overwrote the requested ddmendpoint (ie if the requested ddmendpoint was not allowed), then the trace was not updated as well. Now it is.

Code contributions from D. Cameron, A. Anisenkov, W. Guan, F. Barreiro, P. Nilsson

The travis build has passed but GitHub is not updating the status. Will close and retry the PR


os.getlogin() - OSError: [Errno 2] No such file or directory

[*] Starting module at 2019-02-13-17.09.09

Host: linux-mint-19-2
OS: Linux, #21-Ubuntu SMP Tue Apr 24 06:16:15 UTC 2018
Kernel: 4.15.0-20-generic
Traceback (most recent call last):
  File "./uptux.py", line 748, in <module>
    main()
  File "./uptux.py", line 741, in main
    check_handler(check, check_name, check_desc)
  File "./uptux.py", line 114, in check_handler
    check()
  File "./uptux.py", line 406, in uptux_check_sysinfo
    tee("Current user: {} (UID {} GID {})".format(os.getlogin(),
OSError: [Errno 2] No such file or directory

See: https://bugs.python.org/issue584566
Fix:
diff --git a/uptux.py b/uptux.py
index 70e39c4..41efa8d 100755
--- a/uptux.py
+++ b/uptux.py
@@ -12,6 +12,7 @@ in restricted shells is a pain.

 import os
+import pwd
 import sys
 import argparse
 import datetime
@@ -403,7 +404,7 @@ def uptux_check_sysinfo():
     tee("Host: {}".format(uname[1]))
     tee("OS: {}, {}".format(uname[0], uname[3]))
     tee("Kernel: {}".format(uname[2]))
-    tee("Current user: {} (UID {} GID {})".format(os.getlogin(),
+    tee("Current user: {} (UID {} GID {})".format(pwd.getpwuid(os.geteuid())[0],
                                                   os.getuid(),
                                                   os.getgid()))
     tee("Member of following groups:\n  {}".format(shell_exec('groups')))

Thank you very much for taking the time to submit this!
I will have a bit of time to review later this week. Apologies for the delay, just in the middle of some projects.
Thanks again for your help and patience!

Thanks again for pointing this out.
I see in the python3 docs here that they recommend using getpass.getlogin(), which first checks environment variables and then falls back to your recommended getpwuid:
getlogin()
Return the name of the user logged in on the controlling terminal of the process. For most purposes, it is more useful to use getpass.getuser() since the latter checks the environment variables LOGNAME or USERNAME to find out who the user is, and falls back to pwd.getpwuid(os.getuid())[0] to get the login name of the current real user id.
I just installed Mint 17 in a VM and tested, and this seems to work.
I think I am going to go with that method. Thanks again for bringing attention to this!

OK, not Mint prints further errors as it appears to not use systemd.
This whole tool is written for systemd, so Linux Mint would not be supported unless you have manually migrated it over.
Sounds like I need to write a check for systemd in use.

Yes, getpass.getuser() works.


Proper about page

[WIP: I need to replace square with put image sprite, but other parts are done.]
Closes #254 #273

Content feedback

of its neighboring squares

I think "of its neighboring 8 squares" makes it clear that corner squares count.
I think we need a bit on "flag mode" and why you might want to flag a square.
No one seems to realise that you can click on a number when it's blue. So maybe add a bit about that.
Document right-click behaviour. It does the opposite of the flag mode setting.
Love the debug info!
Design feedback
Do you think it makes sense to make "About" an extra click away? What if we just put the content under the animation setting? (just asked @surma, he agrees)

Ligatures look bad in this font. Add font-variant-ligatures: none; to the HTML element.

Text isn't centered vertically, because the bottom margin is on the image.

Layout breaks at smaller widths.


Please add me to the Kivy Garden

Link for a merged garden PR or a proposed new flower:
https://github.com/avour/garden.filemanager
https://github.com/avour/garden.reloadbehavior

Can you please structure the flower in the new format as described here and then request again? The old garden tool is being deprecated in favor of the new package structure so that it can be installed like any normal Python package without needing custom garden installers. With the new package format, it can be added to setup.py like any normal dependency.
Also, flowers should have screenshots of the flower  so people can see how it looks. Thanks


Field doesn't populate under tabs on option pages.

Hi,
I am using cmb2 on my option page with the vertical tabs using your snippets.
I picked the recent changes that cover compatibility with the option pages.
Data is getting saved but field data is not populating.
Could you provide a quick fix for this as I am in middle of my project and got stuck there if this functionality is under development?
Thanks.

HI,
I have noticed that recent changes in the code demand object_type attribute to be placed.
When I added it under args , it worked perfectly.
You can close this thread.
Thanks!


Update gradle to 6.6.1 + add distribution hash

F-Droid bot complains about:

missing distributionSha256Sum in gradle/wrapper/gradle-wrapper.properties
gradle vs wrapper version mismatch
see https://gitlab.com/fdroid/fdroiddata/-/merge_requests/12216#note_1204171878 for more info

I took the liberty and updated gradle to latest version of 6.6.x series.
I haven't tried the build - can we try running GitHub Action?

command used:
./gradlew wrapper \
  --gradle-version 6.6.1 \
  --gradle-distribution-sha256-sum 7873ed5287f47ca03549ab8dcb6dc877ac7f0e3d7b1eb12685161d10080910ac

hash obtained from:
https://gradle.org/release-checksums/

The build worked.


The "fp downloader" application crashes after starting the application

The "fp downloader" application can only be used after the initial restart after installing the Magisk module, after the next restart of the "fp downloader" application, it repeatedly crashes after starting the application. However, calling the "fp" command in the terminal still works. Is there a similar experience?

I am sorry. I messed up the thread. my issue is invalid.


WinPath.Updater returns %PROGRAMFILES(X86)%

On a x86_64 computer, WinPath.Updater returns %PROGRAMFILES(x86)%, this is a bug and a concern as Windows team is planning to allow the user to change the default hard disk -- as of writing this issue, this value is hard coded.
I looked into this issue and apparently it happens because of a 32-bit application being run on a 64-bit host environment. Please see #64 for more information.

There are several issues with your WinPath.Updater code:

You are hard-coding Program Files and Program Files (x86) folder names in your code. Are you aware that those names are localized in non-English versions of Windows?
Are you aware that writing to Program Files or Program Files (x86) requires elevation (administrator rights)? Are you even checking for that?
You seem to be wrong in your assessment that getting wrong path is a Win32 problem, I believe it is not and you are simply not using the right API. You should use Environment.GetFolderPath and pass ProgramFiles or ProgramFilesX86 as argument.
If your program is 32-bit you should never install it into Program Files on a 64-bit OS, because that is where 64-bit programs are supposed to be installed. It should go into Program Files on a 32-bit OS and into Program Files (x86) on a 64-bit OS. In your current code the if condition seems to be reversed.

Finally, why bother with writing a command line tool (with an updater no less!) when half the functionality already exists in the setx command?
Not to mention that you can download a GUI tool for that :-)

Hello @levicki, thanks for pointing out issues.

There are several issues with your WinPath.Updater code:

You are hard-coding Program Files and Program Files (x86) folder names in your code. Are you aware that those names are localized in non-English versions of Windows?

Yes, that's why it was a concern; that commented code was returning Program Files (x86) regardless of everything.

Are you aware that writing to Program Files or Program Files (x86) requires elevation (administrator rights)? Are you even checking for that?

Yep, originally, the code to start WinPath.Updater is from WinPath, and yes, it does start it as an administrator and handles everything. The code behind very very messy, but that's only because I plan to replace WinPath.Updater entirely with v0.4.0 (see #67 for more details).

You seem to be wrong in your assessment that getting wrong path is a Win32 problem, I believe it is not and you are simply not using the right API. You should use Environment.GetFolderPath and pass ProgramFiles or ProgramFilesX86 as argument.

That's actually the entire problem in the first place. Refer to this image (x64 host running both x86 and x64 binaries respectively):

(in reply to that comment on the other issue; Your problem has nothing to do with this particular issue which is about .Net implementation of environment variables) and when I said And there's really nothing we can do about it as far as I know, only Win32 API team can change this, I meant that there's nothing .NET can do about it, only Win32 API's team, since .NET calls Windows' API, it doesn't magically know it. And there's really no need for a new issue for this, just sharing some information 😄.

If your program is 32-bit you should never install it into Program Files on a 64-bit OS, because that is where 64-bit programs are supposed to be installed. It should go into Program Files on a 32-bit OS and into Program Files (x86) on a 64-bit OS. In your current code the if condition seems to be reversed.

Yea, I don't do that, I know that. The current structure is very confusing I myself should know that, the way it works is that once WinPath (stored in program files x86 on 32-bit hosts and program files on x64 hosts) starts updating, it downloads the new version and the updater. The updater is only 32-bit and arm, once downloaded, it starts WinPath.Updater.exe which then simply copies it to the right location, it's hard-coded, I know, but, that issue. But hey, that issue should be solved thanks to you now.

Finally, why bother with writing a command line tool (with an updater no less!) when half the functionality already exists in the setx command, not to mention that you can download a simple and nice GUI tool for that from my website? :-)

That's pretty much like saying "why make something when it already exists" my good man. But the main reason I wanted to do that is for

A good learning experience, it's a project that I truly find motivation for. I didn't even spend a complete whole year of programming yet.
This project originally started as a side-project for me -- personal use -- which I started because of that delete button from that environment variables window which is really scary and I don't feel great when messing with it.

On a foot note; can you share your gui application of yours, I would love to take a look at it.

Yes, that's why it was a concern; that commented code was returning Program Files (x86) regardless of everything.

Ok, but this issue of yours has to do with Environment.GetFolderPath function, and not with Environment.SetEnvironmentVariable function which was the issue you posted your problem to.
Environment.GetFolderPath is calling  SHGetFolderPathW from Win32 API which is deprecated, and is just a wrapper for SHGetKnownFolderPath.
In the documentation for KNOWNFOLDERID parameter, right next to FOLDERID_ProgramFiles it says "See Remarks for more information", and near the bottom of that quite lengthy page, there is a Remarks section which explains the behavior you are observing.

So, it turns out that 32-bit program cannot get path to 64-bit Program Files on 64-bit OS (and by "cannot" I only mean "cannot in a legitimate way" because I am sure it is possible in some other way). Basically all that boils down to "works as designed", "won't fix", etc, so I think it would be best if you removed your post from that other issue and I can then remove mine where I told you I will respond here.
You have two options:

Drop 32-bit support, and make everything 64-bit only.

OR

Have 32-bit updater for 32-bit app, and 64-bit updater for 64-bit app as it should be.

Any other option will just cause you more trouble.
The above was a comment from a developer perspective, what follows is from a user perspective.
As a computer user I very much dislike the idea of every program having its own update checking (be it on startup, as a scheduled task, or worse as an always running background service), not to mention its own hand-rolled installers and updaters which often do not respect system conventions and have no rollback functionality to restore the system to a known working state if something goes wrong.
If you want to distribute your programs, you either provide them as a simple .zip archive, or you use one of the myriad known good installers that provide full uninstall, rollback in case of failure, update of existing versions, silent installation and uninstallation, 32-bit and 64-bit OS differences, installation of pre-requisites, etc, such as InstallShield, WIX, MSI, or my favorite InnoSetup which is free to use.

That's pretty much like saying "why make something when it already exists" my good man.

I was confused by the need for another command line version of SETX, that's all :-)
As for my GUI version, you can download it from my website downloads section. It's called SmartPathEditor.

Yes, that's why it was a concern; that commented code was returning Program Files (x86) regardless of everything.

...
So, it turns out that 32-bit program cannot get path to 64-bit Program Files on 64-bit OS (and by "cannot" I only mean "cannot in a legitimate way" because I am sure it is possible in some other way). Basically all that boils down to "works as designed", "won't fix", etc, so I think it would be best if you removed your post from that other issue and I can then remove mine where I told you I will respond here.

That's exactly what I said over there, sure it was about Environment.SetEnvironmentVariable but was not limited to that. I think it should be pretty obvious that my side note also refers to other such variables and actions.

You have two options:

Drop 32-bit support, and make everything 64-bit only.

OR

Have 32-bit updater for 32-bit app, and 64-bit updater for 64-bit app as it should be if you want to support both platforms.

Any other option will just cause you more trouble.

Of course, of course, recall the issue I referred to you in my previous message. I'm going to deprecate support (possibly keep backwards compatibility for a bit) for WinPath.Updater. I'm planning to roll this out slowly and slowly by 0.5.0, possibly 0.4.0 (which will be the release after this bug fix release). The project will either be named WinPath.Manager or WinPath.Elevated -- the former makes more sense -- anyhow, the point; this project will be shipped in 32-bit, 64-bit, arm, and arm64.

The above was a comment from a developer perspective, what follows is from a user perspective.
As a computer user I very much dislike the idea of every program having its own update checking (be it on startup, as a scheduled task, or worse as an always running background service), not to mention its own hand-rolled installers and updaters which often do not respect system conventions and/or system preferences, and have no rollback functionality to restore the system to a known working state if something goes wrong.

I agree with this to be honest, and I completely understand that. If you didn't realize that yet, WinPath never checks for update. The user doesn't even need to go beyond v0.1.0, I think it's a "fully shipped" release, it has the core features meant for it. The only way the user can update is by running winpath update and that's only when it does that, I do not do that elsewhere. I think it's highly unnecessary. It's all up to the user, besides, I don't really get anything from forcing an update, it's all up to the user, all responsibility.

If you want to distribute your programs, you either provide them as a simple .zip archive, or you use one of the myriad known good installers that provide full uninstall, rollback in case of failure, update of existing versions, silent installation and uninstallation, 32-bit and 64-bit OS differences, installation of pre-requisites, etc, such as InstallShield, WIX, MSI, or my favorite InnoSetup which is free to use.

I agree, but that makes sense when the application is actually big enough. WinPath is literally just a standalone not even a megabyte big:

I also eventually decide to roll out an installer script that basically sets this up for the user, though not compulsory, it just eases the process of installation.

A final friendly advice:
C# and .Net are great productivity tools. They enable you to churn out enormous amount of functional code in a rather short time. That's great when you are working on client/server architecture projects (ASP.NET, MVC, etc), but when you are working on a Windows platform and writing console, desktop, or Windows service applications you cannot skip the basics.
...
And the list goes on.
For those who started with C/C++, this was the only way to go, but people starting with .Net usually don't have that platform knowledge and aren't aware they will need it as soon as they aren't happy with how some .Net WinForms control or library API is working.

Sorry, what's your point? I don't quite understand what you're trying to say here.

Oh and, thanks for your time, it really helps getting strong feedback.

That's exactly what I said over there, sure it was about Environment.SetEnvironmentVariable but was not limited to that.

It seems we are talking past each other. The issue "over there" was limited to Environment.SetEnvironmentVariable until you posted unrelated information.
The issue with 'Environment.SetEnvironmentVariable' is a bug (or an API design failure if you want) in .Net Framework code which causes 'Environment.SetEnvironmentVariable' method to behave differently compared to underlying Win32 API. It is all well documented in my back and forth with Microsoft engineers.
Your issue was that you did not understand how underlying Win32 API which you are calling through C# wrapper actually works -- you complained about expected and documented behavior.
The fact that both 'Environment.SetEnvironmentVariable' and 'Environment.GetFolderPath' are in the same namespace does not make them related. In fact, each belongs to a different dev team -- SetEnvironmentVariable is a Windows kernel API, while SHGetFolderPath and SHGetKnownFolderPath are Windows shell APIs.

Sorry, what's your point? I don't quite understand what you're trying to say here.

What I was trying to say is that you need much better understanding of the Windows platform before reporting issues with Windows platform API, and I gave you a list of things which I consider necessary knowledge if one is to write Windows applications.
Furthermore, you should always open a new issue for each problem you discover -- developers don't like when people report multiple issues in a single ticket and there are many good reasons to avoid that.
Finally, hijacking discussion threads that you perceive as related to your issue is generally frowned upon in every discussion forum on the internet, and most people consider it rude.

Oh and, thanks for your time, it really helps getting strong feedback.

You are welcome.

Everything is settled, relating to this issue. We will not (and can not by semver) change this current system until v0.4.0 in favor of an elevated manager.
There is nothing to discuss regarding this topic.


Export to ChimeraX

ref: https://github.com/hgb-bin-proteomics/MSAnnika/issues/18

works out of the box
see https://github.com/hgb-bin-proteomics/MSAnnika/issues/18


multi tier cache failed

We’ve encountered below error when tried to run multitier_cache example:

Code snippet:

I am not sure what is the best way to improve this behavior but I’d like to provide you with a quick workaround so that you are not blocked by this issue.
When you configure cache with more than 1 tier, there’s some additional memory reserved for internal needs per each tier. So, in the presence of tiers you can’t allocate a pool whose size is equal to total available size from cache stats. The following formula should work (I tested it for several total sizes and tier numbers) to calculate the max number of bytes available for a single pool:
size_t maxPoolSize = gCache_->getCacheMemoryStats().cacheSize - ((config.getMemoryTierConfigs().size() - 1) * Slab::kSize);
The fact that the more tiers you use the less memory you are left with seems to have a more dramatic effect for the caches whose total sizes are relatively small (e.g. 48Mb in your code). But for larger caches (e.g. 1Gb) the decrease in memory available seems negligible. So, if you need a bigger pool, I’d suggest that you also play with the total cache size.
Please let me know whether you are able to use the provided formula or whether you have any questions. I’ll investigate the issue more and discuss it with the others to make sure that it is fully addressed, if needed, in the meantime.

@chenyou-intel The problem was caused by incorrect memory tier sizes that were used to determine per tier pool sizes. I submitted a fix and new regression tests for review (https://github.com/pmem/CacheLib/pull/76). If you want to try the fixed code before the promotion to develop is complete, please use this branch: https://github.com/pmem/CacheLib/tree/issue75.

@victoria-mcgrath Thanks! I'll try this branch


version number of library

what is last version number of library ?
and please guide me how to use this CircularSlider by an example

@mohammad7hosein You can see the latest version of the library in the document. it's a 0.0.1
and for the example, you can simply clone the repo and look into that.
https://github.com/Mindinventory/AndroidCircularSlider.git


revise xrefs for 'familial amyloid neuropathy'

and add new class ATTRV30M amyloidosis
close https://github.com/monarch-initiative/mondo/issues/7330

thank you!


vulkan backend use a lot of memory

samples at https://github.com/xtuzy/veldrid.maui/tree/vulkan_memory_debug, we see vulkan backend's memory how to change in wpf sample

Vulkan memory manage in VkDeviceMemoryManager class. Allocate and Free memory will use it.

Allocate process
When we use ResourceFactory.CreateBuffer(), https://github.com/xtuzy/veldrid.maui/blob/a904eb99e4b5ba0bcdbe7c4cc72e3d5b77bc4559/src/Samples/Veldrid.Maui.Samples/Veldrid.Maui.Samples.Core/GettingStarted/GettingStarted.cs#L60
it will load https://github.com/xtuzy/veldrid.maui/blob/a904eb99e4b5ba0bcdbe7c4cc72e3d5b77bc4559/src/Veldrid/Vk/VkResourceFactory.cs#L82
to create a VkBuffer, then load https://github.com/xtuzy/veldrid.maui/blob/a904eb99e4b5ba0bcdbe7c4cc72e3d5b77bc4559/src/Veldrid/Vk/VkBuffer.cs#L101

Free process
When create a VkBuffer, will load https://github.com/xtuzy/veldrid.maui/blob/a904eb99e4b5ba0bcdbe7c4cc72e3d5b77bc4559/src/Veldrid/Vk/VkBuffer.cs#L115
ResourceRefCount have _refCount to record count that use this buffer,
https://github.com/xtuzy/veldrid.maui/blob/a904eb99e4b5ba0bcdbe7c4cc72e3d5b77bc4559/src/Veldrid/Vk/ResourceRefCount.cs#L9
When _refCount equal to 0, will load DisposeCore()
https://github.com/xtuzy/veldrid.maui/blob/a904eb99e4b5ba0bcdbe7c4cc72e3d5b77bc4559/src/Veldrid/Vk/ResourceRefCount.cs#L32
DisposeCore will load https://github.com/xtuzy/veldrid.maui/blob/a904eb99e4b5ba0bcdbe7c4cc72e3d5b77bc4559/src/Veldrid/Vk/VkBuffer.cs#L139

From Allocate process,we can decrease https://github.com/xtuzy/veldrid.maui/blob/a904eb99e4b5ba0bcdbe7c4cc72e3d5b77bc4559/src/Veldrid/Vk/VkDeviceMemoryManager.cs#L12 and https://github.com/xtuzy/veldrid.maui/blob/a904eb99e4b5ba0bcdbe7c4cc72e3d5b77bc4559/src/Veldrid/Vk/VkDeviceMemoryManager.cs#L244
How to get suitable size?

TechPizzaDev use 1m and 8m
https://github.com/TechPizzaDev/veldrid/blob/c0807ebcc4e6c4e1efabf9245d4233d0a960804f/src/Veldrid/Vk/VkDeviceMemoryManager.cs#L251
I use Drawable in Veldrid.Maui.Control, i want release resource when change Drawable, so i hope big block will be dispose, not use memory. When set small MinDedicatedAllocationSize, dispose  these big block will release memory immediately, if MinDedicatedAllocationSize is big, block will be recycle to ChunkAllocator. Because ChunkAllocator be divided, when big block be dispose, maybe have small block in ChunkAllocator, we can't  release memory immediately, we need add better Allocator and defragmentation algorithm


Wrong range for Time randomizers

Time randomizers (LocalTimeRandomizer, LocalTimeRangeRandomizer, etc.) do not properly generate values in the expected ranges (up to 23 for hours, 59 for minutes and seconds, etc.), as the upper value is excluded.
The reason for this seems to be in the nextDouble method of AbstractRandomizer which is used under the hoods by these classes. Thus, I suspect even more randomizers may be affected by this.
The bug can be easily reproduced like this.
  @Test
  void hourRandomizerRangeTest() {

    HourRandomizer hourRandomizer = new HourRandomizer();
    int maxHour = IntStream.range(0, 100_000)
      .map(e -> hourRandomizer.getRandomValue())
      .max()
      .orElseThrow();

    assert maxHour == 23 : "Max hour is 23";
  }

That's a valid issue. Thank you for reporting it!


[BACKLOG-40843] Unit test fixes for JDK17

There was an unused mock error coming from the engine StepMockHelper that got used in one test, so I threw a lenient in there, only usual thing about this one

 Note:

Frogbot also supports Contextual Analysis, Secret Detection, IaC and SAST Vulnerabilities Scanning. This features are included as part of the JFrog Advanced Security package, which isn't enabled on your system.

🐸 JFrog Frogbot

:warning: Build finished in 27m 20s
Build command:
mvn clean verify -B -e -Daudit -Djs.no.sandbox -pl \
engine,plugins/xml,plugins/xml/core

:no_entry: Failed Tests
:cloud_with_lightning_and_rain: 1 test(s) failed:
org.pentaho.di.trans.steps.fieldsplitter.FieldSplitterTest.testSplitFields (click to expand)
Output row is of an unexpected length expected:<4> but was:<3>

Tests run: 2407, Failures: 1, Skipped: 13    Test Results

:information_source:  This is an automatic message

Not sure about that failed test. Wingman ran the engine tests because of the StepMockHelper change, but I don't think this would be related to adding a lenient() to  a mock -- we should probably look at that tests, but I don't think it's related to this change...


Always balance page children

Balancing the nodes inside the split function allows that if the only children is a r3f scene, it renders it inside the Canvas layout. Right now if there is only 1 children it's always rendered as a DOM node.

Awesome! Weirdly enough HMR does not work when we edit <Page />, but it is not related to these changes it must be something with nextjs.
Thanks!


OpenSSL warnings with no effect

May be related to #9
(Still migrating vom EM/Thin to Async/Falcon...)
While configuring SSL support for the Falcon::Server instance that runs in one of the processes of our app, I noticed these OpenSSL errors in the log:
With Safari connected the server running inside my Linux VM:
59.39s     warn: Async::Task [oid=0x1e21c] [ec=0x1e230] [pid=3923825] [2024-07-10 19:56:41 +0200]
               | Task may have ended with unhandled exception.
               |   OpenSSL::SSL::SSLError: SSL_accept returned=1 errno=0 peeraddr=<IP_ADDRESS>:62856 state=error: unexpected eof while reading
               |   → /home/user/.gem/ruby/3.3.1/gems/io-endpoint-0.11.0/lib/io/endpoint/ssl_endpoint.rb:16 in `accept'
               |     /home/user/.gem/ruby/3.3.1/gems/io-endpoint-0.11.0/lib/io/endpoint/ssl_endpoint.rb:16 in `start'
               |     /home/user/.gem/ruby/3.3.1/gems/io-endpoint-0.11.0/lib/io/endpoint/wrapper.rb:171 in `block in accept'
               |     /home/user/dev/oss/async/lib/async/task.rb:164 in `block in run'
               |     /home/user/dev/oss/async/lib/async/task.rb:377 in `block in schedule'

With Chromium connected to the server inside the VM:
    1m     warn: Async::Task [oid=0x1e2d0] [ec=0x1e2e4] [pid=3923825] [2024-07-10 19:57:32 +0200]
               | Task may have ended with unhandled exception.
               |   OpenSSL::SSL::SSLError: SSL_accept returned=1 errno=0 peeraddr=<IP_ADDRESS>:62867 state=error: sslv3 alert certificate unknown
               |   → /home/user/.gem/ruby/3.3.1/gems/io-endpoint-0.11.0/lib/io/endpoint/ssl_endpoint.rb:16 in `accept'
               |     /home/user/.gem/ruby/3.3.1/gems/io-endpoint-0.11.0/lib/io/endpoint/ssl_endpoint.rb:16 in `start'
               |     /home/user/.gem/ruby/3.3.1/gems/io-endpoint-0.11.0/lib/io/endpoint/wrapper.rb:171 in `block in accept'
               |     /home/user/dev/oss/async/lib/async/task.rb:164 in `block in run'
               |     /home/user/dev/oss/async/lib/async/task.rb:377 in `block in schedule'
    1m    debug: Async::HTTP::Protocol::HTTPS [ec=0x1e2f8] [pid=3923825] [2024-07-10 19:57:32 +0200]
               | Negotiating protocol "h2"...
    1m    debug: Falcon::Server [oid=0x1e208] [ec=0x1e2f8] [pid=3923825] [2024-07-10 19:57:32 +0200]
               | Incoming connnection from nil to Async::HTTP::Protocol::HTTPS

With Chromium inside the VM:
    6m     warn: Async::Task [oid=0x1e550] [ec=0x1e564] [pid=3923825] [2024-07-10 20:02:23 +0200]
               | Task may have ended with unhandled exception.
               |   OpenSSL::SSL::SSLError: SSL_accept returned=1 errno=0 peeraddr=<IP_ADDRESS>:37782 state=error: sslv3 alert certificate unknown
               |   → /home/user/.gem/ruby/3.3.1/gems/io-endpoint-0.11.0/lib/io/endpoint/ssl_endpoint.rb:16 in `accept'
               |     /home/user/.gem/ruby/3.3.1/gems/io-endpoint-0.11.0/lib/io/endpoint/ssl_endpoint.rb:16 in `start'
               |     /home/user/.gem/ruby/3.3.1/gems/io-endpoint-0.11.0/lib/io/endpoint/wrapper.rb:171 in `block in accept'
               |     /home/user/dev/oss/async/lib/async/task.rb:164 in `block in run'
               |     /home/user/dev/oss/async/lib/async/task.rb:377 in `block in schedule'

Despite these errors, the browser can load the website just fine. How can that be?

This doesn't seem to happen anymore.


Update pl-PL.json

Coverage remained the same at 92.162% when pulling b1331c989078c5d97aeae68559b889a2825e670b on Kun2210:master into 46cfe51351c545223a77b1ff6f95de3dfb57b661 on MarshallOfSound:master.


switch to Aggregation without serde_untagged

refactor result handling
remove Internal stuff
merge different accessors
switch to Aggregation without serde_untagged

Closes https://github.com/quickwit-oss/quickwit/issues/1485

Codecov Report

Merging #2003 (e841842) into main (c1defdd) will decrease coverage by 0.10%.
The diff coverage is 94.30%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main    #2003      +/-   ##
==========================================
- Coverage   94.44%   94.35%   -0.10%
==========================================
  Files         320      320
  Lines       59231    59001     -230
==========================================
- Hits        55943    55669     -274
- Misses       3288     3332      +44

Impacted Files
Coverage Δ

src/aggregation/agg_result.rs
76.92% <ø> (-1.91%)
:arrow_down:

src/aggregation/bucket/mod.rs
96.15% <ø> (ø)

src/aggregation/metric/mod.rs
94.54% <ø> (ø)

src/aggregation/intermediate_agg_result.rs
95.32% <83.96%> (-2.00%)
:arrow_down:

src/aggregation/bucket/term_agg.rs
97.95% <85.18%> (-0.78%)
:arrow_down:

src/aggregation/agg_req.rs
94.11% <86.51%> (+1.03%)
:arrow_up:

src/aggregation/segment_agg_result.rs
93.18% <94.93%> (-2.75%)
:arrow_down:

src/aggregation/metric/percentiles.rs
89.26% <96.29%> (-2.03%)
:arrow_down:

src/aggregation/metric/stats.rs
92.51% <97.82%> (-5.15%)
:arrow_down:

src/aggregation/agg_bench.rs
100.00% <100.00%> (ø)

... and 7 more

... and 2 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Is there a way to make containers with same group the same height?

I am working on an application where it is needed to drag orders from one column to the other. If one column is very large and the target column is almost empty, the user will have to drag the order to the top of the screen before dropping. If the target column is empty, it is very hard to "find" the container (find the place where to drop, so the onDrop function will be called).
Is there a way to make all the columns equal in height? I didn't see it in the demo.

@LisaMaagdenberg Hi!. Maybe I do not understand something, but it seems your problem can be solved by using js. You need to find the height of the 1st column and then apply the same height to the second column.

Hi @LisaMaagdenberg
One solution is like @TitanFighter said you can arrange height of columns by javascript after each drop.
You can also achieve this by overriding css classes of containers and draggables. For card board page in the demo app I added these classes to make them all equal height.
.smooth-dnd-container.horizontal {
  display: flex;
}

.smooth-dnd-container.horizontal .smooth-dnd-draggable-wrapper {
  height: auto;
}

.card-container {
  height: 100%;
}

.card-container .smooth-dnd-container {
  height: 100%;
}


Wltest monsoon composite energy

Hi guys, this is what I'm using right now for Monsoon energy meter support. It's slightly ahead of the one we merged already. I think I need to rebase and push again before we merge this.

I've already merged a patch (c51081abf093b) which fixes the same issue of the first patch in this series, which can then be dropped.
For the second patch instead I need some more time to review/test it.
The last patch should not be required if we rebase the second one on the master branch.


Cache file in memory after first segment fetch

When the client fetches the file, cache the remaining segments in memory so that we don't have to make many subsequent calls to the database.

We don't need this. Fetching works well enough for normal use cases. For better performance, use DPDK branch.