i18n(ja): Add Japanese core-concepts/project-structure.md

What kind of changes does this PR include?

[ ] Minor content fixes (broken links, typos, etc.)
[ ] New or updated content
[x] Translated content
[ ] Changes to the docs site code
[ ] Something else!

Description

Translated the Project Structure page into Japanese.
Deleted markdown file for old project structure pages
Fixed navigation labels to match the page.

Thank you, @kyosuke ! I checked this as closely as I could for anything that seemed out of place using Google Translate! Let's merge it! 🥳


Questions on visual feature aggregation

Converting a 3D asset into SLAT includes four steps:

Extract active voxels that intersect with object's surface.
Render multi-view images of the object, and extract Dino-v2 features of each image.
Back-project Dino-v2 features to the active voxels, and take average of the multiview features within each voxel.
Compress the above voxelized features into SLAT via a shallow transformer encoder.

We plan to release all the code for reproduction, involving data preprocessing, training for SLAT encoder and decoder, as well as training for rectified flow transformers. Please stay tuned!

Originally posted by @YuDeng in #81
Thanks for the great work! I am trying to implement the above four steps to convert a 3D asset into SLAT and meet some problems:

In step 2, which Dino-v2 features are used? Are they the same as the image condition feature but remove clstokens and regtokens? If this is right, the spatial resolution of the Dino-v2 feature is 37*37. https://github.com/microsoft/TRELLIS/blob/f094d9f20ee2bc2e100ecce4d3ea7ea5119d7e21/trellis/pipelines/trellis_image_to_3d.py#L145
In step 3, we can project an active voxel to each view (a total of 150 views) and get the image feature. Then, we can average the 150 image features. However, this way doesn't consider occlusion issues. Do you handle the occlusion issues in projection?
In step 4, the shallow transformer encoder is https://huggingface.co/JeffreyXiang/TRELLIS-image-large/blob/main/ckpts/slat_enc_swin8_B_64l8_fp16.safetensors ?

Looking forward to your help.

@JeffreyXiang can help with a more detailed explanation.

Hi, we just released dataset toolkits which contain code for visual feature aggregation. See dataset_toolkits/extract_feature.py for more details.

Thank you for the information!


Apple Store updates with Flutter 3.0 result in Apple removing all applications

I had a few Flutter applications under my App Store line up and they all were working fine until I started uploading the iPAS made with Flutter 3.0 using the Transporter tool
I had over 30 builds for one of my enterprise applications however, for some reason after build all of them get uploaded to the App Store the error that pops up would be
"The requested app doesn't exist"
There have been similar discussions elsewhere, is there any way to resolve this?

Desktop application may not have such a lifecycle. If there's no such lifecycle, it can't be called.

Hi @justrach. This issue might be an issue on Apple’s end. And your question is best suited for support channels like StackOverflow or Apple forums, so please ask the same there for solutions if any.
Btw, I've just seen there are some fixes that may help you:

https://stackoverflow.com/a/66580295/5921933
https://developer.apple.com/forums/thread/673860

I'm closing this issue as above explanation. If you disagree about this, please drop a comment and I will reopen it.


DNS rewrites not accepting all possible options

Adguard also supports the values 'A' and 'AAA'. These values work well if you have a wildcard such as *.example.com in Adguard but you want another.exmaple.com to be resolved upstream.
Unfortunately, this provider does not support these, here is the error I receive when I try use one of these values:
Attribute answer must be an IP address/CIDR, MAC address, or only contain numbers, lowercase letters, and hyphens, got: A

Can you please provide the HCL you are trying?

Here is an example of where I would use an A record in conjunction with a wildcard rewrite:
resource "adguard_rewrite" "wildcard-example" {
  domain = "*.example.com"
  answer = "<IP_ADDRESS>"
}

The resource that creates the error:
resource "adguard_rewrite" "blog-example" {
  domain = "blog.example.com"
  answer = "A"
}

As you can see here this should be a valid option for Adguard:

Thanks maintaining a great provider!


Convert existing tests and buildHelp.js script to TypeScript

Here, we convert our backlog of existing tests in JS to TypeScript. We also convert buildHelp.js in the same commit, which is a thing that I would normally call out myself on the theory that converting that script would merit its own separate commit. However, there is a dependency in there that's probably not readily obvious, at least, it sure took me a while to figure out why changing the tests over to TS broke the apparently unrelated script. I can go into more detail if you want about that, but it's not all that interesting.

TL;DR: the code in buildHelp.js that generated the list of JS/TS (as opposed to Markdown) files to use was erroneously including tests, i.e. spec.tsx files, which of course never came up before since there weren't any tests. I'm not 100% clear on the details, but it seems that whereas JS accepted this uncritically, TS has higher standards. Fixing that was as simple as throwing in a filter that matched all files except for tests. Everything else I did in that file was just a bit of refactoring, and adjusting a few things that the TS linter didn't like.

@rileyhgrant if you haven't yet, could you please pull this branch and manually verify that the build script is giving good results? Given that this (apparently obvious) script has been trouble a couple of times in a brief period already, I'd feel better if you double-checked this one for me on your own machine.

@rileyhgrant specifically, I'd appreciate if you could test both yarn build and yarn start and verify by eye that the help pages, especially the haplotype ones that you modified recently, haven't regressed

@rileyhgrant if you haven't yet, could you please pull this branch and manually verify that the build script is giving good results? Given that this (apparently obvious) script has been trouble a couple of times in a brief period already, I'd feel better if you double-checked this one for me on your own machine.

@phildarnowsky-broad Oops yeah, I should have clarified when I approved that things are working on my machine. Good points on wanting to be especially sure seeing as its this script.
Both running yarn start, as well as yarn build then checking the npx live-server from the gnomad-browser/browser/dist/public folder result in a functional help page on my machine. The .tsx files that get used as part of the build, and were causing issues for a tiny bit, are working, as are the .md files. The exception of the one noted in issue #969, which is not directly related to the buildHelp.ts file, and has a standalone issue filed.

@rileyhgrant great, thanks--I figured you probably had checked these on your own initiative, but I was just looking over this PR and saw that, while I had meant to ask you explicitly to do so at the time I opened the PR, apparently I forgot to do so. anyway, good to hear that we're in business, I'll merge.


Showing link in add item.

Steps:

clicks on 'add New item' button, show a link without any changes.

Still didn't work for me.


Release for OCaml 5.0

river depends on ocamlnet that is not compatible with OCaml 5, is that possible to replace this dependency or smoething else to make river available on the last compiler release?
Thanks, that would be very useful for ocaml.org!

Absolutely!
I don't think @kayceesrk or myself will have time in the near future, but I'd be happy to review a PR.

I noticed there is already a branch (500) doing that, but I cannot push on it. It's building locally with ocaml 5.0 though.

I've just released 0.4 which is compatible with OCaml 5 (thanks to @aantron). I'm closing this in anticipation of the merge of https://github.com/ocaml/opam-repository/pull/26848


Improve eq operator in where function

Use case 1: find all pages where tags equals ['foo'].
Use case 2: find all pages where tags equals ['foo','bar'].
This is not possible today. You can find all pages where these tags intersect a provided slice, but you cannot compare slices with the eq operator.
The eq operator does work if your front matter is a scalar, e.g.,
title = 'something 1`
tags = 'foo'

But you can't do this:
title = 'something 2`
tags = ['foo']

