When updating a use case, setting the Score threshold to DISABLED does not seem to take effect.

Describe the bug
As per the following documentation, we can configure Kendra's Advanced RAG configurations. However, once we set the Score threshold to a value other than DISABLED, like LOW, we cannot revert it back to DISABLED in subsequent updates. On the web app, there is no error shown, and it appears to be successful, but the change is not reflected.
https://docs.aws.amazon.com/solutions/latest/generative-ai-application-builder-on-aws/advanced-knowledge-base-settings.html
If we update to set the Score threshold to DISABLED, the request body for the PATCH method to API Gateway does not include the ScoreThreshold parameter. In this case, it is considered necessary to perform processing such as deleting the item keys (ScoreThreshold, StorageCapacityUnits) from the DynamoDB table.
To Reproduce

Create a Generative AI Application Builder.
Deploy a new use case. Leave the default settings as is. Select Kendra for the Knowledge base type, and set a value other than DISABLED for the Score Threshold in Advanced RAG configurations.
Update the use case. Set DISABLED for the Score Threshold in Advanced RAG configurations.

As a result, the Score Threshold does not become DISABLED.
Expected behavior
The Score Threshold value in Advanced RAG configurations can be updated to DISABLED.
Please complete the following information about the solution:

[x] Version: v2.0.4
[x] Region: us-east-1
[x] Was the solution modified from the version published on this repository?: No changes have been made.
[x] If the answer to the previous question was yes, are the changes available on GitHub?
[x] Have you checked your service quotas for the sevices this solution uses?: This is not a service quota issue.
[x] Were there any errors in the CloudWatch Logs?: The Lambda function's create_config_string execution was successful.

Thank you @hiraken-w for the detailed bug report!
I was able to reproduce the issue and identify the root cause. The team will have a look at a fix and include a patch in our next release.
We'll let you know here once a fix is available. Thanks again!

Issue resolved in release v2.1.0


MoreCulling Compatibility

So currently using CIT items in item frames with MoreCulling results in the item looking normal.
I've attempted to fix this myself although I don't seem to be able to get CIT to work correctly in my development area.
The fix should be as simple as doing the same mixins from here except for here instead.
GetModel Vanilla - MoreCulling
RenderItem Vanilla - MoreCulling
That should fix all the issues.

I'll keep this in mind for when the item type gets rewritten but for now there won't be a hotfix for the current state of the cit type.


Junicode: update to 1.002 and correct license

Motivation for this change
Update Junicode to the latest available version. Among other changes, Greek alphabet has been moved into a separate font, Foulis Greek, included alongside Junicode variants.
While at it, I have noticed that the license in the metadata doesn't match the one on the project site, and changed that as well.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
The fonts at least look correct in fontforge.
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
Decreased from 3.0M to 2.2M
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.
meta.maintainers is not set; other than that, I think everything fits.

Would you consider adding yourself as a maintainer of this package? As you have already worked on it :)

I would personally prefer for the maintainer to be a bit more knowledgeable about the inner workings of the whole fontconfig thing, but I guess I can at least keep it up-to-date, so why not.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/134

Would you consider adding yourself as a maintainer of this package? As you have already worked on it :)

I would personally prefer for the maintainer to be a bit more knowledgeable about the inner workings of the whole fontconfig thing, but I guess I can at least keep it up-to-date, so why not.

The alternative here is no maintainer, so I'm glad you're willing to pick this up :) Also note that maintainership doesn't have to be exclusive, so if someone else wants to help out in the future they can add themselves too.

Perfect. Thank you and welcome to the team :)


Suggestion: Communications ability with the userport

It is possible to add userport communications ability for use in creating a terminal program or BBS system in the near future?   Enhanced communication routines with supported 2400 or even 9600 baud would be icing the cake.

The extension system is on the way, hopefully it will be ready for testing in a few days. Once it works, any kind of extensions can be added. If you request a new extension, please write a brief description with the following:

What new constants should the extension define (if any)?
What new commands should the extension define?

Command name and arguments
What does the command actually do?

What new functions should the extension define (if any)?

Function name, arguments and return type
What does the function do?

Preferably write some examples, for example how would you implement each functionality in CBM BASIC or assembly?

Need the following "like" commands:
OPEN {speed,parity,bit,stop}    OPEN 9600,N,8,1
GET #{port},A$  GET#1,A$  (Does not wait for A$ <> "")
PRINT #{port},A$
STAT #{port},A (CONNECT,DCD,DTR)

Thanks. I admit I'll have to do some reading in the topic. Is there an existing code library that could be included in the project and we could get its routines wired to the new commands?

There may be some useful code which could be pulled from CCGMS 19
https://csdb.dk/release/?id=173065


LibraryApi done and up to par with FollowApi style

LibraryApi is done and up to par with FollowApi style of coding and commenting.

Hi Daniel,
I got it. I am on it right away as it will not take much time.  I am watching this brilliant interviewer and of course Ronald Regan who is giving very smart responses to his questions.
Thanks for your patience and time.
Best Regards,
Akshay Srinivasan

Hi Daniel,
Sorry took so long. I am beggining now via pattern recognition to understand why these naming conventions of mine are going wrong.  Also the problem that came out of my understanding is the Spotify API naming conventions are inconsistent.  If you are using CurrentUser for Albumns but only using User for Shows then I expect that there should be a different meaning when one time they say Current User and when they say User only. This is why I was getting it wrong.
Thanks for your patience and time.
Best Regards,
Akshay Srinivasan

♥♥♥♥

Hi Daniel,
I could not figure out your logic of which test to mark as [TestCategory("User")].
First I thought by looking at your modifications to FollowApiTests that it is maybe only if the current user account is being modified i.e. the current user account is the target of the action.
Or maybe is it when the user account is being accessed either read or write in any form.
Please clarify this soon as you see this. I am awaiting your response and then can finish off everything and create pull request for EpisodesApi.
Thanks for your patience and time.
Best Regards,
Akshay Srinivasan


Add support for recovering a sandcats domain whose key has been misplaced

I imagine the flow for this is:
In installer, if you type help (like how none is special-cased), then the install script will say:
If you lost your sandcats domain, and you need it back, you can type its
name here.

If you need other help, type help again.
What Sandcats name do you need help with? asheesh.sandcats.io
OK! We've emailed you a token to enter into this installer that will
let you use that domain on this new Sandstorm install.
Please enter the token: acfd0ed6fc274e9d3e272d1bae6279df
OK! We'll use asheesh.sandcats.io for the install.

@phildini got stuck in this state, so this is probably a show-stopper before 1.0.

If you type a name that's already taken, we could present the option to recover that name then. I think that's what most people are likely to try.

I think I actually did try to put the same name in twice, in the hope it would give me some sort of recovery option.

+1, will expose this in the error messages

I believe this is closed now, as of #59.


How to use this plugin?

I want to change the order of import for SafeAreaView like so:
import { SafeAreaView } from 'react-native-safe-area-context'

instead of:
import { SafeAreaView } from 'react-native'

I have added this line in tsconfig but it does not seem to work:
"plugins": [
      {
        "name": "ts-plugin-sort-import-suggestions",
        // Matches `@/`, `../` and `./`, move them up in the suggestions (This is the default config if you leave it empty)
        "moveUpPatterns": ["react-native-safe-area-context", "@/", "\\.{1,2}/"],
        // Move `dist` down in the suggestions, by deafult it's `[]`
        "moveDownPatterns": ["dist"]
      }
    ]

After making this change I ran 'Typescript: Restart TS Server' in VSCode command palette.
How can I make it work? and is it possible to see the logs inside node_modules/ts-plugin-sort-import-suggestions/index.js

Hi @YoushaExT , apologies for not catching your issue earlier.
Have you configured the VSCode to use the TypeScript from node_modules? By pressing Shift+Cmd+P and typing Select TypeScript Version

To view the logs, please refer to the vscode dev doc:
https://github.com/microsoft/TypeScript/wiki/Writing-a-Language-Service-Plugin#setting-up-typescript-server-logging
code -logToFile true -file ./log.txt -level verbose

Thanks @houkanshan, It worked! The logs also worked, I was using console.log instead of info.project.projectService.logger.info.
Before:

After:


Alphabetical ordering and NatSpec in Vm

The current Vm does not have any particular structuring logic besides the Vm <> VmSafe dichotomy. When adding new cheats, it is unclear where they should be added.
Similarly, it would be nice if the comments used the NatSpec tag @dev, so that we get syntax highlighting (via VSCode extensions) and the ability to automatically generate a documentation UI with forge doc if we ever decide that we want to do this.
I happen to maintain an alphabetically ordered + NatSpec tagged implementation in PRBTest:
https://github.com/PaulRBerg/prb-test/blob/8d76ad63d1bfa0b16bb880cfe4620a9e7e6aaa19/src/Vm.sol
So, adhering to these two principles would also make it easier to keep my implementation in sync with Forge Std.

See https://github.com/PaulRBerg/prb-test/pull/33

See https://github.com/PaulRBerg/prb-test/pull/33

I thought they were grouped by category like how the foundry book groups cheats, but this isn't strictly enforced. Maybe grouping by category and sorting them alphabetically within groups so it doesn't turn into one big chunk of cheats?

@PaulRBerg what do you like about alphabetical ordering? It feels strictly worse to me because:

If you're looking for a specific cheat, you'd just cmd+F it anyway
It reduces discoverability, like @ruvaag said we tried to group cheats by category, so this makes it easy to find all cheats involving block properties, or the filesystem, etc

