Add TemporaryFolder.getRoot()

I am in the process of converting a lot of unit tests to JUnit 5. Almost all of these that use the old TemporaryFolder rule use getRoot() to get the temporary folder itself.
With your extensions, I've changed all these calls to TemporaryFolder.createDirectory("") to mimic this behaviour, but this seems hacky.
Also, I think it's a bit surprising that your Extension can only create a single instance of TemporaryFolder. If unit tests define multiple TemporaryFolder-parameters, all of them will be the same instance.

Thanks for raising the issue.
I'm open to making some changes here but I think I need some additional details so that I can understand the rationale for them.

I've changed all these calls to TemporaryFolder.createDirectory("") to mimic this behaviour, but this seems hacky

If you want to get access to the root folder so that you can create a file or directory within the root directory then you should be using createFile(), createDirectory(). However, you've already seen that createDirectory() meets that need so I'm guessing that you have some other use in mind for the root directory. Could you explain - maybe by providing an example - why you want to get access to the root folder?

I think it's a bit surprising that your Extension can only create a single instance of TemporaryFolder. If unit tests define multiple TemporaryFolder-parameters, all of them will be the same instance.

Are you proposing something like this?
  @Test
  @ExtendWith(TemporaryFolderExtension.class)
  public void canInjectATemporaryFolderAsAParameter(TemporaryFolder thisTemporaryFolder, TemporaryFolder thatTemporaryFolder)
      throws IOException {
    ...
  }

If so, I hadn't really considered the possibility that someone would want to inject multiple instances of TemporaryFolder rather than, say, injecting one and calling createDirectory() multiple times. Could you explain - maybe by providing an example - why you might want to inject multiple instances of TemporaryFolder.

The "why" is the same for both questions: I need do convert several hundred of unit tests that all use getRoot() :) So to make it easy, I've just replaced all getRoot() calls with createDirectory(""). Also, I don't see why I should be forced to create a sub-directory inside a directory that is already temporary.
Most of the unit tests test classes that accept a java.io.File as a (constructor-)parameter, als almost all of these tests just pass tmpFolder.getRoot().

And yes, your example of using multiple TemporaryFolder parameters is exactly what I needed. For me, this is not important anymore, as only a handful of unit tests needed more than one temporary folder, and I could manually edit these to just call temporaryFolder.createDirectory("bla[1-2]").

Ok, so your tests do something like
doSomething(temporaryFolder.getRoot());
// ... assert

And the equivalent - using this extension - is:
doSomething(temporaryFolder.createDirectory(someRandomDirectoryName));
// ... assert

So, this extension - in a bid to hide the root folder, since the  extension is responsible for creating and deleting it - is (a) too verbose for your use case and (b) creates a sub directory when you only need to use the root directory.
If that assessment is correct then I can see that exposing getRoot() would help but I still have concerns about the second issue. The TemporaryFolderExtension could be re-worked to inject (and manage) N instances of TemporaryFolder but I wonder whether this is a sufficiently general use case to warrant the additional complexity inTemporaryFolder. Perhaps a public accessor such as anyDirectory() would reduce the verbosity (there would be no need to create someRandomDirectoryName) a little bit whilst still using a single instance of TemporaryFolder. The caller would be forced to invoke a method other than getRoot() but there wouldn't be any material issue with creating additional folders since the net increase in folder creation is only one; create one instance of TemporaryFolder with multiple sub folders vs. create multiple instances of TemporaryFolder.
So your test might look like:
// when a single instance of temporaryFolder (and hence the same root folder) suffices
doSomething(temporaryFolder.getRoot());

// when a single instance of temporaryFolder (and hence the same root folder) does not suffice
doSomething(temporaryFolder.anyDirectory());
doSomething(temporaryFolder.anyDirectory());

Thoughts?

Well, yeah, in theory I agree with everything you said.
In my case, even a single unit test may call getRoot() several times, and of course expect the same result with every call. So instead of temporaryFolder.createDirectory(someRandomDirectoryName) I've opted for temporaryFolder.createDirectory(""), just because it's shorter :) This also does return the root folder itself, but this of course isn't strictly necessary.
Pragmatically speaking, your library advertises itself with offering replacements for popular JUnit 4 Rules, so I think it is safe to assume that many people will choose this library in an attempt to convert legacy tests to JUnit 5. Because of this, I think it would be the right thing to offer a very similar API as the legacy JUnit 4 Rules to ease the migration. As it was possible to define multiple TemporaryFolder-Rules in a single unit test with JUnit 4, I see no reason why your library should differ here. (And if it does, it should at least throw an exception or something to prevent misuse).
Your proposal of temporaryFolder.anyDirectory() would certainly help for tests that originally used multiple TemporaryFolder-Rules, but again, this makes refactoring of several hundreds to thousands unit tests (as in my case) a lot harder than just converting multiple legacy Rules to multiple TemporaryFolder-parameters.

Keep in mind that there is no need for you (of course there is not "need" anyway) to do anything here, as I was able to archive everything I needed to do. These are just my thoughts on things that could have been nicer, because these where the things I stumpled upon while migrating A LOT of unit tests to JUnit 5. Let me use this opportunity to thank you for this library which made my life just a bit easier :)

I'm adding TemporaryFolder.getRoot(), in addition to your specific use case the argument for 'ease of migration' is compelling.
Thanks for raising the issue.

Thanks :) I absolutely agree that it is safer to not expose the root folder, but well, JUnit 4 did... And of course, any malicious caller could just call createDirectory("../bla") anyway (or just call createDirectory("") as I did, which kinda feels malicious).

Yeah, so much for my safety conciousness ... I didn't like exposing the rootFolder but didn't notice the createDirectory("") backdoor :)
Anyway, mirroring the analogous JUnit4 behaviour for 'ease of migration' (as long as that behaviour isn't definitely undesireable) makes sense.
Thanks again, much appreciated.

I have added TemporaryFolder.getRoot() and published a new version of this library - 2.2.0 - to Maven Central.


Link the documentation pages to their source on Github

It would be awesome if you provide a link to the source of the documentation pages for each doc page. It would help the users to fix the typos or even add missing content to the documentation. Add a line like this at the bottom of the documentation page:
To edit this page, kindly [view its source](github-link) on the Github.

links are added at the bottom of each page. Thanks for suggestion

Yes. We can close the issue now.


Fix install instructions

This project uses main instead of master and also has no release tags, so mint tries to install from a non-existing master branch and fails. Specifying @main seems to fix the install and well as the mint run command. As an alternative, you could make a release tag, but I thought making a PR would be more helpful for discussion.

@twostraws No hurry, just mentioning you in case you didn't see this PR.

Yes, I did see this and in fact asked Yonas whether this could be fixed in Mint itself. In the meantime I'll merge this – thank you!


Fix language code returned by ManageIQ rest API

When setting user language is self-service, we rely on user's language preference as returned
by the rest api. In the case of language variants (such as zh_CN), the rest api returns the language
name with underscores (i.e. as 'zh_CN'), while the catalogs as returned from translators
contain a dash (i.e. 'zh-CN').
This mismatch would cause the self-service application not to be able to find the chinese translation
catalog.
@himdel

@himdel Fixed, thanks.

@himdel @dclarizio Review / merge?


Feedback for How to set an interest on or off

This page should contain an example on how to work with it interest through a CRMScript

Document Details
⚠ Do not edit this section. It is required for docs.superOffice.com ➟ Docs Team processing.

Content Source: toggle_contact_interest_entity

Refactoring this to be a section under automation/crmscript/working with.. instead, to consolidate information


Move request-related RPC code

Server-side code shouldn't be exposed to the client, it's very confusing and leads to useless package bloat.

I'll merge it after #2555, there will be minor conflicts.


Mythic Ability: Not a Chance

Here's Idea for armor version of "Always a Chance" mythic ability.
When enemy rolls 20 and attack still doesn't exceed your AC the attack will still miss despite being a 20.

What kind of AC do you plan that would cause a natural 20 to still miss? :D
I already though Always A Chance is too weak for an ability. Hence the patch to buff it.

Well... I had 52 AC on one character.

Added. Please reply if not working.


Ignore processing instructions to fix errors caused by them

Simply ignore processing instructions, or else it will cause schema inference and parsing to fail when included inside an element.
Closes #411

Thanks @srowen.


dropping support for centos 6 in test-kitchen

removing references for my sanity that centos 6 no longer is supported.
https://github.com/docker/docker/issues/14174#issuecomment-118169576

unfortunately I'm gonna have to close this ;)

lol wha?? Why no reason on not merging this in?

Are you able to actually get this to work with centos 6 in test-kitchen? I've had no luck making it work.

Yep. I just fixed up the sysvinit stuff as well.
On Fri, Feb 12, 2016 at 5:15 PM, Chase Bolt<EMAIL_ADDRESS>wrote:

Are you able to actually get this to work with centos 6 in test-kitchen?
I've had no luck making it work.
—
Reply to this email directly or view it on GitHub
https://github.com/chef-cookbooks/docker/pull/642#issuecomment-183392378
.

Aha... I see what was going on... I was using Digital Ocean to test centos6
and they have a modern kernel.
It fails with Vagrant.
one sec...
On Fri, Feb 12, 2016 at 5:16 PM, Sean OMeara<EMAIL_ADDRESS>wrote:

Yep. I just fixed up the sysvinit stuff as well.
On Fri, Feb 12, 2016 at 5:15 PM, Chase Bolt<EMAIL_ADDRESS>wrote:

Are you able to actually get this to work with centos 6 in test-kitchen?
I've had no luck making it work.
—
Reply to this email directly or view it on GitHub
https://github.com/chef-cookbooks/docker/pull/642#issuecomment-183392378
.

aaand it's gone from .kitchen.yml
On Fri, Feb 12, 2016 at 5:26 PM, Sean OMeara<EMAIL_ADDRESS>wrote:

Aha... I see what was going on... I was using Digital Ocean to test
centos6 and they have a modern kernel.
It fails with Vagrant.
one sec...
On Fri, Feb 12, 2016 at 5:16 PM, Sean OMeara<EMAIL_ADDRESS>wrote:

Yep. I just fixed up the sysvinit stuff as well.
On Fri, Feb 12, 2016 at 5:15 PM, Chase Bolt<EMAIL_ADDRESS>wrote:

Are you able to actually get this to work with centos 6 in test-kitchen?
I've had no luck making it work.
—
Reply to this email directly or view it on GitHub
https://github.com/chef-cookbooks/docker/pull/642#issuecomment-183392378
.

Hahaha ok then! ;)

README testing matrix please :)


is it possible to add more methods like RNN or CNN from keras?

Hi all,
Just wonder is it possible if I want to use methods from keras to build CNN? Since there are all basic classifiers from sklearn in the methods for now. And does it accept 3d input?

This would be fulfilled by #113 which provides keras models

Closing in favor of #113 issue.


code question

In the function one_shot_lowest_ranking_filters(),  why use the dictionary self.chains()?  I didn't understand.