You should be able to get both of the pages above with:
{{ where .Pages "Params.tags" eq "foo" }} OR
{{ where .Pages "Params.tags" eq "foo" (slice "foo"}}

Use case 1: find all pages where tags equals ['foo'].
Use case 2: find all pages where tags equals ['foo','bar'].

The above is rather technical use cases ...
I can understand if you want to find all pages tagged with foo, but finding all pages which has the tags foo and bar and only those, that's not a use case I have seen.

finding all pages which has the tags foo and bar and only those, that's not a use case I have seen

Understood. Np problem. If we can do this:
{{ where .Pages "Params.tags" eq "foo" }} OR
{{ where .Pages "Params.tags" eq "foo" (slice "foo"}}

...then we can do it again again with "bar" and union the results.

Go 1.21 introduced the slices package which contains https://pkg.go.dev/slices#Equal.

Go 1.21 introduced the slices package which contains https://pkg.go.dev/slices#Equal.

The functional part of this has never been the challenge. If you look at that implementation you will find a ... loop.

Understood. Will stick with...
{{ $p := where site.RegularPages "Params.tags" "intersect" (slice "a" "b") }}
{{ range $p }}
  {{ if eq (len .Params.tags) 2 }}

{{ .LinkTitle }}

  {{ end }}
{{ end }}

To be clear:

tags = ['foo', 'bar']

{{ where .Pages "Params.tags" eq (slice "foo" "bar" }}

Would be cheapish to implement with strict ordering (['foo', 'bar'] != ['bar', 'foo']), but I'm not sure how useful that would be.

I agree that strict ordering limits its usage; not sure I'd pursue it.


Required fields show/hide error

Required field target shown when clicking on true value but it don't hide when click on false value.

Closing this one as a duplicate of #7, we're going to rebuild the way required arguments work.


remove dependency on TeX

Plot settings were using usetex=True by default which requires TeX installation. This PR makes usetex=False by default and hence requires no TeX installation for things to work smoothly.

I am closing this since the changes are moved to PR #184 from this fork to a  branch. PR from this fork was not starting the CI, PR from the branch does not have this issue.


[1464] raise error when assigning to unpermitted attributes

This changes the behaviour from the default which ignores mass-assigns to
unpermitted attribute, to one that raises an error. For an API it seems
sensible to be more strict about what's coming in and going out this way.
Context
This really starts to make sense for course updates where we are only allowed to update attributes that are in the enrichments, but even so there are fields such as code that we won't allow updating.
By raising an exceptions here it'll be more obvious in our testing that we haven't enabled it. If this gets noisy in production with false negatives, then we can always revert to just logging there.
Changes proposed in this pull request
Change configuration to raise when an unpermitted parameter is being set. Change tests that broke as a result.
Guidance to review
Some testing was done locally to ensure that sign-ins, location updates and vacancy updates still worked. So far everything appears to be good.
Checklist

[ ] Make sure all information from the Trello card is in here
[ ] Attach to Trello card
[ ] Rebased master
[ ] Cleaned commit history
[ ] Tested by running locally

 [M] Course update action in mc-be


Add documentation pages to the Sitemap

They're not included in the SiteURL so it didn't happen automatically. Also we should look into whether DocC outputs anything we can use to grab all of the documentation page paths, not just the /[owner]/[package]/[ref]/documentation page.

I wonder if it would be advantageous to support nested sitemaps?
https://developers.google.com/search/docs/advanced/sitemaps/large-sitemaps
This is where we would generate a OWNER/PACKAGE/sitemap.xml which includes all the pages for that package.
The top level sitemap then references all of those individual sitemaps instead of itself containing every single line.
It may offer more flexibility and scalability as we add more pages under the package route.

It's possible yes. Certainly if we're able to get every documentation path then this makes sense. If we're only able to add documentation to the end, I'd say not. The sitemap right now is < 500KB which isn't huge at all.
Looks like sitemaps can be up to 50MB (uncompressed) and have 50,000 entries. I'd say we're fine for now.

Imagine a world where...
On a server, after generating a set of statically transformed DocC documentation...
Where the jq command had been installed...
We used our ability to execute a command before uploading build results to execute this command...
jq "{ title: .metadata.title, path: .variants[0].paths[0] }" data/**/*.json | jq "[inputs]"

… and received this as the output…
[
  {
    "title": "SemanticVersion",
    "path": "/documentation/semanticversion/semanticversion"
  },
  {
    "title": "!=(_:_:)",
    "path": "/documentation/semanticversion/semanticversion/!=(_:_:)"
  },
  {
    "title": "...(_:)",
    "path": "/documentation/semanticversion/semanticversion/'...(_:)-40b95"
  },
  {
    "title": "...(_:)",
    "path": "/documentation/semanticversion/semanticversion/'...(_:)-bfr8"
  },
  {
    "title": "...(_:_:)",
    "path": "/documentation/semanticversion/semanticversion/'...(_:_:)"
  },
  {
    "title": "..<(_:)",
    "path": "/documentation/semanticversion/semanticversion/'.._(_:)"
  },
  {
    "title": "..<(_:_:)",
    "path": "/documentation/semanticversion/semanticversion/'.._(_:_:)"
  },
  {
    "title": " Implementations",
    "path": "/documentation/semanticversion/semanticversion/-implementations"
  },
  {
    "title": "<(_:_:)",
    "path": "/documentation/semanticversion/semanticversion/_(_:_:)-1ojsm"
  },
  {
    "title": ">(_:_:)",
    "path": "/documentation/semanticversion/semanticversion/_(_:_:)-4ftn7"
  },
  {
    "title": ">=(_:_:)",
    "path": "/documentation/semanticversion/semanticversion/_=(_:_:)-3q5ap"
  },
  {
    "title": "<=(_:_:)",
    "path": "/documentation/semanticversion/semanticversion/_=(_:_:)-9elz8"
  },
  {
    "title": "build",
    "path": "/documentation/semanticversion/semanticversion/build"
  },
  {
    "title": "Comparable Implementations",
    "path": "/documentation/semanticversion/semanticversion/comparable-implementations"
  },
  {
    "title": "CustomStringConvertible Implementations",
    "path": "/documentation/semanticversion/semanticversion/customstringconvertible-implementations"
  },
  {
    "title": "description",
    "path": "/documentation/semanticversion/semanticversion/description"
  },
  {
    "title": "Equatable Implementations",
    "path": "/documentation/semanticversion/semanticversion/equatable-implementations"
  },
  {
    "title": "init(_:)",
    "path": "/documentation/semanticversion/semanticversion/init(_:)"
  },
  {
    "title": "init(_:_:_:_:_:)",
    "path": "/documentation/semanticversion/semanticversion/init(_:_:_:_:_:)"
  },
  {
    "title": "init(from:)",
    "path": "/documentation/semanticversion/semanticversion/init(from:)"
  },
  {
    "title": "isInitialRelease",
    "path": "/documentation/semanticversion/semanticversion/isinitialrelease"
  },
  {
    "title": "isMajorRelease",
    "path": "/documentation/semanticversion/semanticversion/ismajorrelease"
  },
  {
    "title": "isMinorRelease",
    "path": "/documentation/semanticversion/semanticversion/isminorrelease"
  },
  {
    "title": "isPatchRelease",
    "path": "/documentation/semanticversion/semanticversion/ispatchrelease"
  },
  {
    "title": "isPreRelease",
    "path": "/documentation/semanticversion/semanticversion/isprerelease"
  },
  {
    "title": "isStable",
    "path": "/documentation/semanticversion/semanticversion/isstable"
  },
  {
    "title": "LosslessStringConvertible Implementations",
    "path": "/documentation/semanticversion/semanticversion/losslessstringconvertible-implementations"
  },
  {
    "title": "major",
    "path": "/documentation/semanticversion/semanticversion/major"
  },
  {
    "title": "minor",
    "path": "/documentation/semanticversion/semanticversion/minor"
  },
  {
    "title": "patch",
    "path": "/documentation/semanticversion/semanticversion/patch"
  },
  {
    "title": "preRelease",
    "path": "/documentation/semanticversion/semanticversion/prerelease"
  }
]

Wouldn't that be a wonderful world? 😂

If we send back that JSON, we could use it to generate complete sitemaps for all documentation that we host.

It can also spit it out minified with
jq "{ title: .metadata.title, path: .variants[0].paths[0] }" data/**/*.json | jq -c "[inputs]"

We could also generate the sitemap XML as part of the builder, upload the sitemap to S3 along with the documentation archives, and serve the sitemaps through our docc-proxy mechanism.

Of course, since we're running the builder the jq wouldn't even be necessary as we're in Swift at that point.

Possibly of interest. I tack on the option --emit-digest to get the Docc conversion process to dump out a nicely constrained list, matching exactly what I think you're after: docs/linkable-entities.json
I crunch that down (with that lovely jq) into a list of all the entities:
cat docs/linkable-entities.json | jq '.[].referenceURL' -r > all_identifiers.txt
(And my process has a second step where I grep-sed-rage across the results of that to transform them from identifiers into symbol names)
If you grab the .[].path from linkable-entities.json, I think you'll be in fine fettle for what you're after...
(and the -r option helpfully strips the JSON quotes from around the values)

Thanks Joe! It's funny. Before I started looking at just using the directory/file structure, I was sure I had read about an option that could output a set of doc paths. I tried looking for an option on DocC to do this but didn't find --emit-digest. It's not in the DocC documentation that I could find.
Now you remind me of its name, a quick Google later reminds me where I read about it...
https://rhonabwy.com/2022/02/10/tips-for-getting-the-most-out-of-docc/
😂

I learned it from Ethan on the Swift Forums when I was whining about not having a means to see the list of all possible symbols in a package. (I still like to dump all the symbols into a single doc and them sort them out as a curation process)


Xiaofang don't not work

Traceback (most recent call last):
File "/home/pi/.local/lib/python3.5/site-packages/miio/device.py", line 257, in send
data, addr = s.recvfrom(1024)
socket.timeout: timed out
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/pi/.local/lib/python3.5/site-packages/miio/device.py", line 257, in send
data, addr = s.recvfrom(1024)
socket.timeout: timed out
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "", line 1, in
File "/home/pi/.local/lib/python3.5/site-packages/miio/device.py", line 286, in send
return self.send(command, parameters, retry_count - 1)
File "/home/pi/.local/lib/python3.5/site-packages/miio/device.py", line 286, in send
return self.send(command, parameters, retry_count - 1)
File "/home/pi/.local/lib/python3.5/site-packages/miio/device.py", line 286, in send
return self.send(command, parameters, retry_count - 1)
File "/home/pi/.local/lib/python3.5/site-packages/miio/device.py", line 289, in send
raise DeviceException("No response from the device") from ex
miio.exceptions.DeviceException: No response from the device
Can't read output line 1 from command: python3 -c "import miio;result = miio.device.Device('<IP_ADDRESS>', '58686745624155706865793972764530').send('get_ipcprop', ['all']);print(result['p2p_id']);print(result['avID']);print(result['avPass']);"
Error connecting to camera, make sure ip and token are correct.
Error, waiting 10 seconds and trying again.

Don't waste your time on that. I investigated it and it seems that p2p_id, avID and avPass are not available any more.
Use samtap/fang-hacks for legacy cube cam or EliasKotlyar/Xiaomi-Dafang-Hacks if you have the 1S model and get an RTSP stream straight from the camera itself.


链接未经过url编码处理，在请求期间会出现崩溃，建议加入URL编码处理规则

[url stringByAddingPercentEncodingWithAllowedCharacters:
[NSCharacterSet URLQueryAllowedCharacterSet]];

建议不要使用此种方法, apple默认的处理会转换一部分rfc文档未有的字符, 可以参考afn的方法, AFNetworking/Serialization/AFURLRequestSerialization.h
/**
 Returns a percent-escaped string following RFC 3986 for a query string key or value.
 RFC 3986 states that the following characters are "reserved" characters.
    - General Delimiters: ":", "#", "[", "]", "@", "?", "/"
    - Sub-Delimiters: "!", "$", "&", "'", "(", ")", "*", "+", ",", ";", "="

 In RFC 3986 - Section 3.4, it states that the "?" and "/" characters should not be escaped to allow
 query strings to include a URL. Therefore, all "reserved" characters with the exception of "?" and "/"
 should be percent-escaped in the query string.
    - parameter string: The string to be percent-escaped.
    - returns: The percent-escaped string.
 */
NSString * AFPercentEscapedStringFromString(NSString *string) {
    static NSString * const kAFCharactersGeneralDelimitersToEncode = @":#[]@"; // does not include "?" or "/" due to RFC 3986 - Section 3.4
    static NSString * const kAFCharactersSubDelimitersToEncode = @"!$&'()*+,;=";

    NSMutableCharacterSet * allowedCharacterSet = [[NSCharacterSet URLQueryAllowedCharacterSet] mutableCopy];
    [allowedCharacterSet removeCharactersInString:[kAFCharactersGeneralDelimitersToEncode stringByAppendingString:kAFCharactersSubDelimitersToEncode]];

	// FIXME: https://github.com/AFNetworking/AFNetworking/pull/3028
    // return [string stringByAddingPercentEncodingWithAllowedCharacters:allowedCharacterSet];

    static NSUInteger const batchSize = 50;

    NSUInteger index = 0;
    NSMutableString *escaped = @"".mutableCopy;

    while (index < string.length) {
#pragma GCC diagnostic push
#pragma GCC diagnostic ignored "-Wgnu"
        NSUInteger length = MIN(string.length - index, batchSize);
#pragma GCC diagnostic pop
        NSRange range = NSMakeRange(index, length);

        // To avoid breaking up character sequences such as 👴🏻👮🏽
        range = [string rangeOfComposedCharacterSequencesForRange:range];

        NSString *substring = [string substringWithRange:range];
        NSString *encoded = [substring stringByAddingPercentEncodingWithAllowedCharacters:allowedCharacterSet];
        [escaped appendString:encoded];

        index += range.length;
    }

	return escaped;
}


wrong class for Visual bell in documentation

in that section https://thomas-lowry.github.io/figma-plugin-ds/#visual-bell
Loading has wrapper

which must be
 instead.

Thanks for flagging. Are you using this anywhere? I was looking to deprecate this component because there is now a msging system in Figma. If you have a good use case and are using it, I can keep it as part of the package.

Sorry, I didn't get, what do you mean "msging system"? I'm using the component to show progress status.

When I first created this, and added this component, there was no ability to push a toast to Figma.
figma.notify('toast message here');
But Figma has added that since. There is no loading state or warning state option currently provided by Figma, but it reduces the need for this. If you are still using this I will keep it.
Thanks!


Update persistantvolumeclaim.yaml

No need to create a PVC if existingPersistentVolumeClaim there

Hi @Fran-Rg,
Thanks for your contribution! I will merge and release it.
/cc @all-contributors please add @Fran-Rg for code 👌🏻


docs: add FAQ of how to enable spice data set

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license
Summary
(describe what this merge request does)
Implementation highlights
(describe how the merge request does for feature changes, share the RFC link if it has)
Test checklist

[ ] add new test cases
[ ] all code changes are covered by unit tests
[ ] end-to-end tests

[ ] deploy web console with CloudFront + S3 + API gateway
[ ] deploy web console within VPC
[ ] deploy ingestion server

[ ] with MSK sink
[ ] with KDS sink
[ ] with S3 sink

[ ] deploy data processing
[ ] deploy data modeling

[ ] new Redshift Serverless
[ ] provisioned Redshift
[ ] Athena

[ ] deploy with reporting
[ ] streaming ingestion

[ ] with Redshift Serverless
[ ] with provisioned Redshift

Is it a breaking change

[ ] add parameters without default value in stack
[ ] introduce new service permission in stack
[ ] introduce new top level stack module

Miscellaneous

[ ] introduce new symbol link source file(s) to be shared among infra code, web console frontend, and web console backend

SonarQube Quality Gate Result

Result: :white_check_mark: OK
Triggered by @llmin on pull_request

Metric
Status
Value
Error Threshold

Reliability rating
:white_check_mark: OK
1
> 1

Security rating
:white_check_mark: OK
1
> 1

Sqale rating
:white_check_mark: OK
1
> 1

Coverage
:white_check_mark: OK
83.50
< 80

Duplicated lines density
:white_check_mark: OK
5.60
> 30

Blocker violations
:white_check_mark: OK
0
> 0

Bugs
:white_check_mark: OK
0
> 0

Code smells
:white_check_mark: OK
1
> 40

Critical violations
:white_check_mark: OK
0
> 0

Major violations
:white_check_mark: OK
0
> 0

Vulnerabilities
:white_check_mark: OK
0
> 0

View on SonarQube
updated: 5/15/2024, 08:35:45 (UTC+0)

SonarQube Code Analytics
Quality Gate passed

Additional information
The following metrics might not affect the Quality Gate status but improving them will improve your project code quality.
Issues
 0 Bugs
 0 Vulnerabilities
 0 Code Smells
Coverage and Duplications
No data Coverage
No data Duplication

SonarQube Quality Gate Result

Result: :white_check_mark: OK
Triggered by @zxkane on pull_request

Metric
Status
Value
Error Threshold

Reliability rating
:white_check_mark: OK
1
> 1

Security rating
:white_check_mark: OK
1
> 1

Sqale rating
:white_check_mark: OK
1
> 1

Coverage
:white_check_mark: OK
83.40
< 80

Duplicated lines density
:white_check_mark: OK
5.60
> 30

Blocker violations
:white_check_mark: OK
0
> 0

Bugs
:white_check_mark: OK
0
> 0

Code smells
:white_check_mark: OK
1
> 40

Critical violations
:white_check_mark: OK
0
> 0

Major violations
:white_check_mark: OK
0
> 0

Vulnerabilities
:white_check_mark: OK
0
> 0

View on SonarQube
updated: 5/15/2024, 16:12:32 (UTC+0)

SonarQube Quality Gate Result

Result: :white_check_mark: OK
Triggered by @zxkane on pull_request

Metric
Status
Value
Error Threshold

Reliability rating
:white_check_mark: OK
1
> 1

Security rating
:white_check_mark: OK
1
> 1

Sqale rating
:white_check_mark: OK
1
> 1

Coverage
:white_check_mark: OK
83.40
< 80

Duplicated lines density
:white_check_mark: OK
5.60
> 30

Blocker violations
:white_check_mark: OK
0
> 0

Bugs
:white_check_mark: OK
0
> 0

Code smells
:white_check_mark: OK
1
> 40

Critical violations
:white_check_mark: OK
0
> 0

Major violations
:white_check_mark: OK
0
> 0

Vulnerabilities
:white_check_mark: OK
0
> 0

View on SonarQube
updated: 5/15/2024, 16:15:59 (UTC+0)

SonarQube Quality Gate Result

Result: :white_check_mark: OK
Triggered by @zxkane on pull_request

Metric
Status
Value
Error Threshold

Reliability rating
:white_check_mark: OK
1
> 1

Security rating
:white_check_mark: OK
1
> 1

Sqale rating
:white_check_mark: OK
1
> 1

Coverage
:white_check_mark: OK
83.40
< 80

Duplicated lines density
:white_check_mark: OK
5.60
> 30

Blocker violations
:white_check_mark: OK
0
> 0

Bugs
:white_check_mark: OK
0
> 0

Code smells
:white_check_mark: OK
1
> 40

Critical violations
:white_check_mark: OK
0
> 0

Major violations
:white_check_mark: OK
0
> 0

Vulnerabilities
:white_check_mark: OK
0
> 0

View on SonarQube
updated: 5/15/2024, 16:17:48 (UTC+0)


Windows +<EMAIL_ADDRESS>SafeQL crashes with error code 127, error code<PHONE_NUMBER>

Describe the bug
SafeQL throws wrong errors with<EMAIL_ADDRESS>on Windows and MacOS.
Issue also reported in VSCode ESlint repo:

https://github.com/microsoft/vscode-eslint/issues/1900

Misleading errors:

incorrect table name (user instead of users)
missing comma (should be after users.id)
curly brackets instead of parentheses (after ON)

On macOS ESlint throws these errors:
➜  safeql-eslint-broken git:(main) ✗ pnpm eslint . --max-warnings 0
(node:48991) ExperimentalWarning: Importing JSON modules is an experimental feature and might change at any time
(Use `node --trace-warnings ...` to show where the warning was created)
Warning: React version was set to "detect" in eslint-plugin-react settings, but the "react" package is not installed. Assuming latest React version for linting.
Pages directory cannot be found at /Users/lukas/Documents/UpLeveled/safeql-eslint-broken/pages or /Users/lukas/Documents/UpLeveled/safeql-eslint-broken/src/pages. If using a custom path, please configure with the `no-html-link-for-pages` rule in your eslint config file.

/Users/lukas/Documents/UpLeveled/safeql-eslint-broken/database.ts
  13:0  error  Invalid Query: Error: syntax error at or near "."  @ts-safeql/check-sql
  35:0  error  Invalid Query: Error: syntax error at or near "{"  @ts-safeql/check-sql

On Windows ESlint throws an error code 127  "command not found"
$ pnpm eslint . --max-warnings 0
(node:11440) ExperimentalWarning: Importing JSON modules is an experimental feature and might change at any time
(Use `node --trace-warnings ...` to show where the warning was created)
$ echo $?
127

On Windows with these issues, the ESLint VSCode extension crashes after 5 times in 3 minutes

To Reproduce

Clone repo git clone https://github.com/ProchaLu/safeql-eslint-broken
set .env variables
migrate up
Check the code and errors in the database.ts file

Expected behavior
The expected behavior would be that SafeQL throws correct errors.

incorrect table name (user instead of users)

missing comma (should be after users.id)
The error should lead to the missing comma.

export async function getUsers1() {
  return await sql<Users[]>`
    SELECT
      users.id,
      users.name
    FROM
      users
  `;
}

curly brackets instead of parentheses (after ON)
The error should lead to the wrong use of curly brackets.

export const getUsers3 = async (sessionToken: string) => {
  const users = await sql<{ id: number; name: string }[]>`
    SELECT
      users.*
    FROM
      users
      INNER JOIN sessions ON (
        sessions.token = ${sessionToken}
        AND expiry_timestamp > now()
      )
  `;
  return users;
};

OS: Windows
PostgreSQL version 15.7

These are not issues with SafeQL but syntax errors on your side
You're missing a comma in your first query
You're using {} instead of () in your second query

Using the research strategy from issue #261, I've decoded the error code<PHONE_NUMBER> to the hex code 0xC0000409, which is more googlable.
How do you diagnose the exception code 0xc0000409 on Windows? | Stack Overflow

0xc0000409 means STATUS_STACK_BUFFER_OVERRUN.
In other words, something in your program is writing past the current stack frame, corrupting data on the stack. The program has detected this and rather than let it continue, has thrown an exception.

I've also reported this in a new libpg-query issue, to surface the crashes over there too:

https://github.com/launchql/libpg-query-node/issues/81


(PDOC-75) Work with both versions of 'interpret_any'

Prior to this commit, strings would fail with puppet 4.4.0 and
newer. This was because strings was making use of a method that was
marked API private and thus subject to change. The method was changed
in 4.4.0 release of puppet to take two variables, meaning that strings
much adjust how it calls the method based on the version of puppet
it's running with.

@hlindberg @kylog Updated and tests are passing!


Docker build fails

Branch: master
Build command: docker build -t semantic-search .

Sending build context to Docker daemon  1.804MB
Step 1/4 : FROM tiangolo/python-machine-learning:cuda9.1-python3.7
...

  Installing build dependencies: started
  Installing build dependencies: finished with status 'done'
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/opt/conda/lib/python3.7/tokenize.py", line 447, in open
        buffer = _builtin_open(filename, 'rb')
    FileNotFoundError: [Errno 2] No such file or directory: '/tmp/pip-install-a74vq7au/orjson/setup.py

@JohnGiorgi any clue what's going on here?

@jvwong I think I have fixed it. I will open a PR


ENH: allow calling RemoveObserver() from a const itkObject

Exposes a const version of RemoveObserver() API just like the already existing AddObserver() API here .
Helps with application architecture when building observer design pattern with itkObject as subject.
@mentions: @thewtex - I apologize for mentioning you without any sort of prior discussion. I'm a new contributor, and would be very grateful for some feedback on this PR!
PR Checklist

[ ] No API changes were made (or the changes have been approved)
[X] No major design changes were made (or the changes have been approved)
[ ] Added test (or behavior not changed)
[ ] Updated API documentation (or API not changed)
[ ] Added Python wrapping to new files (if any) as described in ITK Software Guide Section 9.5
[ ] Added ITK examples for all new major features (if any)

Thanks for reviewing this change!

You need to fix compile errors, of course.

@tm9k1 thank you for the contribution. :pray:  Please also exercise the addition in the test suite.

Thanks so much for the prompt review! I've added a couple lines within the basic architecture test suite to include testing of the const version of RemoveObserver.
Given that I just wanted to contribute in-passing, I haven't looked much into the way of working of the team here beyond your feedback and a quick look over the general contribution guidelines. So, please do let me know if I missed some important requirements for the patch, including the test.

for some reason the macOS build step failed after 45 minutes.. re-triggered.

for some reason the macOS build step failed after 45 minutes.. re-triggered.

@tm9k1 Just for your information, you may also retrigger a single build by adding a comment of the form /azp run <build machine>. For example: https://github.com/InsightSoftwareConsortium/ITK/pull/4650#issuecomment-2101219120 But in your case: /azp run ITK.macOS.Arm64

Wow, the beautiful unit test has failed! At https://open.cdash.org/tests/1563467945

Error in !shrink->HasObserver(itk::StartEvent())

Thanks for looking into the issue, Niels. I can confirm that your diagnosis was correct. I now remove the AnyEvent observer and then the endEvent observer as const.

@tm9k1 Thank you very much for your contribution, and for addressing our comments so carefully, Piyush!


fix: Check minor version instead of static string

This pull request replaces the static check against "nvim-0.5" with a more-dynamic
check against the minor version.
Currently, the check fails, as nvim 0.7 was released a few days ago.

As clason has stated there's no reason for current check to fail. Can you say why you think so ?
Instead of thinking has('nvim-0.6') as whether user is using nvim version 0.6 think of it as whether user has features from nvim 0.6 which is true for all versions from 0.6 .
The minor version check you're doing will actually be broken if neovim realeses 1.0 while the current has check will keep working.
There are actually several checks like this in the codebase. And I'm currently strongly considering dropping support for nvim-0.5 & 0.6 .

Like many other issues with this  deprecation warning before, the real culprit was another plugin, nvim-lua/lsp_extensions.nvim.
I just happened to find these lines first and after editing them, the warning disappeared- because I had no LSP server for lua installed, so the extension was never used. I'll see myself out.
Thanks for the great work and fast replies, have a good weekend.


Upgrade CI

Maybe we should drop 2.10 support

This might help https://github.com/pico-works/pico-hashids/pull/7

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@41dd019). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #6   +/-   ##
=========================================
  Coverage          ?   89.13%
=========================================
  Files             ?        3
  Lines             ?       92
  Branches          ?       13
=========================================
  Hits              ?       82
  Misses            ?       10
  Partials          ?        0

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 41dd019...ad9a3d1. Read the comment docs.


[Help] Conflict with MUI in Storybook

Standard preview of badge component shows like this:

But when I apply isSmall property

The preview renders with a text size issue:

After looking in navigator inspector, I noticed that .fr-badge--sm has been overriden
fr-badge--sm {
  font-size: 0.75rem; /* overriden */
  line-height: 1.25rem; /* overriden */
  min-height: 1.125rem;
  padding: 0 0.375rem;
  overflow: initial;
  max-width: 100%;
  max-height: none;
}

It appears that p:not([class^="Mui"]) is responsible of this behaviour:
p:not([class^="Mui"]) {
  font-size: 1rem;
  line-height: 1.5rem;
  margin: var(--text-spacing);
}

Preview is OK after disabling font-size and line-height styles:

After looking in project files, p:not([class^="Mui"]) selector is present in dsfr/dsfr(.min).css files which are generated while running yarn build command.
This code in patchCssForMui.ts is probably the root of the problem:
const selectorNotMui = `${selector}:not([class^="Mui"])`;

if (media === "root") {
    if (selector === "[href]") {
        return selectorNotMui;
    }

    if (selector === "p") {
        return selectorNotMui;
    }
}

I think the best approach is to fix the clash on a case by case basis with style override.

Done, thanks again for the detailed report, it helped me understand that the approach was wrong.

Your welcome, thank you for the fix :slightly_smiling_face:


fontawesome in requirements.txt :tf:

i dont even use this anywhere, ill remove it from wherever its imported


ImageRecordIter usage with autoencoder

So I am trying to create an auto encoder with large image sets and I was wondering how to create/read image records. I created a data record using im2rec as shown in http://myungjun-youn-demo.readthedocs.org/en/latest/python/io.html . But this was using standard class labels. Therefore when I use :
loss = mx.symbol.LinearRegressionOutput(data=data, label=reconstructed_data)
I get the error : mxnet.base.MXNetError: [18:40:21] src/symbol/symbol.cc:121: Symbol.InferShapeKeyword argument name softmax_label not found.
I was using mx.io.ImageRecordIter() in the following manner:
train = mx.io.ImageRecordIter(
path_imgrec = os.path.join(args.data_dir, args.train_dataset),
data_shape  = (3,256,256),
batch_size  = 128,
rand_crop   = True,
rand_mirror = True,
num_parts   = kv.num_workers,
part_index  = kv.rank)
So I am wondering :
1)if ImageRecordIter() can be used for autoencoder purposes where input data and label should be same ?
2) or is my creation mechanism through im2rec not advisable for use with  autoencoders ?