There are probably improvements to the groupings we currently have, e.g. making them more explicit, with consistent comment separators, and fixing out of place ones. I'd be supportive of a PR that cleans that up, but I don't think alphabetical as in https://github.com/foundry-rs/forge-std/pull/461 is the way to go

you'd just cmd+F it anyway

Well, that's the thing - I've trained myself not to use CMD+F that often, primarily because I use alphabetical ordering everywhere. And my guess is that, all else being equal, all English speakers would use CMD+F less often when code is alphabetically ordered.
But anyway. I agree it's a minor issue in the grand scheme of things.

It reduces discoverability

Fair point. But it also increases maintainability as per the OP (unclear where to add new cheats, having to remember to group them, etc.)

Thanks. I do still think it's worth keeping the logical grouping instead of alphabetical, and I'll start being better at enforcing the ordering going forward. I'll also create an issue to clean up the existing Vm file and will tackle that soon. Going to close this as a result :)


Fix PageSelectWidget JS syntax

Original JS raise exception: Syntax error, unrecognized expression #id_page_2 option[value]

related to https://github.com/divio/django-cms/issues/4443


fix(multistore): warn about real FC abilities on url translations

What
Currently, FC do not really handle translations properly (it only returns the root URL of the targeted store).
See : https://front-commerce.slack.com/archives/G01EV7MDN84/p1645526399798739?thread_ts=1645453463.274589&cid=G01EV7MDN84
How
Add a warning to inform our integrators the feature is not yet available.

@abailleulfc I overlooked the target branch. Since it is not related to our next release, it should target main directly.
Could you cherry pick the commit there please?

@real34 cherrypicked in main


load_from_disk resets llm_predictor

Is it possible to modify BaseGPTIndex.save_to_disk() to also save ServiceContext or should it be saved/loaded separetly?
The issue is, once an index is saved, important configurations in ServiceContext will have to be reset upon loading the index. For example, users might use gpt-3.5-turbo as their LLMPredictor when they call GPTSimpleVectorIndex.from_documents(docs, service_context=service_context), but then after saving and loading the index, they expect the same LLMPredictor to be set but this is not the case (it resets to default and more expensive davinci text model).

The index and service context are separate
You can pass in the service context when loading from disk
.load_from_disk("index.json", service_context=service_context)

Thanks. This is what I'm doing now but I thought it might be worth logging what service context is being used after loading the index. A lot of examples I'm seeing online overlook this and they would only notice it if they go through their OpenAI bill carefully.

@SajadAzami it's a good point, just put out a PR to address a piece of this


Add gsoci.azurecr.io and remove docker image

You're right, that's a bit better to read, I created a PR
https://github.com/giantswarm/vertical-pod-autoscaler-app/pull/263


[Custom app]: Spaces in beginning of the application's name not cut

EPAM AI DIAL chat version
0.17.0
How to reproduce
Steps:

create custom application with spaces in beginning of its name
Save changes
Open edit form for the app
Create publication request for app
admin review publication request for app

Actual result
3,4,5 - spaces in the beginning of the name was not cut

Expected result
Spaces in the beginning and end of the name should be cut automatically on application add/edit
Confidential information

[X] I confirm that do not share any confidential information

verified on staging successfully


IConnection aggregatedCells list is alway empty

Hi,
Array aggregatedCells in PrimaryCOnnection is always empty.
And Bandwith in neighbourgs cells is not mapped. My netmonster app have this info.
Can you check ?
Thanks

Hi,
this feature is not implemented yet. Field PrimaryConnection.aggregatedCells is part of initial design of the library and might change before stable version is released. If you are already building your codebase using NetMonster Core you can obtain cells serving as secondary for each subscription using this code:
 NetMonsterFactory.get(context).getCells()
   .groupBy { it.subscriptionId }
   .mapValues { (subId, cells) -> cells.filter { it.connectionStatus is SecondaryConnection } }

Please note that NetMonster app currently uses some guessing techniques that do not provide 100 % correct outputs and they will probably not appear in codebase of NetMonster Core.

External database that links earfcn, pci and location of base stations is required to properly identify secondarily serving cells. For this reason additional post-processing cannot be implemented in the library.
If you are willing to guess which cells with NoneConnection could have SecondaryConnection, then implement algorithm that will be able give you eci from pci + current location. Then you can compare cid and guess which cell is secondarily serving. Please note that each network provider uses different rules when assigning cid to each eNb.

I am thinking that now in 2023, maybe half the cells in the United States are affected by this bug, and thus being marked invalid and thrown away. See https://github.com/zamojski/TowerCollector/issues/205 @zamojski


Off by one error: Mininum password length says it needs 10 chars but actually wants 11

As per subject: the password “pass123456” fails but “pass1234567” works.

dupe


DBZ-5390 Reorder relational_property definition to give priority to out_of_line_constraint

https://issues.redhat.com/browse/DBZ-5390

Upstream PR - https://github.com/antlr/grammars-v4/pull/2714

@Naros @ani-sha Applied, thanks


failed to re-register debuggee project-id: Error: Could not refresh access token

Environment details

App Engine Standard nodejs10
@google-cloud/debug-agent version: ^3.0.0

This is happening randomly every few minutes. I can see it in the application logs on stackdriver logging.

This error is typically due to a transient network connection problem. We do a back-off on the network requests and try again later.
If this is not causing any actual issues other than logs, I am inclined to close this issue. If you disagree, please leave a comment here.


Mutual recursion in an inheritance chain

Added 4 test cases in which mutual recursion occurs in an inheritance chain

@APotyomkin I updated the README file and made the format of the YAML strict. Please, modify your PR to make sure it strictly abides the format.

 Please look at the PR

@APotyomkin thanks!


SWDEV-444683 - Use the api provided by rocm-core to get ROCm install path rather than using hard coded paths

RVS library librvslib.so was having hard coded paths, since hard coded strings provided by build scripts were used to get ROCM_PATH. Use the api provided by rocm-core for getting the same. Also added package dependency to rocm-core

Was the code build and tested ? Also, package created, installed and tested ?

The code was build using build scripts and using the steps in RVS git hub.
Package was created and the basic tests mentioned in RVS repo has been executed. No failure observed


(do not merge) test nesting SQL subcommands in nav

Per DOC-5850, we might try nesting SQL subcommands underneath their parent commands in the Reference nav. This PR shows an example of doing so underneath ALTER TABLE (v22.2 doc).
Preview link (see nav): https://deploy-preview-15276--cockroachdb-docs.netlify.app/docs/v22.2/alter-table.html
This is definitely not ready to go, as-is. A few observations:

Depending on browser window width, side nav doesn't expand enough to show most/all of the nested statement name.
Because ALTER TABLE itself has to open the alter-table.md doc, clicking the arrow does not hide the nested subcommands after the list is shown, but only opens alter-table.md again.
When you open a SQL statement doc, the sidebar doesn't actually scroll to show that statement in the nav, which diminishes the utility of using the nav to display the subcommands in context. I may have imagined that this behavior happened at all, and am only realizing it because the SQL statements list is so long (@nickvigilante can you confirm that the nav doesn't behave this way?).

This change is

@taroface Confirmed as far as I know that navgoco (the software we use for our sidebar) doesn't automatically scroll to the position of the open item. Can you log a ticket for that?

I finally got a good chance to look at the output. I think I'm leaning much more toward including all subcommand docs in the main command document, using _includes. It has the added benefit of making the actual command navigation much simpler.

I finally got a good chance to look at the output. I think I'm leaning much more toward including all subcommand docs in the main command document, using _includes. It has the added benefit of making the actual command navigation much simpler.

Thanks for looking! Initial feedback from the Queries team was also that all subcommands on one statement page is preferable to what we have now.

Closing; this will be addressed soon with https://cockroachlabs.atlassian.net/browse/DOC-5850


Pull request from parser to main

This pull request contains all modification added to parse user command while the program is running

Kindly consider removing InputParser of the compilation unit


Please update Brew package for Mac

Current version of Tundra in Brew is 2.01

I'd love to, but the Tundra formula was contributed by someone else. The brew project frowns upon project owners pushing their own stuff, so someone else will have to submit a PR to them with the version bump.

@jurgen-kluft try paging @kainjow

I no longer use Tundra. However looks like you should just be able to update the url and sha256 fields.

This looks like it's been updated upstream in homebrew to 2.06, so someone is tracking Tundra's releases now. Closing.


Use "maximum 32-bit seconds Unix time" as "infinite" expiry

Signed URLs without an expiry time are not intended to work - the
expiry time is intended to be a required query parameter. This
change comes as close to being backward-compatible as is feasible.
cc @frankyn

Thank you both!


Papers to post

Analysis of Impartial Quality Measurements on Indian Broadband Connections https://www.dropbox.com/s/zxqpqgxpj4db76f/ndt.pdf?dl=1

Analysis of End-User QoE in Community Networks http://people.ac.upc.edu/leandro/pubs/ACMDEV15-QoE.pdf


Feat issue 33604 callable key prefix

Trac ticket number

ticket-33604
Branch description
Based on the great work of @alexei  I try to make my first pull request. The origin PR #15553 seems to be abandoned. So I tried implement the things mentioned in the comment. I hope I didn't missed something.
This update allows to set the key_prefix as a callable.
Discussions:
https://code.djangoproject.com/ticket/33604
https://groups.google.com/g/django-developers/c/UbD1DkV1uPo
Checklist

