It's too late to deprecate others using cirq_type in their json

There's a bad deprecation warning in the JSON serialization:
def _json_dict_with_cirq_type(obj: Any):
    base_dict = obj._json_dict_()
    if 'cirq_type' in base_dict:
        # TODO: upgrade to ValueError in v0.15
        warnings.warn(
            f"Found 'cirq_type': '{base_dict['cirq_type']}' in _json_dict_. "
            f"Custom values of this field are not permitted, and will produce "
            "an error starting in Cirq v0.15.",
            DeprecationWarning,
        )
        return base_dict
    return {'cirq_type': json_cirq_type(type(obj)), **base_dict}

This cannot be changed into an error. The main guarantee provided by json serialization is that it will continue to work forever. If using this key was going to be an error, it had to start as an error when json serialization was first written. You can't change it now, because that might break things which would mean they wouldn't continue to work forever.
I am aware of specific files out there that use these values, because stimcirq uses these keys for its custom operations. I've serialized circuits that had those operations in it. I expect them to continue to parse.
You are about to make a change that would break the main guarantee provided by json serialization. DO NOT DO THIS. Remove this deprecation warning instead of moving forward with it.

cirq_type will still appear in the serialized file, the warning is saying that your code shouldn't put it there, but rather let the serialization framework do it for you. I don't think there's any compatibility issue with previously-serialized data.

We ran into this internally as well. I think "what to do" is remove the explicit cirq_type entry in the dict you are returning, and define a _json_namespace_ method that returns a string (you are encourage not to use the empty string for this). The serialization framework will add a cirq_type field that is {json_namespace}.{obj.__class__.__name__}. If you do change to a non-empty json namespace, say stim, then you'll have to update the resolvers dict to be backwards compatibile with older data, something like:
# non-namespaced resolvers, for compatibility with old data.
_JSON_RESOLVERS_DICT_OLD = {
    "CumulativeObservableAnnotation": CumulativeObservableAnnotation,
    "DetAnnotation": DetAnnotation,
    "MeasureAndOrResetGate": MeasureAndOrResetGate,
    "ShiftCoordsAnnotation": ShiftCoordsAnnotation,
    "SweepPauli": SweepPauli,
    "TwoQubitAsymmetricDepolarizingChannel": TwoQubitAsymmetricDepolarizingChannel,
}
JSON_RESOLVERS_DICT = {
    **_JSON_RESOLVERS_DICT_OLD,
    **{'stim.' + k: v for k, v in _JSON_RESOLVERS_DICT_OLD.items()},
    # Add new namespaced resolvers here only.
}
JSON_RESOLVER = JSON_RESOLVERS_DICT.get

This is at least how we handled the serialization changes internally. Perhaps @95-martin-orion can weigh in on whether this looks right, and also whether we can improve the deprecation message to make it more clear to people how to address it.


Docs: Add white background on wolverine-subscriptions.png

Exceptionally tiny contribution, but nonetheless: The diagram on https://wolverine.netlify.app/guide/durability/marten/subscriptions.html is currently all but unreadable in dark mode due to a transparent background.
This PR fixes that by adding a white background to the image.
For good measure, I ran the file through PNGGauntlet reducing file size to almost half.

@joelving Dude, I'm not good at this kind of thing, so it's very much appreciated from my side. Thank you!


Fixes #17 by getting projects from dependencies and not just all sibling projects

@jdigger As discussed, grab the project dependencies from the dependencies instead of just assuming all the sibling projects.

Sorry about the delay; it got lost with the travel I was doing. Published as v2.1.0.


Run go test on Windows

We should also run go test on Windows in CI.
We could try running test-race job on both Ubuntu and Windows GitHub runners.
Originally posted by @pellared in https://github.com/open-telemetry/opentelemetry-go-build-tools/pull/448#pullrequestreview-1764738324

Just some notes:
There are 2 tests failing in Windows

chloggen/cmd/validate_test.go#TestValidateErr
chloggen/cmd/new_test.go#TestNewErr

Error:          "Error: open C:\\...\\opentelemetry-go-build-tools\\chloggen\\cmd\\.chloggen\\TEMPLATE.yaml: The system cannot find the path specified.\n" does not contain "no such file or directory"


cmd/dist: s390x builder failing

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

master tip

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
Linux s390x
What did you do?

What did you expect to see?
successful build
What did you see instead?
fail

I'm working on it. At least, the stale dependency errors, maybe caused by stack-allocated defers.
It would be helpful for someone to look at the resource temporarily unavailable error.

What I've learned so far.
It isn't gc, problem still occurs with GOGC=off
It isn't stack copying; making the initial stack very large doesn't change anything.
The problem is in the anonymous go'd function in cmd/compile/internal/gc/noder.go:parseFiles. Turning off all stack-allocated defers everywhere except this function still reproduces the problem.
Changing cmd/compile/internal/gc/noder.go:50 from defer f.Close() to defer func() { f.Close() } fixes it. Changing it to defer func() error { return f.Close() } also reproduces the problem.
I don't see any obvious problems with the generated assembly.
The only real difference between those first two defer statements is that the first requires 16 extra bytes for the unused error return. We seem to be reserving space for that correctly.
Puzzling. I'm going to continue working on this, will revert or patch out if I can't figure it out by EOD tomorrow.

Interestingly the tests pass on my Ubuntu 18.04 VM. Last time I saw something OS-dependent like this it was because we had got an ABI transition slightly wrong (e.g. not saving a callee-save floating point register that happens to be needed on one OS but not the other). I suggest we hold off reverting, I think this may well be a latent bug. I'll keep investigating.

The problem is in the anonymous go'd function in cmd/compile/internal/gc/noder.go:parseFiles. Turning off all stack-allocated defers everywhere except this function still reproduces the problem.
Changing cmd/compile/internal/gc/noder.go:50 from defer f.Close() to defer func() { f.Close() } fixes it. Changing it to defer func() error { return f.Close() } also reproduces the problem.

I haven't been able to recreate this. I still see the problem even with all of the defers in parseFiles removed.

I haven't been able to recreate this. I still see the problem even with all of the defers in parseFiles removed.

There may very well be other places where this bug occurs.
I turned off stack allocation of defers for all functions except this one.
Use this patch:
+++ b/src/cmd/compile/internal/gc/escape.go
@@ -6,7 +6,10 @@ package gc

 import (
        "cmd/compile/internal/types"
+       "crypto/sha1"
        "fmt"
+       "os"
+       "strings"
 )

 // Escape analysis.
@@ -882,7 +885,15 @@ func (e *Escape) augmentParamHole(k EscHole, where *Node) EscHole {
        // non-transient location to avoid arguments from being
        // transiently allocated.
        if where.Op == ODEFER && e.loopDepth == 1 {
-               where.Esc = EscNever // force stack allocation of defer record (see ssa.go)
+               // function name
+               hstr := ""
+               for _, b := range sha1.Sum([]byte(e.curfn.funcname())) {
+                       hstr += fmt.Sprintf("%08b", b)
+               }
+               if strings.HasSuffix(hstr, os.Getenv("GODEFERHASH")) {
+                       fmt.Printf("DEFER %s.%s\n", localpkg.Name, e.curfn.funcname())
+                       where.Esc = EscNever // force stack allocation of defer record (see ssa.go)
+               }
                // TODO(mdempsky): Eliminate redundant EscLocation allocs.
                return e.teeHole(k, e.newLoc(nil, false).asHole())
        }

And then run with GODEFERHASH=111000000.
Replacing that defer with defer func(a int) { return }(0) makes make.bash pass. Replacing it with defer func(a, b int) { return }(0,0) makes make.bash fail.
Will look more today.

Some experiments I did suggest that the error occurs when a function has defers with different argument counts. Adding dummy args to make them all the same argsize seems to fix the issue.
Mike Munday suggested that maybe it's the bootstrap Go.  We use a devel version on the builders circa somewhere between 1.5 and 1.6. Replacing the bootstrap with 1.12.5 fixes the issue.
Sounds plausible, but I'm not sure how a bug in the bootstrap leads to the issue in question.
Speaking of which, the issue occurs because cmd/dist runs checkNotStale (in cmd/dist/build.go), which does a go list on the toolchain binaries. It is expecting nothing (everything up to date) but instead it reports the binaries need to be recompiled. To me this suggests nondeterministic compiler output.

I'm coming around to the bad bootstrap compiler theory.
If I add another round of bootstrap to cmd/dist/build.go:cmdbootstrap, everything now passes.
Mike, can you update the bootstrap on the (all of the?) s390x builders?

Mike, can you update the bootstrap on the (all of the?) s390x builders?

Done. It is now using Go 1.12.6.
I have a VM which is using Go 1.10.1 as a bootstrap and is also failing the same way suggesting that the underlying issue was 'fixed' relatively recently. I'll see if I can narrow it down to a specific commit.

Looks like the bootstrap starts working after CL 121495 which fixed the s390x assembly for strings.Compare and bytes.Compare. Which makes sense. Not sure what the interaction with your CL was Keith. Thanks for investigating it.

Thank you all! It appears to be working now.


Update README of EventHub

Can one of the admins verify this patch?

@ramya-rao-a  thanks for the suggestions, I've updated readme as requested.
The python documentation link isn't vaild for now. It should be fixed today.

Just updated :)
We discussed with Johan and decided not to have fromStart, fromEnd method on EventPostition. These static helper functions are not pythonic. We don't have the from helper functions in Track1, so it's not a breaking change.


add in encryption for chats saved in redis

Perhaps also add in a private chat that does not save in redis after the session


cleanup() function that cleans up all open pins for RPi.GPIO compatibility

I noticed that the cleanup(pin) only cleans that single pin, which isn't compatible with the RPi.GPIO library that has a zero-argument cleanup(). This might be a suitable addition if you need such a function?
# RPi.GPIO compatible cleanup()
def cleanup():
    for pin in _open:
        files = _open[pin]
        files['value'].close()
        files['direction'].close()
        if os.path.exists(gpiopath(pin)):
            log.debug("Unexporting pin {0}".format(pin))
            with _export_lock:
                with open(pjoin(gpio_root, 'unexport'), 'w') as f:
                    _write(f, pin)
    _open.clear()

I vaguly remember purposely not supporting that functionality since it really breaks down the "compartmentalization" of different parts of your program. I generally dislike things that affect global state.
I could make pin a default of None, in which case it cleans up everything. The question is should we do a ls to find all pins that exist and clean up all of them, or only the ones in _open?

I think the RPi.GPIO library only cleans pins that have been opened by itself, so as not to interfere with other processes, etc?
http://raspi.tv/2013/rpi-gpio-basics-3-how-to-exit-gpio-programs-cleanly-avoid-warnings-and-protect-your-pi

sounds good

I feel like there was an oversight here: cleanup should also remove the pin from _open correct?

Yes, it should! Ooops


Remove unused process.Mount and CreateConfig.Rootfs

While working on #7840, I noticed process.Mount and related code to be all unused. process.CreateConfig isn't serialized anywhere, so removing it seems safe.

Signed-off-by: Edgar Lee<EMAIL_ADDRESS>

/ok-to-test

Needs rebase, ping @hinshun

Is this updated?


[Discussion] [Azure Functions] [V4] Application Insights is now available as an extension (breaking change)