ImageRecordIter outputs 'data' and 'softmax_label'. Your model only uses 'data' but not 'softmax_label'. You need to write a DataIter that wraps around ImageRecordIter and ignores the 'softmax_label' and only output 'data'.

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


Adding an environments array field for policy and an enum for policy targets

There is now an enumeration for targets in policy. This is currently what we have in place in Rode as well as Rode-UI

Name
Number
Description

UNIVERSAL
0
default if no target field is provided

DEBIAN
1

DOCKER
2

FILE
3

MAVEN
4

NPM
5

NUGET
6

PYTHON
7

RPM
8

GIT
9

There is also a field for environments. The word all is reserved to apply to all environments. I would have made an enumeration here as well, but I think there were discussions about allowing this to be user defined.

We're going a different direction with regards to policy assignment / environments, so I'm going to close this.


Image Editor project using Js

Describe the project you want to add with tech stack
I would like to add an image editor project using vanilla JavaScript. The project will include the following features:
Four filters: brightness, saturation, inversion, and grayscale, with adjustable levels up to 100%
Options to rotate and flip the image
Reset button to undo all applied filters
Button to choose an image from the user's device
Button to save the edited image
Expected behavior
The image editor should allow users to apply various filters to their selected image, adjust the levels of brightness, saturation, inversion, and grayscale, as well as rotate and flip the image. The reset button should revert the image to its original state, and the save button should save the edited image.

