fix: OPTIC-1120: Draft lead time incorrectly calculated which can cause annotation lead time to be wrong

PR fulfills these requirements

[X] Commit message(s) and PR title follows the format [fix|feat|ci|chore|doc]: TICKET-ID: Short description of change made ex. fix: DEV-XXXX: Removed inconsistent code usage causing intermittent errors
[ ] Tests for the changes have been added/updated (for bug fixes/features)
[ ] Docs have been added/updated (for bug fixes/features)
[X] Best efforts were made to ensure docs/code are concise and coherent (checked for spelling/grammatical errors, commented out code, debug logs etc.)
[X] Self-reviewed and ran all changes on a local instance (for bug fixes/features)

Change has impacts in these area(s)
(check all that apply)

[ ] Product design
[ ] Backend (Database)
[ ] Backend (API)
[X] Frontend

Describe the reason for change
When the current annotation would be a draft, based on the logic previously there it would assign total accumulated leadTime to both the current draft and any annotations. There was a point in time where the draft and annotation would be loaded into state, and the calculation made here would not properly factor in that drafts accumulate and eventually add the current running total of the draft over to the annotation once submitted. Because of this, we would end up doubling the value stored each time a draft would be saved, and once applied to the annotation.
An example I was able to reproduce was as follows:

Open a task in quick view with no current annotations from my user.
Annotate something and allow a draft to save
Let some time pass noting the approximate time elapsed.
Submit the annotation.
Annotate or change something to produce the next draft
Leave the task
Return to the task after some amount of time noting the approximate time elapsed.
Annotate or change something to produce the next draft save
And continue to cycle draft, draft update, annotation submit, leaving and returning to task to simulate normal usage.
Eventually the value of lead time would become extremely large, larger than even the total time from first time the task was opened and the first draft was produced.

The numbers I was able to find just infrequently working through a task with the above steps over the course of 4 hours, was:

Reported LeadTime: 119 hours
Approximate LeadTime: 1.7 hours (this is what I calculated it should have been)

It cannot be possible to have a lead time that is larger than the time from which the task was first taken and the first interaction made.
This is how it should be calculating:
Draft Created      → 0s   DraftLeadTime
Elapsed                → 10s
Draft Save            → 10s DraftLeadTime
Elapsed                → 5s
Draft Save            → 15s DraftLeadTime
Draft Submitted   → 0 + 15 = 15s LeadTime
Draft Created       → 0s   DraftLeadTime
Elapsed                 → 20s
Draft Save             → 20s DraftLeadTime
Elapsed                 → 25s
Draft Save             → 45s DraftLeadTime
Draft Submitted   → 15 + 45 = 60s LeadTime
The draft values should be rolling up to the annotation it would be converted to on submit, not incorrectly adding all previous annotation values on every draft save.
Does this PR introduce a breaking change?
(check only one)

[ ] Yes, and covered entirely by feature flag(s)
[ ] Yes, and covered partially by feature flag(s)
[X] No
[ ] Not sure (briefly explain the situation below)

Which logical domain(s) does this change affect?
(Annotation, Draft, History) Lead Time

/git merge develop


Fix broken codecov reports

The codecov bot has been commenting very strange results for a while now and we were not sure why, example: https://github.com/gchq/Gaffer/pull/2709#issuecomment-1184543574. In this linked PR, one line has changed in a test but the reports shows completely different files changed.
The strange results often ends up in the bot being ignored, so it should be fixed.
I have discovered that the root of the issue is that the ci workflow (which tests the code and uploads coverage) is not run on develop, therefore, there is not an up to date report to compare against.
I think we should update the ci action to:

run on develop and v2-alpha
use codecov/codecov-action@v3

As well as deleting the bugged historical reports.

Alternatively we could look into codacy

As an additional explanation to the above: I think the pattern of effectively having 2 develop branches (develop and v2-alpha) also can break codecov.
For a proper fix, we need to:

merge v2-alpha into develop (release 2.0.0)
make the coverage upload run on develop
delete the bugged historical reports

Since the CI now does run on develop, I think this issue should just track the move to the GitHub App as suggested by @GCHQDeveloper314

Fixed by https://github.com/gchq/Gaffer/pull/2953


AppMetafields API not consistently returning metafield values

@gavinballard flagged this on https://github.com/Shopify/ui-extensions/issues/1151#issuecomment-1652880371

Thanks @sjanczy2 - I have passed along some information via email.

Thanks @sjanczy2 - I have passed along some information via email.

Hi folks, this issue has been a bit tricky but here’s the situation as I understand it.

Yesterday when an automation closed the issue, a fix went out that should be a partial fix. Specifically, any published extensions should have started properly getting back app metafields. However,
This is only true for published extensions, extensions coming from the CLI are still not always getting back the data
As @patryk-smc pointed out, I’m not seeing the app metafields on initial render, but on subsequent renders they are there(I’m basing this off of console logs)

So far, we think we know the cause for 2 and are diving deeper. We’ll continue investigating that one followed by 3.
If you are still seeing app metafields not returned for published extensions(number 1 above) please let us know
I also saw this issue come in recently, if it does not share the same root cause as the current issue we’ll continue the conversation there

I have been developing my extension for few weeks now and app metafields always seem to be empty array on first render. We just started testing extension on production and on one testing device they were consistently empty, while on other they were empty on first render and fetched afterwards. After few minutes first device started working just fine.

If you are still seeing app metafields not returned for published extensions(number 1 above) please let us know

Not confirmed but I highly suspect at least one of my merchants is also seeing this issue. He reported today checkout UI components were not showing on his checkout, despite showing up fine last week. Upon further inspection on my end I found the component showing, but only sometimes.
Network tab shows the app loading fine and console didn't show any related errors. My app depends on AppMetafields for component data, and the past several times it stopped showing it was also related to problems with AppMetafields. If the AppMetafields doesn't load properly then my app gets no data and renders nothing, which seems to match in description for this issue.
With regards to app metafields showing empty array on the first render, I've noticed that as well for months now. It was actually empty for the first several renders, but data always came through eventually in my testing.

If you are still seeing app metafields not returned for published extensions(number 1 above) please let us know

@sjanczy2 Still seeing it.

Seeing this mostly on one-page checkout where extensions don't consistently load. Not reproducing on three-page. Multiple production merchants reported seeing this behavior.

+1 we are observing the same problem on our production store where first there is metafield related content in an array but subsequently stores as an empty array

Apologies for all the trouble with this folks. We've shipped out a fix – please let us know if you encounter any more unexpected behaviour.

Hi @elskhn,
I am trying to develop an app for Shopify. I have scaffolded an extension, and have not changed the code affecting the API:
const api = useApi();

However, this never gives me the shop metafields, either the metafields or the appMetafields section (I am trying on CLI). Anything I can do to stop this?

hey @SuspenseFallback,
make sure you define the metafields in your extension config first. Then, useAppMetafields() should return the shop metafields you are looking for.

Hi @elskhn,
Thanks for the comment. It turned out that useAppMetafields was asynchronous, which is why it didn't work for me.


Deprecate this bucket in favour of Ash258/Scoop-NirSoft

Refs https://github.com/Ash258/Scoop-NirSoft
Once their bucket is out of WIP status, this one should be deprecated. My idea is to add a depreciation notice to the manifests, and keep updating the bucket for a couple more months, perhaps.
Also make it public in lukesampson/scoop#2808

Since the referenced bucket has converted its manifests to YAML files, this is no longer a viable option as scoop doesn't support YAML files, atm.


Multi Line Edit

Issue Type: Feature Request
Azure Data Studio should have multi line edit (Alt + Shift + arrow up/arrow down) in a manner similar to what SSMS has today.
Azure Data Studio version: azuredatastudio 1.2.4 (1ebfd0235690ad044ed2874a02e65188f5fea01c, 2018-11-02T21:50:52.634Z)
OS version: Windows_NT x64 10.0.15063

in ssms, when a cursor moves to a place after the end of a line, it is automatically filled with spaces. Can this behavior also be implemented in azure data studio?


Cant login

Hello, It seems I cant log in with this it throws an error say I don't have an account.

Hi @DuzAwe
You log in using your hub serial number and the password you use with the app.
It might not be the same as the "myenergi account" website where you log in using an email and a password.

That was it, Thanks


transform ts to java

Search Terms

Suggestion

Use Cases
Client and Server communication often need a common message protocal, like protobuf, it can generate message struct types for different languages.  Now I write those message types in typescript first, and like to transform them to another language like Java.  I need typescript to provide language service api like type information, thus I can write code to transform and generate Java code automatically
Examples
message.ts
interface IUserInfo {
    name: string;
    age: number;
    level: 'small' | 'big';
}

message.java

enum Level {
    small,
    big
}

class IUserInfo {
    public String name;
    public double age;
    public Level level;
}

Checklist
My suggestion meets these guidelines:

[ ] This wouldn't be a breaking change in existing TypeScript / JavaScript code
[ ] This wouldn't change the runtime behavior of existing JavaScript code
[ ] This could be implemented without emitting different JS based on the types of the expressions
[ ] This isn't a runtime feature (e.g. new expression-level syntax)

There are other tools to handle this like protocol buffers, Swagger (for REST), and more, but this isn't something the TypeScript team project aims to serve.


Compile failure with nightly rust

I know you don't really bother about nightlies, but as far as I see, current nightly is in sync with master.
rustc --version
rustc 1.0.0-nightly (44a287e6e 2015-01-08 17:03:40 -0800)

/home/kstep/.cargo/registry/src/github.com-1ecc6299db9ec823/mucell-0.1.9/src/lib.rs:90:17: 90:21 warning: the `uint` type is deprecated; use `usize` or a fixed-sized integer
/home/kstep/.cargo/registry/src/github.com-1ecc6299db9ec823/mucell-0.1.9/src/lib.rs:90 const MUTATING: uint = -1;
                                                                                ^~~~
/home/kstep/.cargo/registry/src/github.com-1ecc6299db9ec823/mucell-0.1.9/src/lib.rs:90:17: 90:21 help: add #![feature(int_uint)] to the crate attributes to silence this warning
/home/kstep/.cargo/registry/src/github.com-1ecc6299db9ec823/mucell-0.1.9/src/lib.rs:90 const MUTATING: uint = -1;
                                                                                ^~~~
/home/kstep/.cargo/registry/src/github.com-1ecc6299db9ec823/mucell-0.1.9/src/lib.rs:96:19: 96:23 warning: the `uint` type is deprecated; use `usize` or a fixed-sized integer
/home/kstep/.cargo/registry/src/github.com-1ecc6299db9ec823/mucell-0.1.9/src/lib.rs:96     borrows: Cell<uint>,
                                                                                ^~~~
/home/kstep/.cargo/registry/src/github.com-1ecc6299db9ec823/mucell-0.1.9/src/lib.rs:96:19: 96:23 help: add #![feature(int_uint)] to the crate attributes to silence this warning
/home/kstep/.cargo/registry/src/github.com-1ecc6299db9ec823/mucell-0.1.9/src/lib.rs:96     borrows: Cell<uint>,
                                                                                ^~~~
