Allow a custom panic handler

Rendered

Could this mechanism be used to support abort-on-panic semantics without the need for a compiler flag?

Yep, though you'd still have all of the panic infrastructure (landing pads, etc) in place, so we'd probably still want to have the flag. This would cover the use case of "I want to find out about panics via a process abort (maybe with debug data logged or something)", but not "I'm running in an environment that I don't want/can't have stack unwinding".

@sfackler
-Z no-landing-pads causes a very ugly crash on panic with the default (unwinding) panic handler, but otherwise it is kind of orthogonal.

cc @rust-lang/lang @rust-lang/core -- while technically a "library" issue, this is a pretty central aspect of the language and should get scrutiny across these teams.

The API design here seems reasonable for the "global resource" approach to panic handling, and I agree with @sfackler that as long as panic handlers are an application-level concern, this probably works just fine. But I have a couple concerns.
Composability/library use
My biggest worry about the design is potential use within libraries; I agree with @alexcrichton that such use is basically inevitable. Global resources give you poor composability between libraries (think: global state like the current working directory).
As others have mentioned, the per-thread handler model could potentially deal with it. The alternatives section mentioned the per-thread model, but dismisses it essentially due to lack of inheritance. But we should be able to provide inheritance for threads spawned via the standard library at least. So I'd like to discuss the tradeoffs here a bit more deeply.
Relatedly, the RFC claims that it'd be easy to add per-thread handlers on top later. I'm curious exactly what you have in mind -- would it be an entirely separate layer prior to calling the global handler?
Pre- vs post-unwinding
The proposal here is to invoke the handler immediately upon panic, rather than after unwinding. IIRC this was motivated in part so that the call stack is available. But it's also a bit counterintuitive, and doesn't play well with recover -- even in cases that panics are "caught" and otherwise "handled" in some way, the generic global handler will still be run.
Is there some other way we could retain the needed stack information but invoke the handler after unwinding? Or is there rationale, perhaps for providing hooks on both ends of unwinding?
Abort semantics
Finally, there are two issues around abort that would be good to settle:

To what degree does this API tie us to the current "abort on double panic" semantics? I know some have argued against that semantics (@nikomatsakis among them, IIRC). I've heard talk of, for example, "bundling together" the payloads from multiple panics. It'd be nice to keep our options open.

Others have raised the issue of using this mechanism for an abort-on-panic semantics, but @sfackler points out that you probably want to invoke a separate compiler flag as well. It'd be nice to have a relatively clear and simple story about how to get abort-on-panic, and how that might play with the handlers here. (Not a blocker for this RFC, more something to discuss a bit just to be sure there's a reasonable plan.)

Follow-up: my previous comment sort of implied that inherited, thread-local handlers "solve" composability, but I think that's not really true; they just mitigate it. It's still possible to use multiple libraries that want to frob the thread-local handler, and they have to play together somehow.

@aturon
Panic handlers are intended to work even without unwinding, and therefore they should run before it. If you want to ignore panics that are caught by recover, you should do your own recover at the event loop.
The interaction with double panics is annoying. I would prefer a version that has panic!("foo") being equivalent to
    let panic_info = PanicInfo {
        location: current_location!(),
        info: box "foo"
    };
    GLOBAL_PANIC_HANDLER.borrow()(&panic_info);
    unwind(panic_info);

In that case, a panic in a panic handler would lead to a safe death via stack overflow. If this is not desirable, a panic handler could have a recursive lock (we need to have some way to handle concurrent panics anyway).

:bell: This RFC is now entering its week-long final comment period :bell:
Note that this entering FCP at the same time as #1100 and the two will likely be decided upon as a unit.

This is a more polished design than the one I proposed and didn't have time to revise, and supports adding thread-local handlers in the future.
:+1:

@aturon
I am not a fan of an inheritance based situation for a couple of reasons:

Many programming languages provide a global exception handler - C++, Java, Python, etc. As far as I can tell, zero languages provide this kind of handler inheritance scheme.
Threads not spawned via std::thread::spawn are left in some kind of limbo state with respect to panic handling. We could have some function you could call to register a handler after the fact, but that makes writing code that interoperates with other languages way weirder.

It's not totally clear to me what use cases you see for libraries wanting to mess with a global handler. From my experience in Java, the only libraries that mess with it are the ones that explicitly advertise that fact and take over the handler to e.g. forward uncaught exceptions from an Android app back out to some server for tracking. I can imagine we'd update the log crate to have a function that will replace the global handler with one that logs errors for people that want to do that.
Libraries certainly might find it convenient to set custom handlers on specific threads that they create and maintain but that doesn't seem like a necessity - panic::recover is a better bet for thread pools and Java is the only language I've looked at that supports per-thread uncaught exception handlers.
Running handlers post-unwinding does seem like a better bet if we can figure out what to do with backtraces. @alexcrichton, is the stack walking part of backtrace generation fast enough that we can reasonably run it for every panic? If not, we might need a separate method to indicate that the handler is interested in backtrace info or something like that.
I don't think this ties us to abort on double panic any more than the Err value for joining a thread does - in both cases we have a Box<Any + Send> which right now implies one payload, though the thing inside of it could of course in the future be something wrapping a Vec<Box<Any + Send>> or whatever.
The reentrancy of the handler itself is a bit more subtle, but I don't feel super strongly about what the behavior is there. It could be treated as a double panic -> abort, or recursively call the handler, or skip the handler the second time around.

@sfackler
From my point of view this is not supposed to be an unhandled exception handler because Rust does not support handled exceptions, it just supports recovery from exceptions. It is just the panic handler, and it should run when panic is called (for example, unwinding may not actually be supported, and even if it is, it may call undesirable destructors).

In terms of composability I think I prefer the ability to only add a handler and unregister just that handler (e.g. the Windows-like style), but in terms of implementation what's proposed here is more general (e.g. you can build the Windows style on this one). As a result, plus some of the comments @sfackler has made, I think I'd lean in favor of this strategy, acknowledging that the composability just isn't that great and more composable solutions can be built up externally. The upside of this proposal is that it's easy to understand and implement and provides at least the foundation for doing more pieces later.

Pre- vs post-unwinding

@aturon, this is an interesting point I hadn't considered before, yeah. I wouldn't necessarily say that there's a hard requirement that it runs pre-unwinding, it's just convenient today that the backtrace is available. That being said, it may not be totally unreasonable to have a scheme along the lines of the outermost recover block is the only one that will run the global handler. For example if you have nested recover blocks on one thread you'll never invoke the global handler until the outermost one catches the panic. We could easily set this up with TLS, and it means that FFI boundaries and thread boundaries would run the global handler, but manual calls to panic::recover wouldn't.
Along those lines, it may not be so critical that we distinguish pre vs post unwinding.

Abort semantics

I agree with @sfackler that I don't think this really changes our story here much. Even if we have to special case the panic-handler panicking that seems like it's not the end of the world, the desire to handle double panics I believe is motivated to work with "most code" rather than all code.

is the stack walking part of backtrace generation fast enough that we can reasonably run it for every panic?

@sfackler, I'm not actually quite sure, but panicking is so slow already it probably wouldn't matter if we just generated a backtrace. That being said I still think we should always have backtraces be optional because including libbacktrace is a nontrivial dependency of the standard library that may not always be desired.

Yep, I figure we'd make any backtrace accessor return an Option<Whatever> like PanicInfo::location does in this proposal.

@sfackler Thanks for the detailed reply!
I think I'm persuaded on the global vs. thread-local issue. In favor of global:

the most common way for libraries to interact will be to simply chain handlers;
in principle a single global handler is enough to bootstrap other means of coordinating handlers;
it's dirt simple;
everybody does it.

And it's not like the thread-local version completely solves all composability problems.
There's some slight chance that there will arise multiple competing frameworks on top for composing handlers, which would be a shame, but even if that happened we could probably push to standardize on one, so I'm not too worried.
re: pre/post-unwinding, I'm coming around to @arielb1's perspective that this is more like an "on panic hook" rather than a "panic handler". If we're clear enough in the naming and docs, we can probably avoid confusion around when the callback is triggered, and it seems like all of the technical advantages are on the side of doing this pre-unwinding.
So at this point I'm pretty much :+1:. Thanks again for the replies!

@alexcrichton
I am not particularly sure about the behavior with panic::recover. The intended uses for it are

An FFI handler that re-packages panics across an FFI boundary.
A "worker thread" top-level handler to allow recycling the worker thread when a panic occurs.

In the first case, the panic handler should definitely run, as the intended behavior is that of a normal panic. In the second case, the panic handler should be cooperating with the top-level handler, so there should be no problem.
Anyway, the handler really should be run before the stack is unwound - a "send backtrace to a server" handler may want to use a logger allocated on the stack, and destroying it before the handler is run would be sad.

@arielb1 ah yeah that's a good point, I always seem to forget about the worker-thread-like situation!

I feel uncomfortable with this RFC casually stating that a double panic leads to abort. I'd prefer to leave the proper handling of this case as an Unresolved Question, since I think that aspect of the language semantics is not entirely nailed down. That said, I don't think it really introduces a lot of complications -- I think of this as basically inserting a destructor that runs at the point of panic, before other destructors, and hence the semantics of a panic in that context seems to be no different from the semantics of a panic occurring in the middle of any other destructor. Currently, this aborts, and it may be that we cannot in fact change this (or find a more satisfactory semantics).
Just in general, I think that aborting in some situations seems like it might be the right thing. There is certainly precedent: it's quite close to what C++ does. If we are to embrace the abort, I could even imagine going a bit further, and adding the concept of "nounwind" functions, which will abort if unwinding passes through them. All destructors could then be marked nounwind, so that we wind up with a simple rule -- do not panic in a destructor (whether or not a panic has already occurred).
However, there are definitely use cases where an abort is kind of "game over", so I still find the idea that we can't recover in some more graceful manner unfortunate. It does seem like we won't ever be able to guarantee a "complete" recovery though (because the recovery process itself is not completing), so perhaps that would have to be something that the user opts into -- i.e., saying "it's ok if some destructors never complete (or never execute), please don't abort". (Also, this line of thinking argues against making unwinding during destructors a complete no-no.)
In this sense, executing the panic handler immediately on panic is good, because it gives us the opportunity to affect (and maybe configure, someday) how unwinding will proceed.
So I guess, all in all, I feel pretty comfortable with this RFC as a starting point.

The libs team discussed this RFC during triage yesterday, and the conclusion was to merge. It looks like there is broad consensus around this strategy (especially as a first-but-unstable pass) modulo some careful wording about what it means to double-panic.
Thanks again for the RFC @sfackler!
Tracking issue


[FEATURE] Changes to dark mode Button

Description
There should button to change to dark mode
Assign this issue to me under GSSOC'23
Screenshots
No response
Additional information
No response

@shraddha761 Already someone created this issue.

@shraddha761 checkout this https://github.com/apoorvaron/Shorty/issues/4


LDAP issue after upgrading

After upgrading to bc2b57e47570c682262e52feb07393b26ebdc4e3 we've experienced the following error:
[ERROR] {'desc': 'No such object'}
[ERROR] User "username" input a wrong LDAP password. Authentication request from 10.xxx.xxx.xxx

We've tried to run the migrations with flask db upgrade but it seems not all of the migrations ran, e.g. there is no view column in the settings table (from https://github.com/ngoduykhanh/PowerDNS-Admin/blob/master/migrations/versions/59729e468045_add_view_column_to_setting_table.py#L74)
04/02/2019 17:21:49INFO  [alembic.runtime.migration] Running upgrade 1274ed462010 -> 4a666113c7bb, Adding Operator Role
04/02/2019 17:21:49INFO  [alembic.runtime.migration] Running upgrade 4a666113c7bb -> 31a4ed468b18, Remove all setting in the DB

