Cannot run local or docker since latest commit

Description
Cannot run local or docker since latest commit 0702ecbe037165224e5fce40dcc73155d138d462
But does work with "git checkout" the commit before : dab1a163559fc21ae7e8c02399a24a4ff7b070ca
Steps to Reproduce the Bug
Git pull
./AGiXT.sh shows menu
Option 1 : Step 1, 2, 3 .... then falls back to the menu and nothing started
When I  try with docker it is the same
Expected Behavior
start backend and frontend
Operating System

[X] Linux
[ ] Microsoft Windows
[ ] Apple MacOS
[ ] Android
[ ] iOS
[ ] Other

Python Version

[ ] Python <= 3.9
[X] Python 3.10
[ ] Python 3.11

Environment Type - Connection

[X] Local - You run AGiXT in your home network
[ ] Remote - You access AGiXT through the internet

Runtime environment

[ ] Using docker compose
[X] Using poetry
[ ] Custom setup (please describe above!)

Acknowledgements

[X] I have searched the existing issues to make sure this bug has not been reported yet.
[X] I am using the latest version of AGiXT.
[X] I have provided enough information for the maintainers to reproduce and diagnose the issue.

Hi I see you checked you're using poetry can you try to git pull with the latest changes since we removed poetry and switched back to requirements.txt you can use the start script called AGiXT.sh

Problem solved now with updates to source code between PR 0702ecbe037165224e5fce40dcc73155d138d462 and 056ecb2c54865f2fbd21f5ac2dfd806dc0747c0f


Add a openFilesInNewWindow setting that opens depending on being inside VS Code or not

Before I get into this bug, thank you for adding this new setting for finer control over how files are opened with Code. Much appreciated!

VSCode Version: 1.9.0
OS Version: Win x64 10.0.14393

Steps to Reproduce:

Set the new openFilesInNewWindow setting to "on"
Activate an existing Code window and choose "File > Open" or "File > Open Recent" to open a file.

