Issues building with fastlane in a Jenkins job

Not sure what exactly is going on, but the same fastlane lane that builds and archives an iOS app that is using Cocoapods (0.35 for iOS 7 support) from the command line fails to find the pod targets during the linking phase when executed in the context of a Jenkins job:
ld: library not found for -lPods-xxx
Has anyone else experienced this? There isn't that much special going on in terms of the gym command being used to build the app and it's fine from the command line. None of the other phases of the lane in Jenkins is a problem (bumping version, getting latest provisioning profiles, etc.).
I'm using the latest fastlane tools including the recent updates to gym from today and Xcode 7 GM/iOS 9 SDK/etc. (Jenkins is at v1.598).

Is your scheme shared? Do you use the cocoapods action in fastlane?

The scheme is shared, but I'm not using a cocoapods action in fastlane as the Pods folder is under source control.
I did just confirm from a fresh repo clone that the xcodebuild command being issued by Fastlane does fail on its own if done on the command line similar to how it fails in Jenkins, but not when executed within Fastlane.
set -o pipefail && xcodebuild -workspace 'XXX.xcworkspace' -scheme 'XXX Beta' -configuration 'AppStore-Beta' -destination 'generic/platform=iOS' -archivePath '/Users/xxx/Library/Developer/Xcode/Archives/2015-09-21/org.xxx.XXX-Beta.AppStore-Beta.ad-hoc.<IP_ADDRESS> 2015-09-21 15.55.58.xcarchive' clean archive CODE_SIGN_IDENTITY='iPhone Distribution: XXX (xxxxxxxx)' | xcpretty

Ok I think I know what's happening. A certain script wasn't running ahead of time.

I experienced the same problem, how to solve it ,can you explaint  it detailedly？

@fanyuRN Thanks for reaching out on this. Can you open a new issue if you are running into a problem? Thanks!

@TKBurner  Thanks!I have resolved it

@fanyuRN Awesome! So glad to hear you are up and running!

@fanyuRN I'm running into this problem, too.  Can you share what you did to solve it?  Thanks!

you should  modify your workspace,


Add new runtime versions to the language list

Environment summary
Install Method (e.g. pip, interactive script, apt-get, Docker, MSI, edge build) / CLI version (az --version) / OS version / Shell Type (e.g. bash, cmd.exe, Bash on Windows)

This should go in the https://github.com/Azure/azure-cli-extensions repo where the extension lives.


[Project Submission] Community Showcase Submission

@TheDiscordian can you help push this through?

@mishmosh just some copy and paste mixup, I forgot to remove it from the template

March 1, 2021
I have just been notified that I will be receiving a microgrant for my submission. I am excited. I have begun looking to contract out a UX / UI Designer in order to help me build out new mocks for front-end. I need to generate new requirements in order to make sure I have all the features that I want in the v1.
I will also be getting some assets made for the NFT when the lots are generated.

March 8:
Working with a designer to scope on direction of the application
Got some direction from a founder that I know
Got some direction from a  employee who works at a VC on a pitch deck
Met with Senior Architect on direction of the backend design

Cool!

When I did the hackathon, I used a json payload for the houses so I can iterate fast during the Eth Global Hackathon.
I had began setting up firebase as a backend because that I was familiar with but I realized I would probably be reinventing the wheel with a lot of the solutions in firebase. Therefore I am opting to use Moralis, a "firebase for crypto" as they say in their docs in order to get the ease of the web2 auth and web3 token management that I don't really want to deal with managing.

So this week i started a new repo. The Hackathon repo I had was a bit too messy so I started from scratch using Moralis.
Things i need to think through is KYC and validation that a user really owns a property

Found a list of APIs for real estate that can prove very useful
https://gist.github.com/patpohler/36c731113fd113418c0806f62cbb9e30

So I have been creating use case diagrams of all the users for the platform
The designer will start breaking ground on the design so we can start implementing the visual

I didn't want to make a mistake of implementing without thinking through features so I decided to put all my use cases down for all the users on the platform
After that I will have a designer that will help me flesh out the designs and make them come to Loren

Started working on the v2 app will upload to vercel so that i can have demo link

![Uploading 163751759...]
![Uploading 163751910...]
Got some isometric images off Fiverr and bought a domain

@Josiassejod1 the images didn't seem to finish loading to GitHub.com. 😃

Got a designer working on figma mocks now too

Got the dev branch up here, realty-dapp-v2-gkpiolvat-josiassejod1.vercel.app

Paid a designer for their services

Haven't left an update in a while, I really have just been building the component library out,


Add support for Azure DevOps

Is your feature request related to a problem? Please describe.
Before Microsoft acquired Github, there was an stll remain Azure DevOps which is a huge part of many enterprise environments, so I would like to see Draft support Azure DevOps pipeline creation.
Mention what platform you want to support the new feature
Azure DevOps pipelines
Describe the solution you'd like
'generate-workflow' should be able to generate a yaml pipeline for Azure DevOps.
Describe alternatives you've considered
Manually writing Azure DevOps pipeline.
Additional context
N/A

Second this.

The official Youtube account for Microsoft Azure commented almost a year ago that it is "on the roadmap":


Clarify meanings of unitSymbol, unitSymbolHtml, and unitSymbolUnicode

The annotations for unitSymbol, unitSymbolHtml, and unitSymbolUnicode could be improved to better indicate what each should be used for. Each of the definitions contains 'm?' as an example for square meter--this appears to be rendering error. The annotations could be rephrased without reference to it. Examples should also be separated into skos:examples.
unitSymbolUnicode could be improved with an example: The unicode for square meter is U+33A1.
gist:_percent uses the special character "%" as its unitSymbol, but unit symbols aren't supposed to have special characters. We could revise the annotation for unitSymbol or the symbol for gist:_percent.

#992 should be discussed first, since it makes this moot.

@dylan-sa Has this been addressed in PR #997? If so, please close this issue.

Yes, #997 cleared things up nicely.


[NOREF] Email Validation Error Message && Use Built-in React USWDS Required Marker in Form Labels

NOREF
Description
Addressing https://cmsgov.slack.com/archives/C01B9TEC65A/p1733169342753359?thread_ts=1733163413.249109&cid=C01B9TEC65A

use built-in react USWDS required marker in form labels

How to test this change

Navigate to the custom solution modals
Put a invalid email address and check to see error message pops up
Enter a valid email and check to see that the error message goes away

PR Author Checklist

[x] I have provided a detailed description of the changes in this PR.
[x] I have provided clear instructions on how to test the changes in this PR.
[ ] I have updated tests or written new tests as appropriate in this PR.
[ ] Updated the Postman Collection if necessary.

PR Reviewer Guidelines

It's best to pull the branch locally and test it, rather than just looking at the code online!
When approving a PR, provide a reason why you're approving it

e.g. "Approving because I tested it locally and all functionality works as expected"
e.g. "Approving because the change is simple and matches the Figma design"

Don't be afraid to leave comments or ask questions, especially if you don't understand why something was done! (This is often a great time to suggest code comments or documentation updates)
Check that all code is adequately covered by tests - if it isn't feel free to suggest the addition of tests.

@garyjzhao Changes look good, but I think snapshots might need to get updated!
https://github.com/CMS-Enterprise/mint-app/actions/runs/12144471382/job/33863890839?pr=1549

@ClayBenson94 pushed up snapshots in the lastest commit


[BUG] The registry.all_estimators() function fails if pytest is not installed

Describe the bug
Listing all estimators via the aeon.registry.all_estimators()-function raises the below error if pytest is not installed. I assume that the function accidentally loads a test module while traversing the aeon source modules.
I also noticed that the first execution takes a considerable amount of time (around one minute), which I find too slow. I cannot reproduce the long runtime in an existing workspace (venv).
Steps/Code to reproduce the bug
from timeit import Timer
from aeon import registry
from aeon import show_versions

show_versions()

timer = Timer(lambda: registry.all_estimators(estimator_types="clusterer", return_tags=True))
try:
    timer.timeit(100)
    print(f"Executing 'registry.all_estimators()' runtime measurements")
except Exception:
    timer.print_exc()

print(timer)

Expected results
The registry lists all clustering estimators with their tags (probably a list of dictionaries).
Actual results
Traceback (most recent call last):
  File "/home/sebastian/Documents/projects/ts-clustering/aeon-registry-bug.py", line 10, in <module>
    timer.timeit(100)
  File "/home/sebastian/.conda/envs/aeon/lib/python3.10/timeit.py", line 178, in timeit
    timing = self.inner(it, self.timer)
  File "<timeit-src>", line 6, in inner
    _stmt()
  File "/home/sebastian/Documents/projects/ts-clustering/aeon-registry-bug.py", line 8, in <lambda>
    timer = Timer(lambda: registry.all_estimators(estimator_types="clusterer", return_tags=True))
  File "/home/sebastian/.conda/envs/aeon/lib/python3.10/site-packages/aeon/registry/_lookup.py", line 249, in all_estimators
    raise e
  File "/home/sebastian/.conda/envs/aeon/lib/python3.10/site-packages/aeon/registry/_lookup.py", line 233, in all_estimators
    module = import_module(module_name)
  File "/home/sebastian/.conda/envs/aeon/lib/python3.10/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1006, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 688, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 883, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/home/sebastian/.conda/envs/aeon/lib/python3.10/site-packages/aeon/testing/test_all_estimators.py", line 15, in <module>
    import pytest
ModuleNotFoundError: No module named 'pytest'

Versions

System:
python: 3.10.13 (main, Sep 11 2023, 13:44:35) [GCC 11.2.0]
executable: /home/sebastian/.conda/envs/aeon/bin/python
machine: Linux-5.15.0-94-generic-x86_64-with-glibc2.35
Python dependencies:
pip: 23.3.1
setuptools: 68.2.2
scikit-learn: 1.4.1.post1
aeon: 0.7.0
statsmodels: None
numpy: 1.23.5
scipy: 1.10.0
pandas: 2.0.3
matplotlib: None
joblib: 1.3.2
numba: 0.56.4
pmdarima: None
tsfresh: None

@MatthewMiddlehurst I think there are a few test files defined outside the tests folder in the testing module. Hence, it is not skipped by MODULES_TO_IGNORE in _lookup.all_estimators causing this issue?

I have also noted all_estimators takes a long time sometimes,

@MatthewMiddlehurst I think there are a few test files defined outside the tests folder in the testing module. Hence, it is not skipped by MODULES_TO_IGNORE in _lookup.all_estimators causing this issue?

good spot! Maybe just include testing in the MODULES_TO_IGNORE? Should also ignore datatypes, datasets and maybe visualisation?

I can reproduce this, but also note that return_tags is not a boolean
return_tags: str or list of str, optional (default=None)

if I remove the return_tags argument in the above example and add testing to the modules to ignore, it works fine. I think it should be set to this
    MODULES_TO_IGNORE = ("tests", "setup", "benchmarking", "utils", "all",
                         "testing", "datasets","registry","datatypes","visualisation")

