getExtension of WebGL shouldn't return any

TypeScript Version:  2.4.0
Code
This is part of lib.d.ts.
    getExtension(name: string): any;

But, there are also several type definitions of  extension returning value.

interface WEBGL_compressed_texture_s3tc {
    readonly COMPRESSED_RGB_S3TC_DXT1_EXT: number;
    readonly COMPRESSED_RGBA_S3TC_DXT1_EXT: number;
    readonly COMPRESSED_RGBA_S3TC_DXT3_EXT: number;
    readonly COMPRESSED_RGBA_S3TC_DXT5_EXT: number;
}

declare var WEBGL_compressed_texture_s3tc: {
    prototype: WEBGL_compressed_texture_s3tc;
    new(): WEBGL_compressed_texture_s3tc;
    readonly COMPRESSED_RGB_S3TC_DXT1_EXT: number;
    readonly COMPRESSED_RGBA_S3TC_DXT1_EXT: number;
    readonly COMPRESSED_RGBA_S3TC_DXT3_EXT: number;
    readonly COMPRESSED_RGBA_S3TC_DXT5_EXT: number;
};

interface WEBGL_debug_renderer_info {
    readonly UNMASKED_RENDERER_WEBGL: number;
    readonly UNMASKED_VENDOR_WEBGL: number;
}

declare var WEBGL_debug_renderer_info: {
    prototype: WEBGL_debug_renderer_info;
    new(): WEBGL_debug_renderer_info;
    readonly UNMASKED_RENDERER_WEBGL: number;
    readonly UNMASKED_VENDOR_WEBGL: number;
};

interface WEBGL_depth_texture {
    readonly UNSIGNED_INT_24_8_WEBGL: number;
}

declare var WEBGL_depth_texture: {
    prototype: WEBGL_depth_texture;
    new(): WEBGL_depth_texture;
    readonly UNSIGNED_INT_24_8_WEBGL: number;
};

These objects are not used.
    getExtension(name:"WEBGL_depth_texture"): WEBGL_depth_texture;
    getExtension(name:"WEBGL_debug_renderer_info"): WEBGL_debug_renderer_info;
    getExtension(name:"WEBGL_compressed_texture_s3tc"): WEBGL_compressed_texture_s3tc;
    getExtension(name:string):any;

This must be fixed with like the code above. There are more extensions also, these also should be defined in lib.d.ts.

looks like there are more extensions available: https://www.khronos.org/registry/webgl/extensions/

PRs welcomed. You can find more information about contributing lib.d.ts fixes at https://github.com/Microsoft/TypeScript/blob/master/CONTRIBUTING.md#contributing-libdts-fixes.

OK, if there were any other developer want to send PR, I will send PR next week.

Related PR was merged. Thanks!

I still need to merge the change into the main TS repo


[POS Validation] Analyze block flood attack

PosConsensusValidator.ContextualCheckBlock has a commented out code that mentions block flood attack. We don't know what this is at the moment and we need to investigate this and decide whether this attack is still valid for our networks. If it is, we should review the proposed code there (it is commented out).
So first, analyze what this is about and write it here. Then we decide what to do next.

also StakeValidator code contains this:
          // TODO: Investigate:
          // The POS protocol should probably put a limit on the max amount that can be staked
          // not a hard limit but a limit that allow any amount to be staked with a max weight value.
          // the max weight should not exceed the max uint256 array size (array size = 32).

let's find out what it is and if it is not relevant, let's remove this todo from codebase, otherwise let's implement it

Fixed with activation merge.


Add section to conditionals about key attribute

@LinusBorg Do you think this is a step in the right direction towards resolving #587? Also, is it short enough and explained in simple enough terms so as not to confuse beginners?

I like it. I had a rough draft a while ago but I like your version much better :)

Yeah this is really good! It shows that you are an educator - the way you introduce the upside of this behaviour before introducing a way to handle the "downside" is great!
And I think the message is transported very clearly.


Games do not stay on lower resolutions in fullscreen with Vulkan

Godot version
4.0.alpha3
System information
Windows 10, Geforce GTX 1650 SUPER, Vulkan
Issue description
The problem is that the resolution I set it at (640x480) is actually at that resolution when windowed. However, when I set it to full screen, it instead resolutes to my whole monitor, instead of staying at the resolution.
Steps to reproduce
1: Start up Godot
2: Start a new project or use one you already have on hand
3: Go to Project settings > Window
4: Set a resolution to a lower number (ex. 640x480)
5: Set to fullscreen
6: Start the scene
Minimal reproduction project
No response

This is intended behavior, and it hasn't changed with the switch to Vulkan. See the Multiple resolutions documentation for more information.
Note that the master branch also allows you to change the 3D rendering scale without affecting 2D rendering (and without requiring a separate Viewport). Use the scaling_3d_mode and scaling_3d_scale project settings for that purpose.


Fix Tests Page UI

Only difference is the position of the "Run" buttons in the table.
Old:

New:

Please fix merge conflict.


envbuilder - pass ssh key

Is it possible to pass the coder pubkey to envbuilder?
#1: 📦 Cloning git@bitbucket.org:lorem/ipsum.git to /workspaces/lorem-ipsum...
#1: 🔑 Using SSH authentication!
#1: 🔑 No SSH key found, falling back to agent!

@reiser Have you tried setting ENVBUILDER_GIT_SSH_KEY_PATH to the path of a private key mounted inside the container?


Decrypt JWE payload with .p12 file and password

I am trying to decrypt using code snippet and documentation provided here : https://github.com/Spomky-Labs/jose/blob/master/doc/operation/Decrypt.md
Facing issue as  Fatal error: Uncaught Assert\InvalidArgumentException: Unable to load the key in C:\xampp\htdocs\Encryption\vendor\beberlei\assert\lib\Assert\Assertion.php on line 285
Please let me know what can be the issue or proper example to do it.
Note : I am using PHP 7.0 and using Xampp on windows 7.

Hi @archana27m,
I am afraid but at the moment there is no possibility to directly load PKCS12 certificates.
You have to load it first and then create a JWK object.
Example:
use Jose\Factory\JWKFactory;

// Load the PKCS#12 file
$res = openssl_pkcs12_read(file_get_contents('/path/to/your/pkcs.p12'),$certs, 'YOUR SECRET IF NEEDED');
if (false === $res) {
    throw new \RuntimeException('Unable to load the certificates.');
}

// Normally you should have an array with a key "pkey" that represents your private key
$jwk = JWKFactory::createFromKey($certs['pkey']);

I think I will add a convenient method.

Hi @Spomky , thanks for the solution. Is it possible for encryption as well using public key?

Great!
I let this issue opened.
That feature will be part of the next major release.

@Spomky,  Just one more thing, is it possible to get json response in decryption, as currently its giving as associative array? Do we need to specify anything for format?

Can you try with the following script ?
$json = (object) $jwe->getPayload();

No, It's creating default object with arrays, is there any specific parameter to be passed while decrypting using loader?

OK. Maybe the accepted answer?

No, I just wanted to know if it converts into array by default? if we have valid json, it does not return json string?

The v6 of this library always tries to convert the payload.
The next major release will never do that and will always return a string (or null is the payload is detached).

Done. See 8fcef2014f52425e97ba128608c10953c525e0b2.
Will be available on v7.0.0.


Close old/obsolete connections in start_user_task handler.

Meant to address https://openedx.atlassian.net/browse/EDUCATOR-4597

Nice find. If you merge this, can you wait for #27 to pass tests before cutting the release?


Bootswatch 5.2.3

Issue #2350

bootswatch/bootstrap 5.2.3 upgrade
bootstrap-icons support
jquery3.7.0

Any update ?

This week makes 6 months that this PR has been here waiting. Any chance it can get reviewed and merged into an upcoming release sometime soon?

Well it's quite sad to see this waiting and maybe getting outdated...

@mariofix, I think you can turn this PR into a package. I think that would be great! 🚀

Since there are no updates for the review, PR closed.

Since there are no updates for the review, PR closed.