[x] This PR targets the main branch.
[x] The commit message is written in past tense, mentions the ticket number, and ends with a period.
[x] I have checked the "Has patch" ticket flag in the Trac system.
[x] I have added or updated relevant tests.
[x] I have added or updated relevant docs, including release notes if applicable.
[x] I have attached screenshots in both light and dark modes for any UI changes.

Cool! Maybe you'll get luckier 👍


use only default https ports for https check

spring boot uses all high ports and there are some trouble if ports end with 443 (e.g. 24443, ...)
I think for a default behavior only the low ports should be used.

@rpahli Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@rpahli Thank you for signing the Contributor License Agreement!

Codecov Report

Merging #2000 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master   #2000      +/-   ##
=========================================
+ Coverage    76.3%   76.3%   +<.01%
=========================================
  Files         209     209
  Lines        6376    6377       +1
  Branches      794     794
=========================================
+ Hits         4865    4866       +1
  Misses       1173    1173
  Partials      338     338

Impacted Files
Coverage Δ

...loud/netflix/ribbon/DefaultServerIntrospector.java
75% <100%> (+8.33%)
: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 7425f97...ccf8f86. Read the comment docs.


Configure globally

It would be good if we could set global defaults for the configuration, particularly for language
$.fn.yadcf.defaults = {}
I think it would just be a simple case of adding $.extend({}, $.fn.yadcf.defaults, params) here https://github.com/vedmack/yadcf/blob/master/dist/jquery.dataTables.yadcf.js#L4642

Do you mean like this?
* Global Parameters (per table rather than per column)
*
* Usage example yadcf.init(oTable,[{column_number : 0}, {column_number: 3}],{cumulative_filtering: true});

The third argument is being applied to all columns

No, I mean global for every table. https://github.com/vedmack/yadcf/pull/287 requires you to set lang on every table you want to change the language? I just want to set a languages definition once and it apply to every yadcf instance.

@vedmack will you accept a PR for this? and when is the next release expected?
I need a single place to set languages rather than
// template1.html
oTable.yadcf([/* unique column definition */], {
  language = {'select': 'Select value'}
});
// template2.html
oTable.yadcf([/* unique column defintion */], {
  language = {'select': 'Select value'}
});

Sure, a PR is always welcomed

Sent https://github.com/vedmack/yadcf/pull/617

thanks! I will try to merge it when I'll find some spare time

@vedmack sorry to ping but any chance of seeing this merged and released?


cant set active / selected items

Hey, i am using ng2-select and i am quiet satisfied.
I spent the last 3 hours to set the initial data of the select.
My first attempt was to use "active:
code:
<ng-select  [active]="active"...
public active = ['Amsterdam'];

Compiler:

Can't bind to 'active' since it isn't a known property of 'ng-select'

After that I tried around with some hacks but so far I couldnt find a solution.
I also want to use it as a multi select, but i am already failing on the single selection.
Unfortunately there is no demo and i am out of ideas..
Another little thought:
CSS height is not working correctly.
I have to set the height initially but if it changes (like on multiselect), therefore the hight stays the same..
Thank you for this very helpful component!
Greetings
Emanuel

Dupe of #379? I too have been trying to get [active] to work. On #379, he says to use [initData]

nope tried that too, same didnt work  :(

I had this problem, it was actually giving me errors for using "data" to populate the initial selected values in rc5, because there is an event called "data" which causes a conflict.
So reading the source I did bump into [initData] and it worked.

I think maybe I maybe had some conflicts with the RC versions. initData its fine, on git readme its still false and should be updated :-)

I'm having the same issue here. It looks like the right attribute should be [active] right now in stead of [initData], but I keep getting this error:
selectedItems.map is not a function

@kuiperr005 same here
[active]="value[0]" then
selectedItems.map is not a function

[active]="value[0]" must be the array

@karmis Do you have a working example?

same here.. selectedItems.map is not a function

I have found the solution, you need to set the active attribute like this:
[active]="[arrayWithObjects[objectIndex]]"
In most cases this means you should refactor your data object to this format to get It working.

Thanks! In my case it was just [active]="[object]", because it was a single object!

@kuiperr005 Are you able to change a value after setting the [active] data binding? I get the error self.context.onAssignedSelected is not a function after changing a value then..

anybody had success with an array of objects as active property? how to do this?

import { SelectComponent } from '../node_modules/ng2-select/';

export class SelectButtonComponent {

@ViewChild(SelectComponent) select: SelectComponent;

     public method(e:any):void {
          this.select.active = [value.toString()];
     }

}

I used the way that directly change its active value in 'SelectComponent',  hope it works for you.

I found this when trying to solve in reactive forms. Need to setValue like this:
this.select.active = [item.category];                     this.form.setValue({                         name:this.item.name,                         description:this.item.description,                         category:[this.item.category.name],                     });

thanks @XidongHuang

just using [active]="myInitialStringArray" worked for me

[items] already contains all items to be displayed .. I just want to select one of those to be selected initially .. how to do that? why i should give the data again via active ?


⚠ Samsung has degraded performance

In 7b9e25d, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2183 ms

Resolved: Samsung performance has improved in 64c411d.


Package chisel4ml.transform us broken

I tried to test your tool, and I followed the installation instructions, However, when I ran testing, I get the following:

How Can I fix this issue?

Hello.
Thanks for trying out the tool. Which branch did you clone?

Yes that is a correct one. hm. this is wierd. Which operating system are you using?

Ah. I think I see the problem. Likely your python version is to low. Could you try with python 3.10 or 3.9?
We still didn't get the continos integration cycle properly running so problems with versions could happen.

I think you are correct I am using Python3.8, I will check this. Thank you :)


Calculate unique IDs for control leaf nodes

Adds a new method ir::Component::control_with_node_ids that modifies the
control program to attach @node_id which unique number for each enable
and invoke node.

I vaguely remember that @EclecticGriffin had to do something similar for the register unsharing implementation. Maybe we can rewrite things to use this method? Also, does it actually help with what you were trying to do?

Relevant issue is #510.

Right, the issue with the register unsharing was that invoke statements needed a unique label or name so that their uses of variables could be correctly propagated, distinguished, and re-written. The solution at the time was originally using attributes but changed to the more transient raw-pointer based approach to metadata. Given that the pass only needs a unique ID, I think it could be re-written with this, though it wouldn't significantly change the pass's logic. I don't necessarily see a benefit to doing this rewrite beyond general aesthetics, but I'm happy to add it to my task list.
I also think that it doesn't necessarily encompass a general notion of metadata (#510) but thus far we haven't seen a need for this type of metadata emerge so perhaps it's a moot point

Closing this for now. #671 implements this for tdcc. We can make it generally available if another pass needs it.


ISSUE-765: Standardize copyright/license headers

Closed since headers don't need to be updated, it turns out.


Merge pull request #1 from netty/4.1

update fork merge
Motivation:
Explain here the context, and why you're making that change.
What is the problem you're trying to solve.
Modification:
Describe the modifications you've done.
Result:
Fixes #.
If there is no issue then describe the changes introduced by this PR.

merge pull


Use redirect flow to acquiretokensilent,not stable!

Library

[ ]<EMAIL_ADDRESS>or<EMAIL_ADDRESS>
Description
I'm using a redirect flow to perform authentication in the mobile browser,It works well in localhost,but after I deploy to the server,sometimes,when acquireTokenSilent ,the authorize will happen twice at the time,I put some details in this link
https://stackoverflow.com/questions/62339746/acquiretokensilent-failed-when-using-msal-js
Examples:

"Why authorize happen twice?"

this error code returns:token_renewal_error

Can you provide the rest of your msal-related code, like where getTokenRedirect is being called?