do you agree @MatthewMiddlehurst?
thanks @CodeLionX for reporting it and @aadya940 for fixing it :) @CodeLionX would you like to fix it by creating a PR changing all_estimators MODULES_TO_IGNORE to the above (assuming matthew agrees to the list)?

I think slow execution is a separate issue, I'll run some tests, iirc its very slow when an incorrect tag is given but needs testing.

I'll prepare a fix for the pytest-issue.

Was away for a bit, thanks for working on this. Merged the PR by @CodeLionX. Agreed that the slowness of the method should be separate.


Way to view offending code location?

When running a normal Trivy config it outputs the code location of the offending lines. With the report here I am unable to figure out how to still get that information. That data is critical for our team to be able to take action on the data that is being presented.

Hi @austin-engle ,
Thanks for reporting this. We are planning to implement that feature in the next release.
Please consider giving us a GitHub star ⭐.
Thank you!

I appreciate you letting me know that is in the roadmap? I'm planning to implement this into our pipeline soon. Do we have a potential timeline to when this would be available?

Yes, it will be available in the first week of January.


Added No records found in ABDM and button placement adjusted in Investigation Reports Page

Proposed Changes

Fixes #8997
Added No records found text in ABDM page
Adjusted the Next Session and Previous Session buttons placement in Investigation Reports Page

Before Changes:

After Changes:

@ohcnetwork/care-fe-code-reviewers
Merge Checklist

[ ] Add specs that demonstrate bug / test a new feature.
[ ] Update product documentation.
[ ] Ensure that UI text is kept in I18n files.
[ ] Prep screenshot or demo video for changelog entry, and attach it to issue.
[ ] Request for Peer Reviews
[ ] Completion of QA

Summary by CodeRabbit
Summary by CodeRabbit

New Features

Added a message to indicate when no consent records are found, improving user feedback.

Bug Fixes

Corrected capitalization for the "No Records Found" message to ensure consistency in localization.

Style

Enhanced styling of the "Previous Sessions" button for improved visual appearance.

LGTM


QA - Site - Product Detail Page - User can not be able to Select Shipping Address in Place Order.

QA - Site - Product Detail Page - User can not be able to Select Shipping Address in Place Order.

Click on search button and go to search page.
Click on any product and go tot detail page.
Select Place Order tab and Select product Color.
Enter Qty in Qty box.
Select Shipping Method.
Observe its not working.

Done.
User Should be Able to Select Shipping Section Now.


Add documentation for custom plugins

[Ch1486]

@martina-if You know the build failed right? All good if so. 👍

I added the conclusion section but it feels like the wrong word for this. Perhaps "next steps" or "what to expect next" or something similar would be better.

The pre tag is causing horizontal scrolling on mobile but most of our traffic is desktop so we can fix it later.


 The grep code in script can only get one digit of minor version, showing 1.1 for version 1.10.

Thanks for that 👍

Hope you saw #264 covering more use cases


Creating Issues template for the project

I've created issues template for those who want to create new issues for an easier issues management inside this project. Please assign me to the task and I'll create PR immediately. Thanks!
-- Bugs report --

-- Features request --

@vatana7 thanks for volunteering! I'll review it shortly.

Released to main.
Thanks @vatana7 !


Add default export

Make ES module - compatible and allow TypeScript users to easily use the package.
This will work after merging the PR:
import ms from 'ms'

I am not a big fan of those double CJS / ES exports myself, but there are a couple of scenarios when you have basically got no other choice.

Update: I just learned there is now finally a way around that issue, see this tweet.
Feel free to turn down if you want :)

Glad you found a fix! Generally, this should be resolved by Webpack anyway... 😊

@andywer how did you get this to work? in NodeJS? Angular?

@ericsnap Just set the esModuleInterop option to true in your tsconfig.json. See here. It's a TypeScript thing, so it works everywhere: frontend, backend, wherever you use TS.

@andywer thanks. yeah, I was having issues compiling it kept saying that it did not have a default export. I did have to also set allowSyntheticDefaultImports. Does the module setting or the target have any impact on this?

@ericsnap Try:
{
    "allowSyntheticDefaultImports": true,
    "esModuleInterop": true,
    "module": "commonjs",
    // ...
}


startup.md incorrectly references "http://example.org"

I was just reading over lrsql/startup.md and I noticed that on line 18 of that file, it says that the default name for authorityURL is "http://example.org". But I think it is in fact "http://mydomain.com" (in config/lrsql.json.example).
Thanks for all you are doing!

@JonathanDudzik Hi, we're going to close this issue since we've determined it was not actually an issue and we don't want to leave it open if that's the case. If you still think there is a problem, just let us know and we can re-open the issue.

Hello @kelvinqian00, sounds great. Thank you for looking into it and helping me understand better. :)


Error caught by duk_pcompile missing line number/filename

It seems that if an error occurs during compilation (a SyntaxError, say), the lineNumber and fileName properties of the resulting error object exist, but are set to undefined.  Bug, or...?

This is covered by #140 so closing.


PPGN Dockerfile

I made a Dockerfile of your PPGN implementation, for GPU use with CUDA 8. Source code available on github: https://github.com/thommiano/ppgn-docker/blob/master/Dockerfile
and it's also available on Dockerhub:
https://hub.docker.com/r/socraticdatum/ppgn/

@thommiano This is awesome! :) Thanks for letting me know.

@anguyen8 I'm going to update the file once I get example 5 working.


fix issue with caching global this object incorrectly

I misunderstood meaning of FunctionCodeGenJitTimeData::isInlined and was trying to use that to save global this address for topfunc only, but that field has a different meaning. Instead, just always store it when creating FunctionCodeGenJitTimeData.

@dotnet-bot test Ubuntu ubuntu_linux_test


Add helpers to get line/col of error message

index is enough for machines but hard to read for humans. It's trivial to get the line/col via
val lines = f.input.take(f.index).lines.toVector
val line = lines.length
val col = lines.last.length

But it's probably worth chucking this (or some more-optimized version of it) into the Failure object for everybody's convenience

Would you want to also report the parser position as line:column in Failure.formatParser? Most calls to this function are not enclosed in the scope of a Failure.
It might be the case to push some additional information into the ParseCtx, like the number of already read lines and the global input index at start of current line.

reporting line:column in formatParser seems reasonable. I'm not sure if we still want to show the index somewhere; it seems like it might be useful but maybe it isn't
You shouldn't need too much additional info; IIRC most places where we call formatParser already have access to the input: String, and that + index is enough to re-derive the line and column

This has landed in #52. Thanks @Alain-Bearez!


Adding formatting spec

This PR adds a spec that details formatting requirements.

The Sway repository is the only one that's adopted this formatting configuration (which could possibly be a chicken-and-egg problem) in this time. I would move to close this rather than require all other repos to adopt a new formatter configuration.

cc @emilyaherbert for the decision

The Sway repository is the only one that's adopted this formatting configuration

Er, is this right? It looks like the Rust SDK has the same file https://github.com/FuelLabs/fuels-rs/blob/7fb18dfeeb12519fc7e6bd87e8da003d2e8dc9ee/rustfmt.toml
I'm open to closing this RFC (let's ignore the fact that it's been open since September haha 😅), aside from the fact that standardized formatting reduces noise and merge conflicts. As the number of outside contributors grows this will be more important, so if it needs to happen, it should be sooner rather than later IMO.

cc @xgreenx might have some extra suggestions :)


report volcanism

I've never used GitHub so I hope I'm doing this right and this is the correct way to ask. I just start using EDDI with ED and absolutely love it. However, I know nothing about programming or scripting.
In the Body scanned script, there is a section that reads:
{_Atmosphere }
{_Volcanism }
I was looking to have this report both Atmosphere and Volcanism in addition to what it already is doing. Currently, it just skips this and moves to rotation.
I've been trying to duplicate what similar scripts say but am having no luck, and Google doesn't seem to be helping either.
What would go here so that it would report these conditions as well?
Thanks so much in advance.

This is on the list to add, but I haven't managed to get around to gathering all of the correct internal names for the different atmosphere and volcanisms.

Thanks for the reply. Seems I was jumping the gun. You've done an amazing job with this! I'll def be on the lookout for future updates

On Feb 20, 2017, at 10:33 AM, cmdrmcdonald<EMAIL_ADDRESS>wrote:
This is on the list to add, but I haven't managed to get around to gathering all of the correct internal names for the different atmosphere and volcanisms.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

This has been incorporated in to EDDI 2.3 (currently in beta).

Oh wow! That is great news! Thanks so much for your work on this. I found EDDI a month or so back and incorporated it into VA. Makes this game so much better and immersive.

On Apr 19, 2017, at 8:48 AM, cmdrmcdonald<EMAIL_ADDRESS>wrote:
This has been incorporated in to EDDI 2.3 (currently in beta).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/cmdrmcdonald/EliteDangerousDataProvider/issues/139#issuecomment-295257770, or mute the thread https://github.com/notifications/unsubscribe-auth/AYtDIcmAOXiuARKwOZV6IjaDbjqfH3HJks5rxgKCgaJpZM4MGOsZ.


Fixes the expandableListView in the streams drawer

This was because in the messages table stream column and the read column have null values which should not be the case, I have uploaded the database of the android app which is dumped after the first login to the app, if any one wants to tinker with the queries (Maybe the SQL queries can be made better)
databases.zip
Previously the queries were dependent on the id's now they are being compared by names!
2e4ce93 is basically removing 0fea6fbc525390f9a50ab476b7126f9203fb3dee commit, not sure though if this is the best method to do this!

Automated message from Dropbox CLA bot
@kunall17, it looks like you've already signed the Dropbox CLA. Thanks!

I tried this, after reading messages in homeview, the unread counts updates in streams drawer but after I tap on anyone of the streams, and then check stream drawer unread count, it increases for that stream.

@kunall17 have you had a chance to look at working on this further?

@timabbott The PR does its job well, but there is something else playing with the read values!
Sam1301 is looking for it!
And after merging https://github.com/zulip/zulip-android/pull/283 the problem should be fixed, just we would have to revert the commit 0fea6fb

superseded by #283


⚠ Bofrost.at has degraded performance

In ec48edc, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2371 ms

Resolved: Bofrost.at performance has improved in 1048899 after 7 minutes.


[openbsd] hide support column and update summary

2 points:

Are we tracking -stable or -release. I'm assuming -release, but maybe we should make it explicit
The releaseCycle needs to be filled, otherwise the first column is blank. If there are no patch releases, we can switch the latest to releaseCycle, and hide the last column with releaseColumn: false.

@captn3m0

Are we tracking -stable or -release. I'm assuming -release, but maybe we should make it explicit

-release

The releaseCycle needs to be filled, otherwise the first column is blank. If there are no patch releases, we can switch the latest to releaseCycle,

How to do it?

and hide the last column with releaseColumn: false.

