Service account without token

Hi,
Something that caught my eye is that this setup does not create a token for the default service account. Have you experienced the same problem? and if so, how did you fix it?
I'm trying to connect a pod to the API master without going through the Insecure option.
Cheers,
Leo.-

It seems that this is a bug on Kubernetes.
https://github.com/kubernetes/kubernetes/issues/19980


Add Explicit Conditions

They allow to express more complex conditions that will be evaluated at the server side. Since they include the field selection in the current conditionFromModel, reuse this way of creating conditions, e.g:
// No conditions falls back to equality, index-based conditions
api.Where(api.ConditionFromModel(&LogicalSwitch{Name:"foo"})).Update(...)

// Now we accept explicit conditions
ls := &LogicalSwitch{}
api.Where(api.ConditionFromModel(ls, Condition{
    Field: &ls.Enabled,
    Function: ovsdb.ConditionEqual,
    Value: []bool{true},
    })).Update(...)

Any number of conditions can be created to select multiple rows
Fixes: #99

Pull Request Test Coverage Report for Build 848968960

87 of 112 (77.68%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.5%) to 73.496%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

client/condition.go
37
41
90.24%

ovsdb/bindings.go
13
17
76.47%

client/api.go
17
22
77.27%

client/orm.go
16
28
57.14%

Totals

Change from base Build 842678591:
0.5%

Covered Lines:
1564

Relevant Lines:
2128

💛  - Coveralls


Support for default IDP using Keycloak as Identity Broker

Can you add support to set extra parameters like kc_idp_hint used by keycloak to choose the default brokered idp?
Here is the documentation: https://www.keycloak.org/docs/latest/server_admin/#default_identity_provider
Thanks

I created an online version of this tool. I added features that I haven't ported back to this repo. Check out [1]. It has the ability to add custom parameters to the calls for the authorization endpoint and token endpoints. Does that let you do what you need to?
Reference:
[1] https://tools.test.idptools.io/debugger.html


Status of tests for bit manip extension ?

Hi,
I had a look at some tests for the bit manipulation instructions and I found something strange.
For example, for the instruction andn, (defined as  X(rd) = X(rs1) & ~X(rs2)) , I can find the following test
inst_0:
// rs1 == rs2 == rd, rs1==x31, rs2==x31, rd==x31,
// opcode: andn ; op1:x31; op2:x31; dest:x31; op1val:0xffffffff;  op2val:0xffffffff
TEST_RR_OP(andn, x31, x31, x31, 0xffffffff, 0xffffffff, 0xffffffff, x1, 0, x2)

(file riscv-test-suite/rv32i_m/B/src)/andn-01.S , line 44)
If I follow the definition of the TEST_RR_OP macro (arch_test.h)
#define TEST_RR_OP(inst, destreg, reg1, reg2, correctval, val1, val2, swreg, offset, testreg)
the result is expected to be 0xffffffff, with rs1 = 0xffffffff and rs2 = 0xffffffff
but 0xffffffff & ~0xffffffff  is obviously not 0xffffffff
We found several strange "errors" like that for bit manipulation instructions so we are wondering if those tests are actually "valid" or just simple placeholders for future development

The expected results are place holders and should be ignored. The tests themselves are correct. Correct results should be obtained by running them on SAIL.

Does this apply only to the Bit Manipulation extension tests ? We have been using rv32i_m/C and rv32i_m/I for some designs for a while without the need to generate correct result with SAIL

Does this apply only to the Bit Manipulation extension tests ?
No.
All tests need to be used with the SAIL model. Although some of the values in the tests may be correct. They should not be treated as golden and they are not a replacement for generating values from SAIL. They are merely a feature to aid in debugging in the future.

Closed as comments now reflect are just placeholder., even if some value is passed in the macros (but is unused)
Closing this


Show code actions for vector descipe there's no actual effect

Reported by missinggo

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Fix #10468] Fix a false positive for Style/FileWrite

Fixes #10468
This PR fixes a false positive for Style/FileWrite when a splat argument is passed to f.write.
There are different method signatures between File.write (class method) and File#write (instance method).

https://ruby-doc.org/core-3.1.1/IO.html#method-c-write
https://ruby-doc.org/core-3.1.1/IO.html#method-i-write

This PR accepts the case of using splat argument as an argument for File#write because static analysis does not know the contents of the splat argument.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Looks good, but please reflect this in the cop docs as well with some example.

Yeah, I've update the doc for this cop.


Null safety migration

Is there any change this library getting null safety support?

Check https://github.com/rodydavis/flutter_vibrate/pull/13

+1


text mouse scroll horizontally

drag text with mouse, move one row at a time for maximum newline reading efficiency


Add prow e2e script

Force-merging this. The Circle-CI tests are failing due to the Go version dependency. This PR will allow removing Circle CI and fixing those tests.

/cherrypick release-1.3

@rlenglet: new pull request created: #151

In response to this:

/cherrypick release-1.3

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


handling of sequences with more than 30 amino acids

The current code as defined in prosit/prosit/constants.py only supports sequences with up to 30 amino acids.
A total of 81194 sequences exist in the human fast file. Only 703 sequences are smaller than 30 amino acids in the human fast file. 80491 sequences are larger than 30 amino acids. Basically 99.13% of the sequences in humans are not covered with the current prosit code version.
The error handling could be improved by stating that "only sequences with up to 30 amino acids are supported in prosit" instead of  the rather cryptic error message "index 30 is out of bounds for axis 1".
Maybe it is even possible to just increase the sequence length in constants.py to 34475 letters, which is the longest sequence in the human fasta file? Not sure how that influences predictive accuracy or if that is possible.
Basically the current error message is not very descriptive for practitioners.
Traceback (most recent call last):
  File "jump.py", line 26, in <module>
    tensor = tensorize.peptidelist(df)
  File "/root/prosit/tensorize.py", line 56, in peptidelist
    "sequence_integer": get_sequence_integer(df.modified_sequence),
  File "/root/prosit/tensorize.py", line 42, in get_sequence_integer
    array[i, j] = constants.ALPHABET[s]
IndexError: index 30 is out of bounds for axis 1 with size 30

Example sequences with 30 and 30 letters
NGGRDYALRGPEHPGSGGAPEPQGWIHFI
MNERREQLRAKRPEVPFPEITRMLGNEWS
DKRQNSSRFSASNNRELQKLPSLKGPPTL
MVKLTAELIEQAAQYTNAVRDRELDLRVS
MYVAESTRKTLLYHMEFSELTSRYIKIIN
MDFVAGAIGGVCGVAVGYPLDTVKGLLALP
MTGELEVKNMDMKPGSTLKITGSIADGTDG
MEPEFLYDLLQLPKGVEPPAEEELSKGVCP
MEDALFLRKSPPYIFSPIPFLGHAIAFGKS
MSDILRELLCVSEKAANIARACRQQEALFQ

IndexError: index 30 is out of bounds for axis 1 with size 30
An index of 30 indicates a amino acids with at least 31 amino acids (zero-based counting).
The reason why we do not include longer amino acids are addressed in detail in our paper. In short, we want to rely on high-quality measurements of synthetic peptides, and there is not enough data to meaningfully train a model.

@gessulat
ok, so creating smaller sequences via in-silico digestion.
Do you have a tool for in-silico digestion that you can recommend, like peptidecutter?
So how do I take a fasta file and prep it for prosit? Online?
I feel there is a lot of information missing in the readme or project WIKI.

Internally I am using pyteomics FASTA digest and other folks use available R libraries. We are working on including a FASTA digest option at our online tool, but the release date is not clear yet. See www.proteomicsdb.org/prosit/
If you are missing any specific information in the readme or wiki please feel free to contribute.


Update Binary usage to reflect GHC changes

Updates to support the changes found in https://gitlab.haskell.org/ghc/ghc/merge_requests/1960/

Hi!  As of today, patches for the ghc-head branch must be done on the Haddock mirror on GitLab, at https://gitlab.haskell.org/ghc/haddock.


Correct keyserver path

While attempting to install SIFT on a fresh install of Ubuntu 16.04.02 under VirtualBox 5.1.18 r114002 it was discovered that the keyserver for the docker repo was unavailable.  Specifically this error message is always returned when sudo salt-call --local --file-root=/tmp/salt state.sls sift is run:
[ERROR   ] Command '['apt-key', 'adv', '--keyserver', 'p80.pool.sks-keyservers.net', '--logger-fd', '1', '--recv-keys', '58118E89F3A912897C070ADBF76221572C52609D']' failed with return code: 2
[ERROR   ] stdout: Executing: /tmp/tmp.7bQXCDbzsA/gpg.1.sh --keyserver
p80.pool.sks-keyservers.net
--logger-fd
1
--recv-keys
58118E89F3A912897C070ADBF76221572C52609D
gpg: requesting key 2C52609D from hkp server p80.pool.sks-keyservers.net
gpg: keyserver timed out
gpg: keyserver receive failed: keyserver error
[ERROR   ] retcode: 2
[ERROR   ] Failed to configure repo 'deb https://apt.dockerproject.org/repo ubuntu-xenial main': Error: key retrieval failed: Executing: /tmp/tmp.7bQXCDbzsA/gpg.1.sh --keyserver
p80.pool.sks-keyservers.net
--logger-fd
1
--recv-keys
58118E89F3A912897C070ADBF76221572C52609D
gpg: requesting key 2C52609D from hkp server p80.pool.sks-keyservers.net
gpg: keyserver timed out
gpg: keyserver receive failed: keyserver error

This appears to be an error with the keyserver path in sift-saltstack/sift/repos/docker.sls
...
    - keyid: 58118E89F3A912897C070ADBF76221572C52609D
    - keyserver: p80.pool.sks-keyservers.net
    - refresh_db: true
...

Fix was to use the recommended path provided by Saltstack Formulas and change as follows
...
    - keyid: 58118E89F3A912897C070ADBF76221572C52609D
    - keyserver: hkp://p80.pool.sks-keyservers.net:80
    - refresh_db: true
...

Closes #145

What version of saltstack is installed?
I'm not opposed to merging this as it seems to work, but the current version works on all my test VMs including a clean one I just built.

Thank you for the contribution!

I'm using Salt: 2016.11.3
Initially I thought it might be a corporate network issue because the default port for the key exchange is weird (port 11371), but it failed on my test network too.  I'll try and dig into understanding why it failed a little better.  Thanks.


Not working disabled metadata

I have setted showMetadata = false in config.toml but it doesn't work.
example
[params]
user = ""
hostname = "renanbastos.dev"
sitedescription = 'Renan Bastos has been working as a Software Engineer for over 9 years with cutting-edge technologies'
showMetadata = false

I see. So basically, if you set it to false it becomes false | true which is evaluates to true

Wait actually the disabling works as expected but it has to be changed individually in each post frontmatter. Globally turning off post metadata is not yet implemented. I'd welcome a PR for this. I'll keep this issue open but I'll close #41