Hey @mariofix, the project is alive again and there is another PR (#2432) related to this. Please check out if you are still interested in 🙏.


Table view width is fixed (or capped) and summary never unwraps even on wide screens

I tried both Chrome and Firefox.

Yes we have been wanting to make this full screen since inception. Glad to see others concur.

@hannes-ucsc yes!  This was part of the original design specification (max width set at 1200).  Dave and I discussed yesterday after issue of wrapping of summary... stay tune!


remove session secret

remove the session secret so our secrets (intended for the server) are not published

So when we run the app locally, do we just insert a secret? The app requires a secret so it seems you can't just now clone the repo and run the app to get it working straight out of the box

Boot strap sets a default https://github.com/UKHomeOfficeForms/hof-bootstrap/blob/master/lib/defaults.js#L23


arm-logic/2015-02-01-preview missing $filter parameter for workflow trigger history

According to https://msdn.microsoft.com/library/azure/mt643788.aspx the 'Get trigger history for the workflow' action has a $filter parameter:
https://{endpoint}/subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.Logic/workflows/{workflowName}/triggers/{triggerName}/histories?api-version=2015-08-01-preview&$filter=status eq 'Succeeded'

However, https://github.com/Azure/azure-rest-api-specs/blob/48bc044a6288328f0d3e545eb099a4732d97820f/arm-logic/2015-02-01-preview/swagger/logic.json#L966 is missing the parameter in the definition.

@tonytang-microsoft-com - Can you take a look at this issue?

Fixed by PR 327


symbol gets converted to >

This media query
@media(max-width: 600px) {
    & > tbody > tr:nth-child(3) {
      table {
        border: 1px solid $c-slate-lighter;
        padding-bottom: 100px;
        position: relative;
      }
    }
}

gets converted to this
@media (max-width: 550px) {
      .solicitud >
      tbody >
      tr:nth-child(3) .cta {
        position: absolute;
        bottom: 30px;
        left: 30px;
        right: 30px;
        width: auto;
      }
}

So I have to fix this manually. I tried to find if there is any configuration option unsuccessfully. Not sure if it's a bug or a misuse from my side.

Same here. Any clues?

It looks like an issue on the premailer / nokogiri side. So times ago I had that problem reversed (entities got decoded). Apart from checking that your meta charset and file encoding is utf8, you could try updating those gems.

Thank you, @dwightjack! I just updated premailer and nogokiri to their latest versions, but the issue still remained. I'll report it on their repos as well.
For now I just added couple lines to your premailer.rb, to revert the chars.


date2num with masked inputs

Fixes #267

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@davidhassell this looks good, but I think you probably did not intend to commit the change to setup.cfg


Split generated AssistedInject_*Module to multiple modules

Currently the generated AssistedInject_*Module contains all the bindings for @AssistedInject.Factory annotated factories in one module. As a result, it's impossible to have dependencies that exist only in subcomponents. For example:
I have two ViewModels, where one of them takes an argument (GithubApi in this case), which exists only in one the subcomponent hosting these view models.
class CommitDetailViewModel @AssistedInject constructor(
    @Assisted handle: SavedStateHandle,
    privatel val githubApi: GithubApi // Should be provided only in CommitDetailActivitySubcomponent
) : BaseViewModel(handle) {
    fun loadDetail() {
        Timber.v("Request commit detail")
        // use githubApi here
    }

    @AssistedInject.Factory
    interface Factory : ViewModelAssistedFactory<CommitDetailViewModel>
}

class CommitListViewModel @AssistedInject constructor(
    @Assisted handle: SavedStateHandle
) : BaseViewModel(handle) {

    fun loadCommits() {
        Timber.v("Request commit list")
    }

    @AssistedInject.Factory
    interface Factory : ViewModelAssistedFactory<CommitListViewModel>
}

interface ViewModelAssistedFactory<T : ViewModel> {
    fun create(handle: SavedStateHandle): T
}

@AssistedModule
@Module(includes = AssistedInject_ViewModelAssistedFactoriesModule.class)
public abstract class ViewModelAssistedFactoriesModule {
}

The generated AssistedInject_ViewModelAssistedFactoriesModule contains the bindings for both factories above. Due to that, I have to include it in the ApplicationComponent so it's visible to my subcomponents. But then compilation fails since GithubApi's binding is only defined in one of the subcomponents.

How do we know which bindings are semantically grouped together?

I was initially thinking that splitting the generated module to one module per factory would be an option, but actually it's not that different than just setting up the bindings manually :/

It's actually worse because with your own modules you could at least group
them
On Wed, Mar 20, 2019 at 5:31 PM Nimrod Dayan<EMAIL_ADDRESS>wrote:

I was initially thinking that splitting the generated module to one module
per factory would be an option, but actually it's not that different than
just setting up the bindings manually :/
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/square/AssistedInject/issues/87#issuecomment-475036761,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEEEbzgtDnbwT_zALSrupmgcX2kpBGqks5vYqjHgaJpZM4b4Esm
.

Could we have the @AssistedInject.Factory take some sort of user-defined @AssistedGrouping annotation (somewhat conceptually like the @MapKey stuff in dagger) that the @AssistedModule would use in its creation?

Today you can choose not to use @AssistedModule and specify the @Binds methods yourself in whichever modules you want. That's not that far off from what you're asking.
Ideally the @ContributesAndroidInjector mechanism would be generalized so we could have @ContributesAssistedFactory Foo.Factory foo() and it would do the right thing.

Today you can choose not to use @AssistedModule and specify the @Binds methods yourself in whichever modules you want. That's not that far off from what you're asking.

Well the issue is still the extra boilerplate creation (which is why @AssistedModule even exists, I assume.) It'd be nice to not have to write those extra binds for every factory.

@Nimrodda https://github.com/google/dagger/issues/1781 might be related to your problem.
Re-including the assisted module in the subcomponents will let the assisted module search the subcomponent for provisions (in your case GithubApi).
Feels unintuitive to reinclude a module in a subcomponent, but it definitely solves the need for splitting the AssistedInject module into multiple modules.

Re-including the assisted module in the subcomponents will let the assisted module search the subcomponent for provisions (in your case GithubApi).

Could you please elaborate on that?

Here's another example to be more specific:
@AssistedModule
@Module(includes = [AssistedInject_AssistedModule::class])
abstract class AssistedModule

@dagger.Component(modules = [AssistedModule::class])
interface AppComponent {
    fun commitDetailActivitySubcomponent(): CommitDetailActivitySubcomponent
}

// Reinclude AssistedModule so that the subcomponent can fulfill the binding.
@dagger.Subcomponent(modules = [AssistedModule::class, GithubModule::class])
interface CommitDetailActivitySubcomponent {
    fun vmFactory(): CommitDetailViewModel.Factory
}

@Module
static class GithubModule {
    @Provides
    fun githubApi(): GithubApi {
        return GithubApi()
    }
}

class CommitDetailViewModel @AssistedInject constructor(
    @Assisted handle: SavedStateHandle,
    private val githubApi: GithubApi
) : BaseViewModel(handle) {
    // ....
}

There's two things happening here.

@Binds (and subsequently AssistedModule) doesn't actually impose the dependency requirement on the component it's attached to. To me, there's an assumption that it does, since Dagger is so strict with types elsewhere, it's actually a little surprising that it's okay with this. That means that even though there's a @Binds which requires GithubApi to fulfill, it's okay for that to exist in a component without the GithubApi, since no one actually uses that binding at the AppComponent level. So AssistedModule is safe to use, even in places where not all the dependencies exist yet.

@Binds does not exactly carry over to subcomponents. If we consider dagger provisions to be sort of a pure mathematical formula (object A + object B lets you build object C), you'd figure that the declaration of this formula at the top level means you'd be able to use this formula anywhere. However, this assumption is also incorrect; the formula only executes with the component context where it's declared.
For example, if we omitted the AssistedModule from the ActivitySubcomponent, the AssistedModule could only search the AppComponent for fulfilling dependencies, and would not be able to get GithubApi for fulfilling the CommitDetailViewModel.Factory binding, even though it's actually being provided.

@davidliu my use case is a little bit more complicated.
I have multiple modules where I have subcomponents. I want to isolate dependencies from these subcomponents.
Here is an example of what I mean:

That's what I want (and as as I understand it's not possible with a single AssistedModule):

Is there a way to make it work still?
How can I manually create an assisted module? Or provide dependencies at a later time?

Like I mentioned, the AssistedModule doesn't impose a dependency requirement. You can put it in each of your components, and it will be fine. Declaring the AssistedModule in your app component won't require you to have the MainRepo at the app level; you just also need to have the AssistedModule at the MainActivity level. Just try it out in code instead of trying to figure it out beforehand.
I'm not sure how to be any more clear in my explanation so I just made an example project for this:
https://github.com/davidliu/MultiLevelAssistedModule/tree/master/app/src/main/java/com/deviange/multilevelassistedmodule

@davidliu thank you for the answer and for the example. Your repo was extremely helpful!
Assisted module doesn't even need to be included anywhere except in a subcomponent that needs it.
My problem was that my assisted module also had dependencies listed in there (that were imposing a dependency requirement).

Like I mentioned, the AssistedModule doesn't impose a dependency requirement. You can put it in each of your components, and it will be fine. Declaring the AssistedModule in your app component won't require you to have the MainRepo at the app level; you just also need to have the AssistedModule at the MainActivity level. Just try it out in code instead of trying to figure it out beforehand.
I'm not sure how to be any more clear in my explanation so I just made an example project for this:
https://github.com/davidliu/MultiLevelAssistedModule/tree/master/app/src/main/java/com/deviange/multilevelassistedmodule

Using the AssistedInject_AssistedModule approach worked like a charm.
@davidliu always saving the day 🙏

Just to leave the note that with Dagger 2.31, Assisted injection is included, this problem (I solved it by removing this dependency) no longer exists.

Just to leave the note that with Dagger 2.31, Assisted injection is included, this problem (I solved it by removing this dependency) no longer exists.

Yep. Realistically, we never were going to fix this. The correct fix is to either allow Dagger to automatically discover modules or to fully upstream the assisted feature. The latter happened.

Yep. Realistically, we never were going to fix this. The correct fix is to either allow Dagger to automatically discover modules or to fully upstream the assisted feature. The latter happened.


SOLUTION: Doesn't work with Button. Cleans all tags.

Replace
var text=$($note.summernote('code')).text();
with
var text=cleanText($note.summernote('code'));

Thanks I fixed it, not sure how I missed that.


Static build with OpenSSL (Visual Studio)

This is duplicate of https://github.com/grpc/grpc/issues/12231 which has been closed as stale and in which I cannot comment.

What version of gRPC and what language are you using?
C++ 1.24/1.25
What operating system (Linux, Windows,...) and version?
Windows 10
What runtime / compiler are you using (e.g. python version or version of gcc)
Visual Studio 2015
What did you do?
Compiled static OpenSSL and attempted to compile static gRPC.
What did you expect to see?
Successful build
What did you see instead?
Undefined references related to symbols in Microsoft's crypt32.lib. Unfortunately I don't have the build log with me currently.
Anything else we should know about your project / environment?
The build I've configured attempts to mimic the vcpkg build - vcpkg works around this issue by patching the CMakeLists file with the following patch:
https://github.com/microsoft/vcpkg/blob/master/ports/grpc/00006-crypt32.patch
(I'm a bit disappointed to see no attempt on vcpkg's behalf to upstream any of the patches in vcpkg.)
I've now built gRPC from my own fork of 1.25.0 successfully by having the patch applied on top of the v1.25.0 branch: https://github.com/Rantanen/grpc/commits/v1.25.0-mfiles
Would gRPC be interested in upstreaming this change? I can prepare a merge request if that's the case. I know there's some shift in focus onto Bazel these days, but I haven't found a way to create a static gRPC archive with Bazel (not that I looked for one that much). Our product toolchain is based on Visual Studio, so currently we are compiling gRPC as static archive and including that in the VS builds.
Also I'm not sure how testable such MR would be, given - as far as I know - the CI doesn't compile static libraries or link against OpenSSL (1.1.x).

As OpenSSL 1.0.2 has end of life on 31st Dec 2019 and we should move to OpenSSL 1.1, this patch from vcpkg should be incorporated into grpc.

I'm packaging grpc for Conan and this patch finally allows me to upgrade OpenSSL from 1.0.2 to 1.1.1
This patch should get merged
https://github.com/inexorgame/conan-grpc/commit/f72a7c47b277845236949c5bc52896133c9ce053

https://github.com/grpc/grpc/pull/21611


Error boundaries should implement getDerivedStateFromError()

This a new warning coming from freshly released React 16.6.

Warning: ErrorBoundaryComponent: Error boundaries should implement getDerivedStateFromError(). In that method, return a state update to display an error message or fallback UI.

I am not sure how it should work just yet, but I suppose it should be adapted to this.

Well noted. I'll publish a patch soon. Thanks for the headsup


Fix modifier one name for 2525D land units

According to MILSTD2525D the name of the "01" modifier for land units
should be "Air Mobile/Air Assault", "not Tactical Satellite Communications".

This is changed in the draft of mil-std-2525d change 1 that I got, and there are a bunch of other things that are according to the draft in the json files as well. The new version is to be released very soon, so I will keep this on hold until we can see the final version and what more needs to be fixed.

Thanks for the heads up. Will they change that modifier in APP-6 D as well?

Not quite sure about that... :-/ They were supposed to be in sync, but there are a quite a lot of things that are different... Might be able to get more info, in an official way, during the fall about APP-6.

Thanks for the update.  Closing pull request.


Add “API group” to glossary #15906

This PR resolves #15906
Created API Group glossary term
Linked glossary term to applicable pages

/assign @sftim

Slightly related to issue #13807

@ashishbharthi would you have time to fix this pull request up for review?

/lgtm
@ashishbharthi would you be willing to rebase / squash the commits in this PR?

@sftim  done.

Have a look at https://deploy-preview-16192--kubernetes-io-master-staging.netlify.com/docs/reference/glossary/?all=true @ashishbharthi
This isn't rendering quite right. Also you could have a think about the length of the short version of the glossary versus the long entry. Most entries have a short version that is succinct.

@sftim I have made short version shorter and moved additional information in the long entry. Please take a look and let me know what do you think?

@sftim I have made the suggested changes.

@ashishbharthi If you accept the change I made, we can get this merged.
/approve

@zparnold I have accepted the change.

/lgtm