Done
--- a/products/openbsd.md
+++ b/products/openbsd.md
@@ -6,6 +6,7 @@ category: os
 command: uname -r
 activeSupportColumn: false
 releaseDateColumn: true
+releaseColumn: false
 sortReleasesBy: 'release'
 releases:
   - release: 2021-10-14

Page looks good: https://deploy-preview-913--endoflife-date.netlify.app/openbsd

Page looks good: https://deploy-preview-913--endoflife-date.netlify.app/openbsd

I like it too. Let's merge it to master.


Simplify kubectl kcp workspace plugin usage and implementation

Always derive the current workspace name and org from the server URL of the current kubeconfig context (instead of reading it from the current workspace kubeconfig context key). This fixes issue #678
Store only one kubeconfig context to store the current workspace context, and one for the previous workspace (instead of the whole history of current contexts)
Correctly manage pretty names, so that they are shown as an alias in the use or current commands as shown below:

$ kubectl kcp workspace --token=another-token create my-workspace
Workspace "my-workspace" created.
$ kubectl kcp workspace --token=user-1-token create my-workspace
Workspace "my-workspace" created.
# It is a distinct workspace, whose org-wide name will be disambiguated. But the name requested by the user was given as an alias in the personal scope.

$ kubectl kcp workspace --token=user-1-token current
Current workspace is "my-workspace" (an alias for "my-workspace--1") in organization "default".