Whether the file is within the open/active folder in the Code window or not, the file is always opened in a new window. I don't find this behavior to be intuitive.
I believe "on" should mean that when I'm working in an existing Code window, any "open file" commands I issue from that window should result in the file(s) being opened in that same window.
If I take action outside any Code window (i.e. from Explorer or double-clicking on a file I've associated with Code (which is technically still Explorer I suppose)) to open the file(s), they should open in a new Code window.
As for "default" and "off", these values work as I would expect them to so far.
(Note that the ctrl+p quick-open command is not affected and works as expected, opening files within the active window.)

I would have the same expectations: that the openFilesInNewWindow setting would only control actions taken from outside VS Code. If I already have a VS Code window active and choose File->Open (or File->Open Recent), I'd expect that my currently-active VS Code window to be the one to receive the file.
The reason I expect this is because when I'm already inside VS Code, it's trivial to get a new window if that's where I really wanted to open a file: just hit Ctrl+Shift+N first, then hit Ctrl+O.
BTW, I can confirm that Linux (specifically, Ubuntu 14.04) has the same behavior as Windows in this regard. Not surprising, of course, since this behavior is what was intended by the code, even if it's not what some users apparently expect. :-)

@stefcameron @rmunn again there are platform differences. Mac users expect choices made from the File > Open menu to open inside a new window (as you can see from https://github.com/Microsoft/vscode/issues/371).
Looks like what is needed here is yet another option (or enum value) to do what you suggest here.

The biggest issue for me is that opening a folder will replace the content of the active window. What I want is to open a new project folder in a new window, and if I don't remember to open a new window before I open a project folder, all the work (and terminals!) in my active window get blown away.

@plmrry Have you set openFoldersInNewWindow: "on"? That sounds like the behavior you're looking for when opening folders instead of files.

Btw independent from the settings you can force open file/folder into a new window by pressing Ctrl on Windows/Linux or Alt on macOS.


Curve.sol function _helper

the desc
@return  x^32/_MARKET_OPENING_SUPPLY^5
        for (uint256 index = 1; index <= _N; index++) {
            x = (x.mul(x)).div(_MARKET_OPENING_SUPPLY);
        }

with index = 1,  x = x^2 / _MARKET_OPENING_SUPPLY
when run to next iteration, x = x^4/(_MARKET_OPENING_SUPPLY^2) / _MARKET_OPENING_SUPPLY
the doc is wrong

Yup, the real calculation is actually
@return x^32 / _MARKET_OPENING_SUPPLY^31
The number in the documentation is far off here...


AltheaC_MMA_Blog_19_10_30

Added new how to for MMA and 10/28 Errata post

Docs Build status updates of commit e4739fa:
:warning: Validation status: warnings

File
Status
Preview URL
Details

documentation/blog/MS-WININTBLOGLP/171376c7-36c0-46cc-8185-9858deb30232.md
:warning:Warning

Details

documentation/blog/MS-WININTBLOGLP/6183f838-c21e-4c44-a5bb-1a84e1864c2e.md
:warning:Warning

Details

documentation/blog/MS-WININTBLOGLP/6344e692-0783-4f0a-bb97-5a07b9167286.md
:warning:Warning

Details

documentation/blog/MS-WININTBLOGLP/734a1e39-4616-4ed7-b2cd-766c25d382c4.md
:warning:Warning

Details

documentation/blog/MS-WININTBLOGLP/81cbb832-8bf1-4e68-b46a-7fdb371ddf6b.md
:warning:Warning

Details

documentation/blog/MS-WININTBLOGLP/91797b07-3a66-488c-9334-5023638c4c87.md
:warning:Warning

Details

documentation/blog/MS-WININTBLOGLP/dd98b93c-0a75-4eb0-b92e-e760c502394f.md
:warning:Warning

Details

documentation/blog/MS-WININTBLOGLP/f1812ab3-d00a-41e1-9e60-b8a3da0cda9a.md
:warning:Warning

Details

documentation/blog/MS-WININTBLOGLP/f6bc5f7d-6b3f-441d-9de1-4994329aeff9.md
:warning:Warning

Details

documentation/blog/MS-WININTBLOGLP/MS-WININTBLOGLP_files/image002.png
:white_check_mark:Succeeded

documentation/blog/MS-WININTBLOGLP/MS-WININTBLOGLP_files/image003.png
:white_check_mark:Succeeded

documentation/blog/MS-WININTBLOGLP/MS-WININTBLOGLP_files/image004.png
:white_check_mark:Succeeded

documentation/blog/MS-WININTBLOGLP/MS-WININTBLOGLP_files/image005.png
:white_check_mark:Succeeded

documentation/blog/MS-WININTBLOGLP/MS-WININTBLOGLP_files/image006.png
:white_check_mark:Succeeded

documentation/blog/MS-WININTBLOGLP/MS-WININTBLOGLP_files/image007.png
:white_check_mark:Succeeded

documentation/blog/MS-WININTBLOGLP/MS-WININTBLOGLP_files/image008.png
:white_check_mark:Succeeded

documentation/blog/MS-WININTBLOGLP/MS-WININTBLOGLP_files/image009.png
:white_check_mark:Succeeded

documentation/blog/MS-WININTBLOGLP/MS-WININTBLOGLP_files/image010.png
:white_check_mark:Succeeded

documentation/blog/MS-WININTBLOGLP/MS-WININTBLOGLP_files/image011.png
:white_check_mark:Succeeded

documentation/blog/MS-WININTBLOGLP/MS-WININTBLOGLP_files/image012.png
:white_check_mark:Succeeded

documentation/blog/MS-WININTBLOGLP/MS-WININTBLOGLP_files/image013.png
:white_check_mark:Succeeded

documentation/blog/MS-WININTBLOGLP/MS-WININTBLOGLP_files/image014.png
:white_check_mark:Succeeded

documentation/blog/MS-WININTBLOGLP/toc.yml
:white_check_mark:Succeeded

documentation/blog/MS-WININTBLOGLP/171376c7-36c0-46cc-8185-9858deb30232.md

[Warning] Author openspecs-office cannot be recognized in GitHub.

documentation/blog/MS-WININTBLOGLP/6183f838-c21e-4c44-a5bb-1a84e1864c2e.md

[Warning] Author openspecs-office cannot be recognized in GitHub.

documentation/blog/MS-WININTBLOGLP/6344e692-0783-4f0a-bb97-5a07b9167286.md

[Warning] Author openspecs-office cannot be recognized in GitHub.

documentation/blog/MS-WININTBLOGLP/734a1e39-4616-4ed7-b2cd-766c25d382c4.md

[Warning] Author openspecs-office cannot be recognized in GitHub.

documentation/blog/MS-WININTBLOGLP/81cbb832-8bf1-4e68-b46a-7fdb371ddf6b.md

[Warning] Author openspecs-office cannot be recognized in GitHub.

documentation/blog/MS-WININTBLOGLP/91797b07-3a66-488c-9334-5023638c4c87.md

[Warning] Author openspecs-office cannot be recognized in GitHub.

documentation/blog/MS-WININTBLOGLP/dd98b93c-0a75-4eb0-b92e-e760c502394f.md

[Warning] Author openspecs-office cannot be recognized in GitHub.

documentation/blog/MS-WININTBLOGLP/f1812ab3-d00a-41e1-9e60-b8a3da0cda9a.md

[Warning] Author openspecs-office cannot be recognized in GitHub.

documentation/blog/MS-WININTBLOGLP/f6bc5f7d-6b3f-441d-9de1-4994329aeff9.md

[Warning] Author openspecs-office cannot be recognized in GitHub.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Having problem to update value

I have no problem reading file but for some reason i can not update the values. I searched and found a simple sample
` Dim ini As New IniFile
ini.Load("C:\Users\Dropbox\Vodex\CNAME\cname.ini")
ini.Sections("Error").Keys("value").Value = "yes"`
When i debug and step over it i hit the ini.Load but it never hits the ini.Sections and the value is never updated ?
What am i missing ?

Hi,
Note that IniFile object is an in-memory representation of that loaded INI file.
In order to see any changes that are done in that object, you need to save it.
In other words, try using something like the following:
Dim ini As New IniFile

ini.Load("C:\Users\Dropbox\Vodex\CNAME\cname.ini")
ini.Sections("Error").Keys("value").Value = "yes"

ini.Save("C:\Users\Dropbox\Vodex\CNAME\cname.ini")

Does this solve your issue?
Regards,
Mario

Yes that did the trick for me. All the samples o the internet in different forums did miss that last step
And I did not see anything that said it was in mem only till executing .save
thanks
From: Mario Zorica<EMAIL_ADDRESS>Sent: Wednesday, March 7, 2018 5:26 AM
To: MarioZ/MadMilkman.Ini<EMAIL_ADDRESS>Cc: Alex Ponnath<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [MarioZ/MadMilkman.Ini] Having problem to update value (#21)
Hi,
Note that IniFile object is an in-memory representation of that loaded INI file.
In order to see any changes that are done in that object, you need to save it.
In other words, try using something like the following:
Dim ini As New IniFile
ini.Load("C:\Users\Dropbox\Vodex\CNAME\cname.ini")
ini.Sections("Error").Keys("value").Value = "yes"
ini.Save("C:\Users\Dropbox\Vodex\CNAME\cname.ini")
Does this solve your issue?
Regards,
Mario
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/MarioZ/MadMilkman.Ini/issues/21#issuecomment-371134141, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Ag--pVXKR9EJLb2fKbGXikf0EyxsWFDEks5tb92FgaJpZM4SfP0U.

Hi,
I'm glad that helped.
Also in case you're interested, I would recommend you to take a look at the first few pages of MadMilkman.Ini.Documentation (1 Getting Started, 2 Overview and 3 Code Samples pages)
I hope this helps as well.
Regards,
Mario


fix: added display: grid; in code element

fixed: issues/104

Thank you! I noticed this shortly after I published v0.10.0 and forgot to push a fix (since it's easy to just to manually add display: grid, and think that's currently on the docs).


MSID: 90893 Version: 2 DOI: 10.1101/2<IP_ADDRESS>8217

MSID: 90893
Version: 2
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>8217
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=90893
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>8217

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/90893v2
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>8217
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/90893/v2/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

warn: [mixed-citation-other-publication-flag] mixed-citation in reference (id=c87) has a publication-type='other'. Is that correct?
warn: [p-all-bold] Content of p element is entirely in bold - 'A.'. Is this correct?
warn: [p-all-bold] Content of p element is entirely in bold - 'B.'. Is this correct?
warn: [article-email-corresp-author-count-equivalence] The number of corresponding authors (1: Helen E Scharfman) is not equal to the number of distinct email addresses (2<EMAIL_ADDRESS>helensch@optonline.net). Is this correct?

https://sciety.org/evaluations/hypothesis:GTXg8l4REe-jwF_ugg30ug/content


[BUG] blob.CreateIfNotExistsAsync(); creates a blob but throws an exception hence doesnt append anything to it as intented

Library name and version
Azure.Storage.Blobs 12.13.1
Describe the bug

Following code doesnt work for the first time i.e when the blob doesnt exist. this code only creates an empty blob without appending the content. From next time onwards it appends the content as expected.

BlobContainerClient containerClient = new BlobContainerClient(_connectionString, container);
await containerClient.CreateIfNotExistsAsync();
            // take Blob's reference to modify
            var blob = containerClient.GetAppendBlobClient(fileName);

            // if blob doesn't exist, the system will create it
            if (!blob.Exists())
            {
                **// this works but no code after this line executed.**
                await blob.CreateIfNotExistsAsync();
                blob = containerClient.GetAppendBlobClient(fileName);
                blob.SetTags(AddTags(new Dictionary<string, string> { { "Id", quoteRequestReference } }));
            }

            var data = "somedata";
            var bytes = Encoding.UTF8.GetBytes(data);
            var stream = new MemoryStream(bytes);
            blob.SetTags(AddTags(new Dictionary<string, string> { { "Id", "someid"} }));

           **// this doesn't execute or work for the first time.**
            **await blob.AppendBlockAsync(stream);**

Expected behavior
The await blob.CreateIfNotExistsAsync();  and await blob.AppendBlockAsync(stream); should both work when function gets called and blob doesnt exists
Actual behavior
The await blob.CreateIfNotExistsAsync(); -- creates an empty blob
await blob.AppendBlockAsync(stream); this doesnt get executed first time when blob gets created by a statement above
Reproduction Steps

Run the function with container present but blob not present
When called first time, the blob gets created with no contents in it
When called second time, the contents gets appended to the blob properly

Environment
No response

@jay-aplan I have tried your code block, but I do not see any issues with it in the first run.  The data "some-data" is getting appended to the blob in the first run itself.

I found a workaround for this. I think the async method has some issue with it "CreateIfNotExistsAsync". I replaced that with "Create" and it worked fine for me.


denali-fastboot addon

An official addon that integrates fastboot support into denali.
Probably with https://github.com/ember-fastboot/fastboot-express-middleware

If we use https://github.com/ember-fastboot/fastboot-express-middleware it looks like we leave Denali users in a bit of a bind re how to handle new Ember app deploys (especially annoying in clustered mode). But if we use the full Express server we're almost guaranteed to run into conflicts with Fastboot's Express server. Theoretically you can swap out the httpSeever in Fastboot but ...

We could just use fastboot directly.

Is this mostly to keep the ember app and denali app served from the same server? I almost prefer to keep them separate.

Yes. The idea is for this to be an addon - you certainly don't have to combine them.
However, having first class support via an addon would allow you to do things like use Denali to inject some preloaded data into the fastboot served page, without needing to hit your own Denali API over the network from a separate fastboot server.

Closing in favor of the RFA on the forums: http://discuss.denalijs.org/t/request-for-addon-ember-fastboot/46


React router v4 upgrade?

Can this be upgraded too to rr4? Thanks so much

Yep no problem, I'll do it this evening 👍

Done and dusted

Thank you!!


Broken link in the document | Microsoft Edge stable release notes

Hi,
There is a link in the document that leads to 404 error not found page.

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

ID: 3ffcaf54-ce86-1b30-e947-540bf07a54c3
Version Independent ID: 81267e97-e284-ff00-b081-476934538326
Content: Microsoft Edge release notes for Stable Channel
Content Source: edgeenterprise/microsoft-edge-relnote-stable-channel.md
Product: microsoft-edge
GitHub Login: @dan-wesley
Microsoft Alias: archandr

@HotCakeX Thanks for letting us know!  I believe the correct URL is https://support.microsoft.com/en-us/topic/recover-lost-or-deleted-favorites-in-microsoft-edge-ac06270e-ba57-4266-8389-8bfd1df1aa5e.  We will work on updating the doc.

@Y-Kelly Thank you ^^


[Feature request] Hope can use custom repo when install CJK

Idea
In some countries, for some special reasons, cannot connect to dl-cdn.alpinelinux.org. So i hope can use custom repo
This is my idea:
Add a environment APKREPO, like APKREPO=mirrors.aliyun.com. And add a row like if [ ! -z $APKREPO ]; then sed -i "s/dl-cdn.alpinelinux.org/$APKREPO/g" /etc/apk/repositories; fi to 10-cjk-font.sh

Maybe I can submit a pr, but I don't find 10-cjk-font.sh in this repo :

This part is shared across multiple containers, both based on Alpine and Debian/Ubuntu.  So I will add a generic solution that works in all cases.

PACKAGES_MIRROR environment variable can now be used for this purpose.


SpawnActor failed because of collision at the spawn location

I've created and imported a custom vehicle into Carla. I followed the Carla docs to create and import it. When I run python3 manual_control.py --filter <my_vehicle_name> I got this error. I couldn't spawn the vehicle.
CARLA version: latest (source build)
Platform/OS: Ubuntu 18.04
Problem you have experienced: Error: Actor not Spawned
What you expected to happen: Spawn a custom vehicle
Steps to reproduce: Run manual_control.py in PythonAPI/examples
Other information (documentation you consulted, workarounds you tried):

Hello,
I'm having a similar issue and I have posted an issue #5674
Can anyone help please?
Thanks!


[Enhancement] Easy checking of CLI output

What this PR does / why we need it:
This PR would add an declarative way to check the output of the structs.
This types of command could be used now :
apiVersion: kuttl.dev/v1beta1
kind: TestStep
commands:
- command: echo "Hello World"
  output:
    stdout:
      contains: "Hello World"
    stderr:
      contains: "Hello World"

Fixes https://github.com/kyverno/kuttl/issues/22

@eddycharly
I am trying to fix this but this is not fixing :
gofmt -w /pkg/test/utils/kubernetes.go

How about something like:
apiVersion: kuttl.dev/v1beta1
kind: TestStep
commands:
- command: echo "Hello World"
  output:
    stdout:
      expected: "Hello ?*"
      useWildcard: true
    stderr:
      expected: "Hello World"

apiVersion: kuttl.dev/v1beta1
kind: TestStep
commands:
- command: echo "Hello World"
  output:
    stdout:
      expected: "Hello ?*"
      match: Wildcard
    stderr:
      expected: "Hello World"
      match: Equals

Okay, the Match could be Equal ( strict ) and contains ( just string contains check )  a wildcard (wildcard check could be passed) .....
Or we should keep match to  equals and wildcard

I think Equals and Wildcard are enough to cover 99% cases.
We might need Contains in very specific situations though and it shouldn't rely on wildcards (when the expected output contains * or ? but we don't want to treat them as wildcard patterns...)

So let's push contains it would be good 👍

So let's push contains it would be good 👍

Ok, fine.

Any schema changes just be sure that the initial post in this PR has a full and complete description of what's available.

type CommandOutput struct {
	Stdout ExpectedOutput `json:"stdout"`
	Stderr ExpectedOutput `json:"stderr"`
}

type ExpectedOutput struct {
	Equals string `json:"equals"`
	Contains string `json:"contains"`
	Wildcard string `json:"wildcard"`
}

Now we could have something like :
For only wild-card match.
apiVersion: kuttl.dev/v1beta1
kind: TestStep
commands:
- command: echo "Hello World"
  output:
    stdout:
      wildcard: " any pattern "

Strictly the stdout should be equal what I define.
apiVersion: kuttl.dev/v1beta1
kind: TestStep
commands:
- command: echo "Hello World"
  output:
    stdout:
      equals: "Hello World"

Check whether the the stdout contains the "hello"
apiVersion: kuttl.dev/v1beta1
kind: TestStep
commands:
- command: echo "Hello World"
  output:
    stdout:
      contains: "Hello"

@shubham-cmyk the changes here do not reflect this:
  output:
    stdout:
      expected: "How are you today ?"
      match: Wildcard

@eddycharly

@shubham-cmyk the code in validateCommandOutput is very similar for stdout and stdedd, can we factorise it ?

is very similar for stdout and stdedd, can we factorise it ?

Yes I can do that

It would be nice to cover this with tests too 🙏

It would be nice to cover this with tests too 🙏

Added unit_test to cover this function.

@shubham-cmyk please raise an error if background is true and output checks are present

@shubham-cmyk please raise an error if background is true and output checks are present

Done


Add support for annotations on Workflows, CronWorkflows and Tasks

Add support for annotations on Workflows, CronWorkflows and Tasks

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Looks great! Thank you for the contribution @RunningIkkyu 🙂 any chance you can update the minor VERSION and add a CHANGELOG entry? We can merge after that.

Thanks @flaviuvadan, the version and CHANGELOG has been updated!


Pixeled mouse cursor on tab bar next to tabs

Issue Type: Bug
The cursor shown on the tab bar, where a double click opens a new tab, is a "hand touch" cursor which cannot be changed in windows settings and for one makes no sense (should be simple pointer) and also is pixeled grocely. (Note: this is not the hand pointer for links!) See attached image taken without any extensions active. My screens are UHD scaled to 150% - all other cursors display fine.
VS Code version: Code 1.63.2 (899d46d82c4c95423fb7e10e68eba52050e30ba3, 2021-12-15T09:40:02.816Z)
OS version: Windows_NT x64 10.0.19044
Restricted Mode: No

System Info

Item
Value

CPUs
11th Gen Intel(R) Core(TM) i7-11700KF @ 3.60GHz (16 x 3600)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
63.85GB (43.38GB free)

Process Argv
--disable-extensions --crash-reporter-id 89fc7b7f-a14e-42c2-a456-7da04cf6efee

Screen Reader
no

VM
0%

Extensions disabled

I think it's a hand because you can drag this area. For example, try this:

Open a file.
Split Editor Right using the button on the upper right of the window.
Double-click in the are to the right of the file's tab in the right-hand split. This creates an untitled file.
Repeat step 3 to create a second untitled file.
Close the right-hand group's copy of your original file.
Now drag the pixelated hand from the right group and drop it onto the left group. This transfers the two untitled files onto the left group and closes the right group.

It also appears when there is no target area to drag the tabs to, which I find irritating. And it is really ugly... ;) Of course it is a Windows-issue that this cursor is nowhere to be changed and has never been updated to HighDPI apparently, but maybe there is another way to achieve this within VS Code? I have never seen this cursor in any other app ever....

I checked the behavior once more; when there is no split view, the active tab can be dragged and shows a pointer-hand (like over URLs), while dragging from the empty space in the bar also drags the active tab, now showing the (ugly) touch-hand, though. This is also inconsistent. The easiest fix would probably be to always use the pointer-hand.

As it was pointed out, this works as designed, the editor group is draggable, i.e. you can move all tabs at once.
The fact that the cursor looks bad is something I reported a while ago, but Chrome does not seem to have any interest in fixing it: https://bugs.chromium.org/p/chromium/issues/detail?id=1035285&q=&can=4


add: rishabh kumar

Description
I have finally added my profile
[Include a brief description of the changes you've made]
added my profile as per the guidelines
Kindly review, check, and merge it to the master branch
Related Issues
[Include any related issues or pull requests that this PR addresses or is related to. Use GitHub's shorthand syntax to link to them, like #1234.]
Changes Proposed
[Describe the changes you've made in detail. Be specific and include any relevant code snippets.]
Checklist

[x] I have read and followed the Contribution Guidelines.
[x] All new and existing tests passed.
[x] I have updated the documentation to reflect the changes I've made.
[x] My code follows the code style of this project.
[x] The title of my pull request is a short description of the requested changes.

Screenshots
Note to reviewers

@rishabh22204901 Thank you so much for your contribution! Your efforts are greatly appreciated and will go a long way in improving our project. Please feel free to share it with others and help us grow even further!


Wrong Description for SeatControlCapabilities, RadioControlCapabilities

The description of SeatControlCapabilities.moduleName mentions the 'light control module'.
The description of RadioControlCapabilities.moduleName mentions the 'climate control module'.
Both cases seem to be copy and paste errors.

Closed via https://github.com/smartdevicelink/rpc_spec/pull/319


Introduce BP_PHP_NGINX_ENABLE_HTTPS to enable https

ssl directive has been deprecated since 1.15.0,
and is removed in nginx 1.25.1.
This is replaced by the configuration option to add the value ssl to the listen directive.
In the pre-deprecation world, a user can add a user-defined conf file with the app to turn on ssl. In the new world, since the listen directive is provided as part of the core nginx conf generated by this buildpack, users are left with no option (you can't override a directive).
This change solves the issue by allowing users to set BP_PHP_NGINX_ENABLE_HTTPS=true
Fixes https://github.com/paketo-buildpacks/php-web/issues/139
Fixes https://github.com/paketo-buildpacks/php-nginx/issues/21
Fixes failure of https://github.com/paketo-buildpacks/php/pull/724 with env var addition in tests.
Links:
http://hg.nginx.org/nginx/rev/0aaa09927703
http://nginx.org/en/CHANGES
http://nginx.org/en/docs/http/ngx_http_core_module.html#listen
Checklist

[x] I have viewed, signed, and submitted the Contributor License Agreement.
[x] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[ ] I have added an integration test, if necessary.
[x] I have reviewed the styleguide for guidance on my code quality.
[x] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

@sophiewigmore Thanks for looking into this.

Fixed integration tests
Added docs to website https://github.com/paketo-buildpacks/paketo-website/pull/696


Use auto nat discover starting in lnd 0.5-beta

This the same issue as Stadicus/guides#249

I followed your guide. Making new build and testing. Much thx.

On my testing it did not work and the lnd made a shutdown getting into a loop of unlock, shutdown, unlock, shutdown - so this setting is not godd as a default ... will put scripts to old state for now.
Maybe add an option later to activate this.

Did you allow your Raspiblitz to create UPnP port forwardings..?

@frennkie the ufw rule "allow local LAN SSDP for UPnP discovery" is active - is that what you mean?

Moving issue to version 1.0 milestone to at least make it an option to activate.

Yes, makes sense.. also I forgot, that there is still an issue with the nat option: lightningnetwork/lnd#2014.
LND will not re-create the port forwarding if - e.g. after a 24 hour DSL disconnect the port forwarding is no longer active on the router. I have been running the code in the PR for two weeks now and it's working fine. But I have to see how and when this can be merged into the official code.

@frennkie can you report on any progress on this issue with lnd?
For version 1.0 want to remember to put "auto nat discover" as an option into the service section, but deactivated by default for now.

I have been occupied otherwise... From my point of view LND needs to be changed.. Not sure how fast such a change can be incorporated..

With the next sd card release the auto-nat feature can be switched on in the servcie section to experiment with. Closing this ticket for now.

Fine with me.
As long as the two PRs for lnd are not merge it would make sense to add some context/additional information for people who think about enabling the option.

If you write up two/three lines with additional info I can add it as an notice during the install process.


Added radare2 plugin.

Solves #3.

Manually integrated this change as we now have a better format for handling multiple submissions. Thanks for the submission.

Thanks Jordan.


Add new API functionality

Implements:

UserGroup class
search_all method in Devices class to retrive all devivces

Sorry to bother you with this again (if you missed my message in Slack):
I am tring to retrive all memeber of usergroup which is synced from AD.

Find the group: GET /api/system/usergroups/search
->

{'Page': 0,
 'PageSize': 500,
 'ResultSet': [{'groupName': 'AirWatch',
                'id': 11,
                'lastSyncOn': '10/29/2018 15:10:43',
                'mergeStatus': 'MergeComplete',
                'rootLocationGroupName': 'redacted',
                'syncStatus': 'SyncComplete',
                'type': 'Directory',
                'users': 3}],
 'Total': 1}

Try to get all members like with regular usergroup: GET /api/system/usergroups/11/users
->

Error #400: UserGroup 11 not found or user does not have access to it.

My API user is OU admin. Any ideas why can I retrive all members of regular usergroup but not from the one synced from AD?

thanks again for the contribution @MichalMMac ! The PR looks good and I will merge it.
Regarding your question, I spoke with the respective teams within R&D and found out this is a current gap in the API today. I've been told it's prioritized to be completed some time in 2019. There is a FR number to track at PR-184954. Thanks!

Cool. Thank you for this information :-) I'll keep an eye on it.


Add and expose metrics

Add various metrics using Prometheus.

As announced on Slack, we (i.e., me and/or some of my team colleagues) would be keen on adding some initial support for Prometheus-based metrics collection.
Is there anything in particular that should be included from the project maintainer's / Ark community's view? The baseline for us would probably be somewhere along the lines of collecting info about

when a backup / restore took place
how many objects were included
how many errors occurred

That would allow us to set up alerts like "notify me when the last successful backup is older than x time units" and "notify me when less than x objects were considered".
Implementation-wise, we usually just include the official Go Prometheus client library. So far, it has served us well.
Happy to discuss design considerations and implementation details up front.

@timoreimann I think all of those are good ideas. I'm also interested in things like

time to list all pods, services, etc
time to write the backup tarball
time to upload a file to object storage
size of tarball, log file, etc
time to run custom backup actions and restorers

+1 to the official client lib. We're using dep now so you can run hack/dep-save.sh, where any arguments you pass are handed directly to dep ensure.
I think it would be a good idea to see a rough sketch of how you envision integrating metrics gathering into the code base.
Thanks!

Appreciate the prompt feedback, @ncdc. I'll try to dedicate some time next week to dig into the relevant code sections and come up with a metrics master plan to post here. :-)

@timoreimann this makes sense, thanks!
FYI I did stumble across this branch where someone was already doing a POC: https://github.com/heptio/ark/compare/master...chadcatlett:stab_at_metrics. Probably worth getting @chadcatlett involved in the discussion 😄
If we do our own registry, would it be a singleton, or would we ever want or need to have multiple?
Another thing to consider is that we're using gRPC based plugins. We'll need to figure out how to get any metrics generated by plugin instances - either by somehow getting their data into the ark server process, or by sending them to a metrics gateway or to a prometheus server directly. Something to think about...

@ncdc I don't think we'll need more one than registry, so it should be a singleton basically. The design options I outlined above should only affect how (easily) we can test things IMHO.
gRPC-based plugins are an interesting point. I'm not too familiar with the plugins architecture in Ark. If the binaries are long-running just like the Ark server, then I'd think that having the server component pull in plugin metrics via a dedicated gRPC function on each Prometheus scrape interval could be an option. OTOH, if the binaries are invoked at certain times only and terminate again while the Ark server keeps on running, letting them submit their metrics through the Prometheus pushgateway makes more sense.
AFAIK, the pushgateway doesn't come with a retention period that cleans up metrics automatically. So when in doubt, the regular pull mechanism seems preferable.
@chadcatlett: I see you have already produced some code. Is there already a PR somewhere? Happy to collaborate / coordinate to get the code into Ark. :-)

Currently, some plugins are long-lived, while others are very short. Our plan, however, is to make all plugins short-lived, as it should make it easier for us to handle how we manage plugins.
My ideal setup would be to have the Ark server pull metrics from a plugin just before it terminates the plugin. I'm not sure if there's an easy way to do this with Prometheus. I stumbled across
https://github.com/prometheus/client_golang/blob/e11c6ff8170beca9d5fd8b938e71165eeec53ac6/prometheus/examples_test.go#L648-L705
which uses a custom Gatherer, which is a model we could perhaps consider, or something like it.
I'd prefer to avoid requiring setting up an external component such as the pushgateway as the sole means of gathering plugin metrics. It just doesn't feel right to me.

Pulling metrics just before the plugin terminates sounds like a viable option if we are able to control the plugin lifecycle.

Yes, we're 100% in control (assuming the plugin doesn't crash).

We just came back to this today too and would be interested in it's progress and potentially some hands on keyboard implementation in my ever abundant free time :)
For what it's worth, I would trend towards simplicity first on this to determine what/how we want to use it much to a few of your observations @timoreimann.  Would it be viable to put in the Global registry without plugin consideration to start?  I think this would give us the biggest bang for our buck. To your point, I could see a more generalized TracingContext argument being valuable and perhaps expand beyond simply a Prometheus exporter there.
I haven't researched closely enough the plugin model, but I could see a somewhat simple first pass at those metrics simply being plugin level RED metrics to start.  This would give us visibility into the duration of the Plugin execution broken out by the plugin name to start.  If it was determined we need more granular trace like information for the internals of the plugin then we could create a "proper interface" for the plugins that could allow the exposing of that data.
I have some similar concerns about the PushGateway integration for this due to the complexity of integration and implementation that may involve.
Would be happy to chat about this with anyone who is passionate about seeing it happen!

@jrnt30 as long as we have a high-level vision of how we plan to integrate plugin process metrics into the overall metrics registry, I would be fine starting with just the Ark server. But I would hesitate to use the global registry if we can identify up front that it's impossible to integrate metrics from external sources into it easily.

As a user I'd love a topline metric for each schedule that just counts backup successes and failures. As I just discovered our backups have been failing for weeks by stumbling upon it.

I'm not familiar enough with the Ark plugin model nor the prometheus client to weigh in on those topics.
That said, I think it's important to differentiate between the consumers of Ark metrics.
@ncdc - as an Ark developer, you likely care about visibility and performance metrics of Ark internals based on the environment it's running in (e.g. how many objects to backup, and how long does it take)
A plugin developer likely will want to have their own metrics as well that are focused on their development process.  These metrics need to be namespaced so that two plugins don't collide on metrics.  (namespacing could be via different metric name, or via Labeling)
Ark users, however, probably care more about RED metrics (Rates - how many backups have been taken, Errors - how many backups failed, Distribution - how long did it take for backups to complete).  @SleepyBrett also makes an excellent point that we may want to have metrics labeled by Backup name and/or Scheduled name.
Quick pass at metrics & types, feedback welcome
# Metric labels for all include:
# * {backup="foo"} -- The name of the backup
# * {schedule="bar"} -- The name of the schedule that triggered this backup
# ark_backup totals for overall success/failure tracking
ark_backup_attempts - counter
ark_backup_completions - counter
ark_backup_failures - counter

# ark_backup_seconds describes the duration of a backup from start to finish
ark_backup_seconds -- histogram
ark_backup_seconds_count -- counter
ark_backup_seconds_sum -- counter

# ark_backup_size describes the size of the resulting tarball
ark_backup_tarball_size_bytes - histogram
ark_backup_tarball_size_count - counter
ark_backup_tarball_size_sum - counter

# ark_backup_tarball_write_seconds describes the time to write the tarball to disk
ark_backup_tarball_write_seconds
ark_backup_tarball_write_seconds_count
ark_backup_tarball_write_seconds_sum

# ark_backup_tarball_upload_seconds describes the time to upload the tarball
ark_backup_tarball_upload_seconds
ark_backup_tarball_upload_seconds_count
ark_backup_tarball_upload_seconds_sum

# ark_backup_actions_seconds describes the time to run custom backup actions
# Extra labels: action_name
ark_backup_actions_seconds -- histograms
ark_backup_actions_seconds_count -- counter
ark_backup_actions_seconds_sum -- counter

### Now restores

# ark_restore totals for overall success/failure tracking
ark_restore_attempts - counter
ark_restore_completions - counter
ark_restore_failures - counter

# ark_restore_seconds describes the duration of a backup from start to finish
ark_restore_seconds -- histogram
ark_restore_seconds_count -- counter
ark_backup_seconds_sum -- counter

# ark_restore_size describes the size of the resulting tarball
ark_restore_tarball_size_bytes - histogram
ark_restore_tarball_size_count - counter
ark_restore_tarball_size_sum - counter

# ark_restore_tarball_download_seconds describes the time to download the tarball
ark_restore_tarball_download_seconds
ark_restore_tarball_download_seconds_count
ark_restore_tarball_download_seconds_sum

# ark_restore_actions_seconds describes the time to run custom restorers
# Extra labels: restorer_name
ark_restore_restorers_seconds -- histograms
ark_restore_restorers_seconds_count -- counter
ark_restore_restorers_seconds_sum -- counter

I did not include log file as I wasn’t sure how important that is @ncdc
I did not include listing pods, services, etc because I don’t know if it matters the type of object, or just how long it takes to get all the objects, regardless of type
Do we want/need to track the duration of time it takes for a cloud-provider snapshots be taken?
I’m not sure about the ark_restore_restorers — feedback welcome

Thumbs up for the RED metrics.

RED metrics are something I'm looking for as well, I want to know if backups are failing most importantly, but all the other things mentioned would be a plus.

+1

Hoping that PR 531 will channel the discussion here and get us the metrics that we want and add more metrics in the future. Keeping the approach simple and as non-intrusive as possible. Feel free to leave feedback on the PR. I will continue to add the RED metrics that seem to be most useful for cluster operators.

@ncdc You can assign this issue to me as I am working on the PR anyway. I am unable to do it myself.

Assigned!

Added another metric to my list:
ark_info used to provide version and other meta data about this Ark instance
* {version:"v0.9.0"}
* {branch:"master"}
ark_info - gauge # always set to 1

Tried out the v0.9.0-alpha.2 Ark with the Prom metrics endpoint support added in https://github.com/heptio/ark/pull/531 (getting the metrics at :8085/metrics works like a charm).
However, with the above metric list/metrics in place now, as the ark_backup_failure_total counter metric is a total, we're not sure how to actually do alerting on this type of metric/the available metrics for failed backups. After one failed backup, this ark_backup_failure_total counter would always be >=1 so alert conditions like "Alert me when ark_backup_failure_total > 0" would always be active/trigger.

You could use the delta or rate as a way of evaluating how that number
has changed over time.
On Tue, Jun 19, 2018 at 12:34 PM James Wen<EMAIL_ADDRESS>wrote:

Tried out the v0.9.0-alpha.2 Ark with the Prom metrics endpoint support
added in #531 https://github.com/heptio/ark/pull/531 (getting the
metrics at :8085/metrics works like a charm).
However, with the above metric list/metrics in place now, as the
ark_backup_failure_total counter metric is a total, we're not sure how to
actually do alerting on this type of metric/the available metrics for
failed backups. After one failed backup, this ark_backup_failure_total
counter would always be >=1 so alert conditions like "Alert me when
ark_backup_failure_total > 0" would always be active/trigger.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/heptio/ark/issues/84#issuecomment-398482281, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAWanIf-bZoLSsxSoD97OOFqfq39-8hGks5t-TYogaJpZM4PWbrz
.

@jrnt30 are you referring to the rate and delta functions in the context of Prometheus? We're unfortunately not using Prometheus. We have an in-house metrics system and we're writing integration scripts/daemons that poll Prometheus metrics endpoints for various components in our clusters and sends/writes those metrics to that in-house metrics system.
It sounds like with the Prometheus metrics paradigm, we wouldn't ever expect the Ark /metrics endpoint to return any kind of rate information and the layer that is processing/polling the Prometheus metrics would need to handle that?

That is true, yes.  Typically with counters in Prometheus they only go up
and the query is then responsible for analyzing the "impact" of that.
rate and delta are two ways of assessing "has this thing changed in my
time window" or "what is the rate at which this has changed".
Something similar would be able to be done with your in-house tooling, but
can I understand how, without this context, that would be a bit confusing.
Does that clarify things at all?
On Tue, Jun 19, 2018 at 12:53 PM James Wen<EMAIL_ADDRESS>wrote:

@jrnt30 https://github.com/jrnt30 are you referring to the rate and
delta functions in the context of Prometheus? We're unfortunately not
using Prometheus. We have an in-house metrics system and we're writing
integration scripts/daemons that poll Prometheus metrics endpoints for
various components in our clusters and sends/writes those metrics to that
in-house metrics system.
It sounds like with the Prometheus metrics paradigm, we wouldn't ever
expect the Ark /metrics endpoint to return any kind of rate information
and the layer that is processing/polling the Prometheus metrics would need
to handle that?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/heptio/ark/issues/84#issuecomment-398488416, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAWanJHC3JSwqXiF5KgHdrCMtFFRavMjks5t-TqzgaJpZM4PWbrz
.

Yup, that makes sense. With our lack of Prometheus and unwillingness to add all this rate/delta calculation logic into the metrics middle layer (between Prom endpoints and our in-house system), we're probably going to try to emit metrics for "time-since-last-backup" by looking at the GCS bucket backing the Ark backups.

That is one way of doing it for sure.  Another approach which I tool while
this was being ironed out was writing a small utility that used the Backup
CRDs in the cluster itself to determine what the state should be
considered.
A word of caution that we ran into is that if you don't have an indicator
of what it was the last time you checked then you won't have a good handle
on what the new number means potentially. In the end I have a feeling you
may end up putting in a similar amount of effort to checkpoint that as you
would simply using the prom metrics.  A hacky way of doing that would be to
just use a configmap to actually write your last known state back into the
cluster each time your process "scrapes" the default metrics.
On Tue, Jun 19, 2018 at 1:05 PM James Wen<EMAIL_ADDRESS>wrote:

Yup, that makes sense. With our lack of Prometheus and unwillingness to
add all this rate/delta calculation logic into the metrics middle layer
(between Prom endpoints and our in-house system), we're probably going to
try to emit metrics for "time-since-last-backup" by looking at the GCS
bucket backing the Ark backups.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/heptio/ark/issues/84#issuecomment-398492252, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAWanNOxNssM3VHyiLQoiWC98HoCu1uEks5t-T2GgaJpZM4PWbrz
.

So the ideal way to set this up is to do aggregation, think rate in prometheus land, and use a look back window. Here is an example of how to setup your alerts
- alert: FailedBackup
  expr: rate(ark_backup_failure_total[62m]) > 0
  for: 30s
  labels:
    severity: "warning"

@ashish-amarnath Hi let me know if you need help in this implementation. We too need this feature.

@munai-das thanks! Please feel free to open a PR to add any metrics you'd like. I'd recommend starting small - pick a single metric or a group of related metrics and do a PR for that.

I've just installed 0.9.0 and and it works very well. I use Prometheus and these metrics are enough for me to create alerts or diagnose problems. Thank you. :+1:

Can anyone please clarify how to use these metrics? I have installed Prometheus Operator (which includes Grafana) and then Velero from Helm but I don't know how to access/use the metrics. Thanks

@vitobotta It's because Velero adds some annotations to inform Prometheus about it's metrics. However, Prometheus now discourages the use of annotations in favor of ServiceMonitors and has disabled the scraping of the annotated targets by default.
But you can have a look at https://github.com/coreos/kube-prometheus/pull/16 on how to add the required config. I'm doing that now and I'll try to remember and give some feedback if it works for me or not :)

Is there any complete documentation on monitoring velero (preferably via prometheus) ?


lvalues

https://eli.thegreenplace.net/2011/12/15/understanding-lvalues-and-rvalues-in-c-and-c/
https://stackoverflow.com/questions/3601602/what-are-rvalues-lvalues-xvalues-glvalues-and-prvalues
But lets not make c++2

Closed by #45


Remove copied props from state

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

55 of 57 (96.49%)  changed or added relevant lines in 7 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.01%) to 97.123%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/reducers/managerReducer.ts
20
22
90.91%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
1417

