net/http: If the response is returned directly and the request body is not consumed, the browser throws an error, but nodejs does not.

What version of Go are you using (go version)?

$ go version
go version go1.20.2 windows/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
set GO111MODULE=
set GOARCH=amd64
set GOBIN=
set GOCACHE=C:\Users\Administrator\AppData\Local\go-build
set GOENV=C:\Users\Administrator\AppData\Roaming\go\env
set GOEXE=.exe
set GOEXPERIMENT=
set GOFLAGS=
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOINSECURE=
set GOMODCACHE=C:\Users\Administrator\go\pkg\mod
set GONOPROXY=
set GONOSUMDB=
set GOOS=windows
set GOPATH=C:\Users\Administrator\go
set GOPRIVATE=
set GOPROXY=https://goproxy.cn
set GOROOT=C:\Program Files\Go
set GOSUMDB=sum.golang.org
set GOTMPDIR=
set GOTOOLDIR=C:\Program Files\Go\pkg\tool\windows_amd64
set GOVCS=
set GOVERSION=go1.20.2
set GCCGO=gccgo
set GOAMD64=v1
set AR=ar
set CC=gcc
set CXX=g++
set CGO_ENABLED=1
set GOMOD=NUL
set GOWORK=
set CGO_CFLAGS=-O2 -g
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-O2 -g
set CGO_FFLAGS=-O2 -g
set CGO_LDFLAGS=-O2 -g
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=-m64 -mthreads -Wl,--no-gc-sections -fmessage-length=0 -fdebug-prefix-map=C:\Users\ADMINI~1\AppData\Local\Temp\go-build2456021619=/tmp/go-build -gno-record-gcc-switches

What did you do?
When sending data to an HTTP server built in Go, the browser throw a network error if the server returns a response without actively consuming the request body (only if the request body is large).  However, this error does not occur when using an HTTP server built with Node.js.
example
https://github.com/tianluanchen/go-issue-62694
What did you expect to see?
No error occurs in the browser
What did you see instead?
The browser throws a network  error

This seems related to #3595.
If the server doesn't read the complete request, there is a race inherent in the HTTP protocol: see RFC 7230 §6.6, starting with the paragraph that begins “If a server performs an immediate close of a TCP connection”.
As far as I can tell, there is no way to avoid that race without also requiring the server to read the entire request, which would allow a misbehaving client to consume a proportionally very large fraction of the server's incoming bandwidth. Instead, the Go HTTP server implementation waits for an arbitrary delay that is usually long enough. Perhaps in your case that arbitrary delay is too short.
If you are not worried about ill-behaved clients, you could have your server Flush its response to the client (to minimize latency) and then io.Copy(io.Discard, req.Body) before returning.
(CC @neild)

This seems related to #3595.
If the server doesn't read the complete request, there is a race inherent in the HTTP protocol: see RFC 7230 §6.6, starting with the paragraph that begins “If a server performs an immediate close of a TCP connection”.
As far as I can tell, there is no way to avoid that race without also requiring the server to read the entire request, which would allow a misbehaving client to consume a proportionally very large fraction of the server's incoming bandwidth. Instead, the Go HTTP server implementation waits for an arbitrary delay that is usually long enough. Perhaps in your case that arbitrary delay is too short.
If you are not worried about ill-behaved clients, you could have your server Flush its response to the client (to minimize latency) and then io.Copy(io.Discard, req.Body) before returning.
(CC @neild)

Thanks for your help.


Feature/db2 implementation

hey @Thanh-NgHoL - introducing db2 for the persistence layer is rad 🤘
Without having looked at the PR itself, I sure hope that you closed it only b/c of git/file/tech thingies...


Remove propertyWithDefaultValue method from hierarchy of all property based tests

This is an internal change to clean up a test hierarchy discovered during work on #22595.
Default property values only make sense for collection properties. This PR
removes extraneous factory methods related to this for non-collection property tests, cleaning
the spec hierarchy a little.

@bot-gradle test this

OK, I've already triggered the following builds for you:

PullRequestFeedback build

@big-guy - This is the refactor I was talking about

@bot-gradle test and merge

OK, I've already triggered a build for you.

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.

Pre-tested commit build failed.

@bot-gradle test and merge

OK, I've already triggered a build for you.


Essential - Team Level Badges

It is important to have Team Level Badges so as to award certain points to Teams on winning team competitions.  Team Level Badges can display below the Individual Team Leaderboard.

Closing this issue.


How To: update accordion title

@apercas we also need to make sure these accordion titles don't mess up the formatting because they are too long. Can you check this?


Some colors stopped applying

Hello, once again VS updates screwed up my favorite extension!
It seems like these don't apply anymore:

Sematic Normal Method
Semantic Property (when used, but it works when declared)

Examples:

base.OnNavigatedTo before was the same yellow as in public override async void OnNavigatedTo
IsBusy = true before was purple (same color as parameters)

I am using VS 2017 v15.8.7
Any way to restore those colors?
Thanks!

NOTE: it looks like that only happens when the methods has warnings. For example here I had a
try {
    //...
}
catch (Exception ex) {
    //...
}

where ex was not used...


Extending GeneratorEnqueuer to handle finite generators.

The GeneratorEnqueuer class is used by Keras to generate batches of data. Even though it's an internal class, it can be used for building fast training and prediction pipelines that do not use the *_generator() methods. One core assumption of the class is that the underlying generator must be able to produce infinite amount of data. This is useful when we perform training but can be limiting when we want to make predictions (especially when you combine Spark + Keras).
This PR extends GeneratorEnqueuer to handle finite generators and it was tested both with threads and processes. The main idea is that we catch the StopIteration which is raised when the underlying generator is exhausted and gracefully exit the thread. When we call get() on the enqueuer instead of sleeping directly when the queue is empty, we check if any of the threads are active. If all of them are inactive we raise a StopIteration exception, else if at least one thread is still running we sleep.
The changes should have no effect if the underlying iterator is infinite. When it is finite, we raise a StopIteration exception once the underlying generator is exhausted to inform the user of the class that the data have been processed.

@fchollet Roger. I'll send an update next week. :)

@fchollet I think it's ready from my part. Please have a look and let me know.
I added two additional tests that cover the case of the finite generator (one with threads and one with processes). I also noticed that the PR is partially covered by the two test_generator_enqueuer_fail_* tests. Apologies for the multiple commits; I had a hard time with PEP8 policies as I don't have it installed on my system. I can squash the commits if you want, but I think you typically do this on merge.


[Explore] Time picker custom setting for timezone

In Advanced range type, user can type in time with free text (this is great! l like this design and implementation!!):

But in Custom range type, user have to set time by click on hour/minute/second, free text will be overwritten when user close calendar.

I think the 3 columns for hour/minute/second is a little extra.

The settings for Advanced vs Custom should be consistent: one place allow user type free text, the other is not, with no obvious reason or hint from UI. When i type in time in Custom range type and then get removed , i thought it was a bug.

Environment
latest mater
@junlincc @zhaoyongjie

Thanks for the nice comment for Advanced! @graceguo-supercat
much more users can do in the free text input, but we haven't had a chance to better document the new features yet.
if you think below would be useful for your users, pls also help test them and give us feedback  🙏 @zuzana-vej  @etr2460

Thanks for the nice comment for Advanced! @graceguo-supercat
much more users can do in the free text input, but we haven't had a chance to better document the new features yet.
if you think below would be useful for your users, pls also help test them and give us feedback  🙏 @zuzana-vej  @etr2460

again, thanks for the valuable feedback, we will get this enhancement done by 1.0 release.
The settings for Advanced vs Custom should be consistent
@zhaoyongjie 🙏

again, thanks for the valuable feedback, we will get this enhancement done by 1.0 release.
The settings for Advanced vs Custom should be consistent
@zhaoyongjie 🙏

This specifications for the new Timepicker is great. I will share with airbnb users after our next release.

This specifications for the new Timepicker is great. I will share with airbnb users after our next release.

Hi, @graceguo-supercat. There are some UX usage issues here.

In the Custom Frame:
The new Datepicker(Antd Datepicker) use mouse onBlur event as shortcut to cancel value input. (so the Datepicker value cannot be saved in time)
If you need to confirm the change, you need to use the enter key or mouse click the Ok button. (submitted Datepicker value)

https://user-images.githubusercontent.com/2016594/103897402-777da300-512e-11eb-80a8-4c428b91f533.mp4

For Advanced Frame vs Custom Frame consistent
Custom Frame is just a subset of the Advanced Frame representation range, Advanced Frame has more rich datetime-expression

Hi, @graceguo-supercat. There are some UX usage issues here.

In the Custom Frame:
The new Datepicker(Antd Datepicker) use mouse onBlur event as shortcut to cancel value input. (so the Datepicker value cannot be saved in time)
If you need to confirm the change, you need to use the enter key or mouse click the Ok button. (submitted Datepicker value)

https://user-images.githubusercontent.com/2016594/103897402-777da300-512e-11eb-80a8-4c428b91f533.mp4

For Advanced Frame vs Custom Frame consistent
Custom Frame is just a subset of the Advanced Frame representation range, Advanced Frame has more rich datetime-expression

Thanks @zhaoyongjie I didn't realize enter key is the key :)
I have a few other things to clarify:

Midnight in Custom type: is it Midnight UTC time zone? or user's local time zone? Is it the last midnight or the next midnight in the future? Please make it clear:

Thanks @zhaoyongjie I didn't realize enter key is the key :)
I have a few other things to clarify:

Midnight in Custom type: is it Midnight UTC time zone? or user's local time zone? Is it the last midnight or the next midnight in the future? Please make it clear:

is it Midnight UTC time zone?

Yes, it's UTC time zone, and we have plan to allow user to set time zone.

is it Midnight UTC time zone?

Yes, it's UTC time zone, and we have plan to allow user to set time zone.

Advanced date type has many new operators, like dateadd, datetrunc etc, which we don't have before. We should offer a full list of feature descriptions for users. Otherwise for those non-tech background users, they don't know how to use it or how to find the secret.

Yes! I am well aware of the need of adding tooltip of advanced syntax description, as I'm one of the non-tech background users😅. We will get both projects started within a couple weeks. @graceguo-supercat

Advanced date type has many new operators, like dateadd, datetrunc etc, which we don't have before. We should offer a full list of feature descriptions for users. Otherwise for those non-tech background users, they don't know how to use it or how to find the secret.

Yes! I am well aware of the need of adding tooltip of advanced syntax description, as I'm one of the non-tech background users😅. We will get both projects started within a couple weeks. @graceguo-supercat

is it Midnight UTC time zone?

Yes, it's UTC time zone, and we have plan to allow user to set time zone. it's a committed enhancement in roadmap @graceguo-supercat

@junlincc Can user set time zone in superset right now?
If yes, how can user do that?


Uncaught SyntaxError: Unexpected token 'export'

