🐛 Bug Report: GitLabOrganizationalData not working in Docker with User "node"

📜 Description
Hello,
the "node" user declared in the Dockerfiles proposed in the backstage deployment documentation in docker mode cannot start the gitlab/org plugin.

deployment/docker: https://backstage.io/docs/deployment/docker
gitlab/org : https://backstage.io/docs/integrations/gitlab/org
code: https://github.com/backstage/backstage/tree/master/plugins/catalog-backend-module-gitlab

👍 Expected behavior
Log with GitlabOrgDiscoveryEntityProvider, backstage stared with root user
[1] Loaded config from app-config.yaml, app-config.local.yaml
[1] 2024-02-12T13:54:07.302Z backstage info Found 5 new secrets in config that will be redacted
[1] 2024-02-12T13:54:07.309Z backstage info Created UrlReader predicateMux{readers=azure{host=dev.azure.com,authed=false},bitbucketCloud{host=bitbucket.org,authed=false},github{host=github.com,authed=false},gitlab{host=gitlab.domaine.com,authed=true},gitlab{host=gitlab.com,authed=false},awsS3{host=amazonaws.com,authed=false},fetch{}
[1] 2024-02-12T13:54:07.323Z catalog info Performing database migration type=plugin
[1] 2024-02-12T13:54:07.524Z backstage info Task worker starting: GitlabOrgDiscoveryEntityProvider:yourProviderId:refresh, {"version":2,"cadence":"PT2M","timeoutAfterDuration":"PT3M"} type=taskManager task=GitlabOrgDiscoveryEntityProvider:yourProviderId:refresh
[1] (node:28940) Warning: Setting the NODE_TLS_REJECT_UNAUTHORIZED environment variable to '0' makes TLS connections and HTTPS requests insecure by disabling certificate verification.
[1] (Use node --trace-warnings ... to show where the warning was created)
[1] 2024-02-12T13:54:07.646Z auth info Configuring "database" as KeyStore provider type=plugin
[1] 2024-02-12T13:54:07.682Z auth info Configuring auth provider: gitlab type=plugin
[1] 2024-02-12T13:54:07.684Z techdocs info Creating Local publisher for TechDocs type=plugin

👎 Actual Behavior with Screenshots
Log without GitlabOrgDiscoveryEntityProvider, backstage stared with node user
Loaded config from app-config-from-configmap.yaml, env
2024-02-12T13:55:42.378440723Z {"level":"info","message":"Found 5 new secrets in config that will be redacted","service":"backstage"}
2024-02-12T13:55:42.390357994Z {"level":"info","message":"Created UrlReader predicateMux{readers=azure{host=dev.azure.com,authed=false},bitbucketCloud{host=bitbucket.org,authed=false},github{host=github.com,authed=false},gitlab{host=gitlab.domaine.com,authed=true},gitlab{host=gitlab.com,authed=false},awsS3{host=amazonaws.com,authed=false},fetch{}","service":"backstage"}
2024-02-12T13:55:42.437977421Z {"level":"info","message":"Performing database migration","plugin":"catalog","service":"backstage","type":"plugin"}
2024-02-12T13:55:42.764153115Z {"level":"info","message":"Configuring "database" as KeyStore provider","plugin":"auth","service":"backstage","type":"plugin"}
2024-02-12T13:55:42.829669573Z {"level":"info","message":"Configuring auth provider: gitlab","plugin":"auth","service":"backstage","type":"plugin"}
{"level":"info","message":"Creating Local publisher for TechDocs","plugin":"techdocs","service":"backstage","type":"plugin"}
👟 Reproduction steps

Add Gitlab org pulgin and config
Start backstage in docker mode with node user

📃 Provide the context for the Bug.
No response
🖥 Your Environment
No response
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
None

This looks like a TLS validation error, which I doubt has any relation to the docker user.  Most likely because the certificate is not trusted by docker.  This could happen if either your gitlab instance is signed by an internal Certificate Authority, or if your company implements some form of TLS Inspection / main in the middle, which re-signs traffic with their own Certificate Authority.
You'll likely need to tell your docker container about these custom CAs.  One way is to set a NODE_EXTRA_CA_CERTS1 environment variable.  Another is to add the cert to your container OS trust store2.

Hi
The certificates are well managed, but the right on the all-ca.crt file is not :(
The same dockerfile with the user "node" the plugin does not work without any error message, if we use the default user (root) the plugin works.
The problem is resolved by adding the "---chmod=777" to the COPY of the ca certificates file.
Can you add the plugin logs in case of malfunction please?
Thanks a lot for your help.


Get release date for all versions?

To follow up on #3 -- this is what I get now for the package I'm interested in:
$ leak astropy
================================================================================
astropy
Community-developed python astronomy tools
--------------------------------------------------------------------------------
Author:              The Astropy Developers
Author mail<EMAIL_ADDRESS>Available versions:  52
Home page:           http://astropy.org
================================================================================
(1.3.3) Most recent. 19/06/2017 17:01 release date
(1.3.2)
(1.3.1)
(1.3)
(1.3rc1)
(1.2.2)
(1.2.1)
(1.2)
(1.2rc1)
(1.1.2)
(1.1.1)
(1.1.post2)
(1.1.post1)
(1.1)
(1.1rc2)
(1.1rc1)
(1.1b1)
(1.0.13)
(1.0.12)
(1.0.11)
(1.0.10)
(1.0.9)
(1.0.8)
(1.0.7)
(1.0.6)
(1.0.5)
(1.0.4)
(1.0.3)
(1.0.2)
(1.0.1)
(1.0)
(1.0rc2)
(1.0rc1)
(0.4.6)
(0.4.5)
(0.4.4)
(0.4.3)
(0.4.2)
(0.4.1)
(0.4)
(0.4rc2)
(0.4rc1)
(0.3.2)
(0.3.1)
(0.3)
(0.2.5)
(0.2.4) Most popular. 127057 downloads
(0.2.3)
(0.2.2)
(0.2.1)
(0.2)
(0.1)

Really what I'm interested in is to list the release data for all the versions, so I'm filing this as a new feature request.

@cdeil It's been a long journey!
Finally the feature has been landed into the lib 🙂
You can upgrade you leak and check dates/downloads for every release available
$ pip install --upgrade leak --force-reinstall

@bmwant - This is super cool! Thank you!


Fixed the Key Vault test-resources.json file to properly configure its deployment script for certificate creation

Credit goes to @heaths for catching the issue and proposing the fix :)

/azp run java - keyvault - tests

/check-enforcer override


Compile errors with gcc 6.1

Hi all
I noticed gcc 6.1 can no longer build the miniz.c inside rust.
See: https://github.com/rust-lang/rust/issues/33246

This fixes the build for me with GCC 6.1.1.
diff --git a/src/rt/miniz.c b/src/rt/miniz.c
index 2b803b0..b3a453d 100644
--- a/src/rt/miniz.c
+++ b/src/rt/miniz.c
@@ -575,7 +575,10 @@ tinfl_status tinfl_decompress(tinfl_decompressor *r, const mz_uint8 *pIn_buf_nex
       {
         mz_uint8 *p = r->m_tables[0].m_code_size; mz_uint i;
         r->m_table_sizes[0] = 288; r->m_table_sizes[1] = 32; TINFL_MEMSET(r->m_tables[1].m_code_size, 5, 32);
-        for ( i = 0; i <= 143; ++i) *p++ = 8; for ( ; i <= 255; ++i) *p++ = 9; for ( ; i <= 279; ++i) *p++ = 7; for ( ; i <= 287; ++i) *p++ = 8;
+        for ( i = 0; i <= 143; ++i) *p++ = 8;
+        for ( ; i <= 255; ++i) *p++ = 9;
+        for ( ; i <= 279; ++i) *p++ = 7;
+        for ( ; i <= 287; ++i) *p++ = 8;
       }
       else
       {
@@ -1393,7 +1396,10 @@ static MZ_FORCEINLINE void tdefl_find_match(tdefl_compressor *d, mz_uint lookahe
         if ((d->m_dict[probe_pos + match_len] == c0) && (d->m_dict[probe_pos + match_len - 1] == c1)) break;
       TDEFL_PROBE; TDEFL_PROBE; TDEFL_PROBE;
     }
-    if (!dist) break; p = s; q = d->m_dict + probe_pos; for (probe_len = 0; probe_len < max_match_len; probe_len++) if (*p++ != *q++) break;
+    if (!dist) break;
+    p = s;
+    q = d->m_dict + probe_pos;
+    for (probe_len = 0; probe_len < max_match_len; probe_len++) if (*p++ != *q++) break;
     if (probe_len > match_len)
     {
       *pMatch_dist = dist; if ((*pMatch_len = match_len = probe_len) == max_match_len) return;

You just had to add line breaks to fix it?
On 21 May 2016 at 00:31, Simon Sapin<EMAIL_ADDRESS>wrote:

This fixes the build for me with GCC 6.1.1.
diff --git a/src/rt/miniz.c b/src/rt/miniz.c
index 2b803b0..b3a453d 100644--- a/src/rt/miniz.c+++ b/src/rt/miniz.c@@ -575,7 +575,10 @@ tinfl_status tinfl_decompress(tinfl_decompressor *r, const mz_uint8 *pIn_buf_nex
{
mz_uint8 *p = r->m_tables[0].m_code_size; mz_uint i;
r->m_table_sizes[0] = 288; r->m_table_sizes[1] = 32; TINFL_MEMSET(r->m_tables[1].m_code_size, 5, 32);-        for ( i = 0; i <= 143; ++i) *p++ = 8; for ( ; i <= 255; ++i) *p++ = 9; for ( ; i <= 279; ++i) *p++ = 7; for ( ; i <= 287; ++i) *p++ = 8;+        for ( i = 0; i <= 143; ++i) *p++ = 8;+        for ( ; i <= 255; ++i) *p++ = 9;+        for ( ; i <= 279; ++i) *p++ = 7;+        for ( ; i <= 287; ++i) *p++ = 8;
}
else
{@@ -1393,7 +1396,10 @@ static MZ_FORCEINLINE void tdefl_find_match(tdefl_compressor *d, mz_uint lookahe
if ((d->m_dict[probe_pos + match_len] == c0) && (d->m_dict[probe_pos + match_len - 1] == c1)) break;
TDEFL_PROBE; TDEFL_PROBE; TDEFL_PROBE;
}-    if (!dist) break; p = s; q = d->m_dict + probe_pos; for (probe_len = 0; probe_len < max_match_len; probe_len++) if (*p++ != *q++) break;+    if (!dist) break;+    p = s;+    q = d->m_dict + probe_pos;+    for (probe_len = 0; probe_len < max_match_len; probe_len++) if (*p++ != *q++) break;
if (probe_len > match_len)
{
*pMatch_dist = dist; if ((*pMatch_len = match_len = probe_len) == max_match_len) return;
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/richgel999/miniz/issues/51#issuecomment-220654322

Yes, exactly.

The error message is

this ‘for’ clause does not guard this statement, but the latter is misleadingly indented as if it is guarded by the ‘for’

So the program would behave correctly if this check was simply disabled. But line breaks do make it much easier to read for humans.

I made a PR: https://github.com/richgel999/miniz/pull/52

So I think this will not be merged?

Should be fixed now with 2.x


Add PytorchBatchConverter

Adding PytorchBatchConverter as discussed in https://github.com/apache/beam/issues/21440 as an initial step to integrate Batched DoFns into RunInference.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @TheNeuralBit

PTAL @TheNeuralBit

It looks like codecov is unhappy since this file looks untested. Presumably because we don't have pytorch installed in the run generating coverage data. I confirmed this test is exercised on Jenkins:


$__contains macro doesn't expand correctly with variable of type custom

I have an application logging custom events with Application Insights and have Grafana (self hosted, v5.3.4) set up with Application Insights data source. I have a multi-value variable defined as $room on the dashboard as a custom type with the following value:
Room 1, Room 2, Room 3, Room 4
I then have a query such as (simplified for example):
customEvents
| where $__contains(customDimensions.roomName, $room)
| project customDimensions.start, customDimensions.roomName

Query inspector shows the query parameter for the API call expands out to:
customEvents
| where customDimensions.roomName in ({Room 1,Room 2,Room 3,Room 4})
| project customDimensions.start, customDimensions.roomName

This is invalid according to Log Analytics query in Azure Portal.
One reason I can see is the values are not quoted.
So if I change the variable to use quotes: 'Room 1', 'Room 2', 'Room 3', 'Room 4' (not ideal as it shows the quotes in the drop down in dashboard).
This expands to:
customEvents
| where customDimensions.roomName in ({'Room 1','Room 2','Room 3','Room 4'})
| project customDimensions.start, customDimensions.roomName

This is also invalid: Query could not be parsed at '{' on line [2,38]
If I remove the { and } in Azure Portal query, then it runs fine.
It appears to be broken and doesn't correspond to the documentation:

$__contains(colName, $myVar) - is to be used with multi-value template variables. If $myVar has the value 'value1','value2', it expands to: colName in ('value1','value2').

p.s. The custom variable I'm just using at the moment to hard code a set of values. I'm having a hard time trying to get a variable to pick values from a query. I just get an error "Template variables could not be initialized: undefined" whatever query I use.

I think the problem here is with the variable definition in Grafana rather than the macro. Using the custom variable type specifying a series of comma separated values, the variable has { and } surrounding it. Also it doesn't put quotes around the values, but manually adding quotes makes the quotes appear in the selection drop down in the UI also.


Whois v1.21 has Trojan/Win32.Swrort

Whois v1.21 has Trojan/Win32.Swrort by VirusTotal
Why?
How its possible?

This is a very low reputation score with only one AV engine finding it. FPs are common with AV products and this is common with VirustTotal


Add type support

Using Flowtype or TypeScript, I want to include support for some kind of static typing. At a minimum, we should add a .d.ts file and a .js.flow file, but I would seriously consider (re)writing the library into either of those directly. Statlz makes a good argument that all libraries should be written in TypeScript, but it's really an argument for types of any kind.
Up to this point, I've favored Flowtype's philosophy, designed to allow adding types incrementally. TypeScript has bigger community uptake, and the development on Flowtype seems slower, so all of these seem like decent arguments for writing it in TypeScript, and then possibly figuring out how to compile it to Flowtype definitions.
Either way, we should eventually pick one and use it.

If we're going to do this, we should seriously consider dropping Ramda. Don't know how difficult that would be, but I'm not sure how well types work with the currying Ramda provides. Which is a great feature, but we might be able to do the same things with arrow functions and make the types much clearer.
The other thought is to keep Ramda, forget Flow/Typescript, and add some kind of runtime type checking with something like tcomb. We're already doing typechecking of sorts for the user-facing code though, so I'm not sure how beneficial that is.

This being done as part of #78.


Erroneous output in chapter04_ex11

I appreciate having this repo available as a resource as I'm working through the book.
This file, however, is just printing n_primes odd numbers, rather than n_primes first primes. It also skips 2.
Updated main, in case you care to update:
https://gist.github.com/pithonsmear/5477912532be67d1926271091214730c

Thanks for letting me know! I'll definitely update that.


Xbox controllers on Windows 10 buttons are incorrectly mapped

Hi! I'm the maintainer of Silk.NET. We're currently experiencing a bug with Xbox controllers, where the buttons are returned incorrectly by GLFW. This could be because Windows 10 hijacks the home button, but:

the home/guide button is never reported as pressed
any buttons that are after GUIDE in the button array are offset by -1 (i.e. LEFT_STICK (index 9) is reported as GUIDE (index 8), RIGHT_STICK as LEFT_STICK etc)

To reproduce:

Use GLFW 3.3
Extract & run this sample: https://filebin.net/9w738tei2i54dqau
Connect a Xbox Controller
Press Left Stick, you will notice in the logs that this will appear as Home (which is our name for the Guide button).
Press Right Stick, you will notice in the logs that this will appear as Left Stick

Sample source: https://github.com/Ultz/Silk.NET/tree/master/examples/InputTest
We have investigated thoroughly and have deduced that this is a problem with GLFW, not Silk.NET.

cc @mikart143

Any news on this?

Thank you for your patience. Which model Xbox controller are you using?

I’m using a Day One 2013 controller with all the latest updates installed on it. I’m also using the latest version of Windows 10 and have the Xbox (Beta) app installed.

Also did you still need that sample program as I noticed that the download link expired?

Thank you for the quick reply! No need to update the download link, though if you have access to the controller it would be useful to know what GUID that GLFW says it has on Windows.

Cannot reproduce this with an Xbox One controller on Windows 10 and the Xbox beta app.

Are you using the sample I provided? Also I'll get the GUID now...

The GUID is 78696e70757401000000000000000000

Thank you! No, I'm using the joysticks test from GLFW.

Ah okay. I've had a deeper look and it looks like glfwGetJoystickButtons is returning a button count of 14 instead of 15 as expected, implying that the Home/Guide button isn't included in the button array.
Here's the repackaged & reuploaded test application I'm using: https://filebin.net/3tj6d9omkriatfi5

Are you using glfwGetGamepadState and reading the buttons from GLFWgamepadstate?
I'm also on Windows 10 with the same controller and glfwGetGamepadState reports the button states as expected. The buttons returned by glfwGetJoystickButtons may not be mapped with the same indexes.

Ah no we were using the joystick APIs alone as we need to support generic
joysticks - not just gamepads.
On Wed, 25 Dec 2019 at 15:36, aloucks<EMAIL_ADDRESS>wrote:

Are you using glfwGetGamepadState
https://www.glfw.org/docs/latest/group__input.html#gadccddea8bce6113fa459de379ddaf051
and reading the buttons from GLFWgamepadstate
https://www.glfw.org/docs/latest/structGLFWgamepadstate.html?
I'm also on Windows 10 with the same controller and glfwGetGamepadState
reports the button states as expected. The buttons returned by
glfwGetJoystickButtons may not be mapped with the same indexes.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/glfw/glfw/issues/1599?email_source=notifications&email_token=ACVEYI4U2UNVQQ6KFTAEXMTQ2N4WDA5CNFSM4JSJKNB2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHUN7LI#issuecomment-568909741,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACVEYIY7RVGDYTOVJ53OMQTQ2N4WDANCNFSM4JSJKNBQ
.

Ah just seen your edit, will implement the suggested changes shortly. Will leave this open for the time being.

Closing as this has been solved.


balena-yocto-scripts: Update to 1.16.0

Update balena-yocto-scripts from v1.14.9 to v1.16.0
Change-type: patch
Signed-off-by: Alex Gonzalez<EMAIL_ADDRESS>

@balena-ci retest

@balena-ci I self-certify!


JSON should support stacked metrics

Needed for grafana

what does this mean? how do you "stack" numbers in json?

Currently the stacked graph code is tied to the Cairo renderer.  We have some stacked graphs at work that fail with grafana's flot renderer but look fine with png. If this is a user error or grafana restriction we can close this.

stacking in grafana is just a viz option in the renderer that should have no effect on the http request or response to/from graphite.  I used the devtools network inspector to confirm this. requests and responses look identical whether stacked is enabled in grafana or not.
I still don't understand what you expect a "stacked" json response to look like, how it should differ from a normal response, or what you believe the problem is in your responses.

Hmm.. I'll poke @Civil to investigate this more on Monday.

Closing.


Strange behaviour with display names with quotes

Double quotes in a from address are getting double escaped, this seems to be something within the field class that is doing this.
I think this may be related to #960
[142] pry(main)> m = Mail.new
=> #<Mail::Message:70265610849840, Multipart: false, Headers: >

[143] pry(main)> addy =<EMAIL_ADDRESS>=> #<Mail::Address:70265610731960 Address<EMAIL_ADDRESS>>

[144] pry(main)> addy.display_name = 'I "am" cool'
=> "I \"am\" cool"

[145] pry(main)> m = Mail.new
=> #<Mail::Message:70265610525820, Multipart: false, Headers: >

[146] pry(main)> m[:from] = addy.format
=> "\"I \\\"am\\\" cool\"<EMAIL_ADDRESS>
[147] pry(main)> puts m.to_s
Date: Sun, 07 Aug 2016 19:56:34 +0100
From: "I \\\"am\\\" cool"<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>Mime-Version: 1.0
Content-Type: text/plain
Content-Transfer-Encoding: 7bit

So the final message from header is double quoted: "I \\\"am\\\" cool"<EMAIL_ADDRESS>I was expecting "I \"am\" cool"<EMAIL_ADDRESS>Little more info, this seems to affect master, and the 2.6 stable branch.
I created a quick spec to confirm:
it "should handle special chars in headers correctly" do
      from =<EMAIL_ADDRESS>      from.display_name = 'foo "quotes" bar'

      to =<EMAIL_ADDRESS>      to.display_name = 'foo "quotes" bar'

      mail = Mail.new do
           from from.format
             to to.format
        subject 'This is a test email'
           body 'This is a body of the email'
      end

      expect(mail.to_s).to match(/From: "foo \"quotes\" bar"<EMAIL_ADDRESS>      expect(mail.to_s).to match(/To: "foo \"quotes\" bar"<EMAIL_ADDRESS>    end

>> addy =<EMAIL_ADDRESS>=> #<Mail::Address:70155625366220 Address<EMAIL_ADDRESS>>
>> addy.display_name = 'I "am" cool'
=> "I \"am\" cool"
>> m = Mail.new(from: addy.format)
=> #<Mail::Message:70155653435720, Multipart: false, Headers: <From: "I \"am\" cool"<EMAIL_ADDRESS>>> puts m.encoded
Date: Tue, 16 May 2017 23:06:04 -0700
From: "I \"am\" cool"<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>Mime-Version: 1.0
Content-Type: text/plain
Content-Transfer-Encoding: 7bit

=> nil
>> puts Mail.new(from: addy.format)[:from].display_names
I "am" cool
=> nil


Exchange script adjustments

Supports the ability to import cert from filesystem, add option to remove old exchange certificates, and cleans up logic
Might be best to commit and squash.

@LBegnaud - How are you feeling about this? My gut says that the script is much better already than the once that's currently being distributed in our releases. Would fixing the parameter type issue at a later date cause problems for people upgrading? I guess what I mean to say is don't let perfect get in the way of good :).

I have reviewed the existing script and the proposed script and I'm confident that what they are both trying to do is going to work fine, but it does seem like the new changes to the proposed script might need a little touching up before merging.

I honestly had forgotten about this. I think that it's fine in its current state. I'm going to make a quick fix to the parameter to be an int that gets passed a 0 or 1 as I don't have time to mess with it too much right now. I'll push a commit then you're good to merge.

K I think it's ready. Thanks for the reminder

No problem, thanks for contributing!


Drag and Drop support

There is a way to implement drag and drop support from winfile to any other program?

Winfile already supports some of the drag/drop and clipboard formats/operations.   The original support was extremely limited and I improved that a few years back.
What specific cases don't work for you?

When i try to drag and drop from other program to winfile, works flawlessly.
But when i try to drag and drop from winfile to elsewhere, doesn't work at all, with the mouse cursor changing to unavailable.

There are some cases from WinFile to other programs that work, but there is no OLE drop source yet (https://docs.microsoft.com/en-us/cpp/mfc/drag-and-drop-implementing-a-drop-source).  The headers have the corresponding definitions, but I never got that far.

This OLE drop source support aren't on the plans, right?

Independent of who does the work, supporting the OLE drop source mechanism makes sense to me.  The work is non-trivial.

At the very least it would be nice to be able to drop from Winfile onto the desktop.

This is...annoying.
Before OLE drag and drop existed, there was a simple drag and drop mechanism used by the File Manager.  It's still documented on MSDN and the OLE documentation refers to it: "OLE drag and drop is different from the File Manager drag-and-drop mechanism, which can only handle filenames and is designed specifically to pass filenames to applications."  That's supposed to be a criticism, but the advantage is that the interface designed for passing around files is really simple to use when the requirement is to pass around files.
What really changed is Explorer is no longer calling DragAcceptFiles to accept files from this simple interface.  It works fine in older versions of Windows - I just used @MartinPayne's port on NT4 to check my sanity, and sure enough, it works fine on NT4 and the same binary can't drag to Explorer on modern Windows.  You can use WinFile on modern versions to drag files to other applications that support its interface, such as my media player. Explorer will even generate the old interface, so objects from Explorer can be dragged to Serenity, but WinFile doesn't register for its own WM_DROPFILES interface, so Explorer won't talk to it in that direction either.
The pragmatist in me realizes that there's no chance of Explorer supporting the old interface again, but it's really annoying that this used to work, and changes in Explorer prevent it from receiving files from WinFile.  I'm also fairly confident that at one point WinFile supported WM_DROPFILES, but the NT 3.1 version doesn't seem to do it, so that's going back a long way.

Independent of who does the work, supporting the OLE drop source mechanism makes sense to me.  The work is non-trivial.

Well I think I did the C implementation of IDataobject IDropsource and related classes, but I'm currently struggling that other Droptargets accept my Dropsource.
Maybe I didn't put the proper types into the dataobject. Have to play a bit.

WinFile doesn't register for its own WM_DROPFILES interface, so Explorer won't talk to it in that direction either.

but it is funny that you can start up 32bit winfile, then start up a 16bit winfile, and then DnD between them. winfile instances happily accept their own DnD mechanism(i.e. WM_QUERYDROPOBJECT and friends).


Ignore columns

This branch adds ignore column functionally by adding any columns you want this app to ignore via command line --ignore as many times as you want. These columns won't show up in diff but will be shown in patches!
Lars,
I can't seem to be able to fix the tests. I don't know why csvdiff_cmd returns exit code -1 . The application works fine and I've manually tested it. Can you help me fix these few failing tests?

Added '--ignore', 'extra' parameters to csvdiff_cmd runner in tests.
Tests still fail.

Hi Dan, thanks for the patch! I'm on summer break and without computer for
another week, I'll take a look once I'm back and see if I can work out why
the tests still fail. Just seeing the exit code being different is a little
frustrating, I agree.
On Fri, 22 Jul 2016 at 02:41, Dan Murad<EMAIL_ADDRESS>wrote:

Added '--ignore', 'extra' parameters to csvdiff_cmd runner in tests.
Tests still fail.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/larsyencken/csvdiff/pull/16#issuecomment-234425064,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACMrL8L6rZInrKoMOV_AaeMuSeMA3UMks5qYBGjgaJpZM4JSWDT
.

No problem Lars, I’ll tell my team to pip install my branch zip.
Enjoy your break!
Talk to you later.
From: Lars Yencken<EMAIL_ADDRESS>Sent: Saturday, July 23, 2016 12:18 AM
To: larsyencken/csvdiff<EMAIL_ADDRESS>Cc: Dan Murad<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [larsyencken/csvdiff] Ignore columns (#16)
Hi Dan, thanks for the patch! I'm on summer break and without computer for
another week, I'll take a look once I'm back and see if I can work out why
the tests still fail. Just seeing the exit code being different is a little
frustrating, I agree.
On Fri, 22 Jul 2016 at 02:41, Dan Murad<EMAIL_ADDRESS><EMAIL_ADDRESS>> wrote:

Added '--ignore', 'extra' parameters to csvdiff_cmd runner in tests.
Tests still fail.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/larsyencken/csvdiff/pull/16#issuecomment-234425064,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACMrL8L6rZInrKoMOV_AaeMuSeMA3UMks5qYBGjgaJpZM4JSWDT
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/larsyencken/csvdiff/pull/16#issuecomment-234702176 , or mute the thread https://github.com/notifications/unsubscribe-auth/AMZ2OpTJLon62CwMlRlOYE68PJOElwDvks5qYbH6gaJpZM4JSWDT .  https://github.com/notifications/beacon/AMZ2OofIoBF0_-gLuBcNTsACLq65vI_jks5qYbH6gaJpZM4JSWDT.gif

Ah, ok. Basically, the existing tests cover the loop of diffing a.csv and b.csv with csvdiff, generating a patch, then applying the patch to a.csv with csvpatch. It should be a closed loop that gives you the same result as b.csv.
In this case, you changed some of the test files, and needed to add the --ignore option to csvpatch too. I'd make new tests instead of changing the existing ones though.
In terms of the code change, a nicer way of doing this would be to use the ignored columns to create a view of the data that doesn't have those columns, when loading records. That way diffing or patching code doesn't need to change.
Got time to try and adjust your pull request, or would you like me to have a crack at this other approach?

Sure take a crack at it. That is a better way, I agree.

I would like to give this a go...

@kevinlr5 go for it

Yeah, things have been more busy for me than I realised. Please take a stab at it!

Just merged https://github.com/larsyencken/csvdiff/pull/19 which implemented this in the view style. Thanks for prompting for this new feature.


Add api/v1.0/ prefix for all REST related calls

We should add "api" and a version path segment

+1

should we also add that prefix to websocket?

Maybe not. As the web socket address will be returned by REST.

It is part of the REST.


[5.5][Async Refactoring] Wrap code in a continuation if conversion doesn't yield reasonable results

Explanation: If we are requested to convert a function to async, but the call in the function’s body that eventually calls the completion handler doesn’t have an async alternative, we are currently copying the call as-is, replacing any calls to the completion handler by placeholders. This is seldom what the developer wants and it would be better to wrap the function in withChecked(Throwing)Continuation and replace all calls to the completion handlers by calls to continuation.resume(throwing/returning:).
Scope: Async refactoring where the completion handler is called from a function that doesn’t have an async alternative
Risk: Low
Testing: Added regression test
Issue: rdar://79304583
Reviewer: @bnbarham (Ben Barham), @hamishknight (Hamish Knight), @ktoso (Konrad Malawski) on original PR #38193

Example 1
Base Code
func testDispatch(completionHandler: @escaping (Int) -> Void) {
  DispatchQueue.global.async {
     completionHandler(longSyncFunc())
  }
}

Old Refactoring Result
func testDispatch() async -> Int  {
  DispatchQueue.global.async {
     <#completionHandler#>(longSyncFunc())
  }
}

New Refactoring Result
func testDispatch() async -> Int {
  return await withCheckedContinuation { continuation in
    DispatchQueue.global.async {
      continuation.resume(returning: longSyncFunc())
    }
  }
}

Example 2
Base Code
func testUrlSession(completionHandler: @escaping (Data) -> Void) {
  let task = URLSession.shared.dataTask(with: request) { data, response, error in
    completion(data!)
  }
  task.resume()
}

Old Refactoring Result
func testUrlSession() async -> Data {
  let task = URLSession.shared.dataTask(with: request) { data, response, error in
    <#completion#>(data!)
  }
  task.resume()
}

New Refactoring Result
func testDataTask() async -> Int?
  return await withCheckedContinuation { continuation in
    let task = URLSession.shared.dataTask { data, response, error in
      continuation.resume(returning: data!)
    }
    task.resume()
  }
}

@swift-ci Please test

@swift-ci Please test


does this support deep linking into your app?

all of the examples show deep linking from your app into another app, for example - like launching facebook. how about deep linking into your own app? does this support that too?

Ey @hsavit1, yeah! The Facebook scheme was an example. Try out your own link!
Add deep link support to your iOS app:
1.- Make sure you have a URL scheme registered for your app in your Info.plist

2.- Add this to your AppDelegate.m
#import "RCTLinkingManager.h"

- (BOOL)application:(UIApplication *)application openURL:(NSURL *)url
  sourceApplication:(NSString *)sourceApplication annotation:(id)annotation
{
  return [RCTLinkingManager application:application openURL:url
                      sourceApplication:sourceApplication annotation:annotation];
}

// Only if your app is using [Universal Links](https://developer.apple.com/library/prerelease/ios/documentation/General/Conceptual/AppSearch/UniversalLinks.html).
- (BOOL)application:(UIApplication *)application continueUserActivity:(NSUserActivity *)userActivity
 restorationHandler:(void (^)(NSArray * _Nullable))restorationHandler
{
 return [RCTLinkingManager application:application
                  continueUserActivity:userActivity
                    restorationHandler:restorationHandler];
}

For Android:
https://developer.android.com/training/app-indexing/deep-linking.html#adding-filters
More info: https://facebook.github.io/react-native/docs/linking.html

@luisfcofv thanks for the reply
so I'm still a bit confused. could you possibly add an example to the guide that shows how to handle a deep link into your app?

what is the difference between a scheme and a route?

@hsavit1 I updated the README and the example code, try to run the example provided (Only iOS works at the moment).
When the app is installed,  go to safari and enter a supported deep link into the address bar.
The supported deep links in the example are: example://test, example://test/:id & example://test/:id/details.
Entering the link example://test/100/details will open the app.
Hope this is clear enough, let me know how it goes.
Cheers.

hello @luisfcofv
all steps are clear for me, and it's workig in the way is described, but...always but
I need to send email with link that open my app, and if I in link send prefix as example:// , the webView not parse link as html... and if I send it without "://" I get link but not opening my app
Pls. help
ty

Hi @franj0, this seems more like format issue with your email app. Have you tried to wrap it in a html hyperlink tag? Something like Open the app

Yes, you are right. Gmail remove href attribute if there exist irl scheeme
e.g. example://....
I have hack solution for this.
Open regular link and on the server side in nginx rewrite do redirect 301
with my url scheme. This is temporary solution until my universal links get
works. Sry for bad eng, ty
On 20 Feb 2018 17:16, "Luis Flores"<EMAIL_ADDRESS>wrote:

Hi @franj0 https://github.com/franj0, this seems more like format issue
with your email app. Have you tried to wrap it in a html hyperlink tag?
Something like Open the app
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/luisfcofv/react-native-deep-linking/issues/4#issuecomment-367030020,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALBLMW37WLy8ZR8V8ZkJi7MKFsICor41ks5tWu_LgaJpZM4MH7SE
.

Great! Glad you figure it out


rename lakefs init to setup and keep init as alias

Closes #468

Codecov Report

Merging #961 (8e5d3d4) into master (4c888fa) will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #961      +/-   ##
==========================================
+ Coverage   44.84%   44.88%   +0.03%
==========================================
  Files         142      142
  Lines       11448    11448
==========================================
+ Hits         5134     5138       +4
+ Misses       5673     5671       -2
+ Partials      641      639       -2

Impacted Files
Coverage Δ

catalog/mvcc/cataloger_create_repository.go
62.96% <0.00%> (+3.70%)
:arrow_up:

catalog/mvcc/cataloger_create_entry.go
100.00% <0.00%> (+5.26%)
:arrow_up:

Continue to review full report at Codecov.

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

Cool!  This is great, I did not know about aliases.  Could you also please rename the file, to make it easier to find the command when searching by filename?

I did name it setup, missed another file?


Fixed plugin ignoring settings allow-end and allow-nether in server

configurations.
Fixes #2634 for end portals. Should also fix for nether ones.

I don’t think this should be fixed by mv side, as it looks like an issue with the server software instead.


"Adding" solutions is slow (casadi.horzcat)

"Adding" solutions can take up a large portion of the overall solution time. The time is spend calling casadi.horzcat.
E.g. in the below example casadi.horzcat takes about 9% of the overall time.
import pybamm
pybamm.set_logging_level("INFO")
options = {
    "sei": "solvent-diffusion limited",
    "sei film resistance": "distributed",
    "sei porosity change": "true",
}
model = pybamm.lithium_ion.DFN(options=options)
params = pybamm.ParameterValues(chemistry=pybamm.parameter_sets.Chen2020)
params.update(
    {
        "Reference temperature [K]": 278.15,
        "Ambient temperature [K]": 278.15,
        "Initial temperature [K]": 278.15,
        "Upper voltage cut-off [V]": 4.21,
    }
)
exp = pybamm.Experiment(
    ["Hold at 4.2V until C/100", "Rest for 4 hours (3 minute period)"]
    + [
        "Discharge at 1C until 2.5V",
        "Charge at 0.3C until 4.2V (3 minute period)",
        "Hold at 4.2V until C/100 (3 minute period)",
    ]
    * 50
)
sim = pybamm.Simulation(model=model, parameter_values=params, experiment=exp)
sim.solve()

Should be fixed, together with #1276 , by reformatting Solution to create a processed variable from its sub-solutions, without first creating solution.y

Should be fixed, together with #1276 , by reformatting Solution to create a processed variable from its sub-solutions, without first creating solution.y


MON-3486: add feature gate for Collection Profiles

Adds a custom feature gate for CMO's Collection Profiles feature. Also preserve the earlier behavior to enable it by default on TP enabled clusters.

/cc @openshift/openshift-team-monitoring

/jira refresh

/retest

@rexagod can you also link enhacemement proposal here?

/retest

/test e2e-azure

/retest

/retest

I think all comments has been addressed and to me this
/lgtm
Iiuc this is simply inactive until we merge the handling code in CMO?

@jan--f There are no blockers for this PR and this can be merged straightaway. Once that's done, a PR updating the api version in COO needs to be raised like so: https://github.com/openshift/cluster-config-operator/pull/377/files, and then pulled into https://github.com/openshift/cluster-monitoring-operator/pull/2153/files#diff-99c0290799daf9abc6240df64063e20bfaf67b371577b67ac7eec6f4725622ffR439 to make FeatureGateMetricsCollectionProfiles available.
Also, as a side note, https://github.com/openshift/cluster-monitoring-operator/pull/2046 is independent of any other PRs and can be merged in parallel.

To me this looks good. Lets rebase this and get api reviewer eyes on this.
/lgtm

@rexagod This will need an update according to #1815.

/retest

Ah, it seems all SingleNode manifests were removed from https://github.com/openshift/api/tree/master/payload-manifests/featuregates, however, update-payload-featuregates ends up recreating those files.

/lgtm

@deads2k seems like the bot merged this when it said its not mergeable...

[From a discussion with Jan earlier today] It seems this commit was pulled into Jan's PR, so when that went it, this was marked as merged as well.


[ESP32S3] btdm_controller_init failing after deinit (IDFGH-8987)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.
v4.4.3-319-g0fb32e11a58
Operating System used.
Windows
How did you build your project?
VS Code IDE
If you are using Windows, please specify command line type.
PowerShell
Development Kit.
ESP32S3-WROOM N8R2
Power Supply used.
USB
What is the expected behavior?
Switch on and off bluetooth multiple times on request to provision devices
What is the actual behavior?
Device crash on ble init the second time provisioning is started.
This was not happening before, we are looking for the commit that changed the behavior.
From the documentation it seems that we are performing some forbidden handling
/**
 * @brief       Initialize BT controller to allocate task and other resource.
 *              This function should be called only once, before any other BT functions are called.
 * @param  cfg: Initial configuration of BT controller. Different from previous version, there's a mode and some
 *              connection configuration in "cfg" to configure controller work mode and allocate the resource which is needed.
 * @return      ESP_OK - success, other - failed
 */
esp_err_t esp_bt_controller_init(esp_bt_controller_config_t *cfg);

/**
 * @brief  De-initialize BT controller to free resource and delete task.
 *         You should stop advertising and scanning, as well as
 *         disconnect all existing connections before de-initializing BT controller.
 *
 * This function should be called only once, after any other BT functions are called.
 * This function is not whole completed, esp_bt_controller_init cannot called after this function.
 * @return  ESP_OK - success, other - failed
 */
esp_err_t esp_bt_controller_deinit(void);

but then how you are supposed to handle a device that needs to update it's own configuration via bluetooth but normally has only wifi?
Steps to reproduce.

Start provisioning of one device
End provisioning -> BLE will be switched off
Restart provisioning without rebooting the device
...

Debug Logs.
I (16:44:23.963) prov_BLE: Unhandled event 0
I (16:44:23.968) prov_BLE: Starting BLE provisioning
I (92942) wifi:mode : sta (7c:df:a1:e0:8b:7c)
btdm: bss start 0x3fcef180, len 36
btdm: data start 0x3fcef174, data start rom 0x40057350, len 12
MAGIC fadebead VERSION 00010006
I (16:44:23.989) BT_INIT: BT controller compile version [02bfb7f]
E (16:44:23.996) simple_ble: simple_ble_start enable controller failed 257
E (16:44:24.004) protocomm_ble: simple_ble_start failed w/ error code 0x101
E (16:44:24.011) wifi_prov_scheme_ble: Failed to start protocomm BLE service
E (16:44:24.019) wifi_prov_mgr: Failed to start service
ESP_ERROR_CHECK failed: esp_err_t 0xffffffff (ESP_FAIL) at 0x40381e80
0x40381e80: _esp_error_check_failed at C:/src/esp-idf/components/esp_system/esp_err.c:42

file: "./components/provisioning/bluetooth.c" line 480
func: provisioning_ble_start
expression: wifi_prov_mgr_start_provisioning(security, pop, service_name, service_key)

we currently applied the following patch to the provisioning app in order to continue to use it

Did you only turn on and off the controller?
I have test init and deinit the bluetooth protocol stack, but no problems have been found so far. Could you please provide your sample code?

@xiewenxiang
The sample code is using the Bluetooth stack of the provisioning app...
https://github.com/espressif/esp-idf/tree/release/v4.4/examples/provisioning/wifi_prov_mgr
Not sure you can reproduce tho, since the provisioning app doesn't allow to go back into provisioning mode 🤔.

@KonssnoK
This is the original demo of IDF, and I have add init & deinit code, tested with this demo, but I couldn't reproduce this issue.

i'm sorry @xiewenxiang we are following on other issues, we currently don't have time to test this one :(
But the code there is not enough i think, you need to changes in order to enable multiple provisioning without restarting the device.

after updating to latest v4.4 7eba5f80027e1648775b46f889cb4d9519afc965
we tried removing the patch and we are not having the failure anymore


Fix #87

map._renderer was undefined if using panes

Amazing, looks like a dupe of this PR tho:
https://github.com/makinacorpus/Leaflet.TextPath/pull/85

Oh, didn't see. Should have been merged for a while then...

It is indeed a duplicate. I'll close the other merge request and we'll review this one soon.
Thanks for your contribution. :pray:

Thanks, for me, i've tested and it fixed error that came up when using latest leaflet with panes etc.


How to identify the start of the video playback

Does this library provide an option to identify the exact time of when the video playback starts happenning?
the actual playback may have been delayed (due to network delays, buffering etc.) but I'd like to know the exact time of when the video starts playing. Thanks in advance.

Yes, add a listener to the player. Read doc for more info.


Discord Handle Validation

Validate discord identity form front end using degen bot

This can be closed since we are going a different route where claiming is handled directly on Discord.


use bufferknife before version 8.4.0 in lib module

Whether to use in module before version 8.4.0? what should i do?

You are required to use 8.2.0 or newer for use in library modules with Android Gradle plugin 2.x. If you are using AGP 3.x you have to use 9.0.0-SNAPSHOT until the next release.


Force xwayland mode for Qt applications

Globally configuring Qt (like via /etc/environment) to run natively in wayland mode QT_QPA_PLATFORM=wayland (or even with QT_QPA_PLATFORM=wayland:xcb to support x11 fallback) will prevent Qt applications to start under gamescope, this change forces Qt apps to run in X11 mode for gamescope to grab them.
The use of QT_QPA_PLATFORM is documented in the arch wiki wayland page.
2024.06.18 edit: fixing typos, adding arch wiki link

Is there any issue with this addition?

Running desktop applications in gamescope is a niche use case, but this seems like a good (and otherwise harmless) thing to have.
Thanks!


Test case failed

websocket/Tests/WebSocketTests/WebSocketTests.swift:106: error:
-[WebSocketTests.WebSocketTests testServerContinuation] :
XCTAssertEqual failed: ("/") is not equal to ("!dlrow ,olleH")

I wasn't sure but the first call in onText closure returned, / and the next one is the correct one( !dlrow ,olleH), The question is why .onText() an initial value of /.
        client.onText { ws, text in
            XCTAssertEqual(text, "!dlrow ,olleH")
            _ = server.close()
        }
        client.send(raw: "Hello, ", opcode: .text, fin: false)
        client.send(raw: "world", opcode: .continuation, fin: false)
        client.send(raw: "!", opcode: .continuation)

Oh I see, the server is sending the path.


GM.xmlHttpRequest not resolvable without reconfiguring minify

Thanks for the plugin, it remains useful!
The default minify ON settings appear to clobber the following:
import {
    GM,
} from '$';

const STREAMING_SUPPORTED = GM.xmlHttpRequest?.RESPONSE_TYPE_STREAM;
console.log('Streaming supported: ', STREAMING_SUPPORTED);

.... using GM.xmlHttpRequest in code instead of GM_xmlHttpRequest, to maintain compatibility with Greasemonkey/GM API 4 while supporting Tampermonkey and others.

Is this intentional? By clobber, I mean these references resolve to "undefined" in Greasemonkey and Tampermonkey, instead of resolving to 'stream' and the appropriate xhr function.
When I add this configuration to Vite, then it works:
  esbuild: {     minifyIdentifiers: false,     keepNames: true,   },
But it doesn't minify as completely as it could. Ideally this would work with default minify.

what is your raw code, build config and product code

So the code in question is https://greasyfork.org/en/scripts/472236-gm-fetch/code but lightly revised to work with GM.xmlHttpRequest only instead of GM_xmlhttprequest, which ensures it works on Greasemonkey too (besides Tampermonkey).
build config
xport default defineConfig({
  plugins: [
    monkey({
      entry: 'src/main.js',
      userscript: {
        author: 'foo',
        description: 'foo',
        name: 'foo',
        namespace: 'bar',
        downloadURL:
          '...',
        updateURL:
          '...',
        match: [
            // ...
        ],
      },
      build: {
        fileName: 'foo.user.js',
        metaFileName: true,
      },
    }),
  ],
  build: {
    minify: true,
  },
  // TODO: https://github.com/lisonge/vite-plugin-monkey/issues/166 default minify settings appear to clobber GM.xmlHttpRequest. Reconfiguring esbuild's minify settings mitigates this, at the cost of a slightly less minified userscript. (+43 kB)
  esbuild: {
    minifyIdentifiers: false,
    keepNames: true,
  },
});

If you want a full printout let me know. Not sure what else you need?

Can you provide the repository url to reproduce this issue?
Or you can upload a repository zip file to the github comment section

output.zip
jstest.user.js.txt
Please let me know if any issues reproducing the issue. Attached a sample user.js with the default minify settings.

This is caused by autoGrant, you can manually set grant to solve it
grant: ['GM.xmlHttpRequest'],

@blakehurd Hi, I'm the maintainer of the said script (gm-fetch) you'd like to adapt. Just want you to know I'd love to accept PRs for it to be compatible with Greasemonkey in a non-breaking way, but of course it's entirely up to you :)


feat:  [M3-8326],  [M3-8055] - Create Linode - Create Using Command Line A/B Test

Description 📝
This PR uses the new feature flag for A/B testing for the Command line tools.
Changes  🔄
List any change relevant to the reviewer.

Created and used feature flag for A/B testing.
Added event emitter to track the metrics for the trackable elements in the DX tool Modal.
Created Metrics in the launch darkly and mapped to A/B testing experiment.

Target release date 🗓
9/16
How to test 🧪

Checkout the branch and run the app
Navigate to Create command line tools or View code snippets Modal and click on tabs, copy icon and resources.
Verify experiments results in LD reflects the clicks. Reach out me for further help.

As an Author I have considered 🤔
Check all that apply

[ ] 👀 Doing a self review
[ ] ❔ Our contribution guidelines
[ ] 🤏 Splitting feature into small PRs
[ ] ➕ Adding a changeset
[ ] 🧪 Providing/Improving test coverage
[ ] 🔐 Removing all sensitive information from the code and PR description
[ ] 🚩 Using a feature flag to protect the release
[ ] 👣 Providing comprehensive reproduction steps
[ ] 📑 Providing or updating our documentation
[ ] 🕛 Scheduling a pair reviewing session
[ ] 📱 Providing mobile support
[ ] ♿  Providing accessibility support

Metrics not firing for me:

❌ Initial opening of modal
❌ Linode CLI Tab Selection
❌ cURL Tab Selection
❌ All Resource Links

Metrics working:

✅ Linode CLI Copy Code Snippet Icon
✅ cURL Copy Code Snippet Icon
✅ Integrations - Ansible - Copy Code Snippet Icon
✅ Integrations - Terraform - Copy Code Snippet Icon
✅ SDKs - Python - Copy Code Snippet Icon
✅ SDKs - Go - Copy Code Snippet Icon

@cpathipa mind pulling in the latest from develop to get the tests passing?

@jdamore-linode done!

Thanks @cpathipa 🙌

Metrics not firing for me:

❌ All Resource Links

Metrics working:

✅ Initial opening of modal
✅ Linode CLI Tab Selection
✅ Linode CLI Copy Code Snippet Icon
✅ cURL Tab Selection
✅ cURL Copy Code Snippet Icon
✅ Integrations Tab Selection
✅ Integrations - Ansible - Copy Code Snippet Icon
✅ Integrations - Terraform - Copy Code Snippet Icon
✅ SDKs Tab Selection
✅ SDKs - Python - Copy Code Snippet Icon
✅ SDKs - Go - Copy Code Snippet Icon

I will create a bug ticket to investigate why event emitters are not firing consistently for resources.

Metrics not firing for me:

❌ All Resource Links

Metrics working:

✅ Initial opening of modal
✅ Linode CLI Tab Selection
✅ Linode CLI Copy Code Snippet Icon
✅ cURL Tab Selection
✅ cURL Copy Code Snippet Icon
✅ Integrations Tab Selection
✅ Integrations - Ansible - Copy Code Snippet Icon
✅ Integrations - Terraform - Copy Code Snippet Icon
✅ SDKs Tab Selection
✅ SDKs - Python - Copy Code Snippet Icon
✅ SDKs - Go - Copy Code Snippet Icon

I will create a bug ticket to investigate why event emitters are not firing consistently for resources.

Looks like events are firing for the resources links


Streaming API

Foremost, this is a terrific project.
I've been trying to integrate it with other apps, but the API is a little bit different compared to other implementations like KobolAI  and its API or textgen-webui and its API examples.
I could get it to work (while the webapp is running) with the following script with my limited knowledge, albeit it's not the best:
import requests
import json
import sys

url = 'http://<IP_ADDRESS>:5005/api/userinput'
data = {'user_input': 'What time is it? Write a very looong essay about time.'}
headers = {'Content-type': 'application/json'}

# send the POST request and stream the response
response = requests.post(url, data=json.dumps(data), headers=headers, stream=True)

# extract the text values from the JSON response
text_values = (json.loads(line).get('text') for line in response.iter_lines())
for text_value in text_values:
    print(text_value, end="")
    sys.stdout.flush() # flush the output buffer

What do you think about the possibility of making a streaming api endpoint on /api/stream that is not connected with the backend user handling and message saving, and is "stateless" so it follows the REST principles? Since it's one of the most performant backends this would surely boost its popularity.

There are some people already working on APIs. But it is on my list. I just need to do a little more research to figure out what the best, minimal stateless API would look like.

@bkutasi I have a (very) basic "stateless" API wrapper for exllama that might point you in the right direction:
https://github.com/disarmyouwitha/exllama/blob/master/fast_api.py
https://github.com/disarmyouwitha/exllama/blob/master/fastapi_chat.html
https://github.com/disarmyouwitha/exllama/blob/master/fastapi_request.py
fast_api.py is just a FastAPI wrapper around the model and generate_simple functions. It takes the -d command for the model directory. It will load the model and start listening on port 7862 for POST requests to http://localhost:7862/generate
You can go to /chat to load the HTML through FastAPI, which will allow you to load the page via browser.
fastapi_chat.html is a demo GUI written in HTML to be more accessible for editing, I stuffed everything into 1 file because I didn't want to clutter up the repo. It uses tailwind.css to make everything responsive (and mobile friendly)
fastapi_request.py is an example script of how to call the API from python.
This is just a quick implementation, I will actually be revisiting this code to work in some of the new improvements Turboderp made... after I get a bit of Diablo4 in this week ^^;

Your implementation looks great, I will try it out right away. Would love to see it merged down the line(in some form) into the main branch.

@disarmyouwitha your fast api is working great, but the web interface is not sending generation requests if its not accessed through the localhost, even when listening (<IP_ADDRESS>). Probably other requests are also not sent, but the page loads.
Basically everything jinja2 related to work but the other two does not.
Sorry for mentioning it here but i didn't see issue reporting active on your repo i hope turboderp wont mind it, otherwise lets move.

@bkutasi oh hm, I never noticed you had to enable issues - I have opened up the issues tab in my repo if you continue to have problems we can follow up there =]
Are you accessing the GUI by clicking the .html file, or by going to http://host:7862/chat?
If accessing it through the HTML file it will always assume localhost:
// Check if the page was loaded from FastAPI or opened independently
if (!window.location.href.startsWith("http://{{host}}:{{port}}/"))
{
    host = "localhost";
    port = "7862";
}

If accessing through /chat it should be trying to determine your host like this:
@app.get("/chat")
async def chat(request: Request, q: Union[str, None] = None):
    return templates.TemplateResponse("fastapi_chat.html", {"request": request, "host": socket.gethostname(), "port": _PORT})

(But maybe I was trying to be too clever and broke something)
I have the FastAPI running on a headless server, so I access the page like this:
http://wintermute:7862/chat
And in fastapi_requests.py I use:
r = requests.post("http://wintermute:7862/generate", json=data, stream=True)
It may be worth mentioning that you will probably need to forward port 7862 to access it from another machine:
sudo ufw allow 7862


PXT Microbit: Roll magnitude is wrong compared to physical microbit

v0.6.33
Input->Rotation Roll is producing values in the range -180 to +180 degrees, where as the physical Micro:bit produces values in the range -90 to +90 (Pitch correctly produces values in the range -90 to +90, as per the physical device).
Attached file demonstrates the issue. Only LEDs 1,1, 1,2, 2,1 and 2,2 should light up. The simulator also allows LEDs in column 0 and 3 to light up.
microbit-pxt_rotation_bug.zip

The simulator code maps the DAL runtime code. If you bring Z to zero, eg microbit standing vertically, you should see value ranging in the 180 for roll.

Okay, just about. The simulator makes it look like the board has only rotated by about 45 degrees though, which is confusing. The simulator board ought to rotate more to show this? And why is it not possible for pitch to rotate through the same distance? Hardware sensitivity issue?
The documentation says the value returned is from 0 to 360 degrees. But it's actually from -180 to 180 degrees. I guess that'll be another issue to submit haha.

It gets really hard to use the simulator when it tilts the full 90%, you can't see anything on it. Maybe we should be displaying x,y,z somewhere to make it obvious about the values. We do that in the microbit.co.uk simulator.

Agreed, 90 degrees would be impossible. But surely 60 or 70 would still be visible and make it obviously different from pitch? Or are they supposed to be in a similar range? I can't tell.
Displaying the values would be good too. The microbit.co.uk editor actually displays some very odd values that are not clamped to -180 to +180 : appears to be directly related to cursor x/y coordinates producing values in the range -1000 to +1000 (possibly larger if I had a bigger screen).

@pelikhan showing the values is also good for an informal "what's the range I'm interested in" and helping people understand that more intuitively which axis is which when testing.

Closing old issues. If this is still an issue please reopen.


All Leads should be on the Team page

Only very few of the Module leads are on the Team page.
http://arquillian.org/invasion/team/

I fixed that a while ago. Let me know if I'm missing someone.


Any chance you can put an MIT license on this?

Sure Matt. Thanks for pointing this out !

Done

Thank you!
On Wed, Oct 9, 2019 at 4:11 AM Ankur Choraywal<EMAIL_ADDRESS>wrote:

Closed #3 https://github.com/ankurnuk365/airflow-backfill-util/issues/3.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ankurnuk365/airflow-backfill-util/issues/3?email_source=notifications&email_token=ABA5EVWW2EPU63OWOE74XBLQNWGZRA5CNFSM4I6TZVC2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOUDISDRI#event-2698060229,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABA5EVXPATLJKZ7EQ2Y4ZOTQNWGZRANCNFSM4I6TZVCQ
.


Add Subtitle to Post

Hi, I was trying to add a subtitle in YAML front to my post (.Rmd) file. However, I cannot get a subtitle shown on my webpage even the YAML front contains subtitle. I use R Blogdown package to build the post. People from R Blogdown told me this was because the Hugo theme I was using doesn't support subtitle. Can I get any idea on how to get a subtitle?Thank you.

This theme supports subtitle for the webpage, but not the posts.
You need to modify the css file yourself. However, this theme is not fully compatible with blogdown, You can refer to another issue #45, which is also related to the theme customization using blowdown.
Unlike blowdown, after theme customization you need to run sass to regenerate the static folder in order to make it function.
I hope it will help.

Sorry for my very delayed response. Thank you for the info and I will look into it.


Feature/tasks

Changes:

Change some files with CRLF to LF to fix EOL
Add SecretsManager missing configuration in file application.ex

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add Google Assistant QUERY support for climate entities and support Fahrenheit.

Description:
Follow-up to #10174
If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] Tests have been added to verify that the new code works.

@balloob I'm not sure what your policy is on making point releases, but this PR seems to unexpectedly be a blocker for using Google Assistant with climate devices at all, so it would be great to get it released sooner than 0.58.

Another side note: for the device query, currently the operation_list from the device is used. However this is empty on my side as Zwave doesn't provide it. This leads to the problem that I cannot tell Google Assistant to "set heating to 25 degrees", as there's no device which includes heat in the modes list. "Set thermostat to 25 degrees" works fine though. Hard coding it to on,off,heat fixed it for me for now, but I'm not sure what the drawbacks might be.
Does anyone else also experience this or might it only be a German localization issue on Google's side? Should we make this configurable (similar to google_assistant_type) or add default values as well?

@sibbl I think that depends on the semantics of your device. Probably (once #10151 is merged), your device should have operation_list = ["on", "off", "heat"] and support setting and querying those if that's how it actually behaves in practice.

@emosenkis okay, but when the Zwave component simply doesn't provide it, there's no chance to add it from the configuration, or is there?

I would expect the zwave component to add it. If it were to be configurable, it should be there.

This looks like something that would benefit from standardizing the operation modes, e.g. as proposed in https://github.com/home-assistant/home-assistant.github.io/pull/3910 , am I correct?

Definitely. The PR mentioned there has also been referenced here several
times since standardizing the modes would make this integration much more
robust. This PR isn't blocked on that, though.
On Sun, Nov 12, 2017, 02:02 Lukas Barth<EMAIL_ADDRESS>wrote:

This looks like something that would benefit from standardizing the
operation modes, e.g. as proposed in
home-assistant/home-assistant.github.io#3910
https://github.com/home-assistant/home-assistant.github.io/pull/3910 ,
am I correct?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/pull/10346#issuecomment-343703434,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABtgTvThU3802wIHZ9cMpmQh-v6DIFGjks5s1jWIgaJpZM4QSGjX
.

Ok to merge when conflicts resolved 👍

Woohoo 🐬  🌮

@emosenkis @balloob I'm using a vera thermostat and I'm still having issues setting temperatures in Fahrenheit through Google Assistant. Is there something I'm missing from my config?
My config looks like this:
homeassistant:
  unit_system: imperial
vera:
  vera_controller_url: http://vera_controller:3480

Did you update to HA 0.58? In what way is it not working? Please provide any relevant logs, if applicable.

Ok, I'm not sure what happened but it's working now. I'm using docker and had updated the image - HA was reporting version 0.58.1, but maybe there was some old cache that caused it not to work. I also went into Google Assistant and removed the HA integration and added it back.

Please open an issue if you suspect a bug. If you need help please use our help channels:
https://home-assistant.io/help/#communication-channels
Merged PRs should not be used for support or bug reports. Thanks!


diagram

would be good to have a basic diagram shows the logical architecture of the system.
is some ways the flux patter is what Gu uses, as well as others.
So i am linking that diagram here to just show the idea:
https://github.com/facebook/flux
Making diagrams with https://github.com/jgraph/drawio-github is the way to go

Nope it doesnt use flux bro. No stores, just a pubsub system. Not much flux
in there to be honest.
On 18 Feb 2017 16:05, "gedw99"<EMAIL_ADDRESS>wrote:

would be good to have a basic diagram shows the logical architecture of
the system.
is some ways the flux patter is what Gu uses, as well as others.
So i am linking that diagram here to just show the idea:
https://github.com/facebook/flux
Making diagrams with
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/gu-io/gu/issues/24, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAICLdkly1IqMcOjEtfIrvlx4ChJ25jXks5rdwhhgaJpZM4MFJSp
.

Remember its a UI library that has no dataflow standards ;)
On 18 Feb 2017 16:05, "gedw99"<EMAIL_ADDRESS>wrote:

would be good to have a basic diagram shows the logical architecture of
the system.
is some ways the flux patter is what Gu uses, as well as others.
So i am linking that diagram here to just show the idea:
https://github.com/facebook/flux
Making diagrams with
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/gu-io/gu/issues/24, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAICLdkly1IqMcOjEtfIrvlx4ChJ25jXks5rdwhhgaJpZM4MFJSp
.

Ok.
Well the pubsub system is nice because it's highly adaptable, and I am
guessing this is your intent ?
If you make another example to show a bit more of the component composition
with some events between it would be good.
G
On Sat, 18 Feb 2017, 17:58 Ewetumo Alexander<EMAIL_ADDRESS>wrote:

Remember its a UI library that has no dataflow standards ;)
On 18 Feb 2017 16:05, "gedw99"<EMAIL_ADDRESS>wrote:

would be good to have a basic diagram shows the logical architecture of
the system.
is some ways the flux patter is what Gu uses, as well as others.
So i am linking that diagram here to just show the idea:
https://github.com/facebook/flux
Making diagrams with
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/gu-io/gu/issues/24, or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AAICLdkly1IqMcOjEtfIrvlx4ChJ25jXks5rdwhhgaJpZM4MFJSp
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gu-io/gu/issues/24#issuecomment-280858480, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ATuCwtTCHPwnGu114_9GEL09BjwGNJ82ks5rdyNMgaJpZM4MFJSp
.

You want an example of a parent component speaking to a component it
composes?
On 18 Feb 2017 19:30, "gedw99"<EMAIL_ADDRESS>wrote:

Ok.
Well the pubsub system is nice because it's highly adaptable, and I am
guessing this is your intent ?
If you make another example to show a bit more of the component composition
with some events between it would be good.
G
On Sat, 18 Feb 2017, 17:58 Ewetumo Alexander<EMAIL_ADDRESS>wrote:

Remember its a UI library that has no dataflow standards ;)
On 18 Feb 2017 16:05, "gedw99"<EMAIL_ADDRESS>wrote:

would be good to have a basic diagram shows the logical architecture of
the system.
is some ways the flux patter is what Gu uses, as well as others.
So i am linking that diagram here to just show the idea:
https://github.com/facebook/flux
Making diagrams with
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/gu-io/gu/issues/24, or mute the thread
<
https://github.com/notifications/unsubscribe-auth/
AAICLdkly1IqMcOjEtfIrvlx4ChJ25jXks5rdwhhgaJpZM4MFJSp
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gu-io/gu/issues/24#issuecomment-280858480, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ATuCwtTCHPwnGu114_
9GEL09BjwGNJ82ks5rdyNMgaJpZM4MFJSp>
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/gu-io/gu/issues/24#issuecomment-280865301, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAICLUffdmC596fNwnHYNjgHp6TnU-Hgks5rdzjTgaJpZM4MFJSp
.

Yes. Just something to show composition.
Just as a starting point.
On Sat, 18 Feb 2017 at 20:28 Ewetumo Alexander<EMAIL_ADDRESS>wrote:

You want an example of a parent component speaking to a component it
composes?
On 18 Feb 2017 19:30, "gedw99"<EMAIL_ADDRESS>wrote:

Ok.
Well the pubsub system is nice because it's highly adaptable, and I am
guessing this is your intent ?
If you make another example to show a bit more of the component
composition
with some events between it would be good.
G
On Sat, 18 Feb 2017, 17:58 Ewetumo Alexander<EMAIL_ADDRESS>wrote:

Remember its a UI library that has no dataflow standards ;)
On 18 Feb 2017 16:05, "gedw99"<EMAIL_ADDRESS>wrote:

would be good to have a basic diagram shows the logical architecture
of
the system.
is some ways the flux patter is what Gu uses, as well as others.
So i am linking that diagram here to just show the idea:
https://github.com/facebook/flux
Making diagrams with
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/gu-io/gu/issues/24, or mute the thread
<
https://github.com/notifications/unsubscribe-auth/
AAICLdkly1IqMcOjEtfIrvlx4ChJ25jXks5rdwhhgaJpZM4MFJSp
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gu-io/gu/issues/24#issuecomment-280858480, or
mute
the thread
<https://github.com/notifications/unsubscribe-auth/ATuCwtTCHPwnGu114_
9GEL09BjwGNJ82ks5rdyNMgaJpZM4MFJSp>
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/gu-io/gu/issues/24#issuecomment-280865301, or mute
the thread
<
https://github.com/notifications/unsubscribe-auth/AAICLUffdmC596fNwnHYNjgHp6TnU-Hgks5rdzjTgaJpZM4MFJSp
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gu-io/gu/issues/24#issuecomment-280868984, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ATuCwhmAYvLmrCEXer85QR4hO_zBY5y8ks5rd0ZvgaJpZM4MFJSp
.

This will be resolved on https://github.com/gu-io/example repo


Implement delivery receipts for PE Report emails

💡 Summary
All reports sent out by the P&E team should request delivery receipts.
Motivation and context
The report mailer is part of this project.  It is the code base that will need to be modified.
This would be useful so that we can tell that the stakeholders Email system isn't rejecting the reports.
Implementation notes
Acceptance criteria
Delivery receipts returned for the P&E reports

[ ] Criterion

Duplicate of Issue-216


Don't Serve routes at Root Route

Just cut the routes. I actually don't think it's useful, and it's a huge payload. Maybe serve a link to the docs instead?

done here https://github.com/CounterpartyXCP/counterparty-core/pull/1928/commits/6ba36840e596da627a1dd24420008a85cef16d37


Rate limiting on OAuth2 web application level (not end consumer) - How?

When configuration the rate-limiting plugin for an API, I currently have two options of doing it: Either for the entire API, or for a specific consumer (by specifying consumer_id when creating the plugin for the API).
When dealing with OAuth2 APIs using the Authorization Code Grant or Implicit Grant, things are a little more complicated though. Let's look at the implicit grant: There I have a web application (which has some kind of client_id to tell me who it is), and when I have authenticated and authorized the user (using my custom code), I will create a Kong Consumer with a registered OAuth2 application and issue a token (using the /oauth2/authorize Kong end point).
Now, if I want rate-limiting on this consumer, I add that using the method above (using consumer_id), and that's fine. But how would I go about rate limiting all consumers of this API for a specific web application? Given that I might have more than one web application which uses the same API?
Have I turned wrong in my thinking here somewhere, or is this nothing you would do? My use case is e.g. billed APIs on a per-web-app-basis, where I want to make sure each web app gets its "share" of the API, disregarding the amount of actual end users using it.
Would I solve this in a different way, granted I want to use the Implicit Grant Flow?
/Martin

I don't think I fully understand what you're trying to achieve, but the logic is that your consumer is the web application, not the end user (OAuth2 credentials - client_id, client_secret - are bound to application/API consumer, not to end user). When you set rate-limiting plugin to that application consumer, you actually limit usage of the API for all users of the application.

Hm. I guess you could do it like that, but I actually want the consumer to be the end user, as I need his identity after the API call has gone through the gateway.
I want to achieve this:

For each call to the API (which is a Web App API used from Angular 2), I want Kong to add the end user's identity (this I can make work)
I want (to be able) to do rate limiting over all traffic from a specific Web App (this I have not managed/I don't know how to do)

To get it the other way around would be quite trivial (rate limiting per consuming web app via registering a single consumer per web app), but then I lose the end user's authenticated identity. Or where's my problem in thinking here?

There are actually 2 identities bound to Bearer token when using either code grant or implicit grant:

consumer identity (usually meaning API consumer, e.g. web app) - identified by client_id & client_secret. This is passed from Kong to upstream as header X-Consumer-ID (and X-Consumer-Username / X-Consumer-Custom-ID if set)

end user identity (if you set your authentication accordingly), which is bind with token by calling /authorize Kong endpoint and is passed to upstream as X-Authenticated-Userid header.

Have a look at OAuth2 plugin documentation, which describes the OAuth2 code grant flow.

OK, thanks for clarifying this. Makes sense in a sense, but gives me a headache with something else. I would still like to be able to get both kinds of rate limiting. Both on a particular end user and on all users of a specific consumer (if you implement it the way you describe it).
I will have to go back to the drawing board a little for this one; if I do it the way I described it, I can pass along more information based on the access token (like licensing information in custom headers), otherwise I'd have to get that information from a different service in my backend.

See also #1759.

I don't quite know how I could miss that, but this is config.limit_by=credential.


Adding info handler, that contains two fields, num objects and memory…

Added an info handler which required updating Cargo.toml to include valkey-module-macros and linkme. The info handler contains two fields bloom_total_memory_bytes and bloom_num_objects. I then updated the creation and copy methods of bloom objects to increment these values where necessary. In order to deincrement added impl Drop for bloom in order to decrease the info fields as necessary.
Testing:
Created a new test file specifically for info. This test file contains for different tests:
test_basic_command_metrics: This tests all create commands and non create commands to check that create commands increase the info metrics while non create dont have an influence.
test_scaled_bloomfilter_metrics: Tests scaling a bloom filter increases the metrics as desired
test_copy_metrics: Tests copying a bloom filter will double the metrics as desired
test_save_and_restore_metrics: Tests saving and restoring a scaled and non scaled bloom filter will increase metrics as desired

Accidently closed the pr, reopening with the following changes:

Moved the info handler to a separate file as well as the info metrics.
Created a new metric for number of filters (Changed the other metric to actually be number of bloom objects)
Updated tests to include this new metric
Moved an increment of metrics to be after a possible thrown error


Spike: Investigate the design for handling panning components in terra-graphs

Feature Request
Description

As a developer, I want to land on a clear design for how we want to handle panning components in terra-graphs
So that we can proceed with the actual work to move that logic to terra-graphs
We'll know we're done when we have a clear idea on how to proceed and additional tasks are logged with information on what needs to be done

Concluded spike by establishing a rough design of incorporating panning in the react components. The design will use React Effects. The next part of this story has moved to #131 to start with a line component.


Clean up the RTE classify code

The RTEFeatureExtractor object in nltk.classify.rte_classify.py has functions outside the class object, it would be good clean up the code by:

having RTE relation extracting  function as staticmethod within the object
porting the demo code into proper unit test in nltk.test.unit

Patched in #1837


Just a kind reminder: please make a plan for .Net 6.0 support at a earlier time.

What makes many developers really unhappy is: .Net 5.0 was released Nov 2020. But SF did not add the support till April 2021, which is about half year delay.
Since .Net 6 is a LTS while .Net 5 is not, I think many people will move from .net 4 to .net 6 directly.
Per schedule, .Net 6.0 will be released on Nov 2021, https://dotnet.microsoft.com/platform/support/policy/dotnet-core. Please make a plan .Net 6.0 support at a earlier time. BTW, .Net 6.0 is in preview 4 now. https://devblogs.microsoft.com/dotnet/announcing-net-6-preview-4/
It would be great if SF .Net 6.0 support could be released together with .Net 6.0 this time.
Thanks.

If it takes until March to get the next LTS version of .NET it'll be sad times at Ridgemont High.
I also see there is interest in Hot Reload (The BIG .NET 6 feature) in #1189

Any idea when (which version) will support .net 6?

.NET 6.0 support is available


Issue with changing server

I installed using pip install PIdata recently. Although I am able to download PI data from the default server, I am unable to add the server as one of the input to the function and download the data. Error: got an unexpected keyword argument 'server'.
It is mentioned in the code that it is now revised to include this provision now.

Hi @Resmi-Suresh,
Thanks for raising the issue (and using PIdata)! You are correct. It seems like the pip version is out of date. I will rebuild it and then let you know when you can update.

PIdata version 0.3.1 is now available. It includes the ability to change server. I have also added some updates to the documentation. @Resmi-Suresh updating should fix the problem:
pip install PIdata --upgrade


Create a timeline-2.html

One need to make this component using HTML and Tailwind CSS. I would suggest to use Tailwind Playgrounds to make things faster and quicker.
Here is a reference to the component.

There would be border outside.
After building the component please raise a PR with a screenshot of the component and add the component in seamless-ui/Timeline/src/timeline-2.html.
If you need to use any icon please use it from Hero Icons
Good luck.

please assign it to me

This we can build @nik132-eng


Add options to customize the "decompiler's" output

Doktr right now assumes that NRT are enabled for example and also decompiles value tuples. However, this might not be something that the user wants, especially if their project doesn't use NRT (basically everything will have a ?, which is correct but might not be what the user wants). A couple of ideas (ranked in order of importance):

Option to disable NRT (the default should be for it to be enabled)
Option to change C# version(?) to keep the "raw" representation of value tuples ([TupleElementNames("One", "Two")] ValueTuple<string, string> instead of (string One, string Two)) and perhaps nullable value types (Nullable<int> instead of int?)

Option to disable NRT implemented in 1a4c418.


If showAlignments is disabled console is spammed with unexpected null receiver

A QCoreApplication::postEvent: Unexpected null receiver warning appears if the showAlignments option is disabled in the settings at the program start. This shouldn't happen and could (maybe?) result in something more nefarious?

Yep, that was bad. QPointer here was effectively a nullptr that I called methods on anyway. I'm a bit surprised it didn't just outright segfault…


Make CSRF failures less confusing

A CSRF failure currently returns a confusing error message:

form-urlencoded POST field did not match cookie

Would be good if that at least linked to relevant docs https://docs.datasette.io/en/stable/internals.html#csrf-protection - since these errors are most likely during development.
On Discord here: https://discord.com/channels/823971286308356157/823971286941302908/1273347775165235391
Relevant:

https://github.com/simonw/asgi-csrf/issues/28

I released asgi-csrf 0.10 with a new feature to enable this: https://github.com/simonw/asgi-csrf/releases/tag/0.10

New design:

Ooops... looks like this is served without text/html as the content-type!

Fixed:


Add web_page context schema

See https://github.com/snowplow/snowplow-javascript-tracker/issues/369 for discussion


Updated README and renamed spring5 profiles

Description

Motivation and Context

Fixes #194
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.

:+1:


Text size setting is not kept upon force close

After updating to v. 3.0.10-530 I noticed that the text size is not maintained over time.
I found that it is deterministically reproducible as follows:

Set text size to anything other than Medium;
Exit the app and verify the widget appearance has indeed changed as specified;
Force close the app;
Open the app and check the widget's text size settings;
Note that the size has reverted to Medium.

The effect might not be immediately visible on the widget as per refresh time, but the setting is not kept.
This happens also without force closing the app, perhaps by the battery optimisation process of android, the text setting gets reverted after some time probably due to that. Force closing the app is the immediate way to test this behaviour.

Hi @slabua I couldn't reproduce the same behaviour yet on my two devices.
Could you try, if other changed settings are lost also. Please change Future events background noticeably and repeat your steps.
I suspect that on your device preferences aren't fully stored at internal card on step 2, and this is why they are lost on "Force close the app".
In any case, so far I don't see, why "Text size" is so special.
Please try the same with other settings.

To all, who want to help,
Please try also the below test:
Test 3.

Change Text size of the widget to "Very small"
Press home button at make sure that changes applied.
Reboot your device normally (via reboot menu item in your device...)
Check if Text size persisted.

Hi @yvolk, I tried test 2 and 3.
I have no problems with test 2, as the settings are correctly saved.
Test 3, though, keeps failing.
I thought that the failing settings were only the radio buttons under Other settings, so I had a try also with the time format, but that one persisted fine.
For some reason, only the text size is not being saved for me.
I'm using a Pixel 3a with Android 10 (QP1A.190711.020.C3)
A little background: before the big-changes-version, it was working fine. After updating and noticing that the widget wasn't working at all, and reading that one had to completely remove the old version and reinstall the new version, I did as advised. So I'm pretty confident that any old data has been wiped during the reinstall process.
But if there's anything I should try, check presence/absence/permission of some files in some path, just let me know.

I found the bug :-), inspecting the code.
Text size scale was not stored properly (absence of strongly typed values...))
I could reproduce the bug in Android 9 emulator (test 3 above) and checked that it is fixed now.
What still puzzles me, is that test 3 passed successfully on my Android 7 tablet: Very small and very Large text stayed as is even after reboot... Need more investigation there...
Please test v.3.0.11 here: #320

@yvolk thank you, it works perfectly now : )

@slabua Thank you for your help!


Can't choose Coffee Lake when using 'Current Version'

i9-9900k + Asus Prime Z390-A
I was trying to fix Current Framebuffer. When I select Framebuffer-Current Version, I cannot choose 'Intel Generation - Coffee Lake', it jumps back to 'Ice Lake(LP)' every time.
This won't happen when using macOS 10.14

Same issue here 👀

You should set to Framebuffer->macOS 10.14. This setting uses the framebuffer dumps from 10.14 instead of from inside the framebuffer binaries which is what you need for >= macOS 10.14.


Testing utilities: Gas usage

An interesting utility that our user we'll want to access is the possibility of displaying gas consumption from their methods.
To build this feature we should get inspiration from the Gas report feature from Foundry.
We should add the capabilities for user to specify for which actors they wish to print gas reports. We can use the same configuration concept as it is done in Foundry.
Gas reports should be generated when testing. the returned value should be the average gas consumed by their methods over all testings.
Thomas' note
Gas consumption tracking for a given method can be done by parsing traces when a test method has been executed. We just need to remove the Gas consumption related to the logic written in the test method and only keep the gas consumed after a call is made to one of the target actor method.

Closed by #57


Fix misc issues and enable v8r64 FPU

This PR fixes some issues and enables the FPU for v8R AArch64

Fix text exception by increasing the stack size

Remove LOG print before MPU is enabled to avoid the potential un-alignment instruction called before MPU is enabled.

Fix the STACK_INIT during the reset

Add cpu_map to have a correct mapping of CPU id and MPID

Fix a deadlock case when using FPU in the spin-locked section (this workaround fixes modified the spinlock, RFC)
Using FPU with FPU sharing in a spin-locked critical section might cause a 'deadlock'. When core 0 got a spinlock that other cores are waiting for, it will 'deadlock' if the core 0 current threads' FPU context is living in another one that is waiting for the spinlock but never receives the FPU IPI. CC @npitre

Enable FPU and FPU_SHARING for v8r aarch64

Tested on all runnable Arm64 platforms including fvp_baser_aemv8r and fvp_baser_aemv8r_smp

@povergoing: The deadlock is a very interesting one.

I think this is ready for review :)


Fix formatting of ORDER BY for MariaDB

Also add a test for ORDER BY to avoid such regression in the future.
Fixes #298

A trivial fix. Merging straight in.


Can't find map data files

Whare are the csv files?
https://handsondataviz.org/peer-3-sample-2.html
...
 var runLayer = omnivore.csv('data.csv', null, customLayer)
...

@flywire
The repository for the My Leaflet map, with the data file, is here: https://github.com/HandsOnDataViz/leaflet-map-simple-instructor-sample


a few emoji display wrongly as blank white spots; may be missing

I was composing a message and tried out the emoji pane, and saw that a spot looked empty:

If I hovered over that spot I saw that it said :one:I tried using that emoji in a message and it displayed as blank white space (but if I hovered over it I saw :one:. After some experimentation I found several other emoji that acted similarly:
:one: :eight: :five: :four: :hash: :six: :three: :two: :zero:
Screenshots:

When I look up the emoji I'm having trouble with in tools/setup/emoji_dump/emoji_map.json, I see that they're all associated with integers starting with 00, e.g., "four": "0034", and none of the other emoji are. And I'm having trouble finding those emoji in tools/setup/emoji_dump/bitmaps/. So maybe those image files simply aren't in our bitmaps directory?

@umairwaheed can you investigate this one?

sure

OK I've figured out what's going on here; these emoji actually do exist, they're just white, and thus are essentially invisible against our white background.  You can sorta see them in a PM message.  I'm not sure what the right solution for these emoji is; it's possible we should either remove them or change their color.

My suggestion: change their color.

Yeah, I think that's reasonable; we'll likely need to do that via adding new code to tools/emoji_dump/build_emoji.


MySQL ODBC example

Demo MySQL ODBC connector based off our internal connector. Demo video shows connecting with and without SSL.

@rosswbrown You're right, connectionHelper.attributePort exists and works. We're just not using it in our other plugins. Changed it in the commit below.


Sync get latest referring params

do not merge, doing a little sanity check of countdownlatch w/ @Sarkar

We need the same implementation for getFirstReferringParams
Time out should be less than 5sec ignored to prevent ANR when called from UI thread
Test function should do it in sync way with out AsyncTask

Other than the above few things this LGTM. great 👍


Reset Query button error, Cannot set property 'value' of undefined"

Hi,
i have 2 tables in a different component and routing, let say user and product. in the first reload to user component the button its working fine, but when i change to  another route/product component, the reset button throwing  error Cannot set property 'value' of undefined" i need to refresh the current route to this button work correctly
im using version 1.1.2

Hi @2IhsanWp
This looks strange, I will give a try on my side and check what causes this error.
If you can make a codepen with this issue, then it would be easy for me to debug.
Cheers,
Ruby.

I have same issue, looking forward to hear about updates. Thanks!

Hi @2IhsanWp @hasangok
I've fixed this bug in the develop branch and tomorrow I'll make a new release with a couple of other fixes.
Please update your package tomorrow to avoid this error.
And, if you like this plugin please star this repository.
Cheers,
Ruby.

Hi @2IhsanWp @hasangok
This issue is fixed in version 1.1.4. So please update your package.
Cheers,
Ruby.


Could NOT find XCB_UTIL (missing: XCB_UTIL_LIBRARY XCB_UTIL_INCLUDE_DIR)

Build Process
Describe how you build polybar, list the exact commands you are using:
sudo ./build.sh from the polybar directory
If you build polybar directly from this repository:

Output of git describe --tags:
3.4.0-50-g1fe679d
If you use some other way (like the AUR):

List exactly where you are building from and which version you are building.
Build Log
Post everything that is output to the terminal while building polybar below. This HAS to include the output of the cmake and make commands, if you used them.
cmake:
** Fetching submodules

** Removing existing build dir (-f)

** Setting build options

** Use GCC even if Clang is installed ----------------------------- [y/N]: N
** Include support for "internal/i3" (requires i3) ---------------- [y/N]: N
** Include support for "internal/alsa" (requires alsalib) --------- [y/N]: N
** Include support for "internal/pulseaudio" (requires libpulse) -- [y/N]: N
** Include support for "internal/network" (requires libnl/libiw) -- [y/N]: N
** Include support for "internal/mpd" (requires libmpdclient) ----- [y/N]: N
** Include support for "internal/github" (requires libcurl) ------- [y/N]: N
** Build "polybar-msg" used to send ipc messages ------------------ [y/N]: N
** Using compiler: clang++/clang

** Executing cmake command

-- Trying to enable ccache
-- Couldn't locate ccache, disabling ccache...
-- The CXX compiler identification is Clang 8.0.0
-- Check for working CXX compiler: /usr/bin/clang++
-- Check for working CXX compiler: /usr/bin/clang++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- No build type specified; using Release
-- Using supported compiler Clang-8.0.0
-- Found PkgConfig: /usr/bin/pkg-config (found version "1.6.1")
-- Looking for C++ include pthread.h
-- Looking for C++ include pthread.h - found
-- Looking for pthread_create
-- Looking for pthread_create - not found
-- Check if compiler accepts -pthread
-- Check if compiler accepts -pthread - yes
-- Found Threads: TRUE
-- Checking for module 'cairo-fc'
--   Found cairo-fc, version 1.16.0
-- Checking for module 'xcb-composite'
--   Found xcb-composite, version 1.13.1
-- Checking for module 'xcb-xkb'
--   Found xcb-xkb, version 1.13.1
-- Checking for module 'xcb-randr'
--   Found xcb-randr, version 1.13.1
-- Checking for module 'xcb-randr>=1.12'
--   Found xcb-randr, version 1.13.1
--  Build:
--    Version: 3.4.0-50-g1fe679d
--    Type: Release
--    CXX: /usr/bin/clang++  -Wall -Wextra -Wpedantic -Wno-error=parentheses-equality -Wno-zero-length-array -O2 -DNDEBUG
--    LD: /usr/bin/ld
--  Install Paths:
--    PREFIX:  /usr/local
--    BINDIR:  /usr/local/bin
--    DATADIR: /usr/local/share
--    DOCDIR:  /usr/local/share/doc/polybar
--    MANDIR:  /usr/local/share/man
--  Targets:
-- [ ]   polybar-msg
-- [ ]   testsuite
-- [ ]   documentation
--  Module support:
-- [ ]   alsa
-- [ ]   curl
-- [ ]   i3
-- [ ]   mpd
-- [ ]   network (wireless-tools)
-- [ ]   pulseaudio
-- [X]   xkeyboard (1.13.1)
--  X extensions:
-- [X]   xcb-randr (1.13.1)
-- [X]   xcb-randr (monitor support) (1.13.1)
-- [X]   xcb-composite (1.13.1)
-- [X]   xcb-xkb (1.13.1)
-- [ ]   xcb-xrm
-- [ ]   xcb-cursor
--  Log options:
-- [X]   Trace logging
-- The C compiler identification is GNU 9.2.1
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Checking for module 'xcb-proto'
--   Found xcb-proto, version 1.13
-- Found PythonInterp: /usr/bin/python3 (found suitable version "3.7.5", minimum required is "3.5")
-- XCB[XCB]: Found component XCB
-- Found XCB_XCB: /usr/lib64/libxcb.so
-- XCB[ICCCM]: Found component ICCCM
-- Found XCB_ICCCM: /usr/lib64/libxcb-icccm.so
-- XCB[EWMH]: Found component EWMH
-- Found XCB_EWMH: /usr/lib64/libxcb-ewmh.so
-- Could NOT find XCB_UTIL (missing: XCB_UTIL_LIBRARY XCB_UTIL_INCLUDE_DIR)
-- XCB[IMAGE]: Found component IMAGE
-- Found XCB_IMAGE: /usr/lib64/libxcb-image.so
CMake Error at /usr/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:137 (message):
  Could NOT find XCB (missing: XCB_UTIL_FOUND)
Call Stack (most recent call first):
  /usr/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:378 (_FPHSA_FAILURE_MESSAGE)
  lib/xpp/cmake/FindXCB.cmake:245 (find_package_handle_standard_args)
  lib/xpp/CMakeLists.txt:28 (find_package)

-- Configuring incomplete, errors occurred!
See also "/home/tatu/projects/polybar/build/CMakeFiles/CMakeOutput.log".
See also "/home/tatu/projects/polybar/build/CMakeFiles/CMakeError.log".
** Failed to generate build... read output to get a hint of what went wrong

Environment:

Distro (with Version):
Fedora 30.

Additional context
Add any other context that you think is necessary about the problem here.

Do you have xcb-util-devel installed?

apparently I didn't have. thank you very much, the build succeeded now.


Update MacSpice to 3.1.7

Thank you for the contribution. It was merged directly as 8b99bc202eb3a3680700af933d366084766ccf7e to keep commit history cleaner. Your contribution is still credited to you.