Relevant Lines:
1455

💛  - Coveralls


Volume cap when using media keys

Hello,
When using my monitor over usb-c with the lid closed and a magic keyboard 2, my volume is capped at 4 bars when using the media keys (12/50 in monitor values).
However, when using ⇧ + ⌥ + Volume up/down, I can increase the volume above 4 bars.
Moreover, once this threshold crossed, I cannot increase volume in normal increments; decreasing works fine.

Using the sliders from the menu bars works even though the volume and brightness sections are greyed out.

Greyed out sections in the menubar screenshot

Troubleshooting:

[x] Restarted macbook
[x] Restarted monitor
[x] Ran ddcctl (see below)
[x] Tried different cables:

USB-C to USB-C
HDMI to USB-C

ddcctl
I compiled ddcctl locally for amd and intel.
Running ./ddcctl -d 1 -v <volume> was successful in changing the volume.
(My monitor volume ranges between 0 and 50)
Information:

Type
Model/Version

Display
BenQ EX2780q over USB-C

Device
MBP 15" (2016)

OS
10.15.4

Monitor Control
2.0.0 (Build 640)

I have the issue with the same monitor + mac mini 2018

I have a similar issue.
At this level, my monitor displays it at 95/100. Any higher and it will go above 100 which doesn't make any sense. Each increment is also not the same value.


