support non-interactive transactions in sql-over-http

Background
My company (Brevity) deploys hundreds (soon thousands) of cloudflare workers for our clients. We also supply each client with a neon database to handle data persistence and querying. Latency is important to our clients so the traditional sql-over-websockets approach has not worked for us because it adds too much latency to each request (no connection pooling!). To mitigate this, we currently deploy a small server for each neon database that manages a connection pool to the database and as a single api for querying data over http.
Needless to say, we are very excited about the sql-over-http feature that is currently in development at neon because it will allow us to completely remove these extra servers from our stack (and all the orchestration that is involved). We see this as a unique value-add for Neon and it is a large reason why we choose Neon in the first place.
Problem
That said, the sql-over-http feature as it is currently implemented does not support our use case because it does not support basic transactions or multi-statement queries. We allow our users to create row level security policies for the data in their database and before we execute a db query on their behalf, we set some db local variables (ex SET LOCAL jwt.claims.sub = '{user_id}') to establish who the current user is, which in turn allows the row level security policies to make decisions about what data the current user has access to. These local variables only live as long as the current transaction which is perfect for our use-case but means that we can not realistically use Neon's sql-over-http feature as it is currently implemented because it does not support transactions (I know it's still a work in progress).
Summary of changes
While I don't really expect this PR to be merged, I wanted to show a proof of concept on how basic transactions could be supported by the http endpoint. With this change, users can choose to send a single query to the http endpoint (which is consistent with the current functionality) or they can send an array of queries which would automatically be wrapped in a transaction and executed serially. Sending an array of queries would return an array of results. This would support our use case.

NOTE: I did not have access to the custom fork of tokio-postgres so for the time being, I changed query_raw_txt back to query_raw. In order to support this fully, we would have to add query_raw_txt support to tokio_postgres::Transaction and the GenericClient trait -- which should be fairly simple.

Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

[ ] Do not forget to reformat commit message to not include the above checklist

cc: @kelvich I'd love to get your thoughts on if you think something like this might be feasible (or if you have an alternate api in mind)

Thank you for posting!
I like that proposal, and makes sense to add that API in proxy. We were mostly thinking of sql-over-http as a communications layer behind postgres js drivers that we are hooking it into. And since the driver is general-purpose it supports interactive transactions, where next query may be constructed only after results for a current query are received. So the API for transaction is more of:

POST /create_tx -> tx_id
POST /execute_query?tx_id=...&query=...&params=...
POST /commit_tx?tx_id=...

However, you highlighted a good special case where the whole transaction could be sent as one array. That both makes a lot of sense and easier to implement compared to the interactive transaction flow.

Thanks @kelvich!
FWIW, Prisma (a very popular ORM for Node) supports both "interactive transactions" as you described and "sequential transactions" as I described by passing in an array of operations.
Long term, it would be amazing if Neon could support both but I suspect you are correct that "sequential" transactions would be much easier to implement in the short term and would unlock many use cases.

This PR looks pretty good, thanks @nicksrandall! I have fixed a small issue that query_raw_txt should be used instead of query_raw, which is from our own patch of rust-postgres, and I believe this PR is good to go. @conradludgate would you please take another pass on this PR and also https://github.com/neondatabase/neon/pull/4793 + https://github.com/neondatabase/rust-postgres/pull/20 tomorrow? Thanks!

Updated commit message:
This PR adds support for non-interactive transaction query endpoint.
It accepts an array of query and parameters, and return an array of
query results. The queries will be run in a single transaction one
after another on the proxy side.

decided to merge this after tomorrow's release so that we can run this for some time in staging and see if things go well :)

Thanks everybody! This will be big for us at Brevity. Is there anyway I can test these changes while in staging?

Staging is for internal access and therefore we will test it and this PR will be released to production next Tuesday. We are also planning to update the serverless driver to have native support for batch SQL. For example, we might have something like:
const result = await batch([
  sql`select 1`,
  sql`select * from t`
]);

probably also in this week and will be released later.

@nicksrandall I'm just catching up here after being away. I've had a play with the proxy support, which looks really nice. Thank you!
@conradludgate @skyzh @kelvich A couple of thoughts:
(1) If we run into an error in one of the queries, could we try to add an indication in the error message of which query caused the problem (and perhaps also confirm that everything got rolled back)?
(2) At the moment, it seems that the isolation level and read-only status are echoed back in the response headers if they were specified in the request headers, but not otherwise. It seems to me like it's would be more valuable to include them in the response when they're not specified explicitly and defaults were therefore used. Would this be easy to do?

(1) If we run into an error in one of the queries, could we try to add an indication in the error message of which query caused the problem (and perhaps also confirm that everything got rolled back)?

Yeah this sounds good, will add today.

(2) At the moment, it seems that the isolation level and read-only status are echoed back in the response headers if they were specified in the request headers, but not otherwise. It seems to me like it's would be more valuable to include them in the response when they're not specified explicitly and defaults were therefore used. Would this be easy to do?

Read-only is always echoed back. For isolation levels, because we have checks when processing the request, and if it is wrongly set, the request will simply fail. If isolation level is not set, it will be using the default isolation level in the Postgres, which will need a SQL to know what's the current level. Therefore, the current implementation should be good that only echoing back when user sets it.

fixed in https://github.com/neondatabase/neon/pull/4958

We did discuss it, but I'm not sure if returning the prior query results is a great idea.
First, they could be misleading — for example if my query is INSERT INTO x (a, b, c) VALUES (1, 2, 3) RETURNING id, then I get back a non-existent id.
Second, I think the way we'll deal with the error in the JS client is by throwing, which means there's no obvious way to get this data back to the user.

FWIW, I agree with @jawj that partial results are not helpful if the transaction was rolled back and therefore data returned is not accurate.
That said, it would be helpful to know what query caused the rollback and what the error was.

I think the way we'll deal with the error in the JS client is by throwing, which means there's no obvious way to get the data back to the user.

I raised this in https://github.com/neondatabase/neon/pull/4958#issuecomment-1674954432 but I think you can throw custom exception types that can have proper data in the objects. eg
try {
    batch_query();
} catch (NeonBatchQueryException e) {
    console.error(e.results);
}

@kelvich

POST /create_tx -> tx_id
POST /execute_query?tx_id=...&query=...&params=...
POST /commit_tx?tx_id=...

Nice! Would like to see any documentation on that interface.
So far from your site it looks that this use-case is not exposed neither from @neondatabase/serverless nor from any other API. Is there an HTTP REST/Swagger/OpenAPI doc on this one?


Loving this fantastic plugin. Just trying to contact the creator and thank him personally.

I'm using your epic/beautiful vocal output plugin for my newest Lollms plugin I plan to sell on the marketplace very soon.
Video Demonstration for my plugin using your beautiful vocal ai output plugin too
Thanks a million for posting this on GitHub instead of on the marketplace.
I'm willing to negotiate an amount of income for your beautiful contribution too. Thanks friend. Your fantastic ElevenLabs Ai vocal output piece is a super important piece for my plugin. Thanks friend!

np, code is free to distribute


[Doc revamp][test doc revamp harsh3].md

This is a dummy PR. Please review.

Docs Build status updates of commit 44c0645:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/resources/create-messaging-extension-using-appstudio.md
:warning:Warning
View
Details

msteams-platform/messaging-extensions/what-are-messaging-extensions.md
:white_check_mark:Succeeded
View

msteams-platform/resources/create-messaging-extension-using-appstudio.md

Line 235, Column 53: [Warning-bookmark-not-found] Cannot find bookmark '#action-commands' in 'messaging-extensions/what-are-messaging-extensions.md'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


NAS-123774 / 24.04 / add InitShutDownScript plugin

Jira URL: https://ixsystems.atlassian.net/browse/NAS-123774


Animate element when in viewport with isVisible?

Hi, thanks for a great plugin.
Is it possible to get an example on how I would animate elements when they come into the viewport? I Guess i use isVisible but I can't get it to work.
[...document.querySelectorAll('.animate-this')].forEach((el) => {
 scrollbar.isVisible(el) ? el.classList.add('is-visible') : false
})

Here's an old demo about isVisible method: http://jsbin.com/noxoho/edit?js
Hope it could help you :)

@idiotWu thanks, perfect=)

@idiotWu This only fires when you interact with the scrollbar, what about if the element is at the top of the page or partially showing? try to get some functionality similar to waypoint.js

@adamwright32 I don't understand what you mean by "interact with the scrollbar". The scrollbar.isVisible() method  just simply compares the bounding rect of the given element with the bounding rect of scrollbar: https://github.com/idiotWu/smooth-scrollbar/blob/8ff9227a73691a937c855ec135db5d2c76d6a868/src/geometry/is-visible.ts#L3-L14

