VertexAISearch throwing 'Request contains an invalid argument'

Environment details

OS: Windows10
.NET version: net6.0
Package name and version: Google.Cloud.AIPlatform.V1" Version="3.4.0

Steps to reproduce
    public string Error(string query)
    {
        string publisher = "google";
        string model = "gemini-1.5-pro-001";
        var generateContentRequest = new GenerateContentRequest
        {
            Model = $"projects/{_googleProject}/locations/{_googleLocation}/publishers/{publisher}/models/{model}",
            Tools =
            {
                new Tool
                {
                    Retrieval = new Retrieval
                    {
                        VertexAiSearch = new VertexAISearch
                        {
                            Datastore = $"projects/{_googleProject}/locations/{_dataStoreLocation}/collections/default_collection/dataStores/{_dataStoreId}",
                        }
                    },
                }
            },
            Contents =
            {
                new Content
                {
                    Role = "USER",
                    Parts =
                    {
                        new Part
                        {
                            Text = query,
                        },
                    }
                }
            },
        };

        return new PredictionServiceClientBuilder
        {
            Endpoint = $"{_googleLocation}-aiplatform.googleapis.com",
        }.Build().GenerateContent(generateContentRequest).Candidates[0].Content.Parts[0].Text;
    }

Throws:
{"Status(StatusCode=\"InvalidArgument\", Detail=\"Request contains an invalid argument.\")"}

Commenting out VertexAISearch tool makes it work
Tested with GoogleSearchRetrieval and that works

Not sure what/how to test/debug this, specified a logger but the only thing I see is Starting asynchronous API call to GenerateContent. and boom, no other information
Any tips on what the issue can be or how to add more logs/debug information? 🙏

Also tested creating data stores in multiple locations or even creating a search app but don't think that's required?

