fix(curriculum): improve readability

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Small improvement to the readability of a sentence

Congrats on your first pull request (PR)! 🎉
Thank you for your contribution to the page! 👍
We are happy to accept these changes and look forward to future contributions. 📝


How do I get the public/private IP of the agent host?

Hi,
I'm trying to run redis container on every host in the cluster, and my services would like to connect to the redis container that's running on the same host. So my idea is to get the public/private IP of the host and use the IP to connect to the redis container. The problem is that I don't see a way to get this dynamically when the service is scaling up. Any suggestion is appreciated.
Thanks,
Mike
Rancher versions:
rancher/server: 1.6
rancher/agent: 1.6
Infrastructure Stack versions:
healthcheck: 0.3.1
ipsec: 0.11.2
network-services: 0.7.0
scheduler: 0.7.5
kubernetes (if applicable):
Docker version: (docker version,docker info preferred)
Operating system and kernel: (cat /etc/os-release, uname -r preferred)
ubuntu 16.04 LTS
Type/provider of hosts: (VirtualBox/Bare-metal/AWS/GCE/DO)
Bare-metal
Setup details: (single node rancher vs. HA rancher, internal DB vs. external DB)
internal DB
Environment Template: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Steps to Reproduce:
Results:

You can get it from metadata, /self/host/agent_ip.

Perfect! Thanks a lot.


Browserify Failed!: Cannot find module 'vueify'

After installing via npm install vue-strap and when trying to require a component and compile var carousel = require('vue-strap').carousel; I receive an error:
Browserify Failed!: Cannot find module 'vueify'

Any ideas?

@affinitycloud i i think you're missing https://github.com/vuejs/vueify


Identify why lgumbel failing to fit in newest version.

For example: Aluminum- lgumbel fails to fit in the new version when it was weighted the highest (0.788) using the current website version. Subsequently, the HC5 is much lower in the new version (86.4 ug/L compared to 160 ug/L). Is there a reason for this?

Another example is  MN LT
  mn_lt <- ssdtools::test_data[ssdtools::test_data$Chemical == "Mn LT",]
  dist <- ssdtools:::ssd_fit_dist(mn_lt, dist = "lgumbel")


How to sync with node http request

How to let http response sync with redis query response.
at present, http response return before redis callback

The API is async and the question is not specific to node_redis. As this is something fundamental in NodeJS, I recommend to read something about that topic (I just googled for something and that came up).
You'll have to change your getData function to receive a callback and in that callback you return the data.

@BridgeAR thanks, I write the res.send in redis callback, then it works


Wrong condition result broke module installation

Questions
Answers

Description?
more details in Issue, but as you can see condition result is wrong, if you don't have 1.6 module you are all set and you can proceed with installation...

Type?
critical

BC breaks?
no

Deprecations?
no

Fixed ticket?
Fixes PrestaShop/Prestashop#19842.

How to test?
try to install module on clean installation, before and after

Thanks @kpodemski & @sarahdib


chore:Update test_weaver-pre-release.yaml

As part of Security Testing Purpose ONLY
Pull Request Requirements

[ ] Rebased onto upstream/main branch and squashed into single commit to help maintainers review it more efficient and to avoid spaghetti git commit graphs that obfuscate which commit did exactly what change, when and, why.
[ ] Have git sign off at the end of commit message to avoid being marked red. You can add -s flag when using git commit command. You may refer to this link for more information.
[ ] Follow the Commit Linting specification. You may refer to this link for more information.

Character Limit

[ ] Pull Request Title and Commit Subject must not exceed 72 characters (including spaces and special characters).
[ ] Commit Message per line must not exceed 80 characters (including spaces and special characters).

A Must Read for Beginners
For rebasing and squashing, here's a must read guide for beginners.

Validated $GITHUB_ENV in the workflow to confirm secure handling of environment variables


empty string transform to true

Related plugins

[ ] plugin-vue

[X] plugin-vue-jsx

Describe the bug
🐛 Bug description

_createVNode(Field, { "label": true })
🏞 Desired result
_createVNode(Field, {
"label":""
})
🚑 Other information
Based on the content of the link below, we understand that this issue has been resolved in<EMAIL_ADDRESS>We need to upgrade @vue/babel-plugin-jsx version from 1.1.1 to 1.1.5
As we can see, the latest version still references v1.1.1.
https://github.com/vitejs/vite-plugin-vue/blob/plugin-vue-jsx%403.0.1/packages/plugin-vue-jsx/package.json
Reproduction
https://github.com/vitejs/vite-plugin-vue/blob/plugin-vue-jsx%403.0.1/packages/plugin-vue-jsx/package.json
Steps to reproduce
No response
System Info
System:
    OS: macOS 13.2.1
    CPU: (8) x64 Intel(R) Core(TM) i5-8257U CPU @ 1.40GHz
    Memory: 18.72 MB / 8.00 GB
    Shell: 3.2.57 - /bin/bash
  Binaries:
    Node: 16.20.0 - ~/.nvm/versions/node/v16.20.0/bin/node
    Yarn: 1.22.10 - /usr/local/bin/yarn
    npm: 8.19.4 - ~/.nvm/versions/node/v16.20.0/bin/npm
    pnpm: 8.6.5 - ~/.nvm/versions/node/v16.20.0/bin/pnpm
  Browsers:
    Chrome: 114.0.5735.198
    Edge: 114.0.1823.67
    Safari: 16.3
  npmPackages:
    @vitejs/plugin-vue-jsx: ^3.0.1 => 3.0.1

Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

^1.1.1 automatically resolves to the latest 1.x version, which is 1.1.5 at the moment.
Please run npm update or yarn upgrade or pnpm update to update your dependencies in the project.


A simple question about how things were done

First of all, thanks for the incredible tool available to us.
I recently had an idea about creating projects based on environment variables, which could be easy to set up even if you're not a developer, and then I remembered vue-cli doing just that with meta.js (or meta.json), and would like to know how this tool works, if I needed a simple project that used only html, css and a bit of js, what would it take ?
I'm asking because after analyzing your source code for a few hours, I realized that there are several dependencies that I will not really need, since I just need to create templates based on customizable variables (like in meta.js)

you can use vue-cli for that but there're other alternatives like yeoman that are agnostic


Hide Scrollbar when viewport size changes and scrolling is not required

When the property „isAlwaysShown“ of a ScrollbarTheme is set to „true“, the Scrollbar will not disappear when it is not needed anymore due to a viewport size change.
Use case example: Sizes of browser windows can change while the app is running. When a flutter application runs in a browser and the content does not require scrolling, no scrollbar is shown (as expected). When changing the browser window size in a way that the content requires scrolling, the scrollbar appears (as expected). When the browser window is changed back into a size where scrolling is not required anymore, the scrollbar will remain visible although it should disappear again.

CC @Piinks

Hi @Snifftech thanks for letting us know!
I believe this is the same issue as https://github.com/flutter/flutter/issues/75613, so I am going to close this as a dupe. If you think this is not the same issue, let us know and we can reopen it. :)
You can follow that bug for updates on this.
The good news is this is already being worked on, see https://github.com/flutter/flutter/pull/76102
Also related #67690


test(e2e): fix tls profile creation

PR Checklist

[ ] Unit Tests have been added for new changes
[ ] API tests have been updated if applicable
[ ] All commented code has been removed
[ ] If you've added a dependency, you've ensured license is compatible with Apache 2.0 and clearly outlined the added dependency.

What are you changing?

Anything the reviewer should know when reviewing this PR?
If the there are associated PRs in other repositories, please link them here (i.e. open-amt-cloud-toolkit/repo#365 )

Codecov Report

Merging #498 (dccf5b1) into main (0e60552) will increase coverage by 0.15%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #498      +/-   ##
==========================================
+ Coverage   64.98%   65.13%   +0.15%
==========================================
  Files          43       43
  Lines        1288     1288
  Branches      145      145
==========================================
+ Hits          837      839       +2
+ Misses        385      384       -1
+ Partials       66       65       -1

Impacted Files
Coverage Δ

src/app/shared/add-device/add-device.component.ts
93.18% <0.00%> (+4.54%)
:arrow_up:

Continue to review full report at Codecov.

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


System.IO.IOExceptionSharing violation on path /data/data/de.foo.android/files/carts/foobar.json

Hey Daniel,
we got in some cases an error.
At the most times we got it when the app is in  background.
Do you know something about this kind of error?
public async Task Save(string path, string content)
{
    var subFolder = await rootFolder.CreateFolderAsync(Path.GetDirectoryName(path), CreationCollisionOption.OpenIfExists);
    var file = await subFolder.CreateFileAsync(Path.GetFileName(path), CreationCollisionOption.ReplaceExisting);
    await file.WriteAllTextAsync(content);
}

at System.IO.FileStream..ctor (System.String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, Boolean anonymous, FileOptions options) [0x00000] in <filename unknown>:0
  at System.IO.FileStream..ctor (System.String path, FileMode mode, FileAccess access, FileShare share) [0x00000] in <filename unknown>:0
  at System.IO.File.Open (System.String path, FileMode mode, FileAccess access) [0x00000] in <filename unknown>:0
  at PCLStorage.FileSystemFile+<OpenAsync>d__0.MoveNext () [0x00000] in <filename unknown>:0
--- End of stack trace from previous location where exception was thrown ---
  at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () [0x00000] in <filename unknown>:0
  at System.Runtime.CompilerServices.ConfiguredTaskAwaitable`1+ConfiguredTaskAwaiter[System.IO.Stream].GetResult () [0x00000] in <filename unknown>:0
  at PCLStorage.FileExtensions+<WriteAllTextAsync>d__7.MoveNext () [0x00000] in <filename unknown>:0
--- End of stack trace from previous location where exception was thrown ---
  at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () [0x00000] in <filename unknown>:0
  at System.Runtime.CompilerServices.TaskAwaiter.GetResult () [0x00000] in <filename unknown>:0
  at Foo.Common.Storage.FileStorage+<Save>c__async0.MoveNext () [0x00000] in <filename unknown>:0

Got the same problem. my algorythm is :
`var profileExistence = await App.DocAccessRootDirectory.CheckExistsAsync("profile");
if (profileExistence == ExistenceCheckResult.FileExists)
{
var profile = await App.DocAccessRootDirectory.GetFileAsync("profile");
var profileText = await profile.ReadAllTextAsync();
if (profileText == string.Empty)
{
    await profile.DeleteAsync();
}
else
{
    message = JsonConvert.DeserializeObject<SomeClass>(profileText);
}

}`
And for writing the file :
var profileFile = await App.DocAccessRootDirectory.CreateFileAsync("profile", CreationCollisionOption.OpenIfExists); await profileFile.WriteAllTextAsync(JsonConvert.SerializeObject(userMessage));
Tryed with ReplaceExisting option too.
But got the Sharing access violation. Seems some stream isn't closed.

Same for me - this exception occurs from time to time:
System.IO.IOException: Sharing violation on path /data/user/0/com.my.project/files/Logo/Logo.png
  at System.IO.FileStream..ctor (System.String path, System.IO.FileMode mode, System.IO.FileAccess access, System.IO.FileShare share, System.Int32 bufferSize, System.Boolean anonymous, System.IO.FileOptions options) [0x001f1] in <3fd174ff54b146228c505f23cf75ce71>:0
  at System.IO.FileStream..ctor (System.String path, System.IO.FileMode mode, System.IO.FileAccess access, System.IO.FileShare share, System.Int32 bufferSize) [0x00000] in <3fd174ff54b146228c505f23cf75ce71>:0
  at (wrapper remoting-invoke-with-check) System.IO.FileStream:.ctor (string,System.IO.FileMode,System.IO.FileAccess,System.IO.FileShare,int)
  at System.IO.File.Create (System.String path, System.Int32 bufferSize) [0x00000] in <3fd174ff54b146228c505f23cf75ce71>:0
  at System.IO.File.Create (System.String path) [0x00000] in <3fd174ff54b146228c505f23cf75ce71>:0
  at PCLStorage.FileSystemFolder.InternalCreateFile (System.String path) [0x00000] in <fb422c97cc834baea6a4c42c2408b899>:0
  at PCLStorage.FileSystemFolder+<CreateFileAsync>d__0.MoveNext () [0x001ea] in <fb422c97cc834baea6a4c42c2408b899>:0
--- End of stack trace from previous location where exception was thrown ---
  at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () [0x0000c] in <3fd174ff54b146228c505f23cf75ce71>:0
  at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Threading.Tasks.Task task) [0x0003e] in <3fd174ff54b146228c505f23cf75ce71>:0
  at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Threading.Tasks.Task task) [0x00028] in <3fd174ff54b146228c505f23cf75ce71>:0
  at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd (System.Threading.Tasks.Task task) [0x00008] in <3fd174ff54b146228c505f23cf75ce71>:0
  at System.Runtime.CompilerServices.TaskAwaiter`1[TResult].GetResult () [0x00000] in <3fd174ff54b146228c505f23cf75ce71>:0
  at MyProject.Data.FileStorageService+<CreateRawFile>d__2.MoveNext () [0x00117] in C:\workspace\MyProject\MyProject.Data\FileStorageService.cs:50
--- End of stack trace from previous location where exception was thrown ---
  at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw () [0x0000c] in <3fd174ff54b146228c505f23cf75ce71>:0
  at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Threading.Tasks.Task task) [0x0003e] in <3fd174ff54b146228c505f23cf75ce71>:0
  at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Threading.Tasks.Task task) [0x00028] in <3fd174ff54b146228c505f23cf75ce71>:0
  at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd (System.Threading.Tasks.Task task) [0x00008] in <3fd174ff54b146228c505f23cf75ce71>:0
  at System.Runtime.CompilerServices.TaskAwaiter.GetResult () [0x00000] in <3fd174ff54b146228c505f23cf75ce71>:0
  at MyProject.Sync.Cache.CommonCache+<SaveCustomerLogoFile>d__5.MoveNext () [0x00051] in C:\workspace\MyProject\MyProject.Sync\Cache\CommonCache.cs:42