/home/kstep/.cargo/registry/src/github.com-1ecc6299db9ec823/mucell-0.1.9/src/lib.rs:248:32: 248:45 error: use of undeclared type name `sip::SipState`
/home/kstep/.cargo/registry/src/github.com-1ecc6299db9ec823/mucell-0.1.9/src/lib.rs:248     fn hash(&self, state: &mut sip::SipState) {
                                                                                ^~~~~~~~~~~~~
error: aborting due to previous error

The fatal error is only with SipState which is private now. Maybe a question for @rust-lang core team?

The fix is in PR#11, but breaks tests (I actually done the same changes on my laptop with the same results, so I didn't post a breaking PR after all).

All fixed in 88f59af993205599ec31c9422082a425d1aa28e4.


Misc fixes for Cosmos SDK

This PR

Adds support for urllib3 connection retries
Makes Offer class extend object instead of dict

Can one of the admins verify this patch?

/azp run python - cosmos - tests


Issue #2801: SA1500 fires for the while clause of do/while statement

Provide new configuration setting, layoutRules.allowDoWhileOnClosingBrace such that when enabled, the following will be allowed:
do
{
    Console.WriteLine("test");
} while (false);

See Issue #2801.

@Kevin-Andrew I may still merge this. Let me give it some thought today.

Thanks for looking at this @sharwell. I see that one of the required automation builds is failing.

I've looked at the information/logs in the build pipeline, but I'm not familiar with it enough to tell what the actual problem might be. But if there is anything more I can help with, I'd be happy to do so.

I restarted the build, and it should pass this time. I'm guessing our timeout is too short to handle all cases, but I can increase it.


Add hard-coded greeting fetched from /data

Add a hard-coded greeting from the '/data' URL that appears upon a button click.

Due to conflicts on another branch, I had to remove the greeting object anyway, as it conflicted with the comment section that is now returned by the servlet at /data.

Sorry for the complicated revisions on this branch; I merged the CL #11 before this one, which resulted in merge conflicts that negated the purpose of this CL in the first place.


step by step to run this project?

Hi! The project is not finished yet. I'm still debugging and the runnable version will come up this week. Cheers!


Dashes in folder names cause initialization crash

Describe the bug
If you name a folder with dashes in it, the dash is converted to a dot (.) by the FunctionParser class. This was while testing a reactive function not a restful endpoint.
To Reproduce
Bad structure:

Good Structure:

In order to see the difference in the generated exports, I modified my index.ts file. If you run the firebase emulator and do something to invoke a reactive function, you'll see the difference. Make one function's folder have a dash in the name and another without to compare logs.
// index.ts

import * as admin from 'firebase-admin';
import { FunctionParser } from 'firebase-backend';

admin.initializeApp();

const exportsTmp = new FunctionParser({
  rootPath: __dirname,
  exports: exports,
  verbose: true,
}).exports;

console.log('exportsTmp', exportsTmp);

exports = exportsTmp;

const backendVersion = 'v1';
const separator = '_';

for (const key in exports) {
  if (Object.prototype.hasOwnProperty.call(exports, key)) {
    const newKey = `${backendVersion}${separator}${key}`;
    console.log('key', key);
    console.log('newKey', newKey);
    exports[newKey] = exports[key];
    delete exports[key];
  }
}

Expected behavior
Dashes should be converted to camel-case.
Screenshots
See above.
Additional context
Keep it up @FilledStacks!

Thanks for filing the issue.
I don't know if we should support a structure we don't want to promote. Js uses camel case so I don't think it's important to supply conversions from kebab case. If you want to do it you can submit a PR, I'll happily include it. But it's not something that I'll be adding.


Can't Install xi2ds

I have .NET Runtime and Desktop  .NET Runtime installed, but it still said to download and install it

global.json file:
Not found
Host:
Version:      6.0.10
Architecture: x64
Commit:       5a400c212a
.NET SDKs installed:
No SDKs were found.
.NET runtimes installed:
Microsoft.NETCore.App 6.0.10 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 6.0.10 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Download .NET:
https://aka.ms/dotnet-download
Learn about .NET Runtimes and SDKs:
https://aka.ms/dotnet/runtimes-sdk-info

I had v6 installed and nothing but v5 seems to work, wired


feat(tier4_autoware_api_launch): add some arguments

Description
This PR aims for flexible use cases for tier4_autoware_api_launch by adding arguments on whether to launch some modules.

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@isamu-takagi Could you review this.


Implementation the new Additional Method nisBR

Create the new addtional method nisBR for validation of NIS/PIS (Número de Identificação Social) - (Social Identification Number) for Brazillians
Closes #1879
Checklist for this pull request
Before submitting a pull request, please make sure to follow these rules:

Your code should contain tests relevant for the problem you are solving.
Your commits messages format should follow the jQuery git commit message format (http://contribute.jquery.org/commits-and-pull-requests/#commit-guidelines).
The pull request should reference existing issues or link to a reproducible demo.
Please review the guidelines for contributing (CONTRIBUTING.md) to this repository for more information.

Description
Please describe your pull request.
Thank you!

Will close for now because of inactivity. Feel free to ask further questions/provide furhter information regarding this topic here. We can re-open if required.


Blazor WASM - Initialising client with realtime produces error 'System.Security.Cryptography.Csp is not supported on this platform.'

Good evening,
I'm using Visual Studio 2022 to build a .NET 6 Blazor Web Assembly app. I've used the supabase-csharp nuget package to connect to login and fetch from the database. However, it doesn't appear the real-time connections work due to the following error:
System.Security.Cryptography.Csp is not supported on this platform.
Here's the error full log i get:

It looks like this is a result of an update from Microsoft:
https://docs.microsoft.com/en-us/dotnet/core/compatibility/cryptography/5.0/cryptography-apis-not-supported-on-blazor-webassembly
So I'm not sure if you are going to be able to address this. I'll do some digging but I doubt i'll get anywhere.
Here is the code I'm using to initialise the supabase client which produces the error:
await Supabase.Client.InitializeAsync(
     supabaseUrl,
     supabaseKey,
     new Supabase.SupabaseOptions
     {
         AutoConnectRealtime = true, // Only running with this open executes without error but realtime object is null
         ShouldInitializeRealtime = true, // Produces System.Security.Cryptography.Csp error
     }
 );

All the best,
John Waring.

Thanks for raising the issue John! Unfortunately supporting blazor websockets would require rewriting the realtime-csharp library using a different Websocket Client dependency (This is discussed a bit on #3). I haven't seen enough interest from people to justify rewriting that library, so the current solution is, well, to disable realtime on blazor apps.
If you have an interest in trying a different Websocket library, I'm all ears! But I can't guarantee it will be implemented.

Thanks for the fast reply. No problem it does look like a lot of work. I'll have a look at what may work in the Blazor Web Assembly environment, or see if i can find a resolution for the cryptography dll issue. But I doubt I'll get anywhere.
On a side note, the Blazor Server seems fine initialising the above code, I'll test it further tomorrow to see how functional it is, but not sure the supabase client in blazor server will work for me.
Kind Regards,
John Waring.

I'm still investigating other libraries I could possibly try to replace the 'WebSocketSharp-netstandard' package. When I finish work I'm going to have an attempt to change the Websocket code to use:
https://github.com/Marfusios/websocket-client
I doubt I'll be successful, but it's worth a try.

@d11-jwaring awesome! Keep me in the loop if you hit any brick walls!

Ok, all seems fine, which I'm quite delightfully surprised:

I built a local nuget package of realtime-csharp with me changes, added it to my local nuget source and spun up a Blazor WASM project and added postgrest nuget package and the local realtime-csharp package. And it connected to the docker containers without issue.
Here's the Blazor WASM project and realtime-csharp nuget package to demonstrate the fix:
https://github.com/d11-jwaring/SupabaseRealtimeBlazorWASM/tree/master
Let me know how I can send the changes over so you can review and ensure it's a good solution and won't impact your other users.

Wow! Did that quick @d11-jwaring! Sounds like you were selling yourself short there - haha! If you could fork the repo, commit the changes you've made, and then open a pull request so I can review (and hopefully merge) the changes, that would be fantastic!

I've setup a PR on the realtime-csharp repo:
https://github.com/supabase-community/realtime-csharp/pull/14

@d11-jwaring will be available in<EMAIL_ADDRESS>and<EMAIL_ADDRESS>- thanks for the help!


[NLP] json parsing error - fails to parse valid json

Likely due to trailing comma
json_str = """```json
    {
    "sub_answers": [
        {
        "reasoning_steps": "The post mentions the recommendation of two papers, indicating a positive sentiment towards them.",
        "candidate_tags": {
            "<dg-observation>": "The post is articulating a positive observation about the recommended papers.",
            "<reading>": "The author is likely looking forward to reading the recommended papers in the future.",
            "<missing-ref>": "There is no explicit link provided to the papers mentioned.",
        },
        "final_answer": ["<dg-observation>", "<reading>", "<missing-ref>"]
        }
    ]
    }
    ```"""

@ShaRefOh  this is an issue with the new multi ref tagger, causing it to fail in some cases. Will work on a fix in the next few days, it only happens some times and it doesn't cause a crash - it causes the model to return an empty prediction

I don't think it's related to #79


Replace RESIN_* variables in docs to BALENA_*

List of all instances of RESIN_* variables remaining in the documentation.
$ git grep 'RESIN_HOST_CONFIG'
pages/learn/develop/local-mode.md:- Changes to device [configuration variables][configuration], for example, `RESIN_HOST_CONFIG_gpu_mem`, will result in the device rebooting and applying those settings.
pages/reference/OS/advanced.md:Variables that start with the `BALENA_HOST_CONFIG_` or `RESIN_HOST_CONFIG_` prefix will be added to the `config.txt` file, replacing any preexisting values in the file. For example, a variable named `BALENA_HOST_CONFIG_start_x` with the value of `1` will result in the following entry in `config.txt`:
pages/reference/OS/advanced.md:Custom configuration variables may be added by clicking _Add custom variable_ and entering a name with the prefix of `BALENA_HOST_CONFIG_` or `RESIN_HOST_CONFIG_` as shown below:
pages/reference/OS/advanced.md:To allow setting several values, devices running {{ $names.os.lower }} version >= 2.12.0  (supervisor >= 7.0.0), will parse the values of `RESIN_HOST_CONFIG_dtoverlay` and `RESIN_HOST_CONFIG_dtparam` in a special way where the value of the configuration variable will be treated as the contents of a JSON array (without the enclosing braces `[]`), so a comma-separated list of quote-enclosed values will be split into several lines.
pages/reference/OS/advanced.md:For example, the default value of `RESIN_HOST_CONFIG_dtparam = "i2c_arm=on","spi=on","audio=on"` will translate into the following entries in config.txt:
pages/reference/OS/advanced.md:Another example would be setting several overlays with their own parameters, e.g. `RESIN_HOST_CONFIG_dtoverlay = "i2c-rtc,ds1307","lirc-rpi"` will translate to:
pages/reference/OS/advanced.md:This parsing will only be done if the value is a valid string, so if it doesn't begin with a quote `"`, the value will be parsed as a single string and not split into several lines. For instance `RESIN_HOST_CONFIG_dtoverlay = i2c-rtc,ds1307` will translate to:
shared/general/config-variables-pi.md:RESIN_HOST_CONFIG_disable_splash | 1 | Enable / Disable the Raspberry Pi rainbow splash screen
shared/general/config-variables-pi.md:RESIN_HOST_CONFIG_dtparam | "i2c_arm=on","spi=on","audio=on" | Define DT parameters
shared/general/config-variables-pi.md:RESIN_HOST_CONFIG_gpu_mem | 16 | Define device GPU memory in megabytes

One needs to go in and edit the docs to replace all the RESIN_ variables to BALENA_

One needs to go in and edit the docs to replace all the RESIN_ variables to BALENA_

It seems that some are still RESIN, e.g.
RESIN_HOST_FIREWALL_MODE
RESIN_SUPERVISOR_DELTA

Looks like this will require checking each case in the source.

Yes, I was hoping the contributor could check and notify the respective product owner. So they at least know about it. We can also remove the deprecated variables which is RESIN_SUPERVISOR_DELTA. This way we move closer to replacing RESIN out of the ecosystem with balena.

This way we move closer to replacing RESIN out of the ecosystem with balena.

Ironically, https://github.com/alexandrosm still links to resin.io :wink:  @alexandrosm


unable to parse yaml playbooks - ERROR "Ansible-lint is not available. Kindly check the path or disable validation using ansible-lint"

Summary
Fresh installation of Ansible extension for VSCode 1.1.34 (Windows 10).  Unable to use the ansible extension to review YAML files for consistency and contextual support.
YAML files are associated with the Ansible extension.
Error shown within Vscode output when opening YAML file;
Path for lint:  undefined
[Trace - 11:20:15] Received request 'window/showMessageRequest - (12)'.
Params: {
"type": 1,
"message": "Ansible-lint is not available. Kindly check the path or disable validation using ansible-lint",
"actions": []
}
Extension version
1.1.34
VS Code version
1.74.2
Ansible Version
Not installed locally.

OS / Environment
Windows 10
Relevant log output
[Trace - 11:20:15] Sending notification 'textDocument/didOpen'.
Params: {
    "textDocument": {
        "uri": "file:///c%3A/Users/locadd/GIT/appAutomation/playbooks/playbook.yml",
        "languageId": "ansible",
        "version": 1,
        "text": "---\r\n# copy gitonfig master ansible -m copy -a \"src=../adhoc/.gitconfig dest=~/.gitconfig\" localhost\r\n\r\n- name: ensure gitconfig is applied to device\r\n  hosts: localhost\r\n  tasks:\r\n  - copy: src=../adhoc/.gitconfig dest=~/.gitconfig\r\n#ansible -m homebrew -a \"name=jq state=latest\" localhost\r\n#ansible -m homebrew -a \"name=bat state=latest\" localhost"
    }
}

[Trace - 11:20:15] Sending notification 'update/ansible-metadata'.
Params: [
    [
        "file:///c%3A/Users/locadd/GIT/appAutomation/playbooks/playbook.yml"
    ]
]

[Trace - 11:20:15] Sending request 'textDocument/semanticTokens/full - (9)'.
Params: {
    "textDocument": {
        "uri": "file:///c%3A/Users/locadd/GIT/appAutomation/playbooks/playbook.yml"
    }
}

[Trace - 11:20:15] Received request 'workspace/configuration - (11)'.
Params: {
    "items": [
        {
            "scopeUri": "file:///c%3A/Users/locadd/GIT/appAutomation/playbooks/playbook.yml",
            "section": "ansible"
        }
    ]
}

[Trace - 11:20:15] Sending response 'workspace/configuration - (11)'. Processing request took 1ms
Result: [
    {
        "ansible": {
            "path": "C:\\Users\\locadd\\.vscode\\extensions\\redhat.ansible-1.1.34",
            "useFullyQualifiedCollectionNames": true,
            "reuseTerminal": false
        },
        "python": {
            "interpreterPath": "python3",
            "activationScript": ""
        },
        "ansibleNavigator": {
            "path": "ansible-navigator"
        },
        "completion": {
            "provideRedirectModules": true,
            "provideModuleOptionAliases": true
        },
        "validation": {
            "enabled": true,
            "lint": {
                "enabled": true,
                "path": "ansible-lint",
                "arguments": ""
            }
        },
        "executionEnvironment": {
            "enabled": false,
            "containerEngine": "auto",
            "containerOptions": "",
            "image": "quay.io/ansible/creator-ee:latest",
            "pull": {
                "arguments": "",
                "policy": "missing"
            },
            "volumeMounts": []
        }
    }
]

cmd 'python3 --version' was not executed with the following error: ' Error: spawn C:\WINDOWS\system32\cmd.exe ENOENT
cmd 'ansible-lint --version' was not executed with the following error: ' Error: spawn C:\WINDOWS\system32\cmd.exe ENOENT
[Trace - 11:20:15] Received notification 'update/ansible-metadata'.
Params: [
    [
        {
            "ansible information": {},
            "python information": {},
            "ansible-lint information": {}
        }
    ]
]

Error: spawn C:\WINDOWS\system32\cmd.exe ENOENT
    at ChildProcess._handle.onexit (node:internal/child_process:283:19)
    at onErrorNT (node:internal/child_process:478:16)
    at process.processTicksAndRejections (node:internal/process/task_queues:83:21) {
  errno: -4058,
  code: 'ENOENT',
  syscall: 'spawn C:\\WINDOWS\\system32\\cmd.exe',
  path: 'C:\\WINDOWS\\system32\\cmd.exe',
  spawnargs: [ '/d', '/s', '/c', '"command -v ansible-lint"' ],
  cmd: 'command -v ansible-lint',
  stdout: '',
  stderr: ''
}
[Trace - 11:20:15] Received response 'textDocument/semanticTokens/full - (9)' in 11ms.
Result: {
    "resultId": "1673868015084",
    "data": [
        3,
        2,
        4,
        2,
        0,
        1,
        2,
        5,
        2,
        0,
        1,
        2,
        5,
        2,
        0,
        1,
        4,
        4,
        3,
        1
    ]
}

Error: spawn C:\WINDOWS\system32\cmd.exe ENOENT
    at ChildProcess._handle.onexit (node:internal/child_process:283:19)
    at onErrorNT (node:internal/child_process:478:16)
    at process.processTicksAndRejections (node:internal/process/task_queues:83:21) {
  errno: -4058,
  code: 'ENOENT',
  syscall: 'spawn C:\\WINDOWS\\system32\\cmd.exe',
  path: 'C:\\WINDOWS\\system32\\cmd.exe',
  spawnargs: [ '/d', '/s', '/c', '"whereis ansible-lint"' ],
  cmd: 'whereis ansible-lint',
  stdout: '',
  stderr: ''
}
Path for lint:  undefined
[Trace - 11:20:15] Received request 'window/showMessageRequest - (12)'.
Params: {
    "type": 1,
    "message": "Ansible-lint is not available. Kindly check the path or disable validation using ansible-lint",
    "actions": []
}

[Trace - 11:20:15] Received notification 'window/logMessage'.
Params: {
    "type": 1,
    "message": "An error occurred in 'onDidOpen' handler: [TypeError] Cannot read properties of undefined (reading 'has')\nTypeError: Cannot read properties of undefined (reading 'has')\n    at c:\\Users\\locadd\\.vscode\\extensions\\redhat.ansible-1.1.34\\out\\server\\src\\server.js:2:44852\n    at Generator.next (<anonymous>)\n    at s (c:\\Users\\locadd\\.vscode\\extensions\\redhat.ansible-1.1.34\\out\\server\\src\\server.js:2:42491)\n    at process.processTicksAndRejections (node:internal/process/task_queues:96:5)"
}

[Error - 11:20:15] An error occurred in 'onDidOpen' handler: [TypeError] Cannot read properties of undefined (reading 'has')
TypeError: Cannot read properties of undefined (reading 'has')
    at c:\Users\locadd\.vscode\extensions\redhat.ansible-1.1.34\out\server\src\server.js:2:44852
    at Generator.next (<anonymous>)
    at s (c:\Users\locadd\.vscode\extensions\redhat.ansible-1.1.34\out\server\src\server.js:2:42491)
    at process.processTicksAndRejections (node:internal/process/task_queues:96:5)
[Trace - 11:20:15] Sending response 'window/showMessageRequest - (10)'. Processing request took 189357ms
No result returned.

[Trace - 11:20:15] Sending notification 'update/ansible-metadata'.
Params: [
    [
        "file:///c%3A/Users/locadd/GIT/appAutomation/playbooks/playbook.yml"
    ]
]

cmd 'python3 --version' was not executed with the following error: ' Error: spawn C:\WINDOWS\system32\cmd.exe ENOENT
cmd 'ansible-lint --version' was not executed with the following error: ' Error: spawn C:\WINDOWS\system32\cmd.exe ENOENT
[Trace - 11:20:15] Received notification 'update/ansible-metadata'.
Params: [
    [
        {
            "ansible information": {},
            "python information": {},
            "ansible-lint information": {}
        }
    ]
]

[Trace - 11:23:13] Sending notification 'update/ansible-metadata'.
Params: [
    [
        "file:///c%3A/Users/locadd/GIT/appAutomation/playbooks/playbook.yml"
    ]
]

cmd 'python3 --version' was not executed with the following error: ' Error: spawn C:\WINDOWS\system32\cmd.exe ENOENT
cmd 'ansible-lint --version' was not executed with the following error: ' Error: spawn C:\WINDOWS\system32\cmd.exe ENOENT
[Trace - 11:23:13] Received notification 'update/ansible-metadata'.
Params: [
    [
        {
            "ansible information": {},
            "python information": {},
            "ansible-lint information": {}
        }
    ]
]

[Trace - 11:23:16] Sending notification 'textDocument/didClose'.
Params: {
    "textDocument": {
        "uri": "file:///c%3A/Users/locadd/GIT/appAutomation/playbooks/playbook.yml"
    }
}

Same error message.
OS Version
❯ cat /etc/lsb-release
DISTRIB_ID=ManjaroLinux
DISTRIB_RELEASE=22.0.0
DISTRIB_CODENAME=Sikaris
DISTRIB_DESCRIPTION="Manjaro Linux"

❯ sudo pacman -Q | grep -i kde
kde-cli-tools 5.26.4-1
kde-gtk-config 5.26.4-1
kdeclarative 5.101.0-1
kdeconnect 22.12.0-1
kdecoration 5.26.4-1
kded 5.101.0-1
kdegraphics-mobipocket 22.12.0-1
kdegraphics-thumbnailers 22.12.0-1
kdelibs4support 5.101.0-1
kdenetwork-filesharing 22.12.0-1
kdeplasma-addons 5.26.4-1
kdesu 5.101.0-1
libblockdev 2.28-1
manjaro-kde-settings 20220130-2
polkit-kde-agent 5.26.4-1
qt5-base 5.15.7+kde+r177-1
qt5-declarative 5.15.7+kde+r25-1
qt5-imageformats 5.15.7+kde+r6-2
qt5-location 5.15.7+kde+r3-1
qt5-multimedia 5.15.7+kde+r1-1
qt5-quickcontrols2 5.15.7+kde+r7-1
qt5-speech 5.15.7+kde+r1-1
qt5-svg 5.15.7+kde+r9-1
qt5-tools 5.15.7+kde+r1-2
qt5-wayland 5.15.7+kde+r58-1
qt5-webchannel 5.15.7+kde+r3-1
xdg-desktop-portal-kde 5.26.4-1

VSCode version
1.74.3 x64
Extension version
v1.1.34

I am having the same issue @fluffydice and @Xenion1987.I do hope this issue can be resolved soon

The default settings expect you to have ansible-lint installed while using the extension for a better diagnostics experience. If you do not have ansible-lint installed, either install it (which is highly recommended), else, you can turn off the usage of ansible-lint from the settings (ansible.validation.lint.enabled: false)

Also, looking at the logs, I can see that you are running the extension on Windows. The vscode-ansible extension is supported only on WSL (if you are using Windows).

After installing ansible-lint locally and setting up the path within the extension settings, it worked as expected.
To be honest, I expected the extension to install ansible-lint by itself, just like some other tools install their dependencies by themselves as well.

Since I'm using Manjaro Linux, I installed ansible-lint via sudo pamac -Sy ansible-lint.
Afterwards, ich checked the path where it has been installed to via type -p ansible-lint and edited the extension settings to match the executable path (here as JSON setting): "ansible.ansible.path": "/usr/bin/ansible"
If you are using Windows, you have to use Windows Subsystem for Linux (WSL) and start developing via WSL Extension  and maybe additionally Remote Development. Inside WSL, you can install ansible-lint. This should fix the described behavior.
Further reading: https://code.visualstudio.com/docs/remote/wsl

Thanks will work on it @Xenion1987

Hi @Xenion1987
I was able to solve it using the links and the resources sent.
Thank you.

Mine was a little different.
I was using virtual-environment, where ansible-lint is installed but still I was getting this error.
I had to go to vscode-settings and enable the execution enabled.
This is how it is done:
via settings.json:
"ansible.executionEnvironment.enabled": true,
via settings UI:

@priyamsahoo It would help each one new to this extension, if you and the team could update the README to guide us:
Either
* install ansible-init for Major OS choices(Win, MacOS, apt-based Linux and Yum-based linux)
Or
* disable it from the setting.
Even if I read through the issue, I still am hesitated in choosing which way to go, and it seems like brew install ansible and pointing VS code to the right python runtime does not solve the issue for me.

Where is the setting to set the path for ansible-lint?  I don't see it in the Ansible extension settings.

I found it under the Remote [WSL: Ubuntu] settings.

Also very confused, I am on Mac OS and have done a brew install Ansible-lint, I can run Ansible-lint --version in my terminal but still VsCode says that It can't find the path, I don't understand why? It's there? Can someone pls explain this to me?

@gdmjoho Did you get it to work on your mac? I have macOS Sonoma on apple silicon and can get ansible-lint to work. Installed with brew and on the virtenv using pip but still get error that it's not installed.

Hi, yes I created a .profile in my home folder also a .zprofile file in same directory
In it I put:
eval "$(/opt/homebrew/bin/brew shellenv)"
After that I have ansible-lint in my $PATH:
echo $PATH
/opt/homebrew/bin:/opt/homebrew/sbin:

sudo apt install ansible-lint

In my case, (macOS Sonoma 14.2)
Ansible VS Code Extension v2.9.118 + Homebrew installed ansible-lint v6.22.1 doesn't work.
After installing ansible-lint using pip, it started working properly.
pip install ansible-lint
Detailed version info:
$ ansible-lint --version
ansible-lint 6.22.1 using ansible-core:2.16.2 ansible-compat:4.1.10 ruamel-yaml:0.18.5 ruamel-yaml-clib:0.2.8


Comands have no output

I'm trying to change the resolution using the console but there is no output even when using msys2

https://github.com/Genymobile/scrcpy/issues/385


protein disaggregase activity

ntr: protein-unfoldase activity
definition
An ATP-dependent molecular chaperone activity that mediates the solubilization of protein substrates.
see cdc48 summary
https://www.yeastgenome.org/locus/YDL126C

here is a reference
VCP/p97-Mediated Unfolding as a Principle in Protein Homeostasis and Signaling
PMID: 29153394
Similarly, p97 can harness unfolding to distort the interfaces between client proteins and their binding partners to trigger dissociation.
ts molecular function is ATP-driven protein unfolding, which is directed by ubiquitin and assisted by a host of cofactor proteins. This activity underlies p97’s diverse ability to pull proteins out of membranes, unfold proteins for proteasomal degradation, or segregate proteins from partners for downstream activity.
"p97 Is an Unfolding Machine
The different cellular functions of p97 imply distinct conceptual demands for p97. However, all p97 activities can be explained in principle by an unfolding enzyme that uses pulling force to break up the tertiary and, to a certain extent, also the secondary structure of its client proteins (Prakash and Matouschek, 2004). This mechanism can pull proteins out of membranes during ERAD, but can also be employed for global protein unfolding in preparation for proteasomal degradation. Similarly, p97 can harness unfolding to distort the interfaces between client proteins and their binding partners to trigger dissociation. Over the past years, the underlying molecular mechanism of p97-mediated unfolding has been controversially discussed."

actually, these are different.
disaggregase is soluabalization of an aggregate, this is soulablization of a protein, I will edit

The paper I am approving PMID:35320724 also provides these refs
"Cdc48 has an N-termi-nal domain, two tandem ATPase domains (D1 and D2), and an unstructured C-terminal tail. In the hexameric complex, D1 and D2 form a double-ring structure with a central pore through which substrates can be threaded, which unfolds the polypep- tide (Banerjee et al., 2016; Bodnar and Rapoport, 2017; Cooney et al., 2019; Ji et al., 2021; Twomey et al., 2019)"

Hi @ValWood
Does this new term I created last week work ?
protein-containing complex destabilizing activity, see https://github.com/geneontology/go-ontology/issues/23065#issuecomment-1076352973
I can add synonyms.

It is slightly different. Although "destabilizing activity" is another function of cdc48 (it specifically extracts ubiquitinated substrates from binding partners) before it unfolds them.
The authors of this paper call this activity "a segregase activity"
Central to the function of Cdc48 is its role as a segregase. Cdc48 separates ubiquitinated targets from their non-modified binding partners or from cellular structures (Stolz et al., 2011; Ye et al., 2017). For example, p97Ufd1/Npl4 extracts ubiquitinated Aurora B kinase from chromatin at mitotic exit to promote chro- matin decondensation and nuclear envelope reformation (Ramadan et al., 2007), and yeast Cdc48 mobilizes the transcrip- tion factors Mga2 and Spt23 from the endoplasmic reticulum membrane (Rape et al., 2001; Shcherbik and Haines, 2007).
I would therefore add
"ubiquitnated protein segregate" as a narrow synonym of  +name: protein-containing complex destabilizing activity
The aforementioned "unfoldase" is a separate activity that occurs after "extraction" by threading the protein through a central pore.
I guess these two activities could be combined in some way, but only if all unfoldases are destabilizers and vice versa.
One is defined as destabilizing a complex (extraction), the second as destabilizing a individual  protein (unfolding), once it is extracted.
It seems to be an important distinction because the extraction may not result in degradation (especially in the context of actin etc) where the extracted protens are recycled, but not unfolded.

In fact, I had created protein disaggregase activity here: #19173
I now changed the label to 'ATP-dependent protein disaggregase activity', and added 'protein unfoldase activity' to 'ATP-dependent protein disaggregase activity' and to 'protein-containing complex destabilizing activity'.
Does this address the issue?
Thanks, Pascale

remember that the 'protein-containing complex destabilizing activity'  was to be a parent for the severing  (If I remember correctly, this isn't in QuickGO yet), and this isn't 'unfolding' the  subunits don't unfold, the polymers are severed into smaller parrtts. So  'protein unfoldase activity' does not work as a synonym for this term.

I think this is done - see comment above
https://github.com/geneontology/go-ontology/issues/23088#issuecomment-1082712628


Incomplete Type definitons for the QRCode component.

I am using the  QRCode component in a typescript project and I have noticed, that the type definitions for the components properties is incomplete. As seen in the JSX code of the component you pass down all properties, so basically all properties that the default HTML elements have can also be given to the QRCode component. However the type definition is far more limited and that for f.e. the id prop can't be given to the component without typescript linting it as an error.

@trm217 can this issue be closed?

The problem it's self is solved by my pr. However I wonder if the JS prop-types declaration might ha e to be extended too?
Also the docs could be updated to mention that all other arguments are passed to the SVG element. If you think this would be a good idea, I'm willing to create a second pr later today that adds this 👍🏻

@trm217 I guess there is no need to change prop types, I think it is just fine.
If you are willing to create another PR with improved docs, I will be happy to marge it for sure! 😃


[Issue 905][api][pulsar-client] add serializable token class

Motivation
When trying to authenticate Pulsar in distributed systems (a very good example of this is the Spark connector, see here: https://pulsar.apache.org/docs/en/adaptors-spark/), the following exception is thrown (this is taken from the log of one of the Spark executors):
21/05/13 11:59:34 WARN PulsarClientImpl: [topic: persistent://tenant/namespace/topic] Could not get connection while getPartitionedTopicMetadata -- Will try again in 380 ms
21/05/13 11:59:34 INFO ConnectionPool: [[id: 0x4d13ed61, L:/<IP_ADDRESS>:43624 - R:broker.svc.cluster.local/<IP_ADDRESS>:6650]] Connected to server
21/05/13 11:59:34 WARN ClientCnx: [broker.svc.cluster.local/<IP_ADDRESS>:6650] Got exception java.lang.RuntimeException: failed to get client token
	at org.apache.pulsar.client.impl.auth.AuthenticationDataToken.getToken(AuthenticationDataToken.java:62)
	at org.apache.pulsar.client.impl.auth.AuthenticationDataToken.getCommandData(AuthenticationDataToken.java:55)
	at org.apache.pulsar.client.api.AuthenticationDataProvider.authenticate(AuthenticationDataProvider.java:133)
	at org.apache.pulsar.client.impl.ClientCnx.newConnectCommand(ClientCnx.java:218)
	at org.apache.pulsar.client.impl.ClientCnx.channelActive(ClientCnx.java:199)
	at org.apache.pulsar.shade.io.netty.channel.AbstractChannelHandlerContext.invokeChannelActive(AbstractChannelHandlerContext.java:230)
	at org.apache.pulsar.shade.io.netty.channel.AbstractChannelHandlerContext.invokeChannelActive(AbstractChannelHandlerContext.java:216)
	at org.apache.pulsar.shade.io.netty.channel.AbstractChannelHandlerContext.fireChannelActive(AbstractChannelHandlerContext.java:209)
	at org.apache.pulsar.shade.io.netty.channel.DefaultChannelPipeline$HeadContext.channelActive(DefaultChannelPipeline.java:1398)
	at org.apache.pulsar.shade.io.netty.channel.AbstractChannelHandlerContext.invokeChannelActive(AbstractChannelHandlerContext.java:230)
	at org.apache.pulsar.shade.io.netty.channel.AbstractChannelHandlerContext.invokeChannelActive(AbstractChannelHandlerContext.java:216)
	at org.apache.pulsar.shade.io.netty.channel.DefaultChannelPipeline.fireChannelActive(DefaultChannelPipeline.java:895)
	at org.apache.pulsar.shade.io.netty.channel.epoll.AbstractEpollChannel$AbstractEpollUnsafe.fulfillConnectPromise(AbstractEpollChannel.java:620)
	at org.apache.pulsar.shade.io.netty.channel.epoll.AbstractEpollChannel$AbstractEpollUnsafe.finishConnect(AbstractEpollChannel.java:653)
	at org.apache.pulsar.shade.io.netty.channel.epoll.AbstractEpollChannel$AbstractEpollUnsafe.epollOutReady(AbstractEpollChannel.java:529)
	at org.apache.pulsar.shade.io.netty.channel.epoll.EpollEventLoop.processReady(EpollEventLoop.java:465)
	at org.apache.pulsar.shade.io.netty.channel.epoll.EpollEventLoop.run(EpollEventLoop.java:378)
	at org.apache.pulsar.shade.io.netty.util.concurrent.SingleThreadEventExecutor$4.run(SingleThreadEventExecutor.java:989)
	at org.apache.pulsar.shade.io.netty.util.internal.ThreadExecutorMap$2.run(ThreadExecutorMap.java:74)
	at org.apache.pulsar.shade.io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30)
	at java.lang.Thread.run(Thread.java:748)
Caused by: java.lang.NullPointerException
	at org.apache.pulsar.client.impl.auth.AuthenticationDataToken.getToken(AuthenticationDataToken.java:60)
	... 20 more

This happens since the token supplier function become null when it gets transferred to eg. a Spark executor. Refer to this issue (which is roughly the same): https://github.com/streamnative/pulsar/issues/905 .
Modifications
Added a new token storage class, SerializableAuthenticationToken, which can be serialized properly (this is tested in its corresponding unit test as well). On the other hand, this class does not use supplier functions to fetch token data, it stores the token as a plain string instead. For use-cases requiring a token supplier, this cannot be used, but can load its configuration from file (using the file:// URL prefix), like the original AuthenticationToken class.
Corresponding unit test were added to the class. Will also add proper documentation prior to merging once the change can be accepted - if that is required.
Verifying this change

[ ] Make sure that the change passes the CI checks.

This change added tests and can be verified as follows:

added unit tests for the new class to ensure that it supports most functionality as the original AuthenticationToken
added an unit test to ensure that the token inside the class remains intact when going through serialization

Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): no
The public API: yes (the Java API allows to use SerializableAuthenticationToken upon connecting as well, apart from the existing authentication methods)
The schema: no
The default values of configurations: no
The wire protocol: no, but if this feature is used, JWT tokens in plain text format will be transferred throughout the network
The rest endpoints: no
The admin cli options: no
Anything that affects deployment: don't know

Documentation

Does this pull request introduce a new feature? yes (in the Java Client API)
If yes, how is the feature documented? brief JavaDocs, will extend public docs as well if needed

thank you @atezs82
We will review this patch
@lhotari PTAL

@atezs82 I'd rather not introduce a new auth provider here duplicating most of the code. Is there a way that this can be fixed/enhanced within the context of the existing AuthenticationProviderToken?

@merlimat thanks for the review, I'll try to modify this accordingly.

@merlimat I pushed a slightly different version without new classes (just modified AuthenticationToken). Let me know what do you think.
Note that in theory it is still possible to provide a not-serializable token supplier, which would still give back that cryptic NullPointerException above.

@atezs82 Change LGTM, just left few minor suggestions.

@merlimat Thanks, I'll fix these and push the changes.

@atezs82 There seems to be an issue reported by spotbugs tool:
Error:  Medium: The field org.apache.pulsar.client.impl.auth.AuthenticationDataTls.certStreamProvider is transient but isn't set by deserialization [org.apache.pulsar.client.impl.auth.AuthenticationDataTls] In AuthenticationDataTls.java SE_TRANSIENT_FIELD_NOT_RESTORED
2473
Error:  Medium: The field org.apache.pulsar.client.impl.auth.AuthenticationDataTls.keyStreamProvider is transient but isn't set by deserialization [org.apache.pulsar.client.impl.auth.AuthenticationDataTls] In AuthenticationDataTls.java SE_TRANSIENT_FIELD_NOT_RESTORED
2474
Error:  Medium: The field org.apache.pulsar.client.impl.auth.AuthenticationDataTls.trustStoreStreamProvider is transient but isn't set by deserialization [org.apache.pulsar.client.impl.auth.AuthenticationDataTls] In AuthenticationDataTls.java SE_TRANSIENT_FIELD_NOT_RESTORED

@merlimat Please excuse my limited knowledge on the pulsar-client. On the other hand, it seems that you have handled this Findbugs issue in the AuthenticationTls class as well (line 49):
@SuppressFBWarnings(value = "SE_BAD_FIELD", justification = "Using custom serializer which Findbugs can't detect")
private Supplier<ByteArrayInputStream> certStreamProvider, keyStreamProvider, trustStoreStreamProvider;

Based on this, I also added a similar suppress annotation to AuthenticationDataTls (I am not exactly sure why this was missing from there though).
Let me know if this approach shall be OK or shall I modify the custom deserializer logic instead. Thanks!

Regarding the CI failure it seems that a unit test for BROKER_GROUP_1 was failing (org.apache.pulsar.broker.loadbalance.LoadBalancerTest.testBrokerRanking). I tried to execute the whole test suite locally by running mvn clean install && ./build/run_unit_group.sh BROKER_GROUP_1, but eventually got a lot of failing tests due to various reasons (Tests run: 1168, Failures: 818, Errors: 0, Skipped: 345, I think most of the skips were happening because of the failures).
On the other hand, I managed to execute just the problematic unit test locally by running mvn clean install && mvn -B -ntp -pl pulsar-broker -Dtest=LoadBalancerTest (just copied the command form the .sh script). That ran without any problems:
[INFO] -------------------------------------------------------
[INFO]  T E S T S
[INFO] -------------------------------------------------------
[INFO] Running org.apache.pulsar.broker.loadbalance.LoadBalancerTest
[INFO] Tests run: 7, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 87.75 s - in org.apache.pulsar.broker.loadbalance.LoadBalancerTest
[INFO]
[INFO] Results:
[INFO]
[INFO] Tests run: 7, Failures: 0, Errors: 0, Skipped: 0
[INFO]

Not exactly sure on how to proceed with this. Shall these test be executed locally, or do they need some kind of CI environment to run? Given that I could execute this locally, I would re-run the workflow on the CI but I think I lack privileges for doing so.


localhost:3000/api ?

I've noticed when I run my nuxt app on local, there is a console log mentioning /api.
Could you provide some information on what that does, and how to use it.

Hi. If you are using axios module this log indicates that default baseurl for requests is /api implementation or proxy passing is up to you! :)


Case insensitive user pools?

Which Category is your question related to?
Auth
Amplify CLI Version
14.13.4
What AWS Services are you utilizing?
Cognito
Per this article, newly created user pools are created to have case-insensitive usernames. Manually created pools have this enabled by default.
However, pools created through amplify push do NOT have case insensitive pools. How do I enable case insensitivity for new or existing pools (i expect the latter is not possible)?

@claym We don't have this feature available on CLI but it's scheduled for release. For the time being you can enable it in the Cloudformation template more information can be found at https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-cognito-userpool.html under UsernameConfiguration

      UsernameConfiguration:
        CaseSensitive: false

That worked. Thanks!

I received the error "Updates are not allowed for property - UsernameConfiguration" when perform amplify push after adding this to my .yml file manually.

It's not really feasible for me to remove auth and recreate it. Is there a fix for https://github.com/aws-amplify/amplify-cli/issues/3494#issuecomment-598194358 that doesn't require that?

Hi there guys, any update on this? The User Pool create via Amplify CLI is yet with the "Enable case insensitivity for username input" unchecked and disabled to be marked, any ideas?

Please help (any kind soul out there)!
I tried the above solution on my "test" env. (Also have dev+prod).

I removed Auth -> userPoolGroups (I still have another Auth (labelled cadcognito))
Tried the changes (UsernameConfiguration: CaseSensitive: false). It did not work. (This is not my concern)
Tried to revert back by:

a) first adding userPoolGroups back and when that didn't work
b) pull my dev env and push that into my test cloudformation.

"Amplify status" now says it wants to Create userPoolGroups , but when I try I get the following error:
UPDATE_FAILED               authcadcognito          AWS::CloudFormation::Stack Thu Aug 06 2020 11:15:26 GMT+0200 (GMT+02:00) Parameters: [allowUnauthenticatedIdentities, identityPoolName, thirdPartyAuth] do not exist in the template

I have checked both in console and locally and allowUnauthenticatedIdentities, identityPoolName, thirdPartyAuth all exist in the template.
I´d like to save my test env if possible but not required, but more alarmingly, amplify now wants to make updates to both my dev and prod envs... so I am afraid that when I do my next push to dev or prod (which are working ok now), amplify will insert the same error that now is crashing test.... :/
I want things back they were before trying this :(
Is there a way to either:

Pull my production env, and add+push to a new "test2" environment, and will this remove any requests from amplify to want to commit changes to my prod env in the future?
If I create "test2", is there an easy way to transfer my dynamodb-data from test to test2?
Would another attempt to remove BOTH  userPoolGroups AND cadcognito and repush possibly work? I am a bit afraid of "update api" at this point, since I have a running prod with clients :/ (Is there a way to print the current api settings?)

I´ve spent 1+ week on this and still feel lost

I'm getting the same 400 error when adding the UsernameConfiguration
Resource Name: us-east-1_24x1azsMU (AWS::Cognito::UserPool)
Event Type: update
Reason: Updates are not allowed for property - UsernameConfiguration. (Service: AWSCognitoIdentityProvider; Status Code: 400; Error Code: InvalidParameterException; Request ID: null; Proxy: null)

I just ran into this issue and for me the problem was you cant update this setting after the pool has been created.
From docs: "This configuration is immutable once it has been set."
https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-cognito-userpool.html#cfn-cognito-userpool-usernameconfiguration
Once I deleted my cloudformation stack and deployed the stack again it all worked.
Hope this helps

I removed Auth, then added it back.
I am using React JS.  The backend file is a JSON and not a YML.  After I edit the JSON and add:
"UsernameConfiguration": {
"caseSensitive": false
}
After I save the file and issue an "amplify push", I notice that the file reverts back to what is pulled from the backend.  I tried to update the YML file in S3 but that didn't seem to work.

Can you tell me when you plan to add the option for case sensitive to the CLI?
The workaround mentioned adding it manually here doesn't seem to work.

Hey folks :wave: this feature has since been released and can be toggled behind the enablecaseinsensitivity feature flag. Closing 🙂


oxidize: Expand the functionality of the "auth" command group

I would love to see the following functionality:

Being able to filter scopes and roles.
Being able to grant permanent scopes to an user or role.
Being able to list your own scopes and (optionally) scopes from roles.
Being able to list someone else's scopes and (optionally?) scopes from roles.

One idea to implement this could be named arguments?
E.g.

!auth scopes --filter "song" - List your own scopes, filtering by song.
!auth scopes setbac --filter "song" - List scopes of setbac, filtering by song.
!auth roles setbac - List roles of setbac.

P.S. The principal argument behaviour should probably be documented, @ targets a role, anything else is an user.

Thanks!
I like the idea of named arguments to expand on functionality. Could hook it up to an existing library, like https://docs.rs/structopt/0.3.15/structopt/ to make it more convenient.


fix(kernel_crawler): fixed ubuntu targets: actually use real ubuntu flavor as target

What type of PR is this?

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

/kind bug

/kind cleanup

/kind documentation

/kind feature

Any specific area of the project related to this PR?

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

/area crawler

/area ci

/area utils

What this PR does / why we need it:
This PR is part of a wider effort to support any ubuntu flavor to build Falco drivers; see:

https://github.com/falcosecurity/driverkit/pull/201
https://github.com/falcosecurity/falco/pull/2178

Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Note that kernel-crawler output wants to be as agnostic as possible from driverkit/test-infra
internal representation/implementation.
Therefore, we should adhere to correct ubuntu flavor as a target.
It is up to the consumers to fix data for their usage.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP
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 [FedeDP]

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, leogr
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 [FedeDP,leogr]

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:<PHONE_NUMBER>a76bc3f6730a2b2012c967a366dc10


issue 428: date-time validator rejects value with Z suffix

Description of defect at https://github.com/apigee-127/swagger-tools/issues/428

Do you mind running npm test and appending the commit with the browser binaries built in browser?  I think this should regenerate those.  Other than that, great job.

Updated with browser-compatible packages, @whitlockjc. Mind merging?

I think the try/catch was there on purpose to make the API return false whenever it failed to parse the offset hours/minutes.  I think we should keep that as-is to avoid any breakages.  Thoughts?

I couldn't figure out what value would cause Number to throw. At worst it should return NaN.

The value in matches[6] and matches[7] will be undefined or a string. Acc. 7.1.3 of ECMA-262, the former will always produce NaN. The rules for the latter are more complicated, but it won't throw. At worst, it should be NaN, and that will tickle the return on line 125.

Thanks for looking into this.  :)


Make the data available

Bootstrapping for yourself takes two days, so maybe I should make the data available to everyone. My SQLite3 file is 1.5 GB (485 MB gzipped), but that's including indexes etc.

Put it on box: https://nyu.app.box.com/file/377542343766

Put new data on OSF: https://osf.io/7g2y8/

Moved to Backblaze, OSF has a ~5GB limit: https://f004.backblazeb2.com/file/rr4-files/filesdb.sqlite3.zst


Update helm for rc2

Codecov Report
Patch coverage has no change and project coverage change: +53.69% :tada:

Comparison is base (a0503c2) 26.41% compared to head (c21a18d) 80.10%.
Report is 6 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1732       +/-   ##
===========================================
+ Coverage   26.41%   80.10%   +53.69%
===========================================
  Files          67        9       -58
  Lines        4527      548     -3979
  Branches     2317      246     -2071
===========================================
- Hits         1196      439      -757
+ Misses       3331      109     -3222

Flag
Coverage Δ

config-deployer-service
?

idp-domain-service
80.10% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 76 files with indirect coverage changes

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


cg - Implelement ZipCodeController Tests

In this PR we implement a joke controller using the API from https://api.zippopotam.us/
Closes #17

I like the joke controller you have, the API call looks good and your link for the request also looks good


Add new trading rules to InferTrade that use the technical analysis rules with conventional recipe

Currently we have rules that calculate a technical analysis rules and then determine positions by linear regression.
The class used to calculate the allocation from the signal is here.
Many technical trading rules do not use a regression to calculate their sizing, but use entry and exit rules based on a formula.
We therefore want to create trading rules that replicate the conventional trading approach along the current option of sizing through regression.
To do so we need three things:

[ ] Choice and description of standardise trading approaches. (Cal)
[ ] Updated trading rule functions that calculate the allocations. (@bi-kash )
[ ] Documentation (LaTeX/PDF) for these new trading rules. (@fhashim )

The new Python rules should be in here.
The tex files and PDFs should be here.
Images that may be needed are here

@fhashim - unfortunately we can't link issues unless they fully address an issue (as issue will close prematurely when PR is merged).

Here is the sample of one of the most basic use tech indicators for traders.
The condition/parameter is to give a buy signal when the price is above ema50 and vice versa give a sell signal when the price is below the ema50. (we can adjust the ema length later depend on user).
We're trying to test it with @bi-kash  and once ready for testing we can add more complicated ones, tech indicators that are used in traditional ways by traders.

DECRIPTION : EXPONENTIAL MOVING AVERAGE (50)
CONDITION/PARAMETER : GIVE A BUY SIGNAL WHEN THE ASSET PRICE CLOSED ABOVE THE EMA50 VALUE AND VICE VERSA GIVE A SELL SIGNAL WHEN THE ASSET PRICE CLOSED BELOW OR LOWER THAN EMA50 VALUE

LINK FOR FURTHER DETAILED EXPLANATION OF EXPONENTIAL MOVING AVERAGE:
https://www.investopedia.com/terms/e/ema.asp

Hi sir @ta-oliver , for your comments regarding this format of recording here in gihub. Thanks

Let's move each trading rule addition to its own GitHub issue? You can use "Reference in new issue" to do this.

noted sir. thanks


Update network-utilization-planning.md

Re-order the "control sync throughput" section to put OneDrive GPO first and recommended, since Windows QoS method is outdated.

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@kaarins PR has been copyedited and is ready for final review, could you please check and merge? If you would like a final copyedit prior to merging, let us know. Thanks!

@kaarins PR has been copyedited and is ready for final review, could you please check and merge? If you would like a final copyedit prior to merging, let us know. Thanks!


update pls :D

You can fork and develop :D

You can fork and develop :D

im not good at py

Im into C++ these days. Don't like python anymore.

Im into C++ these days. Don't like python anymore.

true, i dont like python, but i can re make this tool in c++

This kind of tools are easier with scripting languages since they are IO bound not CPU bound. Performance is not an issue. I've used selenium webdriver to control the page. There isn't any good webdriver is C++. (since no one moves to C++ for this kind of work)

If you like chess and C++, checkout this repo. Its straightforward and easy to get started. There are lots of todos.
https://github.com/vaezim/Shohih

If you like chess and C++, checkout this repo. Its straightforward and easy to get started. There are lots of todos.
https://github.com/vaezim/Shohih

well thx, but i use more c#


Space missing after image

It's minor, but it bothered me.

Hi @edmacauley, would you review the proposed changes? Thanks.

Hi @dangowans thanks for that eagle eye and fixing the space. #sign-off


[QST] abp_pcap_detection

I was wondering if the abp_pcap_detection example was runnable?
When I attempt to run it it crashes in the data prep stage.
I am also curious as to the expected output, what would pred indicate in this case?
This is the "stack trace" from the crash I am seeing.
  /opt/conda/envs/morpheus/lib/python3.8/site-packages/cupy/_creation/from_data.py(76): asarray
  /opt/conda/envs/morpheus/lib/python3.8/site-packages/cudf/core/dataframe.py(372): _setitem_tuple_arg
  /opt/conda/envs/morpheus/lib/python3.8/contextlib.py(75): inner
  /opt/conda/envs/morpheus/lib/python3.8/site-packages/cudf/core/dataframe.py(143): __setitem__
  /opt/conda/envs/morpheus/lib/python3.8/site-packages/morpheus/messages/multi_message.py(152): set_meta
  /workspace/examples/abp_pcap_detection/abp_pcap_preprocessing.py(164): pre_process_batch

WARNING: Logging before InitGoogleLogging() is written to STDERR
E20220926 18:22:10.815363   397 context.cpp:125] main/preprocess-anomaly-2; rank: 0; size: 1; tid:<PHONE_NUMBER>52224: set_exception issued; issuing kill to current runnable. Exception msg: ValueError: Unsupported dtype <U36

@chris-nardone Pipeline on a release container from image morpheus:v22.09.00b-runtime, executed. This is the log.
(morpheus) root@dgx04:/workspace/examples/abp_pcap_detection# morpheus --version
morpheus, version 22.9.0b0+14.g3a2d9c0

(morpheus) root@dgx04:/workspace/examples/abp_pcap_detection# python run.py \
> --input_file ../data/abp_pcap_dump.jsonlines \
> --output_file ./pcap_out.jsonlines \
> --model_name 'abp-pcap-xgb' \
> --server_url localhost:8001
====Registering Pipeline====
====Building Pipeline====
====Building Segment: linear_segment_0====
Preprocessing rate: 0 messages [00:00, ? messages/s]====Building Segment Complete!====
Preprocessing rate: 0 messages [00:00, ? messages/s]====Building Pipeline Complete!====
                                                  ====Registering Pipeline Complete!====
====Starting Pipeline====:00, ? inf/s]
====Pipeline Started==== 0 add-class [00:00, ? add-class/s]
Added source: <from-file-0; FileSourceStage(filename=../data/abp_pcap_dump.jsonlines, iterative=False, file_type=FileTypes.Auto, repeat=1, filter_null=False, cudf_kwargs=None)>
  └─> morpheus.MessageMetaile [00:00, ? to-file/s]