# Now with the org-wide ("all") scope instead of the personal scope, we directly see the workspace real, org-wide name
$ kubectl kcp workspace --token=user-1-token --scope=all current
Current workspace is "my-workspace--1" in organization "default".```

In addition, to use a given context instead of the current context to specify the KCP server URL, no need to reset the current kubeconfig context first. You can now use the standard --current kubectl override:

$ kubectl kcp workspace use my-workspace
Current workspace is "my-workspace" in organization "default".
$ kubectl kcp workspace --context=root use default
Current workspace is "default" in organization "root".

More details:
For historical reasons until now the plugin used to add, inside the current kubeconfig, one new context for each new current workspace (after a "use" call), with the context name containing the name of the workspace and the related scope.
This way of storing current workspace contexts has become inconsistent with the current plugin behavior which builds the workspaces virtual workspace URL automatically based on the current context server URL, by extracting the current workspace logical cluster name from it.

@varshaprasad96 You might be interested by this PR, above which #632 should be much easier to implement.


Set up CI with Azure Pipelines

This branch was supposed to be purely for testing something in dev.azure.


Townspots icons sometime cut off

See around NYC

Related: https://github.com/tangrams/tron/issues/75


Filtering of params in QSA

Hi,
First, Thanks for your work.
I got a question, for a small project, i use github-webhook-handler to handle all of my "webhook"
It's a server receiving QSA to know which webhook must be launched /webhook?nameOfTheWebhook
To made it work, I have to hack your lib to get rid of the two line filtering ?
if (req.url.split('?').shift() !== options.path)
      return callback()

Why do you forbid QSA usage ?
Is there any way to accept them (or to deal with dynamic routing /routing/[a-z]+) without hacking your library ?
Thanks again :)
Kenny

I just think this isn't a use-case that's been dealt with yet, I'd be happy to see a pull request to make it work, just be sure to include tests to verify that it works properly

Hmm i have no problem in making a pull request, but this one will only delete code :)
the two line quoted in my initial post, and the according test for rejecting path with query string.
If there is no reason to forbid QSA, and you confirm i can delete those two things, i will make a pull request for sure :)
if there is a reason, i'll be happy to talk about it.
thanks

hi, any news on that ? Thanks you

@kennydee, I think this would be a fair change. Query strings are still valid within a POST are still valid HTTP, and I think it's reasonable to trust downstream code to filter/validate them as needed.
If you're still up to submit that pull-request (and no-one else expresses concerns) I'd be happy to bring it in!

removing the ? split for a semver-major release


RqForm is broken as described in readme

Hi! I was trying do some form processing with takes accroding to the README
public Response act(final Request req) {
    final String name = new RqForm(req).param("name");
    return new RsWithStatus(HttpURLConnection.HTTP_NO_CONTENT);
}

But when I try to replicate it, it give me a error: 'RqForm' is abstract; cannot be instantiated, then I took look at the source code and RqForm  is an interface which cannot be instantiated.
public interface RqForm extends Request {
    Iterable<String> param(CharSequence var1) throws IOException;
    Iterable<String> names() throws IOException;
}

And the interface's  .param()return value does not return a String, like the example in readme.
Is there something I did wrong(eg. imported the wrong class)?  Please help! Thanks in advance.

@paulodamaso/z please, pay attention to this issue

@ssysm/z this project will fix the problem faster if you donate a few dollars to it; just click here and pay via Stripe, it's very fast, convenient and appreciated; thanks a lot!

@0crat in


Tag and release 0.3

With https://github.com/fractal-analytics-platform/fractal-web/pull/123, milestone 0.3 is in principle over. Let's close as soon as we have some additional checks by me and @jluethi.
Then we should:

Add a 0.3 tag
Make a github release

For the moment the github release is not crucial, but we should start getting some experience on how we want things to be. Let's begin with the simplest option (that is, let's not include the build files), and ater on we can check what's best re:

Should we actually attach the bundle to the release?
Should we build it as part of a github action, so that we have a more predictable output (e.g. we always build with the same node version)?

I checked the minor fixes in https://github.com/fractal-analytics-platform/fractal-web/pull/123 and confirm that they are working as expected.
👍🏻 for a 0.3 release and starting to explore the release process :)

Release: https://github.com/fractal-analytics-platform/fractal-web/releases/tag/0.3.0
Tag: https://github.com/fractal-analytics-platform/fractal-web/tree/0.3.0


How to force swtich language code?

I could not find a function to change the language manually.

to change it programatically :
astroI18n.langCode = "fr"
for a language switcher :
const routes = astroI18n.langCodes
	.filter(langCode => langCode !== astroI18n.langCode) // optional (excludes current language)
	.map(langCode => l(Astro.url.pathname as any, {}, {}, langCode as any))


Speed up backend tests

Summary
This PR speeds up backend tests by removing redundant backend tests. The targets, which always been included in the slowest 20 test durations, are test_elementwise_operations and test_nn_operations.
This PR also cleans up old styles for backend tests including assert_value_with_ref, return_results, shape_or_val, and the related stuff with them. As a result, the test_conv, test_depthwise_conv, and test_pool are now much cleaner than before.
Related Issues
#10972
#10956
#10930
cc @gabrieldemarmiesse

What does cc mean? Should I review this PR?

@gabrieldemarmiesse I mean that please feel free to see the changes :) Because you have been contributed to related issues.

Sure then, thanks!

From what I see, it looks nice and clean now. There doesn't seem to be any weird stuff going on.


Adding health checks with Liveness, Readiness, and Startup probes

Adding health checks with Liveness, Readiness, and Startup probes

It was done in  #97


[openwrt] Fail in "Thread Harness Test" as a Thread Border Router.

I built a OpenWRT FW for Qualcomm Maple board and ported the "ot-br-posix" for implementing the Thread Border Router.

Mother board : Qualcomm Maple board
RCP board : SiliconLabs MG21

The "Thread Border Router" function seems work well.
But it got a lot of FAIL in "Thread Harness Test".
Attached C_5_9_1-3.zip is the test report of Thread Harness Test Case "BR_1" "C_5_9_1".
Really appreciate if anyone can provide suggestions.

What configuration options are you compiling ot-br-posix with? It might be worth comparing what you are using to the reference implementation that is used for certification. Alternatively, given that you may be porting to a new board, it could be a kernel config issue.

Following are the ot-br-posix Makefile & the kernel config file I used :

openwrt ot-br-posix Makefile : openwrt-ot-br-posix-Makefile.zip
Kernel .config : kernel-config.zip

The file "C_5_9_1_teraterm.log" [in C_5_9_1-3.zip] is otbr-agent log (run in foreground) .
There are several "Failed to update ipsets" in the log.
otbr-agent[26301]: 00:01:47.621 [I] Platform------: Execute command ipset flush otbr-ingress-allow-dst-swap = 0
otbr-agent[26301]: 00:01:47.636 [I] Platform------: Execute command ipset flush otbr-ingress-deny-src-swap = -1
otbr-agent[26301]: 00:01:47.636 [I] Platform------: Got an error when executing command ipset flush otbr-ingress-deny-src-swap: No child process
otbr-agent[26301]: 00:01:47.636 [W] Platform------: Failed to update ipsets: Failed_
I am not sure why such ipset Failed happen ?
Really appreciate if anyone can provide some suggestions.

Closing stale issue.


Fix code of fixture error for 32-bit devices

Replaces 0xdeadbeef in test code with 0x01234567.
0xdeadbeef  causes following build error on 32-bit devices.
ResultTests/ResultTests.swift:101:61: error: integer literal '3735928559' overflows when stored into 'Int'
let error = NSError(domain: "com.antitypical.Result", code: 0xdeadbeef, userInfo: nil)

See https://travis-ci.org/ishkawa/Result/builds/83435179 for the full content of xcodebuild log.

Thanks for the PR!
Since this makes error and error2 the same, I wonder if we could just use a single error, or alternatively use different codes (e.g. 1 and 2) while still avoiding overflow on 32-bit.

this makes error and error2 the same

No, there're different each other, 0x01234567  and 0x12345678 . Actually, there're confusing, so I have replaced them with 1 and 2!

Haha, I clearly hadn’t caught that :smile: Nice one!
This is very good, thank you @ishkawa!

:smile:

@ishkawa: I’ve added you as a collaborator. We prefer a PR & review workflow for changes, as usual, but you can feel free to review, tag, etc. now as well :smile:

Glad to hear that! Thank you :sparkles:


Provide filename option to ejs render, allowing template partials

Hello. Using a custom theme, I tried to use include in the ejs to share some stuff between parts of our codebase. The include fails miserably because an include path is not found in the ejs. Digging a bit deeper, I noticed that providing a "filename" option to the ejs renderer makes it find the inclusion path where it searches for these 'template partial'.
In the current behavior, if you for example use a template like this:
hello.

<%- include someotherfile.ejs %>

<%- product.name %>
<%- name %>

You will get an error.
The changes in my PR fixed this for me.

@amireldor Looks good to me! 💯
Thanks for your contribution! Published as 2.0.12.

@amireldor Looks good to me! 💯
Thanks for your contribution! Published as 2.0.13.


Added node management services

Added a docker-compose.yml definition to make it easy to install some node management services in the system.
Do not accept the PR if it's out of scope for the repo.
Please check the use of Environment variables in the docker-compose. Just to be sure.

done as per your suggestion

done as per your suggestion


Correctly record opsbehind prop in events

Description
Item: https://dev.azure.com/fluidframework/internal/_workitems/edit/4662
We found that Data_opsbehind is not correct when the connection is established in parallel to snapshot fetch as then the lastProcessedSeq number would just be 0 and opsbehind would just be lastKnownSeqNumber - 0 instead of lastKnownSeqNumber - snapshotseqnumber.
Also add opsbehind prop in connected event in case of first connection to see what was the diff between lastProcesssedseqnumber and snapshotseqnumber.

'					opsBehind = checkpointSequenceNumber - this.deltaManager.lastSequenceNumber;  '

In the NoJoinOp query we use deltaManager.initialSequenceNumber instead of lastSequenceNumber. Should we update our query ?

I mean it has the same issue. If the snapshot is loaded in parallel to making a connection, then it will be 0, otherwise it will be same as lastSeqNumber. We may want to eliminate cases entirely in which connection was made in parallel.


reader: Fixed race condition

reader: Fixed race condition where reader would be terminated before it was properly initialized.
Currently, when reloading input very quickly (happens when updating the prompt quickly with 'reload' bound to 'change') it is possible for the reader's terminate method to be called before readFromCommand properly spawns the new reader process. This means the process is never killed and input is not reloaded until the process completes naturally or another 'EvtSearchNew' is triggered.
An example where this becomes a problem is when using fzf with rg integration and searching a large directory. Occasionally, the query being processed by rg is not the same as the query displayed by fzf and only after further modifying the query or waiting until rg finishes will the input reload to the correct state. This commit introduces a 'started' condition that the terminate method will wait for before trying to kill the process. The 'started' condition is signalled only when the reader reaches a state where it can be safely terminated.

Is there any way I can easily reproduce the problem without having to rely on ripgrep and a large directory? Maybe using a contrived command with sleep for an artificial delay.

fzf --disabled --bind 'change:reload:echo Current query: {q} && sleep 1d'

When running this command, sometimes when you stop typing a query you may see that the item list does not update for the latest version of the query. You either have to wait 1 day or modify the query to get any new results.
https://github.com/user-attachments/assets/a81a0d94-84ff-404c-83d6-8a11b1c170ab

Please see dcb4c3d84a5bd7985dd70dd7de7a68992693c20c and let me know if it doesn't fix the problem, or you find any flaw in the patch. Thanks.

Thank you! https://github.com/junegunn/fzf/commit/dcb4c3d84a5bd7985dd70dd7de7a68992693c20c does fix the problem I was encountering.
I did notice 3 minor issues when reading through reader.go/core.go.

In restart, if startCommand returns nil, nil then fin is never called. This means the coordinator routine never receives a EvtReadFin event. So, if exec.Cmd.Start fails, no future reload commands will ever be run. I verified this in a contrived way by entering a query, moving my shell, attempting to modify the query (input did not reload as expected), moving my shell back, and then further modifying the query (still the input did not reload even with valid shell). exec.Cmd.Start is unlikely to fail but in the case that it does I think that fin should be called to properly shutdown the event poller and allow future input reloading.

There is still a race condition with ReadSource when !streamingFilter.
https://github.com/junegunn/fzf/blob/d938fdc496ccecfe5d747500927b675d31e1835a/src/core.go#L170-L176
This race condition is much less likely to cause problems then the original one, but still exists. It can be simulated by adding a time.Sleep(time.Second) to ReadSource and then entering a query when fzf launches. When ReadSource wakes up, the reader's termFunc will have already been set to nil. fzf is then in a state where the item list will not be updated until either the initial command completes or the query is modified.

In action:
fzf --disabled --bind "start:reload:echo start command; sleep 1d" --bind "change:reload:echo reloaded input with query {q}"

https://github.com/user-attachments/assets/10ae13cf-67bd-4d06-a1f7-ffd4f2b280d6

When piping data to fzf and then trying to reload the input, the Read call in feed will not return until more data is sent to stdin. This means that when piping data to fzf from a long running program that only occasionally (or never) produces output, attempting to reload input will fail for potentially long periods of time.

Here is a contrived example:
(while true; do echo stdin data; sleep 10; done) | fzf --disabled --bind "change:reload:echo reloaded with: {q}"

https://github.com/user-attachments/assets/b247a46e-710c-47bb-80ca-6c4e81840c08
This is because the stdin file is in blocking mode. Calling util.SetNonblock(os.Stdin, true) before readFromStdin and then appropriately handling syscall.EAGAIN errors from Read seemed to fix the issue on my system.

Thank you very much for the detailed analysis. I'll address the issues and let you know of the progress.

I believe 19495eb addresses the first two issues you mentioned. It restored the previous code structure, but added a barrier using a channel to ensure that the process has started.
I'm not going to think too much about the third issue for now, as I'm afraid trying to fix it might cause other side effects (e.g. affecting the ingestion performance) and we can work around it using a start:reload binding.
fzf --disabled \
    --bind 'change:reload:echo reloaded with: {q}' \
    --bind 'start:reload:while true; do echo stdin data; sleep 10; done'

Thanks for your feedback!

Hmm, the last commit has a bug that can freeze fzf when a process failed to start. Let me see..

Thank you!


recent eslint & prettier changes are clashing over " and '

it seems that we still have some lingering issues with formatting & eslint
prettier expects imports to use " double quotes, but eslint wants single quotes for everything '. if I run a prettier fix on a file, it switches the import to use ", but then eslint fails on this change because it wants single quotes
it seems we need to address this regression that was recently introduced - previously we had a check to ensure that prettier and eslint didn't have clashing configs, it seems that was removed?

Prettier doesn't expect double quotes, it uses single quotes everywhere https://github.com/graphql/graphiql/blob/9a38de29fddf174ba9e793ac5852407537244f87/.prettierrc#L2

the clashing I think I saw was over arrow functions and use of parenthesis actually, i think prettier calls this arrowParens setting, either way I will close this until it seems to be a real issue. I was having this issue only in the github online vscode instance and not on my local, when running "format file", so I think it was because the commits were bypassing the githooks


Better exceptions

Starts on this issue:
Prevents dumping very long stack to the user.
With:
Registering machine group configurations:
Error registering the machine group.
Failed with exception:
 (403)
Reason: Forbidden
HTTP response headers: HTTPHeaderDict({'content-type': 'application/json', 'X-Cloud-Trace-Context': 'e4e6cd3a8faa07f6912d78dcdc5a598b', 'Date': 'Thu, 25 Jan 2024 17:17:20 GMT', 'Server': 'Google Frontend', 'Content-Length': '96'})
HTTP response body: b'{"detail":"Maximum allowed number of machines is exceeded. Maximum allowed: 10. Requested: 80."}'

Without:
Registering machine group configurations:
Traceback (most recent call last):
  File "/Users/augustoperes/Documents/inductiva/../inductiva_test_scripts/launch_too_many_cores.py", line 3, in <module>
    mg = inductiva.resources.MachineGroup(machine_type="c2-standard-16",
  File "/Users/augustoperes/Documents/inductiva/inductiva/resources/machines.py", line 49, in __init__
    self._register_machine_group(num_vms=self.num_machines,
  File "/Users/augustoperes/Documents/inductiva/inductiva/resources/machines_base.py", line 83, in _register_machine_group
    resp = self._api.register_vm_group(body=instance_group_config)
  File "/Users/augustoperes/Documents/inductiva/inductiva/client/paths/compute_group/post.py", line 303, in register_vm_group
    return self._register_vm_group_oapg(
  File "/Users/augustoperes/Documents/inductiva/inductiva/client/paths/compute_group/post.py", line 219, in _register_vm_group_oapg
    raise exceptions.ApiException(status=response.status,
inductiva.client.exceptions.ApiException: (403)
Reason: Forbidden
HTTP response headers: HTTPHeaderDict({'content-type': 'application/json', 'X-Cloud-Trace-Context': '9a0153fca058e663cf6da096de30ace5', 'Date': 'Thu, 25 Jan 2024 17:15:38 GMT', 'Server': 'Google Frontend', 'Content-Length': '96'})
HTTP response body: b'{"detail":"Maximum allowed number of machines is exceeded. Maximum allowed: 10. Requested: 80."}'

QA testing.

Install from this branch;
Launch a script that will fail, for example:

import inductiva

mg = inductiva.resources.MachineGroup(machine_type="c2-standard-16",
                                      num_machines=2,
                                      disk_size_gb=70,
                                      register=True)
mg.start()

mg = inductiva.resources.MachineGroup(machine_type="c2-standard-16",
                                      num_machines=80,
                                      disk_size_gb=70,
                                      register=True)
mg.start()

The first machine group should be launched just fine.
The second should display the error:
Registering machine group configurations:
Registering machine group failed with exception (403)
Reason: Forbidden
HTTP response headers: HTTPHeaderDict({'content-type': 'application/json', 'X-Cloud-Trace-Context': '5418a1ae0ac566c3b4a1c0fe291ea9f9;o=1', 'Date': 'Fri, 26 Jan 2024 10:55:21 GMT', 'Server': 'Google Frontend', 'Content-Length': '96'})
HTTP response body: b'{"detail":"Maximum allowed number of machines is exceeded. Maximum allowed: 10. Requested: 80."}'

There should also exist a file rotated.log(in the directory from which you launched the script with):
2024-01-26 10:59:01|INFO|root|machines_base.py:82 Registering machine group configurations:
2024-01-26 10:59:03|INFO|root|machines_base.py:217 > Name: api-3rmf8k3o5ssjkbgzd6y89ewh3
2024-01-26 10:59:03|INFO|root|machines_base.py:218 > Machine Type: c2-standard-16
2024-01-26 10:59:03|INFO|root|machines_base.py:219 > Disk size: 70 GB
2024-01-26 10:59:03|INFO|absl|machines.py:78 > Number of machines: 2
2024-01-26 10:59:03|INFO|absl|machines.py:79 > Spot: False
2024-01-26 10:59:03|INFO|absl|machines.py:93 Estimated cloud cost for all machines : 1.838 $/h
2024-01-26 10:59:03|INFO|root|machines_base.py:136 Starting machine group. This may take a few minutes.
2024-01-26 10:59:03|INFO|root|machines_base.py:138 Note that stopping this local process will not interrupt the creation of the machine group. Please wait...
2024-01-26 10:59:27|INFO|root|machines_base.py:151 Machine group successfully started in 0:00:24.
2024-01-26 10:59:27|INFO|root|machines_base.py:82 Registering machine group configurations:
2024-01-26 10:59:28|ERROR|root|log.py:17 Registering machine group failed with exception (403)
Reason: Forbidden
HTTP response headers: HTTPHeaderDict({'content-type': 'application/json', 'X-Cloud-Trace-Context': 'faf0b9b9189332476f6d65c5bf6524e1;o=1', 'Date': 'Fri, 26 Jan 2024 10:59:28 GMT', 'Server': 'Google Frontend', 'Content-Length': '96'})
HTTP response body: b'{"detail":"Maximum allowed number of machines is exceeded. Maximum allowed: 10. Requested: 80."}'

It seems good to me! But just to understand better: is it automatic that it sends to the two different loggers now: stdout and file?

Yes

QA
The purpose of the PR is handled correctly. When starting a machine with incorrect parameters, I get the following error:

Then, for the registering I was able to get "Unhandled Exception" when inserting wrong value types:

I guess this will need to be dealt in the Client, since I was able to register successfully a machine with num_machines=-1.

I guess this will need to be dealt in the Client, since I was able to register successfully a machine with num_machines=-1.
@IvanPombo please open a bug


Error in fine-tuning

Hi,
Thanks for the nice repo.
I was trying to do fine-tuning on pre-trained model and used the code like below.
import torch
from enformer_pytorch import Enformer
from enformer_pytorch.finetune import HeadAdapterWrapper

enformer = Enformer.from_pretrained('EleutherAI/enformer-official-rough')

model = HeadAdapterWrapper(
    enformer = enformer,
    num_tracks = 128,
    post_transformer_embed = False   # by default, embeddings are taken from after the final pointwise block w/ conv -> gelu - but if you'd like the embeddings right after the transformer block with a learned layernorm, set this to True
).cuda()

seq = torch.randint(0, 5, (1, 196_608 // 2,)).cuda()
target = torch.randn(1, 200, 128).cuda()  # 128 tracks

loss = model(seq, target = target)
loss.backward()

But it has an error like below.
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
[<ipython-input-7-369343718439>](https://localhost:8080/#) in <module>
----> 1 loss = model(seq, target = target)
      2 loss.backward()

8 frames
[/usr/local/lib/python3.8/dist-packages/enformer_pytorch/modeling_enformer.py](https://localhost:8080/#) in forward(self, x)
    172
    173         if seq_len < target_len:
--> 174             raise ValueError(f'sequence length {seq_len} is less than target length {target_len}')
    175
    176         trim = (target_len - seq_len) // 2

ValueError: sequence length 768 is less than target length 896

Error seems to come from the different length of sequences so I put the 229_376 to make 896 but it's also not working. Do you know any solution for this?
Thanks,

oh yes, I see how that is confusing, will fix tomorrow morning
can you try invoking https://github.com/lucidrains/enformer-pytorch/blob/main/enformer_pytorch/modeling_enformer.py#L370 with the enformer instance before passing it into the head adapter?

@joonan30 hey Joon, your example can be fixed with the addition of a single line before passing the enformer into the HeadAdapter
import torch
from enformer_pytorch import Enformer
from enformer_pytorch.finetune import HeadAdapterWrapper

enformer = Enformer.from_pretrained('EleutherAI/enformer-official-rough')

enformer.set_target_length(200) # <-- add this

model = HeadAdapterWrapper(
    enformer = enformer,
    num_tracks = 128,
    post_transformer_embed = False   # by default, embeddings are taken from after the final pointwise block w/ conv -> gelu - but if you'd like the embeddings right after the transformer block with a learned layernorm, set this to True
).cuda()

seq = torch.randint(0, 5, (1, 196_608 // 2,)).cuda()
target = torch.randn(1, 200, 128).cuda()  # 128 tracks

loss = model(seq, target = target)
loss.backward()

i think i'll offer a mode on the enformer so it can automatically set the length on forward

@joonan30 hey Joon, in 0.6.2, you should be able to run your original script just fine (it will auto set the target length)


Add a new command: UnsupportedCommand to communicate compatibility of JGit

This command serves as a way to communicate to the git plugin which git features
are not supported by JGit.

The reason for not implementing it as a GitCommand implementation is to separate
the need of a JGitAPIImpl or CliGitAPIImpl to check if a certain additional behaviour
of git is supported or not.
This class can be instantiated with no need of any information, it can be decorated by a
GitSCMExtension to fill in the features which are not supported.

How to use it in the Git Plugin?

Step 1: Define a function within the GitSCMExtension to be implemented by extensions which are not supported by JGit.
ex: public void determineSupportForJGit(GitSCM scm, UnsupportedCommand unsupportedCommand) {   }
Step 2: Let's take GitLFSPull as an example, implement a function called:

    public void determineSupportForJGit(GitSCM scm, UnsupportedCommand cmd) {
        List<RemoteConfig> repos = scm.getRepositories();
        cmd.lfsRemote(repos.get(0).getName());
}

Step 3: Use the collected information to determine if JGit can be supported or not using the UnsupportedCommand only.

Checklist

[x] I have read the CONTRIBUTING doc
[ ] I have referenced the Jira issue related to my changes in one or more commit messages
[ ] I have added tests that verify my changes
[ ] Unit tests pass locally with my changes
[ ] I have added documentation as necessary
[ ] No Javadoc warnings were introduced with my changes
[ ] No spotbugs warnings were introduced with my changes
[ ] I have interactively tested my changes

Types of changes
What types of changes does your code introduce? Put an x in the boxes that apply

[x] New feature (non-breaking change which adds functionality)

@MarkEWaite Many thanks for writing the test cases, I was working on the git plugin PRs and couldn't add those in time.
While the automated cases written should be more than enough to cover tests functionality wise, I believe we may have to add more test cases within the git plugin to check if the UnsupportedCommand is able to determine support after being used by various extensions.

@rishabhBudhouliya I saw that the message associated with the first commit in this pull request was missing the blank line after the first line.  That causes the log output of command line git to be "noisy".  I modified the commit message to add that blank line so that the log output would be more tidy.  I also added javadoc comments for all the public methods of the UnsupportedCommand class so that we don't have javadoc warnings added to the build.
I believe this is ready to merge once the CI job passes.


yt-dlp does not always download highest quality video from Funimation

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I remove or skip any mandatory* field

Checklist

[X] I'm reporting a broken site
[X] I've verified that I'm running yt-dlp version 2022.09.01 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
USA
Provide a description that is worded well enough to be understood
yt-dlp does not always download highest quality video from Funimation. For example, when downloading this episode of Code Geass, it downloads the 540p version instead of the 1080p version that is available:
https://www.funimation.com/en/shows/code-geass/turn-24-the-grip-of-damocles
The downloaded file is around 360 MB. The expected file size is well over 1 GB, judging by the few episodes that did download in 1080p.  Why is that?
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['-u', 'PRIVATE', '-p', 'PRIVATE', '--extractor-args', 'funimation:language=english', '--cookies-from-browser', 'chrome', '--no-check-certificate', '-vU', 'https://www.funimation.com/en/shows/code-geass/turn-24-the-grip-of-damocles']
[debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.09.01 [5d7c7d6] (win32_exe)
[debug] Python 3.8.10 (CPython 64bit) - Windows-10-10.0.19041-SP0
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] exe versions: none
[debug] Optional libraries: Cryptodome-3.15.0, brotli-1.0.9, certifi-2022.06.15, mutagen-1.45.1, sqlite3-2.6.0, websockets-10.3
[Cookies] Extracting cookies from chrome
[debug] Extracting cookies from: "C:\Users\Tanner\AppData\Local\Google\Chrome\User Data\Default\Cookies"
[debug] Found local state file at "C:\Users\Tanner\AppData\Local\Google\Chrome\User Data\Local State"
[Cookies] Extracted 2157 cookies from chrome
[debug] cookie version breakdown: {'v10': 2157, 'other': 0, 'unencrypted': 0}
[debug] Proxy map: {}
[debug] Loaded 1670 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version: 2022.09.01, Current version: 2022.09.01
yt-dlp is up to date (2022.09.01)
[funimation:page] Logging in
[debug] [funimation:page] Extracting URL: https://www.funimation.com/en/shows/code-geass/turn-24-the-grip-of-damocles
[funimation:page] code-geass_turn-24-the-grip-of-damocles: Downloading JSON metadata
[debug] [Funimation] Extracting URL: https://www.funimation.com/player/156923
[Funimation] 156923: Downloading player webpage for 156923
[Funimation] turn-24-the-grip-of-damocles: Downloading Uncut english (156923) JSON
[Funimation] turn-24-the-grip-of-damocles: Downloading Uncut english (156923) m3u8 information
[debug] Sort order given by extractor: lang, source
[debug] Formats sorted by: hasvid, ie_pref, lang, source, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), channels, acodec, filesize, fs_approx, tbr, vbr, abr, asr, proto, vext, aext, hasaud, id
[debug] Default format spec: best/bestvideo+bestaudio
[info] 156922: Downloading 1 format(s): 156923-mp4
[debug] Invoking http downloader on "https://video.prd.funimation.com/FunimationStoreFront/V1691872/5b5d5d32-5521-46ef-9d31-ac0a6c1f86e7_index.mp4?t=exp=1662774954~acl=https://video.prd.funimation.com/FunimationStoreFront/V1691872/*~hmac=1719a7bfc79d9c6527017f660aabf662ded934ad4ba59a54111c59d609260ed1"
[debug] File locking is not supported. Proceeding without locking
[download] Destination: Turn 24 - The Grip of Damocles [156922].mp4
[download] 100% of 354.68MiB in 00:08 at 40.99MiB/s

Install ffmpeg https://github.com/yt-dlp/yt-dlp#strongly-recommended

Coulda sworn I had it installed already, huh. That seems to have done the trick! Thanks! Sorry


[Security Assessment] PURIS R24.03

Security Assessment Request for Eclipse Tractus-X
Component/Feature

PURIS (Business Application)
Reason

Preparational Assesment was for R23.12 (see PR)
Scope

Same as previously
Timeline

I won't be available between 2024-02-08 and 2024-02-15.
Meetings early in the morning similar to last time are fine. (e.g. tomorrow or wednesday 7 o'clock, Friday 2024-02-16 7.15 o'clock)

Thanks for notification Tom.
I will schedule a meeting for Friday 2024-02-16 7.15 o'clock.
Enjoy the holidays.
Cheers ;)

Data Flow Diagram PURIS
Below diagram was designed with purpose to pull it with the security assessment documentation crated by Kristian
I just wanted to let you know that I have finished the diagram. I will request for a pull once Kristian's security assessment file will be merged into the repo.
Please feel free to review the diagram and let me know in case I would have to make some changes.
flowchart TD
    A(Customer \n Human User)
    A2(Puris Endpoint App \n Data Provider \n Out of Scope)
    A3(EDC \n Eclipse Data Space Components Connector \n C-X Member \n Out of Scope)

    B(EDC \n ECLIPSE DataSpace Components Connector \n C-X Operator \n Out of Scope)

    C(Vue User Interface)

    D6(Master Data Controller \n Exposed for external system based on API Keys \n Out of scope)
    D7(Product Measures Visualization \n Just visualization of the data \n Out of scope - Not yet developped)

    D(Stock View Controller)
    D2(EDC View Controller)
    D3(Data Request Controller)
    D4(Data Response Controller)
    D5[(PostgreSQL DB \n Main Database)]

    A-->|Main functionality is possibility to view & manage stocks with Business Partners. \n Enter stock information manually. \n View the Supply Dashboard to check a supply situation between Partner & Customer \n for Partner and Supplier. \n HTTPS Protocol|C
    A2-->|Providing Customer data on stocks & supplies \n HTTPS Protocol|A3

    C-->|Read & Write access|D
    C-->|Read access|D2

    A3-->|View Data \n Negotiate Contracts \n Initialize & Perform Data Transfers \n HTTPS Protocol|B

    D-->|Data Read & Write \n TCP9092|D5
    D3-->|Data Read & Write \n TCP9092|D5

    D4-->|Data Read & Write \n TCP9092|D5

    D3-->|Forwarding Data Response \n HTTPS Protocol|B

    B-->|Forwarding Data Requests \n HTTPS Protocol|D3
    B-->|List catalog negotiations and transfers \n Read Access \n HTTPS Protocol|D2

    B<-->|Forwarding Data Response \n HTTPS Protocol|D4

    subgraph Internet Boundary
    A

        subgraph Customer Environment

         A2
         A3

         end

end

subgraph Catena - X Environment

B

    subgraph PURIS Product

        subgraph PURIS Product - View Frontend
         C
         end

         subgraph PURIS Product - Java Backend
         D6
         D7
         D2
         D3
         D
         D5
         D4
          end

    end

    end

Assessment completed and documented in the GitHub Repo for PURIS Product.
Closing the task.


Spelling mistake

'Tams' should be 'Teams'

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

ID: b46d058a-2546-b917-1aff-2d15cf0ea76e
Version Independent ID: 0c35bbb1-08a3-a26f-47de-ff5381522fe8
Content: Release notes and latest version of the CoE starter kit Core components for Teams 1.7 - Power Platform
Content Source: power-platform/guidance/coe/release-notes/coreteams-1.7.md
Service: power-platform
GitHub Login: @manuelap-msft
Microsoft Alias: mapichle

I fixed and created a PR in the private repo. https://github.com/MicrosoftDocs/power-platform-pr/pull/3097


Update attached-properties-overview.md

Fixes indentation for C#, VB and C++/CX. Also removes dev langs as they were not working correctly and just shows the examples in all the languages by default.

It looks like the working branch that was used to create the pull request has been deleted, so a commit can't be added to the pull request. We'll close the PR. Please create a new pull request to update the article per the author's feedback. Thanks.


Api key

Hey, I have an api problem, after generating it and entering it into config when starting the application
teamspeak@xxxx:~/anura$ java -jar web-1.0-SNAPSHOT-jar-with-dependencies.jar
[INFO] de.anura.bot.BotStartupKt - Your Steam API Key is valid

[steam] ; The Steam web api key is used for getting the games of a user ; You can get one here: http://steamcommunity.com/dev/apikey api_key = 26CC97D49BFF6C26DF37E5E80xxxxxx

The message
[INFO] de.anura.bot.BotStartupKt - Your Steam API Key is valid

indicates that your Steam API key is correct (= valid) and can be used. So, everything should be fine.


OpenRGB issues

Describe the bug
The app's autostart functions do not work and the tray icon title is "AppRun.Wrapped". You can add it to the system's autostart, but that does not provide the apps autostart functions.
What did you expect to happen?
I expected the "start at login" option to make the app start with the OS, but it didn't.
I expected the "start minimized" option to make the app start minimized, but it didn't.
I expected the "load profile" option to automatically load a profile on start, but it didn't.
Output of rpm-ostree status
State: idle
Deployments:
  ostree-image-signed:docker://ghcr.io/ublue-os/bazzite:stable
                   Digest: sha256:8833376d2c232931a91e85397072f75406099f24970f13902846eca3acccc394
                  Version: 40.20240807.0 (2024-08-07T07:31:29Z)
            SecAdvisories: 1 unknown severity, 1 important
                     Diff: 79 upgraded, 2 removed, 11 added
          LayeredPackages: goverlay
            LocalPackages: lact-0.5.5-0.x86_64

● ostree-image-signed:docker://ghcr.io/ublue-os/bazzite:stable
                   Digest: sha256:c58e6c54668614acc5620403a2daac520559f9d6fab9ba2c3ad1a07f82353a9d
                  Version: 40.20240803.0 (2024-08-04T06:31:02Z)
          LayeredPackages: goverlay
            LocalPackages: lact-0.5.5-0.x86_64

  ostree-image-signed:docker://ghcr.io/ublue-os/bazzite:stable
                   Digest: sha256:c58e6c54668614acc5620403a2daac520559f9d6fab9ba2c3ad1a07f82353a9d
                  Version: 40.20240803.0 (2024-08-04T06:31:02Z)
          LayeredPackages: goverlay
            LocalPackages: lact-0.5.5-0.x86_64

Hardware
No response
Extra information or context
No response

We don't write this software, you'll want to open an issue with them

But this happens specifically only in this distro. Or at least neither in Manjaro, EndeavourOS, nor Nobara.

But this happens specifically only in this distro. Or at least neither in Manjaro, EndeavourOS, nor Nobara.

Just reproduced this on a stock Fedora 40 VM. Not immutable. You can note that in your issue w/ OpenRGB.


Feature request? tree component?

I think that would be a great addition

Thanks. Can you go into more details to describe what it would be? I'm not entirely sure what you mean by a tree component. Are there examples in the wild? Are there a11y concerns we need to account for in designing one? What would be the ideal features?

Closing this issue as there was no response


Samsung SM-G965F

I got a message saying there was a login through a Samsung SM-G965F. Is this normal. Do you run a virtual machine (something like BlueStacks) in order to stream to Instagram?

Internally we do use a device Id to communicate with Instagram APIs, so it's common to recieve an email from Instagram about the login. But most of the time it should be a Xiaomi MI device. So, if you feel like something is not right, please change your password immediately. The app only communicates your credentials to Instagram and it's not stored anywhere else other than your PC. Make sure the mail you recieved have the correct geo location mentioned as your current location, if not please change your credentials.

Internally we do use a device Id to communicate with Instagram APIs, so it's common to recieve an email from Instagram about the login. But most of the time it should be a Xiaomi MI device. So, if you feel like something is not right, please change your password immediately. The app only communicates your credentials to Instagram and it's not stored anywhere else other than your PC. Make sure the mail you recieved have the correct geo location mentioned as your current location, if not please change your credentials.

Oh ok thanks for letting me know! The location and time seems to be accurate from when I signed into it. I was just a little startled when I saw the device model since I don’t own that one..

On 23 Jan 2021, at 18:53, Musthaq Ahamad<EMAIL_ADDRESS>wrote:
communicate

Oh ok thanks for letting me know! The location and time seems to be accurate from when I signed into it. I was just a little startled when I saw the device model since I don’t own that one..

On 23 Jan 2021, at 18:53, Musthaq Ahamad<EMAIL_ADDRESS>wrote:
communicate


v86 won't build on Ubuntu WSL2

Here's the log:
mkdir -p build/
BLOCK_SIZE=K ls -l build/v86-debug.wasm
ls: cannot access 'build/v86-debug.wasm': No such file or directory
make: [Makefile:182: build/v86-debug.wasm] Error 2 (ignored)
cargo rustc --target wasm32-unknown-unknown -- -C linker=tools/rust-lld-wrapper -C link-args="--import-table --global-base=4096 " -C link-args="build/softfloat.o" -C link-args="build/zstddeclib.o" --verbose -C target-feature=+bulk-memory
**Segmentation fault**
make: *** [Makefile:183: build/v86-debug.wasm] Error 139

I believe maybe this is a issue with WSL.
Some host system info:
Microsoft Windows [Version 10.0.22000.1455]
CPU is Intel(R) Core(TM) i3-10110U CPU @2.10GHz

Can you build rust applications at all? (e.g. cargo init && cargo rustc)?

I got Segmentation fault.

It looks like an issue with your wsl or rust install then. I'd try wiping it, or perhaps trying with a different distribution or installing a different version of rust. However, since the problem is not v86-specific, I'll close this issue.


Jonmv/separate clustercontroller cluster

@hmusum please review.
I need some unit tests, and an actual check of whether an app is "in sync", using the /cluster/v2 API on the CC.
I'm also going to remove support for the <controllers> element under <content>, which is unused.

Thanks! Nice catch.

Lots of tests, and a little fix in NodeSpecification.

4 last commits are new since you reviewed. Conflict, so had to force-push.

Yes, we need that. I also think an integration test is the right way, since we're not keeping the state.


Custom nodes request: KJNodes for rgb masks

It would be great to have KJNodes included as it has the mask tools like rgb to mask that is used with AnimateDiff videos.
https://github.com/kijai/ComfyUI-KJNodes
Nice to have additions would be:
Rgthree-comfy mutie and bypass node utilities
https://github.com/rgthree/rgthree-comfy/tree/main
cg-use-everywhere seed everywhere node is damn useful when there is upscalers in the workflow. Unless there already is a node that does the same.
https://github.com/chrisgoringe/cg-use-everywhere

I'm looking into adding https://github.com/kijai/ComfyUI-KJNodes

Great to hear!
IPAdapter was updated again with some new nodes and one is RGB mask node. I have not tested it yet, but it could work as KJNodes RGB mask works. KJNoses one is not limited to just red green and blue so i have used it with 6 color masks.


Task/create view estudios pendientes

What's this PR do?
It displays the list of socio-economic studies under review and pending
in the dashboard and test the views to be displayed correctly.
Where should the reviewer start?
estudios_socioeconomicos/views.py
estudios_socioeconomicos/test_views.py
estudios_socioeconomicos/urls.py
This is optional if you want to check the data shown in each template.
templates/estudios_socioeconomicos/pendientes.html
templates/estudios_socioeconomicos/revision.html
How should this be manually tested?
python manage.py test estudios_socioeconomicos
Any background context you want to provide?
N/A
This template was adapted ~stolen~ from Quickleft/Sprint.ly

Tests are failing due to the decorators, you should not make a pull request of something that is not tested.

Codecov Report

Merging #31 into develop will increase coverage by 0.08%.
The diff coverage is 99.14%.

@@             Coverage Diff             @@
##           develop      #31      +/-   ##
===========================================
+ Coverage    95.28%   95.36%   +0.08%
===========================================
  Files           88       88
  Lines         1208     1381     +173
===========================================
+ Hits          1151     1317     +166
- Misses          57       64       +7

Impacted Files
Coverage Δ

administracion/test_views.py
100% <100%> (ø)
:white_check_mark:

administracion/urls.py
100% <100%> (ø)
:white_check_mark:

administracion/views.py
75% <77.77%> (-25%)
:x:

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 58e7d67...c3bd09f. Read the comment docs.


DBConnection.Ownership

[x] Initial working implementation with passing tests
[x] Support ownership mode
[x] Use supervisors
[x] Cache with ETS
[x] Docs
[x] Add ownership specific tests

Shall we merge this and we can work on the remaining tasks in separate PRs or would you like to continue with this branch?

Checkin/stop/disconnect should be casts to pids so they dot crash if the pool is gone.
We starting the owner process we should link to the manager until the reply is sent. This ensures the manager goes down if a reply is not sent. When sending a reply from another process it should always be linked directly or indirectly via the supervision tree so that the target of the call exits if a reply isn't sent.

@fishcakez I have made checkin async but ran into issues for disconnect and stop. Can you please give it a try? Everything else is done. :bowtie:

:heart_eyes_cat: great stuff!! I will tag 0.2 in the morning.

Thanks for the remaining fixes! Beautiful!


install precommit

add pre-commit==2.7.1 to requirements.txt

I had to install tensorboard

That should be added to the requirements as part of your story


Importing imp in standard_library raises a PendingDeprecationWarning

The imp package is pending deprecation in favor of importlib as of Python 3.4.
$ python3.5 -Wonce -c "import future.standard_library"
[...]/lib/python3.5/site.py:165: DeprecationWarning: 'U' mode is deprecated
  f = open(fullname, "rU")
[...]/lib/python3.5/site-packages/future/standard_library/__init__.py:66: PendingDeprecationWarning: the imp module is deprecated in favour of importlib; see the module's documentation for alternative uses
  import imp

The recommended alternative is described here.

Any hope to get this fixed? Still happening after 3 years.

Assuming no one has plans to fix this? Currently tracking the source of a DeprecationWarning and found this along the way.

Greetings again from the future. We're now in the days of Python 3.11, with Python 3.12's release looming ever closer. There is an open PR (#574) from 2019 to close this issue.
Is python-future dead? Once Python 3.12 releases, this project will be left in a forever broken state if not updated.
Please merge the pending PR.

The imp module has now been removed from Python 3.12.

https://github.com/python/cpython/issues/98040
https://github.com/python/cpython/pull/98573

Python 3.12 will be in beta in two weeks, and set for release on 2023-10-02 and this library is one of the top 5,000 most-downloaded from PyPI.
Please could you upgrade to use importlib? The imp docs have suggestions on what to use to replace each function and constant.

For people googling, in python 3.12, importing future.standard_library raises “ModuleNotFoundError: No module named 'imp'”:
ImportError: Failed to import test module: myfile
Traceback (most recent call last):
  …
  File "myfile.py", line 41, in <module>
    from future import standard_library
  File "/opt/hostedtoolcache/Python/3.12.0/x64/lib/python3.12/site-packages/future/standard_library/__init__.py", line 65, in <module>
    import imp
ModuleNotFoundError: No module named 'imp'

In my case, I resolved the error by replacing those imports that need future.standard_library.install_aliases() with six instead (see imports2):

urllib.parse --> six.moves.urllib.parse
urllib.error --> six.moves.urllib.error
urllib.parse -->six.moves.urllib.parse
dbm.ndbm --> six.moves. dbm_ndbm
http.server --> six.BaseHTTPServer, six.CGIHTTPServer, or six.SimpleHTTPServer
tkinter.filedialog --> six.tkinter_tkfiledialog
tkinter.simpledialog --> six.tkinter_simpledialog
xmlrpc.server --> six.xmlrpc_server


Feat: Convert to stake modal

What is the purpose of the change

Similar to the upgrades flow, this feature provides the user with a modal accessible from the pools page that allows them to select share pools that are earning a lower APR then the staking APR. From that list, the user can select a share pool to migrate via a custom ConvertToStake message that handles the migration of all forms of gamm shares (available, bonded, unbonding, SF bonding & SF unbonding).

NOTE: blocked on working with the chain devs on issues serializing lockIds with a value of 0, as we had with the migrate position message.
ClickUp Task
ClickUp Task URL
Brief Changelog

Adds a UserConvertToStakeConfig that encapsulates all the logic around querying for stake opportunities, selecting the desired pool, as well as sending the respective stake opportunity via transaction to the chain
Adds a modal that depends on the above ConvertToStake UI config. If the user has not selected a validator set preference, the modal will jump to the validator selection modal before sending the transaction.

Testing and Verifying

[x] Works with available GAMM shares
[x] Works with bonded and unbonding superfluid shares
[x] Works with bonded and unbonding normal GAMM shares
[ ] FIX: refreshing data after converting; pools still showing

Successful un bond and stake for locked shares initiated from frontend:
https://www.mintscan.io/osmosis/transactions/5AE07FA3B61943145FAB7160C8F378539A9DA8789D7FA43654A37970140659E1
Documentation and Release Note

The message and valsetprefs functionality is already present on mainnet and other dev chains.

it currently sends users to the choose superfluid validator modal, even if they already have some OSMO staked (and so should be able to just use the valset prefs msg)
The modal currently says "select superfluid validator", but this isn't for superfluid

@sunnya97

It currently should directly use valsetpref it one is present
I'll fix this
This is left over from prev design. Should I remove it for all use cases?

You should still be able to use the valset prefs stake msg, as long as they have any delegations (even if not via a set valset pref)
👍
Yeah I think so.  It looks weird.


Support commit_description parameter in push_to_hub

Support commit_description parameter in push_to_hub.
CC: @Wauplin

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


feature: not add input listener onBlur and onSubmit modes.

Related #69

awesome thank you 👍

this repo is young, please consider supporting by star the repo and share around the community 🙏


disable-note-block

Timings link
can't be generated
Stack trace
https://gist.githubusercontent.com/100petr/ad0aabe41b5dd02c34c2fcdb40aa2757/raw/5d9ed4033c29a5a8015f2d25a4a70d7085a7d500/gistfile1.txt```

