Vector matching with logical operator doesn't report many-to-many error

Example input:
vector1{x = "a", y="foo"}
vector1{x = "a", y="bar"}

What did you do?
vector1 and on(x) vector1
What did you expect to see?
Error because there's many-to-many results.
What did you see instead? Under which circumstances?
It works - not really sure what the result is...
Note: vector1 / on(x) vector1 reports the error correctly: many-to-many matching not allowed: matching labels must be unique on one side"
Environment

System information: Linux 4.19.0-0.bpo.6-amd64 x86_64
Prometheus version: 2.22.2

It makes more sense to ask questions like this on the prometheus-users mailing list rather than in a GitHub issue. On the mailing list, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.

I wasn't intending to ask a question, but instead was submitting this as a bug report. I only now noticed that the documentation says:
Grouping modifiers can only be used for comparison and arithmetic. Operations as and, unless and or operations match with all possible entries in the right vector by default..
Surprising, but I suppose it's a "feature not a bug" bucket.


Add Ruby as a flair

This PR adds ruby as a flair, resolves #50, and as mentioned in PR #22 I put the SVG through SVGOMG.

Just made a change for how I handle flair, see https://github.com/benawad/vsinder/issues/70


Create a garbage collector for old Draft events

When any user clicks the button to create a new meeting, a new record is created in the Event tables with the State set to DRAFT. These records will stay forever in there if we don't clean them.
We should have a flow that works like a garbage collector, each day, it should delete old records that are still in DRAFT mode.
I'm thinking about deleting any draft that is 30 days or older.

The Release Planner uses DRAFT meetings, so we need to be careful when we delete events.

See if we should expand this to also completed meetings after 1 year of completed. Make sure we do soft/hard deletion strategy here


Clarification on deprecated withServiceAccount() and withApiKey() methods

1. Which version of kreait/firebase-php are you using?
Version 3.2
2. Which version of PHP are you using?
PHP 7.1.9
3. What's the issue?
Methods like withServiceAccount() and withApiKey() were deprecated in favour of new method withServiceAccountAndApiKey(). I'm only using firebase database feature, which can be accessed by using service account credentials, but the new method forces me to pass a web api key. Still I can work with this new method by passing an empty string instead of api key.
Could you tell me why those methods are deprecated?
It doesn't make sense with method name which has service account & api key but I only pass one proper value.
Can you bring back those deprecated methods?

I was planning on removing those methods in the next major version to reduce some complexity in the library. At the moment, I have to check whether the API key is set when Auth features are being used. Performing requests in the name of a given user(-id) is more complex this way as I have to maintain two different authentication methods.
But I understand and respect your valid point and will undeprecate the methods in the next minor release.

@jeromegamez Thanks for your understanding. Eagerly waiting for next minor release.

@vigneshgurusamy Here it is: https://github.com/kreait/firebase-php/releases/tag/3.2.1 \o/


Add model code for putting items on a MergedSierraRecord

What is this PR trying to achieve?
This creates a new class SierraItemRecord, which is currently identical to SierraBibRecord but is a different type for the purposes of type safety.
It adds a way to store item records on a MergedSierraRecord, and adds a new method mergeItemRecord(), which adds information from a new item to the merged record.
This patch only covers the model changes; exercising the model outside our tests is beyond the scope of this patch.
Required for #1195.
Who is this change for?
Anybody who wants more than bib data from Sierra.

I think this is ready for review.


Update desktop-accessibility-keyboard-shortcuts.md

Added missing shortcuts to the DAX formula bar

@davidiseminger
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


README content is not updated after refresh

https://gowalker.org/gopkg.in/ini.v1

Note: only happens with gopkg.in/xx path, it works with github.com/go-ini/ini.


Add requiresObjCLinking to Target

Child of #350. Addresses 2.ii.

I'm not convinced all this complexity and performance hit is worth it. Is there a big downside to either just adding OTHER_LDFLAGS: "$(inherited) -ObjC" to every target by default, or at least to targets that have library.static target dependencies?

Great feedback. I’ll adjust it to not look at the sources and just default the value to true for static libraries. I’ll clean up the documentation so it’s bettter understandable by the average person (and why you would want to adjust the default). I’ll also move this back into their proper places.
I want to keep the property though, since I see it being added to the carthage and framework dependency types for static frameworks.
Ohh, and to answer your question about just putting this on targets: there is a very real binary size hit you take if you use this option without he need for it. I’ll make sure the documentation reflects this.

@yonaskolb Addressed your feedback. Per the other PR's comments, I'll add new commits instead of amending next time.


Tor Source for debian

https://github.com/rootzoll/raspiblitz/blob/4ead6f00d0d53beaeab5bb7209cb854dd5195f34/build_sdcard.sh#L206-L220
Any special reason why  debian is not covered?

As identified here: https://github.com/rootzoll/raspiblitz/issues/1752. I think we should add [arch=arm64] to this when appropriate (64bit OS)

OK for v1.7.2 the build_sdcard.sh is using the new tor.install.sh script by @nyxnor to install Tor for all baseimages from the debian repos. To be final tested if also is working will all images.

@nyxnor got this error during sd card build - do we still need to adapt the install? How best to check if Re-Install was successfull? And manybe add a automatic check & error exit if unsucessfull?
Err:5 tor+http://apow7mjfryruh65chtdydfmqfpj5btws7nbocgtaovhvezgccyjazpqd.onion/torproject.org bullseye InRelease The following signatures couldn't be verified because the public key is not available: NO_PUBKEY 74A941BA219EC810

Full log in context:
*** Adding Tor Sources ***

deb [arch=arm64] tor+http://apow7mjfryruh65chtdydfmqfpj5btws7nbocgtaovhvezgccyjazpqd.onion/torproject.org bullseye main
deb-src [arch=arm64] tor+http://apow7mjfryruh65chtdydfmqfpj5btws7nbocgtaovhvezgccyjazpqd.onion/torproject.org  bullseye main

- OK sources added

*** Reinstall ***
Hit:1 http://deb.debian.org/debian bullseye InRelease
Hit:2 http://deb.debian.org/debian bullseye-updates InRelease
Hit:3 http://security.debian.org/debian-security bullseye-security InRelease
Hit:4 http://archive.raspberrypi.org/debian bullseye InRelease
Get:5 tor+http://apow7mjfryruh65chtdydfmqfpj5btws7nbocgtaovhvezgccyjazpqd.onion/torproject.org bullseye InRelease [3,526 B]
Err:5 tor+http://apow7mjfryruh65chtdydfmqfpj5btws7nbocgtaovhvezgccyjazpqd.onion/torproject.org bullseye InRelease
  The following signatures couldn't be verified because the public key is not available: NO_PUBKEY 74A941BA219EC810
Reading package lists... Done
W: http://deb.debian.org/debian/dists/bullseye/InRelease: The key(s) in the keyring /etc/apt/trusted.gpg.d/torproject.gpg are ignored as the file is not readable by user '_apt' executing apt-key.
W: http://deb.debian.org/debian/dists/bullseye-updates/InRelease: The key(s) in the keyring /etc/apt/trusted.gpg.d/torproject.gpg are ignored as the file is not readable by user '_apt' executing apt-key.
W: http://security.debian.org/debian-security/dists/bullseye-security/InRelease: The key(s) in the keyring /etc/apt/trusted.gpg.d/torproject.gpg are ignored as the file is not readable by user '_apt' executing apt-key.
W: http://archive.raspberrypi.org/debian/dists/bullseye/InRelease: The key(s) in the keyring /etc/apt/trusted.gpg.d/torproject.gpg are ignored as the file is not readable by user '_apt' executing apt-key.
W: tor+http://apow7mjfryruh65chtdydfmqfpj5btws7nbocgtaovhvezgccyjazpqd.onion/torproject.org/dists/bullseye/InRelease: The key(s) in the keyring /etc/apt/trusted.gpg.d/torproject.gpg are ignored as the file is not readable by user '_apt' executing apt-key.
W: GPG error: tor+http://apow7mjfryruh65chtdydfmqfpj5btws7nbocgtaovhvezgccyjazpqd.onion/torproject.org bullseye InRelease: The following signatures couldn't be verified because the public key is not available: NO_PUBKEY 74A941BA219EC810
E: The repository 'tor+http://apow7mjfryruh65chtdydfmqfpj5btws7nbocgtaovhvezgccyjazpqd.onion/torproject.org bullseye InRelease' is not signed.
N: Updating from such a repository can't be done securely, and is therefore disabled by default.
N: See apt-secure(8) manpage for repository creation and user configuration details.
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
tor is already the newest version (<IP_ADDRESS>-1~deb11u1).
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
apt-transport-tor is already the newest version (0.5).
curl is already the newest version (7.74.0-1.3+deb11u1).
gpg is already the newest version (2.2.27-2).
nyx is already the newest version (2.1.0-2.1).
obfs4proxy is already the newest version (0.0.8-1+b6).
python3-stem is already the newest version (1.8.0-3).
torsocks is already the newest version (2.3.0-3).
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.

*** PREPARING BITCOIN ***

The gpg needs to be dearmored.
--dearmor
              Pack or unpack an arbitrary input into/from an OpenPGP ASCII armor.

It is currently like this https://github.com/rootzoll/raspiblitz/blame/dev/home.admin/config.scripts/tor.install.sh#L142
torsocks curl -s https://deb.torproject.org/torproject.org/A3C4F0F979CAA22CDBA8F512EE8CBC9E886DDD89.asc | sudo gpg --no-default-keyring --keyring gnupg-ring:/etc/apt/trusted.gpg.d/torproject.gpg --import

See the original method: https://github.com/rootzoll/raspiblitz/blame/e7f04fd5d5ada068c316d5d54983a0c988e817f7/home.admin/config.scripts/tor.install.sh#L47
torsocks curl --tlsv1.3 --proto =https --connect-timeout 10 https://deb.torproject.org/torproject.org/A3C4F0F979CAA22CDBA8F512EE8CBC9E886DDD89.asc | gpg --dearmor | sudo tee /usr/share/keyrings/tor-archive-keyring.gpg >/dev/null

I would change this now to:
tor_deb_repo_clean="${tor_deb_repo#*tor+}"
curl -s -x socks5h://<IP_ADDRESS>:9050 --tlsv1.3 --proto =https --connect-timeout 10 "${tor_deb_repo_clean}"/torproject.org/A3C4F0F979CAA22CDBA8F512EE8CBC9E886DDD89.asc | gpg --dearmor | sudo tee /etc/apt/trusted.gpg.d/torproject.gpg >/dev/null