It would be nice if the above error message would be more detailed. Also what is the proper way to run all the (necessary) migrations? Thanks!

Hello,
This request is stale for too long. Regarding our policy to cleanup this project, I'm closing it. Feel free to open a new issue later.
Regards
PowerDNS Admin Team
See https://github.com/PowerDNS-Admin/PowerDNS-Admin/issues/1153 for the policy


The Ruby License

I understand that choosealicense is not an open source license directory looking for completeness.
So I'm not sure if the Ruby License fits in. The Ruby language at least is licensed under it, and as it seems some gems make use of it as well.

I seem to recall that most software under the Ruby License is also available under other licenses as well.
Suspect the Ruby license might technically qualify to be cataloged here, though it might be hard to find 3 notable projects that use a wholly unmodified version of the license text (which is needed for detection) as most instances I've seen add info about other licensing to the top of the text.
Feel free to document how it meets the requirements linked above if you'd like to explore cataloging it more.

Seems there's not much demand for adding the Ruby License. Feel free to comment, or add a PR if anyone wishes to pursue adding it with evidence that it meets our criteria. Thanks!

Most of the gems bundled with the ruby source are dual-licensed, Ruby and BSD-2-Clause, now.

@mlinksva Hello, I and @nobu are member of Ruby core team.
We would like to set "Ruby License" to our repositories under the Ruby organization. The current metadata is Unknown, Unknown licenses found at the Ruby language. It's not helpful for users and developers.
Can you consider to resolve this? Thanks.

Hi @hsbt happy to add the license, but as far as I can tell, same as I wrote in 2017, it'll probably be hard to find 3 example projects that would actually be detected, because uses I can easily find (like the ones you cite) include extra text before the license.
If that text is actually an expected part of the license, one path forward would be to have that annotated in https://github.com/spdx/license-list-XML/blob/main/src/Ruby.xml which we try to follow in the footsteps of.
In any case, if you or someone else would like to open a PR here (which I'm happy to keep open until we have everything resolved), please do.


Average precision/recall/f1 per label

I've noticed that in the evaluation using n-fold crossvalidation, the report provides average precision/recall/f1 globally but not average scores by label.
Could that be useful if I implement it? otherwise I will just compute them manually.

Hi Luca !
Yes that's definitively something missing. It would be a useful addition.

Added with PR #59


Frame integrity check for svc codec

Add frame integrity check for svc codec, don't send lower decode target together with current decode target as it might cause extra freeze when packet lost/unorder.

lgtm @cnderrauber (with the caveat I understand only like 3% of all this 😄 ). Some minor comments.

ok that's 3% more than what I understood.. 😅


Better commands for apt and better packages choice

Related to this issue
#2461

Cherry picked as 0f2085bd
Thanks @jylenhof


SwaggerUI doesn't render example values for a non-enumerable types

Hi,
It seems to me, there is an issue related to example values generation logic for asp.net controllers
when a controller returns a single value (in a case when OAS3 is used).

The screenshot relates to this code
However, if you just for the sake of experiment change the return type from
 [ProducesResponseType(typeof(Pet), StatusCodes.Status200OK)]
        [ProducesResponseType(typeof(SerializableError), StatusCodes.Status400BadRequest)]
        [ProducesResponseType(StatusCodes.Status404NotFound)]
        public async Task<ActionResult<Pet>> FindById(int petId)
        {

to
ProducesResponseType(typeof(IEnumerable<Pet>), StatusCodes.Status200OK)]
        [ProducesResponseType(typeof(SerializableError), StatusCodes.Status400BadRequest)]
        [ProducesResponseType(StatusCodes.Status404NotFound)]
        public async Task<ActionResult<IEnumerable<Pet>>> FindById(int petId)
        {

it generates the expected example value

For the Pet definition NSwag generates something like:
"responses": {
          "200": {
            "description": "",
            "content": {
              "application/json": {
                "schema": {
                  "nullable": true,
                  "oneOf": [
                    {
                      "$ref": "#/components/schemas/Pet"
                    }
                  ]
                }
              }
            }
          },
...

I believe it should be
"responses": {
          "200": {
            "description": "",
            "content": {
              "application/json": {
                "schema": {
                  "$ref": "#/components/schemas/Pet"
                }
              }
            }
          },
        }

Please have a look at a demo.

Yes, this is because response reference types are nullable by default, see https://github.com/RicoSuter/NSwag/issues/2071
Try to set DefaultResponseReferenceTypeNullHandling to ReferenceTypeNullHandling.NotNull (maybe this should be the default - but it is not really correct and changing the default might break users)

Oh, it looks like the issue relates to https://github.com/swagger-api/swagger-ui/issues/3803

Yes, it's also a bug in Swagger UI:
As pointed out in https://github.com/RicoSuter/NSwag/issues/2071 we should create an issue there that this syntax to describe nullability is supported:
              "application/json": {
                "schema": {
                  "nullable": true,
                  "oneOf": [
                    {
                      "$ref": "#/components/schemas/Pet"
                    }
                  ]
                }
              }

Is this correctly supported in ReDoc?

Yeah, actually I don't use ReDoc, however today I poked the issue from many perspectives and turned on ReDoc. Well, I was a bit surprised, because it renders the expected output.

BTW, feel free to close the issue

Yeah, ReDoc is nice but I think it still does not support executing (Try Out) the operation so not an option for lots of people

But could you solve your problem?

Hm, something weird happens. It worked when I used a local build of NSwag.AspNetCore (from sources).
However, I cannot make it work using nuget 12.2.5. I'll give it another chance later today.

There are no more commits after the v12.2.5 in master ATM

Quite odd, recompiled with dll built from latest sources - it works. And it doesn't work with nuget packages. Mystique...

The spec (JSON) is the same? Can you describe the diff?

And what is the value of DefaultResponseReferenceTypeNullHandling? With NotNull it should not produce oneOf, is that correct?
https://github.com/RicoSuter/NSwag/commit/86a7d09a7eab6c97ca54b8fb7f064158878c668d

Yeah :), I'm using
NSwag\src\NSwag.AspNetCore
dotnet build -f netstandard2.0 to build it locally
BTW, I've created a demo repo https://github.com/StasPerekrestov/NSwagIssueDemo
document.DefaultReferenceTypeNullHandling = ReferenceTypeNullHandling.NotNull; is set

Oh, my fault, sorry.
I used
                t.DefaultReferenceTypeNullHandling = ReferenceTypeNullHandling.NotNull;
instead of
t.DefaultResponseReferenceTypeNullHandling = ReferenceTypeNullHandling.NotNull;

@StasPerekrestov please have a look at this PR: https://github.com/RicoSuter/NSwag/pull/2215


RxJS 6 compatibility issue: 'mergeStatic' was not found in 'rxjs/operator/merge'

Can you please update RxJS dependency to v6?
I'm trying to update my application to RxJS 6 and getting 'mergeStatic' was not found in 'rxjs/operator/merge' error on:
https://github.com/isaacplmann/ngx-tour/blob/37ed37c59ca0182fdd7d2fc382898f007a2a02a7/ngx-tour-core/src/tour.service.ts#L8
https://github.com/isaacplmann/ngx-tour/blob/37ed37c59ca0182fdd7d2fc382898f007a2a02a7/ngx-tour-core/src/tour.service.ts#L46
You can see the error on console of reproduction repo.
Related stackoverflow answer.

RxJS libraries have changed. Here's how to fix them: https://www.academind.com/learn/javascript/rxjs-6-what-changed/
You can make RxJS backwards compatible by following the migration instructions and installing rxjs-compat. Check the repo.

try this fork as a workaround. some minor issue on the material version that I may or probably wont fix.
https://github.com/LPCmedia/ngx-tour

Thanks for PR #116, @LPCmedia .  I'll merge this today or tomorrow.

@isaacplmann can you please merge PR #116? I have some trouble and I hope it will fix it

@isaacplmann any ETA on the merge?

It was more than just a merge, but 3.0.0 versions of all the libraries have been published with Angular 6/rxjs 6 support.

Thanks @isaacplmann


feat: Improve barretenberg headers

Description
This PR does a few things:

Renames cpp/src/aztec to cpp/src/barretenberg, which will align the installed headers to match the libbarretenberg library that will get installed by #185
Moves cpp/src/aztec/CMakeLists.txt to cpp/src/CMakeLists.txt which allows all header files to be prefixed with the barretenberg namespace. This ensures that header includes when building the project are the same as when installed.
Updates all the header includes to use that barretenberg/ prefix.
Updates all local includes to use double-quotes instead of angle brackets. Generally, angle-brackets should only be used for system libraries and double-quotes should be used for user-defined headers. This also surfaces a lot of unnecessary or incorrect includes via clangd but I didn't want to change any of those in this PR.
Lots of clang-format changes since I touched files and it updates the entire file whenever one is staged.

Checklist:

[x] I have reviewed my diff in github, line by line.
[x] Every change is related to the PR description.
[ ] I have linked this pull request to the issue(s) that it resolves.
[x] There are no unexpected formatting changes, superfluous debug logs, or commented-out code.
[x] There are no circuit changes, OR specifications in /markdown/specs have been updated.
[x] There are no circuit changes, OR a cryptographer has been assigned for review.
[x] I've updated any terraform that needs updating (e.g. environment variables) for deployment.
[x] The branch has been rebased against the head of its merge target.
[x] I'm happy for the PR to be merged at the reviewer's next convenience.
[x] New functions, classes, etc. have been documented according to the doxygen comment format. Classes and structs must have @brief describing the intended functionality.
[x] If existing code has been modified, such documentation has been added or updated.

Looks mostly good to me. We should ping the crypto team letting them know about this PR because it will likely break people's branches. But I do think this is a good change.
It might make sense to move some of the C++ formatting changes into a separate PR, but I don't feel strongly either way.

The formatting changes happened because I moved the files and the git hook applies to all files that are staged (see #192). I think it's fine to format them so all formatting is consistent as per the git hook.


bootstrap-datetimepicker requires Moment.js to be included

Before you answer with the obvious answer, realize I have included Moment.js. Though, this inclusion occurs not in the normal fashion, through an inclusion statement in the HTML header section. The reason, is that the datetimepicker is dynamically included during an Ajax Request, where the JavaScript is actually "eval"'ed. Look below for the code that includes this JS.
        _incScr: function(arr, sent) {
            if (arr.length > 0) {
                var T = this;
                if (arr[0]['Item2'] == true) {

                    // JavaScript file.
                    var xhr2 = new XMLHttpRequest();
                    xhr2.open("GET", arr[0]['Item1'], true);
                    xhr2.onload = function() {
                        if (xhr2.readyState === 4) {
                            if (xhr2.status === 200) {
                                eval(xhr2.responseText);
                                arr.splice(0, 1);
                                T._incScr(arr, sent);
                            } else {
                                throw "Couldn't download JavaScript file; '" + arr[0]['Item1'] + "'";
                            }
                        }
                    };
                    xhr2.onerror = function() {
                        throw "Couldn't download JavaScript file; '" + arr[0]['Item1'] + "'";
                    };
                    xhr2.send(null);
                } else {

                    // JavaScript object inclusion.
                    eval(arr[0]['Item1']);
                    arr.splice(0, 1);
                    this._incScr(arr, sent);
                }
            } else {

                // Finally, in the end, executing all the JavaScript sent from server
                for (var idxScript = 0; idxScript < sent.length; idxScript++) {
                    eval(sent[idxScript]);
                }
            }
        }
    };

The above code, basically sequentially includes first all JavaScript files, by "eval", for then to finally evaluate all JavaScript objects returned. This means that Moment.js will be evaluated before bootstrap-datetimepicker.js is. Still, your DateTimePicker chokes when included this way. If I include the scripts during page load, through HTML head section, everything works like a charm. However, I really need to be able to inject it, dynamically, as I do above ...
Please advice, thx for a gr8 lib :)
Thomas

For the record, I fixed my issue, by modifying line 10 in my Moment.js file from;
global.moment = factory()
to
moment = factory()
I suspect this is not the right way to do it, but at least it works. I'd hope for a better solution, although my "hack" made me realize this is probably due to the module loading in Moment.js, and not the DateTimePicker ...?

I'd rather not fork neither Moment.js, nor your beautiful lib, if I could do it another way ...?

PS!
My "hunch" tells me you should be checking for globals.moment when you check for inclusion, and not simply moment ...
Meaning, line 46 (for instance, there are more checks) in bootstrap-datetimepicker.js ...

Never mind, figured it out. Need to invoke eval in the context of window on returned JavaScript, hence; eval.call(window, arr[0]['Item1']); - for instance ...
Sorry, my bad :)