@vite-pwa/sveltekit: 0.2.8
Sveltekit version: 1.20.2
installed by npm:
vite-plugin-pwa: 0.16.7
workbox-build: 7.0.0
I am getting a runtime error with a production build with injectManifest strategy when the page attempts to load the serviceworker: Uncaught SyntaxError: Unexpected token 'export'.  The plugin is building the SW as a module but workbox-window is using the 'classic' type to register the SW so it fils to load.
I posted https://github.com/vite-pwa/vite-plugin-pwa/issues/597 from a previously working workaround to this issue to force the build to be 'iife' using injectManifest.rollupFormat: 'iife' but that workaround stopped working in the latest version of this plugin and workbox-build.
// vite.config.ts
const config: UserConfig = {
  ...
  plugins: [
    sveltekit(),
    SvelteKitPWA({
      minify: false,
      srcDir: './src',
      mode: process.env.NODE_ENV === 'development' ? 'development' : 'production',
      strategies: 'injectManifest',
      filename: 'sw.ts',
      useCredentials: true,
      injectRegister: false,
      manifest: {...},
      injectManifest: {
        globPatterns: ['client/**/*.{js,css,ico,png,svg,webp,woff,woff2}'],
      },
      devOptions: {
        enabled: true,
        type: 'module',
        navigateFallback: '/',
      },
    }),
  ],
}

// app.ts
  import { useRegisterSW } from 'virtual:pwa-register/svelte'

  const {} = useRegisterSW({...})

I have worked around this issue for now by copying https://github.com/vite-pwa/vite-plugin-pwa/blob/main/src/client/build/register.ts and https://github.com/vite-pwa/vite-plugin-pwa/blob/main/src/client/build/register.ts and changing the workbox-window registration type to module.

You can fix pwa kit plugin version where the pwa plugin build the sw, but once you update Kit it will not work.
I've seen similar errors due to ancient libraries in the repo, that's why we switch to kit build.

So is the recommendation to no longer use this plugin with Sveltekit? I don't understand your response.
I'm working on updating my app to the latest version of sveltekit 1.27.6, and updating other dependencies.

in SvelteKit there is no way to build the service worker on client build since it is build before the server
we need to build the service worker in server build using a custom Vite plugin to be called after prerender process (writeBundle Vite Hook) but before closeBundle SvelteKit Vite build plugin hook: we need the html pages to add them to the sw precache
when building the service worker originally, the output generated by Vite was weird, with exports (kit 1.20 IIRC)
then we change the plugin to delegate the service worker build to SvelteKit

If SvelteKit is unable to build your service worker, you should check the cause:

add mode: 'development'  to Vite config file and check the service worker output or
you can find serviceWorker in the build output and check if adding manually the type module in the registration works (I think it won't work).


Set to first page on search redirect

https://github.com/patrickdemooij9/SeoToolkit.Umbraco/issues/214

Hi @kows. Change looks great! Thank you for the find and the PR


Comet 0.3.0 jar file not present

Describe the bug
I noticed that the file "comet-spark-spark3.4_2.12-0.3.0.jar" is not present in the "jars" folder inside the image of version 0.3.0.
Steps to reproduce
No response
Expected behavior
No response
Additional context
No response

Thanks for reporting this. This is a known issue that was fixed for the 0.4.0 release:
https://github.com/apache/datafusion-comet/pkgs/container/datafusion-comet/307947900?tag=spark-3.4-scala-2.12-0.4.0


mobile: add new preview profile for building apps with separate bundle ID

PR Checklist

[ ] Includes changes to desk files
[X] Describes how you tested the PR locally (test ship vs livenet)
[ ] If a new feature, includes automated tests
[ ] Comments added anywhere logic may be confusing without context

Changes

Configures Android to have preview and production product flavors with differing app icons and names
Configures iOS to have Landscape-preview and Landscape app targets with differing app icons and names
Updates Expo/EAS configs to support preview and production profiles
Updates CI workflows to allow user to choose profile while triggering jobs
Disables the push notification permission prompt for now in this repo so it doesn't conflict with the existing version of tlon-mobile on testers' devices

Unfortunately, I had to change the engines value in package.json because Expo's build servers don't support Node v20 yet. I set the Node engine as >=18.18.0 so we can continue using v20 locally and in other build processes.
Tested by running the eas build commands locally. New bundles are available in Google Play and Apple App Store already.

Hey, could we include the latest changes from tlon-mobile (notifications linking fix) on here too?

We can, although since we still need to add push notifications to this version and we're not planning to use this one for production releases yet, I think it's OK to add the latest from tlon-mobile in a separate PR.

Tracking @patosullivan 's suggestion (syncing changes from the other repo) in Linear. Thanks for surfacing.


InstallLicenseConfigure fails with ArcGISConfigure

@shailesh91
Im installing Portal and webadaptor on a server each. Started with InstallLicensenConfigure and met an error. I tested Install and InstallLicenses. They work fine. But when using InstallLicensenConfigure I first get a warning then invalid MOF definition
Any idea to resolve this?
This is the error in Powershell window:
WARNING: It is not recommended to use domain credential for node 'portalserver'. In order to suppress the warning, you can add a property named
'PSDscAllowDomainUser' with a value of $true to your DSC configuration data for node 'portalserver'.
Write-NodeMOFFile : Invalid MOF definition for node 'portalserver': Exception calling "ValidateInstanceText" with "1" argument(s): "Undefined propety State
At line:32, char:2
Buffer:
"ArcGISConfigure";
;^
Insta
"
At C:\Windows\system32\WindowsPowerShell\v1.0\Modules\PSDesiredStateConfiguration\PSDesiredStateConfiguration.psm1:2369 char:21
...             Write-NodeMOFFile $Name $mofNode $Script:NodeInstanceAlia ...
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidOperation: (:) [Write-Error], InvalidOperationException
FullyQualifiedErrorId : InvalidMOFDefinition,Write-NodeMOFFile

Directory: E:\ArcGIS_Automation\ArcGISConfigure
Mode                LastWriteTime         Length Name

a----       10.05.2019     15:42           7116 portalserver.mof.error
Write-NodeMOFFile : Invalid MOF definition for node 'webadapterserver': Exception calling "ValidateInstanceText" with "1" argument(s): "Undefined property Sate
At line:29, char:2
Buffer:
"ArcGISConfigure";
;^
Insta
"
At C:\Windows\system32\WindowsPowerShell\v1.0\Modules\PSDesiredStateConfiguration\PSDesiredStateConfiguration.psm1:2369 char:21
...             Write-NodeMOFFile $Name $mofNode $Script:NodeInstanceAlia ...
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidOperation: (:) [Write-Error], InvalidOperationException
FullyQualifiedErrorId : InvalidMOFDefinition,Write-NodeMOFFile

a----       10.05.2019     15:42          15296 webadatorserver.mof.error
Compilation errors occurred while processing configuration 'ArcGISConfigure'. Please review the errors reported in error stream and modify your confiuration code appropriately.
At C:\Windows\system32\WindowsPowerShell\v1.0\Modules\PSDesiredStateConfiguration\PSDesiredStateConfiguration.psm1:3917 char:5

throw $ErrorRecord

CategoryInfo          : InvalidOperation: (ArcGISConfigure:String) [], InvalidOperationException
FullyQualifiedErrorId : FailToProcessConfiguration

@grumor can you confirm that you don't have the xNetworking module installed? This error sounds very similar to some of the users have encountered due to submodule naming conflicts.

@shailesh91 i will check during the weekend. Do have a brief explanation how to check if it is install. (I could probably google it to find how, but if you have it by your fingers, i will preseate a quick solution)

@shailesh91  I have identified the reason why this error occurs!  Our servers is created by Powershell DSC by our partner, and are using a newer version of the xFirewall resources that is embedded in the ArcGIS module. In the ArcGIS module version 2.0 is used and my partner use 5.4. The reason is a naming conflict when running the ArcGIS module.
The lead specialist that looked into this suggested, you should rename the xFirewall resource in the code or check if it exists.  It was enough to delete del 'C:\Program Files\WindowsPowerShell\Modules\xNetworking' -Force then it worked fine.  How to proceed to fix this?

@grumor this is being renamed in the upcoming release.

@shailesh91  thx!

Looking forward to test. THX


Add Quickcheck to spec to motivate dev dependency

Over at #68 I had an issue with using Id to express a traversable law - i.e. using an implementation within the spec.
At #66 @phadej suggested we motivate some reason to depend on the spec so we can chart it's use and have a reason for versioning.
I'd like to see a law quickcheck test script included in the spec. Something like: https://github.com/folktale/laws
That would allow people to add fantasy-land as a dev dependency as well as give me a reason to include Id
My vision is something like:
var fl = require('fantasy-land')
fl.quickcheck(MyType, [fl.functor, fl.traversable, fl.foldable]);

I also believe it makes sense to do the derivations from fl, but that might be overboard.

I've moved the quick check lib to fantasy land. It's not as simple as what you've got in your example, but could easily be added via a wrapper!
https://github.com/fantasyland/fantasy-check

I'm going to close this, as we do have a checking library for it now. You can suggest helper methods (see @DrBoolean idea above) in that library now :)


import should default to nuking existing database

Easiest way is to:
DROP DATABASE nitro; CREATE DATABASE nitro;
We could optionally add a flag to import over existing one, but it's probably the edge case.

should prompt before nuking the database.

So we ask for a new database during import, I think we can close this out since we do not use the default nitro database?

Yup, closing.


traefik: 1.6.5 -> 1.7.0

Signed-off-by: Vincent Demeester<EMAIL_ADDRESS>Motivation for this change
Bump to latest version
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 nox --run "nox-review wip"
[x] 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)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build traefik

Success on x86_64-darwin (full log)
Attempted: traefik
Partial log (click to expand)
find: '/nix/store/6h02k9cx361qgh12wfx54aja3jh58by9-traefik-1.7.0': No such file or directory
find: '/nix/store/6h02k9cx361qgh12wfx54aja3jh58by9-traefik-1.7.0': No such file or directory
find: '/nix/store/6h02k9cx361qgh12wfx54aja3jh58by9-traefik-1.7.0': No such file or directory
find: '/nix/store/6h02k9cx361qgh12wfx54aja3jh58by9-traefik-1.7.0': No such file or directory
find: '/nix/store/6h02k9cx361qgh12wfx54aja3jh58by9-traefik-1.7.0': No such file or directory
find: '/nix/store/6h02k9cx361qgh12wfx54aja3jh58by9-traefik-1.7.0': No such file or directory
find: '/nix/store/6h02k9cx361qgh12wfx54aja3jh58by9-traefik-1.7.0': No such file or directory
find: '/nix/store/6h02k9cx361qgh12wfx54aja3jh58by9-traefik-1.7.0': No such file or directory
find: '/nix/store/6h02k9cx361qgh12wfx54aja3jh58by9-traefik-1.7.0': No such file or directory
/nix/store/5azqh334giyq1cf303dmanf7kgv9s8qm-traefik-1.7.0-bin