Hi,
Thank you for your interests, I could have written the comments more clearly.
self.chains is a dictionary that stores the dependency.
For example, if layer A is followed by a depth-wise convolution layer B, then layer A and layer B are chained together. Another example is the residual connection. The operand of the sum (two convolution layers) are chained together.
This chain is built for different networks (currently pseudo-hard-coded) in fp_resnet.py and fp_mbnetb2.py.

Thank you for your reply.
For Resnet, the process of pruning is a little complicated. According to the paper, the importance of a filter is just a transformation of its l2 norm. But in the code these filters seem to be correlative because of the residual connection. Why do you calculate the importance like this?

Hi,
According to the explanation in above, when one prunes off a filter that is chained to some other filters, they all have to be pruned to ensure the resulting network still runs.
The idea behind the implementation is straightforward in the sense that we just add up the l2 of the filters that are chained together. It is because when one prunes away one of them, all of them has to be pruned.
Also, actually MobileNetV2 is more complicated since ResNet does not really need to tie the operands of the summation together, one can prune a filter and pad 0. However, this introduces a lot more implementation overhead and is obviously less attractive from a speed improvement perspective.
Hope this addresses your questions

I understand, thank you very much.


Rename TileTexture to TileTextureIndex or TextureIndex

I think we should rename TileTexture to TileTextureIndex or TextureIndex
It is confusing that the name indicates a texture when it is just an index to a texture:
https://docs.rs/bevy_ecs_tilemap/latest/bevy_ecs_tilemap/tiles/struct.TileTexture.html
It would also improve naming clarity here:
https://docs.rs/bevy_ecs_tilemap/latest/bevy_ecs_tilemap/tiles/index.html

Lets go ahead and do this. 👍 Feel free to open a PR, or I'll get to it in a few days.

I am happy to do it.

This is finished closing this issue now.
@frederickjjoubert Thank you for the PR!


Unwelcome reconnections

Hello,
I've tried your plugin and got it to work with Microsoft Entra ID, many thanks for all the work you put into this !!
I still have two problems with this, and I'm not sure if it's a feature request, a bug or a misconfiguration :

When the TLS soft reset occurs, it relaunch the web auth, popping a browser in the face of the user and sometimes during a presentation, or in the wrong browser session, causing all sort of troubles. Is there a way to prevent the web-auth  durint TLS soft reset ? In other words, is there a way to prevent the server to REAUTH when reneg-sec has expired ? I know I can set reneg-sec to 0 to prevent TLS soft reset, but I don't want to create a security issue

2023-12-21 16:13:41 pc-xxxxxx/A.B.C.D:59833 TLS: soft reset sec=3596/3596 bytes=11766644/-1 pkts=20362/0
2023-12-21 16:13:41 pc-xxxxxx/A.B.C.D:59833 VERIFY OK: depth=1, C=, ST=, L=, O=, OU=, CN= CA, name=, emailAddress=
2023-12-21 16:13:41 pc-xxxxxx/A.B.C.D:59833 VERIFY KU OK
2023-12-21 16:13:41 pc-xxxxxx/A.B.C.D:59833 Validating certificate extended key usage
2023-12-21 16:13:41 pc-xxxxxx/A.B.C.D:59833 ++ Certificate has EKU (str) TLS Web Client Authentication, expects TLS Web Client Authentication
2023-12-21 16:13:41 pc-xxxxxx/A.B.C.D:59833 VERIFY EKU OK
2023-12-21 16:13:41 pc-xxxxxx/A.B.C.D:59833 VERIFY OK: depth=0, C=, ST=, L=, O=, OU=, CN= CA, name=, emailAddress=
2023-12-21 16:13:41 pc-xxxxxx/A.B.C.D:59833 peer info: IV_VER=2.6.8
2023-12-21 16:13:41 pc-xxxxxx/A.B.C.D:59833 peer info: IV_PLAT=win
2023-12-21 16:13:41 pc-xxxxxx/A.B.C.D:59833 peer info: IV_TCPNL=1
2023-12-21 16:13:41 pc-xxxxxx/A.B.C.D:59833 peer info: IV_MTU=1600
2023-12-21 16:13:41 pc-xxxxxx/A.B.C.D:59833 peer info: IV_NCP=2
2023-12-21 16:13:41 pc-xxxxxx/A.B.C.D:59833 peer info: IV_CIPHERS=AES-256-GCM:AES-128-GCM:CHACHA20-POLY1305
2023-12-21 16:13:41 pc-xxxxxx/A.B.C.D:59833 peer info: IV_PROTO=990
2023-12-21 16:13:41 pc-xxxxxx/A.B.C.D:59833 peer info: IV_LZO_STUB=1
2023-12-21 16:13:41 pc-xxxxxx/A.B.C.D:59833 peer info: IV_COMP_STUB=1
2023-12-21 16:13:41 pc-xxxxxx/A.B.C.D:59833 peer info: IV_COMP_STUBv2=1
2023-12-21 16:13:41 pc-xxxxxx/A.B.C.D:59833 peer info: IV_GUI_VER=OpenVPN_GUI_<IP_ADDRESS>
2023-12-21 16:13:41 pc-xxxxxx/A.B.C.D:59833 peer info: IV_SSO=openurl,webauth,crtext
2023-12-21 16:13:41 pc-xxxxxx/A.B.C.D:59833 TLS: Username/Password authentication deferred for username ''
2023-12-21 16:13:41 MANAGEMENT: CMD 'client-pending-auth 1 4 "WEB_AUTH::https://vpn.example.com/oauth2/start?state=XXXX" 180'
2023-12-21 16:13:41 SENT CONTROL [pc-xxxxxx]: 'AUTH_PENDING,timeout 180' (status=1)
2023-12-21 16:13:41 SENT CONTROL [pc-xxxxxx]: 'INFO_PRE,WEB_AUTH::https://vpn.example.com/oauth2/start?state=XXXX' (status=1)
2023-12-21 16:13:41 pc-xxxxxx/A.B.C.D:59833 Control Channel: TLSv1.3, cipher TLSv1.3 TLS_CHACHA20_POLY1305_SHA256, peer certificate: 2048 bit RSA, signature: RSA-SHA256
2023-12-21 16:13:42 MANAGEMENT: CMD 'client-auth 1 4'

When I restart the openvpn server with a .service file, the line management-hold hangs the service, waiting for the openvpn-auth-oauth2 service to restart. It breaks all kind of automation, not releasing the prompt. Is it the expected behavior ? If yes, how should we use tools like Ansible with that kind of behavior ? I feel like I'm missing a parameter such as "wait in the background" but I'm not finding it !

Thank you !

1

Is there a way to prevent the web-auth durint TLS soft reset ?

Yes. OpenVPN has concept of auth-tokens.
Man page:

--auth-gen-token [lifetime]
After successful user/password authentication, the OpenVPN server will with this option generate a temporary authentication token and push that to client. On the following renegotiations, the OpenVPN client will pass this token instead of the users password. On the server side the server will do the token authentication internally and it will NOT do any additional authentications against configured external user/password authentication mechanisms.The lifetime argument defines how long the generated token is valid. The lifetime is defined in seconds. If lifetime is not set or it is set to 0, the token will never expire.
This feature is useful for environments which is configured to use One Time Passwords (OTP) as part of the user/password authentications and that authentication mechanism does not implement any auth-token support.

The OpenVPN auth token acts like an auth cookie here.
Information: From security point of view, it skips the REAUTH on the SSO Provider (Entra ID). If a user gets deactivated after auth, he is able to continue the VPN.
Problem: Due https://github.com/OpenVPN/openvpn/issues/296, the auth-token. does only work, if the client provides username/password (which is pointless, if openvpn-auth-oauth2 is used). A workaround is that the client sends dummy username/password and the server does not validate the credentials (openvpn-auth-oauth2 will not validate any passwords from clients).
An alternative would be a solution inside openvpn-auth-oauth2. But at the moment, openvpn-auth-oauth2 is stateless. I have no clue from security point if its an good idea to store all the access tokens from the clients. I hope, OpenVPN solves the bug in the next version
2

When I restart the openvpn server with a .service file, the line management-hold hangs the service, waiting for the openvpn-auth-oauth2 service to restart. It breaks all kind of automation, not releasing the prompt. Is it the expected behavior ?

No, its not the expected behavior. Omit management-hold would be the short-term workaround. I hope, that is can be solved with systemd. Maybe BindsTo on the unit section of openvpn-auth-oauth2 solves the problem.

it relaunch the web auth, popping a browser in the face of the user and sometimes during a presentation, or in the wrong browser session, causing all sort of troubles

I agree here, that is something which increases the bad user experience dramatically.
#94 add a non-interactive re-authentication against the SSO Provider using refresh token. On each TLS soft reset, openvpn-auth-oauth2 will ask Entra ID to refresh the session using a refresh token from pervious auth. If the users gets blocked in mean time, session will be not refreshed and the VPN connection will be denied and expired.
The process is documented here: https://github.com/jkroepke/openvpn-auth-oauth2/blob/refresh/docs/Configuration.md#non-interactive-session-refresh could you give me please feedback, if the documentation is clear and understandable?

Seems like Azure token on your site have a nonce field which is not expected here. From my point of view, I do not have this behavior.
Since you redacted the value, this it a known value to you?

After a few weeks, I can confirm this is working properly with CONFIG_OAUTH2_NONCE
Thank you for your fix


MAC: Toggling the sound setting doesn't turn off the sound.

Changing the Settings.Sound property doesn't toggle the actual sound off and on. For some reason, this happens only on the MacOS version and could be related to threading.

Fixed


Production Deploy

The latest dev updates (commit history listed below).
This also includes the fix for the Hotjar script and closes ScPCA-Admin #608 (as chatted on 1:1 today, I created this PR for it, thank you @davidsmejia!)

Closing in favor of a new PR.

🗒 @davidsmejia, once https://github.com/AlexsLemonade/scpca-portal/pull/422#event-10122632877 is successfully deployed and confirmed the Hotjar script fix on client, and we'll close the issue ScPCA-Admin #608, thank you.


The window has crashed

Version: 1.44.2 (system setup)
Commit: ff915844119ce9485abfe8aa9076ec76b5300ddd
Date: 2020-04-16T16:33:57.013Z
Electron: 7.1.11
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT ia32 10.0.17763
Steps to Reproduce:

write code

var buffer = Buffer.alloc(1*1024*1024*50 - 1, "1");
const str = buffer.toString();
console.log(buffer.length);

start debug

show "str"

Does this issue occur when all extensions are disabled?: Yes

Thanks for the issue, can you try out our new debugger and see if it fixes this for you?

new debugger

launch.json:
{
  "version": "0.2.0",
  "configurations": [
    {
      "name": "Mocha",
      "type": "node",
      "request": "launch",
      "program": "${workspaceRoot}/node_modules/mocha/bin/_mocha",
      "stopOnEntry": false,
      "args": [
        "test/signed_data_big.js"
      ],
      "cwd": "${workspaceRoot}",
      "runtimeExecutable": null,
      "runtimeArgs": [
        "--nolazy"
      ],
      "env": {
        "NODE_ENV": "development"
      },
      "sourceMaps": false,
      "outFiles": "${workspaceRoot}/buildjs",
    },
  ]
}

