deployment fails with "context deadline exceeded"

In what area(s)?

/area administration

Expected Behavior
./build.sh -a apply works without errors
Actual Behavior
./build.sh -a apply
...
╷
│ Error: retrieving `contact` for KeyVault: keyvault.BaseClient#GetCertificateContacts: Failure sending request: StatusCode=0 -- Original Error: context deadline exceeded
│
│   with azurerm_key_vault.azhop,
│   on keyvault.tf line 6, in resource "azurerm_key_vault" "azhop":
│    6: resource "azurerm_key_vault" "azhop" {
│
╵
╷
│ Error: Error reading queue properties for AzureRM Storage Account "azhoplzxslx57": queues.Client#GetServiceProperties: Failure sending request: StatusCode=0 -- Original Error: context deadline exceeded
│
│   with azurerm_storage_account.azhop,
│   on main.tf line 80, in resource "azurerm_storage_account" "azhop":
│   80: resource "azurerm_storage_account" "azhop" {
│
╵

Perhaps this is related to the fact that I did not specify a value for the key_vault_readers (I did not know how)
key_vault_readers: #<object_id>

Steps to Reproduce the Problem
Follow deployment instructions on Ubuntu 20.04 for az-hop v1.0.9

After some digging, it seems that this error was caused by some outbound firewall rules on the VM I was using to deploy az-hop (no fault of az-hop).
From the above error messages this was not entirely straightforward to understand.
When I tried a second time from scratch I also got
╷
│ Error: Error waiting for Vault: (Name "kvlzxslx57" / Resource Group "azhop-dev1") to become available: Error connecting to "https://kvlzxslx57.vault.azure.net/": Get "https://kvlzxslx57.vault.azure.net/": EOF
│
│   with azurerm_key_vault.azhop,
│   on keyvault.tf line 6, in resource "azurerm_key_vault" "azhop":
│    6: resource "azurerm_key_vault" "azhop" {
│
╵

which made it easier to get to the root of the problem.


An error occurred: Failed to find Firefox cookie file

Hello, any idea about the error?
Thanks

Do you have Firefox installed? Have you tried with another browser (e.g. Chrome or the others supported by browser_cookie3, or perhaps with no browser specified per the README?

Yes i do have Firefox installed.
It's worked before just fine but now i get that error.
I will try with other browser cuz i have both edge and chrome and reply back.
I will no browser as well.

Closed due to lack of activity.


3.7: instance member function becomes instance member property in type references

TypeScript Version:  3.7.0-dev.20191002

Search Terms: instance member property
Code
declare global {
    class A {
        public f(): void;
    }
    class BA extends A {
        public f(): void;
    }
    class CA extends A {
        public f(): void;
    }
    class D extends CA {
        public f: BA['f']; // errors with 3.7
    }
}

export default undefined;

Expected behavior: use declarations without error
Actual behavior: errors with Class 'CA' defines instance member function 'f', but extended class 'D' defines it as instance member property.
Playground Link: https://www.typescriptlang.org/play/?ts=Nightly#code/CYUwxgNghgTiAEBzCB7ARlC8DeAoeB8kUAzifAII76G0AOArmhAJZjwBmAFAJQBc8AG4oWwANw0CAX0lFoZeACEqIAB4AXEADtg5Knlr0mrdt35CR42TNrEFAYRUbtuytUOFGzNp14DhohK0NoR25AAi8GqaOuSO7h7wXiacAsoA2gDkHJkAumLwAPSFUTAwKDDkAO4s6gAW8ADMAHQA7Na4MrhgKFokKBAgzaiIXJkAEiAQqAA08FUVEMAAhJk8ErhqdBXq8KAcUAwQuww6IBwsWiBWQA (compiles there without error)
Related Issues: -

I'm getting the same issue, namely when using react lifecycle methods that have an arrow function.
componentWillMount = () => {...}
/*Throws 'Class component'' Props<State, any>  defines instance member function componentDidMount, but extended class 'X ' defines it as an instance member property
*/
Error is fixed with:
componentWillMount() {....}
The problem with the workaround, is that to my knowledge it will change what is bound to context of 'this' under certain scenarios. Could someone please clarify on this?

In your case the this scope might change as arrow functions use the outer scope as they do not have their own.

This is still the case in 3.7.0-beta. Original issue: https://github.com/Microsoft/TypeScript/issues/17113.

Looks like this is fixed in nightly already, but I haven't tested it myself: https://github.com/microsoft/TypeScript/pull/34494

Confirmed. The issue is solved for me as well since 3.7.0-dev.20191016.


L-1384: POSTing new webhook should return expires_at by default

Small PR to return expires_at for new webhook POST requests.

I'm just gonna merge this since I tested it and it works fine :)


Hello, Clojurescript Track. Again!

@iHiD, I think I'm understanding now V2 (and before) and prepare the exercises. Also, my main goal is V3.
Would you check it out and tell me if I'm on the right way ?

This looks good now :)
Do you want to work out how to set up Continuous Integration for this repo? At the moment it looks like it's now working.

All exercises (nearly, all of the latests) comes from Clojure Track. I'll implement new test cases and rewrite the inherited code.

I'm almost bootstrap the Track.
@iHiD, I have questions;

What is the purpose of the UUID field in every exercise data in the Track config file ?
configlet does not work as we expected. I examine the shell code, I think it is OK. The Travis logs tell us about problem. I'll debug and place a solution, when I'm back home.
20 exercises are ready. Their test cases mostly comes from Clojure Track. I'll be finished it by the end of the week.

What is the purpose of the UUID field in every exercise data in the Track config file ?

This is what allows Exercism to uniquely identify an exercise. It means that if it's other properties change (such as its name), we know that we're refering to the same exercise. Once merged, a UUID can never change.

configlet does not work as we expected. I examine the shell code, I think it is OK. The Travis logs tell us about problem. I'll debug and place a solution, when I'm back home. If it's a known issue, can you throw me a solution ?

I'm not sure about this, but I've added #4 which maybe fixes it? If not, I'll ask on Slack for you and someone might know.

twenty exercises are ready. Their test cases mostly comes from Clojure Track. I'll implement new cases and change the old ones. I'll be finished it by the end of the week. Is there any problem the way it is ?

This sounds amazing 💙

I've added @ErikSchierboom to give this a check once he's back next week 🙁

I've added @ErikSchierboom to give this a check once he's back next week 🙁

There is no rush on me, I'll stick my plan. I want all of the exercises. 😇👨🏻💻

Wow, this looks amazing!

Thank you! 😇🙋🏻♂

The Babashka Track may be of reference, since I had to get rid of anything using JVM-specific features and doing so resulted in more portable code. So porting from Babashka to Clojurescript might even be easier.

The Babashka Track may be of reference, since I had to get rid of anything using JVM-specific features and doing so resulted in more portable code. So porting from Babashka to Clojurescript might even be easier.

Thank you. I'll consider it. 😇

Last two exercise to re-factor and then we are ready to Launch! 😍😇🙏🏻🥳👨🏻💻


Custom preset causes custom field to not show and existing field to show error

Describe the bug
I've added a custom preset as per the documentation inside the file config/ckeditor.ts:
export default {
  presets: {
    custom: {
      editorConfig: {
        toolbar: {
          items: ["heading", "|", "bold", "italic", "strikethrough", "link"],
        },
      },
    },
  },
};

When I run the build and start the development server, I see two issues:

When attempting to add a CKEditor field to an existing collection it doesn't show in the custom tab

The existing CKEditor field I have on one of my collection shows a blank, disabled input

To Reproduce
Outlined above
Expected behavior
I should see the CKEditor custom field with the preset "custom" as defined in my config.
please complete the following information:
config/plugins.ts
export default () => ({});
package.json
{
  "private": true,
  "scripts": {
    "build": "strapi build",
    "deploy": "strapi deploy",
    "develop": "strapi develop",
    "start": "strapi start",
    "strapi": "strapi"
  },
  "dependencies": {
    "@_sh/strapi-plugin-ckeditor": "4.0.6",
    "@strapi/email": "5.0.4",
    "@strapi/plugin-cloud": "5.0.4",
    "@strapi/plugin-color-picker": "5.0.4",
    "@strapi/plugin-users-permissions": "5.0.4",
    "@strapi/strapi": "5.0.4",
    "mysql2": "3.11.3",
    "react": "18.3.1",
    "react-dom": "18.3.1",
    "react-router-dom": "6.26.2",
    "sanitize-html": "2.13.1",
    "styled-components": "6.1.13",
    "zod": "3.23.8"
  },
  "devDependencies": {
    "@types/node": "22",
    "@types/react": "18",
    "@types/react-dom": "18",
    "@types/sanitize-html": "2.13.0",
    "typescript": "5"
  },
  "engines": {
    "node": ">=18.0.0 <=20.x.x",
    "npm": ">=6.0.0"
  },
  "strapi": {
    "uuid": "b1e9e956-72b1-45b2-9cf5-382fd4002568"
  }
}

node version
v20.16.0

Your configuration is wrong. Follow this example:
https://github.com/nshenderov/strapi-plugin-ckeditor/blob/9cecf62a13a78a890de680db980aaf47c1d5a674/README.md?plain=1#L159-L236

Looks like this issue is no longer relevant


proposal: Go 2: Error-Handling Paradigm with !err Grammar Sugar

Go Programming Experience
Intermediate
Other Languages Experience
VB, Pascal, C#, PHP, JS
Related Idea

[ ] Has this idea, or one like it, been proposed before?
[ ] Does this affect error handling?
[ ] Is this about generics?
[ ] Is this change backward compatible? Breaking the Go 1 compatibility guarantee is a large cost and requires a large benefit

Has this idea, or one like it, been proposed before?
No
Does this affect error handling?
No
Is this about generics?
No
Proposal
A more concise and comfortable reading and writing experience
Language Spec Changes
Add keywords
Informal Change
No response
Is this change backward compatible?
I don't know, perhaps it can be avoided.
Orthogonality: How does this change interact or overlap with existing features?
No response
Would this change make Go easier or harder to learn, and why?
No response
Cost Description
No response
Changes to Go ToolChain
No response
Performance Costs
No response
Prototype
This has nothing to do with me; I'm just putting forward an idea; Please don't attack me.
func RecvSend(ws *websocket.Conn) {
	fmt.Printf("recvSendServer %#v\n", ws)
	for {
		var buf string
		err := websocket.Message.Receive(ws, &buf)
		if err != nil {
			fmt.Println(err)
			break
		}
		fmt.Printf("recv:%q\n", buf)
		err = websocket.Message.Send(ws, buf)
		if err != nil {
			fmt.Println(err)
			break
		}
		fmt.Printf("send:%q\n", buf)
	}
	fmt.Println("recvSendServer finished")
}

###Add:  listen: @ok
// Demo 1

@ok {
	fmt.Printf("recv:%q\n", buf)
}!{
	fmt.Println(err)
	return
}

func RecvSend(ws *websocket.Conn) {
	listen: @ok

	fmt.Printf("recvSendServer %#v\n", ws)
	for {
		var buf string
		err := websocket.Message.Receive(ws, &buf).@ok{}.Message.Send(ws, buf).!@ok2 {
			fmt.Println("send err")
			break
		}
		fmt.Printf("send:%q\n", buf)
	}
	fmt.Println("recvSendServer finished")
}

// Demo 2

@ok {
	func Abc() {
		fmt.Printf("recv:%q\n", buf)
	}
}

func RecvSend(ws *websocket.Conn) {
	listen: @ok

	fmt.Printf("recvSendServer %#v\n", ws)
	for {
		var buf string
		err := websocket.Message.Receive(ws, &buf).@ok{ Abc() }.Message.Send(ws, buf).!@ok2 {
			fmt.Println("send err")
			break
		}
		fmt.Printf("send:%q\n", buf)
	}
	fmt.Println("recvSendServer finished")
}

// Demo 3

@ok {
	fmt.Printf("recv:%q\n", buf)
}!{
	return err, nil
}

func RecvSend(ws *websocket.Conn) {
	listen: @ok

	fmt.Printf("recvSendServer %#v\n", ws)
	for {
		var buf string
		err := websocket.Message.Receive(ws, &buf).@ok.Message.Send(ws, buf).!@ok2{
			fmt.Println("send err")
			break
		}
		fmt.Println("recvSendServer finished")
	}
}

I believe the feedback left in previous rounds remain valid, proposed syntax with no explanation of semantics or why they would be a good fit for Go is just a waste of time.


(400) Bad Request starting on 2/22

Hello,
I have 3 flows set up using the tool that have been running successfully for months.  All three started to fail on 2/22 with the error "Bad Request".  code: "0x80040265", message: "Something went wrong: The remote server returned an error: (400) Bad Request."
I checked all the setup and it looks to still be correct and valid.  When running the test for the custom connector, all the GET commands and the POST_GetReportParameters return normally without an error.  If I try to test the _POST_RenderReport, it is always returns an error with the same error message.  I've tested built in and custom reports that do not have parameters with the same result.  I've tried PDF, DOC, XLS for format and all give the same result.
Was there a change in CRM that resulted in breaking this?  Or do you have any tips that I could try to fix this?
Thanks!

I have the exact same Issue, All of my API calls work but the _POST_RenderReport request. I tried various rendering formats but still won´t work. Did you solved it MDrake?

I have not been able to resolve it.  Was the API working for you before or are you new to using it?

I’m new, I saw the previous issues on github and last year some user posted the same, maybe they’ll fix the bug, for now I”ll create my flows and leave them disactive.
If I ever find a solution or it starts working again, I’ll let you know
Scarica Outlook per iOShttps://aka.ms/o0ukef

Da: MDrake3 @.>
Inviato: Friday, March 12, 2021 3:36:17 PM
A: bguidinger/Xrm.ReportScheduler @.>
Cc: Miguel Darío Magaña Heredia @.>; Comment @.>
Oggetto: Re: [bguidinger/Xrm.ReportScheduler] (400) Bad Request starting on 2/22 (#58)
I have not been able to resolve it. Was the API working for you before or are you new to using it?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/bguidinger/Xrm.ReportScheduler/issues/58#issuecomment-797769196, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AD7GIPP3452WP4EJ5PL4UQTTDJ3MDANCNFSM4YZ4V3JQ.

I have the same problem. This is the first time I have set this up. Everything works except for _POST_RenderReport. Hope this gets resolved because this would be an extremely useful tool for me.

Hey guys, I fixed the problem, first I disabled the MFA(multifactor authentication) on the Office Admin Portal -> Users -> Active Users -> Multifactor Authentication -> Disable.
And I checked again the user/password on the Solutions configuration on D365. Advanced Settings -> Solutions -> Render Report(or whatever name you gave it) -> Configuration -> I set a user and password of my organization with admin permissions and that user is also the one creating the Flows.
Once I changed this and Published all Customizations my Flow worked, I´m still having troubles with the PDF rendering(its sending a huge amount of characters instead of a PDF file) but it works perfectly with HTML5.
Let me know if this helps you

I was able to do this too, however, now I realize that I can't use this if MFA is on, which is a deal breaker for me. I need to have MFA turned on for the user. Unless there is a workaround, I'll have to find another solution.


Request for Flash Logging feature

Problem
It's very difficult to connect UART/Jlink to get logs in-field devices.
and some of my devices getting crash after 3 to 4 days but I get no log to debug the issue.
Solution
I am looking for a feature where I can store the normal/crash logs in flash.
Similar to  Nordic_SDK_Flash_Logging_feature
Please give some pointers So I can try it.

See the additional information in #26295

@rhardik I missed this issue in my search. As Carles pointed ut, my recent #26295 duplicates this issue. I have taken the liberty of posting the details from #26295 below:
Is your enhancement proposal related to a problem? Please describe.
All log mechanisms today requires the user/developer to have either RTT/Serial/SWO connected to the board to see log output. Also, if you desire to store logs, this has to be done on the host side of the log link.
In most cases this is fine, but in some situations it would be highly desirable to place logs in on-board persistent storage:
A: A customer experiences an error, but did not have a backend connected. With this solution the log can be retrieved using a debugger from internal flash, USB from external flash or an SD card can be read over USB or put into a PC.
B: Logging from small portable devices. In many cases you have a portable device you want logging where it is cumbersome or impossible to have USB/UART/RTT connected.
E.g. logging battery level over a long period of time.
Describe the solution you'd like
LOG_BACKEND_PERSISTENT_STORAGE or
LOG_BACKEND_EXT_FLASH
LOG_BACKEND_SD_CARD
(LOG_BACKEND_INT_FLASH)
If the corresponding KConfig settings have not been made, you will get an error.
You will of course also have to set a file system, have a filename for the log and some kind of full storage handling.
Alternatively
As a first step, provide a LOG_BACKEND_USER or LOG_BACKEND_BLANK. When enabled, logs will e.g. go to UART and the USER backend at the same time. The user can then handle logs in the USER log callback as desired and the specifics of storing to flash or SD card can be left to the user.

I have submitted #28892 which implements one version of this. But due to the different nature of persistent flash logs versus normal ephemeral logs, I think we want to introduce some new APIs. Feel free to join the discussion in the PR.

https://github.com/zephyrproject-rtos/zephyr/pull/32973/commits/8e05a9bb0fa3e921370e753815f99722fc8ec2d9
Have that PR solved the aforementioned issue?


Correcting the Installation Documentation for Windows users

Pull request type

Documentation content changes
What is the current behavior?

Issue Number: 265, 266
What is the new behavior?

add description of some quirks and problems Windows users might encounter while setting up the development environment

Does this introduce a breaking change?

no
Other information

Pull request checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added or the PR is neither a bug fix nor a feature
[ ] Docs have been reviewed and added / updated if needed or the PR is neither a bug fix nor a feature
[ ] The Unreleased section in CHANGELOG.md has been amended to reflect the changes in this PR if needed
[ ] Build (mvn verify) was run locally and any changes were pushed
[ ] The pull request is against the correct branch (master for bug fixes, develop for new functionality)
[ ] Dependencies and citation templates have been updated where necessary or there are no new dependencies

Checklist for maintainers
Releases

Do NOT push the green merge button on GitHub.
Follow the latest version of the developer/maintainer docs for making releases.
Check that license and citation information for dependencies are complete (each dependency has a folder in /THIRD-PARTY/).

Thanks @bunz-lightyear. As a first step towards merging this, can you please go through the PR checklist in https://github.com/hexatomic/hexatomic/pull/298#issue-601560650? The checklist is meant to be completed by the person putting up the PR :). Thanks!

Mergin regardless of failing check as this is due to SonarCloud not being able to run on PRs with a forked source.


Connect to an existing kernel and run/debug file.py

How after installing the connection on the remote machine, run or debug the file:

if the file is on the local machine?
if the file is on a remote machine, then you can run the command runfile ('path_at_remote_machine / file.py', wdir = 'path_at_remote_machine') in the iPython console, but we will not see debuging?
can synchronize local and remote folders, edit the file in Spidere, and run the manual command in another console or in iPython. But this is inconvenient.

How edit a file in Spider on a local machine and run(F5) and debug(Ctrl+F5) its at remote kernel in the Spider? And see the variables while debugging when will stop at a breakpoint?
Versions and main components

Spyder Version: Spyder 4.0.0.dev0 (Commit: 0608076)
Python Version: 2,7
Qt Version: 4.8.7
PyQt Version: PyQt4 (API v2) 4.11.4
Operating system: Ubuntu 16.04

Dependencies
Please go to the menu entry Help > Optional Dependencies (or
Help > Dependencies), press the button Copy to clipboard
and paste the contents below:
IPython >=4.0;<6.0:  5.4.1 (OK)
cython >=0.21     :  0.25.2 (OK)
jedi >=0.9.0      :  0.10.2 (OK)
nbconvert >=4.0   :  5.2.1 (OK)
numpy >=1.7       :  1.13.0 (OK)
pandas >=0.13.1   :  0.19.0 (OK)
psutil >=0.3      :  5.2.2 (OK)
pycodestyle >=2.3 :  2.3.1 (OK)
pyflakes >=0.5.0  :  1.5.0 (OK)
pygments >=2.0    :  2.2.0 (OK)
pylint >=0.25     :  1.6.4 (OK)
qtconsole >=4.2.0 :  4.3.0 (OK)
rope >=0.9.4      :  0.10.3 (OK)
sphinx >=0.6.6    :  1.5.1 (OK)
sympy >=0.7.3     :  1.0 (OK)

We're aware of all these issues but right there's no solution for any of them, sorry.


Fix bug on elections listing

Hey, long time! 👋
Thanks for putting together MM, I'm stoked to see you launch another project :) (and in SvelteKit no less!)
Noticed and fixed a small bug on the elections list.

Hey!
Thanks so much! I had a feeling I was forgetting something when I removed that array from the layout data lol.


Explicate optional parameter for parser methods

Here is a snippet from the body-parser middleware documentation:

bodyParser.json(options)
Returns middleware that only parses json and only looks at requests where the Content-Type header matches the type option. This parser accepts any Unicode encoding of the body and supports automatic inflation of gzip and deflate encodings.
A new body object containing the parsed data is populated on the request object after the middleware (i.e. req.body).
Options
The json function takes an option options object that may contain any of the following keys:

The options objects that the JSON parser takes is optional, so shouldn't the snippet actually look like this?:

bodyParser.json([options])
Returns middleware that only parses json and only looks at requests where the Content-Type header matches the type option. This parser accepts any Unicode encoding of the body and supports automatic inflation of gzip and deflate encodings.
A new body object containing the parsed data is populated on the request object after the middleware (i.e. req.body).
Options
The json function takes an optional options object that may contain any of the following keys:

Here are the two changes proposed

Add brackets around options parameter in the the method signature to signify optional options object
Change option options object to optional options object (last sentence)

If so, similar changes can be applied to all four parsers (JSON, Raw, Text, URL-encoded).

We use a script to grab the README from the body-parser module directly. Please file an issue (or even better, a PR) to that module and it'll get picked up here :)

@dougwilson I'm sorry. totally misunderstood your directions. body-parser has its own repo... sorry for the troubles.

It's no problem. I figured as soon as you made the issue, you started on the PR and didn't see my issue comment; happens all the time.


RSSFeedItemGUID is being given a wrong value for this rss feed

If you parse this rss feed,
http://allearsenglish.libsyn.com/rss
A wrong value comes in for RSSFeedItemGUID property. It looks like the link value is written for the RSSFeedItemGUID property. It is happening for some recent episodes.
Look at this screenshot.

Hi, @pearl7721User1
That's because the feed is outputting it like that.
Try running in the terminal:
curl http://allearsenglish.libsyn.com/rss

Then have a look at the feed:
<guid isPermaLink="false"><![CDATA[http://allearsenglish.com/?p=6]]></guid>

This is perfectly fine, and an acceptable form of a unique identifier for this use case.

Hi, @nmdias
Turned out it's my mistake. It's all working fine. Thank you for your swift answer.


Feat(eos_designs): l3ls: svi_profiles should support referencing a base svi_profile

Change Summary
Similar to #875 but instead of a port profile support for SVI profiles should be extended to support 2 stage nested profiles.
Related Issue(s)
Fixes #1370
Component(s) name
arista.avd.eos_designs
Proposed changes

svi_profiles:
  < profile_name >:
    parent_profile: < parent svi profile name >
    ...

How to test

molecule
Checklist
User Checklist

N/A

Repository Checklist

[x] My code has been rebased from devel before I start
[x] I have read the CONTRIBUTING document.
[x] My change requires a change to the documentation and documentation have been updated accordingly.
[x] I have updated molecule CI testing accordingly. (check the box if not applicable)

The PR title should be feat or fix.

run molecule converge -s evpn_underlay_ebgp_overlay_ebgp and then molecule converge -s eos_designs-twodc-5stage-clos. This should fix the artifacts.

Anything missing from my side?

Thank you for the feedback @ClausHolbechArista - will add the changes

Added all the requested changed from @ClausHolbechArista


Added Serialization/Deserialization tab in documentation

Description
Added documentation for serialization/deserialization in advanced tab. Gives a good brief on how the functions work (as of now)
Addresses and fixes #539
Type of change
Please delete options that are not relevant.

[- ] Documentation update

Checklist:

[ ✅] My code follows the style guidelines of this project
[ ✅] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ✅] I have made corresponding changes to the documentation
[ ✅] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[ ✅] I have checked my code and corrected any misspellings

Maintainer Checklist

[ ] closes #xxxx (Replace xxxx with the GitHub issue number)
[ ] Made sure Checks passed

@Shady2kOver can you please resolve the conflicts?

@Shady2kOver Could you please resolve the conflicts? We will merge this soon!

thanks for your contribution @Shady2kOver  .
Will have to close this PR as the code has become stale.


Output

This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines

[ ] The PR has modified HISTORY.rst with an appropriate description of the change (see Modifying change log).

Command Guidelines

[ ] Each command and parameter has a meaningful description.
[ ] Each new command has a test.

(see Authoring Command Modules)

Codecov Report

Merging #9 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master       #9   +/-   ##
=======================================
  Coverage   72.23%   72.23%
=======================================
  Files         422      422
  Lines       26357    26357
  Branches     3991     3991
=======================================
  Hits        19040    19040
+ Misses       6089     6086    -3
- Partials     1228     1231    +3

Impacted Files
Coverage Δ

...s/azure-cli-interactive/azclishell/az_completer.py
59.72% <100%> (ø)
:arrow_up:

src/azure-cli-core/azure/cli/core/util.py
72.1% <0%> (ø)
:arrow_up:

...nent/azure/cli/command_modules/component/custom.py
16.23% <0%> (ø)
:arrow_up:

...dback/azure/cli/command_modules/feedback/custom.py
31.25% <0%> (ø)
: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 cea6c8a...8b722ba. Read the comment docs.


BGInfo Version number

The Page for BgInfo (https://docs.microsoft.com/en-us/sysinternals/downloads/bginfo aka https://github.com/MicrosoftDocs/sysinternals/blob/live/sysinternals/downloads/bginfo.md) has v4.26 as the Version number although the actual version of the download is v4.27.

Thanks for reporting. I will get this updated as soon as possible.

I just updated this now. Thanks again for reporting.


test for lint_ext_authority_key_identifier_no_key_identifier

I'm going to go out on a limb and merge this since it's just a unit test. If folks would prefer I wait for more :mag:'s on this sort of PR speak up and I won't do it again :-)


Fixes issue #225

Fixed the reported bug and found another related one we had not encountered yet - input with no type is a text input.
Actually an input of any unknown type is a text input - we probably don't handle that, that should probably be a separate issue..? For example <input type="foo"/> I'm thinking that's an outlier and no point holding up this fix for it.
Also completely ignoring the fact that aria-required should not be used when an exactly equivalent native attribute is available - that should be a different audit anyway because SEVERE is a bit extreme in that case.

@alice PTAL

:+1:


launcher.sh: line 205: can't create : nonexistent directory

Getting below issue from today
[INFO] --- gauge-maven-plugin:1.4.3:execute (default-cli) @ assignment-test ---
mktemp: Invalid argument
launcher.sh: line 205: can't create : nonexistent directory
Error occurred while waiting for runner process to finish.
Error : exit status 1
Error ----------------------------------
Your Environment Information -----------
linux, 1.1.1, 6e0d83f
html-report (4.0.11), java (0.7.8), reportserver (0.2.0), screenshot (0.0.1), spectacle (0.1.4), xml-report (0.2.3)

possibly due to https://github.com/getgauge/gauge-java/pull/422

@shanzee05 - can you give us some information on what environment you are seeing this error?
if it's docker - would you be able to share the image used?
if it's a desktop/vm - what distro are you on?
you can run cat /etc/*-release to get the above information. Thanks.

Can't share the image , its alpine base image and other dependencies are installed on top of it .
bash-5.0# cat /etc/*-release
3.11.3
NAME="Alpine Linux"
ID=alpine
VERSION_ID=3.11.3
PRETTY_NAME="Alpine Linux v3.11"
HOME_URL="https://alpinelinux.org/"
BUG_REPORT_URL="https://bugs.alpinelinux.org/"

Unfortunately Gauge is not tested on alpine docker images. Can you try using ubuntu or debian based images?

I have been using it successfully for few months now (gauge-java 0.7.4 and gauge v1.0.8 )- faced issue only after latest updates(Gauge and gauge-java latest version)
I reverted to

Ok, thanks for verifying that will look into fixing this.

I can confirm that this is an issue with gauge-java 0.7.8 on alpine based images.
Steps to replicate

docker run -it  openjdk:15-jdk-alpine3.11 /bin/sh
apk --no-cache add curl
curl -SsL https://downloads.gauge.org/stable | sh
mkdir foo && cd $_
gauge init java
gauge run specs

Running specs on this image throws:
# gauge run specs
Compatible version of plugin html-report not found. Installing plugin html-report...
........................
Successfully installed plugin 'html-report' version 4.0.11
Installing required plugins.
.......................
Successfully installed plugin 'screenshot' version 0.0.1
sh: 15-ea: bad number
mktemp: Invalid argument
launcher.sh: line 86: can't create : nonexistent directory
Error occurred while waiting for runner process to finish.
Error : exit status 1
Error ----------------------------------

[Gauge]
Failed to start gauge API: Error occurred while waiting for runner process to finish.
Error : exit status 1

Get Support ----------------------------
	Docs:          https://docs.gauge.org
	Bugs:          https://github.com/getgauge/gauge/issues
	Chat:          https://spectrum.chat/gauge

Your Environment Information -----------
	linux, 1.1.1, 6e0d83f
	html-report (4.0.11), java (0.7.8), screenshot (0.0.1)


Update comments in proto

Pull Request Functional Test Report for #1764 / 9cd4a948f482afa02dcb22858f36f33768b30364
No tests identified for validation.
Help

LGTM, @prinikasn please resolve conflicts


[refactor](inverted index) Refactor the idx storage format to avoid cyclic references in Thrift files.

Proposed changes
TInvertedIndexStorageFormat is defined in the AgentService.thrift. When other  Thirft files use TInvertedIndexStorageFormat, need to include AgentService.thrift.
However, AgentService.thrift might already include that file, causing cyclic references.


Web Page Speed Monitor - check_frequency issue

Hi team, I am getting an exception in terraform when creating a web page speed monitor, has anyone seen the same.  I do not have the same issue with a web site monitor.
core doc i used
https://registry.terraform.io/providers/site24x7/site24x7/latest/docs/resources/web_page_speed_monitor
╷output from scalr
│ Error: Check frequency not supported.
│
│   with module.crp_website_mail2_caledoncard_com.site24x7_web_page_speed_monitor.web_page_speed_monitor,
│   on payroc/webpage_speed/main.tf line 17, in resource "site24x7_web_page_speed_monitor" "web_page_speed_monitor":
│   17: resource "site24x7_web_page_speed_monitor" "web_page_speed_monitor" {
│
╵
Error: Failed to perform terraform apply. Unexpected exit code: 1
variables.tf
variable "display_name" {
type = string
description = "// (Required) Display name for the monitor"
}
variable "url" {
type = string
description = "(Required) Website address to monitor."
}
variable "check_frequency" {
type = string
description = "(Optional) Check interval for monitoring. Default: 1 https://www.site24x7.com/help/api/#check-interval for all supported"
default = "5"
}
main.tf
// Data source to fetch a monitor group
data "site24x7_monitor_group" "monitor_group" {
// (Required) Regular expression denoting the name of the monitor group.
name_regex = var.monitor_group_name
}
data "site24x7_user_group" "user_group" {
name_regex = "TOps Notifications"
}
// Website Monitor API doc: https://www.site24x7.com/help/api/#web-page-speed-(browser)
resource "site24x7_web_page_speed_monitor" "web_page_speed_monitor" {
// (Required) Display name for the monitor
// display_name = "mymonitor"
display_name = var.display_name
// (Required) Website address to monitor.
// website = "https://foo.bar"
website = var.url
// (Optional) Check interval for monitoring. Default: 1. See
// https://www.site24x7.com/help/api/#check-interval for all supported
// values.
// check_frequency = "15"
check_frequency = var.check_frequency
}
cal.tf
module "crp_website_mail2_caledoncard_com" {
source = "./payroc/webpage_speed"
display_name = "testmk.caledoncard.com"
url = "https://mail2.caledoncard.com"
//alert=false
monitor_group_name = "cal"
//monitor_ssl = false
}

I note in the file resource_vs_attribute_types.json there does not seem to be an entry for web page speed monitors.
also noticing check frequency is an int for web transaction browser and a string for everything else
not sure if this is related

Hi markkellypayroc,
Thank you for using our service and we are sorry for the issue.Generally "5" minutes check_frequency is not supported in web page speed browser unless you are an "Enterprise" customer.So kindly use "10 min" or above...

Thanks team
Mark Kelly
Mark Kelly  Senior Site Reliability Engineer II
e: @.***
w: www.payroc.com

From: sujithramagesh @.>
Sent: Wednesday, January 17, 2024 7:09:31 AM
To: site24x7/terraform-provider-site24x7 @.>
Cc: Mark Kelly @.>; State change @.>
Subject: Re: [site24x7/terraform-provider-site24x7] Web Page Speed Monitor - check_frequency issue (Issue #229)
Hi markkellypayroc,
Thank you for using our service and we are sorry for the issue.Generally "5" minutes check_frequency is not supported in web page speed browser unless you are an "Enterprise" customer.So kindly use "10 min" or above...
—
Reply to this email directly, view it on GitHubhttps://github.com/site24x7/terraform-provider-site24x7/issues/229#issuecomment-1895190230, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVHFAANTBDB6AXR6TSSTSRDYO52KXAVCNFSM6AAAAABBWFAH4OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJVGE4TAMRTGA.
You are receiving this because you modified the open/close state.Message ID: @.***>


Failed to verify OTP

Hi,
When verifying OTP, I have checked the error message.
Failed to verify OTP, try again: "Rust SDK does not support sharded to enclave auto-migration."
I have tried to test the sample level "Level_Thirdweb" and the result was the same.

I hope to hear from you soon.
Best regards.

Hey! That is correct. The auto migration functionality is imminent but not yet included.

related to CNCT-2482
references https://github.com/thirdweb-dev/thirdweb-rust/pull/39


Conflict CRD when installing kong using helm chart

Summary
Hi, I tried to install latest kong version on the fresh kubernetes 1.17.2 (kops) using helm v3 and got the problem :

Error: rendered manifests contain a resource that already exists. Unable to continue with install: existing resource conflict: kind: CustomResourceDefinition, namespace: , name: kongcredentials.configuration.konghq.com

This is new fresh k8s cluster so I don't think there is a real crd conflict. I also tried to run "kubectl delete crd kongconsumers.configuration.konghq.com kongcredentials.configuration.konghq.com kongingresses.configuration.konghq.com kongplugins.configuration.konghq.com" then re-install again but still got the problem
Any ideas ? Hope receive your help. Thank you so much.
Steps To Reproduce

helm repo add kong https://charts.konghq.com
helm repo update
helm install kong kong/kong

Let's continue the conversation in https://github.com/Kong/charts/issues/33.


Specify input/output format in Readme

The readme should specify exactly which input and output formats exist. Is coordTimes a universally accepted property or something we came up with?

@freenerd Readme updated, it seems like coordTimes is something @tmcw came up with since there is no standard to do this in the geojson spec.

Yep, I would love for this to be slightly more standard but that's a job for mr. @sgillies specification author

Tell me more about coordTimes. Sounds like a good candidate for a GeoJSON extension. What it would need to be a good extension is its own namespace, eg, properties['gpx-ish']['coordTimes'] instead of properties['coordTimes'], and documentation of the protocol to be followed (zipping the coordTimes and coordinates lists together).

If we are going to to standardize this, we should go with timestamps instead of coordTimes.

Since we're expecting users to use this along with togeojson, the change will have to be implemented there first.


pinchzoom issue

open http://openlayers.org/en/master/examples/simple.html on a mobile device
pinch zoom
on pinch release, the intermediate tiles are displayed

dec39151a01dbca5aafa9178d73520a863b70980 is the first bad commit
commit dec39151a01dbca5aafa9178d73520a863b70980
Author: Thomas Chandelle<EMAIL_ADDRESS>Date:   Wed Jan 11 10:21:21 2017 +0100

    Consider pointerdown only if the last pointerup has been emitted

:040000 040000 f3aa493faa6a4e8de125177e3509b62f9ca40486 bddb78f479f2b83800f215e2bed2ae76c5b43dbb M      src

Indeed I can reproduce the issue, Sorry about that, forgot to test it back then.
This commit solved #6350.

diff --git a/src/ol/interaction/pointer.js b/src/ol/interaction/pointer.js
index 048ff27..068b32b 100644
--- a/src/ol/interaction/pointer.js
+++ b/src/ol/interaction/pointer.js
@@ -187,8 +187,7 @@ ol.interaction.Pointer.handleEvent = function(mapBrowserEvent) {
     if (mapBrowserEvent.type == ol.MapBrowserEventType.POINTERDRAG) {
       this.handleDragEvent_(mapBrowserEvent);
     } else if (mapBrowserEvent.type == ol.MapBrowserEventType.POINTERUP) {
-      this.handleUpEvent_(mapBrowserEvent);
-      this.handlingDownUpSequence = false;
+      this.handlingDownUpSequence = this.handleUpEvent_(mapBrowserEvent);
     }
   } else {
     if (mapBrowserEvent.type == ol.MapBrowserEventType.POINTERDOWN) {

fixes the issue but I haven't tested #6350

It solved both issues. But it introduces a new one ...

Go to http://localhost:3000/build/examples/translate-features.html
Select a country, move it, it works
Move it again, it pan the map instead
Move it again, it works again
GOTO 3

diff --git a/src/ol/interaction/dragpan.js b/src/ol/interaction/dragpan.js
index 30cb906..80867c4 100644
--- a/src/ol/interaction/dragpan.js
+++ b/src/ol/interaction/dragpan.js
@@ -93,7 +93,7 @@ ol.interaction.DragPan.handleDragEvent_ = function(mapBrowserEvent) {
 ol.interaction.DragPan.handleUpEvent_ = function(mapBrowserEvent) {
   var map = mapBrowserEvent.map;
   var view = map.getView();
-  if (this.targetPointers.length === 0) {
+  if (this.targetPointers.length <= 1) {
     if (!this.noKinetic_ && this.kinetic_ && this.kinetic_.end()) {
       var distance = this.kinetic_.getDistance();
       var angle = this.kinetic_.getAngle();

Solved the three issues, pass tests. But I'm not confident about it.

See https://github.com/openlayers/openlayers/compare/master...fredj:primaryAction
This revert the fix for #6350 and add a condition to ol.interaction.Translate to discard not "primary action" events. (left click is a primary event, middle and right click aren't).
The same should be done for the zoom box

It doesn't work neither. handleDownEvent_ never receive right click events.

@fredj I like the look of your changes in https://github.com/openlayers/openlayers/compare/master...fredj:primaryAction.  I couldn't tell if @tchandelle's comment above was related to that or to an alternative fix that didn't work.
I'm proposing #6445 as a fix for pinch zooming.  It feels to me like allowing pointer interactions to stop propagation of drag and up events would help the situation.  And it looks like pointer interactions would benefit from the primary action condition.  But if #6445 fixes things for the release, I think it is worth getting in.


Fixed up GetAll Ped/Props/Vehicles/entites

Removed Get ped nearby peds and vehicles
Removed most of the code for Entity pool in favour of calling functions from gta5 exe

Can't merge because you have both a "dev" and a "Dev" branch in your repository, which causes issues. Could delete the invalid one (the lowercase version)?

Is that better?


Add parameter substitution on run

This PR adds the support of parameters substitution when using the run command.
As an example a command written in the README as:
$ echo <YOUR_NAME>

Could be called by running
$ rdme run echo username

The placeholder YOUR_NAME would then be replaced by username in the final command.
PS: it's my first time working with Golang ^^

Hi @hamza-tam. Thank you for your contribution! The idea looks great, however, rdme already supports that using --replace flag. Following your example, it would work like this:
rdme run echo --replace="s/<YOUR_NAME>/Test/"

--replace uses sed syntax. The advantage is that you can use an arbitrary placeholder format.
I also recommend using shell variables as placeholders, for example:
echo $YOUR_NAME

Then you can execute it like this:
YOUR_NAME=MyName rdme run echo

Hello @adambabik, Thank you for your feedback !
The replace command is indeed very handy. In the spirit of the tool, I think adding the feature to input parameters directly too would be very useful. rdme's goal is to remove the need to copy commands manually. While the --replace solves the issue of replacing placeholders with actual values, I think it is more suited for edge cases with weird parameter formatting.
Let's take as an example this command from the gh Cli documentation:
$ gh auth login --hostname <hostname>

you could either run this command using rdme run gh-auth --replace="s/<hostname>/hostname/" which requires some mouse gymnastics, and kind of removes the need for the tool, Or directly type rdme run gh-auh hostname.
Of course other parameter patterns could be added to support more syntaxes.

@hamza-tam You're right about tool's spirit and important aspects of UX and simplicity. Having predictable arguments substitution would be useful indeed as --replace is quite complex.
For the arguments substitution, I would like to see the following:

If any arguments is missing, the error clearly states it, for instance: Argument "hostname" is missing.
All arguments must be provided; we can skip defaults in any form.
It should be possible to skip resolving arguments with --resolve-args=false.

Hello @adambabik, Would it be possible to develop the improvements you asked in a separate PR in order to not make the current one much more complex ? I envisioned that the current one would add the basic feature, and it could be improved upon afterwards.

@hamza-tam totally we can. We are in the middle of some bigger changes for a new use-case but I will try to incorporate your PR this week.

Hi @hamza-tam, sorry about the long delay closing the loop here. Runme's development (fka rdme) has advanced since. It offers full support of ENV var completion: https://docs.runme.dev/configuration/cell-level#set-environment-variables.
This work both in the CLI & Extension. Here's an example:

Closing this issue as "won't fix" for now. Feel free to reopen or file another issue if you encounter any other issues. Thank you!


Always hidden icons inside the always hidden section is still visible.

The icons inside the always hidden section is still visible after expanding.
steps to reproduce:

add any icon to the always hidden section.
collapse the bar.
expand the bar.

app version: 1.8 (12) from mac app store
os: macOS 12.5.1
video for reference:
https://user-images.githubusercontent.com/47196695/188254887-93b32e61-c538-44de-a899-5c092194fc1b.mov

Hi, I seem to be having this issue as well!
Can this please be looked into?

I have the same problem.
App version 1.9 (13); downloaded from github
MacOS 12.5.1
Macbook Air 2020 M1 8C CPU 8C GPU, 8 GB ram

Same issue as well here: https://app.usebubbles.com/vSJnZRX3mF2WWSXfRzz3an/hidden-bar-issue
App version 1.8(App store)
MacOS 12.5.1

My findings show that after you enable the "always hidden" option you have to quit hidden and restart it and then move icons to the appropriate section. It should work after that.

@mlcampbe Thanks, that gets "always hidden" working for me. Odd behavior!

mlcampbe's trick isn't working for me on MacBook Pro (16-inch, 2019) macOS 12.5.1.

Right-click on the arrow button to toggle between always hidden visible and hidden

@shbhnkr-sg Thanks! I completely missed the step to right-click in the popover help. I did need to restart the app, but after that, it's working great.

My findings show that after you enable the "always hidden" option you have to quit hidden and restart it and then move icons to the appropriate section. It should work after that.

So weird but work! Seems like a little bug.

Bug Confirmed, Workaround from Gitbreast too.

我的发现表明，启用“始终隐藏”选项后，您必须退出隐藏并重新启动它，然后将图标移动到适当的部分。它应该在那之后工作。

yes，is useful

My findings show that after you enable the "always hidden" option you have to quit hidden and restart it and then move icons to the appropriate section. It should work after that.

@mlcampbe's fix works for me!
Using Version 1.9 (13) an old mid-2015 MacBook Pro running on Monterey 12.6.

this i tried out several times - this unfortounately does not work for me…
after restart hidden bar no single symbol of hidden bar is shown not the arrow, not even one of the vertikal lines…
nothing…
its a mess...
Am 20.01.2023 um 01:58 schrieb Razi Syed @.***>:
My findings show that after you enable the "always hidden" option you have to quit hidden and restart it and then move icons to the appropriate section. It should work after that.
@mlcampbe https://github.com/mlcampbe's fix works for me!
Using Version 1.9 (13) an old mid-2015 MacBook Pro running on Monterey 12.6.
—
Reply to this email directly, view it on GitHub https://github.com/dwarvesf/hidden/issues/224#issuecomment-1397798553, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALUJYXFGJV7SUSFR3WV5PXTWTHPLHANCNFSM6AAAAAAQDXHNJM.
You are receiving this because you are subscribed to this thread.

@studiohansen are you right-clicking the "<"?

thanks for answering me…
there is no > and no < anymor shown - even no |
i don’t know what happened…
since i tried out the  „always hidden“ version with two |.
before everything worked well…  f…
i made the mistake and uninstalled hidden bar and hoped, the icons would have come back.
but the never came back with or withoiut hidden bar app
after reinstall hidden bar these control signs aren’t shown anymore
it’s a mess
Am 20.01.2023 um 17:07 schrieb Razi Syed @.***>:
@studiohansen https://github.com/studiohansen are you right-clicking the "<"?
—
Reply to this email directly, view it on GitHub https://github.com/dwarvesf/hidden/issues/224#issuecomment-1398606969, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALUJYXGKEFRTKJF3O6M3A7DWTKZ53ANCNFSM6AAAAAAQDXHNJM.
You are receiving this because you were mentioned.

problem solved…
Thank you for the hint with the right mouse button: after I was completely desperate, I wildly right-clicked everywhere in the menu bar. Suddenly an icon appeared that looked familiar: it was the hide menu bar function from the app "Only Switch".
It seems that the two programmes don't get along. Although I had turned off the hide function in Only Switch, it seems that using Hidden Bar simply turns it back on. The two programmes do not get along. The solution is to set the separator symbol for objects to be hidden in Only Switch to the far left. Then press the right mouse button and the symbol disappears. It seems that the two programmes do not get along with each other. Although I had switched off the hide function in Only Switch, it seems to be switched on again by using Hidden Bar. The two programmes do not get along. The solution is to hide the separation symbol for the objects in Only Switch and then press the right mouse button. This hides the symbol and Hidden Bar can then be used normally. Or it is best to do without Only Switch.
Maybe someone else knows more about this, I can only say I am glad to have my usual working environment back.
Hidden Bar and Only Switch: Caution when using them at the same time
Many thanks for your help,
Holger.
Am 20.01.2023 um 17:07 schrieb Razi Syed @.***>:
@studiohansen https://github.com/studiohansen are you right-clicking the "<"?
—
Reply to this email directly, view it on GitHub https://github.com/dwarvesf/hidden/issues/224#issuecomment-1398606969, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALUJYXGKEFRTKJF3O6M3A7DWTKZ53ANCNFSM6AAAAAAQDXHNJM.
You are receiving this because you were mentioned.

The (right-clicking the "<") should be document somewhere perhaps in readme.md - Usage and the overview App screen!

restart app works


[OPTIMIZE] Use extension style for binding-adapters

Update the binding-adapter to kotlin extension style.

@JoseAlcerreca how about the change?

Closing this as binding adapters are obsolete now that the app is in Compose


Update Go modules for CSM release 1.8

Description
Update with released Go client modules.
GitHub Issues
List the GitHub issues impacted by this PR:

GitHub Issue #

https://github.com/dell/csm/issues/885

Checklist:

[X] I have performed a self-review of my own code to ensure there are no formatting, vetting, linting, or security issues
[X] I have verified that new and existing unit tests pass locally with my changes
[X] I have not allowed coverage numbers to degenerate
[X] I have maintained at least 90% code coverage
[ ] I have inspected the Grafana dashboards to verify the data is displayed properly
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[X] I have maintained backward compatibility

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration

[X] Unit tests
[ ] Test B

Manual inspection of the GUI
I have verified that the dashboards show the data properly while generating I/O and storage resources

[ ] Yes
[ ] No

run e2e test

run e2e test

run e2e test

run e2e test

e2e pass


4.17, Wireshark Lab2

I think two images were downloaded in parallel. Because of their waterfall almost overlaps each other.

Thanks for your issue.
I am looking at the offical answer.It also says two images were downloaded in parallel.
In my experiment, i thought two image were starting to download in different time.
The author and you thought the time different can miss.
The experiment does not have correct answer. But we can find others views in talking.
Thank you!
感谢您的疑问，我查阅了官方答案，官方也说明这两张图片是同时下载的。
在我当时实验中，我认为两张图片下载get请求有时间差，所以我说有差别。
我猜测您和作者都觉得这个时间很小可以忽略不记，
这实验没有绝对的答案对错，只愿我们在交流过程中能够发现其他人的想法和找出自己的不足。
再次感谢您！

thank you～


Enable to use basic_auth_only option from winrm gem

This pull request adds an option for the winrm transport named basic-auth-only, which is by default set to false, but if set to true will force basic auth by passing the corresponding basic_auth_only option to the winrm gem.
If there is an easier solution to my issue than to force basic auth, I would gladly welcome any advice.
I set up winRM to use with basicAuth on a machine at our site. I was able to connect properly using plain winRM powershell commands where I can pass the auth method via -Authentication Basic. For bolt this was not possible, the connection failed with an AUTH_ERROR all the time.
I looked into the code and saw that bolt offers no option to enforce basic_auth, while the winrm gem offers this option via basic_auth_only.

CLA signed by all contributors.

@knorx If you're ok with me pushing to your branch I'd be happy to rebase this work and get it cleaned up for merge

Tests are failing since they set basic-auth-only => true and ssl => false further up in the options hash: https://github.com/knorx/bolt/blob/basic-auth-only/spec/bolt/inventory/inventory_spec.rb#L609-L626

CLA signed by all contributors.


Translate "Decorateur" section

I will translate this section.

I finished the translation of the decorator section.
You can watch the changes.


SH doesn't support spreading object properties in evalscript

Example (also in EOB https://sentinelshare.page.link/xtKS )
//VERSION=3
function setup() {
  return {
    input: ["B01","B02","B03", "dataMask"],
    output: { bands: 4 }
  };
}

function evaluatePixel(sample) {
  const bb = {"a": 1, "b": 2};
  const cc = {...bb, "c": 3};

  return [2.5 * sample.B01, 2.5 * sample.B02, 2.5 * sample.B03, sample.dataMask];
}

SH returns error
Failed to evaluate script!
evalscript.js:11: SyntaxError: Unexpected token ...
  const cc = {...bb, "c": 3};

Object.assign(target, source1 [, source2, ...]); should be used instead

in this project:
in src/pg_to_evalscript/node.py
apply_dimension:
https://github.com/openEOPlatform/openeo-pg-evalscript-converter/blob/ce4f3a1e9382c78b3dc0e6385b2f7a2b84f9ed44/src/pg_to_evalscript/node.py#L285
merge_cubes:
https://github.com/openEOPlatform/openeo-pg-evalscript-converter/blob/ce4f3a1e9382c78b3dc0e6385b2f7a2b84f9ed44/src/pg_to_evalscript/node.py#L221
aggregate_temporal:
https://github.com/openEOPlatform/openeo-pg-evalscript-converter/blob/ce4f3a1e9382c78b3dc0e6385b2f7a2b84f9ed44/src/pg_to_evalscript/node.py#L379
solution for our project:
something like
return apply_dimension(Object.assign({}, arguments, {process}))

found because of https://github.com/openEOPlatform/openeo-pg-evalscript-converter/pull/77#pullrequestreview-991852142

Thanks for reminding about this issue, we had noticed it before https://github.com/openEOPlatform/openeo-pg-evalscript-converter/pull/64 but I guess it slipped our minds again.
However, please use https://git.sinergise.com/team-6/openeo-platform/-/issues for issues, closing and republishing there.


no function or associated item named unmarshal found for struct

I'm trying to figure out what is happening here. Compiling the project on its folder has no errors, but compiling my project when it depends on retina, gives this:
error[E0599]: no function or associated item named `unmarshal` found for struct `Header` in the current scope
   --> src/external/retina/src/client/rtp.rs:193:49
    |
193 |             let h = match rtcp::header::Header::unmarshal(&data) {
    |                                                 ^^^^^^^^^ function or associated item not found in `Header`
    |
    = help: items from traits can only be used if the trait is in scope
    = note: the following trait is implemented but not in scope; perhaps add a `use` for it:
            `use webrtc_util::marshal::Unmarshal;`

error[E0599]: no function or associated item named `unmarshal` found for struct `rtcp::sender_report::SenderReport` in the current scope
   --> src/external/retina/src/client/rtp.rs:211:66
    |
211 |                     let pkt = rtcp::sender_report::SenderReport::unmarshal(&pkt)
    |                                                                  ^^^^^^^^^ function or associated item not found in `rtcp::sender_report::SenderReport`
    |
    = help: items from traits can only be used if the trait is in scope
    = note: the following trait is implemented but not in scope; perhaps add a `use` for it:
            `use webrtc_util::marshal::Unmarshal;`

error: aborting due to 2 previous errors; 1 warning emitted

have you had something like this? I have no idea about what is happening

The rtcp crate had an accidental semvar breakage between version 0.2.2 and 0.2.3. The current version of retina only works with the former. So the workaround is to specify rtcp 0.2.2 in your lockfile via something like cargo update --package rtcp --precise 0.2.2.
I'll make sure the next retina release I prepare works without any lockfile manipulation.


app not opening on android

Android 13 / 20-20230813-UNOFFICIAL-lilac
Thorium Version [124.0.6367.218]
i open the app, this small window with the "next" button and something with google-login appears but after pressing next nothing happens.
therefore, app is not working at all.
see screenshot.
if it helps, my phone has no google framework or services.


cordDifference.js:5 Uncaught TypeError: styledComponents_browser_cjs.styled is not a function  at RecordDifference.js:5:14

(api) ivan@IvanPC:~/code/xxxxxplatform/bo (ajsnew)$ npm ls --all | grep styled-com
│ │ ├─┬<EMAIL_ADDRESS>│ │ │ ├─┬<EMAIL_ADDRESS>│ │ │ │ └──<EMAIL_ADDRESS>deduped

in docs for 5.3.9 I see that default import should be used:
import styled from 'styled-components';

But here is a a partial import:

Using the next versions:
 "adminjs": "^7.0.0",
 "@adminjs/logger": "^5.0.0",
 "@adminjs/express": "^6.0.0",
"@adminjs/import-export": "^3.0.0",
"@adminjs/mongoose": "^4.0.0",
"@adminjs/passwords": "^4.0.0",
"@adminjs/themes": "^1.0.0",
"@devforth/adminjs-upload": "^4.0.4",

Closing because it was already fixed in @adminjs/logger 5.0.1


typo in code - should be 'utf-8'

https://github.com/helgeerbe/picframe/blob/b0fddbb9df1b771a9133dc3dd5ea92c18b9165df/picframe/get_image_meta.py#L40

@junglebells Thanks for finding that. But if I understand the definition correctly it should be 'utf_8' but 'utf8' and 'utf-8' are valid aliases.
Das that line makes any problems? Or does it work correctly in your environment?

I am not sure about captions, but my IPTC keywords (tags) are not coming through (they are there in pics, but not in the db)
I was trying to find where the problem could be and noticed the typo.
Can you confirm that if there are multiple keywords on a picture, it would work. Here is a sample picture, if you want to test.

Hi @junglebells,
I can confirm that multiple keywords load correctly into the db's meta table if that's what your asking. Also, I can confirm that the keywords in your above sample image do get stored the db as shown below:

Is that not the case for you?

No, unfortunately it’s not working for me. I don’t see the keywords in the db.
I have the latest code base. Any tips on how to troubleshoot?
Thanks so much.
Ashu

On Jul 19, 2021, at 8:35 AM, Jeff Godfrey @.***> wrote:

Hi @junglebells,
I can confirm that multiple keywords load correctly into the db's meta table if that's what your asking. Also, I can confirm that the keywords in your above sample image do get stored the db as shown below:
Is that not the case for you?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

I have the latest code base

What are you running, specifically? Source code (main, dev, ??) or a packaged release (what version)? There were some meta-data collection issues fixed in the past, but I think they should be widely available now...

pi3d-2.46 picframe-2021.7.5
on Raspberry Pi, installed via python3 pip
On Mon, Jul 19, 2021 at 9:54 AM Jeff Godfrey @.***>
wrote:

I have the latest code base
What are you running, specifically? Source code (main, dev, ??) or a
packaged release (what version)? There were some meta-data collection
issues fixed in the past, but I think they should be widely available now...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/helgeerbe/picframe/issues/149#issuecomment-882703562,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQKQV6P5LVPGKLZ4K3VMB7TTYRKERANCNFSM5ATB35OA
.

pi3d-2.46 picframe-2021.7.5

Hmmm... Yeah, that should definitely have the meta-data fixes I mentioned above. I'm not sure why it's not working for you. Is it failing on all images, or just some subset? Also, do the offending images have a meta table record with no tags data, or is there no meta table record for them at all?
Does the logging show anything interesting?

It's failing for all (except 1) images. These images do have a record in
the meta table, but no tags data.
The only difference in the image that's working and others is in the way
the tag/keyword data is stored.  But I saw your screenshot - that one offending image worked at
your end.
I'd love to explore logs, but could not find where they are stored?
On Mon, Jul 19, 2021 at 11:37 AM Jeff Godfrey @.***>
wrote:

pi3d-2.46 picframe-2021.7.5
Hmmm... Yeah, that should definitely have the meta-data fixes I
mentioned above. I'm not sure why it's not working for you. Is it failing
on all images, or just some subset? Also, do the offending images have a
meta table record with no tags data, or is there no meta table record for
them at all?
Does the logging show anything interesting?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/helgeerbe/picframe/issues/149#issuecomment-882770780,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQKQV6MZBRG6SIM6W44OZZLTYRWIFANCNFSM5ATB35OA
.

I'd love to explore logs, but could not find where they are stored?

You'll probably need to:

Stop the frame service
Activate the logging
Restart the service
Check the logs

To activate the logging, you'll need to open the configuration file (configuration.yaml) and set the following two options:

log_level (defaults to WARNING, which might be enough. For full logging, set it to DEBUG)
log_file - set this to a fully qualified path of the log file you want to create (/home/pi/picframe.log for example)

Also, if you don't have much in your cache database at the moment, I might recommend that you just delete it (or rename it) so it gets recreated. That should force the the caching process to run on all images again, which will hopefully log something helpful should it fail.

I enabled the logs and also recreated the db (deleted the existing one). To my surprise the tags are showing up now!
Not sure what may have been wrong, but this is great... thanks so much for your help!
On a side note, I noticed in the db screenshot you sent earlier, you had people names. I am trying to convert face recognition tags to keywords (in Apple Photos.app) - any suggestion on what might be a good way to do it?
Thanks again for all the help

Hey, that's great news @junglebells. Glad things are working now (though I agree, it's a mystery why it didn't work before).
One thing I should have mentioned earlier... You'll want to be sure to undefine the log file name in the config file (just set it back to ""). Otherwise, it'll continue to concatenate log info to your file and it'll get huge pretty quickly.
Regarding face recognition.... Unfortunately, I don't really have any particular advice here. The tags you saw in my earlier screenshot were added manually, although my DAM software can use Google API's for auto face recognition (but I've never used it).
Perhaps @helgeerbe or @paddywwoof  have some input?

@junglebells good to hear that it is working now. I don't use apple photo. But have a look at https://discussions.apple.com/thread/8595733.

Thank you @jgodfrey @helgeerbe @paddywwoof - I consider this issue resolved. Thanks so much your help and a wonderful product.


[ML] ci: Enable Python 3.12 in test matrix

Description
This pull request is a follow up #31758, which enabled testing for Python 3.12 in the repository but excluded the ML package because of blocking test failures.
These test failures should now be resolved now that the following PRs have been merged:

https://github.com/Azure/azure-sdk-for-python/pull/32381
https://github.com/Azure/azure-sdk-for-python/pull/32364

This pull request enables Python 3.12 testing in our CI.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

cc: @scbedd

Superseded by #32450


crop-video.py / OMP: Error #15: Initializing libiomp5md.dll

Hi everyone,
Today i was try to learn DeepFake Image Animation and than found a tutorial
I follow this tutorial step by step. When i came to 24:02 he's entering the python crop-video.py --inp some_youtube_video.mp4 code.
and this is my version of this code (deepfake) F:\DeepFake>python crop-video.py --inp assets/intro.mp4
but when i run this code

OMP: Error #15: Initializing libiomp5md.dll, but found libiomp5md.dll already initialized.
OMP: Hint This means that multiple copies of the OpenMP runtime have been linked into the program. That is dangerous, since it can degrade performance or cause incorrect results. The best thing to do is to ensure that only a single OpenMP runtime is linked into the process, e.g. by avoiding static linking of the OpenMP runtime in any library. As an unsafe, unsupported, undocumented workaround you can set the environment variable KMP_DUPLICATE_LIB_OK=TRUE to allow the program to continue to execute, but that may cause crashes or silently produce incorrect results. For more information, please see http://www.intel.com/software/products/support/.

getting error like this.
I search a lot of site for that. And i tried this one: https://docs.anaconda.com/mkl-optimizations/
It's passed my error but had an another error :'( and this one;

Traceback (most recent call last):
File "crop-video.py", line 1, in
import face_alignment
File "", line 259, in load_module
File "C:\Users\mertn\anaconda3\envs\deepfake\lib\site-packages\face_alignment-1.1.0-py3.8.egg\face_alignment_init_.py", line 7, in
File "", line 259, in load_module
File "C:\Users\mertn\anaconda3\envs\deepfake\lib\site-packages\face_alignment-1.1.0-py3.8.egg\face_alignment\api.py", line 3, in
File "C:\Users\mertn\anaconda3\envs\deepfake\lib\site-packages\torch_init_.py", line 97, in
from torch._C import *
ImportError: numpy.core.multiarray failed to import

I reinstall the Numpy run again the code;

(deepfake) F:\DeepFake>python crop-video.py --inp assets/intro.mp4
Traceback (most recent call last):
File "crop-video.py", line 1, in
import face_alignment
File "", line 259, in load_module
File "C:\Users\mertn\anaconda3\envs\deepfake\lib\site-packages\face_alignment-1.1.0-py3.8.egg\face_alignment_init_.py", line 7, in
File "", line 259, in load_module
File "C:\Users\mertn\anaconda3\envs\deepfake\lib\site-packages\face_alignment-1.1.0-py3.8.egg\face_alignment\api.py", line 6, in
ModuleNotFoundError: No module named 'skimage'

Skimage is part of that scikit-image (face-alignment requirement)
And then.... run again the code;

(deepfake) F:\DeepFake>python crop-video.py --inp assets/intro.mp4
OMP: Error #15: Initializing libiomp5md.dll, but found libiomp5md.dll already initialized.
OMP: Hint This means that multiple copies of the OpenMP runtime have been linked into the program. That is dangerous, since it can degrade performance or cause incorrect results. The best thing to do is to ensure that only a single OpenMP runtime is linked into the process, e.g. by avoiding static linking of the OpenMP runtime in any library. As an unsafe, unsupported, undocumented workaround you can set the environment variable KMP_DUPLICATE_LIB_OK=TRUE to allow the program to continue to execute, but that may cause crashes or silently produce incorrect results. For more information, please see http://www.intel.com/software/products/support/.

See? Turn back to nothing... Anyone have solution for that?
(Sorry if i wrote anything wrong. My english is little problematic :D)
Thanks in advance for your interest.

Was this ever resolved? I am having the same issue as described.

I've got the same issue.


Bug 58506 - JMS Point-to-point sampler should offer an async using temp queue mode and prevent hangs by supporting a reply timeout

Bug 58506 - JMS Point-to-point sampler should offer an async using temp queue mode and prevent hangs by supporting a reply timeout.
Changed line 'executor = new ExtendedTemporaryQueueExecutor(session, sendQueue, getTimeoutAsInt());'
to accept timeout.

Hello @pmouawad,
Could you please review the changes. We need to have this functionality as part of our project.

Hello,
Thank you for your contribution.
My notes:

The class ExtendedQueueRequestor seems to come from existing code as per "This source code implements specifications defined by the Java Community Process." Are you the author of this code or not ? If not what is the license of this code ?
The code lacks javadoc for  ExtendedQueueRequestor
TemporaryQueueExecutor is not used anymore, it should be deprecated
The ExtendedQueueRequestor needs to be unit tested. Since we've setup Sonar on repository, we try to increase test coverage either with JUnit code or Test Plan that allows testing results. This code should be unit tested using mocks I think

Regards


added chordchart to ref and plugin ts files

Hi @liorpa7, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;


Bug in List View

Pull Request #54 introduced a bug that causes this:
[2016-11-02 12:30:22,130] ERROR in app: Exception on /list [GET]
Traceback (most recent call last):
  File "/usr/lib/python3.5/site-packages/flask/app.py", line 1988, in wsgi_app
    response = self.full_dispatch_request()
  File "/usr/lib/python3.5/site-packages/flask/app.py", line 1641, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/usr/lib/python3.5/site-packages/flask/app.py", line 1544, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/usr/lib/python3.5/site-packages/flask/_compat.py", line 33, in reraise
    raise value
  File "/usr/lib/python3.5/site-packages/flask/app.py", line 1639, in full_dispatch_request
    rv = self.dispatch_request()
  File "/usr/lib/python3.5/site-packages/flask/app.py", line 1625, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/torpaste/torpaste.py", line 137, in list
    status, data, code = logic.get_paste_listing(config, listFilters)
  File "/torpaste/logic.py", line 134, in get_paste_listing
    paste_list = b.get_all_paste_ids(filters)
  File "/torpaste/backends/filesystem.py", line 257, in get_all_paste_ids
    if get_paste_metadata_value(p, k) != v:
  File "/torpaste/backends/filesystem.py", line 214, in get_paste_metadata_value
    if get_paste_metadata(paste_id)[key]:
KeyError: 'visibility'

when listing the pastes from a docker container ran with:
docker run -d -p 80:80 -v /pastes:/torpaste/pastes --name torpaste --restart=always daknob/torpaste

As you can see, the optional variable TP_ENABLED_PASTE_VISIBILITIES is missing.

Okay, so the bug is that the visibility metadata is not available to all pastes, for examples the ones from a previous version of TorPaste. I will patch this and assume the visibility before is public, as always.

More specifically it is an issue in the backend.py filesystem. Patch coming up soon!

S**t, I didn't test this. I don't keep pastes between tests, although I clearly should.
That said, if no visibility meta is available, the "default visibility" (the first one in the config'd list of enabled visibilities) should probably be used, consistently with paste creation.

@j11e So far I have this:
diff --git a/backends/filesystem.py b/backends/filesystem.py
index aa80ca6..40c5f4b 100644
--- a/backends/filesystem.py
+++ b/backends/filesystem.py
@@ -211,9 +211,9 @@ def get_paste_metadata_value(paste_id, key):
              the key wasn't set
     """

-    if get_paste_metadata(paste_id)[key]:
+    try:
         return get_paste_metadata(paste_id)[key]
-    else:
+    except:
         return None

however without any more changes, the previous pastes are treated as unlisted:
if get_paste_metadata_value(p, k) != v:

The above if is simply not true, since I return None.


Enable SVG Code Hints

Mini-plan for improved SVG support:

[x] Split SVG into own language (PR #9969)
[x] Update Brackets SVG Code Hints extension because it's no longer required create an SVG language. Also update documentation because it's no longer necessary to manually switch to SVG language
[x] Merge SVG Code Hints into Brackets Core (after code cleanup, adding some unit tests, and code review).

There are some other SVG extensions in Brackets Registry that may also benefit from these changes.

PR: https://github.com/adobe/brackets/pull/10294

Code is now master and ready for Testing.


The "show all" button does not work after the first click

If you press the "show all" button, then you can not press it again. So to hide the display of the entire table is necessary to change the view mode

The "show all" functionality has now been changed to where it can be toggled on & off by switching between it and the "filtered" view, so closing this issue.


Fix bug in case study previewing

Trello: https://trello.com/c/yWNA28st/295-bug-case-studies-in-2i-don-t-appear-in-content-preview
Background
Case studies were not getting sent to the draft environment on update if they were in the submitted or rejected states.
What happened in the case of this particular document was:

it was created with a certain title, and the document was sent to draft content store properly with path /government/case-studies/honours-nominate-someone-who-has-improved-young-lives--2
it was then submitted to 2nd eyes review moving it into the submitted state
whilst in the submitted state, the title was amended. Because this was the first draft of the document the slug changed. Because of this bug, the document with the new slug was not sent to the draft environment.

Now when the user clicked the preview button they would get the 404 error because the content was missing from the preview environment.
Fix
The edition.draft? guard in the DraftEditionUpdater service object was too narrow. We should allow updates to editions in the submitted or rejected state as well.
This problem was compounded by the fact that there wasn't sufficient error handling around the call to updater.perform! in the Admin::EditionController. The consequence was that the #perform!
call would silently fail and the edition would be updated, but the draft would not be sent to the content preview environment.
Changing this required splitting up the #edit_as method, and rendered it redundant, so I've removed it completely.
I've tightend up the testing around this by adding a new functional test mixin for use in edition admin controllers should_send_drafts_to_content_preview_environment_for, and using this in the admin case study controller test.

:+1: good stuff


CDT Build issues

Warnings over Ricardian contracts are spamming the build, there should be a flag to disable this warning

Build times seem longer than expected, it seems to rebuild more than necessary

Warning about date.h including sstream which bloats wasm, we shouldn’t be bloating wasm just by including standard headers (unless we really need sstream). This may also be contributing heavily to build times

See #752 for discussion of 1.

On develop, I have not seen a noticeable change in build times. There will be one however once the tester stuff comes in, though that is to be expected.

Where are you seeing this? I don't think I've seen it on develop.

Note: these are about building contracts with CDT, especially when using CMake (2, 3).

All of these have been addressed.


Adds support for gunicorn multiprocess process

Solves the issue : https://github.com/vllm-project/vllm/issues/2573

Can you share some benchmarks of this vs master?

hey @NikolaBorisov alas the gpus that I got are taken back, weeping in the back 🥴. will add some benchmarks if I get them back next week, maybe !

@jalotra @NikolaBorisov,
I'm trying to test this and do some benchmarking for you. But encountered this issue:
RuntimeError: Cannot re-initialize CUDA in forked subprocess. To use CUDA with multiprocessing, you must use the 'spawn' start method

Tried adding the following:
torch.multiprocessing.set_start_method("spawn")

But sadly no luck!

I'm, personally a bit skeptical we need the multiprocessing, because usually vllm process just couple maybe 10 requests per second at max. Because of this I didn't think it is will have a big impact to make the webserver multiprocessing. As you saw because the current process was also doing some cuda work is not that trivial to make it multiprocessing. To make this work you will need to maybe split up the parts of vllm that deal with the GPU in another process. So you will end up with architecture like something like this: 10 web process that can take requests, but they have to pass the requests into one vllm engine process to feed the (1,2,4,8) gpu workers.
Anyway, for this PR to work you will need some more code.

hey @NikolaBorisov

vllm process just couple maybe 10 requests per second at max

any resource on this? I believe this should be pivoted on input + output token len, say it looks like 1000 tokens vs 100 tokens, then while batching via the PagedAttention method, the web server should be able to listen and batch to 10x more requests in a single forward pass but here since the web-server becomes a bottleneck, you won't be able to batch 10X, the idea of using gunicorn is to let go of this bottleneck.

So you will end up with architecture like something like this: 10 web process that can take requests, but they have to

pass the requests into one vllm engine process to feed the (1,2,4,8) gpu workers.
yes this could be done, can you try running the process with only 1 WORKER node once ?

hello hello, this work would need some of the core contributor to help out here, I think this is more about using vLLM for production use-cases and using uvicorn for this doesn't makes sense to me !


grpc::ServerContext::TryCancel crashes with segment fault (no matter what!)

I know this issue has be discussed before. But it seems it was dismissed.
In my case, my service starts listening to client streaming requests like so:

initialize grpc::ServerContext.
initialize grpc::ServerAsyncWriter with this context
start listening (AsyncService::Request<...>)
If I call grpc::ServerContext::TryCancel here (from any thread) I get a segmentation fault
If I call grpc::ServerContext::TryCancel after the streaming has started I get a segmentation fault as well.

I believe that this method should either be documented more clearly to explain under what scenario it can be called, or at least log / report the reason for the error. A simple fix that would actually cancel the RPC listening from the server side would be nice as well.
I use grpc v1.5.0 on Windows 10 OS.

Hi @eladm-ultrawis, has the RPC actually been made? Can you provide a quick repro?

@yashykt The RPC from the client side was not yet initiated. Like I tried to exaplain, the Server service is only listening to client requests. I could not find a way to "cancel" the service listening without shutting down the whole server. calling TryCancel always crashes.

yeah, you are right. The API does not have a way to cancel a RequestCall

@yashykt thank you for the update, is there a chance to implement the ability to cancel a RequestCall? I think it makes a lot of sense to be able to cancel such a call.

@eladm-ultrawis Please take a look at the async callback API. https://github.com/grpc/grpc/blob/master/examples/cpp/helloworld/greeter_callback_server.cc

@yashykt I don't follow.

We are moving to the new callback API world for gRPC C++, and you won't be dealing with some of the issues that you are facing with the CQ based API. Please try out the new API :)

@yashykt  I am using asio-grpc which is based on the completion queues. the callback API is not an option. opens up to many threads without being able to control it.

Using async callback API doesn't exactly resolve the root of the issue here. Correct me if I'm wrong, but there doesn't seem to be any way to cancel a request if the server isn't connected yet, even if the callback API is used. The only thing I have found to work around this is to set a deadline in the context so you can reassess on timeout. This polling method seems messy to me because it's difficult to come up with an arbitrary polling period which will work in all cases.


Breaks fonts in lot of pages

For example,
http://csswizardry.com/about/
With this plugin enabled, monospace font is shown, if not enabled, then correct font is shown. After some debugging I figured out that this extension is for some reason overriding font tried to be used by page.

the problem is that jsonfinder always injects its style.css regardless if json data is shown or not.
fonts form style.css priorize curier new on any non mac machines.

Fixed in 0.1.0