CheckBox has fixed Height/MinWidth in Template

Describe the bug
CheckBox has odd sizing due to fixed values in template (Height, MinWidth, etc...), this means if you just want to use the 'check box' without text as part of another layout/setup then it's difficult to just drop-in place.
I'd expect if I had just a <CheckBox/> element for it's bounding box to just be around the check like so:

(Or maybe with some Padding, but the Padding property is ignored by the check and only effects the inner Content.)
But it's not and currently does this:

Version Info

NuGet package version: 2.4.2

Windows 10 version
Saw the problem?

Insider Build (19041)
Yes

November 2019 Update (18363)

May 2019 Update (18362)

October 2018 Update (17763)

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Device form factor
Saw the problem?

Desktop
Yes

Mobile

Xbox

Surface Hub

IoT

Additional context
Fixed values in template:
https://github.com/microsoft/microsoft-ui-xaml/blob/6e99a4150772813f9f2303de162e5698164197dc/dev/CheckBox/CheckBox_themeresources.xaml#L293-L294
https://github.com/microsoft/microsoft-ui-xaml/blob/6e99a4150772813f9f2303de162e5698164197dc/dev/CheckBox/CheckBox_themeresources.xaml#L639-L659

Hopefully with WinUI 3 we can find a more robust solution for CheckBox and RadioButton. Since those controls are not aware of the line height, essentially the padding for the tick/radio control needs to be hardcoded to correctly align with the text.
See #1520 for more on this.

This is in part by design. We set default values to 32 x 120 for minimum touch target considerations. We don't want checkbox to have very small touch footprints by default. If you want to change this any value you set should override the setter.

Hopefully with WinUI 3 we can find a more robust solution for CheckBox and RadioButton. Since those controls are not aware of the line height, essentially the padding for the tick/radio control needs to be hardcoded to correctly align with the text.
See #1520 for more on this.

Hopefully WinUI 3 will allow changes to the text controls to introduce Baseline alignment options, as well as x-height and other typographic metric alignment.

Hopefully with WinUI 3 we can find a more robust solution for CheckBox and RadioButton. Since those controls are not aware of the line height, essentially the padding for the tick/radio control needs to be hardcoded to correctly align with the text.
See #1520 for more on this.

Hopefully WinUI 3 will allow changes to the text controls to introduce Baseline alignment options, as well as x-height and other typographic metric alignment.

This is a cool idea, do you know if we already have a proposal for it?

@StephenLPeters saw you added needs-author-feedback but then this auto-closed the bug ☹
I didn't see an action back to me? I do understand touch targets can be a consideration, but it does make it harder to customize the default experience without this being called out somewhere, most other things like buttons don't have a minimum size like this.

@michael-hawker Does setting the minheight/width not resolve your issue? I guess I'm a little confused.  Also is it true that button doesn't have a min size? I was pretty sure it did for the same reasons.  I guess it expands to fill that space where Checkbox does not.

@StephenLPeters I was able to override the min width/height and work around it I believe, I was just confused at it seemed to behaving differently than the button with blank content vs. content.


It's almost same with SCLAlertView

Did you edit @dogo's https://github.com/dogo/SCLAlertView ? I think you add to bottom
Inspired By SCLAlertView

Hey @invisible66,
I'm aware of SCLAlertView as I've used it in a few projects. However, FCAlertView wasn't inspired or edited from it. It's actually been completely designed for sketch for our app at ShiftRide, you can see the original mockups (done in Sketch) here:

Cheers!

Ok, i got it 👍 :)

Cheers! 😀


remove inner snippet

Related to #1255
Hide whitespace changes for easier diff

Related PR: https://github.com/dotnet/samples/pull/243


Support FR3?

Type of Issue
Select the type of issue:

[ x ] Feature request (to request an additional feature)

Description
Is polymetis planning to support the Franka Research Arm 3? It seems like Franka is not selling Emika Panda arms anymore, so this seems important. We are thinking of buying a Franka FR3 and we'd like to use polymetis :)

Any updates on this?

@exhaustin Hey Austin! Hope you've been well. Would love to hear if you guys might include some internal stuff for FR3. One concrete issue is that the joint limits are pretty different.

Hi @AlexanderKhazatsky! Again sorry for the slow response.
As I am no longer working at Meta, and that the team is no longer using the Franka Panda, support for Polymetis will be dropped.
I know this is disappointing news, I'm happy to discuss offline to see how I can help.