zd1211fw: update from 1.4 to 1.5

Potentially fixes CVE-2012-2187.
Signed-off-by: Edward O'Callaghan<EMAIL_ADDRESS>

Ok I think this package needs an update since year 2008. Let's merge, thanks.


generate routes as code instead of a routes file

so hand-written routes won't be overwritten
see https://www.playframework.com/documentation/2.5.x/ScalaSirdRouter
see #24 for more about the motivation behind this

Yes, this will be the road ahead. Thanks for pointing out.

The easiest way to bind a sird router is as described in Using SIRD router from a routes files, but this implies the the developer should manually add a line like this:
->      /api                        api.ApiRouter
to their conf/routes file. I don't know if this is practical.
I cannot figure out a solution that is fully transparent for the developer.

done in 0.0.12-SNAPSHOT


Update Composer dependencies (2023-05-06-00-08)

Visual regression test passed!
View the visual regression test report


Draft Maintainers Circle Welcome Email

[x] create a welcome email to send to the maintainer lists including an action (example: join the maintainer slack channel) (merged with #19)
[ ] Try and recruit at least one voice per project (this is optional but will be good to have someone/small group to keep in constant contact about contributor related matters) and keep a record somewhere TBD (need clear ownership on this)
[ ] think about what we would want the first meeting to look like or whatever it is that we are delivering. whats a good valuable first step? (need clear ownership on this and discussion)

re: third bullet:
@karenhchu is helping now! \o/
the first session will be advertised as a birds of a feather for introduction, kick off, setting expectations and soliciting discussion/feedback. will have more than one session for different time zones.
opening up a new issue that captures the third bullet in more detail.


Doesn't cleanup after itself

Right now, nrpl leaves behind a nimcache directory, a tmp.nim file, and a tmp executable when it exits. It should remove these upon exit.

Changed them temporary file used from "tmp.nim" to "nrpltmp.nim". Now removes "nimcache" directory and all temporary files upon exit.


ERROR - expected token '('

Hi Ken!
Any idea why TagUI throws an error when attempting to use an if statement with exist() helper function?
This error appears on live mode as well.
The intended use is to check if an email field exists and if it does exist fill it with the email address.
Happens for other strings as well (if exist("email"), if exist('Log-In'), etc). OS is Ubuntu, TagUI version is the latest (6.14).
Did I misunderstand the syntax, or is there something else going on?
All the best,
Gil

if exist(//*[@id="email"])
 type //*[@id="email"] as testy-McTestFace

ERROR - expected token '('

Hi Gil,
Above example should be with single quotes like below because it is a function -
if exist('//*[@id="email"]')
 type //*[@id="email"] as testy-McTestFace

Also, there are a couple of fixes past few months on indentation. You can download the latest copy of TagUI with this and unzip to overwrite your existing installation (please drag the folders under tagui\src to overwrite your existing installation) - https://github.com/kelaberetiv/TagUI/archive/master.zip
Then see if that helps. In the next release, changes in above updated file will become part of the packaged zip files.
PS - lastly, present() can also be used if you don't need to 'test' until the default timeout of 10 seconds but wants a response right away if something is there without waiting for it

Hi Ken!
No luck:
if exist('//*[@id="email"]')
ERROR - expected token '('

Overwriting the old TagUI copy with the latest from master doesn't help.
Thoughts?
PS - isn't it about 01:00 am in Singapore? This is hardly a critical issue and can definitely wait until tomorrow :smile:

Hi @gilgoldman the following works for me, can you try if it works for you?
www.google.com
if exist('//*[@name="q"]')
    type //*[@name="q"] as open-source rpa[enter]
wait

PS - haha I sometimes sleep very late and work late into the night 🤣

The output I have and using TagUI v6.42 (type tagui command without parameters)
START - automation started - Sun Jun 06 2021 10:34:26 GMT+0800 (+08)
https://www.google.com - Google

type //*[@name="q"] as open-source rpa[enter]
wait

https://www.google.com/search?q=open-source+rpa&sxsrf=ALeKk01g5753fWJ3E1X0FhWOT6SIysGyFA%3A1622946866790&source=hp&ei=MjS8YOiwLYPn9QOx6oyQBg&iflsig=AINFCbYAAAAAYLxCQqde-3BxpbIXUgO4d1O1b9JhDx6z&oq=open-source+rpa&gs_lcp=Cgdnd3Mtd2l6EAwyBAgjECcyAggAMgYIABAWEB4yBggAEBYQHjIGCAAQFhAeMgYIABAWEB4yBggAEBYQHjIGCAAQFhAeMgYIABAWEB4yBggAEBYQHjoFCAAQkQI6CwguELEDEMcBEKMCOgUIABCxAzoLCC4QsQMQxwEQrwE6DgguELEDEIMBEMcBEK8BOggILhDHARCvAVCPCVjvFmDEGGgAcAB4AIABXIgBzwaSAQIxNZgBAKABAaoBB2d3cy13aXo&sclient=gws-wiz&ved=0ahUKEwjoke2b_IHxAhWDc30KHTE1A2IQ4dUDCAc - open-source rpa - Google Search
FINISH - automation finished - 10.0s

Hi Ken!
No luck - still the same issue. I also have TagUI v6.42 (confirmed by typing tagui without parameters)
Any idea what it might be?
www.google.com
if exist('//*[@name="q"]')
ERROR - expected token '('

Hi @gilgoldman oh that's right! In live mode there isn't support to do if condition / for loops. This is because it's quite awkward to try to communicate what steps belong to the if condition and for loop.
And for now the live mode compiler runs each line as a single block of JavaScript code instead of building buffer and have some way for user to tell live mode 'hey i have finish entering all the lines i want, now run them'.

Ohhhhhhh that makes a lot of sense, actually. Issue resolved, then :smile:
Thanks again Ken!


CPU load and GC

Hi!
How can I measure process CPU load and free memory after GC?

@UncleVic CPU and memory metrics are based on prom-client

@UncleVic CPU and memory metrics are based on prom-client


int_size IS_8 takes two bytes for each uint8

Overview
I don't know yet if this is an issue. I'm trying to reduce the current size of my payload or encoded stream, and I notice that for each insertion of each uint8, the stream add two bytes, and should be one byte, I guess. I tested different options but nothing.
My proto definition:
syntax = "proto2";

import "nanopb.proto";

message Frame {
  required uint32 lenght = 1 [(nanopb).int_size = IS_16];
  repeated uint32 data = 2 [(nanopb).int_size = IS_8, (nanopb).max_size = 1];
}

My code:
bool encode_uint8_array(pb_ostream_t *stream, const pb_field_t *field, void *const *arg) {
  for (int i = 0; i < chunk_size; i++) {
    if (!pb_encode_tag_for_field(stream, field))
      return false;
    uint8_t val = (out_jpg+chunk_pos)[i];
    if (!pb_encode_varint(stream, val))
      return false;
  }
  return true;
}

Full code

Varints with value >= 128 use two bytes when encoded: https://protobuf.dev/programming-guides/encoding/#varints
For data like that, you probably want to use the bytes type in protobuf, which doesn't use extra space and automatically passes the length too.

Thanks, I didn't know that, and I tried with bytes but I don't remember why it didn't work in the beginning of this exploration. Also with repeated bytes I had issues.. Maybe do you know an example or test using bytes and callback similar a repeated int that I can follow? Thanks

Encoding bytes is the same as encoding string here, just replace strlen() with the actual binary data length:
https://github.com/nanopb/nanopb/blob/master/tests/callbacks/encode_callbacks.c#L9-L17

Thanks, I did a mistake the first time that I try with encoding string, I was close. Thanks for your fast feedback and for this wonderful library. Is working better:
https://github.com/nanopb/nanopb/assets/423856/31a478ff-f916-4ad5-97eb-20238359e0eb


Separate out a Daydream backend

some viewManager refactor

GearVRf-DCO-1.0-Signed-off-by: Mihail Marinov<EMAIL_ADDRESS>

Basic work done; lots of testing left.
We will have three backends; order of loading is determined by the asset/backend_x.txt file where x is the priority. This file gives a factory class name we use to bring up the backend and hook it up.
The monoscopic renderers are still bundled.
Significant refactoring of viewManager but there is more left when it comes to dealing with screenshots, statistics.

Smoke testing showed only a problem with shadow mapping.

The problem with gvr-shadows was caused by another change. Fix merged in master already.

gvr-lodtest doesn't work; doesn't work prior to this change too. Looks to be caused by the recent bounding volume change. Will look into it anyway.

All samples works as well as before. I will experiment with the existing unit tests but this is ready for review already.
lodtest is a separate thing that will have to covered by another pull request.

Will take a look.

I got caught by .gitignore. Will add the daydream backend files.

Ran framework-tests; I don't think they are yet in a state that allows conclusive decisions.

Monoscopic and Nexus is something we can't handle right now. It is a rather special case since Oculus is not there but the monoscopic renderer is bundled with Oculus.
This needs work of separating out the monoscopic renderer(s). @thomasflynn Should this be handled as part of this change or could it be done in the future?

future

Rahul is ok with merging this. Will merge tomorrow morning if there are no further comments.

I tested this today with the experimental gradle build and the normal build. It works OK for the Oculus backend. I don't have a Daydream phone so I did not test that option.


Confusion in README.md in the "very simple JSON-formatted messages" support between single and double quotes

Hi !
Just to let you know that the in the JSON message sent over MQTT, the key must be enclosed in double quotes and not single quotes (as it is written in the README.md). Messages with single quotes aren't recognised by the plugin at all, and it can be painful to debug such a simple mistake... especially since it's mentionned that it's the only supported format. People might want to adhere strictly to it. (no no no, it didn't happen to me 0:) )
If you could make it clearer the README.md, it might save some time to other users!
Sincerely,
Louis

Thank you so much for reporting this Louis. Fixed now in: https://github.com/grafana/mqtt-datasource/blob/main/README.md#known-limitations


feat: endpoint caching

https://github.com/aws/aws-sdk-js-v3/issues/5723
this PR adds a cache for the resolveEndpoint function keyed on two levels, the ruleset object as a first key and the endpoint parameters as the second key.
This makes resolving endpoints repeatedly, such as when generating multiple signed URLs, faster.
The expected effect on endpoint resolution with a cache hit is 1-5ms (depending on hardware) to <0.5ms.

[ ] suggestion: the lifecycle of the cache should follow a client so as not to prevent garbage collection

[ ] benchmark simple ruleset client
[ ] benchmark 100% cache miss diff
[ ] benchmark 50% cache miss diff


Sprint.v1.1.2

Review Sprint V1.1.2

Sprint V1.1.1 was tested on a darwin and linux virtual machine.

@dlewissandy There is nothing for you to review.  I will merge these changes into master.


With tables

:tada: This PR is included in version 5.0.0-next.8 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0-next.1 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Finished downloads not changing to 'available' despite being visible in Jellyfin

Description
Finished downloads not changing to 'available' despite being visible in Jellyfin
After requesting a show it starts download as it should, status in 'requests' changes to 'processing' but when download finishes status becomes empty and is never added to 'recently added'. Logs show no errors or issues. A manual rescan doesn't solve the problem, neither does refreshing metadata in Jellyfin
Version
1.2
Steps to Reproduce
Request show
Sonarr starts downloading and in requests it says "processing"
Download finishes, status returns to "requested" for a while
Then status becomes and remains empty (see screenshot)
Show never loads as "recently added" despite being visible in Jellyfin
Screenshots