Hey, @MohitGupta121 , @pranjay-poddar  Please assign me this issue under GSSOC'23 contribution.


[SPARK-27540][MLlib] Add 'meanAveragePrecision_at_k' metric to RankingMetrics

What changes were proposed in this pull request?
Add 'meanAveragePrecision_at_k' metric to RankingMetrics
How was this patch tested?
Add test to RankingMetricsSuite.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can someone verify this?

How is this related to https://github.com/apache/spark/pull/24458 ? let's have only one PR

Jenkins OK to test


in the search if i press ctrl and backspace the focus moves to the editor and deletes there

Type: Bug
in the search if i press ctrl and backspace the focus moves to the editor and deletes there
STR:

open a file
press ctrl + shift + f to open the find widget
search something
while the focus is on the search textbox, and press ctrl + backspace

observe: the focus goes to the file opened and deletes things in the open file instead of the search box.
same thing happens with ctrl + f and search in the file.
VS Code version: Code - Insiders 1.94.0-insider (d78a74bcdfad14d5d3b1b782f87255d802b57511, 2024-10-02T13:10:18.718Z)
OS version: Windows_NT x64 10.0.22621
Modes:
Remote OS version: Linux x64 4.4.0-22621-Microsoft
Remote OS version: Linux x64 4.4.0-22621-Microsoft

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8665U CPU @ 1.90GHz (8 x 2112)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabledwebnn: disabled_off

Load (avg)
undefined

Memory (System)
15.82GB (0.60GB free)

Process Argv

Screen Reader
no

VM
0%

Item
Value

Remote
WSL: Ubuntu

OS
Linux x64 4.4.0-22621-Microsoft

CPUs
Intel(R) Core(TM) i7-8665U CPU @ 1.90GHz (8 x 0)

Memory (System)
15.82GB (0.60GB free)

VM
0%

Item
Value

Remote
WSL: Ubuntu

OS
Linux x64 4.4.0-22621-Microsoft

CPUs
Intel(R) Core(TM) i7-8665U CPU @ 1.90GHz (8 x 0)

Memory (System)
15.82GB (0.61GB free)

VM
0%

Extensions (29)

Extension
Author (truncated)
Version

vscode-drawio
hed
1.6.6

output-colorizer
IBM
0.1.2

remote-wsl
ms-
0.88.4

ng-template
Ang
18.2.0

vscode-markdownlint
Dav
0.56.0

vscode-eslint
dba
3.0.10

gitlens
eam
15.5.1

EditorConfig
Edi
0.16.4

prettier-vscode
esb
11.0.0

git-graph
mhu
1.30.0

mongodb-vscode
mon
1.9.1

vscode-scss
mrm
0.10.0

vscode-dotnet-runtime
ms-
2.1.7

data-workspace-vscode
ms-
0.5.0

mssql
ms-
1.24.0

sql-bindings-vscode
ms-
0.4.0

sql-database-projects-vscode
ms-
1.4.3

playwright
ms-
1.1.7

live-server
ms-
0.4.15

vscode-typescript-next
ms-
5.7.20241002

angular-console
nrw
18.28.1

vscode-xml
red
0.27.2024082510

vscode-yaml
red
1.15.0

sonarlint-vscode
Son
4.11.0

code-spell-checker
str
3.0.1

vscode-stylelint
sty
1.4.0

win-ca
uko
3.5.1

vscodeintellicode
Vis
1.3.1

vscodeintellicode-completions
Vis
2.0.0

Hmm, I can't seem to reproduce this. Can you debug your keybindings to see what actions are triggering? https://code.visualstudio.com/docs/getstarted/keybindings#_troubleshooting-keybindings

@andreamah,
if you enable the key binding deleteWordPartLeft you should be able to reproduce the issue.

Note: i have tested this on a vscode stable release as well which is almost new (not much setting changes), i use insiders as my daily driver issue can be seen there as well.
if in case it is not reproduceble still i can provide the troubleshooting logs required.

What it say for these shortcuts in the source column? Is it user or extension-set?

user

If you run Open Keyboard Shortcuts (JSON), is that keybinding there? It seems that this is a keybinding that you set yourself.

You can use add editorFocus to the when clause in order to get it to work properly. To look at what can be tested in the when clause, you can inspect the context keys in the editor.
https://code.visualstudio.com/api/references/when-clause-contexts


[TestResources] Doc Updates and Revisions for External Use

Summary
The focus of these changes is to revise the script to better support use by external contributors and others outside of the Azure SDK ecosystem and without access to the Microsoft AAD Tenant.
Details

Creation of a new Test Application service principal is now possible   from a non-Microsoft AAD tenant.

When a new Test Application principal is created, the principle of least  privilege is now applied; the new Test Application is granted ownership of the resource group  associated with the test resources and no longer has access to any other resources in the subscription.

If an existing Test Application principal is specified, it will be assigned ownership of the resource group created.  This supports using a Test Application principal without privileges at the subscription-level.

When no provisioner is specified, the script is now executed in the  context of the caller rather than the Test Application principal.  This supports using a Test Application principal that has restricted privileges and better aligns to the purpose of the Test Application principal.

The $TestApplicationOid is now explicitly bound at the time a new Test Application principal is created rather than having to query for it later.

Common error scenarios resulting from lack of permissions now provide messaging with more context of why the failure occurred and suggest remediation.

Added new examples to illustrate the common call patterns needed by external contributors running the script outside of the Microsoft tenant and Azure SDK ecosystem.

Documentation has been enhanced with additional context to detail the permissions and roles assigned by the script.

Added documentation details for Bicep template use.

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