Hey Austin, sad to hear about the Polymetis support being dropped :(
On another note, I'd love to reach and chat with you offline whenever you have the chance. Is there an email I can contact you at privately?

@exhaustin

In case anyone reads this in the future, I've been using polymetis with an FR3 (after manually modifying the joint limits), and haven't had issues yet.


Update disabled for mat-select does not fully work

Bug, feature request, or proposal:
I'm creating an application with Electron and I have a strange behavior with "mat-select". I have this form:
<form>
  <mat-form-field style="width:400px;margin-right:15px;">
    <input name="directory" matInput placeholder="Directory" [(ngModel)]="directory" (input)="directoryChanged()" />
  </mat-form-field>
  <mat-form-field style="width:400px;margin-right:15px;">
    <mat-select placeholder="Build action" [disabled]="isValidDirectory === false">
      <mat-option [value]="0">None</mat-option>
      <mat-option [value]="1">Incremental</mat-option>
      <mat-option [value]="2">Full</mat-option>
    </mat-select>
  </mat-form-field>
  <mat-form-field style="width:400px;margin-right:15px;">
    <mat-select placeholder="Build configuration" [disabled]="isValidDirectory === false">
      <mat-option [value]="0">Debug</mat-option>
      <mat-option [value]="1">Release</mat-option>
    </mat-select>
  </mat-form-field>
  <mat-form-field style="width:400px;margin-right:15px;">
    <mat-select placeholder="Deployment file" [disabled]="isValidDirectory === false">
      <mat-option *ngFor="let file of files" [value]="file">{{ file }}</mat-option>
    </mat-select>
  </mat-form-field>
  <mat-form-field style="width:400px;margin-right:15px;">
      <input name="Test" matInput placeholder="Test" [disabled]="isValidDirectory === false" />
    </mat-form-field>
</form>

The handler "directoryChanged" is the following:
directoryChanged(): void {
  this.isValidDirectory = true;
}

So it simply toggles the value of the isValidDirectory variable to true. The expected result is that "mat-select" and "matInput" should become visually (and "physically") enabled, however, once I typed something in the first "matInput", the "mat-select" remain underlined with a dotted line (showing them as disabled) while the last "matInput" shows a full line (showing that it became enabled). However, if I click on a "mat-select", the dropdown menu appears and the line becomes full, so it seems that the "mat-select" has to be clicked to notice that it is actually enabled.
What is the expected behavior?
It should be visually obvious that the "mat-select" component became enabled.
What is the current behavior?
The "mat-select" elements looks disabled.

Here is a reproduction
https://stackblitz.com/edit/angular-material2-issue-xdnxes?file=app/app.component.html

And an interim workaround would be to use select.stateChanges.next() to let the form-field know it needs to perform change detection.
https://stackblitz.com/edit/angular-material2-issue-mjjk2j?file=app/app.component.html

And should be fixed by https://github.com/angular/material2/pull/7673.

Closing since https://github.com/angular/material2/pull/7673 has been merged.


Secure download links in UI

Download links in UI are rendered as hypertext links. These need to be converted to something that allows an authentication header

Need to mirror behavior in https://github.com/visulate/visulate-ora2pg/pull/53/files

Fixed in #56 and #57


TMP: Bump dropwizard and pay commons

superseded by https://github.com/alphagov/pay-publicauth/pull/498#pullrequestreview-526247511


SPEC: Make UUID generation more vague.

Make spec more vague around the issue of POD UUIDs.  This
allows for generation of UUIDs to occur on the metadata
service itself if desired by the implementation.
Fixes #324

I agree with your comments -- I was not sure how descriptive to be in technical writing like this. I will make those changes.

@schancel as a general comment based on my experience so far shepherding the spec - it's definitely better to stray on the side of being too verbose and illustrative (with examples, etc), as when we've stuck to purely technical language people haven't always grasped the subtleties/use cases/etc. We can start to clean this up as we approach 1.0 and move towards splitting the spec out into different sections, adopting a more formal structure, etc etc.

ping on this

I liked your wording, so I used that under the UUID definition.  I didn't want to restate much under the Metadata Service section, so there's only a short sentence and link to the UUID section.

:+1: thanks!


Extra empty blue banner is showing for SceneViewer with matterport scene

What happened:
When loading a matterport scene in Grafana v10, there will be an extra empty blue banner showing up that is not interact-able. The same scene will not have the blue banner when using Grafana v9, and non-matterport scene is not having the blue banner in v10

What you expected to happen:
There should be no blue banner.
How to reproduce it (as minimally and precisely as possible):
With Grafana v10, load a matterport scene in SceneViewer panel
Anything else we need to know?:
From console log, there is a failed matterport API call. But since the scene is loading fine, if it's causing the blue banner, then we want to remove it for this case.
Environment:

Grafana version: 10.2.3
Plugin version: 1.11.0

Reaching out to the TwinMaker team at AWS to see their thoughts on this

@sheilaXu is this on Grafana Cloud? Looks like the cloud trial / licence banner
What is SceneViewer?

SceneViewer is a panel visualization that comes installed with the twinmaker datasource/app.

I'm a bit stumped but my current hunch is that there is some kind of style change here that is causing unexpected style changes in Twinmaker's Scene Viewer Panel.
The Blue Bar itself is coming from within the matterport-viewer webcomponent which I don't believe grafana has any direct control over:

What's interesting is that this "error-bar" component exists in the dom in earlier versions of grafana as well it's just invisible:

Is it possible for the Twin Maker team to edit the css in the Matterport Web component?

This blue bar is coming from Matterport, the likely fix is to upgrade the Matterport SDK Version, update the IotAppKit version in the Grafana Plugin, and see. Otherwise, I can reach out to our contacts at Matterport to get some next steps.

Update that TM team may have a fix in for this and is awaiting testing feedback from customer, then we can close this out.

TM team mentioned we can close this out, customer verified. Thank you!


No option to remove timestamps form Model

When saving an instance of a model, the column created_at and updated_at can't be missing in the table.
Option to disable these fields would be nice

This will help http://adonisjs.com/docs/3.0/lucid#timestamps


Bump to our PR onto wasmd

DO NOT MERGE
only for testing purposes

No longer needed for testing, closing.


Body scroll blocking when component attached to container

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[ ] Performance issue
[X] Feature request
[ ] Documentation issue or request
[ ] Support request
[ ] Other... Please describe:

Current behavior
When trying to make a portal or directly call dialog when loading for specific components blocks (lets assume like a dashboard), the body was set with ngneat-dialog-hidden class and blocks entire page scroll.
Expected behavior
Choose which DOM element scroll should be blocked (body, container it attached to etc)
Minimal reproduction of the problem with instructions
constructor(
        private dialogService: DialogService,
        private el: ElementRef
    ) {
      this.dialogService.open(SpinnerBoxComponent, {
            container: this.el,
            closeButton: false,
            enableClose: false,
            width: 'auto',
        })
    }

What is the motivation / use case for changing the behavior?
The possibility to control the dialog component per component scope (like dashboard module stats etc) and to not block page scroll.

Environment

Angular version: 14.1.2

Browser:
- [ ] Chrome (desktop) version XX
- [ ] Chrome (Android) version XX
- [ ] Chrome (iOS) version XX
- [ ] Firefox version XX
- [ ] Safari (desktop) version XX
- [ ] Safari (iOS) version XX
- [ ] IE version XX
- [X] Edge version 116.0.1938.54 (Official build) (arm64)

For Tooling issues:
- Node version: XX
- Platform:

Others:

You're welcome to create a PR


Psr0Fixer - ignore file without name or with name started by digit

Fix for #1522

LGTM :+1:
of the record, how would this fixer handle other file names with characters not supported as a class name, like non-UTF-8 stuff?

Please, give an example.
BTW
D:\GIT\fork
λ cat Żabka.php
<?php

class Żabka {}

D:\GIT\fork
λ php -l Żabka.php
No syntax errors detected in »abka.php

class Psr0FixerTest extends AbstractFixerTestBase
{
    public function testIgnoreFileStartedByDigit()
    {
        $file = $this->getTestFile('$test.php');

        $expected = <<<'EOF'
<?php
namespace Aaa;
class Bar {}
EOF;

        $this->makeTest($expected, null, $file);
    }

I'm just trying this out here to be honest ;)

@SpacePossum let us use same regex to validate classname as PHP does

sound like a solid idea @keradus :)
A PSR4 fixer might benefit from this as well. Do you know where to find such regex, I don't but I'm happy to start looking.

I'll need to update my PSR4 PR with this too.

In, 2.x, maybe we need to ditch the PSR0 fixer, and just have a fixer to fix the namespace, and another to fix the filename.

i like that thought


stdenv: add pkgsCross.powernv.linux_latest to tests.cross.sanity

Description of changes
This commit adds pkgsCross.powernv.linux_latest to tests.cross.sanity now that it builds again.
Targeting staging because this needs #245449.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

@ofborg build pkgsCross.powernv.linux_latest


[OAP Server] Log bridge and receiver package name.

Add commons-logging bridge for elastic search client.
Rename receiver package name.

Coverage remained the same at 22.896% when pulling 7a22f5deaa33940826cf36803cf79f520bca2147 on peng-yongsheng:feature/oap/test into 06165a0359c38c655e71ca8c29c9c06ccd4aec53 on apache:6.0.


Documentation fixes

Capitalization fixes in the 2.0 docs.

Coverage remained the same at 71.972% when pulling d1c3612a5001e2d241d63600388b88ff01b7f16a on DaanDeMeyer:documentation-fixes into b2e162f2d761d8ee8b95adcbceafe1504f81a276 on nnnick:master.


Make transfer use Credit

Addresses https://github.com/DACH-NY/daml-finance/issues/262
This forces holdings allocated to a specific account to have the same type.
It relies on a public party mechanism to disclose the holding factory.
Open questions:

we need one holding factory per custodian, so maybe it makes sense to introduce a HoldingFactoryFactory
upgrade scenario for holdings does look a bit more complex now
are we happy with the public party disclosure mechanism?
given that we are coupling accounts and holdings, should we introduce 1 account implementation per holding type instead of relying on holding factories?

Upgrade scenario for holdings:

introduce V2 holding
for each V1 holding factory, introduce V2 holding Factory
for each account referencing the V1 factory, create new account referencing V2 factory
replace each V1 holding with a V2 holding at the new account

A possibility to simplify the upgrade scenario is to have multiple holding factories per account and use Daml exceptions to make sure the correct factory is used to Credit the asset on a transfer.

I will merge the PR and open issues to track the open items


update governance document link

Thanks for contributing!

What type of PR is this?

release/none

What this PR does / why we need it:
update governance document link
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

ready for merging ?

yes

wait...

it's ready to be merged


jdt-language-server: init at 1.5.0

Motivation for this change
It doesn't seem like there are any Java language servers in nixpkgs so far
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Also addresses #97814

This is several version behind the current (0.66.0) so I'll have to update it. That being said, I also am not sure if a few things are correct - the /tmp runtime directory, the JDK version, etc. It works for me, but is a bit buggy in some projects. This might be due to my particular setup though.

Looking into this more, my issue was that I had set GRADLE_HOME incorrectly.
# incorrect
export GRADLE_HOME="${pkgs.gradle}";

# correct
export GRADLE_HOME="${pkgs.gradle}/lib/gradle";

Also, I think this should probably be set to use JDK15 because with JDK11 it doesn't work in JDK15 projects (probably no project with a JDK version > 11).

@fzakaria are you able to review this? Pretty please with a cherry on top. 🍒

I'm not sure where GRADLE_HOME comes into the picture here ?
I started to look at the submission but haven't tested it locally yet; I'm dealing with some personal issues at the moment (COVID) so i'm a bit behind everything :(
My 2cents on the matter though that the description:

It works for me, but is a bit buggy in some projects. This might be due to my particular setup though.

Does not inspire confidence in submission. Should a package be included if that's the current state ? Maybe it needs the broken attribute then.
Anyways thank you for the contribution and I will try give it a more in-depth review this week.

Thank you @fzakaria! I hope everything is alright. Keep safe, no rush, life always comes first. ❤

@fzakaria Thank you for having a look! I hope everything is alright on your end as well.

I'm not sure where GRADLE_HOME comes into the picture here ?

This was just a response to my previous comment that I was having issues, but I had set GRADLE_HOME incorrectly in the environment where I started the language server.

It works for me, but is a bit buggy in some projects. This might be due to my particular setup though.

Does not inspire confidence in submission. Should a package be included if that's the current state ? Maybe it needs the broken attribute then.

This is true, but I believe said bugginess & uncertainty is a result of two factors. (1) jdt-language-server is unfortunately not well documented and I've had to rely on digging through code to fix issues along the way, and (2) most people use jdt-language-server through vscode-java, but I use neovim and LanguageClient-neovim, for which there isn't tonnes of good info on how it should be integrated. I've been using this setup for a few months now, and it mostly works. Issues that I have, I believe are due to my setup and the language server itself being not that great, and not due to packaging. I wish there were another good option for a Java language server, but it doesn't seem there is.

I just updated my fork and pushed that, and also upgraded to 0.67.0. Everything except for the version upgrade is how I've been running it for the past month or so, I just hadn't updated the PR. Sorry about that.

I'm working in something similar:
https://github.com/NixOS/nixpkgs/pull/107424
I wanted to do the options more configurable, still working on adding gradle support .
Probably we can combine both PRs

@fzakaria Thank you for reviewing this and sorry about the very late response. Things got a bit busy.

@fzakaria Thank you for reviewing this and sorry about the very late response. Things got a bit busy.

Is there any update one this one? I need jdtls and it seems like this PR is working yet not merged?

I just tried out the implementation and it works fine in nixos, just updated the version for 1.3.0. version.

@willcohen Thanks for your comment. I've actually been using 1.7.0 for quite some time now (Dec 19th). I haven't been keeping this branch up to date because it doesn't seem to be going anywhere. But maybe now we can push this through the gate :-)
Could you try out the derivation and make sure it works on your end too?

Result of nixpkgs-review pr 99330 run on x86_64-darwin 1

Trying to run this PR is causing nothing to actually get built for me. That said:
$ nix-build $NIXPKGS -A jdt-language-server
/nix/store/gpam4yj0dg9dzd0jy371q0gjd89w9ny9-jdt-language-server-1.7.0

Result of nixpkgs-review pr 99330 run on x86_64-darwin 1

1 package built:

jdt-language-server

I did not test this but a couple of people said it worked


Hyperlink missing

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
https://docs.github.com/en/webhooks/testing-and-troubleshooting-webhooks/troubleshooting-webhooks
There is a hyperlink missing for smee.io
What part(s) of the article would you like to see updated?
There is a hyperlink missing for smee.io under url-host-localhost-is-not-supported section
Additional information
Adding the missing hyperlink: https://smee.io/

@cmwilson21 I've updated the doc with the minor change.
Thank you!


xsd:long is "int" in validation rules - error with values that not fit

I was forced to turn off validation to avoid errors with values, that already bigger than 32bit integer.
And this is a question, what to do on different platforms - 32bit and 64 bit.

@GroxExMachine Do you think of another better solution than I found?

@mikaelcom I think, that this is a simple(in terms of realization) and good solution.

solved in 2.0.1 and 1.5.1


Typings for put.

Makes T extend Action, so code like:
yield put([])
Fails at compile time, instead of at runtime.

thanks

What if you want to put to a channel of type other than Action?

That's true, sorry about that. Fixed with https://github.com/yelouafi/redux-saga/pull/512

Missed that; thanks @aikoven for the catch


Connection issues, please check your Internet connection

He writes, here is such an error,( Connection issues, please check your Internet connection ) although there is Internet, since I sent this article the same way, restarting and reinstalling did not help, please tell me how to solve this

Unable to connect to Emerald Services. Please check your Internet connection


Incorrect WMS Key Value Pair syntax created by geturl()

geturl() creates incorrect WMS Key Value Pair (KVP) syntax.
The WMS specification tells us:

A URL prefix is defined as an opaque string including the protocol, hostname,
optional port number, path, a question mark '?', and, optionally, one or more server specific
parameters ending in an ampersand '&'.
A client appends the necessary request parameters as
name/value pairs in the form "name=value&". The resulting URL shall be valid
according to the HTTP Common Gateway Interface standard [CGI], which mandates the
presence of '?' before the sequence of query parameters and the '&' between each
parameter.
The URL prefix shall end in either a '?' (in the absence of additional server-specific
parameters) or a '&'.

So we should have a URL like:
http://host[:port]/path?{name[=value]&}
Using the following code as an example we can see that we don't get a closing ampersand instead the key (name) / value pairs seem to be added with a starting ampersand,  so we get a double && when parameters are appended to the advertised service URL
i.e.
...?language=eng&&service=WMS
from owslib import wms as wms

end_point = "http://ogc.bgs.ac.uk/cgi-bin/BGS_AGS_EN_Bedrock_and_Structural_Geology/wms?"

wm_130 = wms.WebMapService(end_point, version='1.3.0')
afmap = wm_130.getmap(layers=['AFG_AGS_1M_BLS'], styles=[], srs="EPSG:32641",
                      bbox=(180307, 2.97704e+006, 1.7569e+006, 4.6101e+006),
                      format="image/png", size=(1205, 735), transparent=True)
print(afmap.geturl())

wm_111 = wms.WebMapService(end_point, version='1.1.1')
afmap1 = wm_111.getmap(layers=['AFG_AGS_1M_BLS'], styles=[], srs="EPSG:32641",
                      bbox=(180307, 2.97704e+006, 1.7569e+006, 4.6101e+006),
                      format="image/png", size=(1205, 735), transparent=True)
print(afmap1.geturl())

The generated URLs are:
http://ogc.bgs.ac.uk/cgi-bin/BGS_AGS_EN_Bedrock_and_Structural_Geology/wms?language=eng&&service=WMS&version=1.3.0&request=GetMap&layers=AFG_AGS_1M_BLS&styles=&width=1205&height=735&crs=EPSG%3A32641&bbox=180307%2C2977040.0%2C1756900.0%2C4610100.0&format=image%2Fpng&transparent=TRUE&bgcolor=0xFFFFFF&exceptions=XML
http://ogc.bgs.ac.uk/cgi-bin/BGS_AGS_EN_Bedrock_and_Structural_Geology/wms?language=eng&&service=WMS&version=1.1.1&request=GetMap&layers=AFG_AGS_1M_BLS&styles=&width=1205&height=735&srs=EPSG%3A32641&bbox=180307%2C2977040.0%2C1756900.0%2C4610100.0&format=image%2Fpng&transparent=TRUE&bgcolor=0xFFFFFF&exceptions=application%2Fvnd.ogc.se_xml

Fixed in master.


Add abstract timestamped model

This PR adds an abstract TimestampedModel that provides created_at and updated_at fields.
The models that have these fields have been updated to inherit from this class.
No migrations are needed since the affected models already included these fields.

Codecov Report

Merging #71 (df89ed8) into develop (bef5c62) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop      #71      +/-   ##
===========================================
+ Coverage    90.65%   90.67%   +0.01%
===========================================
  Files           41       42       +1
  Lines         1220     1222       +2
  Branches       107      107
===========================================
+ Hits          1106     1108       +2
  Misses          90       90
  Partials        24       24

Impacted Files
Coverage Δ

apartment_application_service/models.py
100.00% <100.00%> (ø)

application_form/models.py
100.00% <100.00%> (ø)

users/models.py
97.14% <100.00%> (-0.08%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bef5c62...df89ed8. Read the comment docs.


v1.16 Backports 2024-08-06

[ ] #33438 (@harsimran-pabla)

Once this PR is merged, a GitHub action will update the labels of these PRs:
 33438

/test-backport-1.16


Help with GetPriceRangesAsync

Hello,
i´m trying to use "GetPriceRangesAsync" ...
I tried
var tradePileResponse = await _client.GetTradePileAsync();
var priceRanges = await _client.GetPriceRangesAsync(tradePileResponse.AuctionInfo.Select(ai => ai.CalculateBaseId()));
or
var tradePileResponse = await _client.GetTradePileAsync();
var priceRanges = await _client.GetPriceRangesAsync(tradePileResponse.AuctionInfo.Select(ai => ai.ItemData.Id));
...but  not working...
Someone could help me?

What is not working? Syntax error? Empty response?

Exception in the call...
Message:

Code: NotFound

Stack

at UltimateTeam.Toolkit.Requests.FutRequestBase.MapAndThrowException(Exception exception, FutError futError)
at UltimateTeam.Toolkit.Requests.FutRequestBase.d__22`1.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at UltimateTeam.Toolkit.Requests.PriceRangesRequest.d__2.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task task)
at FutClient2015.Gerenciadores.ManipuladorDaTradePile.d__4.MoveNext()

Your 2nd code example should work - Can you fiddler it and show me the request you are trying to send?

I don´t have fiddler in this machine and can´t install...
The final url request for example is:

https://utas.s2.fut.ea.com/ut/game/fifa16/marketdata/item/pricelimits?itemIdList=249351565296,250263101928,250220706821,250275582346,248928576073,250242086236,249813174275,244901970785,250294301185,250294789742,250236443555,249186881781,247168135328,249979347406,246055662757,245130100703,249898864779,246775317190,247105922225,246312427434,250286551046,248946132848,248115569759,250280513180,241257277685,249920968828,250300047226,249095343834,242813187848,247166941663

Normally this is the behaviour if you didn't call watchlist/tradepile before. This like I do:
            PriceRange priceRange = new PriceRange();
            List<long> savedItems = new List<long>();

            try
            {
                await GetTradepile();
                savedItems.Add(itemId);
                var priceRanges = await futClient.GetPriceRangesAsync(savedItems);

                foreach (PriceRange prices in priceRanges)
                {
                    priceRange.MaxPrice = prices.MaxPrice;
                    priceRange.MinPrice = prices.MinPrice;
                }

                return priceRange;
            }

This is currently working.

Ok, no problem...

Can you try again? It's working again at my app.

Same error... 404 - NotFound

Really weird - It's working in my code - Can proof you are not sending any requests between the tradepile request and the pricerange request?

The code is exactly like this:
var tradePileResponse = await _client.GetTradePileAsync(); var priceRanges = await _client.GetPriceRangesAsync(tradePileResponse.AuctionInfo.Select(ai => ai.CalculateBaseId()));

I'm afraid I cannot help without any Fiddler logs, because you see I'm using exactly the same code and it's working.

I will go home on Thursday and I can install on my PC and then show here the log

Maybe another user can confirm / not confirm the behaviour?

It's not working because you don't have that item in the tradepile.

Yes, that could be the reason. You need to have the item in your previous response or you get the 404.

Great @engate...
I had some items previously listed in TradePile . I removed them in the "select" and then is now working perfectly !
Thank you engate and tringler for the help!
The final code:
var tradePileResponse = await _client.GetTradePileAsync();
var ids = tradePileResponse.AuctionInfo.Where(ai => ai.Expires == -1).Select(ai => ai.ItemData.Id);
var priceRanges = await _client.GetPriceRangesAsync(ids);


Wallet

Implemented tooltips for hosting and wallet to use.

TODO after this PR: Document tooltip & notification system in tutorial

All comments were addressed and/or deferred to the next PR from my 'hosting' branch


Error: Cannot calculate TypeScript semantic issues without a valid project

Hi! I'm getting an error while linting some typescript files:
Error while parsing file.ts
Line undefined, column undefined: Cannot calculate TypeScript semantic issues without a valid project.
`parseForESLint` from parser `@typescript-eslint/parser` is invalid and will just be ignored

file.ts
  13:55  warning  Parse errors in imported module './constants': Cannot calculate TypeScript semantic issues without a valid project. (undefined:undefined)

I found strange lines in eslint-module-utils/parser.js
  // @typescript-eslint/parser will parse the entire project with typechecking if you provide
  // "project" or "projects" in parserOptions. Removing these options means the parser will
  // only parse one file in isolate mode, which is much, much faster.
  // https://github.com/import-js/eslint-plugin-import/issues/1408#issuecomment-509298962
  delete parserOptions.EXPERIMENTAL_useProjectService;
  delete parserOptions.project;
  delete parserOptions.projects;

If I comment this lines, all works perfectly.
        "@typescript-eslint/eslint-plugin": "6.21.0",
        "@typescript-eslint/parser": "6.21.0",

Related to: #1408

We don't yet support v6 of the TS eslint parser (meaning, we don't test on it).

Do you have any plans?

I believe @bradzacher is more suitable to answer the question.

Hard to say much of anything without some information. Would need a repro to be able to provide any advice.
Though the reported error message isn't one of ours.

I suspect this error comes from tsconfig-paths; it’s not one of ours either.

Reproduce repo

To fix this, we'll need to start testing on v6 of the TS eslint parser.

errorOnTypeScriptSyntacticAndSemanticIssues: true,
I had forgotten that even existed. I didn't know people used it - it's literally tge first time I've ever seen it used.
Turn that off and it'll work.

This is a common standard for our company, I'll ask if we can disable it.
But in any case, do you think this is normal behavior?

Yes. The behaviour of deleting the parser options is intended (and implemented by me).
eslint-plugin-import does its own out-of-band parsing so it can analyse exports of dependencies. To do this is uses your configured parser.
If we leave those parser options behind then it means that when import/ attempts to parse a dependency it will cause our parser to attempt to do a type-aware parse. If the file isn't part of a project (eg it's in node_modules) then this causes REALLY REALLY REALLY BAD BEHAVIOUR such as crashes or CRAZY BAD slowness (as you noted - #1408)
When I added those deletes 5 years ago I didn't remember errorOnTypeScriptSyntacticAndSemanticIssues existed.
In v6 we changed the underlying infra so that that option requires type information.
It's an undocumented parser option and shouldn't actually work within eslint anyways. We'll look at removing it entirely (filed https://github.com/typescript-eslint/typescript-eslint/issues/8681).
You should delete it from your eslint config.

Got it, thanks!


[CHANGE] Hover effect and fixed the template issues

Fixed the UI errors and added the hover effects to the product cards
Fixes #94

TO

Thank you for fixing this issue.


Added serach bar in CA an schloarship

Description
> If you are going merge my PR , Please label it as 'hacktoberfest-accepted',as it showing that your repo is not  participating in Hacktoberfest
Added search bar in CA program.html and scholarship.html.I Added a form tag inside li tag in nav bar.Css for search bar has been added to style.css with heading "searchbar"  and also added some change in media queries related only to form tag/search bar.
Fixes # (9)
Type of change

[ ] 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 not work as expected)
[ ] This change requires a documentation update

Checklist:

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

Please share the screenshot or screen recording of your work output for better review

Please share the screenshot or screen recording of your work output for better review

I have sent you a video recording on your instagram handle.My ig handle is choudhary_avishek.

@avishekchoudhary it is not looking same on website https://scodein.tech/Scodex/CA%20program.html as you shared me on insta

@avishekchoudhary it is not looking same on website https://scodein.tech/Scodex/CA program.html as you shared me on insta

It's working fine.I think this website is not mobile responsive.Here I'm sending you a screenshot how it looks on my mobile phone.

Ok Team will check it till then i am removing the labels to make it safe after reviewing the issue we will again label it with hacktoberfest

Ok Team will check it till then i am removing the labels to make it safe after reviewing the issue we will again label it with hacktoberfest

Could you please assure some certain time for reviewing process.As it would be very helpul for me .

within 24 hours we will review it

within 24 hours we will review it

Ok.

Any updates regarding reviewing process.

Sorry your PR is not accepted towards Hacktoberfest

Sorry your PR is not accepted towards Hacktoberfest

Ok


German translation

Hey, i saw the russian translation issue and would like to do it for the language german.
By reading the issue https://github.com/LorisYounger/VPet/issues/271
i am not fully sure about what is needed.
Can you provide all texts, needed to be translated for steam, so the issue can be resolved without much enquire? :)

Translate files in mod\0000_core\lang
you can use mod maker or Manually creating MODs
The Core or other  translation mod increative workshops should give you a reference.
Here you can also download translation tools (demo) specifically written for translation.
https://github.com/LorisYounger/LinePutScript.Localization.WPF
for german. https://learn.microsoft.com/en-us/openspecs/windows_protocols/ms-lcid/a9eac961-e77d-41a6-90a5-ce1a8b0cdb9c
you can create a folder called ge in  langfolder for german truanslate

Also, in github.  the game itself will not add languages other than English, Simplified Chinese, and Traditional Chinese
Because I am unable to confirm the text content and there are currently many versions of translation documents in the workshop, I cannot choose to make one of them an official translator.
You can add [Human Translation] in title that Let everyone know the advantages of your translation
You can also post it in the discussion Let everyone know that there is a French translation available
I will also release a special announcement in the future to inform everyone of the existing community translation content and location
Thank you for support of VPET project


badge links unnecessarily redirect to an HTTP endpoint

Fetching badge links (such as e.g. https://docs.rs/py-comp/badge.svg) requires going through two redirects:
$ wget https://docs.rs/py-comp/badge.svg
--2019-06-03 23:14:30--  https://docs.rs/py-comp/badge.svg
Resolving docs.rs (docs.rs)... <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, ...
Connecting to docs.rs (docs.rs)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 302 Moved Temporarily
Location: http://docs.rs/py-comp/badge.svg?version=0.1.3 [following]
--2019-06-03 23:14:36--  http://docs.rs/py-comp/badge.svg?version=0.1.3
Connecting to docs.rs (docs.rs)|<IP_ADDRESS>|:80... connected.
HTTP request sent, awaiting response... 301 Moved Permanently
Location: https://docs.rs/py-comp/badge.svg?version=0.1.3 [following]
--2019-06-03 23:14:37--  https://docs.rs/py-comp/badge.svg?version=0.1.3
Connecting to docs.rs (docs.rs)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 884 [image/svg+xml]
Saving to: ‘badge.svg’

In trying to fetch the badge, three HTTP calls are made:

https://docs.rs/py-comp/badge.svg --> 302 Moved Temporarily
http://docs.rs/py-comp/badge.svg?version=0.1.3 --> 301 Moved Permanently
https://docs.rs/py-comp/badge.svg?version=0.1.3 --> 200 OK

Besides the fact that this is unnecessary (because the first endpoint can just return the badge for the latest version by default), it messes with downstream uses.
Specifically, VSCode don't follow HTTP links when trying to fetch badges for display in markdown previews when "strict" security is enabled. This means that it stops following links at (2) and refuses to display the badge.
See https://github.com/microsoft/vscode/issues/73531 for reference.

For that specific request, the redirection answer is built using req.url.scheme(): https://github.com/rust-lang/docs.rs/blob/70e5a44a423be65fd1e45505051f7e277b65a1fc/src/web/rustdoc.rs#L300
But this seems to be a general issue with redirects, maybe caused by some tls termination proxy in front of the actual docs.rs instance, you can observe the same behaviour if you monitor the redictions for the crate version resolution as well, e.g. https://docs.rs/image/:
$ wget https://docs.rs/image
--2019-06-04 11:02:28--  https://docs.rs/image
Resolving docs.rs (docs.rs)... <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, ...
Connecting to docs.rs (docs.rs)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 302 Moved Temporarily
Location: http://docs.rs/image/0.21.1/image/ [following]
--2019-06-04 11:02:29--  http://docs.rs/image/0.21.1/image/
Connecting to docs.rs (docs.rs)|<IP_ADDRESS>|:80... connected.
HTTP request sent, awaiting response... 301 Moved Permanently
Location: https://docs.rs/image/0.21.1/image/ [following]
--2019-06-04 11:02:29--  https://docs.rs/image/0.21.1/image/
Connecting to docs.rs (docs.rs)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 44834 (44K) [text/html]

Also affected: Feeling lucky, crate details. This is also a security concern in so far as those plaintext redirections can be intercepted an arbitrarily modified and also leak the requested package url on the network.

Potential fix would be to set and parse the Forwarded header in whatever terminates the https connection and use the encoded original protocol instead of the scheme if available.

@HeroicKatora thanks so much for investigating this! I'll handle the server configuration.


Task3-8

Task 3-8

@Siri82
Task -3
Code verified. change Default activity  names and give annotations

Task -4
Code verified.
Task -5
Code verified.
Task -6
Code verified. change Default activity  names and give annotations

Task -7
Code verified. change Default activity  names and give annotations

Task -8
Code verified


btrfs corruption issues (?)

In the last few days I've been running into a bunch of btrfs corruption issues on my Macbook M2 Air. I initially suspected a single fluke, but it got worse.
Yesterday I entirely re-created the filesystem (luks with --allow-discards), then mkfs.btrfs with default params, and again got btrfs errors.
It seems I can rule out the internal SSD internal, as the same issues also happens on a (somewhat reliable and fast) external SSD (formatted with LUKS and btrfs).
Opening filesystem to check...
Checking filesystem on /dev/mapper/usb
UUID: a4d7d051-44ee-4512-bcfd-3b634526b02a
[1/7] checking root items
[2/7] checking extents
[3/7] checking free space tree
[4/7] checking fs roots
[5/7] checking csums against data
mirror 1 bytenr<PHONE_NUMBER>6 csum 0xc296d77c expected csum 0xe5e91fb3
ERROR: errors found in csum tree
[6/7] checking root refs
[7/7] checking quota groups skipped (not enabled on this FS)
found<PHONE_NUMBER>8 bytes used, error(s) found
total csum bytes: 40380008
total tree bytes: 685703168
total fs tree bytes: 570392576
total extent tree bytes: 63324160
btree space waste bytes: 119804959
file data blocks allocated:<PHONE_NUMBER>0
 referenced<PHONE_NUMBER>2

This was after copying my /nix/store from the host to /mnt, and unmounting.
dmesg of the host:
    6.192083] BTRFS: device label root devid 1 transid 41 /dev/disk/by-label/root scanned by mount (530)
[    6.192241] BTRFS info (device dm-0): first mount of filesystem 5eaac3f0-833c-4f0f-b6f5-df3eb94e4327
[    6.192248] BTRFS info (device dm-0): using crc32c (crc32c-generic) checksum algorithm
[    6.192252] BTRFS info (device dm-0): forcing free space tree for sector size 4096 with page size 16384
[    6.192254] BTRFS info (device dm-0): using free-space-tree
[    6.192254] BTRFS warning (device dm-0): read-write for sector size 4096 with page size 16384 is experimental
[    6.200521] BTRFS info (device dm-0): checking UUID tree
[    6.740028] systemd-journald[807]: Creating journal file /var/log/journal/bbe02739e577495c999bfebef448138d/system.journal on a btrfs file system, and copy-on-write is enabled. This is likely to slow down journal access substantially, please consider turning off the copy-on-write file attribute on the journal directory, using chattr +C.
[    6.817857] BTRFS info: devid 1 device path /dev/disk/by-label/root changed to /dev/dm-0 scanned by (udev-worker) (936)
[ 7896.890101] BTRFS: device fsid a4d7d051-44ee-4512-bcfd-3b634526b02a devid 1 transid 6 /dev/mapper/usb scanned by mount (17224)
[ 7896.890774] BTRFS info (device dm-2): first mount of filesystem a4d7d051-44ee-4512-bcfd-3b634526b02a
[ 7896.890802] BTRFS info (device dm-2): using crc32c (crc32c-generic) checksum algorithm
[ 7896.890811] BTRFS info (device dm-2): forcing free space tree for sector size 4096 with page size 16384
[ 7896.890816] BTRFS info (device dm-2): using free-space-tree
[ 7896.890819] BTRFS warning (device dm-2): read-write for sector size 4096 with page size 16384 is experimental
[ 7896.892854] BTRFS info (device dm-2): checking UUID tree
[ 7896.893323] BTRFS info (device dm-2): last unmount of filesystem a4d7d051-44ee-4512-bcfd-3b634526b02a
[ 7902.339423] BTRFS: device fsid a4d7d051-44ee-4512-bcfd-3b634526b02a devid 1 transid 8 /dev/mapper/usb scanned by mount (17294)
[ 7902.340267] BTRFS info (device dm-2): first mount of filesystem a4d7d051-44ee-4512-bcfd-3b634526b02a
[ 7902.340294] BTRFS info (device dm-2): using crc32c (crc32c-generic) checksum algorithm
[ 7902.340303] BTRFS info (device dm-2): forcing free space tree for sector size 4096 with page size 16384
[ 7902.340308] BTRFS info (device dm-2): using free-space-tree
[ 7902.340312] BTRFS warning (device dm-2): read-write for sector size 4096 with page size 16384 is experimental
[ 8089.623727] BTRFS warning (device dm-0): csum failed root 5 ino 978709 off 1253376 csum 0x81dd87df expected csum 0xe77556aa mirror 1
[ 8089.623738] BTRFS error (device dm-0): bdev /dev/dm-0 errs: wr 0, rd 0, flush 0, corrupt 1, gen 0
[ 8320.442776] BTRFS warning (device dm-0): csum failed root 5 ino 807754 off 49152 csum 0x579bee4a expected csum 0x8a36f543 mirror 1
[ 8320.442794] BTRFS error (device dm-0): bdev /dev/dm-0 errs: wr 0, rd 0, flush 0, corrupt 2, gen 0
[ 8320.450734] BTRFS warning (device dm-0): csum failed root 5 ino 807754 off 94208 csum 0x857c0a3b expected csum 0xc3bf7a9c mirror 1
[ 8320.450739] BTRFS error (device dm-0): bdev /dev/dm-0 errs: wr 0, rd 0, flush 0, corrupt 3, gen 0
[ 8320.462502] BTRFS warning (device dm-0): csum failed root 5 ino 807754 off 49152 csum 0x579bee4a expected csum 0x8a36f543 mirror 1
[ 8320.462507] BTRFS error (device dm-0): bdev /dev/dm-0 errs: wr 0, rd 0, flush 0, corrupt 4, gen 0
[ 8320.470859] BTRFS warning (device dm-0): csum failed root 5 ino 807754 off 49152 csum 0x579bee4a expected csum 0x8a36f543 mirror 1
[ 8320.470866] BTRFS error (device dm-0): bdev /dev/dm-0 errs: wr 0, rd 0, flush 0, corrupt 5, gen 0
[ 8336.179155] BTRFS warning (device dm-0): checksum verify failed on logical<PHONE_NUMBER> mirror 1 wanted 0x9e547725 found 0x87cd70c6 level 0
[ 8336.179673] BTRFS info (device dm-0): read error corrected: ino 0 off<PHONE_NUMBER> (dev /dev/dm-0 sector 2047040)
[ 8651.859183] BTRFS info (device dm-2): last unmount of filesystem a4d7d051-44ee-4512-bcfd-3b634526b02a

Similar issue here: https://github.com/tpwrules/nixos-apple-silicon/issues/196
Should we roll back the kernel? If you have a relatively easy and safe way to replicate, can you try a few past kernel versions?

I'm currently trying ZFS with its own crypto layer, so if it's really dm-crypt (only) I shouldn't be affected anymore.
If that's stable, and everything is set up again, I can do the smoketest with the external drive on various kernel versions and see if there's a pattern.

Might be coincidental, but I hit some bad ext4 corruption yesterday on my M1, also using dm-crypt.
It was rebuilding the kernel+mesa, and the compile started failing with gcc complaining one of the kernel .c files was filled with binary content.
I noticed this in dmesg:
May 16 02:08:23 ZAIR kernel: EXT4-fs error (device dm-1): ext4_lookup:1855: inode #11943410: comm nix-daemon: iget: bad extra_isize 762 (inode size 256)

Rebooted to this, and ran a repair. I don't have the full fsck log, but it was large.
May 16 02:28:42 ZAIR stage-1-init: [Wed May 15 16:26:46 UTC 2024] Passphrase for /dev/disk/by-uuid/2186c706-f18d-4be1-b1e6-cdfb0260843d:
May 16 02:28:42 ZAIR stage-1-init: [Wed May 15 16:26:48 UTC 2024] Verifying passphrase for /dev/disk/by-uuid/2186c706-f18d-4be1-b1e6-cdfb0260843d... - success
May 16 02:28:42 ZAIR stage-1-init: [Wed May 15 16:26:48 UTC 2024] starting device mapper and LVM...
May 16 02:28:42 ZAIR stage-1-init: [Wed May 15 16:26:48 UTC 2024] 2 logical volume(s) in volume group "vg" now active
May 16 02:28:42 ZAIR stage-1-init: [Wed May 15 16:26:48 UTC 2024] checking /dev/disk/by-uuid/07e2eada-bf28-4f4e-b0f0-1fbc05953b2a...
May 16 02:28:42 ZAIR stage-1-init: [Wed May 15 16:26:48 UTC 2024] fsck (busybox 1.36.1)
May 16 02:28:42 ZAIR stage-1-init: [Wed May 15 16:26:48 UTC 2024] [fsck.ext4 (1) -- /mnt-root/] fsck.ext4 -a /dev/disk/by-uuid/07e2eada-bf28-4f4e-b0f0-1fbc05953b2a
May 16 02:28:42 ZAIR stage-1-init: [Wed May 15 16:26:48 UTC 2024] root contains a file system with errors, check forced.
May 16 02:28:42 ZAIR stage-1-init: [Wed May 15 16:26:51 UTC 2024] root: Inode 9608098 seems to contain garbage.
May 16 02:28:42 ZAIR stage-1-init: [Wed May 15 16:26:51 UTC 2024] root: UNEXPECTED INCONSISTENCY; RUN fsck MANUALLY.
May 16 02:28:42 ZAIR stage-1-init: [Wed May 15 16:26:51 UTC 2024] (i.e., without -a or -p options)
May 16 02:28:42 ZAIR stage-1-init: [Wed May 15 16:26:51 UTC 2024] /dev/disk/by-uuid/07e2eada-bf28-4f4e-b0f0-1fbc05953b2a has unrepaired errors, please fix them manually.
May 16 02:28:42 ZAIR stage-1-init: [Wed May 15 16:26:51 UTC 2024] An error occurred in stage 1 of the boot process, which must mount the
May 16 02:28:42 ZAIR stage-1-init: [Wed May 15 16:26:51 UTC 2024] root filesystem on `/mnt-root' and then start stage 2.  Press one
May 16 02:28:42 ZAIR stage-1-init: [Wed May 15 16:26:51 UTC 2024] of the following keys:
May 16 02:28:42 ZAIR stage-1-init: [Wed May 15 16:26:51 UTC 2024] i) to launch an interactive shell
May 16 02:28:42 ZAIR stage-1-init: [Wed May 15 16:26:51 UTC 2024] f) to start an interactive shell having pid 1 (needed if you want to
May 16 02:28:42 ZAIR stage-1-init: [Wed May 15 16:26:51 UTC 2024] start stage 2's init manually)
May 16 02:28:42 ZAIR stage-1-init: [Wed May 15 16:26:51 UTC 2024] r) to reboot immediately
May 16 02:28:42 ZAIR stage-1-init: [Wed May 15 16:26:51 UTC 2024] *) to ignore the error and continue
May 16 02:28:42 ZAIR stage-1-init: [Wed May 15 16:27:10 UTC 2024] Starting interactive shell...
May 16 02:28:42 ZAIR stage-1-init: [Wed May 15 16:28:41 UTC 2024] mounting /dev/disk/by-uuid/07e2eada-bf28-4f4e-b0f0-1fbc05953b2a on /...