@jmckennon
I called my function in componentDidMount,and regarding to the call  this.props.onFetchTemplate();
and define it in my saga file     const res = yield call(repository.findAll, "inputhere");
and here is my call
export class TemplateRepository implements IRepository<ITemplate> {
    constructor() {
    }
    public findAll = async (queryOptions: string): Promise<ITemplate[]> => {
        let token;
        var storage = window.sessionStorage;
        if (isMobile) {
            storage.setItem('getTemplate', '2');
            token = await getTokenRedirect();//TODO lOAD TWICE
        } else {
            token = await getTokenMsal();
        }
        console.log("token when findall templates" + token);
        const reqHeaders = new Headers();
        const bearer = "Bearer " + token;
        reqHeaders.append("Authorization", bearer);
        reqHeaders.append("Access-Control-Allow-Origin", "*");
        reqHeaders.append("Content-Type", "application/x-www-form-urlencoded");
        const options = {
            headers: reqHeaders,
            method: "GET"
        };

        return new Promise<ITemplate[]>((resolve, reject) => {
            fetch(GRAPH_ENDPOINTS.SITETEMPLATE, options)
                .then(response => {
                    // This is where you deal with your API response. In this case, we
                    // interpret the response as JSON
                    if (response.status === 200) {
                        response.json().then((responseJson: any) => {
                            storage.setItem('getTemplate', '1');
                            const result = responseJson;
                            const mapObject = (result as any[]).map<ITemplate>(res => {
                                const createdTemplate = this.mapITemplateFromServiceResult(res);
                                return createdTemplate;
                            });
                            return resolve(mapObject);
                        });

                    } else {
                        console.log("res:" + response.text())
                        const err = new Error("401")
                        reject(err);
                    }
                })
                .catch(error => {
                    reject(error);
                });
        });
    };`

and when first loading the page and tried to get token,such error code will returned.
Thanks!

You shouldn't try to call one of the login APIs when your app is loaded inside an iframe.
You can also try using a blank page as a redirectUri as of<EMAIL_ADDRESS>It is recommended to set the redirectUri for the silent calls to a blank page, say https://yourhost/auth.html instead of the SPA's homepage.
Could you also implement the msal Logger and send us the logs? That will help us see what's going on in this use case. You can read about this on the Microsoft docs.

@jmckennon how would you load the app, if after each login you redirect to a blank page?

@ryandegruyter when you call acquireTokenSilent, it sends off a request to the server. The original call still sits monitoring the iframe for the response, even if you redirect to a different page. A blank page works well because it avoids reloading any application-level or msal code that may unknowingly interfere with the original call. Once you get the response back, you can reload the page or do whatever your app needs to do. This strategy works with silent and pop calls.
Additionally, you can have more than one redirectUri, and also set them per request. This allows you to set the blank page as the redirectUri only for silent calls and not for redirect calls where this strategy wouldn't work.

Ok, we don't use acquireTokenSilent. only loginRedirect and the MSALGuard for Angular

@jmckennon Yes,I used to use a blank page for Popup login,It works well.But if I use blank page It will redirect the application to a blank page.How can I load the application after login? And do you mean I can only use it in popup and silent calls,if so how can I have more than one redirect uri?Does it means I have to maintain different auth variable when call login redirect and silent call? Thanks!

Yes,I used to use a blank page for Popup login,It works well.But if I use blank page It will redirect the application to a blank page.How can I load the application after login?

If you are using the redirect flow, setting the redirectUri as a blank page will not work. This will only work in popup and silent flows.

how can I have more than one redirect uri? Does it means I have to maintain different auth variable when call login redirect and silent call?

You can register multiple redirectUri in the Azure portal, in your app registration. You can then pass in redirectUri to your msal config as a means of using the registered ones, so that a particular request will go to that specific uri. From this documentation page:

redirectUri: Optional. The redirect URI of your app, where authentication responses can be sent and received by your app. It must exactly match one of the redirect URIs you registered in the portal. Defaults to window.location.href.

The idea is that you can make this different for each request type. So only when doing acquireTokenSilent, you may give the request a redirectUri of a blank page.

Closing as the question has been answered and there has been no further activity. Let us know if you're still having issues.


Feature/dashbord

Before it is reviewed this must be merged pool request number 13 to see more clearly what are the changes !
https://github.com/yanachkov-devix/JobriX/pull/13


Canceling drag distribution should not pick up items that existed in slots before drag began.

Also fix (left click drag) even distribution doing the same thing and modifying slots that had items before the drag was started.
Also allow (left click drag) even distribution in the case of starting the click drag on a slot with a pre-existing item matching the held item's id.

All slot item instances should be added to an item instance array, that gets checked when canceling and evenly distributing to ensure slot item instance values do not change.


Multi-Lingual Sentence Embedding

Can someone help me to get an example of multi-lingual sentence embedding? I want to extract the sentence embedding for Hindi using the Multi-lingual sentence embedding model.
Thanks.

Hi, I have the same problem. I have seen that in the go_bot tutorial they use glove embeddings by downloading them and passing them to the embedder like:
`        download_resource(url="http://files.deeppavlov.ai/embeddings/glove.6B.100d.txt",
dest_paths=[bot_path])
     gobot_config['chainer']['pipe'][-1]['embedder'] = {
    "class_name": "glove",
    "load_path": os.path.join(bot_path, 'glove.6B.100d.txt')
}`

However,  I would like to use Spanish embeddings because my dataset is in Spanish but I am not able to know where are they allocated. I have tried to access "http://files.deeppavlov.ai/embeddings/" but it does not work as a link, so I do not know where to look for those pre-trained embeddings.

@PaGul Check if below url helps you for the embeddings. I 'm trying to find a way to use the Multi-ligual sentence embedding for Hindi. Let me know if you find anything useful.
http://docs.deeppavlov.ai/en/master/features/pretrained_vectors.html

Hi, @ashispapu,
You can find an example of getting embeddings using Bert in our documentation: http://docs.deeppavlov.ai/en/master/features/models/bert.html#bert-as-embedder
There is also a configuration for getting only sentence-level embeddings: https://github.com/deepmipt/DeepPavlov/blob/master/deeppavlov/configs/embedder/bert_sentence_embedder.json
you can use it like so:
from deeppavlov import build_model, configs

m = build_model(configs.embedder.bert_sentence_embedder, download=True)

texts = ['Hi, i want my embedding.', 'And mine too, please!']
sent_max_embs, sent_mean_embs, bert_pooler_outputs = m(texts)

Thanks @yoptar


Inquiry: ERC-4337 (Account Abstraction) Support in Tradetrust

Context
I'm exploring integrating Tradetrust with ERC-4337 account abstraction standards and would like clarity on current support.
Specific Questions
Does the current Tradetrust SDK or platform natively support ERC-4337?
Are there recommended approaches for implementing account abstraction with Tradetrust contracts?
If not directly supported, what would be the recommended integration strategy?
Use Case
Implementing gas sponsorship
Enabling flexible wallet interactions
Improving user onboarding experience
Technical Details
Interested in compatibility with:
Web3Auth
Bundler clients (pimlico / biconomy / etherspot)
Smart account implementations
Additional Context
Our goal is to enable seamless, sponsored transactions for document issuance and transfer while maintaining Tradetrust's security and verification mechanisms.
Any guidance or insights would be greatly appreciated.

Hey there.
Just an informal reply. We do not have any guide or recommendation.
We hope to see creative things being done by our collaborators like yourself!


Relocated filetype testing to be app service's responsibility

@adamCaldwell for your visibility:
I changed the location of responsibility for what is testing the filetypes - this is to keep the SQS service generic (it'll read whatever message is on the queue) - the app service is what should care about the message's content. I also had to update the test resources - when you receive a message from SQS it gets wrapped in a whole load of metadata like a MessageId, ReceiptHandle, etc, so this is representative of what would be received.

@adamCaldwell separately, I also had to rework the asynchronicity. By moving the upload to S3 call outside the scope of the async function called by readFile, it means that the S3 upload was happening before the file had been read and opened (and therefore a blank file was being uploaded). I've therefore put it back inside the scope of that async method and wrapped the whole thing in a promise to make the test work


dsl2lsd compiller

Can you make it usable for compilling .LSD dictionary from .DSL source? (without need original ABBYY DSLCompiller)

Compiling DSL wery wery hard process!!


Use context to signal client open state

Continuing #202, this PR replaces the client.State.done atomic int with a context value.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

11 of 11 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 98.603%

Totals

Change from base Build<PHONE_NUMBER>:
0.1%

Covered Lines:
5154

Relevant Lines:
5227

💛  - Coveralls


tag it

according to https://github.com/golang/dep/blob/master/docs/FAQ.md#how-do-i-roll-releases-that-dep-will-be-able-to-use it should be something like x.x.x. Add a prerelease tag if you are not totally sure the api is stable yet, such as x.x.x-beta. Otherwise you might also assume a breaking change api is acceptable, as long as the version does bump appropriately (major).

Good advice, tks.


Windows support

Initial Windows support. Make it compile as host/target for the start.

You can also test locally "./tools/syz-env make presubmit".


Saga-sized dashboard

Sonar has been running on Saga for several years and we have lots of historical data.  The data are old-style - untagged and without gpu information, and the cpu_pct field is not very useful because it has the wrong semantics - but we have them and we can create a Saga dashboard in Jobanalyzer.  This will be a new challenge because Saga has many more nodes than Fox (364) and a linear list of nodes will not be practical.
EDIT: Most of this is working, see subsequent comments and PR #374.  And data are coming in from Saga.  Remaining tasks (evolving):

[x] Proper backend / machine config data for Saga nodes
[x] Dashboard configuration data to display meaningful columns
[x] Dashboard changes to make use of those config data
[x] misc pages should honor config settings, eg machine-details should / should not show violators, zombies depending on settings
[x] Dashboard config data should include the default node selection expression
[x] #381
[x] #384

372 nodes in these groups according to doc:

200 standard compute nodes, with 40 cores and 192 GiB memory each
120 standard compute nodes with 52 cores and 192 GiB memory each
28 medium memory compute nodes, with 40 cores and 384 GiB of memory each
8 big memory nodes, with 3 TiB and 64 cores each
8 GPU nodes, with 4 NVIDIA GPUs and 2 CPUs with 24 cores and 384 GiB memory each
8 login and service nodes with 256 cores in total
6.6 PB high metadata performance BeeGFS scratch file system

Node names appear to be c\d+-\d+, gpu-\d+-\d+, hugemem-\d+-\d+.
The ml/fox dashboard is a machine-centric view with the extra feature that nodes that are in trouble are sorted to the top.  For bigger systems, we'll need to probably have a selection feature so that it's possible to look at a node when information about it is needed, but most of the time the dash needs to have some selection criteria and needs to show the nodes that fit those criteria.  Criteria could be "trouble", "loaded", "idle", "any" maybe + perhaps a selector for the type of node + optionally a regex to match against node name?  Ie the dash becomes a live search tool against the current state of the system.  Node names would still be hyperlinked.

Experimental code for this is live for Saga and Fox (currently Fox is the more impressive demo as we don't have data for Saga quite yet.)

This is done.  I'll spin off the matter of adding more aggregate plots as a separate low-pri bug.


ERROR: for nginx  driver failed programming external connectivity on endpoint

WARNING: Image for service installer was built because it did not already exist. To rebuild this image you must use docker-compose build or docker-compose up --build.
Creating dockerizedmagento_mysql_1
Creating dockerizedmagento_fullpagecache_1
Creating dockerizedmagento_sessions_1
Creating dockerizedmagento_cache_1
Creating dockerizedmagento_solr_1
Creating dockerizedmagento_php_1
Creating dockerizedmagento_nginx_1
ERROR: for nginx  driver failed programming external connectivity on endpoint dockerizedmagento_nginx_1 (f4231c1e80b680a8a59961cb3658c55d2b64cc3a3e980e09f8b94531a608892f): Error starting userland proxy: listen tcp <IP_ADDRESS>:80: listen: address already in use
Traceback (most recent call last):
File "", line 3, in
File "compose/cli/main.py", line 63, in main
AttributeError: 'ProjectError' object has no attribute 'msg'
docker-compose returned -1
caner@vegan:~/magento-caner/dockerized-magento$ ./magento start
dockerizedmagento_fullpagecache_1 is up-to-date
dockerizedmagento_solr_1 is up-to-date
dockerizedmagento_mysql_1 is up-to-date
dockerizedmagento_sessions_1 is up-to-date
dockerizedmagento_cache_1 is up-to-date
dockerizedmagento_php_1 is up-to-date
Starting dockerizedmagento_nginx_1
ERROR: for nginx  driver failed programming external connectivity on endpoint dockerizedmagento_nginx_1 (3e4887cb50ff899b19f660d3886f03d6682cf6332373912c0aa9e3932f4d8e5c): Error starting userland proxy: listen tcp <IP_ADDRESS>:80: listen: address already in use
Traceback (most recent call last):
File "", line 3, in
File "compose/cli/main.py", line 63, in main
AttributeError: 'ProjectError' object has no attribute 'msg'
docker-compose returned -1
caner@vegan:~/magento-caner/dockerized-magento$
after a whihle, the error is when i open the page

403 Forbidden
nginx/1.11.3

which are given from console :

installer_1      |
installer_1      | Fixing filesystem permissions
installer_1      | Installation fininished
installer_1      | Frontend: http://dockerized-magento.local/
installer_1      | Backend: http://dockerized-magento.local/admin
installer_1      |  - Username: admin
installer_1      |  - Password: password123

images are

fabf508632a6        dockerizedmagentomaster_installer   "/bin/install.sh"        4 minutes ago       Up 4 minutes                                                   dockerizedmagentomaster_installer_1
44a8db375577        nginx:latest                        "nginx -g 'daemon off"   4 minutes ago       Up 4 minutes        <IP_ADDRESS>:80->80/tcp, <IP_ADDRESS>:443->443/tcp   dockerizedmagentomaster_nginx_1
73883d1ad863        dockerizedmagentomaster_php         "php-fpm"                4 minutes ago       Up 4 minutes        9000/tcp                                   dockerizedmagentomaster_php_1
6c7e24a5f1b6        dockerizedmagentomaster_solr        "/usr/bin/java -Xmx10"   4 minutes ago       Up 4 minutes        8983/tcp                                   dockerizedmagentomaster_solr_1
824765d3e79d        mysql:5.5                           "docker-entrypoint.sh"   4 minutes ago       Up 4 minutes        <IP_ADDRESS>:3306->3306/tcp                     dockerizedmagentomaster_mysql_1
515fde576688        redis:latest                        "docker-entrypoint.sh"   28 minutes ago      Up 4 minutes        6379/tcp                                   dockerizedmagentomaster_sessions_1
e6a583fda276        redis:latest                        "docker-entrypoint.sh"   28 minutes ago      Up 4 minutes        6379/tcp                                   dockerizedmagentomaster_cache_1
e449459d1355        redis:latest                        "docker-entrypoint.sh"   28 minutes ago      Up 4 minutes        6379/tcp                                   dockerizedmagentomaster_fullpagecache_1

Also

[RuntimeException]
installer_1      |   Magento folder could not be detected
installer_1      |

Hi @caneraydinbey.
The project was in a bad shape a few month ago because it hasn't been updated in a while.
Since then I have create a newer version which should work again.
Please clone the project in a different folder and try again.
Greetings
Andy


config-next: Tracking Issue

This is a tracking issue to keep tabs on the config-next refactoring.
The broad goals of this refactoring are to move away from the domain-specific API that package config currently exposes, to a data-fetching type, and a series of well-factored types and packages throughout the rest of LFS.
Specifically, this refactoring comes in a number of parts:

[x] Introduce Fetcher, implement EnvFetcher, and integrate with *config.Config.
[x] Update tests to not use mutable behavior like Set and SetAll.
[x] Refactor Fetcher type as described in https://github.com/github/git-lfs/commit/e32fabe37fc150b21a5d2d7594a5300c1388c5cb.
[x] Implement the GitFetcher.
[x] Remove mutable .gitconfig-related operations

[x] From package transfer
[x] From everywhere else

[ ]  Piece by piece, remove legacy calls to GetenvBool, replacing them with .Env.Bool (and etc).
[ ] Piece by piece, remove domain knowledge from the *config.Config type, in favor of the methodology described in https://github.com/github/git-lfs/pull/1390#issuecomment-236068550.

Acking for internal script

Acking for internal script.

Piece by piece, remove domain knowledge from the *config.Config type, in favor of the methodology described in https://github.com/github/git-lfs/pull/1390#issuecomment-236068550.

Taking this one off of the tracking issue, since this is now outside the scope of the config-next refactoring. As discussed with @technoweenie, a good first step would be to outline all of the places the config package is used irresponsibly from api.

Woo-hoo! 🎉


Python 3.5.0 image?

Now that Python 3.5 final has been released, it would be good to have an image for it!

See #62. :+1:


docs: add contribution documentation for contributors on Windows machines

Adds to the CONTRIBUTING.md to help contributors on Windows machines with the setup process.
The setup requires extra non-obvious steps, such as using WSL, which merits some documentation.

Thanks @go-to-k! I applied the changes and resolved the conversations.
Also added a small Note to direct users to keep the project in the WSL file system and not the "mounted" Windows C drive.

Also @go-to-k I see the PR docs:  is not supported in the validate PR title

Thanks updated as requested!
I think it may now make sense to add a general "setup" section for Linux/Mac users?
If so we can create a separate issue for this :)

Added the last comment. : https://github.com/open-constructs/aws-cdk-library/pull/17#discussion_r1579252577

I think it may now make sense to add a general "setup" section for Linux/Mac users?
If so we can create a separate issue for this :)

That's good! It's helpful!


Add support for typescript-mode

Since there is already a js-mode support, I don't see why not support typescript-mode too.
Image example:

Thanks. I've added the icon as typescript-mode_icon
@wizardlink Could you point me in the direction of some modes this should be active in? need this for elcord-mode-icon-alist.
And while you're there, could you check what the mode's name is? If it's something confusing/misleading we'd want something special for elcord-mode-text-alist
I'd really appreciate it! Busy week this one.

https://github.com/emacs-typescript/typescript.el
It is called typescript-mode. This is the only name/implementation I know of for this major mode.

Added in 4a53075
Thanks @wizardlink


[stable/prometheus-operator] Enable extra(.*) args for prometheus mountings

stable/prometheus enables a user to add parameters like
extraVolumeMounts
extraVolumes
extraHostPathMounts
extraConfigmapMounts
extraSecretMounts
Because my etcd leaves outside of the cluster it cannot be auto-discovered like normal k8s pods. Also authentication is needed to access it but the prometheus does not have the required secrets mounted.
This is a feature request to implement the availability of extraSecretMounts and extraHostPathMounts on the values.yaml of the stable/prometheus-operator.
Describe the solution you'd like
This parameter will be added under

prometheus.prometheusSpec.extraHostPathMounts
prometheus.prometheusSpec.extraSecretMounts

You should already be able to do this by using prometheusSpec.secrets and prometheusSpec.configMaps. Host path mounts aren't available, and would need implementation in the underlying coreos/prometheus-operator project.

I shall check, thanks

I'm using this feature in my own deployment, should be working

Yes, this worked thank you.


feat: seo improvements for card

Description

Resolves #1615
Switch out containing body to div whilst normal text content body should be p.
Switch out header to h2

Scope

[ ] Patch: Bug Fix
[x] Minor: New Feature
[ ] Major: Breaking Change

we've discussed this offline, and we decided not to work on these - will close all the SEO related PRs/issues


"//" rules seem to match even outside of the build root.

So I have a ghc-pkg (my bootstrap one) in /path/to/ghc-bootstrap/stage1/bin/ghc-pkg. Now hadrian calls need on any builder, which is totally fine. But the rule for Programs, which looks like
"//" ++ stageString stage -/- "bin" -/- programName context <.> exe %> \bin -> do

still fires, even though my build root is /path/to/_build.

I believe this is to be expected behavior:

https://hackage.haskell.org/package/shake-0.16/docs/Development-Shake.html#v:-63--61--61-
http://shakebuild.com/manual#toplevel-variables

@angerman I like your solution and I might factor it out into a separate PR soon (time permitting), becuase it might help simplify a couple of other things too. Hope you don't mind me stealing your trick :)
By the way, this may also be relevant: https://github.com/ndmitchell/shake/issues/376

I think this has now been fixed. If not, please reopen.


Date format output

Moments is deprecating MM/DD/YYYY and other formats; details: https://github.com/moment/moment/issues/1407
When passing the result of next dates occurence from .recur() moment-recur outputs date in MM/DD/YYYY which when passed back to moments is triggering the Date deprecation warning
Deprecation warning: moment construction falls back to js Date. This is discouraged and will be removed in upcoming major release. Please refer to https://github.com/moment/moment/issues/1407 for more info.
The code I'm using to generate the next date array
var interval = moment.recur({
		start: start_date,
	    end: end_date
	}).every( recurring_frequency ).days();
var nextdates = interval.fromDate( today_date ).next(1,"L");

then this is what triggers the warning
console.log( moment(nextdates[0]).format('YYYY-MM-DD') );

Use a format other than "L" - you can output the dates in any format you want.


Update node-canvas dependency

Any chance the version of node-canvas being used could be updated? The version referenced in package.json still uses waf instead of gyp so running it on windows is a nightmare.

Closing for now. If this is still an issue, please bring it up again.


Slack

@jhaber @gchomatas

rebased off of master to get changes to ModuleBuildDao & Service

Other than the above comments LGTM


Update CppDocGen.ts

I would like the @author-tag being generated for each function. Thanks.

Is there a reason you closed this?

Seems like a good feature

It sure does. Needs a bit more than the source code change but I would happily merge it.


Add missing single-quote

Just a typo fix: The fourth curl example was missing a quote.

Thank you so much @kkuchta


feat: Add API server-compatible Argo UI

Very much a WIP
Add a UI that will support all read/write operations from the new Argo API Server
Screenshots (prone to change – will update as features get added):

Closed in favor of #1897


[BUG] - Microsoft.Azure.ServiceBus - System.InvalidOperationException: Can't create session when the connection is closing.

Describe the bug
Getting an InvalidOperation Exception in our ExceptionReceivedHandler about being unable to create a session.
Exception or Stack Trace
System.InvalidOperationException: Can't create session when the connection is closing.
at Microsoft.Azure.ServiceBus.Core.MessageReceiver.d__86.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.Azure.ServiceBus.Core.MessageReceiver.<>c__DisplayClass64_0.<b__0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.Azure.ServiceBus.RetryPolicy.d__19.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at Microsoft.Azure.ServiceBus.RetryPolicy.d__19.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.Azure.ServiceBus.Core.MessageReceiver.d__64.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.Azure.ServiceBus.Core.MessageReceiver.d__62.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.Azure.ServiceBus.MessageReceivePump.<b__11_0>d.MoveNext()
To Reproduce
Not sure exactly how to reproduce this - it occurs intermittently.
Expected behavior
This exception should perhaps be ignored by the message pump similar to other exceptions which are ignored.
Setup (please complete the following information):

OS: [e.g. iOS] Windows
IDE : Visual Studio 2019
Version of the Library used - Microsoft.Azure.ServiceBus 4.1.0 (going to upgrade to 4.1.1 but I don't believe that will resolve this problem)

Additional context
Wondering if InvalidOperationException should be handled in the same way ObjectDisposed and OperationCanceledException are being handled in PR 8449 https://github.com/Azure/azure-sdk-for-net/pull/8449 and this commit:
https://github.com/Azure/azure-sdk-for-net/pull/8449/commits/008bb2b06b1d0a420be33900d62a04257f703fb4#diff-a4508926a30ad8c3ce214614ed2fd446
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x ] Bug Description Added
[x ] Repro Steps Added
[x ] Setup information Added

@schwartzma1, This seems to be same as #6940.
Could you check again if you are indeed using 4.1.0?

@nemakam - Yes so in the release the project that is using this has:

I am wondering not whether it should be ServiceBusCommunicationException or InvalidOperationException, but why is this exception raised to the caller all if the connection is closing? Can't it just be ignored on ServiceBus side like ObjectDisposed and OperationCanceledException are handled in this commit in the MessagePump.cs? : 008bb2b#diff-a4508926a30ad8c3ce214614ed2fd446?

Another occurrence - #11066

Hi @nemakam , to confirm, we are not explicitly closing anything. I can see from the Stacktrace the connection is indeed closing.

Okay connections can get closed once in a while in a distributed world. The only thing we could aim at here is to make sure we translate this exception into communication exception.

Once we change this into communication exception, the retry policy will kick in and retry if the client is not closed. So it will improve the condition drastically

Once we change this into communication exception, the retry policy will kick in and retry if the client is not closed. So it will improve the condition drastically

Does this mean it won't pop-up anymore in AI or the console output?

It will. By default the client will report every communication exception. Having few communication exceptions is expected and can happen and you could ignore it. If there a lot of such exceptions happening frequently then its something we should look at. For this, the client is will always report such exceptions if an operation failed with it.
Only times when exceptions could be swallowed is when the client was explicitly closed and the exceptions are related to that.

Hmm understand. Well, both AI and the console is being spammed a lot with service bus exceptions. Timeouts, exceptions like above, taskcancelled,... It's really annoying as it makes it really difficult to find value in there. Would it be possible for you (or one of your colleagues) and me to have a short call where I can showcase the behavior perhaps?

Small update, we went through AZ (paid) support channels as this issue existed for almost a year already and was consuming not only a lot of AI bandwith, but also taking in the available slots for snapshot debugs.
After a lot of back and forth, we tested one small change; We currently have (or had) a queueclient for sending, and one for receiving, both receiving a connectionstring(so not the servicebusconnection object).
We refactored this to one queueclient and literally all these exceptions have gone away.
So this makes me think there's an issue with having two queueclients, which should still be possible I guess (we have one for sending as this may never be blocked, but receiving we'd like to close upon high pressure, although we don't have that functionality currently).
I noticed the 7.0 supports this, correct? Any ETA for that?

@Mortana89 , sorry I think I didnt get few things. What support are you exactly looking at in 7.0?
Also, would you be able to provide numbers?
How many queueClients did you have earlier?
How many sessions do you have on avg at any time?
What are the sessionHandlerOptions that you used to use, and the values that you use now?

And you did mention "constant" in your description, but I wanted to confirm that again. Do you see these exceptions constantly at the same rate or do you see a burst of exceptions at sometime, and then remaining times you don't see any?

@Mortana89 , sorry I think I didnt get few things. What support are you exactly looking at in 7.0?
Also, would you be able to provide numbers?
How many queueClients did you have earlier?
How many sessions do you have on avg at any time?
What are the sessionHandlerOptions that you used to use, and the values that you use now?

Hi Nekam,
Constant yes, see following screenshot from AI before applying the change;

We had two seperate queueclients per queue, one for sending, one for receiving. Some microservices consume multiple queues and thus have more queueclients than others. We have roughly 60 microservice instances that were generating these exceptions.
It's difficult to put a number on the avg amount of sessions, we have peak load at night but this didnt represent in the AI logging. I'd say at any given time no more than 150.
Sessionhandleroptions are the same, nothing changed there. We use peeklock, maxtimeout of 30min, maxwait of 5sec and 100 concurrency max.

Also, the screenshot is with 50% ingestion sampling enabled!

Could you expand the first column "overall" and send a screenshot / maybe just copy-paste?

Its very surprising that switching from two queueClients to one reduces the exceptions. The pipelines are very independent. Do you think you could provide a sample snippet?
And also, you mentioned queue clients were created using connection string and not the connection object. That would mean even the connections are independent..
And, are you sure the exception you are seeing is from the queueClient that's handling the receives and not the sends?

Hi Nekam,
It looks as if it's for receiving where the exceptions are thrown as they all come from the messagepump;
Microsoft.Azure.ServiceBus.ServiceBusException:
at Microsoft.Azure.ServiceBus.SessionClient+d__37.MoveNext (Microsoft.Azure.ServiceBus, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7e34167dcc6d6d8c)
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at Microsoft.Azure.ServiceBus.SessionReceivePump+d__20.MoveNext (Microsoft.Azure.ServiceBus, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7e34167dcc6d6d8c)
Message:
The connection was inactive for more than the allowed 300000 milliseconds and is closed by container 'LinkTracker'. TrackingId:5abed6bf93f841e484cbe107e93c1c40_G2
It's always per two exceptions. One of above also triggers the taskcancelled exceptions:
Microsoft.Azure.ServiceBus.ServiceBusException:
at Microsoft.Azure.ServiceBus.SessionClient+d__37.MoveNext (Microsoft.Azure.ServiceBus, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7e34167dcc6d6d8c)
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at Microsoft.Azure.ServiceBus.SessionReceivePump+d__20.MoveNext (Microsoft.Azure.ServiceBus, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7e34167dcc6d6d8c)
Inner exception System.Threading.Tasks.TaskCanceledException handled at Microsoft.Azure.ServiceBus.SessionClient+d__37.MoveNext:
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at Microsoft.Azure.Amqp.TaskHelpers.EndAsyncResult (Microsoft.Azure.Amqp, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35)
at Microsoft.Azure.Amqp.IteratorAsyncResult1.StepCallback (Microsoft.Azure.Amqp, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35)    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)    at Microsoft.Azure.Amqp.AsyncResult.End (Microsoft.Azure.Amqp, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35)    at Microsoft.Azure.Amqp.AmqpCbsLink+<>c__DisplayClass4_0.<SendTokenAsync>b__1 (Microsoft.Azure.Amqp, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35)    at System.Threading.Tasks.TaskFactory1.FromAsyncCoreLogic (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at Microsoft.Azure.ServiceBus.Amqp.AmqpLinkCreator+d__11.MoveNext (Microsoft.Azure.ServiceBus, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7e34167dcc6d6d8c)
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at Microsoft.Azure.ServiceBus.Core.MessageReceiver+d__101.MoveNext (Microsoft.Azure.ServiceBus, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7e34167dcc6d6d8c)
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at Microsoft.Azure.Amqp.FaultTolerantAmqpObject1+<OnCreateAsync>d__6.MoveNext (Microsoft.Azure.Amqp, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35)    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)    at Microsoft.Azure.Amqp.Singleton1+d__13.MoveNext (Microsoft.Azure.Amqp, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35)
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at Microsoft.Azure.Amqp.Singleton`1+d__13.MoveNext (Microsoft.Azure.Amqp, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35)
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at Microsoft.Azure.ServiceBus.Core.MessageReceiver+d__83.MoveNext (Microsoft.Azure.ServiceBus, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7e34167dcc6d6d8c)
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at Microsoft.Azure.ServiceBus.RetryPolicy+d__19.MoveNext (Microsoft.Azure.ServiceBus, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7e34167dcc6d6d8c)
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at Microsoft.Azure.ServiceBus.RetryPolicy+d__19.MoveNext (Microsoft.Azure.ServiceBus, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7e34167dcc6d6d8c)
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at Microsoft.Azure.ServiceBus.SessionClient+d__37.MoveNext (Microsoft.Azure.ServiceBus, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7e34167dcc6d6d8c)
Message:
A task was canceled. A task was canceled.
Do you have an e-mailadress I can send the previous sourcecode of our queue interop client to?

This is still happening to us and I am fairly sure we did not call close. Can this be changed from InvalidOperationException to ServiceBusCommunicationException so that the retrying kicks in?

@mladedav  Can you share with us a snippet of your code when you run into this error and we will see if we can repro it?
We could try to translate this exception into communication exception if we can repro. But you will still get the logs on AI. Would that be okay for you?

In our case this processes one message and then we get this exception when trying to read the next message. I was told the load should be about 35 messages/hour. It happens for one job consistently in kubernetes.
We are ok with the exceptions being shown, it's just an issue of retrying. Since there is built-in retrying in the SDK, that would seem like the preferred way, but if that wasn't reliable, we would have to implement it ourselves costing us more time and code.
       public async Task<IList<Message>?> ReceiveAsync(CancellationToken cancellationToken)
        {
            if (!_isReaderOpen)
            {
                throw new InvalidOperationException("Reader was not opned but an attempt to read from receiver was made.");
            }
            if (_receiver is null)
            {
                throw new InvalidOperationException("Receiver was not created but receive was called. Reader seems to have been opened though.");
            }

            var tcs = new TaskCompletionSource<bool>(TaskCreationOptions.RunContinuationsAsynchronously);

            using (cancellationToken.Register(() => tcs.SetResult(true)))
            {
                var cancelTask = tcs.Task;

                while (true)
                {
                    var receiveTask = _receiver.ReceiveAsync(_maxBatchSize, TimeSpan.FromSeconds(10));

                    var completedTask = await Task.WhenAny(cancelTask, receiveTask);
                    if (completedTask == receiveTask)
                    {
                        var messages = await receiveTask;
                        if (messages is null)
                        {
                            continue;
                        }

                        _logger.LogTrace("Received batch of size {count}/{max}.", messages.Count, _maxBatchSize);
                        return messages;
                    }
                    else
                    {
                        break;
                    }
                }
            }

            cancellationToken.ThrowIfCancellationRequested();
            throw new InvalidOperationException("Control is in unexpected parts of code. ServiceBus receiver did not finish reading any messages.");
        }

with the receiver built like
            var builder = new ServiceBusConnectionStringBuilder(connectionString);
            builder.EntityPath = EntityNameHelper.FormatSubscriptionPath(builder.EntityPath, subscription);
            _receiver = new MessageReceiver(builder, ReceiveMode.PeekLock, Microsoft.Azure.ServiceBus.RetryPolicy.Default);
        }