Key Vault's test-resources-post.ps1 is currently broken because we set the TestApplicationOid as the initial Managed HSM owner, which is a hardened security feature. This needs to include the provisioner as well because the post script needs to activate the domain. We either log in as the test application, or we add the Provisioner* properties to the $templateParameters passed to New-AzResourceGroupDeployment.
I might also be able to work around this with a pre-script that adds values to $templateFileParameters (should be in scope, I believe), but I think it better to just pass the provisioner properties through to the deployment via $templateParameters.

Key Vault's test-resources-post.ps1 is currently broken because we set the TestApplicationOid as the initial Managed HSM owner, which is a hardened security feature. This needs to include the provisioner as well because the post script needs to activate the domain. We either log in as the test application, or we add the Provisioner* properties to the $templateParameters passed to New-AzResourceGroupDeployment.

What's involved with activating the domain?   If we assign the provisioner (either passed as params or the interactive user) as the initial owner and the post- script is running in the context of the same provisioner,  would that work or is there something that needs to be done with the provisioner's identifying information?

Key Vault's test-resources-post.ps1 is currently broken because we set the TestApplicationOid as the initial Managed HSM owner, which is a hardened security feature. This needs to include the provisioner as well because the post script needs to activate the domain. We either log in as the test application, or we add the Provisioner* properties to the $templateParameters passed to New-AzResourceGroupDeployment.

An alternate thought:   If we continue to tread the test application as the administrator of the HSM,  couldn't we have the post- script log into the test app, authorize the HRM, and then restore the calling context?    The test application will have Owner rights on the RG as well and I can't think of a reason why that wouldn't work.

The test application SP is granted access via RBAC, but the initial admin OIDs are the only OIDs that can activate the MHSM, so we'd have to log in as the test application which we're trying to avoid here.
The solution is pretty simple: add the ID and OID for the provisioner - either the given or the current - to the possible template parameters just like we do with the test application ID and OID.
I'll try my changes in my local copy and contribute back the changes. Is the PR open that I can push commits? GitHub used to show this obviously, but I don't see it now.

The test application SP is granted access via RBAC, but the initial admin OIDs are the only OIDs that can activate the MHSM, so we'd have to log in as the test application which we're trying to avoid here.

I'd frame it more as we were trying to not use the Test App as the provisioner in the general case.   Given the unique constraints around this scenario,  I don't think it would be a bad thing to use it as the admin and login in the post- script.   That'd be scoped to the specific scenario.
That said, I'm not opposed to pushing the provisioner OID around either - assuming the post- script is also executing in that context, that approach would seem to be cleaner and more generic.

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

GitHub isn't showing the changes to the markdown file as a diff. What did you change?

L592,  changed http to https because Analyze was complaining.

Right - everything during this script is now run as the provisioner (explicit or current context), which is fine. The problem was that the Key Vault test-resources.json and test-resources-post.ps1 assumed the test application likely was that provisioner. Not sure, now that I've run into this problem after your (perfectly fine) changes, how it was working during live test runs by the CI which presumably passed different contexts. Or maybe it didn't. @benbp?

@heaths not quite sure what you are asking, but CI uses the same SP values for test application and provisioner.

@benbp thanks, that explains it. Test app and provisioner were assumed to be the same, but that's no longer true (which is a good change, I agree).

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

One property question and some spelling fixes, but otherwise LGTM.

If there weren't typos, nobody would believe that I wrote it.  :smile:

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

Fixing typos and spelling mistakes

Would've been funnier if you misspelled your commit title.

/check-enforcer evaluate


Broke icon on MacOS (socondary monitors only)

VS Code Version: 1.55.0
OS Version: Big Sur 11.2.3

Steps to Reproduce:

View Icon anywhere on built in screen -> No issue
View icon on any other connected screen -> Icon broke

View screenshots here: https://drive.google.com/drive/folders/1rnm40DtusNxyl_IsgzQfEe5RCVqjU8Rr?usp=sharing
The issue occured first after an update of VS Code Insiders and now after the last update of VS Code.
I am using a MacBook Pro 2019. The issue occurs on every connected screen (testet with Dell DisplayLink and without).

Does this issue occur when all extensions are disabled?: Not testet

Was just about to come and create the same issue!
VS Code Version 1.55.0 & macOS Big Sur 11.2.3 as well, macbook pro 2017

Same issue

Same issue!
VS Code Version 1.55.0 & macOS Mojave 10.14.6 as well.

Same issue!
VSCode - Version: 1.55.0 (Universal)
Macos - Version: BigSur 11.2.3 (20D91)
MacBook pro 13in (2017)
But issue is happening on both the screens, monitor and mac book screen

Same issue!
VSCode - Version: 1.55.0 (Universal)
Macos - Version: BigSur 11.2.3 (20D91)
Mac mini (2018)

Having the same issue on 11.2.3 in Launchpad Folder and Spotlight. VSCode v 1.55 on Mac Mini 2020

Same issue on me. I use macOS Sierra 10.12.4. Issue appers on built-in screen.

/duplicate of #120305


Removed duplicate issue

Purpose

Fix #317

Does this introduce a breaking change?

[ ] Yes
[X] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[X] Documentation content changes
[ ] Other... Please describe:

How to Test
Look at RaspberryPi.md, the Troubleshooting section

Test the code

What to Check
Verify that the following are valid

Spelling corrected, layout fixed, duplicate removed

Other Information

The last commit had two unwanted changes. extra bash tags where they should not be.


Introduce patrol develop command with R button for restarting the entry point

See also #982

Do we need to require developers to complete the native setup to use patrol develop?
The main reasons to remove patrol drive in favor of patrol test were:

inability to get reports in a common format like JUnit
inability to use device farm platforms (like Firebase Test Lab)

none of which make sense for local development of tests.
This would bring back one of Patrol's initial design goals – requiring 0 to minimal native setup (we had that goal abandon when we moved to patrol test). Literally, no setup would be required: just patrol develop -t integration_test/your_test.dart and you get Hot Restart and native automation.
We wouldn't use flutter drive - we don't want it anymore - but flutter run is enough for this case. Alternatively, we could maintain our own Dart VM service connection.

I totally agree with that. If this doesn't mean we have to put the double amount of work to achieve this - patrol develop completely assumes running locally and it's ok to NOT require native setup. However, there will be cases when users want to run patrol develop with their tests having native automation inside - what would happen then?
If we have to throw an exception, we should somehow tell the users (e.g. in the docs) about this limitation.

Scope:

[x] get the core functionality to work (r/R to restart app,  q/Q to quit app)
[x] pipe logs from flutter logs
[ ] don't print "error - execution interrupted" when the user does CTRL+C or Q

However, there will be cases when users want to run patrol develop with their tests having native automation inside - what would happen then?

What cases do you have in mind? I can't think of any but I probably forgot about something.


Integer overflow vulnerability in Escrow

Expected Behavior
Safemath everywhere, always.
Current Behavior
now >= (payment.timestamp + payment.expirySeconds)

reserved for evgeni@limechain


AB#146422 add unconfirmed task list section

Adds "unconfirmed" as a task list status

This is the wrong approach, as this enum is the list of statuses that are part of the GOV.UK Design System.
I have provided support for custom task statuses in plain ASP.NET by doing something like this:
<govuk-task-list-task-status status="Completed">Unconfirmed</govuk-task-list-task-status>
However that's not currently something you can configure in Umbraco, so the right approach would involve adding Umbraco support for that.

I have added a new property on the 'task' block called 'Status text'. Setting a value here will override the value from the dictionary which makes the status independent from the status text.
This property can be overrode in code.


[Feature Request]: Reserve some count public IPv4 / IPv6 for whitelisting

Description
On GKE in our company we use next case:
We reserve 10-20 IPv4 but cluster use only 4 IPv4, but when autoscaling add new cluster node it take from reserve IPv4.
That need when service communicate with external service where need add IPv4 to whitelist. And need to be add all IPv4 nodes. And when k8s autoscaling up or down we don't have issue with access to external services.
Also we have stateful service where run on each k8s node for balance requests to the external services, count stateful services depends with count running pods in cluster.
I think that case happened often on other projects where services communicate with externals SaaS platforms.
Thank you!

We deployed k8s on hetzner and move services from GKE but we need access to Google secrets.
And some cluster nodes got IP from iran. We discuss with Hetzner support and they saying can't verify own IP database.
I add additional 2 nodes use terraform and remove 2 nodes with iran IPs. But now I have problem with terraform update.
  # module.kube-hetzner.module.agents["1-5-agent-large"].random_string.server will be destroyed
  # (because module.kube-hetzner.module.agents["1-5-agent-large"] is not in configuration)
  - resource "random_string" "server" {
      - id          = "kqm" -> null
      - keepers     = {
          - "name" = "h-k3s-staging-agent-large"
        } -> null
      - length      = 3 -> null
      - lower       = true -> null
      - min_lower   = 0 -> null
      - min_numeric = 0 -> null
      - min_special = 0 -> null
      - min_upper   = 0 -> null
      - number      = false -> null
      - numeric     = false -> null
      - result      = "kqm" -> null
      - special     = false -> null
      - upper       = false -> null
    }

Plan: 6 to add, 0 to change, 14 to destroy.

Do you want to perform these actions?
  Terraform will perform the actions described above.
  Only 'yes' will be accepted to approve.

Better need solution for manages IPv4 / IPv6 for reservation.
============
Dear Client,
Thank you for your request.
Unfortunately, some GeoIP databases incorrectly locate some of our IPs in Iran. We cannot influence these databases.
If this leads to issues for you, please create a Snapshot of the server with the incorrect IP location. Then create a new server with this Snapshot. You can then delete the server with the incorrect IP location.
Kind regards
Kevin-Damian Gosa
Hetzner Online GmbH
Industriestr. 25
91710 Gunzenhausen / Germany
Tel: +49 9831 505-0
Fax: +49 9831 505-3
www.hetzner.com
========
Dear Client,
thanks for your message.
Unfortunately such a temporary blockade is unfortunately beyond our control, since this is a decision by the respective service provider.
There is a workaround to change the IP address of your Cloud-Server. To do this, create new servers before deleting the old ones. (If you have data on the old server, take a snapshot before deleting it). Each server should have a new IP address. Just keep the server that is not blocked.
Optional:
To restore the snapshot of your old server, click on the new server and select the 'REBUILD' section.
You should find the snapshot at the bottom of the Image drop down list. Select it and click REBUILD.
Should you require any further assistance, please do not hesitate to contact us.
Kind regards
Tim Stich
Dear Client,
thanks for your answer.
Unfortunately, I have to inform you that it is not offered to remove any IPs from the rotation. In principle, these will be reserved for you for 24 hours, after which alternatives will be issued.
Kind regards
Tim Stich
Hetzner Online GmbH
Industriestr. 25
91710 Gunzenhausen / Germany
Tel: +49 9831 505-0
===================

@CroutonDigital  Yes, this is a real problem with Hetzner IPs. What you could do is use the cilium egress gateway. If you egress from a node that is clean, normally you won't have any issues. Please try it. See kube.tf.example.

@CroutonDigital There is another trick, it's to use reserved IPs to check. Basically hetzner allows you to reserved IPs, you grab one, and you try to pull a container from google registry, if it fails, you reserve keep the IP reserved, if it succeeds you free it. What will happen is that the free IPs will be now just for you to use (that's how Hetzner works, I do not how long it stays for you, but at least  a few hours). So you find as much free IPs as you need to deploy your cluster. Then you deploy, and released all iran IPs that you found and kept reserved. Please if you managed to write a script that does just that, PR most welcome!