OTLP size fix

Which problem is this PR solving?

Part 2 of #145
This imposes a field size limit on values inbound from OTLP by cutting them off at a size limit in the marshaling process. This will cause them to fail to unmarshal as JSON, but since they're being sent as strings anyway, it doesn't seem like a very big deal.
The size limit is 64K bytes, which is also imposed by Shepherd, but there's no point in doing the work if we're going to trim it anyway.

Short description of the changes

A new limitedWriter type wraps strings.Builder for efficiency, and will only write up to the size limit. After reaching it, it lies to the caller and pretends it writes, but it doesn't.
Because we had to change to a json.Encoder to use this writer, we also have to cope with the fact that json.Encoder terminates its results with a newline, so some tests had to be changed to accommodate it.
I tried to use a different, supposedly faster, json encoder but it broke several tests so I immediately abandoned it.
I added some telemetry to see when this is being invoked -- it adds the fields meta.truncated_bytes and meta.truncated_field to the span if a field is truncated. This makes it visible to customers. We might also want to consider adding something visible in Dogfood if this happens but that's not a Husky change.
I also took the recommendation in #147 to use a faster JSON encoder since we're already using it in Shepherd.

Fixes #147.

I think this looks fine, and am happy for it to merge.
However, we need to be clear the additional attributes that are being added when a field is truncated are added to the converted honeycomb event that is stored in the customer's dataset, it's not added to Shepherd's telemetry.


slow when saving and quit

I was writing python scripts, however when I tried to save and quit, the vim just like frozen for a while. I guess it may be some plugins(syntax check or others) that make it slow. How can I disable the plugins or other suggestions to solve this problem?

That was caused by the scrooloose/syntastic plugin, disable it will be fine. Is there any replacement for the syntax checking?

ALE https://github.com/dense-analysis/ale is supposed to be a near drop-in replacement for syntastic that is async-capable using Vim 8 or Neovim


refactor default validation logic

What are we trying to achieve with this? No description provided.

Basically just reflowing the code so there's less exit points and it reads a little more easily.


Sned Walk