What do you expect to do?

The if statement currently checks if .Params.showMetadata is set to false. .Params is set in post frontmatter. To be able to disable post metadata globally it also needs to check .Site.Params.showMetadata. If either of those is false the metadata on that post won't be shown


Perception: fix the issue perception can't work

Hi, does this mean that the detectionComponent is a replacement or an upgrade to the previous SegmentationComponent ?  thanks.

Hi, does this mean that the detectionComponent is a replacement or an upgrade to the previous SegmentationComponent ? thanks.

@ntutangyun Yes, segmentation component has been removed. Perception module is using detection component, which is based on PointPillars method, for 3d object detection now.


BuiltinKindsEntityProcessor error

After upgrading the related frontend and backend package about catalog and scaffolder, I got this error.  And I didn't import any about API catalog, I just imported the kind is Component

Hi @chuliang014 ! Looks like you are ingesting an entity via app-config.yaml using the catalog.locations property. And you have a rule only allowing API kinds.
Can you make sure that you have an allow property like this[1] in your catalog.rules config, which allows kinds other than API?
https://github.com/backstage/backstage/blob/a3ec122170e0205fd3f9c307b98b1c5e4f55bf5f/app-config.yaml#L184-L195

Hey @OrkoHunter. I've allowed these values.

@OrkoHunter fixed the issue after delete node_modules and yarn install

Hi, I am having the same issue, trying to get a component that dependsOn: another component and the app-config is good:
catalog:   import:     entityFilename: catalog-info.yaml     pullRequestBranchName: backstage-integration   rules:     - allow: [Component, System, API, Resource, Location]
tried the deletion of the node_modules folder followed by reinstalling yarn, same happens..


It doesn't catch screenState on Ubuntu

It doesn't run this.handle() when I lock my screen
if (userState === UserIdleState.idle || screenState === ScreenIdleState.locked) {
    this.handleIdle();
}

OS: Ubuntu 20.04.4 LTS
Did you test it?

@dmythro

Sure, I tested it on recent macOS and Chrome at the time of publishing a module (as we used it on a project but then switched to non-isomorph version). Native API was still a draft, so probably something has changed, need to re-check everything now and add some test coverage — I'll see how to make it stable, meanwhile I'd suggest switch to non-isomorph one.

Hey @HyoilKim could you debug this on Ubuntu now? Is it working after you give permissions? Chrome/Firefox. And let me know browser versions, if 'IdleDetector' in window etc.
The screenState feature detection is only available when IdleDetector API is there (isIdleDetectorSupported() or 'IdleDetector' in window).


MuJoCo Rendering Code

Hello,
Thanks for making this resource public! Would you mind pointing me to the rendering code that was used for MuJoCo?
Cheers

Hi! Our codes used in the rendering tests are in the code branch. You can find them here. :)

Hi @YuyangLee, it looks like you are including the physics step in the rendering time computation for MuJoCo but not doing so for other simulators. That seems wrong unless I am misunderstanding the code.

Obviously the sim time is cheap here but I think the benchmark should minimize differences as much as possible between simulators!

Yes, we will update the code and the reported fps. Thank you so much for the information.

Already updated, thank you again!


llm_api_endpoint

I set the url, but there is a bug, what the correct llm_api_endpoint should be like

In your case, could you test the endpoint api.openai.com?

yes, i already test the api_key, which can normally return

I mean the llm_api_endpoint should be set to "api.openai.com" instead of "https://api.openai.com/v1/chat/completions".