Added stage: <deserialize-1; DeserializeStage()>
  └─ morpheus.MessageMeta -> morpheus.MultiMessage
Added stage: <preprocess-anomaly-2; AbpPcapPreprocessingStage()>
  └─ morpheus.MultiMessage -> morpheus.MultiInferenceFILMessage
Added stage: <monitor-3; MonitorStage(description=Preprocessing rate, smoothing=0.05, unit=messages, delayed_start=False, determine_count_fn=None)>
  └─ morpheus.MultiInferenceFILMessage -> morpheus.MultiInferenceFILMessage
Added stage: <inference-4; TritonInferenceStage(model_name=abp-pcap-xgb, server_url=localhost:8001, force_convert_inputs=True, use_shared_memory=False)>
  └─ morpheus.MultiInferenceFILMessage -> morpheus.MultiResponseProbsMessage
Added stage: <monitor-5; MonitorStage(description=Inference rate, smoothing=0.05, unit=inf, delayed_start=False, determine_count_fn=None)>
  └─ morpheus.MultiResponseProbsMessage -> morpheus.MultiResponseProbsMessage
Added stage: <add-class-6; AddClassificationsStage(threshold=0.5, labels=['probs'], prefix=)>
  └─ morpheus.MultiResponseProbsMessage -> morpheus.MultiResponseProbsMessage