Success on aarch64-linux (full log)
Attempted: traefik
Partial log (click to expand)
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/lafxxg388ssdk8p0s7bksvixbnmqwpjq-traefik-1.7.0-bin
shrinking /nix/store/lafxxg388ssdk8p0s7bksvixbnmqwpjq-traefik-1.7.0-bin/bin/traefik
strip is /nix/store/y4ymnvgxygpq05h03kyzbj572zmh6zla-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/lafxxg388ssdk8p0s7bksvixbnmqwpjq-traefik-1.7.0-bin/bin
patching script interpreter paths in /nix/store/lafxxg388ssdk8p0s7bksvixbnmqwpjq-traefik-1.7.0-bin
checking for references to /build in /nix/store/lafxxg388ssdk8p0s7bksvixbnmqwpjq-traefik-1.7.0-bin...
strip is /nix/store/y4ymnvgxygpq05h03kyzbj572zmh6zla-binutils-2.30/bin/strip
/nix/store/lafxxg388ssdk8p0s7bksvixbnmqwpjq-traefik-1.7.0-bin


One jenkins multiple teams

Hi,
we have one Jenkins instance and multiple Slack teams (projects). Current plugin in global settings defines Slack "secrets" so you can't configure it for multiple teams.
So either put this to the build configuration or enable definition of Teams in th global settings where a value can be than picked for a specific build.
This is quite a showstopper for our case.
Can I expect the fix in a reasonable time or do we have to make a change ourselves?
thanks in advance
Peter

There is still the issue here of having multiple jenkins builds on the same instance posting to different rooms. You can specify a build to post to different rooms but it doesnt seem like you can have a single global config post to different rooms given that the integrations all have their own api tokens.

Slack Plugin 1.7 has been released. A fix for this has been included in it. Please try out the new version and if it doesn't resolve this issue then ask for this to be re-opened and provide more details.

I do not believe this is working.
We have Slack plugin 1.8 and jobs will only post to the first channel specified in the list. Any additional channels get nothing.

Encountering the same issue as @spuniun.

@spuniun @cmeury see issue #98

Can you just specify the channel to be notified at the job configuration instead of being global? we have several teams using same jenkins and would be nice just to let them configure the channel to be notified at the job so there is no impact on the global configuration


Convert Attributes to NamedTuple

In order to pass arguments around recipes I convert attributes to namedtuples with the following function:
Base.NamedTuple(attrs::Makie.Attributes) = NamedTuple([pattr.first => pattr.second isa Attributes ? pattr.second : pattr.second[] for pattr in attrs])

Now, when I do this locally that's type piracy.
Would it hurt if the function was defined in Makie.jl ?
I could do a PR in case.
coming from #1019

scratch that. Base.NamedTuple(::Attributes) seems to work fine already. sorry for the noise.

Btw, I wouldn't actually do that, because it creates lots of type information to just then discard it again! How do you use that anyway?
you can either do plot!(...; attributes...) or plot!(recipe_plot, attributes, args...), to pass through attributes to a plot.

I don't quite remember why I ended up getting this road.. Probably I hit some kind of error and it came naturally to convert to NamedTuple, which is used for all julia functions. But I see now that it can work directly with passing attributes. Thanks!


CurveTexture doesn't work on some Android devices

Godot version:
3.0, 3.1
OS/device including version:
Sony F3311 (Mali-T720) and ASUS Z010D (Adreno 306) and many others.
Issue description:
When using CurveTexture, it gets rendered as black on many Android devices.
This issue was created after PR #15774 was closed - there are more details there.

Suggested fix from @reduz in https://github.com/godotengine/godot/pull/15774#issuecomment-366939216


Allow moving layers using the keyboard arrow keys

Summary
When the move tool is selected, this PR allows to use the keyboard keys to move the selected layer by 1px.
Related Issues / Discussions
https://github.com/invoke-ai/InvokeAI/issues/7120
Known issues

~The layer moves even if it's disabled~ fixed
If we are on another menu, then we have double inputs, e.g. if we click on the "Layers" tab then pressing right arrow moves the selected layer AND changes the tab from "Layers" to "Gallery" -> Probably an issue with the focus manager itself

Checklist

[x] The PR has a short but descriptive title, suitable for a changelog

Superseded by  #7368


Fix toolchain build

change the repo hermit is pulled from to hermit-playground.
enable libgomp compilation and remove workaround
enable set -e, so we don't miss important errors (else the dependent compilations will produce weird and hard to find errors)

Currently, compiling the toolchain from source fails, since parts of the cmake files for libhermit-rs got moved (?) to hermit-playground.
What is the reason behind disabling libgomp support in gcc and then trying to install the libgomp.spec file? For me this file does not exist with the old gcc flags. I replaced --disable-libgomp with --enable-libgomp, removed the workaround and everything build fine.
In addition, the toolchain does not build on archlinux with gcc9. This is an upstream binutils issue, there is an unmerged workaround on the mailing list. Currently I have just applied it manually, but since we have forked binutils anyways, we might apply it there aswell?

👍


Description is not working for object CRD fields in the extensions

Description
We have a following object CRD field:
config:
    additionalProperties:
      type: string
    description: Config defines the configurations that can be applied
      to the eventing backend
    type: object

the description is not being rendered as expected:

for usual fields the description is working:
typeMatching:
      description: TypeMatching defines the type of matching to be done
        for the event types
      type: string

is rendered as:

/label bug

/bug


netlify: move to open source plan

Originally opened by @myitcv in https://github.com/cuelang/cuelang.org/issues/83
Consider switching to the Netlify open source plan, which is the same as the pro plan with unlimited team members.
Per https://www.netlify.com/legal/open-source-policy/ we must satisfy the following to qualify:

[x] Includes a license listed on the Open Source Initiative approved license list or a Creative Commons license that includes “attribution” or places the work in the public domain
[x] Features a Code of Conduct at the top level directory of the project repository or prominently in the documentation (with a link in the navigation, footer, or homepage).

Possibly create a specific CONDUCT.md file in the repo root
Need to add link from the site to the conduct file

[x] Must feature a link to our service on your main page

Need to add link to home page

[x] Must not be a commercial project, whether created by a company or an individual. This prohibition includes commercial support and hosting services.

cc @mpvl

This is now resolved.


Debug session continues to step even though docker container stopped

Please answer the following before submitting your issue:
Note: Please include any substantial examples (debug session output,
stacktraces, etc) as linked gists.

What version of Delve are you using (dlv version)?

dlv version
Delve Debugger
Version: 1.0.0-rc.1
Build:

What version of Go are you using? (go version)?

go version
go version go1.8.3 windows/amd64

What operating system and processor architecture are you using?

Client:
Windows 10 Enterprise, version 1607
Intel 64-bit operating system, x64-based processor
Container:
Debian GNU/Linux 8

What did you do?

I started a container process with:
dlv exec ./collector -l <IP_ADDRESS>:2320 --headless=true --log=true -- --config=/go/src/perfhub-worker/configs/collector-config.json --workerid="4"
I set a breakpoint in Visual Studio Code and started a debug session, with the "Attach to perfhub collector" configuration from the following launch.json file.
{
"version": "0.2.0",
"configurations": [
{
"name": "Attach to perfhub collector",
"type": "go",
"request": "launch",
"mode": "debug",
"remotePath": "/go/src/perfhub-collector/collector",
"port": 2320,
"host": "localhost",
"program": "${workspaceRoot}",
"env": {"GOPATH": "C:/Users/rpfister/go",
"GOROOT": "C:/Go"},
"args": [],
"showLog": true
}
]
}
I was then able to step through code.  But, at a certain point, the container exited, but VS Code continued to step through code as if nothing had happened.

What did you expect to see?

I expected the VS Code debug session to somehow notice that there was no response to "next".  A message such as "Connection lost, unable to step over " (or continue, or step into...) would be appropriate,   The it should stop debugging.

What did you see instead?

VS Code debug continued to behave as if it were continuing to control execution in the docker container.

I've studied this and see that I should have "mode": "remote" (not "debug") to connect to the headless dlv server I started in a docker container.  I think I was actually debugging a process started locally by VS Code.  I'm still frustrated; I've spent a lot of time trying to figure the right way to connect to a docker container.  Any help would be appreciated.

You should map the port 2320 to a host port (assuming you aren't already doing that) then connect to it with whatever configuration vscode takes to connect to a remote instance of delve.

Thanks @aarzilli for your response.  I have mapped port 2320 to host port 2320, so that's not the problem.  The "configuration vscode takes" is the problem.  The Visual Studio Code documents essentially say to use the configuration attributes expected by your debugger (https://code.visualstudio.com/docs/editor/debugging), with some generic discussion of attributes that are commonly used.  There is also some hover help, which I have found so far to be misleading.
Visual Studio Code pushes the problem to debugger; Delve doesn't really document - at least as far as I can see - how to use VS Code with Delve.  I'm left in the middle.
Any suggestions from users who successfully debug docker containers from Visual Studio Code would be appreciated.

This is something that's handled by the VSCode plugin, if it's not documented anywhere you'd have to read the source code of that or ask for help here. I can't help you because I don't use VSCode.

Here's what I've figured out about launch.json (the configuration file for the debugger within Visual Studio Code) so far; I'd appreciate any corrections.
“name”: This is a string, user defined, that identifies this particular launch configuration.  It will display in the debug viewlet, but has no other purpose.
“type”: “go” is the only value that works for debugging a go program.
“request”: “launch” is the only value that works for debugging a go program.  Hovering over this value can be misleading because it suggests that the value can be either “launch” or “attach”. It looks like, for Delve, the go debugger, the only acceptable value is “launch”.
“mode”:  The hover help says "One of 'debug', 'remote', 'test', 'exec'." It looks like debug, test, and exec are for testing are for debugging locally.  It seems like remote is the only logical choice, but hasn't worked yet.
“remotePath”: The hover help is "If remote debugging, the path to the source code on the remote machine, if different from the local machine." Seems straightforward.
“port”: The hover help is "The port that the delve debugger will be listening on." Again, seems straightforward, I've set this to the port configured on the remote machine by the dlv exec command.
“host”: The hover help is "The host name of the machine the delve debugger will be listening on." Here I'm a little less sure what to do. In my case both the docker container and VS
Code are on the same machine - my development machine.  In the delve command within the docker container I set it to listen on <IP_ADDRESS>, so I set this configuration attribute to the same value.
“program”:  The hover help is "Path to the program folder (or any file within that folder) when in 'debug' or 'test' mode, and to the pre-built binary file to debug when in 'exec' mode."  Since I'm in remote mode, maybe I should not have this set.
"env": The hover help is "Environment variables passed to the program."  Does this have any applicability when in remote mode?

The current status is that I seem to be able to connect to the remote server, but I'm not able to step through code.  The debugger seems to freeze, as shown in the attached file.
Verbose logs as written to Debug Console:
C:\Users\rpfister\AppData\Local\Temp\vscode-go-debug.txt 11:38:18 AM, 8/9/2017 InitializeRequest InitializeResponse InitializeEvent SetBreakPointsRequest All cleared Creating on: C:\Users\rpfister\go\src\perfhub-collector\perfhub-collector.go (C:/Users/rpfister/go/src/perfhub-collector/perfhub-collector.go) :100 Creating on: C:\Users\rpfister\go\src\perfhub-collector\perfhub-collector.go (C:/Users/rpfister/go/src/perfhub-collector/perfhub-collector.go) :101

The latest is that I can connect to the headless server (the one inside the docker container) easily enough from the command line.  I can set breakpoints, list them, and step through the code.  I just can't do it from Visual Studio Code.

I now see this as a VS Code bug; not a delve problem.

Did you ever succeed in remote debug with VS code? If so I would very much like the step. Thanks in advance. I can tell you that I have also looked at eclipse che as a potential ide. I guess we a all looking for at good ide that supports delve - or some documentations on making it work :-)
Just a note to derekparker: Thanks for the good work on Delve