It didn`t work

Could I have the detailed settings and the terminal response.

ubuntu 22.04
python 3.11
openai 1.25.0
model gpt-3.5-turbo

It is weird, I can run it with Openai API using the same endpoint setting and model.
I suggest the following solutions:

double check the network and VPN (if any), sometimes a proxy leads to network errors.
use other APIs.

thanks


Local Rancher server does not respect proxy variables HTTP_PROXY and HTTPS_PROXY

What kind of request is this (question/bug/enhancement/feature request): bug
Steps to reproduce (least amount of steps as possible):

Start rancher server locally with the HTTP_PROXY and HTTPS_PROXY environment variables set

Result
Start up (1) reaches out to https://releases.rancher.com/ui/latest2/index.html and (2) ignores HTTP Proxy environment variables HTTP_PROXY and HTTPS_PROXY causing the request and UI to fail to load:
2019/07/16 14:41:40 [ERROR] failed to serve UI: Get https://releases.rancher.com/ui/latest2/index.html: dial tcp <IP_ADDRESS>:443: connect: network is unreachable

Possibly related: #20709

This is specific to fetching the UI when running locally and not related to the jail stuff in https://github.com/rancher/rancher/issues/20709


TaskExit event should not fire at init process close but wait until container is closed

Tracking this here so doesn't get forgotten. @jterry75 FYI

This was fixed.


angular.merge loses type information

Assuming I have the following code:
  function Cat(name) {
    this._name = name;
  }

  var myFatCat = new Cat("Garfield");
  var myKitten = new Cat("Nermal");

  if(myFatCat instanceof Cat){
    console.log("I'm a cat!");
  } else {
    console.log("I'm not a cat :(");
  }

  var myCats = {
    cat1: myFatCat,
    cat2: myKitten
  };

  var myPets = angular.merge({}, myCats);

  if(myPets.cat1 instanceof Cat){
    console.log("I'm a cat!");
  } else {
    console.log("I'm not a cat :(");
  }

The type information of the 'deep' object is lost.

This is expected. The merge function just copies enumerable properties across from the source object to the destination object. It is outside the scope of this function to be able to make deep clones that have the same type information.

Agree with @petebacondarwin it is outside the scope of angular.merge to copy type information


A warning causes build to fail, but still says "0 Errors"

Steps to reproduce
Use the attached solution (.NET 4.0 Console app) - MsBuildComIssue_v1.zip
Or, create it manually:

Create a .NET project (I'm using WPF) with reference to a specific COM object MSHTML- "00020430-0000-0000-c000-000000000046" version 2.0.

make sure you are using a machine without Microsoft Office!

use msbuild to build the project.

In the log you can see that it will use the following "csc" command:
C:\Program Files (x86)\Microsoft Visual Studio\<yourVersion>\MSBuild\<yourVersion>\Bin\Roslyn\csc.exe
	/noconfig
	/nowarn:1701,1702
	/nostdlib+
	/errorreport:prompt
	/warn:4
	/define:TRACE
	/highentropyva-
	/reference:"C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.0\Microsoft.CSharp.dll"
	/reference:"C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.0\mscorlib.dll"
	/reference:"C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.0\PresentationCore.dll"
	/reference:"C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.0\PresentationFramework.dll"
	/reference:"C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.0\System.Core.dll"
	/reference:"C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.0\System.Data.DataSetExtensions.dll"
	/reference:"C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.0\System.Data.dll"
	/reference:"C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.0\System.dll"
	/reference:"C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.0\System.Windows.Forms.dll"
	/reference:"C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.0\System.Xaml.dll"
	/reference:"C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.0\System.Xml.dll"
	/reference:"C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.0\System.Xml.Linq.dll"
	/reference:"C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.0\WindowsBase.dll"
	/debug:pdbonly
	/filealign:512
	/optimize+
	/out:obj\Release\myProject.dll
	/ruleset:"C:\Program Files (x86)\Microsoft Visual Studio\2017\Professional\Team Tools\Static Analysis Tools\\Rule Sets\MinimumRecommendedRules.ruleset"
	/resource:obj\Release\myProject.g.resources
	/target:library
	/utf8output
	/deterministic+
	yourFile.cs

Expected behavior
build should either fail or succeed - depending on if you have the COM object registered.
Actual behavior

C:...\Bin\Microsoft.Common.CurrentVersion.targets(2729,5): warning MSB3283: Cannot find wrapper assembly for type library "MSHTML". Verify that (1) the COM component is registered correctly and (2) your target platform is the same as the bitness of the COM component. For example, if the COM component is 32-bit, your target platform must not be 64-bit. [C:\Proj.csproj]
Done executing task "ResolveComReference" -- FAILED

The build will fail with exit code 1!
but you'll still see "0 Error(s)"
Environment data
tested on both:
msbuild /version output:
Microsoft (R) Build Engine version 16.4.0+e901037fe for .NET Framework
Copyright (C) Microsoft Corporation. All rights reserved.

<IP_ADDRESS>107

And
Microsoft (R) Build Engine version 15.9.21+g9802d43bc3 for .NET Framework
Copyright (C) Microsoft Corporation. All rights reserved.
<IP_ADDRESS>4

OS info:

Windows Server 2016 1607 14393.3443

Side note:
As I suspected it's an issue with the content of the GAC, here's a list of the content:
gacutil-list.txt

Issue still present in VS2019 build tools:
Microsoft (R) Build Engine version 16.5.0+d4cbfca49 for .NET Framework
Copyright (C) Microsoft Corporation. All rights reserved.
<IP_ADDRESS>03

Microsoft.Build, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a, processorArchitecture=MSIL
Microsoft.Build.Conversion.v4.0, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a, processorArchitecture=MSIL
Microsoft.Build.Engine, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a, processorArchitecture=MSIL
Microsoft.Build.Framework, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a, processorArchitecture=MSIL
Microsoft.Build.Tasks.v4.0, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a, processorArchitecture=MSIL
Microsoft.Build.Utilities.v4.0, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a, processorArchitecture=MSIL

Try removing those and see what happens. Having our assemblies in the GAC can cause confusing version mismatches. We aren't entirely sure how they get there so often, but we're working on removing them:
https://github.com/dotnet/msbuild/pull/5626

I'm sorry, but I didn't got approval to remove these DLLs from GAC.
To bypass the issue our IT have copied MSHTML assemblies into the GAC.

I've tried to compile the same project on a clean machine with VS2019 16.7.2 and got the following result:

so I suppose that the issue is fixed.


Staircase on hackerrank

Problem Link
https://www.hackerrank.com/challenges/staircase/problem
Coding platform
Hackerrank
Programming language

[ ] C
[ ] C++
[ ] Java
[ ] Python

I want to do it in C++

@Nivedita967 I want to solve this in Java.

I can do this in C

I will pull the  request after the above pr gets merged

please assign it to me ,I want to work on this issue in python.

@JASKAMAL22 Go for it.

@JASKAMAL22 Are you working ?

If he refuses,
I can provide code in Python too


[Android]sdk更换到29，项目换成了AndroidX的包，然后weex在Android10系统上加渲染失败，报degradeToH5|createInstance fail|wx_create_instance_error isJSFrameworkInit==false reInitCount == 1错误

Describe the bug

To Reproduce

Environment

Expected behavior

Screenshots

Additional context

感谢你的反馈，Weex目前是隶属于Apache Software Foundation的项目，因此在Gtihub渠道我们鼓励使用英文反馈问题。下面我将使用Google Trasnlate将你的问题翻译成英语，并以英语回答。

Question:  SDK changed to 29, project changed to AndroidX package, then weex failed to add rendering on Android 10

Weex doesn't support Android X yet.
We're focusing on fixing branding issues recently, and I am afraid, we will not support Android X.
However, you're welcomed to create a PR for supporting Android X. I am happy to discuss design detail with you in mailing list.

怎么解决了吗？怎么解决的？


Hello, does this model heavily rely on the pre existing face detection model? If the face detection model is not very robust, will there be any issues with the model? I found during the experiment that the effectiveness of face detection greatly affects the effectiveness of the facial keypoint localization model. Looking forward to your reply

Yes, I think the detection model will affect the landmark model. In short, you need to make sure the landmark model is trained with images that cropped the same way as the detection model you use for inference. And differende detection model may have different bounding box styles, e.g., some give you more boundaries and some are quite tight, etc.


Integration into Compiler Explorer

I'd be delighted to help integrate directly into Compiler Explorer, if that's something you'd consider useful? (see also https://github.com/mattgodbolt/compiler-explorer/issues/1203 )

That sounds like a great idea!

Let us make it happen (in 2019).

That's a neat idea!
I'll let you know when the application is in a more production ready state. As @lemire said, it will probably happen somewhere in 2019.

Oh. I think it will definitively happen in 2019.


[FLINK-28195][python] Fix the build wheels

What is the purpose of the change
This pull request will fix the build wheels
Verifying this change
This change added tests and can be verified as follows:

Trigger in private Azure

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

https://dev.azure.com/hxbks2ks/FLINK-TEST/_build/results?buildId=1862&view=logs&j=79728727-bed4-5185-8f2e-b43a119f2877 build wheels


Glitch With the Rudders

The rudders can be hooked up and all, and they move with the helm when setup correctly, however they don't actually steer the ship at all, and only if you reach very high speeds can you get an (extremely delayed) slight turn. This feature was working in versions of VS prior to the mod splitting into 3 parts.

Can you name the particular version it's been working in?
Also, it'd be really cool if you could make a vid or similar to demonstrate the difference.


Website is down...

Thanks for reporting! We're investigating the root cause, looks like it might be something upstream with our static site host.
Will update here and on status.simpla.io
Sorry! Will get back up and running ASAP. And I have confirmed that this outage is just for the Simpla.io website, projects and the API are unaffected

This has been resolved upstream by our hosting provider - sorry for the annoyance! Can you check https://www.simpla.io again?

Worked like a charm 3-4 hours after I first reported the incident. Sorry for the late reply @seaneking :)


Enable repository filtering

Users can configure the MavenRepositoryContentDescriptor to only include certain packages from reporitory.

I played around with this change and I could not make it work with both Groovy and Kotlin DSLs (the proposed change works only with Kotlin).
I am not confident that a serializable action as an extension property is the correct approach to solving this problem given that Gradle has its SerializableAction in its internal API org.gradle.api.internal.lambdas.SerializableLambdas. While it does work (somewhat, could not make it work with Groovy) at the moment if we mark an action as serializable, I fear it may break in the future.
I tried overcoming this using a nested property for MavenRepositoryContentDescriptor as documented here (see SiteExtension.java) and an action to configure it, but this can't work because MavenRepositoryContentDescriptor itself is a complex type that can not be serialized.
Eventually, I cherry picked the commits from this change and came up with an alternative that also allows repository filtering, but does so using Gradle's public repository APIs without adding anything to the extension: https://github.com/boundrates/codeartifact-gradle-plugin/pull/23
This way, the extension remains focused just on configuring the CodeArtifact aspect. And anything else can be configured using the facilities that Gradle already provides.


Multi-level is not loadind in case of pop-ups

Multi-level values not loading in case the work item opens in a dialog/pop-up, not in a new tab.
This happened in both:

Create a new work item
Update an existing work item

Hi, Can you please provide an example with a screenshot?


Ghost certificate is being sent for clients without SNI

I was doing a ssllabs test and I found this strange certificate:

I don't know why it's being sent at all. I'm using jwilder/nginx-proxy:alpine with this settings:
VIRTUAL_HOST: domain1 domain2 domain3
VIRTUAL_PORT: 80
LETSENCRYPT_HOST: domain1 domain2 domain3
LETSENCRYPT_EMAIL<EMAIL_ADDRESS>

Can I delete default.crt and default.pem safely to disable this behavior? I don't care about non-SNI clients.

No, they'll get recreated when the container restarts.
If you don't have default certs, non SNI clients (or client trying to access the server by IP) will get a wrong certificate anyway, the certificate of the first proxyed service in the nginx config file if I recall correctly.
What behaviour would you like to see ? Again the default cert is only served to non SNI client, or client reaching the server through IP or non configured domain, if you don't care about them I'm not sure I understand the issue.

I'd like to never display a warning with my URL at all. I don't know what happens when a non SNI client tries to access a SNI server only. I suppose there will be some kind of warning telling them to upgrade or something.
If not, the ideal solution would be to display a blank page in https (maybe with a different subdomain... something like error.domain.com) telling them to upgrade their client if they want to visit the website.
HTTP could work, but all my domains have HSTS so that's not a option. I don't understand why you can't at least have the default certs being a subdomain with a valid cert at all?
If someone access my website with a non-SNI client, I don't want to scare them away. Do you think the average user will understand that it's their fault and not my website?
Also if that website can see I'm serving invalid cert, search engines can too. Will they find that suspicious? I don't know. Why would a domain be serving a valid & invalid certificate?

I don't know what happens when a non SNI client tries to access a SNI server only.

It will most probably reach another site on the same server than the one it's to trying to connect to and will get a certificate mismatch warning.

I suppose there will be some kind of warning telling them to upgrade or something.

Nope. The obsolete client isn't aware of the reason why it is getting a mismatched certificate.

If not, the ideal solution would be to display a blank page in https (maybe with a different subdomain... something like error.domain.com) telling them to upgrade their client if they want to visit the website.

I think you might be able to do this with nginx-proxy default domain option + a custom default certificate. Not sure though. Check the docs.

I don't understand why you can't at least have the default certs being a subdomain with a valid cert at all?

You are free to symlink the default cert and key to an existing cert and private key. As told previously the self signed default certificate only gets created when you don't provide your own default cert and key.

If someone access my website with a non-SNI client, I don't want to scare them away. Do you think the average user will understand that it's their fault and not my website?
Also if that website can see I'm serving invalid cert, search engines can too. Will they find that suspicious? I don't know. Why would a domain be serving a valid & invalid certificate?

This is how name based shared hosting has been working for years if not decades all over the world. Does not seem to be an issue at all in the long run.

I understand. I'll try stuff and report back here if you wish. Thanks

@Nottt news on this ?

I know this is a 'wontfix' issue, but it's probably not the ideal way to do this. It would provide anyone scanning the internet at large with the capability to fingerprint the architecture of the system. For example, now that I'm running this setup, I can tell that anyone running a certificate with the name: letsencrypt-nginx-proxy-companion as the hostname is probably running the two containers that are listed here.
Maybe there is a way to obfuscate the naming and the mechanism used to deliver this certificate to obfuscate this further so it's not so obvious?

@mosesrenegade Yep, I'm aware of that but I did not come up with a better solution yet (one that would both enable the container to recognize if the default certificate / key are its own self generate ones and defeat fingerprinting).
My thought was that a certificate requires a subject name and that subject name is going to be exposed to the outside world no matter what. Going from there I did not see a way to make the subject name not opening up the door to fingerprinting. Even a random subject name would give it away because the way we would implement that would probably be unique enough to be fingerprinted.
I'm open to suggestions and/or PRs.


Fix CI

@defnull I removed Python 3.2 version because they break the build.
Also, this version of Python is obsolete and it should cause people to think bottle in it's
current master branch is not working with their current Python (I assume most people
will not have Python 3.2).

I'm testing 3.2 in a non-public CI anyway, so dropping the tests from public CI is totally fine. Thanks :)


Issue #2170 : Use path for all internal file IO

This changes api, that would mean more process. Let's talk about this

Do we want to revisit this PR? Has been hanging there for long

Right now due to deficiencies in the java.nio.files API any migration to the new API would be incomplete and kind of pointless.  Closing this PR, but I'll keep the branch just in case someone wants to revisit this in the future.


Update get-started.md

Typos "...compeition ewith differentiated.."  fix to "...competition with differentiated..."

@sfrechette : Thanks for your contribution to the Azure documentation! The author, @guyinacube, has been notified to review your proposed change.

@sfrechette Thank you for catching and fixing that!
@kfile
#sign-off

@JennieHubbard: I'm sorry - only the author of this article, @guyinacube, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


Using jsonb_agg/jsonb_build_object to parse to inner structs

Hi,
Whenever I try to get (select/scan) the Groups (outer struct) along with their Collaborators (inner structs), I'm getting the following error :
// sql: Scan error on column index ..., name "collaborators": unsupported Scan, storing driver.Value type []uint8 into type *[]User
the code to fetch from db is :
func (psql *Postgres) GetGroups(someParam string) ([]Group, error) {
   groups := []Group{}
   err := psql.db.Unsafe().Select(&groups, <the query ...>, someParam)
   ....
}

type Postgres struct {
    db      *sqlx.DB
    config  *config.PostgresDB
    timeout time.Duration
}

This is the SQL query :
SELECT groups.id,
       groups.title,
       JSONB_AGG(JSONB_BUILD_OBJECT(
		'id', u.id,
		'first_name', u.first_name,
		'last_name', u.last_name,
		'user_pic_url', u.user_pic_url)) as collaborators
FROM groups
JOIN user_group_permissions p
ON   p.group_id = groups.id
JOIN users u
ON   u.id = p.user_id

These are the structs :
type Group struct {
    Id             string  `json:"id" db:"id"`
    Title          string  `json:"title"   db:"title"`
    Collaborators  []User  `json:"collaborators" db:"collaborators"`
}

type User struct {
    Id            string  `json:"id" db:"id"`
    FirstName     string  `json:"first_name" db:"first_name"`
    LastName      string  `json:"last_name" db:"last_name"`
    ProfilePhoto  *string  `json:"profile_photo" db:"user_pic_url"`
}

I have a simple Group table , a User table and table which represents all users with Permissions to the group :
CREATE TABLE groups (
   id	 int UNIQUE NOT NULL generated always as identity,
   title text
)

CREATE TABLE users (
    id 		 bigint UNIQUE NOT NULL generated always as identity,
    first_name 	 text NOT NULL,
    last_name 	 text NOT NULL,
    user_pic_url text
)

CREATE TABLE user_group_permissions (
   group_id   unsigned_int,
   user_id    unsigned_bigint,
   permission unsigned_smallint,
)

CREATE DOMAIN unsigned_smallint AS smallint
   CHECK(VALUE >= 0 AND VALUE < 32767);

CREATE DOMAIN unsigned_int AS int
   CHECK(VALUE >= 0 AND VALUE <<PHONE_NUMBER>);

CREATE DOMAIN unsigned_bigint AS bigint
   CHECK(VALUE >= 0 AND VALUE <<PHONE_NUMBER>854775807);

sqlx doesn't automatically unmarshal JSON columns; you'll need to use a custom type with a Scan method.
Something like:
type Users []User

// Scan converts the data returned from the DB into the struct.
func (u *Users) Scan(v interface{}) error {
        switch vv := v.(type) {
        case []byte:
                return json.Unmarshal(vv, u)
        case string:
                return json.Unmarshal([]byte(vv), u)
        default:
                return fmt.Errorf("unsupported type: %T", v)
        }
}

sqlx doesn't automatically unmarshal JSON columns; you'll need to use a custom type with a Scan method.
Something like:
type Users []User

// Scan converts the data returned from the DB into the struct.
func (u *Users) Scan(v interface{}) error {
        switch vv := v.(type) {
        case []byte:
                return json.Unmarshal(vv, u)
        case string:
                return json.Unmarshal([]byte(vv), u)
        default:
                return fmt.Errorf("unsupported type: %T", v)
        }
}

Is there any other way to achieve the same but without the need to define new struct with Scan? JSON is kind of a part of postgresql, so I also expected this to work automatically ;/ IMHO it would be a great feature.


fix(override): failed to override finalizers

see: https://github.com/kubewharf/kubeadmiral/pull/328

/lgtm

/approve


[docs] Add ClassKeys migration description for Renaming API

As @mnajdova pointed out https://github.com/mui-org/material-ui/pull/22053#pullrequestreview-460189237, I think we should document the change of classKeys.
This PR for updating already merged two PRs.
#22015
#22050

As working on this, I found anchorOriginBottomLeftCircular is missing in classKeys.
Is this on purpose or just missing?

classname or className?


Bug: File column always resets to far right position.

Describe the bug
The "file" column does not maintain its order and always becomes the farthest right column
To Reproduce
Steps to reproduce the behavior:

go to you database file
drag columns around until the "file" column in to the left (in the middle or farthest left)
switch files
switch back to database file
The "file" column will have relocated back to the farthest right column

Expected behavior
I expect the columns to stay in the order I drag-n-dropped them.

Desktop (please complete the following information):

OS: Windows 11
Version: Obsidian 0.14.7
dbfolder version: 0.0.7

Actually file column is a metadata internal type, not support yet persist the order.   I will try to persist it with future versions

after 0.1.1 that resolves some bugs, I am starting with this one

was fast! Now this feature is avaliable with 0.1.2

It didn't work at first...  I was dragging the other columns to the right of the "file" column then switching files and coming back... the file column would reset to the far right.  (just like in my first post)
BUT, once I grabbed the file column and dragged it, it now remembers where it should be, even if I drag the others.
I think its a bug, just I can't reproduce it now that I've dragged the file column.

I think is a temporal incompatibility with the already created databases. Once is persisted should work.
I leave the thread open by the flys

after deep testing this feature, I closed the issue


Help setting NormalFloat

I'm close to getting the hover window looking proper. I'm not sure how I can make it have the same colour inside of the window?

I'm not really sure why you closed this without commenting at all, is this to trivial to be worth an issue? 😂

You may want to override NormalFloat hlgroup.

I placed in the image right under the open floating window that  FloatBorder, NormalFloat are being set to something.
FloatBorder = { bg = "NONE" }
NormalFloat = { bg = "NONE" }

What would you rather get to show that there's more effort being made in a post?
This is a genuine question, not trying to word it rude/offensive.

Can't reproduce... I guess something is overriding your highlights? btw, if you allow compilation, always remember to run :KanagawaCompile
try
:verbose hi NormalFloat

I placed in the image right under the open floating window that FloatBorder, NormalFloat are being set to something

I did not notice that, perhaps you should emphasize more relevant information. I am sorry anyway, as missing this information slightly tilted me towards the impression that you were asking which hlgroups are responsible for floating window background.

I did not notice that, perhaps you should emphasize more relevant information. I am sorry anyway, as missing this information slightly tilted me towards the impression that you were asking which hlgroups are responsible for floating window background.

No worries, happens to us all :) Anyways, this is the output to your suggested command I should try:
:verbose hi NormalFloat

Can't reproduce... I guess something is overriding your highlights? btw, if you allow compilation, always remember to run

I've got that set to false. This is my entire setup file
vim.opt.laststatus = 3
vim.opt.fillchars:append({
    horiz = '━',
    horizup = '┻',
    horizdown = '┳',
    vert = '┃',
    vertleft = '┨',
    vertright = '┣',
    verthoriz = '╋',
})
-- Default options:
require('kanagawa').setup({
    compile = false,             -- enable compiling the colorscheme
    undercurl = true,            -- enable undercurls
    commentStyle = { italic = true },
    functionStyle = {},
    keywordStyle = { italic = true},
    statementStyle = { bold = true },
    typeStyle = {},
    transparent = false,         -- do not set background color
    dimInactive = true,          -- dim inactive window `:h hl-NormalNC`
    terminalColors = true,       -- define vim.g.terminal_color_{0,17}
    colors = {                   -- add/modify theme and palette colors
        palette = {},
        theme = {
            all = {
                ui = { bg_gutter = "none" }
            },
        },
    },
    overrides = function(colors) -- add/modify highlights
         local theme = colors.theme
        return {
            TelescopeTitle = { fg = theme.ui.special, bold = true },
            TelescopePromptNormal = { bg = theme.ui.bg_p1 },
            TelescopePromptBorder = { fg = theme.ui.bg_p1, bg = theme.ui.bg_p1 },
            TelescopeResultsNormal = { fg = theme.ui.fg_dim, bg = theme.ui.bg_m1 },
            TelescopeResultsBorder = { fg = theme.ui.bg_m1, bg = theme.ui.bg_m1 },
            TelescopePreviewNormal = { bg = theme.ui.bg_dim },
            TelescopePreviewBorder = { bg = theme.ui.bg_dim, fg = theme.ui.bg_dim },

            Pmenu = { fg = theme.ui.shade0, bg = theme.ui.bg_p1 },
            PmenuSel = { fg = "NONE", bg = theme.ui.bg_p2 },
            PmenuSbar = { bg = theme.ui.bg_m1 },
            PmenuThumb = { bg = theme.ui.bg_p2 },

            FloatBorder = { bg = "NONE" },
            NormalFloat = { bg = "NONE" },
            LineNr = { fg = "#C0A36E", bg = "NONE"},
            CursorLineNr = { fg = colors.palette.sakuraPink, bg = "NONE"},
            -- Visual = { fg = "NONE", bg = colors.palette.lightBlue},
        }
    end,
    theme = "wave",              -- Load "wave" theme when 'background' option is not set
    background = {               -- map the value of 'background' option to a theme
        dark = "wave",           -- try "dragon" !
        light = "lotus"
    },
})

-- setup must be called before loading
vim.cmd("colorscheme kanagawa")

-- vim.api.nvim_set_hl(1, "Linenr", {bg = "none", fg = "#C0A36E"})
vim.cmd [[highlight IndentBlanklineContextStart guisp=#FF0000 gui=underline]]

Perhaps this might help? I'm clueless :/

I did however notice, if I go inside of the floating window, then the colour seems to be the correct colour...?

Ah while, trying this out, I found out why it's not working properly... So apparently if I have dimInactive = true it behaves like before. When I have it set to false then it works normally.  I tried this option with compile set to both true and false this had no affect. It's completely dependant on the dimInactive property. :D
Here is the same hover with dimInactive and compile set to false.

Here dimInactive is set to true and compile is false.

Not sure why it behaves like this? I'm using Alacritty as my terminal emulator, if that might with the debugging.


Clarify PIN protected managements keys and PIN derived management keys

https://developers.yubico.com/yubikey-piv-manager/PIN_and_Management_Key.html

I had a look at what yubikey-manager does for protected management keys, and it looks like piv-go has the protected metadata part down, but it doesn't set the right flag in the unprotected metadata, so ykman doesn't know to use the stored key.
https://github.com/Yubico/yubikey-manager/blob/7fb59f633ea1a782d401ff3f03b1f0a859f5fb16/ykman/piv.py#L164
I'll see if I can send a PR to expand the Metadata type and methods.


Refactor out the use of CMILES

I notice that this project is the only one in our stack that currently (explicitly) uses CMILES. Given that it's no longer maintenance and all functionality (should) be in the toolkit, we should probably remove it altogether at some point.

Closed via #413


clues: update Easy Cryptic in Hosidius

The Hosidius Rework moved this clue slightly north west, and into the kitchens

Reported by Prxgmatic on the Discord

Looks like this needs rebase again. Otherwise LGTM


Make goreleaser archive to zip format

When using MacOs and Terraform >= 0.13 I get an error complaining about not finding the zip archive for the provider on the registry.
This might fix it. Not sure if it is related to the archive being tar.gz by default or in the change to the archive naming convention.
I based the changes on the goreleaser configs used by terraform-provider-aws.

Changes look good... I haven't used terraform in a while. I'll cut a new release after I merge this - let me know if it still isn't working.

I cut a new release - I see there's now a terraform registry and someone has forked this repo and published it as their own... if you're using the terraform registry to install, you won't get the update. Not sure what can be done about this...

I published to the terraform registry - looks like they prefix everything with username, so bmatcuk/vagrant doesn't conflict with the guy who forked my repo and published it =)


Nihlathak does not show up as npc object in the level data

in the map data, Nihlathak is not here, i tried to look at the code can't seem to figure it out, i assume npc_is_useless is returning true but not sure how this works.

I took out the npc_is_useless check to see what would happen, But I'm still only getting 1 NPC for Halls of Vaught, NPC code 802 which is too high for the NPC code table.
wine bin/d2-map.exe /app/game --seed 16711935 -d 0 -m 124

I wonder if the developers have hard coded his spawning logic into the game client somehow.

yea that 803 is not correct it goes to a location no where near Nihlathak.
tried to get some info, all i can find is
https://raw.githubusercontent.com/kolton/d2bot-with-kolbot/0ba87a25ef5e9432920e20d93082b936b318e148/d2bs/kolbot/sdk/SuperUniques.txt

The game joins together multiple arrays to make new ids.
with monsters, its all regular monsters then super unique, so having a super unique id of 61 means your actual id is around 790.
The game also has a bunch of hard coded logic which adds things to these lists. For instance the super chests in LK don't exist inside of the object game files at all but is hard coded into the source code eg https://github.com/ThePhrozenKeep/D2MOO/blob/b0951d44f4ef69de8170c3eddae4f6c18fa14086/source/D2Common/src/Drlg/DrlgPreset.cpp#L766
As to where Nihlathak is, his monster id is not shown anywhere on that level as far as I can tell.


Discord.py can't install

-macOS High Sierra
when i try to run the (pip install discord.py) command on my terminal i get this message here.
Command "python setup.py egg_info" failed with error code 1 in /private/var/folders/vn/v0ljrfl92tbf3lh6fh1kq6d40000gn/T/pip-install-WLF4ZL/aiohttp/
any way to fix this?

I'm sorry, but you've reached the wrong repository:  you might try https://github.com/pypa/pip/


test_typhoid_vaccine.py failed

https://jenkins.apps.idmod.org/view/Emodpy/job/Emodpy-Typhoid-Pipelines/job/Emodpy-Typhoid-Pipeline-for-testing/24/console
test_typhoid_vaccine.py:90: in compare_cases_after_vax_age_15
    self.assertTrue(sum(first_15_age_groups_cases_a) >= sum(first_15_age_groups_cases_b))
E   AssertionError: False is not true

so far this passes locally for me

Also passed on Jenkins upon rerun

Closing because cannot repro and everything works now (it does have sampling and it doesn't seem to be an issue)


Update CAC migration with latest sha

Description
I had to do an update on my CAC card to reassociate my teams account, which changed the sha, this migration updates it.
Setup
Ensure that migrations/app/secure/20201020164146_gidjin_cac.up.sql doesn't have the private portion of the migration
Down load the migrations for stg, exp and prd and verify that STG and EXP are good but prd is a stub placeholder
download-secure-migration<PHONE_NUMBER>4146_gidjin_cac.up.sql

Code Review Verification Steps

Any new migrations/schema changes:

[X] Follow our guidelines for zero-downtime deploys (see Zero-Downtime Deploys)
[X] Have been communicated to #g-database
[X] Secure migrations have been tested following the instructions in our docs

[X] Request review from a member of a different team.

Warnings

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

  Generated by :no_entry_sign: dangerJS against 521ecf7b6d12aca6b50258623818d517aeba2c03


Expansion Table short size

Expansion table does not support short table size.
Even after changing the table size to short only the header size changes, but rows are still the same.

This is an issue with carbon-components - open an issue here https://github.com/carbon-design-system/carbon and link it back to this one

https://github.com/carbon-design-system/carbon/issues/4796

https://github.com/carbon-design-system/carbon/issues/4796

Thanks!

Seems to be working correctly in the storybook now.
@vishals9711 do you still experience this issue?

Seems to be working correctly in the storybook now.
@vishals9711 do you still experience this issue?

closing, if still issue, please reopen.


boto3 1.14.59 breaks s3 test_presigned_url_restrict_parameters

s3 generate_presigned_url exception message for unknown parameters has changed (new param added).

This is now fixed as of moto >= 1.3.16.dev5


BMI088

TODO:

Add pull ups on SCL/SDA
NC connected to GND?
CSB1 should go to VDDIO

CSB1 does not need to be grounded in 12c mode


After running eslint - the problems do not show up.

Issue Type: Bug
Hello VSC -
After installing the ESlint plugin from the market place and running eslint there are hundreds of errors picked up in my terminal (Strings must use singlequote for example) but there is not blue light bulb to fix all/anything under the Problems tab or available above in the work space. I occasionally see a yellow lightbulb but no option to fix all and only limited options.
Thank you for the help
Extension version: 2.1.23
VS Code version: Code 1.57.1 (507ce72a4466fbb27b715c3722558bb15afa9f48, 2021-06-17T13:28:32.912Z)
OS version: Darwin x64 19.6.0
Restricted Mode: No

Can you please provide steps on how to reproduce what you are seeing, best by providing a GitHub repository I can clone.

Hi Dirk and thank you for the assistance.

Install ESLint
run lint command
view erros in command line
Go to the problems tab in VS code or the file
The only option via right click in the problems tab is a copy or copy message
The only option via the file is a yellow light bulb that when clicking on show fixes says "Generate get and set accessors" or "Learn more about JS/TS refactorings"

I appreciate your help. At this point I'm thinking it may be faster to uninstall and start with a fresh install of VS Code and then a fresh install of ESLint. Do you have advice on this process or just follow the normal steps?
Thanks again.

Just follow normal steps.

I have followed the steps in the below link (including rm -rf ~/.vscode*) and reinstalled VSCode and ESlint.
lerna run lint produces
'error  Strings must use singlequote                    @typescript-eslint/quotes'
but unfortunately it looks like ESLint is not giving me the option to fix it. Note -  I haven't modified the settings for ESLint extension. Would you suggest I change something in the settings?
[Info  - 3:56:47 PM] ESLint server is running.

https://stackoverflow.com/questions/42603103/how-to-completely-uninstall-vscode-on-mac

Just a follow up that I have tried { "mode": "auto" }  as the "eslint.workingDirectories" in the User and Workspace settings but to no avail. Thank you again for looking into this as my coworkers are enjoying ESLint. They had a permission issue in the past but it was resolved.

@spetes375 best would be you provide me with a GitHub repository I can clone that demos what you are experience.

It is now working post reinstall. Thank and and congratulations on build an awesome tool. Thanks!


MAPPING_INFO command, returning the mapping of extensions to types

The mapping is returned as part of the server info, but also separately:
>rasctl -m
Supported image file extension to device type mappings:
  hda->SCHD
  hdf->SAHD
  hdi->SCHD
  hdn->SCHD
  hdr->SCRM
  hds->SCHD
  iso->SCCD
  mos->SCMO
  nhd->SCHD


Update LINEmsg-balanceInquiry.js

Sorry!
Please dismiss this change.

This update contains "APIKey".
Isn't it a security issue for us to merge this?


Start running only E2E test on PR(Prow)

As a QE I want to run only e2e on PRs and remove prow tests as required to speed up the PR feedback loop
Action Iteams

remove integration tests from PR
make prow test not required for PR

Please check prow jobs related to unit and validation tests. Remove them if they're same as IBM tests.


Add output_type_id_datatype property

In v3.0.1 of the schema, the optional output_type_id_datatype property was introduced to enable hub administrators to configure and communicate the output_type_id column data type at a hub level. This property allows hubs to override default behaviour of automatically determining the simplest data type that can accomodate output type IDs across all output types when creating hub schema.
The setting is also useful for administrators to future proof the output_type_id column from potential issues arising by changes in data type, introduced by new output types after submissions have begun, by setting output_type_id_datatype to the simplest data type from the start, i.e. character.
If the property is not provided or set to "auto" (the default), the hub uses autodetection to determine the simplest hub level data type that can represent all output type ID values across all hub output types and rounds.
Also resolves #12

I understand the motivation for setting the default behavior to "auto" in the schema -- we want to avoid issues with back compatibility.  However, here, might it make sense to use "character" for this setting since that's the most robust setting?

I'm hesitant to do that. I don't fell that should be a decision we make as default but a decision hub admins make consciously, thoughtfully and with a plan to communicate that to teams submitting.
There will be a whole section on the topic in hubDocs shortly (https://github.com/hubverse-org/hubDocs/issues/143) and I feel educating is better than enforcing by default.

a few thoughts:

I agree with the value of education and documentation
I wouldn't use the phrase "...better than enforcing by default": (a) education/documentation are complementary to a default setting, not a replacement for it; we'll hope that admins make a conscious decision either way; (b) we're not enforcing anything, the setting can be updated from the default that's placed into the template, right?
It seems like we should set the default here to be our best guess at what will be appropriate for most hubs.  Do you think "auto" is going to be the right answer more often than "character"?

The property can indeed be updated but I still feel setting the default here should be more flexible (which auto" is).
"auto" is always safe when first developing a config as it ensures all output type ID values can be represented appropriately, It's only if output types are added at a later date that it might produce an issue.
"auto" has also been the default setting so far and only now has the need to be able to override it come up in only one hub. If a hub is set up from the template that is only intending to collect quantiles and means for example and the admins don't change a "character" default setting, they might end up with an output_type_id column that was unnecessarily cast to character and validation errors that they are not sure where they are coming from.
Having said all this, maybe we should just ask users in a dev and/or community call what they would prefer?

discussion on devteam call:

opinions were split about what was preferred
what we do for this setting is low impact
let's just move ahead with this as is


fix(connectors): fixes MGDCTRS-777, connector agent APIs get http status 410 if the cluster has been deleted

Description
connector agent APIs get http status 410 if the cluster has been deleted
Verification Steps
Added integration test to verify.
Checklist (Definition of Done)

[X] All acceptance criteria specified in JIRA have been completed
[X] Unit and integration tests added that prove the fix is effective or the feature works (tested against emulated and non-emulated OCM environment)
- [ ] Documentation added for the feature
[ ] CI and all relevant tests are passing
[ ] Code Review completed
- [ ] Verified independently by reviewer
- [ ] Required metrics/dashboards/alerts have been added (or PR created).
- [ ] Required Standard Operating Procedure (SOP) is added.
- [ ] JIRA has created for changes required on the client side

Follow up for #938 with agent API test.


How to stream infrared frame from rosbag with matlab?

Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

Consider checking out SDK examples.
Have you looked in our documentations?
Is you question a frequently asked one?
Try searching our GitHub Issues (open and closed) for a similar issue.

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
D435

Firmware Version
<IP_ADDRESS>

Operating System & Version
Win 10

Platform
PC

SDK Version
2.21.0

Language
Matlab

Segment
others

Issue Description
Hi,
I have recorded .bag file contain RBG and infrared (IR1 and IR2) data with my d435. I'm using rosbag_example.m to stream both, the RGB and infrared data. From that example, I was able to stream the RGB data but clueless with how to stream the infrared.
And one more thing, it seems the RGB and infrared data have different fps (I check it using realsense viewer). Is there any example of how to stream the data with the timestamp? So I can understand the data better.
Thankyou

Hi MrNash97,
Wonder if this can help you, if not checked yet?
https://github.com/IntelRealSense/librealsense/blob/master/wrappers/matlab/rosbag_example.m
Thanks!

I've tried that code. It works well with depth and RGB data. But, what I want to know is how to display the Infrared data. Can you help me with how to access the IR data? I only know how to access depth and RGB.
depth = fs.get_depth_frame();  This one is for depth
and
color = fs.get_color_frame();  This one is for RGB.
Thank you.

If the bag contains IR files, they should be streaming already. When no streams are explicitly specified to the config object for a bag file, all streams present are opened. fs.get_infrared_frame(0) and fs.get_infrared_frame(1) should allow you to grab the individual frames as they are read from the bag

@lramati Thank you, it's working.
I want to ask one more thing. Is there any way to access the timestamp of each frame in Matlab? I need to synchronize the timing between IR and RGB data because it has different fps (even though I set the recording with the same fps).

Hi MrNash97,
You can refer to the timestamp functions defined in frame.m for how to get timestamp info:
https://github.com/IntelRealSense/librealsense/blob/master/wrappers/matlab/frame.m#L24
https://github.com/IntelRealSense/librealsense/blob/master/wrappers/matlab/frame.m#L27
Hope this can be helpful.
Thanks!

Thank you for your help. It seems that I must enable the frame metadata first to get what I wanted.


Enum with StringEnum should be lowercase, is uppercase

(moved from https://github.com/MangelMaxime/Thoth/issues/133)
I have this enum:
[<StringEnum>]
type Camera = FirstPerson | ArcRotate | IsometricTopDown

When I serialize it on the server, it is serialized with UpperCase ("Camera":"FirstPerson").
Then on the client side it does not get recognized because Fable seems to expect lowerCase ("firstPerson").
Workaround 1:
I think in this case I can just remove the StringEnumbecause that value is not/(no longer) passed to any ts/js
Workaround 2:
disable lowercasing with [<StringEnum(CaseRules.None)>]
Fix 1:
The encoder on the server could detect the presence of StringEnum and check the case rules
Fix 2:
The decoder on the client could detect it and lowerCase the first char.

There doesn't seem to be any reflection info emitted for string enums, so either this would need to be extended in Fable, or fixed on the server side:
[<StringEnumAttribute>]
type MyStringEnum = A | B

type MyRecord = { I : int; E : MyStringEnum }

generates this reflection info:
export function MyRecord$reflection() {
  return record("Test.MyRecord", [], MyRecord, () => [["I", int32], ["E", string]]);
}

Should be fixed by 3.4.0 if you use .NetCore.
I wasn't able to make it work over .NetFramework because of Fable.Core being available only. for .NetCore.
Tests


docs Refactor query-cache docs

Proposed changes
Refactor query-cache docs
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


Direction of edges inverted when going from SimpleWeightedDiGraph to SimpleDiGraph

Hello all,
I hope this is the right place to ask this question!
After the recent update of NetworkDynamics.jl the functions no longer work with SimpleWeightedDiGraph objects, instead taking a SimpleDiGraph object.
In e.g. the FitzHugh-Nagumo tutorial in the docs this is described.
A slightly modified version of setting up that tutorial:
using SimpleWeightedGraphs, LightGraphs

# Create lists of sources and destination vertices
sources = [i for i in 1:3]
destinations = [4 for _ in 1:length(sources)]
weights = [1. for _ in 1:length(sources)]

# Construct the weighted and directed graph
g_weighted = SimpleWeightedDiGraph(sources, destinations, weights)
edge_weights = getfield.(collect(edges(g_weighted)), :weight)

# Convert the weighted graph to a simple directed graph
g_directed = SimpleDiGraph(g_weighted)

However, this procedure inverts the direction of the edges:
julia> collect(edges(g_weighted))
3-element Array{SimpleWeightedEdge{Int64,Float64},1}:
 Edge 1 => 4 with weight 1.0
 Edge 2 => 4 with weight 1.0
 Edge 3 => 4 with weight 1.0

and
julia> collect(edges(g_directed))
3-element Array{LightGraphs.SimpleGraphs.SimpleEdge{Int64},1}:
 Edge 4 => 1
 Edge 4 => 2
 Edge 4 => 3

Of course this is easily fixed by doing reverse(g_directed), but it is confusing nevertheless.
This might be a SimpleWeightedGraphs.jl issue, so I posted it there as well, but I thought you maybe wanted to be aware of this.

Hi Thomas, thanks for reporting this issue. Indeed this should be fixed upstream. I'll leave this issue open until it has been fixed by SimpleWeightedGraphs.

Hi Thomas, thanks for reporting this issue. Indeed this should be fixed upstream. I'll leave this issue open until it has been fixed by SimpleWeightedGraphs.

Fixed upstream.


Swagger APIView PR pipeline automation reports success and failure

In PR 23091 the Swagger APIView PR pipeline automation is reporting success:

AND failure:

I have retriggered the pipeline ... hoping that this will resolve the problem.

This is expected though as they are two different steps in the pipeline. The first is the run of the Swagger Parser which succeeds in creating the APIView token files. The later (which failed) is the call to the APIView endpoint to create and actual APIView using the token file generated.
Perharps we can improve the naming to say SwaggerAPIViewParsing for the first step?

Here is another instance: PR 23616
It looks like the failure was:
  status: 500,
  statusText: 'Internal Server Error',


RageHook crashing loading LSPDFR

Version
V3.5.1
Description
Launching RageHook it crashes midway through load.  If I take it out, it stops crashing.  Numerous people have spoken about this on the LSPDFR forums, please advise.

This is SHVDN, not RageHook, the two are not officially supported together.


new snacks - Jelly Beans

About the PR
Adds packet of Jelly Beans - small bean-shaped sugar candies that takes it roots from USA. Now everyone on the station can taste them in the Discount Dan`s.
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl: Tunguso4ka