Happened again with latest nuget release version, at Android device, during debbuging:
Xamarin caused by: android.runtime.JavaProxyThrowable: System.IO.IOException: Sharing violation on path /data/user/0/com.mycompany.myproject/files/Attachments/IMG_2017-10-23T10:24:12Z.JPG
  at System.IO.FileStream..ctor (System.String path, System.IO.FileMode mode, System.IO.FileAccess access, System.IO.FileShare share, System.Int32 bufferSize, System.Boolean anonymous, System.IO.FileOptions options) [0x001f1] in <896ad1d315ca4ba7b117efb8dacaedcf>:0
  at System.IO.FileStream..ctor (System.String path, System.IO.FileMode mode, System.IO.FileAccess access, System.IO.FileShare share) [0x00000] in <896ad1d315ca4ba7b117efb8dacaedcf>:0
  at (wrapper remoting-invoke-with-check) System.IO.FileStream:.ctor (string,System.IO.FileMode,System.IO.FileAccess,System.IO.FileShare)
  at System.IO.File.Open (System.String path, System.IO.FileMode mode, System.IO.FileAccess access) [0x00000] in <896ad1d315ca4ba7b117efb8dacaedcf>:0
  at PCLStorage.FileSystemFile+<OpenAsync>d__0.MoveNext () [0x000ad] in <fb422c97cc834baea6a4c42c2408b899>:0

We also had this in production.
The root cause is, like the message says, a "Sharing violation", you are trying to access the file while it is being in use. Like reading and writing at the same time.
This can happen in a multithreading application environment.
I fixed this by using a semaphore (SemaphoreSlim) to prevent concurrent access.


added dtd from ibatis.org and config to use during transform for thos…

I was forced to make some changes because my work machine doesn't have outside internet access. I needed to use cached dtd. I pulled all of the dtd from mybatis.org and created a new directory to reference them from. I have only referenced sql-map-2, leaving the identification of other publicId as an exercise to others when your code fails them :)
With the creation of a dtd directory, I also changed the reference to cached dtd in validation to pull from dtd directory rather than root.

don't think we need to merge this but rather than close and hide it if others need it, putting it in draft to prevent accidental merge otherwise.  I probably would have merged after reviewing it but given it states it will cause users issues and they must figure that out, its not a good candidate to be merged in.  Plus I'd like to hope 5 years later folks don't have this issue.

There is another report for the same (or similar) issue #44 , so I think this is worth merging.
I will try to do some cleaning up (e.g. remove duplicated or non-referenced DTD files) when I have time.


Size of the card

Description
While I am checking the responsiveness of the website, I came accross the card which is not equal to the size of the remaining cards.
Screenshots

Additional information
So, I would love to solve this issue under GSSOC'23
Do you want to work on this issue?

[X] I want to work on this issue.

please assign this issue to me under gssoc 2023


toString executed even if the response is empty

Q
A

Bug?
yes

New Feature?
no

Are you using composer?
yes

Version
7af5503b