This will avoid the need to use torsocks if it failed to install before and save the key dearmored inside the apt/trusted.gpg.d folder. This will also use the same url from the tor_deb_repo_clean but without the preprending tor+ needed for apt, as now this is the curl process.

@rootzoll which branch are you on so I can commit?

That sd card build was on dev branch

PR #2845 merged ... TODO: restest sd card build

sd catd builded fine for latest RCs ... closing issue.


[OAP-1680][OAP-CACHE] Decouple FileFormatDataWriter, FileFormatWriter and OutputWriter

What changes were proposed in this pull request?
This pr is sub-task of #1337, use to decouple FileFormatDataWriter, FileFormatWriter and OutputWriter we copy from spark, copies of them are no longer needed after we discard oap data format
How was this patch tested?
GitHua Action Passed

@LuciferYang
Let's  merge this?

@zhixingheyi-tian  oap-Rpmem-shuffle not pass,  retest this first, will merge if it passing

@zhixingheyi-tian done ~


Custom boxShadow theme in tailwind config doesn't properly override default tailwind theme on mobile (iOS and Android).

Describe the bug
Custom boxShadow theme in tailwind config doesn't properly override default tailwind theme on mobile (iOS and Android).
To Reproduce

Define a tailwind theme with custom boxShadow styles

export default {
  theme: {
    boxShadow: {
       // shadow-red works as expected, as it doesn't conflict with the default tailwind naming for shadows
       red: '0px 40px 100px rgba(255, 0, 0, 0.1)',
       // shadow-lg shows the default tailwind shadow-lg, rather than the green one defined below.
       lg: '0px 40px 100px rgba(0, 255, 0, 0.1)',
       // shadow-sm, shadow-xl, etc. are still available on mobile, despite this being a theme override rather than a theme extension
    }
  }
}

Apply the classes and observe the visible difference on web vs mobile.

<View className="shadow-lg bg-elevation-base h-[150px] w-[150px] border border-black"><Text>shadow-lg</Text></View>
<View className="shadow-red bg-elevation-base h-[150px] w-[150px] border border-black"><Text>shadow-red</Text></View>

Expected behavior
Expect mobile to behave the same as web: all default tailwind shadow sizes become unavailable, and the shadows should show as defined in the theme.
Screenshots
I've attached the expected and actual visual results (expected is on web, actual is on iOS).
Expected:

Actual:

The shadow-lg class on mobile should be applying the massive green shadow, like we see on web, but instead, it seems to apply the tailwind default for shadow-lg.

Fixed in 4.0.29.


Add support for distributed tracing

This is a larger change. I tried this with Quarkus 1.2 but failed (most likely pilot error), so I pulled in 1.3 where I got it to work.