I've had some recent corruption issues as well rendering my primary partition unbootable; I don't have a log to provide, but I'm using an ext4 partition on LUKS. I'm not sure what it could be exactly.

I'm currently trying ZFS with its own crypto layer, so if it's really dm-crypt (only) I shouldn't be affected anymore.
If that's stable, and everything is set up again, I can do the smoketest with the external drive on various kernel versions and see if there's a pattern.

I tried reproducing the issue from there, by copying my /nix/store to the external drive with a btrfs inside a luks volume.
I could not immediately reproduce it anymore, though that's a kernel with much more options enabled, essentially a distro kernel built with the asahi kernel sources (https://github.com/yu-re-ka/nixos-m1/tree/minimize-patches).

I can reproduce it with a vanilla linux v6.8.9 on a M1 Pro Macbook (j316). That opens up the possibility to bisect it.
The reproducer I am using is tio's examples/basic-verify.tio on a freshly created dm-crypt volume, which seems to trigger the bug reliably.

Can you post a bit more details on how to reproduce? I don't know tio and a quick search didn't turn up anything helpful in particular.

I double checked with the proper asahi kernel. It is fixed for me with the following commits reverted:

aefbab8e77eb ("arm64: fpsimd: Preserve/restore kernel mode NEON at context switch") (for the context), and
2632e2521769 ("arm64: fpsimd: Implement lazy restore for kernel mode FPSIMD") (as found by git bisect)