Added stage: <monitor-7; MonitorStage(description=Add classification rate, smoothing=0.05, unit=add-class, delayed_start=False, determine_count_fn=None)>
  └─ morpheus.MultiResponseProbsMessage -> morpheus.MultiResponseProbsMessage
Added stage: <serialize-8; SerializeStage(include=[], exclude=['^ID$', '^_ts_'], fixed_columns=True)>
  └─ morpheus.MultiResponseProbsMessage -> morpheus.MessageMeta
Added stage: <monitor-9; MonitorStage(description=Serialize rate, smoothing=0.05, unit=ser, delayed_start=False, determine_count_fn=None)>
  └─ morpheus.MessageMeta -> morpheus.MessageMeta
Added stage: <to-file-10; WriteToFileStage(filename=./pcap_out.jsonlines, overwrite=True, file_type=FileTypes.Auto, include_index_col=True)>
  └─ morpheus.MessageMeta -> morpheus.MessageMeta
Added stage: <monitor-11; MonitorStage(description=Write to file rate, smoothing=0.05, unit=to-file, delayed_start=False, determine_count_fn=None)>
  └─ morpheus.MessageMeta -> morpheus.MessageMeta
Preprocessing rate[Complete]: 537241 messages [00:06, 85728.27 messages/s]
Inference rate[Complete]: 537241 inf [00:14, 38340.77 inf/s]
Add classification rate[Complete]: 537241 add-class [00:15, 35521.08 add-class/s]
Serialize rate[Complete]: 537241 ser [00:15, 35513.64 ser/s]
Write to file rate[Complete]: 537241 to-file [00:20, 26557.39 to-file/s]
====Pipeline Complete====