Logs
2022-10-14T23:26:00.050Z [debug][Jobs]: Starting scheduled job: Download Sync
2022-10-14T23:27:00.004Z [debug][Jobs]: Starting scheduled job: Download Sync
2022-10-14T23:28:00.016Z [debug][Jobs]: Starting scheduled job: Download Sync
2022-10-14T23:29:00.008Z [debug][Jobs]: Starting scheduled job: Download Sync
2022-10-14T23:30:00.004Z [debug][Jobs]: Starting scheduled job: Download Sync
2022-10-14T23:30:00.031Z [info][Jobs]: Starting scheduled job: Jellyfin Recently Added Sync
2022-10-14T23:30:00.033Z [info][Jellyfin Sync]: Jellyfin Sync Starting {"sessionId":"9cb6184d-1ac4-4cce-a1fb-c656af4acf89"}
2022-10-14T23:30:00.043Z [info][Jobs]: Starting scheduled job: Plex Watchlist Sync
2022-10-14T23:30:00.071Z [info][Jellyfin Sync]: Beginning to process recently added for library: Shows
...
2022-10-14T23:30:01.391Z [debug][Jellyfin Sync]: Updating existing title: Nathan For You

Platform
desktop
Device
Raspberry Pi
Operating System
Docker on Raspbian OS 64 bit
Browser
Firefox
Additional Context
No response
Code of Conduct

[X] I agree to follow Overseerr's Code of Conduct

Do you have it set up to change the date of the file to the aired date or so in sonarr/radarr or jellyfin?
Try a full library scan if it does add then that means somehow the date is being changed

Thanks for the prompt response Fallenbagel
'Change file date' is set to 'None' in Sonarr advanced options
Checking the filesystem the last modified date of the video files is 14 October which fits with when Sonarr's history says the download was finished
A manual library scan doesn't solve the problem either I'm afraid
The strange thing is that if I delete the config folder and do a fresh setup the manual scan still doesn't see the downloaded media (in this case Nathan for You). Recently Added seems to be stuck in a situation from a while back. I've since downloaded a few episodes of other shows since they became available but those shows aren't at the front of Recently Added (if that is what is supposed to happen)
Any other ideas? I really appreciate your help

Putting the files in 'Season 1' folder caused Jellyseerr to find them and appropriately update the status

Glad you worked it out! 🥳
Yeah jellyseerr requires season folders as thats how it matches with the tmdb database


Add image for latest 2.8.x

The charm is still on 2.8.x, but there is no PR for 2.8.6, only 2.9.x.
We need a 2.8.6 image (bunch of CVEs).

Do we actually need 2.8.6? Is there a problem with just using 2.9.x?

As soon as we have a 2.9 workload in stable, we could close this issue.


Automate formatting a large repo to test CSharpier changes

I've been doing the following manually and it helps track down edge cases that are missed by our current tests before we get to the release candidate stage. Automating this process would make it easier.
When a PR is created.

Create a new branch pre-[nameOfCSharpierBranch] off of master on https://github.com/belav/aspnetcore
Format the new aspnetcore branch with the current CSharpier master code and commit it.
Format the new aspnetcore branch with the CSharpier code from the PR and commit it.
Automatically add a message the CSharpier PR linking to the new commit.

3 and 4 could be repeated with each new commit to the PR.
This process could also be done using a script locally, to avoid having to create the PR first.
This process also could use a different repo, or be configurable if running locally.

I started working on this, but there are a number of issues to figure out.
This is the current action I had, but I haven't tested it.
name: Validate PR
on:
  pull_request:
    branches: [ master ]
jobs:
  format_repositories:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v2
        with:
          path: csharpier
      - run: echo "##[set-output name=branch;]$(echo ${GITHUB_HEAD_REF #refs/heads/})"
        id: extract_branch
        shell: bash
      - uses: actions/checkout@v2
        with:
          repository: belav/aspnetcore
          path: aspnetcore
      - uses: actions/checkout@v2
        with:
          path: csharpier-master
          ref: master
      - run: dotnet build csharpier-master/Src/CSharpier/CSharpier.csproj -c release
      - run: dotnet csharpier-master/Src/CSharpier/bin/release/net5.0/dotnet-csharpier.dll aspnetcore
      - run: |
          git checkout -b pre-${{ steps.extract_branch.outputs.branch}}
          git add -A
          git commit -m "Before changes"
          git push --set-upstream origin pre-${{ steps.extract_branch.outputs.branch}}
      - run: dotnet build csharpier/Src/CSharpier/CSharpier.csproj -c release
      - run: dotnet csharpier/Src/CSharpier/bin/release/net5.0/dotnet-csharpier.dll aspnetcore
      - run: |
          git checkout -b post-${{ steps.extract_branch.outputs.branch}}
          git add -A
          git commit -m "After changes"
          git push --set-upstream origin post-${{ steps.extract_branch.outputs.branch}}

The issues I know of

It would try to create a new pre-branch on every commit to the csharpier PR
It doesn't yet create the aspnetcore PR from post-branch to pre-branch
It doesn't yet auto add some kind of comment on the csharpier PR to tell someone to go look at the aspnetcore PR

A powershell script was created that mostly accomplishes what we need, but isn't yet run in a github action.

doing this manually is good enough


envoy:<PHONE_NUMBER>9d

Signed-off-by: Jose Nino<EMAIL_ADDRESS>

I bet https://github.com/envoyproxy/envoy/pull/19564/files is making the expectation in the IdleTimeoutTests swift test fail

What course of action do you recommend to address that timeout test failure?


Login page: email login submit on enter

Support submitting the email by pressing enter.

Huh, false alarm!


publish crate

Hi,
Thanks for this awesome project.
Could you please add a license and publish this as a crate ?
I'd like to use it in https://github.com/yazgoo/blockishfire
Right now I'm including your code as a submodule https://github.com/yazgoo/blockishfire/blob/master/Cargo.toml#L11

Sorry for the delay. The crate is now published in crates.io


Incorrect result in computing MulticlassRecall macro average when ignore_index is specified

🐛 Bug
Specifying the ignore_index argument in MulticlassRecall leads to incorrect results when computing the macro average.
To Reproduce
import torch
from torchmetrics.classification.precision_recall import MulticlassRecall

metric = MulticlassRecall(num_classes=2, ignore_index=0, average="macro")
y_true = torch.tensor([0, 0, 1, 1])
y_pred = torch.tensor([
    [0.9, 0.1],  # Correct
    [0.9, 0.1],  # Correct
    [0.9, 0.1],  # Incorrect
    [0.1, 0.9],  # Correct
])
metric.update(y_pred, y_true)
print(metric.compute())  # Prints 0.25, but I would expect the result to be 0.5

Expected behavior
In the toy example above there are two classes (0 and 1) and two instances of each class. I want to ignore class 0 altogether (hence ignore_index=0), so I would expect the first two predictions to be ignored entirely (since they relate to class 0 instances). Of the remaining two predictions only one is correct, so I would expect the recall score to be 0.5.
However, the actual result is 0.25. It looks as if the recall score for the ignore_index class is simply set to 0 before computing the average, causing the average to be computed as (0 + 0.5) / 2 instead of 0.5 / 1.
Environment

TorchMetrics version: 1.3.1 (installed via pip)
Python version: 3.10
PyTorch version: 2.2.1
OS: Ubuntu 22.04

This issue has been there for a while. After v0.9.3, average="macro" does not work with ignore_index.

This issue has been there for a while. After v0.9.3, average="macro" does not work with ignore_index.

thank you for the adjustment, would yo be interested in sending a PR?

Pls assign it to me @Borda

Pls assign it to me @Borda

cc: @SkafteNicki


Add pr welcoming badge

Signed-off-by: Open Source Explorer<EMAIL_ADDRESS>
Add PR welcoming badge :+1:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:

Please fix dco signoff.


Tiledscene upgrade

Some of this PR looked very interesting.  Why did you close it?

@kenvandine Still trying to add some finishing touches but will post a newer update soon. Also need to test and update documentation.
My latest changes are aimed towards making the whole API more declarative. You can check out my changes here if you interested.


Enabling SPI ports on some nucleos