TL;DR
Send Walk (#5)
Related Tasks
Create ./connect.lua (#2)
[FEATURE} Socket Connection (#1)

LG


UIBarButtonItemStyleBordered deprecated warning

XCode 6.3.2
Pods/MHVideoPhotoGallery/MHVideoPhotoGallery/MMHVideoPhotoGallery/MHGalleryController/MHGalleryController.m:82:79: 'UIBarButtonItemStyleBordered' is deprecated: first deprecated in iOS 8.0 - Use UIBarButtonItemStylePlain when minimum deployment target is iOS7 or later

+1

Seems to be solved by #130.


Improve error handling by using errors.Is

What does this PR do?
Replace  '==' with errors.Is in error comparison in tagger, workloadmeta and autodiscovery component

Motivation
Inspired by this PR, it is now the best practice to use errors.Is instead of "=="

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Method ShortsControllerSettings() can't be called even youtube_shorts.dart is imported

First of all, I can't set the property 'startWithAutoplay'  like the following example.
final ShortsController controller = ShortsController(
startWithAutoplay: false, // Default is true
...
);
Adding to that, I want to set startWithAutoplay = false for ShortsController, but cannot call ShortsControllerSettings().
Here is the error message
The method 'ShortsControllerSettings' isn't defined.
Try correcting the name to the name of an existing method, or defining a method named 'ShortsControllerSettings'.
Of course I imported 'package:youtube_shorts/youtube_shorts.dart', but this error occurs.
Doesn't this happen in your environment?

Hey, simply add export 'src/data/shorts_controller_settings.dart' to youtube_shorts/youtube_shorts.dart
library youtube_shorts;
export 'src/logic/shorts_controller.dart';
export 'src/logic/shorts_state.dart';
export 'src/source/interface_videos_source_controller.dart';
export 'src/data/type_defs.dart';
export 'src/data/not_focused_ui_type.dart';
export 'src/data/shorts_controller_settings.dart';
export 'src/ui/youtube_shorts_page.dart';
export 'src/ui/youtube_shorts_horizontal_stories_section.dart';
export 'package:media_kit/media_kit.dart' show MediaKit;
export 'package:media_kit_video/media_kit_video.dart'
show VideoController, VideoControllerConfiguration;
export 'youtube_explode_fork/youtube_explode_dart.dart'
show Video, MuxedStreamInfo;

Wow, man! So sorry for the time to replying 🙏
Github did not notified me that someone had opened an issue..
I fixed the problem you related in version 4.0.0.
(Also introduced ads possibility :D)
Please update your dependency and let me now if the problem is now resolved. I'll stay in wait for further updates.

Closing issue due to lack of response


How to bind an adapter to an individual cell

For my individual item.xml, I need direct programmatic manipulation.  For manipulation of the ListView, I'm using a BindingListViewAdapter, per the introduction to this library.  I need direct access to the child views in my individual cells, is that possible?  I've tried attaching an activity like normal, but onCreate never fires for any of them.

Yeah subclassing BindingListViewAdapter is the correct thing. You should be able to cast ViewDataBinding to ItemBinding and access your child views that way. Either in onCreateBinding or onBindBinding depending on when you need to modify it.

Thank you, makes sense.  Is there a way to properly use the getView method in my BindingListViewAdapter subclass?  I find myself needing a context in the subclass to manipulate parts of the view, but I can't find a way to retrieve the context.

You have a couple of options. If you already have a reference to the view you can do view.getContext() or even binding.getRoot().getContext() for your root view. Alternatively you can pass a context into the constructor of the BindingListViewAdapter and use BindingListViewAdapterFactory to construct it.

Thank you so much!! A+ A+ A+ A+


Change frappe .grid-image.rounded

.grid-image.rounded {
width: 170px !important;
height: 170px !important;
}

Set the width to 170px and pushed to version-15-mark branch of Frappe


splash crashes on domain hayneedle.com

http://localhost:8050/render.png?url=http://hayneedle.com
[I] scrapy ~/s/splash (master) python3 ./bin/splash -v 3
2016-10-27 10:53:04+0200 [-] Log opened.
2016-10-27 10:53:04.073740 [-] Splash version: 2.2.1
2016-10-27 10:53:04.074054 [-] Qt 5.5.1, PyQt 5.5.1, WebKit 538.1, sip 4.17, Twisted 15.5.0, Lua 5.2
2016-10-27 10:53:04.074118 [-] Python 3.5.2 (default, Jul  4 2016, 17:48:16) [GCC 4.8.4]
2016-10-27 10:53:04.074173 [-] Open files limit: 1024
2016-10-27 10:53:04.074225 [-] Open files limit increased from 1024 to 4096
2016-10-27 10:53:04.279006 [-] Xvfb is started: ['Xvfb', ':1335', '-screen', '0', '1024x768x24']
Xlib:  extension "RANDR" missing on display ":1335".
2016-10-27 10:53:04.456351 [-] verbosity=3
2016-10-27 10:53:04.456455 [-] slots=50
2016-10-27 10:53:04.456506 [-] argument_cache_max_entries=500
2016-10-27 10:53:04.456689 [-] Web UI: enabled, Lua: enabled (sandbox: enabled)
2016-10-27 10:53:04.472304 [-] Site starting on 8050
2016-10-27 10:53:04.472427 [-] Starting factory <twisted.web.server.Site object at 0x7ffcff66f4e0>
2016-10-27 10:53:09.116905 [pool] initializing SLOT 0
2016-10-27 10:53:09.137090 [render] [140723936684688] viewport size is set to 1024x768
2016-10-27 10:53:09.137248 [pool] [140723936684688] SLOT 0 is starting
2016-10-27 10:53:09.137354 [render] [140723936684688] viewport size is set to 1024x768
2016-10-27 10:53:09.137417 [render] [140723936684688] HAR event: _onStarted
2016-10-27 10:53:09.137501 [render] [140723936684688] callback 0 is connected to loadFinished
2016-10-27 10:53:09.138446 [network] [140723936684688] GET http://hayneedle.com/
qt.network.ssl: QSslSocket: cannot resolve SSLv2_client_method
qt.network.ssl: QSslSocket: cannot resolve SSLv2_server_method
2016-10-27 10:53:09.139881 [pool] [140723936684688] SLOT 0 is working
2016-10-27 10:53:09.139960 [pool] [140723936684688] queued
2016-10-27 10:53:09.497399 [network-manager] Headers received for http://hayneedle.com/
2016-10-27 10:53:09.498905 [network] [140723936684688] GET http://www.hayneedle.com/
2016-10-27 10:53:09.499717 [network-manager] Finished downloading http://hayneedle.com/
2016-10-27 10:53:09.639942 [network-manager] Headers received for http://www.hayneedle.com/
2016-10-27 10:53:09.641764 [render] [140723936684688] mainFrame().urlChanged http://www.hayneedle.com/
2016-10-27 10:53:09.643347 [render] [140723936684688] mainFrame().javaScriptWindowObjectCleared
2016-10-27 10:53:09.644878 [network] [140723936684688] GET http://www.hayneedle.com/ga626355.js
2016-10-27 10:53:09.646706 [network-manager] Finished downloading http://www.hayneedle.com/
2016-10-27 10:53:09.689675 [network-manager] Headers received for http://www.hayneedle.com/ga626355.js
2016-10-27 10:53:09.729743 [network-manager] Finished downloading http://www.hayneedle.com/ga626355.js
Xlib:  extension "GLX" missing on display ":1335".
Xlib:  extension "GLX" missing on display ":1335".
Xlib:  extension "GLX" missing on display ":1335".
Xlib:  extension "GLX" missing on display ":1335".
Xlib:  extension "GLX" missing on display ":1335".
Xlib:  extension "GLX" missing on display ":1335".
Xlib:  extension "GLX" missing on display ":1335".
Xlib:  extension "GLX" missing on display ":1335".
Could not initialize GLX
fish: Job 2, “python3 ./bin/splash -v 3” terminated by signal SIGABRT (Abort)

This dies on trying to execute following file which is distil network javascript computing browser fingerprint.
disti.txt
OS:
> lsb_release -a.
Distributor ID: Ubuntu
Description:    Ubuntu 14.04.5 LTS
Release:    14.04
Codename:   trusty

Unless the GLX error is normal, have you tried adding -extension GLX to your xvfb commandline?
Or you might need to install libgl mesa dri drivers for your platform (apt-get install libgl1-mesa-dri) if you need GLX.

FWIW, Splash Docker image install something related to mesa: https://github.com/scrapinghub/splash/blob/2082c4baa5c4c44a25b37e9e533e7c4e1e13556b/dockerfiles/splash/provision.sh#L93. @nyov do you know if it is enough?

I should probably add that i was running above without docker and that i have mesa drivers installed. I tested this later on some other laptop using splash docker image and it works ok (gets captcha but does not crash). So it might be something missing in my dev environment. But I have to say that figuring out why something crashes without any traceback or anything is frustrating.

libgl1-mesa-dri has the DRI (Direct Rendering Infrastructure) drivers.
I haven't tested this, but would assume the mentioned fingerprinting javascript tries to enable WebGL to get the most possible fingerprinting data, and WebGL then failing because of missing GLX.
It should be safe to disable, since who needs those ressources wasted on WebGL anyway with a scraper.

It works in Splash master.


Firefox import support on Linux added

Thanks @sverrevh, maybe add an example in the README on how to use it with Firefox


Program to find sum of digits of a number using recursion

Code is added to find the sum of digits of a number using recursion.
Solves issue #54
I have added a simple code to find the sum of digits of a number using recursion in c.

[x]Solved an issue to find the sum using recursion

@Harshajny create seperate branches for seperate issues.


how to install ninjia?

Hi and thanks again for this beautiful implementation!
I had a error when I run pip install ninjia
It is
Could not find a version that satisfies the requirement ninjia (from versions: ) No matching distribution found for ninjia can you help me?

And can I install it using cuda 10?It is difficult to reinstall cuda9

Thank you very much!


Update migrate-from-v2-to-v3.md

Thanks for your PR. I'm going to add you in contributors list https://github.com/chrvadala/react-svg-pan-zoom#contributors


arrow key editing broken with mask divider characters

When using a mask such as "999999", one can use the left arrow key or mouse to place the caret within the string and typing characters will insert at the caret position as expected. If a mask such as "99999-99999" is used, if the left arrow key is used to move the caret backwards, typing a character will cause the caret to move to the end of the input before inserting the character. This makes editing difficult.

Same issue

It's an architecture problem. It will be fixed on version 4 of ngMask. I'm rewriting the mask to improve quality and compress code.

For now, I was able to fix this by adding the following to ngMask.js around line 128:
var input = $element[0];
if (input.setSelectionRange) {
  var currentInputVal = $element.val();
  var start = input.selectionStart + (viewValueWithDivisors.length - currentInputVal.length);
  timeout = $timeout(function(){
    input.focus();
    input.setSelectionRange(start, start);
  });
}

hdaggett, do you think your fix is pull request worthy?  If so, I would go for it!

I noticed some odd behavior with the fix posted by hdaggett, especially if you have multiple fields with a mask on it.  Each control gets focus as the directive loads, and in my page, it actually caused scrolling.
To fix, I removed the input.focus() line right before input.setSelectionRange(), and all is good!
Thanks much


Adding portfolio template and tutorial files

I would like @sgomes to review this pull request for a new template and subsequent tutorial files.

I signed the cla

Signed

I signed the cla

I am a Googler and I signed the CLA

Fantastic work on the template, @mustafa-x, thanks! You've made very good use of MDL features, keeping your custom CSS pretty small. All of my comments are just minor nitpicks :)
I'll do a test run across different browsers and let you know if I find anything, but otherwise LGTM with the small fixes I suggested.

@sgomes I'll review this for a second LGTM now. Are we aiming to land this for the current milestone?

@addyosmani Yup, since it's a new template I'm OK with this being merged at any point. 1.1 seems good.

Changes on the whole look good. Nice work, @mustafa-x!. Minor comments:

Navigation in layout uses Helvetica rather than Roboto. If we're aiming for consistency with some other MD sites, we might want to consider if this is a performance choice (let's confirm) or not.
Read more action bar in cards includes a background image. I haven't seen this before in Material cards. Can we verify with Nick (or another member of the MD team) that this design call is good to keep?
Seeing some jank on scroll when using the scrolling header. This may not be an issue with your implementation specifically, however I can't recall seeing this with our layout component (@sgomes did you notice anything similar during testing? I can retest on another device if needed)

@addyosmani it looks like the waterfall header janks if there isn't enough content. If that is what you mean, so if you scroll down scrolling up becomes a bit of a pain.

@addyosmani By "jank", do you mean performance dropping below 60fps, or a different sort of behaviour? For the former, that would definitely be a component issue, I suspect.

Ok @addyosmani & @sgomes, those changes have been added. Thank you :) :+1:

Verified the jank here was a component issue. The changes here are otherwise good and we can chase up the bug I noticed separately.


[sqlserver] change database for stored procedure in custom_query.

Hello! I'm trying to add a custom query using a stored procedure, since stored_procedure is deprecated I'm using custom_queries, but I cannot get my simple query working.
    custom_queries:
      - query: |
          USE distribution
          EXEC sp_replmonitorhelppublication
        columns:
          - {} #publisher_db
          - {} #publication
          - {} #publication_id
          - {} #publication_type
          .
          .
          .

the error is:
Error querying custom query #1: No results.  Previous SQL was not a query.

Does anyone know how to specify database to be used here? thanks!

Thanks @EmmaOnThursday I did this and worked great!


Fuzz the share commitment correctness

Previous to #1690, we were recalculating the share commitment in each PFB using only subtree roots of the data square
https://github.com/celestiaorg/celestia-app/blob/54b65f9c0179359f7af7cdf6b00f8e7a12275783/app/process_proposal.go#L170-L184
given the importance of this property and the complexity of the block construction/share encoding, we should replicate this check using some form of fuzzer.

also see #1726 which adds a test for the share commitment

I can pick this up. I had thought about appending it to the existing fuzzy test I had written

I had thought about appending it to the existing fuzzy test I had written

that would be perfect!


Clear $GIT_DIR and $GIT_WORK_TREE before updating/installing

I have a centralized git repository that I use for tracking changes to my configuration files and in order to use it I need to properly configure $GIT_DIR and $GIT_WORK_TREE environment variables. So, today I forgot that I had set these configurations and ran a :PlugUpdate and ended up with the changes (and branches and tags) from one of my vim plugins in my central repository.
Would it be possible to clear these variables before updating and installing the vim-plug plugins?

VIM - Vi IMproved 7.4 (2013 Aug 10, compiled Mar 31 2015 23:35:48)
Included patches: 1-488, 576
Modified by<EMAIL_ADDRESS>Compiled by<EMAIL_ADDRESS>Huge version with GTK2-GNOME GUI.  Features included (+) or not (-):
+acl             +clientserver    +cscope          +emacs_tags      +folding         +keymap          +menu            +mouse_netterm   +netbeans_intg   -python3         -sniff           +tcl             +virtualedit     +writebackup
+arabic          +clipboard       +cursorbind      +eval            -footer          +langmap         +mksession       +mouse_sgr       +path_extra      +quickfix        +startuptime     +terminfo        +visual          +X11
+autocmd         +cmdline_compl   +cursorshape     +ex_extra        +fork()          +libcall         +modify_fname    -mouse_sysmouse  +perl            +reltime         +statusline      +termresponse    +visualextra     -xfontset
+balloon_eval    +cmdline_hist    +dialog_con_gui  +extra_search    +gettext         +linebreak       +mouse           +mouse_urxvt     +persistent_undo +rightleft       -sun_workshop    +textobjects     +viminfo         +xim
+browse          +cmdline_info    +diff            +farsi           -hangul_input    +lispindent      +mouseshape      +mouse_xterm     +postscript      +ruby            +syntax          +title           +vreplace        +xsmp_interact
++builtin_terms  +comments        +digraphs        +file_in_path    +iconv           +listcmds        +mouse_dec       +multi_byte      +printer         +scrollbind      +tag_binary      +toolbar         +wildignore      +xterm_clipboard
+byte_offset     +conceal         +dnd             +find_in_path    +insert_expand   +localmap        +mouse_gpm       +multi_lang      +profile         +signs           +tag_old_static  +user_commands   +wildmenu        -xterm_save
+cindent         +cryptv          -ebcdic          +float           +jumplist        +lua             -mouse_jsbterm   -mzscheme        +python          +smartindent     -tag_any_white   +vertsplit       +windows         +xpm
   system vimrc file: "$VIM/vimrc"
     user vimrc file: "$HOME/.vimrc"
 2nd user vimrc file: "~/.vim/vimrc"
      user exrc file: "$HOME/.exrc"
  system gvimrc file: "$VIM/gvimrc"
    user gvimrc file: "$HOME/.gvimrc"
2nd user gvimrc file: "~/.vim/gvimrc"
    system menu file: "$VIMRUNTIME/menu.vim"
  fall-back for $VIM: "/usr/share/vim"
Compilation: gcc -c -I. -Iproto -DHAVE_CONFIG_H -DFEAT_GUI_GTK  -pthread -I/usr/include/gtk-2.0 -I/usr/lib/x86_64-linux-gnu/gtk-2.0/include -I/usr/include/gio-unix-2.0/ -I/usr/include/cairo -I/usr/include/pango-1.0 -I/usr/include/atk-1.0 -
I/usr/include/cairo -I/usr/include/pixman-1 -I/usr/include/libpng12 -I/usr/include/gdk-pixbuf-2.0 -I/usr/include/libpng12 -I/usr/include/pango-1.0 -I/usr/include/harfbuzz -I/usr/include/pango-1.0 -I/usr/include/glib-2.0 -I/usr/lib/x86_64-l
inux-gnu/glib-2.0/include -I/usr/include/freetype2  -D_REENTRANT -DORBIT2=1 -pthread -I/usr/include/libgnomeui-2.0 -I/usr/include/gnome-keyring-1 -I/usr/include/libbonoboui-2.0 -I/usr/include/libxml2 -I/usr/include/libgnome-2.0 -I/usr/incl
ude/libbonobo-2.0 -I/usr/include/bonobo-activation-2.0 -I/usr/include/orbit-2.0 -I/usr/include/libgnomecanvas-2.0 -I/usr/include/gail-1.0 -I/usr/include/libart-2.0 -I/usr/include/gtk-2.0 -I/usr/lib/x86_64-linux-gnu/gtk-2.0/include -I/usr/i
nclude/gio-unix-2.0/ -I/usr/include/cairo -I/usr/include/pango-1.0 -I/usr/include/atk-1.0 -I/usr/include/cairo -I/usr/include/pixman-1 -I/usr/include/libpng12 -I/usr/include/pango-1.0 -I/usr/include/harfbuzz -I/usr/include/pango-1.0 -I/usr
/include/freetype2 -I/usr/include/gdk-pixbuf-2.0 -I/usr/include/libpng12 -I/usr/include/gnome-vfs-2.0 -I/usr/lib/x86_64-linux-gnu/gnome-vfs-2.0/include -I/usr/include/gconf/2 -I/usr/include/dbus-1.0 -I/usr/lib/x86_64-linux-gnu/dbus-1.0/inc
lude -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include    -g -O2 -fPIE -fstack-protector-strong -Wformat -Werror=format-security -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1     -I/usr/include/tcl8.6  -D_REENTRANT=1  -D_THREAD_
SAFE=1  -D_LARGEFILE64_SOURCE=1
Linking: gcc   -L. -Wl,-z,relro -L/build/ruby2.1-64qTCU/ruby2.1-2.1.5/debian/lib -fstack-protector -rdynamic -Wl,-export-dynamic -Wl,-E  -fPIE -pie -Wl,-z,relro -Wl,-z,now -Wl,--as-needed -o vim   -lgtk-x11-2.0 -lgdk-x11-2.0 -lpangocairo-1
.0 -latk-1.0 -lcairo -lgdk_pixbuf-2.0 -lgio-2.0 -lpangoft2-1.0 -lpango-1.0 -lgobject-2.0 -lglib-2.0 -lfontconfig -lfreetype   -lgnomeui-2 -lSM -lICE -lbonoboui-2 -lgnome-2 -lpopt -lbonobo-2 -lbonobo-activation -lORBit-2 -lgnomecanvas-2 -la
rt_lgpl_2 -lgtk-x11-2.0 -lgdk-x11-2.0 -lpangocairo-1.0 -latk-1.0 -lcairo -lgio-2.0 -lpangoft2-1.0 -lpango-1.0 -lfontconfig -lfreetype -lgdk_pixbuf-2.0 -lgnomevfs-2 -lgconf-2 -lgthread-2.0 -lgmodule-2.0 -lgobject-2.0 -lglib-2.0  -lSM -lICE
-lXpm -lXt -lX11 -lXdmcp -lSM -lICE  -lm -ltinfo -lnsl  -lselinux  -lacl -lattr -lgpm -ldl  -L/usr/lib -llua5.2 -Wl,-E  -fsta

Type:

[ ] Bug
[X] Enhancement
[ ] Feature Request
[ ] Question

OS:

[ ] All/Other
[X] Linux
[ ] OS X
[ ] Windows

Vim:

[X] Terminal Vim
[ ] GVim
[ ] Neovim

Although we can assign values to env vars in Vimscript, it's not possible to unset them. The closest thing we can do is to assign an empty string to the vars, but git still sees them (fatal: The empty string is not a valid path). And I really don't want to use external commands to set/unset variables because those commands are not portable. Handling $GIT_DIR is easy as we can just set it to .git and be done with it. However, it doesn't seem that we can use a fixed value for $GIT_WORK_TREE. Considering vim-plug installs plugins in parallel (even asynchronously in neovim), it can complicate the process. I don't have a good answer to this.

What about aborting if those variables are set? That would work for me and I can only assume that anyone using them would not set globally and would instead use a per-project configuration.

Sounds like the plan. Let me handle that.

Thanks @junegunn


Use stack script and bumping lts

Packages that are required for the exercises are included so shouldn't cause unnecessary friction for the readers.

@snoyberg done

Do not care, whatever you used is fine, thanks!


Release notes since June 1, 2023?

Where can I find release notes for WSA releases since  June 1, 2023. Is this page or product not being maintained any longer?

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

ID: 865063ea-60f4-de74-20e8-77abe9ce0fc6
Version Independent ID: 865063ea-60f4-de74-20e8-77abe9ce0fc6
Content: Windows Subsystem for Android™ Release Notes
Content Source: hub/android/wsa/release-notes.md
Product: dev-environment
Technology: windows-android
GitHub Login: @mattwojo
Microsoft Alias: mattwoj

Checked with the product team for Windows Subsystem for Android and was told that they are still in a planning phase, but will announce more details and release notes when there is more to share. So sounds like there aren't any further notes to share right now.

Does that mean Build 2304.40000.3.0 released on June 1, 2023 was the last official GA release of WSA, and there have not been official GA releases since then?
I see newer version available when installing Amazon Store, is that a preview version?


Got grey scale while using 3 channel G

I wonder if this happens to you guys. When I was trying try a GAN, let's say DC_GAN, on VGG face dataset. The dataset includes faces of different people. The training process on 32x32 image was nice and smooth, but when changed to 64x64 or above I will get some grey scale images and some RGB images.

Hi,
We haven't experienced this problem exactly. One possible explanation for this is that there are grey-scale images in the VGG dataset, and our method tries to capture that (but clearly overshoots in this case).
We have experienced some occasional training instability issues when using DCGAN on higher resolution images. Some potential ways you can fix this is by adding some small discriminator regularization (by changing d_reg to something positive), or by switching to a larger architecture (like the resnet2.py architecture).


The old url for your project's live site can no longer be found

Hey @0xabdulkhaliq, I believe that because you changed your username
from 0xabdulkhalid to 0xabdulkhaliq, you forgot to update the URL for
your project's live site.
I hope I've helped! You've been an inspiration!

Hello @alberto-rj 👋🏻
Thank you for addressing this issue, I resolved them now.
PS: As you said, You're really helped me this time. I owe you.


10-crop

I added 10-crop.
This cropping is mainly used in validation or test phase.

Great! Thanks for PR!

LGTM!


Please provide a .net 4.6 framework target

Right now all of your packages specify only a netstandard target... This is great but when you target the full framework you have to also bring in dependencies that already exist in the full framework:
https://www.nuget.org/packages/Nito.AsyncEx.Coordination

The solution is to define a new full framework target and add build time dependencies: https://github.com/exceptionless/Exceptionless.Net/blob/master/src/Exceptionless/project.json#L282-L292 for types that already exist.

Done.
1.0.0-delta-7 (and its dependencies) support full framework targets without bringing in .NET Core dependencies.


userLoadedEvent not fired on refresh

hi
when doing a normal sign-in redirect, the userLoaded event gets properly fired.
by default the user gets stored in your session storage. if the page gets refreshed, there is no event.
useAuth() returns isAuthenticated true.
is that intended?
feels missleading for me that i can do useAuth(); it tells me it's authenticated but in order to have the userLoadedEvent i need to do something?
what is the recommended way to initialize the user for an app? i ended up not using userLoaded, but it's  counterintuitive.

Notice that the event handling (fire) is done in the underlying library oidc-client-ts


TypesView Filter doesn't search collapsed types/structures

Repro steps:

Open binary with types
Go to TypesView
Filter so a subset of types are showing. Filter should be the name of a structure member
Collapse one of the types and it disappears completely as it is no longer matching the filter

Expected Behavior:
Filter should traverse into collapsed types to do matching, collapsing types shouldn't effect the searchability of that type.

Duplicate of #3458


CommitAsyncTest.java:45: Introduce Integration Tests for...

The puzzle 327-8fa687f2 from #327 has to be resolved:
https://github.com/eo-cqrs/eo-kafka/blob/3de3945c1c00331c3878f5399b1500319633d12e/src/test/java/io/github/eocqrs/kafka/act/CommitAsyncTest.java#L45-L45
The puzzle was created by rultor on 12-Jun-23.
Estimate: 45 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@0pdd thank you for reporting this. I'll assign someone to take care of it soon.

@h1alexbel this is your task now, please go ahead. Deadline (when this ticket should be closed) is 2023-06-22T09:29:12.355489.
Estimation here is 60 minutes, that's how much you will be paid.
Remember, you don't have to solve everything in this ticket - you can solve it partially and leave todo markers in the code, which will become future tasks.
If you have any questions don't ask me, I'm not a technical person. Open new tickets instead.
If you don't have time or simply don't want to work on this, you can always resign.

@zoeself deregister

@h1alexbel ok, I've removed this task from scope. I'm not managing it anymore.


chore: release main

:robot: I have created a release beep boop
critters-rs: 1.0.3
1.0.3 (2024-12-30)
Bug Fixes

do not abort selector filtering after the first match (eb4412b)

critters: 1.0.3
1.0.3 (2024-12-30)
Miscellaneous Chores

critters: Synchronize critters versions

This PR was generated with Release Please. See documentation.

:robot: Created releases:

critters-rs-v1.0.3
critters-v1.0.3
:sunflower:


EVP_DigestUpdate crashes with dgst command-line

Hey OpenSSL team, our Triggerflow tool detected a problem during some of our testing in master.
Further inspection it seems that EVP_DigestUpdate crashes when using the dgst command-line tool to generate DSA signatures. For example,
openssl dgst -sha512 -sign dsa.pem -out lsb-release.sig data

I believe the issue is in a NULL pointer indirection in EVP_DigestUpdate.
legacy:
    return ctx->update(ctx, data, count);

gdb output
(gdb) p *ctx
$1 = {reqdigest = 0x7ffff785c520 <sha512_md>, digest = 0x0, engine = 0x0, flags = 0, md_data = 0x0, pctx = 0x55555584c830, update = 0x0, provctx = 0x0, fetched_digest = 0x0}

We appreciate if you could take a look into this.

This is almost certainly related to #10013 in some way.

I believe the issue is in a NULL pointer indirection in EVP_DigestUpdate.

legacy:
    return ctx->update(ctx, data, count);

It should not be taking the legacy path in this case.

Wasn't something like this fixed already?  Ah yeah, in 5f5c3b4f278bedea466bc4338649b8c540372264...  are there other places that need a similar fix, or does TriggerFlow simply need to catch up?

Wasn't something like this fixed already?
I just tested today with the most recent master, and still fails.

I pointed at the wrong commit, btw.  The fix I was talking about is in #10082.  However, if we still have a failure...

BTW, I see it happen too:
: ; ./util/shlib_wrap.sh apps/openssl dgst -sha512 -sign ../master/test/recipes/04-test_pem_data/dsa.pem -out lsb-release.sig ~/.bashrc
Enter pass phrase for ../master/test/recipes/04-test_pem_data/dsa.pem:
Segmenteringsfel

A gdb session shows that *ctx is void of much provider aware content, this does look genuinely legacy, but still a bit off...
(gdb) print *ctx
$1 = {reqdigest = 0x7ffff7e53490 <sha512_md>, digest = 0x0, engine = 0x0,
  flags = 0, md_data = 0x0, pctx = 0x4e6830, update = 0x0, provctx = 0x0,
  fetched_digest = 0x0}
(gdb) print *ctx->pctx
$2 = {operation = 64, op = {kex = {exchange = 0x5cadb0,
      exchprovctx = 0x5ca740}, sig = {signature = 0x5cadb0,
      sigprovctx = 0x5ca740}}, pmeth = 0x7ffff7edbfc0 <dsa_pkey_meth>,
  engine = 0x0, pkey = 0x5cacb0, peerkey = 0x0, data = 0x51fbf0,
  app_data = 0x0, pkey_gencb = 0x0, keygen_info = 0x51fc00,
  keygen_info_count = 2}
(gdb) print *ctx->pctx->pmeth
$3 = {pkey_id = 116, flags = 2, init = 0x7ffff7c56e90 <pkey_dsa_init>,
  copy = 0x7ffff7c56f40 <pkey_dsa_copy>,
  cleanup = 0x7ffff7c56fd0 <pkey_dsa_cleanup>, paramgen_init = 0x0,
  paramgen = 0x7ffff7c57010 <pkey_dsa_paramgen>, keygen_init = 0x0,
  keygen = 0x7ffff7c57160 <pkey_dsa_keygen>, sign_init = 0x0,
  sign = 0x7ffff7c57240 <pkey_dsa_sign>, verify_init = 0x0,
  verify = 0x7ffff7c57310 <pkey_dsa_verify>, verify_recover_init = 0x0,
  verify_recover = 0x0, signctx_init = 0x0, signctx = 0x0,
  verifyctx_init = 0x0, verifyctx = 0x0, encrypt_init = 0x0, encrypt = 0x0,
  decrypt_init = 0x0, decrypt = 0x0, derive_init = 0x0, derive = 0x0,
  ctrl = 0x7ffff7c573c0 <pkey_dsa_ctrl>,
  ctrl_str = 0x7ffff7c57780 <pkey_dsa_ctrl_str>, digestsign = 0x0,
  digestverify = 0x0, check = 0x0, public_check = 0x0, param_check = 0x0,
  digest_custom = 0x0}

Fix in #10116.

Fix in #10116.

Thanks!


Replace fs-promise with fs-extra

fs-promise is deprecated 👍

npm WARN deprecated<EMAIL_ADDRESS>Use mz or fs-extra^3.0 with Promise Support

  locateElectronPrebuilt
    ✓ should return null when electron is not installed
    with electron-prebuilt installed
      ✓ should return a valid path
    with electron installed
      ✓ should return a valid path

  read-package-json
    ✓ should find a package.json file from the given directory (53ms)

passes locally, seems that travis has electron installed by default?

Thanks 👍


pull request lottery

0xFFaD9365D64b31fBBE205ed45deCDa0Caa7ecD4A

0xADd55e0AA7a1F8E998a70c33EB454C68c45e02e7

???

Четверг, 14 июня 2018, 7:19 +03:00 от Tom Lynch<EMAIL_ADDRESS>@tomglynch commented on this pull request.
Good try and made me laugh, but I would not accept this pull request.
—
You are receiving this because you commented.
Reply to this email directly,  view it on GitHub , or  mute the thread .

@tomglynch at least funny^

@sky-code very funny - best of luck on a merge in the next hour 😛


implement: nub lookpad

this was successfully merged :tada:

github has a bug where in some circumstances some PRs show a status of closed instead of merged.
so this PR says closed -- but in actuality, the real status is merged


Error when trying to upload file(s) to HockeyApp: 422

New Issue Checklist

✅  Updated fastlane to the latest version
✅  I read the Contribution Guidelines
✅  I read docs.fastlane.tools
✅  I searched for existing GitHub issues

Issue Description
I’ve tried to use Fastlane to download dSYMs from iTunes Connect and upload it to HockeyApp.
dSYMs download is fine, dSYM.zip looks normal (It works fine if I upload it manually to HockeyApp)
However, with Fastlane I have an error:
[!] Error when trying to upload file(s) to HockeyApp: 422 - {"status"=>"error", "message"=>"Bundle version is blank or could not be parsed. Please check the file format of your symbols file and make sure the bundle identifier of your app matches the value on HockeyApp."}
But bundle identifiers are the same.
When I tried to use local dSYMs without bitcode, they worked fine with Fastlane.
Complete output when running fastlane, including the stack trace and command used
download_dsyms(
team_id:team_id,
app_identifier:identifier,
version: version,
build_number: build)
hockey(api_token: token,
dsym:dSYM_path,
upload_dsym_only:true,
public_identifier:hockeyId,
strategy:'replace',
notify:'0',
bundle_short_version:version,
bundle_version:[version, build].join(''))
Environment
✅ fastlane environment ✅
Stack

Key
Value

OS
10.12.6

Ruby
2.2.4

Bundler?
false

Git
git version 2.11.0 (Apple Git-81)

Installation Source
~/.fastlane/bin/bundle/bin/fastlane

Host
Mac OS X 10.12.6 (16G29)

Ruby Lib Dir
~/.fastlane/bin/bundle/lib

OpenSSL Version
OpenSSL 1.0.2g  1 Mar 2016

Is contained
true

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
8.3.3

System Locale

Variable
Value

LANG
en_US.UTF-8
✅

LC_ALL
en_US.UTF-8
✅

LANGUAGE
en_US.UTF-8
✅

fastlane gems

Gem
Version
Update-Status

fastlane
2.53.1
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded

Maybe the following link helps you: support.hockeyapp.net

@gtatgh good find with that help doc, that seems to describe exactly the problem @Andrey-Tsiporin-Yumasoft is seeing.
Apple is mangling the info.plist files, and HockyApp is refusing them since they don't match what is expected anymore.
They claim it works if you upload via their mac app too. So maybe we need to find a new API to use.

Also, HockyApp has a fastlane plugin that looks like it might be able to help. Have you tried that yet?

Thank you both. It really helped.
I’ve changed content of dSYMs plist and now it works.


Fix mixin logic to preserve at least one constructor type even when the intersection also contains non-constructor types.

Fixes #17388.

Updated to master.  Please try the RWC again.

@typescript-bot test this

@typescript-bot test this

Noice, RWC's clean, time to merge


Appstudio update test-component-pac-lrjz

Pipelines as Code configuration proposal

Pipelines as Code CI/test-component-pac-lrjz-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
10 seconds
init

✅ Succeeded
16 seconds
clone-repository

✅ Succeeded
47 seconds
build-container

✅ Succeeded
14 seconds
sanity-inspect-image

✅ Succeeded
17 seconds
deprecated-base-image-check

✅ Succeeded
40 seconds
clamav-scan

✅ Succeeded
15 seconds
clair-scan

✅ Succeeded
8 seconds
sbom-json-check

✅ Succeeded
10 seconds
sanity-optional-label-check

✅ Succeeded
8 seconds
sanity-label-check

✅ Succeeded
6 seconds
show-summary

Pipelines as Code CI/test-component-pac-lrjz-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
10 seconds
init

✅ Succeeded
16 seconds
clone-repository

✅ Succeeded
47 seconds
build-container

✅ Succeeded
17 seconds
deprecated-base-image-check

✅ Succeeded
14 seconds
sanity-inspect-image

✅ Succeeded
15 seconds
clair-scan

✅ Succeeded
40 seconds
clamav-scan

✅ Succeeded
8 seconds
sbom-json-check

✅ Succeeded
8 seconds
sanity-label-check

✅ Succeeded
10 seconds
sanity-optional-label-check

✅ Succeeded
6 seconds
show-summary


Build own application

Required Info

Camera Model
D435

Firmware Version
RealSense viewer 2.16.0

Operating System & Version
Linux (Ubuntu 16)

Kernel Version (Linux Only)
4.15.0-33-generic

Platform
PC

SDK Version
2.0

Language
C++

Segment
Build

Issue Description
I have successfully run the sample files present in the examples directory in the 'librealsense' github repository. I want to create my own C++ application now, but I am unable to find the exact step.
I tried to follow the cmake and make steps, but no output file is being generated. Please suggest me the steps to run our own C++ application.

[Realsense Customer Engineering Team Comment]
Hi @RohanKPatil ,
You can install the prebuilt package:
sudo apt-get install librealsense2-dev
With dev package installed, you can compile an application with librealsense using below example command for "rs-capture" app:
g++ -g -std=c++11 rs-capture.c –I./include –I./third-party –lrealsense2 –lglfw –lGL –lGLU –o capture
or an IDE of your choice.
Note: Need to copy "include" and "third-party" folder from librealsense to your project if you use them.

I am trying to run a small beginner application. I am encountering the following error -
/usr/bin/ld: cannot find -lGl
collect2: error: ld returned 1 exit status

Thanks for the previous reply. I was successful in building and running few simple applications. I am stuck at a point where I want to send the rs2::frame object over the network. So, probably I want to convert the rs2::frame object into bytes and then send those bytes. But I am unable to find any method in rs2::frame class to convert it into bytes. Please help me with this issue.

There is no full built-in serialization for frames, but you can access the frame content using get_frame_data().

So can a new frame be initialized or constructed using the output of get_frame_data( ), as I need to reconstruct the frame from the bytes transferred at the receiver side.

As per my understanding, this basically returns a pointer to the 1-D array which contains pixel values. What is the size of the array returned by get_frame_data()?

I need to reconstruct the frame on the receiver side as well. But out of the constructors available for rs2::frame, only rs2::frame(rs2_frame *) seems to be useful for me. This is because I can get rs2_frame * by using frame.get() to the sender's frame. Is there any way that I can serialize the struct pointer by rs2_frame * ? What is the size of the struct pointed by rs2_frame * ?

I am trying to generate cv::Mat using the uint8_t array obtained from get_frame_data() for a frame. I am building a client-server application where I am sending the integer array from server to client. At the client side, I am trying to generate cv::Mat from the integer array received. But, the image displayed is some garbage. I know Mat() also takes the void * array but I can't send this array from server to client, I need to convert it into int array first. So, how do I use the int array to convert it into Mat?
I have written the following code. Please help!
rs2::frame color_frame = frames.get_color_frame();
    int height = 480;
      int width = 640;
      int arr[width*height];
      auto pixels = (uint8_t*)color_frame.get_data();
      for(auto i=0;i<(width*height);i++)
      {
      	arr[i] = pixels[i];
      }

    // Creating OpenCV Matrix from a color image
    Mat color(Size(640, 480), CV_8UC3, arr, Mat::AUTO_STEP);

    // Display in a GUI
    namedWindow("Display Image", WINDOW_AUTOSIZE );
    imshow("Display Image", color);

So can a new frame be initialized or constructed using the output of get_frame_data( ), as I need to reconstruct the frame from the bytes transferred at the receiver side.

Is it possible to create a frame from get_frame_data( )?
Thank you.


Preview deformed while rotating

In case I'm enabling landscape in the demo project, the image is stretched while I'm rotating.
surfaceChanged is called only when rotation completed, so i can't use this method.
Any idea?
BTW, great work on the Lib, I want to use wrap it and use it as ReactNative library.

The preview is stretched because you use an aspect ratio in surfaceview different of the stream.
For example:
If you select stream resolution of 1920x1080 aspect ratio = 16:9. So you need create a surfaceview with 16 pixels of width each 9 of height.

Thx, this fixed the problem, I had to call prepareVideo(.....) before startPreview():
  getHolder().addCallback(new SurfaceHolder.Callback() {
         @Override
         public void surfaceCreated(SurfaceHolder surfaceHolder) {
             String resolution = rtmpCamera1.getResolutions().get(0);
             int width = Integer.parseInt(resolution.split("X")[0]);
             int height  = Integer.parseInt(resolution.split("X")[1]);
             rtmpCamera1.prepareVideo(width, height, fps, videoBitrate * 1024, isHardwareRotatation, orientations[1]);
             rtmpCamera1.startPreview();
         }

         @Override
         public void surfaceChanged(SurfaceHolder holder, int format, int width, int height){
         }
         @Override
         public void surfaceDestroyed(SurfaceHolder surfaceHolder) {
             rtmpCamera1.stopPreview();
         }
     });
 }