@chris-nardone The upstream module (cuDF) changes are the causing this issue. This issue has been fixed in version 22.09.
https://github.com/nv-morpheus/Morpheus/blob/bc16f7968b9f7413e10e9d4b090224a604274d32/examples/abp_pcap_detection/abp_pcap_preprocessing.py#L177

I do not seem to have permission to fetch any version other than 22.06
root@ip-172-31-16-179:/home/ubuntu# helm fetch https://helm.ngc.nvidia.com/nvidia/morpheus/charts/morpheus-sdk-client-22.09.tgz --username='$oauthtoken' --password=$API_KEY --untar
Error: failed to fetch https://helm.ngc.nvidia.com/nvidia/morpheus/charts/morpheus-sdk-client-22.09.tgz : 404 Not Found
root@ip-172-31-16-179:/home/ubuntu# helm fetch https://helm.ngc.nvidia.com/nvidia/morpheus/charts/morpheus-sdk-client-22.07.tgz --username='$oauthtoken' --password=$API_KEY --untar
Error: failed to fetch https://helm.ngc.nvidia.com/nvidia/morpheus/charts/morpheus-sdk-client-22.07.tgz : 404 Not Found
root@ip-172-31-16-179:/home/ubuntu# helm fetch https://helm.ngc.nvidia.com/nvidia/morpheus/charts/morpheus-sdk-client-22.06.tgz --username='$oauthtoken' --password=$API_KEY --untar
root@ip-172-31-16-179:/home/ubuntu#