Unable to login as admin or anything else

Describe the bug
I am unable to login as the admin after installing alcali as a salt formula onto a new salt-master running 3004.1.
To Reproduce
Attempt to login via web browser and get invalid login error in red.
Expected behavior
Successful login
Desktop (please complete the following information):

OS: Windows 10 21H2
Browser: Chrome 98.0.4758.102

Additional context
Server OS is the latest release of Almalinux as of this date.
I found no alcali command but created a shell script containing this as a shortcut:
ENV_PATH=/opt/alcali /opt/alcali/.venv/bin/python3 /opt/alcali/code/manage.py "$@"
alcali check works, as does changepassword, createsuperuser, etc.  And the db has the extra tables created by alcali in addition to the default ones needed for salt.  Salt is also actively populating that db.
I have tried with and without LDAP, but my current obfuscated .env shows:
AUTH_BACKEND="ldap"
DB_BACKEND="mysql"
DB_NAME="salt"
DB_USER="alcali"
DB_PASS="dbpassword"
DB_HOST="localhost"
DB_PORT="3306"
MASTER_MINION_ID="saltmaster.dom.pri"
SECRET_KEY="thisisnotagoodsecret.orisit?"
ALLOWED_HOSTS=""
SALT_URL="https://saltmaster.dom.pri:8080"
SALT_AUTH="alcali"
AUTH_LDAP_SERVER_URI="ldaps://ad.dom.ctrl.dom.pri"
AUTH_LDAP_USER_BASE_CN="dc=dom,dc=pri"
AUTH_LDAP_USER_SEARCH_FILTER="(objectClass=)"
AUTH_LDAP_BIND_DN="FullDN of LDAP query user"
AUTH_LDAP_BIND_PASSWORD="binddn_password"
Removing AUTH_BACKEND or setting to "None" has the same effect.
Also, fwiw, my salt formula is being picked up and applied but the /srv/pillar/alcali.sls file appears to not get used at all.
In there I am attempting to install the latest version 3003.2.
I have had to adjust some of the pkg installs since we are also using Percona instead of mysqld or mariadb.  But the db does appear to be working otherwise.