Thanks for providing us with the source code. I will look into this and see if I can repro the error. Will update as soon as I can.

Hi, were you able to reproduce? Would it be possible to change the type?

@mladedav Sorry about the delay in response. I have checked our code in the _receiver.ReceiveAsync path and confirmed that you should not receive InvalidOperationException if the connection is closing. I cannot repro what you are seeing with our SDK.
I am also a bit confused since you have shared 2 stack trace, one is the first one:
'''
System.InvalidOperationException: Can't create session when the connection is closing.
at Microsoft.Azure.ServiceBus.Core.MessageReceiver.d__86.MoveNext()
'''
For this path, you should not see this error is you are using "Microsoft.Azure.ServiceBus" Version="4.1.0". This stack trace corresponds with the code snippet you shared with me:
var receiveTask = _receiver.ReceiveAsync(_maxBatchSize, TimeSpan.FromSeconds(10));
Can you please just double check again if the machine that throws this exception is INDEED using Version="4.1.0" or greater?
Another stack trace you have shared:
'''
It looks as if it's for receiving where the exceptions are thrown as they all come from the messagepump;
Microsoft.Azure.ServiceBus.ServiceBusException:
at Microsoft.Azure.ServiceBus.SessionClient+d__37.MoveNext (Microsoft.Azure.ServiceBus, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7e34167dcc6d6d8c)
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at Microsoft.Azure.ServiceBus.SessionReceivePump+d__20.MoveNext (Microsoft.Azure.ServiceBus, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7e34167dcc6d6d8c)
Message:
The connection was inactive for more than the allowed 300000 milliseconds and is closed by container 'LinkTracker'. TrackingId:5abed6bf93f841e484cbe107e93c1c40_G2
'''
From this stack trace you seem to use SessionClient for receive, can you share a piece of code snippet that causes this stack trace? Which path is InvalidOperationException that you want to change the exception type?