@chris-nardone apologies on the delay here, are you still having problems with this?

Honestly gave up on this.
We had a meeting and discovered that the abp_pcap_detection module was
intended to detect bitcoin mining and that was not what we were looking for.
Thank you for getting back to me
On Mon, Feb 27, 2023 at 4:26 PM Ben Jarmak @.***> wrote:

@chris-nardone https://github.com/chris-nardone apologies on the delay
here, are you still having problems with this?
—
Reply to this email directly, view it on GitHub
https://github.com/nv-morpheus/Morpheus/issues/376#issuecomment-1447116374,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHAKTJ6DZ6ZSL42UJFXNDODWZULZJANCNFSM6AAAAAAQWBVMWY
.
You are receiving this because you were mentioned.Message ID:
@.***>

Understood - if you're looking for any new features, we're always happy to get new feature requests!


update set_reader to return type bool

This PR fixes an inconsistency with the type annotations and return types of the set_reader method in both goldenverba/verba_manager.py and goldenverba/components/managers.py

Hey @jwnz we just released the newest v2 version 😄 We fixed the inconsistency in the newest release, but thanks a lot for the effort


fasclick and swipe elements

Hi,
I'm working for digital advertising company and our partners using fasclick on their web site. And we faced an issue that because of fastclick swipe and other rich media creatives are not working - click triggered instead of swipe. needsclick is also on working as expected. Is it possible at all to skip some elements from fastclick handling, or the only way is to detach fasclick from body and attach to everything, but some elements
Example:  http://euroman.dev.workingpropeople.com/