I added the following to my .env
AUTH_LDAP_USER_SEARCH_FILTER=(sAMAccountName=%(user)s)
and I edited /etc/systemd/system/alcali.service to enable logging to syslog:
ExecStart=/opt/alcali/.venv/bin/gunicorn --log-syslog config.wsgi:application --bind <IP_ADDRESS>:5000 -w 4 -t 300
And now I see the following in /var/log/messages when I attempt to login as my AD user:
Feb 25 08:17:56 saltmaster alcali[116437]: search_s('OU=IT,DC=dom,DC=pri', 2, '(sAMAccountName=%(user)s)') returned 1 objects: cn=first.last,ou=it,dc=dom,dc=pri
Feb 25 08:17:56 saltmaster alcali[116437]: Populating Django user first.last
I ran alcali dbshell and manually updated the db so that my user is_staff=1.
However, I still get the "Invalid Login / Password" message.  This might be because of a logging error.  Not sure if this was introduced by the fact that I am logging at all now and it also wants access log and error log defined for the django instance.  This is not logged every time, and perhaps only after the 3rd or 4th time I try to login:
Feb 25 08:21:22 saltmaster alcali[116437]: --- Logging error ---
Feb 25 08:21:22 saltmaster alcali[116437]: Traceback (most recent call last):
Feb 25 08:21:22 saltmaster alcali[116437]:  File "/usr/lib64/python3.6/logging/handlers.py", line 942, in emit
Feb 25 08:21:22 saltmaster alcali[116437]:    self.socket.sendto(msg, self.address)
Feb 25 08:21:22 saltmaster alcali[116437]: OSError: [Errno 9] Bad file descriptor
Feb 25 08:21:22 saltmaster alcali[116437]: Call stack:
Feb 25 08:21:22 saltmaster alcali[116437]:  File "/opt/alcali/.venv/bin/gunicorn", line 8, in
Feb 25 08:21:22 saltmaster alcali[116437]:    sys.exit(run())
Feb 25 08:21:22 saltmaster alcali[116437]:  File "/opt/alcali/.venv/lib/python3.6/site-packages/gunicorn/app/wsgiapp.py", line 58, in run
Feb 25 08:21:22 saltmaster alcali[116437]:    WSGIApplication("%(prog)s [OPTIONS] [APP_MODULE]").run()
Feb 25 08:21:22 saltmaster alcali[116437]:  File "/opt/alcali/.venv/lib/python3.6/site-packages/gunicorn/app/base.py", line 228, in run
Feb 25 08:21:22 saltmaster alcali[116437]:    super().run()
Feb 25 08:21:22 saltmaster alcali[116437]:  File "/opt/alcali/.venv/lib/python3.6/site-packages/gunicorn/app/base.py", line 72, in run
Feb 25 08:21:22 saltmaster alcali[116437]:    Arbiter(self).run()
Feb 25 08:21:22 saltmaster alcali[116437]:  File "/opt/alcali/.venv/lib/python3.6/site-packages/gunicorn/arbiter.py", line 202, in run
Feb 25 08:21:22 saltmaster alcali[116437]:    self.manage_workers()
Feb 25 08:21:22 saltmaster alcali[116437]:  File "/opt/alcali/.venv/lib/python3.6/site-packages/gunicorn/arbiter.py", line 545, in manage_workers
Feb 25 08:21:22 saltmaster alcali[116437]:    self.spawn_workers()
Feb 25 08:21:22 saltmaster alcali[116437]:  File "/opt/alcali/.venv/lib/python3.6/site-packages/gunicorn/arbiter.py", line 616, in spawn_workers
Feb 25 08:21:22 saltmaster alcali[116437]:    self.spawn_worker()
Feb 25 08:21:22 saltmaster alcali[116437]:  File "/opt/alcali/.venv/lib/python3.6/site-packages/gunicorn/arbiter.py", line 583, in spawn_worker
Feb 25 08:21:22 saltmaster alcali[116437]:    worker.init_process()
Feb 25 08:21:22 saltmaster alcali[116437]:  File "/opt/alcali/.venv/lib/python3.6/site-packages/gunicorn/workers/base.py", line 140, in init_process
Feb 25 08:21:22 saltmaster alcali[116437]:    self.run()
Feb 25 08:21:22 saltmaster alcali[116437]:  File "/opt/alcali/.venv/lib/python3.6/site-packages/gunicorn/workers/sync.py", line 123, in run
Feb 25 08:21:22 saltmaster alcali[116437]:    self.run_for_one(timeout)
Feb 25 08:21:22 saltmaster alcali[116437]:  File "/opt/alcali/.venv/lib/python3.6/site-packages/gunicorn/workers/sync.py", line 67, in run_for_one
Feb 25 08:21:22 saltmaster alcali[116437]:    self.accept(listener)
Feb 25 08:21:22 saltmaster alcali[116437]:  File "/opt/alcali/.venv/lib/python3.6/site-packages/gunicorn/workers/sync.py", line 29, in accept
Feb 25 08:21:22 saltmaster alcali[116437]:    self.handle(listener, client, addr)
Feb 25 08:21:22 saltmaster alcali[116437]:  File "/opt/alcali/.venv/lib/python3.6/site-packages/gunicorn/workers/sync.py", line 134, in handle
Feb 25 08:21:22 saltmaster alcali[116437]:    self.handle_request(listener, req, client, addr)
Feb 25 08:21:22 saltmaster alcali[116437]:  File "/opt/alcali/.venv/lib/python3.6/site-packages/gunicorn/workers/sync.py", line 184, in handle_request
Feb 25 08:21:22 saltmaster alcali[116437]:    self.log.access(resp, req, environ, request_time)
Feb 25 08:21:22 saltmaster alcali[116437]:  File "/opt/alcali/.venv/lib/python3.6/site-packages/gunicorn/glogging.py", line 344, in access
Feb 25 08:21:22 saltmaster alcali[116437]:    self.access_log.info(self.cfg.access_log_format, safe_atoms)

Seems to die right after POST /api/token and the successful ldap check.

Digging much deeper, it seems that login calls validate() in MyTokenObtainPairSerializer from api/serializers.
This reaches into /opt/alcali/.venv/lib/python3.6/site-packages/rest_framework_simplejwt/serializers.py to perform auth against the backend.  With debug added, I pass all of the steps here.
This serializers.py calls out to authenticate() in /opt/alcali/.venv/lib/python3.6/site-packages/django/contrib/auth/init.py
I can get debug out of all of the above, but the original validate function exits:
class MyTokenObtainPairSerializer(TokenObtainPairSerializer):     def validate(self, attrs):         # /opt/alcali/.venv/lib/python3.6/site-packages/django/contrib/auth/__init__.py         # /opt/alcali/.venv/lib/python3.6/site-packages/rest_framework_simplejwt/serializers.py         print("Calling validate for:")         print(attrs)         data = super().validate(attrs)         print("Validated user " + self.user.username) <--------------not printed         print(type(data)) <--------------- not printed         refresh = self.get_token(self.user)         data["refresh"] = str(refresh)

Turns out I am running into this instead upon successful login:
File "/opt/alcali/.venv/lib/python3.6/site-packages/rest_framework_simplejwt/backends.py", line 43, in encode
return token.decode('utf-8')
AttributeError: 'str' object has no attribute 'decode'

python         if (type(token) == str):             return token         return token.decode('utf-8')

https://github.com/latenighttales/alcali/issues/340


The Demo React SSR with renderToPipeableStream may not work as expected.

Just follow the tutorial. Run the command nohup wasmedge --dir .:. /path/to/wasmedge_quickjs.wasm dist/main.mjs,  and then try to visit "http://localhost:8001". Request pending and nothing returns.
But It's worked fine with command node dist/main.mjs.

@ahayaSKBINCN hi, please tell us what your platform is？

@ahayaSKBINCN hi, please tell us what your platform is？And wasmedge version.

I'm sorry for the late reply. The platform of my computer is MacOS, M1 chip.  And the wasmedge version is v0.11.1.

Facing the same issue with wasmedge 0.12.1 , MacOS Ventura 13.3.1 and x86 chip.