Now I have a problem changing orientation on rotating from portrait to landscape (and vice versa), the orientation is set only in Camera1ApiManager constructor, can I change it while previewing? or I'm doing something wrong

I'm not sure if I understand the question.
Do you want change preview orientation on startPreview but with the same stream orientation?
Orientation only change stream result. The preview orientation is setted to the camera automatically to 0 if you are in landscape or reverse landscape activity or 90 if you are in portrait or reverse portrait activity.
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/blob/master/encoder/src/main/java/com/pedro/encoder/input/video/Camera1ApiManager.java#L59
Want do you want exactly? Maybe I can do a little change in the library if the requeriment is usefull
If you explain me it with a picture, it will be more easy for me.

I want to be able to change the orientation on the fly while previewing.
For example, I started with portrait (90), and while previewing I rotated the device to landscape.
and there is no place where i can call camera.setDisplayOrientation(orientation); with the new orientation.
maybe  camera.setDisplayOrientation method should be exposed from RtmpCamera1/Camera1Base
This method can be called while previewing:

Starting from API level 14, this method can be called when preview is active.

https://developer.android.com/reference/android/hardware/Camera.html#setDisplayOrientation(int)

I did a commit in the library to implement it: https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/commit/51cb8dd7b19760d0d358024fd0d85f7035cb9b0d
Now:
rtmpCamera1.prepareVideo(width, height, fps, videoBitrate * 1024, isHardwareRotatation, rotation);
rtmpCamera1.startPreview();