I’m not sure, but for me "to detach fasclick from body and attach to everything, but some elements" is the only way

Yes that obvious solution, but requires more work to be done. Anyways thank you for reply.

@dkot might selective fastclick help you?


WIP: chore(litmus-portal): Auth integration

What this PR does / why we need it:
Auth Integration with frontend
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Checklist

[ ] Does this PR have a corresponding GitHub issue?
[ ] Have you included relevant README for the chaoslib/experiment with details?
[ ] Have you added debug messages where necessary?
[ ] Have you added task comments where necessary?
[ ] Have you tested the changes for possible failure conditions?
[ ] Have you provided the positive & negative test logs for the litmusbook execution?
[ ] Does the litmusbook ensure idempotency of cluster state?, i.e., is cluster restored to original state?
[ ] Have you used non-shell/command modules for Kubernetes tasks?
[ ] Have you (jinja) templatized custom scripts that is run by the litmusbook, if any?
[ ] Have you (jinja) templatized Kubernetes deployment manifests used by the litmusbook, if any?
[ ] Have you reused/created util functions instead of repeating tasks in the litmusbook?
[ ] Do the artifacts follow the appropriate directory structure?
[ ] Have you isolated storage (eg: OpenEBS) specific implementations, checks?
[ ] Have you isolated platform (eg: baremetal kubeadm/openshift/aws/gcloud) specific implementations, checks?
[ ] Are the ansible facts well defined? Is the scope explicitly set for playbook & included utils?
[ ] Have you ensured minimum/careful usage of shell utilities (awk, grep, sed, cut, xargs etc.,)?
[ ] Can the limtusbook be executed both from within & outside a container (configurable paths, no hardcode)?
[ ] Can you suggest the minimal resource requirements for the litmusbook execution?
[ ] Does the litmusbook job artifact carry comments/default options/range for the ENV tunables?
[ ] Has the litmusbooks been linted?

Special notes for your reviewer:

/preview

/preview

/preview

/preview 60

/preview

/preview 30

/preview

/preview 60

/preview

/preview

/preview 60

/preview 60

/preview 60


Repo cleanup...

I see some files like pycache that should probably be cleaned up.  Also make sure your .gitignore is setup properly.

@dacb just fixed that

What do you mean about the .gitignore tho?

If the __pycache__ file is committed it might mean your .gitignore file is not complete enough to avoid this.  It could also be that the file was created before you current .gitignore was in place and was just left over.