Do you want me to forward this report upstream? If yes, two short questions:
Did you do the bisection with a vanilla kernel? Is vanilla 6.9 still showing the same problem? And does a revert help there, too (I assume all of that is the case, but sometimes it's better to be sure)
[side note: I'm the Linux kernel's regression tracker; somebody pointed me here; normally I do not comment on downstream bug trackers, but I make an exception due to the data corruption aspect]

ahh, I see, somebody reported it upstream already<EMAIL_ADDRESS>great, thx!

That was me reporting it, but thanks for the offer.

I double checked with the proper asahi kernel. It is fixed for me with the following commits reverted:
* aefbab8e77eb ("arm64: fpsimd: Preserve/restore kernel mode NEON at context switch") (for the context), and

* 2632e2521769 ("arm64: fpsimd: Implement lazy restore for kernel mode FPSIMD") (as found by `git bisect`)

Hej @mixi,
both commits need to be reverted on top of asahi-6.8.9-5 / v6.8? I'm a little confused since the linux-arm-kernel mail only mentions 2632e2521769 which reverts and builds cleanly.

You are right to be confused. Reverting 2632e2521769 alone is enough, and that is also the commit bisect pointed me to yesterday.
Apparently I reverted one commit too many by accident and guessed I did it for context reasons when writing the comment afterwards.

@tpwrules asahi-6.8.9-6 containing only the revert pushed to AsahiLinux/linux

Latest release contains the revert. Please close the issue if you are satisfied with the fix.

This might have fixed the issue, but the iso is now lacking btrfs support.
I'm pretty sure it had this previously? or am I mistaken?
(I too was bit by this bug, I was unable to rescue my system due to corrupted btrfs root partition)

Bad news: aefbab8e77eb ("arm64: fpsimd: Preserve/restore kernel mode NEON at context switch") also needs to be reverted. See https://lore.kernel.org/all/Zkw9kK0sXIgfqd01@shadowice/ for details, and a new reproducer that found the commit (the old one reproducibly sees the commit as good).
@jannau:

Apparently I reverted one commit too many by accident and guessed I did it for context reasons when writing the comment afterwards.

Correction: Apparently I reverted the right commit for the wrong reasons back then.

Please try this fix, and report on the thread whether or not it works for you:
<EMAIL_ADDRESS>

Just to make sure, is this a fix to be applied on top of any reverts (and if so, which), or an attempt to fix without reverting anything else?

The latter.

With asahi-6.8.9-7 (essentially reverting the other revert(s) and applying that patch) I don't seem to be running into these issue anymore, a lot of other folks on the ML thread also reported the same, and it already got applied to arm64 (for-next/core).
I guess what's left here is bumping linux-asahi in here again, then this can be closed.

PR up at https://github.com/tpwrules/nixos-apple-silicon/pull/202

Thanks @flokli ! I used to reliably get Firefox to crash by running nix-store --verify --check-contents in the background.  With #202 that's no longer happening.

https://github.com/tpwrules/nixos-apple-silicon/pull/202 has been merged (bumping the kernel to asahi-6.8.9-7, including a cherrypick), and a new release of nixos-apple-silicon been created, so we can close the issue here.
On the upstream kernel side, I however noticed the fix only landed in the master branch so far - meaning other aarch64 machines running the mainline kernel might still run into this corruption.
@knurd is there anything else left to be done so this gets cherrypicked to linux-6.9.y, so it'll land in v6.9.2?

@knurd is there anything else left to be done so this gets cherrypicked to linux-6.9.y, so it'll land in v6.9.2?

That's likely too late, as 6.9.2 is in its -rc phase already – and usually Greg does not add any patches at that point aiui. You could ask though. But it likely should go into 6.9.3 dues to the "CC: stable..." tag in the commit.

(trying here, as I don't have that ML subscribed): Hey @gregkh, any chance "arm64/fpsimd: Avoid erroneous elide of user state reload" could still end up in 6.9.2, due to its data corruption nature?

Please send stable requests to<EMAIL_ADDRESS>we can't take stuff from random github repos for obvious reasons.

The commit I linked had a Cc: stable in the message. That's sufficient?

The commit I linked had a Cc: stable in the message. That's sufficient?

Up to Greg, but I'd say it's in the everyone's best interest if you write a quick mail to the list (like with most Linux kernel lists, you don't have to be subscribed!) with Greg CCed (side note: you might ask for the patch to be included in 6.8.y, too) – that among others is also important for the paper trail in case the question "who asked for this to be included" comes up later.

@knurd Sent out an email to stable@, both you and greg are in CC.


Set develop branch as default branch for repo

Currently, JetNews is more or less the source of truth on using jetpack compose. When you first get to this repository you land on master, which is currently very out of date (in the compose world). It would be far more useful to folks for develop to be the default branch for viewing to reduce confusion.

For the record: i only realized the develop branch existed thanks to nick's reply to another tweet here https://twitter.com/crafty/status/1232668784680427520

Ideally master wouldn't be so out of date but we had some issues with dev04. We should be updating to dev05 shortly.
Would being up to date with the latest devXX release plus something in the README calling out the develop branch help?

Yes, I think that would work as well. However, with develop working off of SNAPSHOT builds, could this cause delays in keeping master up to date with devXX builds?

The sample is now dev06.


Beca_BRT-100 working with ZHA

Note that mine is Moes but the zigbee device id reported is this one…

That's fine, Moes is just rebranding the same device. Thanks for the contribution


Cannot get npm test to run

I was trying to get the tests to run so I could contribute by tracking down some other issue I'm having, but I can't actually get the tests to pass, even though Travis says the tests pass.
This is my log:
$  node --version
v5.7.0
$  npm --version
3.7.5
$ git clone https://github.com/jonathantneal/precss.git
...
cd precss
npm install
...
npm test

When I run this, it says two tests fail (expected CSS output is different than actual CSS output), but also gives this error:
npm ERR! Darwin 14.5.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "run" "fixtures"
npm ERR! node v5.7.0
npm ERR! npm  v3.6.0
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>fixtures: `tape test/*.js | tap-spec`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>fixtures script 'tape test/*.js | tap-spec'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the precss package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     tape test/*.js | tap-spec

It also fails if I run tape test/*.js manually.
Any pointers or suggestions?

Should now be running


NWB file: adding group/naming information for LFPs/clusters

Was wondering if you could add electrode and unit names when running load_file() function with an NWB file.
I've added two screenshots, the first shows a DataFrame for the LFP signals, the second shows the group-name and labels of each LFP signal.
Thank you!

Hi John, I am not sure I understand. Do you want labelled columns when loading the LFP?

Hey Guillaume, yes exactly! I can reference back to the original file by like loading the raw nwbfile and aligning the nap ids with nwbfile ids, but I was wondering if I can just have pynapple group that information for me. So in that first screenshot, is there a way for index 0 to have an additional column or replacement column with 'CSC1', which is further grouped into 'Tetrode1' as shown in the second screenshot?
I'm working on a small package for the lab that uses pynapple and am looking for ways for users to index out information by entering the raw file name 'CSC1' or by entering grouped names like 'Tetrode1'.
I hope that my description is more clear this time

Hi John, I think it's possible. If the nwb file has a label columns, it can be used as a column name for the TsdFrame.
Also please note that I am about to bump the version 0.4 (next week) which relies on the numpy array container approach. This is different from inheriting pandas. You can install the pre-release version by doing pip install --pre pynapple to test it and make sure your code still works. You can also check this notebook to get an idea of how it works.

Okay I’ll check it out next week!
On Tue, Oct 17, 2023 at 4:06 PM Guillaume Viejo @.***>
wrote:

Hi John, I think it's possible. If the nwb file has a label columns, it
can be used as a column name for the TsdFrame.
Also please note that I am about to bump the version 0.4 (next week) which
relies on the numpy array container approach. This is different from
inheriting pandas. You can install the pre-release version by doing pip
install --pre pynapple to test it and make sure your code still works.
You can also check this notebook
https://pynapple-org.github.io/pynapple/generated/gallery/tutorial_pynapple_numpy/
to get an idea of how it works.
—
Reply to this email directly, view it on GitHub
https://github.com/pynapple-org/pynapple/issues/187#issuecomment-1767089159,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AK7U57SNHZH7C3YRWINFSLDX73QL3AVCNFSM6AAAAAA55X7F52VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRXGA4DSMJVHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Ok fixed. It's pushed to the main branch. It should appear in the next version soon.


Add 'Table of Contents' in documentation

Add 'Table of Contents' in README.md.

@bhattabhi013 README.md file is short already. But I still what to know what things will be there under table of contents.

Something like this.

@bhattabhi013 Go ahead

Fixed by #36


[BUG] Authentication error when using AppendBlobClient and storage access keys

Library name and version
Azure.Storage.Blobs Version 12.16.0
Describe the bug
I get an authentication error when I use the access keys' connection string and perform an action with the AppendBlobClient (delete an existing blob, create a new blob...)
Server failed to authenticate the request. Please refer to the information in the www-authenticate header.

With that access key, I'm able to create/delete the container, and I'm even able to delete the blob using the BlobClient class.
If instead of using the access key, I generate and use a sas connection string, I don't get any error
Expected behavior
I would expect that the main connection string from the access key has full access to the storage account
Actual behavior
I get an authentication error
Reproduction Steps
[Test]
public async Task Working()
{
    var blobServiceClient = new BlobServiceClient("BlobEndpoint=https://StorageAccount.blob.core.windows.net/;SharedAccessSignature=SasToken");
    var blobContainer = blobServiceClient.GetBlobContainerClient("test-container");
    await blobContainer.CreateIfNotExistsAsync();
    var blobReference = blobContainer.GetBlobClient("TestBlob");
    var appendBlobClient = new AppendBlobClient(blobReference.Uri);
    await appendBlobClient.DeleteIfExistsAsync();
    await appendBlobClient.CreateAsync();

}

[Test]
public async Task NotWorking()
{
    var blobServiceClient = new BlobServiceClient("DefaultEndpointsProtocol=https;AccountName=StorageAccount;AccountKey=AccountKey;EndpointSuffix=core.windows.net");
    var blobContainer = blobServiceClient.GetBlobContainerClient("test-container");
    await blobContainer.CreateIfNotExistsAsync();
    var blobReference = blobContainer.GetBlobClient("TestBlob");
    var appendBlobClient = new AppendBlobClient(blobReference.Uri);
    await appendBlobClient.DeleteIfExistsAsync();
    await appendBlobClient.CreateAsync();
}

The account name, sas token and account key has been removed from the test
Environment
Windows 10, Azure Function 4 with .Net 7
VS Version 17.5.1

@pilarodriguez Thanks for reaching out to us and reporting this issue. Could you please share the x-ms-requestid (Guid ending with 6 zero's) header from the error message, so that I can look at the backend logs for the cause of the issue.
Awaiting your reply.

x-ms-request-id: 0b35fd57-c01e-0050-02e0-db51aa000000
Thanks for the response

@pilarodriguez While I looked at the backend logs for the above failing request, I could see the reason for status 401 is because the request somehow came as Anonymous request while performing the delete operation for the blob https://ba*****ple.blob.core.windows.net/test-container/TestBlob .
Please Update the code as below to perform the delete operation:
    var blobServiceClient = new BlobServiceClient("DefaultEndpointsProtocol=https;AccountName=StorageAccount;AccountKey=AccountKey;EndpointSuffix=core.windows.net");
    var blobContainer = blobServiceClient.GetBlobContainerClient("test-container");
    await blobContainer.CreateIfNotExistsAsync();
    var blobReference = blobContainer.GetBlobClient("TestBlob");

    // Create the AppendBlobClient using the blobContainer and blob name
    var appendBlobClient = blobContainer.GetAppendBlobClient("TestBlob");

    await appendBlobClient.DeleteIfExistsAsync();

If you want to create an append blob you can see the same code here.
Hope this helps.

It works. Thanks a lot for your help

@pilarodriguez


Fix temp dir driver tests to work on the bots.

Example test failure here.
At a cursory glance the issue seems to have to do with SDK discovery.

https://codereview.chromium.org/2976493002/ removes tests that were commented and reference this issue. Recover these tests once the issue is fixed.

@scheglov : I'm tempted to close this (as I'm not tracking it and I don't know how relevant it remains currently).  Thoughts?

SG


Open Collective

Hello,

Can you put Open Collective Badge on top
of Spyder documentation page /github readme ?
Easier for people to contribute financially.

Is it way to finance by open collective some new features dev in Spyder ?

Is there a way to see new feature requests ?

Can you put Open Collective Badge on top

Thanks for the suggestion. We'll add the Github "Sponsor" button that redirects to Open collective.

Is it way to finance by open collective some new features dev in Spyder ?

No, sorry. We use the money as we consider necessary.

Hello,

Can you create a list of
best voting features ? (ie most wanted)

People can also contribute
by Open Collective to those features ?
It makes easier to hire “students” or people.
and will attract much more donator.

By controbuting, it does not mean that it will get done right, but increase attractivity of donators....
What do you think ?

On Sep 2, 2019, at 16:09, Carlos Cordoba<EMAIL_ADDRESS>wrote:
Can you put Open Collective Badge on top
Thanks for the suggestion. We'll add the Github "Sponsor" button that redirects to Open collective.
Is it way to finance by open collective some new features dev in Spyder ?
No, sorry. We use the money as we consider necessary.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@arita37 you can fund any feature you find important with https://gitcoin.co

Thanks
How to link your github features with gitcoin.
Thanks, Regards

On Sep 4, 2019, at 17:38, anatoly techtonik<EMAIL_ADDRESS>wrote:
@arita37 you can fund any feature you find important with https://gitcoin.co
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@arita37 look here - https://gitcoin.co/how/funder- you don't need to create an issue on GitHub - it is possible to select existing one.

Thanks.
Does it appear on the spyder github issues too ?
Regards.

On Sep 4, 2019, at 18:23, anatoly techtonik<EMAIL_ADDRESS>wrote:
@arita37 look here - https://gitcoin.co/how/funder- you don't need to create an issue on GitHub - it is possible to select existing one.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@arita37 gitcoin.co is a parallel bounty service that is independent from this project, and attracts people from outside.

@steff456, please take care of this one so we can display the new Github "Support" button on top of all pages in this repo.


Fix bugs for dealing with processed objects with only 1 cell

While processing data for the Portal, I found an object that only has 1 cell in the processed object so it tested out some of the code we added to handle that scenario. There were a few issues that I had to fix to get this to work and then I also made a design change that we will need to decide on if it's the right move or not.

If any object only has 1 cell then it doesn't have PCA or clusters. We have a check for PCA/UMAP when generating the reports, but not for clusters. Here I added a has_clusters variable to make sure no plots that use clusters are created.
There was also a small error in the add_celltypes_to_sce.R where we were still attempting to add in cell type annotations if the predictions file was empty. So I adjusted the logic there to account for the missing file properly.

The other thing I did was choose to not create the supplemental report if there's only 1 cell in the object. The report was just some text with one table showing the assignment of that single cell in SingleR, but no other plots. This doesn't seem super informative to me, and we would still have the table in the main QC report. What do others think of this choice? This should not affect very many samples, but we would want to make sure we check how this affects zipping up the files for the Portal.
@davidsmejia Would you guys have to make changes when zipping up the files for the Portal if a sample was missing the celltype-report.html file? I assume you have checks to make sure the files are all there, so would we be able to make the existence of this file optional?

@jashapiro are we okay with the decision to not create a supplemental report? I just want to be sure before I merge this and then update the current release.

@jashapiro are we okay with the decision to not create a supplemental report? I just want to be sure before I merge this and then update the current release.

Yes, I am fine with that decision!

Yes, I am fine with that decision!

Can you approve please? 😄


Updated manual.md

Added some info blocks to help the reader install the operator without issues.
What type of PR is this?

Uncomment only one  /kind line, and delete the rest.
For example, > /kind bug would simply become: /kind bug

/kind bug
/kind cleanup
/kind failing-test
/kind enhancement
/kind documentation
/kind code-refactoring

What does this PR do / why we need it:
This is a documentation update to help reader perform manual install of operator without issues.
Have you updated the necessary documentation?

[ ] Documentation update is required by this PR.
[ ] Documentation has been updated.

Which issue(s) this PR fixes:
This is a documentation update.
Fixes #?
How to test changes / Special notes to the reviewer:
I think notes are pretty simple, just guiding the reader to the right direction.

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

Hi @iam-veeramalla,
Would you review this and give feedback?

Hi @iam-veeramalla,
Would you review this and give feedback?

Sure @samanamp


getUsedRange throws error, not return "undefined" as in docs

The problem is with the function getUsedRange of a Range.

getUsedRange(valuesOnly)
Returns the used range of the given range object. If there are no used cells within the range, then this method returns undefined.

When there is an empty range calling that function, it does not return undefined, as mentioned in the documentation, but throws an "RichApi.Error":
name: "RichApi.Error"
code: "ItemNotFound"
traceMessages: Array[0]
innerError: null
debugInfo: Object
code: "ItemNotFound"
message: "The requested resource doesn't exist."
toString: function toString()
errorLocation: "Range.getUsedRange"
statement: "var usedRange1 = range3.getUsedRange(...);"

PS. I remark another doc page on the same subject this one is correct
https://learn.microsoft.com/en-us/javascript/api/excel/excel.range?view=excel-js-preview

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

ID: dcd3b516-e875-6dcf-6c57-81c7999548db
Version Independent ID: fc09db9d-2b47-61a9-0dc6-fbd7a955cdc5
Content: ExcelScript.Range interface - Office Scripts
Content Source: docs/docs-ref-autogen/excel/excelscript/excelscript.range.yml
Product: excel
Technology: scripts
GitHub Login: @o365devx
Microsoft Alias: o365devxgetUsedRange(valuesOnly)
Returns the used range of the given range object. If there are no used cells within the range, then this method returns undefined.

Thanks for reporting this issue @sdudnic.
I'll update the Office Scripts documentation. Let me also tag @wandyezj to confirm this is the correct behavior from the product standpoint.


[README] Support HTML in Markdown

Is your feature request related to a problem? Please describe.
GitHub renders  and  E.g.
This is  text.
This is  text.
Edit by @joelverhagen: the following list of HTML tags should be considered:

 (this issue)
 (this issue and https://github.com/NuGet/NuGetGallery/issues/8913)
, especially with fixed width or height (https://github.com/NuGet/NuGetGallery/issues/8828)

 (https://github.com/NuGet/NuGetGallery/issues/8836, workaround: https://github.com/NuGet/NuGetGallery/issues/8836#issuecomment-985963667)

 with center (https://github.com/NuGet/NuGetGallery/issues/9186)
 (https://github.com/NuGet/NuGetGallery/issues/9186)

 with center (https://github.com/NuGet/NuGetGallery/issues/8828)

 and other sides (https://github.com/NuGet/NuGetGallery/issues/8828)

Describe the solution you'd like
I'd like this to be supported in the rendering of package readme's in the NuGet gallery.
Additional context
 and  are currently not rendered. E.g. from https://www.nuget.org/packages/SimpleExec/8.0.0-alpha.4 —

@adamralph, thanks for the proposal! Indeed, nuget.org doesn't support yet the full set of markdown feature GitHub does. If you are interested in contributing and sending a PR we will gladly work with you on merging and shipping it.

@skofman1 I won't be attempting a contribution here. I wouldn't know where to start.

@adamralph , sure thing. We will keep an eye up for upvotes and prioritize accordingly.

Use of HTML in Markdown - including  and  is extremely common. Here are 10 popular GitHub C# projects that use HTML in their READMEs:

GitHub project
Stars

microsoft/PowerToys
57582

dotnet-architecture/eShopOnContainers
18232

mxgmn/WaveFunctionCollapse
16662

NickeManarin/ScreenToGif
14981

dotnet/roslyn
14697

AvaloniaUI/Avalonia
11675

files-community/Files
11561

Unity-Technologies/ml-agents
11411

jellyfin/jellyfin
11319

huiyadanli/RevokeMsgPatcher
10640

More GitHub projects...

 Unfortunately Markdig does not let you configure an allowlist or blocklist for HTML elements. It seems our options are:

Contribute an allowlist feature to Markdig, or...
Enable all HTML elements but then use something like HtmlSanitizer to block unsafe elements

Hey,
Just want to raise my voice on this issue, as it is causing many of my packages to not render correctly on NuGet. For example: https://www.nuget.org/packages/Tomlyn/
On all my projects, I'm putting usually a html logo aligned on the right side:

Contribute an allowlist feature to Markdig, or...
Enable all HTML elements but then use something like HtmlSanitizer to block unsafe elements

The first option seems like the best path. If we go with the second option we'd definitely would want to get a security review from security experts.

As the main developer of markdig, I would suggest more the second approach. This is what I used for the service babelmark where I'm stripping all javascripts/events from the code with NUglify here. Can't vouch for the security, but HtmlSanitizer looks good.

Since the vast majority of README files are designed to render correctly in GitHub, it would seem sensible to replicate the HTML rendering in GitHub, in the NuGet gallery.

I just added logos to a few of my readmes only to realize after release that they don't show on nuget.org. github/cmark-gfm is the markdown library used by GitHub.

Please support  too (or ignore it by not displaying the HTML tag)

I disagree with @wiz0u, I'd rather see the ugly HTML (so people know something is there, even if it's not currently rendered properly) than to have it hidden away, which developers would find even more perplexing.
And put me on the list of people using HTML in Markdown which looks awful on NuGet.
On github.com

On nuget.org

I disagree with @wiz0u, I'd rather see the ugly HTML (so people know something is there, even if it's not currently rendered properly) than to have it hidden away, which developers would find even more perplexing.
And put me on the list of people using HTML in Markdown which looks awful on NuGet.
On github.com

On nuget.org

Thank you for your feedback. Please free feel to upvote this issue so that we could reprioritize.

@bradwilson Thank you for your feedback. Please free feel to upvote this issue so that we could reprioritize.

If you don't mind me asking, where do I upvote?

@bradwilson Thank you for your feedback. Please free feel to upvote this issue so that we could reprioritize.

If you don't mind me asking, is liking this issue considered an upvote or somewhere else?

Thumb up(👍 )on issue description on the top! currently, it has 20 👍

Currently this issue is 4th most upvoted, while being just 2 years old; other 3 issues are 5+ y/o with not much more votes. Can we hope to see this implemented in the near future?

Just thought I'd chime in that the image tag is not rendering for my Nuget package, despite the image host being on the allowed domain list.

I don't necessarily need to use html. Does Nuget support native image markdown?

@lyndaidaii As of today, this issue has the second most upvotes on this repository, so I think it's fair to say that this is a highly sought-after feature request. That being said, are there any plans to put this task on a roadmap? I am not feeling like anybody (from NuGet) is currently keeping an eye on that.

@StefanGreve, thank you for feedback. I will bring this up to team for planning see if we could prioritize work.

What other tags do people want to see supported or even features from Markdig supported here?
https://github.com/xoofx/markdig?tab=readme-ov-file#features


New config

Describe the pull request:

[ ] Bug fix
[x] Functional change
[ ] New feature
[ ] Code cleanup
[ ] Build system change
[ ] Documentation change
[ ] Language translation

Pull request long description:

Reworked the queues to work with the new setup from CRG.
The setup now has 2 distinct modes, federated and standalone.
Federated
Here we have 2 exchanges sda and to_cega, all messages published in the to_cega exchange are shoveled upstream.
The filesqueue is mirrored down from CRG and the only service interacting with that one is intercept.
There is a number of local shovels that moves messaged published to the sda exchange with the following routing keys; inbox, verified, completed and error to the to_cega exchange while prepending files. to the routing key.
Standalone
This one has no shovels or federation links and only one exchange sda.
Queues
The queues are identical  in both standalone and the federated setup.

archived, for when a file has been ingested
completed, a completed file, ready for download
error, all errors are sent here
files, only used in a federated setup, messaged from CRG are coming here.
inbox, for inbox operations
ingest, for messages that will start an ingestion process
mappings, for dataset mappings
stableIDs, for assigning stable IDs
verified, when a file has been verified

Changes made:

Related issues:

Additional information:

Release note:

Documentation change:

Mentions:

LGTM, this also looks like a breaking change for the python services and some follow-up changes are needed:

sda-helm updated with new queues BROKER_ROUTINGKEY for the sda-pipeline services are fixed at the moment

Yes, and there needs to be some changes to the sda-pipline code as well

sftp-inbox will need to be updated to use the to_cega queue as by default it uses cega and in deployment we need to have to set a special exchange for it different than the other services

That is the problem with hard coding stuff rather than relying on config

Note: if we can change the queues probably i would prefer accessionIDs instead of stableIDs to be more consistent with terminology .. but that is just nitpicking

wilco.


ok to use this libaray for integers

Hi Rob, I've been searching for a simple circular buffer that can calculate averages. My project is for a Sensirion SCD-41 carbon dioxide (CO2) sensor which returns uint16_t values. This library looks ideal. Bonus is all the other min / max functions.
Do you see any problem with me converting these to float so I can use this libary?
Thanks!

I've had a better look at the examples now, and I can see you add ints without even type casting to float, so I think that answers my question. I'll leave this issue open in case you have any comments or things I should look out for.

You can just add the data, it will be no problem.
But also have a look at the runningMedian libraty as that one will handle outliers differently.
RunningAverage includes outliers in the  average, runningMedian won't unless there are so many that it fills up the buffer.

Thanks I've got it running beautifully on an ESP32. I've got 3 circular buffers, one size [12] for history of last minute of raw samples at 5 sec/sample, another size [60] for an hours worth of average of the last minute, and another size [24] for days worth of average of the last hour. This library has saved me so much time, literally coded this in just a few lines of code.
I had a look at runningMedian, but this one serves my purpose ok. I notice you have a few other arduino repositories too 😲

Good to hear you got it all working. Yes been working with Arduino since 2009 or 10 and wrote quite some libs. If you are missing functionality let me know and I will look if I can add it.
Btw which sensor do you use?


[Platforms] KubeCon EU 2024 Recap Blog Post

Contribution Description
The idea was to recap everything that happened at KubeCon EU 2024 and make this post as the go-to post for all things platform engineering that happened at KubeCon. I've tried to cover all the aspects of the event.
Draft: https://docs.google.com/document/d/1_oc5Xy7gory3jUTku090K7JrjOaVlcQWHT4m3bc-rVc/edit?usp=sharing
Open to feedback, suggestions and comments. If there's anything that I've missed (For eg: photos and updates from the TAG-App Delivery booth) please suggest them in the document and I'll accept them.
Thanks in advance :)
PS: I know this is delayed but I was traveling and then was caught up with work.
Related Working Group (WG)
Platforms
Contribution type
Blog (Default)
Why TAG App Delivery?
Since the updates are closely related to the activities that we do as part of the Platforms Working Group, the blog post fits well here.
Related projects/technologies
No response
Affiliation disclosure
No response
Additional collaborators
No response
Additional information
No response

LGTM :)

This is great @techmaharaj , thanks for taking a stab at this! I can see a lot of great work happening on the google doc, what kind of timeline do you have in mind for this and what support do you need to get it over the line? Would love to see this published as it showcases how great Paris was for the group 🎉

Hi @abangser, I know a few folks have reviewed it, was hoping to review it early this week, giving time to folks to review it over the weekend. I plan to make the changes and raise a PR with the final doc tomorrow end of the day.

That is perfect, thanks @techmaharaj !

@abangser I made all the changes and updated whatever I could. There are a couple of things on which I'm waiting for Marsh and Mathieu to revert, once that is done, we should be GTG.

Fantastic @techmaharaj, as we spoke about in slack, please just open a PR at your convenience, and happy to work with you on this if it is helpful!


[WIP] Remove references to MSDN where applicable #2037

#2037
Removing references to MSDN   (below are sections complete)

docs\csharp\language-reference

@isaac2004,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@mairaw here is a running PR... If you want it done a different way, let me know

Thank you @isaac2004!  My only comment so far is that we avoid using absolute URLs for our links (one of the reasons is that otherwise, links in the offline books for VS won't work).
One easy way to do this is to change the https://docs.microsoft.com/dotnet part of the link with ~/docs and add the .md extension.

@mairaw sounds good I can change those

Build seems stuck. Closing and reopening the PR.

@isaac2004,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@isaac2004 I just resolved some merge conflicts that were flagged in the PR. You can see how I change the links in those topics.

@isaac2004 I'd love to merge this. Do you want to address the feedback or do you want me to finish this PR for you? I'm fine either way.

@mairaw it is a big one, I am done with that one section in the OP. There are 100s of files left to scub


Windows Terminal (wt.exe) not whitelisted by default

Issue: on Windows 11, the default command line is now Windows Terminal (wt.exe). If you launch tron.bat from Windows Terminal (i.e you want to use command line switches "tron.bat -a -x -r"), Windows Terminal closes (and thus stops tron.bat) during the job "ProcessKiller".
Fix: add wt.exe to the tron/resources/stage_0_prep/processkiller/whitelist.txt
I know this is an easy fix on the user side, however I think this should be in the default whitelist, as Microsoft has made Windows Terminal the default cli for right click context menu and also for right clicking the start button, thus, this will potentially be one of the preferred ways for admins to run Tron when more users adopt Windows 11.

https://github.com/bmrf/tron/pull/215

Thanks @TylersComputersKC , will add to the whitelist. What is the full path to the exe? I checked on my test VMs and couldn't find it. System32?


baby bugs are below block level

When each of the bug creatures breeds and has a child, the child is small enough that is sinks below the block is stands on, and appears as only a shadow.  Can be seen if they are jumping or falling so they are definitelly there.  Likely just need to adjust the Y value for the transform in the model file

Fixed the issue with the transform.  Should be fixed!

Fixed the issue with the transform.  Should be fixed!


Dump project conditions on test failures

All Submissions:

[X] Have you signed our CLA?
[ ] Put closes #XXXX in your comment to auto-close the issue that your PR fixes (if there is one).
[ ] Update docs/release-notes/release-notes.md if your changes should be included in the release notes for the next release.

traces were not very useful


Generic Plugin Support

Rando4 provides a system for "connections," mods which can add their own items and locations to the randomization pool. Many feature requests we receive are for things which rando4 connections have implemented already, and most well-implemented connections allow any mod to reuse those items and locations.
We should include support for connections in the HK client using a plug-and-play model. This will involve:

A way to fail out of file creation if the required mods are not installed
A way to ingest connection settings on the generator side (likely this means getting the logic extractor running again)
An API for connections with special requirements (e.g. TRJR) to make changes to IC savedata during file creation based on the Archipelago settings

I'm very keen on taking this up but won't assign myself until prereqs are met

related to this, I'm currently avoiding the bit of code in the map mod that deals with logic overrides. probably just going to disable this bit until its figured out how plugin support will work.

Logic-wise I am considering a few possibilities:

All supported connections perform logic edits (somehow) which collects all logic in one place statically which can be pulled in by both the extractor and the map mod
Now that APworlds are a thing that exist we might be able to take an approach closer to rando4, where connections dynamically make their logic edits, and then an APWorld is generated locally with their own variations of static logic

Both of these have pretty major downsides, but even without the map consolidation of logic rules is very concerning (and I think whatever solution gets landed on, whether it's one of those or something else, will probably end up being able to easily accommodate the map just by accident)


Potential regression: notivue stopped working with nuxt v3.13.2

Environment
notivue v2.4.5
nuxt v3.13.2

Reproduction

Use Notivue playground: https://stackblitz.com/edit/nuxt-starter-fnhcmx?file=pages%2Findex.vue
Upgrade nuxt to v3.13.2: nuxi upgrade -f
Run nuxt in dev mode. Example page breaks

Describe the bug
Potential regression, as Notivue v2.4.5 used to work with Nuxt v3.13.1 without issues. Now nuxt dev breaks with:
Cannot destructure property 'isRunning' of 'useNotivueInstance(...)' as it is undefined.

I opened https://github.com/smastrom/notivue/issues/69 against Notivue just in case.
Additional context
No response
Logs
No response

Same issue


Tags support

Hi,
Great theme. Will you add tag support for docs?
Thanks

Thanks for the suggestion. Tags are planned for Hextra's future roadmap. Could you elaborate on how you envision using tags in docs, perhaps with examples?

Hey thanks for the fast response. I want to use tags in  a book type content (aka docs structure)

I'm going to add specific kind of tags support: A way to automatically link docs pages to relevant blog posts:

I'll set tags in docs via cascade in the Hugo config.
In blog posts, I'll manually add tags when I write them.
My template coding will use the .Related function in docs to link to these blog posts with matching tags.

A way to automatically link docs pages to relevant blog posts:

I'll set tags in docs via cascade in the Hugo config.

Could you give some examples?

Could you give some examples?

Sure. I have docs hierarchies for different programming languages. I create these pages with a headless CMS. So, instead of changing the markdown documents, I could do this:
cascade:
- _target:
    path: "python/**"
  tags:
  - python

(Now that I write that out, I dislike how redundant it is. Maybe I would change my CMS to output tags in the frontmatter.)


working with private dependencies in functions

I came across a use-case where I had to use a private git repo as a go import, the git repo had some APIs.
Usually, I can just change my gitconfig file to include the creds and go will include them while building the binary. Is it possible to do the same in functions (or buildpacks maybe)?

There is not anything stopping us from sending the .gitconfig along with the source code for the build, however we would want to implement an explicit directive to ingore this file when actually building the contents of the container image so that those credentials are not part of the final container.
We are not actively working on supporting referenced private dependencies at this time, though it is something we do intend to support.  Have you considered vendoring the dependencies before it is sent to the builder, such that it need not reach out over the network at build time?

I tried the latest func, and did the vendoring, but seems like I am getting the same issue while the image is being built - is this expected?

For reference, Buildpacks added support for this in https://github.com/paketo-buildpacks/go-mod-vendor/issues/140

/remove-lifecycle stale

/reopen


Add translation for OTP concurrency - Portuguese

Related to #122
Note: I couldn't find a good translation for callback, so I leave as it is but in italic.
Note 2: The word handler is not easy to translate, so I used the word "processador" which is something similar.

@pragmaticivan can you review this one?

@philss Just left some comments. Thanks! :beers:

@pragmaticivan thank you! fixed :beers:

Nice, thanks :sparkles: o/


Disable combine behaviour in cli

spoonbill's cli currently does not have any option to disable combining behavior of the library. It may be good to have such, maybe through additional --no-combine option.
Also additional validation level would be required  to --state-file option because analyzer do not track combined locations as child tables, so, changing list of combined types would require new  analysis.

--no-combine sounds good.


feat: adding documentation to cross modal example

Enhancements to the cross modal example documentations in terms of:

[x] Research and use case explanation of cross - modal search

[x] Better explanation of Jina flows for index and query

[x] Results

Results are not properly seen in github?

Results are not properly seen in github?

Result jpg is pushed to this repo as a part of this PR in the results folder, this is embedded in the Results section of the README


Trying to add Badge Component

Trying to create a route /badge  which will render the badge component

Closing for https://github.com/Diversity-In-DSC/Badge-Module/commit/49294b23c2c6462a3e4fd0a4dddf650ea0def94f


Add expanding row to .table

The WEB-TBL-EXP-001 and WEB-TBL-EXP-002 for the small and base tables are missing in v5.
Need to study if the behavior need an implementation to Boosted or if an example is enough.

Close this one for #526.


docs: Update CI badge and versions table in README

coverage: 98.942%. remained the same
when pulling eb6b2a4c415668e8bb07e2427b674aed2905ea93 on docs/update-readme
into ef3aebeed832d14ba3b2c919163f87d5ca369f1a on main.


src folder now takes sources relative to CMAKE_CURRENT_LIST_DIR

I could never get daxa to be included as a subproject so I fixed it.

This should not ever be necessary

It is when src isn't at the root level of a CMakeLists.txt chain

what do you mean "CMakeLists.txt chain"? Are you using add_subdirectory()? or include()?

I have my top level CML.txt @ ~/Workspace/SkillQuest/CML.txt and daxa is in ~/Workspace/SkillQuest/ipotrick/daxa/CML.txt

Okay, add_subdirectory works, this can be closed


Bicep

Need some Bicep code samples here please!

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

ID: 6dada714-f867-2a46-6b94-eaea6e1da966
Version Independent ID: 25dc8f50-52c6-3c2b-ade6-c5d8258b74ec
Content: Configure managed identities on Azure VM using template - Managed identities for Azure resources
Content Source: docs/identity/managed-identities-azure-resources/qs-configure-template-windows-vm.md
Service: entra-id
Sub-service: managed-identities
GitHub Login: @barclayn
Microsoft Alias: barclayn

@solidcloudio
Thanks for your feedback! We will investigate and update as appropriate.

@solidcloudio
Thank you for bringing this to our attention.
I've delegated this to content author @barclayn, who will review it and offer their insightful opinions.

@solidcloudio thank you for reaching out. I have created an internal work item to track your suggestion. #please-close