@idiotWu but thats only triggered when you are listening to the scrollbar. So if the page loads and you don't scroll it won't fire. Or maybe i'm doing it wrong
scrollbar.addListener(function (status) {
            // Content Feature Header
            $(".js-header").each(function () {
                var $this = $(this);

                if(scrollbar.isVisible(this))
                {
                    $this.addClass("animated titleInRight");
                }
            });

@adamwright32 no you don't have to put it in the scroll listener. It's a static check and you can run it everywhere.

@idiotWu
Is there a way for scrollbar.isVisible(element) to check if the element isVisible on page load?
Reference example: http://jsbin.com/hujugirexi/edit?html,js,output (I re-ordered the divs in your previous example, such that "content" is on top). "content" only shows "visible: true" when scrolling at least once (even though it is visible upon page load).
Thanks for the help.

@dtab428 simply call isVisible() at anywhere you want

Hi, can you please give us an example on windows load?


added ttl

added ttl that was in issue #5

I'll have to close the PR, as it's been stale for a few days now, and this needed to be implemented, so I did it myself. However, I would still like to thank you for the contribution, and hopefully you'll be able to contribute in the future.


Thumbnail error takes down Dashboard completely

At the moment my users are regularly experiencing a problem where they're completely locked out of the Dashboard (the login page doesn't even function). The only way I can restore the site is to change the image manipulation library between GD and Imagemagick (by manually editing config files), which for some reason jolts it back into life.
Looking in the error logs, it's always:
Exception Occurred: ......concrete/src/Http/Middleware/ThumbnailMiddleware.php:76 Call to a member function getStatusCode() on null (1)

I don't know which image is causing the problem, and as there's about 900 of them in there manually checking might take some time.
Is there anything further I can do to investigate or fix this? Should the thumbnail middleware really be preventing logging in?

#7115 should fix this issue


feat: multiple highlight area in walkthrough, corrected walkthrough events for each feature

Description

Added support for multiple highlights in walkthrough
Corrected walkthrough events for each feature walkthrough
Added fix for the walkthrough dismissal in case of empty/no schema feature

PR fixes following issue(s)
Fixes #25417
Fixes #25416
Media

A video or a GIF is preferred. when using Loom, don’t embed because it looks like it’s a GIF. instead, just link to the video

Type of change

Please delete options that are not relevant.

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
Chore (housekeeping or task changes that don't impact user perception)
This change requires a documentation update

Testing

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration.
Delete anything that is not relevant

[ ] Manual
[ ] Jest
[ ] Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

/build-deploy-preview skip-tests=true

Few Observation :

The walkthrough for select and chart widget appear intermittently
Content in the walkthrough is only related to table and when Select and chart is populated it becomes irrelevant

@prapullc As per discussion, keeping learning popover only for table, removing it for select and chart

/build-deploy-preview skip-tests=true

/ok-to-test

/build-deploy-preview skip-tests=true

Tested per above discussions, and found the popover to be appearing correctly for Table widget selection.

/ok-to-test

/ok-to-test

This PR created a release blocker : #25941


More documentation regarding hot reloading ?!!!

My mind can't grasp the concept behind doing hot reloading without doing a compile!!!
At first I thought it was a joke, but I guess it's not.
Could you please explain further the concept at least briefly.

Good question. I'll add info about it to the docs.
Let's say you have
#!hot

fn main() {
    for {
        foo()
        time.sleep(1)
    }
}

fn foo() {
    println(1)
}

It will print number 1 every second.
However since we used the hot directive (name is not final 😃), foo() is loaded dynamically, and V will insert a small program that will run in the background and wait for modifications in main.v. Once it's modified, that program will ask V to generate main.so with the modified function and reload foo. So if you change println(1) to println(2) while the program is running, you will get
1
1
1
2
2


Break pyslurm dependency

The pyslurm dependency for interactions with the workflow manager is, to be honest, unnecessary for what we're doing and the source of some difficulty for easy adoption.
This can be easily overcome by decoupling from direct usage of vN.N APIs and using the command line instead. Currently this tool will get in a grump if you can't see to installing the correct interface for Slurm, and this makes it annoying for users.
Update the tool and the instructions for usage.

Still need to do the following:

[ ] Sort out the README / docs for installation
[ ] Ensure this all gets updated in PyPI

PyPI task also linked with #4


Consider changing default date format

Currently the default date format is YYYY-MM-DD, while a more frequently used format is YYYY/MM/DD. Let's consider making that the default.

Not sure if it's worth the change. This is a breaking change for everyone using the Datepicker (and you can simply change to default if you want something else)


Enum rule defined_only not aligned with official protobuf style guide

Note: Only tested with Java.
According to the official style guide https://developers.google.com/protocol-buffers/docs/style#enums the enum value 0 should be used to indicate that the enum constant is UNSPECIFIED or not set at all.
The Enum rule defined_only accepts 0 as a valid enum value.
Either the rule should be updated to disallow 0 or there should be another rule in place.
Workaround: Specify not_in as an additional rule: [ (validate.rules).enum = { defined_only: true, not_in: [0] } ]

I'd treat that style guide as suggestions (i.e. "you really should do this"), not hard-and-fast rules. Since it's not baked into the protobuf compiler, and there is a workaround, I'm hesitant to break everyone who uses 0 as a valid enum value.

I presented 2 possible solutions, and I agree that breaking the current functionality is not the best idea.
But what about adding another rule defined_only_not_unspecified (or similar) to support this style guide recommendation?


Not working on 10.0.19041.3570

.\autoupdate.bat
...
[*] Start searching [10.0.19041.3570] version entry in file "C:\Program Files\RDP Wrapper\rdpwrap_new.ini"...
[-] NOT found "termsrv.dll" version entry [10.0.19041.3570] in file "C:\Program Files\RDP Wrapper\rdpwrap_new.ini"!

[10.0.19041.3570]
LocalOnlyPatch.x86=1
LocalOnlyOffset.x86=C3549
LocalOnlyCode.x86=jmpshort
SingleUserPatch.x86=1
SingleUserOffset.x86=3FDD7
SingleUserCode.x86=Zero
DefPolicyPatch.x86=1
DefPolicyOffset.x86=419A9
DefPolicyCode.x86=CDefPolicy_Query_eax_ecx
SLInitHook.x86=1
SLInitOffset.x86=71088
SLInitFunc.x86=New_CSLQuery_Initialize
LocalOnlyPatch.x64=1
LocalOnlyOffset.x64=92AF1
LocalOnlyCode.x64=jmpshort
SingleUserPatch.x64=1
SingleUserOffset.x64=11E12
SingleUserCode.x64=Zero
DefPolicyPatch.x64=1
DefPolicyOffset.x64=1F315
DefPolicyCode.x64=CDefPolicy_Query_eax_rcx
SLInitHook.x64=1
SLInitOffset.x64=2866C
SLInitFunc.x64=New_CSLQuery_Initialize
[10.0.19041.3570-SLInit]
bInitialized.x86      =E498C
bServerSku.x86        =E4990
lMaxUserSessions.x86  =E4994
bAppServerAllowed.x86 =E499C
bRemoteConnAllowed.x86=E49A4
bMultimonAllowed.x86  =E49A8
ulMaxDebugSessions.x86=E49AC
bFUSEnabled.x86       =E49B0
bInitialized.x64      =125074
bServerSku.x64        =125078
lMaxUserSessions.x64  =12507C
bAppServerAllowed.x64 =125084
bRemoteConnAllowed.x64=12508C
bMultimonAllowed.x64  =125090
ulMaxDebugSessions.x64=125094
bFUSEnabled.x64       =125098

the service is not started o_O even after reboot
this is strange because before the ini change, the service was working

重启后可以多用户连接，非常感谢

[10.0.19041.3570] LocalOnlyPatch.x86=1 LocalOnlyOffset.x86=C3299 LocalOnlyCode.x86=jmpshort SingleUserPatch.x86=1 SingleUserOffset.x86=3FDC7 SingleUserCode.x86=零 DefPolicyPatch.x86=1 DefPolicyOffset.x86=41999 DefPolicyCode。 x86=CDefPolicy_Query_eax_ecx SLInitHook.x86=1 SLInitOffset.x86=70E18 SLInitFunc.x86=New_CSLQuery_Initialize LocalOnlyPatch.x64=1 LocalOnlyOffset.x64=92AF1 LocalOnlyCode.x64=jmpshort SingleUserPatch.x64=1 SingleUserOffset.x64=11E12 SingleUserCode.x64=零 DefPolicyP补丁.x64=1 DefPolicyOffset.x64=1F315 DefPolicyCode.x64=CDefPolicy_Query_eax_rcx SLInitHook.x64=1 SLInitOffset.x64=2866C SLInitFunc.x64=New_CSLQuery_Initialize
[10.0.19041.3570-SLInit] bInitialized.x86 =E498C bServerSku.x86 =E4990 lMaxUserSessions.x86 =E4994 bAppServerAllowed.x86 =E499C bRemoteConnAllowed.x86 =E49A4 bMultimonAllowed.x86 =E49A8 ulMaxDebugSessions。 x86=E49AC bFUSEnabled.x86 =E49B0 bInitialized.x64 =125074 bServerSku.x64 =125078 lMaxUserSessions.x64 =12507C bAppServerAllowed.x64 =125084 bRemoteConnAllowed.x64=12508C bMultimonAllowed.x64 =125090 ulMaxDebugSessions.x64=125094 bFUSEnabled.x64 =125 098

非常感谢

error 1067

привет. Удалось решить?

PDB file here: https://github.com/stascorp/rdpwrap/issues/2357#issuecomment-1756205715

[10.0.19041.3570]
LocalOnlyPatch.x64=1
LocalOnlyOffset.x64=92AF1
LocalOnlyCode.x64=jmpshort
SingleUserPatch.x64=1
SingleUserOffset.x64=11E12
SingleUserCode.x64=Zero
DefPolicyPatch.x64=1
DefPolicyOffset.x64=1F315
DefPolicyCode.x64=CDefPolicy_Query_eax_rcx
SLInitHook.x64=1
SLInitOffset.x64=2866C
SLInitFunc.x64=New_CSLQuery_Initialize
[10.0.19041.3570-SLInit]
bInitialized.x64 =125074
bServerSku.x64 =125078
lMaxUserSessions.x64 =12507C
bAppServerAllowed.x64 =125084
bRemoteConnAllowed.x64=12508C
bMultimonAllowed.x64 =125090
ulMaxDebugSessions.x64=125094
bFUSEnabled.x64 =125 098

[10.0.19041.3570] LocalOnlyPatch.x86=1 LocalOnlyOffset.x86=C3299 LocalOnlyCode.x86=jmpshort SingleUserPatch.x86=1 SingleUserOffset.x86=3FDC7 SingleUserCode.x86=零 DefPolicyPatch.x86=1 DefPolicyOffset.x86=41999 DefPolicyCode。 x86=CDefPolicy_Query_eax_ecx SLInitHook.x86=1 SLInitOffset.x86=70E18 SLInitFunc.x86=New_CSLQuery_Initialize LocalOnlyPatch.x64=1 LocalOnlyOffset.x64=92AF1 LocalOnlyCode.x64=jmpshort SingleUserPatch.x64=1 SingleUserOffset.x64=11E12 SingleUserCode.x64=零 DefPolicyP补丁.x64=1 DefPolicyOffset.x64=1F315 DefPolicyCode.x64=CDefPolicy_Query_eax_rcx SLInitHook.x64=1 SLInitOffset.x64=2866C SLInitFunc.x64=New_CSLQuery_Initialize
[10.0.19041.3570-SLInit] bInitialized.x86 =E498C bServerSku.x86 =E4990 lMaxUserSessions.x86 =E4994 bAppServerAllowed.x86 =E499C bRemoteConnAllowed.x86 =E49A4 bMultimonAllowed.x86 =E49A8 ulMaxDebugSessions。 x86=E49AC bFUSEnabled.x86 =E49B0 bInitialized.x64 =125074 bServerSku.x64 =125078 lMaxUserSessions.x64 =12507C bAppServerAllowed.x64 =125084 bRemoteConnAllowed.x64=12508C bMultimonAllowed.x64 =125090 ulMaxDebugSessions.x64=125094 bFUSEnabled.x64 =125 098

非常感谢

Спасибо тебе, китайский друг :)
Убрал йероглифы из конфига и x86

[10.0.19041.3570] LocalOnlyPatch.x64=1 LocalOnlyOffset.x64=92AF1 LocalOnlyCode.x64=jmpshort SingleUserPatch.x64=1 SingleUserOffset.x64=11E12 SingleUserCode.x64=Zero DefPolicyPatch.x64=1 DefPolicyOffset.x64=1F315 DefPolicyCode.x64=CDefPolicy_Query_eax_rcx SLInitHook.x64=1 SLInitOffset.x64=2866C SLInitFunc.x64=New_CSLQuery_Initialize
[10.0.19041.3570-SLInit] bInitialized.x64 =125074 bServerSku.x64 =125078 lMaxUserSessions.x64 =12507C bAppServerAllowed.x64 =125084 bRemoteConnAllowed.x64=12508C bMultimonAllowed.x64 =125090 ulMaxDebugSessions.x64=125094 bFUSEnabled.x64 =125 098

its work,thx!

Напишите нормально код, без иероглифом, а лучше выложите полный файл с dll

Напишите нормально код, без иероглифом, а лучше выложите полный файл с dll А то какие то пробелы лишние тоже есть.

По инфо - все работает. При подключении все равно выкидывает.

хоть с пробелом, хоть без пробела - ошибка. Служба не стартует. При этом отображается, что ini полная поддержка, а вот служба отказывается запускаться. Ругается на rdwrapper.dll

[10.0.19041.3570] LocalOnlyPatch.x64=1 LocalOnlyOffset.x64=92AF1 LocalOnlyCode.x64=jmpshort SingleUserPatch.x64=1 SingleUserOffset.x64=11E12 SingleUserCode.x64=Zero DefPolicyPatch.x64=1 DefPolicyOffset.x64=1F315 DefPolicyCode.x64=CDefPolicy_Query_eax_rcx SLInitHook.x64=1 SLInitOffset.x64=2866C SLInitFunc.x64=New_CSLQuery_Initialize
[10.0.19041.3570-SLInit] bInitialized.x64=125074 bServerSku.x64=125078 lMaxUserSessions.x64=12507C bAppServerAllowed.x64=125084 bRemoteConnAllowed.x64=12508C bMultimonAllowed.x64=125090 ulMaxDebugSessions.x64=125094 bFUSEnabled.x64=125098

work for me after removing extra spaces, thanks

https://github.com/sebaxakerhtc/rdpwrap.ini/commit/467219de236398ce9517936edce536f90c3e07c6

nice work as always @sebaxakerhtc

ini for ini for 10.0.19041.3570.zip

Reopened. Not included in official INI by Stas'M :)

Fully suported bu could nt connected. Please help

sebaxakerhtc/rdpwrap.ini@467219d

Confirmed! The offsets worked a treat.

I can't edit or replace the rdpwrap "Because it's open in remote desktop services". How do I solve this? I just installed it, didn't open it. I've uninstalled it and re-installed several times without change.

can't edit or replace the rdpwrap "Because it's open in remote desktop services".

CMD as Administrator
net stop termService

Or reboot to safe mode.

Cool, that did the trick. Thanks! But I'd like to use this to connect to my work computer, and connect with an IP-adress (I think) (I have windows 10 home, so no default remote desktop for me). Now it seems to be a pre-determined adress which I don't know where it leads to. Is there a way to configure this?

After replaced the ini file,fullysupported,but not listling and the seviece is stop ,what wrong ?PLS HLP

@oukunhi

After replaced the ini file,fullysupported,but not listling and the seviece is stop ,what wrong ?PLS HLP

Try:

Download RDP Wrapper 1.6.2 and extract this to any directory.
Place rdpwrap.ini in the same directory.
Open CMD as administrator
Execute this commands

cd C:\path\to\directory\
rdpwinst -u -k
rdpwinst -i

Enjoy!

Original message: https://t.me/rdpwrap/49131

I still got this error:

I wanted to try the above solution I even run the cmd as administrator but it shows access denied:

due to active antivirus, i was receiving the above error

I run this command: Net start termservice
And the remote service got unstable and kept stopping on its own
and this is the errors I see in the event log viewer:
The server {F9A874B6-F8A8-4D73-B5A8-AB610816828B} did not register with DCOM within the required timeout.
,
The Remote Desktop Services service terminated unexpectedly. It has done this 13 times (s).
My PC Windows version is: Windows 10 Home 22H2 19045.3570

i tried these steps too:
sc config TermService type= own

If you have antivirus software, try to add rdpwrap.dll to exclusions
Create file rdpwrap.txt in the root of system disk (for example C:\rdpwrap.txt)
Restart PC
Check contents of rdpwrap.txt log file

And this Is the content of rdpwrap.txt log file:

SvchostPushServiceGlobals
Loading configuration...
Configuration file: c:\program files\rdp wrapper\rdpwrap.ini
Initializing RDP Wrapper...
Base addr:  0x00007FF9EA440000
SvcMain:    termsrv.dll+0x0000000000038510
SvcGlobals: termsrv.dll+0x00000000000397A0
Version:    10.0.19041.3570
Freezing threads...
Patch CEnforcementCore::GetInstanceOfTSLicense
Patch CSessionArbitrationHelper::IsSingleSessionPerUserEnabled
Patch CDefPolicy::Query
Hook CSLQuery::Initialize
Resumimg threads...
<<< SvchostPushServiceGlobals
ServiceMain
<<< ServiceMain

rdpwrap.zip

Thanks for the file now the services got stable but still there is one problem:

The Solution to My Problem:
https://github.com/sebaxakerhtc/rdpwrap.ini/blob/master/rdpwrap.ini

The Solution to My Problem:
https://github.com/sebaxakerhtc/rdpwrap.ini/blob/master/rdpwrap.ini

OK!!!! Thanks bro, it worksfor me~

The Solution to My Problem:
https://github.com/sebaxakerhtc/rdpwrap.ini/blob/master/rdpwrap.ini

My nessage above...
https://github.com/stascorp/rdpwrap/issues/2359#issuecomment-1756993375

@sashaqwert    You’re right but the ini file just got updated yesterday and that works now.

My rdpwrap.ini is right, rdpConfig is normal.
But I can't be linked by two PC.

My rdpwrap.ini is right, rdpConfig is normal. But I can't be linked by two PC.

Im having the same problem.

Working!
@pheotry You should also modify this settings in gpedit
"Enable Multiple RDP Sessions
Log into the server, where the Remote Desktop Services are installed.
Open the start screen (press the Windows key) and type gpedit.msc and open it.
Go to Computer Configuration > Administrative Templates > Windows Components > Remote Desktop Services > Remote Desktop Session Host > Connections.
Set Restrict Remote Desktop Services user to a single Remote Desktop Services session to Disabled.
Double click Limit number of connections and set the RD Maximum Connections allowed to 999999."
Source: https://help.matrix42.com/020_ESM/80_MyWorkspace/10MyWorkspace_Guide/HowTo%3A_Enable_multiple_sessions_for_Remote_Desktop_Services

I have problems with Printers forwarding to remote session. Only some printers are forwarder (like PDF), and they are visible only at print dialog window.
How to fix it?

@AlexHighTower

I have problems with Printers forwarding to remote session. Only some printers are forwarder (like PDF), and they are visible only at print dialog window.\nHow to fix it?

See https://github.com/stascorp/rdpwrap/issues/504

See #504

Actually don't see solution (((
There is two suggestions:

set TerminalServices-DeviceRedirection-Licenses-TSEasyPrintAllowed to 0
set SLInitHook.x64 to 0

am I missed smth?

@AlexHighTower

set TerminalServices-DeviceRedirection-Licenses-TSEasyPrintAllowed to 0
set SLInitHook.x64 to 0

Also: install printer driver to server.

We've got some 19041.3570 Win10 Pros on HyperV virtual machines. Tried this RDPWrap with the .ini modification above, but is not working:

only ONE user can access the machines at a time, and
the printers are not connecting up to the remote machine at all.

(Wrapper state is installed, Service's running, Listener is listening and termsrv is fully supported - but it's really not working. Mode is NLA, Shadowing mode is Full with user's permission.)
What else can I try, if the .ini?
I also tried sebaxakerhtc's modification ( https://github.com/sebaxakerhtc/rdpwrap ): that's a little bit better, because multiple users can login to a virtual machine - but printers are also not visible on that.
(And that modification is a one-file installe without .ini, so ini modding is not possible. )
So I really need some working ideas...
Thanks for your help!

@find-a-good-nickname

What else can I try, if the .ini?

Try ask in project Telegram chat (link from repository README): https://t.me/rdpwrap

Does anyone know the search value for 10.0.19041.3570?

Does anyone know the search value for 10.0.19041.3570?

https://github.com/stascorp/rdpwrap/issues/2359#issuecomment-1758965897

I am currently on windows 10 home edition and I am unable to get the port to LISTEN.

I have edited the rdpwrap.ini file to the following.

The Solution to My Problem:
https://github.com/sebaxakerhtc/rdpwrap.ini/blob/master/rdpwrap.ini

I am sure I am missing something but I can't seem to find it.  Please do let me know if there are other information required.
Thank you!

Does anyone know the search value for 10.0.19041.3570?

#2359 (comment)

wrong offset
;SingleUserOffset.x64=11E12
SingleUserOffset.x64=182FB

.text:0000000180011E0E ; 28:   v2 = 0; .text:0000000180011E0E                 xor     edi, edi .text:0000000180011E10 ; 29:   *a2 = 1; .text:0000000180011E10                 mov     dword ptr [rdx], 1 .text:0000000180011E16 ; 30:   v3 = a2; .text:0000000180011E16                 cmp     cs:?bInitialized@CSLQuery@@0HA, edi ; int CSLQuery::bInitialized
Equivalent to setting the value of bInitialized to 1.
So it's wrong.

.text:0000000180011CFF                 mov     rax, [rsi+770h]
.text:0000000180011D06                 lea     rdx, [rbp+arg_0] ; int *
.text:0000000180011D0A                 mov     rcx, [rax+638h] ; this
.text:0000000180011D11                 call    ?IsSingleSessionPerUserEnabled@CSessionArbitrationHelper@@UEAAJPEAH@Z ; CSessionArbitrationHelper::IsSingleSessionPerUserEnabled(int *)
.text:0000000180011D16 ; 30:   v6 = v5;
.text:0000000180011D16                 mov     edi, eax
.text:0000000180011D18 ; 31:   if ( (v5 & 0x80000000) == 0 )
.text:0000000180011D18                 test    eax, eax
.text:0000000180011D1A                 jns     short loc_180011D39
.text:0000000180011D1C ; 53:     _DbgPrintMessage(
.text:0000000180011D1C ; 54:       8,
.text:0000000180011D1C ; 55:       "m_ptrDefaultHelper->IsSingleSessionPerUserEnabled failed: 0x%x in %s",
.text:0000000180011D1C ; 56:       v5,
.text:0000000180011D1C ; 57:       "CSessionArbitrationHelperMgr::IsSingleSessionPerUserEnabled",
.text:0000000180011D1C ; 58:       -2i64,
.text:0000000180011D1C ; 59:       v11,
.text:0000000180011D1C ; 60:       v12[0]);
.text:0000000180011D1C                 lea     r9, aCsessionarbitr_3 ; "CSessionArbitrationHelperMgr::IsSingleS"...
.text:0000000180011D23                 mov     r8d, eax
.text:0000000180011D26                 lea     rdx, aMPtrdefaulthel ; "m_ptrDefaultHelper->IsSingleSessionPerU"...
.text:0000000180011D2D                 mov     ecx, 8          ; int
.text:0000000180011D32                 call    ?_DbgPrintMessage@@YAXHPEBDZZ ; _DbgPrintMessage(int,char const *,...)
.text:0000000180011D37                 jmp     short loc_180011DAB

If mov dword ptr [rdx], 1 is changed to mov dword ptr [rdx], 0, then _DbgPrintMessage will be called, outputting an error

.text:00000001800182EF ; 31:   v3 = 0;
.text:00000001800182EF                 mov     ebx, edi
.text:00000001800182F1                 call    memset_0
.text:00000001800182F6 ; 33:   *v2 = 1;
.text:00000001800182F6                 lea     eax, [rdi+10h]
.text:00000001800182F9                 mov     dword ptr [rsi], 1  <=0
SingleUserOffset.x64=182FB
That's the right offset.


100% coverage

Current coverage is 100.00%

Merging #2 into master will increase coverage by +0.70% as of af35631

@@            master      #2   diff @@
======================================
  Files            1       1
  Stmts          288     287     -1
  Branches         0       0
  Methods          0       0
======================================
+ Hit            286     287     +1
  Partial          0       0
+ Missed           2       0     -2

Review entire Coverage Diff as of af35631

Powered by Codecov. Updated on successful CI builds.


fix(ios): currentPosition and duration may never equal during playing

Problem
When play a remote file on iOS, currentPosition may never get to duration, which makes it difficult to know whether the playing is end.
Solution
Add an observer to native player end event, and in that observer, force trigger another playback event, where currentPosition is set to be duration. Note that at this moment, playerItem.currentTime is still smaller than duration, so we cannot use that value.
may fix #328

same issue, was working fine till yesterday...


WinForms 6 Support

Discussed in https://github.com/ScottPlot/ScottPlot/discussions/1430
Investigate whether a separate .NET 6 package is required for each of the controls
https://github.com/ScottPlot/ScottPlot/blob/d55bfbfb35cbd31c9ce0edb97d06e1a000fcf9d9/src/controls/ScottPlot.WinForms/ScottPlot.WinForms.NUGET.csproj#L4

Works on my system! More discussion in #1430
https://user-images.githubusercontent.com/4165489/146845485-dccd6813-a3aa-427c-9784-1c8aefdfcafb.mp4

Following-up, I found that the ScottPlot.sln was not showing the FormsPlot in the toolbox. I'm still not totally sure why, but adding net6.0-windows as a TargetFrameworks for the control helped, and oddly it had to be first in the list of target frameworks. After making this change it appeared in the toolbox 🤷
  <PropertyGroup>
    <TargetFrameworks>net6.0-windows;net5.0-windows;net472</TargetFrameworks>
    <UseWindowsForms>true</UseWindowsForms>
  </PropertyGroup>

oddly it had to be first in the list of target frameworks.

Very weird, is this already known about? Might be worth reporting to Microsoft.

Might be worth reporting to Microsoft

I tried pretty hard to recreate this bug in a new solution created from scratch and I couldn't do it. I'm guessing it's a super niche corner case for complex solutions with mixed .NET framework and core projects... since I can't reproduce it, I probably won't report it 🤷

is this already known about?

I did find a few instances of posts about older versions of Visual Studio that noted something along the lines of, "when targeting multiple frameworks, Visual Studio designer just uses the first one in the list". It's probably a good rule of thumb to always order the newest target first... and hope people are using a modern version of Visual Studio 😅 #1500
I'm thinking of only targeting .NET 6 for the next major version of ScottPlot that drops System.Drawing dependency 😮 Multi-targeting has so many rough edges with regard to testing, building, packaging, and interacting with Visual Studio. I'd love to stop supporting msbuild, and be able to use the newer language features without too much hacking (records, nullable, spans, etc). I'm curious how much pushback we'd get.

I'm thinking of only targeting .NET 6 for the next major version of ScottPlot that drops System.Drawing dependency 😮

Personally, I would at least wait until a release hits end-of-life before deprecating it. When I was interning I saw most projects were on .NET Core 3.1 or earlier. No one wanted to upgrade to .NET 5 unless they were going to use the new features, as 3.1 was an LTS and 5 was not (this was several months before .NET 6).
Honestly, even if one waits after EOL I think it's a little soon to drop support. Microsoft only gives 3 years for LTS releases, which is pretty short, and that isn't accounting for some failures in backwards compatibility. Tracking EOL honestly sounds like hell anyways with all the mishmash of .NET Framework/Core/Standard, and the different implementations of .NET Standard all have different EOL dates, because why not.

Personally, I would at least wait until a release hits end-of-life before deprecating it

This could become a pretty important discussion topic worth considering carefully!
According to the Lifecycle FAQ .NET Framework through 4.6.1 will reach EOL soon (April 2022) but .NET Framework 4.8 is part of Windows 10 and its lifecycle policy will track its parent OS which is expected to EOL in 2025 😬
I don't love the idea, but maybe a version split would be acceptable, with updates limited to bugfixes and all development/features focused on the newer branch. It will be a huge burden lifted to drop MSBuild and the multi-targeting nuances. The switch gets a little more justification considering Microsoft is forcing our hand by deprecating cross-platform support for System.Drawing now.
An option still on the table is using another level of abstraction, and passing an IRenderer throughout all the existing code. I'm exploring this idea currently as a less-disruptive way to transition to Maui.Graphics (and leave the door open for a raw Skia renderer or something different in the future).
As of this moment, I'm probably leaning toward maintaining a .NET Framework/.NET 5 ScottPlot 4 that uses System.Drawing, and a .NET 6+ ScottPlot 5 that doesn't. I'm still trying to buy a little more time watching Maui.Graphics mature to see if that will make the decision more obvious in the future.

I think a version split has the potential to claw back a lot of the time saved from dropping multi-targeting. While leaving a lot of users without the latest features.


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-2.x" workflow

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

Questions ?
Please refer to the Backport tool documentation


Import order of customSchemaFiles not deterministic

Describe the bug
If one imports multiple schema files with the customSchemaFiles option, the schema files are imported in a non-deterministic order.
https://github.com/jp-gouin/helm-openldap/blob/6c1d9bf38aa27033e1d1ea54321bb1b6dd4dacb2/values.yaml#L122-L127
This is because the schema files end up in /opt/bitnami/openldap/etc/schema/
https://github.com/jp-gouin/helm-openldap/blob/6c1d9bf38aa27033e1d1ea54321bb1b6dd4dacb2/templates/statefulset.yaml#L212-L218
These files will be loaded by the non deterministic function ldap_add_schemas function: https://github.com/bitnami/containers/blob/main/bitnami/openldap/2.6/debian-11/rootfs/opt/bitnami/scripts/libopenldap.sh#L416-L422
Instead we should load these custom schema files by the deterministic ldap_add_custom_schemas function: https://github.com/bitnami/containers/blob/main/bitnami/openldap/2.6/debian-11/rootfs/opt/bitnami/scripts/libopenldap.sh#L450-L455
Expected behavior
custom ldap schemas passed into customSchemaFiles are loaded in a deterministic order, sorted by filename.
Additional context
I probably won't work on this for the next weeks. A possible workaround is to have only one schema file (with multiple schemas in it).

Hey @wkloucek, thanks for reporting it !
Feel free to submit a PR when you got time to work on it :)


UITableView's didSelectRowAt not called after swiping

Short description
There is a really annoying bug when the contentViewController of the FloatingPanelController has a UITableView. When you swipe up or down quickly so that the floating panel bounces a bit at the end of animation, the first tap on UITableView cell is ignored - the delegate method tableView:didSelectRowAt: is not called. It's being called the second time you tap on a cell or when you swipe slowly and then tap. There must be an issue with intercepting some gestures when UIScrollView decelerates but I didn't have much time to investigate properly.
Expected behavior
The tableView:didSelectRowAt: is called every time.
Actual behavior
The tableView:didSelectRowAt: is not called the first time after scrolling quickly.
Steps to reproduce
Can be easily reproduced using the Maps example by adding some logs to tableView:didSelectRowAt: to see they are not printed out after scrolling fast and tapping on a cell.
Code example that reproduces the issue
Environment
Library version
1.4.0
Installation method

[x] CocoaPods
[ ] Carthage
[ ] Git submodules

iOS version(s)
12
Xcode version
10.1

Hi, I have the same issue.
Did you find a solution ?

I acknowledged this issue. I will try to fix it.

How about fix-tap-abort branch(7a51219)?

Yes, this fixes the issue. Thanks.

Still same for me :/

I double checked that on the Maps example and looks like it's only partially fixed. It works now when you swipe up from tip to half and half to full but it's still broken when you swipe from full to half.
@RpX974 can you confirm that?

I'm using from tip to full, so I can confirm for that. I will try the other ways.

Yes, from tip to half, it works

Nevermind, it doesn't work too, I scrolled before when I tried from tip to half.

Ok, I found something. When I try to change the height constraint of a view in "floatingPanelDidChangePosition" for the tip position, I got the issue. But if I don't change the height, I don't have the issue.

This issue might be related to this bug ,https://bugs.swift.org/browse/SR-10257.
See also here, https://stackoverflow.com/questions/55592341/xcode-10-2-protocol-inheritance-issue.

@SCENEE I doubt that. This issue can be reproduced using Xcode 10.1. The bugs you're talking about are related to Xcode 10.2.

Thanks for your quick feedback, @peterstojanowski 😀
I struggled with this issue  for a while and at last I found a solution to make it better!
I know it doesn't defeat this issue completely, but eases the problem.
I pushed 9076ba8 commit. I'm pleased to check it out 👍

Hmm... the commit might not fixed it...

Hmm.... It's also working well partially....

I found out that Apple Maps has the same bug.

Thanks for your sharing, @congmn!
Then I think it's hard to fix this issue completely without modifying a tap gesture handling of UITableView.  So I'm going to merge #185 to master.

I have a suggestion to solve this problem.
You can put the button into the cells of the table.
I succeeded with this solution.

I merged a solution into master to ease this issue 👍 Thanks all 🎉
I closed this issue for now, but I would like to fix it as expected so welcome any ideas and mentions for this :relieved:.

I Still have this issue!


fixed open issue related to Morphing Card - action buttons do not app…

…ear correctly in circle and hexagon

Thanks @mr-joshi for fixing it


Allow importing public-key-only identities in web client

So you can still grant them rights easily, and verify messages.

Underlying code has been changed already, just need UI.


bug(anvil): eth_getCode fails with  BlockOutOfRangeError when Anvil started with --load-state

Component
Anvil
Have you ensured that all of these are up to date?

[ ] Foundry
[X] Foundryup

What version of Foundry are you on?
nightly-e90348416c3a831ab75bb43f6fa5f0a0be4106c4
What command(s) is the bug in?
No response
Operating System
Linux
Describe the bug
1- Start Anvil with --dump-state
2- Deploy a contract
3- Stop Anvil
4- Restart Anvil with --load-state
5- Query the contract with eth_getCode , contract address and 'latest' as the block. It returns ok
6- Query the contract with eth_getCode, contract address and a block number mined before load state
get an error like the one bellow :
{"jsonrpc":"2.0","id":1,"error":{"code":-32602,"message":"BlockOutOfRangeError: block height is 253 but requested was 16"}}

But If i try cast block 16 it returns a valid block.

This error occurs becauseload_state populates the db but not the in-memory historical state (backend.states) in the backend. backend.states is only populated with the most recent block or best_block.

@fmoura fix has been merged and should be released in the next nightly.
You can now preserve the historical states by using the --preserve-historical-states flag along with --dump-state


echo messages not showing in map console.

Need to check to see that the map-console tool is subscribed to, and reporting, all messages.

Fixed on develop


Defaulting to human readable leaves the user with no way to get the exact byte length

I run lsd 0.19.0 on Arch with ls aliased to lsd, I unaliased them to show the differences below.
I tried:
$ lsd --classic -la
drwxr-xr-x gbin gbin  70 B  Fri Mar  5 08:29:43 2021 .
drwxr-xr-x gbin gbin 536 B  Fri Mar  5 08:29:41 2021 ..
.rw-r--r-- gbin gbin 128 KB Thu Mar  4 09:00:51 2021 Mac_ROMs
.rw-r--r-- gbin gbin 128 KB Fri Mar  5 08:28:33 2021 Mac_ROMs_sans_magic
.rw-r--r-- gbin gbin 128 KB Thu Mar  4 09:01:27 2021 vMac.ROM

I would expect the same behavior as ls:
$ ls -la
total 388
drwxr-xr-x 1 gbin gbin     70 Mar  5 08:29 .
drwxr-xr-x 1 gbin gbin    536 Mar  5 08:29 ..
-rw-r--r-- 1 gbin gbin 131076 Mar  4 09:00 Mac_ROMs
-rw-r--r-- 1 gbin gbin 131072 Mar  5 08:28 Mac_ROMs_sans_magic
-rw-r--r-- 1 gbin gbin 131072 Mar  4 09:01 vMac.ROM

(this is a real example where the real size does matter! we have a 4 bytes difference).
I thought I could simply add both the "size_value" and size in the config, but no, the size_value is the bare human readable value with no unit (IMHO it makes no sense here neither as size value will never get parsed as it is inconsistent and you cannot get the unit anyway).
S lsd -la
drwxr-xr-x gbin gbin 70   70 B  Fri Mar  5 08:29:43 2021  .
drwxr-xr-x gbin gbin 536 536 B  Fri Mar  5 08:29:41 2021  ..
.rw-r--r-- gbin gbin 128 128 KB Thu Mar  4 09:00:51 2021  Mac_ROMs
.rw-r--r-- gbin gbin 128 128 KB Fri Mar  5 08:28:33 2021  Mac_ROMs_sans_magic
.rw-r--r-- gbin gbin 128 128 KB Thu Mar  4 09:01:27 2021  vMac.ROM

I found this afterwards:
lsd --size bytes

But I would argue that --classic should still revert completely to the standard ls behavior and size_value should always stay as bytes and not randomly change unit. What do you think?

@gbin Makes sense. We can make --classic change size to bytes.


integrate two more Yocto SDKs into the imx docker image

Two more Yocto SDKs are installed into the imx docker image.

Is there an ETA for deprecating the old version?


Make working with CustomEvent more lean and robust

We have a lot of repeation and verbosity around the creation of custom events, e.g.
new CustomEvent("this-and-that", {
   detail: { someInfo },
   bubbles: true,
   composed: true,
})

My current stance is that it makes a lot of sense to rely on events for control flow in custom components. Therefore it would be cool if it was easier and more robust to emit them. The event configuration is mostly the same (bubbles and composed).

On the event handling side there might also be room for simplification: each dialog has its own event name this-and-that-failure but they are all registered individually. So we could either have a generic operation-failed event, or we keep the individual events but bulk-register them.

Fixed through https://github.com/mtlynch/tinypilot/pull/711 and https://github.com/mtlynch/tinypilot/pull/712.


How to serialize java class generated by protoc.exe using protostuff?

How to serialize java class generated by protoc.exe using protostuff?

For example, in the code below, User is a class generated by protoc.exe, and use the original serialization method to gen bytes rs1

User.Demo.Builder builder = User.Demo.newBuilder();
builder.setId("1").setAge(20).setName("mike").build();
User.Demo userDemo = builder.build();
byte[] rs1 = userDemo.toByteArray();

, but if use protoStuff:
byte[] rs2 = ProtostuffIOUtil.toByteArray(demo
   , RuntimeSchema.getSchema(User.Demo.Class)
   , buffer)

the correct result cannot be obtained (rs2  is longer than rs1 and Chinese is not displayed normally when deserialized), is there any way for protostuff to allow me to serialize User.demo and get result equals to rs1?

or should i  give up and just let these two serialization methods exist in the project?

It will need extra codegen via java_v2protoc_schema
See https://protostuff.github.io/documentation/compiler-options/

Thanks！


XSS vector in the Markdown processor

It's possible to inject javascript: URLs in link targets to execute arbitrary client side scripts. All parsed URLs should be restricted to a white listed set of URI schemas to avoid this.
https://www.owasp.org/index.php/XSS_Filter_Evasion_Cheat_Sheet#Image_XSS_using_the_JavaScript_directive

Note that there is already MarkdownSettings.urlFilter that can be used by application code to avoid this, but this still needs to be safe by default.


Is there a way to generate RSS feed for blog posts?

Will be working on that next week as I got many requests regarding this matter.

Great stuff. I'm currently converting my personal website and blog to katana. I'll send you a link if you want for reference.

Would be interested to check it it out :+1: thanks :)