Oh, I assumed that the __pycache__ was allowed by your current .gitignore.  Maybe add __pycache__/* to your git ignore if it isn't already there to prevent it from coming back in the future.

Done!


Runelite plugins not working due to update

Describe the bug
Runelite plugins stopped working because of an updating in OSRS
To Reproduce
Open Runelite
Additional context
Update Blog Post

MacOS 12.0.1

Why is this marked as invalid?

Read the box, and wait for the update

ok, thanks!


Add bodvcd & bodvrd

Hi @bermejus, thanks for this huge work! I'll review this PR today.
I would have preferred a PR on the dev branch (as mentioned in the repo) but I'll manage it.
Just one thing that caught my attention was that in the latest commit you changed the cspice-sys lib. Can you give me more detail to understand?


ENH Add beta_fragility_heuristic and gpd_risk_estimates functions

Hi,
I thought it'd be interesting to add some risk metrics to empyrical for use in zipline/pyfolio.
Here's the papers I used.

Beta Fragility: https://www.imf.org/external/pubs/ft/wp/2012/wp12216.pdf

GPD Risk Estimates: https://link.springer.com/article/10.1007/s10614-006-9025-7

I wasn't sure about whether or not docstrings and tests are required for subroutines. (I'm missing some docstrings and tests for subroutines of gpd_risk_estimates.)
If the above are required, please feel free to ping me back and I'll work on adding them.
Thanks!
Jeyan

Ah looks like I forgot to check flake.
Will push a patchset.

Thanks @jeyoor, this looks great. I'll try to take a look next week (traveling).

Sorry @jeyoor, I completely dropped the ball on this. This looks great, if you solve the merge conflicts we can go ahead and merge this.

Closing this as it's old. Feel free to reopen if this is still relevant.

@gusgordon ,
Thank you for the reminder on this!
I'm a bit slammed at the moment, but once I get some free time I plan to update the branch per the comments and ping you back.
Thanks again!
Jey

Moved to new pull request here https://github.com/quantopian/empyrical/pull/118


Steam now has different steamSecureLogin for its 6 domains

Only one pair of web cookies is returns in getWebCookies
https://github.com/DoctorMcKay/node-steam-session/blob/master/src/LoginSession.ts#L796
Starting yesterday Steam uses different steamLoginSecure cookie for its different domains (steamcommunity, store, help, checkout, etc.) while getWebCookies returns only one. Thus, it cannot be used for requests to steam store, help and other domains.

7 hours ago, in 1.7.2 arleady updated

npm update


A little problem when  click and press A to add a file

my steps to add a file to a folder:

click a folder
press 'a'
enter file name and press Enter to save

The little problem is in step 1, when a folder is clicked, it is toggled (fold/unfold) automatically, this is not what I expected.
Thanks

Same! I would also prefer if it only toggled when clicking the little arrow 👍

Thanks for contributing!
We noticed that this looks like a duplicate of https://github.com/atom/tree-view/issues/1222 so you can subscribe there if you'd like.
As mentioned in that issue, you may find https://atom.io/packages/dbclick-tree-view helpful.

Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


Add feature weights support

This PR adds support for feature_weights parameter, which is xgb.DMatrix parameter since Xgboost version 1.3.0. This PR does not add this support for RayDeviceQuantileDMatrix class.

@Yard1 I'm not sure how to proceed with the latest pytest failure. May be this is a temporary issue on Azure repo side? The error message says Unable to fetch some archives, maybe run apt-get update or try with --fix-missing?, but I am not sure how to do this. Could you please advise? Thanks!

@mozjay0619 I just reran the job and it passed that point. It looks like a random failure not related to your changes.

hmm @mozjay0619 we seem to have some actual errors, can you check?

@Yard1 Sorry, let me explain the nature of the error since we've had this multiple times. The feature_weights leverages the colsample feature, so it is inherently stochastic. The weights are configured to be monotonically increasing with the last (9th) feature having the biggest contribution. But due to the stochasticity, the second to last feature sometimes overpowers the last feature. I increased the boosting rounds for convergence and lowered the colsample rates as well. And this time, I tested it against 100 random seed values. I can take more drastic measures but I don't want to alter the code too much from the original demo code provided by xgboost official doc...

@mozjay0619 can we also set the seed as a parameter for xgboost? The numpy seed will not be carried over to the xgboost-ray Actors carrying out the training.

@Yard1 Yeah, I had entertained that exact idea. But I couldn't find an easy way to do it since the xgboost.train() method does not have seed parameter, and it relies on numpy random state. I suspect this is why the demo code sets numpy random state. One hacky way might be to create a seed parameter to train method in xgboost_ray and use that number to set the local seed for each Actor? But I'm not sure what this would actually do in the context of distributed xgboost training...

I believe it has a seed learning parameter but if the test passes here then I am fine with that. Thanks for iterating on this!


[SPARK-36808][BUILD][3.0] Upgrade Kafka to 2.8.1

What changes were proposed in this pull request?

This PR backports SPARK-36808 (#34050) to branch-3.2, upgrading Kafka from 2.4.1 to 2.8.1.
Why are the changes needed?

Kafka 2.8.1 was released a few hours ago, which includes a bunch of bug fix.
https://downloads.apache.org/kafka/2.8.1/RELEASE_NOTES.html
As we discussed, we think it would be good to backport this change.
https://issues.apache.org/jira/browse/SPARK-36808?focusedCommentId=17492248&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-17492248
Does this PR introduce any user-facing change?

No.
How was this patch tested?

CIs.

Sorry, but we cannot upgrade like this.

@dongjoon-hyun O.K, I'll close this. Thank you.


expose all deployment's ingresses to the container

It'd be cool to have all the expose.accept HTTP URI's specified by the user in SDL file available in the container via the environment variable.
This way the apps/scripts running within the containers will be aware of all the externally exposed uri's.
As well as having the provider's ingress address exposed as an environment variable will be useful when exposing a service over non-80/443 port, i.e. K8s's nodePort --service-node-port-range (default: 30000-32767).

This makes a degree of sense and shouldn't be too difficult. We'd need to setup a standard for how it is done, but I guess we can just base it off that strange (albeit functional) way that Kubernetes maps internal services between containers to environmental variables.


fix(check-for-changes): Sort by semver and filter tags

What does this PR do?
Re-apply code of #28344 with a modification that validates we only consider release-related tags

Motivation
Fix the create_rc_pr workflow, failing when we have more than 10 release candidates

Additional Notes
cc @Pythyu

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


HDDS-6603. Dynamically refresh debug operations for audit log

What changes were proposed in this pull request?
In HDDS-6562, we introduced the feature to exclude specific operations for AuditLog.
But the usage in PROD cluster is not convenient, we need to update the config and restart service to enable the new configuration.
This ticket is to add a new servlet to refresh the configuration for AuditLog.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-6603
How was this patch tested?
unit test.

@symious Can you add a testcase enabling debug operations dynamically? Correct me if I am wrong I dont see this in this PR.

@dineshchitlangia Thank you for the advice, the reload feature works fine for our cases, I will close this ticket.


feature: export settings to file

Hi
I don't know where you want to store all player's settings when they are made, but I think maybe a good idea is to store them in a XML file or so.
You could then update the player (overwrite executable) and all settings are still in the XML file.
I just hate most players are having default settings when you install an update.
Since the player is cross platform, maybe there is no other choice then a settings file?
I think it's not good to make a QT specific file because the settings should work in another GUI too.
What do you think?

Yes that is the plan (by default Qt will store the settings in your home directory so they will persist between different updates) right now now some temporary settings is saved in the root directory (like playlist and song database) but the plan is to not have them there.


Add ioctl FS_IOC_{G,S}ETVERSION and FS_IOC_{G,S}ETFLAGS

This PR focus on adding support for the constants behind the call of ioctl(FS_IOC_G/SETFLAGS) and ioctl(FS_IOC_G/SETVERSION)
Ressources:
Linux: https://elixir.bootlin.com/linux/latest/source/include/uapi/linux/fs.h#L220
Android: https://cs.android.com/android/platform/superproject/+/master:prebuilts/vndk/v31/arm/include/bionic/libc/kernel/uapi/linux/fs.h;l=155?q=FS_APPEND_FL&ss=android%2Fplatform%2Fsuperproject
Android (bis): https://cs.android.com/android/platform/superproject/+/master:bionic/libc/kernel/uapi/linux/fs.h;l=161;drc=4794e479f4b485be2680e83993e3cf93f0f42d03?q=FS_APPEND_FL&sq=&ss=android%2Fplatform%2Fsuperproject
Note: Thoses calls are a bit of a mess historically, however the constants did not change since

Seems the values are different on i686-unknown-linux-gnu.

Seems the values are different on i686-unknown-linux-gnu.

Indeed, I fail to expand the macros, I did so for all the supported arch now

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

Could you also fix the merge conflict and squash commits into one?

Could you also fix the merge conflict and squash commits into one?

Done

Sorry for the delay! @bors r+

:pushpin: Commit 011c1e1f853f1b36025235913387f67bd4c6e8f8 has been approved by JohnTitor
It is now in the queue for this repository.

:hourglass: Testing commit 011c1e1f853f1b36025235913387f67bd4c6e8f8 with merge aed89a5e211e9dd2836b052560062604f891c2a6...

:broken_heart: Test failed - checks-actions

Could we retry this please? The CI logs have expired so we can't know what went wrong.

@rustbot review
(not sure if this is the way to go, sorry for the noise)

@bors retry

Looks like this is stuck in Bors queue @JohnTitor

@bors retry

@fsavy-tehtris: :key: Insufficient privileges: not in try users

@bors retry

:hourglass: Testing commit 011c1e1f853f1b36025235913387f67bd4c6e8f8 with merge 65dd81118967078c2f14c9c1657f9a6ecedfeb5e...

:broken_heart: Test failed - checks-actions

:hourglass: Testing commit 011c1e1f853f1b36025235913387f67bd4c6e8f8 with merge ecfff1878c99a1d62fe64a089d022bc2bbefcbe6...

@bors retry r-

Superseded by #3396, thanks anyway!


added code for submission summaries

code to generate submission summaries
generate embeddings of summaries
push summary and summary embeddings to supabase table

#25
#23


Add markdown support for custom footer links

Summary
This PR addresses enhancements requested issue #(2959) regarding adding support for markdown hyperlinks in footer environment variables. Previously, markdown links added in these environment variables were rendered as plain text. With these changes, the application now correctly interprets and displays these links.
Change Type
Please delete any irrelevant options.

[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

Testing
This was tested on several cases. Works fine for single and multiple markdowns.
To test these changes, add a markdown link in the CUSTOM_FOOTER environment variable and confirm that the link is correctly interpreted and displayed in the footer by the application.
Example markdown:
CUSTOM_FOOTER="Internal [ChatGPT documentation](https://placeholder-link.com)"
Test Configuration:

Add a markdown link to the CUSTOM_FOOTER environment variable and save changes.
Run the application.
Navigate to main page that displays the footer.
Confirm that the markdown link added in the environment variable shows as a clickable hyperlink in the footer.

Checklist
Please delete any irrelevant options.

[x] My code adheres to this project's style guidelines
[x] I have performed a self-review of my own code
[x] I have commented in any complex areas of my code
[x] I have made pertinent documentation changes
[x] My changes do not introduce new warnings
[x] Local unit tests pass with my changes
[x] Any changes dependent on mine have been merged and published in downstream modules.
[x] A pull request for updating the documentation has been submitted.

Tested and seems to work correctly from testing
Few things i noticed:

When adding a pipe between links the format isn’t matching the privacyPolicy and termsOfservice size
If no link is provided, it will redirect back to the chat window(I guess that is fine for now)
Theres some spacing issue with the custom footer and privacyPolicy  and termsOfservice  in mobile view

When adding a pipe between links the format isn’t matching the privacyPolicy and termsOfservice size

Good catch @Tanvez.
@jakubmieszczak maybe you can map any occurrence of | to separator elements?
<div key={`separator-${index}`} className="h-2 border-r-[1px] border-gray-300" />

Theres some spacing issue with the custom footer and privacyPolicy and termsOfservice in mobile view

The prior comment may help resolve this issue. I can try to address it if you don't figure it out. The important thing would be to allow the custom footer render to possibly be made up of multiple elements on the same plain as privacy/tos

Only other comment I have is that client/src/components/Input/Footer.tsx is no longer used, and you can actually delete it as part of this PR

I've addressed the proposed changes in this PR.

All links in the footer are now displayed horizontally next to each other, rather than vertically one on top of another.
I've removed the unused client/src/components/Input/Footer.tsx file to clean up our repository.
The pipe characters have been correctly converted into div elements so they're visually consistent with the rest of the layout.
Text across all sections in footer is aligned to center.

Please let me know if there are any further changes required. Thank you for your feedback @danny-avila !

I've addressed the proposed changes in this PR.
* All links in the footer are now displayed horizontally next to each other, rather than vertically one on top of another.

* I've removed the unused `client/src/components/Input/Footer.tsx` file to clean up our repository.

* The pipe characters have been correctly converted into div elements so they're visually consistent with the rest of the layout.

* Text across all sections in footer is aligned to center.

Please let me know if there are any further changes required. Thank you for your feedback @danny-avila !

thank you @jakubmieszczak looks good to me!
All I would ask if you would kindly make a PR here to add an additional note for using markdown with CUSTOM_FOOTER
https://www.librechat.ai/docs/configuration/dotenv#app-title-and-footer
https://github.com/LibreChat-AI/librechat.ai/blob/main/pages/docs/configuration/dotenv.mdx

Can we merge this PR and include changes in v0.7.3? @danny-avila


System.DllNotFoundException: sqlite3 with Xamarin Studio

Description
This issue only occurs using Xamarin Studio, the project builds fine with visual studio.
Note
This issue is only occurring for Android, and only with Xamarin Studio
Sample Project with Issue
Stactrace
System.DllNotFoundException: sqlite3
  at at (wrapper managed-to-native) SQLitePCL.SQLite3Provider+NativeMethods:sqlite3_open_v2 (byte[],intptr&,int,byte[])
  at SQLitePCL.SQLite3Provider.SQLitePCL.ISQLite3Provider.sqlite3_open_v2 (System.String filename, System.IntPtr& db, System.Int32 flags, System.String vfs) [0x0000f] in <filename unknown>:0
  at SQLitePCL.raw.sqlite3_open_v2 (System.String filename, SQLitePCL.sqlite3& db, System.Int32 flags, System.String vfs) [0x00000] in <filename unknown>:0
  at Akavache.Sqlite3.Internal.SQLite3.Open (System.String filename, SQLitePCL.sqlite3& db, System.Int32 flags, System.IntPtr zVfs) [0x00000] in <filename unknown>:0
  at Akavache.Sqlite3.Internal.SQLiteConnection..ctor (System.String databasePath, Akavache.Sqlite3.Internal.SQLiteOpenFlags openFlags, System.Boolean storeDateTimeAsTicks) [0x00030] in <filename unknown>:0
  at Akavache.Sqlite3.Internal.SQLiteConnection..ctor (System.String databasePath, System.Boolean storeDateTimeAsTicks) [0x00000] in <filename unknown>:0
  at Akavache.Sqlite3.SQLitePersistentBlobCache..ctor (System.String databaseFile, System.Reactive.Concurrency.IScheduler scheduler) [0x00026] in <filename unknown>:0
  at Akavache.Sqlite3.Registrations+<>c__DisplayClass6.<Register>b__0 () [0x0004e] in <filename unknown>:0
  at System.Lazy`1[T].CreateValue () [0x00075] in /Users/builder/data/lanes/3511/b5fafb24/source/mono/mcs/class/referencesource/mscorlib/system/Lazy.cs:438
  at [MVID] 62f381ebb54b44df949b4ae3da486002 3,4,5,6,7
  at [MVID] bd286ccd2dcb40a2ab0cf2f472b25b51 8
  at [MVID] c84d16e159704a2c86a47d94d4e17163 1,2

I too am having this same issue. However, I am only having the issue on our build agent that we use with our Team City CI system. My local dev machine also has Xamarin Studio and I don't have any issues. I am going to double check that I am using the same version of Xamarin Studio and I will report back.

Are you on Alpha channel? This might be related to: https://developer.xamarin.com/releases/android/xamarin.android_6/xamarin.android_6.2/#Mono.Data.Sqlite.dll_and_libsqlite.so

I was on Alpha channel on our build server (where it wasn't working). I just reverted the Build server back to beta channel to match my local machine. After reverting back to Xamarin Studio 6.0.2 I am back up and running.

Reverting back to the Beta Channel and Xamarin Studio 6.0.2 fixed my issue @tim-hoff

Reverting to the Beta Channel fixed it.
@martijn00 @btripp, Thanks for the help.