Thanks for raising this issue. Unfortunately this sounds like it's really a question about the AI Platform API itself rather than about the .NET client library. The maintainers of this repository aren't experts in all the different Google APIs that are available - only how the .NET client libraries can be used to make requests to those APIs. (We can help customers who know what their API call should look like but are having trouble with how to make that request using C#, but not with figuring out the request to start with.) I've assigned this issue to another engineer who may be able to help you, but otherwise the language-neutral AI Platform support routes are probably your best bet.
One thing you could do to see if there's more information is to check the exception for more status details:
catch (RpcException ex)
{
    foreach (var detail in ex.GetRpcStatus()?.UnpackDetailMessages() ?? Enumerable.Empty<IMessage>())
    {
        // Or whatever logging you have
        Console.WriteLine($"{detail.Descriptor.Name}: {detail}");
    }
}

Hi @Martin-Molinero, we have a VertexAiSearch sample here: https://github.com/GoogleCloudPlatform/dotnet-docs-samples/blob/main/aiplatform/api/AIPlatform.Samples/GroundingVertexAiSearchSample.cs
I just tested with with a datastore and it worked fine for me.
Could you please give it a try and let us know?

Hi!
Thank you for the help, looking at the exception was able to find more information about the error, not sure how to solve it yet

Status(StatusCode="InvalidArgument", Detail="max_extractive_answer_count must be not specified when the datastore is using 'chunking config'")

Maybe it's how you crate the datastore? This is how I create it: https://github.com/meteatamel/genai-beyond-basics/tree/main/samples/grounding/vertexai-search

Hey @meteatamel! Thank you
Yes I was enabling chunk documents when creating the data store, which seemed nice to have, that was causing the issue I had. Not sure how to change max_extractive_answer_count on request, seems internal configuration, any ideas?
Closing the issue!

I'm not sure how to change max_extractive_answer_count but glad you got it working now


[Tooltip] Fix flip invalid CSS property error

flip is not a valid css property.

[x] I have followed (at least) the PR section of the contributing guide.

https://validator.w3.org/nu/?doc=https%3A%2F%2Fwww.a11ywatch.com%2F

@j-mendez The property is targeted at https://github.com/alitaheri/jss-rtl. Did you face a specific issue with it?

@j-mendez It's a great first pull request on Material-UI 👌🏻. Thank you for working on it!

@oliviertassinari thank you and your welcome!


[CY-4991]  invalid DOCKER_HOST must be unset or unix:///var/run/docker.sock

The codacy-analysis-cli-action execution fails when DOCKER_HOST is a different socket value than unix:///var/run/docker.sock.
Run codacy/codacy-analysis-cli-action@master
+ '[' '' == true ']'
+ echo 'Skipping GoSec'
Skipping GoSec
+ '[' '' == true ']'
+ echo 'Skipping StaticCheck'
Skipping StaticCheck
+ '[' -n '' ']'
+ echo 'Skipping Clang Tidy'
Skipping Clang Tidy
+ '[' -n '' ']'
+ echo 'Skipping Faux Pas'
Skipping Faux Pas
--2021-09-13 21:59:35--  https://raw.githubusercontent.com/codacy/codacy-analysis-cli/7.0.5/bin/codacy-analysis-cli.sh
Resolving raw.githubusercontent.com (raw.githubusercontent.com)... <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, ...
Connecting to raw.githubusercontent.com (raw.githubusercontent.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 3107 (3.0K) [text/plain]
Saving to: ‘STDOUT’

     0K ...                                                   100% 39.9M=0s

2021-09-13 21:59:35 (39.9 MB/s) - written to stdout [3107/3107]

We encountered a problem with your Docker setup:
  > invalid DOCKER_HOST=unix:///run/user/<REDACTED>/docker.sock, must be unset or unix:///var/run/docker.sock

Please check https://github.com/codacy/codacy-analysis-cli for alternative instructions.

Error: Process completed with exit code 3.

It looks like the codacy-analysis-cli.sh is explicitly checking the value of DOCKER_HOST.

Hi  @mickelglobbins can you share a bit about your use case, and why you need it to be different? We do have some restrictions that we require to be set as unix:///var/run/docker.sock.
Share whatever information you can on your use case, so I can bring it to the team, to consider adding it to our backlog, as it may not be a straightforward change.


Spell: Grim Tendrils missing persistent damage

In packs/spells/grim-tendrils.json:
system.description.value should be changed from

Tendrils of darkness curl out from your fingertips and race through the air. You deal 2d4 void damage and @Damage[(@item.level)[bleed]] damage to living creatures in the line. Each living creature in the line must attempt a Fortitude save.
\n
\n
Critical Success The creature is unaffected.
\n
Success The creature takes half the void damage and no persistent bleed damage.
\n
Failure The creature takes full damage.
\n
Critical Failure The creature takes double void damage and double persistent bleed damage.
\n
\n
Heightened (+1) The void damage increases by 2d4, and the persistent bleed damage increases by 1.

to

Tendrils of darkness curl out from your fingertips and race through the air. You deal 2d4 void damage and @Damage[(@item.level)[persistent,bleed]] damage to living creatures in the line. Each living creature in the line must attempt a Fortitude save.
\n
\n
Critical Success The creature is unaffected.
\n
Success The creature takes half the void damage and no persistent bleed damage.
\n
Failure The creature takes full damage.
\n
Critical Failure The creature takes double void damage and double persistent bleed damage.
\n
\n
Heightened (+1) The void damage increases by 2d4, and the persistent bleed damage increases by 1.

In Player Core p. 334, Grid Tendrils deals 1 persistent bleed damage.
Apologies, I haven't done a MR for this repo before. Let me know if that would be helpful for such a minor change.

Bleed damage is persistent by definition, so it's not necessary to add it to the inline damage roll. Give it a try and roll it, it will be persistent already.


Feat: Distributing Auth Token

What is this PR?
This PR adds a new recipe that explains the easy distribution of an auth token to an api to allow for authenticated requests. This comes on the heels of this community slack thread and this pattern being used on at least 2 IR client projects.
This is my first cookbook PR and recipe so I'd love any and all feedback!

@ecroce going to take a look at this!


add storage.py & test

to run the sample:
python storage.py

I don't think this is a solution. You need to modify existing code to
make it work with the gs file system. Did you read the hints I provided?
On Fri, 4 Nov 2016 at 6:37 PM, sunxiaobiu<EMAIL_ADDRESS>wrote:

to run the sample:
python storage.py
You can view, comment on, or merge this pull request online at:
https://github.com/hunkim/word-rnn-tensorflow/pull/23
Commit Summary

add storage.py & test

File Changes

A storage.py
https://github.com/hunkim/word-rnn-tensorflow/pull/23/files#diff-0
(100)
A storage_test.py
https://github.com/hunkim/word-rnn-tensorflow/pull/23/files#diff-1
(11)

Patch Links:

https://github.com/hunkim/word-rnn-tensorflow/pull/23.patch
https://github.com/hunkim/word-rnn-tensorflow/pull/23.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/hunkim/word-rnn-tensorflow/pull/23, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AA3DV2LAdu5SbF1iMqpwT2t4Lku7AEgbks5q6wrbgaJpZM4KpZR_
.

According to the hints you mentioned before, I have commit the code that change native python IO to tensorflow.python.lib.io.

Did you test this code? I am not sure this is even working.

I am so sorry that I haven't test the code because in mainland China, my credit card can not successfully register the google cloud platform account. So I haven't got a avaliable static bucket name to use.


Run test options: Polish APIs

Testing https://github.com/microsoft/vscode/issues/74436
Create a TS extension as described in #74436 and look at runTest.ts. It invokes runTests from vscode-tests .
https://github.com/microsoft/vscode-test/blob/master/lib/runTest.ts
With vscode-tests becoming more prominent, its APIs should have same quality as our vscode extension APIs. Here are some areas we need to improve before releasing to the public:

vscodeExecutablePath: I assume this is the path to the VSCode Electron exe, not the cli command. An usage example would be helpful.

TestOptions

extensionPath: If I understand right, this maps to 'extensionDevelopmentPath'. Since last milestone we now also support defining multiple extension dev paths. So maybe this should be <string | string[]>
testRunnerPath: See #74555
testWorkspace: Is this only for .code-workspace files, or any kind of input (folder, folders, files)? Is it a path or a URI? How do I specify an empty window? Given the many possibilities for inputs I suggest to remove that property in favour of letting the user define these in launchArgs
additionalLaunchArgs. Is the example listed in the spec useful? What happens if you define that list but also define extensionPath and testWorkspace ?

ExplicitTestOptions
are much cleaner. Do we really want to provide both TestOptions & ExplicitTestOptions?
It avoids some of the problems listed above, I'd suggest to remove TestOptions and just have ExplicitTestOptions.
For convenience we can always add factory methods that create a ExplicitTestOptions. That's IMO the better approach than the TestOptions interface.
runTests
Needs specs. What is the number that is returned?

Files/Folders/path-to-workspace-files. The first argument passed to code executable. I can improve the doc a bit.
Workspace is not a good name for that. Maybe pathsToOpen. Can be one, multiple, none. Can be paths, URI (--file-uri...). Give the many variants (and who knows what's in the funture), I think it's better the user just uses launchArgs for that.

ExplicitTestOptions and TestOptions are not clear to me, ExplicitTestOptions is a subset of TestOptions and the doc doesn't tell what's the major difference between the two.
For most people, IMO they should only write mandatory options and then testing can work as expected. Optional options are advanced users, they need to understand TestOptions and tweak as wanted. Right now there are actually three tiers,

Users who previously didn't need to customize anything, now they need to specify extensionPath
Users who wants to add additional launch args, change workspace, etc
Users who wants to take control of all options and use ExplicitTestOptions

It would be great if we can simply them. If it's not practical, at least we document better.

The initial idea of splitting them was to make things as simple as possible for the 90%+ of users. We generate this in the yeoman generator don't we, maybe that's simple enough already if we replace TestOptions with ExplicitTestOptions?

Created branch https://github.com/microsoft/vscode-generator-code/tree/vscode-test-and-types-at-code with all the commits.
Set master back to the previous version to avoid that the changes get pushed inadvertently.


Review Ticket Guide

Review Ticket Guide
Each assignment we will be required to make a review ticket. Most assignments provided will have a "Hacks" section. A Hack is a term for coding, a fun term

Screen Captures are nice for Hacks
Short Description
Code fragments

This is emphasis ``` Markdown
Explore the Computer Science Pathway at Del Norte High School and invest in your technical skills. All Del Norte CompSci classes are designed to provide a real-world development experience. Class time includes tech talks (lectures), peer collaboration, communication with teachers, critical thinking while coding, and creativity in projects. Grading is focused on time invested, participation with peers, and engagement in learning. - Introduction to concepts and requirements by the teacher
Project-based learning with teacher support, performing Agile/Scrum development
Coding, frontend, backend, devops, version control and algorithmic thinking
Creativity, research, design, data structures, and utilizing ChatGPT
Performing team work, team communication and collaboration, peer reviews/grading
Focus on tehnical communications through project presentations and student led teaching
Grades are on projects, learnt concepts, and live reviews between student(s) and teacher
Classroom work time is 3-4 hours per week. Homework expectations are approximately 2-3 hours per week. Homework is scheduled over a Sprint, approximately 2-4 weeks. Time lost is extremely hard to make up as all materials are cummulative.
Computer Science and Software Engineering (CSSE) 1,2; Grades 9-12
CSSE 1,2 prepares students for the AP Computer Science pathway. This course focuses on teaching the JavaScript programming language, object-oriented programming and inheritance, and developing algorithmic thinking skills. Through game development projects, students will engage in engineering skills, learn fundamentals of programming, work with data structures, and foster collaboration skills with their peers. Tech talks will be conducted by teachers to introduce concepts, provide guidance on tools, and support ideas to establish development requirements. By performing development and exploration, this course aims to raise students' awareness of the tremendous capabilities of computers and


Outlook 2013 doesn't need a re-enable of Edit Mode when reattaching, before editing text

Best guess is that OL 2013 automatically enables Edit Mode when the file being reattached is added to the list of attachments.
Regardless, the .CommandBars.ExecuteMso("EditMessage") invocation appears to be universally not needed for OL 2013, so a test needs to be added to skip this .ExecuteMso if Left(Application.Version, 2) = "15"

Fixed at f5baed5.
A more general solution might just be to ignore any exceptions raised (or, perhaps, the specific RTE thrown when the Inspector is already in edit mode) on the CommandBars.ExecuteMso("EditMessage") call, since the only anticipated exception condition is when it's already in edit mode, in which case no action needs to be taken.


Update public api documentation

Refer to this page for effective dart documentation.

Hello @fa-fifi,
I'm Michael and I would like to work on this issue.
Can you clarify more about what you mean by updating the public API documentation?

@DeleMike Just put comments for every publicly available methods explaining their purpose. No need to be too long. Just simple and concise, straight to the point. Make sure the grammar is correct and no typos. I already did some of them, and I don't expect you to do all of them. Just do what you can and I will double check them later. Please refer to this link on how to do documentation properly. Thanks btw. 😄

Hi @fa-fifi,
Many thanks for your update. I discovered that most of the Public APIs are already documented. I just added docstring to where I believe there should be comments. What do you think?
Also, I hope I am not missing anything, am I?


Unexpected behaviour in reftran.exe MeasurementIDs removed

Description
When refran is used to produce xml files. It removes the MeasurementID from each measurement.
In the following commands I have imported a station and measure file. The measure file contains measurement IDs for each measurement. However the xml file produced by reftran with the option --export-xml does not have these Measurement IDs.
import -n 20220901_(02)_WA_RENAME 20220901_(02)_WA_RENAMEstn.xml 20220901_(02)_WA_RENAMEmsr.xml
reftran 20220901_(02)_WA_RENAME -r GDA2020 --export-xml
Screenshots

Operating system

Windows

Dynadjust version

tested with Dynadjust 1.2.5 & 1.2.6

Thanks @KentWheeler.  A fix for this will be incorporated in the next release.


typo: Double word "like"

@nschonni : Thanks for your contribution! The author, @david-stanford, has been notified to review your proposed change.

@nschonni  Thanks for the fix.  #sign-off

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


cpu load requires core detection

In my case, the script detects "0" cores. Hence it can't calculate load_avg.
As a second best option I propose:
[ $numberOfCores = "0" ] && numberOfCores=1
It's certaintly not 0, let's assume 1.
My cpuinfo
Processor       : ARMv7 Processor rev 2 (v7l)
BogoMIPS        : 99.26
Features        : swp half thumb fastmult vfp edsp thumbee neon vfpv3
CPU implementer : 0x41
CPU architecture: 7
CPU variant     : 0x2
CPU part        : 0xc08
CPU revision    : 2

Hardware        : herring
Revision        : 0034

That sounds very reasonable to me. I've added the following: https://github.com/afaqurk/linux-dash/blob/master/server/modules/shell_files/load_avg.sh#L4-L6


breakpoint would not excute

more details？

@kuyu132 Don't wait for the process to finish. Go on. It will working.


How long does it take to update to minecraft 15.0

I just want an estimate time on how long it'll take for blocklauncher pro

We don't know yet, sorry

They have been TRYING TO update for 2 gd weeks

Plz, close this report cause, were spending pore time dealing with reports than porting BL!
Thx!

It takse lonk -_- since microsoft has remake some methods in way they think is "better"* IT SUCKS so zhuowei needs to re hook a lot of thinks..but dont worry it will be done soon

Long*


run error

diff: /../Podfile.lock: No such file or directory
diff: /Manifest.lock: No such file or directory
error: The sandbox is not in sync with the Podfile.lock. Run 'pod install' or update your CocoaPods installation.

pod install

pod install／update都没用

我也发现了这个问题。pod uodate没有用

更新之后使用DFTimelineView.xcworkspace进入工程


Automatically check if README.md examples are working when running "cargo test"

Since rustdoc nightly now provides "cfg(test)" when running on test mode, we can now use this macro on test mode only to check if README.md examples are working as expected.

It failed at this step:
Install g++-multilib

Doesn't seem related to my changes. :)

Thanks for the PR, but I would personally prefer to keep it manually up-to-date and would prefer to avoid taking on a new dev-dependency.


First time user:  Application crashes on "SELECT * from commits" on a repo with no commits yet

Describe the bug
Running select * from commits on an empty repo with no commits throws exception.
Note: Installed it for the very first time.
Installation:  scoop install gitql
To Reproduce
Steps to reproduce the behavior:

Create a folder
Do git init
gitql
select * from commits

Expected behavior
I am not sure what should be the expected behavior since I am using it for the first time. Generally speaking I would expect it to show 0 commits or at least don't crash!
Screenshots

GQL (please complete the following information):
GitQL version 0.10.0
Additional context
OS: Windows

Hello @f0restOfHimalayas,
Thank you for reporting, i have fixed it now and will be released on package managers next week.
Thank you,
Amr Hesham

Thank you @AmrDeveloper


test: Use built-in mock with cast functions instead of manual casting.

This is a cherry-pick from master for compatibility with GCC5.
Signed-off-by: Philip Tricca<EMAIL_ADDRESS>

review for this happened on the master branch


Feature request: add changelog to annotated tag message

Hi,
Annotated tags (the kind dune-release uses) have messages attached to them (like commits).
At the moment the string "Distribution x.y.z" is used (can be overridden on the commandline).
I'm thinking it could contain the changelog section corresponding to this release (the one that's used in the github release and opam PR).
How does that sound?

I like the idea that one could run git show <tag> and see the changelog without having to browse github, can be helpful when working with a local copy!

(this piece of information is displayed by git show $tagname)


add longtaplistener

Looks good but may I suggest renaming it to OnLongPressListener?
It's just that a tap is per definition short and it stays in line with system naming.


add "button prompts galore"

Button Prompts Galore
https://github.com/ZeusOfTheCrows/sdh-controller-prompts: changes the xbox button prompts to sony or nintendo styled icons (colour or grey options)
Checklist
Failure to complete this checklist or deleting boxes from the checklist will result in the closing of your pull request unless this is a theme update. Please write any comments regarding this checklist at the bottom of your pull request.
Check every box.

[x] I am the original author of this theme or have permission from the original author to make this pull request.
[x] All copyright of this theme's contents belong to the listed author or is cited in the repository linked above.
[x] This theme's target has been marked appropriately and only styles said target.
[x] This theme works properly on the latest versions of SteamOS for Steam Deck, decky-loader and SDH-CssLoader.
[x] This theme only uses * or !important if absolutely necessary.
[x] This theme is under 4MB in size and uses the least disk space possible.
[x] This theme's preview image does not include text unless it is necessary to describe changes that can be made.
[x] This theme is safe for work and does not contain any sexual, drug-related, or profane content.
[x] This theme prefixes any CSS variables with a unique identifier.

Check one box.

[x] I am not bundling a part of another theme with this theme to encourage mixing and matching themes.
[ ] Themes included with this theme are toggleable using a patch.

Check one box.

[ ] This is a keyboard theme applied to the default keyboard.
[ ] This is a system-wide theme applied to the default keyboard. The keyboard is toggleable using a patch.
[x] This theme does not target the keyboard.

i just want to check how release.json works: can i use .gitignore syntax? i.e. /folder/* to exclude the entire contents of a folder? and readme.md to ignore that filename in all folders, regardless of nesting?
(if i cannot, don't accept this but tell me and i'll write it out by hand)

[x] This theme's target has been marked appropriately and only styles said target.

i believe so? i was torn between tweak & system-wide

[x] This theme's preview image does not include text unless it is necessary to describe changes that can be made.

it does not, but why?

thank you for your work on css loader

Hi, would you mind contacting me on discord? I can more easily explain how stuff works. Release.json is not a glorified .gitignore, it just unlinks all paths in it. (Why do you need to ignore folders in the first place? You can add subpaths to navigate to a folder in the repository, before checking the theme )

alright i am doing this too late and making stupid mistakes. i will see if i am smarter in the morning

sorry for wasting your time yesterday - it shows that no matter how simple something appears, i should never do it late at night


Performance degrades in such pattern when using swagger API for rest document

Hi All,
We have swagger enabled in our Spring Boot Microservice project. It really works great. During performance testing we have identified one of request at every 30 seconds takes 8 times higher then the actual time. For instance our API takes usally 30 to 50 ms (milli seconds) to process the request, but the request comes at 30th second will take around 450 ms. We are using routing and load balancer components like Zuul, Eureka, Ribbon and thought some of these components might impact the performance, after removing all these 3 components we still had that performance issue.
We never had a thought about Swagger, but at last we removed the Swagger completely and gained all the performance back, like all request takes only 30 to 50 ms only. We don't know what Swagger is trying to do at every 30th second, even don't have any idea how disable that or atleast extend the 30th second to some other number.
We understand that 450 ms at 30th second costs less only, but we are having millions of requests everyday so we have to fix that.
So did anyone had this problem before or know anything about this issue, please let me know.
Thanks for your time to read this.

Difficult to say, there should be nothing in swagger-core v1 or v2 to cause this behaviour; which project/maven artifact and version are you using? (e.g. io.swagger:swagger-jaxrs:1.5.21 , io.swagger.core.v3:swagger-jaxrs2:2.0.6), and how is that configured?

Ok, so you are Springfox which is a different project, please post this issue in related project

Thanks for the right direction.


Add a "none" log level to disable logging completely on a particular logger

Sometimes I want to disable a particular logger completely via configuration, how about adding a NoneLevel constant that maps to -1?
Use case is that my app logs to various targets via different loggers and users may want to completely disable a particular target.

I'm happy to put up a PR if you like the idea.

Wont hurt to have such option IMHO

@fasaxc it it still matters to you, a PR would be appreciated. We need to be careful about the Fatal and Panic call which imo should still panic and exit when the logger tracelevel is set to this NoneLevel.
There was another issue requested for the same feature but with a MuteLeve, I'm not sure which name is the best though.

D'oh, looks like Level is defined as a uint32 so we can't use -1.  Making NoneLevel == 0 and moving PanicLevel to 1 could work but it would be a breaking change :-/
Maybe this desire could be handled with a separate Logger.SetEnabled(bool) method?

We can't change the value of the level in a v1 release/branch, it can have too many unexpected behaviour. A boolean would be backward compatible indeed, if the default value can keep the current behaviour.

Its not perfect, but if I run logrus.SetLevel(50) it silences the log. I'm sure it probably unintentionally silences everything i.e. fatal errors but that's the tradeoff.

Closing per the project being put into maintenance mode


Flag batch changes requiring manual review by zone name

Changes in this pull request:

Allow flagging of batch changes that require manual review by zone name match
Update tests
Update docs

can you add a test for this in the approve_batch_change_test


Datetime and number format incorrectly falls back when not defined

Reporting a bug?
When there is no dateTimeFormats or numberFormats for a given language, the translate methodss use the fallback locale to format the given input.
Expected behavior
The d(new Date(), 'formatName') and n(1234567.89, 'formatName') calls should respect the current locale value and only use of the dateTimeFormat / numberFormat option from the fallback.
Reproduction
StackBlitz playground: vue-i18n-dn-test

Initialise Vue I18n with two languages but define a date time and number format only for the fallback:const i18n = createI18n({
  locale: 'hu-HU',
  fallbackLocale: 'en-GB',
  messages: {
    'en-GB': { /* ... */ },
    'hu-HU': { /* ... */ },
  },

  datetimeFormats: {
    'en-GB': {
      shortDate: { dateStyle: 'short' },
    },
    'hu-HU': {
      // empty
    },
  },

  numberFormats: {
    'en-GB': {
      forintCurrency: {
        currency: 'HUF',
        currencyDisplay: 'narrowSymbol',
        style: 'currency',
      },
    },
    'hu-HU': {
      // empty
    },
  },
});

full source: main.ts
Set the locale to the one without the defined formats
Use d() or n() with the format defined in the fallback locale:

<template>

dateTest: {{ dateTest }}

numberTest: {{ numberTest }}

</template>

full source: App.vue

Alternatively, use <i18n-d> and <i18n-n> and do the same:<template>
  <i18n-d format="shortDate" :value="new Date()" />
  <i18n-n format="forintCurrency" :value="1234567.89" />
</template>

Actual behaviour

The date is displayed as 30/11/2024 instead of 2024. 11. 30.
The number is displayed as Ft 1,234,567.89 instead of 1 234 567,89 Ft

System Info
System:
  OS: Linux 5.15 Ubuntu 22.04.5 LTS 22.04.5 LTS (Jammy Jellyfish)
  CPU: (12) x64 Intel(R) Core(TM) i5-10600 CPU @ 3.30GHz
  Memory: 56.79 GB / 62.62 GB
  Container: Yes
  Shell: 5.8.1 - /usr/bin/zsh
Binaries:
  Node: 20.15.1 - /run/user/1000/fnm_multishells/2890037_1732975662492/bin/node
  pnpm: 9.5.0 - /run/user/1000/fnm_multishells/2890037_1732975662492/bin/pnpm
npmPackages:
  @intlify/core-base: ^10.0.4 => 10.0.4
  @vitejs/plugin-vue: ^5.1.3 => 5.1.3
  @vue/eslint-config-typescript: ^13.0.0 => 13.0.0
  @vue/test-utils: ^2.4.6 => 2.4.6
  @vue/tsconfig: ~0.5.1 => 0.5.1
  vite: ^5.4.6 => 5.4.6
  vite-tsconfig-paths: ^4.3.2 => 4.3.2
  vitest: ^2.1.1 => 2.1.1
  vue: ^3.5.6 => 3.5.6
  vue-i18n: ^10.0.4 => 10.0.4
  vue-tsc: ^2.1.6 => 2.1.6

Screenshot

Additional context

I opted to declare en-GB instead of en to avoid ambiguity around the date part order.
Added "Intl test" lines for the playground to show how the date and number are properly formatted with hu-HU locale. These tests use the locale set in vue-i18n but call the Intl methods directly.

Validations

[X] Read the Contributing Guidelines
[X] Read the Documentation
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussions

Thank you for your feedback!
I've just see the your reproduction.
I have checked at your App.vue, but there are no options specified for useI18n. Therefore, on App.vue, the locale operates in the global scope. In other words, it is hu-HU.
(about locale scope, you can see in there: https://vue-i18n.intlify.dev/guide/essentials/scope.html )
Since hu-HU is empty, en-GB is used for dateTimeFormats and numberFormats.
So , This report behaves as expected for vue-i18n
Thank you!

I have checked at your App.vue, but there are no options specified for useI18n. Therefore, on App.vue, the locale operates in the global scope. In other words, it is hu-HU.

that's exactly what I'd expect: the locale is set to Hungarian and therefore the labels are shown in that language too!
I see now where my expectation and Vue I18n's behaviour differs: while the package handles the situation by using the fallback locale's definition and the locale itself, I expected to only use the format option but keep the current locale 🤔
I could reason why my thinking makes sense, but I also see the implemented approach has benefits, just wasn't necessary obvious when I was reading the documentation!
If I'm mistaken the this forces me to define all datetime and number formats for every language, or to create a mechanism that assigns the missing definitions to every locale when assigning the i18nOptions. I don't think it's a big deal, but certainly creates friction when the format options don't deviate too much.
In any case, it's good to know the behaviour is intended and no further action is needed, thank you for the explanation Kazuya!


chore(carbon): refactor layout to match current dashboard

Summary

This PR refactors the side navigation and the general layout to match the current greenhouse dashboard.
Changes Made

Layout refactored
Side navigation in new dashboard does not flicker

Related Issues

https://github.com/orgs/cloudoperators/projects/4/views/9?pane=issue&itemId=84503372&issue=cloudoperators|juno|552

Screenshots (if applicable)

Layout
Before:

After:

Side Navigation
Greenhouse  Carbon
Testing Instructions

cd apps/carbon
npm i
npm run dev

Checklist

[x] I have performed a self-review of my code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have added tests that prove my fix is effective or that my feature works.
[ ] New and existing unit tests pass locally with my changes.
[ ] I have made corresponding changes to the documentation (if applicable).
[x] My changes generate no new warnings or errors.

It looks like this PR breaks the tests for Greenhouse. We should investigate and fix the issue.


Cluster is not accessible from new UI when the agents are in a bad state; Cluster is accessible in Ember UI in this case

Rancher Server Setup

Rancher version: master-head latest 3499046

Describe the bug

To Reproduce

Currently when the cluster is Unavailable, is when in Ember UI, the projects/workloads are not accessible.
For instance When the cluster(RKE1) has come up, but the worker nodes are in provisioning state and agents and other system workloads are deploying, from Ember this cluster is still accessible.
But from Dashboard now, the cluster has to come up Active completely for us to access the pods.

Result

The cluster should be accessible from the new UI also.
Important usecase, is sometimes when things are bad on the cluster, the cluster is still accessible in Ember UI, user can go to System project and look at the pods/workloads.
But now, from new UI this wont be possible. Only option would be to use kubectl commands.

In RKE1, when the cluster is in error state, I am unable to access the system project.
The new UI requires to have the kube-apiserver up and running in order to select the cluster and see al the k8s resources running on it.
In the old UI, you would be able to see the state of the "Nodes", but with v2.6, we've differentiated between machines and kube nodes. The kube nodes are the nodes that you can find with "kubectl get nodes" and that is found if you click on the cluster from the main drop down. The machines which are not dependent on the downstream cluster API to be up can be found through the cluster management -> cluster view.


More verbose run_all_tests testcase

and  test-lib: move image size info to final show results

[test-all]

[test-all]


course statistics: the last term missing

but eg. ohtuprojekti has also later attainments

Checked the frontend logic and it seems correct. I think the problem is that max_attainment_date is wrong in the database.

Checked the frontend logic and it seems correct. I think the problem is that max_attainment_date is wrong in the database.


docs: Add a README and reference the OpenAPI document

cc @slp

If we're moving reference-kbs here, it may be worth waiting a bit to avoid having to amend the link. Perhaps is also worth linking kbs-types?

@jialez0 Fixed and removed the reference-kbs link for now.


feat: add github actions for deploying eos mainnet and eos testnet in…

🗃 Github Issue #1889
Implements the github actions for deploying production ( on MainNet ) and development ( on TestNet ) instances of DHO on EOS.
[x] I have tested all my changes.

@arsenijesavic I wouldn't merge it until we polish out everything on EOS and clean up the initial load for Telos. How initial data is loaded is not the same for EOS and Telos.


Could this feasibly support uploading images?

Use case would be to speed up an image import / note taking process like this one
iOS shortcuts could feasibly use this to make a whole image based capture workflow really efficient:

(ios-shortcut) capture the most recent image
(ios-shortcut) get text from that image
(ios-shortcut) base64 encode the image
upload that image to the vault with obsidian://advanced-uri?vault=<your-vault>&filepath=my-file.png&data=${base_64}
(ios-shortcut) create a markdown snippet like ![TEXT_FROM_IMAGE.slice(0,32)](PATH_TO_IMAGE)
append that snippet to a daily note with obsidian://advanced-uri?vault=<your-vault>&filepath=my-file.md&data=${TEXT}&mode=append

I have huge problems testing this, because the base64 data of a test image is 800000 character long. I don't even know if Obsidian can handle such long uri.

It's kind of a crazy idea, I grant you. I have been using it though -- ios shortcuts let you resize an image, and also convert it to base64 data, so it works just fine to embed that into a note if the size is <300px (think: thumbnails, not high-res).
Limit on url length is pretty long these days, the bigger practical impact is that edit view of your notes is a mess if there's too much base64 in there.
Would be much better if there were a way to put an image into the vault and reference that local vault image "url", but I haven't gotten there yet.


Neon.Service logging broken

All of the neon kube services are crashing because the Service.Logger is null.

Yeah, I hadn't integrated the logging changes into NeonService yet.  That is done now.
We have a couple more pending issues:

[ ] DiagnosticPools is not looking like a great idea.  We're currently assuming that things like pool attributes can be returned back to the pool after calling ILogger.Log() but we really need to wait until all processors have completed processing the log.  The internal MSFT OpenTelemetry code does this with reference counts, but we don't have access to any of that.
We may need to ditch pooling.  It also might be worth requesting a feature where loggers could configure a callback where logging extensions like us could return any assets.

[ ] It would be nice to disable the message ASPNETCORE writes to the console when it starts out to avoid polluting the logs.  This can be accomplished setting the environment variable: ASPNETCORE_SUPPRESSSTATUSMESSAGES=true.
An alternative is to create a custom IHostApplicationLifetime.  The nice thing about this is that we could associate this with our ProcessTerminator to signal ASP.NET to shutdown gracefully.  This is probably the best approach.

[ ] @marcusbooyah saw some JSON logs that we're split along two lines of text.  I'm not sure how this can happen.

Assigning to @marcusbooyah

Closing this, we'll track the remaining issue at #1675


[queue] getQueueClient QueueNotFound error detail doesn't align

Which service(blob, file, queue, table) does this issue concern?
queue
Which version of the Azurite was used?
3.3.0-preview
Where do you get Azurite? (npm, DockerHub, NuGet, Visual Studio Code Extension)
npm
What's the Node.js version?
v12.13.0
What problem was encountered?
Unit test failure:
  QueueServiceClient
    × can create and delete a queue
      HeadlessChrome 79.0.3945 (Windows 10.0.0)
    AssertionError: Error does not contain details property

    // deletes the queue
    await queueServiceClient.deleteQueue(queueName);

    let err;
    try {
      await queueServiceClient.getQueueClient(queueName).getProperties();
    } catch (error) {
      err = error;
    }
    assert.equal(err.details.errorCode, "QueueNotFound", "Error does not contain details property");
    assert.ok(err.message.includes("QueueNotFound"), "Error doesn't say `QueueNotFound`");
  });

Debug
Azurite puts QueueNotFound in err.details.Code.

Latest dev branch doesn't have this issue.

√ can create and delete a queue (60ms)


Feature_Request: ENABLE GITHUB ACTIONS to Greet Users and Comment on PR

Is your feature request related to a problem? Please describe.
No.
Are you contributing under any open-source program ?
I am a MENTOR in GSSOC'24 raising this issue for contributors.

I am calling all the Contributors to Check out this ISSUE & comment first to get this issue Assigned to you.
The issue will be assigned on First come first serve basis.

@apu52 can I work on this?

@apu52 can I work on this?

Yes you can. Thanks for Commenting first . You are assigned. You can start working


Xamarin.Forms batch updates

Hi, I accidentally found an api in Xamarin Forms which is probably meant to process batch updates on views. BatchBegin and BatchCommit. Maybe this methods should be used when updating views? I am not completely positive that this methods are meant for this, but people on the internet are using them in this way.
https://docs.microsoft.com/en-gb/dotnet/api/xamarin.forms.visualelement.batchbegin?view=xamarin-forms
Probably this can give some speed up on updates.

Hi. Thanks for the pointer.
I took a look at those BatchBegin/BatchCommitted methods.
From the sources, It appears to trigger a BatchCommitted event.
https://github.com/xamarin/Xamarin.Forms/blob/d99264507dc1623aecb83daf81aadd2effae62ae/Xamarin.Forms.Core/VisualElement.cs#L593-L594
This event is listened on the platforms and treated as a redraw.
It seems to force a relayout of the native control.
https://github.com/xamarin/Xamarin.Forms/blob/6d3c41fb2dc0911979cbddda4987e8f76ef7845f/Xamarin.Forms.Platform.iOS/VisualElementTracker.cs#L141
But it doesn't batch the properties changes, so I don't think it will really benefit Fabulous.
Will try to play with it in a C# XF app to see if it has any impact.


Updated docker build to use a cpu only version of torchvision

What this PR does / why we need it:
Made changes in grpc image to use cpu only version of torchvision.
The image size has been brought down from 3.7Gb to 1.5Gb
Transformer grpc image build and publish has been merged with same workflow as transformer to re-use the image layers.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #2514
Type of changes
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

Special notes for your reviewer:
Release note:

NONE

/lgtm
/approve


Update documentation examples for Rails 7.0 Support

Feature Description
It can be frustrating to find documentation that suits your use case, but end up with an example that continues to raise errors.
Let's take some time to verify the examples in the Ruby Agent's documentation work on Rails 7.0. All examples should be technically accurate and functional for Rails 7.0 to complete this story. If code changes will take time that exceeds the estimate, reach out to the team to separate changes out into additional issues.
Documentation to Verify

[x] Ruby Custom Metrics
[x] Ignoring Specific Transactions

[x]  Blocking all instrumentation - transactions blocked.
[x] Ignoring specific actions with Rails

[x] :only
[x] except

[x] Ignoring Apdex contributions
[x] Block browser instrumentation
[x] Ignoring transactions dynamically

[x] NewRelic::Agent.ignore_transaction
[x] NewRelic::Agent.ignore_apdex
[x] NewRelic::Agent.ignore_enduser

[x] Ignoring transactions by URL with configuration

[x] Sending Handled Errors
[x] Ruby Custom Instrumentation

[x] Tracing in class definitions
[x] Tracing initializers
[x] Tracing blocks of code
[x] Naming transactions

Additional context
Currently, there is a disclaimer listed at the top of articles with examples that have not been tested with Rails 7.0. This should be removed once examples are tested and, if necessary, updated.

Team discussion notes
Size: Small

Technical accuracy in this story
Wordsmithing can occur in other stories
Important to have this done so that we can be clear about what's supported in Rails 7
If things aren't working in Rails 7 and the time exceeds the estimate, it can go into a different ticket
May be able to wrap examples in the #executor blocks to resolve issues if they arise

Priority: High - important to have a published source of truth about custom instrumentation for our most popular framework
Helping support, ourselves, and our customers be their best advocates by having access to accurate information

@hannahramadan

Look at similar documentation
Review current documentation
Make a plan for what needs to change

Completed testing documentation examples with Rails7. No issues with any example and Rails7 compatibility.
Docs PR resolving Rails7 warnings: https://github.com/newrelic/docs-website/pull/11056#pullrequestreview-1252544730

Closing this issue! I think it remained open by accident.


Template parameter constraints based on other template parameter has issues

model Base {
  a: string;
}

model SubtypeOf<TBase, T extends TBase> {
   ...T;
}

model ExtensionFoo is SubtypeOf<Base, { a: string; }> {}; // <-- error unassignable Type '(anonymous model)' is not assignable to type 'TBase'

model SubtypeOfBase<TBase, T extends Base> { <-- works if I use the explicit type
   ...T;
}

model ExtensionBar is SubtypeOfBase<Base, { a: string }> {};

Playground

As talked offline I think this is a bug in our assignment checker, the following works great in typescript
interface Base {
  a: string;
}

interface SubtypeOf<TBase, T extends TBase> {
  t: T;
}

type ExtensionFoo = SubtypeOf<Base, { a: string }>;


fixed typo on req.txt sample

@midesa : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Fix detection of ROS build

CMAKE_PREFIX_PATH is not an environment variable, but CMake's
internal one that contains a list of items (strings). This
list needs to be concatenated into a single string in order
to become "match"-able against a regex.
Signed-off-by: Dmitry Rozhkov<EMAIL_ADDRESS>

@mdhorn could you please comment? I have no objections to this.

CMAKE_PREFIX_PATH is an environment variable if ROS setup.bash file is sourced, but this is a better solution that works for non-catkin_make builds even if the ROS setup.bash was sourced.
👍  Approved. Please merge.

Conflict resolved and commit merged into master, thank you


Jetson Voice Invalid sample rate

I am running jetson voice container follow https://ngc.nvidia.com/catalog/containers/nvidia:jetson-voice
I tried 2 difference micro devices (micro usb and micro built in usb camera) but they still do not work.
And My camera info at /proc/asound/card4/stream0
Capture:   Status: Stop   Interface 3     Altset 1     Format: S16_LE     Channels: 1     Endpoint: 4 IN (ASYNC)     Rates: 8000, 11025, 16000, 22050, 24000, 44100, 48000     Data packet interval: 1000 u
My microphone infor at /proc/asound/card2/stream0
`C-Media Electronics Inc. USB PnP Sound Device at usb-3610000.xhci-2.1, full spe : USB Audio
Capture:
Status: Stop
Interface 1
Altset 1
Format: S16_LE
Channels: 1
Endpoint: 2 IN (ADAPTIVE)
Rates: 48000, 44100`
Here is information of microphone devices inside jetson voice container :when I run  ./scripts/list_microphones.sh
Input Device ID 24 - USB PnP Sound Device: Audio (hw:2,0) (inputs=1) (sample_rate=44100) Input Device ID 26 - USB Camera: Audio (hw:4,0) (inputs=1) (sample_rate=44100)
When I run  python3 src/test_asr.py --mic 24
The usb microphone got error:
[Errno -9997] Invalid sample rate Traceback (most recent call last):   File "src/test_asr.py", line 66, in <module>     asr_engine.start(on_asr_transcript)   File "/workspace/src/asr/jarvis/asr_engine.py", line 59, in start     self.audio_generator, "mic", transcript_callback)   File "/workspace/src/asr/jarvis/speech_utils.py", line 875, in streaming_recognize     for response in responses:   File "/usr/local/lib/python3.6/dist-packages/grpc/_channel.py", line 416, in __next__     return self._next()   File "/usr/local/lib/python3.6/dist-packages/grpc/_channel.py", line 706, in _next     raise self
in case the camera: When I run  python3 src/test_asr.py --mic 26
The microphone build in camera auto turn off. But the code maybe works but because micro off so it doesn’t receive signal , by this reason the results is so wrong:
"Phrase: Fine. (1.000000)
Phrase: O e. (1.000000)
Partial: Ae (1.000000)
Partial: O a (1.000000)
Phrase: A e e. (1.000000)
Phrase: E o o. (1.000000)
"
Do you have any suggestion to reduce the sample rate of microphone? or any suggestion?
Thank you
Dat

Hi Dat, I don't have resampling built in, so you would need to use a microphone that supports 16KHz sampling rate.
BTW I have a different/newer version of this project at https://github.com/dusty-nv/jetson-voice
but it also currently requires 16KHz sample rate.
BTW what microphone are you using?   I would like to get one that doesn't support 16KHz so that I can implement/test the resampling.


Add docs explaining t method

Context

https://discord.com/channels/836637622432170028/836637623048601633/1033381727630012477
https://discord.com/channels/836637622432170028/836637623048601633/1033434262868861070

Details
We already had some docs about finding translation keys in the indirection docs, but it still seemed a little unclear as to how this was working under the hood, so I provided an example with the Memberships show page and explained how we're overriding the original I18n translation method.

@scottharvey
Tagging you in case you had any other thoughts

Nice one, that explains everything perfectly 👍


Build native images for the different OSes in the GH release actions?

Possibly:

Linux (x86 / arm)
Windows
OSX (x86 / arm)

arm is only possible on GH with external runners, macOS prohibitive expensive when one picks AWS. For Linux arm I have one.

The binaries attached to https://github.com/neo4j-contrib/sql2cypher/releases/tag/early-access are binaries based on jlink, running with a stripped down JVM on the module path. Cross compilation can be achieved here as the compilation only needs the source jvm for the target OS in contrast to GraalVM, that needs it running (and glibc and all of that fancy stuff I have no clue about)


Faq Scrollbar like home page and added favicon

Related Issue
issue no #794
Description
Add Scrollbar like home page also added Favicon to this page.
Type of PR

[x] Bug fix
[ ] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)

Checklist:

[x] I have performed a self-review of my code
[x] I have read and followed the Contribution Guidelines.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.

@Suchitra-Sahoo Please check it and merge !


RFC deposit: filter depositions based on user roles

Problem
Currently for all visitors of the index page of deposit /deposit all depositions are shown.
We would like to have some filtering done based on the roles of the current user so that different users can see different submissions.
Proposal
https://github.com/inveniosoftware/invenio/blob/pu/invenio/modules/deposit/views/deposit.py#L113
ctx = dict(
        deposition_types=DepositionType.all(),
        my_depositions=Deposition.get_depositions(current_user),
        prefill_data=draft_cache.data,
    )

Instead of DepositionType.all() we can have a new function DepositionType.all_authorized(current_user) that only returns deposition types the user is allowed to see.
We will need a new attribute in the DepositionType model that represents the roles that are authorized to see such Deposition type (we probably don't need a config variable for this, if a new role needs to be added to a deposition type, it can be changed in the DepositionType definition)
Then for all views that access a certain deposition type, like:
@blueprint.route('/<depositions:deposition_type>')
@blueprint.route('/<depositions:deposition_type>/create',
                 methods=['POST', 'GET'])
@blueprint.route('/<depositions:deposition_type>/<int:uuid>/<draft_id>',
                 methods=['POST'])

There should be a protection so that, something like a 404 is returned if the user is not authorized. Should this be implemented as a decorator for those views?
What more should be taken into account? Suggestions for the implementation?
CC @lnielsen

What about using the native RBAC model of Invenio, hence instead of having Deposit connecteded to role we continue the current usage of the submit WebAccess action. It's probably not going to be as fast  as the proposed filter, but would be more consistent with the authorization model of Invenio.
Basically given the list of deposition types, you would display the deposition type X if the current user is authorized for the action deposit with deposition_type=X.

It would indeed be nice to connect this to the access module using parameterized authorizations.

As a sidenote from our workflow at join2 it would indeed be necessary to hide sumbissions in the first place. (All submissions are/have to be treated as visible for a given group only till further down the road they are approved to be public.) This would die if everyone could see all submissions. @kaplun our users are allowed to deposit every type, however, they are not allowed to see every submissions by others. (A submits a journal article to private collection a, B does so to b. A must not be allowed to see submissions from B.) If that's in your model I'd agree that consistency of the quite complex rules would be nice.

I think this is a duplicate of #1946?

Taking https://github.com/inveniosoftware/invenio/issues/1946#issuecomment-50313526 and https://github.com/inveniosoftware/invenio/issues/2724#issuecomment-72089962 into account the solution seems to be:

[ ] add acl.py with new usedeposit action
[ ] usedeposit can accept type argument
[ ] calculate precached_allowed_deposition_types when user visit /deposit
[ ] "decorate" rest of deposit urls with decorator that consistently checks the access right

WDYT?

That looks good to me. I will work on this end of next once I am back. Any more comments are welcome until then.


English version ?

Can we have it internationalized ?
English version would be awesome.

@RRadziejewski  For English version see: https://github.com/Randy8080/reference


cli: Fix secret cleanup when no infra ID is specified

Before this commit, running destroy cluster with no infra ID argument silently
failed to clean up CLI-created secrets because the code was using un-defaulted
input which in this case is an empty value.
This commit fixes the code to use the post-validation/defaulting infra ID value.

/lgtm


Linux install script is different from main site install procedure

The Linux installation script is installing .NET Core 2.1 instead of 2.2
The installation procedure described in the document is very different from the main .net web site at (e.g. https://dotnet.microsoft.com/download/linux-package-manager/ubuntu18-04/runtime-current). Search engines often return this document for the keywords "dotnet install linux", and may cause confusion, since it seems not to be the recommended way to install dotnet on Linux

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

ID: a2e484a9-5e0c-eb3e-1413-7b507c4fa07d
Version Independent ID: 5bc7caa5-f1c2-d7d5-fe02-80c65cca9672
Content: dotnet-install scripts - .NET Core CLI
Content Source: docs/core/tools/dotnet-install-script.md
Product: dotnet-core
GitHub Login: @mairaw
Microsoft Alias: mairaw

@mairaw Who is in charge of the script?

@natenho Hi! The script is intended for non-admin installs and automation scenarios. It says this in the first paragraphs of the article that you should go to the main install site. We can't really do anything about the search engine problem as the page title is accurate.
We can contact the script author to update the script to 2.2 though 😄

@johnbeisner is the owner AFAIK

@natenho we'd need a bit more information in order to investigate this issue properly.

Is the first issue still happening? If so, which command exactly are you using? For product issues like that, it's probably easier to file an issue at https://github.com/dotnet/cli/issues, where the product team reviews them.
I cannot repro the issue of seeing this page over the marketing site results on a search engine, but I could add a sentence to this topic pointing to the main downloads site. Would that help?

Yes it is, I've just checked, here is the output:

[root@centos-181 ~]# ./dotnet-install.sh
dotnet_install: Warning: Unable to locate libunwind. Probable prerequisite missing; install libunwind.
dotnet_install: Warning: Unable to locate libicu. Probable prerequisite missing; install libicu.
dotnet-install: Downloading link: https://dotnetcli.azureedge.net/dotnet/Sdk/2.1.701/dotnet-sdk-2.1.701-linux-x64.tar.gz

I could not reproduce the search engine behavior anymore. The first result is going to https://dotnet.microsoft.com/download/linux-package-manager/ubuntu16-04/sdk-current

I think the description in the document could emphasize that the "regular way" of installing dotnet on linux and link to the download website.
The current text of the description says:

We recommend that you use the stable version that is hosted on .NET Core main website. The direct paths to the scripts are:
[...]

IMHO, the recommendation is not very clear. Is it about the stable SDK or the stable installation scripts? At least I can't find those scripts in the main website, could you?
Thanks! :)

Yes, I think we can improve the text. But it's about the stable version of the installation scripts.
For issue #1, as I mentioned earlier, I suggest you file an issue at https://github.com/dotnet/cli, so the product team can review that.

Yes, I think we can improve the text. But it's about the stable version of the installation scripts.

So in this case it would be even better to put the actual link to the installation scripts download page. Where can we find it?

https://docs.microsoft.com/en-us/dotnet/core/tools/dotnet-install-script
You can find the installation scripts' URLs under the 'Description' section.


Kanban Board Setup

Link to GitHub Project
There are two people in our group.

Hi @TEAM , 👋
STATUS: Approved  🎉
Great work on making the changes requested by a previous reviewer :1st_place_medal:
Your project is complete!  There is nothing else to say other than... it's time to merge it
Congratulations! 🎉  🎉 🎉

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please ping me @moise10r when you comment so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Update README.md

Added introduction

Reference: https://www.codementor.io/olatundegaruba/password-reset-using-jwt-ag2pmlck0


[Bug] RELAYHOST_PASSWORD printed out

I assume this is a bug :)
‣ NOTE  Forwarding all emails to [mail.example.com]:587 using username<EMAIL_ADDRESS>and password (redacted).
[mail.example.com]:587 relay@example.com:MYPASSWORD
‣ INFO  Using default private network list for trusted networks.

Set via docker compose env variables:
      RELAYHOST: ${SMTP_RELAYHOST:?}
      RELAYHOST_USERNAME: ${SMTP_RELAYHOST_USERNAME:?}
      RELAYHOST_PASSWORD: ${SMTP_RELAYHOST_PASSWORD:?}

Not sure which line is responsible for the print of the password, but I assume somewhere here: https://github.com/bokysan/docker-postfix/blob/a1a2082ae1652fb5ea90c2ba82020676440b5685/scripts/common-run.sh#L201-L208
Maybe the grep should be redirected to /dev/null ?

Thank you for the report.
grep's missing -q.


Windows Spyder Shortcuts

From<EMAIL_ADDRESS>on 2013-02-08T13:33:34Z
Small polish issue on Windows. The shortcuts that start Spyder should call pythonw.exe, rather than python.exe. This way, we don't see the transient command shell that pops up between clicking the icon and the Spyder splash screen. http://stackoverflow.com/a/9706045
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=1260

From pierre.raybaut on 2013-02-10T04:54:41Z
I agree that seeing this flickering command window at startup may seem 'unprofessionnal'. However, note that this is necessary to avoid side effects of pythonw.exe implementation: this alternative executable without command window is not redirecting the standard input/output streams and that may be a problem in an IDE like Spyder which may be used to run programs in separate processes, hence loosing the I/O streams (no output in console).
See this post for more details: https://groups.google.com/group/spyderlib/tree/browse_frm/month/2010-12/df9532490ab0eef9?rnum=11&lnk=ol
Status: WontFix


 It simplifies expressions suggested by Copilot while working on another PR.

Rebased, will merge once 🟢.


log level changed from warn to debug

Changed the log level for whitespace update from warn to debug.
This solves #803

closing in favor of #806


Support for "ignoring a list of specific fields"

I was using this tool to generate our regression baselines, the result is a JSON string like:
{
"id": "4163B50C4DD9911804894F827078B218",
"instanceId": "020B24C54D891A89B44AAABDBCFCE057",
 "status": 1
}

The problem is that there are some fields such as the instanceId field that is using UUID that will always change, and I don't want this false alert.
One workaround would be:

We provide the ignores=["instanceId",] to the data_regression function, and the value of this field will be ignored when comparing the result.

Or, we can replace the value of specific field using special string, such as "{% ignore %}" or something else;

Hi @hdw868,
Couldn't you preprocess the data yourself before passing to the fixture? Btw, are you using data_regression or file_regression?

Hi @nicoddemus,
Yes, I could process the data before passing to the fixture, but usually, those are very common fileds such as "id", ''creationDate" and etc, I think it would much more convenient to provide a list of keys to be ignored.

Hi @hdw868,
While it might seem convenient to ignore just some top level keys, this doesn't cover nested structures:
{
  "orders": [
     {
       "id": "aksjsnj-10291",
     }
  ],
  "product" : {
    "id": "215",
  },
}

If a user wants to remove the id key from orders but not from product, they will need to do it themselves anyway.
For the simple use case, the proposal to ignore just a top-level key, the code:
del data["id"]
data_regression(data)

Becomes:
data_regression(data, ignore_key=["id"])

Which is not a big win IMHO.
So particularly I'm 👎 on adding this, but let's hear what others think.

We had the same issue using data_regression for SQLAlchemy models.
Below is a recipe for a new fixture based on data_regression that would ignore some fields of the data given to the check method. The recipe uses regex patterns, but I think you can simplify it for your own purpose. You can write this code into your project conftest.py, update the class member fields_to_ignore and use the db_model_regression.check instead.
@pytest.fixture()
def db_model_regression(data_regression):
    return DbModelRegression(data_regression)

class DbModelRegression:

    fields_to_ignore = ['updated_on', 'created_on', r'\w*_id', '^id$']

    def __init__(self, data_regression):
        self.data_regression = data_regression

    def check(self, data, **kwargs):
        return self.data_regression.check(self._prepare_for_regression(data), **kwargs)

    def _should_record(self, field_name):
        for pattern in self.fields_to_ignore:
            if re.match(pattern, field_name):
                return False
        else:
            return True

    def _prepare_for_regression(self, data):
        if isinstance(data, list):
            prepared_data = []
            for item in data:
                prepared_data.append(self._prepare_dict(item))
            return prepared_data
        elif isinstance(data, dict):
            return self._prepare_dict(data)
        else:
            return data

    def _prepare_dict(self, data):
        prepared_data = {}
        for key in filter(self._should_record, data):
            value = data[key]
            if isinstance(value, list):
                prepared_list = []
                for item in value:
                    prepared_list.append(self._prepare_for_regression(item))
                prepared_data[key] = prepared_list
            elif isinstance(value, dict):
                prepared_data[key] = self._prepare_for_regression(value)
            else:
                prepared_data[key] = value
        return prepared_data

I agree with you that we must find a more elegant way of address that. But I don't think that adding an ignores option is the answer. Maybe adding some highlevel fixtures that knows how to deal with objects instead of dicts. I'll try to put some thought on it.

Hi @hdw868,
While it might seem convenient to ignore just some top level keys, this doesn't cover nested structures:
{
  "orders": [
     {
       "id": "aksjsnj-10291",
     }
  ],
  "product" : {
    "id": "215",
  },
}

If a user wants to remove the id key from orders but not from product, they will need to do it themselves anyway.
For the simple use case, the proposal to ignore just a top-level key, the code:
del data["id"]
data_regression(data)

Becomes:
data_regression(data, ignore_key=["id"])

Which is not a big win IMHO.
So particularly I'm 👎 on adding this, but let's hear what others think.

Yes, provide a ignore option only may make things complicated anyway, however, the workaround provided by @igortg seems to have the same problem.  Just wondering if there is a more elegant way to handle this situation.

A more elegant way would be building your own serializer to convert objects to dicts without the unwanted fields. Pydantic has something like that, but you have to specify the attribute you want to ignore also in the nested objects.

Hmm, would adding exclude options like pydantic be useful? I may try to create a PR if it's worth trying.

As an idea, I think it would be nice to have a more generic solution that can handle not just ignoring specific fields, but also custom comparison behaviour.
For example, one may want to be able to

ignore the value of id but assert that id field is present
ignore the value but assert that it is a positive integer
assert that a float equals expected value with some tolerance
assert that a list contains expected items ignoring order
assert that a string matches a regex

One way to achive this could be by providing a way to override fields' values with any object that defines an __eq__ method. Something like
from unittest.mock import ANY
from pytest import approx

data = {
    "orders": [{
        "id": "aksjsnj-10291",
    }],
    "product" : {
        "id": "215",
    },
    "gravity": 9.81,
    # ...100500 other fields
}

data_regression.check(
    data,
    {'product': {'id': ANY}, 'gravity': approx(10, 0.1)}
)

One way to achive this could be by providing a way to override fields' values with any object that defines an eq method.

At first glance I like this idea, seems to include a nice general way to customize how things are compared.

I forked the library (https://github.com/adamzev/pytest-regressions) and implemented a solution that works for my use case. I focused just on data regression yml files. For lines you want to ignore you can add a # ignore comment on them in the yml file and it will ignore differences in those lines. So you don't have to manually enter what lines to ignore, I added a --force-ignore flag which will tag every line which is currently showing up as different (for me, running the tests in a different order produces different model ids which can be deeply nested keys or values) to ignore in the future.
If this would work for other people I'd be happy to look at what needs to be done to make this able to be brought in.

FWIW, here is a relevant discussion in a similar library https://github.com/syrusakbary/snapshottest/issues/21.  I think pointing out that jest does this with their "property matchers" is nice https://jestjs.io/docs/en/snapshot-testing#property-matchers.


Example code for int8 quantized network inference on arm compute library

Hello !
I want to run a int8 quantized tflite version on ACL latest version.
Is there any example available to use ACL interface using int8 quantized weights

Hi @abhajaswal
The best option to run models through ACL is by using ArmNN's ExecuteNetwork
You can download a tflite model from https://storage.googleapis.com/download.tensorflow.org/models/tflite_11_05_08/mobilenet_v2_1.0_224_quant.tgz and then run it as shown below:
root@acl_hikey_1:~/tmp/pabtel01/armnn# LD_LIBRARY_PATH=.:$LD_LIBRARY_PATH  ./ExecuteNetwork -f tflite-binary -m mobilenet_v2_1.0_224_quant.tflite -i input -o output -c CpuAcc
For more information about ArmNN please see https://github.com/ARM-software/armnn/tree/branches/armnn_21_11/tests/ExecuteNetwork
Hope this helps.


Is not working

Firefox 84.0.2 (64-bits)
Linux Mint 20.0 Cinnamon x64 X11
Page Translator Revised 2019.5.23 update 2020-01-22

When I click on the extension icon, it calls google translation, but does not translate the page

example:
https://www.linuxfoundation.org/en/about/
English to Brazilian Portuguese

Well, the extension is loading the translator site URL just fine, so I don't think it's a problem I can fix. When I try translating that page, it doesn't show most of the site with uBlock Origin enabled, but it translates fine with uBO disabled. So I'd suggest disabling uBO to see if that works. You could also try selecting Microsoft Translator in the preferences.

Well, the extension is loading the translator site URL just fine, so I don't think it's a problem I can fix. When I try translating that page, it doesn't show most of the site with uBlock Origin enabled, but it translates fine with uBO disabled. So I'd suggest disabling uBO to see if that works. You could also try selecting Microsoft Translator in the preferences.


Adding @babel/preset-react to docusaurus does not enable "Support for the experimental syntax 'jsx'"

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Prerequisites

[X] I'm using the latest version of Docusaurus.
[X] I have tried the npm run clear or yarn clear command.
[X] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[ ] I have tried creating a repro with https://new.docusaurus.io.
[X] I have read the console error message carefully (if applicable).

Description
I'm getting this error:
 ParseError: Support for the experimental syntax 'jsx' isn't currently enabled (62:23):

Add @babel/preset-react (https://github.com/babel/babel/tree/main/packages/babel-preset-react) to the 'presets' section of your Babel config to enable transformation.
If you want to leave it as-is, add @babel/plugin-syntax-jsx (https://github.com/babel/babel/tree/main/packages/babel-plugin-syntax-jsx) to the 'plugins' section to enable parsing.

when I try to execute a custom remark plugin that contains the following code:
import {visit} from 'unist-util-visit';
const plugin = (options) => {
  const transformer = async (ast) => {
    visit(ast, 'inlineCode', (node) => {
      if (mapping[node.value] !== undefined) {
        node.type = 'html';
        node.value = (

);
        console.log(node)
...

My babel.config.js:
module.exports = {
  presets: [
    require.resolve("@docusaurus/core/lib/babel/preset"),
    require.resolve("@babel/preset-env"),
    require.resolve("@babel/preset-react"),
  ],
};

Although I've followed the instructions found in the log messages (as of my current understanding), it didn't make the code run.
Reproducible demo
No response
Steps to reproduce

Create a remark plugin as above
Add the plugin to Docusaurus as described in the docs
Run yarn clear
Run yarn dev
Observe the behavior

Expected behavior
Code compiling and running.
Actual behavior
Code not compiling and not running.
Your environment

Docusaurus version used: 3.5.2

Self-service

[X] I'd be willing to fix this bug myself.

We don't transpile MDX components with Babel to build faster, because the output of MDX is already "runnable" by Node and browsers.
This is a tradeoff: you can't use fancy new features in MDX files, but this should be fine most of the time and you can extract your code to another component.
That being said, this limitation doesn't seem super related to your case here.
I can't help because I don't have a minimal repro. I don't see the full error message, the input source, the full remark plugin code, and can't investigate.
Please provide a minimal repro. Also, run your plugin in a standalone MDX playground to see what's the compiled output, and how it differs from what you expected to get.

Most likely it's a bug in your remark plugin, and you shouldn't use an element of type "html"
node.type = 'html';
node.value = (

);

That looks highlightly suspicious to me. Please explain what you expect this to do. And please run this outside of Docusaurus first to see if it does what you expect.

@slorber The use case I'm trying to realize here is this: dynamic conversion of inline-code elements (text within single backticks in markdown) into html markup -> the final html is supposed to output a popup element showing a description and more info on the inline-code.
The reason I'm trying to make this work automatically is to avoid cluttering the markdown with unnecessary component calls (we're talking about ~100 individual components I'd otherwise have to create and also call within markdown.). Hence the need to replace the inline-code elements with html markup. the html node type is documented here: https://github.com/syntax-tree/mdast?tab=readme-ov-file#html and seems to be the most suitable node type for me to use from my understanding.
Can you make use of this explanation? I'll more than happily provide a minimal repro as soon as I can to further explain myself.

Remember that MDX compiles to JSX. html is a valid MDAST / HAST type, but I'm not sure a type like html is supported by the MDX. A HTML string might not be well-formed and might not even be convertible to JSX.
What you probably want as output is

And for that the AST is:
    {
      "type": "mdxJsxFlowElement",
      "name": "div",
      "attributes": [
        {
          "type": "mdxJsxAttribute",
          "name": "dangerouslySetInnerHTML",
          "value": {
            "type": "mdxJsxAttributeValueExpression",
            "value": "{__html: \"
My HTML markup
\"}",
            "data": {
              "estree": {
                "type": "Program",
                "body": [
                  {
                    "type": "ExpressionStatement",
                    "expression": {
                      "type": "ObjectExpression",
                      "properties": [
                        {
                          "type": "Property",
                          "method": false,
                          "shorthand": false,
                          "computed": false,
                          "key": {
                            "type": "Identifier",
                            "name": "__html"
                          },
                          "value": {
                            "type": "Literal",
                            "value": "
My HTML markup
",
                            "raw": "\"
My HTML markup
\""
                          },
                          "kind": "init"
                        }
                      ]
                    }
                  }
                ],
                "sourceType": "module",
                "comments": []
              }
            }
          },
        }
      ]
    }

This is out of scope of Docusaurus, and more of a MDX help so I'm closing.
The best to troubleshoot this would be to run the compilation using your remark code outside of Docusaurus, until you get the JSX output you want. You should write unit tests.


Use PSR ContainerInterface instead of Interop

Q
A

Documentation
no

Bugfix
yes

BC Break
yes

New Feature
no

RFC
no

QA
no

Description
When using a PSR11 Container such as elie29/zend-di-config it is not possible to use the included delegator factory as it is hardcoded to use the Interop\Container\ContainerInterface, even though that interface has extended Psr\Container\ContainerInterface for some time now.
This PR swaps out for the Psr version thus allowing PSR11 containers to be configured.
Fixes #24

Note to myself: since the class was final and container-interop implements the psr interface, we do still accept interop interfaces and thus this should not be a BC break.
Have to double check this when I have some more time, but afaik we do conflict in laminas-cache with container-interop <1.2

No worries. I'll be honest, I was confused about the BC break but I wasn't going to argue with the maintainer ;)

Thanks, @cooperaj!


Column Selection by mouse should begin where the mouse click starts

In other editors like VS, i can hold down the Alt key, then left click the mouse at the place where I want to start the column selection, and begin dragging immediately to make the selection.
In VSC, I am required to place the cursor first then Shift+Alt, left click, drag.
This difference in starting the operation led me to thinking that either VSC was missing the feature despite documentation stating otherwise, or that the feature was bugged or otherwise unusable.

@StingyJack You can configure "editor.multiCursorModifier": "ctrlCmd" and then the alt key toggles between normal selection and column selection.
Regardless of the above setting, you can also use the middle mouse button to do column selection.

@alexandrudima -  I'm not talking about the difference between Alt vs Shift + Alt. I think i was pretty clear stating it was how the operation was started, not the hotkeys used.

In other editors like VS, i can hold down the Alt key, then left click the mouse at the place where I want to start the column selection, and begin dragging immediately to make the selection.

Use "editor.multiCursorModifier": "ctrlCmd"


Update cleanup.ps1 so it compares semantic (vs string) versions.

Type of Change

[x] Bug fix (non-breaking change, fixes # )
[ ] New feature (non-breaking change, adds functionality, fixes # )
[ ] Breaking change (effects multiple commands or functionality, fixes # )
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] If new file reference added for test, has is been added to github.com/sqlcollaborative/appveyor-lab ?
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose
With the patch section in the dbatools version number having just gotten over 100, the cleanup script is not removing versions prior to 100 since (for example) from a string perspective 100 -lt 90.
Approach
Casting both the current and previous versions as the version datatype allows the comparison to work as intended.
Commands to test
cleanup.ps1

looks good, one pass 👍

Thank you!


Feat remove thor devkit

Remove thor-devkit from dappkit and replace it with vechain-sdk-core

Already implmented


getInspectorState / mixed namespaced modules between tabs

Version
6.0.1
Reproduction link
codesandbox.io
Steps to reproduce
Create namespaced store with some modules (can be empty).
What is expected?
To work ;)
What is actually happening?
Many errors like that:
An error occurred in hook getInspectorState registered by plugin org.vuejs.vue2-internal

At first I couldn't get to work a project & new vue-devtool plugin I've started yesterday (it used to work with old plugin). Today after update I got many "An error occurred in hook getInspectorState registered by plugin org.vuejs.vue2-internal" errors.
What's important it seems that there's a problem with Vuex namespaces and tab context. I don't have path "ns/otherNs/ in the project but when I created bug example and reloaded my project's page i saw this error: Missing module "ns" for path "ns/otherNs/" which is quite imposible becouse there's no such namespace in my project as it only exists in example.

And the other way round. I reloaded example page and got error about not existing namespace orderBox which... is created in other browser's tab - not in example tab.

Interesting issue :eyes:
Is it also misbehaving in other parts of the devtools such as the components inspector and the timeline?

I didn't notice that. It all worked until I tried to check vuex state.

Looks like archaic Vuex version (3.0.1....) caused all the problems.
With 3.6.2 all works like a harm :)


Add a chatbot

Want to add a chatbot to our website.

can I use jquery to add a chatbot to this website?

Yes you can

I have added the chatbot but alongwith some existence js code it is not working properly otherwise it is working completely fine after removing preloading function code.
How should I resolve that problem @SahityaRoy ?

After changing and removing preloder you can simply Push your code in your branch

Ok
On Sat, Oct 16, 2021, 10:18 PM Tanwi Kumari @.***>
wrote:

@tanwi1234 https://github.com/tanwi1234 ???
Yeah, I am working on it will update you by tomorrow.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/SahityaRoy/E-Learning-freesite/issues/47#issuecomment-944952160,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARLSWZDOKAAUL66VQ5CKPSTUHGUHPANCNFSM5AI3GCFA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

any chatbot service provider available.....?

I want to solve this issue can I work on this


Introduce a group journaled convenience class

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6171/
Test PASSed.

alluxio-bot, merge this please


Recording sometimes stops, rmm and mqttv4 killed

Sometimes the camera stops recording videos. Then the camera needs to be restarted and everything works well again.
After the error and a reboot, I ran the command "ps". These processes were no longer present during the error:
1276 root     42436 S    ./rmm
1724 root       880 S    mqttv4
My Camera Dome Guard r30gb YRS 9.0.05
yi-hack-Allwinner-v2 0.3.1
It might be important to note that I bought the camera with the pre-installed version 0.2.4 and it was working well for over a month until I updated it to 0.3.1.

It's a known problem.
The cam runs out of memory and the kernel kills some processes.
Did you enable swap file?
Disable all unneeded features.

Were there any major changes affecting resource consumption between versions 0.2.4 and 0.3.1?
With version 0.2.4 I had about 20000 KB of memory free. With version 0.3.1 it’s only 12000 KB.
Swap: 15
MQTT is disabled
Active features:
Disable Cloud
Recording without Cloud
RTSP low aac
Enable/disable time OSD
NTPD
HTTPD
As an automatic solution to keep the camera active (reboot) after an error, I’m currently using this script and the cron entry:
#!/bin/sh

if ! ps | grep "./rmm" | grep -qv grep; then
    reboot
fi

*/5 * * * * /tmp/sd/yi-hack/bin/checkrmm.sh

Were there any major changes affecting resource consumption between versions 0.2.4 and 0.3.1?

Too many changes. It's not simple to know what change is responsible for this memory consumption.
About restart, wd_rtsp.sh already contains a check to restart the system when rmm is not running.

Thank you very much for your help.
During some testing I had disabled the RTSP server and was already wondering...
Even without my extra script, the camera restarts automatically after crashes.


Wilson

changed the 3 different use case factories to be done in a single GetPlaylistUseCaseFactory

It looks good. But just a heads up, after my pull request is merged, I will probably make changes to this. I'll explain later. (but just make sure to approve and merge my pull request)


Add container's short-id as default network alias

- What I did
link feature in docker0 bridge by default provides short-id as a container alias. With built-in SD feature for user-defined networks, providing a container short-id as a network alias by default will fill that gap.
Without the short-id alias, compose like applications are forced to perform artificial steps such as -

create a container in a network & get the container-id
disconnect the container from the network
connect the same container again to the same network and this time with --alias={short-id}

By providing the short-id as a default alias (similar to links), we can avoid such artificial steps
This also helps with https://github.com/docker/libnetwork/issues/996.
Not targeting this for 1.11 release
Signed-off-by: Madhu Venugopal<EMAIL_ADDRESS>

@bfirsh @dnephin @taylorb-microsoft @aboch FYI. since this is of interest to the compose + TP5 integration, I don't think this is of urgency for the 1.11 release.

If you think it would be good for 1.11, let me know, it's a small diff.

@thaJeztah I think @mavenugo is on vacation - could you potentially find someone else to carry it?

@friism I don't think it needs a carry, just a review

@thaJeztah @calavera addressed your comments. PTAL

LGTM

ping @jhowardmsft PTAL

Seems reasonable to me, but adding @msabansal  to comment on networking from the windows side. Shame none of those tests actually run on Windows though :(

@jhowardmsft @taylorb-microsoft @PatrickLang can you ask the Windows networking folks have a look at this? I haven't tested yet, but I believe it's what's gonna get compose v2 files working with windows
cc @mavenugo

@msabansal  ^^   (@frissm, You want Sandeep for networking questions as I put above)

LGTM for Linux. Windows doesn't have name resolution support at this time. Once we bring in that support we should be able to support this.

@msabansal Just to be clear on the intention of this PR, it solves 2 problems -

Adds short-id alias by default to the name-resolution
Because the short-id alias is added by default, Compose V2 need not perform the disconnect/connect routine. Hence Compose V2 can be used in windows for TP5 (since windows network driver doesn't support disconnect / connect options).

We are looking for a :+1: from Windows side to make sure Compose V2 can be used (with https://github.com/docker/compose/pull/3337) for TP5.

@mavenugo I understand. I don't see any problem there. Some of the compose features will obviously not be supported for windows and we will add support as we progress.

LGTM


Carousel nav causes scroll to bottom of chat window

After a carousel is displayed and then a few more messages are sent and received, if I scroll up to the carousel and navigate it left or right, after clicking on the nav button the chat window scrolls to the bottom (autoscroll, presumably).
See video: https://kodu.blob.core.windows.net/kodu/eanders/capture-2.mp4

No longer happening since the Redux rewrite, where I touched the autoscroll code quite a bit.


[Documentation] Create lookup table with the Build Analysis processor events

We have multiple events processed by build analysis processor that determines what is done and expected for a message, sometimes an specified kind of event is failing and that change a lot how to approach and investigate them, because of this we need to have a documentation with this info and link it to the "Abandoned message" alert.

Release Note Category

[ ] Feature changes/additions
[ ] Bug fixes
[ ] Internal Infrastructure Improvements

Release Note Description

PR: https://dev.azure.com/dnceng/internal/_git/dotnet-eng-wiki/pullrequest/32411

PR for adding link into table: https://dev.azure.com/dnceng/internal/_git/dotnet-helix-service/pullrequest/32470


Update the code that uses the experiments npm package

We're using old deprecated API isCachedFlightEnabled has been deprecated

Cannot fix this as this is used by code here https://github.com/microsoft/vscode-jupyter/issues/11243

This has veen resolved, experiments packages have been updated.


Set up backend endpoints for submitting a dossier

As an initiator, I want to choose the approval pipeline, so I can request the required approvals of relevant parties.
[Story Points: 4]
[Priority: High]
[Risk: Medium]
This is a subtask of #13
Demo steps:
Given I’m an initiator
When I create a dossier for a change
I can send an API request containing the list of groups to the submission endpoint
and I receive a confirmation that the dossier has been submitted.

dossiersubmission.webm