### Actions to reproduce (if known)

1. In purpur.yml, set in blocks.disable-note-block-updates: true
2. Restart the server
3. Paste schematic https://filebin.net/tolbde4hxniip118

### Purpur version

Checking version, please wait...
[15:30:34 INFO]: This server is running Purpur version git-Purpur-1374 (MC: 1.17.1) (Implementing API version 1.17.1-R0.1-SNAPSHOT) (Git: 8250199 on ver/1.17.1)
[15:30:34 INFO]: You are running the latest version

### Agreements

- [X] I am running the latest version of Purpur available from https://purpur.pl3x.net/downloads.
- [X] I have searched for and ensured there isn't already an open issue regarding this.
- [X] My version of Minecraft is supported by Purpur.

### Other

_No response_

The issue tracker is for reporting bugs, this is not one. Plus, you have already asked for support and been given an answer in our discord.


Quickstart and data_tools broken in README.

Thanks for raising this issue. I have corrected the docs and now all commandlines in the documentation should work as described, and all broken links should be fixed.


Update 13.8.1

This PR bump djs deps to 13.8.1, had some issues with 13.8.0
When some shards reconnecting and the client suddenly leaving from the voice channels

i got sidetracked and completely forgot about this, thanks!


Wrong determinant results for large batch

Description
The bug can be reproduced using the following code:
import jax.numpy as jnp
import jax.random as jr
import matplotlib.pyplot as plt