This is incorrect. You have startPreview method with a resolution. Remember that preparevideo will override resolution of startPreview when call to startStream.
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/commit/51cb8dd7b19760d0d358024fd0d85f7035cb9b0d#diff-cfc8e034d6d6419c1a64088e9cb13dcbR193
setPreviewOrientation() method change the orientation of your preview camera on fly. This can be called when you want (before, after or while the stream). If you never call this method library will select the orientation for you in the rtmpCamera1 constructor. Remember this change only preview no result stream so if you change it while you are stream the result will not rotate only preview rotate.
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/commit/51cb8dd7b19760d0d358024fd0d85f7035cb9b0d#diff-cfc8e034d6d6419c1a64088e9cb13dcbR213

Thanks a lot, ill try and update


Update to UTC times

Change from using the local time to UTC time. This includes schedules, processing, and one-time items. Alternate is to deal with local times only (and accept errors caused by transition to/from Daylight Saving Time).

Reduced number of calls to DateTime.Now in branch https://github.com/jeremybytes/house-control/commit/6523bc02d416b3b547b64259cd8c9b6187909047
There are still calls in tests that will be removed after the TimeProvider abstraction is put into place. See issue https://github.com/jeremybytes/house-control/issues/2


Improve test for OCI controller

Fixes #836