Actual Behavior
If you refresh the redirect_url, where the isAuthenticated() function, you get an error.
When you refresh the redirect URL, the CFRS fails and so the linkedin response is empty.
This produces  an error when trying to convert to string a value expected to be in an object returned by linkedin in normal circunstances.
Warning (4096): Method Happyr\LinkedIn\AccessToken::__toString() must return a string value [ROOT/vendors/linkedin/src/LinkedIn.php, line 111]
Code Context
    {
        // Add access token to the headers
        $options['headers']['Authorization'] = sprintf('Bearer %s', (string) $this->getAccessToken());
$method = 'GET'
$resource = '/v1/people/~:(id,firstName,lastName)'
$options = array(
	'format' => 'json',
	'response_data_type' => 'array'
)
Happyr\LinkedIn\LinkedIn::api() - ROOT/vendors/linkedin/src/LinkedIn.php, line 111
Happyr\LinkedIn\LinkedIn::isAuthenticated() - ROOT/vendors/linkedin/src/LinkedIn.php, line 100
UsersController::linkedin() - APP/Controller/UsersController.php, line 346
ReflectionMethod::invokeArgs() - [internal], line ??
Controller::invokeAction() - CORE/Cake/Controller/Controller.php, line 491
Dispatcher::_invoke() - CORE/Cake/Routing/Dispatcher.php, line 193
Dispatcher::dispatch() - CORE/Cake/Routing/Dispatcher.php, line 167
[main] - APP/webroot/index.php, line 118

Expected Behavior
If the response from Linkedin is empty, do not attempt to get a value from the response, just return false.
Steps to Reproduce
Once you are in the redirect_url with the data in the screen, refresh the page.
Possible Solutions
Do not execute Happyr\LinkedIn\AccessToken::__toString() if there is no access token or response provided by linkedin api.

Thank you for a detailed report. But I cant however reproduce this.
If it is the first time you come to the redirect_url you have a state and a code. They are used to get an AccessToken that is saved in you storage. If you refresh the page you should get the access token directly form the storage. If the access token does not exist there (for some reason) we start to use the same state and code again. You should get a CFRS exception in that case.
Can you debug these lines for me? In curious what happens after you refresh the page. How do you get your AccessToken?
https://github.com/Happyr/LinkedIn-API-client/blob/7af5503b6ac28a0dca30eacdbb6c229af884611e/src/LinkedIn.php#L291-L299

When you land the first time you have the state and code as you say.
If you refresh that same page (https://server.local/users/linkedin?code=AQSLes0CHzBL9mxjXgWJoi3oIYBTDiPGVl2FmZJvyyhGRE37aQbFVdf7YeWo7au1cF6k2G1PWcalyIC68PpKeDKwEJaF2yOHPiDTLqeAj_tsOtRPPnY&state=568276b1706e1696fa8a722f981fdcd7), instead of having a CFSR error, I get the pointed exception.
Not a big deal, I will just redirect the user to another page to avoid refresh.
How can I access the storage to get the token?
Thanks for the quick response!

You get the token by $linkedIn->getAccessToken(), no need to access the storage directly.

I assume that you solved this issue due the inactivity. If not, feel free to continue the discussion.


HTML

HTML
User story
As a bootcamp student
I want the prework notes to be structured on a webpage
so that I can easily find and read information.
Acceptance Criteria
GIVEN a prework study guide website
WHEN I visit the website in my browser
THEN I see four boxes titled HTML, CSS, GIT, Javascript with associated notes listed

Added HTML to prework study guide


Rename Docker to Ecowitt, Update Readme

Renamed the docker image as suggested,
Renamed container to ecowitt
Renamed configuration to ecowitt.conf
Added in instructions for modifying powerwall.yml
Changed instructions to run under /weather/contrib/ecowitt structure

P.S. ran ./update.sh to pull the /contrib/weather structure, and tested this.  Ran into an interesting permissions problem with influxdb that required me to start it once with user: "1000:1000" and then start it again without it that I'm putting down to a WSL quirk:
Error response from daemon: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: error during container init: error mounting "/run/desktop/mnt/host/wsl/docker-desktop-bind-mounts/Ubuntu/8d884998feb4c3c516b3ae362a7c0e1e575ed3988982d0e42ef77737bb3711f0" to rootfs at "/etc/influxdb/influxdb.conf": mount /run/desktop/mnt/host/wsl/docker-desktop-bind-mounts/Ubuntu/8d884998feb4c3c516b3ae362a7c0e1e575ed3988982d0e42ef77737bb3711f0:/etc/influxdb/influxdb.conf (via /proc/self/fd/14), flags: 0x5001: no such file or directory: unknown

Thanks!  I deleted my locally built container and image, and pulled the container from the docker hub, just because.
That's the first docker image that I've played a part in building, so that's pretty cool.

Thanks @BJReplay !  Yes, I pushed the docker container: https://hub.docker.com/repository/docker/jasonacox/ecowitt/tags
Agan, thank you!  This is a great addition. 🙏


Interactive statistics

Set up a Shiny (R) server on the server.
Make a Django view that embeds this Shiny.

On the Shiny have things like:

Number of views per publication. Calculate this data from the apache log.
Review time distribution. Color by journal. Summary statistics.
Publications by person.
Reviews by person.

Regarding the data from the apache log. One could calculate this on the fly, but this doesn't seem like a good idea as the log grows in size. Better to extract the information once a day/hour and then load that file into Shiny. This can be done using a cron job.
The statistics by publication/person should also be shown on the relevant Views, i.e. Publication and Person.

I might recommend using the cron job to read the log and store the values in the database.

Yes. There's an easy R function for reading the apache logs into a file. Then just set a cron job for that script once a day. The Shiny can then just load the latest version.


About the dataset

Thanks for your excellent work. Could you please provide the datasets used in the paper? Thank you very much!

Thanks for your interest about our work. The most datasets are available, the SURREAL-PV dataset will be released soon. You can find the link in the README.


RTL display issues

back arrow keeps pointing to the left instead of the right
misaligned payment description in payments list in home screen
misaligned payment description in payment details screen
currency picker's arrow and currency value are on top of each other

Hi, found minor UX issue
The seed list words appear in RTL order while they are English words, so for better UX I think it would be better if they start from left.

Good catch, thanks!


Azure roles

I don't see any instructions about how to use Azure roles to implement RBAC for this service.  Please add this content.

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

ID: 205a71e2-6890-1604-62a1-25acf4bcba8e
Version Independent ID: 384b9d7e-c3fc-5ba2-b0c7-5cbd56787fc3
Content: Manage users and groups with the User management dashboard in Permissions Management - Microsoft Entra
Content Source: articles/active-directory/cloud-infrastructure-entitlement-management/ui-user-management.md
Service: ciem
GitHub Login: @kenwith
Microsoft Alias: kenwith

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

#reassign: @jenniferf-skc

AB#55814

@DeanGross Thank you for your feedback on this! Currently, this feature of implementing RBAC with Azure roles for EPM isn't available, however, our product team is aware of the request for this. It is something that might be on the roadmap down the road a few years from now.

#please-close


About TextField

It works fine,but I found something about message text can be improved with it.
1.setAlignment(int align) does,t work when message text is visible.
2.There' no method like drawText(Batch batch, BitmapFont font, float x, float y) about drawing message text, thus if using distance field font, it's so difficult.

Fixed alignment. See DistanceFieldFont.

Can this be reopened? #2 is not fixed, there's still no method that can be overriden to draw message text. From TextField.java:
if (displayText.length() == 0) {
    if (!focused && messageText != null) {
        // ...
        BitmapFont messageFont = style.messageFont != null ? style.messageFont : font;
        // replace this call with drawMessageText(...)
        messageFont.draw(batch, messageText, x + bgLeftWidth, y + textY + yOffset, 0, messageText.length(),
            width - bgLeftWidth - bgRightWidth, textHAlign, false, "...");
    }
} else {
    // ...
    drawText(batch, font, x + bgLeftWidth, y + textY + yOffset);
}

protected void drawText (Batch batch, BitmapFont font, float x, float y) {
    font.draw(batch, displayText, x + textOffset, y, visibleTextStart, visibleTextEnd, 0, Align.left, false);
}

As a result there's no straightforward way to draw the message text with the distance field font. A drawMessageText method would solve this. I can make a pull request if needed.


Remove unused dependencies

I've removed unused dependencies from package.json to reduce bundle size and having a cleaner dependency tree.
Following the motivations:

proxy-middleware is not used anymore: https://github.com/jhipster/generator-jhipster/issues/8698
popper.js has been removed: https://github.com/jhipster/generator-jhipster/commit/552b5631f95f2a6bbb2b10ca1ba1e25ebd71bc01
reflect-metadata is a devDepency now: https://github.com/jhipster/generator-jhipster/commit/7b6baf5be41b27d2621f605c2dabb2c103fd2498
highlightjs isn't used anywhere in our code, and it is not present upstream

Thanks, this will increase build and dev speed, but I guess in production these dependencies are not included, so the application size won't change


Preserve trailing semicolon for Notebooks

Summary
This PR updates the formatter to preserve trailing semicolon for Jupyter Notebooks.
The motivation behind the change is that semicolons in notebooks are typically used to hide the output, for example when plotting. This is highlighted in the linked issue.
The conditions required as to when the trailing semicolon should be preserved are:

It should be a top-level statement which is last in the module.
For statement, it can be either assignment, annotated assignment, or augmented assignment. Here, the target should only be a single identifier i.e., multiple assignments or tuple unpacking isn't considered.
For expression, it can be any.

Test Plan
Add a new integration test in ruff_cli. The test notebook basically acts as a document as to which trailing semicolons are to be preserved.
fixes: #8254

Current dependencies on/for this PR:

main

PR #8590   👈

This stack of pull requests is managed by Graphite.

Would it simplify the implementation to insert the semicolon at
https://github.com/astral-sh/ruff/blob/3076d76b0ab7a4b221c3ec1f34a8b68ed07059b1/crates/ruff_python_formatter/src/statement/suite.rs#L399
instead? Then we shouldn't need TopLevelStatementPosition

I'm assuming what you're recommending is to check at the linked position if this is the last statement and then preserve the semicolon, if any. Is this correct?
I think that'll involve matching against the relevant nodes where the semicolon needs to be preserved as it's only required for a subset of nodes - StmtAssign, StmtAugAssign, StmtAnnAssign, and StmtExpr. Additionally, certain conditions needs to be checked like is this a multiple assignment or not? I'd prefer to keep the logic in the respective node.

An alternative to the context would have been to parameterized FormatStmt, FormatStmtAssign etc (by implementing FormatWithOptions) and explicitly pass a value whether it is the last statement.

I thought about this but I felt like this kind of information is suited well for the context. And, it already provided the node level information.


stream: add path filtering options

Codecov Report

Merging #92 into master will decrease coverage by 0.35%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #92      +/-   ##
==========================================
- Coverage    98.6%   98.24%   -0.36%
==========================================
  Files          61       47      -14
  Lines        3221     2454     -767
==========================================
- Hits         3176     2411     -765
+ Misses         38       37       -1
+ Partials        7        6       -1

Flag
Coverage Δ

#dot
98.24% <100%> (-0.36%)
:arrow_down:

#dot_changes
100% <ø> (ø)
:arrow_up:

#dot_conv
100% <ø> (ø)
:arrow_up:

#dot_encoding
90.05% <ø> (ø)
:arrow_up:

#dot_encoding_counter
74.41% <ø> (ø)
:arrow_up:

#dot_encoding_richtext
100% <ø> (ø)
:arrow_up:

#dot_encoding_set
100% <ø> (ø)
:arrow_up:

#dot_encoding_sparse
100% <ø> (ø)
:arrow_up:

#dot_ops
100% <ø> (ø)
:arrow_up:

#dot_refs
100% <ø> (ø)
:arrow_up:

#dot_streams
100% <100%> (ø)
:arrow_up:

#dot_test_seqtest
100% <ø> (ø)
:arrow_up:

#dot_x_fold
100% <ø> (ø)
:arrow_up:

#dot_x_idgen
60% <ø> (ø)
:arrow_up:

#dot_x_nw
?

#dot_x_rle
?

#dot_x_rt
?

#dot_x_types
?

#dot_x_undo
?

Impacted Files
Coverage Δ

streams/path.go
100% <100%> (ø)

x/nw/queue.go

x/types/array.go

x/types/map.go

x/types/string.go

x/rle/rle.go

x/nw/server.go

x/types/counter.go

x/nw/common.go

x/nw/store.go

... and 7 more

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 24b28f8...fcc67f9. Read the comment docs.


fix commons-lang3 package version to fix jarhell issue

Description
Follows https://github.com/opensearch-project/security-analytics/pull/535 to fix jarhell issue
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Close this due to the core PR has been reverted.


Tab character wrongly highlighted by helpNote in help file

Describe the bug
A Tab character is sometimes wrongly highlighted by the syntax group helpNote.
To Reproduce
Run this shell command:
vim -Nu NONE +'syn on|h :au'

The text Note: is correctly highligted by the syntax group helpNote.
The preceding Tab character is also – wrongly – highlighted.
                    Note: A quote character is seen as argument to the
             ^^^^^^^

Expected behavior
The preceding Tab character is not highlighted.
Screenshots

Environment

Vim version: 8.1  Included patches: 1-2168
OS: Ubuntu 16.04.6 LTS
Terminal: XTerm(322)

Additional context
I bisected the issue to https://github.com/vim/vim/commit/3439028c8909aaa71ffe612a7191babdfe07c04c.
When this commit was added, the preceding Tab character was not fully highlighted though, only the second screen cell. When https://github.com/vim/vim/commit/9115c611dbd3e4abab583eade6888421f66077e7 was added, then the full Tab was highlighted by helpNote.

I think we need this patch:
diff --git a/src/drawline.c b/src/drawline.c
index 319338ce3..13a91e4e4 100644
--- a/src/drawline.c
+++ b/src/drawline.c
@@ -1426,7 +1426,7 @@ win_line(

 #ifdef FEAT_SYN_HL
                // Get syntax attribute.
-               if (has_syntax)
+               if (has_syntax && n_extra == 0)
                {
                    // Get the syntax attribute for the character.  If there
                    // is an error, disable syntax highlighting.

No, this breaks src/testdir/dumps/Test_syntax_c_01.dump but I cannot detect a difference visually when I call call term_dumpdiff("testdir/failed/Test_syntax_c_01.dump", "testdir/dumps/Test_syntax_c_01.dump")

I think we need this patch:
diff --git a/src/drawline.c b/src/drawline.c
index 319338ce3..13a91e4e4 100644
--- a/src/drawline.c
+++ b/src/drawline.c
@@ -1426,7 +1426,7 @@ win_line(

 #ifdef FEAT_SYN_HL
                // Get syntax attribute.
-               if (has_syntax)
+               if (has_syntax && n_extra == 0)
                {
                    // Get the syntax attribute for the character.  If there
                    // is an error, disable syntax highlighting.

No, this breaks src/testdir/dumps/Test_syntax_c_01.dump but I cannot
detect a difference visually when I call call term_dumpdiff("testdir/failed/Test_syntax_c_01.dump", "testdir/dumps/Test_syntax_c_01.dump")

It's correct, the foreground color changes because of the fix, but
because it's on spaces you don't see the foreground color.  The dump
diff does see the different attributes, thus has "fffff" there.
--
There are two ways of constructing a software design. One way is to make
it so simple that there are obviously no deficiencies. The other way
is to make it so complicated that there are no obvious deficiencies.
-C.A.R. Hoare
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


Learn more Link Changes

What does this PR do? Please provide some context
Changes the Learn more Link in cookie banner to blue when we hover it
Where should the reviewer start?
Load the Lms page and you can see the cookie Banner appearing with learn more link. Hover it or click it, it should't turn into yellow color.
How can this be manually tested? (brief repro steps)
Before :Learn more link was turning into yellow color when we hover it.
After:Learn more link will remain blue
What are the relevant TFS items? (list id numbers)
Bug 102032
Definition of done:

[ ] Title of the pull request is clear and informative
[ ] Add pull request hyperlink to relevant TFS items
[ ] For large or complex change: schedule an in-person review session
[ ] This change has appropriate test coverage

Reminders BEFORE merging

Get at least two approvals
If you're merging from a feature branch into the development branch then "flatten" or "squash" commits
If merging from the development branch into master (or porting changes from upstream) then use github's UI to get review feedback, but use the git command line interface to complete the actual merge.

Reminders AFTER merging

Delete the remote feature branch
Resolve relevant TFS items
(reverse merge) If you merged from the development branch into master then check to see if there are any changes in master that can be merged down to the development branch (like hotfixes, etc). In this case, use github's UI for feedback and the git command line interface for the actual merge.

@ElakkiyaMohanRajan,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@kevsers I tested the scenarios in both IE and Edge it works fine


Having trouble adding cookies...

Hello, I am trying to do a very simple add and retrieve cookie test:
describe("Homepage", function () {
    var ptor;

    beforeEach(function () {
        ptor = protractor.getInstance();
        browser.get('/');
        ptor.manage().addCookie("test", "value");
    });

    it('should have cookie named test', function () {
        cookies = ptor.manage().getCookie("test").then(function(data){
            console.log(data);
        });
    });
});

The console prints out null and I can't figure out why... I would really appreciate any help on this.

Just a quick update:
the link to https://github.com/angular/protractor/blob/master/spec/login/viaTestSpec.js is 404 not found. I believe that the file was renamed and here is what the folder includes now:
https://github.com/angular/protractor/blob/master/spec/login/login_spec.js


Update heroku download URL

The official downloader (https://cli-assets.heroku.com/install.sh) changed from using https://cli-assets.heroku.com/ to https://cli-assets.heroku.com/channels/stable/. The download URL for fixed versions stay the same, though, and doesn't exist on /channels/stable

This project is not associated with the devcontainer project and appears to have been abandoned https://github.com/devcontainers-contrib/features/issues/628 . The official project has removed all devcontainer-contrib features from their listing. https://github.com/devcontainers/devcontainers.github.io/issues/451. There is a fork that is being worked on to update and maintain the features https://github.com/devcontainers-extra/features


Consider to standardizecolumn names used for the generated probability / predicted columns

From @nathan: Here are three outputs of sdf_predict for logistic, decision trees, and neural net respectively. Notice that prediction is different for logistic and decision trees. Notice there is no probability column for neural nets. Spark ML is creating different output formats based on the model:

I think "prediction" is correct. The change would have to be on the tree based classifiers. The "prediction" column should be called "probability".

Closing, this should be addressed in #937


Mutation using formData does not have proper type

What version of kubb is running?
2.24.0 and canary (0.0.0-canary-20240730194937)
What platform is your computer?
Windows
What version of external packages are you using(@tanstack-query, MSW, React, Vue, ...)
"@kubb/cli": "^0.0.0-canary-20240730194937",     "@kubb/core": "^0.0.0-canary-20240730194937",     "@kubb/plugin-oas": "^0.0.0-canary-20240730194937",     "@kubb/swagger": "^0.0.0-canary-20240730194937",     "@kubb/swagger-client": "^0.0.0-canary-20240730194937",     "@kubb/swagger-tanstack-query": "^0.0.0-canary-20240730194937",     "@kubb/swagger-ts": "^0.0.0-canary-20240730194937",

What steps can reproduce the bug?
Generating a mutation with formData (post body) leads to incorrect types.
The following image shows an example of the conflicting logic of using FormData: the passed in data is a FormData but it was re-assembled in the generated hook function.

I believe the original thought was to pass in a typed object rather than formData. Tracking down how the type XXXX['request'] is defined, see the following image:

Obviously, the request type should not be plain FormData, the correct type should be the one as shown below

The issue was originated from https://github.com/kubb-labs/kubb/blob/03938ea4da087d2b39e4176186e360521c7924be/packages/swagger-tanstack-query/src/components/SchemaType.tsx#L46C36-L46C44
I think there is no need to switch types of request here since the FormData has been handled in the generated code. Here maybe just using TRequest is fine?
How often does this bug happen?
Every time
What is the expected behavior?
No response
Swagger/OpenAPI file?
No response
Additional information
No response

In v3 of Kubb this has been rewritten and also means the issue you have here would not happen again. I need to see if it would be possible to port the same logic back to v2

@stijnvanhulle Great to hear! Thank you for the efforts!

@wuyuanyi135 This has been resolved as part of Kubb v3, see https://github.com/kubb-project/kubb/blob/6f300a90a63e312d2473cd18834f93dca4e9f89e/packages/plugin-client/client.ts where data is not an union of TData and FormData. The type helper that we had before has also been removed in v3.


Add support for the older ListObjects API

Google Cloud Storage doesn't support the newer ListObjectsV2 call.
This is a rebase of https://github.com/durch/rust-s3/pull/175. (That PR was closed with no explanation, I'm not sure what happened there..)

@hlinnaka sorry about that, not sure either. Thanks for the contribution, will look this one over :)


Updated Python dependencies

Updated Python dependencies
Minor fix in gen-dependency-info.py to avoid missing fields
Added pytest_mock to devcontainer as @sbbroot forgot to add it after introducing tests using it.
Updated documentation for Python package management

/azp run


Jade native psbt

latest jade api to recognise more recently added hw
remove code which walked psbt and mapped it into Jade's legacy 'sign_tx' json format - instead pass PSBT directly to Jade for signing, since this has been supported in fw now for well over a year and seems to be working well.

Concept ACK

Thanks - any idea what's up with the CI ?  Always feel more confident when CI looks good ...

Thanks - any idea what's up with the CI ? Always feel more confident when CI looks good ...

You need to rebase.

Probably best superseded by #763

We prefer to support as old firmware as possible. This bumps the minimum supported firmware version, but it looks like it would be trivial to keep all of the old code while still using native PSBTs for newer firmware versions.

Rebased onto #768 (so this PR implicitly includes that PR), and retained support for legacy signing for old firmwares.
Thanks.

Rebase for #768 and also there's an actual CI failure.
hwilib/devices/jade.py:380: error: Returning Any from function declared to return "PSBT"  [no-any-return]


Fix hello-document support in shell-session language

Information:

Prism version: 1.20.0
Plugins: none
Environment: Browser

Does the latest version of Prism from the download page also have this issue?
yes
Description
First, thanks so much for Prism.
Putting together the regular expressions to pull this off must be painstaking!
Consider the following code block using the shell-session language.
$ cp /etc/ipsec.secrets /etc/ipsec.secrets.backup

$ cat << "EOF" > /etc/ipsec.secrets
: RSA vpn-server-a.key
EOF

$ cat << "EOF" > /etc/ipsec.secrets
: RSA vpn-server-a.key
# : RSA vpn-server-b.key
EOF

Issues:

Everything after cat is part of the output token. I believe the above code block should render three command tokens with no output.

Lines that start with # (or $) inside hello-documents become new commands. I believe commands inside of hello-documents should be ignored as they are part of a parent command.

See following screenshot.

@zeitgeist87 Perhaps you an help with this issue?

@sunknudsen You mean like this, right?

The problem is that our here doc strings assumed that there's nothing after the identifier.
Is it correct to assume that every after the identifier (so e.g. > /etc/ipsec.secrets but not the lines below) is a command and not part of the string? I'm asking because, as you can see, bash isn't highlighting the strings correctly and I don't know the correct interpretation.

@RunDevelopment I am fairly new to here documents, but my gut feeling is only cat should be considered a command and everything else after it (including the delimiting identifier) a string.
This is how it is illustrated on Wikipedia (see https://en.wikipedia.org/wiki/Here_document) and how Visual Studio Code handles it in markdown codeblocks and shell scripts (coloring the delimiting identifier is a nice addition).

Thank you for the clarification! I'll make a PR that fixes your issue.

@RunDevelopment Amazing, thanks Michael!


⚠ Bofrost.be has degraded performance

In 279d8fb, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2934 ms

Resolved: Bofrost.be performance has improved in 63ab358.


Define rule: Unmodifiable attributes of biomedical concepts should not be changed

Assuming that there are attributes of biomedical concepts (and their properties) that should not be modified - like the required flag - check that the values of the unmodifiable attributes in both BiomedicalConcept and BiomedicalConceptProperty match the values specified in the CDISC Library definition of the biomedical concept.
If necessary, this rule could be split into separate rules for BiomedicalConcept and BiomedicalConceptProperty

This rule requires defining which attributes of a biomedical concept are unmodifiable.  Suggestions for current COSMOS conceptual BC attributes:
bc_categories -- modifiable
bc_id -- unmodifiable (current codes are CDISC-defined, an implementation might have its own ids)
ncit_code -- unmodifiable (essential to semantic meaning of BC)
parent -- not sure (not all BCs have a parent; not sure how relationship to parent would be used)
short_name -- unmodifiable (human-readable version of ncit_code)
synonyms -- modifiable
result_scale -- unmodifiable
definition -- unmodifiable (essential to semantic meaning of BC)
change_history -- sort of assumes a version, which is not currently defined.


T-SNE

Dear Dr. zhou,
Thank you for your contribution!
Could you please share the T-SNE code which can plot the Figure 4?

the code has been updated
please refer to README in imcls/ for the details

thank you!

Hi，could you please share the T-SNE code for the style statistics(bottom) in Figure 4? I can only find the code for visualization of flattened feature maps... or could you please tell me what does the single point(bottom) in the Figure 4 mean? Thanks a lot!

I still can not understand what 'style statistics' stands for. Could you please clarify how you get Fig.4 a little bit further? Thank you so much.


Update Android compileSdkVersion and buildToolsVersion

The current Android compileSdkVersion, targetSdkVersion and buildToolsVersion uses the older but default version provided by the react-native init tool. However updating to the latest version might support possible Android build issues in scenarios where other third-party libraries require the root project to have their compiledSdkVersion and buildToolsVersion to be 25 or above. Have tested these changes in our own project and seems to be working fine without any issues.

The below issue mentions the need to update the SDK version to 26 in detail.
https://github.com/facebook/react-native/issues/17287

First of all, thank you for your PR.
We already had several PRs bumping the build tools version, while I would love to merge it and already did once, we will bump it as soon as facebook raises their minimum requirement.
Until then people would need to bump it on their own.

As soon this https://github.com/facebook/react-native/pull/17741 is merged we can merge this PR.

since RN 0.56.0 this is default gradle configuration
    buildToolsVersion = "26.0.3"
    minSdkVersion = 16
    compileSdkVersion = 26
    targetSdkVersion = 26
    supportLibVersion = "26.1.0"

can we update this appropriately and merge?

Please note that the build tools in this pull request are set to 26.0.2. The latest version is 26.0.3.

Thank you again for the PR, I am closing this one in favor of https://github.com/getsentry/react-native-sentry/pull/452


Connector plugin design is available in the repo

AC:

[ ] A simple design doc is added to the design/ folder based on the Plugin README. As community members will be reading this, the TODO sections in the plugin README should not be included
[ ] The CONTRIBUTING guide is updated with current info on contributing plugins (eg we support connectors, link to the design doc for now for more info)

Closing as a duplicate of #847


Operator projects using the removed APIs in k8s 1.22 requires changes.

Problem Description
Kubernetes has been deprecating API(s), which will be removed and are no longer available in 1.22. Operators projects using these APIs versions will not work on Kubernetes 1.22 or any cluster vendor using this Kubernetes version(1.22), such as OpenShift 4.9+. Following the APIs that are most likely your projects to be affected by:

apiextensions.k8s.io/v1beta1: (Used for CRDs and available since v1.16)
rbac.authorization.k8s.io/v1beta1: (Used for RBAC/rules and available since v1.8)
admissionregistration.k8s.io/v1beta1 (Used for Webhooks and available since v1.16)

Therefore, looks like this project distributes solutions via the Red Hat Connect with the package name as anchore-engine and does not contain any version compatible with k8s 1.22/OCP 4.9. Following some findings by checking the distributions published:

anchore-engine.v1.0.0: this distribution is using APIs which were deprecated and removed in v1.22. More info: https://kubernetes.io/docs/reference/using-api/deprecation-guide/#v1-22. Migrate the API(s) for CRD: (["anchoreengines.charts.anchore.io"])

NOTE: The above findings are only about the manifests shipped inside of the distribution. It is not checking the codebase.
How to solve
It would be very nice to see new distributions of this project that are no longer using these APIs and so they can work on Kubernetes 1.22 and newer and published in the Red Hat Connect collection. OpenShift 4.9, for example, will not ship operators anymore that do still use v1beta1 extension APIs.
Due to the number of options available to build Operators, it is hard to provide direct guidance on updating your operator to support Kubernetes 1.22. Recent versions of the OperatorSDK greater than 1.0.0 and Kubebuilder greater than 3.0.0 scaffold your project with the latest versions of these APIs (all that is generated by tools only). See the guides to upgrade your projects with OperatorSDK Golang, Ansible, Helm or the Kubebuilder one. For APIs other than the ones mentioned above, you will have to check your code for usage of removed API versions and upgrade to newer APIs. The details of this depend on your codebase.
If this projects only need to migrate the API for CRDs and it was built with OperatorSDK versions lower than 1.0.0 then, you maybe able to solve it with an OperatorSDK version >= v0.18.x < 1.0.0:

$ operator-sdk generate crds --crd-version=v1
INFO[0000] Running CRD generator.
INFO[0000] CRD generation complete.

Alternatively, you can try to upgrade your manifests with controller-gen (version >= v0.4.1) :
If this project does not use Webhooks:

$ controller-gen crd:trivialVersions=true,preserveUnknownFields=false rbac:roleName=manager-role  paths="./..."

If this project is using Webhooks:

Add the markers sideEffects and admissionReviewVersions to your webhook (Example with sideEffects=None and admissionReviewVersions={v1,v1beta1}: memcached-operator/api/v1alpha1/memcached_webhook.go):

Run the command:

$ controller-gen crd:trivialVersions=true,preserveUnknownFields=false rbac:roleName=manager-role webhook paths="./..."

For further info and tips see the blog.
Thank you for your attention.

Thanks, we will take a look. The operator itself lives in https://github.com/anchore/engine-operator so I may transfer this issue to that repository after triage.


Fix #3637 Bigtable RowMutation should allow passing of a Mutation

Fix #3637 Bigtable RowMutation should allow passing of a Mutation
Will add TableName in following PR as that needs to be consistent with other models BulkMutation, ConditionalRowMutation.

@igorbernstein2 @sduskis can you please review?

@igorbernstein2 updated as per feedback. Please review.

@igorbernstein2 sorry I misunderstood your earlier feedback. Reverted that commit and updated as per feedback.


Major re-org to crate / repository structure

Description
A major re-organization of the components of roslibrust per: https://github.com/orgs/RosLibRust/discussions/214
Some facets of this:

The core roslibrust crate now includes no backends by default. The rosbridge backend is now hidden behind the rosbridge feature flag. This means you don't pay to compile any backend that you aren't using.
The rosbridge backend moved under the rosbridge namespace in the roslibrust crate so it is no longer treated as a special backend in the naming conventions.
RosApi trait and implementations moved out of roslibrust and into their own crate. Also now generically written so they can be used with any backend.

Fixes
Closes: number
Checklist

[ ] Update CHANGELOG.md

@lucasw / @ssnover this is ready for a review now.
It is a pretty big change to overall crate structure, but I think aligns things much more where I want development to head.
I think it sets things up well to eventually include a ros2 backend with either zenoh or dds, and I think it sets us up very well to start making the generic traits for TopicProvider and ServiceProvider the a first class API that both allows folks to swap backends around and to use MockRos for testing.

Reading on mobile right now, but just going to ask clarifying questions and I'll review later:

Is the intended entry point crate roslibrust_common now?
Should it maybe be called roslibrust_core?

For users the intended entrypoint is still roslibrust
The "standard" usage would be to depend on roslibrust with at least one backend enabled via a feature flag.
Depending on roslibrust with no backends enabled can be done (see rosapi) to implement generic behaviors without a particular backend specified as well.
Not expecting anyone outside of roslibrust development to depend on roslibrust_common and I'm super happy to rename that to roslibrust_core or potentially roslibrust_traits <- that is most specifically what is actually inside it at the moment.


custom class name for components

Apologies for the simple question, but is there a way to assign custom classes to components? I'm trying to make a button 100% width.
#347 mentions some techniques, but I'm not sure if it is the same thing as what I'm trying to accomplish:
This doesn't work, with a friendly warning from vite Unused CSS Selector.
<Button icon={Add16} class="earth">Continue</Button>

If I inline the css, it works fine though:
<Button icon={Add16} style="width:100%">Continue</Button>

and as always, shortly after posting this, I discovered about the global option:
:global(.earth) {
        width: 100% !important;
    }

If you could please let me know if this is acceptable, would there be a better way?

That is a sensible approach. Another way would be to wrap components with a div to use Svelte's scoped styles. However, the downside is that it adds another more markup.

Thank you, closing.


rclone dedupe not working in GoogleDrive on latest beta

I noticed the dedupe command didn't seem to be working so I tested it and confirmed.
rclone.exe version

rclone v1.35-153-g1cc58e4β

rclone.exe md5sum gdrive:testdupes

2017/03/03 14:40:52 INFO  : Google drive root 'testdupes': Modify window is 1ms
cad1670909ccc3a6e858fa26dd50b5f2  Readme (2).html
cad1670909ccc3a6e858fa26dd50b5f2  Readme.html
cad1670909ccc3a6e858fa26dd50b5f2  Readme (1).html

rclone.exe --verbose=5 dedupe gdrive:testdupes

2017/03/03 14:41:16 DEBUG : rclone: Version "v1.35-153-g1cc58e4β" starting with parameters ["C:\\Users\\**\\Documents\\bin\\rclone.exe" "
--verbose=5" "dedupe" "gdrive:testdupes"]
2017/03/03 14:41:17 INFO  : Google drive root 'testdupes': Modify window is 1ms
2017/03/03 14:41:17 INFO  : Google drive root 'testdupes': Looking for duplicates using interactive mode.
2017/03/03 14:41:17 DEBUG : Google drive root 'testdupes': Reading ""
2017/03/03 14:41:17 DEBUG : Google drive root 'testdupes': Finished reading ""
2017/03/03 14:41:17 DEBUG : Go routines at exit 4
2017/03/03 14:41:17 DEBUG : rclone: Version "v1.35-153-g1cc58e4β" finishing with parameters ["C:\\Users\\**\\Documents\\bin\\rclone.exe"
"--verbose=5" "dedupe" "gdrive:testdupes"]

rclone.exe ls gdrive:testdupes

2017/03/03 14:41:29 INFO  : Google drive root 'testdupes': Modify window is 1ms
    16844 Readme.html
    16844 Readme (2).html
    16844 Readme (1).html

EDIT: To add on, after testing I observed the same behavior on OneDrive as well as with the linux version, I don't use all remotes but dedupe seems to be broken.

I believe dedupe needs to have the same name, same path, sane md5.  Your names are different so they are different.

This looks to be solved. If you still have an issue, feel free to post on the forums and ask.


gpt-engineer

Trying to use gemini-openai-proxy with the gpt-engineer
I invoke gpt-engineer as following OPENAI_API_KEY=1234 OPENAI_API_BASE=http://localhost:8080/v1 gpt-engineer Tic-tac-toe gpt-3.5-turbo and get the following error on proxy side
2023/12/31 07:50:03 genai get stream message error googleapi: Error 400:
[GIN] 2023/12/31 - 07:50:03 | 200 |  740.287576ms |             ::1 | POST     "/v1/chat/completions"

Not sure if the problem is on proxy side or gpt-engineer side tho

This error means that the request parameter has an issue, you should try printing the request parameter
	bs, _ := json.Marshal(req)
	fmt.Printf(string(bs))

This error means that the request parameter has an issue, you should try printing the request parameter
	bs, _ := json.Marshal(req)
	fmt.Printf(string(bs))

{"model":"","messages":null,"max_tokens":0,"temperature":0,"top_p":0,"n":0,"stream":false}2024/01/01 17:04:05 genai get stream message error googleapi: Error 400:
[GIN] 2024/01/01 - 17:04:05 | 200 |  649.504851ms |             ::1 | POST     "/v1/chat/completions"

{"model":"","messages":null,"max_tokens":0,"temperature":0,"top_p":0,"n":0,"stream":false}2024/01/01 17:04:05 genai get stream message error googleapi: Error 400:
[GIN] 2024/01/01 - 17:04:05 | 200 |  649.504851ms |             ::1 | POST     "/v1/chat/completions"

message in request data cannot be empty, please confirm if client has bug, this issue is not caused by gemini openai proxy


Document necessary configuration for sending expiration e-mails

Added some basic information on using Mailgun to https://github.com/wfhio/tramcar/wiki/Prod-Deploy.


Support npm-publish

npm-publish requires node and npm client.
It's not present so we get the error:
Info] Running npm Publish
[Error] exec: "npm": executable file not found in $PATH

Related with #348

Thank you for this PR @ruionwriting!
Can you please look at the Pull Requests Guidelines section in the README?
Specifically, please join our contributors community bt signing the CLA and also submit the PR on the dev branch, instead of the master.
In addition, I believe that npm is also needed for the npm-install command and not only for npm-publish, so I suggest correcting the comment in the Dockerfile.

@ruionwriting,
Are you planning to continue working on this PR?

@ruionwriting,
I'm closing this PR for now. Please let us know in case you do want to continue working on this.


Make file modifications atomic in install-cni.sh

This PR depends on https://github.com/istio/cni/pull/111 being merged first.
This will prevent CNI plugins and kubelet from potentially reading garbage during updates.

Still need to review in detail, but has an issue been seen with this?  either in real deployments or in integration testing?

Still need to review in detail, but has an issue been seen with this? either in real deployments or in integration testing?

This is fixing a race condition between the file updates and container creations. This has not been reported yet since it should be a rare occurrence. But it's a real bug and I'm fixing it proactively.
What are your concerns?

Maybe one reason why this has not been found is that there are no tests installing the CNI while pods are getting created?

Still need to review in detail, but has an issue been seen with this? either in real deployments or in integration testing?

This is fixing a race condition between the file updates and container creations. This has not been reported yet since it should be a rare occurrence. But it's a real bug and I'm fixing it proactively.
What are your concerns?

It's more about the priority for getting this in.

Now that 1.2 has been released, please review this again? Thanks!

Not stale. Still waiting for review for more than 4 months.

/test gencheck_cni

/cherry-pick release-1.4

@rlenglet: new pull request created: #216

In response to this:

/cherry-pick release-1.4

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


Unable to Clone Repository in Dockerfile

Preliminary Checks

[X] This issue is not a duplicate. Before opening a new issue, please search existing issues.
[X] This issue is not a question, feature request, or anything other than a bug report directly related to this project.

Description
Since the introduction of the zed-ros2-interfaces submodule, I cannot git clone the repository in a Dockerfile anymore. I suppose the issue has something to do with the SSH URL of the submodule. I also tried the zed-ros-wrapper, which also has a submodule but with an HTTPS URL, and that works.

ENTRYPOINT [ "./entrypoint.sh" ]

and the entrypoint.sh file
#!/bin/bash
# shellcheck disable=SC1090

source "${ROS_ROOT/foxy/foxy\/install}/setup.bash"
source "${ROS_WS}/install/setup.bash"

exec "$@"

Steps to Reproduce
Build with
docker build --build-arg ZED_SDK_VERSION=3.6 --build-arg L4T_VERSION=r32.6.1 --build-arg ROS_DISTRO=foxy --platform linux/arm64 --tags test .

Expected Result
Docker builds the image without failures.
Actual Result
I get the following error
[...]
 You can look around, make experimental
#10 132.3 changes and commit them, and you can discard any commits you make in this
#10 132.3 state without impacting any branches by performing another checkout.
#10 132.3
#10 132.3 If you want to create a new branch to retain commits you create, you may
#10 132.3 do so (now or later) by using -b with the checkout command again. Example:
#10 132.3
#10 132.3   git checkout -b <new-branch-name>
#10 132.3
#10 134.4 Submodule 'zed-ros2-interfaces' (git@github.com:stereolabs/zed-ros2-interfaces.git) registered for path 'zed-ros2-interfaces'
#10 134.8 Cloning into '/workspaces/foxy/src/zed-ros2-wrapper/zed-ros2-interfaces'...
#10 134.8 fatal: unable to fork
#10 134.9 fatal: clone of 'git@github.com:stereolabs/zed-ros2-interfaces.git' into submodule path '/workspaces/foxy/src/zed-ros2-wrapper/zed-ros2-interfaces' failed
#10 134.9 Failed to clone 'zed-ros2-interfaces'. Retry scheduled
#10 135.1 Cloning into '/workspaces/foxy/src/zed-ros2-wrapper/zed-ros2-interfaces'...
#10 135.1 fatal: unable to fork
#10 135.1 fatal: clone of 'git@github.com:stereolabs/zed-ros2-interfaces.git' into submodule path '/workspaces/foxy/src/zed-ros2-wrapper/zed-ros2-interfaces' failed
#10 135.1 Failed to clone 'zed-ros2-interfaces' a second time, aborting
------
executor failed running [/bin/bash -c apt-get update &&     apt-get install -y --no-install-recommends git &&     git clone -b "${ROS_DISTRO}-v${ZED_SDK_VERSION}" --recurse https://github.com/stereolabs/zed-ros2-wrapper.git &&     git clone -b "${ROS_DISTRO}" https://github.com/ros/diagnostics.git &&     git clone -b ros2 https://github.com/ros-perception/image_common.git &&     git clone -b ros2 https://github.com/ros/xacro.git]: exit code: 1

ZED Camera model
ZED2i
Environment
OS: Ubuntu 20.04

Anything else?
The git clone command works for zed-ros-wrapper, though. This is the reason why I suspect the submodule's URL.

Hi @kevinanschau
I tested the command $ git clone --recurse-submodules https://github.com/stereolabs/zed-ros2-wrapper.git and everything seems to work correctly:
$ git clone --recurse-submodules https://github.com/stereolabs/zed-ros2-wrapper.git
Cloning into 'zed-ros2-wrapper'...
remote: Enumerating objects: 1444, done.
remote: Counting objects: 100% (657/657), done.
remote: Compressing objects: 100% (436/436), done.
remote: Total 1444 (delta 371), reused 436 (delta 202), pack-reused 787
Receiving objects: 100% (1444/1444), 1.86 MiB | 1.65 MiB/s, done.
Resolving deltas: 100% (777/777), done.
Submodule 'zed-ros2-interfaces' (git@github.com:stereolabs/zed-ros2-interfaces.git) registered for path 'zed-ros2-interfaces'
Cloning into '/home/walter/temp/zed-ros2-wrapper/zed-ros2-interfaces'...
remote: Enumerating objects: 48, done.
remote: Counting objects: 100% (48/48), done.
remote: Compressing objects: 100% (34/34), done.
remote: Total 48 (delta 13), reused 38 (delta 10), pack-reused 0
Receiving objects: 100% (48/48), 713.85 KiB | 1.39 MiB/s, done.
Resolving deltas: 100% (13/13), done.
Submodule path 'zed-ros2-interfaces': checked out '5fab23c62ae6199021c35e6802d2c87ac87276a9'

you are using this command that is slightly different:
git clone -b "${ROS_DISTRO}-v${ZED_SDK_VERSION}" --recurse https://github.com/stereolabs/zed-ros2-wrapper.git
What are the values of the ROS_DISTRO and ZED_SDK_VERSION variables?
Why do you use --recurse and not --recurse-submodules? I looked in the documentation and --recurse seems to not exist.

Thank you for your reply, @Myzhar.
I also tested your command locally and it works. But when I use the same command in the Dockerfile, it doesn't, and it outputs the same error I got above.
I run this command
docker build --build-arg ZED_SDK_VERSION=3.6 --build-arg L4T_VERSION=r32.6.1 --build-arg ROS_DISTRO=foxy --platform linux/arm64 --tags test .

and so the command becomes
git clone -b foxy-v3.6 --recurse https://github.com/stereolabs/zed-ros2-wrapper.git

--recurse worked for me outside the Dockerfile. Don't know why, but I see that it's not in the documentation. Regardless, changing it to --recurse-submodules or --recursive didn't solve the issue.
Oddly enough, using the same command inside the Dockerfile to clone the zed-ros-wrapper works.

Hi @Myzhar,
commit 6ce5488 fixed this issue. Thank you very much.


Issues with multiple views inside a UIScrollView

Hi there,
First of all, I want to say that this library is really awesome and is one of the reasons that I'm using autolayout right now. Been using it for a new project and everything is working like a charm, really appreciate it.
But, I'm trying to use it on a ScrollView and I'm having a lot of trouble. I tried to solve this in multiple ways and different approaches and I'm not having any success at all. The more I try the less I feel I understand how Autolayout and Masonry works. Maybe that's the problem, not Masonry but the way Autolayout works in ScrollViews.
The main issue I have is that the scrollView only scrolls horizontally and not vertically, it's stuck.
Found this issue https://github.com/cloudkite/Masonry/issues/12 and tried to always set the width of my subviews. I even copied exactly what he did there, but then I created like 50 views in the same scrollView and it doesn't scroll.
Been reading this https://developer.apple.com/library/ios/technotes/tn2154/_index.html but in this case, they use only one big view, and not multiple ones.
At this point, after so many hours trying, I think that I'd better see an example and start again than try to fix my own one so I created a new ViewController and added this code to the init:
    self.scrollView = [[UIScrollView alloc] initWithFrame:CGRectZero];
    _scrollView.backgroundColor = [UIColor grayColor];
    [self.view addSubview:_scrollView];

    [_scrollView makeConstraints:^(MASConstraintMaker *make) {
        make.edges.equalTo(self.view);
    }];

    NSNumber* height = [NSNumber numberWithInt:350];

    UIView *view1 = [[UIView alloc] initWithFrame:CGRectZero];
    view1.backgroundColor = [UIColor blueColor];
    [_scrollView addSubview:view1];

    UIView *view2 = [[UIView alloc] initWithFrame:CGRectZero];
    view2.backgroundColor = [UIColor greenColor];
    [_scrollView addSubview:view2];

    UIView *view3 = [[UIView alloc] initWithFrame:CGRectZero];
    view3.backgroundColor = [UIColor redColor];
    [_scrollView addSubview:view3];

    [view1 makeConstraints:^(MASConstraintMaker *make) {
        make.top.equalTo(@0);
        make.left.equalTo(@0);
        make.width.equalTo(_scrollView.width);
        make.height.equalTo(height);
    }];

    [view2 makeConstraints:^(MASConstraintMaker *make) {
        make.top.equalTo(view1.mas_bottom);
        make.left.equalTo(@0);
        make.width.equalTo(_scrollView.width);
        make.height.equalTo(height);
    }];

    [view3 makeConstraints:^(MASConstraintMaker *make) {
        make.top.equalTo(view2.mas_bottom);
        make.left.equalTo(@0);
        make.width.equalTo(_scrollView.width);
        make.height.equalTo(height);
    }];

This code adds the first view and the next two correctly, one using the bottom as a top reference, but the UISCrollView is not scrolling at all. I also tried to add everything inside a UIView. I even tried to get the last object Y position, but I get a very high number that doesn't make sense.
Should I have to set the contentSize of the scrollView ? but that's why I wanted the Y position for, I'm feeling that this is the wrong direction.
At this point I hope that somebody can help me, I will really appreciate it, and even better, seeing some example. I think I'm missing something here.
Thanks in advance :)

As I couldn't find this solution anywhere else but here, in case someone needs it...
I didn't have to add the footer view.
Just set the bottom constraint of the last view in the scrollview and it works fine.
In the example code above, the last useful view would be the view3.
In Swift:
view3.snp.makeConstraints { make in
        make.top.equalTo(view2.snp.bottom)
        make.left.equalToSuperview()
        make.right.equalToSuperview()
        make.bottom.equalTo(scrollView.snp.bottom).offset(-20)
}


ERROR in app: Exception on /triggers/manual [POST]

Hello - I am getting the error below for some paths.  The paths do exist and are accesible by Plex.

[2024-02-07 07:56:24,572] WARNING in app: Starting directory scan of: /RAID/deleted/TV
[2024-02-07 07:56:24,578] ERROR in app: Exception on /triggers/manual [POST]
Traceback (most recent call last):
File "/usr/local/lib/python3.11/site-packages/flask/app.py", line 2525, in wsgi_app
response = self.full_dispatch_request()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/flask/app.py", line 1822, in full_dispatch_request
rv = self.handle_user_exception(e)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/flask/app.py", line 1820, in full_dispatch_request
rv = self.dispatch_request()
^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/flask/app.py", line 1796, in dispatch_request
return self.ensure_sync(self.view_functions[rule.endpoint])(**view_args)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/app/app.py", line 30, in trigger
metadata_files = plex_api.find_metadata_from_dirs(directory=directory)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/app/plexapihandler.py", line 33, in find_metadata_from_dirs
media_name = pathlib.PurePath(directory).parts[section_parts]
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^
IndexError: tuple index out of range
<IP_ADDRESS> - - [07/Feb/2024 07:56:24] "POST /triggers/manual?dir=/RAID/deleted/TV HTTP/1.1" 500 -

Are you using the 0.2.0 version or 0.1.1?
Does it happen in all the scans or only in this particular case?

Because of the line number I can infer that you are using the old 0.1.1 version. Nevertheless the new version should still have that bug in it.
Looks like the bug is caused by Plex having a different media location than the instance triggering the update (sonarr, radarr, bazarr o whatever service is calling the adapter). This is generally due to mount discrepancies.

I have just updated to the latest container and I've double-checked the paths - they are all the same and are consistent with the host paths themselves as Plex runs natively.

@jrcalvert hmm looks like I've found the issue but to confirm it I need you to provide the full path to the media file that produced the error. The full path should include the media file. You can mask any username if you'd like for security. I will also need the path to the media library that you added in Plex.
As a summary what I think is happening is: for paths which have a subdirectory the scanning will work okay, but for the absolute paths that contain the media file it won't.
Lets say you have the media library  /RAID/deleted/TV in plex. If you scan for /RAID/deleted/TV/Walking_Dead/Season_1/S01E01.mkv it will work. Lets now assume you have /RAID/deleted/TV in plex but your file is located at /RAID/deleted/TV/S01E01.mkv, this will not work.
See how the first example has subpaths Walking_Dead and Season_1 in it after the media library path, but the second example doesn't. This happens because the script assumes a sonarr/radarr-like file structure which is not always the case.
https://github.com/dantebarba/autoscan-adapter/blob/8e143fe8b636467146ecde93aa249055cf7b903c/plexapihandler.py#L32
If you can confirm all this by providing the full paths I will push a version with a fix. Thanks

Thank you.
Sorry, I have been away
On Sun, 18 Feb 2024, 21:46 Dante Barba, @.***> wrote:

@jrcalvert https://github.com/jrcalvert the issue has. been solved in
https://github.com/dantebarba/autoscan-adapter/tree/0.3.0
—
Reply to this email directly, view it on GitHub
https://github.com/dantebarba/autoscan-adapter/issues/5#issuecomment-1951454778,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADXRBWUYODAUQ3WMYFZHJMLYUJZDFAVCNFSM6AAAAABC6HYNFSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJRGQ2TINZXHA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Be more specific to the permission required

When customer restore a storage account and it is blocked by Azure Policy, the portal doesn't tell more details about the reason, only return error: "The storage account 'xxxx' could not be recovered. If the issue persists, we may not be able to recover this storage account."
To the prerequisite mentioned in this document "The user who is recovering the storage account must be assigned an Azure RBAC role that provides the Microsoft.Storage/storageAccounts/write permission. For information about built-in Azure RBAC roles that provide this permission, see Azure built-in roles."
To be more specific, we need that RBAC permission "Microsoft.Storage/storageAccounts/write" and the permission and activity should not be blocked by any other features like Azure Policy.

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

ID: 19bf80a6-331d-dbc1-1a4d-067eff42e159
Version Independent ID: 06c51db4-0789-f196-aed3-9753a2e0af57
Content: Recover a deleted storage account - Azure Storage
Content Source: articles/storage/common/storage-account-recover.md
Service: storage
Sub-service: common
GitHub Login: @jimmart-dev
Microsoft Alias: jammart

@uonie
Thanks for your feedback! We will investigate and update as appropriate.

Thanks @uonie. I am the customer in this case and raised the issue with support originally. The main crux of feedback is that a self-service SA recovery is a time-critical action. In this case, we wanted to be able to have the SA recovered as quickly as possible. The problem we had was that the self-service recovery feature failed without providing a reason. This then required us to raise a support case. This slowed down our recovery substantially.
I get that there are limitations to these types of capabilities. I feel it is critical though that:
a) MS fully disclose ALL the limitations that may prevent self-service recovery from working
b) When the self-service recovery fails, provide explicit feedback to the customer about WHY it failed
As the capability current stands, there is zero mention that any Policy controls may impact self-service recovery.

@ausfestivus from my personal view, the platform provides all kinds of features like soft-delete, snapshot, backup, RBAC permission control etc. to avoid such accidential deletion but it still happens, the PG released this feature for the customers who have deleted their storage accounts to recover SA by them-selves in a more convenient way, and we can also contact support team for the recovery anyway, which means we can always have a workaround for this with Microsoft best effort basis.
We appreciate your feedback, and I agree with you that from a feature perspective, with such requests the product could be more complete, and it gonna take time for the enhancement.

Thanks for your contribution to our documentation
Unfortunately, we have been unable to review this issue in a timely manner. We sincerely apologize for the delayed response. We are closing this issue. If you feel that the problem persists, please respond to this issue with additional information.
?
Please continue to provide feedback about the documentation. We appreciate your contributions to our community.
#please-close

#please-open
Closing issues because Microsoft doesnt have the time or people to resolve a reported problem is very poor form. Closing an issue indicates that the problem is resolved. This is not resolved as far as can be seen on this issue's history.
GitHub 101 folks.


GraphQL aliases

How do you specify aliases ?
e.g.:
{
  empireHero: hero(episode: EMPIRE) {
    name
  }
  jediHero: hero(episode: JEDI) {
    name
  }
}

Currently you can't do aliases, you can do separate the requests in different objects and use batching to limit the network requests (wrapping inside a Promise.all)
there is an example here

I won't implement this, i will add native batching support instead

@remorses I think I have a scenario where batching is not a valid replacement for aliases.
Suppose I'm fetching a list of items with nested details like this:
query getOrders {
  orders {
    id
    name
    allTasks: tasks_aggregate {
      aggregate { count }
    }
    completedTasks: tasks_aggregate(
      where: { task: { completed_at: { _is_null: false } } }
    ) {
      aggregate { count }
    }
  }
}

To replicate this with batching I would have to fetch the list of items twice and this doubles the cost of the query! 😬

Maybe I could add support for this passing an array of objects queries and merge them using something like https://github.com/domasx2/graphql-combine-query
This way you would get the same power of aliases without batching, but I would need to add more complex logic to rename the implicit aliases back to their original name (only when you query fields with same name)

How batching resolves this issue exactly?

How batching resolves this issue exactly?
 AN
type X {
value: A
}
so, N clients needed?

@remorses for some reason I got the notification only now.
Anyway, looks like it could be a solution, although I'm not sure this way the API would be as intuitive as with using GraphQL literals.
In terms of API maybe something like this might better resemble the alias DX. Basically it adds a third array item to specify aliases.
orders: [
  {},
  {
    id: true,
    name: true,
    allTasks: {
      aggregate { count }
    },
    completedTasks: [{
      where: { task: { completed_at: { _is_null: false } } }
    }, {
      aggregate { count }
    }]
  },
  {
    allTasks: "tasks_aggregate",
    completedTasks: "tasks_aggregate"
  }
]

@remorses for some reason I got the notification only now.
Anyway, looks like it could be a solution, although I'm not sure this way the API would be as intuitive as with using GraphQL literals.
In terms of API maybe something like this might better resemble the alias DX. Basically it adds a third array item to specify aliases.
orders: [
  {},
  {
    id: true,
    name: true,
    allTasks: {
      aggregate { count }
    },
    completedTasks: [{
      where: { task: { completed_at: { _is_null: false } } }
    }, {
      aggregate { count }
    }]
  },
  {
    allTasks: "tasks_aggregate",
    completedTasks: "tasks_aggregate"
  }
]


Team for Tuesday

Value: $0.00Closes: Tuesday, April 17, 2018Location: Victoria Some in-person work requiredOpportunity DescriptionWe know some stuffAcceptance CriteriaHow to ApplyGo to the Opportunity Page, click the Apply button above and submit your proposal by 16:00 PST on Tuesday, April 17, 2018.We plan to assign this opportunity by Friday, April 20, 2018 with work to start on Tuesday, April 17, 2018.Proposal Evaluation Criteria

This was a test.


Flutter support for variable font attribute FontFeature('wght', x) request

Internal: b/195883187

Use case
A device platform using Flutter required us to establish and maintain a completely separate icon font (250±) just to use icons at a different size/weight from platform to platform.

Proposal
Would like Flutter to add support for the weight attribute of variable fonts. It seems to support all other attributes other than weight.

Per Jason, exposing the feature is easy enough. It would be good to have an instance of the font so we can verify the implementation.

@Hixie I see the customer label. Can you cross reference with the internal issue so we can follow along on any internal discussions please?

Per Jason, exposing the feature is easy enough. It would be good to have an instance of the font so we can verify the implementation.

It would be the Google GM icons font. Nothing otherwise custom.

The FontFeature API only supports feature tags.
'wght' is a variation axis used by variable fonts.  Flutter does not currently support variable fonts, but this has been a longstanding feature request (https://github.com/flutter/flutter/issues/33709)
I looked into this and confirmed that Skia and the font shaping libraries used by Flutter can handle variable fonts.  As an experiment I tried patching Libtxt to use SkFontArguments::setVariationDesignPosition and SkTypeface::makeClone to create an SkTypeface based on an OpenType variable font.  The font rendered as expected.
More work will be required to expose this as an API.  This will also need to be supported in SkParagraph.

This is going to be implemented in both the SkParagraph and libTXT backends.

This doesn't seem to be a blocker just yet. It is on Jason's TODO list for later in the month. Adjusting priority.

Is it possible we can just skip the libtxt implementation?

Perhaps, although I don't expect that Libtxt-specific work would be a major part of the overall effort.
On the engine side we need to set up the API and verify that the font loading/rendering libraries that we are using work as intended with these features.  If we do a Libtxt implementation, it's mostly serving as a way to validate those parts and possibly for backward compatibility until everyone is migrated.

As https://github.com/flutter/flutter/pull/100978 is merged, can this be closed?

Yea, I've created an issue for the remaining work. #103131


Make MDN more reliable by separating the service of file attachments, which uses AWS EFS, from the rest of MDN.

Description
General
From experience we know that a majority of the pods for the web deployment within the MDN Kubernetes cluster are required simply to handle the requests for file attachments (e.g., /files/2767/hut.jpg). The actual file attachments are stored within an AWS EFS instance, so any pod serving a request for a file attachment depends upon EFS.
Problem
We recently experienced downtime which seemed to be due to significantly increased latency with our EFS instance. The increased latency caused the pods that were serving file attachments to take longer, which in turn did two things. First, it forced more and more pods to occupy themselves with serving the load of incoming requests for file attachments, rather than MDN documents. Second, more of the pods trying to serve file attachments, since they were waiting on EFS, started to fail their liveness and readiness probes, which then made matters even worse since those pods were restarted or removed from the incoming traffic. In the end, this vicious cycle left a dwindling number of pods available to handle actual MDN document requests, and users began to see huge page-load times.
Proposed Solution
This current vulnerability of MDN can be mitigated by serving file attachments with a separate group of pods. With that in place, if EFS were to have latency issues again, the requests for file attachments would have problems, but since the requests for MDN documents would be served by a separate group of pods, they would not be affected. MDN pages would load, but they might be missing their file attachments.
User Story
As an MDN user, I want MDN to avoid downtime or latency based on serving file attachments, so that I can quickly access the information I need.
Acceptance Criteria

Create a new certificate with domains mdn.mozillademos.org and mdn-demos-origin.moz.works for prod.
A new certificate exists with domain stage-files.mdn.moz.works for stage.
An "attachments" K8s service exists.

This would be exactly the same as the current "web" service (built upon https://github.com/mozmeao/infra/blob/master/apps/mdn/mdn-aws/k8s/cert.svc.yaml.j2), but with a SERVICE_NAME of attachments and the certificate from above.
A new Makefile target (k8s-attachments-svc) exists, modeled after k8s-web-svc to create the service, and add it as a dependency to the k8s-services target.
The appropriate environment variables exist in the setup scripts under https://github.com/mozmeao/infra/tree/master/apps/mdn/mdn-aws/k8s/regions.

An "attachments" K8s deployment exists.

This would be exactly the same as the current "web" deployment (built upon https://github.com/mozmeao/infra/blob/master/apps/mdn/mdn-aws/k8s/kuma.deploy.yaml.j2), but with a different name.
-A new Makefile target (k8s-attachments) exists, modeled after k8s-web to create the deployment, and add it as a dependency to the k8s-kuma-deployments target.
The appropriate environment variables exist in the setup scripts under https://github.com/mozmeao/infra/tree/master/apps/mdn/mdn-aws/k8s/regions.
This deployment is configured to use a separate New Relic label for reporting purposes.
1.The new "attachments" service and deployment are deployed to the mdn-stage namespace.

The DNS for stage-files.mdn.moz.works is updated from the ELB for the "web" service to the new ELB for the "attachments" service.
Tests pass on stage.
The new "attachments" service and deployment are deployed to the mdn-prod namespace.
The DNS for mdn-demos-origin.moz.works is updated from prod.mdn.moz.works to the new ELB created as part of creating the K8s "attachments" service above.
Tests pass on prod.

This has been completed.


Strip "hdfs://" from path

Hi all,
I was thinking about opening a PR to ensure we strip hdfs:// from the path (both in the file system and file objects).
For the file system object, the best place would be to put it in ensure_bytes, while it could be in the __init__ of the file object.
Would anyone object to such an approach?
The rationale is that we often do
path = 'hdfs://something'
df.write.format('parquet').save(path)
# do more here
hdfs = hdfs3.HDFileSystem()
hdfs.ls(path)  # or other methods

and we currently need to strip path otherwise hdfs3 will fail to find the path.
What do you think? (Again, I volunteer to do the work, add tests, etc.)

Firstly, I should say that hdfs3 is no longer unedrgoing much development, as pyarrow's hdfs implementation is now a mature replacement. You may find, that if you use that implementation, this issue is already solved for you.
On the specifics, ensure_bytes would not be a good place to put this code, since it is not only called with path strings. Instead, you would end up doing something like all of the if path.startswith('s3://') for the s3fs case or its generalization.
If you would still like to take this on, I would be happy to review it.

Hi Martin
I didn't know about the new pyarrow situation. I guess I'll have to embrace the JNI way of doing things :)
I'll close the issue, I haven't done any work on it yet. We might always reopen.


Add Accio to Homebrew Core

Now that Accio has passed the 50 stars, before everyone gets used to installing Accio via its Tap too much, we should try adding it to Homebrew Core. That also seems reasonable now given that Accio has stabilized a lot lately.

Closing this, as Accio is now deprecated.


minor improvement in the readme

Thanks for the script. I made small changes in the readme.
I didn't read the blog post. Directly came here from google search. If we have the details in the readme it would be helpful.
Once again, thanks.

Thanks for the changes!


Adjust mask bits from 57 into 32

I believe this should be faster than the current version. In my own benchmark, it indeed was fast by a small amount.
I think changing the mask bits from 24 into 32 for 3~4 digits case might be also beneficial at least on x64, because there is no perf difference between imul for 32-bit registers and imul for 64-bit registers. But it seems that the difference seems to be not even measurable, so I will leave it as is.

Thanks. I'll see if I can repeat the performance difference. On msvc I saw a slight boost to using the 57 shift on my system so I went with that for this version. 32 bit shifts would likely be better for x86

I tested it again...can't get consistent results


Add ImpalaClient.close method that closes Impyla sessions and drops temp tables

Close #533

@mariusvniekerk I'll drop this in 0.4 -- give it a shot and report bugs as they arise!


Develop Branch: Admin user email is hardcoded in build/templates/test/master.json

Expected Behavior
Admin email address should not be hardcoded in build/templates/test/master.json
Suggest prompt for email address, and/or require email address as environment var or parameter.
Actual Behavior
Admin email address is hardcoded to:
"Email"<EMAIL_ADDRESS>When stack is created non interactively by a different user (using "npm run stack test/master up"), the Cognito QnABot signup email is not received.
Steps to Reproduce the Problem

As a user that is not jcalho, create stack from cloned repo (develop branch) using 'npm run stack test/master up'
try to log in to Content Designer UI

Specifications

Version:
Platform:

maybe add the script in npm install that clears that field, or have it tied to /config.json

fixed in v2


Refactoring parsing-related code to remove almost-duplicate types.

ParseTreeVisitor now directly generates Terms, Atoms, Rules, etc. instead of creating intermediate ParsedTerm, ParsedAtom, etc.
Removing unnecessary ParsedTerm, ParsedAtom, etc.
Moving ChoiceAtom and RuleAtom from common to grounder packages.
Adapting all test cases.

Coverage decreased (-0.5%) to 70.472% when pulling 5697f8a62792f299188b071963ff52dde771b87b on parser_refactoring into 59d24ca8d88a883c351472ac0376f68d4894bd24 on master.

It seems that the AppVeyor CI needs some missing configuration files. Lorenz?

Cool. I wanted to do this for quite some time ... Will take a closer look soon.

Coverage decreased (-0.5%) to 70.472% when pulling 0dc2fc97f9b014d4b56c5b0b037a76a8ed4ad5a2 on parser_refactoring into 59d24ca8d88a883c351472ac0376f68d4894bd24 on master.

Coverage decreased (-0.5%) to 70.472% when pulling d3376ae798ab155656692b3bf17a970195a3691c on parser_refactoring into 59d24ca8d88a883c351472ac0376f68d4894bd24 on master.

Coverage decreased (-0.5%) to 70.475% when pulling 2a270dc80d81b7a61d32b968272fd5b1e0384cb5 on parser_refactoring into 59d24ca8d88a883c351472ac0376f68d4894bd24 on master.

Coverage decreased (-0.5%) to 70.475% when pulling 2a270dc80d81b7a61d32b968272fd5b1e0384cb5 on parser_refactoring into 59d24ca8d88a883c351472ac0376f68d4894bd24 on master.


lua53: revision for mkpath

Versions follow-up to https://github.com/Homebrew/homebrew/pull/48242 fix.

:+1:


py-spy: 0.3.14 -> 0.3.14-unstable-2024-02-27; fix build, pin python311

Description of changes
pin python to 3.11 to fix build and update to unstable to fix build on darwin
https://github.com/benfred/py-spy/compare/v0.3.14...8dd54929106916a3c961cc57c1172793ce126180

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 325422 run on x86_64-linux 1

4 packages marked as broken and skipped:

python311Packages.vllm
python311Packages.vllm.dist
python312Packages.vllm
python312Packages.vllm.dist

4 packages failed to build:

python311Packages.kserve
python311Packages.kserve.dist
python312Packages.kserve
python312Packages.kserve.dist

5 packages built:

py-spy
python311Packages.ray
python311Packages.ray.dist
python312Packages.ray
python312Packages.ray.dist

fix kserve https://github.com/NixOS/nixpkgs/pull/325423

at least  x86_64-darwin passes :-)

Result of nixpkgs-review pr 325422 run on aarch64-linux 1

8 packages marked as broken and skipped:

python311Packages.kserve
python311Packages.kserve.dist
python311Packages.vllm
python311Packages.vllm.dist
python312Packages.kserve
python312Packages.kserve.dist
python312Packages.vllm
python312Packages.vllm.dist

1 package built:

py-spy

Result of nixpkgs-review pr 325422 run on x86_64-linux 1

4 packages marked as broken and skipped:

python311Packages.vllm
python311Packages.vllm.dist
python312Packages.vllm
python312Packages.vllm.dist

4 packages failed to build:

python311Packages.kserve
python311Packages.kserve.dist
python312Packages.kserve
python312Packages.kserve.dist

5 packages built:

py-spy
python311Packages.ray
python311Packages.ray.dist
python312Packages.ray
python312Packages.ray.dist

Result of nixpkgs-review pr 325422 run on aarch64-darwin 1

8 packages marked as broken and skipped:

python311Packages.kserve
python311Packages.kserve.dist
python311Packages.vllm
python311Packages.vllm.dist
python312Packages.kserve
python312Packages.kserve.dist
python312Packages.vllm
python312Packages.vllm.dist

1 package built:

py-spy

Result of nixpkgs-review pr 325422 run on x86_64-darwin 1

8 packages marked as broken and skipped:

python311Packages.kserve
python311Packages.kserve.dist
python311Packages.vllm
python311Packages.vllm.dist
python312Packages.kserve
python312Packages.kserve.dist
python312Packages.vllm
python312Packages.vllm.dist

1 package built:

py-spy

@GaetanLepage wow thanks for such prompt response !
I'll leave some time for @SuperSandro2000 to have a look

@ofborg build py-spy