It's online now. Thank you for this great tool @themsaid
http://mariobasic.com/

Woo-Hooo :)
Looking good

What i've done is create a normal blade file which sorts the blogs in reverse order of date and shows the latest 10 https://gist.github.com/dailysleaze/13897eff30e54ee787ca3406779ddf59

@dailysleaze but it'll be converted to an html file instead of .xml then.

True, but it the browser seems to detect it as a feed if it's using the content rather than the mime type. Agree it's not perfect.


Restore of restic backups does not work

What steps did you take and what happened:
If I follow the restic guide at https://github.com/heptio/ark/blob/master/docs/restic.md the data of the volume is successfully backed up, but never restored.
The ark backup is created successfully, the PodVolumeBackupResource is successfully created, and in S3 you can also see the created restic backups in the restic bucket.
If you download the backup though, the snapshot.ark.heptio.com/ annotation that should be added by the backup to the pod metadata is missing.
By adding further logging, I can see that the SetPodSnapshotAnnotation method is definitely called with a valid snapshot id, but the itemBytes that are written to the tar archive (https://github.com/heptio/ark/blob/1c26fbde320a2f192d712af8710bfac3d6f4e4f3/pkg/backup/item_backupper.go#L258), do not contain this annotation, which as far as I can see results in ark, never trying to restore the volume with restic.
The bug was likely introduced with https://github.com/heptio/ark/commit/11c176c490bbe00d21a2260f96192a3a1af21388 and the updated metadata is never written back to the obj that is persisted.
What did you expect to happen:
When you restore a backup with restic the data of the volume should be restored.
Environment:

Ark version (use ark version):  tested in 0.9.3 and master
Kubernetes version (use kubectl version): 1.11.2
Cloud provider or hardware configuration: OpenStack with Quobyte S3
OS (e.g. from /etc/os-release): Ubuntu 16.04

@bashofmann thanks for the report. This is a duplicate of #822, and we have a fix pending in #834.

Awesome, thanks and sorry for the duplicated report.


13.2.Searching in matrix (2nd approach (BS))

My code is clear and well understanding
Please assign it to me

Need to add a description and your approach. Thanks, Closing as of now.


Update to monaco-vscode-api 3.2.1

Using the standalone packages from monaco-editor works. Transformation of workers to esm bundles also works fine. Loading the workers directly via the factory function does not. I don't understand yet why. Editor worker is no problem.

What is the errors with the workers?

First, one guy called Kai is unable to type properly. Then vite gets confused if you specify this (js extension):
typescript: () => new Worker(new<EMAIL_ADDRESS>import.meta.url), { type: 'module' })