Yes, it does now work.  This is not a delve problem, but a bug in the go plugin for Visual Studio Code (vscode-go) that was causing problems.  That was corrected and put into an August; so if you have the a version of the plugin no older than that you should be fine.
Have you tried the sample project at https://github.com/lukehoban/webapp-go/tree/debugging ?
Be sure you're on the debugging branch and look for the launch.json file in the .vscode directory.  Be sure to read the comments that explain how to use it.  If you still have questions you can send me an email at my github user name at yahoo dot com.

Yes, fantastic it actually works in the latest - thanks Robert!


JdbcDbClient uses get/setString for binary data

TL;DR: does it make sense to replace setString/getString with setBytes/getBytes?
Current table structure for Jdbc is FIELDN TEXT, however the stored data is binary. It causes String <-> byte[] conversions.
JDBC has multiple ways to upload binary:
void setBinaryStream(int parameterIndex, java.io.InputStream x)
void setBinaryStream(int parameterIndex, java.io.InputStream x, int length)
void setBytes(int parameterIndex, byte x[])
It might easily be the case that setBytes(int, byte[]) turns out to be more effective than setString(int, byteIterator.toString())
On top of that, certain JDBC drivers might implement extensions to stream bytes from a source without intermediate byte[] copy (see https://github.com/pgjdbc/pgjdbc/pull/953 )
On the read side, JDBC has
byte[] getBytes(int columnIndex)
java.io.InputStream getBinaryStream(int columnIndex)
Read part can have extensions as well via public <T> T getObject(int columnIndex, Class<T> type).
See
https://github.com/brianfrankcooper/YCSB/blob/d6e57c3f575914c32556ca4eb5802073681b288c/jdbc/src/main/java/com/yahoo/ycsb/db/JdbcDBClient.java#L385
https://github.com/brianfrankcooper/YCSB/blob/d6e57c3f575914c32556ca4eb5802073681b288c/jdbc/src/main/java/com/yahoo/ycsb/db/JdbcDBClient.java#L413

yeah this would be a nice improvement.


HTTP 406 error code on mail send

Issue Summary
A summary of the issue and the environment in which it occurs. If suitable, include the steps required to reproduce the bug. Please feel free to include screenshots, screencasts, or code examples.
Randomly we receive exception on sending email:
message: "Not Acceptable"
stack: "Error: Not Acceptable
    at node_modules/@sendgrid/client/src/classes/client.js:146:29
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (internal/process/task_queues.js:97:5)"

Steps to Reproduce
Code to reproduce:
const sgMail = require('@sendgrid/mail');
sgMail.setApiKey(sendgridApiKey);
sgMail.send(data);

No way to reproduce as it happens randomly. I confirmed that data is not really important here. I could send mail with same data that was previously failing.
Code Snippet
const sgMail = require('@sendgrid/mail');
sgMail.setApiKey(sendgridApiKey);
sgMail.send(data);

Exception/Log
message: "Not Acceptable"
stack: "Error: Not Acceptable
    at node_modules/@sendgrid/client/src/classes/client.js:146:29
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (internal/process/task_queues.js:97:5)"

Technical details:

sendgrid-nodejs version: "@sendgrid/mail": "^7.6.0"
node version: 12.19.1

We're also seeing this error.
I don't think it's an issue with this library, but with the actual SendGrid API. We're been using version 6.5.3 of this library for more than a year now with no problems.
We haven't changed anything on our end in a long time. This error just started popping up in the last week.

@childish-sambino @thinkingserious @twilio-dx @eshanholtz can you help with this? Thanks you

Update - We haven't seen this error since Feb 14 @ 7:22pm CST. Prior to that, we were getting the error ~10 times a day. So it might be fixed? Or maybe just minimized?

Thanks for submitting a GitHub issue! We are very sorry that you are running into this problem. In order to better serve you, as this does not present itself as a library specific issue, we would like to ask that you reach out to our support team at support.sendgrid.com.


Problem of latency using NeuralAmpmodeler

Thanks for taking the time to write a bug report! Use the following prompts to help you describe what's going on.  The more info you provide, the easier it'll be to understand how to fix it for you without having to come back and ask you questions.
Describe the bug
Hi,
I am currently using Neural Amp Modeler and i have a Univerasl Audio Volt 276. I have a guitar conected to the Volt 276.
I have a latency time of 2s between the strum of the strings and the sound in the speakers or in the headphones
Tha,nk you for your help and support
Roland
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
See attached file

Desktop (please complete the following information):

OS [e.g. Windows 10,
Plugin version 04/04/2024 - v0.7.9
Standalone, VST3, or AU
DAW (if applicable)
Audio interface: Volt 276
Sample rate 44.1 kHz
Buffer size 4096

Additional context
Add any other context about the problem here.

Closing since I assume this was fixed w the above advice.

Dear Steven,Yes it is closed Thank you for your help and support Kind regards RolandEnvoyé depuis l'application Mail Orange----------------De: "Steven Atkinson" @.>Envoyé: lundi, 27 mai 2024, 1:23À: "sdatkinson/NeuralAmpModelerPlugin" @.>Cc: "roland6939" @.>;"Author" @.>Objet: Re: [sdatkinson/NeuralAmpModelerPlugin] Problem of latency using NeuralAmpmodeler (Issue #469)
Closing since I assume this was fixed w the above advice.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Dear DomThank you for  your reply I do apologize for my late  answerKind regardsRoland Envoyé depuis mon appareil Galaxy
-------- Message d'origine --------De : Dom Mcsweeney @.> Date : 13/05/2024  00:36  (GMT+01:00) À : sdatkinson/NeuralAmpModelerPlugin @.> Cc : roland6939 @.>, Author @.> Objet : Re: [sdatkinson/NeuralAmpModelerPlugin] Problem of latency using NeuralAmpmodeler (Issue #469)
To lower your latency, choose a lower buffer. 8192 is too long. Try something like 64 or 128 @ 48khz, and change 'driver type' from Direct Sound to ASIO.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


ImportError: cannot import name 'build'

Hi, developers of BRNet. I have installed BRNet following the getting_started.md. All prerequisites and their corresponding version are meet the requirements. (python==3.6.9  cuda==10.1  pytorch==1.5.0 torchvision==0.6.0 mmcv==1.3.14  mmdet==2.17.0  mmdet3d==0.17.0)

But when I run the train.py, I get error like this:

Traceback (most recent call last):
File "tools/train.py", line 212, in
main()
File "tools/train.py", line 176, in main
test_cfg=cfg.get('test_cfg'))
File "/home/BRNet/mmdetection3d/mmdet3d/models/builder.py", line 58, in build_detector
cfg, default_args=dict(train_cfg=train_cfg, test_cfg=test_cfg))
File "/home/BRNet/mmcv/mmcv/utils/registry.py", line 212, in build
return self.build_func(*args, **kwargs, registry=self)
File "/home/BRNet/mmcv/mmcv/cnn/builder.py", line 27, in build_model_from_cfg
return build_from_cfg(cfg, registry, default_args)
File "/home/BRNet/mmcv/mmcv/utils/registry.py", line 45, in build_from_cfg
f'{obj_type} is not in the {registry.name} registry')
KeyError: 'BRNet is not in the models registry'

And when I test the installation by "from mmdet3d.apis import init_detector, inference_detector", I get error like this:

Traceback (most recent call last):
File "", line 1, in
File "/home/BRNet/mmdet3d/apis/init.py", line 1, in
from .inference import inference_detector, init_detector, show_result_meshlab
File "/home/BRNet/mmdet3d/apis/inference.py", line 11, in
from mmdet3d.models import build_detector
File "/home/BRNet/mmdet3d/models/init.py", line 1, in
from .backbones import *  # noqa: F401,F403
File "/home/BRNet/mmdet3d/models/backbones/init.py", line 3, in
from .nostem_regnet import NoStemRegNet
File "/home/BRNet/mmdet3d/models/backbones/nostem_regnet.py", line 2, in
from ..builder import BACKBONES
File "/home/BRNet/mmdet3d/models/builder.py", line 3, in
from mmdet.models.builder import (BACKBONES, DETECTORS, HEADS, LOSSES, NECKS,
ImportError: cannot import name 'build'
Thanks for your help if you know how to install BRNet to avoid these problem.

You may refer to this issue.
#5
You should take care of the version of mmdet and mmdet3d, mmdet==2.10.0 and mmdet3d==0.11.0 are recommended.
Newer versions of mmdet and mmdet3d will bring some bugs due to code changes.

Thank you~


[Monikerization] [.NET Docs Repo] Update docfx. Files can share path across monikers

.NET needs a bit more additional work here. It doesn't currently have a fusionToCConfig or real monikers to speak of right now. Need to add that before we can make further progress on the docfx.

[ ] .NET Repo monikerized
[ ] DocFX Updated to allow identically named files across monikers
[ ] CI Updated If Necessary
[ ] Necessary Changes to targeting are merged in archetype--release stage

See parent epic for details.

Approaches to fix:

Split the .NET ref docs into separate repos by moniker (legacy, latest, preview)
Feature work from the ref docs team

Chatted w/ @maggiepint -- This work can be done as part of Grouped ToC onboarding.

Docs internal feature for this: https://dev.azure.com/ceapex/Engineering/_workitems/edit/657647

Monikerization work is done. https://learn.microsoft.com/en-us/dotnet/api/overview/azure/?view=azure-dotnet&branch=main
The issue could close. Will move to .NET Docs.Ms automation.


Is Prometheus Federation feature available here now ???

Hi
This is just a question , do we have Prometheus Federation feature available with Prometheus Operator today ??? How we can implement the same
Best Regards
Sujith P V

Prometheus federation simply uses a different metrics endpoint than /metrics. You can configure the metrics path in the ServiceMonitor object. So yes, you can do federation with the Prometheus Operator.


fix ShOperator. add shell option.

Hello.
I want use login shell.
In addition, I want to choose shell environment variable because it is fixed in "/bin/sh".
Thanks.

This PR failed test?
My Local OSX is passed test.

Is difference of shell implementations significant? What kinds of shell do you use?
I think that workflow definition should be reproducible on many environments as much as possible. This change makes workflow more dependent on the execution environments. Alternative idea is to add shell: /bin/sh as an option.

Is default_ prefix necessary? I think shell option is the most flexible:
_export:
  sh:
    shell: zsh -l
+step1:
  sh>: echo this uses zsh with -l option

The default shell is /bin/sh.

@frsyuki
Ah, OK!
Thanks good advise.

@frsyuki

The default value of shell option is /bin/sh
fixed.

@danielnorberg
Thanks advise!
fixed bellow settings.
Please review.
_export:
  sh:
    shell:
      - /bin/zsh
      - '-l'
      - '-c'

including -c is not good idea because it's assuming internal behavior of ShOperator

@danielnorberg I think it's better if sh operator can accept string as well as array because of #136 or #25 issue. Necessary update is to parse the input using a better tokenizer. Maybe that's a future improvement (because I can't come up with easy code to parse).

I added a commit and created #143. shell option is now more similar to shebang. So this also works:
+calling:
  sh>: p 1 + 1
  shell: [/usr/bin/env, ruby]
  #=> 2

@frsyuki
Oh, thanks.
That is easy to understand.


ResonanceMolSupplier hangs on NO2

I can create a molecule for NO2, and a ResonanceMolSupplier, but asking the supplier if it is at the end results in a hang.
>>> from rdkit import Chem
>>> mol = Chem.MolFromSmiles('O=[N+][O-]')
>>> supp = Chem.rdchem.ResonanceMolSupplier(mol)
>>> supp.atEnd()
Killed: 9

I waited for a while but it does not return anything.  I haven't tested all the flags exhaustively, but I do find that if I use
Chem.ResonanceFlags.ALLOW_INCOMPLETE_OCTETS | Chem.ResonanceFlags.UNCONSTRAINED_CATIONS| Chem.ResonanceFlags.UNCONSTRAINED_ANIONS

then it works just fine.
The hang is present in rdkit versions 2021.09 and 2021.03, but was not present in 2019.03 (that's all I have installed at the moment).

@jasondbiggs Many thanks for reporting this, I’ll take a look and see what’s going on.

Hi,
I'm having the same bug with CN(C)C=[N+]=[N-].
Tested with the following versions of RDKit:

2020.03.1, 2020.03.6 --> works
2020.09.1, 2020.09.3 --> works, but supp.atEnd() returns True
2021.03.1, 2021.09.5 --> hangs forever, unless the resonance flags are set as in the first post.

I've been having some problems using the flags Chem.ResonanceFlags.ALLOW_INCOMPLETE_OCTETS | Chem.ResonanceFlags.UNCONSTRAINED_CATIONS| Chem.ResonanceFlags.UNCONSTRAINED_ANIONS throughout, because it is generally too permissive and creates too many resonance structures.
At the same time, I needed a reliable solution for this because it would bring my entire workflow to a standstill with no way of catching the error.
What has worked for me is to check the following to conditions:

Check that total charge of molecule is 0
Check that some atoms have non-zero formal charges.
If both conditions are True, I use the flag Chem.ResonanceFlags.ALLOW_CHARGE_SEPARATION, which so far has always worked for me.

It seems that not allowing charge separation but then providing a molecule with separated charges leads to the issue we observed (not sure that makes sense?!).

I was too optimistic, this still fails for c1ccc[s+]c1, which requires Chem.ResonanceFlags.UNCONSTRAINED_CATIONS to not freeze.

Fixed by #5701


Chat model selector cleanups

Various cleanups to the chat model selector:

For Cody Free users, made the model selector more usable and glanceable by toning down the Cody Pro badges and reducing the opacity on models they can't use
Removed by $provider in favour of a tooltip, given we have the icon already conveying that information
Fixed some of the vector provider icons that were miscentered in their 16x16 frame
Tweaked colors/opacities
Swap recommended between GPT-4 Turbo and GPT-4o
Added prominent link to documentation (which will help for discovery of Ollama, experimental and BYOK models)

Screenshots

Free plan:

Pro plan:

New docs link:

[x] Make Documentation link work

Test plan

CI

Love it! Pushing a commit to make the link work in Cody Web and not need to pass a new message to the exthost. Now it just uses an <a href>. Looks like cmdk will add a link soon and then we could slightly simplify the code: https://github.com/pacocoursey/cmdk/issues/258.

@sqs oh nice, thanks for the fix. I tried to get it working as an <a> firstly, and failed. Much nicer!

I noticed telemetry was removed. Was that intended?


Is there a CHANGELOG?

Hi everytone,
I'm using version 2.0.1, and I want to upgrade to the latest, but I can not find any changelog about that. IDK if it's broken my custom npm package. So, is there any changelog to review?
Thanks.

Hey @mksglu, there is no changelog (will be adding one moving forward, thanks for brining this to my attention)
between v2.0.1 and v3, there should be no api changes, there were a few changes internally to fix bugs such as maintaining the order the css was imported and ensuring code splitting worked properly. Other than that, nothing changed that should break your code.
If you want to share a code snippet or create a minimal reproduction, I would love to take a look and help you get the latest version working for you.

Thanks for the quick reply. I'll bump the version to the latest.
Thanks again for the package.


Ensure all servers listen on <IP_ADDRESS>

The upstream issue with <IP_ADDRESS> vs. local host strikes again (#6207).

[ ] ensure that all servers are listening on <IP_ADDRESS> (or :port, which is equivalent)
[ ] ensure that all servers log this endpoint at startup (we may be only logging ports in some places)
[ ] why didn't our CI for hotrod/all-in-one catch this?

I was referring to HotROD integration tests that includes hotrod app sending data via OTLP to all-in-one
I am aware of those two ports, my question was about all other ports that jaeger opens
your curl command is not authoritative as some of the endpoints do not accept GET requests so getting 404 does not mean misconfiguration. If anything it confirms the opposite - the server has to be reachable to respond with 404.
same point as in 2 above

I was referring to HotROD integration tests that includes hotrod app sending data via OTLP to all-in-one ---> Thanks for clarifying! will check the HotROD Integration test and update the conversation.
the server has to be reachable to respond with 404. ---> Yaa actually, my point was the same. 404 means reachable. and the curl gave 404 with the --set receivers.otlp.protocols.http.endpoint=<IP_ADDRESS>:4318 --set receivers.otlp.protocols.grpc.endpoint=<IP_ADDRESS>:4317 over-rides . Without setting above over-rides, it was giving "Recv failure: Connection reset by peer jaeger".
2 and 4. question was about all other ports that jaeger opens ---> Understood. would check for the rest of the ports that Jaeger opens and if the hostPorts are logged.

appreciate your inputs! Thanks!

Did a quick check on [1] , for the jaeger v2 docker CI, we do set the below over-rides in docker-compose-v2.yml . This should allow the hotrod example to send metrics to otlp collector [https://github.com/jaegertracing/jaeger/blob/main/examples/hotrod/docker-compose-v2.yml#L29](over http://jaeger:4318).
`

--set=receivers.otlp.protocols.grpc.endpoint="<IP_ADDRESS>:4317"
--set=receivers.otlp.protocols.http.endpoint="<IP_ADDRESS>:4318"
`

Right, but we added them after the OTEL upgrade was done and presumable out build was still green. I am verifying that now: #6226

Thanks for taking it forward @yurishkuro ! Was trying to follow various issues around this.

#6231 ---> We are introducing an environment variable JAEGER_LISTEN_HOST which gets used from all-in-one/v2 config. This defaults to localhost when not defined. In case of running from docker, it overrides to <IP_ADDRESS>
Suppress warning about <IP_ADDRESS> ---> As a result of running at port <IP_ADDRESS> from inside docker, we get an warning as below currently which needs to be suppressed appropriately.

Using the <IP_ADDRESS> address exposes this server to every network interface, which may facilitate Denial of Service attacks.

ensure that all servers log this endpoint at startup (we may be only logging ports in some places) ---> Initial Point 3 of this Issue still needs to be answered. Will check and update the conversation.


Help me run example code

When you run the example code, you will see "package org.qcri.rheem ... does not exist".
Is there any other condition besides running Rheem's build.sh in order to run the example code?

me too


refactor: new builder api

Codecov Report
Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (e594055) 99.49% compared to head (31aa84f) 98.90%.

Files
Patch %
Lines

src/help.ts
97.67%
1 Missing :warning:

src/utils.ts
90.90%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #63      +/-   ##
==========================================
- Coverage   99.49%   98.90%   -0.59%
==========================================
  Files           8        7       -1
  Lines         197      183      -14
  Branches       50       51       +1
==========================================
- Hits          196      181      -15
- Misses          1        2       +1

Flag
Coverage Δ

unittests
98.90% <98.78%> (-0.59%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


px4io causing inner loop latency

The inner loop of PX4 involves several tasks, such as sensors, ekf2, mc_att_control, mc_pos_control, and px4io.
And in this loop, px4io is expected to read values from actuator_controls_0 topic and update PWM registers every 4ms.
However, px4io does several other things consuming CPU.
Every 20ms, it reads and publishes raw RC input and update arming state.
And every 200ms, it does DSM paring and parameter update handling.
Thus px4io sometimes does some or all of these jobs before updating PWM registers
and consequently increases inner loop latency, time elapsed from when sensors task reading gyro to when px4io task updating PWM
And another source of the latency is that px4io's priority (245) is lower than those of other tasks involved the inner loop (250).
We can find out this in "perf" result showing "io latency" meaning elapsed time between beginning of ekf2's main loop and px4io's PWM update
as shown in below perf result, maximum "io latency" is 7427us while average is 4775us indicating the inner loop suffers from significant latency
io latency: 61066 events, 291478904us elapsed, 4773us avg, min 3956us max 7108us 1007967.375us rms
As a temporary solution, I boosted px4io's priority from 240 to 250 right after mc_acc_control publishes to actuator_controls_0 topic and restore the priority right after px4io completes PWM update with the published data.
Below is the perf result with modified PX4.
Max "io latency" decreases to 4937us and avg decreases to 3787us.
io latency: 164447 events, 622789254us elapsed, 3787us avg, min 3221us max 4949us 718777.500us rms
This seems to remove one of the latency sources, px4io's lower priority.
But the another source, px4io's lower frequency jobs, is still remaining.
Thus I think below is one better solution.

divide px4io's original jobs into three groups according to their periods
create three tasks instead of px4io
assigns each job group to each task
give different and appropriate priorities to them
(For example, 250 to px4io_4ms, 245 to px4io_20ms, and 240 to px4io_200ms)

Which version of PX4 are you using?
Many of the priorities were already reviewed and adjusted a few months ago (after the last stable release). https://github.com/PX4/Firmware/blob/master/src/platforms/px4_tasks.h#L116
Before splitting the px4io into multiple threads (which requires a lot of additional memory) I'd recommend profiling the other routines (raw RC input, arming state, DSM, etc)
Here's the io update perf counter (surrounds everything you mentioned) from a random log (https://logs.px4.io/plot_app?log=167834f7-6d3f-4eb7-be9c-3d9116bc1e89).
io update: 128950 events, 74825278us elapsed, 580us avg, min 262us max 2294us 913.672us rms

The very first thing the loop does is send the actuator controls to the physical px4io (a separate processor), so I don't know that you're actually going to be able to decrease latency by splitting it.
One minor optimization is that we could probably skip things like param update and DSM pairing after arming. Param updates will cause significant jitter and in some cases aren't safe while flying.

A found a bigger problem. The MC attitude controller one of the highest priority tasks and updates when there's new gyro data, but the timestamp sample is still being set from the vehicle_attitude message.
gyro -> attitude controller -> rate controller -> px4io
io latency: 73243 events, 44045138us elapsed, 601us avg, min 162us max 1922us 410.388us rms

https://github.com/PX4/Firmware/pull/8415

I believe we can consider this resolved by https://github.com/PX4/Firmware/pull/8415.


InvalidConfigurationException: Unrecognized options "auto_mapping" under "doctrine.orm"

Hi,
after following the instructions to install this bundle, Im getting this error:

InvalidConfigurationException: Unrecognized options "auto_mapping" under "doctrine.orm"

I found other issues related to this problem but i did not find a solution..
This is my config.yml:

doctrine:
    orm:
        auto_generate_proxy_classes: %kernel.debug%
        auto_mapping: true
        entity_managers:
            default:
                mappings:
                    StofDoctrineExtensionsBundle: ~

and this below is my composer.phar:

{
    "name": "symfony/framework-standard-edition",
    "description": "The \"Symfony Standard Edition\" distribution",
    "autoload": {
        "psr-0": { "": "src/" }
    },
    "require": {
        "php": ">=5.3.3",
        "symfony/symfony": "2.1._",
        "doctrine/orm": "2.2._",
        "doctrine/doctrine-bundle": "dev-master",
        "twig/extensions": "dev-master",
        "symfony/assetic-bundle": "dev-master",
        "symfony/swiftmailer-bundle": "dev-master",
        "symfony/monolog-bundle": "dev-master",
        "sensio/distribution-bundle": "dev-master",
        "sensio/framework-extra-bundle": "dev-master",
        "sensio/generator-bundle": "dev-master",
        "jms/security-extra-bundle": "1.1._",
        "jms/di-extra-bundle": "1.0._",
        "gedmo/doctrine-extensions": "dev-master",
        "friendsofsymfony/user-bundle": "_",
        "stof/doctrine-extensions-bundle": "dev-master",
        "knplabs/knp-menu-bundle" : "1.1._",
        "knplabs/knp-menu" : "1.1.*",
        "sonata-project/exporter" : "dev-master",
        "sonata-project/jquery-bundle" : "dev-master",
        "sonata-project/block-bundle" : "dev-master",
        "sonata-project/cache-bundle" : "dev-master",
        "sonata-project/admin-bundle" : "dev-master",
        "sonata-project/user-bundle" : "dev-master",
        "sonata-project/doctrine-orm-admin-bundle" : "dev-master"

Javi

Using this config and SYmfony 2.7
orm:
        auto_generate_proxy_classes: "%kernel.debug%"
        naming_strategy: doctrine.orm.naming_strategy.underscore
        auto_mapping: true
        dql:
            datetime_functions:
                month: DoctrineExtensions\Query\Mysql\Month
                year: DoctrineExtensions\Query\Mysql\Year

I have an error :
Attempted to load class "Month" from namespace "DoctrineExtensions\Query\Mysql".
Did you forget a "use" statement for another namespace?
I tried to add a use statement in my reposotory...
If anyone as an idea ?

Same issue... I have an error :
Attempted to load class "Month" from namespace "DoctrineExtensions\Query\Mysql".
Did you forget a "use" statement for another namespace?

@BigWenceslas These classes don't belong to this bundle, nor to the Gedmo extensions. So this is the wrong place to report this.


Add new branch and new algorithm - BankerAlgorithm

Codecov Report

Merging #19 (306ae6a) into master (be263c2) will increase coverage by 0.03%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master      #19      +/-   ##
============================================
+ Coverage     33.02%   33.05%   +0.03%
+ Complexity     1786     1784       -2
============================================
  Files           239      238       -1
  Lines         13939    13910      -29
  Branches       2635     2631       -4
============================================
- Hits           4603     4598       -5
+ Misses         9095     9069      -26
- Partials        241      243       +2

Impacted Files
Coverage Δ

...t/algorithms/datastructures/skiplist/SkipList.java
92.13% <0.00%> (-4.50%)
:arrow_down:

...set/algorithms/strings/BoyerMooreStringSearch.java
77.77% <0.00%> (-3.71%)
:arrow_down:

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


Add function get_variables for class calculator

Sometime we want to know what variables we should give into TokenMap before we evaluate the expression.
Change-Id: I50e1ebb70d6e1437eda8d72e412cfa73f7e020ba

Hey @peacalm thanks for this improvement, looks good I just need to understand if we need to add that push() and pop() functions on the TokenQueue_t.
I also added some suggestions tell me what you think of them too =]

😄, your suggestions are great, I accepted them all.
I add push() and pop() for TokenQueue_t just want to make it compatible with existing code, because I changed TokenQueue_t from type std::queue to std::deque to make items in the queue visitable.
Or another way: just modify typedef std::queue<TokenBase*> TokenQueue_t; to typedef std::deque<TokenBase*> TokenQueue_t;, and modify every push() to push_back(), every pop() to pop_front().
All I want is a feature to know what variables are required in some given expression, so I must search them in TokenQueue, but items in std::queue are not iterable.
My code looks like a short hack to support what I want, maybe you have more consideration about whether to support this feature, or maybe you have better implementation. Never mind, it's just a little idea after I reading this project. Thanks very much for your feedback.

@peacalm no it's a great addition, yeah I will think about the implementation details, I will try reading these changes more carefully tonight and thinking about whether there is anything I should consider about this type change.
Thanks for contributing.

Thanks for this contribution =]

You are welcome @peacalm, thank you actually.
Yeah, so I think that for actually choosing which tool to use between std::queue and std::deque we should actually write some benchmark. I also believe this probably behaves the same as before so this is probably not a real issue. If we wanted to really tackle this problem I would start by writing a benchmark, but I will probably not have time for that =P.
And I am happy it's been helpful to you =].


Using -b to select bin config name is not working.

For example, I have a below config with "b" as bin name.
{     "bin": [         {             "name": "b",             "host": "https://privatebin.net"         }     ] }
When I try to create a paste using below command

privatebin create --bin b
privatebin create -b b

The application shows

Error: unknown flag: --bin
Error: unknown shorthand flag: 'b' in -b

By the way, it works if I set the bin name to "".

Fixed in the v2.0.1


Repository does not include license information

Awesome project, please add a license to the root of your repository :)

Each file had a license header, but yes, that's a  nice suggestion.
I added a LICENSE file to the root, just now (finally! ... sorry for such delay)


add npm build

I found this to be useful for my deploys, since I was manually adding a cap task for this already.

Thanks but this Gem is supposed to be pretty bare-bones and not opinionated about a project's npm usage. So even though this is optional it doesn't really make sense to include it since who knows how popular npm build really is.


No Humidity

My thermostat does not have a humidity sensor. Is there something I can return for the humidity value  to make it ignore it?

Humidity is more when managing both heating and cooling. If you don't have cooling system, as a quick fix, you can return an hard coded value (maybe even -1).
As a more long term solution, the HAP documentation lists the mandatory and optional functions to implement. You can have a look at it and comment these lines :)
Is this answering your question?

That's what I figured. I was hoping you'd know of a way I could just hide those options, but no worries. Thanks - it's working really well :D

You should be able to just comment out the current and target relative humidity from the Thermostat service (and the implementation in the prototype) as they are both optional to home kit in thermostats :)


Fixed - scroll issue on categories screen - 1736

##Fixes Issue
closes #1736
Changes proposed

Removed class py-2 from footer which was causing the issue.

Screenshots

##Note to reviewers
Please check it @Anmol-Baranwal @rupali-codes

@ketansaresa
Please stick to the PR template, there are specific keywords for linking the issue to the PR like fixes or closes which you can read at official docs.

Here it is not linked.

This will most likely keep the issue open even if the PR is merged, so just keep in mind for future purpose.
I'm doing it for you this time.

Thanks for contributing @ketansaresa! 😊 It was wonderful working with you! 😄 If you want to stay updated on LinksHub, join our Discord community! :) We'd love to have you! :)


feat: support qwik

Description
Add support for qwik.
Related to #224

I don't know how to inspect the compilation process. I tried the vite-plugin-inspect but it didn't work for qwik.  And I couldn't find another plugin to inspect the compilation process yet.

It seems that React.createElement was generated by esbuild. When I set esbuild: { jsx: 'preserve'} in vite.config.ts, vite told me:
Unexpected token (Note that you need plugins to import files that are not JavaScript)

Does this mean that the unplugin-icons process code after qwikVite?
But I don't understand why this is useful in DEV mode


[Bug] AutoScheduler not work when kernel contains conv2d+gradient and has a large size

Hello:
I am trying to generate source code of an operator which involves conv2d_nchw and gradient. I tried to achieve this with AutoScheduler, but the search period will fail when the operator has a relative large shape, and when the task.apply_best(log_file)  was run, it will crash and print Cannot find any valid schedule for ["conv2d_layer"] in file conv2d.json.
To be exact, the program works well when the layer looks like:
@auto_scheduler.register_workload
def conv2d_layer():
    x = tvm.te.placeholder((1,64,7,7), name='x')
    w1 = tvm.te.placeholder((64,64,3,3), name='w1')
    z1=topi.nn.conv2d_nchw(x,w1,(1,1),(1,1),dilation=1,out_dtype="float32")
    [dw1] = tvm.te.gradient(z1, [w1])
    return [x,w1,dw1]

but will fail when it looks like:
@auto_scheduler.register_workload
def conv2d_layer():
    x = tvm.te.placeholder((1,128,7,7), name='x')
    w1 = tvm.te.placeholder((128,128,3,3), name='w1')
    z1=topi.nn.conv2d_nchw(x,w1,(1,1),(1,1),dilation=1,out_dtype="float32")
    [dw1] = tvm.te.gradient(z1, [w1])
    return [x,w1,dw1]

the whole program looks like this.
import os

import numpy as np
import tvm
from tvm import te, auto_scheduler, topi
from tvm.topi.testing import conv2d_nchw_python

@auto_scheduler.register_workload
def conv2d_layer():
    x = tvm.te.placeholder((1,128,7,7), name='x')
    w1 = tvm.te.placeholder((128,128,3,3), name='w1')
    z1=topi.nn.conv2d_nchw(x,w1,(1,1),(1,1),dilation=1,out_dtype="float32")
    [dw1] = tvm.te.gradient(z1, [w1])
    return [x,w1,dw1]

target = tvm.target.Target("cuda")

task = auto_scheduler.SearchTask(
    func=conv2d_layer, args=(), target=target
)

print("Computational DAG:")
print(task.compute_dag)

log_file = "conv2d.json"
measure_ctx = auto_scheduler.LocalRPCMeasureContext(min_repeat_ms=300)
tune_option = auto_scheduler.TuningOptions(
    num_measure_trials=1000,  # change this to 1000 to achieve the best performance
    runner=measure_ctx.runner,
    measure_callbacks=[auto_scheduler.RecordToFile(log_file)],
    verbose=2,
)
task.tune(tune_option)

sch, args = task.apply_best(log_file)

del measure_ctx

print("Lowered TIR:")
print(tvm.lower(sch, args, simple_mode=True))

print("CUDA source code:")
print(task.print_best(log_file, print_mode="cuda"))

the last part of debug message is below:
==================================================
No: 1000        GFLOPS: 0.00 / 0.00     results: MeasureResult(error_type:RuntimeDeviceError, error_msg:Traceback (most recent call last):
  File "/root/tvm/python/tvm/auto_scheduler/measure.py", line 1120, in _rpc_run
    random_fill(empty_array)
  File "/root/tvm/python/tvm/_ffi/_ctypes/packed_func.
...
----------
An error occurred during the execution of TVM.
For more information, please see: https://tvm.apache.org/docs/errors.html
---------------------------------------------------------------
  Check failed: (code == RPCCode::kReturn) is false: code=1
, all_cost:13.82, Tstamp:1631297770.85)
==================================================

..........

[18:16:10] /root/tvm/src/auto_scheduler/measure.cc:299: Warning: Too many errors happened during tuning. Switching to debug mode.

Time elapsed for measurement: 426.86 s
----------------------------------------------------------------------
------------------------------  [ Done ]
----------------------------------------------------------------------
No valid state found in this search round. Check if it has traversed all of the search space.
/root/anaconda3/lib/python3.6/site-packages/xgboost/training.py:17: UserWarning: Old style callback is deprecated.  See: https://xgboost.readthedocs.io/en/latest/python/callbacks.html
  warnings.warn(f'Old style callback is deprecated.  See: {link}', UserWarning)
MeasureInput with old format workload key ["conv2d_layer"] should be updated using the script from https://github.com/apache/tvm/pull/7317.
MeasureInput with old format workload key ["dense_layer"] should be updated using the script from https://github.com/apache/tvm/pull/7317.

Traceback (most recent call last):
  File "conv_layer_tuning_grad.py", line 38, in <module>
    sch, args = task.apply_best(log_file)
  File "/root/tvm/python/tvm/auto_scheduler/search_task.py", line 522, in apply_best
    "Cannot find any valid schedule for %s in file %s" % (self.workload_key, log_file)
RuntimeError: Cannot find any valid schedule for ["conv2d_layer"] in file conv2d.json

The commit id of my tvm build is:
commit f1ca91d4e401096d04e962c982d62b1f2669c9f5
Date:   Wed Aug 25 18:25:29 2021 -0700

I am wandering that what is wrong with it. Can anyone help me?

You need to specify head when calculating the gradient:
x = tvm.te.placeholder((1,128,7,7), name='x')
w1 = tvm.te.placeholder((128,128,3,3), name='w1')
z1=topi.nn.conv2d_nchw(x,w1,(1,1),(1,1),dilation=1,out_dtype="float32")
dy = tvm.te.placeholder(z1.shape, name="dy")
[dw1] = tvm.te.gradient(z1, [w1], head=dy)

According to https://github.com/apache/tvm/blob/main/python/tvm/te/autodiff.py#L33, if head is None, then the identity tensor of shape output.shape + output.shape will be used. It means the size of dy will be super large and unlikely to fit into the GPU memory.

That really works! I do met a few more bugs like timeout at runtime but they are quiet easy to fix. Thanks a lot!

Weird. When I use auto_scheduler to search conv2d on V100, it shows "No valid state found in this search round. Check if it has traversed all of the search space." and lots of messages like
No: 3	GFLOPS: 0.00 / 0.00	results: MeasureResult(error_type:RuntimeDeviceError, error_msg:Traceback (most recent call last):
  File "/home/test/tvm/python/tvm/auto_scheduler/measure.py", line 1124, in _rpc_run
    random_fill(empty_array)
  File "/home/test/tvm/python/tvm/_ffi/_ctypes/packed_func.py", line 237, in __call__
    rai
...
----------
An error occurred during the execution of TVM.

My workload is as follows.
@auto_scheduler.register_workload
def my_workload():
    A = tvm.te.placeholder((128, 64, 224, 224), name='input0')
    W = tvm.te.placeholder((64, 64, 3, 3), name='input1')
    C = topi.nn.conv2d(A, W, (1,1), (1,1), (1,1), layout='NCHW', out_dtype=A.dtype) # (128, 64, 224, 224)
        return [A, W, C]

After setting MAX_TRACEBACK_INFO_LEN=8192, I can get more details:
Get 64 programs to measure:
........................................................
*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E*E==================================================
No: 1	GFLOPS: 0.00 / 0.00	results: MeasureResult(error_type:RuntimeDeviceError, error_msg:Traceback (most recent call last):
  File "/home/test/tvm/python/tvm/auto_scheduler/measure.py", line 1124, in _rpc_run
    random_fill(empty_array)
  File "/home/test/tvm/python/tvm/_ffi/_ctypes/packed_func.py", line 237, in __call__
    raise get_last_ffi_error()
tvm._ffi.base.TVMError: Traceback (most recent call last):
  4: TVMFuncCall
  3: _ZNSt17_Function_handlerIFvN3tvm7runtime7TVMArgsEPNS1_11
  2: tvm::runtime::RPCWrappedFunc::operator()(tvm::runtime::TVMArgs, tvm::runtime::TVMRetValue*) const
  1: tvm::runtime::RPCClientSession::CallFunc(void*, TVMValue const*, int const*, int, std::function<void (tvm::runtime::TVMArgs)> const&)
  0: tvm::runtime::RPCEndpoint::CallFunc(void*, TVMValue const*, int const*, int, std::function<void (tvm::runtime::TVMArgs)>)
  File "/home/test/tvm/src/runtime/rpc/rpc_endpoint.cc", line 801
TVMError:
---------------------------------------------------------------
An error occurred during the execution of TVM.
For more information, please see: https://tvm.apache.org/docs/errors.html
---------------------------------------------------------------
  Check failed: (code == RPCCode::kReturn) is false: code=kShutdown

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/test/tvm/python/tvm/auto_scheduler/measure.py", line 1185, in _rpc_run_worker
    res = _rpc_run(*args)
  File "/home/test/tvm/python/tvm/auto_scheduler/measure.py", line 1143, in _rpc_run
    dev.free_raw_stream(stream)
  File "/home/test/tvm/python/tvm/_ffi/runtime_ctypes.py", line 456, in free_raw_stream
    check_call(_LIB.TVMStreamFree(self.device_type, self.device_id, stream))
  File "/home/test/tvm/python/tvm/_ffi/base.py", line 348, in check_call
    raise get_last_ffi_error()
tvm._ffi.base.TVMError: Traceback (most recent call last):
  42: 0xffffffffffffffff
  41: _start
  40: __libc_start_main
  39: main
  38: Py_Main
  37: 0x000000000063886a
  36: PyObject_Call
  35: 0x000000000058945c
  34: 0x0000000000507cd3
  33: _PyEval_EvalFrameDefault
  32: 0x000000000050a3fc
  31: 0x00000000005099ff
  30: 0x0000000000507cd3
  29: _PyEval_EvalFrameDefault
  28: 0x000000000050a22e
  27: 0x0000000000516284
  26: 0x0000000000507cd3
  25: _PyEval_EvalFrameDefault
  24: 0x000000000050a3fc
  23: 0x00000000005099ff
  22: 0x0000000000507cd3
  21: _PyEval_EvalFrameDefault
  20: PyObject_Call
  19: 0x00000000005893d9
  18: 0x0000000000507cd3
  17: _PyEval_EvalFrameDefault
  16: PyObject_Call
  15: 0x000000000058931a
  14: 0x0000000000507cd3
  13: _PyEval_EvalFrameDefault
  12: 0x000000000050a3fc
  11: 0x00000000005096c7
  10: _PyEval_EvalFrameDefault
  9: 0x000000000050a532
  8: _PyObject_FastCallKeywords
  7: 0x00007f2b9b1f5c12
  6: _ctypes_callproc
  5: ffi_call
  4: ffi_call_unix64
  3: TVMStreamFree
  2: tvm::runtime::RPCDeviceAPI::FreeStream(DLDevice, void*)
  1: non-virtual thunk to tvm::runtime::RPCClientSession::FreeStream(DLDevice, void*)
  0: std::_Function_handler<void (tvm::runtime::TVMArgs, tvm::runtime::TVMRetValue*), tvm::runtime::RPCEndpoint::Init()::{lambda(tvm::runtime::TVMArgs, tvm::runtime::TVMRetValue*)#2}>::_M_invoke(std::_Any_data const&, tvm::runtime::TVMArgs&&, tvm::runtime::TVMRetValue*&&)
  File "/home/test/tvm/src/runtime/rpc/rpc_endpoint.cc", line 681
TVMError:
---------------------------------------------------------------
An error occurred during the execution of TVM.
For more information, please see: https://tvm.apache.org/docs/errors.html
---------------------------------------------------------------
  Check failed: (code == RPCCode::kReturn) is false: code=1
, all_cost:11.04, Tstamp:1636202883.20)
==================================================
Placeholder: input0, input1
blockIdx.x nn.0@ff.0<EMAIL_ADDRESS>(0,16384)
  vthread nn.1@ff.1<EMAIL_ADDRESS>(0,4)
    threadIdx.x nn.2@ff.2<EMAIL_ADDRESS>(0,64)
      for rc.0 (0,64)
        for rx.0 (0,3)
          threadIdx.x ax0@ax1@ax2<EMAIL_ADDRESS>(0,64)
            input1.shared = ...
          for ax0@ax1@ax2<EMAIL_ADDRESS>(0,126)
            threadIdx.x ax0@ax1@ax2<EMAIL_ADDRESS>(0,64)
              pad_temp.shared = ...
          for ry.1 (0,3)
            for nn_c.3 (0,2)
              for yy_c.4 (0,7)
                for xx_c.4 (0,7)
                  compute.local = ...
      for nn.3 (0,2)
        for yy.3 (0,7)
          for xx.3 (0,7)
            compute = ...

==================================================

hi, @comaniac . Is it due to the large problem size of conv2d?  I think it's a common setting of CNN model.

It's not due to the large problem size. Did you set USE_RANDOM ON when building TVM?
For future discussions of this issue, please post to the discuss forum.

It's not due to the large problem size. Did you set USE_RANDOM ON when building TVM?
For future discussions of this issue, please post to the discuss forum.

Sure. --> https://discuss.tvm.apache.org/t/autoscheduler-failed-to-find-a-valid-schedule/11001


Offline indicator is missing on Welcome to Expensify page

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.31-1
Reproducible in staging?: y
Reproducible in production?: n
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Navigate to staging.new.expensify.com
Create new account
Welcome to Expensify page will appear
Go offline

Expected Result:
Offline indicator should appear on Welcome to Expensify page when user goes offline
Actual Result:
Offline indicator is missing on Welcome to Expensify page
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/7c16d958-aaa0-44c3-8353-3b9319ee0227
View all open jobs on GitHub

This is new feature from #32154
cc: @stitesExpensify

Yeah, I don't think that this needs to be a blocker. If the user goes offline and chooses an option then everything will work just fine, and then they will see the offline indicator on the main page.

Hey @dannymcclain!
I want to understand if we really want to add the offline indicator here, since in phone screens this is actually a modal

Just like our actions modal, which doesn't show an offline indicator

I want to understand if we really want to add the offline indicator here

I would say no. I agree with you that since this is a modal and not an actual screen/view, it probably should not include the offline indicator. cc @Expensify/design in case anyone disagrees.

Agree with you!
On a separate note.. is it just me or does the illustration here feel massive? cc @stitesExpensify

It does feel quite large.

I am going to close this one then, but should we do something about the image?

@stitesExpensify thoughts on following up and cleaning up the image styles?

This was the original mock:

We changed the illustration to the animation, but to avoid cutoff it needs to be scaled up. Perhaps we can match it closer to the height of the original mock?

Yeah, I'm thinking we should ask Augenblick to get us a version of that animation where the hands are way longer so we can make a more wide illustration as opposed to something more square in proportion.

 Basically, will we get rid of this particular page once the Wave 9 stuff is in place?

AFAIK this will be replaced when wave9 comes out


fix: overloaded requests

The changes in https://github.com/sse-secure-systems/connaisseur/pull/1274 tried to fix the memory leak problem by directly using an event loop. This unfortunately caused performance issues, so that the load tests regularly failed. Using nest-asyncio fixed the performance issue.
Checklist

[x] PR is rebased to/aimed at branch develop
[x] PR follows Contributing Guide

Codecov Report
Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (3b20e00) 96.42% compared to head (42dff17) 96.27%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1280      +/-   ##
===========================================
- Coverage    96.42%   96.27%   -0.15%
===========================================
  Files           23       23
  Lines         1315     1317       +2
===========================================
  Hits          1268     1268
- Misses          47       49       +2

Files
Coverage Δ

connaisseur/__main__.py
0.00% <0.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Fix wasmJs target on kotlin 2.1.0

I've just added sweet-spi to my project and it works perfect except wasmJs targets fails with
IllegalArgumentException: Service `ServiceName` wasn't registered.

I'd also consider returning empty list of services instead of throwing exception in case no service available. It seems a legit use case and more intuitive response to me.


markdown for 'onlyblocks' and 'onlyjs'

I need to write documentation specific to blocks or specific to javascript. I need to have markdown to allow only blocks to be shown, or only JS to be shown.

Just use no name for js
```
```

or just for blocks
```block
```

Updated pxt docs.


Removing comments

Removing uneeded TODO comments.
Signed-off-by: Diogo Monica<EMAIL_ADDRESS>

LGTM


Working Razor component drag events example

Could you provide working example of using drag events i.e. ondragstart, ondragover and ondrop? Example should show how to put data into DataTransfer and DataTransferItem .NET objects and how data is received in ondrop event.
The problem was discussed here and postponed until .NET 5 was released.

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

ID: aa724f22-3a49-c7f6-1500-ac2d8dde5894
Version Independent ID: 5e8ef502-ccf9-659d-f25a-0f8f3c75f6e0
Content: ASP.NET Core Blazor event handling
Content Source: aspnetcore/blazor/components/event-handling.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

Hello @Jarek300 ... We might be able to add such an example, but it will take a long time to reach this issue due to many high priority issues in the queue over the next several months. In the meantime if that cross-linked issue doesn't help, reach out to the community for assistance. We recommend the usual community support forums ...

Stack Overflow (tag: blazor)
ASP.NET Core Slack Team
Blazor Gitter

I'm afraid that you don't understand.
Working example is not needed to better understand how to use 'DataTransfer and DataTransferItem that hold dragged item data.'
Working example is needed because there is no information at all  about DataTransfer and DataTransferItem classes.
DataTransfer and DataTransferItem are part of Blazor from the beginning (almost two years) and in that time nobody in the community  has figured out how to use them. Honestly, I suspect that DataTransfer and DataTransferItem classes do not work at all and never did.

It looks like they're going to address at least some drag+drop functionality on https://github.com/dotnet/aspnetcore/issues/18754. You can provide feedback on the work that they're doing on that issue. WRT having JS interop examples, I think we'll have to see what they come up with for 6.0. There's no point in showing JS interop approaches that their new built-in features will address with C#. In theory, we'll have the new features, whatever they are, documented by 6.0's release.

Leave this issue open ... I'll use this issue to track the work for 6.0.

I'm going to move this to P3 ("Priority 3") and mark it for the PU ("product unit") ...

Eventually, they'll schedule work at https://github.com/dotnet/aspnetcore/issues/18754. If they say tho that they plan to build something into the framework for .NET 7, that's too far out to maintain an issue for here. We'll place a new issue mid-2022.
When they have a chance to run down the doc issues either in December or 22Q1, they might provide a sample/example for this sooner than providing built-in framework feature. In that case, we'll use this issue to track the work. Stand-by a bit longer because they can't be pinged about this right now. They're all 🏃😅 on the mop-up of the 6.0 release.

We're going to go with Option 1 ...

Eventually, they'll schedule work at https://github.com/dotnet/aspnetcore/issues/18754. If they say tho that they plan to build something into the framework for .NET 7, that's too far out to maintain an issue here. We'll place a new issue mid-2022.

I usually generate doc issues for upcoming framework releases in Q2 each year from the new features tracking issue that will appear on the PU repo. We'll open a new issue for this when that tracking issue appears for .NET 7. As I mentioned earlier, everyone is welcome to provide feedback to the PU on their open issue for the new feature.

well they mentioned that they will NOT implement this feature source


Spike for Create separate process to display/capture seed words.

Spike for https://github.com/cosmos/voyager/issues/1359.

I would table this until we go web. What do you think?

Feasibility was proven


pairs from integer array whose sum is equal to given number

You have given an integer array and a number, you need to write a program to find all elements in the array whose sum is equal to the given number. Remember, the array may contain both positive and negative numbers, so your solution should consider that too
accept input from user and also the target sum from user
output= the pairs which give sum equal to the target sum

I would like to work on this issue...can you plz assign this issue to me???

I would like to work on this issue
Sannidhi Nandgaonkar
Sy comp

I am Arti Tambade, TY IT
I would like to work on this issue, Could you assign this issue to me?

Hi there!
I would Like to work on this issue
Can you please assign it to me?

Hi there!
I would like to work on this issue!
Can you please assign it to me?

I am from FY

Hi there! I would like to work on this issue! Can you please assign it to me?

Hi there! Can you please assign me this issue?

Hi there! I am not aware which issue I have been assigned which is yet to be submitted because all the issues which were assigned to me were completed and merged as well. Can you please let me know the issue no?


If a chart creates a .tools folder in a mounted volume, delete it

Currently, some applications may create a subfolder in the '.tools' folder in mounted volumes. When the application has been deleted, this folder is no longer needed, so we should have some way of removing it. One option is to use a delete-hook that mounts the volume and deletes the subfolder.

its added to all the required charts


infer taxonomies in models resulting from queries

When a query returns a model, there should be a little inference run on the model to have additional triples. For example if a triple has
bdr:W123 :hasGenre bdr:T234 .
bdr:T234 :taxSubclassOf bdr:T567 .

the new model should also contain
bdr:W123 :hasGenre bdr:T567 .

The same mechanism should be used for the LanguageTaxonomy (in the ontology).
(no need to do that now)

I'm not sure this is the right place to do that, or even that it's really a good idea, closing


Fix narrow page width for read the docs

Issue: I have scroll the page left and right several times to read the content of the tables

Ref: https://github.com/readthedocs/sphinx_rtd_theme/issues/295#issuecomment-455226058

Thank you for fixing it 👍

Thank you for fixing it 👍


Baking crosstalk check fix

Follow up of #1598, fix the crosstalk shaper space check making sure we use the optimised processor to allow typical collapse of matrices. There is another issue related to the use of NamedTransform as shaper space which appear to not work currently because we are using LookTransform which doesn't support these.

Tagging as draft while I investigate the issue with NamedTransform.


Reduce the number of points in Canvas for this lab

The two parts of this lab are collectively worth nearly 100 points, which is a bit out of proportion to the value of the other labs. The Segmented File Server is only worth 24, i.e., a quarter of this, and it really should be more like 50-75 points compared to this.

Actually, I don't think that the problem is the number of points here, but is more the number of points in other labs. In general the labs are worth between 30 and 50 points "per week", which seems fairly plausible. This is a bit heavy, but it's a lot of work. I think I should up the points on some of the other labs (like Segmented File Server) rather than reduce the points here.

I've gone through all the rubrics and tried to rationalize things, so I'm going to close this.


optimistic-pruning-copy: Avoid copying segments when no segment is pruned

We expect that most of the time segments are valid and their schemas updated. Therefore ValidSegmentPruner and DataSchemaSegmentPruner should not be pruning segments most of the time. But these are the only SegmentPruners that use the default SegmentPruner.prune(list, context), which always copy all the segments in order to have a simpler implementation.
This means that most of the time these two segments are duplicating the array of segments twice for no reason. Usually this shouldn't be a problem, but in cases where Pinot has thousand of segments, the copy cost can be noticeable.
This PR makes the default SegmentPruner implementation a little bit more complex by making the optimistic assumption that sometimes the pruner will in fact not prune. To do that it has two loops:

The first one iterates until the first pruned segment is found. If no segment is pruned, the segment list received as parameter is returned and therefore no copy is made. In case at least one segment is pruned, then a new array list is created, all previously not pruned segments are added and a new loop is executed.
The second loop starts from the index of the first segment that wasn't tested and every time a segment is not pruned it is added to the new list, which is finally returned as result.

I've also include a some tests to verify the new implementation.

I think there's a better approach than this, given the nature of the segment pruners which exist:

pruner
applicability
purpose
pruning likelihood
complexity
can be disabled?
empirical cost

valid
always
don't query empty segments
rare
one liner
no
negligible

dataschema
always
don't query segments which don't have the right columns (enforces debatable semantics)
only in the (real time) window between schema migration and segment refresh
one liner
yes
medium

columnvalue
queries with filters
acts a bit like a sparse index
depends on selectivity of filter
high
yes
high

selectionquery
select queries without a filter
applies the row limit to simple select queries
high
medium
yes
usually terminates early

The first two pruners generally lead to unnecessary list creation because they seldom prune segments, but they could just be inlined into the second two pruners which would eliminate two list copies automatically. Following on from that, SegmentPrunerService could inspect the query context to check if there is a filter or if there is a limit in a select query, and only invoke the right pruners for the query. This would result in only ever allocating one new list since the last two pruners are mutually exclusive, which is the best this change could achieve.


Feature request: a way to add custom snippets

I can't see on the docs anywhere that show me how to create custom snippets!

I can't see on the docs anywhere that show me how to create custom snippets!

You can't yet. They are all defined here : https://github.com/alm-tools/alm/blob/master/src/app/codemirror/addons/autocomplete/snippets.ts#L22

If these snippets are general purpose I can add them for you? :rose:

Hi @basarat No, there are not general purpose, currently Im working with nativescript and angular 2 frameworks so I want a way to save time typing code since most of the code are boilerplate. But I think we need a way to add custom snippets for doing this kind of things :smile:


Ray Tracing feature not added for compatible games

the title kinda says everything,
since i would love to filter on RayTracing,
it doesn't seem to work even if the PCGamingWiki link shows it supports raytracing

Looks like ray tracing in the articles was renamed from Ray tracing to Ray tracing (RT)

Fixed in 1.2.3


IPFS HELP

Everytime I start up IPFS Desktop I see this message  "Could not connect to the IPFS API" and steps on installing IPFS daemon. I NEED HELP

Hi @jimmynsfw! I'm sorry you are not being able to use IPFS Desktop as you'd hope to. Could you please answer the following questions:

What version of IPFS Desktop are you using? Have you used it before?
Did you already have an IPFS repository?
Can you show us a screenshot of the error?