outFiles should be an array rather than a string

Now with debugger work nice. But if change code and alloc ~100mb on start debug зrocess exited with code<PHONE_NUMBER>

This is the process running out of memory when it's inspected. Not ideal, but not avoidable. Node.js doesn't give us a head's up before it runs out of memory, and internally we must enumerate all properties in order to inspect array values.


add tests

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Summary by CodeRabbit

Tests

Introduced new test functions to enhance coverage for the Scheduler's event recording functionality.
Added scenarios to validate behavior during resource binding and cluster resource binding.
Expanded test cases to include new configuration options for the Scheduler's behavior.
Included utility tests to verify string representation of target clusters across various scenarios.

@coderabbitai Review tests


iOS support

Any plans on adding iOS support?
Flutter is a cross-platform dev tool, would only hope it's packages are too 😢

Nvm found this great library that works cross platform https://github.com/justsoft/video_thumbnail/


[OpenAPI 3] Emitter crashes with templated union

Playground
union Response<T> {
  T;
  { hi: "there" };
}

Error seems to be in the compiler where we don't resolve template Parameter in union, error also in json schema emitter playground


Simulator autoconnect

automatically connects to the simulator when in debug mode
also adds a flag to prevent connections

OK the retry stuff looks ok, had one comment about the flag you added. otherwise looks good.

Thanks kyle! Do you think we could group a couple more of these pulls into the next release? eg. https://github.com/mixpanel/mixpanel-iphone/pull/218/files FYI. the build is failing b/c its a non-mixpanel pull.
And im updating the documentation anyway, so ill be happy to add a blurb about this feature in the ios docs. :)

@acanyon yeah, i'll be adding that pr to the release


Add ocid namespace to annotation

Signed-off-by: Mrunal Patel<EMAIL_ADDRESS>

That field isn't even used atm, I guess we can merge this since I will refactor this later in #34,your call to merge

Yeah, can wait.

closing in favor of https://github.com/kubernetes-incubator/ocid/pull/34


fix(.circleci): correct publishing command

Signed-off-by: Leonardo Grasso<EMAIL_ADDRESS>
What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

If contributing rules or changes to rules, please make sure to also uncomment one of the following line:

/kind rule-update

/kind rule-create

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area engine

/area rules

/area tests

/area proposals

What this PR does / why we need it:
Quick hotfix before releasing
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
/milestone 0.23.0
Does this PR introduce a user-facing change?:

NONE

@leogr: The provided milestone is not valid for this repository. Milestones in this repository: [0.24.0, 1.0.0]
Use /milestone clear to clear the milestone.

In response to this:

Signed-off-by: Leonardo Grasso<EMAIL_ADDRESS>
What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

If contributing rules or changes to rules, please make sure to also uncomment one of the following line:

/kind rule-update

/kind rule-create

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area engine

/area rules

/area tests

/area proposals

What this PR does / why we need it:
Quick hotfix before releasing
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
/milestone 0.23.0
Does this PR introduce a user-facing change?:

NONE

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

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

Needs approval from an approver in each of these files:

OWNERS [fntlnz,leodido]

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

LGTM label has been added.  Git tree hash: 77d2bcb0b5a2cc23f44515d9c477fcab1aecd3f5

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

Needs approval from an approver in each of these files:

OWNERS [fntlnz,leodido]

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


Cannot open AssetLib and build templates

Godot version:
Godot 3.2 stable official
OS/device including version:
Manjaro i3
cat /etc/lsb-release:
DISTRIB_ID=ManjaroLinux
DISTRIB_RELEASE=19.0.2
DISTRIB_CODENAME=Kyria
DISTRIB_DESCRIPTION="Manjaro Linux"
Issue description:
Whenever I open the AssetLib tab it does only show "Loading..." and never actually finishes or crashes. When starting Godot from the command line there is no output too.
Steps to reproduce:

Open Godot
Click on the AssetLib tab

Minimal reproduction project:
Does never work even with an empty project.

Just downloaded 3.2.1 on the same system and the problem persists.

The cause could be similar to #36418.

Thanks for the reply.
I did the ping from #36418f and got this result:
ping godotengine.org
PING godotengine.org(web.tuxfamily.net (2a02:2178:1000:201::4)) 56 data bytes
--- godotengine.org ping statistics ---
5 packets transmitted, 0 received, 100% packet loss, time 4062ms