instead of this:
typescript: () => new Worker(new URL('@codingame/monaco-vscode-standalone-typescript-language-features/worker', import.meta.url), { type: 'module' })

@CGNonofr first, one guy called Kai is unable to type properly. Then vite gets confused if you specify this (js extension):
typescript: () => new Worker(new<EMAIL_ADDRESS>import.meta.url), { type: 'module' })

instead of this:
typescript: () => new Worker(new URL('@codingame/monaco-vscode-standalone-typescript-language-features/worker', import.meta.url), { type: 'module' })

Yep, that's because as soon as you have an exports field in your package.json, you can't import any other file directly. It's not really a vite specificity. What is specific to vite is that if the url of the worker can't be resolved, it just gives up transforming it without errors.
So everything is fine now?

What is specific to vite is that if the url of the worker can't be resolved, it just gives up transforming it without errors.

It says the file is not there and that is just not true. It has no visibility due to the extra export. I like that you did that actually.

So everything is fine now?

Yes

@CGNonofr I disabled the yarn verification. It is making trouble all the time because it can't resolve local dependencies.

@CGNonofr this is now really ready.


Update to latest Swift, add fullscreen support

Duplicate of #97. I screwed up the git history over there.
That is, it can now float over fullscreen apps.
Most of the Swift updates came automagically from Xcode; I hope they don't clobber any backwards compatibility.
Here's a cute animal picture for your trouble:

I'm a little hesitant to make the app automatically visible on all spaces. Is there any way to allow it on top of a fullscreen app without making it space-independent?

If not, I think I'd like this "show on all spaces" to be a preference in the menu bar rather than the new default behavior.

@JadenGeller I think the purpose of the app dictates that it should be space-independent, no? I'm disappointed this didn't make it into the latest release.

I'm totally happy to add a setting if it can work, but I won't get to that for a week or so—a bit slammed at the moment.
In any case, I'd rather share my changes with upstream than not, just in case you want 'em!

@net I don't think that's necessarily true. Personally, it really frustrates me when windows follow me from space to space.
@wlonk When you have a chance, sure! It might even be reasonable to make it the default behavior—not totally sure though. I'd have to think about it more. I noticed some weird graphical glitches when you invoke expose while the app is hovering over a fullscreen app. Not a big deal, but a bit annoying.

@wlonk I went ahead and added it myself, hope that's ok. I'm not very familiar with swift so my code might not be great. #108

Woot, nice!

@net Looks great, thanks!


Fixed missing "CONFIG_" perfix to enable LWM2M_FIRMWARE_UPDATE_OBJ_SUPPORT_MULTIPLE

Fixed missing '_CONFIG' prefix when enabling LWM2M_FIRMWARE_UPDATE_OBJ_SUPPORT_MULTIPLE and CONFIG_LWM2M_FIRMWARE_UPDATE_OBJ_INSTANCE_COUNT through prj.conf
Signed-off-by: henbro brom<EMAIL_ADDRESS>

Sorry this is my first contribution and I am not authorized, @rlubos could you merge this PR?

Sorry this is my first contribution and I am not authorized, @rlubos could you merge this PR?

No, I don't have merge permissions, it just needs to wait for a maintainer.


Allow point style to be overridden for Legend points

Feature Proposal
I've got a chart I'm working on where the designer specifically wanted a style point type that looks good in the chart but they don't like how it looks in the legend and the tooltip.
The tooltip has a labelPointStyle callback that allows you to override the point styles (though there is a bug there) but there's no matching override for the legend.
Possible Implementation
Possibly implement this the same way as it's implemented in the tooltips? Though hopefully with the needed fixes applied.