Hello,
While testing something on SPI that required to use stm32's, and since I had these nucleos, I enabled the SPI ports relevantly there.
I could successfully test nucleo_f103rb and nucleo_f334r8
@erwango My nucleo_f030r8 st-link acting weirdly as you know I could not test yet (I'll do you trick of using an external st-link). But I believe it should be fine. The only thing which I have a doubt about is the NSS pinmuxing: should it has a pull up?

@tbursztyka : You might want to rebase on #5883.
For nucleo_f030r8, I'll do the test

Codecov Report

Merging #6172 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #6172   +/-   ##
=======================================
  Coverage   52.86%   52.86%
=======================================
  Files         412      412
  Lines       40284    40284
  Branches     7802     7802
=======================================
  Hits        21296    21296
  Misses      15773    15773
  Partials     3215     3215

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4c123a5...7874721. Read the comment docs.

@erwango thx, missed that pr. there is most probably redundant things then

Rebased on top of  #5883 and updated : I removed 2 patches that were then unnecessary.

recheck

@tbursztyka rebase on #5883 again. There were some CI problems solved yesterday by #6223

@ydamigos I did not do it on purpose: why enabling a port by default if it's not in use? I'd rather let the user do that, depending on his need.

@tbursztyka The user might get confused to see them enabled in the dts file. For example, spi_loopback compiles fine but it won't work on the board.

@ydamigos ok, well after users should know what they are doing so if they have unused spi port there, up to them to disable these. Will update

@ydamigos can you review it again? Sounds like it's good to go (hopefully)


Cut yaml lines

for readability

Only way to tell is to run it and check.
On Tue, 21 Feb 2023 at 2:57 am, Étienne @.***> wrote:

@.**** approved this pull request.
Thanks for improving the readability. I can't tell about the deployment
but on the content part LGTM 👍
—
Reply to this email directly, view it on GitHub
https://github.com/ocurrent/ocaml-ci/pull/764#pullrequestreview-1306042799,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABJXOLGTSP2YVIYYBFKE3TWYOIAJANCNFSM6AAAAAAVCA7L7U
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


more macOS testing feedback

brew install libnatpmp qrencode cppzmq
https://github.com/hebasto/bitcoin/assets/152159/2a609840-40f5-4e78-94ff-6563847edc51
cmake version 3.29.2
Darwin DeepSpaceMBPro.local 22.6.0 Darwin Kernel Version 22.6.0: Wed Jul  5 22:21:56 PDT 2023; root:xnu-8796.141.3~6/RELEASE_X86_64 x86_64
changes below were used to generate build on macOS Ventura 13.5 (22G74)

close PR after reading

@RandyMcMillan
Could you please clarify what the problem is?

in the ccmake console - there were extraneous .....
probably doesnt matter...

BUILD_DAEMON has port...

feel free to close - just posting things i noticed.

I can't reproduce it.
On my macOS 14.5 with Homebrew's CMake 3.29.6:

It is essential to tell whether the issue is specific to our project or is related to ccache.


WSL Terminal does not open

Issue Type: Bug
I installed the remote WSL sub system on VS code.
When I selected that terminal it just exits.
I do not actually want to use that terminal, however when I click on terminal option it just exits without giving me a chance to change the terminal to cmd.
What Do I do ?

can you pls provide a gif? @Dialhub


[BUG] Error during the validation

I'm getting the following error:
Epoch 0: 100% 119/119 [02:51<00:00,  1.44s/it, v_num=0]
Validation: 0it [00:00, ?it/s]
Validation:   0% 0/151 [00:00<?, ?it/s]
Validation DataLoader 0:   0% 0/151 [00:00<?, ?it/s]
Validation DataLoader 0:  13% 20/151 [00:08<00:56,  2.32it/s]
Validation DataLoader 0:  26% 40/151 [00:14<00:39,  2.78it/s]
Validation DataLoader 0:  40% 60/151 [00:20<00:30,  2.98it/s]
Validation DataLoader 0:  53% 80/151 [00:25<00:22,  3.09it/s]
Validation DataLoader 0:  66% 100/151 [00:31<00:16,  3.15it/s]
Validation DataLoader 0:  79% 120/151 [00:37<00:09,  3.22it/s]
Validation DataLoader 0:  93% 140/151 [00:42<00:03,  3.27it/s]
Validation DataLoader 0: 100% 151/151 [00:46<00:00,  3.28it/s]Error executing job with overrides: []
Traceback (most recent call last):
  File "/content/kaggle-child-mind-institute-detect-sleep-states/run/train.py", line 73, in main
    trainer.fit(model, datamodule=datamodule)
  File "/content/kaggle-child-mind-institute-detect-sleep-states/.venv/lib/python3.10/site-packages/pytorch_lightning/trainer/trainer.py", line 532, in fit
    call._call_and_handle_interrupt(
  File "/content/kaggle-child-mind-institute-detect-sleep-states/.venv/lib/python3.10/site-packages/pytorch_lightning/trainer/call.py", line 43, in _call_and_handle_interrupt
    return trainer_fn(*args, **kwargs)
  File "/content/kaggle-child-mind-institute-detect-sleep-states/.venv/lib/python3.10/site-packages/pytorch_lightning/trainer/trainer.py", line 571, in _fit_impl
    self._run(model, ckpt_path=ckpt_path)
  File "/content/kaggle-child-mind-institute-detect-sleep-states/.venv/lib/python3.10/site-packages/pytorch_lightning/trainer/trainer.py", line 980, in _run
    results = self._run_stage()
  File "/content/kaggle-child-mind-institute-detect-sleep-states/.venv/lib/python3.10/site-packages/pytorch_lightning/trainer/trainer.py", line 1023, in _run_stage
    self.fit_loop.run()
  File "/content/kaggle-child-mind-institute-detect-sleep-states/.venv/lib/python3.10/site-packages/pytorch_lightning/loops/fit_loop.py", line 202, in run
    self.advance()
  File "/content/kaggle-child-mind-institute-detect-sleep-states/.venv/lib/python3.10/site-packages/pytorch_lightning/loops/fit_loop.py", line 355, in advance
    self.epoch_loop.run(self._data_fetcher)
  File "/content/kaggle-child-mind-institute-detect-sleep-states/.venv/lib/python3.10/site-packages/pytorch_lightning/loops/training_epoch_loop.py", line 134, in run
    self.on_advance_end()
  File "/content/kaggle-child-mind-institute-detect-sleep-states/.venv/lib/python3.10/site-packages/pytorch_lightning/loops/training_epoch_loop.py", line 249, in on_advance_end
    self.val_loop.run()
  File "/content/kaggle-child-mind-institute-detect-sleep-states/.venv/lib/python3.10/site-packages/pytorch_lightning/loops/utilities.py", line 181, in _decorator
    return loop_run(self, *args, **kwargs)
  File "/content/kaggle-child-mind-institute-detect-sleep-states/.venv/lib/python3.10/site-packages/pytorch_lightning/loops/evaluation_loop.py", line 122, in run
    return self.on_run_end()
  File "/content/kaggle-child-mind-institute-detect-sleep-states/.venv/lib/python3.10/site-packages/pytorch_lightning/loops/evaluation_loop.py", line 244, in on_run_end
    self._on_evaluation_epoch_end()
  File "/content/kaggle-child-mind-institute-detect-sleep-states/.venv/lib/python3.10/site-packages/pytorch_lightning/loops/evaluation_loop.py", line 326, in _on_evaluation_epoch_end
    call._call_lightning_module_hook(trainer, hook_name)
  File "/content/kaggle-child-mind-institute-detect-sleep-states/.venv/lib/python3.10/site-packages/pytorch_lightning/trainer/call.py", line 146, in _call_lightning_module_hook
    output = fn(*args, **kwargs)
  File "/content/kaggle-child-mind-institute-detect-sleep-states/src/modelmodule.py", line 97, in on_validation_epoch_end
    preds=preds[:, :, [1, 2]],
IndexError: index 2 is out of bounds for axis 2 with size 2

Set the environment variable HYDRA_FULL_ERROR=1 for a complete stack trace.
Epoch 0: 100% 119/119 [03:57<00:00,  2.00s/it, v_num=0]

@atamazian
The bug was created when the code was UPDATED.
Fixed.See PR below.
https://github.com/tubo213/kaggle-child-mind-institute-detect-sleep-states/pull/37

https://github.com/tubo213/kaggle-child-mind-institute-detect-sleep-states/pull/37

@atamazian
Please comment on Disscussion for a summary, as exchanges on github may fall under the category of private sharing.

Done


[VS Code] The host's network conditions are poor or they are no longer connected to the Live Share cloud relay. Ask them to check their network conditions, restart the collaboration session, and try again.

Error:
The host's network conditions are poor or they are no longer connected to the Live Share cloud relay. Ask them to check their network conditions, restart the collaboration session, and try again.
Steps to Reproduce:

Version Data

extensionName
VSLS

extensionVersion
0.3.834

protocolVersion
2.2

applicationName
VSCode

applicationVersion
1.28.1

platformName
Windows

platformVersion
10.0.17134

Hi,
Thanks for reporting this issue.
Could you please provide some more information? What were you doing when you saw this error message (it looks like you were a guest in the middle of a Live Share session)? How often does this happen? Is it possible that this was a valid error message? Did restarting collaboration session help to resolve this issue?
I'm going to resolve it as a dupe of #555.
Please feel free to reply either here or on #555.


Plugin assets bundle

Have a look at Map.php row 364.
I think this method should call $this->getPlugins()->registerAssetBundles... isn't it?
public function registerClientScript($position = View::POS_END)
{
    $view = Yii::$app->getView();
    MapAsset::register($view);
    $this->getPlugins()->registerAssetBundles($view); // <---- IS THIS MISSING??
    $view->registerJs($this->getJs(), $position);
}

Not really, MapAsset is the one doing it on the line above.

mmmm.... where?
nobody call PluginManager::registerAssetBundles

You know? Please, forgive me. I thought this was leaflet extension (on my mobile).  Good catch.

So is it the right place to put that call?
Anyway... my pleasure to help if I can!


Trailing commas should be used to properly format the code

Current Behaviour:
In the whole code base of the app we are not using the trailing commas to format the code vertically
Expected Behaviour:
We should use the trailing commas to properly format the code
For more info: view this link

@i-am-ijaz can you use this issue to implement more linting rules. For final and constant keywords etc...
max length line : 80 char etc...

@JordyHers Sure, I will implement additional linting rules in the same branch.


build: fix up containerized release build process

rkt run was doing some weird stuff on fedora machines with SELinux in enforcing mode, so we're back to docker for the containerized build wrapper.
After this, we'll be ready to cut v0.9.1.
\cc @mumoshu

@colhom Thanks for this! 👍
It seems that the build failed with missing glide command in the Travis env.
Would you mind fixing it or shall I do it after merging?

@mumoshu will fix

@mumoshu do you thinks it's desirable to have ./build do glide install- or should we simply implicitly trust the checked-in ./vendor folder? I realize now that may have been the original intent, in which case I will back out the modification to ./build here.

@colhom I'm ok with the latter!
// but that's because I have been blindly assuming that's why we've committed ./vendor into the repo.
// off topic but I began to think that we should document the pre-req/implication of ./build somewhere regardless of how we fix this build.

Current coverage is 56.90% (diff: 100%)

Merging #47 into master will not change coverage

@@             master        #47   diff @@
==========================================
  Files             4          4
  Lines           949        949
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            540        540
  Misses          329        329
  Partials         80         80

Powered by Codecov. Last update b02c31c...8b684a1

@colhom LGTM 👍 Thanks


[manuf]: add code to create Cloud KMS signed CA certificate

The modification makes use of the openssl ability to interact with Google Cloud KMS when the appropriate configuration set.
When the appropriate environment variables are set to point to a local instance of Google KMS integration library
(https://github.com/GoogleCloudPlatform/kms-integrations) and Cloud KMS configuration, the second certificate, signed by the configured external crypto engine is also generated.

hi Tim, I know you worked on CA cert name changes, I can update this if necessary. Not even sure we want to merge it, but this is an example of how the KMS signed CA can be created.

hi Tim, I know you worked on CA cert name changes, I can update this if necessary. Not even sure we want to merge it, but this is an example of how the KMS signed CA can be created.

No problem, I will wait to do the renaming until we get this and the following PR (#23069) in to avoid blocking you.

Note, since this is passing CI, I think you can cleanup the comments I mentioned above in #23609, since I made other suggestions on that PR too.

sorry for the typos I should set up vscode to spellcheck bash comments. Will update in the other PR


variable values where possible

If a value could be changed, why does it only appear fixed? It would be much better to use and configure the heat pump if these were numerical entities.
For example, 233 t4_ibh_on.

The answer to your "why" is that I did not implement it yet. It is on my todo list. But in such cases a pull request with working/tested code is always very welcome!

Pull request #11 is merged where a number of entities are made configurable, including #233.


V4: Tooltip border color

Possible to set only border color for tooltip
See: http://jsfiddle.net/2htZe/1852/

Hi @yasinkocak!
You appear to have posted a live example (http://fiddle.jshell.net/2htZe/1852/show/light/), which is always a good first step. However, according to Bootlint, your example has some Bootstrap usage errors, which might potentially be causing your issue:

line 35, column 7: W013: Bootstrap version might be outdated. Latest version is at least 3.3.6 ; saw what appears to be usage of Bootstrap 2.3.2

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!
(Please note that this is a fully automated comment.)

@yasinkocak Please describe your request in greater detail.

@cvrebert on http://jsfiddle.net/2htZe/1852/ u can see the tooltip have border color blue with white background.
If you can make it possible to set variables where u fill in border color and the background of the tooltip.
Now u can set only background color with the same border-color

Hi @yasinkocak!
You appear to have posted a live example (http://fiddle.jshell.net/2htZe/1852/show/light/), which is always a good first step. However, according to Bootlint, your example has some Bootstrap usage errors, which might potentially be causing your issue:

line 35, column 7: W013: Bootstrap version might be outdated. Latest version is at least 3.3.6 ; saw what appears to be usage of Bootstrap 2.3.2

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!
(Please note that this is a fully automated comment.)

Going to punt on adding a border here, thanks though!


Add pouchdb-find support

Should be easy by just adding some kind of custom implementation capability like I did for map/reduce.

I'm able to use the db.find with worker.

Yeah, but I believe if could be more efficient if worker-pouch included find support. Right now it still has to execute a lot of queries between the UI thread and the worker thread, when it could isolate more of them to the worker thread.

:+1:

+1

Does not work for me at all. Pity - this makes this promising project unusable to me. :-(

Yeah, but I believe if could be more efficient if worker-pouch included find support. Right now it still has to execute a lot of queries between the UI thread and the worker thread, when it could isolate more of them to the worker thread.

I would like to tackle this issue and make a PR.
Could you elaborate a bit more and give me some pointers? Thanks! 🙏


Logo Added (Fixed)!

New Material Logo

Logo added
plus typo error in logo fixed

Thanks for this amazing icon @geekykant... merging it.
There is one thing I wanted to ask. When the the timer is running, other apps display monochrome logos. This app however, only displays a square or a circle, depending on the device. Would you know how to fix it and how difficult it is to achieve?
Thanks.
Fixes #25


Introducing new primitive type: Capsule!

This capsule currently spams:
Zero normal found, this leads to invalid polyhedron-point tests
In a lot of cases. I am not sure why. If anyone is willing to help figure it out then please do!
I got the algorithm from this forum thread. I made sure the bottom and the top of the capsule get triangulated.

This capsule currently spams:
Zero normal found, this leads to invalid polyhedron-point tests

I'm not able to reproduce this specific bug, but I do get this error:

Placing a capsule's tip on another brush results in this, and placing an icosphere like so:

produces the same error you are having:

Zero normal found, this leads to invalid polyhedron-point tests

@Kerfuffles It appears to be a "bug" with SabreCSG itself. Loading a perfect capsule mesh yields the same result. Until this brush (or rather the CSG algorithm) is more stable it won't be promoted to the toolbar but I will make it available in the primitive brush inspector.


A few issues

Very cool!
A few comments/issues; apologies for lumping them into one GitHub issue.

README: racket-mode also displays images in the REPL. I'd be grateful if you could mention it, too.

README: A third dependency people need to raco pkg install is quickcheck.
quickcheck seems to have a mistake -- it provides but doesn't define check-result?. After I commented out that, it built fine. I reported that to quickcheck.

Your load-plug-in assumes the current directory. So when I made this little test program in my /tmp folder:

#lang racket
(require identikon/identikon)
(identikon 300 300 "racket")

I got:
; default-load-handler: cannot open module file
;   module path: #<path:/tmp/rules/default.rkt>
;   path: /tmp/rules/default.rkt
;   system error: No such file or directory; errno=2
; Context:
;  /tmp/id.rkt:1:1 [running body]
;  /Users/greg/src/racket/identikon/identikon.rkt:63:0 core16

I worked around it by setting the curdir to your collection:
#lang racket
(require identikon/identikon)
(parameterize ([current-directory "/Users/greg/src/racket/identikon"])
  (identikon 300 300 "racket"))

Then it displays an awesome identikon!
But this probably isn't the way it should work. Maybe you need define-runtime-path to reach files relative to your own source files. Let me know if you have any questions, or if you'd like me to try to submit a pull request.
Again, very cool!

I opened an issue for quickcheck: https://github.com/ifigueroap/racket-quickcheck/issues/6

I submitted a pull request for the 3rd issue.

Thanks @greghendershott - good catch on racket-mode, I use it in Emacs (and it works great btw) but spaced when writing the blog post. Will get that updated.

Will also update the information for quickcheck. This is really a question for @ifigueroap but I wasn't able to use both rackunit and quickcheck in the tests submodule as it looks like quickcheck overwrites check. Do you have any experience working with both of them together?

This is really a question for @ifigueroap but I wasn't able to use both rackunit and quickcheck in the tests submodule as it looks like quickcheck overwrites check. Do you have any experience working with both of them together?

The usual way of handling name conflicts like that is that require has some optional sub-forms like only-in, prefix-in and rename-in.
So you could do something like (require rackunit (prefix-in q: quickcheck)): check would be from rackunit and q:check would be from quickcheck. Or you could prefix-in both. As you prefer.
I haven't used quickcheck much, and never alongside rackunit, so I don't know for sure if it's that simple, but usually it is.  If you do ping @ifgueroap maybe you could suggest they add that as a tip to their README.

Thanks for updating the README to add quickcheck and racket-mode!
One nit: That screenshot is of Geiser (I'm pretty sure from the racket@> style prompt).
I'm definitely not objecting to that. (And BTW I wasn't asking you to remove Geiser from the README, just to add racket-mode).
But I do want to make sure you realize you're using Geiser not racket-mode. Especially if you give me a bug report about it someday. :P

Ahh funny okay, I have racket-mode installed, but didn't realize I was getting the REPL through Geiser.

Allrighty, I just needed to configure racket-mode to point to where I keep Racket, and I got the REPL working:

Very cool. Thanks for your help on this!

Maybe this is a bit late, but now it is possible to use both RackUnit and quickcheck. First, there is no conflict regarding the check symbol. Now quickcheck exports quickcheck and quickcheck/config symbols. Furthermore, the new rackunit/quickcheck collection defines some forms, check-property and check-property/config, where failed properties raise proper RackUnit exceptions. It is also possible to add property-instance-specific metadata to the eventual exceptions using the add-property-check-info form, designed in the same spirit as with-check-info.

very cool, thanks for the heads up @ifigueroap


Programs Not Excluded During Installation

Hi everyone. First, thank you for this resource and tool. Very much appreciated.
I'm hoping you can help out.
I've created a script to exclude certain programs from Office 2016. I'm not using the Office 365 version, I'm using the standalone version of Office 2016.
As I understand it, the Product ID is ProjectProRetail.  I found the Product ID here:
https://support.microsoft.com/en-us/help/2842297/product-ids-that-are-supported-by-the-office-deployment-tool-for-click
In the command prompt, which I've launched as Administrator, I run:
setup.exe /configure configuration.xml
I've included the script generated from the system. Would appreciate any insights on how I can exclude the programs listed.
Many thanks.
Jeffrey

All, I resolved the issue.
There were 2 problems.

Product ID is ProPlusRetail
I was missing the command to download the installer files
setup.exe /download configuration.xml

Once both of those were implemented everything worked.
Again, thank you for this resource


Shadow issues when using path

Hey!
For one of my apps I need to display an SVG ticket made with a path and it needs a drop shadow effect.
Now, since the package doesn't support filters in the master repo (yet?) I'm just using the out of the box shadows of react native.
This works fine on iOS but when i try it on android it's all messed up.
You can't see the content I have on top of the SVG etc.
This is what it looks like on iOS.

And this is what it looks like on Android.

As you can see, Android doesn't follow the SVG path whilst iOS does.
This is the code that the card uses:
import React from 'react'
import { View, Text, StyleSheet, TouchableOpacity } from 'react-native'
import { Svg, Path } from 'react-native-svg'
import PropTypes from 'prop-types'
import Colors from '../../../Themes/Colors'
import Fonts from '../../../Themes/Parser/Fonts'
import BlackTieIcons from '../../../Config/BlackTie/BlackTieIcons'
import AttendeeList from '../Lists/AttendeeList'

const TicketCard = ({eventTitle, location, time, navigate, attendees}) => {
  return (
    <TouchableOpacity onPress={() => navigate('Tickets')}>
      <View
        style={[{alignItems: 'center'}]}>
        <View style={styles.cardStyles}>
          <Svg width="345" height="185" viewBox='0 45 350 185'
               style={[{transform: [{rotate: '3deg'}], elevation: 4, borderRadius: 10}]}>
            <Path fill={'#fff'} fillRule={'evenodd'}
                  d={'M5,116.005446 L5,51 C5,48.2385763 7.23857625,46 10,46 L345,46 C347.761424,46 350,48.2385763 350,51 L350,116.005446 C337.804495,116.271345 328,126.240691 328,138.5 C328,150.759309 337.804495,160.728655 350,160.994554 L350,226 C350,228.761424 347.761424,231 345,231 L10,231 C7.23857625,231 5,228.761424 5,226 L5,160.994554 C17.1955053,160.728655 27,150.759309 27,138.5 C27,126.240691 17.1955053,116.271345 5,116.005446 Z'}/>
          </Svg>
        </View>
        <View style={styles.cardStyles}>
          <Svg width="345" height="185" viewBox='0 45 350 185'
               style={[{transform: [{rotate: '-3deg'}], elevation: 4, borderRadius: 10}]}>
            <Path fill={'#fff'} fillRule={'evenodd'}
                  d={'M5,116.005446 L5,51 C5,48.2385763 7.23857625,46 10,46 L345,46 C347.761424,46 350,48.2385763 350,51 L350,116.005446 C337.804495,116.271345 328,126.240691 328,138.5 C328,150.759309 337.804495,160.728655 350,160.994554 L350,226 C350,228.761424 347.761424,231 345,231 L10,231 C7.23857625,231 5,228.761424 5,226 L5,160.994554 C17.1955053,160.728655 27,150.759309 27,138.5 C27,126.240691 17.1955053,116.271345 5,116.005446 Z'}/>
          </Svg>
        </View>
        <View style={styles.cardStyles}>
          <Svg width="345" height="185" viewBox='0 45 350 185' style={{elevation: 4, borderRadius: 10}}>
            <Path fill={'#fff'} fillRule={'evenodd'}
                  d={'M5,116.005446 L5,51 C5,48.2385763 7.23857625,46 10,46 L345,46 C347.761424,46 350,48.2385763 350,51 L350,116.005446 C337.804495,116.271345 328,126.240691 328,138.5 C328,150.759309 337.804495,160.728655 350,160.994554 L350,226 C350,228.761424 347.761424,231 345,231 L10,231 C7.23857625,231 5,228.761424 5,226 L5,160.994554 C17.1955053,160.728655 27,150.759309 27,138.5 C27,126.240691 17.1955053,116.271345 5,116.005446 Z'}/>
          </Svg>
        </View>
        <View style={{
          justifyContent: 'center',
          alignItems: 'center',
          height: '100%',
          marginTop: 2,
          width: '77.5%',
          marginLeft: 5
        }}>
          <View style={{
            width: '80%',
            height: '100%',
          }}>
            <View style={{
              height: '30%',
              justifyContent: 'center',
              borderBottomColor: Colors.lightPink,
              borderBottomWidth: 0.5
            }}>
              <Text style={{
                fontFamily: Fonts.Bebas,
                fontSize: 25
              }}>{eventTitle}</Text>
            </View>
            <View style={{height: '40%', justifyContent: 'center'}}>
              <View style={{
                flexDirection: 'row',
                alignItems: 'center',
                marginBottom: 8
              }}>
                <BlackTieIcons name={'uniF0CA'} type={'regular'} color={Colors.cloudyBlue} size={16}
                               containerStyle={{width: 19}} iconStyle={{paddingLeft: 1}}/>
                <Text style={{
                  fontFamily: Fonts.Bebas,
                  fontSize: 18,
                  color: Colors.cloudyBlue
                }}>{location}</Text>
              </View>
              <View style={{
                flexDirection: 'row',
                alignItems: 'center',
                marginBottom: 8,
              }}>
                <BlackTieIcons name={'uniF019'} type={'regular'} color={Colors.cloudyBlue} size={15}
                               containerStyle={{width: 19}}/>
                <Text style={{
                  fontFamily: Fonts.BebasRegular,
                  fontSize: 18,
                  marginBottom: -5,
                  color: Colors.cloudyBlue
                }}>{time}</Text>
              </View>
            </View>
            <View style={{
              height: '30%',
              justifyContent: 'center',
              borderTopColor: Colors.lightPink,
              borderTopWidth: 0.5
            }}>
              <AttendeeList attendeeList={attendees} attendeeExtra={10}/>
            </View>
          </View>
        </View>
      </View>
    </TouchableOpacity>
  )
}

const styles = StyleSheet.create({
  cardStyles: {
    justifyContent: 'center',
    alignItems: 'center',
    position: 'absolute',
    shadowColor: Colors.banner,
    shadowOffset: {width: 0, height: 2},
    shadowOpacity: 0.8,
    shadowRadius: 3,
    width: 350,
  }
})

TicketCard.propTypes = {
  eventTitle: PropTypes.string.isRequired,
}

export default TicketCard

I know, I know, it's a bit messy ;P
Does anyone know if it's possible to achieve this on both android and iOS?

On android, it would require either filters, or implementing ViewOutlineProvider and getOutline, and it only works with convex paths, so the svg would need to be converter to that first. https://books.google.fi/books?id=rSuuDQAAQBAJ&pg=PA606&lpg=PA606&dq=ViewOutlineProvider+getOutline+convex+path&source=bl&ots=5MfYqIqu6L&sig=ACfU3U2GnuB9HvapQtnCZPa8COm_rjncFQ&hl=en&sa=X&ved=2ahUKEwiNjeT5iNfhAhVkz6YKHdhmAVAQ6AEwB3oECAkQAQ#v=onepage&q=ViewOutlineProvider getOutline convex path&f=false

You could probably use https://github.com/react-native-community/react-native-blur as well

Thanks for the recommendations, I'll try it out.

@msand is the ebook you sent comparable with this package? https://github.com/folofse/androw

Androw actually seems like a better/more general solution. I would recommend trying that as a first choice.

Alright thanks! Then i'll try using that.

Using the androw package and changing the shadowradius depending on the OS I was able to get the shadows to work correctly.

Great! Could you post a screenshot? Would be interesting to see how well it works, with regards to prioritizing implementation of filters for android (can be achieved using e.g. renderscript).

Sure! This is what it looks like on iOS right now:

(still need to fix the size, this is on an iPhone 5)
And this is the result on Android:

Also doesn't have the right size yet but that's not relevant for this thread ;P
Using the package is extremely easy. You just use it as a view and it works immediately on both iOS and Android. Kudos to the creator!

Looks good. I tried optimizing the implementation a bit, could you try my pr? https://github.com/folofse/androw/pull/1
https://github.com/msand/androw/commit/b8626c403a3087173bc9ed44a4d1d8a07b05e9ea

The latest commit in my pr seems to perform quite nicely, the dispatchDraw takes 30 000 - 300 000 nanoseconds when animating opacity, color and offset of the shadow. Animating radius of the blur/shadow is slower, but probably needed less often as well.

I also tried making an experiment with RenderScript, but it was about ten times slower: https://github.com/msand/androw/commit/c3a70d376b895b0a544af35421b43e72f2f6d687

I'll try both out tonight and post the results on here!

@1070rik hey can you please share your code for shadow effect for Android device? It's working only on iOS for me.

const styles = StyleSheet.create({
shadow: {
shadowColor: '#000',
shadowOffset: {
width: 0,
height: 3,
},
shadowOpacity: 0.29,
shadowRadius: 4.65,
elevation: 7,

},
});
This is what I am trying to do.

@1070rik, @msand hey can you please share your code for shadow effect for Android device using Androw? It's working only on iOS for me.
gist.txt
This is what I am trying to do. Any help will be appreciated. Thanks!

@karan-gujral Sure, this is the code that I ended up with and which is working right now.
https://gist.github.com/1070rik/3649de9004dbe0f8d5cb1770cd192f2d
Hope it'll help you out.

Thank you so much @1070rik !! It's working now. Seems like the Androw Library does not support Groups in SVG for Android. I removed the groups from my SVG (same as yours) and it worked.

@karan-gujral No problem, glad I could help!


HPX does not build with Boost 1.58.0

I am trying to build HPX 0.9.10 with Boost 1.58.0. This is on OS X using clang 3.6.0. The build aborts with the error message
/Users/eschnett/software/boost-1.58.0/include/boost/serialization/vector.hpp:216:18: error: comparison of integers of different signs: 'int' and 'base_type' (aka 'unsigned long') [-Werror,-Wsign-compare]
    for(i = 0; i < count; ++i){
               ~ ^ ~~~~~

This looks as if Boost misbehaved, iterating over the element of std::vector using int (why???). HPX then adds -Werror to the mix, leading to this build failure.
It is customary to disable -Werror in released versions for this reason.
This problem sounds oddly familiar; was this discussed before?

You reported that before (see #1475). As said, Boost V1.58 was not released at the point when we released HPX V0.9.10.

Until the next release of HPX please patch your local copy of HPX by removing the -Werror flag locally (https://github.com/STEllAR-GROUP/hpx/blob/0.9.10/CMakeLists.txt#L818).


Title hide in viewlet when should to be show.

I want title for this viewlet.

I don't understand what you mean. And your screenshot is the same one that you posted at #121649

Oh! Sorry! I hope you understand, I thank you for your efforts.
[en][Bing/me Translate]
In this open viewlet (popup) by pressing the green button in the lower left corner according to the image for Open a Remote Window. It should have a title like Open a Remote Window in viewlet and in window.
[pt-br][Original]
Nessa viewlet (popup) aberto ao pressionar o botão verde no canto inferior esquerdo de acordo com a imagem para Open a Remote Window. Devia ter um title like Open a Remote Window na viewlet and na window.

pinging @aeschli because #121180 also relates to this quickpick. Personally I don't think it needs a title. Perhaps some prompt text, thike the language selector has?

pinging @aeschli because #121180 also relates to this quickpick. Personally I don't think it needs a title. Perhaps some prompt text, thike the language selector has?

Your idea is interesting! We can get on with it!
A sua ideia é interessante! Podemos prosseguir com ela!

No sure what the title would be to be helpful. Select a remote window command. ?
Also, the regular command palette also has no title.

I agree with @aeschli . When the Command Palette quickpick appears it doesn't tell me to "Enter a command", probably because it assumes I know that I invoked it for the purpose of doing exactly that.

No sure what the title would be to be helpful. Select a remote window command. ?
Also, the regular command palette also has no title.

[pt-br][Original]
O título deve pegar o atributo title do botão que foi clicado antes de abrir e deve-se deixar implementado para no futuro permitir encapsular esse título como em uma concatenação.
Se o título na barra de status já existir, faz-se um join dos títulos com '>', caso o resultado do título final vá quebrar o layout então deve-se mostrar o primeiro e o último título separados por " > ... > ".
Esse é um recurso de acessibilidade.
[en][Bing Translate]
The title must take the title attribute of the button that was clicked before opening, and it must be left implemented to allow you to encapsulate that title as in a concatenation.
If the title in the status bar already exists, you join the titles with '>', if the final title result is going to break the layout then you must show the first and last heading separated by " > ... > ".
This is an accessibility feature.

I need than Microsoft implements.


Refactors tests to use promises

These changes are in reference to #7166. I was hoping to get some feedback to ensure this is the desired direction before moving forward with updating more of the tests.
For the most part I just removed the done() callback and .catch();, but I also had to update 2 tests as they failed when removing the callbacks, which makes me suspect they weren't functioning as intended anyway.
Please let me know if this is going in the right direction and I can continue updating more of the tests, unless someone else has already taken on this work.
Cheers!

@jeffjewiss this is definitely heading in the right direction!
Only thing I would say is whenever you see any checking going on in a catch block, make sure there's a then block before it that throws an error.

@acburdine thanks for the quick feedback!
Okay sounds good, I'll keep going with updating the tests and make sure to fix the ones I changed with then/catch blocks. I'll keep committing to this branch and ping you when I've gone through them all.

Hey @jeffjewiss, wondering if this is this something you're still working on?

@ErisDS thanks for checking in, I wasn't able to work on this due to moving to a different country. I'm still working on this and should be able to fix the failures in a few days.

Hi @ErisDS, I think I've removed as many callbacks as possible without altering how the tests are written. This PR should be ready for review now, and I'm assuming I'll need to squash the commits too, but I'd prefer to do that after review if that's alright. Thanks!

@jeffjewiss I apologize that it's taken so, so long to get to this. Would you mind fixing up the merge conflicts once you have time (don't feel the need to do so until after the 🎄 season 😉)

@acburdine thanks for following up on this. Yeah, I should have time over the next few days. I'll let you know once I've gone through it and pushed up the changes. Cheers.

@jeffjewiss would love to give you a hand with this. Will have a look at some of the tests using superagent which haven't been re-worked yet.

@jeffjewiss Could you please tell me if you would like to rebase and resolve your conflicts or if i should close this PR?
Thanks for your help 👋

@janvt thanks for the help and going through the tests.
@kirrg001 thanks for checking in, yeah I'll give it a go this weekend. Started a new job and haven't had as much time lately.

@jeffjewiss Hey ✌🏻 any update on this? 🙃

@kirrg001 thanks for your patience on this. I took a look this weekend, but it's going to take me longer than expected to rebase it off master due to how far behind this branch got. I'm going to chip away at it this week. I'll aim to post updates when I get through different sections of tests.

Hey again 👋 Let me know if i should close this PR.

Hey @jeffjewiss we're cleaning up post 1.0 so I'm going to close this as it's almost a year old. If you get a chance to rebase, you can always reopen it :)


Publish Envelope

Summarise the changes this Pull Request makes.
This is a step towards the "envelopes" proposal that separates out message metadata from the body. In this PR it is only on the Publish path, as the Publish path is a simpler first step.
Earlier investigation has raised that the envelope metadata used to configure a publish is different to the the metadata received with the message at the other end of the process. So these can be approached separately.
There are two fields on the Message base class that are optional and are used only on Publish. they are DelaySeconds and MessageAttributes. They are important, but are used in a minority of cases.
These are removed from the Message base class, and are on the PublishEnvelope class instead.
We now are facing more overloads - with/without an Envelope, with/without the CancellationToken so the overloads are moved to extension methods in MessagePublisherHelpers. (they could also be default interface methods, when this feature ships).
This should not affect the "wire format" of messages sent.
please review, naming etc is still up for discussion.
Please include a reference to a GitHub issue if appropriate.

My understanding of default interfaces was that they require runtime support, so aren't usable by .NET 4.8.

This looks good, however I don't think it warrants the symmetry of doing the same for handlers (I know I'm getting ahead of myself). I have a little POC locally to illustrate a competing idea, not saying it's right, just worth exploring a few options IMO.

There is another route to go here, and that is a seperate object, e.g.
public async Task PublishAsync(Message message, PubishMetadata metadata = null, CancellationToken ctx = CancellationToken.None)

We can go that route if people prefer it.

I really like that option, it feels cleaner

The real goal here is: remove Publish metadata from the Message base class, in preparation for thinking about similar with respect to Subscribing.

The intent is: On the Message base class, remove metadata that is only relevant to a Publish and put that on a separate object.  Do as much of this as is possible without changing the "wire format". int? DelaySeconds is a good example of publish metadata, as it is an instruction on how to publish the message This is in preparation for the similar but larger task of doing same with respect to Subscribing.
In subscribing we particularly want to go from a "fat" message base class, to a way to get at the complete message metadata that AWS will supply, but that is a later step.
Option "envelope"
Now we have 2 related option on publishing. The First is detailed in this PR, and using it would be like
var envelope =  new PublishEnvelope(message);
envelope.DelaySeconds = 3;
await publisher.PublishAsync(envelope, CancellationToken.None)

Along with various overloads and helpers to make this easier for the default case where the envelope contains nothing but the message.
Option "metadata"
The second option is a separate object for publish metadata instead of a wrapper
var metadata = new PublishMetadata();
metadata.DelaySeconds = 3;
await publisher.PublishAsync(message, metadata, CancellationToken.None)

Along with various overloads and helpers to make this easier for the default case where the metadata contains nothing.

My initial thoughts are to prefer the envelope approach, as the other would seem at first glance to require more overloads to maintain to keep it like-for-like for consumption just using Message classes.

Dump of thoughts:
In terms of publishing envelope we should consider if the message is published to SNS topic where those attributes would not make much sense (as far as I'm concerned) or to SQS queue where this makes much sense.
Does this mean we will have to distinguish SNS/SQS envelope?
JustSaying is designed to work with AWS SDK. Specifically with SQS/SNS combo.
We are abstracting underlying infrastructure, and will have to extend contracts each time AWS will introduce another feature in SQS or SNS.
There is a direct transformation or map from Message to SNSPublishRequest or SQSPublishRequest, I was thinking maybe it would be a little bit more resilient to changes in AWS SDK if we offer some sort of user defined manipulation on SNS/SQS requests, something like middleware.

Dmitry has a good point: Publishing happens one of 2 ways

Publishing to SQS happens in SqsPublisher where it builds a Amazon.SQS.Model.SendMessageRequest which has DelaySeconds but no MessageAttributes

What's more common is the publish to SNS, which happens in SNSTopicBase, where it builds a Amazon.SimpleNotificationService.Model.PublishRequest which has MessageAttributes but no DelaySeconds.

So the PublishEnvelope is a collection of these two properties,  the union of what is needed in 2 different cases. And actually it doesn't overlap.
Still, I'd rather section this problem into something that is clearly only about publishing, rather than have it on the base class of ever message sent or received.
What would the middleware look like? allow the user to supply optional Action<PublishRequest> and Action<SendMessageRequest> in order to transform them? or Func<PublishEnvelope, PublishRequest> to generate them?
Would this replace or add to the DelaySeconds or MessageAttributes properties?

After some lurking in codebase found this:
This is what available for publishing:
sns request

public string Message { get; set; }
public Dictionary<string, MessageAttributeValue> MessageAttributes { get; set; }
public string MessageStructure { get; set; } // not exposed
public string PhoneNumber { get; set; } // not exposed
public string Subject { get; set; }  // implicit
public string TargetArn { get; set; } // not exposed
public string TopicArn { get; set; } // implicit

sqs request

public int DelaySeconds { get; set; }
public Dictionary<string, MessageAttributeValue> MessageAttributes { get; set; } // exposed but not used in sqs path
public string MessageBody { get; set; }
public string MessageDeduplicationId { get; set; } // not exposed
public string MessageGroupId { get; set; } // not exposed
public string QueueUrl { get; set; } // implicit

Not sure that phone stuff is particularly interesting but MessageGroupId  and MessageDeduplicationId  are very interesting for fifo cases
As for middleware, (or maybe a hook ?) It would be usefull for it to have shape of

Func<TSource, SNSPublishMessage, SNSPublishMessage,>
Func<TSource, SQSPublishMessage, SQSPublishMessage,>

What about adding some builder overloads when registering publishers to use one of these?

Also, based on the HttpRequestMessage we could introduce some store for user state (Dictionary<string, object> ? ) which would not be transferred on the wire, but could be used during interception phase.

Sample code for simple user transform in envelope, actioned by SnsTopicBase:
var request= new PublishRequest
{
	TopicArn = Arn,
	Subject = messageType,
	Message = messageToSend,
	MessageAttributes = BuildMessageAttributes(envelope)
};

if (envelope.SnsTransformer != null)
{
	request = envelope.SnsTransformer.Invoke(request);
}

return request;```

I've but WIP on this on the basis that we don't know which candidate is the winner yet between this and #449.

Some quick thoughts I had about this that I think would be easier with envelopes, so I'm putting it here.
In some cases, attributes would be homogeneous to all messages, rather than per message, right? For example, all messages of type OrderCreated come from domain=order, and there might be no other attributes to add.
In these cases, we could introduce something centrally for publishing that could apply a "global" set of attributes for a message type that always get added prior to publishing onto SNS/SQS. This would then allow code to be updated at registration time to add the attributes, with the call sites remaining untouched and the extra attributes for the messages just works.
So a quick hacky-not-in-an-IDE example:
config.AddMetadata<OrderCreated>("domain", "order")
      .AddMetadata<OrderCreated>("importance", "high")
      .AddMetadata<OrderCreated>("foo", "bar")
      .AddMetadata<OrderPaid>("domain", "payments");

This would then transparently add these three attributes to all OrderCreated messages that are published. Any metadata specified at publish time by the caller would overwrite and/or add to the properties:
var envelope = new Envelope(new OrderCreated());

envelope.MessageAttributes.Add("id", "abc123");
envelope.MessageAttributes.Add("importance", "low");
envelope.MessageAttributes.Add("ordertype", "test");

await publisher.PublishAsync(envelope);

Which would yield a message with the attributes:
domain=order   // Global
importance=low // Was overridden at publish time
foo=bar        // Global
id=abc123      // Added at publish time
ordertype=test // Added at publish time

I agree, some attributes will indeed be the same across a lot of messages, and it would be better to set them up at startup. I think touched on this somewhere. In a later PR maybe.

If we wanted to do C# attributes, I'd suggest that as an alternative that's available in-box for those so minded. As attributes can only really done done at compile-time, having them as the only way would preclude other approaches such as applying attributes from a config file, or ones that need to be different per-message.

Good point about the compile-time restriction! I wasn't implying that the attribute way should be the only way though.

The consensus reached in the discussion we had over hangout this afternoon was to pursue the metadata option (#449) over envelopes.


Add sample crops ml code

@Subash-Lamichhane please sign your commit.
You can do that with:
git rebase HEAD~1 --signoff
git push -f

@Tpuljak Thanks for help,
I have signed the commit.
Could you check this PR once again.


Image clipping/buggy

Hello, I am trying to make a UI like TikTok but the images are behaving weirdly. It works fine on my 11 pro max but not on a screen size smaller, it sort of jumps around. Screenshot
Pages image issue (clipping)

Decided to open it up here since it is a separate issue

Could you send a short reproducible example so I can test it on my end?


Vulnerability roundup 97: libgcrypt-1.5.6: 1 advisory [4.9]

search, files

[ ] CVE-2018-12437 CVSSv3=4.9 (nixos-20.09, nixos-unstable)

Scanned versions: nixos-20.09: 50c8d1e0deb; nixos-unstable: 83cbad92d73.

See also: #90845

@9999years spotify-unwrapped is using this

@9999years spotify-unwrapped is using this

What exactly needs to be done here? I'm not familiar with these vulnerability roundups.

What exactly needs to be done here? I'm not familiar with these vulnerability roundups.

We should remove libgcrypt_1_5 from nixpkgs. Spotify is one of the packages preventing us from doing so. It would be nice if someone could look into using libgcrypt for Spotify.

We should remove libgcrypt_1_5 from nixpkgs. Spotify is one of the packages preventing us from doing so. It would be nice if someone could look into using libgcrypt for Spotify.

Based on my understanding we're only using libgcrypt: https://github.com/NixOS/nixpkgs/blob/master/pkgs/applications/audio/spotify/default.nix#L3 though I could be mistaken.

Based on my understanding we're only using libgcrypt: https://github.com/NixOS/nixpkgs/blob/master/pkgs/applications/audio/spotify/default.nix#L3 though I could be mistaken.

https://github.com/NixOS/nixpkgs/blob/59e73f2/pkgs/top-level/all-packages.nix#L24797

https://github.com/NixOS/nixpkgs/blob/59e73f2/pkgs/top-level/all-packages.nix#L24797


gNMI-1.8_Configuration_Metadata-only_Retrieve_and_Replace : Adding ho…

gNMI-1.8_Configuration_Metadata-only_Retrieve_and_Replace : Adding hostname before doing gnmi.get of hostname, not assuming  that it is pre configured

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 50.147%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
854

Relevant Lines:
1703

💛  - Coveralls

/gcbrun

/gcbrun

LGTM, pending internal presubmit test.

internal b/267812765

@sachendras also LGTMed offline.


Workspace-Gene ID appears empty in summary box

The B. ovis genome is not private.  I like to capture the peg id from the summary box as I can't cut and paste from the table.  When I look at the summary box while in the Features, I see that the geneID is listed as 0.  Usually it has the peg ID there.

Actually, belay that.  I guess when I found the peg in there, that was the unusual case.  I'll try to repeat it.

gene_id refers to the NCBI/GenBank gene id. For some reason, when it is not present, it got stored in the database as zero instead of null.
In any case, It should not have peg id in there.

There are many cases where if gene id and gi numbers (and probably other numeric identifiers) where null, they got stored as 0 in the database. This may be slightly confusing but there are no side effects or unintended consequences here. Hence, instead of updating millions of records in the database, I am leaning towards of leaving them as it is in the database, unless someone disagrees.
-Maulik


Fix z-order for text field/area placeholder

This changes the order in which the placeholder and actual text field are laid out in the TextField and TextArea code. Since the two have the same zIndex (the default value, 0f) their placement order is reflected in the drawing order: the last placed is drawn last, on top of the rest.
This fixes a minor cosmetic issue, where the placeholder would be drawn on top of the cursor when the TextArea/TextField is empty.
Fixes #610

Thank you!!

You're welcome 🤗 it's available in 0.25

Is jewel 0.25 compatible with  IJ 241? Just want to make sure.

Yes. Just make sure you use the -241 artefacts.
The supported IJP versions table in the readme is up to date


Can't display some field columns

There should be geo, machine, event columns after the response column in kibana_sample_data_logs, but they are not showing up for query select * from kibana_sample_data_logs.

Geo is because we didn’t set proper JDBC type for geo type. But for others, is the behavior inconsistent with sql plugin?

Geo is because we didn’t set proper JDBC type for geo type. But for others, is the behavior inconsistent with sql plugin?

The table is using aliases in JDBC response as column names even if aliases are empty, I submitted a pr that fixes this.