@DarumaDocker Can you have a look? Thanks.


Webhook: cert-manager and FlowSchema

Describe the bug:
I am the maintainer of a cert-manager webhook (OVH). The webhook uses this example https://github.com/cert-manager/webhook-example but due to dependencies on the cert-manager part, there's a repeating error message.
W0307 09:07:34.843358       1 reflector.go:424] pkg/mod/k8s.io/client-go@v0.26.2/tools/cache/reflector.go:169: failed to list *v1beta3.FlowSchema: the server could not find the requested resource
E0307 09:07:34.843372       1 reflector.go:140] pkg/mod/k8s.io/client-go@v0.26.2/tools/cache/reflector.go:169: Failed to watch *v1beta3.FlowSchema: failed to list *v1beta3.FlowSchema: the server could not find the requested resource
W0307 09:07:46.213292       1 reflector.go:424] pkg/mod/k8s.io/client-go@v0.26.2/tools/cache/reflector.go:169: failed to list *v1beta3.PriorityLevelConfiguration: the server could not find the requested resource
E0307 09:07:46.213356       1 reflector.go:140] pkg/mod/k8s.io/client-go@v0.26.2/tools/cache/reflector.go:169: Failed to watch *v1beta3.PriorityLevelConfiguration: failed to list *v1beta3.PriorityLevelConfiguration: the server could not find the requested resource

The issue has been describe in https://github.com/cert-manager/webhook-example/issues/48 by others and myself but since dependencies are related to cert-manager packages, I'm raising an issue here in hope someone will provide some guidance or a resolution to https://github.com/cert-manager/webhook-example/issues/48
Expected behaviour:
The FlowSchema error doesn't appear due to dependencies on<EMAIL_ADDRESS>
Steps to reproduce the bug:

Build the cert-manager webhook
Deploy the webhook
Check the deployment logs

Anything else we need to know?:
Just for clarity, I don't think this is a direct bug in cert-manager, but since the webhook relies on cert-manager dependencies, this may be the right place to raise this issue.
Environment details::

Kubernetes version: 1.25
Cloud-provider/provisioner: bare-metal
cert-manager version: 1.10
Install method: helm

/kind bug

Hi, thanks for opening the issue and detailed description.
The errors seem to come from API Priority and Fairness controller that runs as part of the extension apiserver set up here.
I cannot reproduce the issue though (tried with Kubernetes 1.25 and 1.23 and cert-manager 1.11, but I only see some RBAC errors).
Could you please confirm whether you've disabled API Priority or Fairness in your main kube apiserver/disabled beta features or set any APF related flags and whether the cert-manager version (1.10) in the issue description refer to the cert-manager libraries pulled in by your webhook as well?
I'm wondering if there is any value in running the APF controller in the webhook apiserver at all, x-post https://kubernetes.slack.com/archives/C0EG7JC6T/p1684407032236939

I think a long term solution would be to make the extension apiserver configurable by default.
A short term solution for this specific problem could be to just switch the APF controller off.


[BUG] Cannot .launch Executable In latest KDE Plasma versions

Describe the bug
Attempting to launch an executable directly or launching a .desktop on KDE has unanticipated behavior/fails.
To Reproduce
Steps to reproduce the behavior with a minimum self-contained file.

Create a file main.py with:

app = typer.Typer()

@app.command()
def launch_exec():
    typer.launch("/usr/bin/ls")

@app.command()
def launch_desktop():
    typer.launch("ls.desktop")

if __name__ == "__main__":
    app()

Create a file ls.desktop in the same directory with:

[Desktop Entry]

# The type as listed above
Type=Application

# The version of the desktop entry specification to which this file complies
Version=1.0

# The name of the application
Name=ls

# A comment which can/will be used as a tooltip
Comment=run ls

# The path to the folder in which the executable is run
Path=/

# The executable of the application, possibly with arguments.
Exec=/usr/bin/ls

# Describes whether this application needs to be run in a terminal or not
Terminal=true

To test with an executable:

python main.py launch-exec

launch-exec outputs (See screenshots):

A new window displays the following information:
Error -- KIO Client
For security reasons, launching executables is not allowed in this context.

But I expected it to output:

It should request permission to launch if permission is required (or it should launch)

To test with a desktop file:

python main.py launch-desktop

launch-desktop outputs (See screenshots):

It opens the desktop file in KATE (KDEs text editor) for editing.

But I expected it to output:

It should request permission to launch if permission is required (or it should launch)
Expected behavior
When attempting to launch an application, the application should launch or request permissions to launch
Screenshots
for an executable

for a .desktop file

Environment

OS: Linux, Manjaro KDE Edition

Typer Version 0.3.2

Python version: Python 3.9.1