@methnen as workaround, I think  you could leverage on generateLabels option of legend.labels (even if it doesn't work like the tooltip callback).
Using jsfiddle of the other issue:
"legend": {
  "display": true,
  "position": "bottom",
  "labels": {
    "font": {
      "weight": "bold"
    },
    "usePointStyle": true,
    "generateLabels": function(chart) {
      // gets items by default generateLabels
      const items = Chart.defaults.plugins.legend.labels.generateLabels.call(chart, chart);
      items.forEach(function(item) {
        item.pointStyle = 'triangle';
      });
      return items;
    }
  }
},

Hmm... I get the idea, but this line doesn't see to return what you're implying:
const items = Chart.defaults.plugins.legend.labels.generateLabels.call(chart, chart);

All I get back is the entire chart object.

Nevermind, I was calling it incorrectly. I think I might be able to make this work.

OK, this works, I needed to add it inside of an inline plugin in my use case so I attached it to beforeLayout and it runs early enough. So definitely a viable work around but not idea of course.
Thanks a ton for the help on this and the other point styling item. Really appreciate it.

Closing since this seems to be resolved


Figure out a way to get version to read from package.json

It's rather silly to have to update the version in the footer every time I push a new one...

Fixed in 0.8.5


curl example does not work

In the "Base URLs" example:
    // Pass secret key and region using headers to a custom endpoint
    curl -X POST " my-ch-n.cognitiveservices.azure.com/translator/text/v3.0/translate?to=fr" \
    -H "Ocp-Apim-Subscription-Key: xxx" \
    -H "Ocp-Apim-Subscription-Region: switzerlandnorth" \
    -H "Content-Type: application/json" \
    -d "[{'Text':'Hello'}]" -v

curl fails due to a malformed url  (" " prefix)
there is no https prefix and curl defaults to http
The URL doesn't match the section text which says the base URL should be "api.cognitive.microsofttranslator.com" or similar
The example text suggests this is an example to force regional use of Switzerland, it might be better in the "Authenticating with a regional resource" section which covers regional resources, rather than the "Base URLs" section?

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

ID: 69823924-7def-6548-4f85-4ab9b8075d46
Version Independent ID: 9593dc98-656c-a73a-cbe6-ce201e497b11
Content: Translator V3.0 Reference - Azure Cognitive Services
Content Source: articles/cognitive-services/Translator/reference/v3-0-reference.md
Service: cognitive-services
Sub-service: translator-text
GitHub Login: @laujan
Microsoft Alias: lajanuar

@chrbsg
Thanks for your feedback! We will investigate and update as appropriate.

@chrbsg
Thanks for reporting this! We have created a PR for this issue and the changes should go live soon.

#sign-off


feat: use fetch-cookie package for cookie based sticky sessions

Motivation and Resolution
The PR adds cookie handling to allow cookie based sticky sessions. It is using a fork of fetch-cookie as the PR is not merged yet.
RPC version (if applicable)
Usage related changes

The change allows to use cookie based sticky sessions

Development related changes

Checklist:

[x] Performed a self-review of the code
[x] Rebased to the last commit of the target branch (or merged it into my branch)
[x] Linked the issues which this PR resolves
[x] Documented the changes in code (API docs will be generated automatically)
[x] Updated the tests
[x] All tests are passing

Looks good to me. Let us know if this can wait for the linked PR or if it makes sense to merge sooner.
As an aside, I edited this PR to target develop since the v6 beta has been merged and made into a pre-release.

Reply from Elliot, I assume @piwonskp's boss :)
"I think we should not wait for the linked PR because it could easily take many months at that repo's current pace."


Compress-Archive skips files

Be carefull when using Compress-Archive
It skips hidden files on most powershell build, so the complete folder .git is skipped (and the history is lost)
https://github.com/PowerShell/Microsoft.PowerShell.Archive/issues/154

Done with #5


polish tralsnation - misspellings

Thank you!
Dou you want to stay on the translation list on the overview?


Formalize Semantics

Formalize semantics.  The IR model is simple enough that it should easily permit a formal syntax and semantics.

off-topic


switch back to PLAYER mode

if nothing was pressed after 10 seconds in the SETTINGS mode

in fact the switch is to the LIBRARY mode -> if nothing was pressed after 10 seconds in mode SETTINGS switch back to LIBRARY mode


tracklets icon label act as a rectangle

according to  xviz and streetscape.gl,the tracklets icon sometimes act as a white rectangle,do not show the original font icon

It may be that the object's label is changed or unknown.  In order to debug this you would have to look at the state at the timestamp when this occurs and verify the object's state.
Ideally we would have a widget that allows you to see such data, but we haven't built that one yet :(


Add libgdiplus

I checked the list of things that depend on SpriteText and there's quite a few, so until that's sorted out, bundle libgdiplus. This makes the game runnable from Steam as-is on Linux without having to install any additional stuff.
Cherry-pick this into the current Steam build too. I tested putting it into the game directory on clean Ubuntu 19.10 with a clean Steam Quaver install, and it made the game run, whereas without it it doesn't run.

 Turns out it tries loading libgdiplus.so with dlopen, and it tries to load that from libdl.so (the development library) and not libdl.so.2 (the actual non-development library). So that's where the libdl.so dependency has been coming from. And you really don't want want to bundle it either because it's a low-level system library.
Anyway, I found that the development libdl.so is installed out of the box on Ubuntu 19.10 and Fedora 31 and other distros, so that's less of an issue. Libgdiplus.so is the main offender. Just good to know that getting rid of libgdiplus will solve both of these unnecessary dependencies.

Hmm, this new version makes the text look kinda bad though. I think it needs this line removed:
https://github.com/Quaver/Wobble/blob/2a374890564836620e56e9588dedf0aece5e228e/Wobble/Graphics/Sprites/SpriteText.cs#L123

Alright, looks good with that removed.


Not working with IOS 13.4.1 on Iphone XS

Hello,
The Unc0ver not working with IOS 13.4.1 on Iphone XS.
Is an update planned?
Thank you.

@Dge-06 we don't know. There needs to be an exploit which there is not yet.

Hello,
Thank you for the answer. Yet I understand that unc0ver works on iphone xs?
French article:
https://iphoneaddict.fr/post/news-282425-jailbreak-iphone-11-xs-xr-unc0ver-mis-jour-corriger-premiers-bugs
https://infoidevice.fr/unc0ver-3-7-0-disponible-jailbreak-a12x-iphone-xs-xs-max-xr-ipad-pro/
We talk about it everywhere ...

Yes it works on all devices 11.0-13.3. 13.3.1 to 13.4.1 aren't supported because there is no exploit, its nothing to do with the device.

Ok I understand better, thank you.


Adding a StringShape and enum attribute

This commit updates the shape builders to utilize a new concrete shape
type, StringShape. StringShape exposes a property that makes it easier
to inspect the enum attribute of a string shape. StringShape is the only
concrete shape type to expose this property because, at present, the only
shapes that have the enum property are string shapes.
In addition to this feature, the unused SHAPE_CLASSES constant has been
deleted from ServiceModel. If removing this constant is not acceptable, I can
put it back and rebase.

:shipit:  Looks good to me.  I think it's safe to remove the SHAPE_CLASSES class attr.  I think that's just an artifact from a previous refactoring where that attr wasn't removed from the old location after being moved to a different class.


wxwidgets fails to build on OSX

Issue description
Here's the output that I get:
In file included from ./include/wx/osx/private.h:4:
./include/wx/osx/core/private.h:55:10: fatal error: 'Carbon/Carbon.h' file not found
#include <Carbon/Carbon.h>
         ^
4 warnings and 1 error generated.
Makefile:28754: recipe for target 'basedll_filefn.o' failed
make: *** [basedll_filefn.o] Error 1
builder for ‘/nix/store/ab4il1siki6x8vbr8lmjr0mp472674bi-wxwidgets-3.0.2.drv’ failed with exit code 2

Steps to reproduce
E.g. nix-env -i wxmaxima.
Technical details

System: OS X 10.11.6 Beta (15G7b)
Nix version: nix-env (Nix) 1.11.2
Nixpkgs version: "16.09pre83147.df89584"

Do you know whether wxPython 3.0 functioned before #15764 ?

It seems like it should just be a matter of adding some dependencies for Darwin/OSX.
@copumpkin were you using Nix on Darwin?

@FRidh I don't, unfortunately. I've only tried this today, for the first time.

Same error here:

System: OS X 10.9.5
Nix version: nix-env (Nix) 1.11.2
Nixpkgs version: "16.09pre-git"

Pulled from github.com/nixpkgs this morning.

I guess darwin.Carbon needs to be added

@FRidh Dunno.  @matthewbauer pull request says it fixes this issue; I'm trying that now.

Seems it will use Cocoa now https://github.com/NixOS/nixpkgs/pull/16192/files#diff-820eed189b664d8e60bb47d4055b0189R32

Had wxmac installed and got this error message after building @matthewbauer's patch:
collision between ‘/nix/store/v937kg1xdjjdnr9pk2hn79jvqs9d12hk-wxwidgets-3.1.0/bin/wx-config’ and ‘/nix/store/8abyzj37rfxk82bdbmq48zsqsyc8in8k-wxmac-3.0.2/bin/wx-config’; use ‘nix-env --set-flag priority NUMBER PKGNAME’ to change the priority of one of the conflicting packages
builder for ‘/nix/store/gw8cigzlbmyj56a91hmvsvqiim8y2bs1-user-environment.drv’ failed with exit code 255
error: build of ‘/nix/store/gw8cigzlbmyj56a91hmvsvqiim8y2bs1-user-environment.drv’ failed

uninstalling wxmac worked
~/src/nixpkgs$ nix-env --uninstall wxmac
uninstalling ‘wxmac-3.0.2’
~/src/nixpkgs$ nix-env -i wxwidgets-3.1.0
warning: there are multiple derivations named ‘wxwidgets-3.1.0’; using the first one
installing ‘wxwidgets-3.1.0’
building path(s) ‘/nix/store/nsmdkv3bl1cias1y9qdg2dxl17fxryqc-user-environment’
created 513 symlinks in user environment
~/src/nixpkgs$

This still fails when building 2.8 (default version) due to wxGTK being unable to find specific framework versions (10.6) of Carbon. The default go-to is to download the 10.6 Carbon version and pass the argument to the configure script. How should we go about fixing this?
I have a PR-ready version of the package which pulls in all appropriate frameworks (Carbon, Cocoa, CoreServices).

(triage) Steps to reproduce from this ticket doesn't fail currently (probably because it uses wxMac30 now). Is there still a problem?

Looks like we can close this.

Yes, seems OK on Hydra.


Show Policy Audit Violations

We need to have the option in the violations tab in clusters and the new violations tab in policy to show the audit violations if they are published to Kubernetes events.
Story Description
As a user, I need to view audit violations in the UI.
Acceptance Criteria [WIP]
We might want to have a toggle in the violation tables to show audit/enforce violations. Another option is to have the table originally filtered to enforce violations, and let the user use the current filtering mechanism.

Duplicate of https://github.com/weaveworks/weave-gitops-enterprise/issues/2851


Documentation

In the web documentation in the section "Get started> Third party options> Ember" it is specified that "ember-cli-materialize" can be used, however this library only supports the 0.97.0 version of materialize. Remove this recommendation to use version 1.0.0 or even version 0.100.0 since this library is deprecated and has no maintenance by its creator.

+1, this is a good suggestion and I think it's easy to correct
Can you change the title of the issue to something more descriptive (like "remove the (very deprecated) ember package from the getting started documentation page" ?)

Anyone who wants to rock this one? @materializecss/members


Cudnn conv dgrad algo filtering

Description
I've learned that for cudnn versions in the range [7.3.1,7.5.0), the Convolution dgrad algorithm 3 (CUDNN_CONVOLUTION_BWD_DATA_ALGO_FFT_TILING) may produce incorrect results for some strided convolutions.  This is not something that would generally appear in the current CI, which builds against cuda9.1 and cudnn7.1(so before the problem was introduced).
I've created a test to expose the problem, as well as a fix.  I've noticed that there is a CI build for tensorrt that uses cuda10.0 and cudnn7.4, which should exhibit the problem.  Thus, the plan is:

Do an initial PR submission that includes the problem-exposing test, and with a temporary addition of that test to the tensorrt CI test suite.
If hopefully this demonstrates the problem, then push the fix (which involves filtering-out algo 3 from the results of cudnnFind when necessary).
When the fix is shown to work, remove the explicit running of the test from the tensorrt CI, allowing the normal running of the test as part of test_operator_gpu.py to catch any return of the problem.

Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[ ] Changes are complete (i.e. I finished coding on this PR)
[X] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[X] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[X] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

@mxnet-label-bot add [pr-awaiting-review]

Finished first step of this PR: 'Do an initial PR submission that includes the problem-exposing test, and with a temporary addition of that test to the tensorrt CI test suite.'  CI passed, except for the one newly-introduced test (as expected):
======================================================================

FAIL: test_operator_gpu.test_conv_deconv_guards

----------------------------------------------------------------------

Traceback (most recent call last):

  File "/usr/local/lib/python3.6/dist-packages/nose/case.py", line 198, in runTest

    self.test(*self.arg)

  File "/work/mxnet/tests/python/gpu/../unittest/common.py", line 173, in test_new

    orig_test(*args, **kwargs)

  File "/work/mxnet/tests/python/gpu/test_operator_gpu.py", line 543, in test_conv_deconv_guards

    check_consistency([sym, sym_no_cudnn], [ctx, ctx], tol=tol)

  File "/work/mxnet/python/mxnet/test_utils.py", line 1379, in check_consistency

    raise e

  File "/work/mxnet/python/mxnet/test_utils.py", line 1374, in check_consistency

    equal_nan=equal_nan)

  File "/work/mxnet/python/mxnet/test_utils.py", line 495, in assert_almost_equal

    raise AssertionError(msg)