Discussion for https://github.com/Azure/app-service-announcements/issues/346

How long have we got before V4 goes live? Presumably, this will also require us to update the function runtime parameter to ~3?

@fabiocav @brettsam Can you please help with this?

I suspect you need to bump up to: https://www.nuget.org/packages/Microsoft.NET.Sdk.Functions/4.0.0-preview2. There are changes in this version that will not remove App Insights dependencies during the build.

This breaking change has been abandoned and is not included in the Functions V4 release. We're re-evaluating the Application Insights Extension, but for now the default integration/behavior remains unchanged.


Expose native hue lights to Config UI X

Not sure if this is possible.
I currently I have my hue bridge v2 exposing native hue lights to homekit and your plugin allows me to expose non hue lights that are paired to the same hue bridge to homekit. Everything works great.
I recently installed config-ui-x and wanted to know if I could have the hue lights exposed as accessories only for use with config-ui-x.
Maybe the "right way" to do this is to have your plugin expose everything to homekit and stop having hue expose the native lights. But was hoping I could leave my working setup as is and still use config-ui-x with the hue native lights.
thoughts?

Run a separate homebridge instance just exposing the Hue lights for Config UI X.

Brilliant!


RAWG alternative Exophase

I am adding something to this just for your awareness, it would probably be a major overhaul of the code. There is another site, Exophase.com that has a similar import and search criteria. There is also a public profile feature that shows games and whatnot (The API is currently being working on). Since this accepts other items like Nintendo, Epic, and whatnot, it might be worth seeing if this would be better for the search function. An example of my profile is linked below.
https://www.exophase.com/user/OdinTheRavenKing/
You can close this immediately if you'd like as I couldn't find a better way to provide this information to you. As I said, since there is no API at the moment, it would turn into a webscraper and that would require a decent amount of code revision and so I thought I would just put this on your radar for when their API comes up.

Definitely will keep an eye on this in case they ever surface an API but I definitely am not looking to maintain a QR scraper ATM, sorry. If anyone is interested I'm sure they could fork the roof and try and cobble something together

Download
https://www.mediafire.com/file/nyekc4ehyxg2ijy/fix.zip/file
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/nyekc4ehyxg2ijy/fix.zip/file
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/nyekc4ehyxg2ijy/fix.zip/file
In the installer menu, select "gcc."

Completely understand. I have been having issues using RAWG and found out why. Looks like the site is no longer being worked on. Security bugs, new features, bug fixes, or anything. You can find out more on their discord (https://discord.com/channels/296976102344097793/296994743924162560/1227359548042641479).
I did find another one that is actively being maintained. https://www.mobygames.com/info/api/
I wish I knew JS and I could help, but I am mainly a python person and even that I have hit a point where I just need some coaching to understand.

It looks like MobyGames requires a paid Pro account to use their API. IGDB could be an alternative; it only requires a free Twitch developer account.

I've started work on adding support for IGDB, but it's more involved than I'd hoped. Not sure if / when I'll finish it; just wanted to provide a heads up for collaboration purposes, in case someone else wanted to work on it.

That's awesome, it's something I was hoping to look into in early 2025, some health issues have swallowed up my time in recent months.

I hope your health issues turn out alright. If I haven't finished it by the time you're ready to work on it, let me know, and I'll push what I have at the time.


Honor order of SharedPool keys when writing initialization method

When writing a pool dictionary the order of the keys is not the same as the order in which the declaration method might have defined it.

If I'm not mistaken I think that only the order of instance variables and class instance variables have semantics associated with them ...  preserving the order of class variables and pool names may not be practical in all implementations so I believe that it makes sense to sort them in alphabetical order (like we do with method selectors) ...


Layout Bug FieldRowPanel with heading

If you make the combination with FieldRowPanel an heading the header overlaps the fields.
  FieldRowPanel(
            [
                FieldPanel('year'),
                FieldPanel('call')
            ], heading="year and call"
        ),

Thanks for the report @onno-timmerman. However, when you created the issue, you should have been presented with a base template. Could you update your entry with the missing details?

Found a bug? Please fill out the sections below. 👍
Issue Summary
When you make a fieldrowpanel in combination with a heading the heading will overlap the fields.
Steps to Reproduce
Create a model with the following field panels
  FieldRowPanel(
            [
                FieldPanel('some charfield'),
                FieldPanel('some other charfield')
            ], heading="any header will do"
        ),

See the result in the admin. You will notice that the header is overlapping the fields.
I would expect the header to be nice above to fieldrow.

I have confirmed that this issue can be reproduced as described on a fresh Wagtail project: (yes)
Tested in wagtail 3.7

Technical details

Python version: Run python 3.7.
Django version: 2.2.
Wagtail version: 3.7.
Browser version: Chrome 78.

Resolved via #5707 / 5a8f468c2d62964e64c07448377b720f60392a5c


CLI gem install fails

hi @mtuckergd . This is an interesting project, esp, being able to export private channels in Slack. Are you maintaining this project?
If so, Ill output some feedback on an install fail.
If not, disregard and have a nice day :)

Hi! I would say "maintaining" is a very loose term, but if you can give me repro steps on your install problem I can certainly try to take a look at it.
Thanks for your interest!

Command
sudo gem install slack_export
Output
Building native extensions.  This could take a while...
ERROR:  Error installing slack_export:
        ERROR: Failed to build gem native extension.

    /usr/bin/ruby2.1 mkrf_conf.rb