Additional context
I did a bit of digging into this.
The launch function is recreated in Typer but this issue can be recreated in Click (Not surprising since it's almost identical).
On the backend these both use [xdg-open](https://wiki.archlinux.org/index.php/User:Larivact/xdg-open#:~:text=xdg-open is a desktop,file-opener application (eg.) which is a DE agnostic tool for opening files and URLs. Inside a DE this will pass along to the running DE's file opener app.
Here is the default application that will be used to open an executable or a desktop file on my KDE environment.
# An executable file type does not appear to have a default.
(cdda-manager) ~/r/b/cdda-manager ❯❯❯ xdg-mime query default "application/x-executable"
# The default for desktop files is to open in kate, which is the behavior we observed.
(cdda-manager) ~/r/b/cdda-manager ❯❯❯ xdg-mime query default "application/x-desktop"                                                                                ✘ 4
org.kde.kate.desktop
(cdda-manager) ~/r/b/cdda-manager ❯❯❯

One way that does seem to work to open these files from a terminal is this:
# Both of these execute properly without even requesting a user prompt.
kioclient5 exec ls.desktop
kioclient5 exec /usr/bin/ls

Also of note is that you DO see the expected behavior in Dolphin (KDEs File Manager) if you double click an executable or desktop file as it appears to have it's own settings for how to handle executables (or .desktop files which it consider executable as well)
See attached screenshots.

Working examples
Here are some working examples for getting execs and desktops to run.
@app.command()
def working_exec1():
    typer.echo("I'm running the executable directly")
    c = subprocess.Popen(["/usr/bin/ls"])
    return 0

@app.command()
def working_exec2():
    typer.echo("I'm running the executable via kioclient")
    c = subprocess.Popen(["/usr/bin/kioclient5", "exec", "/usr/bin/ls"])
    return 0

@app.command()
def working_desktop1():
    typer.echo("I'm opening the .desktop via kioclient")
    c = subprocess.Popen(["/usr/bin/kioclient5", "exec", "ls.desktop"])
    return 0

@app.command()
def working_desktop2():
    typer.echo("I'm opening the .desktop via dolphin")
    c = subprocess.Popen(["dolphin", "+", FULLPATHTOFILE])
    return 0

This was posted accidentally adding additional information.

Wanted to note that I'm still facing this issue as of today on the latest version of Endeavor OS

From what I understand you're facing an issue with KDE and how it handles programs and permissions, not with Typer itself, right? I suspect the right place to ask would be on KDE. There's probably something you have to do to your program or to Python in general to make KDE accept that interaction.


Can not inject ngMap: Error: $injector:unpr Unknown Provider

Hello,
I'm following the get started but get the error when inject.
// index.html
script src="assets/js/ng-map.min.js"
script src="http://maps.google.com/maps/api/js"
// app.js
app = angular.module('app', ['ngRoute', 'ngResource', 'ngMessages', 'ngMaterial', 'ngMap']);
// controller
angular.module('app').controller('myController', myController);
myController.$inject = ['$scope', '$mdToast', 'ngApp'];
function ContactEditController($scope, $mdToast, NgApp) {...}
The map was show as normal, but whenever i inject ngMap to use in controller it fire an error like this:
Error: [$injector:unpr] http://errors.angularjs.org/1.4.8/$injector/unpr?p0=ngMapProvider <- ngMap.
Error: $injector:unpr
Unknown Provider
Unknown provider: ngMapProvider <- ngMap
Could you help to give a hint?

Try NgMap instead of ngMap.

try to use these scripts

I've got the same error and solved it updating the library to the last version.
Also, in my case, I needed to run bower remove ngmap and bower install ngmap to get the very last version.

Here is a
https://plnkr.co/edit/TlpqI3k4haGCPEFz4mQR?p=preview

@browniebroke This works, https://plnkr.co/edit/iYYRyr?p=preview

Great, thanks for the fast response ! :+1:

+1 I also faced the same issue. Changing it to NgMap when injecting in controller fixed it.
Don't know why there's this inconsistency when injecting in module vs controller.
Thanks,
Anirudha


feat: add image optimization using api

Description

This PR is about optimizing the github user's profile image using image optimizing api
This api is used to convert png image to webp during load time, an png image may size more than 150kb but after conversion to webp the same image may size more than 40kb
We don't need to fear about the failure of api, because it can cache the image for more than 2 days so once an image the next it will immediately serve the process imaged within seconds
This is way more important to increase the site performance by decreasing the consuming bandwidth
This PR closes the ISSUE: #304

Pull Request Checklist

[x] The app is working correctly after applying the changes.
[x] The app is responsive and displays properly on different screen sizes and devices.
[x] The changes are thoroughly documented in the appropriate files or documentation sections.
[x] The coding conventions and style guidelines are followed consistently.

Checklist

[x] I have tested the changes locally and ensured that the app is functioning as expected.
[x] I have verified that the app is responsive and looks good on various screen sizes and devices.
[x] I have reviewed the documentation and ensured that it accurately reflects the changes made.
[x] I have followed the coding conventions and style guidelines defined in the project.
[x] I have run the automated tests and all of them pass.
[x] I have considered the performance implications of the changes made.
[x] I have considered the security implications of the changes made.
[x] I have checked for any potential regressions or side effects caused by the changes.
[x] I have performed a self-review of my code.
[x] Starred the repository.
[x] Added My Name to the Contributor's List.
[x] Followed the repository's Contributing Guidelines.
[x] I ran the app and tested it locally to verify that it works as expected.

Reviewers
@tusharbansal22 , @AmanJain18

@anmode , The image from the Github API is served in PNG format which it can sizes about 150kb and greater but our idea is to optimize the image by converting it during the load itself using an Third party API
The converted webp only sizes lesser than 50kb, the optimized image is cached on api's server once requested so the next time we request it can serve the image without any further process.
The Image Optimization API is well suited for this scenarios, can convert any images into webp, this is a next-generation format introduced by Google.
Nowdays we want to optimize the resources in every aspect to improve the site's performance 🚀

Ohh okay got that. Thanks  @0xabdulkhalid

As per request of @AmanJain18 I have edited on another stuff on this same issue.
This edit is about fixing the improper transition of Profile card image during 'hover'
For some reason i can't add the video footage of before and after

Ya i got it but still it can't be merged...

@AmanJain18 Can i know why ?


[SPIKE][DO NOT MERGE] Create IE-compatible configuration merging function

Spike to see if I could make a version of the configuration merging function that works back to IE8.
This exists as a helper function (getModuleConfig) that takes an unlimited number of configuration objects, flattens them into a non-nested list of key-value pairs, and merges them into one object. Objects are merged from left to right, in the order they're passed into the function (aka, the last object has the highest priority).
Also includes a polyfill for Element.prototype.dataset. This is derived from the one formerly served by polyfill.io, with some modifications:

The regex used to detect data-* attributes has been altered so that it catches attributes containing full stops (.), as we're using these to namespace i18n attributes.
A feature detection for Object.prototype.__defineGetter__ and Object.prototype.__defineSetter__ has been added, as these are required by the polyfill but unavailable in IE8, and the existing Object.defineProperty polyfill doesn't cover these. Instead, a (very hacky) fallback is used if these aren't detected.

Seeing it in action
The Accordion component has been modified to accept i18n options and to use the helper function. Currently, no visual changes take place and the resulting configuration object is logged in the console instead (in a way that IE8 won't fall over on!)
A new example has been added to the review app, 'Accordion with localisations', which overrides the i18n.showSection and i18n.showAllSections strings, exhibiting how the default configuration and data-* attribute configuration options get merged and flattened.
If all is working as expected, you should get this logged to the console on every browser:
Accordion configuration:
    i18n.hideAllSections: Hide all sections
    i18n.hideSection: Hide this section
    i18n.showAllSections: Show everything
    i18n.showSection: Show something
    module: govuk-accordion

The team seems to be happy with the state of this. Will close this PR for now. This branch could serve as a starting point for #2699 when we get to that.


Road to 2.0 - New Challenger Potion needs to consider Sacred Bark

Ok, this potion does not need to be affected by Sacred Bark 👍
Issue done, without a single line of code.

Ok, this potion does not need to be affected by Sacred Bark 👍
Issue done, without a single line of code.

Nope. Let's make it affect Challenger Coin, just make the first potion used give an extra one if Sacred Bark is available.

Ok, Gk gave a better idea.
Instead of giving an extra potion, make the potion transform the room into an Burning Elite room and make the fight give one extra Relic.

Ok, JohnnyBazooka89#8560 gave his code, and it helps, but I have some hiccups in terms of how things work for his code and mine:

His code works by adding a field to rooms and calling functions to make the Emerald Elite rooms work. My code works changing the rooms and calling code after the load to redo the necessary changes.

His code has a relic to ask when the need of a Emerald Elite room is needed, I don't, all that I can count is save and load.

My code needs to give an extra relic.

So, I will have to reread my code to see where my patches will have to diverge from his.

Ok, good news. It seems that I had already counted with EmeraldElite because of Act 3 and the need of the Emerald Key, so making the relic spawn Emerald Elits won't be much of an issue.
My problem is with adding an extra relic.

Ok, if I got this right, the game does not expect that more than one Burning Elite room would show up.
So, I will have to use code that reference stuff from New Challenger Potion to apply the effects AND the extra relic. Will still need to use Bazooka's code.

Also I'm giving the wrong key -_-

Ok, I still have to commit the new code, but it's not working: for some reason, the function to add the buffs is not adding the buffs? Have to check how it is working.

I referenced the issues on the wrong place. Again. sigh
Very well. I have two options at this point.
1 - I manage to piggyback on a function of AbstractRoom to apply the effects when they should be applied, ignoring the whole Emerald Room dillema, and leave the burning not working on all modes;
2 - Patching.
Will try 1, if that does not work, 2.

Also, have to make sure that everything works even without an Emerald Key being given thanks to a specific chosen mode.

WITHOUT A PATCH O/


fix: update typescript and loaders

:shipit:

@sumitarora How about this one? If you'd like me to merge them, just let me know :) Thanks!


Screenshot & screen recorder

Description of the new feature / enhancement
Screenshot & screen recorder feature.
Scenario when this would be used?
take screenshot or record screen.
Supporting information
No response

For screenshot you can use Snipping Tool (preinstalled on Windows 11 and downloadable from Windows 10)
For screen recording you could use the XBox gamebar (win+g).


Difference unique visitors in HTML and in Console

I'm having a slight issue with the unique visitors, as the count in the console and in the report.html file is hugely different.
In the first picture (console) you can see that the count is 38

While in the html file it is at 1365

Does anyone know why there is such a huge discrepancy?

Good question. I see the T.X amount is 0 on the TUI. Are you sure you are using the right format on both sides? also, was the log exactly the same in both runs? (diff  # of total requests is different).

It absolutely was the format problem! If they are the same, the output is identical again. Thanks for your time


Make Scrollables restorable

Description
Enabled the Scrollable widgets (ListView, GridView, PageView, etc) to restore their scroll offset with the state restoration framework.
The change is slightly awkward because Scrollables already had a restoration mechanism with PageStorage. The plan is to eventually replace that mechanism with the new state restoration approach once state restoration supports keeping state information around even when the owning widget is getting disposed.
Related Issues
https://github.com/flutter/flutter/issues/62916
Tests
I added the following tests:

Added for all migrated Scrollables and the new RestorationManager.flushData method.

Checklist
Before you create this PR, confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: https://flutter.dev/go/breaking-changes-template Replace this with a link to a pull request that adds the migration guide to https://flutter.dev/docs/release/breaking-changes

LGTM modulo the async logic in the restoration code

I removed the async logic in the restoration code. It's not necessary and was just leftover from a different approach.

I'll intend to land this when the tree goes green unless I hear otherwise.


add ofppt-edu.ma domain (morocco)

The school official website URL : https://www.ofppt.ma/
add ofppt-edu.ma domain .
L'Office de la formation professionnelle et de promotion du travail. (OFPPT)
Office of Vocational Training and Employment Promotion. (OFPPT)

@Bilalmesk I am sorry, the domain ofppt-edu.ma is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Release 2017-08-09

Coverage increased (+1.3%) to 82.544% when pulling 694f29e78a5612f19c006a3c7fadd098af8b6780 on develop into 5a528a025f253845e5f73811efa81e9ee6192aee on master.

Coverage increased (+1.3%) to 82.544% when pulling 694f29e78a5612f19c006a3c7fadd098af8b6780 on develop into 5a528a025f253845e5f73811efa81e9ee6192aee on master.

Coverage increased (+1.3%) to 82.544% when pulling 694f29e78a5612f19c006a3c7fadd098af8b6780 on develop into 5a528a025f253845e5f73811efa81e9ee6192aee on master.


When using websocket_client.close() the worker never terminates

This is what I have in my test. (stop_client being called in tearDown)
    def start_client(self, path='', **kwds):
        client = connect('ws://localhost:8765/' + path, **kwds)
        self.client = self.loop.run_until_complete(client)

    def stop_client(self):
        self.client.close()
        print("$ Wait for client to terminate")
        self.loop.run_until_complete(self.client.worker)
        print("$ Client stopped")

We have verified with @ametaireau that it works with a Javascript Websocket client so the server is not incriminated.

The correct way to do so is actually to call close in the loop:
def stop_client(self):
    self.loop.run_until_complete(self.client.close())

Indeed, it's a coroutine.


request.scheme with a proxy like nginx

I think the best solution is used Nginx in front and sanic in back (and maybee Gunicorn in middle but I don't understand interest for the moment).
When I test with Nginx, the value for "request.scheme" is not the real value in front but the value for sanic server.
I can find the correct value with for exemple "request.headers['x-forwarded-proto']".
So, I think it's better to specify in the documentation, and/or create a other param for "remote_scheme" for example, because I think it will be the main configuration in production (nginx + sanic).
Sorry for my english ...

@patic-fr so are you using https in nginx, but sanic is seeing http? It would be useful to have a few more details. Maybe you need to add proxy_set_header X-Forwarded-Proto $scheme; in your nginx location block?

Sorry, it's a problem of understanding.
I added "proxy_set_header X-Forwarded-Proto ..." in Nginx of course.
But, I think novice people like me who could trust the "request.scheme" property without thinking more.
I am working on designing authentication and I want to apply all good security practices.
One of them is to force the connection on HTTPS and deny the HTTP. I looked at the extension flask-sslify for ideas.
Of course, I can force redirection in the configuration of Nginx, but I prefer to control this more at the application level.
I think it would be nice to specify in the documentation that the "request.scheme" property does not return the original schema behind a proxy but just the schema of the connection between the proxy and Sanic.
So I think we should just improve the documentation on this point to be more explicit for novice.

For help you, there is two places where we can add informations :

API Reference

Just before "scheme" there is "remote_add" where I can read "Attempt to return the original client ip based on X-Forwarded-For. Returns: original client ip." Perhaps, you could add the same thing for scheme with remote_scheme in the API.
In the documentation, we could write for exemple : "remark : behind a proxy, it's not the original schema. You can try to use request.headers.get('x-forwarded-proto', 'http'), it depends proxy server and configuration."

Request Data

same thing
note : be careful my english, perhaps correction need

@stale nope

This seems to be already addressed in current git master. I suggest closing.


How to format time for simple output

How to format and print as follows:
09-24 08:03:57 DEBUG msg="Info message"
09-24 08:03:58 INFO  msg="Info message"
09-24 08:03:59 ERROR msg="Info message"

thanks

as: https://stackoverflow.com/questions/77422213/how-to-hide-all-keys-when-using-slog-in-golang

You can use slog.TextHandler from the stdlib or https://github.com/lmittmann/tint


Ruby 3.3.0 error

Hello
Trying to update my Rails app to run with Ruby 3.3.0 and getting this loading error:
<internal:~/.asdf/installs/ruby/3.3.0/lib/ruby/3.3.0/rubygems/core_ext/kernel_require.rb>:37:in `require': cannot load such file -- premailer-rails (LoadError)
Did you mean?  premailer/rails
	from <internal:~/.asdf/installs/ruby/3.3.0/lib/ruby/3.3.0/rubygems/core_ext/kernel_require.rb>:37:in `require'
	from ~/.asdf/installs/ruby/3.3.0/lib/ruby/gems/3.3.0/gems/bootsnap-1.17.0/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:17:in `require'
	from ~/.asdf/installs/ruby/3.3.0/lib/ruby/gems/3.3.0/gems/bundler-2.4.7/lib/bundler/runtime.rb:60:in `block (2 levels) in require'
	from ~/.asdf/installs/ruby/3.3.0/lib/ruby/gems/3.3.0/gems/bundler-2.4.7/lib/bundler/runtime.rb:55:in `each'
	from ~/.asdf/installs/ruby/3.3.0/lib/ruby/gems/3.3.0/gems/bundler-2.4.7/lib/bundler/runtime.rb:55:in `block in require'
	from ~/.asdf/installs/ruby/3.3.0/lib/ruby/gems/3.3.0/gems/bundler-2.4.7/lib/bundler/runtime.rb:44:in `each'
	from ~/.asdf/installs/ruby/3.3.0/lib/ruby/gems/3.3.0/gems/bundler-2.4.7/lib/bundler/runtime.rb:44:in `require'
	from ~/.asdf/installs/ruby/3.3.0/lib/ruby/gems/3.3.0/gems/bundler-2.4.7/lib/bundler.rb:195:in `require'
	from ~/programming/rails/blinka/config/application.rb:21:in `<main>'
	from <internal:~/.asdf/installs/ruby/3.3.0/lib/ruby/3.3.0/rubygems/core_ext/kernel_require.rb>:37:in `require'
	from <internal:~/.asdf/installs/ruby/3.3.0/lib/ruby/3.3.0/rubygems/core_ext/kernel_require.rb>:37:in `require'
	from ~/.asdf/installs/ruby/3.3.0/lib/ruby/gems/3.3.0/gems/bootsnap-1.17.0/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:32:in `require'
	from ~/.asdf/installs/ruby/3.3.0/lib/ruby/gems/3.3.0/gems/railties-7.1.2/lib/rails/commands/server/server_command.rb:139:in `block in perform'
	from <internal:kernel>:90:in `tap'
	from ~/.asdf/installs/ruby/3.3.0/lib/ruby/gems/3.3.0/gems/railties-7.1.2/lib/rails/commands/server/server_command.rb:136:in `perform'
	from ~/.asdf/installs/ruby/3.3.0/lib/ruby/gems/3.3.0/gems/thor-1.3.0/lib/thor/command.rb:28:in `run'
	from ~/.asdf/installs/ruby/3.3.0/lib/ruby/gems/3.3.0/gems/thor-1.3.0/lib/thor/invocation.rb:127:in `invoke_command'
	from ~/.asdf/installs/ruby/3.3.0/lib/ruby/gems/3.3.0/gems/railties-7.1.2/lib/rails/command/base.rb:178:in `invoke_command'
	from ~/.asdf/installs/ruby/3.3.0/lib/ruby/gems/3.3.0/gems/thor-1.3.0/lib/thor.rb:527:in `dispatch'
	from ~/.asdf/installs/ruby/3.3.0/lib/ruby/gems/3.3.0/gems/railties-7.1.2/lib/rails/command/base.rb:73:in `perform'
	from ~/.asdf/installs/ruby/3.3.0/lib/ruby/gems/3.3.0/gems/railties-7.1.2/lib/rails/command.rb:71:in `block in invoke'
	from ~/.asdf/installs/ruby/3.3.0/lib/ruby/gems/3.3.0/gems/railties-7.1.2/lib/rails/command.rb:149:in `with_argv'
	from ~/.asdf/installs/ruby/3.3.0/lib/ruby/gems/3.3.0/gems/railties-7.1.2/lib/rails/command.rb:69:in `invoke'
	from ~/.asdf/installs/ruby/3.3.0/lib/ruby/gems/3.3.0/gems/railties-7.1.2/lib/rails/commands.rb:18:in `<main>'
	from <internal:~/.asdf/installs/ruby/3.3.0/lib/ruby/3.3.0/rubygems/core_ext/kernel_require.rb>:37:in `require'
	from <internal:~/.asdf/installs/ruby/3.3.0/lib/ruby/3.3.0/rubygems/core_ext/kernel_require.rb>:37:in `require'
	from ~/.asdf/installs/ruby/3.3.0/lib/ruby/gems/3.3.0/gems/bootsnap-1.17.0/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:32:in `require'
	from bin/rails:4:in `<main>'

Any idea why it cannot load? Something with naming?

I think Ruby might have translated require "foo-bar" to require "foo/bar" in the past.
You're supposed to require "premailer/rails" for this gem. There is no file lib/premailer-rails.rb in this gem, but there is a file lib/premailer/rails.rb.
See also the RubyGems guide on naming.
Does that fix your problem?

There is no mention in the README of doing any require and it has worked for previous ruby versions.
So this is something that has changed in Ruby 3.3.
Not sure how to fix it in a way that works with Zeitwerk.
Will try to add Ruby 3.3 to the test matrix tomorrow and see if I can find anything.

It failed because my version of css_parser was not updated to a Ruby 3.3-compliant version.
I could find this error when I added a manual require premailer/rails in my initializer instead.


Local development setup

How do I integrate this into my local development setup? I couldn't find any documentation or examples on this. Do I have to specify a value like localhost:3000 for the domain?
Thank you.

By default, the image component will be pointed to it's origin. (being described here).
But if you want to change it's default behavior and point to somewhere else, you can create your own loader function to change resolve url to your target server.

Hello, how's it going? Is you issue resolved?

Hey Riffy! Thanks a bunch for checking in. I haven't had a moment to test it out yet, but I'm planning to give it a go in my local environment this weekend. 😊 I'll keep you updated on the progress!

closed as stale


How to install it in debian server?

How to install it in debian server，is there any detailed instruction?

I'm very sorry, I haven't used a Debian server.
However, I think you can start by using npm run build to package the project into the dist  folder. Then deploy it via nginx. (Because that's how I did it on my centos server).

In this project, I used a proxy service to accomplish the functionality needed for function calling. I can open-source it if you want. 🤝

thank you very much,i will try

Thank you for using it!
I have open-sourced the proxy service: https://github.com/Caojiahao-Coder/gptnext_proxy

This is a Nuxt project that you can deploy using Vercel, but you need to register an Api Token with OpenWeather API that has Function Calling enabled.


Change Start to Start<?> ?

Is there a better way to get the generics working? Maybe use a wildcard ? instead of Void?

Semantics matter. Void implies that there is no parent while ? means there might be a parent.

Since the only valid Start class for a descriptor is the main entry, which is by default Start<Void>, it can make sense to push the generic back down, but only for the highest level block. For internal use we need a marker interface which preserves the return type even for the descriptor block.
Every block can have a Head<T> interface, but only the descriptor block can have Start extends Head<Void>.


HA-proxy incubator/haproxy not working

issue with the helm chart
[ALERT] 248/171348 (31) : Starting proxy error503noendpoints: cannot bind socket [<IP_ADDRESS>:8181]

Same issue
I1024 20:38:00.348766       6 controller.go:209] HAProxy[pid=17] output:
[ALERT] 296/203800 (21) : Starting proxy error503noendpoints: cannot bind socket [<IP_ADDRESS>:8181]
E1024 20:38:00.348978       6 controller.go:337] unexpected failure restarting the backend:
exit status 1


add message tests

Codecov Report

Merging #8 into master will increase coverage by 10.26%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           master       #8       +/-   ##
===========================================
+ Coverage   21.62%   31.88%   +10.26%
===========================================
  Files           3        3
  Lines          74       69        -5
===========================================
+ Hits           16       22        +6
+ Misses         53       42       -11
  Partials        5        5

Impacted Files
Coverage Δ

pkg/server/message.go
100% <100%> (+100%)
: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 2390e68...a90e9f3. Read the comment docs.


Please invite me to the GitHub Community Organization

Name
Subhampreet
Discord Username (if applicable)
Subhampreet#1760
Additional Context
I am an Open-Source Enthusiast, and I absolutely love contributing to open-source projects. I am looking forward to learning more about open-source from you and grow with the other members of this awesome community!

You have received an email to join, or press this link - https://github.com/Jaidevstudio - , and you'll see a banner, for you to join the organisation! 🎉
You can also show that you are part of this org, by making yourself public. This will show our profile org on your profile:
Follow This:

After you join the organisation, search your name in the list of people. 🔍
Follow this below: ⬇

If you need anything- help or support - please reopen this issue. Thank you.


Base64 encode SVG

Web Page Performance Test for https://guitar-tuner.appspot.com
From: Dulles, VA - Motorola G - Chrome - 3G
Before: First view Visually Complete: 9.913s
After: First view Visually Complete: 7.893s
Before: Repeat view Visually Complete: 4.087s
After: Repeat view Visually Complete: 4.142s
http://www.webpagetest.org/result/150703_AX_KJ8/1/details/cached/
http://www.webpagetest.org/result/150703_J6_M0S/1/details/cached/

Ah booooo. It has conflicts :(

Conflicts have been fixed @paullewis
Sorry for that!

(gentle ping)

@paullewis This should be good to go afaik. Might want to ask for commits to be squashed for a cleaner merge history, but otherwise should be fine.

Sorrrrrreeeee, total slip on my part.


When exporting the Wiki I expect that I can see almost the same in wiki as in the PDF

This is how it looks on my wiki

and this is the content in the PDF-file

I think the issue might be that the tool is not including the page names as headers in the output, thus making it much more readable
Content of .order is
0.-Introduction
1.-Onboarding
2.-Applications
3.-Architecture
4.-Object-and-data-model
5.-Deployment
9.-Obsolete-systems
and I would expect that as an index in the PDF
Best regards
Thorvald

actually this should be the case. when did you try it the last time?
Maybe try the --heading parameter.

It is long time since I worked on the project, where we used the tool, and I will probably no get access to the documentation again, so it is ok to close this, if you think it is working ok


Adding Lights Out Game, Resolving issue #280

Description 📜
This is a complex game designed on 5 * 5, 6 * 6 & 7 * 7 boards.
The basic idea behind the game is that initially some squares will be of color X and others will be of color Y, where X denotes light on and Y denotes light off.
You have to turn the light of all squares off or make them of color Y so as to say by flipping the tile, but the catch is that once you flip the tile to change its colo/light mod, not only its color change but the color of its adjacent neighbors in the X & Y direction also changes, so the complexity increases as you keep flipping the tile, neighbors also get changed.
In the end, player wins if all tiles are turned off, i.e. all neighbors of all tiles are turned off.
Please include a summary of the change and which issue is fixed.
Fixes #280

Type of change 📝
Please delete options that are not relevant.And in order to tick the check box just but x inside them for example [x] like this

[x] Add a new project
[ ] Modified or enhanced the existing projects
[ ] Improve the readme of any project

Domain of Contribution 📊
Please delete options that are not relevant.And in order to tick the check box just but x inside them for example [x] like this

[x] Project using only HTML & CSS
[x] Project using HTML, CSS & JS
[x] Project using ReactJs

Checklist ✅
Please delete options that are not relevant.And in order to tick the check box just but x inside them for example [x] like this

[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GSSoC'22 contributor

Screenshots / Gif (Optional) 📸
Add the screenshot of the changes you have made

@khushi-purwar can you please check this, I have done the game and it seems to be working fine, I have also added a woring video of the game in the public folder of the repository.

@khushi-purwar also, once you check this, please add a label to the PR for evaluation.

@khushi-purwar can you please check this once

You cannot add the django project in this repo. This repo is only for HTML, CSS , JS AND REACTJS. I am closing this PR as it is not valid.


make:tactician file generation issue

php artisan make:tactician /src/Commands/CommandName
creates the files in /app/CommandBus whereas I have clearly specified a root path.
Current Behavior
/app/CommandBus/Commands/CommandNameCommand
/app/CommandBus/Handlers/CommandNameHandler
Expected behavior
at leaset
/src/Commands/Commands/CommandNameCommand
/src/Commands/Handlers/CommandNameHandler
OR
/src/Commands/CommandNameCommand
/src/Commands/CommandNameHandler
Also, It would be nicer if there were a flag like --bare to skip creating Commands and 'Handlers' folders.

Sorry this took too long, I hope is still relevant but take a look at this, https://github.com/joselfonseca/laravel-tactician/commit/4a0f824ad4a5c992e40b365caa5f948c679b643c .Will tag it soon with support for laravel 6.