Missing focus outline on selected sortable table headers

Detailed description

Describe in detail the issue you're having.
In a sortable table when the user clicks on a header to sort the data, the focus outline is missing on the clicked header cell.

Is this a feature request (new component, new icon), a bug, or a general issue?
Bug

Is this issue related to a specific component?
table component

What did you expect to happen? What happened instead? What would you like to see changed?
It should show the focus outline for the selected header. It is not showing the outline.

What browser are you working in?
Firefox, Chrome and Safari

What version of the Carbon Design System are you using?
Carbon v11

What offering/product do you work on? Any pressing ship or release dates we should be aware of?
IBM App Services Portal

Steps to reproduce the issue
Carbon Table Component
Additional information

I see, we should have outline on sortable table header. It is supported in carbon-react

I'm picking this bug.


Omnisharp-vim debug project failed

Details
I created a new console app and create vimspector.json file using :OmnisharpCreateDebugConfig. Then try to debug project using command :OmniSharpDebugProject and the error happened.
But when I using Vimspector command :VimspectorLaunch, it's ok.

Configuration
Neovim version
nvim --version
NVIM v0.7.2
Build type: Release
LuaJIT 2.1.0-beta3
Compiled by<EMAIL_ADDRESS>
Features: +acl +iconv +tui
See ":help feature-compile"

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/opt/homebrew/Cellar/neovim/0.7.2/share/nvim"

Run :checkhealth for more info

OS Version
MacOS Monterrey 12.4 - M1 Arm 64

.NET Version and SDK
dotnet --info
.NET SDK (reflecting any global.json):
 Version:   6.0.301
 Commit:    43f9b18481

Runtime Environment:
 OS Name:     Mac OS X
 OS Version:  12.4
 OS Platform: Darwin
 RID:         osx.12-x64
 Base Path:   /usr/local/share/dotnet/x64/sdk/6.0.301/

Host (useful for support):
  Version: 6.0.6
  Commit:  7cca709db2

.NET SDKs installed:
  3.1.420 [/usr/local/share/dotnet/x64/sdk]
  5.0.408 [/usr/local/share/dotnet/x64/sdk]
  6.0.301 [/usr/local/share/dotnet/x64/sdk]

.NET runtimes installed:
  Microsoft.AspNetCore.App 3.1.26 [/usr/local/share/dotnet/x64/shared/Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 5.0.17 [/usr/local/share/dotnet/x64/shared/Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 6.0.6 [/usr/local/share/dotnet/x64/shared/Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 3.1.26 [/usr/local/share/dotnet/x64/shared/Microsoft.NETCore.App]
  Microsoft.NETCore.App 5.0.17 [/usr/local/share/dotnet/x64/shared/Microsoft.NETCore.App]
  Microsoft.NETCore.App 6.0.6 [/usr/local/share/dotnet/x64/shared/Microsoft.NETCore.App]

Hi, thanks for the detailed report.
This looks to me like a neovim bug. What happens is that the configuration is built as a vim dict which is sent to vimspector, which converts it to a python object with :py3 vim.eval(). This command succeeds in vim and fails in neovim, because when neovim converts this dict it converts boolean values to strings:
" vim:
:py3 print(vim.eval('{"a":1,"b":v:true,"c":"a string"}'))
" {'a': '1', 'b': True, 'c': 'a string'}

" neovim:
:py3 print(vim.eval('{"a":1,"b":v:true,"c":"a string"}'))
" {'a': '1', 'b': 'True', 'c': 'a string'}

I've started talking to puremourning about it, not sure if the best way forward right now is to try to work around it, or to get it fixed upstream.

@puremourning gave me a tip to use vimspector type coersion to work around this, and now launching the debugger for a project works as expected in neovim.


Tensorflow GPU version support with skflow

I have been trying to execute text_classification.py example of skflow but it is generating an error. Please see the attached document for an error as well as the version of tensorflow and the skflow.
the error is tensorflow.python.framework.errors.InvalidArgumentError: Cannot assign a device to node 'train/update_words/words_embeddings/Assign_1': Could not satisfy explicit device specification '' because the node was colocated with a group of nodes that required incompatible device '/job:localhost/replica:0/task:0/GPU:0'

Try updating your version of TensorFlow: 0.5 was the original release and there have been a ton of bug fixes and improvements since then.

how to update gpu version of tensorflow

Try reading through: https://www.tensorflow.org/versions/r0.7/get_started/os_setup.html#download-and-setup

(Feel free to comment if, after upgrading, you still have the problem, and we can debug further!)


Updated to support JDK 9+

#67 - Updated to use latest RI tools version, made endorsed bootclasspath argument contingent on JDK version, added option to automatically set accessExternalSchema.

Can you have a look at my pull request #71? It also contains fixes for invocation under Java 9-11, but doesn't add an extra option as your PR.


There is a problem with CodeLens (setup sugar) in "v0.27.4"

Capture (mp4)
https://user-images.githubusercontent.com/188642/130305329-9db0f72c-7715-4391-8887-aa211863a17a.mp4
Error
[Error - 10:01:29] Request workspace/executeCommand failed.
  Message: Request workspace/executeCommand failed with message: Cannot read property 'loc' of null
  Code: -32603
(node:74853) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'loc' of null
    at getAddMissingImportsEdits (/Users/yaegassy/.vscode/extensions/johnsoncodehk.volar-0.27.4/node_modules/vscode-vue-languageservice/out/commands/useSetupSugar.js:40:57)
    at /Users/yaegassy/.vscode/extensions/johnsoncodehk.volar-0.27.4/node_modules/vscode-vue-languageservice/out/commands/useSetupSugar.js:32:39
    at async Object.<anonymous> (/Users/yaegassy/.vscode/extensions/johnsoncodehk.volar-0.27.4/node_modules/vscode-vue-languageservice/out/services/executeCommand.js:26:13)
(Use `Code Helper (Renderer) --trace-warnings ...` to show where the warning was created)
(node:74853) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 2)
(node:74853) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
(node:74853) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'loc' of null
    at getAddMissingImportsEdits (/Users/yaegassy/.vscode/extensions/johnsoncodehk.volar-0.27.4/node_modules/vscode-vue-languageservice/out/commands/useSetupSugar.js:40:57)
    at /Users/yaegassy/.vscode/extensions/johnsoncodehk.volar-0.27.4/node_modules/vscode-vue-languageservice/out/commands/useSetupSugar.js:32:39
    at async Object.<anonymous> (/Users/yaegassy/.vscode/extensions/johnsoncodehk.volar-0.27.4/node_modules/vscode-vue-languageservice/out/services/executeCommand.js:26:13)
(node:74853) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 3)
(node:74853) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'loc' of null
    at getAddMissingImportsEdits (/Users/yaegassy/.vscode/extensions/johnsoncodehk.volar-0.27.4/node_modules/vscode-vue-languageservice/out/commands/useSetupSugar.js:40:57)
    at /Users/yaegassy/.vscode/extensions/johnsoncodehk.volar-0.27.4/node_modules/vscode-vue-languageservice/out/commands/useSetupSugar.js:32:39
    at async Object.<anonymous> (/Users/yaegassy/.vscode/extensions/johnsoncodehk.volar-0.27.4/node_modules/vscode-vue-languageservice/out/services/executeCommand.js:26:13)
(node:74853) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 4)
[Error - 10:01:29] Request textDocument/semanticTokens/range failed.
  Message: Request textDocument/semanticTokens/range failed with message: Cannot read property 'loc' of null
  Code: -32603
[Error - 10:01:29] Request textDocument/codeLens failed.
  Message: Request textDocument/codeLens failed with message: Cannot read property 'loc' of null
  Code: -32603
[Error - 10:01:30] Request textDocument/semanticTokens/range failed.
  Message: Request textDocument/semanticTokens/range failed with message: Cannot read property 'loc' of null
  Code: -32603
[Error - 10:01:30] Request textDocument/documentLink failed.
  Message: Request textDocument/documentLink failed with message: Cannot read property 'loc' of null
  Code: -32603
[Error - 10:01:34] Request textDocument/codeLens failed.
  Message: Request textDocument/codeLens failed with message: Cannot read property 'loc' of null
  Code: -32603

I'm in the same situation as you，back to v0.27.3 is ok

Thanks in 0.27.5, thanks for the report!

@johnsoncodehk back to v0.27.3 is ok,but v0.27.5  is no。thanks
https://user-images.githubusercontent.com/24860389/130316241-f01a0e14-744d-4bb4-800d-a1e8e2a0926f.mp4

@Half-A-Turnip Could you try to run debug mode with this commit? https://github.com/johnsoncodehk/volar/commit/df2a3acb031c69fb894ce138ff61730bb2c7e33b
Run debug mode: https://github.com/johnsoncodehk/volar/blob/master/.github/STEP_TO_PROFILING.md#run-volar-on-debug-mode

@

@Half-A-Turnip Please try v0.27.6.

@johnsoncodehk i tried. latest branch is ok. now 0.27.6 is good. thanks

v0.34.11 and having the same problem over here

@TrayHard Please check if you can update your vscode version, if you can't fix please open a new issue and describe your error.


rename 'stylesheets' directory to 'styles'

stylesheets directory is deprecated in latest atom.

This PR fix #28


Max Events option not working

The max_event option is not working for v2.x branch.

Yes it does :)


Enhancement: Testnet, aspect ratio of Native Wallet Wombat seems wrong

Description
Wombat Logo is blurry
Reproduce
1: Click Connect
2: Select Native Wallet
Expected Behavior
Clear Logo of accepted Wallets

Environment Used
OS: OSX
Browser: Chrome

@DestinyDevTeam  this is what is coming back from the api for their icon, i've noticed it elsewhere as well.

Looks better now, I guess this is what we get from that image source.


[FEA] Support org.apache.spark.sql.catalyst.expressions.ReplicateRows

Is your feature request related to a problem? Please describe.
This is a feature request to support org.apache.spark.sql.catalyst.expressions.ReplicateRows.
ReplicateRows is an internal function used by optimizer to rewrite EXCEPT ALL and INTERSECT ALL queries as per the Spark source code.
Below is a mini repro:
import org.apache.spark.sql.Row
import org.apache.spark.sql.types._
import org.apache.spark.sql.functions._

val data = Seq(
    Row(Row("Adam ","","Green"),"1","M",1000.1, "2019-01-01",List("Java","Scala")),
    Row(Row("Bob ","Middle","Green"),"2","M",2000.2, "2019-01-02",List("Java","Python")),
    Row(Row("Cathy ","","Green"),"3","F",3000.3, "2019-01-03",List())
)

val schema = (new StructType()
  .add("name",new StructType()
    .add("firstname",StringType)
    .add("middlename",StringType)
    .add("lastname",StringType))
  .add("id",StringType)
  .add("gender",StringType)
  .add("salary",DoubleType)
  .add("birthdayStr",StringType)
  .add("language",ArrayType(StringType))
             )

val df = spark.createDataFrame(spark.sparkContext.parallelize(data),schema)
df.withColumn("birthday", to_date(col("birthdayStr"))).write.format("parquet").mode("overwrite").save("/tmp/testparquet")
val df2 = spark.read.parquet("/tmp/testparquet")
df2.createOrReplaceTempView("df2")
df2.printSchema