@mladedav Also just a follow up to the first stack trace:
'''
System.InvalidOperationException: Can't create session when the connection is closing.
at Microsoft.Azure.ServiceBus.Core.MessageReceiver.d__86.MoveNext()
...
at Microsoft.Azure.ServiceBus.MessageReceivePump.<b__11_0>d.MoveNext()
'''
I saw it is throwing from MessageReceivePump, are you by any chance using OnMessageHandler in your code?

I haven't actually sent any stack traces, that must have been someone else. I must admit I cannot seem to find them anywhere and I can't reproduce the error with our older code. When I get home I will try to hack around a bit if I can break it again.
At this time I only remember that this was the exception with this message (by which I found this thread). I thought since there are already two I wouldn't clutter this place more with stack traces but lesson learned for next time

@mladedav  Thanks for your reply. Are you by any chance using OnMessageHandler in your code?

No, we're using only the MessageReceiver directly to receive directly. Only other service-bus relevant information I can think of is we were failing to renew messages before locks expired when these issues happened but we fixed that at the same time we added some defensive retrying so I can't say whether it's relevant.

Got it, then the first log trace shows 'MessageReceivePump' in the trace which is only coming from receiving using a OnMessageHandler. So I am not sure how that error trace is thrown.
We will need a reliable error trace to continue investigating since we cannot repro. Can you repro it and send us the log and timestamp? thanks!

