Speech

** Please do not merge this PR - this is for review only, until the checklist has been completed **
Must haves:

[ ] SSML modification to fit cognitive service format
[ ] Localization of the strings (e.g. "Listening")
[ ] Uncomment CognitiveService.js file in speech sample
[ ] Unit test for microphone button (tapping on button starts speaking / stop listening)
[ ] Unit test for microphone visible if configured
[ ] Unit test for activity with speak field -> synthesizer
[ ] Design pass by design team
[ ] Expand unit test page to enable/disable speech
[ ] Add link to speech sample in readme
[ ] Add built files to .gitignore (CognitiveServices.js)

[ ] first checkbox

@w-Yuan & @danmarshall - What's the current status of the couple of items on the check list?


Can't access API because of permission denied

Hi, Thank for your hard work on this repo.
My work for now required to integrate the JWT with OAuth2 so that I came here and use your repo to solve my problem. But look like your authenticate method only work for AnonymousUser?
class JWTAuthentication(BaseAuthentication):
    """
    Token based authentication using the JSON Web Token standard.
    Clients should authenticate by passing the token key in the "Authorization"
    HTTP header, prepended with the string specified in the setting
    `JWT_AUTH_HEADER_PREFIX`. For example:
        Authorization: JWT eyJhbGciOiAiSFMyNTYiLCAidHlwIj
    """
    www_authenticate_realm = 'api'

    def authenticate(self, request):
        """
        Returns a two-tuple of `User` and token if a valid signature has been
        supplied using JWT-based authentication.  Otherwise returns `None`.
        """
        jwt_value = self._get_jwt_value(request)
        if jwt_value is None:
            return None

        try:
            payload = decode_jwt(jwt_value)
        except jwt.ExpiredSignatureError:
            msg = 'Signature has expired.'
            raise exceptions.AuthenticationFailed(msg)
        except jwt.DecodeError:
            msg = 'Error decoding signature.'
            raise exceptions.AuthenticationFailed(msg)
        except jwt.InvalidTokenError:
            raise exceptions.AuthenticationFailed()

        self._add_session_details(request, payload)
        return AnonymousUser(), payload

I follow this method and see that you return an AnonymousUser instead of current user who use the token to access. Could you explain for me why you do that and how if I want to add permission for my views.
Thank you so much.

Done in PR: https://github.com/Humanitec/django-oauth-toolkit-jwt/pull/10


add the APKs package so that the use of RAM is smaller

Please stop creating feature requests that only consist of a title. Half of the time I don't know what you mean. But even if I do, others might not. If you don't invest the time to write down a description, why should anyone invest the time trying to make sense of it?


Examples for reading position?

This github repo is a great idea by Thorlabs management and is a great first step to addressing the severe lack of instructional examples.
The entirely auto-generated .NET documentation makes the learning curve extremely steep for end users.
It does not tell you that the Position property only updates at the polling cycle (which is often 250 ms or 500 ms in the examples).
It does not explain that the deceptively named function, RequestPosition(), does not actually request a new position -- it returns the stored Position from the last poll, which is likely wrong ... because ... it ... is ... moving ...
Calling RequestStatus() and reading the .Status.Position property will give a "live" position.  That took me lots of trial and error to figure out.
Please keep improving and adding to this repo!  Thanks.

Hello and thank you for your comment! We appreciate the feedback and are in the process of creating some additional documentation on the Kinesis SDK, which will include a better explanation of the device polling loop and status updates. These documents will be added to this repository when ready. If you have any other suggestions for improvement or questions, please feel free to comment here or to email us at<EMAIL_ADDRESS>

Thanks for completing.

@manzelmi-thorlabs Thanks for completing.


Enable face culling in RenderNodes where it would make sense.

It appears that Face Culling is currently disabled for a number of render nodes that should probably have it, i.e. OpaqueBlocksNode - which renders most of the landscape.
We need to investigate which nodes could take advantage of face culling and add the line
addDesiredStateChange(new EnableFaceCulling());
in the constructor of those nodes for which it makes sense.

This was tackled by readv in #3291.


Tooltips missing from play/cancel/trash icons for MemoryLogger

There are some other gtAction buttons that might possibly need textual guidance like a tooltip:


MODELIX-252: Trigger missed release for package renamings

It would probably be a good idea to add a BREAKING CHANGE trailer to the commit message explaining the changes to readers of the changelog. The text put under this footer section is expanded into the changelog. https://www.conventionalcommits.org/en/v1.0.0/#commit-message-with-description-and-breaking-change-footer

I would have approved this if the GitHub UI would allow to reach the approve button :D

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


corrected todosDeleted export to todosToggled in converting the todos…

… reducer section

name: :memo: Documentation Fix
about: Fixing a problem in an existing docs page
Checklist

[ ] Is there an existing issue for this PR?
https://github.com/reduxjs/redux/issues/3956
[ ] Have the files been linted and formatted?
No

What docs page needs to be fixed?

Converting the Todos Reducer:
Tutorials -> Redux Fundamentals -> Part 8: Modern Redux with Redux Toolkit:

What is the problem?
export const { todoAdded, todoDeleted } = todosSlice.actions
todoToggled should be exported instead of todoDeleted
What changes does this PR make to fix the problem?
changed todoDeleted to todoToggled in export

Thanks!


Ability to test actual routing without starting application/webserver

Hello. I'm using ASP.NET Web API 2 in production right now. At the beginning of this pleasant journey my team had issues with routing: when you add or modify routes, there is a chance to break currently working ones because of default parameters, one route being a substring of another and other ways. I searched for a way to simulate whole Web Api pipeline and observe route match results (controller and its method) without actually running an application. Finally I've digged into Web Api sources and made a RouteTester. It builds actual routes like Web Api does and tries to match given HttpRequestMessage to controller and method. It is also capable of calling the selected method, keeping the whole pipeline.
Is there any feature like this in the upcoming vNext? If no, i'd like to contribute :) The feature is important IMHO because you can write tests and be sure that your routing is not broken and every piece of magic works as you expect.

You can use TestServer to host an MVC app in memory and then run integration tests including routing in memory. Take a look at the following examples in the MVC tests: https://github.com/aspnet/Mvc/blob/dev/test/Microsoft.AspNet.Mvc.FunctionalTests/RoutingTests.cs.

Thank you for the link. As far as i can see, running a TestServer does the thing. However, i have questions now:

With TestServer, you have to actually call an action, not just select it. What if you are testing just routes? //correct me if i'm wrong

Who does the magic of putting RoutingResult to response body? If you want to test actual response body, where to get it?

You compare result.action and result.controller with desired values as strings. Maybe it's better to compare actual Type of controller and MethodInfo of action instead of hard-coding? It is possible to get them from the ControllerDescriptor and ReflectedHttpActionDescriptor (I'm talking in terms of WebApi2, maybe now things are changed). For example, my tests look like this:
var requestString = string.Format("http://somehost/api/v1/debug/{0}", command);
var request = new HttpRequestMessage(HttpMethod.Get, requestString);
routeTester.Perform(request).CompareControllerAndMethod((DebugController x) => x.Get(command));
// threre are also CompareCode() and CompareContent() methods to test HTTP code and response. The action gets called only if you call one of them.

I had no idea how to test parameters binding, just can be sure that right action overload selected.