val querytext="""SELECT gender from df2 except all (select gender from df2 where salary <> 10)"""
sql(querytext).explain

The Spark Driver log messages:
  !Exec <GenerateExec> cannot run on GPU because not all expressions can be replaced
    !NOT_FOUND <ReplicateRows> replicaterows(sum#99L, gender#76) cannot run on GPU because no GPU enabled version of expression class org.apache.spark.sql.catalyst.expressions.ReplicateRows could be found

The APIS already exist in CUDF TableView.repeat. The hardest part will be memory management and combining it with the existing code that is rather explode and pos_explode specific.


标签数据和Range-angle数据不对齐

标签csv文件中的px、py转换到range-angle坐标系下，在距离稍远的地方出现较大的误差，其中ramap根据雷达参数计算，想请问是自动化标注产生的距离估计误差还是我计算过程出现了问题

Can you show an example? thanks

Thank you for your reply. Here is an example, i read a raw adc file from date sequence 2019_04_09_bms1000, frame 000808.mat, for instance.
The annotation info is
22,80,-1.11408487,11.31695161,0.6,1.7

We can get px = -1.11 m and py = 11.31 m, which are presented in Cartesian coordinate, so we can calculate the range and angle-of-arrival of the target by $r=\sqrt{x^2+y^2}$ and $a=\arctan{x/y}$ : r=11.37 m, a=-5.62 deg.
Then I used the function labelmap2ra provided in RAMPCNN source code, to map the target into confmap.
Here I draw the ra-view of the radar data, strictly using the mmwave radar parameters provided in https://github.com/Xiangyu-Gao/Raw_ADC_radar_dataset_for_automotive_object_detection/blob/main/config.py, and I draw the target point on the figure as well. (According to the radar config, the resolution of range-bin is about 0.22m. )

Obviously there is a disalignemnt between the ramap and the annotation. But in some other frames, when the target is close to the radar, the location of annotation looks correct, for example, frame 000220.mat.

I am trying to use the dataset for a semantic segmentation task, but the problem do bother me a lot.
I wonder if I understood the mapping px and py to the RA coordinate correctly, or is there any operation or setting that I missed, or it's simply due to estimation errors caused by semi-automated annotation.
Thank you!

I also considered the crop operation mentioned in https://github.com/Xiangyu-Gao/Radar-multiple-perspective-object-detection/blob/aff04b354f07f2689dfd8fcf6fcb68a9404eb73a/utils/mappings.py#L15, but the impact on resolution is small and the results are not improved.

The crop operation should have little effect on that. I assume this mismatch is an error from the auto-labeling. The further the distance, the larger the error usually.


Way to verify that all stories render without errors

Is your feature request related to a problem? Please describe.
What happened a bunch of time to me is

I had a working stories for some component
I updated the component by adding new required prop - without it the component crashes
I forget to update the stories
Since the stories use the controls Story<Props> type + .args pattern (that is AFAIK the recommended way to do it in the docs) the story crashes

We do have tsc + eslint + ladle build in our CI pipeline and none of these catches that the story is no longer working
Describe the solution you'd like
2 solutions come to my mind

have some new command (like ladle verify) that would actually render all of the stories and report if some of them render error?
change something with how the story is defined or its types, so that this would get caught on type level?

Additional context
component
export type TestComponentProps = {
  requiredA: () => number;
  requiredB: () => number;
};

export const TestComponent = ({ requiredA, requiredB }: TestComponentProps) => {
  return
a+b = {requiredA() + requiredB()}
;
};

story
export const Test: Story<TestComponentProps> = (props) => <TestComponent {...props} />;
Test.args = {
  requiredA: () => 1,
  // requiredB: () => 2,
  // ^ if this one is missing, tsc/linter/build passes but the story crashes when rendered
};

I think not applying Partial here would fix this this (threw an error) https://github.com/tajo/ladle/blob/main/packages/ladle/lib/app/exports.ts#L94C5-L94C12
@wojtekmaj any thoughts on this?

The problem I was aiming to address with Partial is in my opinion bigger than the one reported here, and I don't know if I have a solution to make everyone happy. In reality many stories would change one, maybe two props, leaving others at their defaults, making them pretty good, isolated and readable test cases. Consider the following:
export const Test: Story<TestComponentProps> = (props) => <TestComponent requiredB={2} {...props} />;

Test.args = {
  requiredA: () => 1,
  // ^ tsc/linter/build would crash despite the fact in reality it would have rendered just fine
};

should we have Story and StoryRequired types?

Sounds like a plan!

Yea I think the StoryRequired should solve this.
One think to consider might be: the pattern mentioned in https://github.com/tajo/ladle/issues/515#issuecomment-1742446858, that would break if Partial was missing, seems not to be that common (at least by briefly looking at our stories, it is less than 10% of stories), so maybe we could make the Story type strict (without Partial) and introduce StoryPartial for these case? This would probably be a breaking change, so not sure if it is worth it, but having safer default seems like a good thing to do.

I think it is quite valuable to have some way to render all stories. Missing props is just one possible reason for a story to be broken. It's important is to check the stories in runtime, not just static type checking (if it does both, even better).
In my opinion, best way is if this could be done by just creating a test for each story. Not sure how difficult is to deal with support for different test runners though.
But if it's done by some separate command like ladle verify as suggested (possibly using Vitest, since Ladle already uses Vite), that's fine too.


IP archive

Where can I find archive for old releases? I have upgraded from U 5.52 to U 2017.4 LTS and I am receiving errors on unloading scene, causing the entire scene graph to get dirty, making me unable to restart the scene. I have bought the original product and the old website is dead.

This is the error:
MissingReferenceException: The object of type 'UIWindowInteractive' has been destroyed but you are still trying to access it.
Your script should either check if it is null or you should not destroy the object.
UnityEngine.Component.GetComponent[Animator] () (at C:/buildslave/unity/build/artifacts/generated/common/runtime/ComponentBindings.gen.cs:48)
Devdog.InventorySystem.UI.UIWindow.get_animator () (at Assets/InventorySystem/Scripts/UI/UIWindow/UIWindow.cs:123)
Devdog.InventorySystem.UI.UIWindow.HideFirst () (at Assets/InventorySystem/Scripts/UI/UIWindow/UIWindow.cs:328)
Devdog.InventorySystem.UI.UIWindow.LevelStart () (at Assets/InventorySystem/Scripts/UI/UIWindow/UIWindow.cs:187)
Devdog.InventorySystem.UI.UIWindow.m__0 (Scene scene, LoadSceneMode loadMode) (at Assets/InventorySystem/Scripts/UI/UIWindow/UIWindow.cs:169)
UnityEngine.SceneManagement.SceneManager.Internal_SceneLoaded (Scene scene, LoadSceneMode mode) (at C:/buildslave/unity/build/artifacts/generated/common/runtime/SceneManagerBindings.gen.cs:245)


Version 4.5.0

Find and auto-start JavaScript modules from markup:
data-module="module-name"(PR #227)

Also includes the explicit setting of a ruby version
https://github.com/alphagov/govuk_frontend_toolkit/compare/v4.4.0...master, might be worth mentioning? Otherwise happy to merge.

Talked to @fofr about the ruby bump and agreed it doesn't need to be in the changelog.
The ruby version only affects the dev/testing part of this repo, not the things exported in packages, eg, Gem/NPM.


Enable the serde feature when the schemars feature is turned on

When enabling the schemars feature on the accesskit crate, it is necessary to also enable the serde feature. Failing to do so will result in an invalid JSON schema being generated due to the fact that serde attributes that are applied on our types won't be present.
Here is a piece of the JSON schema that gets generated when the serde(rename_all = "camelCase") attribute is not applied to the Role enum:
...
  "definitions": {
    "Role": {
      "description": "...",
      "type": "string",
      "enum": [
        "Unknown",
        "InlineTextBox",
...

This PR enables the serde feature if the schemars feature is enabled, so that it is impossible to accidentally forget to enable both of them.

Woops @mwcampbell  clippy is unhappy!


Updated Demo video with README

@asa1997
Please review and share your feedback
@sumodgeorge

@asa1997
Please approve it.
@panickervinod


SDK version support -- build error -- SDK Build Tools revision is too low for project

My project is using SDK version 26.1.1 and React Native 0.55. Is this supported?
While building, the error looks like this:

A problem occurred configuring project ':react-native-billing'.
The SDK Build Tools revision (23.0.1) is too low for project ':react-native-billing'. Minimum required is 25.0.0

@kezzico I'm having the same issue after upgrading to RN .56 and then I did some updates to android studio, now I'm lost figuring this out.  If you find the solution, please let me know and I'll do the same.  Good luck!

@kezzico @mycoberago place this code in your android/build.gradle file:
(you may need to use 25 and 25.0.3 depending on you setup)
subprojects { subproject ->
    afterEvaluate{
        if((subproject.plugins.hasPlugin('android') || subproject.plugins.hasPlugin('android-library'))) {
            android {
                compileSdkVersion 26
                buildToolsVersion "26.0.3"
            }
        }
    }
}

This should be fixed in 2.10.0

@cbrevik Can you explain how this was solved? I'm still facing the same issue, adding subprojects config as suggested by @joserocha3 doesn't seem to solve it either...

Ok, I have found the solution — documenting this for other people running into the same problem.
If you take a look at https://github.com/idehub/react-native-billing/commit/959d1ed870b033c89049729b85d1fe4690ea002c , the safeExtGet method was added, which allows you to overwrite the different parameters used in the build process.
You can define those parameters in your android/build.gradle file:
ext {
    compileSdkVersion = 28
    targetSdkVersion = 26
    minSdkVersion = 16
    buildToolsVersion = "28.0.3"
    supportLibVersion = "28.0.0"
}

You can then also use this configuration in your android/app/build.gradle file:
android {
    compileSdkVersion rootProject.ext.compileSdkVersion
    buildToolsVersion rootProject.ext.buildToolsVersion
    ...
}

In my case I also had to add the following line in android/gradle.properties:
android.enableAapt2=false

Hi @stefanwuest
Sorry was busy this weekend so unable to answer. That is indeed the correct answer.


docs: fips notes for mariner skus

What type of PR is this?

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

Fixes #
Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

tested in https://github.com/Azure/AgentBaker/pull/3336

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall First build on ace/fipsNotes at 61.476%

Totals

Change from base Build<PHONE_NUMBER>:
61.5%

Covered Lines:
2558

Relevant Lines:
4161

💛  - Coveralls


Memory creep over time

Hi, again, thanks for the great project! I'd appreciate some help on this:
I'm running g3 in production as a straightforward forward proxy with pretty modest load (~2.1kRPS peak), and I've noticed that memory consumption has been trending upwards over time -

I considered the possibility of hanging tasks, but the total number of tasks don't seem to reflect the same trend.

There is also no configuration to cull long running tasks, or to limit the memory consumption by each thread. I'm not sure if there is a possiblity of memory leak, or some other issue (looking at tokio issues, memory fragmentation seems to be a common problem)?

the memory utilization is the total memory used by g3proxy?

Hi @zh-jq , It's the memory utilisation captured by node-exporter running on the instance. I have other things running on the instance, but main memory consumption (and growth) is from g3 threads.

18% is far less than 70% in the first graph. Is it possible for you to add a RES memory usage graph for the g3proxy process?

And the mempry utilization in first graph is total-free or total-available?

Ah, good catch. Turns out my metric was misguided - the utilisation is (1- MemFree) / MemTotal which doesn't reflect the memory availability in the system. Due to large number of files opened by the process (I presume) the buffers/cache get quite large over time:
MemTotal:        7950368 kB
MemFree:         1902296 kB
MemAvailable:    5806396 kB

But it's not actually a cause for concern since it's memory that the system can reclaim anyway.
I'll amend my metric, sorry for the bother!


Add types for QueryFilters which flow down to Query<> and QueryKey/DataTag types

At tRPC we're working on a new react query client which only provides factories to React Query interfaces. Many of these factories already support full type-safety, for instance a QueryKey can have a DataTag which hints at the query data type.
QueryKey (with DataTag) and Query<> both support typesafety, but QueryFilters hides this and only exposes the defaults. This means our queryFilter factory method cannot be type-safe without replicating the tanstack query types into the tRPC codebase.
I propose upstreaming this type-safety so that everyone can benefit and tRPC can utilise it natively
Here is the work I'm doing on the tRPC side: https://github.com/trpc/trpc/pull/6265

test-d.ts

Done
Good I'm doing this. I think getQueryState could be next on my agenda, DataTag has no way to communicate an error type but getQueryState can get you back the error. I've left a test with a comment for now to document the current behaviour


UX/UI Confirmation Messages for Namespace Decommissioning

UX for Delete Namespace and Delete Environment to show messaging for the warning, impact and confirmation
UX for Delete Authorization Profile and Delete Product to block deletion based on above requirements

GIVEN I have namespaces
AND the namespaces currently have active consumers
WHEN I go to delete those namespaces
THEN I see a message informing me why my namespace was not deleted
GIVEN I have namespaces
AND the namespaces currently have (?)
WHEN I go to delete those namespaces
THEN I see a message informing me why my namespace was not deleted
@ikethecoder did I understand the scenarios correctly?

@ikethecoder please review the updated changes

@elisafw-ux I don't think the link is valid anymore - do you have an updated one?

@ikethecoder  I sent you a Figma invite to your email. Once you accept it, click on this https://www.figma.com/file/00SZt0Ezx1DGz1x1tlcNLL/APS-Portal-Designs?node-id=28532%3A15765&t=LGPxFB4qWqI4oRA8-1 to view the mockup.

Sorry, @elisafw-ux can you send a new link?

@ikethecoder it's the same link but you have to accept my invite first bc only invited people can see it.

@elisafw-ux I reviewed the Figma designs - all looks great.

@ikethecoder we probably need to brainstorm this again when you get a chance.
The criteria is going to be:
If there are active services and consumer credentials in a namespace, users cannot delete it and they would need to contact us.
I adjusted the designs to reflect this but this creates a cascading problem where every time a user wants to delete an environment or a product, they would also need to contact us and I am not sure that's good UX and it also defeats the purpose of self-serve.

@elisafw-ux do you want to propose an alternative more in line with guiding principles, or should we discuss as a team?

Requirements:

The confirmation modal will prevent users from deleting a namespace, a product, a product environment, or an authorization profile if there are consumer credentials associated with it.

If the user tries to delete any of these with linked consumers, they will see a message asking them to remove all consumers before proceeding with the deletion.

For authorization profiles, the user needs to delete any associated product environments before they can delete the profile.

However, the confirmation modal will let users delete a namespace, a product, a product environment, or an authorization profile if there are no consumer credentials associated with it.

Designs Could you please review the messages on the confirmation modals. If good to go I'll pass it on for implementation @ikethecoder

Thanks, @elisafw-ux! This looks great to me.

@Jonesy ready to implement. Requirements above. Let me know if you have any questions!

@ikethecoder I have a simple proof of concept set up and am using the Environments section of the Products page to test it out. When the user clicks Delete Environment... the modal will make a GraphQL request like this:
  query CheckDeleteEnvironment($id: ID!) {
    deleteEnvironmentPermissions(id: $id) {
      allowed
      reason
    }
  }

and get back something like this if they're not allowed:
{
    "data": {
        "deleteEnvironmentPermissions": {
            "allowed": false,
            "reason": "The PharmaNet Electronic Prescribing other environment cannot be deleted as there are consumer credentials associated with it. Please remove all consumers linked to this environment before proceeding with the deletion."
        }
    }
}

and obviously an allowed: true if it's all good would have the danger red confirmation button as per the designs. Open to suggestions or additional API/GraphQL composition or naming.
@elisafw-ux one catch where I might not be able to completely match the design is highlighting the product name and environment with bold text and a label if the entire reason is returned back as a string from the API. But if we get parts back I could probably compose something that would be able to match it correctly.


Puts me into spectator mode

Sometimes when I join a match in a modded lobby it puts me into spectator mode and sometimes it says "I don't own the song"

Hey, this is most likely related to MultiplayerExtensions. That mod is still in active development so things can break.
Your issue sounds like it might happen if custom a song fails to download from BeatSaver on time.

Sometimes when I join a match in a modded lobby it puts me into spectator mode and sometimes it says "I don't own the song"

The best way to make sure this doesn't happen is to click on the song that will be played under suggested songs on the right. That makes sure that the song gets downloaded and imported and it will show up in the selected song menu once it's done downloading. I discovered this over the past couple of days of testing.

more likely the song won't show up under suggested songs. and sometimes I can play the song but the song info will not show up


Do not call UpdateJsonSerializerSettings if method does not exists.

When the code generation preference generateUpdateJsonSerializerSettingsMethod is set to false we should not make call to UpdateJsonSerializerSettings
Unable to compile client because of the following error
Severity	Code	Description	Project	File	Line	Suppression State
Error	CS0103	The name 'UpdateJsonSerializerSettings' does not exist in the current context

The idea is to set generateUpdateJsonSerializerSettingsMethod = false and provide it in the base class or set it to true and it is generated...

Sorry, this is a little bit confusing... maybe we should put this information in the wiki? Anyway, thanks for the PR!


Discussion: should we include zero?

It is debatable if zero is a positive integer or not (example). It probably depends on the context, so maybe we different versions of numbers.txt?
I would like to use numbers.txt to iterate through my lists (starting with index zero), so this would really help!

This is a very important question, and I'm not sure I can see an easy answer. I'd like to open this up to further discussion before proceeding.

This would require a test update.

That is an excellent point @deplicator!

To get a feel for the mood out there on this issue, I have posted this poll on the fediverse: https://phpc.social/@andrewfeeney/109938989159254475

A reasonable compromise would be to include +0 but not -0 (which are represented differently in IEEE 754)

@kikito You're the first to bring actual standards to the discussion and now you're blowing this wide open again. 🤯


CosmosDb: Remove blocking calls from CFP

Remove blocking calls from ChangeFeedProcessor implementation and other various fixes.

/azp run java-cosmos-tests

/azp run java - cosmos - tests

in response to your question:

milismsft: We need to loop/wait before making another check to retrieve the state. Can you please recommend another way to do this in a non-blocking fashion?

Take a look at Flux.repeat() and Flux.takeUntil() you can build a non-blocking while loop using them.

Work items created for two remaining issues:
https://github.com/Azure/azure-sdk-for-java/issues/5015
https://github.com/Azure/azure-sdk-for-java/issues/5013
There are two E2E unit tests in practice will ensure coverage for most of the functionality in CFP. There's one work item (in progress) which adds the coverage for extensive usage and handles potential splits. Additionally I'm looking at adding tests similar to what we have in .NET for CFP on opportunistic bases.


Allow custom time filter

What filter does this issue target?
https://letsblock.it/filters/youtube-shorts
What changes should be made?
It would be good if we could manually select a threshold for the length of the video so that we can try to lower the amount of false positives.
For example https://www.youtube.com/watch?v=H4HAuU13eMU is under 60 seconds however it is not a Youtube short.
It would be nice if we could manually adjust it like we can for https://letsblock.it/filters/youtube-video-title
Youtube no longer automatically adds "shorts" to the end of the video and as such just a name filter no longer works
For example: https://www.youtube.com/shorts/U0BBf2Pi6pg
However since Youtube uses the "shorts" subdomain is it possible to have the filter look at the link of the video and hide them that way?
Thanks!
Did you write a uBlock filter solving this?
No response
Additional notes
No response

For your second and last paragraph, see Pull Request https://github.com/letsblockit/letsblockit/pull/317.

For your second and last paragraph, see Pull Request #317.

I added your two new rules you wrote in that pull request however they dont seem to work or did i do something wrong?

www.youtube.com##ytd-browse ytd-grid-video-renderer:has(span.ytd-thumbnail-overlay-time-status-renderer[aria-label="Shorts"])
www.youtube.com##ytd-browse ytd-rich-item-renderer:has(span.ytd-thumbnail-overlay-time-status-renderer[aria-label="Shorts"])
www.youtube.com##ytd-search ytd-video-renderer:has(span.ytd-thumbnail-overlay-time-status-renderer[aria-label="Shorts"])
www.youtube.com##ytd-watch-next-secondary-results-renderer ytd-compact-video-renderer:has(span.ytd-thumbnail-overlay-time-status-renderer[aria-label="Shorts"])
www.youtube.com##ytd-watch-next-secondary-results-renderer ytd-shelf-renderer:has(span.ytd-thumbnail-overlay-time-status-renderer[aria-label="Shorts"])
www.youtube.com##ytd-browse[page-subtype="subscriptions"] ytd-video-renderer span.ytd-thumbnail-overlay-time-status-renderer[aria-label="Shorts"]:upward(ytd-item-section-renderer)

is replaced by:
www.youtube.com##ytd-browse[page-subtype="home"] ytd-rich-item-renderer:has(.ytd-thumbnail[href^="/shorts/"])
www.youtube.com##ytd-browse[page-subtype="subscriptions"] ytd-grid-video-renderer:has(.ytd-thumbnail[href^="/shorts/"])
www.youtube.com##ytd-search ytd-video-renderer:has(.ytd-thumbnail[href^="/shorts/"])
www.youtube.com##ytd-watch-next-secondary-results-renderer ytd-compact-video-renderer:has(.ytd-thumbnail[href^="/shorts/"])
www.youtube.com##ytd-watch-next-secondary-results-renderer ytd-shelf-renderer:has(.ytd-thumbnail[href^="/shorts/"])
www.youtube.com##ytd-browse[page-subtype="subscriptions"] ytd-video-renderer .ytd-thumbnail[href^="/shorts/"]:upward(ytd-item-section-renderer)

They only work at the homepage, subscription-page, search-page, and watch-next column.

Hello @bennieboi20, the outdated rules filtering by duration have been removed, you'll get the fix within 24 hours, or earlier if you refresh your list to grab it!


fix: add dlt repo copy to the dockerfile

The docker-run-evidence command does a mdsbox make run serve.
The run part cd into the dlt folder, but it was never copied in the Dockerfile, thus generating an error.

Thanks @benjaminsicard - will merge


Install systemd-networkd package, if any

On Red Hat systems networkd is packaged as systemd-networkd. In particular:

EL7
EL8/EL9 shipped in EPEL
Fedora since 33

#318 is the issue for this.


feat(plugins): add support wildcard config for scoped package plugin

If you wish load all scoped package plugins by @*/karma-* like karma-*, karma doesn't handle it.
It'll be modified to<EMAIL_ADDRESS>internally, and karma doesn't care scoped package's sub directories.
In this PR, add features below.

support plugin name with multiple * (replace ALL * to .*)

@*/karma-* is now valid
karma-* also work same as before
however, if anyone already force using multiple * (such as karma-*-foo-.*), there is a risk of incompatibility

when finding plugin, get scoped package name now correctly (before: @foo, now: @foo/bar)

Fixes #3656

:white_check_mark: Build karma 2936 completed (commit https://github.com/karma-runner/karma/commit/632ce319f5 by @hdmr14)

@googlebot I signed it!

:white_check_mark: Build karma 2941 completed (commit https://github.com/karma-runner/karma/commit/7e8c7cd539 by @hdmr14)

@devoto13 Thanks for review and implement the tests!
PR now supports Node 10(replaced flatMap to map and reduce).

@googlebot I signed it!


extend leveldb with science against key index

https://github.com/jamesturk/go-jellyfish

Started work.

match jargon prefix fo
152 25 matches found
jargon "foaf"
jargon "fod"
jargon "fold case"
jargon "followup"
jargon "fontology"
jargon "foo"
jargon "foobar"
jargon "fool"
jargon "fool file"
jargon "foonly"
jargon "footprint"
jargon "for free"
jargon "for the rest of us"
jargon "for values of"
jargon "fora"
jargon "foreground"
jargon "fork"
jargon "fork bomb"
jargon "forked"
jargon "formosa's law"
jargon "fortrash"
jargon "fortune cookie"
jargon "forum"
jargon "fossil"
jargon "four-color glossies"
.
250 ok
match jargon levenshtein foo
152 3 matches found
jargon "fod"
jargon "foo"
jargon "fool"
.
250 ok

next up, tuning distance and adding more. we can precompute others, magic with #15

science done


LOGIN/SIGNUP

LOGIN/SIGNUP
Acceptance Criteria
GIVEN a book search engine
WHEN I click on the Login/Signup menu option
THEN a modal appears on the screen with a toggle between the option to log in or sign up

Login and Signup loads.


Ecto 3 support

Also contains the changes from #178 .
@scottmessinger here are my latest changes in support of Ecto3.  Please can you take a look and merge if you're happy?  I believe you could close #178 since I pulled in all your changes (and those of our past contributors).

@joeapearson This looks good! Quick question about compatibility -- while the upsert will require mongodb 4.4, will anything break if you use this with Mongo 4? If not, I'm +1 on merging this after I take a closer look! A quick review looked good.

@scottmessinger I'm pretty sure older versions of Mongo will continue to work in non-upsert cases but honestly I haven't checked.  I say live dangerously and we'll fix the bugs later :).

Let's merge this :)

@scottmessinger please can you OK / not OK this and we'll merge.  Need to keep this thing moving I suppose!.

@joeapearson Sorry for being MIA over the past 1.5 months! My goal is to cut a beta release this week!

@joeapearson @johnnyshields This PR is in 1.0.0-beta.2! The initial work I had done is in 1.0.0-beta.1, so we should be able to see if anything in this PR is breaking things more easily.
Thanks everyone for the help! If everything goes well, we cut 1.0.0 by the end of 2022?

I hope you mean by the end of 2021!

@johnnyshields 🤦  yup! By the end of 2021!

Hey @scottmessinger, is there anything blocking a 1.0.0 release or is there anything I can do to help get it across the line?
Lets make a 1.0.0 happen by the end of 2022 after all 😀

@d-staehler Nothing I can think of! If this works, we can cut it this month! Btw, are you interested in getting involved? If so, I'm happy to provide any help I can as you navigate the code base!


Antifeature/remove possibility to set static creds

What this PR does / why we need it:
Remove the possibility to store credentials on the disk.
It was insecure.
How can this PR be verified?
Try to deploy with credentials, but without setting the vm_extensions
Is there any change in kubo-deployment?
Yes.
Is there any change in kubo-ci?
No.
Does this affect upgrade, or is there any migration required?
If someone has been using credentials, they have to migrate to use VM Service Account in GCP or Instance Profile in AWS
Release note:
- Removed ability to set static credentials for GCP and AWS instead of using vm Service Account or Instance Profile

I am not interested in this anymore


Using Godot.mono headless export on linux doesn't embed or generate dotnet assemblies

Tested versions
Using 4.3-mono stable build
System information
Debian 12 bookworm (docker image: stalker2106x/godot-ci:4.3-mono)
Issue description
I am currently writing a CI workflow on github that exports my game for windows x86-64.
It uses the stable Godot 4.3 mono build, and runs  using
godot --headless --export-release "Windows Desktop" "$WIN_BUILD_PATH/$EXPORT_NAME.exe"
export_preset.cfg contains
[preset.0]

name="Windows Desktop"
platform="Windows Desktop"
runnable=true
advanced_options=true
dedicated_server=false
custom_features=""
export_filter="all_resources"
include_filter=""
exclude_filter="maps/*"
export_path="export/OpenRomu_v1.3.1/OpenRomu.exe"
encryption_include_filters=""
encryption_exclude_filters=""
encrypt_pck=false
encrypt_directory=false
script_export_mode=2

[preset.0.options]

custom_template/debug="/usr/local/share/godot/templates/windows_debug_x86_64.exe"
custom_template/release="/usr/local/share/godot/templates/windows_release_x86_64.exe"
debug/export_console_wrapper=0
binary_format/embed_pck=false
texture_format/s3tc_bptc=true
texture_format/etc2_astc=false
binary_format/architecture="x86_64"
codesign/enable=false
codesign/timestamp=true
codesign/timestamp_server_url=""
codesign/digest_algorithm=1
codesign/description="OpenRomu"
codesign/custom_options=PackedStringArray()
application/modify_resources=true
application/icon="res://OpenRomu.ico"
application/console_wrapper_icon="res://OpenRomu.ico"
application/icon_interpolation=4
application/file_version=""
application/product_version=""
application/company_name="OpenRomu"
application/product_name="OpenRomu"
application/file_description="OpenRomu"
application/copyright="OpenRomu"
application/trademarks=""
application/export_angle=0
application/export_d3d12=0
application/d3d12_agility_sdk_multiarch=true
ssh_remote_deploy/enabled=false
ssh_remote_deploy/host="user@host_ip"
ssh_remote_deploy/port="22"
ssh_remote_deploy/extra_args_ssh=""
ssh_remote_deploy/extra_args_scp=""
ssh_remote_deploy/run_script="Expand-Archive -LiteralPath '{temp_dir}\\{archive_name}' -DestinationPath '{temp_dir}'
$action = New-ScheduledTaskAction -Execute '{temp_dir}\\{exe_name}' -Argument '{cmd_args}'
$trigger = New-ScheduledTaskTrigger -Once -At 00:00
$settings = New-ScheduledTaskSettingsSet
$task = New-ScheduledTask -Action $action -Trigger $trigger -Settings $settings
Register-ScheduledTask godot_remote_debug -InputObject $task -Force:$true
Start-ScheduledTask -TaskName godot_remote_debug
while (Get-ScheduledTask -TaskName godot_remote_debug | ? State -eq running) { Start-Sleep -Milliseconds 100 }
Unregister-ScheduledTask -TaskName godot_remote_debug -Confirm:$false -ErrorAction:SilentlyContinue"
ssh_remote_deploy/cleanup_script="Stop-ScheduledTask -TaskName godot_remote_debug -ErrorAction:SilentlyContinue
Unregister-ScheduledTask -TaskName godot_remote_debug -Confirm:$false -ErrorAction:SilentlyContinue
Remove-Item -Recurse -Force '{temp_dir}'"
dotnet/include_scripts_content=false
dotnet/include_debug_symbols=false
dotnet/embed_build_outputs=true
texture_format/bptc=true
texture_format/s3tc=true
texture_format/etc=false
texture_format/etc2=false

The machine does build an .exe and its .pck, but doesnt generate any data_* folder nor embeds the assembly build outputs inside .exe
What am I missing? Is this unsupported ? I have Dotnet 8.0 Installed on the docker image.
Thanks!
Steps to reproduce
Just run an "export-release" using windows export templates 4.3-mono with godot headless on debian system.
Minimal reproduction project (MRP)
No MRP yet sadly D:

Created a MRP here: https://github.com/Stalker2106x/godot-ci-test
Ran pipeline here: https://github.com/Stalker2106x/godot-ci-test/actions/runs/11363233692
Which does work. So I'm trying to understand what is exactly going on with my project.
I believe the fact that i'm using a C# Addons kills the export, I will update this thead

This is not a bug, but is interesting:
I have developed an addon which extends EditorImportPlugin
but I forgot to wrap it in
#if TOOLS
#endif

which caused godot to export everything but not the assembly.
Closing issue.


chore: Replace use of flakeheaven with Ruff for flake8-return

Related:

Closes #8457
https://github.com/meltano/meltano/issues/7383

ruff check --select=RET --add-noqa src tests

Question: Which RET error code should we consider, and which one should we put on the ignore list? For eg: RET505 to RET508 are the return type we consider more pythonic.
https://pypi.org/project/flake8-return/

Question: Which RET error code should we consider, and which one should we put on the ignore list? For eg: RET505 to RET508 are the return type we consider more pythonic.

Let me think about it. Regardless, we should probably look at the astral/Ruff docs instead.


Python: upper/lower bounded versioning

cc @bjwatson
For Python requirements generated in setup.py and requirements.txt, instead of generating
  grpc-{api}==0.{n}.{m}

let's generate
  grpc-{api}>=0.{n}.{m},<0.{n+1}.0

For major version 0, we can adopt the convention of updating the minor version for each breaking change, and the patch version for each non-breaking change.

Closed via #79


Add miniconda to macOS 13 and 14 images (again)

Tool name
miniconda
Tool license
3-clause BSD License
UPDATE (as reported by @zooba): Miniconda is covered by a licensing agreement between Microsoft and Anaconda Inc:

https://devblogs.microsoft.com/python/anaconda-licensing/
https://www.anaconda.com/partners/azure

Add or update?

[X] Add
[ ] Update

Desired version
23.11.0
Approximate size
No response
Brief description of tool
Miniconda was part of the macOS images in the past, until macOS 12, but has not been part of the macos 13 and 14 images.
It is currently part of the Ubuntu and Windows images.
Miniconda is used by a huge number of people in all industries, from Open Source maintainers (e.g. bioconda, conda-forge) to Enterprise customers, particularly in the scientific and AI communities (~40 million users according to Microsoft partner Anaconda Inc). Conda is free to use.
In https://github.com/actions/runner-images/issues/8422#issuecomment-1754613250 @vpolikarpov-akvelon replied:

We've decided not to include Miniconda in the mac OS 13 image to help reduce maintenance. We're trying to strike a balance between convenience and keeping things streamlined and Miniconda isn't a super common tool for everyone.

If you'd like to have Miniconda preinstalled, you can use custom self-hosted runners. They're a great way to tailor your environment to your specific needs.

I'm responding here in a new issue since I haven't gotten a reply to my comment in the original issue. Running custom self-hosted runners is not going to solve this problem, since conda is a self-service package manager and GitHub user would not run a self-hosted runner to make use of it. You wouldn't make the same argument for tools like homebrew or pipx.
I'm looking to get feedback from GitHub on how we can add miniconda to the macOS 13 and 14 images again. What are the constraints currently that block adding it back, and how can we overcome them?
URL for tool's homepage
https://docs.conda.io/projects/miniconda/en/latest/
Provide a basic test case to validate the tool's functionality.
conda --version

Platforms where you need the tool

[ ] Azure DevOps
[X] GitHub Actions

Runner images where you need the tool

[ ] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] macOS 11
[ ] macOS 12
[X] macOS 13
[X] macOS 13 Arm64
[X] macOS 14
[X] macOS 14 Arm64
[ ] Windows Server 2019
[ ] Windows Server 2022

Can this tool be installed during the build?
See the installation of miniconda in the Windows and Ubuntu images, as well as the macOS builds prior to macOS 13.

Tool installation time in runtime
30 seconds
Are you willing to submit a PR?
Yes

Hello @jezdez!
Due to certain technical constraints and maintenance difficulties, we have decided to streamline latest macOS images and remove some programs from them that can be installed quickly and easily at runtime or are infrequently used. The Miniconda package manager tool is also included in this list. Unfortunately, at the moment, we have no plans to reintroduce any of the removed packages, but we will certainly notify the community if plans change.
I would like to note that installing a Miniconda in runtime using the official instructions presented on the site takes less than a minute (about 20 sec to be honest) and does not require special skills or extreme complexity of the workflow code.

Is this kind of thing supported and regularly tested in actions?
After installing, initialize your newly-installed Miniconda. The following commands initialize for bash and zsh shells:

~/miniconda3/bin/conda init bash
~/miniconda3/bin/conda init zsh

I know there have been issues in the past where updating shell settings doesn't apply properly to subsequent steps, which is one of the main reasons we've had to have stuff preconfigured.

Probably want to update your documentation that still tells you that it's built in https://learn.microsoft.com/en-us/azure/devops/pipelines/ecosystems/anaconda

The Azure pipelines documentation (which links here as the image source) still states that conda is built-in: https://learn.microsoft.com/en-us/azure/devops/pipelines/ecosystems/anaconda

The explanation offered here is specious if you're not also going to drop it from the ubuntu and windows images, particularly given @ndevenish's point that the documentation still tells you $CONDA is defined for macOS. To make this all the more confusing, the macos-latest image just jumped from macos-12 to macos-14 between October 1st and October 2nd.

To make this all the more confusing, the macos-latest image just jumped from macos-12 to macos-14 between October 1st and October 2nd.

Ah! I had wondered what caused our CI to break all of a sudden. I thought it might have been an image change, but couldn't find any notices about it and thought "Wait, 14 was last year, maybe something else happened".


add training/hyperparam yaml files- not the actual data

Add coco.yaml and hyperparam.yaml to allow yolov3 scripts to run. Note that this doesn't actually add any dataset, just points the scripts to the correct download location.

Please repro the issue locally to validate the fix.

Please repro the issue locally to validate the fix.

See PR description


check if error message exists

Fixes https://github.com/probot/probot/issues/673 by checking if err.message exists and adding a test to expose this issue.

@hiimbex Is the project still maintained?

@Eddman yes. But it’s volunteer run so we can only do so much :)
Do you still run into this issue? An error instance should always have a .message property. How do you reproduce the problem? Maybe it was resolved meanwhile?

Also Adam, if possible, don’t ever ask if a volunteer run Open Source project is still maintained. It can put a lot of pressure on the maintainers. I know you don’t have bad intentions, but that’s a question we see a lot as maintainers of bigger Open Source projects and it can have the opposite effect of what you try to achieve.
Instead I would recommend to look into the recent changes in the project. There clearly have been recent updates :) If there is a bug in a project you use that nobody is taking care of, offer to help yourself. There are great talks on maintaining an open source project and what burden it can be, e.g. I recommend https://www.youtube.com/watch?v=_iToM2KC0QE by Henry Zhu.
We have weekly office hours, that’s a great place to become a maintainer yourself if you are interested

@gr2m the problem is that it does not have to be an instance of an Error. You can literaly throw any object. So it is still valid issue.

I intentionally abandoned this PR because it adds unnecessary code to Probot core. The solution is to choose to throw Error objects as opposed to other things, which was an intentional design choice. If that doesn't work for your use case, feel free to merge this code in on a fork.


feat: Unify SSO configuration under .spec.sso in backward compatible way

What type of PR is this?

Uncomment only one  /kind line, and delete the rest.
For example, > /kind bug would simply become: /kind bug

/kind bug
/kind cleanup
/kind failing-test
/kind enhancement
/kind documentation
/kind code-refactoring

What does this PR do / why we need it:
This PR contains the changes required to enable gitops-operator to consume the SSO unification changes that will go into argocd-operator
It ensures the default instance is populated with .spec.sso.provider=dex instead of .spec.dex out of the box
and also adjusts the reconciliation logic for the same
It updates the Argo CD CR to contain the new SSO fields
Have you updated the necessary documentation?

[ ] Documentation update is required by this PR.
[ ] Documentation has been updated.

Which issue(s) this PR fixes:
Fixes #?
Test acceptance criteria:

[ ] Unit Test
[ ] E2E Test

How to test changes / Special notes to the reviewer:
See instructions here https://github.com/argoproj-labs/argocd-operator/pull/646#issue-1232494732

/retest

/retest

/retest

/retest

/retest

/retest

/test v4.7-e2e

/test v4.7-e2e

/test v4.7-e2e

/lgtm
/approve


Feature enable create flow on Goerli

What does this PR do and why?
Provide a description of what this PR does. Link to any relevant GitHub issues, Notion tasks or Discord discussions.
Screenshots or screen recordings
If applicable, provide screenshots or screen recordings to demonstrate the changes.
Acceptance checklist

[ ] I have evaluated the Approval Guidelines for this PR.
[ ] I have tested this PR in all supported browsers.
[ ] I have tested this PR in dark mode and light mode (if applicable).

Current dependencies on/for this PR:

main

PR #2341   👈

This comment was auto-generated by Graphite.

Graphite Merge Job
Current status: ⏳ Queued to merge
This pull request is currently queued to be merged as part of a stack.
This comment was auto-generated by Graphite.

Job Reference: 85wgSlSoKEYO8rUIoLww


Pipper not working when apps are full screened - macOS

This is a great program and I use it every day! One thing I noticed is that in macOS the Pipper window does not show when other apps are fullscreened (or splitscreened with other apps) in new 'desktops'. Would be useful to have an option to have the Pipper window persist even when apps are fullscreened. Thanks anyways!

Hi @Fabian-AD, thanks for reaching out, it's actually very easy to implement, I think I will add another option, Stay over all spaces, it will be off by default.
Update should be coming in a few days!


RWIB Mission 5: Documentation Page

Link.
Today, the documentation landing
page lives in the Rust tree, and
is generated for each release channel. As Rust grows though, more
documentation lives out of tree and needs to be discoverable.
Let's reorganize this so that the website itself is the main landing
page for finding docs.
Start by moving the contents of the existing documentation index into
rust-www, reducing the existing doc index to links to the in-tree
docs, as well as a link to the website for more information.
Add a link to the new website-based FAQ.
Brainstorm about other documentation we might want to link to and what
standards of quality and stability we might require before officially
endorsing them.

I would like to take this on after #193.

@efindlay You got it! Thanks!

It looks like my first step should just be a pure refactor with no visible changes (except for URL). I'm guessing the tricky part will be finding and changing links to the current documentation landing page.

Set of possible links:
https://doc.rust-lang.org
https://doc.rust-lang.org/
https://doc.rust-lang.org/index.html
https://doc.rust-lang.org/nightly
https://doc.rust-lang.org/nightly/
https://doc.rust-lang.org/nightly/index.html
https://doc.rust-lang.org/stable
https://doc.rust-lang.org/stable/
https://doc.rust-lang.org/stable/index.html

Grepped with,
git grep -En 'doc.rust-lang.org[^/]'
git grep -En 'doc.rust-lang.org/[^a-z]'
git grep -En 'doc.rust-lang.org/index.html'
git grep -En 'doc.rust-lang.org/nightly[^/]'
git grep -En 'doc.rust-lang.org/nightly/[^a-z]'
git grep -En 'doc.rust-lang.org/nightly/index.html'
git grep -En 'doc.rust-lang.org/stable[^/]'
git grep -En 'doc.rust-lang.org/stable/[^a-z]'
git grep -En 'doc.rust-lang.org/stable/index.html'

Oh. Only links were part of #![doc ... html_root_url = https://doc.rust-lang.org/nightly/ which doesn't need to change, does it?

This is done.


Extrinsic parameters for the annotation tool

Hi,
I am using the online version of your annotation tool and I would like to ask how did you calibrate the extrinsic parameter for camera and lidar? I have used the dataset from Dair-V2X (https://thudair.baai.ac.cn/index) and the calibration seems not quite accurate. Could you suggest some useful tools or methods for extrinsic parameter calibration? Thank you very much.

Check out this docs:
https://docs.xtreme1.io/xtreme1-docs/product-guides/lidar-annotation-tool/coordinate-system
If you still have question, please let me know.


ModuleNotFoundError: No module named 'cryptoxlib'

i have 2 error..

ModuleNotFoundError: No module named 'cryptoxlib'

`C:\Python>pip install git+https://github.com/nardew/cryptoxlib-aio.git@master
Collecting git+https://github.com/nardew/cryptoxlib-aio.git@master
Cloning https://github.com/nardew/cryptoxlib-aio.git (to revision master) to c:\users\mero\appdata\local\temp\pip-req-build-y9j6cl7k
Running command git clone --filter=blob:none --quiet https://github.com/nardew/cryptoxlib-aio.git 'C:\Users\mero\AppData\Local\Temp\pip-req-build-y9j6cl7k'
WARNING: Did not find branch or tag 'master', assuming revision or ref.
Running command git checkout -q master
error: pathspec 'master' did not match any file(s) known to git
error: subprocess-exited-with-error
× git checkout -q master did not run successfully.
│ exit code: 1
╰─> See above for output.
note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error
× git checkout -q master did not run successfully.
│ exit code: 1
╰─> See above for output.
note: This error originates from a subprocess, and is likely not a problem with pip.`

Sorry for coming only now. The default branch was renamed from master to main, hence you need to run pip install git+https://github.com/nardew/cryptoxlib-aio.git@main. I will update the README. Let me know if it works for you.


Prevent babelrc package import failure on relative current path

Summary

Relates to https://github.com/danger/danger-js/issues/261 .
This PR attempts to fix cases of reading babelrc via package.json. Using current code path, below case is possible:
//directory is '.', relative current
const packageJsonFilePath = path.join(directory, PACKAGE_JSON);
//now packageJsonFilePath === PACKAGE_JSON, plain filename only as 'package.json'
//fs can check file exists
if (fs.existsSync(packageJsonFilePath)) {
  //node module resolution failure cause it doesn't have paths,
  //node try to resolve it as node_modules -> exception thrown
  const packageJsonFileContents = require(packageJsonFilePath);
...

I thought of using different mechanics to detect relative current (i.e path.relative(directory, '.'), path.resolve(directory, __dirname)..) but took simplest comparison instead.
Test plan
I've verified by replacing this module uses it as dependency (https://github.com/danger/danger-js/issues/261)

Codecov Report

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

@@           Coverage Diff           @@
##           master    #3723   +/-   ##
=======================================
  Coverage   62.64%   62.64%
=======================================
  Files         183      183
  Lines        6709     6709
  Branches        6        6
=======================================
  Hits         4203     4203
  Misses       2503     2503
  Partials        3        3

Continue to review full report at Codecov.

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

Thanks!


Contex loss on classes that implements indirect interfaces

Running reachable methods analysis on the following program:
public class Test {
  public static void main(String[] args) {
    I x = new C();
    x.test();
  }
}

interface I {
  public void test();
}

interface J extends I {}

class C implements J {
  public void test() {
    System.out.println("Test called");
  }
}

We would expect it to produce C.text as a reachable method but it doesn't.
Petablox properties
petablox.datalog.engine=bddbddb
petablox.main.class=Test
petablox.run.analyses=reachable-methods
petablox.err.file=/dev/stderr
petablox.out.file=/dev/stdout
petablox.jvmargs=-Xmx4096m
petablox.runtime.jvmargs=-Xmx4096m
petablox.class.path=classes
petablox.reflect.kind=dynamic
petablox.run.ids=default
petablox.reflection.timeout=300000
petablox.dynamic.timeout=300000
petablox.reflection.haltonerr=false
petablox.args.default=

And reachable methods:
<Test: void main(java.lang.String[])>
<C: void <init>()>
<java.lang.Object: void <init>()>
<java.lang.Object: void <clinit>()>
<java.lang.Object: void registerNatives()>

Hi. Thanks for the report.
I have fixed it and merged the commits into the main branch. Could you check it again?
Also I have fixed several performance and functionality bugs, so the analysis can become slightly different.

Hi Christian. Can we close this issue? Did you check it on your side?

I didn't get around to test it yet. I'm out traveling, but I will try to run it as soon as I have time. Is it merged into master?

Thanks! Yes. It is merged into master.

On the newest version of master 59180a9f898cb40086126caf8a5d1db252c689a8, I get:
Buildfile: /tmp/nix-build-petablox.drv-0/petablox-59180a9/build.xml

compile:

compile-j-part:
    [mkdir] Created dir: /tmp/nix-build-petablox.drv-0/petablox-59180a9/classes
    [javac] Compiling 467 source files to /tmp/nix-build-petablox.drv-0/petablox-59180a9/classes
    [javac] warning: [options] bootstrap class path not set in conjunction with -source 1.6
    [javac] /tmp/nix-build-petablox.drv-0/petablox-59180a9/src/petablox/analyses/invk/DomI.java:31: error: DomI is not abstract and does not override abstract method visit(JInvokeStmt) in IInvokeInstVisitor
    [javac] public class DomI extends ProgramDom<Unit> implements IInvokeInstVisitor {
    [javac]        ^
    [javac] /tmp/nix-build-petablox.drv-0/petablox-59180a9/src/petablox/analyses/inst/RelMgetClassInvkInst.java:24: error: RelMgetClassInvkInst is not abstract and does not override abstract method visit(JInvokeStmt) in IInvokeInstVisitor
    [javac] public class RelMgetClassInvkInst extends ProgramRel
    [javac]        ^
    [javac] Note: Some input files use or override a deprecated API.
    [javac] Note: Recompile with -Xlint:deprecation for details.
    [javac] Note: Some input files use unchecked or unsafe operations.
    [javac] Note: Recompile with -Xlint:unchecked for details.
    [javac] 2 errors
    [javac] 1 warning

Oh, sorry. I have fixed it. Please try it again.

Using the newest version of petablox 9fd8280b7e5503234cc303a19e2b0d5f8c282531, it still does not work. It only produces the following methods:
<Test: void main(java.lang.String[])>
<C: void <init>()>
<java.lang.Object: void <init>()>
<java.lang.Object: void <clinit>()>
<java.lang.Object: void registerNatives()>

Oh I see. I fixed one more bug.
Now I get
<test.Test: void main(java.lang.String[])>
<java.lang.Object: void <init>()>
<java.lang.Object: void <clinit>()>
<java.lang.Object: void registerNatives()>
<java.lang.System: void <clinit>()>
<java.lang.System: void registerNatives()>
<test.C: void <init>()>
<test.C: void test()>

Please check it again.

Now it works, I think we can consider this issue closed.


added data binding support

Added data binding support to avoid the expensive id look up using findViewBy

So with the binding support, you don't have manually bind it like this,
EditText name = (EditText) findViewById(R.id.edt); ?

Very interesting, to avoid confusion with the other library Butter Knife we can keep them both , we will use Butter Knife to bind handler with view if you want.

We can try to fix this. 😊. I will look into Butter knife.
On Feb 13, 2018 04:10, "Maher Soua"<EMAIL_ADDRESS>wrote:

Very interesting, to avoid confusion with the other library Butter Knife
we can keep them both , we will use Butter Knife to bind handler with view
if you want.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/Madonahs/GrowWithGoogleTeamProject/pull/10#issuecomment-365216095,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALBoGytmxnBd2dWRszRHZqZjHLhvMbnDks5tUV-ngaJpZM4SDOZJ
.

I think DataBinding support library is cleaner and we can you use it to bind data to UI as well. It is up to you guys though but I prefer we stick with core android libs as possible 😃

I agree the case that we may use it like click handler something like that
@OnClick(R.id.button_name) public void anyAction() {}
Take a look at its documentation and you can see where and when we can use it
http://jakewharton.github.io/butterknife/

Can you look at master,  I noticed some couple of errors. Not yet good with
binding. 😀 we can try to fix this.
On Feb 13, 2018 07:39, "CleverChuk"<EMAIL_ADDRESS>wrote:

I think DataBinding support library is cleaner and we can you use it to
bind data to UI as well. It is up to you guys though but I prefer we stick
with core android libs as possible 😃
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/Madonahs/GrowWithGoogleTeamProject/pull/10#issuecomment-365269082,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALBoG1FeM5IOfsO8UO9vyvdxxjUGUS18ks5tUZCKgaJpZM4SDOZJ
.

Sure

@ch thanks @MaherSoua Looks interesting.

Resolving the conflicts between my pull request and emmaunels
Adding Databinding support to onboarding activity


Problems with StorageQueue in 4.0.56-beta4

Hi.
I have just updated from 4.0.45-beta4 to 4.0.56-beta4 and are running into some compiler errors afterwards on a project running .NET Core 3.1.1
The compiler error is
C:\Users\vaxh\.nuget\packages\functionmonkey.compiler\4.0.56-beta.4\build\netstandard1.0\FunctionMonkey.Compiler.targets(29,5): error : Error compiling function: HttpAlteryxJobExecutionEventPost.cs(36,144): error CS0234: The type or namespace name 'CloudQueueMessage' does not exist in the namespace 'Microsoft.WindowsAzure.Storage.Queue' (are you missing an assembly reference?) [C:\Temp\Git\AzureFunction\Novo.Azure.Function.Ada\src\Novo.Azure.Function.Ada.Function\Novo.Azure.Function.Ada.Function.csproj] C:\Users\vaxh\.nuget\packages\functionmonkey.compiler\4.0.56-beta.4\build\netstandard1.0\FunctionMonkey.Compiler.targets(29,5): error : Error compiling function: HttpAlteryxJobExecutionEventPost.cs(208,71): error CS0234: The type or namespace name 'CloudQueueMessage' does not exist in the namespace 'Microsoft.WindowsAzure.Storage.Queue' (are you missing an assembly reference?) [C:\Temp\Git\AzureFunction\Novo.Azure.Function.Ada\src\Novo.Azure.Function.Ada.Function\Novo.Azure.Function.Ada.Function.csproj]
The code I'm trying to compile looks a bit like this
Functions(functions => functions.HttpRoute(alteryxJobCompletedRoute,         route => route.HttpFunction<AlteryxJobExecutionEventGetCommand>("/{jobname}", HttpMethod.Get).                        HttpFunction<AlteryxJobExecutionEventPostCommand>(HttpMethod.Post).                        OutputTo.StorageQueue("alteryxJobStorageQueueName")));
These are some of the nuget packages I have referenced in the project
    <PackageReference Include="FunctionMonkey" Version="4.0.56-beta.4" />     <PackageReference Include="FunctionMonkey.Compiler" Version="4.0.56-beta.4" />     <PackageReference Include="FunctionMonkey.FluentValidation" Version="4.0.56-beta.4" />     <PackageReference Include="Microsoft.Azure.Functions.Extensions" Version="1.0.0" />     <PackageReference Include="Microsoft.Azure.ServiceBus" Version="4.1.3" />     <PackageReference Include="Microsoft.Azure.Storage.Common" Version="11.1.4" />     <PackageReference Include="Microsoft.Azure.Storage.Queue" Version="11.1.4" />
I have also tried to update to latest .NET Core version (3.1.3) but with the same result.
I have tried to run dotnet build --force but with same results.
Any idea on what I might be doing wrong.
/Anders

@jeckyl2010 did you ever find out what the issue was? I've recently started using this project and first time I introduced OutputTo.StorageQueue I'm getting the same compiler exception as you.
I had a quick look at the compiler source code, and I can see BuildCandidateReferenceList (https://github.com/JamesRandall/FunctionMonkey/blob/45f4b7efc843f13a59899cdaaf470753d99bb212/Source/FunctionMonkey.Compiler.Core/Implementation/AzureFunctions/AzureFunctionsAssemblyCompiler.cs#L70) that it doesn't appear to include a reference to the Microsoft.WindowsAzure.Storage assembly in the list - so this could possibly be the problem.

@inuvano-labs Unfortunately this excellent package is more or less out-of-maintenance, so I'm no longer using it. I have taken some of the principles and carried over into my own code, but my code is very limited and only suiting me needs.
As things progresses rapidly on .NET Core 3, 5 & 6 - it will not be long before this package is too high a risk to use in corporate code, if not someone take over the gauntlet :(


clarify effect of command enablement

Tweaks the wording for enablement's description to be more clear on what will be affected. That property only affects the UI, which is not clear to extension developers from the current description.

CI failure seems unrelated.


~~Please, Add .lex file extension support on the plugin ~~

I have been working with some code samples from the web of Flex/Bison and the majority of those projects have the extension file of .lex instead of just .l which is quite annoying going file by file changing the extension form .lex to .l just for the plugin reconniced.
Thank you

@MelvinG24 Hi Melvin, sure, I can add it in the next update which is in a couple of days, cheers.

Thank you @babyraging, I thing I found were the extension file are located in the plugin and I already did my pull request
https://github.com/babyraging/yash/pull/21

@MelvinG24 should be now fixed with the new update. Feel free to reopen for any issues.


Unable to login via CLI (error: UNABLE_TO_GET_ISSUER_CERT_LOCALLY)

Describe the bug
When trying to login via CLI, I get an error UNABLE_TO_GET_ISSUER_CERT_LOCALLY.
Steps:

generate API key on redocly website
execute command npx @redocly/cli login --verbose -> prompts for my API key
paste API key from step 1
error below

  🔑 Copy your API key from https://app.redocly.com/profile and paste it below:

  Logging in...
FetchError: request to https://api.redocly.com/registry failed, reason: unable to get local issuer certificate
    at ClientRequest.<anonymous> (/Users/jamesleegovtech/dev/openapi-starter/node_modules/node-fetch/lib/index.js:1491:11)
    at ClientRequest.emit (events.js:400:28)
    at TLSSocket.socketErrorListener (_http_client.js:475:9)
    at TLSSocket.emit (events.js:400:28)
    at emitErrorNT (internal/streams/destroy.js:106:8)
    at emitErrorCloseNT (internal/streams/destroy.js:74:3)
    at processTicksAndRejections (internal/process/task_queues.js:82:21) {
  type: 'system',
  errno: 'UNABLE_TO_GET_ISSUER_CERT_LOCALLY',
  code: 'UNABLE_TO_GET_ISSUER_CERT_LOCALLY'
}
(node:10499) UnhandledPromiseRejectionWarning: Error: Authorization failed. Please check if you entered a valid API key.
    at RedoclyClient.<anonymous> (/Users/jamesleegovtech/dev/openapi-starter/node_modules/@redocly/openapi-core/lib/redocly/index.js:129:23)
    at Generator.throw (<anonymous>)
    at rejected (/Users/jamesleegovtech/dev/openapi-starter/node_modules/@redocly/openapi-core/lib/redocly/index.js:6:65)
    at processTicksAndRejections (internal/process/task_queues.js:95:5)
(Use `node --trace-warnings ...` to show where the warning was created)
(node:10499) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:10499) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Expected behavior
expected to login successfully
Additional context
Tried different solutions from stackoverflow but unsuccessful:

Turning off strict ssl: npm config set strict-ssl=false
Changing the registry to http instead of https: npm config set registry http://registry.npmjs.org/

Not sure what I should do to generate local cert?
Please help!

Can you share your node version @bookofjames
node -v

Also, what OS and version are you running?

hi Adam,
Node Version v14.18.1
Mac OS X Ventura v13.0.1

Maybe this issue can be related and you have some restrictions or custom certificate handling?

managed to login by using
export NODE_TLS_REJECT_UNAUTHORIZED=0


Cosmo conditioned denoiser

Updated the convdae model such that it can be conditioned by more than just the noise level.
Conditioning can be done now in 2 ways :

2d maps as extra channels in the input
scalars (e.g. cosmo params) that are broadcasted and concatenated to the input and bottleneck layers

Hi @nicolas-cerardi , this looks cool! Let me know when it's ready for review and I'll be happy to do it.

Thanks @b-remy ! well it's ready, it was just small changes

Indeed, but for nscalar > 1 there was a bug here:
condition*jnp.ones_like(out) cannot be performed between shapes (nbatch, 30, 30, 128) and (1, 1, 1, nscalar) with nscalar > 1.

Rigtht :-) I am merging it, thanks!


Visual Genome

Hello! Do I understand correctly that you use the crops of regions with their descriptions for training in case of Visual Genome?

Hi, we consider each region description as an image caption, we didn't use the image crop, in total we have ~5M pairs for visual genome.

Thanks


fix/issue-not-delete-temp-folder-invoking-protos

Codecov Report

Merging #16 (003b8f9) into master (2191d66) will increase coverage by 3.21%.
The diff coverage is 87.50%.

@@             Coverage Diff              @@
##             master      #16      +/-   ##
============================================
+ Coverage     88.99%   92.21%   +3.21%
- Complexity      271      283      +12
============================================
  Files            21       21
  Lines          1109     1104       -5
  Branches         77       77
============================================
+ Hits            987     1018      +31
+ Misses           96       66      -30
+ Partials         26       20       -6

Impacted Files
Coverage Δ

...java/vn/zalopay/benchmark/util/ExceptionUtils.java
62.16% <ø> (+9.88%)
:arrow_up:

...zalopay/benchmark/core/protobuf/ProtocInvoker.java
93.98% <87.50%> (+16.88%)
:arrow_up:

...main/java/vn/zalopay/benchmark/GRPCSamplerGui.java
91.89% <0.00%> (+1.68%)
:arrow_up:

...pay/benchmark/core/specification/GrpcResponse.java
100.00% <0.00%> (+14.28%)
:arrow_up:


The Password and password confirmation must match.

Package
filament/filament
Package Version
1.13.4
Laravel Version
8.78.1
Livewire Version
2.8.2
PHP Version
7.4.27
Bug description
On the create user page, the password field and confirm password are the same but an error message is generated saying: The Password and password confirmation must match.
Steps to reproduce

go to "Manage Users"
Click on "Create"
Fill de fields
Click on "Create" or "Create & Create another"

Relevant log output
No response

Hi, we've dropped bug fix support for v1. However, if you create a PR then I will merge it and tag a new release. Please upgrade to v2 :)


Split catalog itself into separate package

Maybe

Move config into package?

After discussion with colleagues at ACCESS-NRI, we decided not to do this. It is easier and cleaner to leave as is


Document change to withSSLContext method

We'll be deprecating *AuthContext.withSSLContext(SSLContext) and replacing it with *AuthContext.withSSLContext(SSLContext, X509TrustManager) so the HTTP client can have direct access to the TrustManager without resorting to a fragile reflection approach.  As always, the deprecated method will continue to work until the next major release.

Here's another change I don't see any bug filed for. We'll be deprecating HttpClientConfigurator and publishing OkHttpClientConfigurator as a replacement.  That's because HttpClientConfigurator was tied to Apache HttpClient which we no longer ship or use.  If people keep using it, nothing should break but it will never be called.  That is as we've always stated in the javadocs.  The same will happen with OkHttpClientConfigurator if we ever deprecate it.

Here are the release notes I added. Assigning to you for review, Sam.
Deprecation:

Java Client API 4.0.2 introduces the following deprecations.
The com.marklogic.client.extra.httpclient.HttpClientConfigurator interface is deprecated and will be removed in a future release. Use the new com.marklogic.client.extra.okhttpclient.OkHttpClientConfigurator interface instead. Attaching a configurator based on HttpClientConfigurator to a DatabaseClientFactory object no longer has any effect on the HTTP configuration.
The single parameter version of DatabaseClientFactory.SecurityContext.withSSLContext has been deprecated and will be removed in a future release. Instead, use the new version that requires an X509TrustManager as its second parameter. This change affects all classes that implement DatabaseClientFactory.SecurityContext, such as DatabaseClientFactory.CertificateAuthContext, DatabaseClientFactory.KerberosAuthContext, and DatabaseClientFactory.DigestAuthContext.

Incompat:

Java Client API 4.0.2 Ignores HttpClientConfigurator
As of version 4.0.2, the Java Client API uses OkHttp as its HTTP client for communicating with MarkLogic over HTTP. This change should be transparent to most applications, but imposes the following backward incompatibility on applications that customize their HTTP configuration:
Attaching a configurator based on HttpClientConfigurator to a DatabaseClientFactory object no longer has any effect on the HTTP configuration. Use the new com.marklogic.client.extra.okhttpclient.OkHttpClientConfigurator interface instead. The HttpClientConfigurator interface is deprecated and will be removed in a future release.

Those look great to me!


ghostwriter: use qt5's mkDerivation

Motivation for this change
Fixes #65685
ping @suhr for testing.
Things done
I don't see the error in mention anymore.

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @

With this change it runs fine.

Darwin failure is not a regression https://hydra.nixos.org/build/97304751.
Merging.


Update 00_intro.html

PR Checklist:

[ ] make all edits to the docs in the src directory, not in docs or in the html files
[ ] note any issues closed by this PR with closing keywords
[ ] put any other relevant information below

Nice! Seems to be working. I wonder if we should've have a better way though than a linter, like completely hide the part that should not be edit.. Our docs setup is a bit awkward and users should only see src, not docs.


Add AccountID on Bank transactions line items

This change is adding the field AccountID on the line items for GET bank transactions endpoint.

We are exposing the AccountID for the chart of account associated on the line item. It is not available for GET and not supported for PUT and POST just yet.
Types of Changes

[ ] Bug fix (non-breaking change that fixes an issue)
[x] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

@kchandola - Chris is correct.  Look at line 23030 in accounting.yaml.  That's the component you'll want to add the AccountID property to.

Hey @SerKnight @SidneyAllen  I have made the changes here. Please review.
Also @SerKnight this field is only exposed via Bank Transactions endpoint, would you be able to confirm its optional for others, like you mentioned in the comment above.
Thanks

Built and confirmed that the account_id param is serialized back with bank transactions and not required for the other objects using it.
Thank you 😺  - I will get this OAS version merged and released with a few other small changes today.


Slow simtools related to import times

Applications and unit tests are a bit slow and we need to review our way of importing modules and external packages.
Easiest way to test it with python3 -X importtime -c 'import simtools.layout.array_layout' .

Had a quick look into the import times when starting the simtools applications. This is the ranking (slowest at the bottom):
167335 | simtools.applications.db_inspect_databases
167345 | simtools.applications.db_add_model_parameters_from_repository_to_db
168765 | simtools.applications.db_add_value_from_json_to_db
169231 | simtools.applications.db_add_file_to_db
169515 | simtools.applications.simulate_prod_htcondor_generator
172347 | simtools.applications.db_get_parameter_from_db
173259 | simtools.applications.generate_default_metadata
176360 | simtools.applications.db_get_file_from_db
182133 | simtools.applications.production_generate_simulation_config
205534 | simtools.applications.submit_model_parameter_from_external
205794 | simtools.applications.generate_regular_arrays
210282 | simtools.applications.validate_file_using_schema
214927 | simtools.applications.convert_all_model_parameters_from_simtel
220035 | simtools.applications.submit_data_from_external
230099 | simtools.applications.convert_model_parameter_from_simtel
275130 | simtools.applications.db_development_tools.write_array_elements_positions_to_repository
282637 | simtools.applications.db_get_array_layouts_from_db
284077 | simtools.applications.production_scale_events
294935 | simtools.applications.convert_geo_coordinates_of_array_elements
302181 | simtools.applications.generate_array_config
366625 | simtools.applications.simulate_prod
420676 | simtools.applications.plot_array_layout
424615 | simtools.applications.derive_mirror_rnda
436728 | simtools.applications.validate_camera_efficiency
444550 | simtools.applications.validate_camera_fov
460691 | simtools.applications.validate_cumulative_psf
493091 | simtools.applications.simulate_light_emission
498366 | simtools.applications.validate_optics
510308 | simtools.applications.derive_psf_parameters
725179 | simtools.applications.calculate_trigger_rate
937551 | simtools.applications.generate_corsika_histograms
958691 | simtools.applications.generate_simtel_array_histograms

Times are in us, so the last applications have almost a 1s startup time (on my laptop).
Below some graphs. Here is the summary:

the three slowest are those importing ctapipe.io.
for all others are as expected matplotlib and astropy the dominating imports (which are hard to avoid).
one could discuss in camera.py to move the import scipy.spatial into the actual functions, as they are used only when edge pixels / FOVs are calculated (and therefore not needed for most imports); this would reduce imports of a couple of applications by 15%.

Overall, I think we can close this issue. Different PRs (linked above) solved the worst issues, the rest seems to be python pain. Opinions?

one could discuss in camera.py to move the import scipy.spatial into the actual functions, as they are used only when edge pixels / FOVs are calculated (and therefore not needed for most imports); this would reduce imports of a couple of applications by 15%.

It's also used to calculate the fill factor, but that is used only in one/two applications. So it seems like a very easy fix that can help a bit. I don't see why not.


img tag affect other modules

Hello,
Since commit #58 (4a0af4b7619015f5d7fa3562e9d203efd0567a9a) todoist module is breaking images from other modules (e.g. MMM-Wallpaper or MMM-reddit).
I think it's because of the css file in the .img section, it affects ALL images.
I'm not very familiar with MM² or JS, so i'm not able to find a clean fix for it and do a PR.
I have temporarily commented the img section in /MMM-Todoist.css

Thanks I'll take a look at it.

It's fixed. I should have more closely reviewed Pull requests.


[BugFix] Fix query rewrite based on MV failed after refresh MV

Signed-off-by: Youngwb<EMAIL_ADDRESS>What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #17066
Problem Summary(Required) ：

problem:
query rewrite failed aftre refresh mv, and can rewrite successfully a few seconds later.
reason:
In refresh mv task, we would refresh external table to get latest data, but refresh table will generate a new table which lost  relatedMaterializedViews info, we should add it after refresh table.
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[x] 2.5
[ ] 2.4
[ ] 2.3
[ ] 2.2

run starrocks_admit_test

[FE PR Coverage Check]
:heart_eyes: pass : 7 / 7 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/scheduler/PartitionBasedMaterializedViewRefreshProcessor.java
7
7
100.00%
[]

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

@Mergifyio backport branch-2.5


chore(acir): refactor bb CLI interface

Resolves #1671
Checklist:
Remove the checklist to signal you've completed it. Enable auto-merge if the PR is ready to merge.

[ ] If the pull request requires a cryptography review (e.g. cryptographic algorithm implementations) I have added the 'crypto' tag.
[ ] I have reviewed my diff in github, line by line and removed unexpected formatting changes, testing logs, or commented-out code.
[ ] Every change is related to the PR description.
[ ] I have linked this pull request to relevant issues (if any exist).

nit: We should print out byte arrays as hexstrings

I think acir is the wrong scope for this PR. bb instead?

I think acir is the wrong scope for this PR. bb instead?

Thats a good point -- I'm not sure what scope this should be, though bb is better than acir because other parts might use acir when talking about the simulator


[candle-api] Change strategy.

Codecov Report

Merging #42 (ff57038) into main (a3436b7) will decrease coverage by 0.06%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##               main      #42      +/-   ##
============================================
- Coverage     96.39%   96.32%   -0.07%
+ Complexity      509      508       -1
============================================
  Files           141      141
  Lines          1330     1332       +2
  Branches         73       73
============================================
+ Hits           1282     1283       +1
- Misses           38       39       +1
  Partials         10       10

Impacted Files
Coverage Δ

...dle/api/domain/model/strategy/OptimizeParamsSet.kt
100.00% <100.00%> (ø)

...andle/api/domain/model/strategy/TradingStrategy.kt
100.00% <100.00%> (ø)

...rader/candle/api/domain/service/StrategyService.kt
95.45% <100.00%> (ø)

...trader/common/domain/model/time/DateTimeFactory.kt
80.00% <0.00%> (-20.00%)
:arrow_down:

Continue to review full report at Codecov.

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


ARM64 Docker Image of the Rule Engine

Hello,
just wanted to ask if there are any plans to release the rule engine as an ARM64 image?
We run our actions on self-hosted runners which are ARM64 based, hence we currently can not use gitstream properly.
I saw that docker hub as a single arm based image which hints it might be possible to provide such an image :)

Hi, you can switch to V2 which doesn't use docker image anymore:
:point_right: How to Upgrade from v1 to v2:

Go to the CM repo or your dedicated repo
Navigate to the file .github/workflows/gitstream.yml
Change the line: uses: linear-b/gitstream-github-action@v1 to uses: linear-b/gitstream-github-action@v2

Sadly this will not fix the arm issue due to
The version '3.x' with architecture 'arm64' was not found for this operating system.
The list of all available versions can be found here: https://raw.githubusercontent.com/actions/python-versions/main/versions-manifest.json

This is due it the versions-manifest.json not containing any arm64 for linux.
But this might be solve by https://github.com/actions/python-versions/pull/274


Player "quick stats"

Under each player’s name it would be cool to have some basic “quick stats”.
This could exist as the following three elements:
•	Amount of accurate bids vs rounds (Example - 7 from 10)
•	Bid percentage (Example - 75%)
•	“Perfect Game” - If the player is on 100% it says “Perfect Game”. Doesn't need to say the bids vs rounds / percentage etc.

@intellibruce Isn't this what the charts are showing?

Kind of. The charts are great, but this would just display the exact figure under the name to complement the charts with a simple accurate bid amount and percentage as a faster reference.
Sally Bob
2/4 (50%)         4/4 (100%)
                   Perfect Game!

No changes to the charts required, this is just a nice to have


Joystick is frozen in Safari and Chrome on iOS 15.3 but not in Firefox [BUG]

Unable to move the joystick on iOS 15.3. The only browser that works is Firefox. The same site worked a month ago. The ones on the demo page still work.

I have same question, are you resolve this problem ?


[Enhancement] [New Feature] Make sniping better

There is a program called PokeSniper 2 that uses pokesnipers.com for rare Pokemon locations. It will teleport you to the Pokemon then enter battle and teleport back to actually catch it, (preventing you from being soft banned). After it catches it, it will move on to the next Pokemon. It would be amazing if pokesniper was apart of this bot.

It's a very nice idea, but I'm pretty sure that site will be offline due to a DMCA or similar within days.
If we could find out where it gets its data, that would be helpful. I'm guessing twitter?

+1. They have a Public API here: http://pokesnipers.com/api/v1/pokemon.json
I kinda highly doubt that they'll commit to any warnings from Niantic. So I suggest to add support for it regardless.

Seems like this would be easy to detect.

Just realized sniping is already supported: https://github.com/PokemonGoF/PokemonGo-Bot/wiki/MoveToPokemonMap-configuration#snipe

It is, but only in the radius the map bot searches. Very small chance to have 1 very rare pokemon, let alone multiple, of course.
A crowdsourced list of pokemon and coordinates would be best for this purpose, but it's very easy for niantic to detect this and ban your account.

Please move your feature request/s to our feature hub - http://feathub.com/PokemonGoF/PokemonGo-Bot

https://github.com/PokemonGoF/PokemonGo-Bot/issues/3843#issuecomment-239708680

Sniping is almost certainly going you banned now.


[Bug] Search Bar not responsive in Accordion page

Description
The search bar is overflowing out of the main container on the Accordion page which is creating unresponsive website on small devices
Screenshots

Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[X] I am a SWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

I suppose this issue is not just in Accordions page, it is for every category pages @pb0126

I'll fix this for every page

Go ahead @pb0126


Question regarding GFF file generation

Hey, thanks for this nice pipeline!
I'm running the pipeline with the --list argument providing a sample sheet to it to run on  ~100 samples. But for all the viral sequences that are identified by the pipeline I only get one GFF file in the output directory of the first sample from the sample sheet. I think this is cause by this  line of code and I was just wondering what the reasoning behind taking only runnning the GFF script on the first contig file as a reference sequence is.
I get that in the case of running the pipeline on a single sample this doesn't make a difference, but when dealing with multiple samples wouldn't it make sense to generate a GFF file for every sample where viral sequences have been found? Just wondering if I'm missing something here!
Thanks in advance!

Hey @fischer-hub
That looks like a bug to me, we usually run the pipeline one assembly at the time. So I think it's just an unhappy coincidence. I'll change it and will test the pipeline.
Cheers

Thanks again!


Return false if OTP value is nil, instead of ROTP exception

Refs #63, Refs #27.

Ah, I should clarify this addresses the instance you directly access the validate_and_consume_otp! method which is useful in some API designs and 2FA enrolment. Specifically, I require my users to verify they got their 2FA secret onto their phone by providing the current OTP before toggling otp_required_for_login to true

Looks good to me. @ShaneWilton?


Correct pure link regex to reject invalid characters

Regex is using character range +-_ which would match any character between ascii character range of 43 (+) to 65 (_), which includes invalid characters like < and >
ref: https://www.rfc-editor.org/rfc/rfc3986#section-2
Fixes #90

very well spotted, thank you very much

Pull Request Test Coverage Report for Build ccc818ba1f2367693ea50fc498134ed65e648d6f-PR-91

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+2.4%) to 95.908%

Totals

Change from base Build de2216ca0622a9d2491ea2295d0be1bedcaf64c6:
2.4%

Covered Lines:
750

Relevant Lines:
782

💛  - Coveralls

Pull Request Test Coverage Report for Build ccc818ba1f2367693ea50fc498134ed65e648d6f-PR-91

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+2.4%) to 95.908%

Totals

Change from base Build de2216ca0622a9d2491ea2295d0be1bedcaf64c6:
2.4%

Covered Lines:
750

Relevant Lines:
782

💛  - Coveralls

Pull Request Test Coverage Report for Build ccc818ba1f2367693ea50fc498134ed65e648d6f-PR-91
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
8 unchanged lines in 6 files lost coverage.
Overall coverage increased (+0.6%) to 96.518%

Files with Coverage Reduction
New Missed Lines
%

lib/earmark_parser/ast/renderer/ast_walker.ex
1
83.33%

lib/earmark_parser/helpers/lookahead_helpers.ex
1
94.12%

lib/earmark_parser/helpers/string_helpers.ex
1
75.0%

lib/earmark_parser/parser.ex
1
89.38%

lib/earmark_parser/parser/list_parser.ex
1
91.18%

lib/earmark_parser/helpers/attr_parser.ex
3
87.1%

Totals

Change from base Build de2216ca0622a9d2491ea2295d0be1bedcaf64c6:
0.6%

Covered Lines:
887

Relevant Lines:
919

💛  - Coveralls