Think this looks good, would be great if you can smash the latest two commits in the original commits which introduced the changes.


Port Analytics SDK pages to docs.snowplowanalytics.com

Project: Scala Analytics SDK
Wiki lists 1.0.0 as the latest version, but there is a newer version.

Thanks, Gabriel. I updated the page, but all docs should be migrated to docs.snowplowanalytics.com/ soon.

Hi @gvacaliuc ,
We've now migrated documentation related to the Analytics SDK to our documentation website.


refactor!: remove deprecated cli module and update dependencies

Remove the cli module, which has been deprecated since pyprql 0.5.10 (released on PyPI on 2022-12-12) (#93), and update dependencies.
This PR also includes a configuration change for python-semantic-release (major_on_zero = false), making the release after this disruptive change 0.7.0 instead of 1.0.0.

The CLI is very well done and I am impressed, but there are so many dependencies that it will probably need to be split into separate packages if it is to be revived.

Shall I release 0.7.0 from the main repo? It's slightly simpler of the versions stay aligned, unless we have a reason to deviate

Shall I release 0.7.0 from the main repo? It's slightly simpler of the versions stay aligned, unless we have a reason to deviate

If that is your intention, I am happy to hold off on releasing pyprql.
(I just thought it was worth releasing because of the many additional features related to IPython magic.)


Display list of random samples on welcome screen

Instead of default response samples list (which most time will be the same)
list of random samples can be displayed.
Which might more interesting to see and find something new

set to display latest


Update zip.py

imported valid_ip.py as a module and implemented is_valid() function from the module.

Thank you @KUndn for your contribution!

Thanks
On Sat, 3 Oct, 2020, 12:53 PM Anshul Choudhary<EMAIL_ADDRESS>wrote:

Merged #14 https://github.com/achoudh5/Manipulating_Excel_Sheet/pull/14
into main.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/achoudh5/Manipulating_Excel_Sheet/pull/14#event-3836318497,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOUUEMNWYYJIOGHBXXTOMCTSI3GPFANCNFSM4SCPG6KA
.

Hi @KUndn
Upon running, it gave me error:-
   wb = openpyxl.load_workbook(i) NameError: name 'i' is not defined
Seems like valid_ip.py doesn't know about the variable i you transferred from zip.py
Can you fix it?

Hi @achoudh5
wb = openpyxl.load_workbook(i)
in above statement i is input variable and valid.py doesnot take any input
i is not passed from zip.py  folder.
that's all i can deduce from what i know,
if that's correct i can just uncomment the input statement .
Correct me if  i am wrong.

Yea, that's correct understanding! Make sure that is_valid function is working maybe use a print statement.
If you can paste some output as well that'll simply things for me 😃


Playing with this and I got this odd error.

C:\code\sold>python driver.py flag.jpg Images
'python' is not recognized as an internal or external command,
operable program or batch file.
C:\code\sold>c:\Python27\python.exe driver.py flag.jpg Images
Reading tiles from 'Images'...
Processed 60 tiles.
Processing main image...
Main image processed.
Building mosaic, press Ctrl-C to abort...
Traceback (most recent call last):
Traceback (most recent call last):
File "driver.py", line 234, in
File "", line 1, in
File "c:\Python27\lib\multiprocessing\forking.py", line 381, in main
mosaic(sys.argv[1], sys.argv[2])
File "driver.py", line 227, in mosaic
self = load(from_parent)
File "c:\Python27\lib\pickle.py", line 1384, in load
compose(image_data, tiles_data)
File "driver.py", line 199, in compose
return Unpickler(file).load()
File "c:\Python27\lib\pickle.py", line 864, in load
Process(target=build_mosaic, args=(result_queue, all_tile_data_large, original_img_large)).start()
dispatchkey
File "c:\Python27\lib\multiprocessing\process.py", line 130, in start
File "c:\Python27\lib\pickle.py", line 886, in load_eof
self._popen = Popen(self)
File "c:\Python27\lib\multiprocessing\forking.py", line 277, in init
raise EOFError
EOFErrordump(process_obj, to_child, HIGHEST_PROTOCOL)
File "c:\Python27\lib\multiprocessing\forking.py", line 199, in dump
ForkingPickler(file, protocol).dump(obj)
File "c:\Python27\lib\pickle.py", line 224, in dump
self.save(obj)
File "c:\Python27\lib\pickle.py", line 331, in save
self.save_reduce(obj=obj, *rv)
File "c:\Python27\lib\pickle.py", line 425, in save_reduce
save(state)
File "c:\Python27\lib\pickle.py", line 286, in save
f(self, obj) # Call unbound method with explicit self
File "c:\Python27\lib\pickle.py", line 655, in save_dict
self._batch_setitems(obj.iteritems())
File "c:\Python27\lib\pickle.py", line 687, in _batch_setitems
save(v)
File "c:\Python27\lib\pickle.py", line 286, in save
f(self, obj) # Call unbound method with explicit self
File "c:\Python27\lib\pickle.py", line 554, in save_tuple
save(element)
File "c:\Python27\lib\pickle.py", line 286, in save
f(self, obj) # Call unbound method with explicit self
File "c:\Python27\lib\pickle.py", line 731, in save_inst
save(stuff)
File "c:\Python27\lib\pickle.py", line 286, in save
f(self, obj) # Call unbound method with explicit self
File "c:\Python27\lib\pickle.py", line 655, in save_dict
self._batch_setitems(obj.iteritems())
File "c:\Python27\lib\pickle.py", line 687, in _batch_setitems
save(v)
File "c:\Python27\lib\pickle.py", line 313, in save
(t.name, obj))
pickle.PicklingError: Can't pickle 'ImagingCore' object: <ImagingCore object at 0x02F74830>

I suspect this issue is similar to the one described in this StackOverflow post, I don't have a fix at the moment but would be happy to accept a patch if anyone wants to investigate


Preview 1.82.10 Hotfix

What's new? - 1.82.10

[Fix] NullReferenceException crash caused by null voice_packs property, by @neon-nyan
[Fix] Some migration dialog still provides install choices, by @neon-nyan
[Fix] TimeSpan overflow + Number to Percentage and TimeSpan converter refactor, by @neon-nyan and @bagusnl
[Fix] Minor Audio Selection UI bug, by @neon-nyan
[Fix] More SharingViolation exceptions on InstallManagerBase, by @neon-nyan
[Fix] DirectoryNotFoundException on HSR's Game Repair -> RemoveHashMarkFile() method, by @neon-nyan
[Fix] Relative URL for the Velopack packages getting cut-off, by @neon-nyan

Changelog Prefixes
  **[New]**
  **[Imp]**
  **[Fix]**
  **[Loc]**
  **[Doc]**

As per internal discussion, no reviews. Merging this PR ahead


If I toggle st.secure_text_entry, I cannot restore the font

I have a button to "hide/show" a password which changes the secure_text_entry boolean. If it gets hidden then re-revealed, it loses the font I applied to it with RMQ.
  @password.style do |st|
    st.secure_text_entry = false
    st.font = font.size_18_light
  end

Even if I reapply the font, it still shows the system default font (or whatever serif font it is).

Apparently, it's an iOS bug. Found a work around:
  @password.style do |st|
    @password.get.resignFirstResponder
    st.secure_text_entry = false
    @password.get.becomeFirstResponder
  end


bmap! (and alikes) to use ToOwned

99% of the time users will want collections with owned values. Why wrap everything in s! etc. then? Possibly name bmap_owned to make the cloning explicit.

Good point, but which value should be converted to the owned: key? value? both?

Both. My understanding is that ToOwned of an already owned value is a no-op anyway (just a move). So the problem with ToOwned is only that it makes easy to implicitly get an extra allocation, which is generally avoided in most standard APIs.


DTSPO-2880 - adding sds-toffee-shared-infrastructure to prod environment

should probably add the other toffee repoes?

updated


Cycle 4: Moritz to work on astropy.io.ascii

Project Description
Address (by fixing or closing) open issues in astropy.io.ascii
Project / Work
At the time of writing there are 66 open issues in astropy tagged io.ascii, 57 of which are older than 18 months.
While astropy clearly thrives with those bugs and issues, each of them represents at least one user
who was stopped in their work and found this annoying enough to open an issue. We know anecdotally, that
many users who do not think of themselves as "developers" don't open issues on github (e.g. they might not
even have an account) so each bug probably represents several or more users who have run
into a problem. For a smooth user experience, we should close out bugs and fill in feature requests
where we can.
Not all of these issues are actionable (e.g. some
require upstream fixes or
would require fairly invasive changes that would break backwards compatibility in a way that does not seem warrented);
however most can be addressed given some developer time.
Moritz proposes to pay a sub-grant to MIT to pay for some of his time to
work on those issues. Based on previous PRs, I estimate that I can address
on average one issue in one to two work days, so paying for 10% of my work time for one year should allow me to address the majority of the currently open issues.

Since this is a subaward and is to do work which is likely to extend for more than one year (io.ascii maintenance), please consider submitting a budget for 2-3 years instead of just one.
This request could also mention your other efforts in the Project.

I do not anticipate working on this for more than one year at the level of effort I'm asking for. There are about 50 open issues on io.ascii. I'm not qualified to address all of them (e.g. in the fast reader in C code) and some issues are not actionable, e.g. they are asking for things so ill-defined that it cannot be implemented. One year is probably enough to address the issues that can be addressed by me.
Re: "could also mention my other efforts in the project": I was planning to continue my other work as unfunded volunteer. This proposal is to pay for time for astropy that I would not have otherwise, not to get me funded through astropy for things I'm doing anyway. However, I realize that it might not be clear to others "what I would be doing anyway" so I'll try to broaden the language to include any aspects where ambiguity might arise.

The release of the president's budget request two days ago indicates that NASA intends to wind down Chandra operations as soon as possible. While that's only a request and not an approved budget and might still change, it is possible (or even likely) that I won't be able to use my "science time" as paid for by Chandra to volunteer much longer. I'm thus following @kelle suggestion to broaden the scope so that, if granted, this could also be used to pay for my other roles on astropy and I'm suggesting a a possible path to a 2-3 year time line.