AssertionError:

Items are not equal:

Error 19129.703125 exceeds tolerance rtol=0.100000, atol=0.100000.  Location of maximum error:(1, 21, 49, 24), a=-2059.578857, b=-0.076599

 a: array([[[[ -200.87648   ,   365.06775   ,   -56.88517   , ...,

            100.96652   ,  -134.89548   ,   156.62936   ],

         [ -239.07794   ,   498.35898   ,   138.86926   , ...,...

 b: array([[[[ -200.87619   ,   365.0675    ,   -56.885128  , ...,

            100.96669   ,  -134.89523   ,   156.62917   ],

         [ -239.07797   ,   498.35883   ,   138.86902   , ...,...

-------------------- >> begin captured logging << --------------------

common: INFO: Setting module np/mx/python random seeds, use MXNET_MODULE_SEED=1727291009 to reproduce.

common: INFO: Setting test np/mx/python random seeds, use MXNET_TEST_SEED=781016024 to reproduce.

``

I've now finished the second step of the PR: 'push the fix (which involves filtering-out algo 3 from the results of cudnnFind when necessary).'  From the CI logs for the tensorrt build (the only one that previously failed due to its use of cudnn 7.4), now we have:
----------------------------------------------------------------------

XML: /work/mxnet/nosetests_trt_gpu.xml

[success] 100.00% test_operator_gpu.test_conv_deconv_guards: 21.4772s

----------------------------------------------------------------------

Ran 1 test in 21.479s

OK

This PR is in a good state to review.  I hope you guys like the test-driven-development ;-)
@szha @eric-haibin-lin @marcoabreu

This PR as been ready to go for a week.  Should I be concerned there are no reviewers/assignees yet?  @eric-haibin-lin @szha
The idea with this is to protect MXNet users from issues present in more advanced cudnn versions than what is used in the standard MXNet builds and CI.

Still looking for reviewers of this fairly small PR. @marcoabreu @larroy ?

I have pinged a few reviewers, you should get a response shortly. Please excuse the delay

Yes, the test case was designed to trigger the failure (see earlier post).  With the fix in place, the algo_exclusion==True path is activated to ensure the problematic algo is bypassed.


try db.run() inside an UIAlertController

Hello,
After update to swift2 the following code I am struggling with:
alert.addAction(UIAlertAction(title: "OK", style: UIAlertActionStyle.Default, handler: {(alert:UIAlertAction) in
// get user input
let TF = (inputTextField! as UITextField).text
// make dbase query for note name
let rcd = users.filter(Expression("_id") == self.RecordNumber)
// put TF in record note
if !TF!.isEmpty { // OK is clicked
try db.run(rcd.update(Expression("RM") <- TF!)) {}catch{}
// rcd.update(Expression("RM") <- TF!) // previous working code
}
} ))
This code update the dbase if OK is clicked, the rcd.update worked well however the new structure (try db.run) It gives an error in the handler of the alert and I have no idea how to solve it. Can anyone help me?
Much regards,
Erik

It looks like you aren't wrapping the try in a do block (required by Swift). Check out the error handling documentation here:
https://developer.apple.com/library/prerelease/ios/documentation/Swift/Conceptual/Swift_Programming_Language/ErrorHandling.html

Thanks very much, that was exactly the problem!


Bso Herblore Master Cape Issue

🐛 Bug Report
Write a description of the bug here, give all the information you can about what it is, how it happened.
Hello Dearest People,
When mixing potions on BSO with the herblore master cape, you currently receive extra potions.
However, this regretfully stops the "press Y to repeat" action from working.
Additionally, you cannot do "=m" then click the "repeat herblore trip" button as then it states you cannot go over X time period.
So now you have to type the whole command in to repeat, alot slower with slash commands.
Could the Button or repeat instinctively make the maximum a trip takes and not what you received back.
Eg when you get double loots this doesnt effect it.
Thank you,
Kindest Regards,
Robson

Fixed in next update


[Bug] To load the App on localhost:3000/OHIF rather that localhost:3000

Describe the Bug
I want to load the UI of a different path then the default one
I want to load it on http://localhost:3000/ohif rather than just http://localhost:3000 but I cannot se anywhere in the file to customize this can anyone help over here?
Steps to Reproduce
NA
The current behavior
The app loads on http://localhost:3000
The expected behavior
I want app to be loaded on http://localhost:3000/ohif
OS
Windows 11
Node version
16.20.2
Browser
Chrome

https://docs.ohif.org/deployment/build-for-production
please refer here on how to build for Build for non-root path

Extract from the doc:

If you would like to access the viewer from a non-root path (e.g., /my-awesome-viewer instead of /), You can achieve so by using the PUBLIC_URL environment variable AND the routerBasename configuration option.

use a config (e.g. config/myConfig.js) file that is using the routerBasename of your choice /my-awesome-viewer (note there is only one / - it is not /my-awesome-viewer/).
build the viewer with PUBLIC_URL=/my-awesome-viewer/ APP_CONFIG=config/myConfig.js yarn build (note there are two / - it is not /my-awesome-viewer).

Basically, you can try to change the value here:

please don't use issue in the repo for asking questions. There are other channels available for that. Thanks


Sync from Init suppresses error

In practice (and in documentation) no one checks the error returned from liblog.Sync(). Searching sourcegraph repos for log.Sync() has 22 results, all of which ignore the returned error.
This commit removes the error return to make this explicit. Additionally this will remove the "errcheck" linter complaining about all these call sites.
Note: When updating sourcegraph there is one callsite which explicitly ignores the error return which will need updating. https://github.com/sourcegraph/sourcegraph/blob/93bc1093d7169bba30ba6387ae4bb30cd2171f55/dev/sg/main.go#L210
Test Plan: go test

Actually there is one callsite which does explicitly check the return value, but I reckon this can also just ignore it https://github.com/sourcegraph/sourcegraph/blob/93bc1093d7169bba30ba6387ae4bb30cd2171f55/enterprise/internal/authz/perforce/cmd/scanprotects/main.go#L37-L41

How did we miss all these linter complaints 🤔


iree/runtime library cannot support bare-metal thread-less fileIO-less configuration

iree/runtime library needs some clean up to support bare-metal configuration

[x] Split up the dependency of iree/hal/driver (task executor --> thread dependency).
[ ] Split up fileIO depenency.

We should have a subset of APIs that can support bare-metal config and a full APIs to support more sane systems.

What do you think about an iree/base/config.h define for whether to enable file IO (IREE_FILE_IO_ENABLE)? This could be used to disable iree_runtime_session_append_bytecode_module_from_file while keeping iree_runtime_session_append_bytecode_module_from_memory for example.

That could be helpful, thanks! However, we need to create customized CMakeLists.txt instead of the one generated from BUILD.
In addition, I am not sure we have fixed the thread dependency for iree/hal/driver yet. It would require https://github.com/google/iree/issues/4298 to be solved first.

we need to create customized CMakeLists.txt instead of the one generated from BUILD.
what do you mean?

I am thinking about https://github.com/google/iree/blob/331946c/iree/runtime/CMakeLists.txt#L46 and how we can add it only when if(${IREE_ENABLE_FILE_IO}) in CMakeLists.txt.

Ah - we would just make the file_io code no-op (like how we make the atomics and mutex stuff no-op) - file_io only relies on stdio, so it pulls in no deps you would otherwise not want


Double  insertion on GitHub Enterprise

It inserts the  tag twice on GitHub Enterprise :(
No issue on github.com
Version: 23.11.6

Thanks for reporting! This is a tough one as I don't have access to GitHub Enterprise 😅
Just to confirm a couple of things:

Is it inserting the same string twice? e.g.

GitHub recently updated their comment box, do you have this UI in enterprise? e.g.

You only have one version of the extension installed (check in chrome://extensions)? In the past I have seen this bug occur if you have the version on the Chrome Webstore as well as the extension locally

Interestingly, since reporting this and then re-checking today it have stopped doing the double inserts. I did install a updated version of Edge in the mean time, so perhaps that was the culprit.

Yes, this is exactly what happend
No, I only have one version installed
Yep, this is the enterprise UI

But since this now no longer reproduces, I'll close the ticket again.
Thanks for a great and fun browser extension :-)

Thank you for following up. Please let me know if you have any issues with the extension going forward.


fix: stop timeout timer immediately when unused

Timers created with time.After() are recovered only after it fires.
Therefore, it is not recommended to be used inside loops or when efficiency is a concern.
@swoga
Note that my main branch includes both this timeout-cleanup (PR #6) and atomic-tag (PR #5).

Thanks for your PRs, I missed the first one entirely.
Good finds, will merge them later then!

This changes the behavior of the timeout. Currently the timeout is reset after each sentence is received, which is needed so that commands with slow output are not aborted.

Hi, as far as I know. My change has the same behavior regarding per-sentence-timeout. New timer is created in every loop iteration and thus new deadline (timeout is reset).
The differences are:

After sentence is received, the timer is immediately stopped to allow GC to recover timer-related resource.
If timeout duration is 0 or less, timeout channel is nil which means blocking/waiting forever.

Sorry you are right, it was Monday morning and I misread the diff.

Thanks for your PRs!


Startup issue Collectd and Freeradius

@fichtner You did some follow-up changes to the rc.conf.d files, but I'm not sure how they behave.
I tried a fresh install of 17.7RC and collectd, but the service doesn't start when enabling it. It only starts after a reboot of the machine. When enabling the service and pressing start there's no error in browser debug mode and CLI, status returning ist "unknown". Also the rc.conf.d file is colloectd_enable="no".
I checked the setup script and it seems fine, but not executeable. Then I checked with Freeradius because you did the same changes there. The service starts fine after installtion, no reboot required. I also wanted to check how the setup script looks like, but it's not there?!
Now I'm confused :)

Ah, the bootup log show a permission denied for collectd setup.sh and a file not found for freeradius setup.sh

should be fixed on master now, sorry.

Great! What about the permissions for collectd file?

;)
I don't have time to confirm, so please close if really solved and sorry about the trouble.

Hm, when I look at the freeradius commit I wont see collectd :)
Ok, so for collectd it's fixed but for freeradius I get a:

:D

ok, found the issue

All errors gone, thanks :)

collectd doesn't reload the template on save

Really?
Jul 24 12:38:39 OPNsense configd.py: [28d76e73-5c2c-4929-b55c-9e438638f569] generate template OPNsense/collectd
Jul 24 12:38:39 OPNsense configd.py: [8b67e5c1-f284-47bf-8831-b14a72433a34] starting Collectd
Jul 24 12:38:39 OPNsense configd.py: [af61973f-cf32-4e53-9cad-d4f5b608424f] request Collectd status

Have you tested with a clean config.xml? I hit enable, save, it saves but does not rewrite the template so the service won't start as well.

On 24. Jul 2017, at 12:40, Michael<EMAIL_ADDRESS>wrote:
Really?
Jul 24 12:38:39 OPNsense configd.py: [28d76e73-5c2c-4929-b55c-9e438638f569] generate template OPNsense/collectd
Jul 24 12:38:39 OPNsense configd.py: [8b67e5c1-f284-47bf-8831-b14a72433a34] starting Collectd
Jul 24 12:38:39 OPNsense configd.py: [af61973f-cf32-4e53-9cad-d4f5b608424f] request Collectd status
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.

.. cloning a fresh clone ...

https://github.com/opnsense/plugins/blob/master/net-mgmt/collectd/src/opnsense/mvc/app/controllers/OPNsense/Collectd/Api/ServiceController.php#L136
Capital C I think ...

my bad then, once again. will fix in a sec :)

No, my bad using Github Desktop for Windows :)

no, no, my bad for being such a sloppy reviewer :)

In all seriousness: thanks for finding it, confirmed fixed now.


StoreEnhancer has incorrect parameter type

Prior Issues
Introduced in https://github.com/reduxjs/redux/pull/3524.
What is the current behavior?
Code
type MyEnhancerType = { enhancedStuff: number };

function createEnhancer(): StoreEnhancer<MyEnhancerType> {
  return (createStore) => <S, A extends Action>(
    reducer: Reducer<S, A>,
    initialState?: PreloadedState<S>
  ): Store<S, A, never, MyEnhancerType> & MyEnhancerType => {
    // This store that I just created does not have the enhancer applied to it yet
    // It also doesn't know what enhancers have been applied to it
    const nextStore = createStore(reducer, initialState);

    // And yet we can access a property that we have yet to add.
    // This should produce a type error, but it doesn't.
    const propertyThatsNotOnStoreYet = nextStore.enhancedStuff;

    // It's not until here that we have an enhanced store
    // And should be able to access that property
    const enhancedStore = enhanceStore(nextStore);
    const correctPropertyAccess = enhancedStore.enhancedStuff;

    return enhancedStore;
  };
}

function enhanceStore<S, A extends Action>(store: Store<S, A, unknown, unknown>): Store<S, A, never, MyEnhancerType> & MyEnhancerType {
  return {
    ...store,
    enhancedStuff: 5,
  }
}

Output
function createEnhancer() {
    return (createStore) => (reducer, initialState) => {
        // This store that I just created does not have the enhancor applied to it yet
        // It also doesn't know what enhancors have been applied to it
        const nextStore = createStore(reducer, initialState);
        // And yet we can access a property that we have yet to add.
        const propertyThatsNotOnStoreYet = nextStore.enhancedStuff;
        // It's not until here that we have an enhanced store
        // And should be able to access that property
        const enhancedStore = enhanceStore(nextStore);
        const correctPropertyAccess = enhancedStore.enhancedStuff;
        return enhancedStore;
    };
}
function enhanceStore(store) {
    return Object.assign(Object.assign({}, store), { enhancedStuff: 5,
        // Ignore this for now. See https://github.com/reduxjs/redux/issues/3767 for more discussion
        replaceReducer(nextReducer) {
            return store.replaceReducer(nextReducer);
        } });
}

Compiler Options
{
  "compilerOptions": {
    "noImplicitAny": true,
    "strictNullChecks": true,
    "strictFunctionTypes": true,
    "strictPropertyInitialization": true,
    "strictBindCallApply": true,
    "noImplicitThis": true,
    "noImplicitReturns": true,
    "alwaysStrict": true,
    "esModuleInterop": true,
    "declaration": true,
    "experimentalDecorators": true,
    "emitDecoratorMetadata": true,
    "moduleResolution": 2,
    "target": "ES2017",
    "jsx": "React",
    "module": "ESNext"
  }
}

Playground Link: Provided
Steps to Reproduce
See code above.
What is the expected behavior?
Trying to access enhanced properties right after calling createStore should result in a type error.
Environment Details
N/A

Partially addressed by https://github.com/reduxjs/redux/pull/3772, but there's more to be done after that PR gets merged.

Nevermind, just made https://github.com/reduxjs/redux/pull/3776 as a separate PR that can get merged whether https://github.com/reduxjs/redux/pull/3772 gets merged or not.

If the code works in JS but fails only because of typing in TypeScript, I'd suggest you to use "any" to ignore it: const store = () =>   createStore(     reducer,     initialState as any   )

@alestor123 I'm aware that I can use any to ignore basically any problems with types. My goal in creating this issue is to fix the types not just ignore them (otherwise I might as well be using JS). This problem is more extensive than just the single example provided above. I will provide more examples of why the current types are problematic if that would be helpful (for example getting the correct type after composing enhancers is not possible with the current types).
I have also a PR attached to this issue that fixes the problem. Let me know if you think the proposed solution is incorrect.


Add .codeclimate.yml

Added .codeclimate.yml to stop Code climate from tracking tests, ZeroBin scripts and vendor scripts.

Hi @ayushgp! I merged an earlier PR with the code climate file, but it lacks some directories and has the wrong test directory. If you want to update this PR to resolve the merge conflicts before the two later PRs in my queue get to it, I'll give it a merge. Thanks!
Also, I recommend doing all your development on a feature branch. You are going to run into issues when the upstream develop branch changes, like it did here.

@smcgregor what do I do about this pull request now that you've merged the other one?


Update jobs.html

Simple design to remove line under every link

Testing plan:
GIFs or screenshots:

@habfem can you talk about why you want to make this styling change?
Closing since in any case we'd want to do this via a CSS file.


Several object size reductions

The first patch:
"Merging shared code to reduce object size by several percent.
Rolls LoadBytes and merge SubBytes and InvSubBytes to share the non-linear kernel.
Also fixed MixColumns indentation.
Appears to have no performance impact."
The second:
"Rolls MixColumns/InvMixColumns to reduce object size several percent.
This has a non-trivial negative performance impact on O3 -march=native builds.
(On the order of a 14% slowdown)"
Because of the slowdown I don't think we should take the second, but I thought I'd share it in any case.

Merged the first commit.


Fixes #61: About us in Navigation Menu Crash fix

Description
Fixed a crash in navigation menu. When there's no internet availability , on clicking about us option in navigation drawer it shows a toast and a textview in the layout stating "Check your internet connection".
Fixes #61
Screenshot:

Good job! Congrats on your fırst contribution!

Good job! Congrats on your fırst contribution!


Vulnerability roundup 63: gnome-keyring-3.28.2: 1 advisory

search, files

[ ] CVE-2018-19358 (nixos-19.03)

Scanned versions: nixos-19.03: 5847485e3ec. May contain false positives.

See also #55387

Still waiting as there hasn't been any progress on this issue.

Same comment as in #55387: upstream is not going to fix this, disagrees that this is a problem. I think we should close this.


Update in conservative discretization

refactor methods to different files

add new "StepIntersect" method that intersects one step forward and one step backward for any method (see fig below)

extend CorrectionHull with exp=:base and scalar matrix such that it doesn't add the correction hull matrix F (not needed)

example with the single degree of freedom problem: the step size is on purpose chosen to be relatively "large".

Pull Request Test Coverage Report for Build 610920968

118 of 185 (63.78%)  changed or added relevant lines in 5 files are covered.
3 unchanged lines in 2 files lost coverage.
Overall coverage increased (+2.0%) to 47.737%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Continuous/NoBloating.jl
26
33
78.79%

src/Continuous/Forward.jl
48
56
85.71%

src/Continuous/Backward.jl
0
10
0.0%

src/Continuous/CorrectionHull.jl
44
64
68.75%

src/Continuous/StepIntersect.jl
0
22
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/Algorithms/BOX/reach_homog.jl
1
37.5%

src/Algorithms/TMJets/reach.jl
2
93.99%

Totals

Change from base Build 605753146:
2.0%

Covered Lines:
2046

Relevant Lines:
4286

💛  - Coveralls


Create upnode.toml for testnet 15

Description

Website: https://upnode.org/
Security Architecture: https://blog.upnode.org/upnode-security-architecture/
We are providing professionally managed, institutional-grade blockchain infrastructure. Beyond offering top-tier validator and staking services, we deploy and monitor your web3 API seamlessly on your servers, bringing true decentralisation to web3 apps.
Upnode employs an Active-Standby validator system across multiple regions for maximum availability, protected by a VPN network managed by a Fortigate firewall. Servers maintain advanced firewalls with limited port exposure to prevent unattended access. Private keys are stored securely using YubiHSM2. We employ two different OS for each of our datacenters, one relies on Proxmox VMs for node isolation, while the other relies on Ubuntu bare metal for optimal performance, which is our primary active node. We have an intensive monitoring and alerting stack with Tenderduty, Grafana, Loki,  Prometheus Alertmanager, and more with alerts directed to our engineer 24/7 through PagerDuty for incident management.
We have >6M$ delegated on our Celestia validator
https://www.mintscan.io/celestia/validators/celestiavaloper133t4gpv4vhpqgfn9gr8l4u423zrglg8rkqeupr
We have successfully launched our post-genesis namada validator on public testnet 14
Controller
atest1v4ehgw36xazrsdp489znjvpc89ryvd3ngger2wz9xc6nzsfsgycrxsjpggcyyv2xxfq5gvehlf6f7j

Validator
atest1v4ehgw36xazrsdp489znjvpc89ryvd3ngger2wz9xc6nzsfsgycrxsjpggcyyv2xxfq5gvehlf6f7j

Validator Tendermint Address
E80C31BAF701676C0BD1DCBF8C345C7B0F3EA875

However, we are a below capacity set validator as we don't have enough staked token
If this is an UPDATE for a previous genesis validator
Checks are made against your net_address. If this has changed since the previous testnet, make sure you provide links of previous prs merged from your previous git username.
Thanks in advance!
Checklist before merging

[X] Only one toml is added in this PR
[X] The file being added is indeed a .toml file
[X] The toml being added is to the correct folder, and only to the correct folder
[X] The eth_hot_key and eth_cold_key are present
[X] The email, discord, elements telegram, and twitter fields are present and valid

This validator was not recognised as a previous validator and has update in the title. Please read the description.


Add links to docs for all SaaS connector pages

Add links from saas connector templates to UI
https://www.figma.com/file/dmEwdK3xZwjKfGVQ9t66xe/Fides-v.2-Master-Working-Files?type=design&node-id=7683%3A154756&mode=design&t=HdpAKv2NAKQcPNpc-1

moving to done