I too have been receiving this error intermittently for quite some time from our Azure function instances running in Kubernetes.
We have a functions pod running .NET Core 2.2.8 with functions v2 in our Production cluster and a separate functions pod running .NET Core 3.1.5 with functions v3 in our Sandbox cluster after recently upgrading and the exceptions are being received from both pods still.
The production pod references Microsoft.Azure.ServiceBus v3.4.0 and the sandbox pod references Microsoft.Azure.ServiceBus v4.1.3.
Exception message:
Message processing error (Action=Receive, ClientId=MessageReceiver12account-events/Subscriptions/new-account-setup, EntityPath=account-events/Subscriptions/new-account-setup, Endpoint=sndbx-sb-project-au.servicebus.windows.net)
Stack Trace:
System.InvalidOperationException: Can't create session when the connection is closing.   Module "Microsoft.Azure.ServiceBus.Core.MessageReceiver", in OnReceiveAsync   Module "System.Runtime.ExceptionServices.ExceptionDispatchInfo", in Throw   Module "System.Runtime.CompilerServices.TaskAwaiter", in ThrowForNonSuccess   Module "System.Runtime.CompilerServices.TaskAwaiter", in HandleNonSuccessAndDebuggerNotification   Module "Microsoft.Azure.ServiceBus.Core.MessageReceiver+<>c__DisplayClass64_0+<<ReceiveAsync>b__0>d", in MoveNext   Module "System.Runtime.ExceptionServices.ExceptionDispatchInfo", in Throw   Module "System.Runtime.CompilerServices.TaskAwaiter", in ThrowForNonSuccess   Module "Microsoft.Azure.ServiceBus.RetryPolicy", in RunOperation   Module "System.Runtime.ExceptionServices.ExceptionDispatchInfo", in Throw   Module "Microsoft.Azure.ServiceBus.RetryPolicy", in RunOperation   Module "System.Runtime.ExceptionServices.ExceptionDispatchInfo", in Throw   Module "System.Runtime.CompilerServices.TaskAwaiter", in ThrowForNonSuccess   Module "System.Runtime.CompilerServices.TaskAwaiter", in HandleNonSuccessAndDebuggerNotification   Module "Microsoft.Azure.ServiceBus.Core.MessageReceiver", in ReceiveAsync   Module "System.Runtime.ExceptionServices.ExceptionDispatchInfo", in Throw   Module "System.Runtime.CompilerServices.TaskAwaiter", in ThrowForNonSuccess   Module "System.Runtime.CompilerServices.TaskAwaiter", in HandleNonSuccessAndDebuggerNotification   Module "Microsoft.Azure.ServiceBus.Core.MessageReceiver", in ReceiveAsync   Module "System.Runtime.ExceptionServices.ExceptionDispatchInfo", in Throw   Module "System.Runtime.CompilerServices.TaskAwaiter", in ThrowForNonSuccess   Module "System.Runtime.CompilerServices.TaskAwaiter", in HandleNonSuccessAndDebuggerNotification   Module "Microsoft.Azure.ServiceBus.MessageReceivePump+<<MessagePumpTaskAsync>b__11_0>d", in MoveNext
Another interesting piece of info, is that I am also receiving this exception as well at essentially the same time:
System.ObjectDisposedException: Cannot access a disposed object. Object name: '$cbs'.   Module "Microsoft.Azure.ServiceBus.Core.MessageReceiver", in OnReceiveAsync   Module "System.Runtime.ExceptionServices.ExceptionDispatchInfo", in Throw   Module "System.Runtime.CompilerServices.TaskAwaiter", in ThrowForNonSuccess   Module "System.Runtime.CompilerServices.TaskAwaiter", in HandleNonSuccessAndDebuggerNotification   Module "Microsoft.Azure.ServiceBus.Core.MessageReceiver+<>c__DisplayClass64_0+<<ReceiveAsync>b__0>d", in MoveNext   Module "System.Runtime.ExceptionServices.ExceptionDispatchInfo", in Throw   Module "System.Runtime.CompilerServices.TaskAwaiter", in ThrowForNonSuccess   Module "Microsoft.Azure.ServiceBus.RetryPolicy", in RunOperation   Module "System.Runtime.ExceptionServices.ExceptionDispatchInfo", in Throw   Module "Microsoft.Azure.ServiceBus.RetryPolicy", in RunOperation   Module "System.Runtime.ExceptionServices.ExceptionDispatchInfo", in Throw   Module "System.Runtime.CompilerServices.TaskAwaiter", in ThrowForNonSuccess   Module "System.Runtime.CompilerServices.TaskAwaiter", in HandleNonSuccessAndDebuggerNotification   Module "Microsoft.Azure.ServiceBus.Core.MessageReceiver", in ReceiveAsync   Module "System.Runtime.ExceptionServices.ExceptionDispatchInfo", in Throw   Module "System.Runtime.CompilerServices.TaskAwaiter", in ThrowForNonSuccess   Module "System.Runtime.CompilerServices.TaskAwaiter", in HandleNonSuccessAndDebuggerNotification   Module "Microsoft.Azure.ServiceBus.Core.MessageReceiver", in ReceiveAsync   Module "System.Runtime.ExceptionServices.ExceptionDispatchInfo", in Throw   Module "System.Runtime.CompilerServices.TaskAwaiter", in ThrowForNonSuccess   Module "System.Runtime.CompilerServices.TaskAwaiter", in HandleNonSuccessAndDebuggerNotification   Module "Microsoft.Azure.ServiceBus.MessageReceivePump+<<MessagePumpTaskAsync>b__11_0>d", in MoveNext
Let me know if you require any more information.