bump Quarkus to 1.3.0.Final
+- Change test setup to use QuarkusTestResource
Pull in MP-OpenTracing and set it up in application.properties.
Separate out Rbac retrieval from IncomingRequestFilter, as otherwise it would not be part of the outer span (that only starts when the endpoint is hit, but the IRF is a @PreMatch filter, which runs before
Implement a Rbac cache: This is a simple annotation that creates the cache. Entries expire after 2mins (see application.properties)

With engine also instrumented :

Signed-off-by: Heiko W. Rupp<EMAIL_ADDRESS>

Note: I pulled the entire team in as reviewer just as a FYI


How can I use it in an offline env?

hello, I want to know how to replace the datasource by my own data, i have a private openstreetmap server in a offline environment, and I want to know how to use it.

You would have to generate some vector tiles using https://github.com/StrandedKitty/planetiler and serve them locally, and update the vector tiles URL in the config: https://github.com/StrandedKitty/streets-gl/blob/dev/src/app/Config.ts#L143

@ch0s1n Did you manage to run streets-gl with your own local data?


No traces for go applications even though process has found by instrumentation container.

I have configured otel auto-instrumetation for three of my go services.
I am getting below logs for each of them:
{"level":"info","ts":1706790698.5967553,"logger":"go.opentelemetry.io/auto","caller":"cli/main.go:94","msg":"building OpenTelemetry Go instrumentation ...","globalImpl":false}
{"level":"info","ts":1706790700.5980866,"logger":"Instrumentation.Analyzer","caller":"process/discover.go:69","msg":"found process","pid":19}
But after this I don't see any logs, even though I hit the api calls for my go services, I don't see any traces getting generated.
I am using below image.
image: ghcr.io/open-telemetry/opentelemetry-go-instrumentation/autoinstrumentation-go:v0.10.1-alpha
also I have set below env variables:

name: OTEL_SERVICE_NAME
value: target-service-name
- name: OTEL_EXPORTER_OTLP_ENDPOINT
value: http://my-collector-service:13575
- name: OTEL_TRACES_EXPORTER
value: otlp

Environment

OS: LINUX
Go Version: 1.7.5
Version: v0.10.0-alpha
more system info:
/ $ uname -r
5.10.205-195.804.amzn2.x86_64
/ $ uname -m
x86_64
/ $

is your go version really 1.7.5? If so that might be the problem. The logging output hanging at found process means it's stuck looking for functions to instrument

is your go version really 1.7.5? If so that might be the problem. The logging output hanging at found process means it's stuck looking for functions to instrument

Yes it is 1.7 and 1.7.5. The go apps which I am trying to instrument is developed  long back. so can't we auto-instrument it as the go version is 1.7?

@jyoti-borade that might be it. Could you please share the full output of go version so I can check?
The real bug is that whatever is causing the instrumentation to fail is not reporting an error. What you're seeing is actually the code hanging trying to report the error. I proposed a fix for this here: https://github.com/open-telemetry/opentelemetry-go-instrumentation/pull/638. You could try building with that PR to see what the exact error is you're getting

@jyoti-borade that might be it. Could you please share the full output of go version so I can check?
The real bug is that whatever is causing the instrumentation to fail is not reporting an error. What you're seeing is actually the code hanging trying to report the error. I proposed a fix for this here: #638. You could try building with that PR to see what the exact error is you're getting

https://github.com/microservices-demo/catalogue/tree/master  This is the repository for the Go application I'm trying to auto-instrument—an open-source sample microservice developed in Go. The specified Go version in the Dockerfile is 1.7. I'm attempting to auto-instrument the publicly available image [version](image: weaveworksdemos/catalogue:0.3.5). Now, I'll check if I can retrieve the actual error details with the mentioned PR.

@damemi With the new build( #638 ) I am getting below error:
{"level":"info","ts":1706858639.801028,"logger":"go.opentelemetry.io/auto","caller":"cli/main.go:94","msg":"building OpenTelemetry Go instrumentation ...","globalImpl":false}
{"level":"info","ts":1706858641.8052044,"logger":"Instrumentation.Analyzer","caller":"process/discover.go:70","msg":"found process","pid":7}
{"level":"error","ts":1706858641.8076577,"logger":"go.opentelemetry.io/auto","caller":"cli/main.go:103","msg":"failed to create instrumentation","error":"not a Go executable","stacktrace":"main.main\n\t/app/cli/main.go:103\nruntime.main\n\t/usr/local/go/src/runtime/proc.go:267"}
{"level":"info","ts":1706858727.7907786,"logger":"go.opentelemetry.io/auto","caller":"cli/main.go:94","msg":"building OpenTelemetry Go instrumentation ...","globalImpl":false}
{"level":"info","ts":1706858729.8552575,"logger":"Instrumentation.Analyzer","caller":"process/discover.go:70","msg":"found process","pid":7}
{"level":"error","ts":1706858729.860884,"logger":"go.opentelemetry.io/auto","caller":"cli/main.go:103","msg":"failed to create instrumentation","error":"not a Go executable","stacktrace":"main.main\n\t/app/cli/main.go:103\nruntime.main\n\t/usr/local/go/src/runtime/proc.go:267"}
I have set below env variable:

name: OTEL_GO_AUTO_TARGET_EXE
value: /app

and 'app' is the go executable present in home directory of container.

Looks like that error comes from getGoDetails() when trying to parse the go build info. I think that the go version or build environment is the issue, but it seems like a bug in our code to not handle it.
@edeNFed it looks like a lot of this was copied from the Go buildinfo package, is there any reason not to use that package directly? Wonder if the extra handling they do supports older versions like this but it's a little over my head

This version is not supported
For reference: https://github.com/open-telemetry/opentelemetry-go-instrumentation/tree/main?tab=readme-ov-file#compatibility

@pellared we should clarify the readme that the go support is for instrumenting applications and for building the auto-instrumentation binary. I don't think it's clear right now, and our offsets track way earlier than the previous 2 go versions (xref https://github.com/open-telemetry/opentelemetry-go-instrumentation/issues/639 for dropping this support)

@edeNFed it looks like a lot of this was copied from the Go buildinfo package, is there any reason not to use that package directly? Wonder if the extra handling they do supports older versions like this but it's a little over my head

Using buildinfo package makes total sense, I don't remember the reason for not using it in the first place.
Also, just noticed go1.22 has a new go/version package we can probably leverage as well.

Ok, I think we can close this issue as the version of Go in the original post is not supported. I'll open another issue to track refactoring that code to use buildinfo.


ObjectProjectionSelector schema is not correct

The example for a KnowledgeStore object includes a key property that is currently being rejected by api-version=2019-05-06-preview:
The request is invalid. Details: skillset : The property 'key' does not exist on type 'Microsoft.Azure.Search.ObjectProjectionSelector'. Make sure to only use property names that are defined by the type.

When I remove the key field, the request succeeds, but I can see that there are 2 new undocumented properties on a subsequent GET: inputs: [] and sourceContext: null. I'm guessing these serve the same function as they do on skill inputs?
Is it possible to include a full schema reference for the new types?

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

ID: ebfca41a-9842-5c60-672a-307ccb8bef6a
Version Independent ID: 803b4775-cb02-604b-efd3-4cc2d1826826
Content: How to get started with Knowledge Store (preview) - Azure Search
Content Source: articles/search/knowledge-store-howto.md
Service: search
GitHub Login: @HeidiSteen
Microsoft Alias: heidist

Preview is upper case --- 2019-05-06-Preview.

Ill try it, but everything else works with this version after I remove “key”, including GET and PUT

Yes, but I think its working because skillset in the 2019-05-06 (GA version) supports those properties.  The key, source, sourceField are only available in the Preview version.

IIRC I got “unknown type” when I tried to create a KnowledgeStore before changing the version. Isn’t the store preview only now? Or is it GA?

It is preview-only. It worked last week but maybe something broke.  I will test again myself.

Circling back on this thread, I confirm there is a problem with the sample Postman collection, which is getting fixed.

@ctstone No rush but when you get a chance to try out the new collection, could you mark this issue as closed??  https://github.com/HeidiSteen/azure-search-postman-samples/tree/master/Caselaw

Updated object sample looks correct now, closing


You play as a little archeologist navigating an environment to push buttons which open paths to artifacts.
How do you play your game?
WASD to move. Navigate to the red buttons which open paths/removes barriers to reach the objects locked behind them. Reach all the objects in the level to progress to the next and ultimately win.
Code
Check off the items that are true.

[x ] The game was made using the Sprig editor.
[x ] The game is placed in the in the /games directory.
[ x] The code is significantly different from all other games in the Sprig gallery (except for games labeled "demo").
[x ] The game runs without errors.
[x ] The name of the file/game contains only alphanumeric characters, -s, or _s.
[x ] The game name is not the same as the others from gallery

Thanks for your PR!

Neat game!  I'd recommend you extend this a little bit and add a map or two sometime in the future!  That being said, the gameplay you did implement was pretty clean / fun!
If you're a teen, please fill out this form for your very own Sprig device!  Also, feel free to join Hack Club's Slack and show of your game in the #sprig channel!

If you're a teen, please fill out this form for your very own Sprig device! Also, feel free to join Hack Club's Slack and show of your game in the #sprig channel!

Hi thank you so much for your feedback! I think you might not have linked the form? Also to update the game, is it the same process as this? Thank you again :)


Checking userinfo with the uri method

Fixed the basic auth decision in the http client so that it is not affected by ReDoS, which occurs in Ruby 3.1 and below.
close: https://github.com/q9f/eth.rb/security/code-scanning/10
Ruby 3.1.2
irb(main):001:0> RUBY_VERSION
=> "3.1.2"
irb(main):002:0> measure
TIME is added.
=> nil
irb(main):003:0>  /:.*@.*:/i =~ "https://example.com" * 1000
processing time: 0.069204s
=> nil
irb(main):004:0>  /:.*@.*:/i =~ "https://example.com" * 10000
processing time: 4.696144s
=> nil
irb(main):005:0>  /:.*@.*:/i =~ "https://example.com" * 50000
processing time: 119.181579s
=> nil

Ruby 3.2.0
irb(main):005:0> RUBY_VERSION
processing time: 0.000062s
=> "3.2.0"
irb(main):006:0> measure
processing time: 0.000090s
=> nil
irb(main):007:0>  /:.*@.*:/i =~ "https://example.com" * 1000
processing time: 0.001817s
=> nil
irb(main):008:0>  /:.*@.*:/i =~ "https://example.com" * 10000
processing time: 0.019022s
=> nil
irb(main):009:0>  /:.*@.*:/i =~ "https://example.com" * 100000
processing time: 0.101864s
=> nil

Codecov Report

Merging #233 (b520064) into main (3c3c806) will not change coverage.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main     #233   +/-   ##
=======================================
  Coverage   99.66%   99.66%
=======================================
  Files          77       77
  Lines        4432     4432
=======================================
  Hits         4417     4417
  Misses         15       15

Impacted Files
Coverage Δ

lib/eth/client/http.rb
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Added my portfolio website

Good work. I am merging your PR.
Do give us a star by hitting the star button of this repo!


Show Estimated Time of End of Work for current Day

Is your feature request related to a problem? Please describe
Yep, I have to manually check time left to work actively to know at which hour i will end work.
Describe the solution you'd like
Simple caption with estimated end of work moment.
Describe alternatives you've considered
Leave as is.
Additional context
N/A

Closed by #54


Base class BaseApi.php properties like $this->baseUrl have no value in subclasses

When I make some local fixes so that I can get to the point where I can instantiate a MySportsFeed object for version 2.0, I get an object in which the base properties have not been filled in.
My code
   $msf = new MySportsFeeds("2.0", true, null, '');
   echo print_r($msf, true);

gives me this:
MySportsFeeds\MySportsFeeds Object
(
    [buildVersion] => 2.0.0
    [version:MySportsFeeds\MySportsFeeds:private] => 2.0
    [verbose:MySportsFeeds\MySportsFeeds:private] => 1
    [storeType:MySportsFeeds\MySportsFeeds:private] =>
    [storeLocation:MySportsFeeds\MySportsFeeds:private] =>
    [apiInstance:MySportsFeeds\MySportsFeeds:private] => MySportsFeeds\API_v2_0 Object
        (
            [auth:protected] =>
            [baseUrl:protected] =>
            [verbose:protected] =>
            [storeType:protected] =>
            [storeLocation:protected] =>
            [storeOutput:protected] =>
            [version:protected] =>
            [validFeeds:protected] => Array
                (
                    [0] => seasonal_games
                    [1] => daily_games
                    ...
                    [21] => players
                    [22] => seasonal_standings
                )
        )
)

Note in the "apiInstance" object, that the properties (like auth, baseUrl, ...) which are only ever initialized in BaseApi, are all blank (uninitialized?).
When I try to getData() using the MySportsFeed object, the CURL url has no "baseUrl" portion. It is missing the starting "https://api.mysportsfeeds.com/..." portion.
I was able to correct this by making sure that every subclass of BaseApi had an explicit constructor, and all of those subclass constructors calls the parent constructor first thing:
parent::__construct($version, $verbose, $storeType, $storeLocation)
I don't know if my solution was the best or proper way to fix this.

This issue can be closed, it was handled in pull request #14 Won't-run-due-to-PHP-errors


RandomScale function comment section

class RandomScale(DualTransform):
"""Randomly resize the input. Output image size is different from the input image size.
Args:
scale_limit ((float, float) or float): scaling factor range. If scale_limit is a single float value, the
range will be (1 - scale_limit, 1 + scale_limit). Default: 0.1.
interpolation (OpenCV flag): flag that is used to specify the interpolation algorithm. Should

In the above comment section should not it be
range will be (1 + scale_limit, 1 + scale_limit)
rather than
range will be (1 - scale_limit, 1 + scale_limit)
?
because the implementations of versions 0.2.2 and 0.2.3 just add your scale limit to 1.
I had scale limit (0.5 - 1.5) and i ended up scaling everything to (1.5-2.5)
so to make it work properly i had to give as input (-0.5, 0.5)

At last version works properly.


where is my MYX

Is there a way for me to contact Monyx?
I bought in the ICO at 0.50 but I see that it all got sold on the exchange for like 0.12, on Feb 7th.

they are gone buddy...
On Mon, Feb 19, 2018 at 10:27 PM, jakbtc<EMAIL_ADDRESS>wrote:

Is there a way for me to contact Monyx?
I bought in the ICO at 0.50 but I see that it all got sold on the exchange
for like 0.12, on Feb 7th.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/monyxcoin/monyx-coin/issues/256, or mute the thread
https://github.com/notifications/unsubscribe-auth/AijuYtqLx8EKvmT7QFo9lUIgAW_v3Zl7ks5tWagDgaJpZM4SK2bt
.

they will exit soon

@jakbtc what is your username? There was a bug in the system on 7th Feb.


Doors are aligned.

Describe the bug
When the mission graph is disabled, doors connected to passages are aligned. The issue is that the passages have different identifiers, making them distinct at the ID level. Please consider some method, such as interrupting passage generation midway when a passage leading to the desired room is found.
Screenshots

Sometimes, even though passages are supposed to be shared, they are not. The program needs a solution, such as searching for shared gates before generating passages or discarding unnecessary gates after passage generation.

Ver. 1.6.6 has been released.


Nuget install disables listviews' context action  in Android

Like the title says, this Nuget disables the context actions in listviews on android (ios still works). Just by installing it!!
Thats very sad, because aside from this issue this CarouselView is awesome.
Update.
This issue only appears in Debug-Mode. After distributing and installing the apk-file on my Android phone it is working.

This control has been merged into Xamarin.Forms master branch. Please, wait for it to be released as part of Xamarin.Forms and submit any issues there https://github.com/xamarin/xamarin.forms/tree/feature-carouselview. Thanks


Example page totally borked

Chrome 41.0.2272.118 on OSX 10.10

Thanks for the info! Will be fixed soon.

Fixed.


[Backport] Integration tests for Kafka & Kinesis ingestion

Backport of https://github.com/apache/druid/pull/9576 to 0.18.1.
Backport of https://github.com/apache/druid/pull/9724 to 0.18.1.

This PR only modify files in integration-tests with the exception of:

.travis.yml : to add the new integration tests to our Travis CI pipeline
docs/development/extensions-core/kinesis-ingestion.md : to fix a minor typo in existing documentation
pom.xml : to update testng version

There were three merge conflicts which were simple to resolve.

integration-tests/pom.xml
integration-tests/src/test/java/org/apache/druid/tests/coordinator/duty/ITAutoCompactionTest.java (this one was deleted from the PR since ITAutoCompactionTest is not in 0.18.1)
integration-tests/src/test/java/org/apache/druid/tests/TestNGGroup.java


Move Cache-Control header handling into frontend init

This doesn't work with the API.  The way it works is really designed for
frontend sites.

because the api doesn't set response cache headers?


remove duplicate import

pumpify was imported twice with two different names. removing the unused import

Thanks @anotherjesse


Incompatible with existing datasette-lite db?

If you have an existing datasette database that you want to serve via datasette-lite the saved queries are treated as just another table. My use-case is that having configured a datasette db that has queries I'd like others to see and use, I'd like to host that in the browser. Is there a way for datasette-lite to treat saved_queries as the special table it is in datasette?

NB. ?install=datasette-query-links is a decent workaround for anyone else looking at this.


Bump Dapr and CLI versions for integration tests

Description
Bump Dapr and CLI versions for integration tests
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: NA
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[x] Created/updated tests
[ ] Extended the documentation

No longer needed.


Implement StructureTemplate save and load

Implemented StructureTemplate.save() and StructureTemplate.load()
Description

Following how they are implemented for ItemStackBase, I added them for StructureTemplate. Makes it easier to modify structures
Motivation and Context

They are both methods I need to use and implementing them here is cleaner than doing it in my own code
Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the code style of this project.
[x] I have tested my changes and confirmed that they are working

Thank you for contributing.


Kafka source schema registry and sasl plaintext auth implementation

Description

Implemented the Confluent schema registry
Added SASL_PLAINTEXT auth configuration

Issues Resolved
Github issue https://github.com/opensearch-project/data-prepper/issues/254
Check List

[ ] Implementation of schema registry and sasl plaintext auth include testing.
[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed with a real name per the DCO

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

We are closing this PR#2706 because we brought all the functionality of this PR in another PR#2708.


Consecutive request with curl fails

Hi,
I realised, that any consecutive request will fail, if curl is used. The reason I found was, that the headers returned by Apache are postfixed with "\r\n" therfore my suggestion is to trim the second part as well in parseCookies()
https://github.com/matasarei/JsonRPC/blob/master/src/JsonRPC/HttpClient.php#L374
What are your thoughts?

Hello @tigerduck42!
Yes, seems you right. Here you can see changes I've made:
https://github.com/matasarei/JsonRPC/pull/6/files
Please try this fix with your code:
composer require fguillot/json-rpc:dev-fix-consecutive-request-with-url-fails

Waiting for your feedback.

Hello Yevhen,
looks good, my unit tests are successful.
Ready to ship and pack ;-)


update default repo branch names and delete branch clutter

move from master to main which I believe will be the github default soon

Also remove merged/no longer valid branches


Request for running PyMFEM build test in CI

Hi Team,
I propose to run a build-test of Python wrapper when something is pushed to master.
If you have MPI and Python in a test environment, this could be done by something like.
pip install numpy mpi4py six swig
git clone git@github.com:mfem/PyMFEM.git
cd PyMFEM
python setup.py install --with-parallel --mfem-branch=master
cd test
python run_examples.py -serial -verbose  -ex ex1
python run_examples.py -parallel -verbose -ex ex1p

There is also this
https://github.com/mfem/PyMFEM/blob/master/.github/workflows/test_with_MFEM_master.yml,
which build PyMFEM build in a various setting. We could modify it to call this when a change is pushed to master.

I think that's a good idea @sshiraiwa, do you want to propose a PR?

It's probably best to run this not just on master but all PRs -- that way we can catch issues before they are merged in master. To do that, we'll need to replace --mfem-branch=master with a local MFEM directory. Is that supported in the PyMFEM installer?

@v-dobrev , @tzanio let me look at this this weekend. Meantime, please point me a CI script which you think is good to add a call to test the wrapper.

Hi @sshiraiwa,
It's probably easiest to use a file like https://github.com/mfem/mfem/blob/master/.github/workflows/builds-and-tests.yml as a starting point -- copy it with a different name and edit that. A lot of the things in that file can probably be removed.

@sshiraiwa, @justinlaughlin, this was already addressed in #4283, right?
If not -- feel free to reopen.


Add Support for EVTX Files

Add Support for EVTX Files
Feature Request: Implement EVTX file support in toolong for importing, and reading  Windows Event Viewer logs.
Why This Matters:

Relevance: EVTX is a crucial format for Windows event logs, widely used in IT security and troubleshooting.
Benefit: Facilitates direct log analysis within toolong, enhancing its utility for Windows system users.

Thank you for considering this enhancement.

If you could link me to something that describes the format, or an example file, that would be very helpful.

Sure, these 2 links might be helpful.
Info on the file format
A cross-platform parser for the Windows XML EventLog format


fix: ensure fee payer (if custom) is refunded when accounts are closed

Ensure that payer where relevant is where rent exemption fees are refunded to, rather than purchaser when data accounts are closed.


Too many tries in fill_missing_atoms error

Hi,
I get the following error while executing it on the Google Collab notebook, while executing the Run BindCraft cell.
File: /home/benchmark/rosetta/source/src/core/conformation/Conformation.cc:1933
[ ERROR ] UtilityExitException
ERROR: too many tries in fill_missing_atoms!

Please make sure you trimmed your input pdb correctly and there are no partial residues

This is generally because the structure of the input rosetta does not meet its standards. I recommend adding a repack processing step in the rosetta filter section to ensure that the structure meets the specifications.


IDE no longer lets you generate method

Alt+Enter no longer shows an option to add a method on target class.
When writing
myClass.myNewMethod();

myNewMethod is highlighted in red, but when pressing Alt+Enter on it IDE usually allows to add a method with this name and signature to myClass implementation, supporting both classes and interfaces.
In latest update option to generate method no longer appears.

C# ?

Yes, C#

Latest Update didn't resolve the issue - generating methods on other classes does not work

Please provide simple example

public class MyClass1 {

    }

    public class MyClass2 {
        private MyClass1 myClass1;

        public MyClass2() {
            myClass1.init();
        }
    }

MyClass1 does not have method init(), so init is highlighted in red color. Usually in Intellij products pressing Ctrl+Enter shows a menu, that lets you add missing method on the target class automatically, generating it. This was working in Consulo before, but no longer works.

All works. What your C# plugin version? Current is 2269 (updating only Consulo is not enough, due it dont contains plugins)

I don't see update to 2269. I have 2257 now

i think u need update Consulo

Consulo is 1201 - latest I got on Download link

Try now - webserver have lags (open plugin list, then right click on C# - reload plugin list)

Generation is fixed, thanks.
A related issue, that was present before.
In the following example generated result should be void init();, not public void init() {}

public interface MyClass1 {

    }

    public class MyClass2 : MyClass1 {
        private MyClass1 myClass1;

        public MyClass2() {
            myClass1.init();
        }
    }

I created issue
https://github.com/consulo/consulo-csharp/issues/333

fixed


Feature/stan model interface

An interface to allow easy generation of media model stan code from a python interface.
Huge advantage of the Bayesian approach with smaller datasets like these is incorporation of domain expert knowledge through priors
Motivations

Experiementing with the right model requires hardcoded stan model changes, can be error prone.
Making this pipeline flexible to different models.

Changes
To make this pipeline quick to experiment, more changes will be needed in the helper functions (mmm_decompose() etc. etc.) A logical place to create a foundation for those changes is in the model configuration interface.

Validate a media model config though command line
Run pipeline with priors specific to each media variable, through command line
(see file modeling workflow in examples folder)

WIP / TODO

[ ] pipeline fails saving mmm (model dictionary variable ) to file if a non default config used. So currently can only help to explore effect of different priors, not proceeed to diminishing returns model. (BIG todo)
[ ] config editing from command line

To compare 2 media models

First, save current media model to examples folder;
cp mmm1.json examples/mmm_A.json

Specify a config file:
python -c "import json, model_configs; print(json.dumps( model_configs.default_config() ))" > "config.json"

Edit the config file manually, to assign new priors to a group of media variables.

Validate the config:
path/to/python mmm_stan.py --validate-config config.json

Run the model pipeline with config:
path/to/python mmm_stan.py --config config.json

(WIP: currently it will fail after saving the media model output.)

Save the new model
cp mmm1.json examples/mmm_B.json

Compare two models in notebook: examples/media.ipynb

An interface to allow easy generation of media model stan code from a python interface.
Huge advantage of the Bayesian approach with smaller datasets like these is incorporation of domain expert knowledge through priors
Motivations

Experiementing with the right model requires hardcoded stan model changes, can be error prone.

Making this pipeline flexible to different models.

Changes
To make this pipeline quick to experiment, more changes will be needed in the helper functions (mmm_decompose() etc. etc.) A logical place to create a foundation for those changes is in the model configuration interface.

Validate a media model config though command line

Run pipeline with priors specific to each media variable, through command line

(see file modeling workflow in examples folder)
WIP / TODO

[ ] pipeline fails saving mmm (model dictionary variable ) to file if a non default config used. So currently can only help to explore effect of different priors, not proceeed to diminishing returns model. (BIG todo)

[ ] config editing from command line

To compare 2 media models

First, save current media model to examples folder;

cp mmm1.json examples/mmm_A.json

Specify a config file:

python -c "import json, model_configs; print(json.dumps( model_configs.default_config() ))" > "config.json"

Edit the config file manually, to assign new priors to a group of media variables.

Validate the config:

path/to/python mmm_stan.py --validate-config config.json

Run the model pipeline with config:

path/to/python mmm_stan.py --config config.json

(WIP: currently it will fail after saving the media model output.)

Save the new model

cp mmm1.json examples/mmm_B.json

Compare two models in notebook: examples/media.ipynb

@sibylhe I, and it looks like quite a few other people have really valued your implementation of this algorithm. So I wanted to pay it back with this PR 🛠☺


Cannot enable "Mode ZHA or zigbee2mqtt"

Hello,
I have installed the recommended custom firmware for the Xiaomi Gateway 3 (v1.5.0_0102), but when I want to enable ZHA mode, I receive the following exception in the logs:
2022-11-15 10:40:28.285 ERROR (MainThread) [aiohttp.server] Error handling request
Traceback (most recent call last):
  File "/usr/lib/python3.10/site-packages/aiohttp/web_protocol.py", line 435, in _handle_request
    resp = await request_handler(request)
  File "/usr/lib/python3.10/site-packages/aiohttp/web_app.py", line 504, in _handle
    resp = await handler(request)
  File "/usr/lib/python3.10/site-packages/aiohttp/web_middlewares.py", line 117, in impl
    return await handler(request)
  File "/usr/lib/python3.10/site-packages/homeassistant/components/http/security_filter.py", line 60, in security_filter_middleware
    return await handler(request)
  File "/usr/lib/python3.10/site-packages/homeassistant/components/http/forwarded.py", line 100, in forwarded_middleware
    return await handler(request)
  File "/usr/lib/python3.10/site-packages/homeassistant/components/http/request_context.py", line 28, in request_context_middleware
    return await handler(request)
  File "/usr/lib/python3.10/site-packages/homeassistant/components/http/ban.py", line 82, in ban_middleware
    return await handler(request)
  File "/usr/lib/python3.10/site-packages/homeassistant/components/http/auth.py", line 236, in auth_middleware
    return await handler(request)
  File "/usr/lib/python3.10/site-packages/homeassistant/components/http/view.py", line 136, in handle
    result = await result
  File "/usr/lib/python3.10/site-packages/homeassistant/components/config/config_entries.py", line 241, in post
    return await super().post(request, flow_id)
  File "/usr/lib/python3.10/site-packages/homeassistant/components/http/data_validator.py", line 73, in wrapper
    result = await method(view, request, data, *args, **kwargs)
  File "/usr/lib/python3.10/site-packages/homeassistant/helpers/data_entry_flow.py", line 110, in post
    result = await self._flow_mgr.async_configure(flow_id, data)
  File "/usr/lib/python3.10/site-packages/homeassistant/data_entry_flow.py", line 280, in async_configure
    result = await self._async_handle_step(
  File "/usr/lib/python3.10/site-packages/homeassistant/data_entry_flow.py", line 367, in _async_handle_step
    result: FlowResult = await getattr(flow, method)(user_input)
  File "/config/custom_components/xiaomi_gateway3/config_flow.py", line 236, in async_step_user
    raise Exception("Can't update zigbee firmware")
Exception: Can't update zigbee firmware

What could be the problem?
Thanks very much for your support!

Try to increase log level


saveImage Future does not resolve on Android when first requesting permission

In GallerySaver.kt there’s a mismatch between saveMediaFile and onRequestPermissionsResult.
In the latter case, success.await() and  finishWithSuccess() don’t get called, so the Future that was returned from saveImage never resolves.

There may be other things around onRequestPermissionsResult actually… it seems that the photo is not is not being saved even upon allowing the permission. Maybe the permissionGranted check is inconsistent with isWritePermissionGranted?
This is on Android 10, with <uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE"/> and android:requestLegacyExternalStorage="true" in my AndroidManifest.xml.

I haven't looked at the iOS code but from looking through the Android side, I feel like a lot of the platform boilerplate surrounding permissions could be replaced with simple calls to permission_handler at the Flutter level. It has an MIT license and is well maintained so I'd really recommend the maintainers consider using it to reduce the scope of code that they're maintaining.


Reading input byte by byte causes lines with even numbers of characters

Windows Terminal version
1.18.3181.0
Windows build number
10.0.22621.0
Other Software
This bug report is a minimized reproduction of https://github.com/oven-sh/bun/issues/8504.
Steps to reproduce
Demo application "input.c"
#include <stdio.h>
#include <windows.h>

int main()
{
    HANDLE hStdin;
    DWORD len;
    char buf[1];

    hStdin = GetStdHandle(STD_INPUT_HANDLE);
    if (hStdin == INVALID_HANDLE_VALUE)
        printf("GetStdHandle failed with %lu\n", GetLastError());

    while (1)
    {
        if (!ReadFile(hStdin, buf, 1, &len, NULL)) // number of records read
            printf("ReadConsoleInput failed with %lu\n", GetLastError());

        printf("byte: %d\n", buf[0]);
    }
    return 0;
}

(This program is mimicking the behavior of the Zig standard library function std.io.Reader().readUntilDelimiterAlloc, which does a byte by byte read until it hits a specified character)
clang input.c (I used llvm 16.0.4, but I do not think this matters)
Run in Windows terminal. .\a.exe
If you type 0 or an even number of characters, the \n byte 10 is not read. only the \r byte is.
If you type an odd number of characters, everything is as normal.
Expected Behavior
I expect all lines to end with \r\n, to match the behavior of conhost.exe, ConEmu, Visual Studio Code's Integrated terminal, and also various terminals on Linux/MacOS over an ssh connection.
Actual Behavior
my inputs are: 1<return>12<return>123<return><return>, and then an S pressed by mistake when trying to take a screenshot.

I've found that this PR fixed the issue: #16313
The issue was caused by this PR: #14745
It's planned to be shipped in the next service release for both 1.18 (Stable) and 1.19 (Preview). I'm sorry that it's been taking some time to ship this! But since the bug only started appearing in 1.18, I personally believe it's fine to not hotfix Zig just yet. (Although I do sort of think that using a buffered reader is a great idea, if it's an option. The ReadFile syscalls are very expensive.)
If you want to test the fix in the meantime, you can find the nightly Canary build here: https://aka.ms/terminal-canary-installer

Thank you.
In Bun i have changed our code to use a buffered reader for all platforms. This makes the readByte call less expensive since it will buffer an amount already.
It probably makes sense for us to leave that commit in regardless because it improves performance slightly. These are only used for terminal input, where the ReadFile is going to always be blocked on user input (bun init and the prompt() JS api).
Maybe I'll revert my change if there are issues consuming stdin from a user program while using the prompt api, but I suspect this case is very unlikely, and by the time this becomes an issue for anyone, Windows Terminal should be propagated to everyone.


NBDev[1.1.3] nbdev_build_docs not working with latest fastcore[1.3.1] 'list' object has no attribute 'map_filter'

Latest/current/1.1.3 version of NBDev does not work with latest/current/1.3.1 fastcore while running nbdev_build_docs command.
Error with message 'list' object has no attribute 'map_filter' is shown and no html file is created.
It seems that fastcore is not compatible with NBDev after this commit  https://github.com/fastai/fastcore/commit/e72fe807d0822206e947be1fb015e7691e2a63c3 where function L.map_filter was removed.
In NBDev the function L.map_filter is used in function add_show_docs https://github.com/fastai/nbdev/blob/master/nbs/03_export2html.ipynb (https://github.com/fastai/nbdev/blob/master/nbdev/export2html.py#L276).
Temporary solution is to install fastcore in previous version pip install fastcore==1.3.0.

This is also breaking in CI: https://github.com/fastai/nbdev/pull/300/checks?check_run_id=1339437962#step:4:70
cc: @jph00
I'm investigating now.

related issue https://github.com/fastai/fastcore/issues/164
Found the answer in the release notes just need to change this to

Redundent L.map_filter removed (since L.map.filter does the same thing)

Working on it now

Looks like @jph00 beat me to it this is fixed by https://github.com/fastai/nbdev/commit/fd4cfac2a85875e9b6f9dad3012b34fc9e8a6a67


Add exception handling to LEAVE_SCRIPT_IF

External calls create exception boundaries which we handle in LEAVE_SCRIPT_START_EX but don't handle in LEAVE_SCRIPT_IF if not currently in script.
Unified both macros to use BEGIN_NO_EXCEPTION/END_NO_EXCEPTION.

@dotnet-bot test OSX static_osx_osx_debug please


Support /reports API

There is new beta API to generate repoort. Currently, this API is still not available on this SDK.
https://developers.xendit.co/api-reference/#reports

#144

Closing this as resolved, thanks @David-Kurniawan


Can't read from server. It may not have the appropriate access-control-origin settings.

I just triggered a weird bug.
I have an error handling middleware which does this when an exception is thrown.
private Task HandleExceptionAsync(HttpContext context, Exception exception)
        {
            var code = HttpStatusCode.InternalServerError;

            var result = JsonConvert.SerializeObject(new { Code = code, Error = exception.Message,
StackTrace = exception.StackTrace }, Formatting.Indented)
.Replace("\\r\\n", "\r\n");

            return context.Response.WriteAsync(result);
        }

This causes the UI to throw the error "Can't read from server. It may not have the appropriate access-control-origin settings.".
Usually, it prints the json when attempting to load the definitions. The offending code is .Replace("\\r\\n", "\r\n"); which I am attempting to use to format the error.

Can you navigate directly to the Swagger JSON endpoint, e.g. /swagger/v1/swagger.json? This may give a little more insight into what's causing the server-side error.

Closing due to inactivity

Am also getting the same error
Can't read from server. It may not have the appropriate access-control-origin settings.
Also i am not able to navigate to json endpoint but can access the  http://localhost:31002/api/clm/v1/swagger/docs


tools: Add AsyncChecker

Signed-off-by: Ryan Northey<EMAIL_ADDRESS>Commit Message: tools: Add AsyncChecker
Additional Description:
Allows use of asyncio for checkers
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]
[Optional API Considerations:]

in the immediate case the network connectivity is to the docker.sock - which is currently setup if you run with ./ci/run_envoy_docker.sh ... which will mount the docker.sock into the container - so its an assumption baked into the current local test env
as a general thing - for testing envoy itself its inevitable that we have to use tools on top to communicate with it - i have tried to keep these to fairly standard ~unix utils, and curl, jq etc
in terms of python libs i would generally prefer using asyncio libs over the sync ones - but my initial use for this is more because i was struggling to get docker-compose to build/run on arm64 - so it was easier to do it in python

Yeah, HTTP is fine for data plane tests on localhosts (assuming we don't assume anything about specific ports etc.). I was more thinking that the checkers we have here are developer tools for validating things like code format. Is there a plan to evolve this into a more general testing framework?

Is there a plan to evolve this into a more general testing framework?

not really - im trying to keep it reasonably simple - just the runner and checker classes
theres probably some stuff we could re/move to bazel - i did it in python i think because that was how i knew to do it - but mostly what is there is working quite well
the code that we register with envoy_py_* gets tested and the coverage is still at 100%

holding off on this while we figure out a way forward for #16899

on the basis that this is generally useful and will i think be used in #16899 im gonna land it


Update index,html

I've added some essential meta tags to this index page

Good deal, thank you for your contribution!


Password is sent by email after registering with ospc.org

After registering with ospc.org, I got a confirmation email including my username and password. Sending passwords by email is a security risk; see http://plaintextoffenders.com/faq/devs

@PeterDSteinberg and @brittainhard,
What's your view on this comment (#571) from a new TaxBrain user?
@MaxGhenis said:

After registering with ospc.org, I got a confirmation email including my username and password. Sending passwords by email is a security risk; see http://plaintextoffenders.com/faq/devs

It seems like the way things work now is not very safe.
Shouldn't this be changed?
@MattHJensen @GoFroggyRun

We do not currently provide any additional functionality for logged in users. I suggest resolving this issue by hiding/removing the signup/login capability.

The docs page (https://www.ospc.org/docs/) suggests registering for the mailing list (https://list.ospc.org/mailman/listinfo/users_list.ospc.org). Should that be removed?

The docs page (https://www.ospc.org/docs/) suggests registering for the mailing list (https://list.ospc.org/mailman/listinfo/users_list.ospc.org). Should that be removed?

Yes. That never took off.
so this issue requires:

[ ] remove suggestion to register for mailing list
[ ] hide signup/login capability until security risk is resolved and we actually use signup for something.


[gui error report] Error: Initializing daemon...

👉 Please describe what you were doing when this error happened.
Specifications

OS: darwin 22.6.0
IPFS Desktop Version: 0.28.0
Electron Version: 19.1.9
Chrome Version: 102.0.5005.167

Error
Error: Initializing daemon...
Kubo version: 0.20.0
Repo version: 13
System version: amd64/darwin
Golang version: go1.19.8

Error: Your programs version (13) is lower than your repos (14).
Please update ipfs to a version that supports the existing repo, or run
a migration in reverse.

See https://github.com/ipfs/fs-repo-migrations/blob/master/run.md for details.

    at errorTemplate (/Users/user/Desktop/IPFS Desktop.app/Contents/Resources/app.asar/src/daemon/migration-prompt.js:78:49)
    at Object.loadWindow (/Users/user/Desktop/IPFS Desktop.app/Contents/Resources/app.asar/src/daemon/migration-prompt.js:129:26)
    at startIpfsWithLogs (/Users/user/Desktop/IPFS Desktop.app/Contents/Resources/app.asar/src/daemon/daemon.js:191:25)
    at process.processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async startDaemon (/Users/user/Desktop/IPFS Desktop.app/Contents/Resources/app.asar/src/daemon/daemon.js:216:27)
    at async startIpfs (/Users/user/Desktop/IPFS Desktop.app/Contents/Resources/app.asar/src/daemon/index.js:48:17)
    at async setupDaemon (/Users/user/Desktop/IPFS Desktop.app/Contents/Resources/app.asar/src/daemon/index.js:116:3)
    at async run (/Users/user/Desktop/IPFS Desktop.app/Contents/Resources/app.

You need to use the latest IPFS desktop which uses kubo 0.21.0 and repo version 14. https://github.com/ipfs/ipfs-desktop/releases/tag/v0.29.0


[HPRO-582] Update Docker configuration to use GAE image

Note: @jt2k points out that this will put on the Google App Engine Flex environment image, rather than Standard. So the actual benefit here may end up somewhat moot. Still, we're moving closer to the versions used in one of the environments available rather than guessing.

This appears to run without issue. I left the install steps in place for the emulator, git, etc., but we could try removing those one-by-one during more comprehensive testing. Right now, they aren't hurting anything to be side loaded.
HPRO-582

Confirmed that the gcloud SDK is not installed by default on the image. As best I can tell, everything else we install/load is necessary.
$ docker run -it gcr.io/gcp-runtimes/php72:2020-02-19-11-00 bash
root@9875135223b9:/app# which gcloud
root@9875135223b9:/app#

We'll revisit this at some point in the future.


[bug]: Stacked CLI #47 appears to have broken line length formatting

Describe the bug
After updating to stacked_cli 1.13.1, line_length variable in stacked.json appears to be ignored. Files auto-generated by Stacked are being formatted with a default line length of 80.
It appears that the PR #46 and merge #47 in the stacked_cli project have undone this behaviour that was once working.
What operating system do you use?
macOS
Information about the installed tooling
[✓] Flutter (Channel stable, 3.24.0, on macOS 14.6.1 23G93 darwin-arm64, locale en-GB)
    • Flutter version 3.24.0 on channel stable at /Users/cord.schneider/Development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 80c2e84975 (3 weeks ago), 2024-07-30 23:06:49 +0700
    • Engine revision b8800d88be
    • Dart version 3.5.0
    • DevTools version 2.37.2

[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
    • Android SDK at /Users/cord.schneider/Library/Android/sdk
    • Platform android-35, build-tools 32.1.0-rc1
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.10+0-17.0.10b1087.21-11609105)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.4)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15F31d
    • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2024.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.10+0-17.0.10b1087.21-11609105)

[✓] VS Code (version 1.92.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.94.0

[✓] Connected device (4 available)
    • slothlet's iPhone (mobile)      • 00008130-000968160C38001C • ios            • iOS 17.6.1 21G93
    • macOS (desktop)                 • macos                     • darwin-arm64   • macOS 14.6.1 23G93 darwin-arm64
    • Mac Designed for iPad (desktop) • mac-designed-for-ipad     • darwin         • macOS 14.6.1 23G93 darwin-arm64
    • Chrome (web)                    • chrome                    • web-javascript • Google Chrome 127.0.6533.120

[✓] Network resources
    • All expected network resources are available.

• No issues found!

Steps to reproduce the issue

Create a stacked project using pre-1.13.1 stacked_cli
Set line_length to 150 in stacked.json
Run stacked create view layoutOne --template=empty in an existing project
Review line length of generated files
Update to 1.13.1 stacked_cli
Run stacked create view LayoutTwo --template=empty in an existing project
Review line length of generated files

Expected behavior
Expected the line length defined in stacked.json to be honoured and all generated files created with stacked_cli 1.13.1 to be correct.
Screenshots
No response
Additional Context
No response

Note that the hotlinks to #46 and #47 are incorrect as Github has referenced theme for stacked rather than stacked_cli. Unfortunately it is not possible to create an issue in the stacked_cli project.

Also note, this appears to be happening for stacked create view commands. I've run stacked generate and it behaves as expected.

Thanks Cord. Will add to the list of fixes

I think we can pass in the line length or set it in the config, we probably didn't update the config.


Dardar

Module to read and resample dardar data with the same methods as for the Cloudsat DPC data
Code to collocate with training and test data

There is now a class to read and resample DARDAR data following the same methods as used for the data from Cloudsat DPC data. Even the same functions are imported and used, except for subsample_iwc_and_height, as the weights should be computed differently.

The code to collocate with training and test data is a script in scripts/collocate_dardar.py.
Notes:

The DARDAR data needs to be downloaded previously to the collocation (it's not done on-the-fly).

Not all CloudSat DPC granules are foun in DARDAR data, implying that there will be less DARDAR collocations

Few DARDAR pixels are missed in the collocation (see figure below). This can be fixed by extending the time bounds, e.g., with one second, but if we do that then it can mean that we actually get more pixels. Either way, it's only a few pixels and the geostationary data temporal resolution is much larger than the DARDAR temporal resolution, and I think it can be considered insignificant differences of 1 second.

Collocation example, black: 2C-ICE, blue: DARDAR. The differences are because of the time boundaries (playing with the time boundaries can make them match).

The code is not intended to support the coarser CPCIR data prepared ("cpcir2").

Some DARDAR profiles can have or consist only of NaN IWC values. An unfortunate resampling can happen if only one, full-NaN IWC profile falls in a bin, resulting in a "valid nan IWP". I adapted ccic.data.write_scenes to discard these profiles when writing the sparse file.

The code can issue the following RunTimeWarning:
Traceback (most recent call last):
  File "/home/amell/ccic/scripts/collocate_dardar.py", line 191, in <module>
    resample_to_scene(f, GRIDSAT_GRID, dardar_files_dict, dst_gridsat)
  File "/home/amell/ccic/scripts/collocate_dardar.py", line 103, in resample_to_scene
    ds_resampled = resample_data(
  File "/home/amell/ccic/ccic/data/cloudsat.py", line 512, in resample_data
    cloudsat_file.add_retrieval_targets(
  File "/home/amell/ccic/ccic/data/dardar.py", line 147, in add_retrieval_targets
    iwc, height = subsample_iwc_and_height(iwc, source_height)
  File "/home/amell/ccic/ccic/data/dardar.py", line 41, in subsample_iwc_and_height
    iwc = convolve(iwc, k, mode="valid", method="direct")
  File "/home/amell/miniconda3/envs/ccic/lib/python3.10/site-packages/scipy/signal/_signaltools.py", line 1430, in convolve
    return correlate(volume, _reverse_and_conj(kernel), mode, 'direct')
  File "/home/amell/miniconda3/envs/ccic/lib/python3.10/site-packages/scipy/signal/_signaltools.py", line 262, in correlate
    z = _sigtools._correlateND(in1, in2, out, val)
RuntimeWarning: invalid value encountered in cast

but neither can I identify any consequence of this nor can I reproduce it. It seems to happen arbitrarily, and this is used before any random sampling is done.

Thanks for the comments. Updated with your comments and with black formatting (except for one line, which black just makes it uggly; I kept that line identical to that in ccic.data.cloudsat). I also took the chance to handle an edge case which I came across after running the script for several hours.


using Multiple GPU for LLM model

Problem
I wanted to run my hugging face model over multiple gpu while deploying the model over runpod.
Code used
I used this code from documentation.
gpu_count = 2

pod = runpod.create_pod(
    name="Llama-7b-chat-brand_mapping",
    image_name="ghcr.io/huggingface/text-generation-inference:0.9.4",
    gpu_type_id="NVIDIA RTX A5000",
    data_center_id="EU-CZ-1",
    cloud_type="SECURE",
    docker_args="--model-id Vishal24/brand_mapping"
    gpu_count=gpu_count,
    volume_in_gb=50,
    container_disk_in_gb=30,
    ports="80/http,29500/http",
    volume_mount_path="/data",
)
pod

I got to understand that the docker image gets created using text-generation-interface.
But according to text-generation-interface to use multi gpu have to pass argument like
docker run --gpus all --shm-size 1g -p 8080:80 -v $volume:/data ghcr.io/huggingface/text-generation-inference:1.1.0 --model-id $model
Question
Now how do i pass "--gpus all" and "--shm-size 1g" into the docker_args parameter to run on multiple GPU.

With runpod you do not need to pass --gpus all or --shm-size 1g we will take care of that for you.


Add methods to get and set shader attribute locations

Implements #987 (forum post).
Fixed attribute locations may be assigned before loading the shader source, and/or the locations assigned by the shader compiler may be retrieved afterward.
Usage:
    // Vertex shader
    #version 330
    uniform mat4 projection;
    uniform mat4 modelview;
    in vec2 position;
    in vec2 texCoord;
    out vec2 v_position;
    out vec2 v_texCoord;
    void main() {
        gl_Position = projection * modelview * vec4(position, 0, 1);
        v_position = position;
        v_texCoord = texCoord;
    }

    // Assign locations before loading
    sf::Shader myShader;
    myShader.setAttribLocation("position", 0);
    myShader.setAttribLocation("texCoord", 1);
    myShader.loadFromFile("shaders/tilemap.vert", "shaders/tilemap.frag");
    glEnableVertexAttribArray(0);
    glEnableVertexAttribArray(1);
    glVertexAttribPointer(0, 2, GL_FLOAT, GL_FALSE, 16, 0);
    glVertexAttribPointer(1, 2, GL_FLOAT, GL_FALSE, 16, 8);

    // Retrieve locations after loading
    sf::Shader myShader;
    myShader.loadFromFile("shaders/tilemap.vert", "shaders/tilemap.frag");
    int loc_position = myShader.getAttribLocation("position");
    int loc_texCoord = myShader.getAttribLocation("texCoord");
    glEnableVertexAttribArray(loc_position);
    glEnableVertexAttribArray(loc_texCoord);
    glVertexAttribPointer(loc_position, 2, GL_FLOAT, GL_FALSE, 16, 0);
    glVertexAttribPointer(loc_texCoord, 2, GL_FLOAT, GL_FALSE, 16, 8);

This is a start, but I was already considering an API that allowed newer GLSL to be used with sfml-graphics rendering as well. Your code only benefits users who use sf::Shader as a lightweight GL object wrapper. It would be nicer if this feature could benefit sfml-graphics users (i.e. those who use sf::RenderTargets for drawing) allowing them to e.g. make use of newer GLSL versions in combination with their usual rendering.

This is a start, but I was already considering an API that allowed newer GLSL to be used with sfml-graphics rendering as well.

Can you explain what you have in mind exactly? How could this new stuff be combined with sfml-graphics, and how would you modify the API to allow that?

Just like how you can bind a GLSL sampler to sf::Shader::CurrentTexture, you should be able to bind your position/texture/colour attributes to fixed attribute enums that a sf::RenderTarget can bind the vertex array to in order to render sf::Drawable objects. Something like sf::Shader::AttributePosition, sf::Shader::AttributeTexCoord and sf::Shader::AttributeColor. All that matters is that there is a common convention that is used for binding between the user's GLSL and the sf::RenderTarget. There would also be an overload for arbitrary binding as in @criptych's PR.

Actually, I'd thought about something like this too, but wasn't sure how to go about it.  The thing is, SFML currently uses glVertexPointer et al., and according to the OpenGL docs
"Applications are not allowed to bind any of the standard OpenGL vertex attributes using this command, as they are bound automatically when needed."  (Notwithstanding, apparently some cards do alias the standard attributes to known locations.) SFML itself would likely have to switch to using shaders internally in order to properly support this.
On a side note, I also found the following in the docs for 3.0+:  "the location of the attribute specified in the shader's source [with location] takes precedence".  My implementation as written doesn't account for this if an attribute location was set by the user (it assumes the set location is definitive), so I'll need to rethink that as well....

The thing is, SFML currently uses standard vertex attributes through glVertexPointer et al., and according to the OpenGL 2.1 docs, "Applications are not allowed to bind any of the standard OpenGL vertex attributes using [glBindAttribLocation], as they are bound automatically when needed." (Notwithstanding, apparently some cards do alias the standard attributes to known locations.) SFML itself would likely have to switch to using shaders internally in order to properly support this.

Not necessarily. There are 3 cases:

No shader bound as part of a RenderState -> Do things as usual.
Shader bound as part of a RenderState without explicit binding -> Do things as usual.
Shader bound as part of a RenderState with explicit binding -> Use glBindAttribLocation instead.

Updated PR; now allows for locations overridden in the shader.

Not necessarily. There are 3 cases:

No shader bound as part of a RenderState -> Do things as usual.
Shader bound as part of a RenderState without explicit binding -> Do things as usual.
Shader bound as part of a RenderState with explicit binding -> Use glBindAttribLocation instead.

Of course, sorry.  I was only thinking about case 3. :)
Unfortunately, though, I still don't really see a way to do it the way you're suggesting:

you should be able to bind your position/texture/colour attributes to fixed attribute enums that a sf::RenderTarget can bind the vertex array to in order to render sf::Drawable objects

The standard attributes gl_Vertex etc. can only be used one-to-one with glVertexPointer etc.  The attributes can't be bound (mentioned above) and the functions don't take a location parameter.  So unless I'm misunderstanding, either you have to use the standard names in your shader (case 2); or you have to use custom names and the generic glVertexAttribPointer with a location.
I'll keep looking at this, though, and I'm open to suggestions.

@binary1248 will this be obsolete with the legacy shader changes you have in the pipeline?

Possibly.

@binary1248 I assume the situation has not changed since 2018?

@Bromeon It hasn't.


[Bug] Package Xamarin.CommunityToolkit.Markup 1.2.0 is not compatible with xamarinios10, monoandroid10.0, netstandard2.0, netstandard2.1

Description
Developing my app on Visual Studio 2019 16.10.4 using Xamarin.Forms <IP_ADDRESS>2.
After updating to 2019 16.11, updating to Visual Studio 2022 Preview 3 and installing the latest MAUI workloads yesterday using maui-check (.NET 6 Preview 7) my app stopped building because of NuGet Package Xamarin.CommunityToolkit.Markup 1.2.0 that could not be resolved anymore. Reinstalling the Markup package produced following errors:
NU1202: Package Xamarin.CommunityToolkit.Markup 1.2.0 is not compatible with xamarinios10 (Xamarin.iOS,Version=v1.0). Package Xamarin.CommunityToolkit.Markup 1.2.0 does not support any target frameworks.
NU1202: Package Xamarin.CommunityToolkit.Markup 1.2.0 is not compatible with xamarinios10 (Xamarin.iOS,Version=v1.0) / win. Package Xamarin.CommunityToolkit.Markup 1.2.0 does not support any target frameworks.
NU1202: Package Xamarin.CommunityToolkit.Markup 1.2.0 is not compatible with xamarinios10 (Xamarin.iOS,Version=v1.0) / win-x64. Package Xamarin.CommunityToolkit.Markup 1.2.0 does not support any target frameworks.
NU1202: Package Xamarin.CommunityToolkit.Markup 1.2.0 is not compatible with xamarinios10 (Xamarin.iOS,Version=v1.0) / win-x86. Package Xamarin.CommunityToolkit.Markup 1.2.0 does not support any target frameworks.
NU1202: Package Xamarin.CommunityToolkit.Markup 1.2.0 is not compatible with monoandroid10.0 (MonoAndroid,Version=v10.0). Package Xamarin.CommunityToolkit.Markup 1.2.0 does not support any target frameworks.
NU1202: Package Xamarin.CommunityToolkit.Markup 1.2.0 is not compatible with netstandard2.1 (.NETStandard,Version=v2.1). Package Xamarin.CommunityToolkit.Markup 1.2.0 does not support any target frameworks.

Tried to downgrade to .NET Standard 2.0 but same error messages accordingly
...
NU1202: Package Xamarin.CommunityToolkit.Markup 1.2.0 is not compatible with netstandard2.0 (.NETStandard,Version=v2.0). Package Xamarin.CommunityToolkit.Markup 1.2.0 does not support any target frameworks.

Downgrading XCT.Markup to 1.1.0 works. App will also start debugging.
Steps to Reproduce

Create new Xamarin.Forms app (Android, iOS)
Install XCT 1.2.0 (works)
Install XCT.Markup 1.2.0 (fails)

Basic Information

Version with issue:

Visual Studio 2019 16.10.4
Visual Studio 2019 16.11
Visual Studio 2019 16.11 Preview 4
Visual Studio 2022 Preview 3

Last known good version:

2019 16.10.4 (before updating machine to VS 2022 Preview 3, .NET 6 Preview 7)
2019 16.11 Preview 3

Platform Target Frameworks:  Does not apply, installing NuGet package in shared project already fails.

Edit:  attached screenshot

"Package Xamarin.CommunityToolkit.Markup 1.2.0 is not compatible with netstandard2.1 (.NETStandard,Version=v2.1). Package Xamarin.CommunityToolkit.Markup 1.2.0 does not support any target frameworks."

@JunkyXL86 If I'm not wrong you should install the Markup package just in your shared project. There's no need to install them into the platform projects.

I didn't even install it in the Android proj. As I said, error occurs when I try to install the package in the shared project.
I tried it over and over again using VS 2019 16.11 and VS 2022 P3.
New clean fresh Xamarin.Forms Mobile App, simply try to install the XCT.Markup 1.2 package.

@JunkyXL86 could you attach a small repro? I couldn't reproduce the error

Meanwhile I got a new PC, Windows 11. On this machine using VS 2022 Preview 3.1 installing XCT.Markup 1.2 works as expected. Maybe something is broken on my old machine.
As the error is pretty certain, I'm still interested what the real cause is.
I've attached the Solution and a GIF showing the steps. Nothing special to it.
XCT.Markup.1.2.zip

@JunkyXL86 I tried to reproduce it and everything works as expected. This error is very odd, since our package has a dependency on .netstandard 2.0. So this should be something in your environment.


Feature request: Key stroke to delete files / Option for safe deletion

thank you for a very fast and useful extension.
Having generating a lot of craps and test generations during the process. I would love to request for DEL keystroke to delete file when in Preview mode. Or at least a Delete button in the Viewer.
With an option to "Do not ask again" in confirmation box. And check box option in settings for this "Confirmation box".
I don't know whether or not the extension can send deleted files to "Recycle Bin" or not. This could be helpful to restore during mass selection deletion mistake. Or the extension could implement what Android apps usually do: by adding ".trash.$original_name" to the deleted files so the extension can skip loading those. Then flush the trashes later at once.
Many thanks again. I hope these could easily implement into the extention.

Deleting images using keyboard shortcuts is now supported, as seen in #159.

Deleting images using keyboard shortcuts is now supported, as seen in #159.

thank you very much. It's extremely useful. But please also implement an option to toggle "Delete confirmation" box. save extra keystoke, save extra time :D

Deleting images using keyboard shortcuts is now supported, as seen in #159.

thank you very much. It's extremely useful. But please also implement an option to toggle "Delete confirmation" box. save extra keystoke, save extra time :D

The implementation of this feature is still in progress

The delete shortcut isn't working for me. "Due to technical limitations, the first image cannot be deleted when opening the fullscreen preview."

The delete shortcut isn't working for me. "Due to technical limitations, the first image cannot be deleted when opening the fullscreen preview."

This is a normal limitation of the system. If you want to delete an image, you can do so in the grid view preview.
If you want to delete an image in full-screen mode, you can only delete the second, third, fourth, etc. images after opening the full-screen preview, due to technical limitations

Deleting images using keyboard shortcuts is now supported, as seen in #159.

thank you very much. It's extremely useful. But please also implement an option to toggle "Delete confirmation" box. save extra keystoke, save extra time :D

I have added this feature that allows you to delete a single image without confirmation

Love this, is there any way to save the key selection for this, or give it a default? Or am I missing something by having to reset this option every time I restart the Webui (in Global Settings)?
Also, could we get a Send to Save Folder key as well? It seems to me that that's might be more appropriate than the Download one. Awesome plugin anyway...

Any chance of implementing Send2Trash as mentioned in the comments in #149?

Love this, is there any way to save the key selection for this, or give it a default? Or am I missing something by having to reset this option every time I restart the Webui (in Global Settings)? Also, could we get a Send to Save Folder key as well? It seems to me that that's might be more appropriate than the Download one. Awesome plugin anyway...

This saves now which is awesome, thanks!


Use normalizeEvent so that addon works if app uses jQuery

After #141, this addon no longer works for apps that use jQuery. Sorry to @ you @dgavey, but this is a show stopper 😫
Several event properties used by this addon are not available when the consuming app uses jQuery, so this PR uses normalizeEvent so that it works no matter what.
More info on events and supporting jQuery and non-Jquery apps here

Thanks for this.  I no longer use this addon in production, so I'm glad someone can alert me of these things.


Make CARDINALITY return INT in Trino

We convert the size function in Hive to cardinality in Trino (FKA PrestoSQL). However, size returns an int while cardinality returns bigint. This causes type validation issues in Trino when reading the view to fail with the error <view_name> is stale; it must be re-created.
When unparsing the cardinality function in Trino, we should add CAST to an integer to retain the semantics of Hive's size and Calcite's cardinality

Closing this PR given #185 has been merged.


Nuget packages update

Downgraded the Microsoft.CodeAnalysis.Common package to version 1.1.0
Installed the package Microsoft.CodeAnalysis version 1.1.0
The result of the analysis can be seen in the dexter-result.json. However, they still do not appear in the console for some reason.

There should be only 2 commits in this PR (6775bb2 and 921a2fc).
All other are redundant (they are already in repo).


Handle exception in update_parent in solr

Summary:
Related issue:
Suggested reviewer(s):
Reviewer checklist:

[ ] The headers of all files contain a reference to the repository license
[ ] 100% test coverage of new code - meaning:

[ ] The overall test coverage increased or remained the same as before
[ ] Every function is accompanied with a test suite
[ ] Tests are both positive (testing that the function work as intended with valid data) and negative (testing that the function behaves as expected with invalid data, e.g., that correct exceptions are thrown)
[ ] Functions with optional arguments have separate tests for all options

[ ] Examples are supported by doctests
[ ] All tests are passing
[ ] All names (e.g., files, classes, functions, variables) are explicit
[ ] Documentation (as docstrings) is complete and understandable

The checklist is based on the S-ENDA conventions and definition of done (see General Conventions). The above points are not necessarily relevant to all contributions. In that case, please add a short explanation to help the reviewer.

@johtoblan pytestcov 3.7 is stuck here. Can we remove that in worflow?

It is removed from the workflow but not from the repo requirements, I can fix it


fix: data model attribute updates

Data model event streaming code sends the object as callback argument, which was incorrectly interpreted as the value. The response is supposed to contain the value so a more complete solution would change that callback interface to pass the value. For now I am just erasing the cache item.
An additional - previously ignored - issue concerns the accumulation of callbacks for a single attribute, which are never cleared. The object returned from add_on_attribute_changed() can be used to unregister. An object should do so upon destruction.  I am assuming that that is necessary. In terms of accumulation of callbacks for a live objects, we could track what is already registered.

@seanpearsonuk Thanks for investgating the issue. I'm fixing this in #2673 by consuming the attribute value from the stream (we already have the complete server-side infrastructure).
I'll look into the lifetime issue of the subscription objects spearately.

@mkundu1 is doing the more architectural fix from the calling side - passing only the value and not the object


select and delete multiple components

Fixes
https://issues.redhat.com/browse/HAC-544
Description

Added provision to select multiple rows
the delete button is disabled if no rows are selected
when one or more components are selected, the delete button is enabled which on clicked will delete the component CRs

Type of change

[x] Feature
[ ] Bugfix
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

Screen shots / Gifs for design review

How to test or reproduce?
Goto: https://prod.foo.redhat.com:1337/beta/hac/app-studio/components?application=
Browser conformance:

[ ] Chrome
[ ] Firefox
[ ] Safari
[ ] Edge

/assign @rohitkrai03

/cc @rottencandy

When pressing delete, there is no indication that components are being deleted, maybe we can use isLoading for the button to show a spinner while the deletion happens. WDYT?
Also, not sure if it would be good should show a notification once deletion is completed.
0.mp4

^ @beaumorley @rohitkrai03

When pressing delete, there is no indication that components are being deleted, maybe we can use isLoading for the button to show a spinner while the deletion happens. WDYT?
Also, not sure if it would be good should show a notification once deletion is completed.
0.mp4

^ @beaumorley @rohitkrai03

@debsmita1 Can you start a conversation around this in slack to get faster turnaround?

@debsmita1 You might have to rework this a bit because designs have changes around this page.

@rohitkrai03  Updated. PTAL !


[FEATURE REQUEST] File watcher for the generator

When scripting and designing websites, as well as in react etc you can enable a watch feature so it re-compiles or re-generates files on change.
Could this be a desired feature for the generator? I've found myself missing this feature quite a lot lately when having to debug and develop templates.

Oh yes, definitely. I feel your pain, my friend. That would be cool.

Fixed by #160


Home Page clean up

[ ] separate home page content from layouts sections
[ ] hyphening seems too aggressive. Remove or reduce

@conorgil , can you show examples where you found the hyphening to be too aggressive?


pythonPackages.pivy: fix build

Motivation for this change
ZHF #68361
I have absolutely no idea what changed so that we now need to disable qmake and cmake explicitly.  Anyway, it builds now.
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @

Backported as 31c575190c0ebf91c6963f1be5af19f163e9d90a

It's because of the new setup hooks based buildPythonPackage. It no longer
sets configure, build and install phases.
On Tue, Sep 10, 2019, 18:41 Gabriel Ebner<EMAIL_ADDRESS>wrote:

Backported as 31c5751
https://github.com/NixOS/nixpkgs/commit/31c575190c0ebf91c6963f1be5af19f163e9d90a
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/68433?email_source=notifications&email_token=AAQHZ36R6S2LQNRFZ7MCQV3QI7EZ5A5CNFSM4IVKA4SKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6LXX4Y#issuecomment-530021363,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAQHZ3433XTGCYKJH67MQCDQI7EZ5ANCNFSM4IVKA4SA
.

Thanks for the explanation!


Using namespaced classes does not work

When using this gem with namespaced classes, this gem breaks.
Ex:
class ModuleA::ModuleB::ClassA < ApplicationRecord
  has_paper_trail

  include Monetizable

  # Relationships
  belongs_to :class_c, class_name: 'ModuleA::ClassC'
  belongs_to :class_d, -> { includes(:class_e) }, class_name: 'ModuleC::ClassD'

  counter_culture [:class_d, :class_e], column_name: "orders_count"

throws:
ActiveRecord::StatementInvalid: PG::UndefinedTable: ERROR:  invalid reference to FROM-clause entry for table...

HINT:  Perhaps you meant to reference the table alias "modulec_class_d".

Whenever I call ModuleA::ModuleB::ClassA .counter_culture_fix_counts

Just to add mode context, modulo A has this table_prefix:
module ModuleA

  def self.table_name_prefix
    'modulea.'
  end
end

Something like explained here:
https://icelandcheng.medium.com/ruby-on-rails-using-table-name-prefix-and-namespace-to-organize-models-a08199deaa69
Also, I have seems this issue in some other libs, like:
https://github.com/RolifyCommunity/rolify/issues/75
They seems to have fixed it somehow.

Can you open a PR with a failing test case for this? I believe you it's failing but it's hard to debug with just that error, without a stack trace etc. I'd love a PR with a fix, too, if you can identify that.

Alright, I will work on it!


binom_vanilla_omp is unstable

Investigating different (and pretty complex ideas) I reached to this simpler solution, that gives speedup and the answer is almost correct (best accuracy and speedup when threads = 2). Using different number of threads changes the answer fairly a lot.
Probable problem: since I'm doing calculations using one vector, then there is a possibility that some calculations are performed with wrong values.
Goal:  make the solution accurate and consistent without sacrificing speedup too much.

This issue is fixed.
Fixed problems were:

End index was +1 to big;
variables should have been defined in pragma omp parallel (except for results variable).

I looked the wrong column, error persists.

Not implementing binomial vanilla method.


refactored memory

Created langchain_community.memory. Moved there two classes from langchain.memory
This movement triggered another changes:

There is a lint rule that langchain_community code should not depend on langchain. Below, I show these dependencies:
`ZepMemory`
  --> langchain.memory.buffer.`ConversationBufferMemory`
       --> langchain.memory.chat_memory.`BaseChatMemory`
            --> langchain_community.chat_message_histories.in_memory.`ChatMessageHistory`
       --> langchain.memory.utils.`get_prompt_input_key`  # the same as in MotorheadMemory

`MotorheadMemory`
   --> langchain.memory.chat_memory.BaseChatMemory   # then the same dependency as in ZepMemory
        --> langchain_community.chat_message_histories.in_memory.ChatMessageHistory
   --> langchain.memory.utils.get_prompt_input_key

So, these classes and a function were moved into `core` because of the above dependencies:
- `ConversationBufferMemory`
- `BaseChatMemory`
- `ChatMessageHistory`
- `get_prompt_input_key`

Then the next lint error forced a change in experimental:
mkdir -p .mypy_cache; poetry run mypy langchain_experimental --cache-dir .mypy_cache
langchain_experimental/autonomous_agents/autogpt/memory.py:3: error: Module "langchain.memory.chat_memory" has no attribute "get_prompt_input_key"  [attr-defined]
-- this was a bug in langchain_experimental/autonomous_agents/autogpt/memory.py:3 (wrong import)

FYI @baskaryan

@efriis Fixed the import bug in the separate #17078. It is not mixed with this PR anymore.


Psalm error on use RouteCollectorInterface::addGroup()

RouteCollectorInterface::addGroup() accepts parameter with type Group, but Group::create() returns GroupInterface:

Should use interface if possible.

No, we don't can use interface in addGroup() because need object with getters, but interface contain only setters.


Pip install flask-mysqldb results in error

Having trouble using the library because I can only install the .whl, and for some reason it's not "registered" so my app picks it up.
My error is here:

Command "c:\users\there\appdata\local\programs\python\python37\python.exe -u -c "import setuptools, tokenize;file='C:\Users\there\AppData\Local\Temp\pip-install-7gfeybon\mysqlclient\setup.py';f=getattr(tokenize, 'open', open)(file);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, file, 'exec'))" install --record C:\Users\there\AppData\Local\Temp\pip-record-2i1evwnn\install-record.txt --single-version-externally-managed --compile" failed with error code 1 in C:\Users\there\AppData\Local\Temp\pip-install-7gfeybon\mysqlclient\

I guess this is why a virtual environment is so beneficial

issue is with mysql client.
Wheels are available here
https://www.lfd.uci.edu/~gohlke/pythonlibs/