add: Check your local Discount Dan`s for new Jelly Beans.

Glad to see this came back in a form more immediately recognizable in most places of the world. As a totally-not-biased jelly bean enjoyer, it was a bummer when the first one was closed.

This still looks like https://www.roshen.com/ro/en/products/boxed-sweets/crazy-bee-frutty and isn't recognisable as jelly beans considering their distinct shape and bright colors https://www.google.com/search?q=jelly+beans&client=firefox-b-d&sxsrf=APwXEdc8kX3647HUS-HNhYu5cTNtMj9-QA:1682865185387&source=lnms&tbm=isch&sa=X&ved=2ahUKEwiD9NCs6dH-AhWWpVYBHW_fATsQ_AUoAnoECAEQBA&biw=1411&bih=1456&dpr=1.36#imgrc=uT9Rxny-9YNgxM

@Tunguso4ka are you coming back to this?

@Tunguso4ka are you coming back to this?

i dont think so... You can close it.


Auto expand sidebar when hovering over it with the mouse

there are posibilities that the sidebar compact to icons for default and expand automatically when the cursor is positioned over it.
This is a cool alternative for smaller screens or when two panels side by side are open

I love it

I love it

@primorolo would you expect it to collapse again once the mouse is no longer hovering over it?

@primorolo would you expect it to collapse again once the mouse is no longer hovering over it?

Of course, like the "vertical Tabs" in Microsoft edge.

Thank you for your feedback, we don't have plans to implement this particular feature request but we'll continue to keep it in mind.

Thank you for your feedback, we don't have plans to implement this particular feature request.


Notifications swallow error messages

Hi there,
First off, thanks for the great addon. I just have a quick question that I can't seem to find anything about: how do I prevent notifications from swallowing error messages during automated acceptance testing?
In my testing environment, I have it autoClear pretty fast so that tests aren't artificially slowed down. But when errors come up when debugging acceptance tests, the error is swallowed up completely and the message is only flashed for an instant. I have to increase the timing just to read it, and I don't get a stack trace for further debugging.
Any suggestions? I get this is probably just the nature of promise chains handling exceptions, but it seems like there should be a better way to handle errors for debugging.

do you have an example test? 🤔 I'm struggling to think of the use case but I'm pretty sure I would get it instantly with an example 😂

did you have the chance to try reproduce this with an example test? I'm happy to put some work into trying to fix this for you but I'm struggling to understand the issue 😂

Sorry @mansona I didn't get a chance. I will put one together. This is more of a feature request if I recall (I can't believe how long ago this was!).


Orchestrator recognizer: Initialize model method slows the first request

Initialize model method in orchestrator recognizer is ran only once, but can take several seconds to complete, and all threads will be blocked on this and potentially leading to degraded user experiences. In voice environments such as telephony channel, time is even more important, where delays of several seconds are not a possibility.
Evaluate whether we can perform initialization on load so that we do not do all the initialization in the first call to RecognizeAsync.

The latest OrchestratorRecognizer refactored (both NodeJS and DotNet versions) code in 'main' can directly be instantiated using the constructor (before, the constructor was only used for mock-up in unit tests). Therefore, this cold-start problem has a work-around: using the constructor with configuration information read from NodeJS .env file or DotNet appsettings.json.


findByIdempotencyKey: return type

Hey,
The function findByIdempotencyKey in IIdempotencyDataAdapter has the return type Promise<IdempotencyResource>. However, the InMemoryDataAdapter returns null which doesn't fit. Can you please clarify?
Thanks

@joscelynjean thanks a lot. Can you please also adapt the return type in the interface to Promise<IdempotencyResource|null> if you don't choose the exception way?

@kasium Thanks for the suggestion. I will check this out.

I would second liking to see the type for findByIdempotencyByKey update to return Promise<IdempotencyResource | null}. Would you welcome a PR updating this?

I will prepare for an official release this week. Thank you @dmmulroy

Now fixed in version 1.0.5


[Bug] Border router does not start on TH

Describe the bug
I want to start Border Router on TH for commissioning, but without success.
I created a project on the TH web tool, advertised the device and started the Commissioning test, but nothing happens.
Steps to reproduce the behavior
1.Install the Ubuntu Server 22.04.1 LTS (64-bit) in the SD card.
2.After the SD card has been flashed, place it in the Raspberry Pi’s memory card slot.
3.Power on the Raspberry Pi and ensure that the local area network, display monitor and keyboard are connected.
4.git clone -b v2.10.2+spring2024 https://github.com/project-chip/certification-tool.git
5.cd certification-tool
6.git submodule update --init --recursive
7../scripts/pi-setup/auto-install.sh
8.Wait about 10 minutes.
9.At the end of the script, select option 1 to restart the RaspberryPi.
10.Insert the Nordic RCP dongle with the firmware written into the TH and reboot.
11.TH will automatically spin the OTBR docker image when executing thread-related test cases, so we will start the Test as is.
Expected behavior
Thread Border Router starts and the commissioning test begins.
Log files
No response
PICS file
Test_PICS.zip
Screenshots
No response
Environment
TH version: v2.10.2+spring2024
OS:ubuntu Server 22.04.1 LTS (64-bit)
Additional Information
No response

The json file is set as follows.
"network": {
"wifi": {
"ssid": "TP-Link_7B8F",
"password": "10861110"
},
"thread": {
"rcp_serial_path": "/dev/ttyACM0",
"rcp_baudrate": 115200,
"on_mesh_prefix": "fd11:22::/64",
"network_interface": "eth0",
"dataset": {
"channel": "15",
"panid": "0x1234",
"extpanid": "1111111122222222",
"networkkey": "00112233445566778899aabbccddeeff",
"networkname": "DEMO"
},
"otbr_docker_image": null
}
},
"dut_config": {
"discriminator": "3840",
"setup_code": "20202021",
"pairing_mode": "ble-thread",
"chip_timeout": null,
"chip_use_paa_certs": true,
"trace_log": true
},
"test_parameters": {
"endpoint": 0,
"manual-code": "34970112332",
"paa-trust-store-path": "/credentials/development/paa_roots",
"storage-path": "admin_storage.json"
}


added trigger for syncing based on the repository.created event

i decided to go this route afterall because it was a simpler implementation (stays forcused on a single repository for now) and works without needing to listen to additional events (we already listen for repository.edited).


Module 1.1 Assignment

Given what you have learned about Git, why would you want to use Git in your classroom?

Git is a source control system that allows to track versions of your work in project containers called repositories.
I'm going to use Git to collaborate with my classmates during the group projects.


Why is nginx container is accessible with http://localhost on Windows and not with <IP_ADDRESS> ?

Hi,
docker network inspect dockersymfony_default tells the IP of nginx container is <IP_ADDRESS>. I can access nginx with this IP from other containers in network (php, db, ...).
BUT from the host machine (Windows) nginx container is not accessible by <IP_ADDRESS>. On the other hand nginx is accessible by <IP_ADDRESS> or localhost.
[
{
    "Name": "dockersymfony_default",
    "Id": "12b4f5a663450bb44e87f9635860bbcede354f2fea27f93b056d7159251dc465",
    "Scope": "local",
    "Driver": "bridge",
    "EnableIPv6": false,
    "IPAM": {
        "Driver": "default",
        "Options": null,
        "Config": [
            {
                "Subnet": "<IP_ADDRESS>/16",
                "Gateway": "<IP_ADDRESS>"
            }
        ]
    },
    "Internal": false,
    "Containers": {
        "1eef50ee350782654bb96e7d16d09e3b9fe54abca97cc339a89791083e08563c": {
            "Name": "dockersymfony_db_1",
            "EndpointID": "8b48ea4934a01703ac23a7f27f8cee0ff9226b7b5401550859b22fbf17a4c10a",
            "MacAddress": "02:42:ac:13:00:03",
            "IPv4Address": "<IP_ADDRESS>/16",
            "IPv6Address": ""
        },
        "1faf199dd6285c6700640b570fd842212962c299d762c531b011205f58598102": {
            "Name": "dockersymfony_nginx_1",
            "EndpointID": "a9be2a542565262d31c93f7d1960a13e373f5a701c299cedf3dc0510c8de9bf4",
            "MacAddress": "02:42:ac:13:00:05",
            "IPv4Address": "<IP_ADDRESS>/16",
            "IPv6Address": ""
        },
        "2c2b994c92895e7a83c189d1e5002d2eb7d88f62761f8c324f00ffdece4efb4a": {
            "Name": "dockersymfony_php_1",
            "EndpointID": "98a0ad7bef2f2c963c68561bb2b56bc0feb113c390a29c0dd9aefd6e32b7e5be",
            "MacAddress": "02:42:ac:13:00:04",
            "IPv4Address": "<IP_ADDRESS>/16",
            "IPv6Address": ""
        },
        "65ee8088e6bc2405b80c56ae21d83a4f4f7fff252a7e782c4046629d670f7b74": {
            "Name": "dockersymfony_redis_1",
            "EndpointID": "80fcd76b2fe0326be8d30abaffe2310ef58f23935f0f67f7476fa4fad951cba6",
            "MacAddress": "02:42:ac:13:00:02",
            "IPv4Address": "<IP_ADDRESS>/16",
            "IPv6Address": ""
        },
        "86665f35aa1599028d8e91aa45f86505a112b44496386e674b7039458dcda45f": {
            "Name": "dockersymfony_elk_1",
            "EndpointID": "c82ef8cc18f1cc345e97c3bdd0e04f0bf398efc0c38f9876b0e922a1e6dd494c",
            "MacAddress": "02:42:ac:13:00:06",
            "IPv4Address": "<IP_ADDRESS>/16",
            "IPv6Address": ""
        }
    },
    "Options": {},
    "Labels": {}
}
]

Why is that ?
I suppose that this problem is the cause of another one: xdebug doesn't work.

It might be related to #21
But I'm not sure, I drop my Windows... Try with your IPAM > Config > Gateway value. According to your JSON, it should be: <IP_ADDRESS>
... but your host is not in a Unix System. In Windows&Mac, I think that the only one access point is localhost (or <IP_ADDRESS>).
Did you try the getting started with Windows? Does it works? (especially the step 4)

@vria, This is ok for you? I can close this issue?

Yes, thank you.


Flatten ns require forms.

Some tools, such as babashka, struggle with nested ns forms such as
(ns io.aviso.columns
  (:require [clojure.string :as str]
            [io.aviso
             [ansi :as ansi]]))

while flattening these aren't problematic
(ns io.aviso.columns
  (:require [clojure.string :as str]
            [io.aviso.ansi :as ansi]]))

Ideally babashka can be taught to handle these, but it's probably not a terrible thing to use the more common flattened require forms here.

Also note that handling nested lib specs is a known, open issue for sci:
https://github.com/borkdude/sci/issues/399

This was some of the earliest code I wrote in Clojure!

Cheers! Yeah, it's interesting seeing some of the older projects before common practices really coalesced. Thanks for the fast turnaround!


[BugFix]: NPE when BE report exceptional status

What type of PR is this：

[x] bug
[ ] feature
[ ] enhancement
[ ] others

Which issues of this PR fixes ：

Fixes #5769
Problem Summary(Required) ：

Fix the NPE when backend report exceptional status.

[FE PR Coverage check]
:disappointed: fail : 0 / 3 (00.00%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/qe/Coordinator.java
0
3
00.00%

Also fixes https://github.com/StarRocks/starrocks/issues/5714

When some BE nodes report error like Status::RuntimeError or something,  this error is not properly handled by FE, and mysql client does not specific error, instead of showing empty set.

Is it the log like
2022-05-06 10:03:31,383 WARN (thrift-server-pool-169|421) [QeProcessorImpl.reportExecStatus():126
] ReportExecStatus() failed, fragment_instance_id=ba5dc209-cce0-11ec-aab7-00163e0e2d06, query_id=
ba5dc209-cce0-11ec-aab7-00163e0e2cf9, error: null

It seems #5761 is the same problem

Is it the log like
2022-05-06 10:03:31,383 WARN (thrift-server-pool-169|421) [QeProcessorImpl.reportExecStatus():126
] ReportExecStatus() failed, fragment_instance_id=ba5dc209-cce0-11ec-aab7-00163e0e2d06, query_id=
ba5dc209-cce0-11ec-aab7-00163e0e2cf9, error: null

It seems #5761 is the same problem

Probably the same bug.

run all

run all

[FE PR Coverage check]
:disappointed: fail : 15 / 27 (55.56%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/qe/Coordinator.java
15
27
55.56%


RestClient marked deprecated but still used in all the docs - replacement API is unclear

com.netflix.niws.client.http.RestClient is marked as deprecated, but it is not clear what the replacement API is. The javadoc comment says "Please see ribbon-rxnetty module for the Netty based client", but that didn't help me figure out what the replacement API is and how to use it. The docs still talk about RestClient, e.g. https://github.com/Netflix/ribbon/wiki/Getting-Started. If the class is truly deprecated, please update the docs with examples of using the replacement API. Otherwise, undeprecate it.

Hi there. There's also https://github.com/Netflix/ribbon/issues/239 that I guess is the same. Luckily someone would mark one of them as duplicate and would answer it, but it seems that no one at netflix is currently working on this project right now.

I missed that one. Thanks.
Yeah, I also noticed that the project appears dead.

I had the same question...
I think we are meant to use: io.reactivex.netty.protocol.http.client.HttpClient
Something like this:
ConfigurationManager.loadCascadedPropertiesFromResources(serviceName);
IClientConfig clientConfig = IClientConfig.Builder.newBuilder(url).build();
            clientConfig.set(IClientConfigKey.Keys.ListOfServers, url);
LoadBalancingHttpClient<ByteBuf, ByteBuf> ribbonClient = RibbonTransport.newHttpClient(clientConfig);

Nice! I read they are accepting PRs, so maybe you could create one :)


Hide On-Chain Attributes option on Collection level

What happened?
Create Collection form asks for value in onchain attributes section.
Feels a bit odd as attributes are defined at NFT level..
if this to capture what the master set of attributes allowed with this collection, then the value field needs to changed to multiselect or something.
Please reproduce in steps
Select BSX chain, then goto create collection.
Expected Behavior
Expected the attributes section here to accept LOVs against each attribute
What browsers are you seeing the problem on?
Chrome, Safari
At which address did you encounter bug?
kodadot.xyz
Are you logged in?
Yes
Which wallet you are using?
PolkadotJS
At which chain did you encounter bug?
Basilisk
Screenshots

Relevant log output
No response
Payment link for reward
https://kodadot.xyz/transfer/?target=<My_Kusama_Address_check_https://github.com/kodadot/nft-gallery/blob/main/CONTRIBUTING.md#creating-your-ksm-address>
Code of Conduct

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

hey @razi429 thanks for reporting this :) I understand, this can indeed produce some amount of confusion, we're currently not using the on-chain attributes on collection for anything meaningful and given your report, I think it would be best to hide it for now to not confuse future users.

👋


[BUG] Multiple-inheritance classes cannot be cimported and extended.

There is no way to indicate in the pxd file that a cdef class multiply inherits from an ordinary class, and so must use our PyType_Ready hacks.

Fixed by the referenced PR.


Does Not Support Filestream?

"Filestream: RLS is incompatible with Filestream."
What does this mean?  If Filestream is on a database,

then RLS cannot be used on any table?
Then RLS cannot be used on a table referencing the blob?
Then RLS cannot be used with a predicate on the file?

Can you provide clarity in this document?

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

ID: 9c96c3ee-5443-004f-d189-3701f8bef6c9
Version Independent ID: efcc952b-eadb-4579-54c7-00da3bc107ae
Content: Row-Level Security - SQL Server
Content Source: docs/relational-databases/security/row-level-security.md
Product: sql
Technology: security
GitHub Login: @VanMSFT
Microsoft Alias: vanto

Hi @keithnicholson - Thanks for the question! It's not that RLS can't be used when FILESTREAM is enabled. FILESTREAM is secured by granting permissions at the table or column levels, and if a user has permissions to the FILESTREAM column, the user can open the associated files, and hence it's incompatible with RLS since it defeats the purpose.

Thank you for the insight.  I'll think through this and consider how else my design could go.
Regarding this "issue", would it feasible to edit the line stating something similar with a reference to https://docs.microsoft.com/en-us/sql/relational-databases/blob/filestream-sql-server?view=sql-server-ver15#integrated-security  That might help others.
If not, just say so and feel free to close this comment out.
In my scenario, it is not so much that we are protecting anything specific or secret in the documents as much as mitigating what a user thinks they have access to without trusting the developer to filter on the client side.
I might play with the idea of using RLS for users accessing through a middle-tier app to a look up table (Scenarios C and D in this doc).  That table would then provide some identifier to access the document, The middle tier would not return this identifier to the client, but would rather permit the middle tier service account view/read-only access to the table with the FileStream column.
Peace!


Redirect to Plunker, Codepen and JSFiddle

Redirects to Plunker, Codepen or JSFiddle with corresponding template. For usage in issuetemplate, issues, StackOverflow etc.
Usage Examples:

http://leafletjs.com/edit.html // Defaults to Plunker, latest version
http://leafletjs.com/edit.html#provider:codepen;version:1.1.0 // Codepen, version 1.1.0
http://leafletjs.com/edit.html#provider:jsfiddle;version:master // JSFiddle, master version.

(edit 2017-10-25: fix examples URL domain extension)

Hi,
Wow that is super nice! :+1:
How about linking to dist/leaflet-src.js instead of the minified version, to facilitate debugging?
I do not know Plunker API at all, but is it possible to specify tag attributes as well?
(typically to include SRI)
I guess that would be impossible in the case of CodePen and JSFiddle.

Yes, agree with @ghybs - it's definitely better to link leaflet-src.js, debugging with the minified source is a real headache.

I knew Ghybs would be happy 👍 Will change to leaflet-src.js and send a PR. As for the tags in Plunker, they can be changed. Just change the HTML template. As for SRI (which i'm not so familiar with),  i was assuming the SRI differs per version so that would be more difficult to implement since i don't know where to get the hash or how to generate it. But if that's needed, i'm willing to look into it. Just point me in the right direction.

Hi iH8,
Yes you are absolutely right, SRI is per file, therefore it is different for each version.
I do not think it is very critical for a debugging tool, but since we show a full HTML code in Plunker, I would try to show a good example that enforces SRI directly… :smile:
As for implementation, I see that you just need to pass Plunker an HTML string indeed, that you have nicely templated.
In that case, you could try to match the requested version, and in case SRI hashes are known for that version, include them into the <link> and <script> tags.
I have just made such a list for some of Leaflet versions:
https://github.com/ghybs/Leaflet.MarkerCluster.Freezable/blob/ddb0f9a7df983d884c855f8fcbb7517d9f7b411b/docs/examples/mcg-freezable-libs-setup.js#L21-L55
You can also use the online SRI Hash Generator tool for other files and versions if needed.

We already have the SRI hashes for the latest version here: https://github.com/Leaflet/Leaflet/blob/master/docs/_config.yml - we could but hashes for older versions there, or in a new YAML file in the _data directory, if we want to leverage Jekyll to solve this for us.

Ok, i understand and i am all for good complete examples. I wrote this with in the back of my mind that we wouldn't have to maintain it or at least keep maintenance to a minimum. So i did not hardcode the versions, it's completely oblivious as to which version is latest. In order for that to work we would have to keep a list of versions and corresponding SRI's (which i don't think is a bad thing to have) and keep it up to date automaticly when doing a release. Now i see @perliedman's response and indeed, something like that would work. I'm working on the tooling, landed at release script at the moment and can easily take this along 👍 Only obstacle for me is that i would like it as a JSON file since that's the easiest to read from browser and write from node script. It's must more transportable than YML. Just output the SRI's to the config.yml like is done now with an ugly string replace and on top of that have a sri.json somewhere:
{
    "v1.2.0": {
        "source": "sha512-YLT+I34kEPlk5OqR5XObf40B7sInrIU+bGe5VcwSpfR5OrFVjExFxfhVoJQEPZQWMyB53o3AU/bb5J91nc8CPA==",
        "minified": "sha512-lInM/apFSqyy1o6s89K4iQUKg6ppXEgsVxT35HbzUupEVRh2Eu9Wdl4tHj7dZO0s1uvplcYGmt3498TtHq+log==",
        "css": "sha512-M2wvCLH6DSRazYeZRIm1JnYyh22purTM+FDB5CsyxtQJYeKq83arPe5wgbNmcFXGqiSH2XR8dT/fJISVA1r/zQ=="
    },
    "v1.1.0": {
        ...
    }
}

Jekyll can handle JSON as well, AFAIK, so that should be good. I'm just thinking it might be good if we can use the generated hashes in Jekyll templates (like we do today) as well as dynamically reading it from the client, if we need that.


README.md still have old link to documentation

Old link to documentation (https://github.com/still-scene/t3/wiki/user-interface) needs to be replaced by https://github.com/tooll3/t3/wiki/T3UserInterface
Installation guide (https://github.com/still-scene/t3/wiki/Installation) there is an automatic redirection to https://github.com/tooll3/t3/wiki/Installation

Thanks for pointing out: Fixed with 78409b52426fdba09649234f39fa086d002d66a8


Admin interface must implement 3.1.2 Language of Parts for multilingual interfaces

Issue Summary
Wagtail supports multilingual admin interfaces, with "partial" language translations where an admin view will have content in multiple languages. For accessibility, it’s important that parts of the page in different languages are correctly marked up with the lang attribute. This currently isn’t the case.
See: WCAG SC 3.1.2: Language of Parts (Level AA)
Steps to Reproduce

Go to https://static-wagtail-v5-2.netlify.app/admin-german/, or any other non-english variation of the CMS interface with english content
Note how some terms like "What's new in Wagtail 5.2" do not use the lang attribute, despite the lack of a translation

Possible solutions
I have no idea how to approach this.

Related: #11375, #4694

As above the tags can contain the lang attributes to the code .

@vanamaabhi2004 thanks for taking a look at this. What you’re screenshoting here is Wagtail content being in a different language, which isn’t what this issue is about. Please see #11375.
This issue is specifically about the Wagtail admin interface being in multiple languages when someone uses it with a language we only have partial translations for.

@thibaudcolas I also can't see a good way of tackling this, short of totally reworking the way the gettext functions work - since those functions don't provide any way to identify when they've failed to find a translation and fallen back on the English text.
Is this something that actually needs addressing, though? For any given string that is missing a translation and not marked up with lang="en", our preferred remedy would clearly be "add the translation" rather than "add the attribute". Obviously, I'm not suggesting that "just add 100% coverage of all languages and the problem will go away 😇 " is a realistic solution to this issue... just that we've always treated translation coverage as a work in progress that will never really be complete.
In the absence of a mechanism for inserting lang="en", achieving full ATAG compliance for a given language will be conditional on having 100% translation coverage for that language. On the other hand, with such a mechanism in place we might technically tick a box on the checklist but I'm not sure that we can call the interface truly usable and accessible in that language. So, wouldn't it be reasonable to treat full cross-language ATAG compliance as the same work-in-progress effort - i.e. say "currently we can claim full ATAG compliance for the English, French, German, Dutch (...) versions of the Wagtail admin; for all other languages, we're working towards it and would welcome help"?
(On the other hand, if there's pressure on us to achieve full (technical) ATAG compliance across all languages on a shorter timescale than reaching 100% translation coverage, then I guess my argument is invalid and we do indeed need lang="en" as a stopgap.)

@gasman thank you for looking into this ☺ Yeah, honestly short of a very creative gettext re-implementation I don’t really know whether/if/how we should tackle this.
This is actually a WCAG AA consideration (SC 3.1.2: Language of Parts (Level AA)), so in some ways more pressing for us than ATAG. It’s only helpful for screen reader users though, and only changes how English words would be pronounced. Something we should certainly try to solve but for now I suspect even those people who could use it would welcome other improvements most.
For now I’ve reached out for help on the web-a11y Slack. The suggestion so far was "have gettext return HTML with the lang attribute in it", which wouldn’t be that simple in practice, and I’m not sure is a good idea with Django. This feels like a very broad "crowdsourced partial translations" problem so I’d hope we can learn from other projects on this – will have to think of other places where to raise this.

I want to give it a try can you tell me the file location

@riddhisharma-sudo we don’t have any proposal on how to tackle this currently. At least in theory, the issue is with all of the parts of the admin interface which contain labels. If you want to work on this please review the problem first, make sure you fully understand it, and share possible solutions here.


add docker-credential-acr-env

Fixes:
Related:
Pre-review Checklist

[ ] IMPORTANT: 'image-request' tag has been applied if this PR is adding any images, including new versions or variants

For new image PRs only
If you have an apko.yaml file in this PR you need to follow this checklist, otherwise feel free to remove.

[ ] Include tests, sufficient enough that you would trust this image running in production.
[ ] The version included is the latest GA version of the software
[ ] The latest tag points to the newest stable version
[ ] There is a dev tag available that includes a shell and apk tools (by depending on 'wolfi-base')
[x] The image runs as nonroot and GID/UID are set to 65532

[ ] Alternatively the username and GID/UID may be a commonly used one from the ecosystem e.g: postgres

[x] ENTRYPOINT

[ ] For applications/servers/utilities call main program with no arguments e.g. [redis-server]
[ ] For base images leave empty
[ ] For dev variants set to entrypoint script that falls back to system

[x] CMD:

[ ] For server applications give arguments to start in daemon mode (may be empty)
[ ] For utilities/tooling bring up help e.g. –help
[ ] For base images with a shell, call it e.g. [/bin/sh]

[ ] Consider where and how the image deviates from popular alternatives. Is there a good reason and is it documented?
[x] Add annotations e.g:

annotations:
  "org.opencontainers.image.authors": "Chainguard Team https://www.chainguard.dev/"
  "org.opencontainers.image.url": https://edu.chainguard.dev/chainguard/chainguard-images/reference/busybox/ # use the academy site here
  "org.opencontainers.image.source": https://github.com/chainguard-images/images/tree/main/images/bazel # use github here

[ ] Check if environment variables are needed e.g. to set data locations
[ ] Ensure the image responds to SIGTERM

[ ] docker kill $(docker run -d --rm cgr.dev/chainguard/nginx)

[ ] Documentation. Let's make this excellent. Include usage example.
[ ] Error logs write to stderr and normal logs to stdout. DO NOT write to file.

I've just added that package because as you said "kaniko" was using these cred helpers while packing the container image, so, I thought, we can use this package while we are packaging "kaniko"

I've just added that package because as you said "kaniko" was using these cred helpers while packing the container image, so, I thought, we can use this package while we are packaging "kaniko"

Packaging it as an apk makes sense, and including that in the kaniko image, if we want to have one.
That doesn't require us to package the ACR cred helper as an image though, and I'm not sure how we'd use a cred helper packaged as an image alone. There's no upstream equivalent image to compare to, so I don't think we need this image.


[AIRFLOW-1124] Do not set all tasks to scheduled in backfill

Dear Airflow maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[X] My PR addresses the following Airflow JIRA issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-1124

Description

[X] Here are some details about my PR, including screenshots of any UI changes:

Backfill is supposed to fill in the blanks and not to reschedule
all tasks. This fixes a regression from 1.8.0.
Tests

[X] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Tests added

Commits

[X] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

@mistercrunch @aoen @criccomini

Codecov Report

Merging #2247 into master will increase coverage by 0.1%.
The diff coverage is 40%.

@@            Coverage Diff            @@
##           master    #2247     +/-   ##
=========================================
+ Coverage   68.77%   68.88%   +0.1%
=========================================
  Files         142      142
  Lines       10926    10929      +3
=========================================
+ Hits         7514     7528     +14
+ Misses       3412     3401     -11

Impacted Files
Coverage Δ

airflow/jobs.py
74.48% <40%> (+1.2%)
:arrow_up:

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 6684597...1c33ec7. Read the comment docs.