Hi,
Some our services in k8s make more requests to external services where need balance requests. service on k8s run on each node and when start get Secrets from Google Secrets.
I make blacklist but not easy workflow.

I create cluster and check external ips on node;
Remove cluster node with wrong IPs;
Make reservation IP use rename IP;
and make terraform apply for created removed k8s nodes with other IPv4;

but some issue when cluster auto add new k8s nodes, and now I plan setup pod distribution for services who access to Google Secrets run on core k8s node.
Can I use egress with multiply IPv4?
Thank you!

I think cilium egress can work with multiple nodes/ips, of course.


Command still hasn't been written into connection! Increase nettyThreads and/or retryIntervalNode source:

Hi，i used redisson-spring-boot-starter in springboot 2.1.12-release with redis Sentinel mode ,1 master and 2 slave. The following error log is encountered while the program is running.  how can i fix this? thanks !
my configration like this:
spring:
redis:
host: <IP_ADDRESS>
port: 6378
password:  redis
sentinel:
master: mymaster
nodes: <IP_ADDRESS>:26379, <IP_ADDRESS>:26379, <IP_ADDRESS>:26379
lettuce:
pool:
max-active: 32
max-wait: 10000ms
max-idle: 32
min-idle: 0
org.springframework.dao.InvalidDataAccessApiUsageException: Command still hasn't been written into connection! Increase nettyThreads and/or retryIntervalNode source: NodeSource [slot=0, addr=null, redisClient=null, redirect=null, entry=MasterSlaveEntry [masterEntry=[freeSubscribeConnectionsAmount=1, freeSubscribeConnectionsCounter=value:50:queue:0, freeConnectionsAmount=1, freeConnectionsCounter=value:0:queue:64, freezed=false, freezeReason=null, client=[addr=redis://<IP_ADDRESS>:6379], nodeType=MASTER, firstFail=0]]], connection: RedisConnection@2115931524 [redisClient=[addr=redis://<IP_ADDRESS>:6379], channel=[id: 0x64f5fb71, L:/<IP_ADDRESS>:52154 - R:<IP_ADDRESS>/<IP_ADDRESS>:6379], currentCommand=null], command: (HGET), params: [[53, 69, 56, 57, 53, 48, 48, 51, 45, 57, ...], [54, 69, 54, 55, 52, 68, 53, 69, 45, 48, ...]] after 3 retry attempts; nested exception is org.redisson.client.RedisTimeoutException: Command still hasn't been written into connection! Increase nettyThreads and/or retryIntervalNode source: NodeSource [slot=0, addr=null, redisClient=null, redirect=null, entry=MasterSlaveEntry [masterEntry=[freeSubscribeConnectionsAmount=1, freeSubscribeConnectionsCounter=value:50:queue:0, freeConnectionsAmount=1, freeConnectionsCounter=value:0:queue:64, freezed=false, freezeReason=null, client=[addr=redis://<IP_ADDRESS>:6379], nodeType=MASTER, firstFail=0]]], connection: RedisConnection@2115931524 [redisClient=[addr=redis://<IP_ADDRESS>:6379], channel=[id: 0x64f5fb71, L:/<IP_ADDRESS>:52154 - R:<IP_ADDRESS>/<IP_ADDRESS>:6379], currentCommand=null], command: (HGET), params: [[53, 69, 56, 57, 53, 48, 48, 51, 45, 57, ...], [54, 69, 54, 55, 52, 68, 53, 69, 45, 48, ...]] after 3 retry attempts
at org.redisson.spring.data.connection.RedissonExceptionConverter.convert(RedissonExceptionConverter.java:48)
at org.redisson.spring.data.connection.RedissonExceptionConverter.convert(RedissonExceptionConverter.java:35)
at org.springframework.data.redis.PassThroughExceptionTranslationStrategy.translate(PassThroughExceptionTranslationStrategy.java:44)
at org.redisson.spring.data.connection.RedissonConnection.transform(RedissonConnection.java:224)
at org.redisson.spring.data.connection.RedissonConnection.syncFuture(RedissonConnection.java:219)
at org.redisson.spring.data.connection.RedissonConnection.sync(RedissonConnection.java:449)
Ps：the version of redis is 5.0.8
Thanks

Did you try to increase nettyThreads?

sorry , 1 don't know how to config  the nettyThreads of Sentinel mode.
it like follow config?
spring:
redis:
nettyThreads: 100

some body help me ,thanks

1 don't know how to config the nettyThreads of Sentinel mode.
it like follow config?

common spring boot settings don't allow to do this. You need to either use Redisson config file or Redisson settings. Examples are here https://github.com/redisson/redisson/tree/master/redisson-spring-boot-starter


(PUP-11869) Retry failed CA/CRL refreshes sooner

Prior to this commit, any time a node attempted to refresh its certificate authority (CA) or certificate revocation list (CRL), regardless of whether it was successful in doing so, Puppet would update the modified timestamp (mtime) on the CA or CRL file.
Puppet uses the mtime to decide when to attempt to update a CA or CRL. As a result, if Puppet attempted but failed to updates either of those file, it would need to wait the full refresh interval before trying again.
This commit changes Puppet to only update the modified timestamp on the CA or CRL files if they have been successfully refreshed. This means that if Puppet fails to update its CA or CRL, it will try again the next run interval instead of waiting for the full CA or CRL refresh interval (which should be greater than the run interval).

LGTM, might be good to add a test (for the CA and CRL) to show that the last modified time isn't updated if there's a failure. It looks like we already test the positive case here (for CA) https://github.com/puppetlabs/puppet/blob/6b400f35193e5d6871fd679d6ad49776d1cb6072/spec/unit/ssl/state_machine_spec.rb#L490
The negative case might be something like:
stub_request(:get, %r{puppet-ca/v1/certificate/ca}).to_raise(Errno::ECONNREFUSED)

expect_any_instance_of(Puppet::X509::CertProvider).to receive(:ca_last_update=).never

state.next_state


MNT: rerender

Hi! This is the friendly automated conda-forge-webservice.
I've rerendered the recipe as instructed in #119.
Here's a checklist to do before merging.

[ ] Bump the build number if needed.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


disableMixerHttpReports: does not remove http filter from inbound.

With  meshconfig.disableMixerHttpReports: true
mixerClient http filter is not inserted in any outbound listeners.
However, it is still inserted in the inbound listener.
disable_report_calls: true is present, but filter should not be injected at all.

Enabling TCP only shows workload as unknown:unknown

is the problem that the filter is present, or that the report calls are still happening?  the code merely sets a value in the service config for the filter, it does not remove the filter altogether.
Removing the filter altogether seems problematic as Check and Report(TCP) may still be desired.


Update fields.mjs to v9.235

Closes #988.

This one can be reviewed an merged without any prerequisites.


ci(openshift): Increase test timeout

Signed-off-by: Kalya Subramanian<EMAIL_ADDRESS>
Description:
Increase e2e test timeout as 3h is insufficient
Potentially closes #5242

Testing done:

Affected area:

Functional Area

New Functionality
[ ]

CI System
[X]

CLI Tool
[ ]

Certificate Management
[ ]

Control Plane
[ ]

Demo
[ ]

Documentation
[ ]

Egress
[ ]

Ingress
[ ]

Install
[ ]

Networking
[ ]

Observability
[ ]

Performance
[ ]

SMI Policy
[ ]

Security
[ ]

Sidecar Injection
[ ]

Tests
[ ]

Upgrade
[ ]

Other
[ ]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project?

Did you notify the maintainers and provide attribution?
n/a

Is this a breaking change?
no
Has documentation corresponding to this change been updated in the osm-docs repo (if applicable)?
n/a

Codecov Report

Merging #5248 (a6d675e) into main (376a826) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #5248   +/-   ##
=======================================
  Coverage   70.30%   70.30%
=======================================
  Files         195      195
  Lines       15657    15657
=======================================
  Hits        11007    11007
  Misses       4595     4595
  Partials       55       55

Flag
Coverage Δ

unittests
70.30% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


REQUESTS ARE NOT EXAMPLES - THEY ARE SYNTAX

syntax:
Get-MgSiteListItem -SiteId $siteId -ListId $listId -ExpandProperty "fields(select=Name,Color,Quantity)"
example:
$ContosoIceCreamSiteID = '[please provide an example; either a GUID, or how to get the GUID]'
$ContosoIceCreamSiteFlavorsID = '[please provide an example; either a GUID, or how to get the GUID]'

Get-MgSiteListItem -SiteId $ContosoIceCreamSiteID -ListId $ContosoIceCreamFlavorsID -ExpandProperty "fields(select=name,allergens,vegan)"

[Enter feedback here]

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

ID: 2524650e-06e7-dc8c-b2ae-0c2fbffdb541
Version Independent ID: 3eb41d73-2b60-1459-e921-20403004655c
Content: Retrieve items from a SharePoint list - Microsoft Graph v1.0
Content Source: api-reference/v1.0/api/listitem-list.md
Product: sharepoint
Technology: microsoft-graph
GitHub Login: @spgraph-docs-team
Microsoft Alias: MSGraphDocsVteam

Hi,
Thanks for your feedback. We are currently working on improving docs and we'll do our best to keep your feedback into account while doing that. In the meantime, I'm going to close this item. Thanks for your understanding.


fix attr cop implementation bbatsov/rubocop#2483

extend the attr cop implementation to respect the additional boolean param.
Fixes #2483

Looks good! I would suggest that you squash these 3 commits into a single commit. Do you know how to do this?

Update the commit message to match our style - [Fix #...] Differentiate...

i merged the two if s. don't know why i did not see that before.
the "default value" thing was introduced to avoid a too long line (87).
i think the changed version should be clearer now.
can you explain me why the build failed?
https://travis-ci.org/bbatsov/rubocop/jobs/96621263

@bbatsov do you know why the build failed?

do you know why the build failed?

There's a bug in the new version of the bundler gem, which has broken the build for every single Ruby project which runs tests on JRuby.

The commit message should still be updated.

:+1:

Great job, @weh!


add option for ignoring ssl_verify

I encountered an error when requesting to ghe, because our GitHub Enterprise uses self-signed certificate.
So we need the option for ignoring ssl_verify.

Thank you Yusuke :)
Could you please confirm this contribution is under the terms of the Apache 2.0 license? Thanks.

Yeah, I confirmed it.
Is there anything I have to apply for?

Nope - merged in. Thank you :)


In hard shutdown ND wheels fail to remember their position

Does homing solve this in a way that is Workable (if not ideal)?

Yes.
On Mon, Apr 23, 2018 at 9:01 AM, Kyle Sunden<EMAIL_ADDRESS>wrote:

Does homing solve this in a way that is Workable (if not ideal)?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/wright-group/PyCMDS/issues/167#issuecomment-383586059,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APD6EYqlTz2uVbZKK0D7kT43tsRvV_j7ks5trd7SgaJpZM4Tf8bk
.


Expose github-resource and task-toolbox digests for use by tenants

... and pin task-toolbox used in configure-namespace
So people with write access to DockerHub can't change code running in the live clusters.

I do worry about what effect this may have on on-demand clusters as they need to pull the latest image, with this they will probably end up with the concourse resource version digest being empty string and never finding a matching version to do anything...

https://github.com/concourse/concourse/issues/6166


Wallet client doesn't handle RBF deposits

When a deposit transaction uses RBF the new txid is not monitored. This prevents users from being credited once the RBF transaction is confirmed.
More context in this discord thread.

I'm prioritizing this fix and we can hopefully include in 0.3.2.
cc: @elsirion

Watching. I can help test and review with this when it's ready too.

Any updates here?

Any updates here?

We prioritized fixing another issue we discovered that prevents the wallet from recognizing change outputs (https://github.com/fedimint/fedimint/issues/5298). I'm currently working on this and will hopefully have a fix ready next week.
@benthecarman I see in the referenced issue the federation is shutting down June 6th. I don't know if there will be enough time to cut a releases with the fix since we're still working through the release cycle for 0.3.2.

During the most recent deep dive call, we discussed multiple approaches for a fix and decided to go with a more holistic refactor of the wallet client deposit state machine that we'll aim to get in 0.4. @dpc has begun work on this approach, so I'm reassigning this issue.
References:

Experimental branches

https://github.com/bradleystachurski/fedimint/tree/rbf-deposits
https://github.com/bradleystachurski/fedimint/tree/sandbox/mempool-behavior

Recording (https://bitcointv.com/w/82LCumWCbnRbL7NLwTL3Gg)

Closed by https://github.com/fedimint/fedimint/pull/5473


Adding events doesn't work with security plugin enabled

Describe the bug
After enabling security plugin, all events goes to global tenant regardless of the current tenant
To Reproduce
Steps to reproduce the behavior:
go to admin private tenant
create a channel first
create an event using send test message
GET /_plugins/_notifications/feature/test/<channel-id>
response:
{
  "event_id" : "<event-id>"
}

query the event id, see error
GET /_plugins/_notifications/events/pOrN7noBi0kFiZoXkBqz
response:
{
  "error" : {
    "root_cause" : [
      {
        "type" : "status_exception",
        "reason" : "Permission denied for NotificationEvent pOrN7noBi0kFiZoXkBqz"
      }
    ],
    "type" : "status_exception",
    "reason" : "Permission denied for NotificationEvent pOrN7noBi0kFiZoXkBqz"
  },
  "status" : 403
}

and switch to admin global tenant, the same event API works.
Configs (channels, senders, email groups) are working as expected.

Expected behavior
A clear and concise description of what you expected to happen.
Plugins
Please list all plugins currently enabled.
Screenshots
If applicable, add screenshots to help explain your problem.
Host/Environment (please complete the following information):

OS: [e.g. iOS]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

closing since we support backend roles instead of tenants now


Bump Dependencies #5379769702

Bumps dependencies for product-is. Link : https://github.com/wso2/product-is/actions/runs/5379769702

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.jenkins-is-staging seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Add non-breaking spaces between units

Proposed documentation enhancement
There should be a non-breaking space between the number and its unit. I don't know if this is supported here, I would use either   or   (preferably the latter). This should be consistent throughout our documents.
e.g. 250 ms

Is this issue still open?  I would like to fix this if still open.

Yes, this is still open. Have fun with it! :)

Can you give me a little direction? (or some resources)

Should I change across all  files? or specific type files? (python/html/rst)?
How do I check which space is used? (is there some tool or some way)

I am actually new in this project, and a little uninformed about variation of spaces.
Probably I am asking for very beginner stuff. But I  an eager to contribute to this project.

Throughout our documentation, so that is the various .rst files in the doc/ folder and the RST markup code in the .py files in the examples and tutorials folders, we have text explaining things. This text occasionally contains numbers with a unit. For example 100 ms, or 23 µV. At the moment, there is usually a normal, breaking, space between the number and the unit. Sometimes this causes awkward linebreaks. It would be better if we were to use a non-breaking space between them.
You can tell what type of space it is because anything other than a regular space probably needs to be written as a unicode directive in order for the documentation system to output a nonbreaking space. Question is how to exactly do this. This StackOverflow tread has some hints: https://stackoverflow.com/questions/11830242/how-can-i-make-a-non-breaking-space-in-restructuredtext
If it is true that we need some sort of custom RST substitution command to generate nonbreaking spaces, we might want to make a sphinx extension that defines a custom command for "units", so we could do something like :unit:`ms`

FYI one way it's done (using html escape &nbsp) is seen in this tutorial:
https://github.com/mne-tools/mne-python/blob/11bc09911a2dbb6b73a86b31d8140ccd8a93b1c3/tutorials/preprocessing/25_background_filtering.py#L728
Other approach (using a \uXXXX style escape, in this case for optional hyphens) is seen here:
https://github.com/mne-tools/mne-python/blob/11bc09911a2dbb6b73a86b31d8140ccd8a93b1c3/doc/conf.py#L880-L884
Yet another approach is the rST substitution syntax:
https://github.com/mne-tools/mne-python/blob/11bc09911a2dbb6b73a86b31d8140ccd8a93b1c3/doc/conf.py#L1074
git grep "|ensp|" will show you that one in-use.
All of those seem to me to make the rST source rather difficult to read. Making a custom rST interpreted text role (this is the :unit:`ms`  style mentioned above) might be the nicest approach, especially if it were designed to take both the number and the unit, like this: :unit:`5 ms`  and automatically replace the ordinary space with a non-breaking (possibly thin) space.

there's already some custom commands defined in doc/sphinxext you could use as example for how to create a new command.

:+1: for thin space!

Thank you for these resources.

After taking  some time for studying about rst and sphinx,I came up  with this custom role.
from docutils import nodes

def unit_role(name, rawtext, text, lineno, inliner, options={}, content=[]):
    try:
        value = float(text.split(' ')[0])
        if len(text.split(' ')) != 2:
            raise ValueError
    except ValueError:
        msg = inliner.reporter.error(
            'The structure should be a value(int/float) and a unit(str) separated by a normal space.Ex:`5 ms`', line=lineno)
        prb = inliner.problematic(rawtext, rawtext, msg)
        return [prb], [msg]
    node = nodes.Text("\u202F".join(text.split(' ')))
    return [node], []

def setup(app):
    app.add_role('unit', unit_role)

This results in :unit:`5 ms`. Does  this seem okay?

this is a great start! Here's a proposed cleanup:
from docutils import nodes
from mne.utils import _validate_type

def unit_role(name, rawtext, text, lineno, inliner, options={}, content=[]):
    parts = text.split()

    def pass_error_to_sphinx(rawtext, text, lineno, inliner):
        msg = inliner.reporter.error(
            'The :unit: role requires a space-separated number and unit; '
            f'got {text}', line=lineno)
        prb = inliner.problematic(rawtext, rawtext, msg)
        return [prb], [msg]

    # ensure only two parts
    if len(parts) != 2:
        return pass_error_to_sphinx(rawtext, text, lineno, inliner)
    # ensure first part is a number
    try:
        _validate_type(parts[0], 'numeric', 'Numeric part of :unit: role')
    except TypeError:
        return pass_error_to_sphinx(rawtext, text, lineno, inliner)
    # input is well-formatted: proceed
    node = nodes.Text('\u202F'.join(parts))
    return [node], []

def setup(app):
    app.add_role('unit', unit_role)

Can you open a PR that adds this role, and uses it instead of the &nbsp here:
https://github.com/mne-tools/mne-python/blob/11bc09911a2dbb6b73a86b31d8140ccd8a93b1c3/tutorials/preprocessing/25_background_filtering.py#L728
I don't think we test any of our sphinx extensions, but it ought to be fairly easy to test this.  @larsoner where would you suggest adding a test for our sphinx extensions?

@larsoner where would you suggest adding a test for our sphinx extensions?

We don't currently test them. But we could add a mne/utils/_sphinxext.py and mne/utils/tests/test_sphinxext.py or so. I'm not sure how easy it will be to set up the infrastructure for it, the hoops we had to jump through in sphinx-gallery were not trivial. But we could figure it out

@drammock The cleanup has a little bug.  When we split the text into  parts = text.split(),   parts[0] is  also a  string. Thus, whether it is 1 or  one, it raises an error. I am not sure, if  _validate_type has a correct form of application here.

from docutils import nodes

def unit_role(name, rawtext, text, lineno, inliner, options={}, content=[]):
    parts = text.split()

    def pass_error_to_sphinx(rawtext, text, lineno, inliner):
        msg = inliner.reporter.error(
            'The :unit: role requires a space-separated number and unit; '
            f'got {text}', line=lineno)
        prb = inliner.problematic(rawtext, rawtext, msg)
        return [prb], [msg]

    # ensure only two parts
    if len(parts) != 2:
        return pass_error_to_sphinx(rawtext, text, lineno, inliner)
    # ensure first part is a number
    try:
          value = float(parts[0])
    except ValueError:
        return pass_error_to_sphinx(rawtext, text, lineno, inliner)
    # input is well-formatted: proceed
    node = nodes.Text('\u202F'.join(parts))
    return [node], []

def setup(app):
    app.add_role('unit', unit_role)

I am seeing one  form of solution to  be this, with ValueError. Any suggestions?

Ah good point. Let's forget about using validate_type then

I  added a PR

Hello...everyone. As the non-breaking space role is added, Now I will like to go through the documentations (As I plan to do to strengthen my understanding of the project, and future effective contributions), and fix all the places, that should contain non-breaking space, but doesn't.
I will probably list those somewhere(could be in this issue comment, as this is related to this topic, as well as you guys can check if all are appropriate) so that I can fix the all in one commit later.
Will this be alright?

Preprocessing:
https://github.com/mne-tools/mne-python/blob/6608b720b214529732542cb2aac11b401770eddf/tutorials/preprocessing/10_preprocessing_overview.py#L39


Dovecot crashing under rosetta

Describe the bug
I tried running the official dovecot container under rosetta and it crashes.
To Reproduce

Enable "Use Rosetta" under Orbstack settings
In a terminal run docker run --rm --platform linux/amd64 dovecot/dovecot
The container will crash with the following errors:

assertion failed [!mmap_result.is_error && mmap_result.value == kSlabAllocatorInterval.left]: Could not mmap data for the VMTracker slab allocator: 12
(VMAllocationTracker.cpp:318 init)

If you disable rosetta and try again, the container start up correctly

Expected behavior
The container shouldn't crash
Diagnostic report (REQUIRED)
OrbStack info:
Version: 1.7.4
Commit: e03df71b90121e11c379c6273bd0a63166b33957 (v1.7.4)
System info:
macOS: 15.0 (24A335)
CPU: arm64, 11 cores
CPU model: Apple M3 Pro
Model: Mac15,6
Memory: 18 GiB
Full report: https://orbstack.dev/_admin/diag/orbstack-diagreport_2024-09-30T13-50-56.787223Z.zip

Screenshots and additional context (optional)
No response

Under Docker Desktop it's giving me a slight different error:
rosetta error: unable to mmap ExecutableHeap: 12

Update: It seems that docker desktop it's now working correctly with Rosetta enabled.
It seems that now it's a orbstack issue only.
The error has changed to:
rosetta error: mmap_anonymous_rw mmap failed, size=1000

Any updates on this?


Can't push single app while specifying manifest

I am trying to push a single app at a time while specifying a manifest and I am getting an error that doesn't seem to be accurate:
C02X2PA8JG5H:armada akrush$ cf bgd armada-ui -f manifest-uat.yaml
Using manifest file manifest-uat.yaml

Could not push new version - Incorrect Usage. Command line flags (except -f) cannot be applied when pushing multiple apps from a manifest file.

It looks like it thinks I'm trying to push multiple apps at the same time? Also, it's saying flags except -f can't be specified and i'm only using -f

Could you share your manifest?

---
applications:

- name: armada-gateway
  path: ../../armada-gateway/target/armada-gateway-1.0.0-RELEASE.jar
  buildpack: java_buildpack_offline
  routes:
  - route: armada.apps.pcf.devfg.rbc.com
  env:
    SPRING_PROFILES_ACTIVE: dev,cloud
    TZ: GMT
  services:
  - sso-service
  - vault-service

- name: armada-ui
  path: ../../armada-ui/build/
  buildpack: staticfile_buildpack
  routes:
  - route: armada-ui.apps.pcf.devfg.rbc.com

- name: armada-security
  path: ../../armada-security/target/armada-security-1.0.0-RELEASE-exec.jar
  buildpack: java_buildpack_offline
  routes:
  - route: armada-security.apps.pcf.devfg.rbc.com
  env:
    SPRING_PROFILES_ACTIVE: dev,cloud
    TZ: GMT
  services:
  - sso-service
  - vault-service

- name: armada-scanning
  path: armada-scanning/target/armada-scanning-1.0.0-RELEASE-exec.jar
  buildpack: java_buildpack_offline
  routes:
  - route: armada-scanning.apps.pcf.devfg.rbc.com
  env:
    SPRING_PROFILES_ACTIVE: dev,cloud
    TZ: GMT
  services:
  - sso-service
  - vault-service

- name: armada-cmdb
  path: armada-cmdb/target/armada-cmdb-1.0.0-RELEASE-exec.jar
  buildpack: java_buildpack_offline
  routes:
  - route: armada-cmdb.apps.pcf.devfg.rbc.com
  env:
    SPRING_PROFILES_ACTIVE: dev,cloud
    TZ: GMT
  services:
  - sso-service
  - vault-service

- name: armada-mission-control
  path: armada-mission-control/target/armada-mission-control-1.0.0-RELEASE-exec.jar
  buildpack: java_buildpack_offline
  routes:
  - route: armada-mission-control.apps.pcf.devfg.rbc.com
  env:
    SPRING_PROFILES_ACTIVE: dev,cloud
    TZ: GMT
  services:
  - sso-service
  - vault-service

- name: armada-virtualization
  path: armada-virtualization/target/armada-virtualization-1.0.0-RELEASE-exec.jar
  buildpack: java_buildpack_offline
  routes:
  - route: armada-virtualization.apps.pcf.devfg.rbc.com
  env:
    SPRING_PROFILES_ACTIVE: dev,cloud
    TZ: GMT
    JAVA_OPTS: -XX:MaxMetaspaceSize=300M
  services:
  - sso-service
  - vault-service
  memory: 2048M

- name: armada-population
  path: armada-population/target/armada-population-1.0.0-RELEASE-exec.jar
  buildpack: java_buildpack_offline
  routes:
  - route: armada-population.apps.pcf.devfg.rbc.com
  env:
    SPRING_PROFILES_ACTIVE: dev,cloud
    TZ: GMT
    JAVA_OPTS: -XX:MaxMetaspaceSize=300M
  services:
  - sso-service
  - vault-service
  memory: 3072M

- name: armada-provisioning
  path: armada-provisioning/target/armada-provisioning-1.0.0-RELEASE-exec.jar
  buildpack: java_buildpack_offline
  routes:
  - route: armada-provisioning.apps.pcf.devfg.rbc.com
  env:
    SPRING_PROFILES_ACTIVE: dev,cloud
    TZ: GMT
    JAVA_OPTS: -XX:MaxMetaspaceSize=300M
  services:
  - sso-service
  - vault-service
  memory: 2048M

- name: armada-vsight
  path: armada-vsight/target/armada-vsight-1.0.0-RELEASE-exec.jar
  buildpack: java_buildpack_offline
  routes:
  - route: armada-vsight.apps.pcf.devfg.rbc.com
  env:
    SPRING_PROFILES_ACTIVE: dev,cloud
    TZ: GMT
  services:
  - sso-service
  - vault-service

- name: armada-slack-bot
  path: slack-bot/target/slack-bot-1.0.0-RELEASE.jar
  buildpack: java_buildpack_offline
  routes:
  - route: armada-slack-bot.apps.pcf.devfg.rbc.com
  env:
    SPRING_PROFILES_ACTIVE: dev,cloud
    TZ: GMT

- name: armada-servernames
  path: armada-servernames/target/armada-servernames-1.0.0-exec.jar
  buildpack: java_buildpack_offline
  routes:
  - route: armada-servernames.apps.pcf.devfg.rbc.com
  env:
    SPRING_PROFILES_ACTIVE: dev,cloud
    TZ: GMT
  services:
    - sso-service
    - vault-service

Thanks, that's helpful. In general support for multiple applications in the same manifest is weak at the moment (see, for example, https://github.com/bluemixgaragelondon/cf-blue-green-deploy/issues/26). The issue you're seeing looks closely related to https://github.com/bluemixgaragelondon/cf-blue-green-deploy/issues/9, although the error messages are different so I suspect the codebase has moved on a bit from #9.


[crash] Draconis mod invokes on_punch() in other mods with incorrect arguments

EdenLost updated the draconis nmod a few days ago.  Players are now reporting that the game crashes when they punch other entities with draconis weapons.
Detailed analysis (including stack traces and code review) here:
https://github.com/EdenLostMinetest/edenlost/issues/45
TL;DR: Draconis is not passing the 5th argument damage (integer) when calling entity:on_punch(), so its getting converted to nil, which is then causing the crash.
The API spec states that entity:on_punch() takes 5 arguments, the last being integer damage.
https://github.com/minetest/minetest/blob/master/doc/lua_api.txt#L2081

Fixed as of latest commit.


Session 4 anirudh prabhakaran

These do not have the bonus tasks implemented yet. I will work on the bonus tasks for session 3 and 4 now

I have also completed some of the bonus tasks. I could not complete the AJAX queries and the limited number of private articles per person task. Do I also have to complete the 5th session in order to be considered for an interview at IRIS?

I have also completed some of the bonus tasks. I could not complete the AJAX queries and the limited number of private articles per person task. Do I also have to complete the 5th session in order to be considered for an interview at IRIS?

No, it's fine - they are "bonus" tasks for a reason and you don't have to complete the fifth session either.

Okay thank you! Shall I close my pull requests then?

Okay thank you! Shall I close my pull requests then?

Yeah, sure


Character string representing post categories doesn't wrap for small screens

I have these categories listed in the YAML of one of my posts:
categories: ["social class", "unethical behavior", "driving", "attention", "Bayesian models", "hierarchical models"]
These get rendered as a comma-separate string on the Recent Posts widget in my blog:
https://jeremyboyd.org/#posts
When the screen size is too small (e.g., on mobile), the categories string spills out of the container. Looks like the string isn't responsive to screen size--it doesn't automatically wrap like it should. I've included a screenshot that shows what happens--text spills out the right side of the screen. If you shorten the number of categories in the category string then no spillage occurs and everything renders correctly on small screens.

Link to your GitHub project: https://github.com/jeremyboyd/personal_website
Wowchemy Version (from your go.mod): 1.15
Hugo Version (run hugo version): 0.87.0
Browser/OS: Safari on macOS & iOS
Wowchemy Template (e.g. Academic): Academic

The category system wasn't intended to be used with so many labels per page. Does the tag system not meet your requirements for tagging pages with many labels?

I like categories over tags because the category labels appear in the posts widget. That way readers can see a list of posts with titles, abstracts, and keywords (categories) all at a glance.
If there's not a way to get categories to wrap, is there a way to get tags to show up in the posts widget?
Love the theme. Really appreciate all of the work you've done!


Problem with export Lens GroupBY

I'm trying to export to excel and I'm getting an error:

my Lens code
public static function query(LensRequest $request, $query)     {         return $request->withOrdering($request->withFilters(             $query->select(self::columns())                   ->groupBy('products.variant_title')         ));     }
 protected static function columns()     {         return [             'products.variant_title',             DB::raw('sum(products.variant_quantity) as quantity'),         ];     }
please advise how to fix it. thanks :)

Perhaps good to check these: https://mattstauffer.com/blog/strict-mode-and-other-mysql-customizations-in-laravel-5-2/ The query you pass has some issues in strict mode


⚠ Bofrost.at has degraded performance

In 5341bf0, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1811 ms

Resolved: Bofrost.at performance has improved in 1feb98a.


Add support for Govee Lyra RGBICWW Corner Floor Lamp (H6072)

It would be great if support for the H6072 lamp could be added to the plugin.

Sure. Does this light support segments? (If you go to the Govee App, go to your light there is a diagram of your light with some options along the light) If it does - how many segments does it have? Adding the light is pretty simple, however it may not be perfect. I spent a lot of time reverse engineering the communication protocols with every device I could get my hands on and most follow the same pattern - but some do not.

Technically this is an RGBICWW - which I haven't had a chance to determine the commandset. I made an assumption that it follows the RGBIC commands. Try the latest release 2.88.0 - if it doesn't behave acceptably I will have to pull it out of the code


Ensure code examples in Counter App - Multiple Files are complete

I'm currently following along with the book in order to learn Ratatui.
I think the section on breaking the Counter App into multiple files could be made a bit more beginner friendly by including more explicitly when certain crates are imported, reminding the reader to add the mods to their crate, etc. - it seems like some possibly obvious code has been excluded for brevity.
As an intermediate rustacean, I do enjoy the extra practice filling in some of the gaps, but it occasionally has been a bit frustrating or felt incomplete.
As I'm going through the book at the moment, I am actively taking notes on places where I've had to add to the example code. I would be happy to share those notes or make a pull request including my edits, if either would be helpful.

I would be happy to share those notes or make a pull request including my edits, if either would be helpful.

I'd love to see your notes! If you can make a PR that'll be even better. I did take a stab at cleaning up some of the issues in a linked PR but I'm sure there's more that can be improved.

I'll start a PR, just adding the bits of code to the examples I had to add to appease the compiler and maybe some small explanations.

So I rebased from main and then tried to rebase my remote branch from my local branch. Now, there is just a lot more noise from all the diffs between the main and my PR :( Sorry about that -- I haven't done much rebasing before, how can I clean this up for you?

Do you feel like this issue has been addressed? Can we close it?

I'll close this issue! Let us know if there's more work that can be done to improve things.