Hi,
The design of routing has changed significantly since the system.web days, and doesn't really support a testing model like you may have used in WebAPI. If you take a gander at https://github.com/aspnet/Mvc/blob/dev/src/Microsoft.AspNet.Mvc.Core/MvcRouteHandler.cs you'll see that the entry point into MVC is itself an IRouter.
The main difference is that there's no longer a separation between routes and handlers. Everything that does routing is still on the call stack when you invoke a handler.
So, to answer your first question, there really is no testing 'just routes'. If you wanted to do something as close as possible, you could add a resource filter in startup code that would return information about what action was chosen (this is pretty similar to what our test code does.
public class RoutingDataFilter : IResourceFilter
{
    public void OnResourceExecuting(ResourceExecutingContext context)
    {
        // get data into a format you like from context.RouteData.Values and context.ActionDescriptor
        var data = ...
        context.Result = new ObjectResult(data);
    }
}

To the second, I'm not sure what you mean exactly. These are normal action methods that return data, the JsonFormatter is writing it out.
Thirdly, these tests are written as client-server test. If you do a build in our repo you'll see them run on your box, but we have done the exercise of running them on a remote server - something we're ramping up as we get close to a v1 release. This isn't meant to be a pattern specifically for routing testing, it's across all of our functional/integration testing.

Thanks for all the info!


chore: prepare Cargo.toml, README.md, DEVELOPMENT.md ready for initial release

closes #36

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

Project coverage is 82.86%. Comparing base (b6dec8b) to head (a4f4557).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #48   +/-   ##
=======================================
  Coverage   82.86%   82.86%
=======================================
  Files          16       16
  Lines        2241     2241
  Branches     2241     2241
=======================================
  Hits         1857     1857
  Misses        246      246
  Partials      138      138

Flag
Coverage Δ

rust
82.86% <100.00%> (ø)

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

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


Mutual lambda interpolation expansion and escaping problem.

There is a rule for each of the first two codes, but if you trying using them mutually, like in the third example, it creates a problem.
Lambda Escaping (code 1) lambda rule 5
var data = {
  'lambda' : function(){
    return '>';  // it is escaped by default (Lambda Expansion)
  }
};

var template = "<{{lambda}}{{{lambda}}}";

// Result = "<>>"

Lambda Interpolation Expansion (code 2) lambda rule 2
var data = {
  'planet' : 'world',
  'lambda' : function(){
    return {{planet}};  // Interpolation Expansion
  }
};

var template = "Hello, {{lambda}}";

// Result = "Hello, world"

Now simply change the value of data['planet'] from 'world' to '<' and there is the problem:
var data = {
  'planet' : '<',
  'lambda': function(){
    return "{{planet}} "  // the planet tag is escaped twice
  }
};

// the first escaping of  '<' becomes '<'  (Interpolation Expansion)
// and the second escaping becomes  '&lt;' ( Lambda Escaping)

var template = "Hello, {{lambda}}";

// Result = "Hello &lt;"  // makes no sense

And this is what the rule currently specifies. Is there a clause in the rules or specification that prevents this?

FYI, the GRMustache way of dealing properly with a lambda that renders "{{planet}} {{{planet}}}" is the following.
First with the general rendering solution:
let data = [
    "planet": Box("<"),
    "lambda": Box({ (info: RenderingInfo) -> Rendering in
        try! Template(string: "{{planet}} {{{planet}}}").render(info.context)
    })
]
let template = try! Template(string: "Hello, {{lambda}}")
// Hello, < <
let rendering = try! template.render(Box(data))

Or with a template instead of a lambda (this technique is also called dynamic partial:
let lambdaTemplate = try! Template(string: "{{planet}} {{{planet}}}")
let data = [
    "planet": Box("<"),
    "lambda": Box(lambdaTemplate)
]
let template = try! Template(string: "Hello, {{lambda}}")
// Hello, < <
let rendering = try! template.render(Box(data))

First of all, thank you for taking some time out of your schedule to read and write a response to my problem / question.
I thought as much that the specification is broken. If those two rules weren't implemented together, there would be no problem, or if interpolations didn't have lambdas as an option, then there wouldn't have been this issue.
While the triple lambda solves this problem, the triple lambda also has it own problem with delimiters
for example:
{{regular-tag}} {{{triple-mustache}}} {{=[ ]=}} [regular-tag] [{triple-mustache}]

You're welcome, @perchingeagle. But don't expect the spec to change: it's stuck (it hasn't changed in years despite the work done by many implementors).
The way to avoid HTML escaping with custom delimiters is & tags: {{{foo}}} is equivalent to {{&foo}}, and to {{=[ ]=}} [&foo]. & was introduced specifically for custom delimiters. Quoting https://mustache.github.io/mustache.5.html:

You can also use & to unescape a variable: {{& name}}. This may be useful when changing delimiters (see "Set Delimiter" below).

I guess the current solution would be the choice of which tag to use for each situation:
{{& tag }} // for delimiters
{{{ tag }}} // for mutual lambda interpolation expansion and escaping

Thanks :+1:

I guess it is safe to say that this issue is now closed
again @groue thank you

Why did you close this issue? The lambda spec has an issue - there's no point hiding it under the carpet, is it?

I have reopened it.


Exporting auto_ml models to use in other programming languages.

Is there any way to export auto_ml models to use in other languages?
Sort of like there is PMML for many other models like lgbm/xgboost/sklearn?

Possibly. When a model is saved, it's saved as a .dill file. dill is a drop-in replacement for pickle, and it seems possible to load .pkl files in other languages, at least based on a cursory google search: https://www.google.com/search?q=loading+a+pkl+file+in+java
It's not something we currently natively support, and we don't really have any plans to in the near future. The pipeline itself uses enough Python logic that it's probably kinda difficult.
What I really want to do is build in interfaces to interact with auto_ml in other languages. We'd probably spin up a python server on localhost on that machine, and send data to and from that server from whatever the non-python language is.
If you want to take a crack at that, I'd love to see it!
You can also use auto_ml to find what works, do all the rapid iteration testing and development, and then write the production code in a different language once you've got a pretty good direction figured out.
Let me know what you end up doing- I'm curious.


Unkown problem with SiaD v1.0.3

Deleted the consensus and this output happened at some point in the last 25% from Siad. Not sure if related, but I moved a wallet from .6 to 1.0.3.

goroutine 108955 [sleep, 3 minutes]:
time.Sleep(0x2b8a118e00)
/usr/local/go/src/runtime/time.go:59 +0xef
github.com/NebulousLabs/Sia/modules/consensus.(*ConsensusSet).managedAcceptBlock.func1.1(0xcc03000000000000, 0xd9bc983986330ac0, 0x5036899bad38571f, 0x47d38da4d
574dc5a, 0x5a6800aa01ec0044, 0x5845f63c, 0xc0423c6000, 0x1, 0x1, 0xc04278cf00, ...)
/home/luke/go/src/github.com/NebulousLabs/Sia/modules/consensus/accept.go:254 +0x84
created by github.com/NebulousLabs/Sia/modules/consensus.(*ConsensusSet).managedAcceptBlock.func1
/home/luke/go/src/github.com/NebulousLabs/Sia/modules/consensus/accept.go:260 +0x190
goroutine 104339 [chan receive]:
github.com/NebulousLabs/muxado/proto.(*Session).Accept(0xc0423b2820, 0xc04202c4f0, 0xc042686580, 0x0, 0xc043411ea8)
/home/luke/go/src/github.com/NebulousLabs/muxado/proto/session.go:159 +0x62
github.com/NebulousLabs/muxado.(*sessionAdaptor).Accept(0xc044179510, 0xc043411ed0, 0x4f77f9, 0x459010, 0xc043411f10)
/home/luke/go/src/github.com/NebulousLabs/muxado/adaptor.go:33 +0x3e
github.com/NebulousLabs/Sia/modules/gateway.(*peer).accept(0xc04417d280, 0xa57730, 0xc042068dc0, 0xc9a0a0, 0xc04308a280)
/home/luke/go/src/github.com/NebulousLabs/Sia/modules/gateway/peers.go:51 +0x3f
github.com/NebulousLabs/Sia/modules/gateway.(*Gateway).threadedListenPeer(0xc042068dc0, 0xc04417d280)
/home/luke/go/src/github.com/NebulousLabs/Sia/modules/gateway/rpc.go:159 +0x117
created by github.com/NebulousLabs/Sia/modules/gateway.(*Gateway).addPeer
/home/luke/go/src/github.com/NebulousLabs/Sia/modules/gateway/peers.go:62 +0x91
goroutine 108968 [sleep, 3 minutes]:
time.Sleep(0x2b12dbfa00)
/usr/local/go/src/runtime/time.go:59 +0xef
github.com/NebulousLabs/Sia/modules/consensus.(*ConsensusSet).managedAcceptBlock.func1.1(0xcc03000000000000, 0xd9bc983986330ac0, 0x5036899bad38571f, 0x47d38da4d
574dc5a, 0x5a6800aa01ec0044, 0x5845f63c, 0xc044674600, 0x1, 0x1, 0xc04278d500, ...)
/home/luke/go/src/github.com/NebulousLabs/Sia/modules/consensus/accept.go:254 +0x84
created by github.com/NebulousLabs/Sia/modules/consensus.(*ConsensusSet).managedAcceptBlock.func1
/home/luke/go/src/github.com/NebulousLabs/Sia/modules/consensus/accept.go:260 +0x190
goroutine 106585 [select]:
github.com/NebulousLabs/muxado/proto/ext.(*Heartbeat).check(0xc0425ed950)
/home/luke/go/src/github.com/NebulousLabs/muxado/proto/ext/heartbeat.go:57 +0x141
created by github.com/NebulousLabs/muxado/proto/ext.(*Heartbeat).Start
/home/luke/go/src/github.com/NebulousLabs/muxado/proto/ext/heartbeat.go:48 +0xc4
goroutine 104287 [IO wait]:
net.runtime_pollWait(0x2d5cac0, 0x72, 0xc042534b80)
/usr/local/go/src/runtime/netpoll.go:160 +0x60
net.(*pollDesc).wait(0xc043e0e470, 0x72, 0xc042534b80, 0xc043659c28)
/usr/local/go/src/net/fd_poll_runtime.go:73 +0x3f
net.(*ioSrv).ExecIO(0xc04205a0f0, 0xc043e0e360, 0x9e764c, 0x7, 0xa57c78, 0x4, 0xc043659d48, 0x445366)
/usr/local/go/src/net/fd_windows.go:171 +0x183
net.(*netFD).Read(0xc043e0e300, 0xc042928700, 0x8, 0x8, 0x0, 0x0, 0x0)
/usr/local/go/src/net/fd_windows.go:426 +0x14a
net.(*conn).Read(0xc0423fa070, 0xc042928700, 0x8, 0x8, 0x0, 0x0, 0x0)
/usr/local/go/src/net/net.go:173 +0x77
github.com/NebulousLabs/muxado/proto/frame.(*BasicTransport).Read(0xc042a4c000, 0xc042928700, 0x8, 0x8, 0xc043659f48, 0x7e4e8b, 0xc042bc8780)
:139 +0x72
io.ReadAtLeast(0xc90d00, 0xc042a4c000, 0xc042928700, 0x8, 0x8, 0x8, 0x9c9860, 0x0, 0xc90d00)
/usr/local/go/src/io/io.go:307 +0xab
io.ReadFull(0xc90d00, 0xc042a4c000, 0xc042928700, 0x8, 0x8, 0x9a8900, 0x0, 0x0)
/usr/local/go/src/io/io.go:325 +0x5f
github.com/NebulousLabs/muxado/proto/frame.(*BasicTransport).ReadFrame(0xc042a4c000, 0xc965c0, 0xc042a4c070, 0x0, 0x0)
/home/luke/go/src/github.com/NebulousLabs/muxado/proto/frame/transport.go:34 +0x6e
github.com/NebulousLabs/muxado/proto.(*Session).reader(0xc04215e280)
/home/luke/go/src/github.com/NebulousLabs/muxado/proto/session.go:276 +0xa8
created by github.com/NebulousLabs/muxado/proto.NewSession
/home/luke/go/src/github.com/NebulousLabs/muxado/proto/session.go:104 +0x4e3
goroutine 104284 [semacquire]:
sync.runtime_notifyListWait(0xc0428ac810, 0xc0000008c9)
/usr/local/go/src/runtime/sema.go:267 +0x130
sync.(*Cond).Wait(0xc0428ac800)
/usr/local/go/src/sync/cond.go:57 +0x87
github.com/NebulousLabs/muxado/proto/buffer.(*Inbound).Read(0xc042573560, 0xc042928a08, 0x4, 0x4, 0x0, 0xc04215e2c8, 0xc0437ffec0)
/home/luke/go/src/github.com/NebulousLabs/muxado/proto/buffer/inbound.go:155 +0x170
github.com/NebulousLabs/muxado/proto.(*Stream).Read(0xc042bc8880, 0xc042928a08, 0x4, 0x4, 0x0, 0x4, 0x4)
/home/luke/go/src/github.com/NebulousLabs/muxado/proto/stream.go:73 +0x5e
io.ReadAtLeast(0x2d677d8, 0xc042bc8880, 0xc042928a08, 0x4, 0x4, 0x4, 0x9c94e0, 0x0, 0x2d677d8)
/usr/local/go/src/io/io.go:307 +0xab
io.ReadFull(0x2d677d8, 0xc042bc8880, 0xc042928a08, 0x4, 0x4, 0x0, 0x0, 0x0)
/usr/local/go/src/io/io.go:325 +0x5f
github.com/NebulousLabs/muxado/proto/ext.(*Heartbeat).respond(0xc0421a7350)
/home/luke/go/src/github.com/NebulousLabs/muxado/proto/ext/heartbeat.go:81 +0x121
created by github.com/NebulousLabs/muxado/proto/ext.(*Heartbeat).Start
/home/luke/go/src/github.com/NebulousLabs/muxado/proto/ext/heartbeat.go:46 +0x80

was there anything above this message? It looks like a partial stack trace, which is usually generated by a panic or a critical error. There should be additional information at the top of the trace containing an error message. Without that, it's hard to diagnose what went wrong.

Unfortunately, this was the entire buffer. Unless there is a log someplace..

Did siad exit after the message? Can you clarify the steps you took? Something like:

Stopped siad
Deleted consensus.db
Moved old wallet.dat to wallet folder
Started siad
siad began downloading blocks
Problem occurred when siad was within 25% of full sync

There are various logs in the Sia directory. You can check them manually, or you can use the new UI and view them with Ctrl+Shift+L.

Closing this because it's not clear how to reproduce, and it may be fixed. Please reopen if you have the issue again.


Made the java binary location customizable

Closure compiler docs (https://code.google.com/p/closure-compiler/wiki/FAQ) recommend the use of 32bit JVM for performance improvements.
I was able to improve my project compile time from 70 seconds to 15 seconds by using their recommendations. The problem is the java binary is hard coded to use your system default.
This is not ideal because most people would be running a 64bit JVM and the 32bit JVM would only be used for a specific purpose, so trying to run a 32 bit JVM alongside a 64bit one would take a lot of tinkering.
With this patch you can specify this in the grunt task config:
{
    javaBin: '/opt/jdk1.8.0_20/bin/java -client -d32',
    closurePath: ...,
}

You can download any JDK and place it somewhere on the system and without setting any environment variables or editing anything etc, you can use it just for doing the compiling to gain the speed increase.
Also, is there any reason this.options() is not used in the task? Seems it would be more efficient for this.

Could someone tell me when this PR is going to be merged?  As this would be a pretty helpful addition.

+1

Wow, this was so close to being merged...


Service principal authenticate with AAD token to Azure Devops repo?

Hi @alexott

Is it impossible to  authenticate service principal with AAD token to Azure Devops repo?

For Azure Devops repo configuration with Azure Devops PAT, how do you configure this for a service principal?

Yes, right now it's impossible to use service principal to authenticate to Azure DevOps (at least this doc lists only user-related tokens).
For 2nd item - right now it's not possible to set DevOps PAT for service principal as corresponding REST API is missing. As I understand it should be added in the near future

@alexott thank you for confirming 👍

@alexott is it possible to create the root repo folder with api or cli ? I cannot seem to find a way to do it and your instructions are to manually pre create them.

Yeah, it’s not obvious because you need to use workspace API or cli command:
databricks workspace mkdirs /Repos/directory

It’s easier a bit with terraform that creates intermediate directories automatically. Like this: https://gist.github.com/alexott/6a8740e6757e2ac0c6ffc3672afdf2a6

@alexott thnx for sharing, now I need to learn Terraform 😄

It was just an example. You can use Databricks CLI as I have shown

Yeah I know, but now I have a reason to go learn Terraform 😂

Thanks for some of this! I had spent the last few weeks trying to get the authentication between Azure Databricks and Azure DevOps to work and couldn't find examples of it or how to get it to work.
I kept getting this error:
{"error_code":"PERMISSION_DENIED","message":"Invalid Git provider credentials. Go to User Settings > Git Integration to ensure that:\n1. You have entered a username with your Git provider credentials.\n2. You have selected the correct Git provider with your credentials.\n3. Your personal access token or app password has the correct repo access.\n4. If you have SSO enabled with your Git provider, be sure to authorize your token."}
Any updates on when this changes would be appreciated!

Unfortunately, right now it's only possible to authenticate using Azure DevOps PAT, not service principal ID/secret

@matthieumarshall Git Credentials API was just released, so you can now set DevOps PAT for service principal as well


experimental/rpm.bzl: use rpmbuild toolchain, re-enable RPM testing in CI

This change modifies the pkg_rpm rule in pkg/experimental/rpm.bzl to use the
rpmbuild toolchain currently in use by the existing RPM packaging rule in
pkg/rpm.bzl.  Logic was largely copied from pkg/rpm.bzl.
Additionally, the RPM tests were removed from the disabled test set -- the
toolchain skipping system should be enough to take care of this on systems that
don't have rpmbuild (everything but CentOS in CI, right now).
Documentation was also updated to reflect bazel version compatibility.  Other
pkg_filegroup-related changes require 4.0.0.
Fixes #299, resolves #183.

Please disregard the CLA message -- I had originally committed using the wrong e-mail.


1st Experience: Preprocessing, Integration, Clustering

Hi,
again just a few little things I ran into while running the steps Preprocessing, Integration, and Clustering.

Preprocessing:

Specifying the parameter "output_logged_mudata" in the pipeline.yml did not work for me, it threw an error. But in the pipeline.yml it is stated as "TODO" that this parameter is supposed to go, so I guessed that this error isn't so important. When leaving this parameter empty, the preprocessing worked completely fine

Integration:

Running "panpipes integration make plot_pcas --local" resulted in an error, stating:
"Target task 'plot_pcas' is not a pipelined task in Ruffus. Is it spelt correctly ?"

Clustering:

So that others don't experience the same error, it may be helpful to mention that one needs to specify more than one resolution, otherwise clustree throws an error:
"Error: Less than two column names matched the prefix: leiden_re
Execution halted"

Best,
Sarah

Thanks Sarah,
Preprocess:
Did you specify a True/False value here or a path? I believe it is supposed to be a path. But also you are correct that we don't intend to maintain that functionality. Since we are utilising scanpy layers instead.
Integration:
You are correct that "plot_pcas" does not exist as a task. You can always work out what the tasks are called by running panpipes integration show full. I would expect the pcas to be plotted in the preprocess step, did they not plot?
Clustering:
I have referenced the clustree issue in the yml, and in the documentation on the website. https://panpipes-pipelines.readthedocs.io/en/latest/workflows/clustering.html

I will leave this open to remind myself to remove the "output_logged_mudata" parameter

Thank you!
Regarding the integration, I ran panpipes integration make plot_pcas --local because it is still mentioned in the README.md: https://github.com/DendrouLab/panpipes/blob/main/README.md, namely: 2. Run panpipes integration make plot_pcas and assess the post filtering qc plots, and pca outputs
But yes, the PCA is plotted in the preprocess step

Great! I will change the README. I'm hopeful that people will start using
the website over the github. But its important to have it be correct anyway!
On Tue, 22 Aug 2023, 19:01 Sarah, @.***> wrote:

Thank you!
Regarding the integration, I ran panpipes integration make plot_pcas
--local because it is still mentioned in the README.md:
https://github.com/DendrouLab/panpipes/blob/main/README.md, namely: 2.
Run panpipes integration make plot_pcas and assess the post filtering qc
plots, and pca outputs
But yes, the PCA is plotted in the preprocess step
—
Reply to this email directly, view it on GitHub
https://github.com/DendrouLab/panpipes/issues/45#issuecomment-1688665057,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGEUUCU5XIDBT2TWOEE5NI3XWTXV3ANCNFSM6AAAAAAX6JPT6Q
.
You are receiving this because you commented.Message ID:
@.***>


[FIX] Flush constraints system to files with incremental names to avo…

…id "Out-of-Memory" on some systems.

Solved in another PR.


Minor refactoring and fixes to the LCI parcelport and pingpong_performance2 benchmark

For the LCI parcelport:

Add an option to enable/disable in-buffer assembly of header messages
Fix the lci_progress_poll in cases where there are multiple pus per core.
Minor changes to the log output.

For the pingpong_performance2 benchmark

Add warmup

Performance test reportHPX PerformanceComparisonBENCHMARKFORK_JOIN_EXECUTORPARALLEL_EXECUTORSCHEDULER_EXECUTORFor Each(=)??(=)InfoPropertyBeforeAfterHPX Datetime2023-05-10T12:07:53+00:002024-01-11T05:48:22+00:00HPX Commitdcb541576898d370113946ba15fb58c20c8325b2f6806e7401269ef9b1b067f0680ae3665016ca77Compiler/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1Datetime2023-05-10T14:50:18.616050-05:002024-01-10T23:54:33.744654-06:00EnvfileClusternamerostamrostamHostnamemedusa08.rostam.cct.lsu.edumedusa08.rostam.cct.lsu.eduComparisonBENCHMARKNO-EXECUTORFuture Overhead - Create Thread Hierarchical - Latch(=)InfoPropertyBeforeAfterHPX Datetime2023-05-10T12:07:53+00:002024-01-11T05:48:22+00:00HPX Commitdcb541576898d370113946ba15fb58c20c8325b2f6806e7401269ef9b1b067f0680ae3665016ca77Compiler/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1Datetime2023-05-10T14:52:35.047119-05:002024-01-10T23:56:47.487942-06:00EnvfileClusternamerostamrostamHostnamemedusa08.rostam.cct.lsu.edumedusa08.rostam.cct.lsu.eduComparisonBENCHMARKFORK_JOIN_EXECUTOR_DEFAULT_FORK_JOIN_POLICY_ALLOCATORPARALLEL_EXECUTOR_DEFAULT_PARALLEL_POLICY_ALLOCATORSCHEDULER_EXECUTOR_DEFAULT_SCHEDULER_EXECUTOR_ALLOCATORStream Benchmark - Add(=)(=)=Stream Benchmark - Scale+(=)(=)Stream Benchmark - Triad(=)=(=)Stream Benchmark - Copy(=)-(=)InfoPropertyBeforeAfterHPX Datetime2023-05-10T12:07:53+00:002024-01-11T05:48:22+00:00HPX Commitdcb541576898d370113946ba15fb58c20c8325b2f6806e7401269ef9b1b067f0680ae3665016ca77Compiler/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1Datetime2023-05-10T14:52:52.237641-05:002024-01-10T23:57:04.260199-06:00EnvfileClusternamerostamrostamHostnamemedusa08.rostam.cct.lsu.edumedusa08.rostam.cct.lsu.eduExplanation of SymbolsSymbolMEANING=No performance change (confidence interval within ±1%)(=)Probably no performance change (confidence interval within ±2%)(+)/(-)Very small performance improvement/degradation (≤1%)+/-Small performance improvement/degradation (≤5%)++/--Large performance improvement/degradation (≤10%)+++/---Very large performance improvement/degradation (>10%)?Probably no change, but quite large uncertainty (confidence interval with ±5%)??Unclear result, very large uncertainty (±10%)???Something unexpected…

@hkaiser I think this PR is ready to be merged.


PreEvaluate/PostEvaluate stages

Following the discussion engaged about an Execute Hook #47,  I would see value in the ability for developers to hook into the PreEvaluate & PostEvaluate stages of modules loading.
The PreEvaluate stage contains the state of the runtime prior to module evaluation, and the PostEvaluate stage contains the state of the runtime after module evaluation.
The access to those stages may allow for various scenarios both in development and in production, like runtime diffing and analysis, orchestration and advanced tracing strategies, modules based state-machines, etc, that would not be possible within the instantiation stage only.
Update: renaming to PreEvaluate and PostEvaluate for conformity with the spec.

Similarly, and echoing the issue #18 about hooks and state, and the issue #89 about manually canceling pipelines stages, I would find quite useful to have, instead of single points entries into the loader pipeline, dual point entries referring to the beginning and the ending of each stage:
preFetch/postFetch
preTranslate/postTranslate
preInstantiate/postInstantiate
preEvaluate/postEvaluate

Does it make any sense ?

you can achieve this by simply defining a hook that applies AOP for the existing (or default) hook, and return a promise that allow you to run an async process before and after.

hmm, that would work if there was an evaluate hook, but there is no way apparently that an import statement informs its context when it has been evaluated.
One could do it with System.import returning a promise but coding everything like this would result in lots of spaghetti code, and won't take into account the declarative "import" statements:
Loader.import(module).then(function(module){
//postEvaluate Stage
}).then(Loader.import(othermodule).then(function(othermodule){
//postEvaluate Stage
}));

Whereas:
import exp from 'module';
import otherexp from 'othermodule';

and
function evaluate(){
//PreEvaluate Stage

//evaluate

//PostEvaluate Stage
}

would be simpler...

no, that's not what I meant, and importing a module does not imply evaluation. but before I try to explain how to achieve this, I wonder is you have more use-cases, specifically things that you might do in production?
it seems that all the use cases that you mentoned are related to instrumentation, which is very low-level feature, and having a hard time to achieve is probably not an issue. I think the realms and the instantitate hook will be enough here.

perhaps it is, and am going beyond the functionality of the loader.
besides instrumentation, debugging or more advanced introspective usage of the loader functionality (because of its intrinsic knowledge of modules), I was thinking, as one use-case, modules like sandboxed processes in an Actor Model context, with the Loader acting as a global registry of processes.
Once we import the modules, they have to communicate with each other, and the Loader seems to be a good center of communication between the modules, as it holds all the information about each and every module loaded at runtime, in his registry.
Once we know when a module is evaluated, and therefore that it is an active actor of the application, that it is loaded, we can make it to participate to the communication pipeline between modules, and therefore messages can be sent to and from it.
This would require getting the information for when a module is ready, like the onload event of a script tag, as I conveive a postEvaluate hook somehow as a ready stage.
If the ready stage does not have a hook, would the modules be evented, with a ready event to listen to, like an onload event ?

Ok, that confirms my suspicious! you will probably create a realm, and evaluate any model-type module in that realm, maybe using a proxy as the global of the realm to have full control of what each module is attempting to do.
I am not trying to judge your architecture, just to make sure that what we have is flexible enough to support it. But I must say that if what you need is a communicacion layer between similar modules, you could probably do the same using a module that they are required to import to access this communicacion api.

I see, I'll have to get more information about the realm. Thanks for having taken the time to answer and for the suggestions. Have to ponder more about all this anyway. Closing the issue for now, as it is not a main one. A

all questions are welcome! it helps us to validate what we have here.


table-hover regression

The behavior of bootstrap 4 when doing a mouse-over on a table row, the row would get a light gray background color. That gray color filled the row perfectly and the gray color is the same shade throughout.
With bootstrap 5, the table hover gray color has two shades. An outer block shade that is the light gray I'm familiar with then an inner block shade that is a darker gray and it doesn't look right.
FireFox 85.02 on Microsoft Windows [Version 10.0.19042.804]

                    Brand Name

                      brand name goes here

                      delete button goes here

</div

In

At bottom of

This appears to be because of the .table class on the
. Removing that clears the issue for me.

Example: https://codepen.io/pen


sensor: drivers: Panasonic sngcja5 particle matter sensor

Support for Panasonic's SN-GCJA5 Particle Matter Sensor.
Added sensor drivers and sensor sample for example usage.

Hello @MaureenHelm
sorry was caught on other projects on last weeks, but will go on here in the next days. Could you please remove the stale label?

Please rename the sample and update the documentation to reflect the change making it generic to any particle sensor driver

Done in last update. Changed naming and documentation from sample sngcja5 to particulate_matter.

@MaureenHelm
Did you already found some time to review?

Hi @galak @avisconti @teburd ,
can maybe someone else review?

@MaureenHelm
could we maybe reopen this PR and remove the stale label?
It's been a while, but in comment https://github.com/zephyrproject-rtos/zephyr/pull/55765#issuecomment-1579962884 I said, that last requested changes from you site are added. So can we merge this PR now? @avisconti already gave the approval in the meantime.

Hi @teburd and @avisconti
had to rebase to match the compliance check and update manifest to pass all checks after removal of stale label.

Hi @MaureenHelm, @avisconti,
could you maybe find some minuets to check if we can merge the PR?
Thanks a lot.

Hi to all listed reviewers,
if @MaureenHelm and @avisconti are not available, could someone else maybe review?
Thanks a lot!

Hello to the reviewers in the list @avisconti, @nashif, @galak, @yperess, @kartben, @tristan-google,
the Pull-Request already got two times the stale label. Can maybe someone else review if @MaureenHelm is not available?
Thanks a lot!

@MaureenHelm and other core contributors to the sensor API,
could you confirm, that I should change sensor channels for particle count, like @kartben suggested here: https://github.com/zephyrproject-rtos/zephyr/pull/55765/files#r1391162757
Thanks a lot.


lua contexts without menu entries don't repeat

duh, they play the intro, then are immediately done repeating the empty menu statements, then hang up. Which is not a good outcome for contexts with only intro statements. sigh, maybe make a special menuless context that repeats the intros
    apology_about = util.context(
        {intro_statements={
             "apologies-are-not-recorded-or-listened-to",
             "this-apology-service-is-an-homage-to-allen-bridges-apology-line",
             "we-hope-you-find-the-act-of-apologizing-helpful"},
         menu_entries={},
         statement_dir="apology-service"})

or, also sigh, add a menu_statements arg that is played within the menu repeats but doesn't contribute menu entries

two other solutions:
just use a menu entry with a destination that goes to same menu, instead of intro statements
context type which bounces to previous context when done, instead of hanging up


doc: add github-merge.py documentation

This has been ported from the bitcoin/bitcoin repo, with some small changes to make it more general.

@real-or-random or @jonasnick, either of you might be interested in looking over this as part of getting setup with the secp256k1 repo?


Added built in parsing of DOM data attributes to populate this.data in base Component

Suggestion for built in default data configuration and automatic detection of DOM data attributes.
Also added a this.bind(func1, func2) to bind many event handlers in one call.

I think it's great, look forward to start using the defaults and bind functionality. Also, unit tests for the win!

Looks great so I merged it.  Look forward to start using the defaults and bind functionality. Also, unit tests for the win!


error correction

I noticed an error, probably a wrong copy/paste.

Hey @xplosionmind, thanks for the fix!


feat: support IS [NOT] NULL in queries

foo=null  ---> WHERE "foo" IS NULL
foo!=null ---> WHERE "foo" IS NOT NULL
foo=NuLl  ---> WHERE "foo" IS NULL

I've the feeling we are repeating a mistake here: https://www.wired.com/story/null-license-plate-landed-one-hacker-ticket-hell/

NiceOrExcellent!

Haha. Fwiw, it was one of your recent commit messages that inspired the new name.


Updated WSUS Maintenance URL

Updated to https://support.microsoft.com/en-us/help/4490644/complete-guide-to-microsoft-wsus-and-configuration-manager-sup-maint
Summarize the change in the pull request title
An update to the article was published and is now the most current, relevant information.
Fixes #Issue_Number (if necessary)

@dhedges01 : Thanks for your contribution! The author, @mestew, has been notified to review your proposed change.

@dhedges01 : Thanks for your contribution! The author, @mestew, has been notified to review your proposed change.

#sign-off


Formatting ReScript file fails with error

When attempting to format a ReScript file, Onivim fails to format the file and throws an error.
Steps to Reproduce

Download and install rescript-vscode extension vsix file.
Launch Onivim 2.
:enew
:w test.res
Cmd+Shift+P
Type "format" and select "Formatting: Format Document."
:enew
Switch tab back to first file.
Cmd+Shift+P
Type "format" and select "Formatting: Format Document."

Expected Result
The ReScript file (res) is formatted. No errors are displayed.
Actual Result
The ReScript file (res) is not formatted. Errors are displayed.
The first format call produces: Invalid range specified.
The second format call produces: ParseFailedException("Expected null or:\n while decoding a list:\n element 0:\n in field \"range\":\n in field \"endColumn\":\n Expected an int, but got 1.7976931248623157e+308\n".

Environment

macOS 11.0.1
Onivim 2 0.5.1-nightly 0.5.1718

Thanks @dcalhoun for all the details and the screenshot. Very helpful!
I pushed up a fix for the parse exception - #2825 - we weren't handling some types of numerical values correctly in our JSON parsing for ranges.
I'm still seeing some quirks though with the ReScript formatter, even after that - possibly related to / similar issue as: https://github.com/onivim/oni2/issues/2196

@bryphe I didn't necessarily test this again, but figured I'd close this based upon your comments/work within #3233. Feel free to reopen if I am mistaken.

Thanks @dcalhoun for closing! Indeed, I did a quick test w/ #3233 - and I wasn't able to reproduce the issue anymore.
Of course, keep me posted if you hit any other issues with formatting :+1:


NRCS-NCSS Web of Soils

Web of Soil version of NRCS-NCSS https://websoilsurvey.nrcs.usda.gov/app/WebSoilSurvey.aspx

[x] download data
[x] annotate data
[x] shoestring data

Is this covered off by soilDB?
Apologies if this is daft, or has already been covered, but I find the relationship between their data and their outward facing products/services confusing.

@brandonnodnarb Yes, we should link into the package and I also wanted to go through the annotation process with the database as well. I'm aware of this data package but haven't dug into the details here yet. If it links to the backend used to generate the web of science version of the database then the structure is a bit different.

The report and read function is drafted and could use more comments/details.

More comments/details have been added to the report and read function. Note: section 2 (Soil organic carbon data processing) of the report file could use more details/comments from someone who fully understands the statistical value/idea that the charts bring/showcase.

@vaasukimarupaka Can you take a look at the NRCS report and read function documentation?


"instances i-xxxxxxxx" dies with error

~/Repos/bash-my-aws$ instances i-xxxxxxxx
Bad value for --query Reservations[].Instances[][InstanceId,[Tags[?Key==Name].Value][0][0]] i-13d381cd: Unexpected token: i: Parse error at column 72, token "i" (UNQUOTED_IDENTIFIER), for expression:
"Reservations[].Instances[][InstanceId,[Tags[?Key==Name].Value][0][0]] i-xxxxxxxx"

oops, my bad. I forgot to delete an alias. :)


Speed up eq

Create the __eq__ as a chain of ANDed comparisons, instead of a tuple.
On my computer, a class with two attributes now compares almost twice as fast: 141 ns vs 222 ns.
This will introduce a very slight incompatibility which I think is worth it for us (and dataclasses already introduced): attributes that compare equal by identity but not by value (= NaNs) will behave differently.

You might want to update the comment above those lines that says we can't do exactly what we're doing now 😄
https://github.com/python-attrs/attrs/blob/13e9a6aaeac156bf792878eceaba10321728fa36/src/attr/_make.py#L1535-L1536

You might want to update the comment above those lines that says we can't do exactly what we're doing now 😄
https://github.com/python-attrs/attrs/blob/13e9a6aaeac156bf792878eceaba10321728fa36/src/attr/_make.py#L1535-L1536

thanks, it's so hard to find quality free labor 🤪


Need to add note about System.Drawing.Common being deprecated in Linux in .NET 7

According to @svick  noted here (https://github.com/dotnet/runtime/issues/80148), the support for System.Drawing.Common has now been completely deprecated in .NET 7; thus, the System.Drawing.EnableUnixSupport setting in runtimeconfig.template.json is no longer valid.
The list of Breaking changes in .NET 7 does not include any information about the deprecation of Unix support for System.Drawing.Common or the deprecation of the System.Drawing.EnableUnixSupport config setting.  This info should be added to the list of Breaking Changes so that developers are aware of this change.

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

ID: fda38618-d1fc-5b45-9a09-1a02643d99ae
Version Independent ID: e59e516b-dd93-937d-f9a1-bce4b5eb3aa9
Content: Breaking changes in .NET 7 - .NET
Content Source: docs/core/compatibility/7.0.md
Product: dotnet-fundamentals
GitHub Login: @gewarren
Microsoft Alias: gewarren

@udlose This was technically a breaking change in .NET 6, not .NET 7, and as such is listed here: https://learn.microsoft.com/en-us/dotnet/core/compatibility/core-libraries/6.0/system-drawing-common-windows-only. This article also contains a note that the switch to re-enable support was removed in .NET 7 (and I don't believe the removal of the switch is considered an additional breaking change).

Related to #33653.


Fix new clippy lints, added null-by-default fields in httpd_ssl_config_t"

new fields added in esp-idf/master, all defaulting to NULL
fix new clippy lints

Thanks!


great idea, currently it dies ;(

hey, testing the repo, but it fails in the simplest of cases, hooking to a UIView, in latest bits of Xcode:
"..<UICustomObject 0x170011580> setValue:forUndefinedKey:]: this class is not key value coding-compliant for the key targets. ..."

My bad, sorry, i thought i set both the class and the module but missed the module ;) It's all good now, and a groovy little repo!


moved the file to the correct directory 'edu', inside the directory '…

…MX'.

@Gnaevs First stage of verification passed, official domain is lania.mx as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision. Proof of domain ownership: https://www.lania.mx/

@Gnaevs Pull request merged. Thank you.


bug: congr application type mismatch

The following minimal reproducer:
import Duper

class Magma (α : Type _) where
op : α → α → α
infix:65 " ◇ " => Magma.op

abbrev Equation1689 (G: Type _) [Magma G] := ∀ x y z : G, x = (y ◇ x) ◇ ((x ◇ z) ◇ z)
abbrev Equation2 (G: Type _) [Magma G] := ∀ x y : G, x = y

example (G: Type _) [Magma G] (h: Equation1689 G) : Equation2 G := by duper [h] {portfolioInstance := 2}

Hits the following exception:
repro.lean:11:71: error: application type mismatch
  Eq.mp (congrArg (fun x => e!_0 x (e!_0 ✝ [anonymous]) = x) (Eq.symm heq)) h
argument
  h
has type
  e!_0 (e!_0 (e!_0 (e!_0 ✝ [anonymous]) (e!_0 ✝ [anonymous])) (e!_0 ✝ [anonymous])) (e!_0 ✝ [anonymous]) =
    e!_0 (e!_0 ✝ [anonymous]) (e!_0 ✝ [anonymous]) : Prop
but is expected to have type
  e!_0 (e!_0 (e!_0 ✝ [anonymous]) (e!_0 ✝ [anonymous])) (e!_0 ✝ [anonymous]) =
    e!_0 (e!_0 ✝ [anonymous]) (e!_0 ✝ [anonymous]) : Prop

Note that this example is taken from https://github.com/teorth/equational_theories/
In particular I mention duper here on Zulip.

This bug is now resolved in the most recent Duper version (v0.0.20)


Adding/changing a thumbnail for a Favorite bookmark creates a duplicate Favorite bookmark

What steps will reproduce the problem?

Create or use an existing Favorite bookmark.

Under the Favorites tab, right-click on that bookmark to bring up the Edit A Favorite window

Click the Browse button and select an image for the thumbnail, then click OK.

What is the expected behavior? What do you get instead?
Expected behaviour: Favorite bookmark's thumbnail would be updated with the newly selected thumbnail.
Actual result: The Favorite bookmark thumbnail is updated with the newly selected thumbnail, then a duplicate Favourite bookmark is created.
What version of the program are you using? On what operating system?
Grabber v4.7.0 (commit a18dc84)
Windows 10 Pro 64-bit
Please provide any additional information below

The duplicate Favorite bookmark persists between program shutdowns.

Deleting either of the two bookmarks will delete both the duplicate and the original bookmark.

Bug is reproducible by editing the thumbnail of any favourite tag under the Favourites tab.

Strange bug indeed. While fixing issue #615, I also fixed this.
The fix should be included in the next release. Thanks for the report! :smile:


After Marking a date it should not refreshed or notified

i solved this issue just add notifyObservers() to brlove method in MarkedDates.java
public MarkedDates add(DateData dateData){
data.add(dateData);
this.setChanged();
notifyObservers();
return this;
}

Merged. Thanks.


Autocomplete merge type

Fixes #269
I never did any development with azure devops, and I could not figure out how to test this properly.
Does anyone mind lending me a hand?
Thanks!

@btastic thanks for taking time to contribute.
The value extracted from AZURE_SET_MERGE_STRATEGY env needs a default in ruby values so that usage out of Azure DevOps (e.g. using a Kubernetes CronJob) continues to work without needing it to be specified. You could have ENV['AZURE_SET_MERGE_STRATEGY'] || "2",

@mburumaxwell Thanks! Just pushed the change

I also renamed the variable from AZURE_SET_MERGE_STRATEGY to AZURE_MERGE_STRATEGY, since I originally copied the flow of the AZURE_SET_AUTO_COMPLETE variable


Crash when Python method returns a string

Given a C++ parent class like this:
UCLASS()
class UEP_CRASH_API AParentActor : public AActor
    ...
    UFUNCTION(BlueprintCallable, BlueprintImplementableEvent) FString ReturnStr();
    UFUNCTION(BlueprintCallable, BlueprintImplementableEvent) int ReturnInt();

and a Python subclass of it like this:
class PyChildActor(ParentActor):
    def ReturnInt(self) -> int:
        return 1234

    def ReturnStr(self) -> str:
        return 'foo'

If you place a PyChildActor in your scene and then have a different Blueprint call ReturnInt, it works. But if a Blueprint calls ReturnStr, UE4 crashes in CallPythonCallable when writing the return value into the return property:
if (ue_py_convert_pyobject(ret, return_property, frame, 0)) {

The crash is in SetPropertyValue_InContainer.
I'm using UE4.19 + Python 3.6.7 and the latest from UnrealEnginePython's master branch.
Attached is a self-contained project that demonstrates the problem. It has the C++ base class and then 2 subclasses, one in BP and one in Python. Both subclasses implement ReturnInt and ReturnStr. When you launch the game, the level BP spawn the BP subclass and calls its ReturnInt and ReturnStr methods to verify that they work. It then spawns the Python subclass and calls its same methods. The call to ReturnInt works, the call to ReturnStr triggers the crash.
uep_crash.zip

FYI I've been working on issue #621 and the fix for it seems to have fixed this problem too. I'm doing some more testing but hope to soon submit a patch for your consideration.


Custom raster provider

Hi,
Is it possible to make attributes of QgsRasterLayer protected to be able to customize provider creation.
It can be usefull to create a custom class, which inherit QgsRasterDataProvider and implement data access in it.
This will allowed to create custom provider without plugins.
Thanks,
Clément

Please could you ask such question to the developer mailing list: http://qgis.org/en/site/getinvolved/mailinglists.html#qgis-mailinglists and http://qgis.org/en/site/getinvolved/faq/index.html#how-to-ask-a-qgis-question
Thanks,


RC Test: Graphing Quadratics 1.1.0-rc.1 PhET-iO

@pixelzoom @kathy-phet @amanda-phet @arouinfar @samreid @chrisklus @zepumph, graphing-quadratics/1.1.0-rc.1 is ready for RC testing.
The goal is to have this sim published by Friday, July 19th. Document issues in https://github.com/phetsims/graphing-quadratics/issues or https://github.com/phetsims/phet-io/issues (if appropriate) and link to this issue. This simulation is the first phet-io publication in over two years. Good work team for getting this far!
Assigning to @ariel-phet for prioritization.

General RC Test

What to Test

Click every single button.
Test all possible forms of input.

Test all mouse/trackpad inputs.
Test all touchscreen inputs.

If there is sound, make sure it works.
Make sure you can't lose anything.
Play with the sim normally.
Try to break the sim.
Test all query parameters on all platforms. (See QA Book for a list of query parameters.)
Download HTML on Chrome and iOS.
Make sure the iFrame version of the simulation is working as intended on all platforms.
Make sure the XHTML version of the simulation is working as intended on all platforms.
Complete the test matrix.
Don't forget to make sure the sim works with Legends of Learning.
Check this LoL spreadsheet and notify AR or AM if it not there.
If this is rc.1 please do a memory test.
When making an issue, check to see if it was in a previously published version

Link(s)

Simulation
iFrame
XHTML
Test Matrix
Legends of Learning Harness

PhET-iO RC Test

What to Test

Make sure that public files do not have password protection. Use a private browser for this.
Make sure that private files do have password protection. Use a private browser for this.
Make sure standalone sim is working properly.
Make sure the wrapper index is working properly.
Make sure each wrapper is working properly.

Please give extra focus to studio, mirror-inputs, and state wrapper.

Launch the simulation in Studio with ?stringTest=xss and make sure the sim doesn't navigate to youtube
For newer PhET-iO wrapper indices, save the "basic example of a functional wrapper" as a .html file and open it. Make sure the simulation loads without crashing or throwing errors.

Link(s)

Wrapper Index
Test Matrix

FAQs for QA Members

There are multiple tests in this issue... Which test should I do first?
Test in order! Test the first thing first, the second thing second, and so on.

How should I format my issue?

We typically assign the developer who opened the issue in the QA repository.

My question isn't in here... What should I do?
You should:

Consult the QA Book.
Google it.
Ask Katie.
Ask a developer.
Google it again.
Cry.

Testers: General matrix is up.  PhET-iO matrix will go up as well as an update to the QA book with diff wrapper instructions.  Will update this comment when that is finished.

Diff wrapper instructions, now in QA book:

Dif Wrapper Test
What? This is a wrapper that allows you to compare the differences between a previous build and the current build.
How? To test this wrapper you will need to test two different cases by entering a url into the box. The first url should be for the wrapper currently being tested, and it should show no differences. The second url will be provided in the issue and should show some difference.

Note that there is no usable second url at this time.  So for this test just use the first url (the wrapper url we are currently testing.)  If this changes I will let you know and post the new url.

@ariel-phet since this is such a long RC (phet-io) and is on a tight schedule I'm going to move it to top priority unless you have any objections.

Metacog recording should only be done by:

[ ] Win 10 chrome
[ ] Mac 10.4 safari
[ ] Chrome OS

The URL is: https://phet-dev.colorado.edu/html/graphing-quadratics/1.1.0-rc.1/phet-io/wrappers/login/?wrapper=record&validationRule=validateDigits&encryption=encryptNumberV1&numberOfDigits=9&publisher_id=5d4c8ae1&key_name=phet-study_2018_q4&widget_id=graphing-quadratics-test_rc&phetioEmitInputEvents=true&phetioEmitStates=true&phetioEmitStatesInterval=60&metacog
See Katie for the id to get the data

On Edge (Phet Brand) I am seeing a fairly sizable https://github.com/phetsims/energy-forms-and-changes/issues/203 style issues where parts of the line will disappear or be drawn outside of the graph.

@KatieWoe that does not warrant an issue, I think we are going to generally ignore odd rendering issues on Edge and wait them out until the chromium version unless they are truly egregious. It seems the issues you had discussed were happening every now and then, not all the time.

QA is done

Since all of the issues discovered during this RC have been addressed, we are planning to maintenance patch the fixes into the RC branch, publish an RC and have it spot checked.


Replace idea-test-kit-fix script with plugin

Coverage remained the same at 99.107% when pulling 34929b8bf69cba0bb062524345a02bf5c8f36232 on nmiyake:ideaPlugin into 498e73fbba9b173a3a72ae21cb0e07b9a7acd1b3 on palantir:develop.

Is this a dup of #65?

Yup, this is a dupe -- forgot about that one :). Will close this one and update the other.


[MTLDebugRenderCommandEncoder setRenderPipelineState:]:1298: failed assertion `renderPipelineState must not be nil.

Description
I have been trying to migrate my ios application to sdk 11.4 and using the existing metal api. After application launch in iPad through xCode, I got a white screen and an error indicating that renderPipelineState must not be nill. Here is the code which has been failing:
id<MTLRenderCommandEncoder> cEncoder = _context.GetCurrentRenderEncoder(); [cEncoder setRenderPipelineState:quadShader->GetRenderPipelineState( SQShaderMetal::BE_StandardAlphaBlend, &_context )];
After analyzing more I found that some MetalQuad library has not been opened successfully and that code have been failed with an error. Here is the code to open MetalQuad library

In the above image iMTLLibraryData is MetalQuad.metallib libary, which eventually not gives any new library for a MTLDevice and gives error that this format is not supported or built on old tools.
Reproduces How Often
Everytime
Xcode
xCode 9.4
iOS Version
iOS SDK 11.4
iOS Device
iPad pro

@raakshash Sorry for being late in responding back. In order to debug the problem further, can you provide following information -

Twilio Video SDK version
Is it possible for you to explain the used case you are trying to achieve?
Are you writing your custom renderer using TVIVideoRenderer APIs? Please note that the video SDK provides Metal and OpenGL ES renderer by default, but you are free to write your own custom renderer.

@piyushtank I am not sure regarding the renderer. also I dont have any used cases.
But as I see that my application is not running further because of this error.
Also one thing that we are using a custom code builder which uses command line tool with some options

is possible that due to code builder this problem is coming? Also if the options given with the metal command are old then will it build the metal APIs in old format? Because the error is saying that the library format is not supported on the platform or build with old version of the tool

@raakshash It is very hard to debug the problem without knowing what you are trying to achieve using video sdk. Are you able to reproduce the problem on our sample app?

Hi @raakshash,
From the discussion so far, you don't appear to be using TVIVideoView, but rather your own Metal renderer. Are you looking for advice on how to solve a problem in your application renderer code? If so, please provide the source code for at least the renderer (if not a runnable version of the app) and we will do our best to help.
I'm closing this ticket, because its not actionable at this time. If you have more information to share please feel free to reopen this issue.
Thanks,
Chris


Change mixer_codegen.sh to use protoc dockerfile

This is starting the work discussed in https://github.com/istio/istio/pull/6606/files

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: gargnupur
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: myidpt
If they are not already assigned, you can assign the PR to them by writing /assign @myidpt in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@gargnupur: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
6f805d028598d4184bd54f2ebffc9fd5d9a340f0
link
/test e2e-bookInfo-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@gargnupur: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
6f805d028598d4184bd54f2ebffc9fd5d9a340f0
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/e2e-simpleTests.sh
85a7b108068e0d373908de9ebb83f149455c6178
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@gargnupur: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
16e03052eaa94288cac199e55fc797b5e98d3101
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@gargnupur: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
16e03052eaa94288cac199e55fc797b5e98d3101
link
/test e2e-simple

prow/e2e-bookInfoTests-v1alpha3.sh
16e03052eaa94288cac199e55fc797b5e98d3101
link
/test e2e-bookInfo-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@gargnupur: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
16e03052eaa94288cac199e55fc797b5e98d3101
link
/test e2e-simple

prow/e2e-bookInfoTests-v1alpha3.sh
16e03052eaa94288cac199e55fc797b5e98d3101
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/e2e-dashboard.sh
16e03052eaa94288cac199e55fc797b5e98d3101
link
/test e2e-dashboard

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@gargnupur: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
16e03052eaa94288cac199e55fc797b5e98d3101
link
/test e2e-simple

prow/e2e-bookInfoTests-v1alpha3.sh
16e03052eaa94288cac199e55fc797b5e98d3101
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/e2e-dashboard.sh
16e03052eaa94288cac199e55fc797b5e98d3101
link
/test e2e-dashboard

prow/e2e-mixer-no_auth.sh
16e03052eaa94288cac199e55fc797b5e98d3101
link
/test e2e-mixer-no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@gargnupur: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
16e03052eaa94288cac199e55fc797b5e98d3101
link
/test e2e-simple

prow/e2e-bookInfoTests-v1alpha3.sh
16e03052eaa94288cac199e55fc797b5e98d3101
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/e2e-dashboard.sh
16e03052eaa94288cac199e55fc797b5e98d3101
link
/test e2e-dashboard

prow/e2e-mixer-no_auth.sh
16e03052eaa94288cac199e55fc797b5e98d3101
link
/test e2e-mixer-no_auth

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
b9e2fe2619edba34db51a7be1716f5fc51ecc73f
link
/test istio-pilot-e2e-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@gargnupur: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
16e03052eaa94288cac199e55fc797b5e98d3101
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/e2e-dashboard.sh
16e03052eaa94288cac199e55fc797b5e98d3101
link
/test e2e-dashboard

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
b9e2fe2619edba34db51a7be1716f5fc51ecc73f
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-mixer-no_auth.sh
b9e2fe2619edba34db51a7be1716f5fc51ecc73f
link
/test e2e-mixer-no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@gargnupur: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
16e03052eaa94288cac199e55fc797b5e98d3101
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
b9e2fe2619edba34db51a7be1716f5fc51ecc73f
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-mixer-no_auth.sh
b9e2fe2619edba34db51a7be1716f5fc51ecc73f
link
/test e2e-mixer-no_auth

prow/e2e-dashboard.sh
b9e2fe2619edba34db51a7be1716f5fc51ecc73f
link
/test e2e-dashboard

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@gargnupur: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
b9e2fe2619edba34db51a7be1716f5fc51ecc73f
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-mixer-no_auth.sh
b9e2fe2619edba34db51a7be1716f5fc51ecc73f
link
/test e2e-mixer-no_auth

prow/e2e-dashboard.sh
b9e2fe2619edba34db51a7be1716f5fc51ecc73f
link
/test e2e-dashboard

prow/istio-unit-tests.sh
7f85fa1ab7c63905fb9926dccf372d7a70b2936b
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@gargnupur: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
7f85fa1ab7c63905fb9926dccf372d7a70b2936b
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@gargnupur: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
7f85fa1ab7c63905fb9926dccf372d7a70b2936b
link
/test e2e-simple

prow/istio-unit-tests.sh
0938f4ec4d3be9be2096a8203260fc0a522cd095
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@gargnupur: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
0938f4ec4d3be9be2096a8203260fc0a522cd095
link
/test istio-unit-tests

prow/e2e-simpleTests.sh
0938f4ec4d3be9be2096a8203260fc0a522cd095
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@gargnupur: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
0938f4ec4d3be9be2096a8203260fc0a522cd095
link
/test istio-unit-tests

prow/e2e-simpleTests.sh
0938f4ec4d3be9be2096a8203260fc0a522cd095
link
/test e2e-simple

prow/e2e-mixer-no_auth.sh
0938f4ec4d3be9be2096a8203260fc0a522cd095
link
/test e2e-mixer-no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@geeknoid , @ozevren , @mandarjog , @douglas-reid : all the tests passed, can you guys please take a look at this again for lgtm?

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: myidpt
If they are not already assigned, you can assign the PR to them by writing /assign @myidpt in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@gargnupur: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@gargnupur: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
9b0664dd4cfa7487b6f441a3bd786df05bac9c16
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@gargnupur: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
9b0664dd4cfa7487b6f441a3bd786df05bac9c16
link
/test istio-unit-tests

prow/istio-presubmit.sh
9b0664dd4cfa7487b6f441a3bd786df05bac9c16
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@gargnupur: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
9b0664dd4cfa7487b6f441a3bd786df05bac9c16
link
/test istio-presubmit

prow/istio-unit-tests.sh
034f39766f0855bcde9210d28a4c0c695eabc505
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/lgtm
/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: geeknoid, ozevren
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [geeknoid]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@gargnupur: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
034f39766f0855bcde9210d28a4c0c695eabc505
link
/test istio-unit-tests

prow/istio-pilot-multicluster-e2e.sh
034f39766f0855bcde9210d28a4c0c695eabc505
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@gargnupur: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
034f39766f0855bcde9210d28a4c0c695eabc505
link
/test istio-unit-tests

prow/istio-presubmit.sh
034f39766f0855bcde9210d28a4c0c695eabc505
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@gargnupur: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
034f39766f0855bcde9210d28a4c0c695eabc505
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test e2e-simple

@gargnupur: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
034f39766f0855bcde9210d28a4c0c695eabc505
link
/test e2e-bookInfo-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Remove circleCI and update repository references

Removed unnecessary circle CI config.
Update all the references to github.com/3scale/kourier to knative/net-kourier.

Update:

Fixed: The changelog is broken due to tags missing in this repo.

utils/setup-circle-ci.sh should be deleted as well, right?

/lgtm

/lgtm

/lgtm

/hold

Superseded by #2


can we add "activate" method in docs?

Hi there,
Can we add "activate" method in docs?
it's would be helpful to open select programmatically. I can't found any describe of this method.
Example of usage:
...
<Multiselect ref="multiselect"/>
...
this.$refs.multiselect.activate()
this.$refs.multiselect.deactivate()
...

This can be done  >>
this.$refs.multiselect.toggle()

@ishpartko Done via https://github.com/shentao/vue-multiselect/pull/1609


Updated submodule pointer for sunflower-tools

Checked out branch issue-144, ran
~/git/sunflower-simulator git submodule sync
Synchronizing submodule url for 'sunflower-toolchain'

and can confirm that now the remote is correct for the toolchain:
~/git/sunflower-simulator/sunflower-toolchain git remote -v
origin	https://github.com/physical-computation/Sunflower-toolchain.git (fetch)
origin	https://github.com/physical-computation/Sunflower-toolchain.git (push)


Create a better FPS Counter (Frame Time Graph)

Provide this via an irr::ext::perf_counter, get the times via GPU time elapsed query ( actually have two modes to measure elapsed time queries between start_lap/stop_lap, one for CPU and one for GPU time taken) .
Keep the last N frametimes (in uint32_t microseconds) in a pre-allocated uint32_t* buffer (use %N addressing).
Doesn't matter if measuring GPU or CPU timestamp.
Provide an irr::ext::perf_graph extension to visualize graphs on screen (with anti-aliased lines with blending), with a user specified sample delay (show data from K frames behind).
Preferrrably irr::ext::perf_counter would output into a persistenly mapped IGPUBuffer which would be used by irr::ext::perf_graph as an UBO for the pixel shader that would draw the graph.
We want to measure multiple things and have multiple graphs.
Also irr::ext:perf_counter should have an option to serialize to a std::string .csv style.

Should probably integrate Tracy or some other CPU and GPU profiler


Question about sparse resnet

Hello:
Thank you for your job. I am interested in your sparse resnet. I am not quite sure how do you implement it. Or you just replace the "concat" operations with "add"?

I do indeed share the same question, Like since the shortest path of the gradient is O(1) in a resnet due to the sum of all the previous feature maps (kind of implicitly), how exactly is the path then extended using sparsenet ?


How to import custom model in tfc_end2end_verification

Discussed in https://github.com/Xilinx/finn/discussions/585

Originally posted by B-Willems April  6, 2022
I want to use the tfc_end2end_verification notebook to verify my finn model but am encountering a few issues.
In this notebook in the second cell, a model gets imported with the command get_test_model_trained("TFC", 1, 1). How can i import my own brevitas export/quantized model in this notebook?
Secondly in the verification of the example network using PyVerilator another weird thing happens. When trying to evaluate, the golden_output is an array of 10 values (as expected), but the output of the rtl simulation is 2. If I look closely at these values, the softmax of golden_output is also 2, so the prediction was successful, however the format in which these are given is not the same. Are there any known fixes for this issue?
An image of the executed cell with added prints can be found below.
Thanks in advance.

Hi @B-Willems , if you have your model in the form of an .onnx model file, then you can simply importing it using ModelWrapper in FINN.
Otherwise, have a look at the basics notebook about Brevitas FINN-ONNX export (https://github.com/Xilinx/finn/blob/main/notebooks/basics/1_brevitas_network_import.ipynb), section 2 and 3. Or in one of our test cases: https://github.com/Xilinx/finn/tree/main/tests/brevitas
The error with global out should be solved by now, please let me know if your issue is solved!

I am closing this due to inactivity. Please feel free to reopen or create a new issue if the problem persists!


feat: add sort items alphabetically test case

What: Extra test case for alphabetical sorting.

Why: Because alphabetical sorting is expected and desired behavior.

How: Someone will have to make tests happy again :D

Checklist:

[ ] Documentation
[x] Tests
[ ] Ready to be merged

Would you be willing to add your suggested change to fix this?

Yeah, will do asap.

I've thought about a different breaking change I want to make. I don't like the CASE_RANKINGS feature at all and that's what's causing these issues. So I'll remove that feature (and show a workaround) and that'll fix this. Thanks!

:tada: This issue has been resolved in version 5.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

@kentcdodds Cool, that's even better :)


Add dependencies-badges

Can you please move them in line with the title? See for example https://github.com/angular/angular.js

I have changed it next the title as what you requested. There, it is please.

Looks good, thanks

You are welcome!!!


Attest Bun support

https://github.com/arktypeio/arktype/issues/880 was addressed, but Bun support is currently blocked by @prettier/sync failing in Bun with the following error:

Once this is implemented in Bun, we can do some final testing to make sure there are no other hurdles and support it officially.

https://github.com/oven-sh/bun/issues/10768


Support/document how to handle raw YAML URLs

Is your feature request related to a problem? Please describe.
Some applications, like https://cert-manager.io/docs/installation/kubernetes/, just stick a YAML file at some URL. This is easy to apply with kubectl apply -f url.yaml. Its a bit harder to do with kpt packages.
Describe the solution you'd like
I am quite new to kpt, so maybe I am "doing it wrong", but I would expect to be able to do standard pkg operations like on a git repo, but with a raw URL.
Describe alternatives you've considered
If this doesn't actually makes sense, which is plausible, then it would be great to have a snippet in the docs recommending what I do in these cases. Should I just curl it and do kpt pkg init in the folder, or is there something better?
Additional context

Hi @howardjohn no issue with asking questions, we are revamping our documentation and understand that the current docs are not sufficient.
First you correct that the on the fly pull of remote resources is a harder with kpt, but we would argue that kpt opinionated approach enforces some good practices.  We would argue that it's creates a more stable environment if you save whatever YAML you are applying before just applying it.
Of course when you are dealing with prototypes it seems like an unnecessary overhead, but in my opinion the overhead is minimal and it's a good practice to adopt early in your development.
Here is an alternative flow:

you create a git repo which will mirror whatever it is that you have applied last to your cluster.
you pull in your remote resources and commit whatever you pulled to git, possibly having a branch that maps to the environment you are working with.
at this point in time you have whatever you intended to apply saved in version control, so if anything happens to the cluster or if you just want to roll back you have that power.
you can then apply the resources.

Let me know if this makes sense or if you'd like to see exact kpt commands there.

Closing for now.  Please let us know if you have any more questions and don't hesitate to re-open the ticket.


Entity groups

What problem does this solve or what need does it fill?
In non-trivial games, entities commonly need to be bundled together to share common behavior, despite being mostly heterogenous. As a few concrete examples:

character models with multiple moving parts
visual effects associated with sprites
UI widgets composed with multiple parts, each of which has its own transform, style, sprite etc

Possible Solutions

Allowing multiple copies of each component per entity (like nodes in Godot) is a Pandora's box of special-casing and is likely to do horrible things to our storages and conceptual clarity.
Using a parent-child-like hierarchy tree and traversing it to apply behavior. This results in a lot of wasted recomputation.
Storing a "WidgetID" component (or analogous) on each entity, then coordinating their behavior by looking it up within each relevant system.
Adding a unique marker component for each entity in the entity group, then querying for it. We generally can't define these at run time, since the number of entity groups is unknown, so ~min const generics might work well here~ we need dynamic components (see #623) (thanks @BoxyUwU and @TheRawMeatball).

Solutions 1 is clearly a poor fit. Solution 2 comes with performance costs (hundreds of tiny, fragmented vecs, recomputation) and hurts conceptual clarity. Solution 3 is probably fine, but can result in broad queries and doesn't leverage the ECS super well (e.g. for data fragmentation).
Regardless of the solution that we reach consensus on as best-practice, we should make sure it's properly supported and has good examples. These patterns are surprisingly tricky, and I've seen various complaints about how hard these designs are to implement in ECS.
Additional context
This challenge / pattern was brought up in the context of widgets, prompting this issue.
The pattern selected here should be used when designing our UI framework: see #254.

Solution 3 could be implemented with / improved by the use of index (see #1587, #1205).

Its worth pointing out that (4) is similar to the "entity relationships" that flecs has, which is similar to Tags, as defined here #1527

What I've been doing in my widgets prototype is to store the entity id of the "widget root" inside the other entities of the widget, and also the reverse when needed (saving the entity id of a "widget child" in the root to make it accessible to all). It feels a little bloated but it works well.

Hi, just commenting to say that having groups of Entity would allow me to more easily the switch between "outside" and "inside" since currently I have to iterate over all "outside" Entity and set their visibility to false (and the other way around when I go outside). So this feature would be really great!
bevy is great, thanks a lot for your work on it!


Make the npm page show a link back to github

Right now it doesn't show a link back to the Github repo on the npm page.

Great, thanks. :)


Is pip package planned?

Is pip package planned? Current version has not pip package.

A pip package is not planned at this time, but this is something that we may explore in the future depending on interest.

Without package like pip, it needs to locate file manually.


Issue with table extraction

I have tried to extract tables from a scanned PDF with landscape orientation using OCR Tesseract. The extraction fails to detect the entire table and hence, misses the column on the left margin ('ALDERSHO' in the screenshot below)  and column on the right margin ('36,974/n17,786/n8,552/n1,038' in the screenshot below). Is there a way to fix this? Thanks!

From what I can see, the outside left and right columns are not being identified because they are not delimited by a border, contrary to the other columns.
As of now, I do not really have an idea on how to handle those cases without breaking table detection on "regular" table layouts.
We can keep this issue open as a reminder

Do you have any suggestions on how I can solve this issue for this task? Other methods that I can use to supplement my current extraction or different approaches that I should look into and consider? Thanks!

You can check out different table recognition frameworks, most of them are deep learning methods (Paddle has an easy to use implementation)
They will probably give you some good results over your documents.


Update us-mn-scott.json

Trying this again, think I've got it now. This service really breaks up the address, so we have a lot of options.

Looks okay to me!


Add Feature::fid()

To test this, the fids in roads.geojson were changed from string to
numeric, because GDAL only supports numeric fids. As a result it no
longer shows up in the schema.

Hmm, any idea why Travis dislikes this PR only on Trusty? It seems to be seeing differences in geometry where there should be none, as I didn't change any of the geometry in roads.geojson, only ids (just double-checked).

maybe it uses a different order for the elements in the geojson?
thread 'vector::tests::tests::test_wkt' panicked at 'assertion failed: `(left == right)`

  left: `"LINESTRING (26.1009423 44.4318179,26.1008957 44.4315723,26.1008417 44.431218,26.1007684 44.4309207)"`,

 right: `"LINESTRING (26.1019276 44.4302748,26.1019382 44.4303191,26.1020002 44.4304202)"`', src/vector/tests/mod.rs:188:13

You're right, it seems the features are sorted by id if id is numeric, but returned in input order otherwise. The order is apparently dependent on the GDAL version and probably also on the driver.
I have added a function to get a feature by its fid, which I could use in the tests but is also useful in general.

Travis is happy now.


Unexpected behavior in rowset::begin()

This ticket is related to old report posted to soci-users in Unexpected behavior in rowset begin():

rowset<row>::const_iterator i = rs.begin();
    for (i=rs.begin(); i != rs.end(); ++i)
    {
       // do something
    }

The calls to rs.begin() increment the row counter in backend fetch for each call.  The first begin returns the first row, the subsequent begin call increments the row counter and returns the next row, not the first row.

First, by design, the rowset iterator is of category of input iterator, so it is not guaranteed that it is possible to pass through the same input iterator twice.
In this situation, however, there are two distinct iterators acquired from the subsequent calls to begin(). The problem here is that all the iterators share the same source, they walk through he same rowset, so they share all the side effects caused by the iteration.
Thus, additional restrictions apply to the rowset iterator. It's similar to istream_iterator- you can't have two iterators walking over the same range of input stream.
So, short answer is that the current behaviour is by design. It is not a bug.
Long answer, I reckon is that it should be possible to change the current implementation, so each iterator will manage its own distinct descriptor of rowset range. It is important to avoid copying of rowset. In fact, it may be impossible to make deep copy of rowset.
If anyone is interested in implementing it, that would be great.
I'm leaving this as a task for future work.

Let's close with the by design conclusion.
rowset_iterator is an input iterator. period.


Error: 1 on launch

This does not seem to affect the process nor its features (my keymetrics.io shows me all server-monit informations I want), but I got this log in pm2 everytime I'm starting the pm2-server-monit:
pm2-server-monit-26 Error: 1
    at /home/dev/.pm2/node_modules/pm2-server-monit/lib/os.js:35:26
    at /home/dev/.pm2/node_modules/pm2-server-monit/node_modules/shelljs/src/exec.js:142:7
    at ChildProcess.exithandler (child_process.js:758:5)
    at ChildProcess.emit (events.js:110:17)
    at maybeClose (child_process.js:1015:16)
    at Socket.<anonymous> (child_process.js:1183:11)
    at Socket.emit (events.js:107:17)
    at Pipe.close (net.js:485:12)

This usually appears when we could not determine which OS you are running on, do you see anything appear in the Operating System widget?

That's right, the OS widget show N/A. What could be the cause?

We only detect the most common linux distributions atm (Debian, Ubuntu, CentOS, Red Hat). What are you using?

Ubuntu 14.04.3 LTS

Im on the same version and it seems to be working for me, is the output of cat /etc/issue -e like this?
Ubuntu 14.04.3 LTS \n \l$
$

The file is empty, it seems the installation process used by our provider (OVH) is the cause of it (http://www.virtualmin.com/node/33336). Here is the workaround I used to get it back:
$ cp /etc/issue.net /etc/issue
$ cat /etc/issue -e
Ubuntu 14.04.3 LTS$

It works now :) Thank you


Improve authenticateRequest observability

Type of change

[ ] 🐛 Bug fix
[ ] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

Packages affected

[ ] @clerk/clerk-js
[ ] @clerk/clerk-react
[ ] @clerk/nextjs
[ ] @clerk/remix
[ ] @clerk/types
[ ] @clerk/themes
[ ] @clerk/localizations
[ ] @clerk/clerk-expo
[ ] @clerk/backend
[ ] @clerk/backend-core
[ ] @clerk/clerk-sdk-node
[ ] @clerk/edge
[ ] @clerk/shared
[ ] build/tooling/chore

Description

[ ] npm test runs as expected.
[ ] npm run build runs as expected.

Fix an error parsing issue
Introduces the auth.debug function in the backend package

Example usage:
app.use(clerk.expressWithAuth())
app.get('/', async (req: WithAuthProp<Request>, res: Response) => {
  const { userId, debug } = req.auth;
  console.log(debug());
  res.json({ auth: req.auth });
});

Decorate the response in clerk-sdk-node with the requestState observability headers when applicable:

These headers are:

  AuthStatus: 'x-clerk-auth-status',
  AuthReason: 'x-clerk-auth-reason',
  AuthMessage: 'x-clerk-auth-message',

@nikosdouvlis Let's update the docs of @clerk/backend with the new debug utility.


Flutter web shows just blank screen (no error)

This is horrible!!! I've lost more than a day on this and still haven't found any solution. Everything works great on iOS and on Android. Once I run it on the web: blank screen & no error!

flutter run -d chrome

It's hard to reproduce. What's even more confusing that out of maybe 100 times that I tried it, 4 times it worked without changing anything on my side. Very frustrating, since I'm developing it mainly for the web.

Logs

[✓] Flutter (Channel master, 1.19.0-3.0.pre.5, on Mac OS X 10.15.4 19E287, locale es-ES)
    • Flutter version 1.19.0-3.0.pre.5 at /Users/tomas/Development/flutter
    • Framework revision f4d26a3b85 (19 minutes ago), 2020-06-01 12:33:01 -0700
    • Engine revision 923687f0e7
    • Dart version 2.9.0 (build 2.9.0-11.0.dev 6489a0c68d)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /Users/tomas/Library/Android/sdk
    • Platform android-29, build-tools 29.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.5)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.5, Build version 11E608c
    • CocoaPods version 1.9.1

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

[✓] Android Studio (version 3.6)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 44.0.2
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)

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

[✓] Connected device (2 available)
    • Web Server • web-server • web-javascript • Flutter Tools
    • Chrome     • chrome     • web-javascript • Google Chrome 83.0.4103.61

• No issues found!

I think I'm getting to the core problem: Firestore cloud!!! I need it very urgently to work RIGHT NOW! :(
I just add cloud_firestore_web: ^0.1.1+2 or   cloud_firestore: ^0.13.6 in the pubspec.yaml and bang the website just wouldn't work at all!

Might be related to: https://github.com/flutter/flutter/issues/50704

Hi @tomasbaran
From what I can see, the issue is related to a FlutterFire plugin rather than to Flutter itself. Please open the issue in the dedicated repository.
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments, possibly providing a minimal reproducible code sample that does not use flutter fire or other 3rd party plugins, and I will reopen it.
Thank you


Google Chrome freezes the app

New version of Google Chrome freezes the app. Any idea how to fix and what flags to check in flags tab?

solved by PIN the tab. but if you'll find solution with flags it would be better )

Update: now with august's update chrome discards even pinned tabs so the only solution Disable automatic tab discarding extenston.
Idk about other platforms but it's all relevant to Windows OS


Update ionicons

Current ionicons version is v2.0.1 (tool-bar is using v1)
Some icons added :)
This is relate to suda/toolbar-main#5

Thanks, that's probably it :+1: :clap:


fix footnotes in chapters 19-23

This issue will be closed when someone has updated the footnotes in these chapters and someone else has reviewed the pull request.

[x] chapter 19 reviewed
[x] chapter 20 reviewed
[ ] chapter 21 reviewed
[ ] chapter 22 reviewed
[ ] chapter 23 reviewed

@Casey308 to review Tyler's work

There is a line (line 296) in chapter 20 that I was not sure was a footnote. And if it is, I am not sure how I would go about fixing it. Any guidance or opinion as to how we should address it would be much appreciated. 43f016a

Similarly, for chapter 22, on line 1013 there is a 1 (presumably representing a footnote), however there is no footnote that corresponds to it. I will need to know if we are either missing a footnote here or if the scan just copied an extra 1 here.

Based on looking in the PDF of the book it doesn't look like the chapter 20 one is a footnote, but the one in chapter 22 is.  I've attached the footnote in question, so maybe we can just copy it from the PDF manually

It seems that Nick and Justin have been putting the footnotes in brackets on a new line, but I don't think you did that Tyler.  I don't know if that matters, but it may be helpful to stay consistent.

@Casey308 I don't think that it's necessary to start a new line with the footnote, though it may be something to confirm as we finalize our workflow.

PS I'm investigating the issues in chapters 20 and 22...

Here's my proposed fix for chapter 20: can you please review @tknightly24?

@Casey308 thanks for the screenshot: that was most helpful!
@tknightly24 here's my proposed fix: https://github.com/STAT325-S24/HistoryAmherstCollege/commit/b93e961c4ce327af268f5df7f61e84daef083807

This all looks good.  I think we can close this issue now unless there were any lingering questions.