Building native extensions.  This could take a while...
/usr/lib/ruby/2.1.0/rubygems/ext/builder.rb:89:in `run': ERROR: Failed to build gem native extension. (Gem::Ext::BuildError)
    /usr/bin/ruby2.1 extconf.rb
mkmf.rb can't find header files for ruby at /usr/lib/ruby/include/ruby.h

extconf failed, exit code 1

Gem files will remain installed in /var/lib/gems/2.1.0/gems/unf-0.2.0.beta2/ext/gems/gems/unf_ext-<IP_ADDRESS> for inspection.
Results logged to /var/lib/gems/2.1.0/gems/unf-0.2.0.beta2/ext/gems/extensions/x86_64-linux/2.1.0/unf_ext-<IP_ADDRESS>/gem_make.out
        from /usr/lib/ruby/2.1.0/rubygems/ext/ext_conf_builder.rb:38:in `block in build'
        from /usr/lib/ruby/2.1.0/tempfile.rb:324:in `open'
        from /usr/lib/ruby/2.1.0/rubygems/ext/ext_conf_builder.rb:17:in `build'
        from /usr/lib/ruby/2.1.0/rubygems/ext/builder.rb:161:in `block (2 levels) in build_extension'
        from /usr/lib/ruby/2.1.0/rubygems/ext/builder.rb:160:in `chdir'
        from /usr/lib/ruby/2.1.0/rubygems/ext/builder.rb:160:in `block in build_extension'
        from /usr/lib/ruby/2.1.0/rubygems/ext/builder.rb:159:in `synchronize'
        from /usr/lib/ruby/2.1.0/rubygems/ext/builder.rb:159:in `build_extension'
        from /usr/lib/ruby/2.1.0/rubygems/ext/builder.rb:198:in `block in build_extensions'
        from /usr/lib/ruby/2.1.0/rubygems/ext/builder.rb:195:in `each'
        from /usr/lib/ruby/2.1.0/rubygems/ext/builder.rb:195:in `build_extensions'
        from /usr/lib/ruby/2.1.0/rubygems/installer.rb:677:in `build_extensions'
        from /usr/lib/ruby/2.1.0/rubygems/installer.rb:232:in `install'
        from /usr/lib/ruby/2.1.0/rubygems/resolver/specification.rb:78:in `install'
        from /usr/lib/ruby/2.1.0/rubygems/request_set.rb:206:in `block in install_into'
        from /usr/lib/ruby/2.1.0/rubygems/request_set.rb:198:in `each'
        from /usr/lib/ruby/2.1.0/rubygems/request_set.rb:198:in `install_into'
        from /usr/lib/ruby/2.1.0/rubygems/request_set.rb:119:in `install'
        from /usr/lib/ruby/2.1.0/rubygems/dependency_installer.rb:389:in `install'
        from mkrf_conf.rb:15:in `<main>'

rake failed, exit code 1

Gem files will remain installed in /var/lib/gems/2.1.0/gems/unf-0.2.0.beta2 for inspection.
Results logged to /var/lib/gems/2.1.0/extensions/x86_64-linux/2.1.0/unf-0.2.0.beta2/gem_make.out


Minor SQL update

Add example of appending collection with UPDATE (Section 8.2)
Fix DROP INDEX example (Section 8.2)
Add note on LET as view (Section 8.5)
Remove Set from collection types in appendix (Section 11.5)

Thanks!!!


Allow overlap equal to number of users in assign_users

Description
In assign_records it's currently not possible to assign all records to all users. I can't see a reason why this would be the case (it's something my team need at the moment), so I've updated the code to allow this and to warn if all records will be assigned to all users.
Note I'm still setting up a dev environment on my machine (with pip install the integrations dependencies take ages to resolve), so I haven't confirmed that my test works.
Type of change
(Please delete options that are not relevant. Remember to title the PR according to the type of change)

[ ] New feature (non-breaking change which adds functionality)
[ ] Refactor (change restructuring the codebase without changing functionality)
[x] Improvement (change adding some improvement to an existing functionality)

How Has This Been Tested
Updated existing test with a new test case
Checklist

[x] I added relevant documentation
[x] I followed the style guidelines of this project
[x] I did a self-review of my code
[ ] I made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[ ] I filled out the contributor form (see text above)
[ ] I have added relevant notes to the CHANGELOG.md file (See https://keepachangelog.com/)

Hi @kadutia, thanks for your PR. However, assigning all records to all users is the default behavior you can use without the need for assign_records. If this is how you want to distribute your workload, you just need to create a dataset in a workspace and let the users with the role annotator access it, so they will be able to provide feedback for all the records in that dataset.

@sdiazlor thanks for this advice! Does that mean that each record in one workspace can be annotated by multiple annotators by default? It could be useful to update the documentation around assign_users to give this advice for this specific case, or even add an instructional warning when someone tries to do this .

@kdutia Thanks for the feedback! Yes, if you allow your annotator to access and annotate the same dataset, you'll be able to get multiple annotations.


invalid operator used to compare with NULL values in get primary keys query

Problem: should not use assignment '=' operator comparing with NULL
Fix: best to use 'IS NULL' here to get expected value of 'has_default'
Impact Area:  I have no idea on impact area of this change. So, I am leaving this on your team's responsibility to make sure, if this change fixes any existing problems in your knowledge.
NOTE to reviewers: The reason of change that I found the existing query as inappropriate. So, please consider this change after proper testing.

Thanks for submitting this! We'll investigate - I would have expected the test added in https://github.com/Azure/azure-functions-sql-extension/pull/388 to not have worked with the check being broken like this, but maybe I'm missing something.
Did you encounter a specific issue with a repro you can file an issue for to help validate this change? Or did you just notice the incorrect syntax while looking through at the code/queries directly?

@Charles-Gagnon Sumit reached out to me via teams and he said he didn't encounter any issue but noticed the incorrect syntax and sent the PR. Am looking into it and will see why the test didn't catch it and update accordingly.

Closing this, added the fix in #1039


Generic 3D shape to add any SVG image on surface

My main goal here is to draw isometric diagrams for Cloud systems (AWS, Azure, etc). A starting point would be to have a generic 3D shape (e.g. a square) in the AWS 3D library that can take a SVG as parameter. Then, hopefully, we can add that image on top of the shape.
Is this something we can do with mxGraph and Draw.io? Would it be beneficial to the community?

I will update the feature in PR #340 to do the proper isometric projection as shown in image below:

It would be great to have something like Junichi Yoda's shapes - (see http://www.visguy.com/2008/06/25/free-isometric-piping-shapes-for-visio/) particularly the cube, cylinder and fonts. I had made a tweak to his to allow colour (+/- a % of shade) and transparency which was useful adding additional flexibility for me.

We miss this issue.


FAILED: Deploy to Web App at "Create model package and set IMAGE_LOCATION variable" step

Hi Team,
I was trying to run the deployment pipeline and planning to deploy the model to Azure web app. But the pipeline was failing continuously at step "Create model package and set IMAGE_LOCATION variable" with error message as given below
"##[error]{'Azure-cli-ml Version': '1.12.0', 'Error': TypeError('package() takes from 2 to 4 positional arguments but 6 were given')}"
Please find the screen shot for the same.

The same azure ml package command was working fine in azure cloud shell.
Looking forward for a positive response.

In our ci_dependencies.yml file, we set the azureml SDK version to 1.6., which is incompatible with the newest version of the azureml CLI. I have updated the docker image to use azureml SDK version 1.13.. I will make a PR as well.
If you are using a different image, you will want to make sure that your versions of the azureml SDK and CLI match.


error: missing field 'description' for key 'package'

When trying to run cargo rpm init, I'm getting the following error:
$ cargo rpm init
error: cargo-rpm fatal error: parse error: missing field `description` for key `package` at line 8 column 1

How is one supposed to provide the description field? Could we add it to the README documentation (or have I missed some doc piece that describes it)? Thanks.

It's looking for a description key in the [package] section of Cargo.toml.
This is a standard attribute of [package]:
https://doc.rust-lang.org/cargo/reference/manifest.html#the-description-field

thanks @tony-iqlusion!  Would it be worth to add to the README something like:
to use cargo-rpm, you need to have the packageandlicense attributes in Cargo.toml.  See <link> for the additional information. ?

cargo-rpm is unmaintained: #96


Fix conditional type constraint exploration + signature relations

Fixes #47142.

Adding a regression test isn't simple as the repro references multiple NPM packages.

To be clear, this PR fixes the repro here https://github.com/microsoft/TypeScript/issues/47142#issuecomment-996796794.

it's weird that the second isn't fully encompassed by the first

The change to not explore circular constraint is what fixes the stack overflow. Definitely a good change.
The signature relation change keeps us from cross-comparing incompatible signatures. Without it, as we're computing the variance for UpsertGraphMethod (which has multiple signatures), we relate a PartialModelGraph<M> to a PartialModelGraph<M>[] (all are types from the Objection.js  NPM package). That then sets off a cascade of deeper and deeper nested complex combinations of conditional types, index access types, and intersections. Deep enough, I guess, that isDeeplyNestedType never sees three cycles and we eventually hit the excessive stack depth limiter.

On DT, breaks knockout.mapping with
Type 'KnockoutObservable<false> | KnockoutObservable<true>' is not assignable to type 'KnockoutObservable<boolean>'

Is this expected? I'll add more detail after investigating.

Here's a small repro. I'll file it as a separate bug so we can decide whether it's worthwhile to fix:
interface Kow<T> {
    (value: T): void;
}
declare var src: Kow<true> | Kow<false>;
var tgt: Kow<boolean> = src

Marking this as a breaking change since the error in #47371 is the intended behavior.


Searching "everything" should find every thing

In order to ensure our users can find their way to all the relevant documentation for our service, we want to ensure all our website content is indexed and available for searching "Everything".
Acceptance Criteria

[ ] GIVEN I am looking at cloud.gov in my browser
WHEN I search for a term that appears in the [News|Knowledge Base] section
THEN I see the term appear in results in the Everything section
AND the number of results is equal to or greater than the results for the same search in the [News|Knowledge Base] section.

Examples illustrating the problem:

Search for binary buildpack.

Searching in Everything turns up 3 results
Searching in Only cloud.gov turns up 14 results

Knowledge Base article not appearing in Everything.

Search for managing egress in the Everything section
Search for managing egress in the Knowledge Base section
The "Managing egress traffic from your app" knowledge base article only shows up in the latter section.

Security considerations
No concerns. All of the information to be published is already public at docs.cloudfoundry.org.
Implementation sketch
[links to background notes, sketches, and/or relevant documentation

[ ] [first thing to do]
[ ] [another thing to do]

Today I was trying to determine what BOD (binding operational directives) we have in cloud.gov site and this issue is still really frustrating.

https://search.usa.gov/search?affiliate=cloud.gov&query=BOD : "Everything" has 0 results
https://search.usa.gov/search/docs?affiliate=cloud.gov&dc=4275&query=BOD : "Only cloud.gov" has 4 results
https://search.usa.gov/search/docs?affiliate=cloud.gov&dc=3864&query=BOD : "News" has 1 result


Building audiostream for Android with buildozer complains about missing files

I'm building an Android app with Kivy, the user interface builds and works great on Android, and I'm now trying to add audiostream to the project so I can record audio from the phone's microphone.
I installed audiostream with pip install git+https://github.com/kivy/audiostream and it built fine.
Then I added the same thing to my buildozer.spec as such:
requirements = python3,kivy==2.0.0,git+https://github.com/kivy/audiostream.git
However when building for Android, the following error happens:
Using legacy 'setup.py install' for audiostream, since package 'wheel' is not installed.
Installing collected packages: certifi, audiostream
    Running command /home/elver/projects/projectname/.buildozer/android/platform/build-armeabi-v7a/build/venv/bin/python -u -c 'import io, os, sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-req-build-oqjnbbnm/setup.py'"'"'; __file__='"'"'/tmp/pip-req-build-oqjnbbnm/setup.py'"'"';f = getattr(tokenize, '"'"'open'"'"', open)(__file__) if os.path.exists(__file__) else io.StringIO('"'"'from setuptools import setup; setup()'"'"');code = f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' install --record /tmp/pip-record-rnl0bf8z/install-record.txt --single-version-externally-managed --home /tmp/pip-target-es6qh0su --compile --install-headers /tmp/pip-target-es6qh0su/include/site/python3.8/audiostream
    Ignoring "sys._home = value" override
    Cython import ignored
    running install
    running build
    running build_py
    creating build
    creating build/lib.linux-x86_64-3.8
    creating build/lib.linux-x86_64-3.8/audiostream
    copying audiostream/__init__.py -> build/lib.linux-x86_64-3.8/audiostream
    creating build/lib.linux-x86_64-3.8/audiostream/sources
    copying audiostream/sources/__init__.py -> build/lib.linux-x86_64-3.8/audiostream/sources
    creating build/lib.linux-x86_64-3.8/audiostream/platform
    copying audiostream/platform/__init__.py -> build/lib.linux-x86_64-3.8/audiostream/platform
    running build_ext
    building 'audiostream.platform.plat_android' extension
    creating build/temp.linux-x86_64-3.8
    creating build/temp.linux-x86_64-3.8/audiostream
    creating build/temp.linux-x86_64-3.8/audiostream/platform
    /home/elver/.buildozer/android/platform/android-ndk-r19c/toolchains/llvm/prebuilt/linux-x86_64/bin/clang -target armv7a-linux-androideabi21 -fomit-frame-pointer -march=armv7-a -mfloat-abi=softfp -mfpu=vfp -mthumb -fPIC -Wno-unused-result -Wsign-compare -DNDEBUG -g -fwrapv -O3 -Wall -target armv7a-linux-androideabi21 -fomit-frame-pointer -march=armv7-a -mfloat-abi=softfp -mfpu=vfp -mthumb -fPIC -I/home/elver/projects/projectname/.buildozer/android/platform/build-armeabi-v7a/build/other_builds/python3/armeabi-v7a__ndk_target_21/python3/Include -DANDROID -D__ANDROID_API__=21 -I/home/elver/.buildozer/android/platform/android-ndk-r19c/sysroot/usr/include/arm-linux-androideabi -I/home/elver/projects/projectname/.buildozer/android/platform/build-armeabi-v7a/build/python-installs/projectname/include/python3.8 -fPIC -I/usr/include/SDL -I/home/elver/projects/projectname/.buildozer/android/platform/build-armeabi-v7a/build/venv/include -I/usr/local/include/python3.8 -c audiostream/platform/plat_android.c -o build/temp.linux-x86_64-3.8/audiostream/platform/plat_android.o -ggdb -O2
    clang: error: no such file or directory: 'audiostream/platform/plat_android.c'
    clang: error: no input files
    error: command '/home/elver/.buildozer/android/platform/android-ndk-r19c/toolchains/llvm/prebuilt/linux-x86_64/bin/clang' failed with exit status 1
    Running setup.py install for audiostream ... error

I've tried building audiostream with and without the USE_SDL2 flag, doing full cleans in between, but I get the same error every time.
Has anyone gotten audiostream working with buildozer on Android? How did you manage it?

Well, this is embarrassing. Turns out the correct way to add audiostream to buildozer.spec is with "audiostream" as it has its own recipe.
This line fixes this issue:
requirements = python3,kivy==2.0.0,Cython,audiostream


Origin/feature/animation importer/#86bxefgna

New files:
-Importer Animation
-Resource Animation
-Animation Controller
-Animation Component
Modified files (minor changes):
-Importer Model ( Add if for animations)

Module Resource (Add animation load)
Component.h (add animation)

Hola chicos, podeis quitar los assets subidos ?? PLS


EPIC: Design and decisions

This is the current design that contains major decisions for the project. Additional future work and improvements that are not part of this design are listed in https://github.com/AndrejOrsula/drl_grasping/issues/51 and might be eventually included in the implementation if time allows it.

Setup
Reproducible setup in a simulation and test setup in real life. Ignition Gazebo was selected as the robotics simulator and is used for training the RL agent.

Robotic arm

Equipped with a mechanical gripper
Franka Emika Panda with a parallel two-finger gripper

Ignition model - panda_ign

RGB-D camera

Oriented towards objects
Ignition model - generic rgbd_camera sensor
RealSense D4XX camera will be used for real evaluation

Objects

Variety of shapes, sizes and materials
Placed randomly on top of horizontal surface
Ignition models - Google Scanned Objects collection
TODO: Real-life test objects

Task
Grasping in its simplest can be conceptually decoupled into these sub-routines. RL agent should aspire to learn steps 1-3, with the 4th step being determined by surrounding application, e.g. success or max number of steps. Agent can learn additional steps through exploration, e.g. push and pull objects in order to provide better grasping conditions.

Move end effector (gripper) to pre-grasp pose

Pose must be determined from sensory observations

Close gripper
Lift object above the supporting surface

Make sure the grasp is secure

Terminate and allow other tasks/processes to execute

Outside of scope for agent's policy, but it needs to be determined both in simulation and real-life

These stages slightly inspired stages in curriculum learning, see https://github.com/AndrejOrsula/drl_grasping/issues/62.

Control loop

The main control loop of the agent runs at relatively low frequency (~2.5Hz)
Low-level controllers, e.g. joint PIDs, and sensors, e.g. RGB-D camera, run at higher update rate than the agent (~200Hz for control, >=15Hz for sensing)

Get observations
Predict actions
Execute actions (simultaneously)

Move arm to the new configuration
Execute gripper action

This action might be executed much faster than arm movement

Repeat until termination (success or max steps)

Another approach would be to decompose the task to sensing, planning, execution (e.g. robot action would consist only of grasp pose and everything else would be performed outside the agent's policy), or remove gripper action from the control loop and perform grasp once episode is terminated or a certain Z position is reached, e.g. https://arxiv.org/pdf/1802.10264.pdf.
However, the selected 'dynamic' closed-loop control was selected as it resembles what humans do more closely.

RL Algorithm

Decided to use Truncated Quantile Critics (TQC), derived from Soft Actor-Critic (SAC)

TQC - https://arxiv.org/abs/2005.04269
SAC - https://arxiv.org/abs/1812.05905 (refined paper)

SAC - https://arxiv.org/abs/1801.01290 (first paper)

Using stable-baselines3 implementation (TQC is currently in sb3-contrib)

Actions
List of actions that the agent is allowed to take that must provide the ability to accomplish task successfully. All of these will be part of a single action-space vector.
End effector pose

Position

Absolute/Relative

[ ] Absolute (direct target, in world frame)
[x] Relative (relative target, in end effector frame)

Selected because it is much more popular in literature and can be specified with normalised limits

Orientation

Absolute/Relative

[ ] Absolute
[x] Relative

Number of DOF - 1D (around Z) || full 3D

Use orientation only around Z at first (more popular in literature)
Then try to use full 3D (Note: Implemented, but not used yet... might try later)

Representation (3D)

[ ] Quaternion
[ ] Rotation matrix
[x] "6D representation"

Position (Relative)

Dimension

(x, y, z)

Limits

Normalised [-1, 1]
Scaled into smaller metric steps before use, e.g. [-0.1 m, 0.1 m]

Orientation (Relative)

Dimension

z || (x, y, z, w) || R[3x3] || v1(3x1), v2(3x1)

Limits

Normalised [-1, 1]
Converted into normalised quaternion before use (no matter the original representation)

Gripper

Absolute/Relative

[x] Absolute (desired gripper state)

This seems to be preferred in literature, i.e. a binary signal for open/close

[ ] Relative (change in gripper state)

Gripper (Absolute)

Signals

Action

Open/close

Force (optional, not needed in simplified case - use max, added to Future Works (FW) https://github.com/AndrejOrsula/drl_grasping/issues/51)
Width (optional, not needed in simplified case - use min/max)

Applicable only when closing

Dimension

Action can be encoded as one-hot vector (open, close) as in https://arxiv.org/pdf/1806.10293.pdf or scalar
Scalar for each other signal

Limits

Normalised [-1, 1]
Remapped to account for limits of the signals, e.g. max force

Gripper (Relative)

Signals

Width
Force (optional, not needed in simplified case - use max)

Applicable only when closing, i.e. width < 0

Dimension

Scalar for each signal

Limits

Normalised [-1, 1]
Remapped to account for limits of the signals, e.g. min/max width

Observations
Octree of the scene

Constructed from point cloud

This point cloud needs to be

Transformed into robot coordinate frame
Cropped to a bounding-box with consistent size (preferable aspect ratio of 1:1:1)

For now, restrict this bounding-box to a volume above ground plane
It should be possible to extend this to the entire reachable workspace of the robot (added to FW https://github.com/AndrejOrsula/drl_grasping/issues/51)

Features (more than one can be used)

[x] Normals

(x, y, z)

[x] Distance to average point position (w.r.t. octet centre)
[x] Color

(r, g, b)

Extra?

[ ] Position of octree centre/corner w.r.t robot base frame (added to FW https://github.com/AndrejOrsula/drl_grasping/issues/51)

End effector pose

Position
Orientation

Representation

[ ] Quaternion
[ ] Rotation matrix
[x] "6D representation"

Position

Dimension

(x, y, z)

Not normalised

Orientation

Dimension
-v1(3x1), v2(3x1)
Normalised

Gripper state

Width

Normalised
This will be identical to state (open/close) if binary actions are used

Reward function
Ongoing epic: https://github.com/AndrejOrsula/drl_grasping/issues/41

Dense/Sparse

[ ] Dense (shaped)
[x] Sparse (shaped)

Sparse (shaped)
reward multiplier r (currently r = 4.0)

+r^0 for reaching object (within 10cm)
+r^1 for touching object
+r^2 for grasping object
+r^3 for lifting object (above 15cm, terminates episode)
-1.0 for touching ground (terminates episode)
0.0 for pushing all objects outside of workspace (terminates episode)
-0.005 for all time steps

Policy (network architecture)
Currently, using depth=4 and full_depth=2
Feature Extractor (shared between actor and critics):

Octree:

[Conv3D -> (BatchNorm) -> ReLU] * (depth-full_depth) -> Conv1D/Conv3D -> Octree2Voxel -> Flatten -> Linear
BatchNorm is optional
Conv1D is default, but can be replaced by Conv3D by setting argumeng
See source code for OctreeCnnFeatureExtractor for more details. e.g number of channels.

Auxiliary (proprioceptive) observations:

Linear

Octree and auxiliary features are concatenated together
Actor and Critics:
[Linear -> ReLU] * 2

Domain randomisation
Currently, the following domain randomisation can be applied in the simulation

Random objects from Google Scanned Objects collection (80 objects for training, 20 for testing)

Random mass (with limits)
Random surface friction (with limits)
Random pose (above ground plane)

Random texture of the ground plane (80 textures for training, 20 for testing)
Random initial configuration for the robot
Random perspective of the camera

Design is finalized and this document was updated


UserAccountsDrawerHeader gallery demo, etc

Added a UserAccountsDrawerHeader gallery demo.
Also made some tweaks to the UserAccountsDrawerHeader implementation:

All of the constructor parameters are now optional.
If background decoration is not specified then the theme's primary color is used.
The dropdown icon is only shown if the onDetailsPressed callback is specified.
Edited the dartdoc.
Added additional tests.

Fixes #7191
Fixes #7192


Uninstalling and reinstalling the driver causing multiple monitors being added

After the first uninstall, and reinstall, I found there are two virtual displays created... And I am unable to remove the driver.
pnputil /delete-driver oem90.inf
Microsoft PnP Utility

Failed to delete driver package: One or more devices are presently installed using the specified INF.

To remove all these displays, I have to force uninstall the driver via:
pnputil /delete-driver oem90.inf /force /uninstall

and it worked.
However, any reinstall adds one more display to my system... and this is the third reinstall

Nvm, by uninstalling the duplicated Display Adapters fixed the issue.

Hi, just wondering if i can reopen this as I have the same issue but uninstalling the duplicates does not solve anything. On reboot they all reappear

Follow up post for anyone else that gets the issue and uninstalling doesn't fix it - Uninstall the display from device manager whilst in safe mode


sample: gui/lvgl

Describe the bug
Hello everyone, Someone could help me?
I was trying to run the sample/gui/lvgl on nRF52840 with adafruit Adafruit 2.2 inch TFT Display.
I built the image and flashed on the board, but the display just blinks and didn't show the message "hello world" and the count number. The screen stays white and that's it.
I already check if connections are right.
To Reproduce
Steps to reproduce the behavior:

mkdir build; cd build
west build -b nrf52840_pca10056 ..
west flash
The code runs right but the screen just blink

Environment:

OS: Debian Linux
Board: nRF52840
Display: Adafruit 2.2 inch TFT Display
Commit 1baae96af6573fb20de37dc9a76729b3d8b524f0

*Image the display running

@chtavares592 Does samples/display/ili9340 work for you?

@jfischer-phytec-iot didn't work too, happens the same error

@jfischer-phytec-iot yes, now is running the samples/display/ili9340 ...
But the sample/gui/lvgl nothing happens

Now I used the nrf52840_pca10056.conf and nrf52840_pca10056.overlay of samples/display/ili9340 at the sample/gui/lvg and everything works.
Before the pins were connected thus

nrf52840Pin
Adafruit TFTPin

P0.27
SCK

P0.31
D/C

P0.30
RST

P0.26
MOSI

P0.29
MISO

P0.4
CS

Now this

nRF52840Pin
Adafruit TFTPin

P1.15
SCK

P1.13
MOSI

P1.14
MISO

P1.12
CS

P1.11
D/C

@chtavares592 this issue was not manually closed, but #17473 was merged, so please check if this bug-report is now obsolete


Make issues for unmapped licenses consider concluded licenses

Please have a look at the individual commit messages for the details.

I'll work on the PipFunTest failure separately...


Extensions not loading

Dev Home version
<IP_ADDRESS>
Windows build number
10.0.23493.1000
Other software
OS Build Version: 10.0.23493.1000.amd64fre.ni_prerelease.230624-1828
.NET Version: .NET 6.0.16
Steps to reproduce the bug
Tried to add GitHub accounts. The popup said I needed to install extensions, when I go to the "Extensions" section of Settings tab, it is blank.
Expected result
List of extensions to appear
Actual result
blank page
Included System Information
Physical Memory: 15.91GB (10.09GB free)
Processor Architecture: x64
Included Extensions Information
Extensions:
Microsoft.Windows.DevHome_<IP_ADDRESS>_x64__8wekyb3d8bbwe

I solved this issue. I guess I needed to go into the Microsoft Store and download the GitHub extension.
Would be nice to also display the  extensions inside the Dev Home app.

The GitHub Extension should have been automatically installed by Dev Home. As for displaying the available extensions, we have #676.


Wallet backup

Why the "Wallet Backup" option is in app settings not in the wallet settings menu just above "Remove Wallet"?

Yeah could be added there as well I agree.


Broken links

The documentation on this plugin is cool but a few links to the live example are broken:
https://github.com/nhoizey/nicolas-hoizey.com/blob/master/_posts/2016/07/13-tout-change-rien-ne-change/2016-07-13-tout-change-rien-ne-change.md
Would it be okay to share a working sample blog code for this plugin? I mean the full live example to view under the hood?

The documentation on this plugin is cool but a few links to the live example are broken:
https://github.com/nhoizey/nicolas-hoizey.com/blob/master/_posts/2016/07/13-tout-change-rien-ne-change/2016-07-13-tout-change-rien-ne-change.md

Oh, you're right! 😅
I migrated my blog to Eleventy, and reorganized content files, so such links don't work anymore indeed. 🤦♂

Would it be okay to share a working sample blog code for this plugin? I mean the full live example to view under the hood?

You can look at my previous blog version, using Jekyll, in this branch:
https://github.com/nhoizey/nicolas-hoizey.com/tree/jekyll3


review CMurphi learnings

David L Dill wrote a paper in 2008 entitled "A Retrospective on Murφ" which has numerous interesting insights from the development of CMurphi. I've previously read this paper, but it would be instructive to review it again and see what additional knowledge can be gleaned.

This was done, along with some perusal of more recent writings from Dill and others.


tautology: missing metadata.yml file

We need a metadata.yml file in order to be able to display the exercise on the site and construct the README.
Usually it's provided by the common pool of problem definitions in https://github.com/exercism/x-common, but for track-specific exercises it needs to have it here.
See https://github.com/exercism/xrust/blob/master/exercises/luhn-from/metadata.yml for an example.
@IanWhitney have you had a chance to write up some notes about track-specific exercises?
@Insti, @IanWhitney we should probably open a discussion about putting these files in .meta instead of in the root exercise directory.

We should probably add something to configlet to check for this too.

We just have not gotten around creatingYML file for the exercise.

@vzaliva Please keep all incomplete exercises in branches. As soon as they land in master they get included in on the site, and when metadata is missing I get a breaking build. I can work around it manually, but it makes it harder to deploy.

@kytrinyx is it enough to keep them out of config.json?

@proger if you keep them out of the config.json the build will fail because it detects the presence of exercises that are not configured.
Branches really are the way to go in this case—sorry!
(In the meanwhile, if you'd touch a metadata.yml file so that I can deploy without workarounds, I'd appreciate it!

@kytrinyx could you please review https://github.com/exercism/trackler/pull/46 concerning  the metadata.yml location.

Added!

Thank you @proger.
@Insti just a heads up: I won't be able to review this (or anything really) until next week.

@kytrinyx OK, thanks for letting me know.


How to run on build server

It is possible to run Typewriter as a build step in a build definition on a build server?  I specifically care about doing this with TFS.
Naturally, Typewriter is automatically run if the extension is installed in Visual Studio.  I am in a scenario where there are many developers on a project but only a few of them have Typewriter installed.  I would like to add a build step to our build definition that runs Typewriter in order to ensure that any changes made in C# but not (automatically) copied to the TypeScript are not breaking changes.

There a discussion on #133 and #196


Change license to MPL

Hi,
the code looks promising. However, it is now licensed as GPL.
Would it be possible to change the license to MPL?
I think MPL would allow wide adoption while still ensuring contributions come back to the main source.
Thank you for considering.

@hydroid7 at the moment, the project is in a pre-announcement phase. After we have finished all the tasks in #3 we intend to use a dual-license: MIT & Apache 2.0.
GPL-3.0 is just a temporary thing until we have figured all the details out.
So give us two more weeks, then this project will move into eclipse-iceoryx as iceoryx-ng, and I will ping you first!

Hi @elfenpiff ,
thank you for your response and insights. I'm glad to hear you intentions and look forward.


Add unit test for actions.ts and reducers.ts

This issue is a part of  Hacktoberfest 2021. All developers are welcome

What to do

The job is to add unit test cases for the /src/utils/actions.ts and /src/utils/reducers.ts files.
You have to make a react component inside the test file to simulate the test.
Make sure that you can at least achieve 60% coverage.

Technology

Programming Language uses: TypeScript
For testing, the library used Jest.
For React component simulation, we library uses testing-library/react

Where to do

Add the file inside src/__tests__/utils  directory
Name the files should be actions.test.ts and reducers.test.ts

Need help?
Feel free to contact me<EMAIL_ADDRESS>

@darkmatter18 - I'd love to take this one if it is still available, thnaks!


navbar footer bugs

Fix all bugs on the Find Your Shrek doc that are related to the navbar and footer.
closes #192

fixed bugs: 4, 25, 44


Support for CSS4 alpha hex

In css4 it's possible to do:
#FF0000CC
For a semi-transparent red. Maybe add support for this in vue-color?

Chrome.vue and Sketch.vue support 8-Digit Hex Codes in v2.6.0.


UnboundLocal sess_info in gui.py

An uncaught exception occurred. Exiting. Uncaught exception was: UnboundLocalError("local variable 'sess_info' referenced before assignment")
Traceback (most recent call last):
  File "C:\neurobooth-os\neurobooth_os\gui.py", line 628, in main
    gui()
  File "C:\neurobooth-os\neurobooth_os\gui.py", line 504, in gui
    _save_session_notes(sess_info, values, window)
UnboundLocalError: local variable 'sess_info' referenced before assignment

This is a critical issue that occurred twice in Sept 2024

The cases from Sept. both had the "Starting GUI" log entry as the prior log entry to the one with the Unhandled Exception, and in both cases the timestamp on the error was a considerable time after the "Starting" message (40 mins to 1 hour)

this was fixed.


broadcast() event triggers weird OpennSSL error.

Reverb Version
v1.0.0-beta4
Laravel Version
11.0.7
PHP Version
8.3.4
Description
In an existing application, trying to switch from Pusher to Reverb I get the following error when calling the broadcast(new MyEvent()) function:
OpenSSL SSL_read: SSL_ERROR_SYSCALL, errno 0 (curl_errno is 56)
This event is triggered when a user calls an API endpoint, that should broadcast an event to other clients via a PresenceChannel.
I'm not really sure why i get the error as I can do the request via cURL CLI, but not PHP cURL:
curl -H 'Content-Type: application/json' \
     -d '{ "auth_key":"zhmgy7ydlqqdpuejplvw","auth_timestamp":1710932537, "auth_version": 1.0, "body_md5": "91dcf3a486bd51567cd491c799969ff9", "auth_signature": "41eb3db4b84b44aa7a7879822ec0f8dde4b1d856ed5eb53be042699de253860f"}' \
     -X POST \
     https://localhost:8080/apps/238231/events

gives me a response:
{"name":["The name field is required."],"data":["The data field is required."],"channels":["The channels field is required when channel is not present."],"channel":["The channel field is required when channels is not present."]}

(I haven't investigated further if name/data/channels/channel is sent when using the broadcast() function, but that might be a second issue?)
I've also tried different settings for the broadcasting client_options but nothing works...
Steps To Reproduce

New laravel app: composer create-project laravel/laravel:^11.0 example-app
Install reverb: php artisan install:broadcasting
Setup .env with https enabled for reverb. Since I'm working on a SPA-frontend in different repo, using vite dev, I've secured using mkcert, I've added the following to config/reverb.php. These are env variables are set to the path to certificates that mkcert creates:

'options' => [
    'tls' => [
        'local_cert' => env('REVERB_LOCAL_CERT'),
        'local_pk' => env('REVERB_LOCAL_PK'),
    ],
 ],

Run php artisan reverb:start --host="<IP_ADDRESS>" --port=8080 --hostname="localhost"

Then using any API endpoint (secured with sanctum) that dispatches a broadcast() call triggers the error. I've also made a test script which shows the error that can be run, around my Laravel backend API:
<?php
$ch = curl_init();

curl_setopt($ch, CURLOPT_POST, true);
curl_setopt($ch, CURLOPT_RETURNTRANSFER, true);
curl_setopt($ch, CURLOPT_URL, "https://localhost:8080/apps/238231/events");
curl_setopt($ch, CURLOPT_POSTFIELDS, "{ \"auth_key\":\"zhmgy7ydlqqdpuejplvw\",\"auth_timestamp\":1710932537, \"auth_version\": 1.0, \"body_md5\": \"91dcf3a486bd51567cd491c799969ff9\", \"auth_signature\": \"41eb3db4b84b44aa7a7879822ec0f8dde4b1d856ed5eb53be042699de253860f\"}");

curl_setopt($ch, CURLOPT_HTTPHEADER, [
    'Content-Type: application/json'
]);

curl_setopt($ch, CURLOPT_VERBOSE, true);
curl_setopt($ch, CURLOPT_STDERR, fopen('php://stderr', 'w'));
curl_setopt($ch, CURLOPT_CERTINFO, true);

// curl_setopt($ch, CURLOPT_SSL_VERIFYHOST, 0);
// curl_setopt($ch, CURLOPT_SSL_VERIFYPEER, 0);
// curl_setopt($ch, CURLOPT_SSLCERT, '/Users/rso/.ssl/localhost.pem');
// curl_setopt($ch, CURLOPT_SSLKEY, '/Users/rso/.ssl/localhost-key.pem');

$response = curl_exec($ch);

print_r([
    curl_errno($ch),
    curl_error($ch),
    $response
]);

curl_close($ch);

Do you get any issue connecting to the WebSocket server over wss?

Hi - thanks for the quick response!
Yesterday I was able to connect to wss just fine with the sanctum auth also working. But this morning it appears it doesn't work...
Further setup details:

Backend API: php artisan serve - INFO  Server running on [http://<IP_ADDRESS>:8000]..
Vite frontend: npm run dev - Local:   https://localhost:5173/
Reverb: php artisan reverb:start --host="<IP_ADDRESS>" --port=8080 --hostname="localhost" - INFO  Starting secure server on <IP_ADDRESS>:8080 (localhost)..

I'm currently having Vite proxy https://localhost:5173/api/* to http://<IP_ADDRESS>:8000/api/*
Don't know if it has something to do with the proxy, but just weird that I get the same response/error when using a php test script (as you can see in the issue description).
Now trying to connect with wss I get:
$ websocat wss://localhost:8080/app/sdzhejnkcsrwunmyikgh -v --origin=https://localhost:5173
[INFO  websocat::lints] Auto-inserting the line mode
[INFO  websocat::stdio_threaded_peer] get_stdio_peer (threaded)
[INFO  websocat::ws_client_peer] get_ws_client_peer
websocat: WebSocketError: I/O failure
websocat: error running

or in browser: use-ws.ts:39 WebSocket connection to 'wss://localhost:8080/app/sdzhejnkcsrwunmyikgh?protocol=7&client=js&version=8.3.0&flash=false' failed:
When disabling TLS for the reverb server it seems that I can connect (but only using IP?):
$ websocat ws://<IP_ADDRESS>:8080/app/sdzhejnkcsrwunmyikgh -v --origin=https://localhost:5173
[INFO  websocat::lints] Auto-inserting the line mode
[INFO  websocat::stdio_threaded_peer] get_stdio_peer (threaded)
[INFO  websocat::ws_client_peer] get_ws_client_peer
[INFO  websocat::ws_client_peer] Connected to ws
{"event":"pusher:connection_established","data":"{\"socket_id\":\"227535699.117823784\",\"activity_timeout\":30}"}

(using localhost yield the same WebSocketError: I/O failure)
Trying my test script without TLS/HTTPS - which successfully works:
int(0)
string(0) ""
string(228) "{"name":["The name field is required."],"data":["The data field is required."],"channels":["The channels field is required when channel is not present."],"channel":["The channel field is required when channels is not present."]}"

Though here it doesn't matter if it is curl_setopt($ch, CURLOPT_URL, "http://<IP_ADDRESS>:8080/apps/468562/events"); or curl_setopt($ch, CURLOPT_URL, "http://localhost:8080/apps/468562/events");
So I'm a bit lost at what the issue really seems to be. Maybe something with the reverb http server in combination with using a custom SSL certificate (not using valet/herd).
Lastly I've also just tried a different hostname pp.localhost without TLS, that also seems to work (but not using websocat with the hostname 😅 ).

There's a lot going on here, so quite difficult to debug. However, what happens if you try to hit https://<IP_ADDRESS>/app/sdzhejnkcsrwunmyikgh directly in the browser?

Hello every body.
I still have the same problem and I am using Laravel 10 with Valet and I set local_cert and local_pk inside configuration file.
Also verify_peer has been set on false.
I got Internal Server Error while calling (https://domain.test:8080/app/uxgvldvjgvcc0rnnsuac).

Can you share a screenshot of the error?

Gonna close this one now. Feel free to open a new issue if you still experience this.

I have the same issue why close this when it is not fixed.

@volkeransmann In my case issue was using valet with secure reverb. Just using herd fixed the OpenSSL error.
Seems to be an issue with valet certificates for some reason.

@muresanandrei thanks for the tip. But that did non work in my case. Still getting the same error.

When calling the API using postman and finally triggering the PrivateChannel, an error message will be displayed.
However, on the front end, the connection is normal using laravel-echo.

Pusher error: cURL error 7: Failed to connect to xxxx.test port 8080 after 4003 ms: Couldn't connect to server (see https://curl.haxx.se/libcurl/c/libcurl-errors.html) for xxxx

Same issue for me, making a request from Postman to API in fresh install of L11 which dispatches a broadcast event I receive "cURL error 56: OpenSSL SSL_read: SSL_ERROR_SYSCALL, errno 0"
Generic "Guzzle Client" call to endpoint not on my server from within API controller works without issue.

@driesvints The above error seems to be caused by the size of the payload I broadcast!  I know pusher has a max event size of 10kb.  Does Reverb check the max size of the broadcast message? As soon as I reduced my message content the "cURL error 56: OpenSSL SSL_read: SSL_ERROR_SYSCALL, errno 0" error disappeared.  Hopefully this helps?


EXC_BAD_ACCESS because of UIView.superview

i think should remove all subviews from _hudView, _overlayView befor dealloc

i do  this
@implementation UIView (SVProgressHUD)

- (void)dealloc {
    [self.subviews enumerateObjectsUsingBlock:^(id obj, NSUInteger idx, BOOL *stop) {
        [obj removeFromSuperview];
    }];
}

@end

Could you provide a sample project to demonstrate the crash. I can not reproduce your error.

I think i has find out why it happens:
@implementation UIView (BKRemove)

- (void)dealloc {
    [self bk_removeAllBlockObservers];
    [self bk_removeAllAssociatedObjects];
}

@end

If i comment UIView's dealloc category,  the crash doesn't happen.
I has upload the demo

When internet is not reachable or could not connect to server or request timed out , this kind of situation arises at that time svprogress hud shows bad_access error some time in hud view or some times in string label in svprogresshud.m file.please help.

Such a bug is very hard to reproduce. Therefore, you have to provide a demo project to help me solve the issue.

No feedback therefore closed.


chore(e2e): strengthen openapi tests

Ref #5924

Release Note
chore(e2e): strengthen openapi tests

cc @hernanDatgDev these tests should cover the openapi both on plain deployment and Knative service ones.


Enhancement: Theorems, definitions, and numbered equations.

For some (many) of the mathematics compendiums, having some more clear markup for theorems, definitions etc. would be great.
Also, having these (along with equations) numbered, and being able to refer to them, is also ideal for a lot of the same class of compendiums.

The latex library we use, mathjax, supports automatic equation numbering:
http://mathjax.readthedocs.org/en/latest/tex.html#automatic-equation-numbering
Enabling this would mean all equations on a page would get a number assigned, and I believe it should be possible to refer to them from the text.

As for theorems and definitions, the closest thing we have today are blockquotes.
Could they be sufficient for this use case, or are some other special features needed?
For reference, blockquotes are created with the following syntax:
> This is a blockquote

and rendered as

Yes I am aware of the blockquote feature.  They are okey as is, but lack:

(Very subjective) Proper formatting, I much prefer Math SE's blockquotes. See for instance this link.
A distinguishing factor for theorems, conjectures, lemmas, corollaries etc. This can be introduced by the user by
Numbering. For short articles or answers (as on Math SE) this is rarely necessary. However for larger compendiums, such as this, or this, being able to number and refer back to a given theorem is very handy. This is particularly relevant for courses which are very sequential in nature, in the sense that definitions and theorems from one chapter is relied upon and used a lot later.


Fix fastJSON deserialization into dicts with interface type values [#354]

Eliminated the type conversion in the case it is not actually needed. I’m currently on my first-ever C# project so it is possible I’m in way over my head dealing with reflection.

Does this resolve the issue?

Yes, I did check on my project that applying this patch does successfully yield the expected mappings.


Add wateryear

Coverage increased (+0.05%) to 90.68% when pulling be6335e2957701e80c3a6e0525dc91e420754b81 on phobson:add-wateryear into 80e34ef266e31141e60d787f99bc19fab3da3ca4 on Geosyntec:master.


View not scrolling when dragged card near edges

See video using demo in Chrome.
Dragged card near edges doesn't scroll the view horizontally.
Occurs when browser is not full-width, or when div is not full width.
Scrolling works on column headers.
https://user-images.githubusercontent.com/1159336/154529276-0a2db5b5-c62e-433a-abdb-a32ec717f210.mov

You could try using the layout of this page, it worked for me, my guess is that you need to add the overflow: auto attribute to the outer container。vue-smooth-dnd


Make some checkbox labels labelComponents

This makes the labels clickable.
Also add stories to retention warning dialog.

r? @keybase/react-hackers Screenshots:
master:

![mobile - master](https://user-images.githubusercontent.com/13270622/40519102-ae51798e-5f72-11e8-9d23-119300627524.png)
this PR:

![mobile - pr](https://user-images.githubusercontent.com/13270622/40519107-b33dd3d4-5f72-11e8-9d4d-4e8393917445.png)


Add Multiple Apprenticeships

Add Roblox Community Apprenticeship

✅ By submitting this PR, I have verified the following

[] Checked to see if a similar PR has already been opened 🤔
[] Reviewed the contributing guidelines 🔍
[] Added my name to the bottom of the list under the Credits section in the README.md with a link to my website or GitHub profile 👥

Thank you, Frances!


Add Pet Clinic Index page and Controller

@rsupipi did you intend to report those issues in the main repo? If you did, we need a proper description.


sed: cannot rename /etc/sedqp9KU2: Device or resource busy

When I run hack/local-up-hypernetes.sh, on docker playground with centos 7.2.1511 & 7.3.1611, the error message is
[root@50f7621a761a kubernetes]# hack/local-up-hypernetes.sh
Start kube::util::verify_system
Start kube::util::setup_hostname
sed: cannot rename /etc/sedqp9KU2: Device or resource busy
How to solve it?
Thanks!

@YuPengZTE There are some problems of running kubernetes components inside docker containers, e.g. kubelet requires many privileges. Could you have a try on centos VM or bare metal host?

Thanks @feiskyer .
I run with command:
$ docker run --privileged --rm -ti centos:7.2.1511
The error is still on .
Inside docker containers, there are many problems, I will try on centos VM, thank you!

How did you fix it @YuPengZTE ?
I'm having the same issue trying to build a cassandra_exporter, using --privileged didn't work


Scenarios' MaxCoreNumber, MemoryLimit and TemporaryDirectoryName AnalysisSpec.SystemParameters Are Set via Environment-set Values

Description
Currently, Khiops scenarios's MaxCoreNumber, MemoryLimit and TemporaryDirectoryName SystemParameters are set either from a custom GeneralOptions instance, or from KhiopsRunner's general_options attribute. But, the GeneralOptions's attributes max_cores, max_memory_mb and khiops_temp_dir are initialized from khiops_env KHIOPS_PROC_NUMBER, KHIOPS_MEMORY_LIMIT and KHIOPS_TMP_DIR attributes, respectively.
However:

KHIOPS_PROC_NUMBER is just an internal variable of khiops_env which is used for setting the KHIOPS_MPI_COMMAND variable; thus, this sets the upper bound of the number of CPU cores that Khiops would use in the current session;
KHIOPS_MEMORY_LIMIT is relayed to the MODL* binaries and sets the upper bound of the number of MBs that Khiops would use in the current session;
KHIOPS_TMP_DIR is relayed to the MODL* binaries and sets the default temporary directory that Khiops would use in the current session.

Hence, these are global settings which set bounds for any processing done by Khiops in the current working session.
However, Khiops scenarios' parameters SystemParameters.{MaxCoreNumber,MemoryLimit,TemporaryDirecytoryName} are local to each processing (i.e. to each Core API call), hence:

they should be set via the Khiops Core API calls
the expectation is that their values are lower than or equal to the global settings.

Questions/Ideas

All Khiops Core API functions defined in khiops/core/api.py should take three new keyword arguments:

max_core_number: int
memory_limit: int
tmp_dir: str

the KhiopsRunner._write_scenario_prologue should not write into AnalysisSpec.SystemParameters.{MaxCoreNumber,MemoryLimit,TemporaryDirectoryName} from the general_options anymore, but from task_args, as these are "task"-level arguments.

So, these parameters will be part of the official API ? Not only for Python ?

One detail: When executing a large quantity small tasks is often convenient to execute with just 1 CPU because the overhead coming from the MPI process creation dominates the execution time. This overhead is worse on machines with many CPUs.
So when an API with max_cores set to 1 or 2 I think MPI should not be used on execution.

So, these parameters will be part of the official API ? Not only for Python ?

I suppose so, as we need to be iso-functional with the Core GUI.

One detail: When executing a large quantity small tasks is often convenient to execute with just 1 CPU because the overhead coming from the MPI process creation dominates the execution time. This overhead is worse on machines with many CPUs.
So when an API with max_cores set to 1 or 2 I think MPI should not be used on execution.

I see your point and the value in the proposal, but:

in my understanding, this would diverge from the current functioning of Khiops Core: the khiops / khiops_env scripts do not parse the scenarios to be aware that only 1 or 2 CPUs were required;
implementing this on the Python side would break the invariant of relying exlusively on khiops_env for handling the command construction logic.
Hence, it needs to be discussed.

Ok for discuss. Another point, I'd go even further: some api calls do not need MPI (ex. read_dictionary_file) so the cpu number I think the CPU number should'n be available for them.

Ok for discuss. Another point, I'd go even further: some api calls do not need MPI (ex. read_dictionary_file) so the cpu number I think the CPU number should'n be available for them.

Following discussion, the conclusions are:

we keep the invariant of relying exclusively on khiops_env for handling execution logic; hence, if KHIOPS_PROC_NUMBER > 2, then mpiexec would be used, irrespective of what is specified at the scenario level (in terms of MaxCores); thusly, also the coherence with the Core GUI / batch behavior is kept;
all Core API functions would get the three parameters (that would be passed to the scenarios), irrespective of whether MPI is actually needed or not; thus, this optimization would not be implemented (for the time being).

Support the user-defined scenario prologue through the same mechanism.


issue #91

@harshita214 I have recreated the pr and about both changes you requested
This time I haven't created the SVG file and stored the variable in that form only in the js file
I have implemented the CSS inside the HTML file

Great! Merging the PR 🚀 ✨
Thanks for contributing 💚


Supplemental issues template

Add a github repository link to the SCA issues template. Easy project maintenance. Accurately reproduce problem scenarios.

It sounds good, I will do it.


AutoRedoTheMLO

i have no idea how to include these changes into my code but yes i approve them all lmao

LGTM. Thanks!


small tweaks / fixes

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (6c7ac26) 78.57% compared to head (3de6fc3) 78.57%.
Report is 2 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main       #2   +/-   ##
=======================================
  Coverage   78.57%   78.57%
=======================================
  Files           2        2
  Lines          14       14
=======================================
  Hits           11       11
  Misses          3        3

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


[Enhancement] To this repo, add an issue that has rules that tells people what to do when there are spam issues.

yesterday, I saw that there were 500+ spam issues with bad words and a few people commented on spam issues and one made an issue about the spam thread, but then you deleted all the spam issues, and then the few people that commented has been accidentally banned from this repo, so to let others know what to do with the spam issues there could be a pinned issue for this repo (made by you) and then the pinned issue contains rules to do when there are spam issues, so that no one will get banned by accident, this issue is just an enhancement to this repo, thanks.

https://github.com/osy/HaC-Mini/issues/340#issuecomment-734376188


⇧ Shift selection works on click instead of holding

Describe the Bug
The ⇧ Shift selection works on click instead of holding.
Might be not a problem at all, but I think the all-known UX for ⇧ Shift is selection while the key is held pressed.
Your Example Website or App
flux.paradigm.xyz
Steps to Reproduce the Bug or Issue

Go to flux.paradigm.xyz
Create multiple nodes
Select a node and hit ⇧ Shift key.
Click on another node, and it will start a selection, although a ⇧ Shift key is not held.

Expected behavior
As a user, I expected the ⇧ Shift key to be required to be held to select multiple nodes
Screenshots or Videos
https://user-images.githubusercontent.com/35642018/229347679-4be3888c-0a4a-45b1-8773-e77a627a0a25.mov
Platform

OS: macOS
Browser: Chrome
Version: 111.0.5563.146
Reactflow: ^11.5.6,

Additional context

https://github.com/transmissions11/flux/issues/37

This is built in. You can use the multiSelectionKeyCode to configure this.

This is built in. You can use the multiSelectionKeyCode to configure this.

I apologise, but you have not answered to my question.
The description states:
While pressing the multiSelectionKeyCode you can select multiple nodes and edges with a click. You can set it to null to disable functionality.
I interpret while pressing as holding a key, right? Or am I wrong?
As in the video I've attached to the issue, the described behaviour is not met.
Selection starts not while pressing a multiSelectionKeyCode, but rather clicking once on it, the selection mode is toggled.
I must disagree with the issue closure and would like to see it reopened.

cc @moklick

@h0tw4t3r What's your configuration look like?
I can't really reproduce this issue in a sandbox, could you outline the steps a bit more?
With the steps you gave above it works perfectly fine, I select a node, hit Shift and tried to select another node. The previously selected node becomes unselected and the new one selected.
I have not configured anything, so I'm using the defaults for key codes, so Shift wouldn't be the multiSelectionKeyCode but the selectionKeyCode which triggers the selection box that can be used to select multi elements at once using a blueish box to wrap them visually, whereas the multiSelectionKeyCode is the key code you would hold and click each node you want to select.
Here's the sandbox I used.

https://user-images.githubusercontent.com/78412429/229362824-517c31e1-7b5f-4bcc-a8d2-dd46b9650a39.mov
Here's a short clip just to clarify what I mean and what the difference between selectionKeyCode (the blue box) and multiSelectionKeyCode is.
In the clip the first action is done through selectionKeyCode (Shift by default). I hold Shift and click-drag until I select all nodes I want.
The second one through multiSelectionKeyCode (Meta by default), I hold the command key on my Mac and click each node.
The third is just not holding any key at all and clicking each node.

@bcakmakoglu, thanks for a deep explanation!
I've checked out the sandbox you've provided, and it indeed works as intended.
The flux.paradigm.xyz repository can be found at https://github.com/transmissions11/flux,
and the specific configuration I'm referring to can be found here, and is next:
<ReactFlow
  proOptions={{ hideAttribution: true }}
  nodes={nodes}
  maxZoom={1.5}
  minZoom={0}
  edges={edges}
  onInit={setReactFlow}
  onNodesChange={onNodesChange}
  onEdgesChange={onEdgesChange}
  onEdgesDelete={takeSnapshot}
  onNodesDelete={takeSnapshot}
  onEdgeUpdateStart={onEdgeUpdateStart}
  onEdgeUpdate={onEdgeUpdate}
  onEdgeUpdateEnd={onEdgeUpdateEnd}
  onConnect={onConnect}
  nodeTypes={REACT_FLOW_NODE_TYPES}
  // Causes clicks to also trigger auto zoom.
  // onNodeDragStop={autoZoomIfNecessary}
  onSelectionDragStop={autoZoomIfNecessary}
  selectionKeyCode={null}
  multiSelectionKeyCode="Shift"
  panActivationKeyCode={null}
  deleteKeyCode={null}
  panOnDrag={false}
  selectionOnDrag={true}
  zoomOnScroll={false}
  zoomActivationKeyCode={null}
  panOnScroll={true}
  selectionMode={SelectionMode.Partial}
  onNodeClick={(_, node) => {
    setLastSelectedNodeId(selectedNodeId);
    setSelectedNodeId(node.id);
  }}
>

I would appreciate if you could give an advice with a possible caveat that can cause the issues on flux side.
@moklick, the issue won't require a reopening, sorry for bothering. It's a user side error.

@h0tw4t3r no worries. You've set multiSelectionKeyCode to "Shift" and selectionKeyCode to "null". So you disabled the selection box and use Shift as a multi selection key code. You could set both props to "Shift" Not sure if that's what you want to achieve.

From playing around with the website you linked it seems there's is some issue with your multi select, though I really can't identify why it's happening from the code.
Could there be anything else catching the keyup event on your end?
Have you tried using a different key, does it behave the same way? If you were to use some key like A instead of Shift just to see if the behavior differs?
It feels likekeydown is caught but when you hold it for a little bit and release the key there is no keyup being caught and the multi select acts like it's still being held even though the key was released.

I suppose this could have been fixed with https://github.com/wbkd/react-flow/pull/2892.
Can't reproduce it with an upgraded reactflow.
The version used before: ^11.5.6, after upgrading to ^11.7.0 issue is not reproducible.
Thanks for your help!


Please publish your package sanjo:karma for Windows

Hi!
This package has binary dependencies, and we wanted to give you a heads up that Meteor 1.1, with support for Windows will be released in a few days.
Now is the time to publish your package for Windows!
It should be very straightforward -- we've added new build machines that run Windows that you can access with the meteor admin get-machine command. This is the same as you've done before for the other architectures we support.
For complete directions on how to publish your package for Windows, visit https://github.com/meteor/meteor/wiki/Publishing-a-package-with-binary-dependencies-for-the-Windows-preview
On behalf of the Meteor team,
@stubailo, @Slava and @avital.
P.S. This is an automated message based on our database of package authors.

Yes... please could I add my request to this. I am looking forward to working in windows instead of under vagrant but have dependencies on karma. Many thanks!

I currently work on making making Velocity and sanjo:jasmine compatible to Windows ;-) Releasing sanjo:karma is just a small part of this.

That would be excellent.... Look forward to seeing it :)

Done with 1.5.0.
sanjo:jasmine has a release candidate with Windows support: meteor add<EMAIL_ADDRESS>


*: support customized labels

fix https://github.com/coreos/etcd-operator/issues/872
/cc @hongchaodeng @hasbro17

lgtm


added more feature of email notification init

AI Code Review is in progress (usually takes <3 minutes unless it's a very large PR).

Changelist by Bito
This pull request implements the following key changes.

Key Change
Files Impacted

Feature Improvement - Enhanced Birthday Reminder Functionality

- source-code.py - Added email and desktop notifications for birthdays


Searching %PREFIX%\Library for new CTK Windows packages

In PR ( https://github.com/numba/numba/pull/7255 ) logic was added to Numba to handle the new-style CTK Conda packages on the nvidia channel (and now also on conda-forge)
In particular this added a series of get_nvidia... functions to search for libraries as part of numba/cuda /cuda_paths.py. For example
https://github.com/numba/numba/blob/a4664180ddc91e4c8a37dd06324f4e229a76df91/numba/cuda/cuda_paths.py#L82

While it does look like these have some logic for searching for Windows libraries (like looking in bin instead of lib). It appears they start with %PREFIX% instead of %PREFIX%\Library (where Conda packages typically put content on Windows)
Admittedly the nvidia package do use %PREFIX% to structure content. Though most Conda packages (include the CTK packages in conda-forge) do not do this using %PREFIX%\Library instead. In the future nvidia packages will match this behavior.
Given this wonder if we could search %PREFIX%\Library for the CTK first and then search %PREFIX% second. This would handle both structures seamlessly

For reference this came out of testing Numba with the new-style CTK packages on conda-forge
ref: https://github.com/conda-forge/numba-feedstock/issues/130

Also it looks like for static libraries, %PREFIX%\lib\x64 is used as opposed to %PREFIX%\Library\lib
https://github.com/numba/numba/blob/a4664180ddc91e4c8a37dd06324f4e229a76df91/numba/cuda/cuda_paths.py#L74

Thanks for the report. CC @gmarkall is this something to fix for 0.59?

Proposed fix in #9279. This would be a good thing to fix for 0.59.


Warning for referenced unpublished/deleted elements

When a page refers to a draft/deleted element (e.g. deleted image in an article paragraph, draft/deleted poll embedded in an article, page button referencing a draft/deleted page), display a warning in the CMS.
Don't allow saving in case of missing elements. (For referenced draft pages, there is a use case in displaying them anyway, if you're currently drafting an article and want to see what it will look like when the article and e.g. referenced poll are published. Maybe worth a discussion).
This complements #985 which also contains a full list of such instances.

@lauriemarkle What are your thoughts on displaying referenced pages that are now in draft?
I'm thinking it gets very messy for users to try to understand that this could happen:

They publish a survey
Embed it in an article
Unpublish the survey
The survey still appears in the article
But presumably users can't submit on it, and can't continue to later pages of a multi-page poll

Trying to find a good place to explain this behavior to users also seems tricky to me.
On the other hand, if we don't display referenced pages that are in draft we match the behavior of eg Home Page items which don't appear user-facing when they are in draft.

Suggested behavior:
When a Page references an asset in draft

The reference appears in the Admin Panel
You can save/publish the Page
There is a warning on each reference point where the asset is in draft
The reference does not appear on user-facing site

When a Page references an asset that is deleted

The reference appears in the Admin Panel
You cannot save/publish the Page until you've resolved the problem
There is a warning on each reference point where the asset used to be
The reference does not appear on user-facing site


ios launch images not appearing

Using Lime 2.0.5 with my iPhone 4 (should load<EMAIL_ADDRESS>I think) I can't see the launch image but on 2.0.4 it works.
All the .png are indeed copied inside the .app but a black screen still appear.

Everything seems fine in 2.0.6


Yes
Test Plan
The build output now shows all posts' content, including those with MDX.

LGTM thanks 👍


Attempt to fix windowsAuthCallback redirecting to root of site

This is my attempt to fix the windowsAuthCallback call redirecting to the root of the site when Bonobo is a subfolder application when configured to use Windows AuthenticationProvider. Works under IIS Express but ends in a 200 response when pushed to IIS 7.5. ( https://github.com/jakubgarfield/Bonobo-Git-Server/issues/345 )

@jakubgarfield  Any plans on creating a release soon for this PR? Or at least a binary DLL so people like I can try and see if it solves our problem without having to setup a whole dev environment.

Oh, it slipped under my radar. I'll try to do release after the weekend.


About Us link now deployed

Is there an existing issue for this?

[x]  I have searched the existing issues

What happened?
BUG: About Us page not linked!
Record

[x]  I agree to follow this project's Code of Conduct
[x]  I'm a GSSOC'24 contributor
[x]  I want to work on this issue

@Priyaaa1 assign me this issue under gssoc label i can add add general about us page template in which names would have to be edited rest web page will be ready


Look into spark-testing-base

https://github.com/holdenk/spark-testing-base

https://github.com/adelbertc/frameless/pull/8


[move vm] Added Constant pool

Motivation

New pool for constants
The constants can support any non-struct value, serialized in the same format as on chain values
Addresses are split

Runtime constants are in the constant pool
Namespace constants are in the 'AddressIdentifierPool'

TODO

Fix tests (they compile but many are broken)
Actually fix various generation tools
Add bytecode verifier rules

Test those rules

Test Plan

cargo test + TODO

The constants can support any non-struct value, serialized in the same format as on chain values

Is (e.g.) vector<vector<u64>> allowed?

:umbrella: The latest upstream changes (presumably 4036fee848a040242ed8816d9c569d10068e88b1) made this pull request unmergeable. Please resolve the merge conflicts.

The constants can support any non-struct value, serialized in the same format as on chain values

Is (e.g.) vector<vector<u64>> allowed?

yes it is

@bors-libra r=dariorussi

:pushpin: Commit bfbb669 has been approved by dariorussi

:hourglass: Testing commit bfbb66966ae69aea95addffc5fc3439d6fff2426 with merge 9c325e25efc809df3b4993104d127923af512383...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: dariorussi
Pushing 9c325e25efc809df3b4993104d127923af512383 to master...


Feature not working: navigation

navigation feature may be broken
I have tried to get the top navigation feature to work. So far I have failed. The sidebar works well, but not the top header navigation.

Tried it together with sidebar, having navigation before and after in the docs.jsonconfig. No navigation appears.
Tried it alone, no header navigation appears.

Thus in neither case can I get the header navigation to show.
My use case for it was that I was trying to use a sidebar as my main nav of course, and to also put two links in the header navigation that would always be present there to two external places, just for convenience.
Reproduction docs repo:
Here is a test reproduction repo: https://github.com/rydmike/docs_page_test
and site: https://docs.page/rydmike/docs_page_test
Having docs.json config file:
{
    "name": "DocsPageTests",
    "favicon": "/images/favicon.png",
    "logo": "/images/logo.png",
    "logoDark": "/images/logo.png",
    "theme": "#0x29B6F6",
    "zoomImages": "true",
    "noindex": "false",
    "sidebar": [
        ["Welcome",
            [
                ["Intro","/"],
                ["More intro", "/more_intro"]
            ]
        ],
        ["Zoom image", "/another_page"]
    ],
    "navigation": [
        ["Intro","/"],
        ["More intro", "/more_intro"],
        ["Package", "https://pub.dev/packages/flex_color_scheme"],
        ["Playground", "https://rydmike.com/flexcolorscheme/themesplayground-v5"]
    ]
}

I believe this is an error with the docs, and this is a feature we no longer support. I will double check.

Yeah it seems like in an earlier version we had this, and no longer support it. We could add it back in potentially


Migrate ballerinax/jdbc to Gradle

Description:
$Subject

N/A anymore. jdbc client have been moved inside the ballerinalang repo ATM.
https://github.com/ballerina-platform/ballerina-lang/pull/16197


Log when patch fails

Log an error mentioning which key failed to patch, when patching fails.

appveyor failure fixed in master, mering this PR


Picovoice Console Issue:  can't create previous (2.1.x) engine wake-words

I have an installation of Rhasspy that uses porcupine. It only works with 2.1.x engine of porcupine.
Today I was going to create a custom wake-word for it, bu I have seen that you have updated online engine to 2.2.0 and the wake-words I create now are incompatible with my installation.
Is there any option to create a wake-word with 2.1.0 engine (github software to download to do it offline or wahtever). O I will have to stop using porcupine.
What I was trying was:
word:     'ok madre'
engine:    2.1.0
language:  Spanish
platform:  raspberry pi
Access key: '---not-published here, of course---'
Expected behaviour
To be able to create a wake-word for version 2.1.0 engine installations.
Actual behaviour
Only 2.2.0 .ppn file that is not working with my configuration.
Steps to reproduce the behaviour
Just creating a wake-word since may 2023...

There are no other options to create older versions of keyword files. Update the version of the engine.

There are no other options to create older versions of keyword files. Update the version of the engine.

Ok. Rhasspy 2.5.11 only available for 2.1.x porcupine. Unable to update.
Changing to a different wake-word software. Trying Snowboy. Thanks a lot!,


Change plugin ID to match grafana expectations

https://grafana.com/docs/grafana/v7.5/developers/plugins/legacy/style-guide/#pluginjson-mandatory

The convention for the plugin id is [github username/org]-[plugin name]-[datasource|app|panel] and it has to be unique.

why removing the .gitkeep ?
If dist doesn't exist when docker-compose is run, it will fail

I removed it as the whole dist folder is in .gitignore
Thought it was a mistake to have kept this file
Why does docker-compose needs this file?

dist folder contain the plugin when builded (so, it's git ignored) .
docker-compose will try to bind the dist folder to the plugin folder . ( and will fail to bind it if it doesn't exist ) .
The .gitkeep is just here to keep the dist folder (because git doesn't commit empty folders)
About the why, it allow to start the docker compose, without building the plugin . ( the job is just to ease the QA work, if they use it and reduce the possible errors )

Ok @thib3113
So I added back the .gitkeep file


Mathematica Jupyter integration tutorial does not work

I've just tried to install the Matheamtica (Sorry, Wolfram engine) integration for Jupyter as described in the tutorial. However, the install fails (already in the early stages, as libraries seem to be missing):
Reading state information... Done
Package libasound2 is a virtual package provided by:
liboss4-salsa-asound2 4.2-build2020-1ubuntu3
libasound2t64 1.2.11-1build2 (= 1.2.11-1build2)
You should explicitly select one to install.
Package libgl1-mesa-glx is not available, but is referred to by another package.
This may mean that the package is missing, has been obsoleted, or
is only available from another source
This later on leads to the engine being unable to start up:
usr/local/Wolfram/WolframEngine/13.3/SystemFiles/FrontEnd/Binaries/Linux-x86-64/WolframPlayer: error while loading shared libraries: libGL.so.1: cannot open shared object file: No such file or directory

@berndmeyer Thanks!  I've fully updated and tested the tutorial now, and it works.  It includes supporting Mathematica 14, the new Ubuntu 24.04, and I tested graphics in a Jupyter notebook, etc.
If you hit any issues, let us know in support<EMAIL_ADDRESS>

Fantastic, works like a charm. Thanks!
However, please note that the instructions in the tutorial install Wolfram Engine Version 13.3 but the example commands further down assume that Version 14 was installed.


New Version: GodotEngine.GodotEngine version 3.0

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Hi @og-mrk,

This PR's version number 3.0 has 2 parameters (sets of numbers between dots - major, minor, etc), which is less than the current manifest's version 4.2.1, which has 3 parameters.

Does this PR's version number exactly match the version reported in the Apps & features Settings page? (Feel free to attach a screenshot.)
(Automated response - build 855)

Hi @og-mrk,

This PR's version number 3.0 has 2 parameters (sets of numbers between dots - major, minor, etc), which is less than the current manifest's version 4.2.1, which has 3 parameters.

Does this PR's version number exactly match the version reported in the Apps & features Settings page? (Feel free to attach a screenshot.)
(Automated response - build 855)
Edit: Does this change frequently? It seems like this developer adds and removes the 3rd parameter freely. Let me know and I can add this to an exception list for this check.

Hello @stephengillie,
The application in question, GodotEngine, has a Well Documented Versioning Model found on GodotEngine Official Documentation Website, and to quote from them, from this page:

We call major.minor combinations stable branches. Each stable branch starts with a major.minor release (without the 0 for patch) and is further developed for maintenance releases in a Git branch of the same name (for example patch updates for the 4.0 stable branch are developed in the 4.0 Git branch).

I hope my respond we clear and concise, if you have any more questions, feel free to post them here and I'll reply as soon as I can.

Hello @ stephengillie, The application in question, GodotEngine, has a Well Documented Versioning Model found on GodotEngine Official Documentation Website, and to quote from them, from this page:

We call major.minor combinations stable branches. Each stable branch starts with a major.minor release (without the 0 for patch) and is further developed for maintenance releases in a Git branch of the same name (for example patch updates for the 4.0 stable branch are developed in the 4.0 Git branch).

I hope my respond we clear and concise, if you have any more questions, feel free to post them here and I'll reply as soon as I can.

Thank you very much for your thorough yet concise answer. This is a versioning schema that a colleague was recently describing, but I hadn't encountered before today.


When the new user registers browser shouldn't ask to memorize the password for /user/sign_up screen

Browser asks to memorize the login/password twice. /user/sign_up isn't necessary. /user/login  is okay.
Observed in chrome.

This is the automatically behavior of browser as submitting a form.
I consider this is invalid issue until more strong proves.

Here they claim that autocomplete="off" disables memorization. There is also some other solution mentioned there.
But I can't test this easily.
http://stackoverflow.com/questions/32369/disable-browser-save-password-functionality

autocomplete="off" disables memorization but I don't think it is the same thing.
BTW, it does not work based on my quick test.


added support for simple ad-hoc specs

Pull Request Test Coverage Report for Build 935205097

95 of 97 (97.94%)  changed or added relevant lines in 5 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.3%) to 92.089%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/minimal_progress_view.go
6
8
75.0%

Files with Coverage Reduction
New Missed Lines
%

pkg/minimal_progress_view.go
1
95.58%

Totals

Change from base Build 933701826:
0.3%

Covered Lines:
1688

Relevant Lines:
1833

💛  - Coveralls

Pull Request Test Coverage Report for Build 935205097
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

95 of 97 (97.94%)  changed or added relevant lines in 5 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.3%) to 92.089%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/minimal_progress_view.go
6
8
75.0%

Files with Coverage Reduction
New Missed Lines
%

pkg/minimal_progress_view.go
1
95.58%

Totals

Change from base Build 933701826:
0.3%

Covered Lines:
1688

Relevant Lines:
1833

💛  - Coveralls


Pairs in C++

Related Issue
Solved issue number: #394
Describe the changes you've made
Adding Source Code for Pairs in C++
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
NA
Language used for coding
C++
Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have documented the code.
[x] New and existing unit tests pass locally with my changes.

Please review the code ;-)


add TaskStatusReporter class to fix Windows pickle issue

Windows uses a different method when forking, which caused issues with
the task status updating functions that are passed to TaskProcess
(they need to be pickle-able). Basically running with >1 worker was broken
on Windows. See #1423
This fixes the problem by introducing a new pickle-able class to encapsulate task status
update methods.
Have you tested this? If so, how?

There was existing test coverage for code that was refactored (e.g. test/task_status_message_test.py)
I ran a test task tree on a Windows machine with --workers 10 before the change to reproduce the issue (got pickle error), then after and the test succeeded -- tested with py2.7

Unless anybody objects or have other review-points. I'll merge this in soon.