@mhvk : You already gave a "thumb up". Note that the comment by @kelle  and the release of the president's budget request two days ago have let me to significantly increase the scope for this proposal to 3 years.

Definitely another :+1: to the idea to bring the best from X-ray spectral modeling to specutils! It will be nice to have it in a package aimed at all wavelengths: modeling multi-band photometry is not all that different from modelling X-ray CCD spectra. I also more generally like seeing proposals to add missing pieces of functionality! (Though your "grunt" work is always very much appreciated too, of course!)

While obviously we'd love to fund this for the full amount listed, if we need to squeeze the budget is there a minimum number of hours per year that would make sense?

The minimum is defined here: https://github.com/astropy/astropy-project/pull/385/files#diff-15111b979d86e382215dfe1ee880579cbc96ce543e042c956d8c166bb7a68237R53
Copying that line below for convenience:
The minimum useful time is determined by how much effort it is to set up that sub-award. I can work on one issue in 8 hours of work, but neither MIT nor NumFOCUS would want to write and process a subaward contract for that little money [8h /year]. Realistically, it's probably not worth the effort if less than $5-10k  are allocated to this project.

Thanks Moritz; dunno how I missed that first time!

Please react to this comment to vote on this proposal (👍, 👎, or no reaction for +0)

I'm writing on behalf of Astropy's Finance Committee regarding the outcome of this funding request. We are pleased to be able to let you know that, following consultation with the community, we are able to approve this request. We can currently fund the Year 1 amount of US$ 5000 to carry out the project. Funding beyond this amount will be contingent upon the availability of funds. (We will be using the full budgets of all of the approved requests to craft future grant and funding proposals.) We assume you will be using this full year 1 budget, if that is not the case please contact us immediately.
Ana Gabela and I will be your contacts on the Finance Committee to facilitate this award. Please get in touch with us if you have any questions or concerns. Please do not reach out to NumFOCUS directly. Also, after merging this PR, I will open an issue where we can discuss progress on this project.
In addition, new to this funding cycle, is the assignment of a COTR (Contracting Officer's Technical Representative) to each funded project. This concept is borrowed from government funding agencies, although it is to be stressed that Astropy's goal is to make the COTR role as low-overhead as possible. The COTR’s primary responsibility is to make sure the work is happening at the expected pace and, if necessary, to be a liaison between the funded project and the Finance Committee or CoCo. The COTR for your project will be assigned shortly and we’ll also be sending out more details about how we see this working.
We have already been in touch with both NumFOCUS and MIT on getting this subaward setup.
Congratulations --- we are really looking forward to seeing you put these funds to good use!
Kelle, on behalf of the Astropy Finance Committee


The "relocation truncated to fit" error occurs on a compilation of a code generated for the "ml" target.

After increasing a number of instructions for ml target in the commit 2b2abda6fa0fdd35435cca615861fe3145996b14 the successful generation of a assembly code is only possible when  generation timeout is increased by "--gen_timeout".
While it is not a big problem, but when code is finally generated in most cases it cannot be compiled with gcc because of a address relocation problem:
Mon, 12 Jul 2021 08:19:35 INFO     Creating output directory: sim_out_ml
Mon, 12 Jul 2021 08:19:35 INFO     Processing regression test list : .../riscv-dv/target/ml/testlist.yaml, test: all
Mon, 12 Jul 2021 08:19:35 INFO     Found matched tests: riscv_rand_test, iterations:1
Mon, 12 Jul 2021 08:19:35 INFO     Processing simulator setup file : .../riscv-dv/yaml/simulator.yaml
Mon, 12 Jul 2021 08:19:35 INFO     Found matching simulator: riviera
Mon, 12 Jul 2021 08:19:35 INFO     Building RISC-V instruction generator
Mon, 12 Jul 2021 08:20:11 INFO     Running RISC-V instruction generator
Mon, 12 Jul 2021 08:20:11 INFO     Generating 1 riscv_rand_test
Mon, 12 Jul 2021 08:20:11 INFO     Running riscv_rand_test with 1 batches
Mon, 12 Jul 2021 08:20:11 INFO     Running riscv_rand_test, batch 1/1, test_cnt:1
Mon, 12 Jul 2021 08:41:30 INFO     Compiling sim_out_ml/asm_test/riscv_rand_test_0.S
/tmp/cc4iIFSO.o: in function `sub_2_j6':
(.text+0x1e9234): relocation truncated to fit: R_RISCV_JAL against `sub_4'

The compilation command looks in this case something like:
'riscv64-unknown-elf-gcc', '-static', '-mcmodel=medany', '-fvisibility=hidden', '-nostdlib', '-nostartfiles', 'sim_out_ml/asm_test/riscv_rand_test_0.S', '-I/.../riscv-dv/user_extension', '-T/.../riscv-dv/scripts/link.ld', '-o', 'sim_out_ml/asm_test/riscv_rand_test_0.o', '-mno-strict-align', '-march=rv64imc', '-mabi=lp64'

After increasing a number of instructions for ml target in the commit 2b2abda the successful generation of a assembly code is only possible when generation timeout is increased by "--gen_timeout". While it is not a big problem, but when code is finally generated in most cases it cannot be compiled with gcc because of a address relocation problem:
Mon, 12 Jul 2021 08:19:35 INFO     Creating output directory: sim_out_ml
Mon, 12 Jul 2021 08:19:35 INFO     Processing regression test list : .../riscv-dv/target/ml/testlist.yaml, test: all
Mon, 12 Jul 2021 08:19:35 INFO     Found matched tests: riscv_rand_test, iterations:1
Mon, 12 Jul 2021 08:19:35 INFO     Processing simulator setup file : .../riscv-dv/yaml/simulator.yaml
Mon, 12 Jul 2021 08:19:35 INFO     Found matching simulator: riviera
Mon, 12 Jul 2021 08:19:35 INFO     Building RISC-V instruction generator
Mon, 12 Jul 2021 08:20:11 INFO     Running RISC-V instruction generator
Mon, 12 Jul 2021 08:20:11 INFO     Generating 1 riscv_rand_test
Mon, 12 Jul 2021 08:20:11 INFO     Running riscv_rand_test with 1 batches
Mon, 12 Jul 2021 08:20:11 INFO     Running riscv_rand_test, batch 1/1, test_cnt:1
Mon, 12 Jul 2021 08:41:30 INFO     Compiling sim_out_ml/asm_test/riscv_rand_test_0.S
/tmp/cc4iIFSO.o: in function `sub_2_j6':
(.text+0x1e9234): relocation truncated to fit: R_RISCV_JAL against `sub_4'

The compilation command looks in this case something like: 'riscv64-unknown-elf-gcc', '-static', '-mcmodel=medany', '-fvisibility=hidden', '-nostdlib', '-nostartfiles', 'sim_out_ml/asm_test/riscv_rand_test_0.S', '-I/.../riscv-dv/user_extension', '-T/.../riscv-dv/scripts/link.ld', '-o', 'sim_out_ml/asm_test/riscv_rand_test_0.o', '-mno-strict-align', '-march=rv64imc', '-mabi=lp64'

Hello @dariuszst ,
Can you please try to change the "-mcmodel=medany" to "-mcmodel=medlow" in the run.py file and again try to compile and run, please let me know if its working.
Thanks

yes, same problem.... i have tried to change the "-mcmodel=medany" to "-mcmodel=medlow", but still the same

@mukesh891


Get rid of issue_template.md

Description
It was copied from somewhere else (nlohmann/json maybe?). Most of the time when I create new issue this template is irrelevant. Other user "jcoder58" ignored it completely and I don't mind.
So if it is not enforced and irrelevant, then it's easier not to have one. At least in the current state, when there is no tens and hundreds of issues from random users, when it makes sense to somehow formalize issues.
At the same time pull_request_template.md looks legit for now. But I gave up creating PR for a while now, so maybe it is also outdated or irrelevant.

Done


Streaming message chunks from nodes

Currently it is possible to stream results from graphs per node, but is it possible to stream message chunks from LLM invocations  per node as well? Currently UX is not great having to wait for invocations to finish entirely before message could be display to user.

hi,
You can use astream_events for this purpose, eg.
for await (
  const event of await graph.streamEvents(
    { messages: [["user", "What's the weather like today?"]] },
    { streamMode: "values", version: "v1" },
  )
) {
  if (event.event === "on_llm_stream") {
    const msg = event.data?.chunk.message as AIMessageChunk;
    if (msg.tool_call_chunks && msg.tool_call_chunks.length > 0) {
      console.log(msg.tool_call_chunks);
    } else {
      console.log(msg.content);
    }
  }
}

See longer example here https://github.com/langchain-ai/langgraphjs/blob/main/examples/how-tos/stream-tokens.ipynb


objectDifference.ts - enhancements and fixes; Unit test - Jest;

objectDifference.ts - support for removal from link/block lists;
objectDifference.ts - support for change resulting in empty link/block list;
objectDifference.ts - fix bug from version 2.4.5 (isArray called with origObj instead of value);
Add unit tests for objectDifference.ts;
Add unit testing framework - jest;
Update typescript to 5.1.6;
Addresses #35 among other things.
Version 2.4.5 introduced a bug that is causing script not to fire in multiple situations (isArray called with origObj instead of value).
Here's an update to objectDifferences.ts that addresses that.
I rewrote the function to additionally add support for moments when elements are being deleted from list (be that blocks or ref links).
Added Jest Unit Test framework and wrote a couple of unit tests for the difference function.
I am not a java script developer so I might not be following some insider best practices.
Please let me know if something should be corrected or adjusted.

I am still noticing some undesired behavior in browser so further updates might be required (to the FieldExtension.tsx)

Updated script triggering mechanism.
For each modified flattened path the code will be checked agains presence of the oldest ancestor from the modified path up to the level where field holding the script resides.
{
  computed_field
  text_field
  blocks {
    block_computed_field
    block_text_field
    inner_blocks {
      inner_block_text_field
    }
  }
}

compute field
modified path
code checked for presence of

computed_field
textField
textField

computed_field
blocks.0.block_text
blocks

block_computed_field
textField
ignored

block_computed_field
blocks.0.block_text
block_text

block_computed_field
blocks.0.inner_blocks.0.inner_block_text_field
inner_blocks

@gswierczynski Thanks for the extensive PR.
I like the change in the object differences function, but I see some code style changes (like removed spaces or single/double quotes https://github.com/voorhoede/datocms-plugin-computed-fields/pull/36/files#diff-4ac47e410f008329ee7cbc652eb14e4fa23c06c36c411278561ac51e8108f225R1). Can you make sure that stays consistent?
I think we don't want to add unit testing in this PR. Since we have a lot of other plugins we want to keep them consistent. Next to a github action that tests the code we also want it to be implemented consistent to other plugins (like we did in this plugin: https://github.com/voorhoede/datocms-plugin-translate-fields/tree/main).
Is it okay if we cherry pick some test code and implement that in a different PR?
Next to that (just to have a small and clean PR) updating any packages I want to do in a seperate PR.

Perfect, thank you! I will do a last check and merge it


Fix http.status_code is not propagated correctly in Span RED Metrics

Fix https://github.com/wavefrontHQ/wavefront-spring-boot/issues/85

PR for this github issue: https://github.com/wavefrontHQ/wavefront-spring-boot/pull/86

I think that rather than duplicating the issue that was fixed in the SDK, those issues should be about general dependency upgrades. The issue isn't in this project so the current description is a bit misleading IMO.

Superseded by #90