ping -4 godotengine.org
PING godotengine.org (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from web.tuxfamily.net (<IP_ADDRESS>): icmp_seq=1 ttl=48 time=42.4 ms
^C
--- godotengine.org ping statistics ---
1 packets transmitted, 1 received, 0% packet loss, time 0ms
rtt min/avg/max/mdev = 42.413/42.413/42.413/0.000 ms

Manjaro uses piv6 as default so I guess my ipv6 connection is somehow broken too :(

Thanks for confirming this! I'll close this as a duplicate of #36418 then.


Cannot start container with iptables error

Error response from daemon: Cannot start container fa7fb0fdb81cef2d79d36579190f5c4ec1b4e8793a97714ab060934eae519aa6: iptables failed: iptables --wait -t filter -A DOCKER ! -i docker0 -o docker0 -p tcp -d <IP_ADDRESS> --dport 443 -j ACCEPT: iptables: No chain/target/match by that name.
Please help...

I don't think this error is related to this image it's seems to be a docker daemon pb

looks like a docker daemon bug where the rules regarding docker in iptables mysteriously disappear. Restarting the docker daemon will fix the issue as a work-around as the rules will be reapplied in iptables.


Update opentelemetry-exporter-jaeger, ... to 1.25.0

About this PR
📦 Updates

io.opentelemetry:opentelemetry-exporter-jaeger
io.opentelemetry:opentelemetry-exporter-otlp
io.opentelemetry:opentelemetry-sdk

from 1.16.0 to 1.25.0
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.opentelemetry" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "io.opentelemetry" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #283.


Couple of Bug fixes

Some bug fixes and cleanup from previous PRs.
I didn't feel that each of these warranted their own PRs.

utACK

Updated with 2 more bug fixes, also rebased.
bip32utils was removed from the README
find_deices is updated to work with enumerate() needing args.

ACK https://github.com/achow101/HWI/pull/44/commits/4ecf7e2879b2f4b08c628e642b449a1a929f42f3


treat schema parameter as case sensitive

Do all the necessary escaping internally so schema parameter can be treated as case sensitive

The current version does allow you to use case-sensitive keyspace, table, and column names.  For example, you can do
-schema 'myks."MyTable"(col1, "Col2")'
Does that cover the use case you are looking to address?

That gets unnecessarily cumbersome very fast.

The double quotes standard around upper case keyspaces and columns is a cassandra standard. You have to do it in cqlsh, at the driver, etc.
I see your point about it being a bit cumbersome however and am having trouble thinking of downsides of the change. What happens if there's a table called myTable and another table called mytable.

Based on this https://docs.datastax.com/en/cql/3.3/cql/cql_reference/ucase-lcase_r.html I think requiring double quotes to get case sensitivity is necessary (even if a bit cumbersome).

With JSON, this becomes a bigger issue because now we're requiring the column names to come escaped in the JSON file. Nobody will have this. I think it's time to re-evaluate

Here's a new PR to deal with the JSON issue.


5次查看演示地址，4次进不去，可否把修改密码功能禁用？

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

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

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

有人把账号停用了。遇到这个问题，可以添加微信群，及时告知，谢谢！


Scheduler Ips - When upgrading ipsec services, lb instances that get recreated may get bound to all ips because of scheduler not being available for sometime.

Setup - Fresh install of rancher-server v1.6.10 with hosts having scheduler Ips .
Deploy  lb services . Exposed ports get bound to  scheduler Ips as expected.
Upgrade to v1.6.11-rc4.
Upgrade Network-manager.
Upgrade ipsec.
Notice that all Lb services get recreated because of health check failing . Scheduler service contaienrs also get recreated.
Some of the Lb services end up having ports bound to all ip addresses ("<IP_ADDRESS>") . This seems to happen because of scheduler not being available when Lb instances were getting recreated.
This results in same ports on the same host ip being used for different Lb services.
root@ip-172-31-37-102:/home/ubuntu# iptables-save | grep 8080
-A CATTLE_RAW_PREROUTING -d <IP_ADDRESS>/32 ! -i docker0 -p tcp -m tcp --dport 8080 -j MARK --set-xmark 0x1068/0xffffffff
-A CATTLE_RAW_PREROUTING -d <IP_ADDRESS>/32 ! -i docker0 -p tcp -m tcp --dport 8080 -j MARK --set-xmark 0x1068/0xffffffff
-A CATTLE_RAW_PREROUTING ! -i docker0 -p tcp -m tcp --dport 8080 -j MARK --set-xmark 0x1068/0xffffffff

Tested with rancher-server build from v1.6
Steps to validate the issue:
Scenario1:
Have a setup that has hosts with scheduler ips set.
Make the scheduler to go to "unhealthy" state  ( by upgrading service and setting incorrect health check parameters)
Create a service with exposed ports.
Container deployment fails with allocation error:
Failed to allocate instance [container:1i486]: Bad instance [container:1i486] in state [error]: Allocation failed: Cannot perform IP scheduling because external scheduler is unavailable
Make the scheduler to go to "healthy" state.
Container deployment succeeds and the port gets bound to one of the scheduler ips of the hosts as expected.
Scenario2:
Have a setup that has hosts with scheduler ips set.
Create a service with exposed ports and health check enabled.
Make the scheduler to go to "unhealthy" state  ( by upgrading service and setting incorrect health check parameters)
Make service container go unhealthy.
Once the service container gets marked as "unhealthy" and deleted , new container fails to start with error -
 Unhealthy (Failed to allocate instance [container:1i502]: Bad instance [container:1i502] in state [error]: Allocation failed: Cannot perform IP scheduling because external scheduler is unavailable
Make the scheduler to go to "healthy" state.
Container deployment succeeds and the port gets bound to one of the scheduler ips of the hosts as expected.


Uncaught Error: Class 'Aws\CloudFront\CloudFrontClient' not found

I'm hoping this is very much something local I can fix with guidance.
Running the plugin with CloudFront enabled.  Big red X for error, error.log shows the following;
==> error_www.colourw.com.au.log <==
[Mon Jun 11 04:54:53.691309 2018] [:error] [pid 25207] [client <IP_ADDRESS>:1830] PHP Fatal error:  Uncaught Error: Class 'Aws\CloudFront\CloudFrontClient' not found in /var/www/colourw.com.au/www.prod/wp-content/plugins/static-html-output-plugin/library/StaticHtmlOutput.php:905\nStack trace:\n#0 /var/www/colourw.com.au/www.prod/wp-content/plugins/static-html-output-plugin/wp-static-html-output.php(92): StaticHtmlOutput->cloudfront_invalidate_all_items()\n#1 /var/www/colourw.com.au/www.prod/wp-includes/class-wp-hook.php(286): wp_static_html_output_ajax('')\n#2 /var/www/colourw.com.au/www.prod/wp-includes/class-wp-hook.php(310): WP_Hook->apply_filters('', Array)\n#3 /var/www/colourw.com.au/www.prod/wp-includes/plugin.php(453): WP_Hook->do_action(Array)\n#4 /var/www/colourw.com.au/www.prod/wp-admin/admin-ajax.php(99): do_action('wp_ajax_wp_stat...')\n#5 {main}\n  thrown in /var/www/colourw.com.au/www.prod/wp-content/plugins/static-html-output-plugin/library/StaticHtmlOutput.php on line 905, referer: http://wp.colourw.com.au/wp-admin/tools.php?page=wp-static-html-output-options
Has been happening since I installed 2.5, now on 2.6

Thanks for finding this, @alanatgji I haven't tested with CloudFront for a while, I'll check it out on my end. It sounds like a file may have been omitted from including in the plugin. Possibly a quick fix, I'll look into it now.

@alanatgji please try this build. I replaced the aws sdk with the latest version and updated the plugin to use the new syntax required for cloudfront invalidations. Please let me know if that works for you and I'll include in the next release: https://github.com/leonstafford/wordpress-static-html-plugin/blob/master/cloudfront_fix.zip

That seems to have worked.   A few unlink errors left in the error log, of files not cleaning up i assume.   But the cloudfront function went through it seems.

[Mon Jun 11 05:48:33.452331 2018] [:error] [pid 27462] [client <IP_ADDRESS>:2896] PHP Warning:  rename(/var/www/colourw.com.au/www.prod/wp-static/wp-static-html-output-1-1528696103-alanatgji/contents/plugins,/var/www/colourw.com.au/www.prod/wp-static/wp-static-html-output-1-1528696103-alanatgji/contents/lib): No such file or directory in /var/www/colourw.com.au/www.prod/wp-content/plugins/wordpress-static-html-plugin/library/StaticHtmlOutput.php on line 1126, referer: http://wp.colourw.com.au/wp-admin/tools.php?page=wp-static-html-output-options
[Mon Jun 11 05:48:33.452387 2018] [:error] [pid 27462] [client <IP_ADDRESS>:2896] PHP Warning:  unlink(/var/www/colourw.com.au/www.prod/wp-static/wp-static-html-output-1-1528696103-alanatgji/wp-login.html): No such file or directory in /var/www/colourw.com.au/www.prod/wp-content/plugins/wordpress-static-html-plugin/library/StaticHtmlOutput.php on line 1131, referer: http://wp.colourw.com.au/wp-admin/tools.php?page=wp-static-html-output-options
[Mon Jun 11 05:48:33.452399 2018] [:error] [pid 27462] [client <IP_ADDRESS>:2896] PHP Warning:  unlink(/var/www/colourw.com.au/www.prod/wp-static/wp-static-html-output-1-1528696103-alanatgji/wp-login.php): No such file or directory in /var/www/colourw.com.au/www.prod/wp-content/plugins/wordpress-static-html-plugin/library/StaticHtmlOutput.php on line 1132, referer: http://wp.colourw.com.au/wp-admin/tools.php?page=wp-static-html-output-options
[Mon Jun 11 05:49:40.231676 2018] [:error] [pid 27489] [client <IP_ADDRESS>:2922] PHP Warning:  unlink(/var/www/colourw.com.au/www.prod/wp-static/wp-static-html-output-1-1528696103-alanatgji.zip): No such file or directory in /var/www/colourw.com.au/www.prod/wp-content/plugins/wordpress-static-html-plugin/library/StaticHtmlOutput.php on line 1181, referer: http://wp.colourw.com.au/wp-admin/tools.php?page=wp-static-html-output-options

OK, great!
Yes, there is an unlink currently where it tries to delete the zip file even when the user hasn't chosen it as an option.
And likely a rename warning around the plugins dir, which most export will not have, unless there is some publicly viewable file linked from a plugin's dir.
Safe to ignore those for now, but I should keep the error log clean when making a release!

Perfect! 👍
Thanks for your time, greatly appreciated.

My pleasure, Alan. If you wouldn't mind, could you please try this one out, too, it should leave you with a clean error log:
https://github.com/leonstafford/wordpress-static-html-plugin/blob/master/clean_error_log.zip

Tested and works.
CloudFront function runs, error log is clean.

Awesome, closing this now, it will go into the next release. Thanks again for raising the issue and your help in testing!


Disable autologon for empty passwords

See #2474
Thanks for raising this, @igh-gr
Although there is nothing in the specification to prevent automatic logons with empty passwords, this is not a secure default.
The autologon flag INFO_AUTOLOGON ([MS-RDPBCGR] <IP_ADDRESS>.1.1) is now ignored for empty passwords.
Tested with Remmina 1.4.29 which sets INFO_AUTOLOGON if a username is specified without a password.

For V0.9.22 see #2509


Same issue

Delete previous builds, same issue.
What is the most stable build?

Hi @thbestforyourbizdeployment , thanks for the feedback. The most stable would be the latest, however someone a minority of users face new issues that did not occur during testing. I do not recommend installing older version for your safety and security, our team is constantly working on improving the app and solving its issues. We regularly do releases, so if you are not in a big hurry I would suggest staying on the latest version and wait for a fix. If the issue is widespread enough you will likely see a fix in under a week's time. Thank you for your understanding. I've tagged in the the issue relevant to what you're facing.

Hi @thbestforyourbizdeployment , thanks for the feedback. The most stable would be the latest, however someone a minority of users face new issues that did not occur during testing. I do not recommend installing older version for your safety and security, our team is constantly working on improving the app and solving its issues. We regularly do releases, so if you are not in a big hurry I would suggest staying on the latest version and wait for a fix. If the issue is widespread enough you will likely see a fix in under a week's time. Thank you for your understanding. I've tagged in the the issue relevant to what you're facing.


Web API with same route but different versioning- unable to generate multi version swagger document

we adopted versioning approach for our public api and version can be accepted via query string or headers. below are same resource but different store to connect data.
[ApiVersion("1")]
[RoutePrefix("api/items")]
items1controller: apicontroller
{
GET api/items?api-version =1
}
[ApiVersion("2")]
[RoutePrefix("api/items")]
items2controller: apicontroller
{
GET api/items?api-version =2
}
api/items/1?api-version =1
api/items/1?api-version 2
we are unable to generate swagger documentation for multi version api. I think this question was asked many times in different forum. but looking for some solution/direction so that api explorer does not break and we are able to generate swagger document.

Have you looked at #59, #60, and their related links? First-class support for documentation is on the roadmap, but getting out bugs and updates are the highest priority for me. I've almost got all of the other open issues burned down and then I'll have time to focus a lot more effort to formally supporting documentation.  Until then, there have been a lot of discussions, code examples, and community ad hoc implementations.
If there are questions that you can't find answered in any of the questions, then I'll answer what I can for you. Thanks for your patience.

Since there don't seem to be any other questions on this topic and I've already linked the existing, active issues related to this topic, I'm going to close this issue out. I am actively looking into supporting the required documentation features for Swagger, etc. Follow the linked issues for updates. Thanks for your support and interest in the success of the project.

I was got the same bug

Can you elaborate? Are you using Swagger? Have you included the API Explorer package for API Versioning? All of this information is documented in the API Documentation wiki topic. The NuGet packages, samples, and quick starts all have links on the home page as well.
Let me know how else I can help.


Slow Classification

@bamos Hey Brandon, nice project! Our team are really interested and currently we are developing a real time video analyzing tool.
One question i have is that when i try to use the classifier.py to make a prediction, it may take couple of seconds to get the result, and i noticed it is slower than the web demo.
Are those two classification methods different? Is there any way to speed up calling classifier.py, like small photos for example?

Hi @aaaaalucard - thanks! classify.py and the web demo use the same classification methods. I have some timing code in the comparison demo that shows up with the -verbose option. You could add something similar to the classification demo to get a slightly better idea of the bottlenecks. Happy to merge these additions into the master branch if you make them.
My guess is that the web demo doesn't have to re-start when classifying, but every time the classification demo is started, it spends a few seconds loading a large model in a Lua/Torch subprocess. We're interested in moving the subprocess to a server that the Python scripts can communicate with in #4. We probably won't make this change for a few months and are happy for help if you're interested.
The performance differences could also be due to image sizes. And something like #50 might slightly improve the performance for everything.
I'm closing this issue and delegating the improvements to #4 and #50 for now.
-Brandon.

I did as Bamos advised and as he guessed the most of the time (at least on my machine) is due to libraries and Openface models:
./demos/classifier.py --verbose infer ./classifier.pkl image_to_classify.jpg > classifier_output.log
real	0m2.883s
user	0m3.104s
sys	0m0.624s
cat classifier_output.log
Argument parsing and import libraries took 0.496383905411 seconds.
Loading the dlib and OpenFace models took 1.60048508644 seconds.
Original size: (480, 640, 3)
Loading image to classify took 0.00619888305664 seconds.
Find face took 0.425406932831 seconds.
Align image took 0.253262996674 seconds.
Make prediction took 0.000334978103638 seconds.
Predict luca with 0.72 confidence.
How can make you merge the additions i made do classifier.py?

Hi @lucafeudi - thanks! Can you send in a pull request?
See https://help.github.com/articles/using-pull-requests


Provide Dex as a managed app

User story
- As a Giant Swarm customer, I want to have Dex managed by Giant Swarm so that I know it is up-to-date, reliable, and secure.
Description
Right now we offer (dex-app](https://github.com/giantswarm/dex-app) in playground catalog. We want to move it to giantswarm catalog to provide it to our customers (including Giant Swarm) as a managed app.
TODO

[x] https://github.com/giantswarm/giantswarm/issues/18257
[x] https://github.com/giantswarm/giantswarm/issues/17038

[x] scrape Dex
[x] public Grafana dashboard

[x] #629
[x] https://github.com/giantswarm/giantswarm/issues/19195

[ ] #701

[ ] https://github.com/giantswarm/roadmap/issues/547
[ ] Other development / implementation tasks which is left
[ ] Docs
[ ] Communication to other teams/ customers. .etc

Docs which we can add for customers:
providers we support
what we need for configuring provider X
questions to ask to a customer who asks "can I use AAD/GitHub/...?"
How to monitor your SSO success and failures ?
How to troubleshoot when SSO is not working ?
Can I add a fallback OIDC Provider in case the main one is not accessible ?
How do I login when SSO is not working ?


ci(HMS-3056): Testing hms-integration trigger frontend

Waiting for app-interface MR 89367 to get merged.

/retest

/retest


Improve documentation for publishing-api and rummager

This PR adds some extra documentation to the publishing-api and rummager clients and updates the readme.
It doesn't cover all the methods in Publishing API. Where documentation is missing I've added links to the publishing-api documentation in the repo. This may help new starters find their way around GOV.UK a little better.

Thanks for this, @tijmenb!

Something I find useful is to always have an example of the result/output of the method, like Elixir and Python have with DocTest

:+1: Lots of good stuff here. I commented on a few things inline but don't treat these as a blocker for merge.

Updated PR.


[OpenGL] game crashes to desktop

may be related

OPENGL: GL_INVALID_OPERATION error generated. Function glPushAttrib is deprecated and not available in preview contexts.
OPENGL: GL_INVALID_OPERATION error generated. Function glPopAttrib is deprecated and not available in preview contexts.
Mapping string: MissionStartPhase3Ack to index: 2
Game/c4 -> AFXDemoGame -> AFXDemoGame::onClientEnterGame
Game/c4 -> AFXDemoGame -> AFXDemoGame::spawnPlayer
Mapping string: MissionStart to index: 8
*** Initial Control Object
*** Control Object Changed
*** Initial Control Object
Mapping string: /cp/c11Orc Shaman/co to index: 9
Mapping string:  to index: 10
Mapping string: Grolhed to index: 11
Mapping string: Bar Pishnak to index: 12
Mapping string: At gro-Olzog to index: 13
Mapping string: Gridush to index: 14
Mapping string: Etnarud to index: 15
Mapping string: Agycap to index: 16
Mapping string: Nagnus to index: 17
Mapping string: Rool to index: 18
[Material::mapMaterial] - Cannot map unnamed Material
[Material::mapMaterial] - Cannot map unnamed Material

It appears that this is crashing while trying to access the fxaa shaders, disabling antialiasing in the options seems to bypass the crashing.
I do recall that i get a message stating that antialiasing is disabled due to advanced lighting in the generic torque template, need to look into this more

I should note here that while the stack is scrolled all the way down, its because the top 100 lines or so are the same as the top 5 visible here


toString('pgn') produces invalid pgn

Hey there, I found a bug in your lib, at rare occasions the toString('pgn'); produces a invalid pgn by placing the * after the move without a space, could you look into this?

I'm pretty sure it's because of this block of code:
    lineLen += wip.length;
    if (lineLen > MAX_LINE) {
      out += "\n";
      lineLen = wip.length;
    }

Once you check if max line length is reached you put a newline but what if that perfectly cuts off the whitespace and causes the astrix to get placed on the move


Refactor formatting and remove UnitsWidth

I thought I would share this patch, feel free to reject tho, I understand this might be opinionated.
I also played around with UnitsWidth and couldn't make a difference, I fail to see where its trimmed, so effectively has no effect.

This patch unifies and centralises all unit formatting under the formatter.String() umbrella.
It also gets rid of the UnitsWidth attribute, which even if used by format, assigning values to it did not have any real effect on the final output.

Hm.. 'formatter' looks better. But i think we're may lost part of this PR #61

I will take a deeper look later on.

My fault, they were really working.
Now it is good.

Awesome!


Custom tasks

Is it possible to use custom tasks (e.g.: cake, https://github.com/cake-build/cake-vso)?

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

ID: 87686bea-7c46-cd47-d1a9-128e16b63a23
Version Independent ID: ace78317-e562-e35d-781d-23cd7d2dfff2
Content: CI Build in code using YAML
Content Source: docs/pipelines/build/yaml.md
Product: devops
GitHub Login: @andyjlewis
Microsoft Alias: alewis

Yes just install them into the account and reference them as you would any other task.

Yeah, but how?

How do I get a reference of all the tasks proper names and parameters?

Example of using the cake task:
- task: Cake@0
    displayName: Run cake script (Build task)
    inputs:
      script:build.cake
      target: Build

Yeah, so what about every other possible extension I could install?
You guys have to realize that YAML builds are nowhere near approachable for someone who doesn't have an intimate understanding of VSTS internals as well as the naming of all the extensions they install.
There needs to be some kind of centralized reference of all names and extensions available per-pipeline.  Not as inevitably out of date and difficult to discover documentation either.  It has to be highly contextualized within the pipeline itself, as a flyout or modal that people can view at a glance.
Bonus points for a Visual Studio code plugin while editing build YAML.
Many other CI/CD systems do this to show applicable values in-context.

Thanks for your feedback. We're working on a VS Code extension as well as a web editor which will give you contextual, per-task help for YAML definitions. The preview release will have all the in-box tasks, and then we plan to immediately follow it up with a version that logs into your particular Azure DevOps instance to fetch installed tasks. The VS Code plugin will be announced on our DevOps Blog when it's available.

@vtbassmatt - In the interim, do you know how I can use custom pipeline addons from the marketplace in my YAML builds?
Is there any kind of reference or convention I can use to know how to call them?

Is there any kind of reference or convention I can use to know how to call them?
Create a designer build (the UI editor), add the task, and click "view as YAML". It will give you the snippet.

Also, an example of how to provide inputs?
The "view as YAML" link will also help with this.

Also also, how do I know when a plugin will work in Linux vs. Windows?
Good question. My default assumption is that a task is cross-plat unless clearly marked otherwise.

We've asked task authors to start including YAML snippets in their documentation, which should help with all of this.

What if you guys auto-generated a placeholder based on your own internal metadata in the marketplace?

Good idea, thank you. We don't have full auto-generation of docs from our own in-box tasks yet, but we're working on it. I'll see if we can extend the effort to cover third parties.


pkg-config: Use CMake GnuInstallDirs FULL vars

Hello,
This commit uses GNUInstallDirs absolute path variables instead of creating such paths manually.

Use CMAKE_INSTALL_FULL_LIBDIR instead of CMAKE_INSTALL_PREFIX}/CMAKE_INSTALL_LIBDIR
Use CMAKE_INSTALL_FULL_INCLUDEDIR instead of CMAKE_INSTALL_PREFIX}/CMAKE_INSTALL_INCLUDEDIR

This fixes an installation problem in Nix packages, as non-FULL variables are already absolute paths in this case.

Thanks!


add nssc.ac.cn

@lyziew I am sorry, JetBrains does not generally grant free educational licenses to organizations which are not educational, but belong to Academies of sciences. If you are a PhD student there, please apply with official document ("OFFICIAL DOCUMENT" tab at https://www.jetbrains.com/shop/eform/students).

Sorry, I don‘t know why you decide we are not educational
you can see the list in master at swot/lib/domains/cn/ac/
we are all below Chinese Academy of Sciences
our url is http://english.nssc.cas.cn/


Update README.md with star history

@luongngocminh I don't think we need a star history in README, if who care about that, simply goto that site

I saw some project did it, just for showcasing purpose


pref(deps): remove lodash and implement lodash functions used in plugin

fix #8
I removed the Lodash library from the codebase and implemented the reduce, flatMap, uniq, and get functions using pure JavaScript. This helped reduce dependencies, optimize the code, and decrease the bundle size for better performance.

@horuskol
@strothj
please review this pull request

@horuskol @strothj
please review this pull request

Please be patient - it is the weekend and your PR came in at 10pm on a Friday.

@horuskol @strothj
please review this pull request

Please be patient - it is the weekend and your PR came in at 10pm on a Friday.

I apologize if I came across as pushy in my previous message. I assumed that you might not be actively contributing to the plugin due to the time gap since your last contribution

Thanks for your interest and time in putting together the PR. Removing the lodash dependency is something that has been on my list for a while, but I never seem to get the time to do it myself.
If you could resolve the comments I've made on the code changes, I'll see about getting this released very soon.

Thanks for your interest and time in putting together the PR. Removing the lodash dependency is something that has been on my list for a while, but I never seem to get the time to do it myself.
If you could resolve the comments I've made on the code changes, I'll see about getting this released very soon.

I appreciate your quick response and feedback. Thank you again for your time and consideration!

done

@horuskol
hi
please check this MR

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

Comparison is base (084f0a2) 100.00% compared to head (f977006) 96.96%.
 Consider uploading reports for the commit c42314e to get more accurate results

Files
Patch %
Lines

src/util/lodash-fns.js
92.30%
2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##              main       #9      +/-   ##
===========================================
- Coverage   100.00%   96.96%   -3.04%
===========================================
  Files            4        5       +1
  Lines           40       66      +26
  Branches         3       13      +10
===========================================
+ Hits            40       64      +24
- Misses           0        2       +2

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

Hi,
Just reviewed and about to merge in the changes. Thanks for you time and help improving this package.


AfNetworking Domain=NSURLErrorDomain Code=-1004 after few launch app  Ask

The below issue appeared sometimes when I launch the app immediately when the push notification received and if I wait more than 10 seconds after receiving the push then open the push , the request is succeeded and In case of failure the http status code is zero and all requests are https and please note that the request failed after a few seconds and I use the default time out of AFnetworking which is 60 seconds.
Domain=NSURLErrorDomain Code=-1004 "Could not connect to the server." UserInfo={NSUnderlyingError=0x170454940 {Error Domain=kCFErrorDomainCFNetwork Code=-1004 "Could not connect to the server." UserInfo={NSErrorFailingURLStringKey=, NSErrorFailingURLKey=, _kCFStreamErrorCodeKey=61, _kCFStreamErrorDomainKey=1, NSLocalizedDescription=Could not connect to the server.}}, NSErrorFailingURLStringKey=, NSErrorFailingURLKey=, _kCFStreamErrorDomainKey=1, _kCFStreamErrorCodeKey=61, NSLocalizedDescription=Could not connect to the server.}

Any idea?

I have same issue

@barneychen  & @sakshijoshi  The issue is from the server side. Please check the SSL certificate installation .

For general help, please ask on StackOverflow.


Document thought-process for managing dependabot updates

See Anna's guidance in reply to my question:

Do you have any notes on the thought-process you apply when reviewing dependabot updates please?

The text to use is in by @AnnaKornfeldSimpson in https://github.com/obsidian-tasks-group/obsidian-tasks/issues/1080#issuecomment-1232215839
Suggested destination is either in comments in the relevant files, or in CONTRIBUTING.md.

Better still would be to provide a way to auto-merge dependency changes that don't need any testing, so that only the ones that need inspection or testing remain.

Thank you! I can deal with auto-merge down the line, don't worry about that. The important thing is the knowledge!


Incorrect version values from version.Get()

I vendored client-go v0.19.0 in my project.
I just fetch the version and log it on server startup, and I see something like
{"component":"kubevirt-template-validator","level":"info","msg":"kubevirt-template-validator using kubevirt client-go (v0.0.0-master+$Format:%h$ $Format:%H$ 1970-01-01T00:00:00Z)","pos":"app.go:77","timestamp":"2019-07-26T13:21:30.760879Z"}

I was expecting

GitVersion [v]0.19.0
GitCommit
BuildDate a valid date

it seems these fields aren't properly set when the tag is cut.

How did you get that version? I assume with the version pkg? Well, it works as designed: the version package was designed to report the KubeVirt version, and values are set during the build process via ldflags. The values you see are the default values.
So even when you update your build process to set those values, it would report the version of your own component, and not client-go's version. Not sure if we can/want to support the latter. @rmohr wdyt?
You can get KubeVirt's server version if that helps:
virtClient, _ := kubecli.GetKubevirtClient()
info, _ := virtClient.ServerVersion().Get()

Not sure if we can/want to support the latter.

Yep I don't think it makes much sense to support this.
@slintes can we exclude anything related to version from client-go, when we sync with staging?

can we exclude anything related to version from client-go, when we sync with staging?

I think just excluding something won't work. But we could refactor it to only keep the version.Info struct in client-go (it's used in kubecli for getting the server's version), and move everything else back to kubevirt.

@slintes yep, you basically got it. Looks like I misunderstood the purpose of the version package in the client-go module.
I'd like a way to report (= print in the logs) the release of client-go which my binary has compiled in; I can think of few workarounds which don't require any change, so we can close this issue.

if you have a workaround which we could easily add to client-go, let us know :slightly_smiling_face:
/close


📖 Stable is release-0.21.1

Summary
This PR updates deploy-docs.sh to consider release-0.21.1 to be the stable release, instead of 0.3.
BTW, a better fix would take the stable number from the VERSION file.
This PR is based in #1948. We can ether just merge this one, or merge that one first and then I will rebase this one.
Related issue(s)
Fixes #

/lgtm
/approve


Support for the R programming language

Metaflow is currently a Python library. Provide R bindings that would allow a Flow to be written entirely in R and use the Python library as a backend.

Very excited for this feature, both to use in my projects and to learn how Netflix combines R and python!
I am under the impression that this initial work requires porting the internal metaflow codebase to this open source project, but I wanted to ask if I there's any way I can provide assistance toward this feature. 🍻

There is a virtual workshop scheduled for Aug 7th 4PM-7PM UTC for this feature. More details here.

The R bindings are GA now. Documentation is available here - https://docs.metaflow.org/v/r/

This is still listed as future work on this page:
https://docs.metaflow.org/introduction/roadmap


Training epoch loss and validation epoch loss are calculated wrong in Trainer

Description
Since Both loop() for training epoch and validation epoch are assigned as epoch_loss, when training with both training data and validation data, the epoch loss used in callbacks have no meaning.
It should be assigned with training_epoch_loss and validation_epoch_loss separately.
To Reproduce
In gluon-ts/src/gluonts/mx/trainer/_base.py:
In  def loop:
                epoch_loss = loop(
                    epoch_no,
                    train_iter,
                    num_batches_to_use=self.num_batches_per_epoch,
                )

                should_continue = self.callbacks.on_train_epoch_end(
                    epoch_no=epoch_no,
                    epoch_loss=loss_value(epoch_loss),
                    training_network=net,
                    trainer=trainer,
                )

                if is_validation_available:
                    epoch_loss = loop(
                        epoch_no, validation_iter, is_training=False
                    )

                    should_continue = (
                        should_continue
                        and self.callbacks.on_validation_epoch_end(
                            epoch_no=epoch_no,
                            epoch_loss=loss_value(epoch_loss),
                            training_network=net,
                            trainer=trainer,
                        )
                    )

## Error message or code output
no error message, only when using custom callbacks I had some issues due to wrong calculation

put error or undesired output here

## Environment
- Operating system: tried on mac m1 and linux
- Python version: tried with 3.8 and
- GluonTS version: 0.8.1
- MXNet version: 1.6.0

@longdht note that the snippet you posted is not from within the definition of loop.
I do think the code would be less confusing if loop would be defined in at the module’s top level, but I don’t see the bug here: epoch_loss takes a certain value first, which is given to the on_train_epoch_end callback, and then another value, which is passed to the on_validation_epoch_end. That seems ok to me, but maybe I misunderstood your point.

@lostella thank you for your comment,
ah yes its not in the loop, that was my mistake
Sorry this is my first time reporting issues on github so I think I didnt explain well
I was saying that it is better to call it by different name: training_epoch_loss and validation_epoch_loss, otherwise it will confuse people, and also on many callbacks we use epoch_loss, so we should know exactly which epoch_loss we are talking about.
Here is the problem
on the callbacks.on_epoch_end():
when we train without validation data, loss_value(epoch_loss) here is training_epoch_loss, while if we train with validation data, loss_value(epoch_loss) becomes validation_epoch_loss. It should not be confusing like that in my opinion
                should_continue = (
                    should_continue
                    and self.callbacks.on_epoch_end(
                        epoch_no=epoch_no,
                        epoch_loss=loss_value(epoch_loss),
                        training_network=net,
                        trainer=trainer,
                        best_epoch_info=best_epoch_info,
                        ctx=self.ctx,
                    )
                )

I just tried to explain the issue in more detailed, hope I can get some comment from everyone


Blog post

I wrote a yet to be published blog post at https://medium.com/@dabit3/managing-react-state-with-resub-20ea57dee902 and was wondering if someone had some time to review it to make sure I am not misrepresent anything.
Thanks for the library, I'm liking it so far!

Nicely done. Here are a few small comments and suggestions. Feel free to ignore any of these if you think they're off the mark.

I noticed that you are using yarn rather than npm to install components. Do you think that most readers will already have yarn installed and/or know how to install it? Maybe it has become popular enough at this point to make that assumption.
Your store is managing two different pieces of data - a text input and the todo list. This would be a great opportunity to demonstrate how to use ReSub trigger keys. You can create one key (any string is fine) for each data type. Then you can modify the trigger calls to specify only the relevant key, and change the autoSubscribe decorators to subscribe to the relevant key. This won't improve performance in your simple example, but in other cases this mechanism can greatly reduce the number of trigger events, and therefore reduce unnecessary state updates and render operations. Then again, maybe you don't want to go into this more advanced topic in your introductory example. It could be a "part 2" of the blog if you see fit to publish a follow-up.
It's a bit unusual to use a store for a simple text input value. I'd normally just store that in the component state, since its lifetime is limited to that of the component. But for illustration purposes, it's probably fine to keep it in the store. I mention this because some readers might wonder why you're storing it there.
When you're using TypeScript, it's good form to explicitly mark variables and methods as "private" and "protected" where appropriate. For example, "addTodo" and "updateTextField" should probably be private. (We also use the naming convention that all private variables and methods begin with an underscore, but that's probably not a universal convention.)
You're declaring your "styles" array as type "any", which throws away all type information. In general, that's not a good idea because you won't get the benefits of type checking. If you move the definition of the object to the top of the file where you currently declare it, TypeScript will automatically infer the types.

Eric! Thank you so much, this was extremely helpful. I appreciate you taking a look and providing such great feedback, and I've gone ahead and updated the post and code to reflect most of these items.
One thing I'm not sure about is if I have  this wired up right, regarding the second point you made (resub trigger keys):
import { StoreBase, AutoSubscribeStore, autoSubscribeWithKey } from 'resub'

@AutoSubscribeStore
class TodosStore extends StoreBase {
    private _todos: string[] = ['hello','world']
    private _textInputValue: string = ''

    static AddTodoKey = 'ADDTODOKEY'
    static UpdateTextInputKey = 'UPDATETEXTINPUTKEY'

    updateTextInputValue(value: string) {
        this._textInputValue = value;
        this._todos = [...this._todos, 'hellow world adfads']
        this.trigger(TodosStore.UpdateTextInputKey)
    }

    addTodo() {
      if(this._textInputValue === '') return
      this._todos = [...this._todos, this._textInputValue]
      this._textInputValue = ''
      this.trigger(TodosStore.AddTodoKey)
    }

    @autoSubscribeWithKey(TodosStore.AddTodoKey)
    getTodos() {
        return this._todos
    }

    @autoSubscribeWithKey(TodosStore.UpdateTextInputKey)
    getTextInputValue() {
        return this._textInputValue
    }
}

export = new TodosStore()

Is this correct, and also do I need to update my App component at all to work with this?
Thanks.

Likewise, as Eric said it looks good!
A couple additional minor comments and suggestions:
1

We call autoSubscribe to make sure that whenever the data changes...

The typescript concept behind how @autoSubscribe works is called Decorators.  It might be useful to change this to something like the following:

We add the autoSubscribe decorator to make sure that whenever the data changes...

2
Your usage of trigger keys looks good.  We often create a single object that has all the trigger keys.  It only really becomes beneficial to do this when you end up creating manual subscriptions.
export const TriggerKeys = {
    InputTextKey: 'input',
    TodosChangedKey: 'todos'
};

3
Given that there are many different versions of the typescript compiler, you might want to reference a version of the typescript compiler installed in the package scope.
To do this (in npm, I assume yarn is similar), we do the following in package.json.  This allows a developer to run npm run build without having any globally installed packages
{
  "devDependencies": {
    "typescript": "^2.2.2"
  },

  "scripts": {
    "build": "tsc"
  }
}

As Brent said, your use of trigger keys looks correct. There is no need to change anything in your App component.

@berickson1 @erictraut thanks for all of the great feedback!
I've updated the blog post with most of this, and will be publishing on Monday.
If you have any other insight, feel free to share. If not, you're welcome to close this issue.
If I end up doing another post diving deeper into this, I'll be sure to reach back out if that's ok!


adding code link of Medical Sam Adapter

Thank you for your great collection!
I am the author of the paper "Medical SAM Adapter", and our code can actually be found at: https://github.com/KidsWithTokens/Medical-SAM-Adapter
It would be appreciated if you could add our code link, thank you very much.

Hi, @WuJunde Thank you for your interest in my work! You can add your work to the repository and submit a pull request. I will review it and merge it into the main branch. Thank you!

@WuJunde I have updated your code link, thanks.


Update PHONY

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for you're reviewer:
Does this PR introduce a user-facing change?:

/retest

/lgtm
/approve


Placeholder

How can I have dynamic placeholder value?
I would like to get the value from a function or use pipe ( | translate)

Hi @Hesesses,
I tried the following and it was working :
// Template
<ng-select [placeholder]="placeholder">
// Component code
get placeholder() {
return 'your placeholder';
}
Hope it helps,
Guillaume.

works 👍


Backend Framework

Express vs Fastify

This is completed in https://docs.google.com/document/d/1Chivu4jVpx_qLJuKyoYX-M3NSyS_dEXiCynBF5O1rkg/edit


[TOPIC-BLE-LLCP] Bluetooth: controller: allow multiple version requests

At the linked layer level we are only allowed to send one VERSION_IND
per connection, but the host may request version information
multiple times.
We need to ensure that we fill out data correctly in case of
multiple requests, even when we only send one VERSION_IND
over the air. We need to set the response_opcode correctly
since that is used for selecting how to decode/fill in correct data
for the host
Signed-off-by: Andries Kruithof<EMAIL_ADDRESS>

Add unit test for sending 2 VERSION_IND

The unit test that you mention in the comments is missing.

Indeed; I wonder where it went. In any case I'll put it back in

The unit test that you mention in the comments is missing.

I found out; I added the test on the branch for implementing HCI interface. The test is now added here.


Play audio in iOS with Notification

I don't know how to program in objective c, so if you can, you are more then welcome to help me!

im currently waiting for @sonysm to submit the PR regarding IOS implementation


template://experimental/armv7l doesn't boot (No log after Exiting boot services...)

$ uname -srm
Darwin 22.6.0 x86_64

$ sw_vers --productVersion
13.5

$ limactl -v
limactl version 0.16.0-119-g7e99eb6

$ qemu-system-arm -version
QEMU emulator version 8.0.3
Copyright (c) 2003-2022 Fabrice Bellard and the QEMU Project developers

$ limactl start template://experimental/armv7l
? Creating an instance "armv7l" Proceed with the current configuration
INFO[0000] Attempting to download the image              arch=armv7l digest="sha256:f9baf349e9f4bb4acc6cbd14dc82aa2aa514fd76e06c74678e5453f0e6717fd2" location="https://cloud-images.ubuntu.com/releases/22.04/release-20230518/ubuntu-22.04-server-cloudimg-armhf.img"
INFO[0000] Using cache "/Users/suda/Library/Caches/lima/download/by-url-sha256/09e51f47f49935a58ea1139721e702484feed9b97cd0423d4ff66e9178de7aa5/data"
INFO[0001] [hostagent] Starting QEMU (hint: to watch the boot progress, see "/Users/suda/.lima/armv7l/serial*.log")
INFO[0001] SSH Local Port: 56331
INFO[0001] [hostagent] Waiting for the essential requirement 1 of 5: "ssh"
...
^C

$ cat -e ~/.lima/armv7l/serial.log
UEFI firmware (version edk2-stable202302-for-qemu built at 17:13:00 on Mar 15 2023)^M$
Error: Image at 000FFD84000 start failed: Not Found^M$
Error: Image at 000FFCEE000 start failed: Unsupported^M$
Error: Image at 000FFC85000 start failed: Not Found^M$
Tpm2SubmitCommand - Tcg2 - Not Found^M$
Tpm2GetCapabilityPcrs fail!^M$
Tpm2SubmitCommand - Tcg2 - Not Found^M$
Connect: PciRoot(0x0)/Pci(0x4,0x0): Not Found^M$
^[[2J^[[01;01H^[[=3h^[[2J^[[01;01H^[[2J^[[01;01H^[[=3h^[[2J^[[01;01HBdsDxe: failed to load Boot0001 "UEFI QEMU QEMU CD-ROM " from PciRoot(0x0)/Pci(0x1,0x0)/Scsi(0x0,0x0): Not Found^M$
BdsDxe: loading Boot0002 "UEFI Misc Device" from PciRoot(0x0)/Pci(0x7,0x0)^M$
BdsDxe: starting Boot0002 "UEFI Misc Device" from PciRoot(0x0)/Pci(0x7,0x0)^M$
EFI stub: Entering in SVC mode with MMU enabled^M$
EFI stub: Booting Linux Kernel...^M$
EFI stub: Using DTB from configuration table^M$
EFI stub: Exiting boot services...^M$

Tried Ubuntu 22.04, 23.04, 23.10, none of them works

Reported to QEMU: https://gitlab.com/qemu-project/qemu/-/issues/1801

I used /usr/share/AAVMF/AAVMF32_CODE.fd, from qemu-efi-arm

Same results with 23.04, just bigger and slower:
==> /home/anders/.lima/armv7l/serialv.log <==

Ubuntu 23.04 lima-armv7l hvc0

lima-armv7l login:

Sorry, the VM was actually working but the dmesg was not shown, as ttyS0 was not added.
(The default -serial is ttyAMA0)


add StollyStruct ollydbg v1 plugin

This brings support of structures like in ollyv2 but just for v1 :)
https://tuts4you.com/download/1275/

Thank you for your feedback! We've been working on major updates to FLARE VM over the last year.
The now revamped FLARE VM has just been released and will make the project more open and maintainable.
Please check out our blog post at https://www.mandiant.com/resources/blog/flarevm-open-to-public and give the new installation a try.
If this problem still persists with the new installation, please report:

new tools or tool-related issues at https://github.com/mandiant/VM-Packages/issues
ideas and issues related to the installer script and configuration at https://github.com/mandiant/flare-vm/issues

Please note that we use this message to close all legacy issues in this repository. We look forward to your feedback and support for the next generation of FLARE VM.


How to setup swipe to dismiss in one direction?

Love using this! But not sure how to setup a swipe to dismiss in one direction, and a swipe for action in the other direction. anything specific I need to do or this lib not meant for that?

Yea swipe wasn't designed with dismiss animations in mind, but can you try removing your item from its data-set when the dismiss action is performed?

Cool, wasn't sure if there was more to it. Cheers!

Let me know how it goes! I won't mind adding an explicit removal animation in the future if the current experience isn't great.


Feedback: Catalogue of Federal Domestic Assistance and Title

This is the place to leave your feedback and questions about the following data elements:

Catalogue of Federal Domestic Assistance Number
Catalogue of Federal Domestic Assistance Title

The proposed definition of the above elements:
http://fedspendingtransparency.github.io/whitepapers/cfdaprogramnumber-title/
Federal Spending Collaboration home page: http://fedspendingtransparency.github.io/

Given that the Uniform Guidance requires certain levels of payment for indirect cost unless a cap exists in federal statute, it would be EXTREMELY helpful if the CFDA would include whether there is a cap on indirect costs  for each listing and if so what that cap is. As part of this (or through a technical correction  in the Uniform Guidance) the distinction between administrative cost caps and indirect caps needs to be clarified. It seems to be more common to find an administrative cap in statute, but this is often being interpreted and applied as the same as a cap on indirect costs.

As it is outlined in the “white paper”, I agree with the recommended definitions and the impact that the use of current CFDA format and practices will have in the context of the DATA Act and the alignment of financial data between CFDA and USAspending. Since the primary intent of the Data Act is transparency, the federally maintained financial data included in CFDA should also include deobligations like USAspending.
Additionally, if the CDFA number becomes a required data element for recipient subaward reporting, the CFDA number should be prepopulated in the reporting site designated by the federal government. Currently, for FFATA reporting we do not pass the CFDA when reporting on subawards. The CFDA is prepopulated in The Federal Funding Accountability and Transparency Act Subaward reporting System (FSRS) and this practice must continue for future recipient subaward reporting.

AGA established a work group charged with responding to inquiries by the federal government, regarding implementation of the DATA Act. The following response is being submitted on behalf of AGA's work group, which is comprised of state, local, tribal and university practitioners from across the country. In reviewing this data element, group members had the following comments:
The term “area of work” is used in both definitions, but it is not defined. It is not clear what an “area of work” means or how it differs from a program or a cluster of programs.

If CFDA is tracked at the Award Level it may not be necessary to track it at the Sub Award level since it shouldn't really change, it would be redundant reporting.

Update 8/22: Thank you for your feedback on Catalog of Federal Domestic Assistance and Title. We will consider your feedback as we continually assess how we can improve data definitions. You may still contribute the discussion for other data elements.
Check http://fedspendingtransparency.github.io/dataelements/ to find data elements with open feedback pages.


ePSF building and PSF fitting tools

You should be able to merge this yourself. Do you see a merge button?

@mcara Is this PR ready to go?  You should be able to merge it.  Do you want me to merge it?

:clap:


Empowr

EMPOWR ei ole pahavara.!
https://github.com/409H/EtherAddressLookup/issues/105
https://github.com/409H/EtherAddressLookup/issues/10437

Closing. Duplicate of https://github.com/409H/EtherAddressLookup/issues/1038


[vite:asset] Plugin error - Unable to get file name for unknown file

Describe the bug
Error when try to build my project with vite version 2.3.4, work ok with vite 2.3.3
Plugin error - Unable to get file name for unknown file "bd901e3b"

Reproduction
This my private project, sorry I cannot show it
System Info
  System:
    OS: Windows 10 10.0.19041
    CPU: (8) x64 Intel(R) Core(TM) i7-7700 CPU @ 3.60GHz
    Memory: 9.07 GB / 15.92 GB
  Binaries:
    Node: 14.16.0 - C:\Program Files\nodejs\node.EXE
    npm: 6.14.11 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Spartan (44.19041.964.0), Chromium (90.0.818.66)
    Internet Explorer: 11.0.19041.1
  npmPackages:
    @vitejs/plugin-vue: ^1.2.2 => 1.2.2
    vite: ^2.3.4 => 2.3.4

Used package manager: npm
Logs

Still see this issue in vite 2.3.5

After remove inline worker, everything work fine


Post-agg correlated subqueries

Normally, a subquery can refer to a column of the outer query, and this is called a correlated subquery. For example,
create table t (v1 int, v2 int);
select v1 from t where v1 > (select v2);

However, the presence of aggregates can put a limit on which columns are available, even without the complexity of subquery:
select min(v1), max(v2) from t;  -- ok
select min(v1), v2 from t group by v2;  -- ok
select min(v1), v2 from t;  -- error: cannot access v2

select 1 from t having min(v1) > max(v2);  -- ok
select 1 from t group by v2 having min(v1) > v2;  -- ok
select 1 from t having min(v1) > v2;  -- error: cannot access v2

This rule should apply to correlated subqueries as well:
select min(v1), (select max(v2)) from t;
select min(v1), (select v2) from t group by v2;
select min(v1), (select v2) from t;  -- expected error: subquery uses ungrouped column "t.v2" from outer query

select 1 from t having min(v1) > (select max(v2));
select 1 from t group by v2 having min(v1) > (select v2);
select 1 from t having min(v1) > (select v2);  -- expected error: subquery uses ungrouped column "t.v2" from outer query

In the first example, select min(v1), (select max(v2)) from t, although max(v2) appears inside the subquery, it actually participates the aggregate of the outer query, and the subquery itself is actually aggregate-free and should not be limited by the rule above:
create table t2 (v3 int, v4 int);
select
  min(v1),
  (select max(v2) + v3 from t2)  -- access to v3 is ok, no need to wrap inside agg call or group by v3
from t;

We do not have an urgent need to support this, but the binder & planner should report an error rather than generating invalid plans that would cause panic when executed.
Quoted from PostgreSQL:

When an aggregate expression appears in a subquery (see Section 4.2.11 and Section 9.22), the aggregate is normally evaluated over the rows of the subquery. But an exception occurs if the aggregate's arguments (and filter_clause if any) contain only outer-level variables: the aggregate then belongs to the nearest such outer level, and is evaluated over the rows of that query. The aggregate expression as a whole is then an outer reference for the subquery it appears in, and acts as a constant over any one evaluation of that subquery. The restriction about appearing only in the result list or HAVING clause applies with respect to the query level that the aggregate belongs to.

It is still possible to construct a case resulting in a panic. Any idea to ban this case in the binder?
create table t (v1 int, v2 int);
create table t2 (v3 int, v4 int);
select count(*), (select count(*) from (select * from t2 where t2.v3 = t.v2) X) from t group by t.v2;

This is a test case I found in the DuckDB subquery test. This SQL only returns one row in PG, but we return 100000 rows in RW. It appears that our binder and planner handle it differently from PG.
SELECT
(
  select count(*) as value
  FROM (
    SELECT UNNEST(array_agg(distinct id))
  ) a
)
  as total_seats
FROM (
SELECT i id, 1 as val FROM generate_series(0, 100000) i
) as m;


Opening a server generates two sessions (macOS)

Bug description
When I open an SSH session in WebSSH 18.0 beta (build 901), it opens up two sessions to the same SSH server within the same tab.
Screenshots

Steps to reproduce

Double click an SSH session
Two session open up at the same time
Have to eject/close down both sessions to reveal the server list

Good catch @jarrah31 !
Does the double click is a real daily use?
I think the problem was already there but « not visible ».
I’m fixing it anyway

Apologies, it just dawned on me that it's my double-clicking that causes two sessions to appear! I hadn't realised what I was doing - double clicking seemed a natural way to open an ssh session in that moment. I'm now not sure if I was double or single clicking on previous versions... :)
So I guess the fix will be to just open a single session even if double clicking? That works for me. :)
btw, I really like having the bookmark bar (is that the right name for it?) appear on the left hand side - it feels like a more logical way of displaying sessions and navigating WebSSH.
Are there any plans to use a similar bookmark bar on the iPad version to mimic the way safari works on there too?
Also, would it be worth listing saved connections in the left-hand bookmark bar, within Inventory -> SSH (as well as SFTP and Tunnels), that is then organised by the same folder structure that can be expanded and closed?
e.g. if I open a new tab, all my servers are listed in the pain window. It would be useful if that same list of folders and servers were shown in the left-hand bookmarks bar as well.
It feels like a natural place to list saved Connections as there's a lot of available space there. Perhaps the layout and navigation could mimic Safari's bookmarks bar? Maybe even use the same type of icons for showing the tab overview, etc...
Got carried away with myself there - seeing the bookmarks bar in the new version made me think of how Safari works. I don't mind if not though, I just wanted to share my thoughts on that. :)

Apologies, it just dawned on me that it's my double-clicking that causes two sessions to appear! I hadn't realised what I was doing - double clicking seemed a natural way to open an ssh session in that moment. I'm now not sure if I was double or single clicking on previous versions... :) So I guess the fix will be to just open a single session even if double clicking? That works for me. :)

Yes I'm fixing the double click trouble so double click will only result to open only one connection 😉

btw, I really like having the bookmark bar (is that the right name for it?) appear on the left hand side - it feels like a more logical way of displaying sessions and navigating WebSSH. Are there any plans to use a similar bookmark bar on the iPad version to mimic the way safari works on there too?

Apple call it the "sidebar" and in some apps you're right that's the bookmark bar one. On Safari there are bookmarks and also grouped tabs.
Yes I'm also planning to add the sidebar on iOS (iPad and iPhone) but I need to have more feedback like yours on this new implementation on macOS. I have tried to implemented it in the last 60 days and I was wondering if it will be or not a great enhancement for WebSSH' users.

Also, would it be worth listing saved connections in the left-hand bookmark bar, within Inventory -> SSH (as well as SFTP and Tunnels), that is then organised by the same folder structure that can be expanded and closed? e.g. if I open a new tab, all my servers are listed in the main window. It would be useful if that same list of folders and servers were shown in the left-hand bookmarks bar as well. It feels like a natural place to list saved Connections as there's a lot of available space there. Perhaps the layout and navigation could mimic Safari's bookmarks bar? Maybe even use the same type of icons for showing the tab overview, etc...

That's sound great!
My first attempt have used expanded entries like this screenshot :

I have reverted back to an easier implementation because (as sais before) I would like users' feedback to drive my work.
Could you open an issue with all your wishes? It will help me a lot 🙏

Got carried away with myself there - seeing the bookmarks bar in the new version made me think of how Safari works. I don't mind if not though, I just wanted to share my thoughts on that. :)

The sidebar is now used by a lot of apps and WebSSH will use it as much as it can help users in their daily works.
I love your feedbacks and discussions, thank you again @jarrah31

Now available inside WebSSH 18.0 RC2 (macOS)
Have a nice day ☀


hang on MacOS Node16

Verify canary release

[X] I verified that the issue exists in the latest Turborepo canary release.

Link to code that reproduces this issue
https://github.com/repobuddy/repobuddy/pull/152
https://github.com/repobuddy/repobuddy/actions/runs/7603548091
What package manager are you using / does the bug impact?
pnpm
What operating system are you using?
Mac
Which canary version will you have in your reproduction?
1.11.4-canary.1
Describe the Bug
When running the CI on macos-latest with NodeJS 16, it hangs when the script completes:
  Test Suites: 39 passed, 39 total
  Tests:       2 todo, 45 passed, 47 total
  Snapshots:   0 total
  Time:        29.598 s
  Ran all test suites.
// ... hangs ...

This also happens at work in some private repos occasionally (it sometimes go through).
But it seems to occur every time in this CI.
Expected Behavior
Script ends properly.
To Reproduce
Seems to be specific to macos and Node16.
Here is my turbo.json:
{
  "$schema": "https://turborepo.org/schema.json",
  "pipeline": {
    "build": {
      "dependsOn": [
        "^build"
      ],
      "inputs": [
        "src/**",
        "ts/**",
        "package.json",
        "tsconfig.*"
      ],
      "outputs": [
        "cjs/**",
        "esm/**",
        "tslib/**",
        "*.tsbuildinfo"
      ]
    },
    "build:doc": {
      "dependsOn": [
        "^build"
      ]
    },
    "clean": {
      "cache": false
    },
    "coverage": {
      "dependsOn": [
        "@repobuddy/jest#build",
        "build"
      ],
      "inputs": [
        "src/**",
        "ts/**",
        "package.json",
        "tsconfig.*"
      ],
      "outputs": [
        "coverage/**"
      ]
    },
    "depcheck": {
      "dependsOn": [
        "build"
      ]
    },
    "dev": {
      "cache": false
    },
    "lint": {
      "dependsOn": [
        "^build"
      ],
      "inputs": [
        "src/**",
        "ts/**",
        "package.json"
      ]
    },
    "size": {
      "dependsOn": [
        "build"
      ]
    },
    "test": {
      "dependsOn": [
        "@repobuddy/jest#build",
        "build"
      ],
      "inputs": [
        "src/**",
        "ts/**",
        "package.json",
        "tsconfig.*"
      ]
    },
    "test:list": {
      "dependsOn": [
        "build"
      ],
      "inputs": [
        "src/**",
        "ts/**",
        "package.json"
      ]
    }
  }
}

Additional context
https://github.com/repobuddy/repobuddy/actions/runs/7603468790/job/20705243674
Raw Logs:
https://pipelinesghubeus10.actions.githubusercontent.com/qoBhiCmYCrBG87ARc31AlGNdWTiuZARDXdgMr9hyyjJ9wtK7tJ/_apis/pipelines/1/runs/1017/signedlogcontent/8?urlExpires=2024-01-21T20%3A29%3A27.4976374Z&urlSigningMethod=HMACV1&urlSignature=f1T%2F%2B%2FayOIiZgktv714K52G%2FxUPVw2wcqSs8RtWnEqc%3D

Hi, thanks for the issue. I can't seem to reproduce on my local machine, because I get a task failure around coverage ( Module @swc/jest in the transform option was not found. is the error if you have any ideas). In the meanwhile, can you run the turbo command with verbose flags applied? I.e. add -vvv to the turbo command or prefix the command with TURBO_LOG_VERBOSITY=3

Hi, for @swc/jest, it seems like some setting issue. I have added it as a dev deps so that should go away now.
Note that I also can't repro it locally.
But I do experience it from time to time on other private projects on macos and Node16.
This happens to other team members working on those private projects too.
A sad note (or a happy note) is that the latest CI is passing now. So can't repro it atm.

Trying to enable the Mac + Node16 CI again on latest code to see if this can be repro:
https://github.com/repobuddy/repobuddy/pull/157

We support the active and maintenance versions of Node (currently 20 and 18, respectively). If this issue does not reproduce in Node 18, we are unlikely to be able to dedicate any effort to fixing this problem. We may be able to consider a PR, but that is unlikely as well and may take time. I would suggest that you either pin to an older version of turbo or update to a later version of Node.

It also happen on Node 20
https://github.com/repobuddy/repobuddy/actions/runs/8490047264/job/23260750272
Also encountered it locally on Macbook.

It is happening to both Node 18 and Node 20


Update screenshots

Following https://github.com/WebBluetoothCG/ble-test-peripheral-android/pull/83#issuecomment-290571925, here are screenshots with a width of 320px.
R: @g-ortuno

Thanks!

FYI, just finished updating the play store listing.


Fix ingress relation service name bug

Fixes #32.  Also changes the suffix on charm-created objects to make it less confusing which objects are the charm (juju-generated) and which are the workload objects generated by the charm to do the actual work

Closing PR as it does not apply anymore. @ca-scribner please feel free to re-open if you think there's a mistake and this code should be merged.


Adding changelong for v0.5.0-rc2

What type of PR is this?
/kind feature
What this PR does / why we need it:
This will trigger one final release candidate before the v0.5.0 release.
Does this PR introduce a user-facing change?:
NONE

/hold for consensus

/lgtm
/approve

A second RC is a good idea, while I finish cleaning up my two outstanding things, thanks @robscott!. Sorry to be slow.
/lgtm

/lgtm
/approve

/hold cancel


[Doc] Add more examples of adding attributes in Live Components

Q
A

Bug fix?
no

New feature?
no

Issues
-

License
MIT

Note: I tried this in a Symfony app and it didn't work ... so I'm opening this PR to ask you if this should work or if addAttribute() is only for empty attributes that don't define a value.
Thanks!

It only works with attributes in a toggle way .. you may want to use addClass there i think

Thanks for the review. I updated the PR to show an example that won't work and explain that explicitly. Thanks.

I did all the changes that you suggested.

@javiereguiluz, did you miss pushing these changes?

@kbond yes, sorry 😊


Paginated requests don't use max_results value

I'm using the package as follows:
   const result = await roClient.v2.following(twitterId, {
      asPaginator: true,
      max_results: 1000,
    });

    for await (const following of result) {
      followings.push(following);
    }

But I can only fetch 2400 result before reaching the 15 API queries rate limit, which I assume is one query for the first 1000 items and then 14 queries for 100 items (1 * 1000 + 14 * 100 = 2400).
So the issue here is that subsequent retries are not using the max_results parameter. I also tried fetchLast but it seemed to have the same issue.

Related: how do I tell fetchLast to just keep going and get as many results as possible? fetchLast(99999) or is there a cleaner way?

Hello, thank you for the report.
Parameter max_results should stay the same between requests, I'll check why it's not.
For fetchLast method, you could give Infinity as first parameter to trigger endless download (as long as rate limit allows you to pull from Twitter).

Should now be fixed in 1.5.2 just pushed on npm :)

Awesome, this is a huge help! Thanks for fixing it so quickly :)