a = jr.normal(jr.key(0), (1500000, 20, 20))
d = jnp.linalg.det(a)
plt.plot(d)

The values at the end are obviously incorrect.
I have tested different batch and matrix sizes. The bug only happens when the full matrix size (in the example 1500000*20*20) exceeds 2**29. As also shown in the figure, the determinant values are wrong for batch index > 2**29 / (20*20).
I also tested on different devices. This bug happens on different types of GPUs while not on the CPU.
System info (python version, jaxlib version, accelerator, etc.)
jax:    0.4.35
jaxlib: 0.4.35
numpy:  2.0.2
python: 3.12.4 | packaged by Anaconda, Inc. | (main, Jun 18 2024, 15:12:24) [GCC 11.2.0]
device info: NVIDIA A100 80GB PCIe-1, 1 local devices"
process_count: 1
platform: uname_result(system='Linux', node='alcc145', release='5.15.0-94-generic', version='#104-Ubuntu SMP Tue Jan 9 15:25:40 UTC 2024', machine='x86_64')

Thanks for the report – I also can repro this on a GPU but not CPU. It looks like the issue has something to do with failure of batched LU decomposition in this domain (LU is used to compute det) – you can see how the statistics of this distribution change where the erroneous results crop up:
import jax.numpy as jnp
import jax.random as jr
from jax import lax
import matplotlib.pyplot as plt

