The home page unpinned Onedrive folder crashed

Description
The home page unpinned Onedrive folder crashed.
Steps To Reproduce
1.Pin the Onedrive folder on the home page
2.Unpin
Requirements
Hope to fix
Files Version
<IP_ADDRESS>
Windows Version
10.0.22631.2428
Log File
debug.log

Thanks for the report. What do you mean by pin to the home page? Are you using the favorites section?

Yes.

发件人: @.*** @.> 代表 Josh65-2201 @.>
发送时间: 2023年10月18日 06:35
收件人: files-community/Files @.>
抄送: xiaodu-du @.>; Author @.***>
主题: Re: [files-community/Files] The home page unpinned Onedrive folder crashed (Issue #13531)
Thanks for the report. What do you mean by pin to the home page? Are you using the favorites section?
―
Reply to this email directly, view it on GitHubhttps://github.com/files-community/Files/issues/13531#issuecomment-1768475776, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A77JTC4PLASRNARRSIP3Z43X77LKTAVCNFSM6AAAAAA6FRGBROVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRYGQ3TKNZXGY.
You are receiving this because you authored the thread.Message ID: @.***>

Can you see if this is an issue in the preview version. https://files.community/download

There is a good chance this is fixed in the preview version, but it'll be good if you can confirm before we close the issue.

Tried it in the latest preview and it has been resolved

发件人: @.*** @.> 代表 Josh65-2201 @.>
发送时间: 2023年10月28日 06:34
收件人: files-community/Files @.>
抄送: xiaodu-du @.>; Author @.***>
主题: Re: [files-community/Files] The home page unpinned Onedrive folder crashed (Issue #13531)
Can you see if this is an issue in the preview version. https://files.community/download
―
Reply to this email directly, view it on GitHubhttps://github.com/files-community/Files/issues/13531#issuecomment-1783812387, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A77JTC3ANHPGBMOMZXZMCUDYBUCWXAVCNFSM6AAAAAA6FRGBROVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBTHAYTEMZYG4.
You are receiving this because you authored the thread.Message ID: @.***>

Thanks for checking.


add sentry provider

This PR adds a sentry provider for pulumi based on the terraform bridge.

Do not add tags yet as a way to try and release your provider. We are trying to get hold of one required package repository account. Once this is resolved, I will inform you.
Got it!

@henriiik this repository has been equipped with the following secrets:
NPM_TOKEN (for npmjs.org)
NUGET_PUBLISH_KEY (for nuget.org)
PYPI_API_TOKEN (for pypi.org)
RELEASE_GITHUB_TOKEN (for GoReleaser to create a Github Release)

You should be able to start releasing this provider.

@ringods thanks!


Results Page Overview and arbitrary Risk Tolerance

Hello! Love this project. I send links to it all the time in groups, to show specific activities.
My use case for this tool is usually to tell people: "Going to the grocery store won't kill you.  Eating on a patio is a little unsafe, but you aren't spreading the coronavirus by doing so (at least in our county with very low risk levels)."
My issue is that the Results may suggest something is "Very Dangerous" without context for "how often you are choosing to do this".
If someone is isolating generally, going to the grocery store for 30 minutes once a week, and are curious if it was okay to go eat on a patio every two weeks, that activity is totally safe.
While I respect your team has chosen the 1% number for "risk per week" - I would very much appreciate more information about that number besides that it "Feels right" seeing that you are basing the amount of danger on that number. Allow us to choose our own chances. My personal researched weekly risk tolerance is 2%. As said above, this arbitrary risk tolerance level, while I respect your opinion, adds a level of confusion to people I send this tool to.
Suggestion: Before calling something "Very dangerous" etc, ask "How often would you do this activity while otherwise distancing?"
Second Suggestion: Add a 2% option to Risk Tolerance. Or, use an external source for recommended weekly Risk Tolerance. Or, deprioritize the "Moderate" / "Dangerous" / "Very Dangerous" section. At this point this is only noise because everyone has their own risk tolerance.

https://twitter.com/PatJWu/status/1299918900381937665
This twitter thread shows someone making awesome steps to reduce the levels of microcovids, but he still calls his work a "failure" even though he is looking at a relatively risky situation (0.05) and shows changes that make it go down to a 0.01% chance.
If a 0.01% chance of Covid is "high" something is wrong. He could go to the gym for 5,000 days before he had a 50-50 chance.

Last, because, I'm not sure how flexible the team is on this information, this is also in the whitepaper:

Here are our calculations. We think as many as 1 in 10 COVID cases could potentially lead to long-term negative health consequences for the infected person; we don't think the ratio is likely to be higher than that, due to sources like the app-based COVID Symptom Study that found one in ten cases weren't fully recovered after 3 weeks. That would mean that a 1% chance of COVID is the same as up to a 0.1% chance of ongoing negative health consequences from COVID. The average American drives 13,476 miles per year. In 2018 there were 2,491,000 car-crash-related injuries (of any severity) spread across 3,240,327,000,000 vehicle miles travelled (see the NHTSA website for more detailed breakdowns). A study of Swedish car crash injuries found that 11.4% of them met the criteria for at least “1% permanent medical impairment” five years later. (“Permanent medical impairment” is a legally-relevant concept that attempts to quantify disability, and the 1% level was the lowest level of disability that was discussed in the Swedish car crash study.) Multiplying this out, we get a 0.11% chance of ongoing negative health consequences from driving for one year.

One small issue here is that the 1% number from car crashes is 5 years later as opposed to the one in ten cases after three weeks. How many people in the car crash study still had impairment after three weeks?
I think your team has done an amazing job of comparing various risk factors, and I would just ask that you apply that same level of comparison to estimating the total risk acceptable. If we were to peg it on Car Crashes (an anecdote I also often use), I would also assume that the number would be much higher after 3 weeks. However, while COVID has sometimes a slightly longer recovery time, I do not think that those that don't recover for 3 weeks is the appropriate benchmark here. Why not 4 weeks? Why not 8?
Last, I don't see the chance of asymptomatic Covid in this discussion.
So, the more philosophical question here is "Should this calculator maximize caution, or usefulness? How can we provide accurate data with assumptions, without tacking on the risk amounts that "felt right""? How do we balance useful information with the potential for the calculator to cause harm, both to people who are unnecessarily over-isolating themselves, and to those who are unnecessarily over-exposing themselves?

Hi @bbqturtle,
I mostly agree with you that our labels are... not the most helpful. They're designed to be comparable to similar sites (like covidcanidoit) that don't work with budgeting, which doesn't serve users as well as it could. I also agree that # of times per week could be useful. Although at that point, you really get into the territory where the spreadsheet shines - the web ui is insufficient for budgeting out more than 1 kind of activity at a time. Clarifying some of the information about when/what to use the calculator for vs what to use the spreadsheet for is something we will take up soon.
We've generally found that no one was able to quantify their risk tolerance at all; they've been 0, 100, or "whatever I feel like". I'm super curious how you calculated your 2%.
If you ask most humans what their tolerance for risking death or injury is, they have no idea. You are, of course, welcome to choose your own tolerance. Again, the spreadsheet is great for this.

If a 0.01% chance of Covid is "high" something is wrong. He could go to the gym for 5,000 days before he had a 50-50 chance.

I'm not quite sure what you're trying to say by bringing up 50-50. That's an insane risk for any duration of time. If everyone had a 50% chance of getting covid, a) hospitals would get overloaded b) even if the hospitals magically had capacity to treat everyone, 0.5% of the population would die. That's 1.5 million deaths in the US alone. We absolutely cannot condone behavior that would lead anywhere close to that.
0.01% for 300 days is a 3% chance of getting covid. Maybe that's fine if its all you do. Presumably this person wants to use some of their risk budget for doing anything else.
Another important consideration when deciding a risk budget is to effects on society - for the caseload in an area to stay flat, the average person's weekly risk should be the prevalence in the area. If you want the prevalence to go down, people on average need to be safer than that. Picking a weekly risk tolerance above your regional prevalence means that you need to take extra precautions to self-isolate, otherwise you are personally making the pandemic worse (don't personally make the pandemic worse if you can help it).

One small issue here is that the 1% number from car crashes is 5 years later as opposed to the one in ten cases after three weeks. How many people in the car crash study still had impairment after three weeks?

This is a problem with dealing with a pandemic as its happening. We don't know how many people will really never recover from COVID. If you have newer data that suggests the long term impairment rate is lower, please send it over; we'd love to injest it.

So... Since I live in a household of 2 people, allow me to adjust my risk tolerance to 4%. It seems odd to share out a tool without that functionality, assuming we all live in a 8 people household! The average household size is 2.6 people, so if you came to the 1% number with math, that is a 3% average household risk threshold.

Each person living at Ibasho targets an annualized risk of 1% including everyone else's activities - each person has an individual budget of just 60 microCovids per week (a person living alone would have 200 / week). We do need to add the tooling to reduce budget based on # off housemates. And again, you can set your risk tolerance to whatever you want - in the spreadsheet.

Last, I don't see the chance of asymptomatic Covid in this discussion.

We touch on this in https://www.microcovid.org/paper/2-riskiness#fn2 and https://www.microcovid.org/paper/7-basic-method (I suggest going to whitepaper->all if you want to search for a topic). We basically treat asymptomatic people the same as if they were symptomatic. There is evidence that they may be 50%-70% as infectious as symptomatic patients. Note presymptomatic != asymptomatic. Presymptomatic people are the most contagious. Risk of transmission is highest for a 24 hour window before symptoms start.

Hi @beshaya ! I'm flattered you would read my response and reply in such great detail. I truly appreciate this project, and I hope my comments did not come across as not appreciative. Please take my notes below as answering your questions, and not as arguments.

"for the caseload in an area to stay flat, the average person's weekly risk should be the prevalence in the area"

I completely agree with this. So, for my area, the case rate is 2%. So, at a WEEKLY risk budget of 1%, I would help that go down. However, your household is doing a 1%/52 or a 0.02% weekly risk budget, which, while admirable, is much lower than you are suggesting in this case, unless I don't understand what a 1% annual risk rate vs weekly risk budgeting is.

"There is evidence that they may be 50%-70% as infectious as symptomatic patients."

This is in reference to the automobile comparison - if asymptomatic cases happen 20% of the time, and have no symptoms, those numbers should be inputted into the budget setting process that ended at 1% per year.

Each person living at Ibasho targets an annualized risk of 1% including everyone else's activities

I didn't understand this - thank you for clarifying.

you can set your risk tolerance to whatever you want - in the spreadsheet.

I really like sending this to friends, family, coworkers, and it's much harder to do that in the spreadsheet. I think the idea of setting a covid budget is very compelling - even a few tips or separate worksheet would be helpful. (What are your confounding variables? This budget means you will likely catch it within X weeks. ETC.).
Again, my feedback is that setting a risk budget higher than 1% a year (like, maybe, up to 4% a year) would cause people to not reject your calculator so easily for saying everything is too high out of their risk budgets. Combine that with helpful information (fake example: 1% - chances of getting into a car crash annually. 2% - chance of getting the common cold annually. etc).

I completely agree with this. So, for my area, the case rate is 2%. So, at a WEEKLY risk budget of 1%, I would help that go down. However, your household is doing a 1%/52 or a 0.02% weekly risk budget, which, while admirable, is much lower than you are suggesting in this case, unless I don't understand what a 1% annual risk rate vs weekly risk budgeting is.

Excellent point. We designed this at a time and place where prevalence was .2%. That translates to about 10% anualized risk for the population of SF. A 1% risk budget was comfortably 10x lower than the average, so we felt good about not contributing to the pandemic (this is separate from the automobile rationale, but was a good check).
We hadn't really thought through what happens at 2% prevalence...if your region maintains that level for a year, we'd naively calculate 100% of the population to be infected. If you anticipate people continuing their current behavior, then your personal behavior won't make a difference in the population level results. But also that means your region is on track for 1% of the population to die 😬.

if asymptomatic cases happen 20% of the time, and have no symptoms, those numbers should be inputted into the budget setting process that ended at 1% per year.

Oh! I think you're saying that asymptomatic cases don't have long term health impacts, so getting an asymptomatic case shouldn't be considered when thinking about long term risk of covid. Reports suggest that asymptomatic cases also have heart and lung damage: https://www.forbes.com/sites/robertglatter/2020/08/17/covid-19-can-cause-heart-damageeven-if-you-are-asymptomatic/?sh=3d6431436cef so we believe that having an asymptomatic case may be just as bad for you long term. You can choose not to believe that, in which case you'd double your personal risk budget.

I really like sending this to friends, family, coworkers, and it's much harder to do that in the spreadsheet.

I'm curious why its harder with the spreadsheet - it's a google sheet, which lets you send links to individual cells. One of our current priorities is figuring out how to help power users get the most out of the spreadsheet.

Again, my feedback is that setting a risk budget higher than 1% a year (like, maybe, up to 4% a year) would cause people to not reject your calculator so easily for saying everything is too high out of their risk budgets.

Yeah, I see how the 1% budget feels constraining in your area. I'm a little scared to allow larger budgets, because if we give out a budget that's 4x higher and the caseload increases 4x then we'll be back having the same discussion; the calculator will say you should be isolating from everyone. I'm curious, is there a regional prevalence rate at which you would agree that everything is too high out of your risk budget?

Combine that with helpful information (fake example: 1% - chances of getting into a car crash annually. 2% - chance of getting the common cold annually. etc).

Agree that we need more discussion about choosing risk thresholds. I'll work with the team on this.


Detect background volume changes

Would it be possible to detect volume changes if the application is running in the background?
Thanks

Doesn't this already work if you do this as per the instructions:
    _subscription = VolumeController().watchVolume().listen((volume) {
      setState(() => _volumeListenerValue = volume);
    });

It seems to be working here for me on iOS anyway.


Types only checking when explicitly defined, unsound when implicit (inlined)

These two versions are semantically identical and differ only in the placement of types:

working: extracted types
broken: inline types

The only change is that MakeReducers and Operations types are now inlined and the return type of MakeReducers is implicit. Note however that the returned function has exactly the same signature as the type declared in the first version, i.e. the inferred type should be identical to the explicitly declared one.

same on latest version

Closing since the repro uses $Call which is no longer supported


Support for clicking and double clicking certain position of an element

Originally submitted to Google Code by palani.selvam on 26 Apr 2011

Explain the new feature or other enhancement idea you have.
Please add keywords for clickAt, doubleClickAt Selenese commands
Purpose:
One of my pages needed clickAt instead of click. I have gone ahead and extended SeleniumLibrary.
I would like to have keywords for clickAt and doubleClickAt
Methods added to SeleniumLibrary in click.py :
def _click_point(self, locator, coordString='0,0', dont_wait=''):
    """ Added by Palani for EYC
    """
    self._selenium.click_at(locator, coordString)
    if not dont_wait:
        self.wait_until_page_loaded()
def click_element_point(self, locator, coordString="10,5",

dont_wait=''):
"""Click element identified by locator on the given relative
position.
    Key attributes for arbitrary elements are `id` and `name`. See
    `introduction` for details about locating elements and about

meaning
of dont_wait argument.
    'locator' is an element locator
    'coordString' is specifies the x,y position (i.e. - 10,20) of

the mouse event relative to the element returned by the
locator.
"""
# Added by Palani for EYC
self._info("Clicking element '%s' on relative position '%s'" %
(locator, coordString) )
self._click_point(self._parse_locator(locator), coordString,
dont_wait)

Originally submitted to Google Code by @yanne on 29 Apr 2011

This issue was updated by revision 8930394b90bf.
We decided to add coordinate support for Click Element instead of adding new Click At keyword. We also noticed that there is no Double Click Element keyword and will add that separately.

Originally submitted to Google Code by @pekkaklarck on 29 Apr 2011

Double Click Element keyword is  issue 192 . We'll implement it so that it has support for coordinates.

Originally submitted to Google Code by @yanne on 1 May 2011

Now both Click Element and Double Click Element support giving position as an argument.

Originally submitted to Google Code by @yanne on 6 Aug 2012

 Issue 244  has been merged into this issue.


Latest is pointing to 2.x

https://api.github.com/repos/helm/helm/releases/latest
is not pointing to 3.x

Duplicate of #7592

Closing as a duplicate


Create reusuable service - SQLLITE

https://developers.sap.com/tutorials/cp-cap-java-reusable-service.html#55485b03-5e1e-4ca6-9660-c4e02aee7184
i am using visual studio..
i see the sqllite db in my project

if i change the profile from dev to default then it does not build

CURL runs thru and i can query i see data in

I tried looking up data in SQLITEDB via extension no data was there.
AdminService.Java is deleted and no longer part of the project
after that i do a CTRL+C and then bring up the application again
now when i query http://localhost:8080/odata/v4/AdminService/Categories(10) .. i do not get data back, so there is no persistence yet
could you please help

this was the missing step.. ( if possible please add this to the tutorial)
mvn spring-boot:run -D spring-boot.run.profiles=dev
credit: https://blogs.sap.com/2020/12/08/get-start-for-cap-java/

Hi @muru82,
The issue should be resolved with the update we did yesterday, so I am closing the issue. If you still have questions, feel free to reopen the issue.
Best regards,
Iwona


Use purescript-react, fix #35

@ethul Would you be able to code review this, please?

Will do. I will take a look.

Looks good to me!

Welcome!
On Friday, September 4, 2015, Phil Freeman<EMAIL_ADDRESS>wrote:

Thanks very much for the review!
—
Reply to this email directly or view it on GitHub
https://github.com/paf31/purescript-thermite/pull/36#issuecomment-137749798
.


Replaced React.PropTypes reference with PropTypes

Description
Replaced one area which was referencing React.PropTypes to use PropTypes from the prop-types library.
Motivation and Context
Should fix PropTypes warnings which this library causes in itself, and other projects.
How Has This Been Tested?
Test suite ran, PropTypes warning no longer appears in tests.
Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

why is this not merged yet?

@wehriam Any chance this could be merged?

What is going on with this library, multiple PRs for bugs in master that haven't been merged

@capaj and @aanguswilliams this is a community supported project. We do our best to address PRs and issues, but unfortunately have other professional obligations that preempt this work.
If you're interested in custom features or professional support for this project, please feel free to reach out to me directly.
I've added @JonShort as a contributor and  humbly request he evaluate/merge this and other PRs as soon as he has a chance.

Apologies, I do 100% appreciate the contributions yourself and others have made to the open source community and understand there's no obligation to keep it up to date. Just wasn't sure what was going on with the timelines and was hoping to determine whether contributions were even still being taken. Thanks for your work.

@wehriam no problem, I was merely asking. I know activity in the PR thread increase the chance the PR will get the attentions from maintainers. Awesome that you've made @JonShort a contributor.


ft(roles-permission): Setting up roles and permissions

What does this PR do?
Setting up roles and permissions
Description of Task to be completed?

Created roles and their related permissions

How should this be manually tested?
step1: clone git repo if not yet done
step2: cd Avengers-Phantom-BE
step3: npm i or npm install to install dependencies
step4: npm run start:dev to start server
Use postman for testing APIs

This PR have conflicts


The best tagline ever

I have a feeling we could have a tagline that better describe most of the awesomenes.
What do you think about

Versatile javaScript SQL database library for relational data, noSQL (schemaless) data, and graph manipulation supporting localStorage, IndexedDB, and Excel files for import/export

Are we missing the JSON part? Are we missing the "select your SQL flavour" ?
Come with inputs

May be "... relational tables and nested JSON data"

What about

Versatile javaScript SQL database for both your (traditional) relational tables or nested JSON data (noSQL) - supporting localStorage, IndexedDB, and Excel files for import/export

Hmmm. Perfection is when there is nothing more to remove.... but... Thinking this might be more spot on:

Versatile SQL database for browser or node. Handles both your (traditional) relational tables and nested JSON data (noSQL). Export, store, and import data from localStorage, IndexedDB, or Excel.

Future versions could add

Choose your SQL compliant flavor: Postgress, MSSQL, MySQL, OrientDB.

(and so on...)

Ok.

Great - can you put it as the header of the Repo? (I dont have access)
Ill place it in the package.json file

I am such an idiot.... I merged the change of description into master. awwww. Sorry for that :-/

Don't worry. No problem! I did not touch README anywhere. I will just replace it when we will release new version. Please, move all changes to wiki, so we can change it together.

Check !

I updated the tag line in repository

:+1:


nbconvert option results in duplicate IPython imports

Testing #8034
With the nbconvert option:

Create a notebook with magics
Export the notebook to a Python script. The Python script starts off by importing IPython
Run all the cells in the Python script in order, which ends up running them in the IW
Export the IW contents to a new Python script
:bug: IPython gets imported twice

This behaviour could be by design, though.

Closing as will not be done, not a major issue and no one else has reported this, been 2 years, no point leaving this open


Support for codes in spec

Ethereum, Stellar, Torrent codecs and Multiformat codecs. Added test cases too. Closes #4.

@whyrusleeping this pr includes ethereum codes, but I see you commited (07b394e2bc36bd37559456b1ac6138ac590f040e) after I commited this pr, Should I remove the ethereum codecs in this pr? The only difference is you used the short name (eth) and I used the full name (ethereum).

@tabrath Oh, sorry for committing that over you. Yeah, if you want to remove the ethereum names that would be great. I named them with just the Eth prefix to align a bit better with the javascript codes

Alright, will do that. 👍

@whyrusleeping Removed my eth codes and cleaned up a little.


[CSL-959] Add withSocketDo

And without MonadBaseControl IO unlike in previous commit :(

Comment to withSocketsDoLifted added

Okay, I overestimated number of missing MonadBaseControl IO instances in cardano, let's close this issue as soon as https://github.com/input-output-hk/cardano-sl/pull/281 gets merged


getBoundingBoxes should be float instead of int

Hi @lozuwa I think in "ImageAnnotation.py" and getBoundingBoxes function bounding box coordinates should be float instead of int.
def getBoundingBoxes(self, objects = None):
		boundingBoxes = []
		for i in range(len(objects)):
			# Find bndbox
			coordinates = objects[i].find("bndbox")
			# Get coordinates
			xmin = float(coordinates.find("xmin").text)
                        xmax = float(coordinates.find("xmax").text)
                        ymin = float(coordinates.find("ymin").text)
                        ymax = float(coordinates.find("ymax").text)
			boundingBoxes.append([xmin, ymin, xmax, ymax])
		return boundingBoxes

Hi @lozuwa,
after changing bounding box coordinates into float(as per above comment), i am getting below error. Kindly help me to resolve.
python3 reducing_big_images.py
  0%|                                                                                | 0/62899 [00:00<?, ?it/s]
Traceback (most recent call last):
  File "reducing_big_images.py", line 38, in <module>
    main()
  File "reducing_big_images.py", line 35, in main
    obda.reduceDatasetByRois(offset = offset, outputImageDirectory = images_output_path, outputAnnotationDirectory = annotations_output_path)
  File "/home/adas/.local/lib/python3.6/site-packages/impy/ObjectDetectionDataset.py", line 391, in reduceDatasetByRois
    outputAnnotationDirectory = outputAnnotationDirectory)
  File "/home/adas/.local/lib/python3.6/site-packages/impy/ObjectDetectionDataset.py", line 522, in reduceImageDataPointByRoi
    Util.save_img(frame = frame[RoiYMin:RoiYMax, RoiXMin:RoiXMax, :],
TypeError: slice indices must be integers or None or have an __index__ method

Original Image size 1820x940 and i am reducing into 910x470.  And Original images name and xmls are in the format of C02032-US_20181105_144014_00450.xml/C02032-US_20181105_144014_00450.jpg


fix: update spanner lroResponseMappers to map types for metadata and response separately

Update spanner lroResponseMappers to comply with changes in Core in PR #2536
This PR should wait for #2536 to merge in order to update google/cloud-core dependency version.

Currently blocked on

[ ] Update spanner lroResponseMappers to comply with changes in Core in PR #2536
[ ] new release of google/cloud-core

@AVaksman can you update this against the latest master? that should fix the error we're seeing in kokoro.

Can this be merged now before the next release(#2877 ) lands?

Yes. google/cloud-spanner is only updated when a new tag is created, and google/cloud installed at dev-master will have the core changes required.


Limit stack to prevent/notice future spaceleaks

Use this simple weird trick to avoid space leaks in your code ;-).
See more in http://ndmitchell.com/downloads/slides-plugging_space_leaks_improving_performance-06_oct_2016.pdf

"The job exceeded the maximum time limit for jobs, and has been terminated.", so maybe not an issue with PR itself?


[Bug][TCGC]: @clientName not work on namespace

Describe the bug
Check this spec: https://github.com/Azure/cadl-ranch/blob/bbd186faeb224113ff81dba82baccf15e2c58362/packages/cadl-ranch-specs/http/client/naming/main.tsp#L146-L147
namespace Model should be renamed to ClientModel through @clientName, but TCGC still return the original name.

Reproduction
Just parse the spec in the link above.
Checklist

[X] Follow our Code of Conduct
[X] Check that this issue is about the Azure libraries for typespec. For bug in the typespec language or core libraries file it in the TypeSpec repo
[X] Check that there isn't already an issue that request the same bug to avoid creating a duplicate.
[X] The provided reproduction is a minimal reproducible example of the bug.

fixed in: https://github.com/Azure/typespec-azure/pull/1203


Adding option to remove outliers

Added option in HybridNew to remove outliers from test-stat distributions before calculating p-values/limits.
This option removes test-statistic values from the test-statistic distributions that are further than $X\sigma$ from the mean of the distribution, where $\sigma$ is the standard deviation of the distribution and $X$ is the value passed to the option, before calculating $p-$values and limits. Setting this to some reasonably large number (eg 50) should remove the spurious values. By default, no removal of outliers is performed.
Added text to documentation about this option.

Obviously would be better to calculate median rather than mean before outlier removal, will update PR with that


Fix wrong behaviour when move multiple files out of watched folder on…

… mac (#41963)
Backport of https://github.com/dotnet/corefx/pull/41963
Fixes https://github.com/mono/mono/issues/16778

@steveisok could you review this please?


Setting html_context:css_files results in no rtd-theme css files being exported

Problem
I am one of the contributors to the https://github.com/apache/trafficserver project. We were pinned to Sphinx 3.5.4 and using rtd 0.5.2. We were happy with that setup for a while, but we need to start supporting Python 3.10 which those releases no longer support. We upgraded Sphinx recently to 6.1.3 with rtd remaining at 0.5.2 (which was the version installed by pipenv with our sphinx-rtd-theme = '*' specification). This seemed at first glance to be fine, but we eventually noticed issues recorded in #1115. This is not surprising since rtd 0.5.2 is pretty old.
However, when we upgrade to any 1.x rtd theme, none of our rendered html pages link in the rtd themed css content. Essentially the theme is not being used.
Reproducible Project
Reproduction is rather easy using the traffic server docs README.md here:
https://github.com/apache/trafficserver/blob/master/doc/README.md
Here are reproduction steps with docker:
docker pull ci.trafficserver.apache.org/ats/rockylinux:8
docker run -it -p 60123:8888 --name rtd_not_rendered ci.trafficserver.apache.org/ats/rockylinux:8 /bin/bash

# Now, within the docker container.
cd /var/tmp
git clone https://github.com/apache/trafficserver.git
cd trafficserver/doc

# Edit the Pipfile so that `sphinx = "==6.1.3"` and `sphinx-rtd-theme = "==1.2.0rc2"`

# Now, create a pip environment with those Sphinx package versions and enter it.
pipenv install && pipenv shell

# Build the docs.
cd ../ && autoreconf -fi && ./configure --enable-docs && cd doc && make -j8 html

# Create a server to serve the docs.
cd docbuild/html/
python3 -m http.server 8888

# Now view the rendered docs from outside the container with the exported 60123 port.
# You'll notice that the theme's relevant css files are not exported.

Here is the Pipfile after my above edits:
(doc) [root@d51f9c68d2d6 doc]# cat Pipfile
#  Licensed to the Apache Software Foundation (ASF) under one
#  or more contributor license agreements.  See the NOTICE file
#  distributed with this work for additional information
#  regarding copyright ownership.  The ASF licenses this file
#  to you under the Apache License, Version 2.0 (the
#  "License"); you may not use this file except in compliance
#  with the License.  You may obtain a copy of the License at
#
#      http://www.apache.org/licenses/LICENSE-2.0
#
#  Unless required by applicable law or agreed to in writing, software
#  distributed under the License is distributed on an "AS IS" BASIS,
#  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
#  See the License for the specific language governing permissions and
#  limitations under the License.

[[source]]
name = "pypi"
url = "https://pypi.org/simple"
verify_ssl = true

[dev-packages]

[packages]

sphinx = "==6.1.3"

sphinx-rtd-theme = "==1.2.0rc2"
sphinxcontrib-jquery = "*"
sphinxcontrib-plantuml = "*"
# i18n
sphinx-intl = "*"

# For parsing Doxygen XML output, to add links from an API description
# to the source code for that object
lxml = "*"

polib = ">=1.0.3"

[requires]
python_version = "3"

Here is our conf.py file:
https://github.com/apache/trafficserver/blob/9b47f6d514c54aa46fb4a1845d02a8de3c24475c/doc/conf.py
The relevant section that gets executed is:
        import sphinx_rtd_theme
        html_theme = 'sphinx_rtd_theme'
        html_theme_path = [sphinx_rtd_theme.get_html_theme_path()]

I tried adding 'sphinx_rtd_theme', to our extensions, but that doesn't seem to help either.
Is there something about the configuration of the rtd-theme that has changed since version 0.5.2 that we are not doing correctly? I searched the change log but I did not notice anything significant there, but I may have missed it.
Error Logs/Results
N/A
Expected Results
I expect the rtd-theme to render correctly, but the css files for the theme are linked in the HTML files.
Environment Info

Python Version: 3.8.12
Sphinx Version: v6.1.3
RTD Theme Version: 1.2.0rc2

(doc) [root@d51f9c68d2d6 doc]# pip freeze
alabaster==0.7.13
Babel==2.11.0
certifi==2022.12.7
charset-normalizer==3.0.1
click==8.1.3
docutils==0.18.1
idna==3.4
imagesize==1.4.1
importlib-metadata==6.0.0
Jinja2==3.1.2
lxml==4.9.2
MarkupSafe==2.1.2
packaging==23.0
polib==1.1.1
Pygments==2.14.0
pytz==2022.7.1
requests==2.28.2
snowballstemmer==2.2.0
Sphinx==6.1.3
sphinx-intl==2.0.1
sphinx-rtd-theme==1.2.0rc2
sphinxcontrib-devhelp==1.0.2
sphinxcontrib-htmlhelp==2.0.0
sphinxcontrib-jsmath==1.0.1
sphinxcontrib-plantuml==0.24.1
sphinxcontrib-qthelp==1.0.3
sphinxcontrib-serializinghtml==1.1.5
sphinxcontrib.applehelp==1.0.3
sphinxcontrib.jquery==3.0.0
urllib3==1.26.14
zipp==3.11.0

Thank you in advance for the help. My guess is that with the updated rtd-theme we need to do something different in conf.py to enable the theme, but I'm not sure what that is. Any help is appreciated.

Hi @bneradt :wave:
We have just added a new pre-release, sphinx-rtd-theme==1.2.0rc3
There is an issue with a workaround, kindly refer to the workaround here: https://sphinx-rtd-theme.readthedocs.io/en/latest/changelog.html#known-issues

Thank you @benjaoming . I don't think that jquery is the issue though. Rather a coworker noticed that the issue is related to this code block:
https://github.com/apache/trafficserver/blob/9b47f6d514c54aa46fb4a1845d02a8de3c24475c/doc/conf.py#L286-L298
# Include a stylesheet that overrides default table styling, to provide
# content wrapping.
html_context = {
    'css_files': [
        '_static/override.css'
    ]
}
if os.environ.get('READTHEDOCS', None) == 'True':
    html_context = {
        'css_files': [
            'https://media.readthedocs.org/css/sphinx_rtd_theme.css',
            'https://media.readthedocs.org/css/readthedocs-doc-embed.css',
            '_static/override.css'
        ]
    }

If we remove that block, the theme renders fine (but not with our overide.css of course). It seems that worked fine with 0.5.2 but not with the latest release. Is there a preferred method to add an override.css file?
It seems maybe what we're seeing is related to: https://github.com/readthedocs/readthedocs.org/issues/2116

If we remove that block, the theme renders fine (but not with our overide.css of course). It seems that worked fine with 0.5.2 but not with the latest release. Is there a preferred method to add an override.css file?

Needed to read through all these old issues. You need to use this configuration instead:
https://www.sphinx-doc.org/en/master/usage/configuration.html#confval-html_css_files
It was added in place of all these old hacks. I think this is what you'd want?

Actually, we wound up using add_css_file in a setup function in conf.py. That solves our issue:
https://github.com/apache/trafficserver/pull/9316
Thanks for the help!


MRC not aligned with perspective of pv camera (XRSDK)

Describe the bug
When using Mixed Reality Capture holograms aren't aligned with pv camera  (XRSDK). It seems that the transformation offset isn't considered when rendering from the pv camera. With mixed reality built-in xr platform integration hologram alignment ist correct. The problem exists only with XRSDK.
To reproduce
Steps to reproduce the behavior:

Using mrtk InteractableExamples-scene
Added mrtk + xrsdk
Attach TrackedPoseDriver to MainCamera
Using cloned DefaultXRSDKConfigurationProfile, just enabled Render from PVCamera
Build XR SDK app to Hololens (ARM)

Expected behavior
Aligned holograms with PV camera
Screenshots

Your setup (please complete the following information)

Unity 2019.4.9f1 and Unity 2020.1.6f1
Mixed Reality Toolkit 2.4.0
Windows XR Plugin 4.0.2
XR Plugin Managment 3.2.15

Target platform (please complete the following information)

HoloLens 2

This is currently a known issue that we're tracking with Unity. I'll update this thread when I have more info!

Additional detailed repro steps captured in a duplicate issue https://github.com/microsoft/MixedRealityToolkit-Unity/issues/9169#issue-793922581

Additional detailed repro steps captured in a duplicate issue https://github.com/microsoft/MixedRealityToolkit-Unity/issues/9169#issue-793922581

This is currently a known issue that we're tracking with Unity. I'll update this thread when I have more info!

Have there been any updates on this issue or is there a Unity bug tracker for it?

Apologies about my lack of update here. This should be fixed on the latest Windows XR Plugin across the latest Unity versions. I can narrow down the versions if needed, and please let me know if you're still seeing issues on the latest!

@keveleigh I can confirm that it's resolved and records correctly in the only configuration I tested:
Unity 2020.3.12f1 with Windows XR Plugin 4.5.0
The warning from https://github.com/microsoft/MixedRealityToolkit-Unity/pull/9170 is still present in the MRTK MRC settings as of MRTK <IP_ADDRESS>:

Thanks!

@camnewnham Ah, thanks for pointing that out! It's probably been enough time that we can remove that now. I'll get it sorted tomorrow!

@keveleigh Hi, I meet the same issue recently and I’m using MRTK3. How to set the “render from PV camera” function in MRTK3?

@keveleigh Hi, I meet the same issue recently and I’m using MRTK3. How to set the “render from PV camera” function in MRTK3?

I believe it's on by default now when you're using the Mixed Reality OpenXR Plugin. If you're looking to turn it off, there's a checkbox in the Mixed Reality Features settings:

@keveleigh Hi, I meet the same issue recently and I’m using MRTK3. How to set the “render from PV camera” function in MRTK3?

I believe it's on by default now when you're using the Mixed Reality OpenXR Plugin. If you're looking to turn it off, there's a checkbox in the Mixed Reality Features settings:

Thank you for your reply! I’m using holographic remoting app now, is it possible to implement “render on PV camera” in this mode?

Thank you for your reply! I’m using holographic remoting app now, is it possible to implement “render on PV camera” in this mode?

I don't believe it used to be, but someone from Microsoft can comment on the current state of things (@AMollis? @yl-msft?)


Remove basil from 5 plugins

Description
I adopted these plugins to release fixes related to the removal of Commons HttpClient 3.x from core. The fixes have been merged and released and no regressions have been reported, so my work here is done. I have ensured each plugin has the adopt-this-plugin label for easy adoption for the next maintainer.

https://github.com/jenkinsci/extra-tool-installers-plugin
https://github.com/jenkinsci/github-pr-coverage-status-plugin
https://github.com/jenkinsci/ivy-plugin
https://github.com/jenkinsci/jdepend-plugin
https://github.com/jenkinsci/nexus-artifact-uploader-plugin

Submitter checklist for adding or changing permissions

Always

[x] Add link to plugin/component Git repository in description above

When enabling automated releases (cd: true)

[ ] Add a link to the pull request, which enables continous delivery for your plugin or component.
Follow the documentation to ensure, your pull request is set up properly.

When adding new uploaders (this includes newly created permissions files)

[ ] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[ ] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[ ] Make sure to @mention the users being added so their GitHub account names are known if they require GitHub merge access (see below).
[ ] All newly added users have logged in to Artifactory and Jira at least once

Reviewer checklist (not for requesters!)

[ ] Check this if newly added person also needs to be given merge permission to the GitHub repo (please @ the people/person with their GitHub username in this issue as well). If needed, it can be done using an IRC Bot command
[ ] Check that the $pluginId Developers team has Admin permissions while granting the access.
[ ] In the case of plugin adoption, ensure that the Jenkins Jira default assignee is either removed or changed to the new maintainer.
[ ] If security contacts are changed (this includes add/remove), ping the security officer (currently @Wadeck) in this pull request. If an email contact is changed, wait for approval from the security officer.

There are IRC Bot commands for it

Thank you!


Issue 290 error removing toil/src directory from python path (resolves #290)

Catch ValueError when toil is installed into virtualenv from pip.
Resolves #290

Can one of the admins verify this patch?

Jenkins, test this please.

Okay, this works for me

Alden, could you rebase and squash into one commit? Then remove the needs work label. Thank you!

Are we sure that each entry in sys.path is a real path with symlinks resolved? If not, this may still break in other situations, e.g. when a sys.path entry has symlinks in it, when it is a relative path, etc. I'm willing to let this slide since it is just a temporary hack to begin with.

You're right, sys.path can contain symlinks. I've gone back to your original idea.

Yes, concurrent array modification has unpredicted behavior:
    >>> a=[1,2,4,3]
    >>> for x in a:
    ...     if x % 2 == 0:
    ...         a.remove(x)
    ...
    >>> a
    [1, 4, 3]


formula_desc_cop: Whitelist "etc." and fix the full stop autocorrect

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[] Have you successfully run brew tests with your changes locally?

As per https://github.com/Homebrew/brew/pull/2795#issuecomment-340287453, here's a PR to whitelist "etc." only in the full stop format_desc_cop, and fix the autocorrect so it works with quotes at the end of the line too. I ran a brew audit --strict and it stopped complaining about formulae with etc. at the end.

Looks good, thanks @issyl0! Would you mind writing some tests to check these work as expected? Thanks again!

@issyl0 Any news on that? If you'd rather not: no worries.

Sorry @MikeMcQuaid, this week has been stupidly busy and I'm unlikely to get time to do this soon. Sorry for leaving this hanging too. Also, from what I can see, the existing tests always test that violations are reported, not that they're ignored, if that makes any sense, and negating that for this one case feels odd.

Will merge without tests this time but please write some in future. Thanks @issyl0!


table Display misses parentheses

This is the output of a query's .to_string, from https://github.com/PRQL/prql/pull/1413/files#diff-238f7d8cbd582e19ac0d1da76d1d7ec3985c0204c0140c571967815ba42abec8R6 .
The s-string needs surrounding parentheses; this wouldn't be valid.
table grouping = s"
  SELECT SUM(a)
  FROM tbl
  GROUP BY
    GROUPING SETS
    ((b, c, d), (d), (b, d))
"

This would be a good first issue — it should be quite reasonable to add parentheses to Table's Display trait output.

Shall I work on this issue? @max-sixty

@max-sixty a consequence of removing onboarding label in favor of good-first-issue is that all good-first-issues should have a detailed explanation...
But regarding the issue: I think it'd be best to change how parser works, because having parenthesis here is not really necessary.

OK very much agree on the quality of the issue, sorry @Rishav1707 . We'll write a better issue.
But @aljazerzen re how we do this — check out my comments on https://github.com/PRQL/prql/issues/1394#issuecomment-1369334315 — how are you envisaging parsing it without parentheses? We could have the rvalue in the parser be (s_string | nested_pipeline), though it would be special-casing that.

I think we can parse an regular expr_call here, because this would also make sense:
table a = take 4 (from employees)

So for the normal case of using a pipeline, you'd have to use parenthesis, just like a regular pipeline.

You're absolutely right — #1422


[QUESTION] How to make a Sequencer send events to its Synthetizer?

Documentation Confirmation
Please confirm that you have read the documentation and have not found an answer to your question there.
Yes but I'm still new to JS, I'm very sorry if this is trivially obvious.
Question's Topic
Both spessasynth_lib and the demo js code
The Question
In demo_main.js, I only modified demoInit to add the following:
await window.manager.ready; // Existing
// NEW CODE FOR DYNATUNE
// Create worker with proper URL
let dynaTuner = new Worker(new URL('../js/worker/dynatuner.js', import.meta.url), { type: 'module' });

// Set up message handler first
dynaTuner.onmessage = (e) => {
    window.manager.synth.systemExclusive(e.data);
};

// Set up direct note event handlers
window.manager.synth.eventHandler.addEvent('noteon', 'dynatune-noteon', e => {
    if (e.channel !== 10) { // Skip drum channel
        dynaTuner.postMessage({ type: 'noteon', note: e.midiNote, velocity: e.velocity });
    }
});

window.manager.synth.eventHandler.addEvent('noteoff', 'dynatune-noteoff', e => {
    if (e.channel !== 10) {
        dynaTuner.postMessage({ type: 'noteoff', note: e.midiNote });
    }
});

window.manager.synth.eventHandler.addEvent('stopall', 'dynatune-stopall', () => {
    dynaTuner.terminate();
    dynaTuner = new Worker(new URL('../js/worker/dynatuner.js', import.meta.url), { type: 'module' });
});
// Unsure if I need this? Doesn't hurt for now.
window.addEventListener('unload', () => {
    dynaTuner = new Worker(new URL('../js/worker/dynatuner.js', import.meta.url), { type: 'module' });
});

The only other modification I made to the library was adding in a /worker/dynatune.js Worker that runs every ~0-50 ms depending on the load of the matrix operations. It communicates with the main thread via the MessageEvent system, e.g. the onmessage function in demo_main.js. This is purely to allow forwarding the System Exclusive message to the Synth whenever DynaTuner calls postMessage(messageData);
This is my first JS project, and it's been a blast making all these mistakes until finally getting a working version. The only thing that doesn't work is when I load a MIDI file into the example demo_main.js site, the window.manager.synth.eventHandler does not trigger on any noteOns, noteOffs, etc. It works just fine for live input via the mouse/keyboard. So I'm guessing the Sequencer does not notify the Synthetizer of noteOn events despite still rendering the note?
I'm sure there is an easy solution. I even tried modifying play.js from:
// skip note ons
const info = getEvent(event.messageStatusByte);
// Keep in mind midi ports to determine channel!!
const channel = info.channel + (this.midiPortChannelOffsets[this.midiPorts[trackIndex]] || 0);
switch (info.status)
{
    case messageTypes.noteOn:
        break;
    case messageTypes.noteOff:
        break;
    case messageTypes.keySignature:
        break;

to
// skip note ons
const info = getEvent(event.messageStatusByte);
// Keep in mind midi ports to determine channel!!
const channel = info.channel + (this.midiPortChannelOffsets[this.midiPorts[trackIndex]] || 0);
switch (info.status)
{
    // For DynaTune
    case messageTypes.noteOn:
        this.synth.noteOn(channel, event.messageData[0], event.messageData[1], false);
        break;
    case messageTypes.noteOff:
        this.synth.noteOff(channel, event.messageData[0]);
        break;
    case messageTypes.keySignature:
        break;

The behavior was exactly the same, e.g. my DynaTuner Worker is still never notified of which midiNote and velocities are 'On' or which midiNote is 'Off'. If the solution involves something else, I'll revert these changes locally. If you could tell me how you think I can make the Sequencer send 'noteon' and 'noteoff' events, the dynamic tuning algorithm will finally be complete other than adding some UI changes :))
Thank you so much for this wonderful library and all the support you give for it. It means so much. I wouldn't have started this project if SpessaSynth didn't exist!

Hi,
The sequencer does notify the event system.
Adding
await manager.ready;
console.log(window.manager.synth);
window.manager.synth.eventHandler.addEvent("noteon", "test-noteon", e =>
{
    console.log(e);
});

seems to work fine in my case:

I don't know what your worker does, but it could be the cause of your issue. Try console.logging the events in the event handler.
OR
You may have forgotten to recompile demo_main.min.js. The index.html points to the min version, not the demo_main.js file itself!


Talos reporting Lambda memory address - Suggestion

Currently a parameter dictionary with arbitrary Lambdas simply spits out Lambda Object <mem address> for any reporting. This is perhaps not the nicest way of expressing this.
Perhaps a simple approach would be, if there's no decent (defined or otherwise) string conversion to reference them as the parameter entry key, and the index.
So "my_param_key": [Lambda x: x**2, Lambda x: x**3] would then report "my_param_key_0" and "my_param_key_1" in the reporting tables.
I haven't delved too deep into the Talos codebase to see the feasibility of this. Was just wanting to report this behaviour, as it's somewhat annoying to the user as they are unsure which lambda was selected.
PS - I'm aware that a richer object could be used to wrap this, which has its own __repr__ method defined. But as far as baked in implementations go, this could be a small tweak to bring more user friendly output.

I guess one simple way to allow a broader ability to have custom labels for parameters would be to support using a tuple as well as just values. So then one might have [(val1, name1), (val2, name2)] instead of [val1, val2]. I think this would be easy to implement, and might be useful in other cases as well. What do you think?

I was thinking perhaps use of a dictionary. At the moment we have Lists for discrete selections, and tuples for ranges. Dictionaries seems to me to be the perfect entry.
E.g
params = {
  "batch_norm": [ {"Batch Norm": Lambda x: BatchNorma...}, {"No Batch Norm": Lambda x: x} ]
}

Enables a check to see if the params values within List are of type dict, then we can go to assign a special label, otherwise it will use repr ?

I can have a look at it. Not sure when I'd get the time with work ramping up in the New Year. Is there an easy way to contact you? I believe I have my e-mail address on my profile if that works.

What do you think if we just instead make it so that if the format is dict then the label will used. That way it supports any arbitrary codes.


Skills menu doesn't appear with skills unlocked

Close this as it was already fixed. Forgot to reference it in the commit


Update jsoniter-scala-core, ... to 2.23.5

About this PR
📦 Updates

com.github.plokhotnyuk.jsoniter-scala:jsoniter-scala-core
com.github.plokhotnyuk.jsoniter-scala:jsoniter-scala-macros

from 2.21.4 to 2.23.5
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.github.plokhotnyuk.jsoniter-scala" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.github.plokhotnyuk.jsoniter-scala" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, version-scheme:early-semver, commit-count:1

Superseded by #432.


Unable to set log file path in server.conf

This on a windows box. When attempting to set the server.conf option for log-file, it is throwing a Permission Denied error (even though the folder has full permissions)
the path to the server log file
log-file = c:\logFolder
===
Traceback (most recent call last):
File "counterparty-server.py", line 17, in
File "counterparty-server.py", line 16, in server_main
File "C:\counterparty\counterparty-cli\counterpartycli\server.py", line 109, in main
File "C:\counterparty-cli-win32-1.0.0\library\counterpartylib\server.py", line 301, in initialise
log.set_up(root_logger, verbose=verbose, logfile=config.LOG)
File "C:\counterparty-cli-win32-1.0.0\library\counterpartylib\lib\log.py", line 36, in set_up
fileh = util_windows.SanitizedRotatingFileHandler(logfile, maxBytes=max_log_size, backupCount=5)
File "C:\Python34\lib\logging\handlers.py", line 150, in init
BaseRotatingHandler.init(self, filename, mode, encoding, delay)
File "C:\Python34\lib\logging\handlers.py", line 57, in init
logging.FileHandler.init(self, filename, mode, encoding, delay)
File "C:\Python34\lib\logging_init_.py", line 1006, in init
StreamHandler.init(self, self.open())
File "C:\Python34\lib\logging_init.py", line 1030, in _open
return open(self.baseFilename, self.mode, encoding=self.encoding)
PermissionError: [Errno 13] Permission denied: 'c:\logFolder'

hum.. log-file should contains a filename not a folder.. however it not should be a problem (logFolder can be a file name) unless the folder 'c:\logFolder' already exists ?

yes. it look like it has to be the full path including the log file name.
As for the standalone binaries (this one https://github.com/CounterpartyXCP/counterparty-cli/releases/tag/v1.0.0 ?? ). It complains about a missing package...json file. I have to first git and install the python counterparty-cli tools and run that once. Only then can I run the standalone libraries at https://github.com/CounterpartyXCP/counterparty-cli/releases/tag/v1.0.0

great.
I opened an issue for the second problem: https://github.com/CounterpartyXCP/counterparty-cli/issues/20 thanks.


opensearch: add selector to volumeClaimTemplates

Signed-off-by: Pierre Peloille<EMAIL_ADDRESS>Description
This pr adds the ability to specify a selector in the volumeClaimTemplate for the opensearch chart.
Issues Resolved
Fixes #239
Check List

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

For any changes to files within Helm chart directories:

[x] Helm chart version bumped
[x] Helm chart CHANGELOG.md updated to reflect change

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

Hi @Pale-whale would you address @prudhvigodithi comment?
Thanks.

Hello everyone, sorry I did not update this pr sooner.
I corrected the templating thanks to the suggestion of @prudhvigodithi and updated the Chart and the CHANGELOG to match version 1.10.
Let me know if I need to make any other changes, thanks :)

Thanks @Pale-whale
@TheAlgo @prudhvigodithi please review again I will correct any version errors if any once you approve.
Thanks.

Hey @Pale-whale, just wanted to know if helm install of the OpenSearch chart worked for you with adding as ?
matchLabels:
    type: gp2

I see its breaking at
Error: INSTALLATION FAILED: YAML parse error on opensearch/templates/statefulset.yaml: error converting YAML to JSON: yaml: line 34: mapping values are not allowed in this context

This error is actually misleading, more details I have added to the issue.
@Pale-whale

Hi @Pale-whale any updates based on @prudhvigodithi comments?
Thanks.

@Pale-whale you didnt sign DCO again I think.

This change will break the chart installation as mentioned in issue https://github.com/opensearch-project/helm-charts/issues/239 .

Hey @Pale-whale close this PR, please check the #239 for more details.


Garbage Collection

Hi,
I know that the spec is still in development. But, I would like to know more how the garbage collection works with Kotlin and implications for Strong References.
I would like to understand better how Kotlin interacts with the garbage collector.
When variables are ready to be destroyed, how we can manage memory leaks...
I tried to search, but I haven't found much information.
Thanks in advance

I know there are sessions 11.6.2. | 9.9 |  18.2 | 9.7.3.
But, all of them are still tagged as TODO 😢

As of now (May 2020), details of garbage collection are platform-dependent and should be covered in the respective platform-specific parts of the specification.


E: Unable to locate package python3.9

Problem
When using a Dockerfile to build the CyberBattleSim image, you will encounter an error message similar to the following:
...
E: Unable to locate package python3.9
E: Couldn't find any package by glob 'python3.9'

Cause
The problem occurs because of the Dockerfile. It uses the image FROM mcr.microsoft.com/azureml/onnxruntime:latest-cuda, which is based on Ubuntu version 18.04 (Bionic). Inside the init.sh script of this image, the command add-apt-repository ppa:deadsnakes/ppa is used to add a repository to the apt package manager. However, this repository is no longer available for Ubuntu Bionic (18.04) because it reached its end-of-life on June 6th, 2023. As a result, the package manager cannot find the Python 3.9 package.
You can find more information about this issue in this link: https://github.com/deadsnakes/issues/issues/251.
Solution
To fix this issue, you have two options:

Manually install Python 3.9 within the Dockerfile.
Modify the Dockerfile to use a different base image. An alternative option is to use FROM openvino/onnxruntime_ep_ubuntu20:latest, which is based on Ubuntu version 20.04 (Focal), and this Ubuntu version is still supported by ppa:deadsnakes/ppa

I choose second solution as a workaround because the first option might be a bit cumbersome. The image openvino/onnxruntime_ep_ubuntu20:latest can be obtained from this source: https://github.com/microsoft/onnxruntime/tree/main/dockerfiles#openvino


Not able to see my previous chat

Describe the bug
When I open bolt.new and open the side menu I can't see my previous chat.  I had a full app that I was almost finishing it and now I can't find it.
Link to the Bolt URL that caused the error
https://bolt.new/
Steps to reproduce
As you can see I don't see any project here in this screenshot

But if I open https://stackblitz.com/ I can see my previous project, only the ones that I created a repo. But unfortunately the project that I was working on I didn't create the repo and I don't see how can I recover it.

Expected behavior
I should be able to see all my projects.
Screen Recording / Screenshot
No response
Platform

OS:  macOS
Browser: Chrome
Version: Version 130.0.6723.117 (Official Build) (arm64)

Additional context
No response

Appreciate the feedback! We are aware of this issue with chat persistence. Temporary workarounds and updates can be found here: #39. (Go to stackblitz.com, login (same credentials as Bolt), click Collections on the left-hand side, click Bolt Collection). Appreciate your patience as improvements continue to be made.


Game Request: Civ 5

Feature Request
Feature Information:

If possible setup civ 5 pitboss or something with amp.
I confirm:

[x] that I have searched for an existing feature request matching the description.

Not possible, Civ 5 does not have a headless server. Also wrong issue template used.


Implement Subobjects, as well as the subobject presheaf

This PR defines a notion of subobject, and also implements the subobject presheaf. This PR builds off of #202, so that should get merged before this. This should also close #203.
Notes
Interestingly, we don't need to say that 𝒞 is well-powered for this to work! I'm a bit of a novice on size issues in general, but I suspect that the notion of "well-poweredness" does not apply for this library.
References
https://ncatlab.org/nlab/show/subobject
Categories For the Working Mathematician, V.7
Sheaves in Geometry and Logic, I.3

Rebased on top of current master.


Feat/create simplify app

This simplifies the app scaffolding. It's a combination of things we've discussed here and there, and some new things. It's a big change, I'd like to get an approval from the whole core team one this one :)
The goal for me is to reduce the amount of files and package.json scripts which are generated, and to make it a general use case for all types of projects.
Most of the generated files comes from reusing the component generator for each of the app components. This creates a readme, index.js etc. which don't really fit the components in an app project. I changed this to just static files in the app generator.
Overview of changes:

index.js is removed, we just keep MyElement.js and my-element.js
README.md is removed
components/my-component/src is removed, apps don't have public/private files.
stories/my-element.stories.mdx is changed to stories/my-element.stories.js since that fits app components better
demo/index.html is removed, there is already an app-level demo

The final generated file structure if you turn on all options:
├── owc-app/
│   ├── .storybook/
│   │   ├── main.js
│   │   └── preview.js
│   ├── components/
│   │   ├── owc-app/
│   │   │   ├── stories/
│   │   │   │   └── owc-app.stories.js
│   │   │   ├── test/
│   │   │   │   └── owc-app.test.js
│   │   │   ├── aboutTemplate.js
│   │   │   ├── open-wc-logo.js
│   │   │   ├── owc-app.js
│   │   │   └── OwcApp.js
│   │   ├── page-main/
│   │   │   ├── stories/
│   │   │   │   └── page-main.stories.js
│   │   │   ├── test/
│   │   │   │   └── page-main.test.js
│   │   │   ├── page-main.js
│   │   │   └── PageMain.js
│   │   ├── page-one/
│   │   │   ├── stories/
│   │   │   │   └── page-main.stories.js
│   │   │   ├── test/
│   │   │   │   └── page-one.test.js
│   │   │   ├── page-one.js
│   │   │   └── PageOne.js
│   ├── .editorconfig
│   ├── .gitignore
│   ├── custom-elements.json
│   ├── index.html
│   ├── karma.conf.js
│   ├── LICENSE
│   ├── package.json
│   ├── README.md
│   └── rollup.config.js

I also made some other changes in this PR:

Removed snapshot testing scaffolding. Snapshot testing is useful, but not for all types of projects. Having it in by default doesn't feel right, we have it in our docs so people can choose to add it when it makes sense for their project. Removing it reduces the amount of package.json scripts by 2.
Removed browserstack testing scaffolding. I don't feel 100% certain about this one. Browserstack testing is very useful, but not every project can or will make use of it. It feels a bit overkill to generate the files for it when a lot of people need to remove it later on. Removing this reduces 1 generated file in the root of the project, and one package.json script.

@daKmoR @thepassle @bennypowers @Westbrook

You can try out the changed in the generator by checking out the branch, doing cd packages/create and yarn build && node dist/create.js

Very nice 😊
Could we simplify even more? E.g. create-react-app just scaffolds a App.js, App.css, and App.test.js. I think it would be really nice to cut down the total amount of files to:
├── owc-app/
│   ├── .storybook/
│   │   ├── main.js
│   │   └── preview.js
│   ├── components/
│   │   ├── owc-app/
│   │   │   ├── stories/
│   │   │   │   └── owc-app.stories.js
│   │   │   ├── test/
│   │   │   │   └── owc-app.test.js
│   │   │   ├── open-wc-logo.js
│   │   │   ├── owc-app.js
│   │   │   └── OwcApp.js
│   ├── .editorconfig
│   ├── .gitignore
│   ├── custom-elements.json
│   ├── index.html
│   ├── karma.conf.js
│   ├── LICENSE
│   ├── package.json
│   ├── README.md
│   └── rollup.config.js

I think it makes things a lot more clear. We could also then remove the 'pseudo router' which is now causing a11y tests to fail for skip-links. As another added bonus, it looks even less like a "fake monorepo", which is a real issue that users still get confused by

@thepassle so we wouldn't have pages anymore in that case?

@LarsDenBakker correct

It feels like we would miss out on some guidance to users. Perhaps we should put effort again into owc-starter-app with more opinionated project structure, library usage etc.?

My personal experience is that whenever I use our scaffolding, the first thing I do is remove the pseudo router, and remove the PageOne and PageMain. I dont think the pseudo router is really that great of a pattern that we should be recommending as a default. I'm also not sure what guidance they'd miss out on exactly by removing those pages
It would be interesting to hear some feedback from users to see what they think and how they use it.
I also dont think we should separate things again by having a separate owc-starter-app and the generator, keeping the generator up to date proves to be hard enough already.

There are so many things that have come to mind, in the history of our little team's usage. Here are some out of order observations:

super nice to be able to generate a more complete organization of an app, even just to see how to organize our own. In that view, I select everything.
ditto on each comment/change above
There is no ts flavor, that would be nice to be able to select that and generate ts style instead
We seem to have two realities for our wc. Simple for a project unshared unstoried, and more canonical storybook like that has the fully featured. Some wc are just dumb containers for other wc or such
Our simple unshared wc has no src folder, no separate index, just a folder named for the wc and an index.ts which holds everything. No testing or storybook or anything else, just that one file.
the act of removing generated files seems innocuous to me but others on the team get weird as if "owc sucks" so there is that illogical reality - so now you know.
a folder named components would be better for us as wc - shorter and more accurate
what about non-wc libraries such as redux, router etc.  Yes, I know it works for everyone else in the world, I wistfully look over my shoulder and see others using the generator without this problem. Call the whaaaambulance.

After thinking about it some more, I agree with @thepassle . The generator should be a way to set up the project and you shouldn't need to delete a lot of stuff before getting started.
I think it's useful to provide guidance on structuring your application, but there are so many ways to do that there is no one size fits all.

I'm for @thepassle 's minimal case

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: LarsDenBakker:x: Pascal SchilpPascal Schilp seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Helm Charts for 0.12.1 and 0.12.2 are missing in helm charts repo

While the releases for

0.12.1 https://github.com/keptn/keptn/releases/tag/0.12.1
0.12.2 https://github.com/keptn/keptn/releases/tag/0.12.2
are on GitHub, the helm charts are not in https://github.com/keptn/helm-charts/tree/gh-pages/packages

Thanks for the issue! This happened due to some release pipeline changes to being backported to the 0.12.x maintenance branch. Therefore the new changes regarding how Keptn releases helm charts were not there, which in turn means that the 0.12.* charts were not pushed.
I fixed this manually now, and it should update on ArtifactHub in a few minutes.


Change Homepage

How can i make the products page in the example
the home page, the page when the root url visited?
Thank you.

That's actually more of a Gatsby issue, so I don't personally know off the top of my head, but they have a few options for getting support: https://www.gatsbyjs.org/community/

@erquhart, check out my fork for an example of changing the default home:
https://github.com/rustygreen/gatsby-starter-netlify-cms


How do I extend the semantics of an inherited grammar?

What I'm trying to do is to extend the es5 grammar and add some custom semantics.
First off I am able to essentially eval the es5 code like this:
var ohm = require('ohm-js')
var es5 = require('ohm-js/examples/ecmascript/es5')
var code = '2 + 3'
var grammar = es5.grammar
var match = grammar.match(code)
var semantics = es5.semantics
var result = semantics(match)
assert(eval(result.asES5) === 5)

This appears to actually work. But what I would like to do is to extend es5 and add my own semantics which then allow me to expand those semantics into javascript.
For example suppose I wrote my own grammar like this:
Example <: ES5 {
  example = "example"
  keyword += example
  Declaration += ExampleDeclaration
  ExampleDeclaration = "example" #(sc)
}

Now, I modify my original code to be more like this:
var ohm = require('ohm-js')
var es5 = require('ohm-js/examples/ecmascript/es5')
var code = 'example;'
var grammar = ohm.grammar(fs.readFileSync('./g.ohm'), { ES5:  es5.grammar })
var match = grammar.match(code)

var semantics = // ??

var result = semantics(match)
assert(eval(result.asES5) === 5)

What I would like to do, for this example, is to essentially transform the matched ExampleDeclaration into 2 + 3. How can I use the es5 semantics object but modify it to do custom stuff just for my matched semantics?

Here's how you'd create a semantics for your Example grammar that inherits from the semantics that you created for the ES5 grammar:
var semantics = grammar.extendSemantics(es5.semantics);

You can extend / override the semantic actions of the operations and attributes that semantics inherited from its super-semantics using the extendOperation and extendAttribute methods, respectively. Here's a link to the relevant part of the documentation: https://github.com/cdglabs/ohm/blob/master/doc/api-reference.md#user-content-semantics

Ok great, specifically I did this:
var modifiedSourceActions = {
    ExampleDeclaration: (example) => {
        return '2 + 3'
    }
}
var semantics = grammar.extendSemantics(es5.semantics)
semantics.extendAttribute('modifiedSource', modifiedSourceActions);

One last question if I may!
Suppose I wanted to return an AST here instead of  calling asES5?
For example:
var ast = result.asAST
var t = transformer.match(ast) // another grammar that transforms the ast further

I think I want to take multiple transformation steps before reducing down to javascript.

This is pretty sweet! Inheriting from es5 lets me write tests directly in the code file being tested:
model Foo {
   field x
}

assert(new Foo().x)

Then my test code just spins through all of my source files and evals them basically. Very cool!


words.txt lacks words that are in words_alpha.txt

Example :
# cat words_alpha.txt|grep ^ned
ned
nedder
neddy
neddies
nederlands
# cat words.txt|grep ^ned
nedder
neddies
#

The documentation states that words_alpha.txt is a subset from words.txt which apparently is not the case as of now.

I think this is just a case sensitivity issue.
$ cat words.txt|grep -i ^ned
NED
Neda
NEDC
Nedda
nedder
Neddy
Neddie
neddies
Neddra
Nederland
Nederlands
Nedi
Nedra
Nedrah
Nedry
Nedrow
Nedrud

While it would be nice for these files to be perfectly formatted, this is a good reminder to clean your data before doing calculations.

This problem does exist, however. I found 25 missing words with these python3 commands (pasted here for reference):
> import requests
> r = requests.get('https://raw.githubusercontent.com/dwyl/english-words/words.txt')
> r.status_code
200
> w = set(r.text.lower().split())
> len(w)
466546
> r = requests.get('https://raw.githubusercontent.com/dwyl/english-words/words_alpha.txt')
> r.status_code
200
> wa = set(r.text.lower().split())
> len(wa)
370103
> missing = wa - w
> len(missing)
25
> missing
{'preinferredpreinferring', 'stegnosisstegnotic', 'tangantangan', 'false', 'sturdiersturdies', 'peroxidicperoxiding', 'gynecicgynecidal', 'coevolvedcoevolves', 'preobtrudingpreobtrusion', 'kestrelkestrels', 'aliyahaliyahs', 'coracoprocoracoid', 'cylindrocylindric', 'killeekillee', 'antinganting', 'epigonousepigons', 'snailfishessnailflower', 'outwardsoutwarred', 'regeneratoryregeneratress', 'cryptocurrency', 'quadriquadric', 'subsultorysubsultus', 'brigantinebrigantines', 'caducecaducean', 'hypophypophysism'}

Note that there's this other problem of there seemingly being several words that have been merged together somehow, but it's also true that not all words in words_alpha.txt are in words.txt (ex "false").


docs: Leader Receiver Creator ADR

Description
Changes proposed in this pull request (what was done and why):

Changes refer to particular issues, PRs or documents:

https://github.com/kyma-project/telemetry-manager/issues/1003

Traceability

[ ] The PR is linked to a GitHub issue.
[ ] New features have a milestone set.
[ ] New features have defined acceptance criteria in a corresponding GitHub Issue, and all criteria are satisfied with this PR.
[ ] The corresponding GitHub issue has a respective area and kind label.
[ ] The follow-up issues (if any) are linked in the Related Issues section.
[ ] Adjusted the documentation if the change is user-facing.
[ ] The feature is unit-tested
[ ] The feature is e2e-tested

/hold

/test pull-telemetry-manager-build

/unhold


[Tiktok] Unable to download webpage: 'An existing connection was forcibly closed by the remote host'

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2021.12.17
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

C:\yd>youtube-dl.exe https://www.tiktok.com/@klrdubs/video/7020527490435140870 -v
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['https://www.tiktok.com/@klrdubs/video/7020527490435140870', '-v']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2021.12.17
[debug] Python version 3.4.4 (CPython) - Windows-2012ServerR2-6.3.9600
[debug] exe versions: none
[debug] Proxy map: {}
[TikTok] Setting up session
ERROR: Unable to download webpage: [WinError 10054] An existing connection was forcibly closed by the remote host (caused by ConnectionResetError(1005
4, 'An existing connection was forcibly closed by the remote host', None, 10054, None)); please report this issue on https://yt-dl.org/bug . Make sure
 you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\extractor\common.py", line 634, in _request_webpag
e
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\YoutubeDL.py", line 2288, in urlopen
  File "C:\Python\Python34\lib\urllib\request.py", line 464, in open
  File "C:\Python\Python34\lib\urllib\request.py", line 482, in _open
  File "C:\Python\Python34\lib\urllib\request.py", line 442, in _call_chain
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpupik7c6w\build\youtube_dl\utils.py", line 2737, in https_open
  File "C:\Python\Python34\lib\urllib\request.py", line 1186, in do_open
  File "C:\Python\Python34\lib\http\client.py", line 1227, in getresponse
  File "C:\Python\Python34\lib\http\client.py", line 386, in begin
  File "C:\Python\Python34\lib\http\client.py", line 348, in _read_status
  File "C:\Python\Python34\lib\socket.py", line 378, in readinto
  File "C:\Python\Python34\lib\ssl.py", line 748, in recv_into
  File "C:\Python\Python34\lib\ssl.py", line 620, in read

Description

Tiktok parsing fails every time with "Unable to download webpage: [WinError 10054] An existing connection was forcibly closed by the remote host" error. I tried with different video URLs and also from different countries. The latest official build was used.
There was a similar problem in the past (Issue 10084), one of the top developers investigated it but had to close because it did not reproduce. It'd be good to check it the same way now, since the call stack is identical. In the past the problem was with a different site, and now it seems to be 100% reproducible.

To repeat the response to #30545...
The TikTok extractor seems to fail in all sorts of interesting ways, even with PR #30224, PR #30479 or other patches. The yt-dlp extractor for TT, which doesn't try to look at the actual webpage, succeeds with your target.

Thank you so much, sir! I saw that fork in the past, but I am happy to hear a confirmation from you. Hope to see porting of that logic to the original repository.

From my experience this error is related to https requests on Windows. Never experienced it on POSIX.
--no-check-certificate option might help...

From my experience this error is related to https requests on Windows. Never experienced it on POSIX. --no-check-certificate option might help...

Tried, unfortunately did not work for me. On Ubuntu it hangs on '[TikTok] Setting up session'.

What about --user-agent 'Mozilla/5.0', or perhaps some real UA string fro a current browser?

I have an alternate tool that works:
PS C:\> tiktok<PHONE_NUMBER>435140870
GET http://api2.musical.ly/aweme/v1/aweme/detail/?aweme_id=7020527490435140870
GET http://v16m-default.akamaized.net/26569ec19350e923d1e04d11e8effe98/61eb2540/video/tos/useast2a/tos-useast2a-ve-0068c004/1b518c34ba36434a9fca617d5458ecb7/

https://github.com/89z/mech

you can also download the video if you have an browser add-on
that allows right click,  just rith click on the video and select "save video as "
to download the video as mp4.

I have an alternate tool that works:
...

a) as the log shows, it uses the same tactic as yt-dlp: see https://github.com/ytdl-org/youtube-dl/issues/30538#issuecomment-1015864370;
b) thanks to your test it is clear that some unwanted or non-default header is causing the issue;
c) for form's sake: that's nice..

What about --user-agent 'Mozilla/5.0', or perhaps some real UA string from a current browser?

Tried both ways, same error.

@dirkf did you ever manage to create your own repository? I dont see one:
https://github.com/dirkf?tab=repositories&q=&type=source
Its OK, one day you will learn how to program.

I have an alternate tool that works:
PS C:\> tiktok<PHONE_NUMBER>435140870
GET http://api2.musical.ly/aweme/v1/aweme/detail/?aweme_id=7020527490435140870
GET http://v16m-default.akamaized.net/26569ec19350e923d1e04d11e8effe98/61eb2540/video/tos/useast2a/tos-useast2a-ve-0068c004/1b518c34ba36434a9fca617d5458ecb7/

https://github.com/89z/mech

Thank you sir, it works and is very FAST.

From discussions elsewhere, HTTP/2 may be the discriminator that TT is using. According to the Go net/http documentation, HTTP/2 is the default, which would explain why the above tool isn't blocked. yt-dl Python versions only know about HTTP/1.1.

Thank you sir, it works and is very FAST.

How do you get this to work? The author barely provided instruction on how to use it.

You should ask that question on the other repo

You should ask that question on the other repo

They specifically said in the description that they only help patreon/paid users.

OK fair enough, but why you do think asking a question about another repo here is a good idea?

OK fair enough, but why you do think asking a question about another repo here is a good idea?

Where else? Do you expect me to pay them if I instantly use their stuff? That is unfair

this is YouTube-DL repo, so you should probably limit you questions to issues regarding YouTube-DL

I suppose @koopa990 realises that "them" is @89z?

I suppose @koopa990 realises that "them" is @89z?

yes.

I am sorry for the distraction my original comment (from January) caused. I removed it, but someone quoted it, so here we are. I would repeat that anyone with a question regarding that comment, should not be asking about it on the YouTube-DL tracker.

I am sorry for the distraction my original comment (from January) has caused. I removed it, but someone quoted it, so here we are. I would repeat that anyone with a question regarding that comment, should not be asking about it on the YouTube-DL tracker.

Thank you very much for the repo.


feat: avm/res/db-for-postgre-sql/flexible-server

Description
Changed "highAvailability" parameter default value to "zoneredundant" to comply with WAF resiliency pillar.
Pipeline Reference

Pipeline

Type of Change

[ ] Update to CI Environment or utilities (Non-module affecting changes)
[X] Azure Verified Module updates:

[ ] Bugfix containing backwards-compatible bug fixes, and I have NOT bumped the MAJOR or MINOR version in version.json:

[ ] Someone has opened a bug report issue, and I have included "Closes #{bug_report_issue_number}" in the PR description.
[X] The bug was found by the module author, and no one has opened an issue to report it yet.

[ ] Feature update backwards compatible feature updates, and I have bumped the MINOR version in version.json.
[ ] Breaking changes and I have bumped the MAJOR version in version.json.
[X] Update to documentation

Checklist

[X] I'm sure there are no other open Pull Requests for the same update/change
[X] I have run Set-AVMModule locally to generate the supporting module files.
[X] My corresponding pipelines / checks run clean and green without any errors or warnings

Hey @arnoldna, please beware the conflicts


popovers: Show user and group popovers when pill clicked in stream and group settings.

Clicking on the user/group pill in stream/group settings UI to select user/group pill opens up the user/group popover. Applies to both user/group creation and editing UI.
Hover style on the x button of pill occurs only when the x button is hovered, not when the pill is hovered.
Additionally one of the commit optimizes the removePill function of input_pill.ts module as discussed in the CZO.
CZO Thread
Fixes: #28687

Screenshots and screen captures:
X button hover

Group Settings

Stream Settings

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[ ] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Thanks! The PR mostly looks good in manual testing, but I noticed that if I navigate "back" while a popover is open, it sticks around, rather than disappearing like it should.

Thanks for spotting this out Alya, I'll put up a fix for this.

I meant clicking the "back" button in my browser.

@alya, thanks for the clarification. I've fixed the bug you pointed out.

Thanks! @sahil839 are you up for reviewing this next? I haven't re-tested.

@ecxtacy Hello! Good work here, are you still working on this PR? I noticed that it has been a while since you've updated it, I'm willing to pick it up if you've decided to work on other things. peace

Hi @PieterCK , you can pick up the work. In case you need assistance feel free to contact me. Thanks.

OK, thanks for confirming. I'll fork your PR and co-author it.

Closing in favor of https://github.com/zulip/zulip/pull/29898 for this issue.


Add "system queue" semantics for node to node communication for Azure Service Bus, SQS, Rabbit MQ

See if this can sneak in by 1.0

It's already in for Rabbit MQ. I'm breaking this apart.


[QUESTION] How to rescue Faulted volumes

Question
After a node crash, I end up with two faulted volumes that can not be attached, they can't be mounted to /dev/longhorn/, but the longhorn pods works fine, and the image files looks fine to me.
time="2022-07-22T09:01:17Z" level=info msg="All replicas are failed, set engine salvageRequested to true"
accessMode=rwo controller=longhorn-volume frontend=blockdev migratable=false node=node1 owner=node1
state=detached volume=pvc-3cc715b2-aaa2-4c1d-a788-ffc71905874c

time="2022-07-22T09:01:17Z" level=info msg="All replicas are failed, set engine salvageRequested to true"
accessMode=rwx controller=longhorn-volume frontend=blockdev migratable=false node=node1 owner=node1
shareEndpoint= shareState=stopped state=detached volume=pvc-04e953eb-5411-4433-82a4-e6e54aa7fb92

I wonder if I can try to fsck or somehow repair the filesystem?
Environment

Longhorn version: 1.2.4 (recently upgraded from 1.2.3)
Kubernetes version: v1.24.2+k3s2
Node config

OS type and version debian bulleye
CPU per node: 20
Memory per node: 128G
Disk type RAID10 HDD
Network bandwidth and latency between the nodes: singleton node

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal): K3S on debian

Additional context
I have another volume can be mounted without errors.

We need more information about the volume to understand the current state of the volume. Can you past the output of:

kubectl get volumes.longhorn.io pvc-3cc715b2-aaa2-4c1d-a788-ffc71905874c -o yaml -n longhorn-system
kubectl get replicas.longhorn.io -n longhorn-system | grep pvc-3cc715b2-aaa2-4c1d-a788-ffc71905874c . Then for each replicas, `kubectl get replicas.longhorn.io -n longhorn-system  -o yaml
kubectl get engines.longhorn.io -n longhorn-system | grep pvc-3cc715b2-aaa2-4c1d-a788-ffc71905874c . Then `kubectl get replicas.longhorn.io -n longhorn-system  -o yaml

We need more information about the volume to understand the current state of the volume. Can you past the output of:

kubectl get volumes.longhorn.io pvc-3cc715b2-aaa2-4c1d-a788-ffc71905874c -o yaml -n longhorn-system
kubectl get replicas.longhorn.io -n longhorn-system | grep pvc-3cc715b2-aaa2-4c1d-a788-ffc71905874c . Then for each replicas, `kubectl get replicas.longhorn.io -n longhorn-system  -o yaml
kubectl get engines.longhorn.io -n longhorn-system | grep pvc-3cc715b2-aaa2-4c1d-a788-ffc71905874c .

Can you manually try to salvage the volume by:

Make sure the node node1  and the disk /data/longhorn are schedulable
Scale down the statefulset lab-shihs to 0
Goto Longhorn UI -> click on the volume to go to volume detail page -> click on the top right menu bar -> select salvage replica ->  select the replica
Scale up the statefulset lab-shihs to 0

On the other hand, can you check the setting Automatic salvage in Longhorn UI -> setting -> general setting to see if it is ON

Can you manually try to salvage the volume by:

Make sure the node node1  and the disk /data/longhorn are schedulable
Scale down the statefulset lab-shihs to 0
Goto Longhorn UI -> click on the volume to go to volume detail page -> click on the top right menu bar -> select salvage replica ->  select the replica
Scale up the statefulset lab-shihs to 0

On the other hand, can you check the setting Automatic salvage in Longhorn UI -> setting -> general setting to see if it is ON

The disk is not schedulable, it alerts this message:
Last Transition Time: 17 days ago
Message: the disk default-disk-7f5c98b5a858e751(/data/longhorn) on the node node1 has<PHONE_NUMBER>200 available, but requires reserved<PHONE_NUMBER>00, minimal 25% to schedule more replicas
Reason: DiskPressure
Status: False

However<PHONE_NUMBER>00/4531630899200 = 0.071, which is enough for the minimal.
When try to salvage the replica, the output is
unable to salvage volume pvc-04e953eb-5411-4433-82a4-e6e54aa7fb92: Disk with UUID 10540b0a-c68e-43f9-8dd0-91dc4cca4e1c on node node1 is unschedulable for replica pvc-04e953eb-5411-4433-82a4-e6e54aa7fb92-r-0eeb2c9e

And the Automatic salvage setting is ON.
Thank you for your help.

Can you go to Longhorn UI -> click on node tap -> take a screenshot and post it here.
The log message Message: the disk default-disk-7f5c98b5a858e751(/data/longhorn) on the node node1 has<PHONE_NUMBER>200 available, but requires reserved<PHONE_NUMBER>00, minimal 25% to schedule more replicas might be misleading as there are different conditions that can lead to the disk being unschedulable: storage available < storage reserved AND StorageScheduled < maximum allocable storage. See code https://github.com/longhorn/longhorn-manager/blob/4fa1cc197e893c0243599398d3c4bbf021e7b0e7/controller/node_controller.go#L759-L765
TODO: Longhorn should improve the log here

Can you go to Longhorn UI -> click on node tap -> take a screenshot and post it here.
The log message Message: the disk default-disk-7f5c98b5a858e751(/data/longhorn) on the node node1 has<PHONE_NUMBER>200 available, but requires reserved<PHONE_NUMBER>00, minimal 25% to schedule more replicas might be misleading as there are different conditions that can lead to the disk being unschedulable: storage available < storage reserved AND StorageScheduled < maximum allocable storage. See code https://github.com/longhorn/longhorn-manager/blob/4fa1cc197e893c0243599398d3c4bbf021e7b0e7/controller/node_controller.go#L759-L765
TODO: Longhorn should improve the log here

Thanks

Thanks @simsicon How about kubectl get nodes.longhorn.io node1 -n longhorn-system -o yaml?

@simsicon There is another setting that controls the disk scheduling, the picture below. You can low it down to make the disk schedulable

By the way, it seems that there are a lot of disk space are using by other applications on your disk besides Longhorn. Is it correct?

kubectl get nodes.longhorn.io node1 -n longhorn-system -o yaml

apiVersion: longhorn.io/v1beta1
kind: Node
metadata:
creationTimestamp: "2022-03-01T07:41:00Z"
finalizers:

longhorn.io
generation: 14
name: node1
namespace: longhorn-system
resourceVersion: "64222787"
uid: 3f3c1355-f697-4ab2-af7b-c8df523ca12c
spec:
allowScheduling: true
disks:
default-disk-7f5c98b5a858e751:
allowScheduling: true
evictionRequested: false
path: /data/longhorn
storageReserved:<PHONE_NUMBER>00
tags: null
engineManagerCPURequest: 0
evictionRequested: false
name: node1
replicaManagerCPURequest: 0
tags: null
status:
conditions:
MountPropagation:
lastProbeTime: ""
lastTransitionTime: "2022-03-01T07:41:00Z"
message: ""
reason: ""
status: "True"
type: MountPropagation
Ready:
lastProbeTime: ""
lastTransitionTime: "2022-08-02T00:59:44Z"
message: Node node1 is ready
reason: ""
status: "True"
type: Ready
Schedulable:
lastProbeTime: ""
lastTransitionTime: "2022-03-01T07:41:00Z"
message: ""
reason: ""
status: "True"
type: Schedulable
diskStatus:
default-disk-7f5c98b5a858e751:
conditions:
Ready:
lastProbeTime: ""
lastTransitionTime: "2022-07-10T12:24:23Z"
message: Disk default-disk-7f5c98b5a858e751(/data/longhorn) on node node1
is ready
reason: ""
status: "True"
type: Ready
Schedulable:
lastProbeTime: ""
lastTransitionTime: "2022-07-17T05:47:07Z"
message: the disk default-disk-7f5c98b5a858e751(/data/longhorn) on the node
node1 has<PHONE_NUMBER>000 available, but requires reserved<PHONE_NUMBER>00,
minimal 25% to schedule more replicas
reason: DiskPressure
status: "False"
type: Schedulable
diskUUID: 10540b0a-c68e-43f9-8dd0-91dc4cca4e1c
scheduledReplica:
pvc-3cc715b2-aaa2-4c1d-a788-ffc71905874c-r-14116c2d:<PHONE_NUMBER>00
pvc-04e953eb-5411-4433-82a4-e6e54aa7fb92-r-0eeb2c9e:<PHONE_NUMBER>00
storageAvailable:<PHONE_NUMBER>000
storageMaximum:<PHONE_NUMBER>1072
storageScheduled:<PHONE_NUMBER>00
region: ""
zone: ""

After lower the minimal storage percentage, the node is back to schedulable, and the faulted disks back to degraded but attachable,

now I can see it is attached to /dev/longhorn, but I can not use it in my pods, the pods output is
 Normal   SuccessfulAttachVolume  3m26s                attachdetach-controller  AttachVolume.Attach succeeded for volume "pvc-3cc715b2-aaa2-4c1d-a788-ffc71905874c"                                                                                │
│   Warning  FailedMount             85s                  kubelet                  Unable to attach or mount volumes: unmounted volumes=[vol-01vd4], unattached volumes=[volume-localtime vol-xqhgx vol-vzmve vol-iwxcc vol-01vd4 vol-bhyma vol-zqby7 vol-gz │
│ wz volume-devmem vol-z7i0r vol-6rzut vol-5v1jh kube-api-access-9dfc8[]: timed out waiting for the condition                                                                                │
│   Warning  FailedMount             70s (x9 over 3m20s)  kubelet                  MountVolume.Setup failed while expanding volume for volume "pvc-3cc715b2-aaa2-4c1d-a788-ffc71905874c" : Expander.NodeExpand found CSI plugin kubernetes.io/csi/driver.lon │
│ ghorn.io to not support node expansion

Oh yes, I have a lot of other data using local-path provision, I am evaluating switching to longhorn.

I have upgraded the longhorn to 1.2.4 from 1.2.3 before the issue, I don't think it's has a relation to the issue, but worth mention it.  Did you try to increase the size of the PVC? Can you check kubectl get pvc  lab-shared-pvc -n lab -o yaml ?

Why does Kubelet want to expand this volume? Did you try to increase the size of the PVC? Can you check kubectl get pvc lab-shared-pvc -n lab -o yaml ?

Yes, back to then the issue just happened, I thought it was caused by volume size not enough, so I had clicked the expand volume button.
apiVersion: v1
kind: PersistentVolumeClaim
metadata:
  annotations:
    pv.kubernetes.io/bind-completed: "yes"
    pv.kubernetes.io/bound-by-controller: "yes"
    volume.beta.kubernetes.io/storage-provisioner: driver.longhorn.io
  creationTimestamp: "2022-03-15T08:58:06Z"
  finalizers:
  - kubernetes.io/pvc-protection
  name: lab-shared-pvc
  namespace: lab
  resourceVersion: "14715141"
  uid: 3cc715b2-aaa2-4c1d-a788-ffc71905874c
spec:
  accessModes:
  - ReadWriteOnce
  resources:
    requests:
      storage: 100Gi
  storageClassName: longhorn
  volumeMode: Filesystem
  volumeName: pvc-3cc715b2-aaa2-4c1d-a788-ffc71905874c
status:
  accessModes:
  - ReadWriteOnce
  capacity:
    storage: 100Gi
  phase: Bound

Ok. It seems that the filesystem has not been appropriately resized. I would have to review the expansion flow from the code perspective to determine the root cause


enable uploader to support non-uid uploads

closes #158

LGTM

LGTM, pending one comment.

Let's leave this hanging for a moment. I'm almost ready to test.