Hi @mr-davidc, thanks for reaching out. Are you from the same team as @mladedav? If not, can you create another thread and share with us a piece of code that has the exceptions thrown? This thread is already really long and hard to navigate. We need to tailor for each repro to see what's the problem there since these kinds of problems are hard to debug.

@mr-davidc, @mladedav any updates on the logs? If not, we will close this incident.

@mr-davidc, @mladedav any updates on the logs? If not, we will close this incident.

Hi @DorothySun216, mladedav and I are from separate teams so I can't speak for them but from my end I created a separate issue for the problems I am getting here: #13637
Thanks

We have rolled out a fixed https://github.com/Azure/azure-sdk-for-net/pull/17023 on latest release 5.1.0 and can you test if with this new nuget package, are you still seeing the same issue? https://www.nuget.org/packages/Microsoft.Azure.ServiceBus/5.1.0

I'm still getting a similar issue with Microsoft.Azure.ServiceBus 5.1.2. If I close all the sessions and the SubscriptionClient with CloseAsync then ExceptionReceivedHandler continues to fire with exception below:
      Exception handled during connection refresh: [Microsoft.Azure.ServiceBus.ServiceBusException: The operation was canceled.
       ---> System.OperationCanceledException: The operation was canceled.
         at Microsoft.Azure.Amqp.AsyncResult.End[TAsyncResult](IAsyncResult result)
         at Microsoft.Azure.Amqp.AmqpCbsLink.SendTokenAsyncResult.<>c__DisplayClass13_0.<GetAsyncSteps>b__3(SendTokenAsyncResult thisPtr, IAsyncResult r)
         at Microsoft.Azure.Amqp.IteratorAsyncResult`1.StepCallback(IAsyncResult result)
      --- End of stack trace from previous location where exception was thrown ---
         at Microsoft.Azure.Amqp.AsyncResult.End[TAsyncResult](IAsyncResult result)
         at Microsoft.Azure.Amqp.AmqpCbsLink.<>c__DisplayClass4_0.<SendTokenAsync>b__1(IAsyncResult a)
         at System.Threading.Tasks.TaskFactory`1.FromAsyncCoreLogic(IAsyncResult iar, Func`2 endFunction, Action`1 endAction, Task`1 promise, Boolean requiresSynchronization)
      --- End of stack trace from previous location where exception was thrown ---
         at Microsoft.Azure.ServiceBus.Amqp.AmqpLinkCreator.CreateAndOpenAmqpLinkAsync()
         at Microsoft.Azure.ServiceBus.Core.MessageReceiver.CreateLinkAsync(TimeSpan timeout)
         at Microsoft.Azure.Amqp.FaultTolerantAmqpObject`1.OnCreateAsync(TimeSpan timeout)
         at Microsoft.Azure.Amqp.Singleton`1.GetOrCreateAsync(TimeSpan timeout)
         at Microsoft.Azure.Amqp.Singleton`1.GetOrCreateAsync(TimeSpan timeout)
         at Microsoft.Azure.ServiceBus.Core.MessageReceiver.GetSessionReceiverLinkAsync(TimeSpan serverWaitTime)
         at Microsoft.Azure.ServiceBus.RetryPolicy.RunOperation(Func`1 operation, TimeSpan operationTimeout)
         at Microsoft.Azure.ServiceBus.RetryPolicy.RunOperation(Func`1 operation, TimeSpan operationTimeout)
         at Microsoft.Azure.ServiceBus.SessionClient.AcceptMessageSessionAsync(String sessionId, TimeSpan operationTimeout)
         --- End of inner exception stack trace ---
         at Microsoft.Azure.ServiceBus.SessionClient.AcceptMessageSessionAsync(String sessionId, TimeSpan operationTimeout)
         at Microsoft.Azure.ServiceBus.SessionReceivePump.SessionPumpTaskAsync()]

Any suggestions?

@ColeSiegelTR Thanks for reaching out. This is a very unpredictable issue and we attempted to repro it many times but couldn't repro. Do you have retry mechanisms that can recover from this exception? Can you provide me followings for us to keep investigating?

Provide me the exact timestamp of this exception and your service bus namespace name, topic name and subscription name. If this is sensitive information, please email to<EMAIL_ADDRESS>Can you enable client side tracing using the dotnet trace tool to repro this exception? This can give us better sense of what happened when the exception is thrown.
The command you can run after you downloaded the tools is:
dotnet-trace collect --process-id  --providers Microsoft-Azure-ServiceBus

The default should be all keywords (0xffffffff) and verbose logging level (5).  In case you need to specify it explicitly:
dotnet-trace collect --process-id  --providers Microsoft-Azure-ServiceBus:ffffffff:5
Please share the nettrace to the email address above. Thanks!

We upgraded to the latest Azure SDK and this still happens (Azure.Messaging.ServiceBus)


send a dispose command before killing the child process

When stopping/restarting the backing process from the VS Code extension, we blindly call .kill(), but some kernels, particularly remote proxies, might need to clean up some resources.  We should first send DisposeCommand or similar, give the backing process a short amount of time to respond, and only then would we manually send the .kill() signal if it didn't gracefully exit on its own.

The child process should be able to handle its own disposal on termination rather than needing a custom gesture for it. Is there a specific case where this is not happening?


SDCICD-1185: remove rosa version check

it is causing unnecessary churn and jobs to be broken, lets rely on the
version check that rosa CLI does on its own to ensure we are running at
least the minimum version.
Signed-off-by: Brady Pratt<EMAIL_ADDRESS>

https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/openshift_release/45379/rehearse-45379-pull-ci-openshift-osde2e-main-hypershift-pr-check/1745450581675216896
 2024/01/11 14:26:05 e2e.go:87: Failed to set up or retrieve cluster: could not launch cluster: create cluster failed: error: failed to wait for command to finish: exit status 1, stderr: ERR: Failed to create cluster: Creating hosted control plane cluster requires ROSA 1.2.31 or higher. Go to https://console.redhat.com/openshift/downloads#tool-rosa to download the latest version.


Support drag and drop between flatlists.

How to drag and drop between flatlist. example : https://rafaelquintanilha.com/apps/sortabletargets/