a = jr.normal(jr.key(0), (1500000, 20, 20))
d = jnp.linalg.det(a)
lu, _, _ = lax.linalg.lu(a)

fig, ax = plt.subplots(2, sharex=True)
ax[0].plot(d)
ax[0].set_ylabel('det(a)')
ax[1].plot(lu.std(axis=[1, 2]))
ax[1].set_ylabel('std(lu(a))');

The block of high standard deviation values also suggests something else is amiss with outputs in other parts of the batch.

Thanks for this bug report and for the simple reproduction! I've fixed (at least part of) this issue over in https://github.com/jax-ml/jax/pull/24846, which should be available in a day or two in the nightly version of jaxlib. I've added a test for the failures towards the end of the batch, but I'm interested to understand if there are other numerical issues here that we can see in Jake's plots. I'll take a closer look ASAP!

@jakevdp @dfm, Thanks for checking and fixing the bug. Regarding the high standard deviation values, I also found something weird. It can be reproduced by the following code
import jax.numpy as jnp
import jax.random as jr
import matplotlib.pyplot as plt

a = jr.normal(jr.key(0), (1500000, 20, 20))
d = jnp.linalg.det(a)
std = jnp.std(a, axis=(1, 2))
plt.plot(std)
plt.show()

But it works normally without d = jnp.linalg.det(a)

Probably it's also related to lax.linalg.lu.

Whoa! Yeah, I expect that is the same nasty overflow bug, but I'll double check. Thanks for digging!

I have confirmed that https://github.com/jax-ml/jax/pull/24846 also fixes this issue. Thanks again for the bug report!
For reference, I introduced this bug in https://github.com/jax-ml/jax/pull/23054, so this has been a problem for JAX versions 0.4.33-0.4.35. The next nightly and next official release will include this fix!
I'm going to close this issue as fixed, but please comment or open a new issue if you run into any further problems.


Support user-defined cgroups in NixOS

Ideally we should be able to declare cgroups in Nix (in a way similar to cgconfig.conf (5)) and probably reusecgconfigparser to setup the cgroup filesystem, and also support the cgrulesengd daemon which distributes processes to cgroups automatically.
This should be an easy task and has some fairly useful use cases, such as being to able to limit affinity/memory usage per user (as opposed ulimit which is per process).

Is there a status on this ticket?

(triage) Given the inactivity and that it was already bumped, can we close? @shlevy

What was already bumped? Also FWIW I've never tried using cgroups, I'd ask @PkmX if this is closable.

@shlevy The commit above me asked about the status in late 2015, but no reaction.


Issue templates for feature request and bug report

Apart from the suggestion I made, it looks all good to me!


Use null coalescing operator

The null coalescing operator saves code and reduces cognitive overload 💆
:octocat:

I ❤ code reviews!

Please update rector.php and add the following:
use Rector\CodeQuality\Rector\Ternary\ArrayKeyExistsTernaryThenValueToCoalescingRector;

$services->set(ArrayKeyExistsTernaryThenValueToCoalescingRector::class);

and run:
vendor/bin/rector process --clear-cache

to ensure all places applied for array_key_exists ternary then value coalescing transformation and rector will take care of this check in the future.

Oh, just see there is ternary_to_null_coalescing config for that, so the rector rule for it seems unneeded.
Just need rebase then.

Done! 💪


refactor: remove pretty bytes

pretty bytes show KiB, MiB, GiB... format in base 2 values, it's correct but is usual to see the format in KB, MB, GB...
Brave

Edge

Windows

Github

LGTM


[docs] Add link to live demo to KPI example

[x] I've read and followed the contributing guide on how to create great pull requests.
[x] I've updated the relevant documentation for any new or updated feature
[x] I've linked relevant GitHub issue with "Closes #"
[x] I've added a visual demonstration under the form of a screenshot or video

https://deploy-preview-2341--mui-toolpad-docs.netlify.app/toolpad/examples/npm-stats/

I have pushed a change with what makes the most sense to me. I have found it absurd not to have the step to run the example locally.

When someone visits the app from GitHub folder, the'll see the broken screenshot and will certainly miss the render hosted app.
If this can't be fixed, I think we should revert: https://github.com/mui/mui-toolpad/pull/2341/commits/efad60d865b552f43a85f1a6b0b153b34b2fac17

If this can't be fixed, I think we should revert: efad60d

We should be able to fix this by adding full URLs in the src attribute of the img tags. (The commit you mentioned isn't relevant anymore since its changes were overwritten).

@prakhargupta1 Added the full URLs, this should work once this branch is merged and a new version of the docs is deployed (the preview will be broken since the files which are renamed by this PR itself aren't deployed yet)

Ok, let's merge it then.👍


feat: Bows!

Please someone fix Velocity, Yaw, and Pitch. It was working before systems so im not sure what could've broken it.

Yay will look at this later today


Extended AppWindow titlebar top border not visible on Windows 10

Describe the bug
When having a custom titlebar using ExtendsContentIntoTitleBar the top border of the app will dissapear.
Steps to reproduce the bug

Create an empty project
Configure the titlebar

public MainWindow()
{
     AppWindow.TitleBar.ExtendsContentIntoTitleBar = true;
}

Run the app

Expected behavior
I would expect to see the top border, same way we can see it when using MainWindow.ExtendsContentIntoTitleBar.
Now that the API is supported on Windows 10 too, it should work as it's not a compatibility problem anymore.
Screenshots

This is how it looks on Windows 10.

This is how it looks on Windows 11.
NuGet package version
Windows App SDK 1.3 Experimental 1: 1.3.230202101-experimental1
Packaging type
Packaged (MSIX)
Windows version
Windows 10 version 21H2 (19044, November 2021 Update)
IDE
Visual Studio 2022
Additional context
This issue is in  WinAppSdk 1.2 as well.

https://github.com/microsoft/terminal/blob/main/src/cascadia/TerminalApp/TitlebarControl.h
The title bar implementation of windows terminal is currently working fine under Win10. But it's a real pain to strip it out.

This is also an issue in Windows 11 - when using HighContrast modes.

Even if you try to compensate for the missing top border, by adding one - there doesn't seem to be an Inactive state/event you can detect to swap out resources.


Remove most uses of boost

I removed most uses of boost's iterator_interface. This needs testing. I haven't touched BSTArray yet as that's the most complicated one and I'm unsure how to go about it.

Perhaps check with SSE and how it was removed out from there?

The SSE version never had boost added to it. I don't understand enough about BSTArray to make the proper changes to it. I might just leave it for someone else to tackle.

This isn't something that code review will really help with. Did you do some basic testing that the containers seem to work?

Not extensively, most of these containers don't have constructors defined so I can't make automatic tests for them. Besides BSTHashMap, the only way to test them is to grab an already constructed one from the game and iterate over its data. This involves making a plugin.

If you've done some basic testing, I'd be ok with merging. My concern is if this was completely blind since it's not a fix or feature but more some refactoring. That would mean holding it out until some testing could be run since it's a potentially significant change.
That said, given some testing I'll merge it in.

Ok, I tested some more using data from the game where available and all of the iteration scenarios I know of still work. I can't verify everything else as I can't come up with more advanced examples, and the lack of constructors limits things.


Base codecov poc

Change Overview

Pull request type
Please check the type of change your PR introduces:

[ ] :construction: Work in Progress
[ ] :rainbow: Refactoring (no functional changes, no api changes)
[ ] :hamster: Trivial/Minor
[ ] :bug: Bugfix
[ ] :sunflower: Feature
[ ] :world_map: Documentation
[ ] :robot: Test
[ ] :building_construction: Build

Issues

fixes #issue-number

Test Plan

[ ] :muscle: Manual
[ ] :zap: Unit test
[ ] :green_heart: E2E

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Please upload report for BASE (master@b6d930f). Learn more about missing BASE report.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##             master    #3078   +/-   ##
=========================================
  Coverage          ?   27.66%
=========================================
  Files             ?      338
  Lines             ?    22744
  Branches          ?        0
=========================================
  Hits              ?     6292
  Misses            ?    15854
  Partials          ?      598

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Fixed PHP sample app.

GRPC extension is now enabled.
"credentials" key is now added to the $ops parameter.
Updated dependencies.

I signed it!

Squashed the commits. Thx!


Hardened.nix crashes firefox

Describe the bug
Using the hardened.nix profile, my firefox (developer edition) crashes, but what is strange is, when I start it in safe mode, it works fine.
Steps To Reproduce
Steps to reproduce the behavior:

Use hardened.nix profile
Use i3 (probably is not the issue, but why not)
Try to start up firefox developer edition

Expected behavior
I expected that hardened.nix may slow down my performance, and some minor inconveniences in development, but not something such as firefox crashing.
Screenshots

Additional context
I have used  Nix/NixOS for about a month, and I am still grappling to understand the programming language. I am also a lover of security, and so I am trying to implement many security features on this distro.
Notify maintainers
@joachifm @talyz

Metadata

system: "x86_64-linux"
host os: Linux 6.1.38-hardened1, NixOS, 23.05 (Stoat), 23.05.1935.98da3dd0de6
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.13.3
channels(root): "nixos-23.05, nixos-unstable"
channels(jabbu): ""
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

jabbu@tattooine:~/ > nix-shell -p nix-info --run "nix-info -m"
these 11 paths will be fetched (53.19 MiB download, 241.63 MiB unpacked):
  /nix/store/2fr6714pqybnns0sfm9swa4dlaxg8sjz-binutils-wrapper-2.40
  /nix/store/3w02zcld9m60jkjvwmrs2c5g330qrvjq-binutils-2.40-lib
  /nix/store/5wdlffjhcr4i43sfajxcvx3lvnkryh5y-stdenv-linux
  /nix/store/88dbrfrmj0lqxwyk4q6h7w5glkgkfvmn-mpfr-4.2.0
  /nix/store/d2skw3y1jcg27ml4rgz38lrn18q0wrs1-gmp-6.2.1
  /nix/store/d9fndiing52fkalp5knfalrvlb3isi6w-gcc-wrapper-12.2.0
  /nix/store/dg38bhsf0pl4m7qp65cf3aw7xyyw5g8d-libmpc-1.3.1
  /nix/store/dx8hynidprz3kf4ngcjipnwaxp6h229f-binutils-2.40
  /nix/store/f2z9hlzn0ajm4i296r77v3ihcfi77ld8-isl-0.20
  /nix/store/hqbh8ibqaq8x6riwz48xvyx4dvvldd9f-gcc-12.2.0
  /nix/store/kv5bkjsxp75dvz31bvb4gmamycxyz959-expand-response-params
copying path '/nix/store/kv5bkjsxp75dvz31bvb4gmamycxyz959-expand-response-params' from 'https://cache.nixos.org'...
copying path '/nix/store/d2skw3y1jcg27ml4rgz38lrn18q0wrs1-gmp-6.2.1' from 'https://cache.nixos.org'...
copying path '/nix/store/3w02zcld9m60jkjvwmrs2c5g330qrvjq-binutils-2.40-lib' from 'https://cache.nixos.org'...
copying path '/nix/store/f2z9hlzn0ajm4i296r77v3ihcfi77ld8-isl-0.20' from 'https://cache.nixos.org'...
copying path '/nix/store/88dbrfrmj0lqxwyk4q6h7w5glkgkfvmn-mpfr-4.2.0' from 'https://cache.nixos.org'...
copying path '/nix/store/dx8hynidprz3kf4ngcjipnwaxp6h229f-binutils-2.40' from 'https://cache.nixos.org'...
copying path '/nix/store/dg38bhsf0pl4m7qp65cf3aw7xyyw5g8d-libmpc-1.3.1' from 'https://cache.nixos.org'...
copying path '/nix/store/hqbh8ibqaq8x6riwz48xvyx4dvvldd9f-gcc-12.2.0' from 'https://cache.nixos.org'...
copying path '/nix/store/2fr6714pqybnns0sfm9swa4dlaxg8sjz-binutils-wrapper-2.40' from 'https://cache.nixos.org'...
copying path '/nix/store/d9fndiing52fkalp5knfalrvlb3isi6w-gcc-wrapper-12.2.0' from 'https://cache.nixos.org'...
copying path '/nix/store/5wdlffjhcr4i43sfajxcvx3lvnkryh5y-stdenv-linux' from 'https://cache.nixos.org'...
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.38-hardened1, NixOS, 23.05 (Stoat), 23.05.1935.98da3dd0de6`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.13.3`
 - channels(root): `"nixos-23.05, nixos-unstable"`
 - channels(jabbu): `""`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Try a different environment.memoryAllocator.provider like libc
https://search.nixos.org/options?channel=23.05&show=environment.memoryAllocator.provider&from=0&size=50&sort=relevance&type=packages&query=environment.memoryAllocator.provider
https://github.com/NixOS/nixpkgs/blob/6e14bcc2d749af230a8b8c5abdba2fb82e42dcb3/nixos/modules/profiles/hardened.nix#L22
Usually that's the part that causes issues

@joachifm @emilazy

Yeah it's probably the allocator.
I had great plans to make the hardened profile more secure and more usable a few years ago but it fell by the wayside, sorry. Hopefully some day. As it stands you probably want to read it carefully and know what to override to use it, especially for desktop use. For instance turning off unprivileged user namespaces might be a net security regression on desktop because it can make browsers disable their sandbox.

Alternatively, build Firefox with jemallocSupport = false or try out --enable-replace-malloc. I have done the former, and it works, although I have not tried running Firefox with a hardened allocator. As for the latter, if someone tries it, and it works, it might be worth creating an issue and gathering input to see if it can be added to the default flags since if I recall correctly it just exploits linking tricks to make LD_PRELOAD work for replacing the allocator.