Document that pipenv is not designed to run as privileged user (root, administrator)

From https://github.com/pypa/pipenv/issues/3128#issuecomment-434286020

Pipenv is not supposed to run as root either. If people are doing that, they're using it wrong.

The official documentation of pipenv does not warn that pipenv is not designed to be executed as privileged user. As bare minimum, please advise consumers of pipenv, that pipenv must not be executed as root. Please also inform the user, that most container runtimes (e.g. Docker) execute all commands as root user by default. Users must switch to an unprivileged user with e.g. USER command https://docs.docker.com/engine/reference/builder/#user.

pipenv must not be executed as root

I'm not super knowledgeable about docker deployments and I agree that it is a good idea to warn against it, but I'm not sure that it must not be used as root.  Many people use pip in docker as root, and many people use pipenv install --system --deploy to deploy their dependency state to their docker system python installation.
I have no idea why this is a common practice, and it definitely continues to confuse me whenever I try to use docker, but my impression was that since containers themselves are throwaways, the tradeoff is between managing additional tools/tooling for user access management inside the container vs the infrastructure to just scrap the whole thing and start over (which is how I think people are using pipenv atm?).
On the other hand, I know security is your area of expertise, and while the commenter in question isn't a maintainer of pipenv unfortunately, I can see that they are obviously knowledgeable in this space as well. My primary concern is that this would be confusing to many users who already have established workflows around the whole python ecosystem (pipenv included) which relies on root permissions to deploy system level python packages. I think we need to be careful with wording something like this to provide a clear path forward for those users

I was puzzled by the fact, too. I was under the impression that pipenv is designed a full replacement for and wrapper around pip + venv. Both tools are designed to work as root. (Although it's discouraged to run pip install as root, but that's for different reasons.)
I mentioned containers, because I assumed that lots of people are using pipenv in containers. It's a common misconception that containers are isolated, secure enclaves. Container are not a security mechanism, containers don't contain, e.g. https://www.projectatomic.io/blog/2014/09/yet-another-reason-containers-don-t-contain-kernel-keyrings/ . Non-priviliged container processes in combination with additional safety layers like SELinux or AppArmor are pretty solid.

And presumably if you would run things inside the container as root as a normal user, it would typically be because you are too impatient to try and troubleshoot or untangle the landscape around system administration / separation inside the container. Since most users in that situation are trying to test and deploy code, rather than infrastructure, I’m thinking this is exactly the type of problem we are able to offer some serious help with (but we need to get our documentation cleaned up).
I can’t really imagine why you’d need to deploy as root, so I think you’re pretty spot on. All of that aside, it makes it basically impossible to troubleshoot anything because it tends to have many unintended consequences.

In the container case, a lot of people are both deploying and running software as root by default. After all, it's the default user/group setting in Docker. Users have to jump throw some hoops in order to deploy as non-root.

Personally, I don't think there's any reason why Pipenv shouldn't be able to be run by root. It is not encouraged, none of the maintainers use it like that, but if something does not work, it is more likely due to oversights in edge cases rather than an explicit design decision. Patches addressing them will be welcome. Don't expect active progress since none of the maintainers are very interested in the use case, but everyone is welcome to jump in and help.

+1 - I'm writing a lightweight script that needs to be running as Administrator. It's unfortunate that I can't use pipenv to manage a handful of dependencies.

This is a very old issue, and while @tiran is right about the fact that in general software should not run as root, pipenv should not install anything outside the virtualenv it creates.
People which run a virtualenv and pipenv in a container should be made aware that one can use:
pipenv lock -r > requirements.txt

and use that with the OS native pip.

@TTimo, you should be able to run pipenv as root, or Administrator on Windows. However, this is discouraged. You could end up breaking your Python installation.


Port to NetStandard, new Project Layout, and Tests run multi-platform

All project work is done.  To run Tests for supported platforms, you must use the dotnet cli.  I did not change any version numbers, and the Package Meta data is now in the package, and I didn't change that either.
Let me know if you have any other questions.  We always pair to finish this off.

Any timeline on this?


Move WriteBatch from engine to engine_traits/rocks

What have you changed?
More engine abstraction, re https://github.com/tikv/tikv/issues/6402
This ports tikv to use RocksWriteBatch from engine_rocks, and removes WriteBatch from the engine crate.
It does not convert tikv to use generic WriteBatches yet. That will be future work.
There's nothing particularly tricky to note here. Just a lot of code moving around. I reworked this patch many times so history is a bit messy.
What is the type of the changes?

Engineering (engineering change which doesn't change any feature or fix any issue)

How is the PR tested?
cargo test --all --no-run
Does this PR affect documentation (docs) or should it be mentioned in the release notes?
no
Does this PR affect tidb-ansible?
no
Refer to a related PR or issue link (optional)
https://github.com/tikv/tikv/issues/6402
Benchmark result if necessary (optional)
Any examples? (optional)

cc @Rustin-Liu  more engine abstraction.

Oh one thing of note here is that the abstract WriteBatch API is slightly different from the old - write batches are created with a factory method on KvEngine, and themselves contain a reference to the engine, such that an engine reference doesn't need to be carried around with them.
Tikv itself mostly still pairs its DB instances with writebatch.
This patch also adds many new temporary calls to c(), to convert engine::rocks::Engine instances to RocksEngine. These will go away eventually.

One tricky part of this patch is that each writebatch needs to be paired with the correct database, kv or raft. This is true moreso now that writebatch is created from a factory method on engine. It's possible I made mistakes in the porting such that a mixed up writebatches with the wrong database. This type of error may not manifest for a long time since rocks doesn't really care which database creates a given writebatch.

@sre-bot  /run-all-tests

rest LGTM

/test

@hicqu PTAL

/merge


(FACT-2848) Improve cache performance

CLA signed by all contributors.

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


Nested collection queries: System.InvalidOperationException: Unable to resolve generated expression

Some very odd behaviour I've found recently whilst using this library on a project. Rather hard to explain.
I've put an example project together which when ran will trigger an exception just like in our project's code:
https://github.com/benbristow/entityframework-projectables-bug
The main affected code:
Console.WriteLine("test 1");
await context.Students
    .Select(s =>
        new
        {
            CommentCount = s.Comments.SearchComment("foo").Count(),
            CourseCount = s.Courses.SearchCourse("bar").Count(),
        })
    .ToListAsync();
Console.WriteLine("pass 1");

Console.WriteLine("test 2");
await context.Students
    .Select(s =>
        new
        {
            CommentCount = s.Comments.Search("foo").Count(),
            CourseCount = s.Courses.Search("bar").Count(),
        })
    .ToListAsync();
Console.WriteLine("pass 2");

System.InvalidOperationException: Unable to resolve generated expression for ProjectablesExample.Extensions.IHasNameExtensions.SearchComment.
   at EntityFrameworkCore.Projectables.Services.ProjectionExpressionResolver.FindGeneratedExpression(MemberInfo projectableMemberInfo)
   at EntityFrameworkCore.Projectables.Services.ProjectableExpressionReplacer.TryGetReflectedExpression(MemberInfo memberInfo, LambdaExpression& reflectedExpression)
   at EntityFrameworkCore.Projectables.Services.ProjectableExpressionReplacer.VisitMethodCall(MethodCallExpression node)
   at System.Dynamic.Utils.ExpressionVisitorUtils.VisitArguments(ExpressionVisitor visitor, IArgumentProvider nodes)
   at System.Linq.Expressions.ExpressionVisitor.VisitMethodCall(MethodCallExpression node)
   at EntityFrameworkCore.Projectables.Services.ProjectableExpressionReplacer.VisitMethodCall(MethodCallExpression node)
   at System.Dynamic.Utils.ExpressionVisitorUtils.VisitArguments(ExpressionVisitor visitor, IArgumentProvider nodes)
   at System.Linq.Expressions.ExpressionVisitor.VisitNew(NewExpression node)
   at System.Linq.Expressions.ExpressionVisitor.VisitLambda[T](Expression`1 node)
   at System.Linq.Expressions.ExpressionVisitor.VisitUnary(UnaryExpression node)
   at System.Dynamic.Utils.ExpressionVisitorUtils.VisitArguments(ExpressionVisitor visitor, IArgumentProvider nodes)
   at System.Linq.Expressions.ExpressionVisitor.VisitMethodCall(MethodCallExpression node)
   at EntityFrameworkCore.Projectables.Services.ProjectableExpressionReplacer.VisitMethodCall(MethodCallExpression node)
   at EntityFrameworkCore.Projectables.Infrastructure.Internal.CustomQueryCompiler.Expand(Expression expression)
   at EntityFrameworkCore.Projectables.Infrastructure.Internal.CustomQueryCompiler.ExecuteAsync[TResult](Expression query, CancellationToken cancellationToken)
   at Microsoft.EntityFrameworkCore.Query.Internal.EntityQueryProvider.ExecuteAsync[TResult](Expression expression, CancellationToken cancellationToken)
   at Microsoft.EntityFrameworkCore.Query.Internal.EntityQueryable`1.GetAsyncEnumerator(CancellationToken cancellationToken)
   at System.Runtime.CompilerServices.ConfiguredCancelableAsyncEnumerable`1.GetAsyncEnumerator()
   at Microsoft.EntityFrameworkCore.EntityFrameworkQueryableExtensions.ToListAsync[TSource](IQueryable`1 source, CancellationToken cancellationToken)
   at Program.<Main>$(String[] args) in C:\Users\BenjaminBristow\dev\misc\ProjectablesExample\ProjectablesExample\Program.cs:line 29
   at Program.<Main>(String[] args)

Now if I remove the [Projectable] attribute from all of the extension methods in the below file, the program runs to completion with no issues (I don't think these examples actually need this library as EF seems to compile queries OK, but is different in our project where this library is required but to seems to be the same issue).
https://github.com/benbristow/entityframework-projectables-bug/blob/master/ProjectablesExample/Extensions/IHasNameExtensions.cs
If I leave the [Projectable] attribute on the uniquely named methods as below, the program also runs to completion with no issues, e.g.
    public static IEnumerable<Comment> Search(this IEnumerable<Comment> source, string search)
        => source.Where(s => s.Text == search);

    [Projectable]
    public static IEnumerable<Comment> SearchComment(this IEnumerable<Comment> source, string search)
        => source.Where(s => s.Text == search);

    public static IEnumerable<Course> Search(this IEnumerable<Course> source, string search)
        => source.Where(s => s.Name == search);

    [Projectable]
    public static IEnumerable<Course> SearchCourse(this IEnumerable<Course> source, string search)
        => source.Where(s => s.Name == search);

However, if I swap this around, it generates the same exception
    [Projectable]
    public static IEnumerable<Comment> Search(this IEnumerable<Comment> source, string search)
        => source.Where(s => s.Text == search);

    public static IEnumerable<Comment> SearchComment(this IEnumerable<Comment> source, string search)
        => source.Where(s => s.Text == search);

    [Projectable]
    public static IEnumerable<Course> Search(this IEnumerable<Course> source, string search)
        => source.Where(s => s.Name == search);

    public static IEnumerable<Course> SearchCourse(this IEnumerable<Course> source, string search)
        => source.Where(s => s.Name == search);

My hypothesis here is that I think there's an issue where the library isn't picking up the generic argument as being a different method and is getting confused.
Example run:
https://github.com/benbristow/entityframework-projectables-bug/actions/runs/6788027570/job/18452129581

Thanks for reporting this and apologies for not coming back to this earlier!
The problem is due to method overloading. This is currently not supported by this library. Each method projected with a Projectable attribute with generate a companion expression in a new source file. The name of his generated source file is currently derived roughly as: {Namespace}_{ClassName}_{MethodName}. When overloading methods, these generated source files are no longer unique which will  cause generation to fail in which case during runtime, we'll be unable to locate the generated expression.
We should ensure that the generated file names are unique. One way would be to include the line / column of actual method that we generate a projectable expression for as part of the file name.
Meanwhile the workaround would be to not use method overloading and give each method a unique name.

This is tracked by #58


Add libvirt podvm image support

- Description of the problem which is fixed/What is the use case
Add libvirt image creation job and libvirt image type for image generation.
- What I did
- How to verify it
- Description for the changelog

This PR is ready for review. Since I do not have write access, unable to remove the 'draft' tag. Can someone remove this tag please?

This PR is ready for review. Since I do not have write access, unable to remove the 'draft' tag. Can someone remove this tag please?

Since you are the PR creator, you should be able to change this "Ready".  I believe you raised this PR as a draft pull request.

@supritai @savitrilh can you please rebase the PR..

@supritai @savitrilh can you please rebase the PR..

Sure @bpradipt

Thanks! I like the idea, added few comments

/ok-to-test

General remarks :

I see multiple merge commits in this PR. Please reorganize this PR to be a list of regular commits on top of current devel.
Please amend all commits to have proper description and Signed-off-by tags

@supritai please do proper rebases instead of merging upstream devel again and again in your repo. We don't want this complexity in our git log. A PR should be a list of consecutive regular non-merge commits that apply to the target branch.
Each commit must have a description that helps understanding the change and have proper DCO (Signed-off-by).
This is hard requirement for this PR to get merged in addition to the requests from reviewers.

@supritai please do proper rebases instead of merging upstream devel again and again in your repo. We don't want this complexity in our git log. A PR should be a list of consecutive regular non-merge commits that apply to the target branch.
Each commit must have a description that helps understanding the change and have proper DCO (Signed-off-by).
This is hard requirement for this PR to get merged in addition to the requests from reviewers.

@gkurz Sure. I will do that. I was waiting for review to reach final stage so that I can squash all the commits.

@supritai once you have resolved a specific comment, can you please click on the associated "Resolve conversation" button, so that it's evident which comments have been addressed and what is still pending.

@supritai once you have resolved a specific comment, can you please click on the associated "Resolve conversation" button, so that it's evident which comments have been addressed and what is still pending.

Sure @bpradipt , most of them I have marked as resolved. Only where agreement of reviewer needed are not closed

Thanks @supritai for addressing the comments.
I added a minor suggestion to incorporate.  In case you have tried the latest code on s390x, can you post the test results.


S3 failing sometimes with missing reducedMeasurementData

RuntimeError: Xpath returns no href: .//dataObject[@ID='reducedMeasurementData']//fileLocation
(reducedMeasurementData) - e.g. d36dc413-fa01-4284-b993-0873ae3f12b7

our data don't contain reducedMeasurementData, to avoid the errors, removed the constant form the lib's constants.


Add RTL sample images

Questions
Answers

Description?
Add RTL version of sample images and improve image upload. (Redo for #70 )

Type?
improvement

BC breaks?
no

Deprecations?
no

Fixed ticket?
-

How to test?
-

@Progi1984 @matks
I forget what I did on #70
So I tried to create a new one from scratch to make sure it works fine.

Thanks @mparvazi !
I think you do a very interesting thing but can you explain it?
I see you added RTL versions for the images and enabled the code to use RTL images when using RTL language. What happens for custom images? If I add my own item to the slider how can I provide a RTL and a not-RTL image?
We need to explain this to users 😁 else nobody will use it.

The PS is not responsible for end users that who want to add RTL language to shop,
but doesn't know how to create an image for slides in RTL.
These sample images will be used for PS demo (RTL language users thinks PS have problem in RTL for slider in first experience).
It will be useful for theme developers to see how their theme looks like correctly  in RTL.
I just tried to flip current images horizontally to be used in RTL (fast solution),
but it would be better to design new sample images for LTR and RTL (by UX Team).

@matks
Users are able to upload slider images for any language, this PR adds some demo images to RTL languages, to display it properly, that's it 👍

@NeOMakinG @mparvazi Is the following scenario possible:

I use ps_imageslider
I add a a custom slider element, using image landscape.png
I setup a RTL language
A customer visits my shop and select the RTL language
The ps_imageslider module tries to show the custom slider element and looks for landscape_rtl.png image but does not find it

The _rtl suffix only used in installSamples() function, doesn't impact in other functions.
installSamples() only will use on installation or reseting module.
let's back to scenario:
when you setup image slider for lang1, and then you add lang2 to the shop,
the images of lang1 (default shop lang) will be used for lang2.
if you add lang3 then the images of lang1 will be used for lang3.

Let's think hummingbird is ready to preview for demo
This is what RTL language users see:

But I think something like this is better:

hi @mparvazi, it seems that this PR fixes at least 1 issue (PrestaShop/PrestaShop#27777) and add a new behavior when installing the module with an RTL language. Can you be more explicit on what your PR is doing so that the QA team can validate it easier?
Thanks

hi @mparvazi, it seems that this PR fixes at least 1 issue (PrestaShop/PrestaShop#27777) and add a new behavior when installing the module with an RTL language. Can you be more explicit on what your PR is doing so that the QA team can validate it easier? Thanks

Hi @atomiix
For QA:

It's possible to change image of slide for one language without need to change images for all languages.
It adds RTL version of SAMPLE images for fresh install of module.

Hello @sLorenzini
Please check this PR: #74
After merging that PR, we can leave target URL empty.
Btw I didn't change anything about required fields.
Thanks.

Hello @sLorenzini
Please check this PR: #74
After merging that PR, we can leave target URL empty. Btw I didn't change anything about required fields.
Thanks.

Hello @mparvazi,
Indeed, the target URL field could be empty with the last merged PR on the module.
In that case, your PR is QA approved ✅ .
Thank you


yt-dlp: use PyPI tarball

Motivation for this change
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 136010 at f9d989c1 run on x86_64-linux 1

5 packages built successfully:

python38Packages.yt-dlp
python38Packages.yt-dlp-light
yt-dlp (python39Packages.yt-dlp)
yt-dlp-light (python39Packages.yt-dlp-light)
tartube-yt-dlp

1 suggestion:

warning: python-include-tests
Consider adding a checkPhase for tests, or if not feasible, pythonImportsCheck.
Near pkgs/tools/misc/yt-dlp/default.nix:42:0:
   |
42 |     description = "Command-line tool to download videos from YouTube.com and other sites (youtube-dl fork)";
   | ^


Error creating LLDB target at path

Description
When iOS app is build, it crashed on start and get this logs in XCode
Warning: Error creating LLDB target at path '/Users/damirrigo/Library/Developer/Xcode/DerivedData/loyalty-eqbitahuqdywnefiuzwugdkdjweh/Build/Products/Debug-prod-iphonesimulator/loyalty.app'- using an empty LLDB target which can cause slow memory reads from remote devices.
2023-08-28 10:38:08.771130+0200 loyalty[26420:1628391] [native] No bundle URL present.

Make sure you're running a packager server or have included a .jsbundle file in your application bundle.
2023-08-28 10:38:08.776539+0200 loyalty[26420:1628391] *** Terminating app due to uncaught exception 'RCTFatalException: No bundle URL present.

Make sure you're running a packager server or have included a .jsbundle file in your application bundle.', reason: 'No bundle URL present.

Make sure you're running a packager server or have included a .jsbundle file in your application bundle.'
*** First throw call stack:
(
	0   CoreFoundation                      0x000000010e1bc78b __exceptionPreprocess + 242
	1   libobjc.A.dylib                     0x000000010c5a3b73 objc_exception_throw + 48
	2   loyalty                             0x0000000100bb23f5 RCTGetFatalHandler + 0
	3   loyalty                             0x0000000100bcbbd5 __28-[RCTCxxBridge handleError:]_block_invoke + 581
	4   libdispatch.dylib                   0x00000001105a954f _dispatch_call_block_and_release + 12
	5   libdispatch.dylib                   0x00000001105aa7ec _dispatch_client_callout + 8
	6   libdispatch.dylib                   0x00000001105bb6e2 _dispatch_main_queue_drain + 1462
	7   libdispatch.dylib                   0x00000001105bb11e _dispatch_main_queue_callback_4CF + 31
	8   CoreFoundation                      0x000000010e11a6cc __CFRUNLOOP_IS_SERVICING_THE_MAIN_DISPATCH_QUEUE__ + 9
	9   CoreFoundation                      0x000000010e114fbe __CFRunLoopRun + 2429
	10  CoreFoundation                      0x000000010e114264 CFRunLoopRunSpecific + 560
	11  GraphicsServices                    0x0000000114b8e24e GSEventRunModal + 139
	12  UIKitCore                           0x00000001233bd7bf -[UIApplication _run] + 994
	13  UIKitCore                           0x00000001233c25de UIApplicationMain + 123
	14  loyalty                             0x0000000100afc2b0 main + 96
	15  dyld                                0x000000010adb7384 start_sim + 10
	16  ???                                 0x0000000202854310 0x0 +<PHONE_NUMBER>
)
libc++abi: terminating due to uncaught exception of type NSException
Message from debugger: Terminated due to signal 6

React Native Version
0.71.6
Output of npx react-native info
System:
OS: macOS 13.1
CPU: (8) arm64 Apple M1 Pro
Memory: 255.19 MB / 32.00 GB
Shell: 5.8.1 - /bin/zsh
Binaries:
Node: 16.17.0 - ~/.nvm/versions/node/v16.17.0/bin/node
Yarn: 1.22.18 - /opt/homebrew/bin/yarn
npm: 8.15.0 - ~/.nvm/versions/node/v16.17.0/bin/npm
Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
Managers:
CocoaPods: 1.11.3 - /opt/homebrew/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 22.4, iOS 16.4, macOS 13.3, tvOS 16.4, watchOS 9.4
Android SDK:
API Levels: 28, 29, 30, 31, 32, 33
Build Tools: 28.0.2, 30.0.2, 30.0.3, 31.0.0, 32.0.0, 32.1.0, 33.0.0
System Images: android-28 | Google ARM64-V8a Play ARM 64 v8a, android-30 | Google APIs Intel x86 Atom, android-32 | Google APIs ARM 64 v8a, android-32 | Google Play ARM 64 v8a
Android NDK: Not Found
IDEs:
Android Studio: 2021.3 AI-213.7172.25.2113.9123335
Xcode: 14.3.1/14E300c - /usr/bin/xcodebuild
Languages:
Java: 19.0.1 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 18.2.0 => 18.2.0
react-native: 0.71.6 => 0.71.6
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce
Install deps and pods in app then run the app inside XCode.
Snack, screenshot, or link to a repository
https://github.com/facebook/react-native/assets/11744443/cb0c70f2-6022-4ba5-bd6b-e40d8e4dbafc

Any updates ?

We can't help without a reproducer

@rigodamir the "LLDB target at path" warning is not the issue here, as that warning will slow down development but not crash the app. The real issue here is the next errors in the console:

No bundle URL present.

and

Make sure you're running a packager server or have included a .jsbundle file in your application bundle.

This happens when (for example) you have not provided a valid bundleURL to initWithBundleURL, but I'd need to see the code for your AppDelegate to help debug it more.

Any updates? I face the same error message when I upgrade to Xcode15, everything in the old Xcode(14.1) is working fine.  I clear all cache in Pods/ builds/ in ios but it still not work???

I have the same problem here!
Note: I use MacOS Sonoma with M2 Pro

I have the same problem here!
I use MacOS Sonoma with M2 Max

I had this issue because for some reason project configuration names were not Debug and Release. After renaming them correctly to Debug and Release app started successfully.

Hi bumping it up, any fixes?

Hi, facing the same issue. any fix? did you find anything? @JustineUIT  @JulioAugustoS @namangoel11 @shashwats-greytip

@SaatwikRishi , @faizantariq2, for it fixed after upgrading the react-native version to 72.5.

Hi, facing the same issue. any fix?

react-native:0.72.5:
Steps How I do fixe this issue:
1- Open iOS project in Xcode
2- Select -> Target → Build Phases
3- under Copy Bundle Resources click on the plus button as following screenshot.
4- It will show you the prompt as shown below. Then select the file main.jsbundle and click on the Add button. ( If main.jsbundle file is not there then create first by running this command in terminal into your root project folder: react-native bundle --entry-file='index.js' --bundle-output='./ios/main.jsbundle' --dev=false --platform='ios'
)
5- Try to re-run your app
The crash will gone by using these steps but that warning is still there: Warning: Error creating LLDB target at path '/Users/hafizranamawais/Library/Developer/Xcode/DerivedData/ShopCash-gdtskdgmiaxjtselannjqozvodmi/Build/Products/Debug-iphonesimulator/ShopCash.app'- using an empty LLDB target which can cause slow memory reads from remote devices.

Seems like there's a lot of different issues reported here related to crashes.
I am not having any crashes, just the warning on slow memory reads on React Native 0.72.7
Is there anything that can be done about this?

HI, Did anyone find solution , i'm having same issue in release build of IOS in debug build it is working for me .
*** Terminating app due to uncaught exception 'RCTFatalException: Unhandled JS Exception: TypeError: Cannot read property 'emitExpiredEvent' of null, js engine: hermes', reason: 'Unhandled JS Exception: TypeError: Cannot read property 'emitExpiredEvent' of null, js engine: hermes, stack:
anonymous@90969:54
anonymous@50006:27
emit@1620:39
emit@15290:49
anonymous@49982:24
__invokeCallback@2400:22
anonymous@2200:33
__guard@2317:14
invokeCallbackAndReturnFlushedQueue@2199:20
'
*** First throw call stack:
(
0   CoreFoundation                      0x00000001804ae138 _exceptionPreprocess + 172
1   libobjc.A.dylib                     0x0000000180087db4 objc_exception_throw + 56
2   weightwonder                        0x00000001046bf794 RCTFormatError + 0
3   weightwonder                        0x000000010472d14c -[RCTExceptionsManager reportFatal:stack:exceptionId:extraDataAsJSON:] + 488
4   weightwonder                        0x000000010472d960 -[RCTExceptionsManager reportException:] + 1308
5   CoreFoundation                      0x00000001804b4760 invoking + 144
6   CoreFoundation                      0x00000001804b1ac4 -[NSInvocation invoke] + 276
7   CoreFoundation                      0x00000001804b1d5c -[NSInvocation invokeWithTarget:] + 60
8   weightwonder                        0x00000001046ee77c -[RCTModuleMethod invokeWithBridge:module:arguments:] + 388
9   weightwonder                        0x00000001046f07d0 _ZN8facebook5reactL11invokeInnerEP9RCTBridgeP13RCTModuleDatajRKN5folly7dynamicEiN12_GLOBAL__N_117SchedulingContextE + 456
10  weightwonder                        0x00000001046f0420 ___ZN8facebook5react15RCTNativeModule6invokeEjON5folly7dynamicEi_block_invoke + 112
11  libdispatch.dylib                   0x0000000105f97ec4 _dispatch_call_block_and_release + 24
12  libdispatch.dylib                   0x0000000105f9973c _dispatch_client_callout + 16
13  libdispatch.dylib                   0x0000000105fa1a30 _dispatch_lane_serial_drain + 916
14  libdispatch.dylib                   0x0000000105fa2774 _dispatch_lane_invoke + 420
15  libdispatch.dylib                   0x0000000105faf1a8 _dispatch_root_queue_drain_deferred_wlh + 324
16  libdispatch.dylib                   0x0000000105fae604 _dispatch_workloop_worker_thread + 488
17  libsystem_pthread.dylib             0x0000000105f23924 _pthread_wqthread + 284
18  libsystem_pthread.dylib             0x0000000105f226e4 start_wqthread + 8
)
libc++abi: terminating due to uncaught exception of type NSException
Thanks

Did you get any solution

Did you get any solution

Yes, It was the problem with react-native-fetch-blob package. I removed that package and then it worked for me.

i dont have this package in my package.json file although i have
react-native-blob-util package
On Mon, Apr 22, 2024 at 11:38 AM ragini sahu @.***>
wrote:

Did you get any solution
Yes, It was the problem with react-native-fetch-blob package. I removed
that package and then it worked for me.
—
Reply to this email directly, view it on GitHub
https://github.com/facebook/react-native/issues/39180#issuecomment-2068561099,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BF6DSI3AMS4YVZCVMA6LHPTY6SSMHAVCNFSM6AAAAAA4BC5FWSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRYGU3DCMBZHE
.
You are receiving this because you commented.Message ID:
@.***>


Chrome v. 80.0.3987.87 info_outlin

Chrome v. 80.0.3987.87 info_outlin
Originally posted by @deathtrap512 in https://github.com/dotnet/core/issues/4226#issuecomment-582594490

It's not clear what you're asking here; please open a new issue with some detail if this was not created in error.


Support typing an opaque type definition in the editor.

Requires #2221 to be done.

Editor is no longer being worked on, and has been removed from the repo. Closing.


Guidance by question still in view on 'themed guidance' page

Legacy guidance by question data was migrated into 'annotations' on the customisable templates view in an earlier release, but is still visible under the themed guidance view on org-admin pages too.
To see an example check university of Glasgow guidance related to MRC q7.
Here is the view on the template customisation showing it correctly as an annotation, but it also wrongly shows in the themed guidance list.

@sjDCC Is this still an issue?
I think it's been fixed by the overhaul of these views

yes, appears so. Closing out


fix: Truncate long text within buttons

What changed?
The Button in the EditNameSettings.tsx component was updated to address the issue with overly long user names.

Added the max-w-[200px] class to limit the button width.
Applied the truncate class to truncate long strings.
Implemented a tooltip that appears on hover to show the full name for names longer than 15 characters.

Fixes #302 (Truncate long text within buttons)
Why?
This change addresses the issue where long user names were overflowing within the button. It ensures that buttons maintain a consistent width and provides a way to view the full name using a tooltip on hover.
How was this change made?

The max-w-[200px] class was applied to restrict the button width.
The truncate class was added to cut off overflowing text.
A tooltip component was added that displays the full user name for names exceeding 15 characters when hovered over.

How was this tested?

Tested with different user names to verify that names longer than 15 characters are properly truncated.
Hovered over long names to ensure the tooltip correctly displays the full name.
Verified the layout and button resizing under different screen sizes to ensure responsiveness.

Anything else?

No further adjustments are needed unless additional features are requested. Let me know if anything should be modified.

Note: we should also add a changeset to this PR! You can add one by running pnpm changeset, selecting "patch", and entering a comment from the user's point of view like "Truncate long text within buttons"


Mavlink mission item executed not working

Hello,
Has the mavlink state mission item executed topic and parameter (CommonMavlinkStateMissonItemExecuted) been successfully tested or run successfully? I have enabled it through the driver configuration paramters and when I subscribe to the topic and run a mavlink flight plan on the Bebop 2 nothing is published.
This information is extremely important for me to make the next step I make in my research easier.

@thomas-bamford Sorry for joining this thread late. Have you made any progress on this? Maybe @Voidminded can also help with this one.


^M newlines break bash scripts, typically pipefail error

This scenario occurs when downloading ARM template.zip from Azure. It seems the .sh scripts in ARM templates have Windows CRLF line endings, ie. they appear as ^M.
Example from deploy.sh as seen by "cat -net deploy.sh":
#!/bin/bash^M$
set -euo pipefail^M$
When trying to run this in WSL terminal:
bash deploy.sh
: invalid option namet: pipefail
After switching to LF linefeeds it works in WSL as expected.
This probably can not be fixed in WSL, but seems to be quite common error and it was quite hard to figure out what was the root cause. E.g. VSCode doesn't give user any warning about this when using integrated WSL terminal.
Probably something to add to documentation / FAQ.

I think you'll have better luck filing this over at https://github.com/Microsoft/Console/ .

@toniwillberg - Can you point me to the scripts so I can follow up with the team that owns them?  There's nothing that WSL can do to handle how Linux applications handle Windows-style line endings.

@benhillis I'll send you email with details about the scripts.
Meanwhile feel free to close this issue.

@toniwillberg - Sounds good, thanks!

There's nothing that WSL can do to handle how Linux applications handle Windows-style line endings.

There isn't. But Cygwin's set -o igncr patch could be upstreamed to Canonical's launchpad, as mentioned a while back. You can't win this war one script at a time.


Deleted events show up in Contributions

Describe the bug
Events which are deleted/don't exist show up in the contributions section for a collective.
To Reproduce
Steps to reproduce the behaviour:

Go to https://opencollective.com/maddhruv
Click on any event card in the contributions section
The page says - Ooops, an unexpected error seems to have occurred  🤕 which is because the event actually doesn't exist anymore - as I deleted the collective which was hosting the event.

Expected behaviour
The archived/deleted events should not show up here
Screenshots

Crash is now prevented, and the policy for events deletion has been reviewed:

Preventing Deletion of Collective with Events opencollective/opencollective#2577


Event for more than one day?

I have implemented this here: https://github.com/aryaxt/PDTSimpleCalendar
Will open a PR after done with improving scroll performance


Inferno Test Kit $import operation returns HTML without Content-Location

This is a cross-post from the Inferno UDS Plus Test Kit issue, but to keep all the UDS+ issues in one spot:  the Inferno Test Kit is returning an HTML page without a Content-Location header in response to the $import operation call.

The POST tests in the UDS+ Inferno Test Kit (e.g., Receive UDS+ Import Manifest via POST request, which is one of our first requests when a UDS+ Volunteer Tester is practicing connectivity) are only acting as a rudimentary data receiver compared to the UDS+ Sandbox (which actually tests against an endpoint (valid conformant server/s). These tests are not a valid test of the FHIR submission process, but an initial practice run to see if your synthetic test data is healthy and likely ready for submitting to the UDS+ sandbox. If you are looking for a JSON response instead of HTML, then we advise obtaining access to the UDS+ sandbox.


[Fix] Fix cylinder3d

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
Modify cfg and mv cylinder3d blocks from sparse_layer to backbone.
Modification
Modify cfg and mv cylinder3d blocks from sparse_layer to backbone.
BC-breaking (Optional)
Does the modification introduce changes that break the back-compatibility of the downstream repos?
If so, please describe how it breaks the compatibility and how the downstream projects should modify their code to keep compatibility with this PR.
Use cases (Optional)
If this PR introduces a new feature, it is better to list some use cases here, and update the documentation.
Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
If the modification has potential influence on downstream projects, this PR should be tested with downstream projects.
The documentation has been modified accordingly, like docstring or example tutorials.

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


AttributeError's on disnake.Message instantiation due to Intents.none()

Summary
When using Intents.none(), disnake.Message.thread will fail to resolve on un-cached guilds resulting in errors.
Reproduction Steps
I do not have the full reproduction steps, I fixed this awhile back but forgot to make an issue upstream. However, the description following this should provide enough reason to believe me.
Minimal Reproducible Code
Unsure if this works, however this is the rough idea.
# Pop your guild from the state cache
message = await channel.fetch_message(message with thread)

When attempting to build a message instance, Message.thread is called internally. Specifically here
Expected Results
The message object would be fetched/instantiated correctly.
Actual Results
AttributeError: 'Object' object has no attribute 'get_thread'
Intents
Intents.none()
System Information
Still applicable on master.

Checklist

[X] I have searched the open issues for duplicates.
[X] I have shown the entire traceback, if possible.
[X] I have removed my token from display, if visible.

Additional Context
This code fixes the issue while keeping the end API the same.
Modify disnake.Message.thread to be this

#712 would also need to be fixed if this is fixed. I thought about adding it to this issue but I felt I'd left it a little late and its in a separate section


Provide APIs to read inf/NaN sticky registers

Add compute API so compute kernels could insert the calls to read register values to trisc and put to DPRINT or to some other memory location (as indicated by the programmer).
Milos
Sticky bits are in debug register RISCV_DEBUG_REG_FPU_STICKY_BITS.
To read sticky bits register: reg_read(RISCV_DEBUG_REG_FPU_STICKY_BITS);
Sticky bits definition:

Bit 10: integer saturation  [blackhole]
Bits 9-7: L1 accumulation (overflow/underflow/saturation)
Bits 6-4: fpu (overflow/underflow/saturation)
Bit 3: sfpu nan detected
Bit 2: sfpu inf detected
Bit 1: sfpu denormalized detected
Bit 0: sfpu overflow detected

merged as part of https://github.com/tenstorrent/tt-metal/pull/13502.
New API has been added, that executes on MATH thread
ALWI uint32_t get_compute_special_value_flags()
ALWI uint32_t get_compute_special_value_flags_fpu(uint32_t special_value_flags_reg)
ALWI uint32_t get_compute_special_value_flags_sfpu(uint32_t special_value_flags_reg)
ALWI void clear_compute_special_value_flags()
ALWI void store_compute_special_value_flags_to_l1(uint32_t l1_addr)

It seems like the above APIs don't have documentation yet.
So I guessed how to use them. Is this a correct to call like this?
uint32_t flags = get_compute_special_value_flags();
uint32_t fpu_flags = get_compute_special_value_flags_fpu(flags);    // <-- give flags argument returned from the first API call
uint32_t sfpu_flags = get_compute_special_value_flags_sfpu(flags);  // <-- give flags argument returned from the first API call

// do something depeding on the 'fpu_flags' or 'sfpu_flags' values

@ttmtrajkovic
Would you confirm the above usage?

@razorback3,
Yes, that is the correct usage. I will add comments to the compute_api.h as well. Those are helper functions to extract fpu or sfpu bits only.
Other functions for reading, clearing and dumping bits to L1 should be self-explanatory.
milos


Weird predictions after conversion to NCNN

After training a custom model and converting it to NCNN, I'm getting unexpected weird detections as below. There were no issues with torch or ONNX inference. Anyone able to solve it?
When I convert the given pretrained model to NCNN, it works fine. This is strange behaviour. I suspect it has something to do with code changes that happened after 2021

I got a similiar problem. I trained a custom model which class num=1 and export ncnn.
I can use https://github.com/RangiLyu/nanodet/blob/main/demo_ncnn/nanodet.cpp to run predict, and got a right object,but the box is always rot right.
It feels like the x and y always be smaller than right label.For example,  for a input 320x320 test image, the right box is [21,11,285,314] in [x1,y1,x2,y2] order, but the predict is [56,0,255,275].
I have change the setting in nanodet.h which not help:
    int input_size[2] = {320,320};
    int num_class = 1;
    int reg_max = 7; // `reg_max` set in the training config. Default: 7.
    std::vector<int> strides = { 8, 16, 32};


d3.rgb() incorrectly parses hex string values when used as a map() callback

Context
I wanted to make a custom color scale, so I used d3.scale.category10() as an example. category10() uses d3_rgbString() as a map callback. Not having access to that function directly, I figured I would use d3.rgb() since it seems like an ideal iterator and the documentation for a color string doesn't explicitly say it shouldn't be used as such.
For reference, my code looks like this:
['#b3dbf4', '#2894cf'].map(d3.rgb);

Problem
Array.map() passes three arguments to its callback function. d3.rgb() does a check for arguments.length < 2 which in this case will always be false, which in turn causes d3.rgb() to return a new instance of itself with incorrect RGB values. Note that g is set to the map index parameter.
Previous example with output:
['#b3dbf4', '#2894cf'].map(d3.rgb); // [ {"r":0,"g":0,"b":0}, {"r":0,"g":1,"b":0} ]

Expected
I expect the map function to return the correct RGB values for the array elements.
Previous example with expected output:
['#b3dbf4', '#2894cf'].map(d3.rgb); // [ {"r":179,"g":219,"b":244}, {"r":40,"g":148,"b":207} ]

Once I figure out why the arguments length check is being performed, I'll try to whip up a PR.
Thank you for such a killer visualization package! :beers:

I think you want rgbString?

@tristanreid Thank you for the response. My apologies though, I'm not looking for a solution (I'm fine just returning the hex strings for my purposes). I just noticed this while playing with the color scales and thought it odd / a shame and figured I would report it. If nobody cares, this case may be closed.

Thanks for raising this issue. I don’t consider this behavior a bug but an ambiguity caused by the overloaded signature of d3.rgb. Specifically:

d3.rgb(r, g, b) constructs a color for r, g and b channel values
d3.rgb(string) constructs a color from the specified color specifier string

(I’m simplifying slightly: d3.rgb can also accept a single argument that is another color, which is then coerced to a string if it is not already RGB, for conversion from other color spaces.)
The two versions of d3.rgb are disambiguated by the number of arguments. If you call d3.rgb with three arguments you get the first version; if you call it with one argument you get the second version; anything else has undefined behavior. This is implied by the d3.rgb documentation, but things are always going to be a little vague given the nature of JavaScript.
Now note that array.map invokes the specified callback with three arguments: an array element, the corresponding array index, and the array itself. Thus, this form is interpreted as d3.rgb(r, g, b) and not d3.rgb(string).
You could argue that we should use type inference (typeof r === "string", e.g.) rather than argument length checking to disambiguate. However, I prefer argument length checking because it is less ambiguous: it can be determined statically by looking at the code, rather than needing to know the runtime type of a value. Also, note that it’s easy for values to be strings unexpectedly in JavaScript—for example when loading data from a CSV file. Coercing values to numbers is desirable in these cases.
At any rate, you can’t always disambiguate perfectly. I like the argument length strategy, and I think it’s also good to apply that strategy consistently rather than try to special-case the method behavior for array.map.

@mbostock Thanks for the official word. I played around with type checking but ran into edge cases (as you point out) in the tests. Seeing as you prefer the argument length checking, I'll just leave it be. Thank you for your time :smile:


go get showing this library as +incompatible in go.mod

When upgrading from go 1.10 to 1.11 and starting to use go.mod I noticed this was an issue.  I was indirectly using this by using
github.com/appleboy/gin-jwt v2.3.1
but even trying to use it directly I get this +incompatible marking.. not sure what it means or why actually...
ithub.com/dgrijalva/jwt-go v3.2.0+incompatible // indirect
??

Go mod requires some special flags. When I last looked at this, dep was the recommended thing and go mod was just a proposal. Since it's included in go 1.11, it would appear it's time to add support. I'll look into this over the weekend.

Great thank you.  There is a good video on pitfalls of dep and why they went another route.  Finally having a real module Management system in go tooling is huge so glad it's being standardized

I also faced same issue after going through go mod docs I found a way to solve this.
Commit the changes and push it will work after this
@dgrijalva  I tried to create a pr but I was not able to push(do not have remote push rights)
refrences: https://github.com/golang/go/wiki/Modules#semantic-import-versioning
https://github.com/golang/go/wiki/Modules#releasing-modules-v2-or-higher
https://github.com/marwan-at-work/mod

@dgrijalva any update on this?

Are there any plans to fix this?  If so, any idea when the fix will be available?

Support for go modules is coming in v4. I don't think it's possible to add support as a non-breaking change.

Thanks for the response about it coming in v4 - what is the projected timeframe of v4?

I faced some issues with go modules and jwt-go package, and taking a look in the format of other require modules inside my go.mod file I realized you can "import" them in a certain format, like so:
github.com/dgrijalva/jwt-go v0.0.0-20180308151308-06ea1031745c // inderect
20180308151308 -> this part is the date of the last commit made on master branch
06ea1031745c ->  it's the first 12 letters of last commit hash on master branch.
It's working for me, but I'm waiting for v4 to get go mod support, hope it helps some else.

github.com/dgrijalva/jwt-go v0.0.0-20180308151308-06ea1031745c // inderect

did you just use go get on that?


Background size

Hey guys,
Suggestion: change background-size to cover instead of 100% 100% on elements #home and #why.
The UI will look better. 🤘

@psicotropicos thanks! We appreciate your feedback :smile:


add crd source if running on vmware

We need the crd source type when running on vmware clusters.

@QuentinBisson i'm assuming here that the app's values will container provider: vmware?

You might have to change it here
https://github.com/giantswarm/external-dns-app/blob/9f90ed8c8953a2389b333a6a856837cff92b475b/helm/external-dns-app/templates/_helpers.tpl#L139

good spot - i completely forgot about that


helm search --regex case sensitivity

helm/tiller version 2.5.1
I had added a couple repos:
$ helm repo add helmDevel http://example.com/devel
$ helm repo add helmStable http://example.com/stable
However doing various searches:
works $ helm search helmDevel
works $ helm search helmStable
works $ helm search --regex 'helm'
fails $ helm search --regex 'helmDevel'
fails $ helm search --regex 'helmStable'
fails $ helm search --regex 'helm[DS]'
Since the first three work, that leads me to believe there is a problem with how regex is handling mixed casing.

I have confirmed that helm search --regexp helmUpper returns no results but helm search --regexp helm[Uu]pper and helm search --regexp helmupper work as expected.
Marking this as a bug because the UX is confusing. User will see the mixed case repo name but cannot search for it by upper case and regex.

I find the man who wrote code here was to ignore all the capital letters and helm search [--regexp] only uses lowercase letters. I upload a PR about case sensitive. Whether to merge is up to you. @jascott1


[Snyk] Upgrade: @fortawesome/fontawesome-svg-core, @fortawesome/free-regular-svg-icons, @fortawesome/free-solid-svg-icons

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade multiple dependencies.
👯♂ The following dependencies are linked and will therefore be updated together.

:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

Name
Versions
Released on

@fortawesome/fontawesome-svg-corefrom 6.4.0 to 6.4.2
1 version ahead of your current version
21 days agoon 2023-08-02

@fortawesome/free-regular-svg-iconsfrom 6.4.0 to 6.4.2
1 version ahead of your current version
21 days agoon 2023-08-02

@fortawesome/free-solid-svg-iconsfrom 6.4.0 to 6.4.2
1 version ahead of your current version
21 days agoon 2023-08-02

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Check out a preview of your pull request here: https://alphasite-appservice-dev-pr-preview-0.azurewebsites.net


Initial BBS+ signature support crate

Details will assuredly change before a release, as the spec (2022 version) is still in development. This is not compatible with the BBS+ 2020 spec.

WIP @tplooker @mikelodder7


[CLI] chat_completion CLI

TL;DR

adding llama-stack-client inference chat_completion to quickly test a client example w/o getting apps

Test
Non-Streaming
llama-stack-client inference chat_completion --message "hello, what model are you"

Streaming
llama-stack-client inference chat_completion --message "hello, what model are you" --stream

chat-completion instead of chat_completion maybe?


Bug with pageparts in a group (in the Webclient)

Describe:
When view and edit modes are changed on the pages with pageparts in a group, the error "Sorry, something went wrong. Refresh the page and try again." appears.
Specific conditions to reproduce:
•  The page contains a group, which contains a subpage.
•  The subpage has editable fields.
•  When page opens, the group with the subpage should be collapsed.

If the group is expanded before changing of the mode to/from View/Edit, does not show the same error and works fine.
Business Central Versions:
14.1: the error "Sorry, something went wrong. Refresh the page and try again." appears.
14.0: nothing happens with dialog window "Working on it ..."
13.0: everything works perfectly.

Please report this issue via the standard support channel.


Regenerate adif_pb2

I am getting the following error when trying to execute the demo:
TypeError: Descriptors cannot not be created directly.
If this call came from a _pb2.py file, your generated code is out of date and must be regenerated with protoc >= 3.19.0.
If you cannot immediately regenerate your protos, some other possible workarounds are:
 1. Downgrade the protobuf package to 3.20.x or lower.
 2. Set PROTOCOL_BUFFERS_PYTHON_IMPLEMENTATION=python (but this will use pure-Python parsing and will be much slower).

More information: https://developers.google.com/protocol-buffers/docs/news/2022-05-06#python-updates

Hope it has an easy fix by updating the adif_pb2 file

In order to update the adif_pb2 file I would have needed to upgrade my local protoc. Since I installed that via apt, it's pretty far out of date. Rather than experimenting with different versions of protoc, I've depended on a slightly older version of the python protobuf library.


Markdown links inside code blocks are interpreted

I seem to have run into a problem when using {% markdown-include %} to include a markdown file that contains a code block, which in turn contains something that looks like a valid markdown link (but in fact it's a C++11 lambda definition).
Here's an example. I have a markdown file examples/lambda.md that contains a C++ code snippet, with C++11 lambdas:
examples/lambda.md
This is an example of a C++11 lambda:

```cpp
const auto lambda = []() { .... };
````

If I include this markdown file into another document using {% include-markdown %}, the []() string in the code block gets interpreted as an empty link
docs/main.md
Hello! Let's learn C++!

{% include-markdown "../examples/lambda.md" %}

The output then looks like this:

Hello! Let's learn C++!
This is an example of a C++11 lambda:
const auto lambda = [](../examples) { ... };

Notice the "../examples" string rendered in the lambda codeblock. I suspect the reason is that []() is a markdown syntax for links, which somehow gets interpreted during the include process, even though it is inside of a code-block. If I change the code example to contain space between [] and (), then it gets rendered correctly (i.e. no extra strings get injected)
const auto lambda = [] () { ... };

Thank you for the detailed report. It's fixed and released in v3.2.2.


difference of version

What is the difference between version suffix with "AAPL"?

Apple does not allow github to be linked. In order to comply the App Store rule, some of the feature was removed.


Make as_array consistent with as_variable

chainer.as_variable can be used to easily write code that accepts either Variable or ndarray as input. However, the opposite is not possible directly from chainer, but we must write chainer.variable.as_array. I think it's inconsistent that chainer.variable.as_variable and chainer.as_variable exist, but only chainer.variable.as_array and not chainer.as_array. I propose that we make the API consistent by exposing the method from chainer. What do you think?

Looks good.
Could you add documentation here?
https://docs.chainer.org/en/stable/reference/variable.html#variable-classes-and-utilities

Jenkins, test this please

Jenkins CI test (for commit 00b8f7de4ab7b399b76c17ed5c7770f841452a75, target branch master) failed with status FAILURE.

Jenkins seems to fail due to some error unrelated to this PR. Could you please try Jenkins again?

@tkerola Sorry, Jenkins currently has some problem. We're investigating.

Jenkins, test this please

Jenkins CI test (for commit 00b8f7de4ab7b399b76c17ed5c7770f841452a75, target branch master) failed with status FAILURE.

Jenkins, test this please

Jenkins CI test (for commit 00b8f7de4ab7b399b76c17ed5c7770f841452a75, target branch master) failed with status FAILURE.

Jenkins, test this please

Jenkins CI test (for commit 00b8f7de4ab7b399b76c17ed5c7770f841452a75, target branch master) failed with status FAILURE.

Jenkins, test this please

Jenkins CI test (for commit 00b8f7de4ab7b399b76c17ed5c7770f841452a75, target branch master) failed with status FAILURE.

Jenkins, test this please

Jenkins CI test (for commit 00b8f7de4ab7b399b76c17ed5c7770f841452a75, target branch master) failed with status FAILURE.

Finally, the test failure seems irrelevant.
LGTM!


API change stop_strategy to stopping_strategy

Issue #273

Codecov Report

Merging #274 (ede885d) into main (08f82b0) will decrease coverage by 0.00%.
The diff coverage is 78.57%.

@@            Coverage Diff             @@
##             main     #274      +/-   ##
==========================================
- Coverage   62.66%   62.66%   -0.01%
==========================================
  Files          41       41
  Lines        2100     2108       +8
  Branches      372      374       +2
==========================================
+ Hits         1316     1321       +5
- Misses        688      690       +2
- Partials       96       97       +1


loading sets from "script-output"

Hi,
is it possible to automatically load (set-)scripts in "script-output" directory?

I'm confused. What do you wish to do exactly?

okay, I'll try to explain:
currently, the mod has a fixed list of sets loaded by default (loader.addSets "settings/...")
I thought about kind of "set-directory" where users can place there set files, which are loaded then automatically (from outside the mod's zip file)
so you dont have to enter those complicated commands to create sets inside the game and ones can share there sets

I don't think there is clear method to read those files. Workaround might exist, but I would like a official method that does not desynch.
Heres modding interface request relate to this.
Client-side scripts to send custom events to sim; readfile()


Add access webmachine connection properties

This is experimental.

@manderson26 Rebased and moved this PR here: https://github.com/chef/chef-server/pull/165


Support a11y navigation for Swing interop

Proposed Changes
Enable a11y navigation, so focus change inside Compose component will be properly handled by a11y support

BTW forgot to notice that it is not handled by skiko, yet. So we need to implement it in skiko and it will work. But anyway this change in Compose is needed


Fix docker image build

The new task does a few things:

Having an extra build target makes the workflow file cleaner
it downloads the release targets which we want to build docker images for
it extracts the tar files into sub folders. We need to extract into sub folders to so that our image can fine the binaries. By providing platform to docker buildx the variable TARGETPLATFORM will be available when building the image. This variable is either linux/amd64/, linux/arm64/ or linux/arm/v7. Hence we need to extract into subfolders.
We create 2 images, one for the maker and one for the taker. The images are then pushed to our organisation's docker registry.

Great work, much better! Thank you!
One question: Why do we have to publish to the organization's packages? I would have expected the images to appear here: comit-network/hermes/packages

Good question. I don't know unfortunately. Let's see if this works and then try to publish somewhere else.

Testing to publish under the repo here: https://github.com/bonomat/hermes/actions/runs/1339699306


Compling for chipKit WF32 board

I'm trying to compile the video streaming app for my chipKit WF32 board.  It works on an Uno without problems.   With the chipKit set as the baord (under Arduino 1.6.9) I get the following errors:
Arduino: 1.6.9 (Windows 10), Board: "chipKIT WF32"
In file included from ... Projects\ArduCAM\Arduino Sketches\ArduCAM_Mini_OV2640_Video_Streaming\ArduCAM_Mini_OV2640_Video_Streaming.ino:23:0:

...\Documents\Arduino\libraries\ArduCAM/ArduCAM.h:338:3: error: 'regtype' does not name a type
   regtype *P_CS;
   ^

...\Documents\Arduino\libraries\ArduCAM/ArduCAM.h:339:3: error: 'regsize' does not name a type
   regsize B_CS;
   ^

exit status 255
Error compiling for board chipKIT WF32.

Has anyone got these to compile on chipKit boards?

@mstoer, we have simple porting for chipkit, please download from www.arducam.com/downloads/porting/Chipkit/testArducamAPI.zip
It is appreciate if you can help us improve the code for chipkit.

Thank you, I'll give it a try.


Upgrade to latest jackson version

Signed-off-by: Taylor Gray<EMAIL_ADDRESS>Description
Upgrades jackson dependencies from 2.13.1 to 2.13.2
Issues Resolved
Closes #1182
Check List

[x] New functionality includes testing.
[x] New functionality has been documented.

[x] New functionality has javadoc added

[x] Commits are signed with a real name per the DCO

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

Codecov Report

Merging #1183 (16fd2d4) into main (e048846) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #1183   +/-   ##
=========================================
  Coverage     93.99%   93.99%
  Complexity     1056     1056
=========================================
  Files           140      140
  Lines          3016     3016
  Branches        274      274
=========================================
  Hits           2835     2835
  Misses          130      130
  Partials         51       51

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 e048846...16fd2d4. Read the comment docs.


Connection in relation

Hello, first of all thank you for the good work. The graphql-comopose libraries are amazing and make life really easier !
I just have a question about making connection on a relation field.
Imagine I have User and Post schemas. A User has afield posts with the id of its posts.
How could I have a connection from the array of ids of the field posts in a relation. I mean something like this:
UserTC.addRelation(
  'friends',
  () => ({
    resolver: postTC.getResolver('connection'), // But I would like this connection to be from the array of id
                                                                              contained in the field posts
    args: {
      _ids: (source) => source.posts,
    },
    projection: { posts: 1 }, // point fields in source object, which should be fetched from DB
  })
);

If you could help me doing this using your libraries it would be awesome !
Thanks in advance

You are using old API. Let's migrate on new 2.0.0 API:
UserTC.addRelation( 'friends', {
  resolver: () => PostTC.getResolver('findByIds'),
  prepareArgs: {
    _ids: (source) => source.posts,
  },
  projection: { posts: 1 },
});

For making relations via resolvers you just need to see its arguments via graphiql, and understood which args you should provide to them.
You may do it here https://graphql-compose.herokuapp.com/user/ see DOCS in right panel for query

So for connection  you should prepare filter argument in such way:
UserTC.addRelation( 'friends', {
  resolver: () => PostTC.getResolver('connection'),
  prepareArgs: {
    filter: (source) => ({ _ids: source.posts }),
  },
  projection: { posts: 1 },
});

Thank you a lot.
It works perfectly !


Edit a pause card/Story 3

Value Proposition
As a User
I want to save a duration for my pause timings
So that I do not have to remember for how long I need to rest during my exercises.
Description

Acceptance Criteria
Homescreen

[ ] By clicking on the pause button the user is redirected to the form page.
[ ] The default duration of the pause displayed is "00:00".

Formpage Screen

[ ] By clicking on the back button the user is redirected to the home screen without saving the pause duration.
[ ] The default and minimum value of the pause timer is "00:00", the maximum value is "59:59".
[ ] By sliding on the inputbar the user can increase or decrease the duration of the pause.
[ ] By clicking on the checkmark-button the pause duration is saved and the user is redirected to the home screen. The home screen then shows the pause duration the user saved.

Tasks
(for developers team)

[ ] create a button on the homescreen which brings you on a formpage
[ ] the button need to show the time we set on the formpage
[ ] create a paragraph which describes to click the button
[ ] create a digital timer which you can set
[ ] create a link in form of an icon which brings you back to the homepage
[ ] create a submit button which confirms your setted time

Complexity
L-XL


Consider removing just sane

CI is faster now that running just sane locally, this makes the command not really that useful anymore.

It doesn't really hurt having the command and it may be useful still.


sanity check date prior to submission

write one function to do sanity check (date_utils.check_date)
add one option to config (max_submit_within_days)
logics for version compatibility

unfamilliar with this implementation. may need to be modified.

@changukshin You happy for me to rebase this?

@ThorpeJosh Sure. Thank you. I have something to do so I have no slot to follow this. Sorry for this delay.

@ThorpeJosh Sure. Thank you. I have something to do so I have no slot to follow this. Sorry for this delay.

No stress. Raised a new PR #57 with this work


Retry if DMG BOM is empty.

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

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

This error is pretty much impossible to reproduce and only happens rarely on CI.
My naive guess is that mounting a DMG on CI is slow sometimes meaning we try finding files which aren't mounted yet. Not sure if that's actually how hdiutil works but we'll see.
Fail in both cases, so we actually see if retrying fixes this or not.

Makes sense, thanks @reitermarkus!

Review period skipped due to critical label.


[BUG] SkiaSharp not working when deploy Blazor client side application (.NET 7.0) to Azure App Service (Windows) using GitHub Actions (CI/CD Pipeline)

Description
We are using SkiaSharp in Blazor WebAssembly (client side) application. When deploy Blazor client side application (.NET 6.0) to Azure App Service (Windows) using GitHub Actions (CI/CD Pipeline), SkiaSharp works fine without any issue. But when we deploy Blazor client side application (.NET 7.0) to Azure App Service (Windows) using GitHub Actions (CI/CD Pipeline) we are facing libSkiaSharp assembly not found issue.
Code
@page "/skiasharptest"
@inject Microsoft.JSInterop.IJSRuntime JS
@inject HttpClient client
@using SkiaSharp;
@using System.IO

Hello, world!

Click the button to draw the text using SkiaSharp.

<button class="btn btn-primary" @onclick="@Create">Create</button>

Status: @msg

@functions {
    public string msg = string.Empty;
    async void Create()
    {
        try
        {
            //Get the text to draw.
            String drawText = "Hello";
            //Create sKImageInfo object
            SKImageInfo sKImageInfo = new SKImageInfo(612, 792);
            SKSurface sKSurface = SKSurface.Create(sKImageInfo);
            SKCanvas canvas = sKSurface.Canvas;
            //Create font.
            SKTypeface sKTypeface = SKTypeface.FromFamilyName("Calibri Light", SKFontStyle.Bold);
            SKPaint paint = new SKPaint();
            paint.TextSize = 11f;
            paint.Typeface = sKTypeface;
            canvas.DrawText(drawText, 0, 0, paint);
            msg = "End";
        }
        catch (Exception ex)
        {
            msg = ex.ToString();
            msg += ex.Message;
            msg += ex.StackTrace.ToString();
        }
    }
}

Expected Behavior
SkiaSharp should work properly when deploy Blazor client side application (.NET 7.0) to Azure App Service (Windows) using GitHub Actions (CI/CD Pipeline).
Actual Behavior
The following exception is thrown.
System.TypeInitializationException: TypeInitialization_Type, SkiaSharp.SKImageInfo ---> System.DllNotFoundException: libSkiaSharp at SkiaSharp.SKImageInfo..cctor() Exception_EndOfInnerExceptionStack at Blazor_SkiaSharp.Pages.SkiaSharpTest.Create()TypeInitialization_Type, SkiaSharp.SKImageInfo at Blazor_SkiaSharp.Pages.SkiaSharpTest.Create()
Basic Information

Version with issue: 2.88.3
IDE: Visual Studio 2022 (17.5.0)
Platform: ASP.NET Core Blazor (client-side)
Target Frameworks: .NET 7.0
Deploy to: Azure App Service (Windows) using GitHub Actions

Reproduction Link
Attached both .NET 7.0 and .NET 6.0 samples along with workflow file,
Blazor-GitHub-Actions.zip
Reference videos for how to deploy app to Azure using GitHub Actions (CI/CD Pipeline)
https://www.youtube.com/watch?v=UpWeffxf790
https://www.youtube.com/watch?v=QP0pi7xe24s
https://learn.microsoft.com/en-us/azure/app-service/deploy-github-actions?tabs=applevel

Maybe it solves your problem:
https://github.com/mono/SkiaSharp/issues/2767


Find a better way to ensure visibility of variables in NodeView

If variable names are to long the label competely overlays the actual field value. We need to handle this pereferrably via .uss

should be done in user code via custom stylsheet as this can depend on the use case


Center map on startup!

Right now, the default coordinates (0, 0) are not very useful for many resolutions, and most of the markers start off the end of the map.
We should start the map at the center of the markers we know about instead, something like:

lat: (max(lat) + min(lat)) / 2)
lng: (max(lng) + min(lng)) / 2)

Relevant lines here: https://github.com/reach4help/reach4help/blob/master/map/src/components/map-utils/google-maps.ts#L7

Hi @s0,
I think we'd better use the average.
Here are the results for both on a smaller screen and a wider screen:
Case 1:

lat: (max(lat) + min(lat)) / 2)
lng: (max(lng) + min(lng)) / 2)

Case 2:
lat: average(lat)
lng: average(lng)

@btuerker hmm okay that makes sense, thanks for the screenshots. Please feel free to do averages instead and open a PR :)


Fold consecutive const items as use statements in VSCode

In latest rust-analyzer extension of VSCode. It is possible to make use statements folded as below.

I think const(and static) items should have the same behavior.

Implemented in #8231


[core] Fix that cannot read UPDATE_BEFORE of ignore-delete table

Purpose

To maintain compatibility with 0.7, #3139 put ignoreDelete in reader, causing that UPDATE_BEFORE cannot be read in streaming reading. So the ignore-delete have to be put into merge engines.

Tests

API and Format

Documentation

+1


Poor "snabb config get lwaftr /" performance with multiple instances and ietf-softwire-br

The v2017.11.01 release fixed a number of performance problems related to "snabb config get".  However there are still some remaining problems related to schema translation, and that hits us in an obvious way when the user does a "snabb config get" without specifying a --schema argument, as in that case ietf-softwire-br is the default schema.

If you configure 10 devices, the ietf-softwire translation ends up translating those to 10 separate instances.  However each of those instances has its own binding table, which is a problem because that kind of flexibility is not supported by the snabb lwaftr.  The whole point is that a fleet of instances has just one binding table.  We should change the schema translation to produce just one binding table.

Evidently the schema translation is consuming too much memory.  Users report seeing "snabb config get lwaftr /" slow down, issue warnings about hugepages, and eventually crash the manager process (!), taking down the whole lwaftr dataplane.  This needs fixing.

Possibly related to https://github.com/Igalia/snabb/issues/948

Not related to #948 AFAIU, this was with a binding table with only 64K softwires and the v2017.11.01 release fixed many issues related to #948 (though not all of them).


show docs in codegen server pallet path completions

Unfortunately, FRAME Metadata does not currently expose pallet-specific documentation (see paritytech/frame-metadata#47).
As an experiment, I attempted to simply fetch from the pallet-name-corresponding Readme of the paritytech/substrate repo. This is of course not a viable solution, as it doesn't account for version nor differences between chain-specific pallets / pallets with the same names as those in FRAME.
Is there a good way of determining pallet documentation source? If so, I believe the experience is merited.

No longer relevant as we don't have completions anymore


Readme update

just wondering if there are some typos like
var limiters[id] = new RateLimiter({
id: id,
limit: 100, // 100 tokens
duration: 60000 // 60s duration
}, redisClient);
and if you have to require the library as well at the top?

+1, when I copy your sample code (plus including the library, using var, properly getting a redis client, etc.) I get an error Cannot read property '1' of null from line 134 for index.js.

The return of
this.redisClient.multi()
    .get(this.storageId)
    .pttl(this.storageId)
    .setnx(this.storageId + ':lock', 1) // lock the record for other workers
  .pexpire(this.storageId + ':lock', LOCK_TTL) // set expire in case this program crashes
  .exec(onResult.bind(this));

is of the form [null,-2,1,1] right now, which means that res[0][1] is bad. Looks like all the returns from multi().exec() are integer or string literals and the module is treating them like nested arrays/objects

+1 getting this too

You might want to try the version 0.1.3 if you're getting some redis-related errors. I will try to fix or revert the changes in 0.2.0.


Support for kubernetes 1.15

Signed-off-by: Christian Simon<EMAIL_ADDRESS>Support for kubernetes 1.15

Can you have a look at the diff
/assign @JoshVanL

/test verify quick

Looks fine to me
/lgtm


Error initializing AppsFlyer SDK: Code 41 - No dev key

`    const afDevKey = "5dhdKJKvd8******gwiS";
const appId = "";
AppsFlyerOptions appsFlyerOptions = AppsFlyerOptions(afDevKey: afDevKey,showDebug: true);
AppsflyerSdk appsflyerSdk = AppsflyerSdk(appsFlyerOptions);

appsflyerSdk.startSDK(
  onSuccess: () {
    _showMessage("000=>AppsFlyer SDK initialized successfully.");
  },
  onError: (int errorCode, String errorMessage) {
    _showMessage("000=>Error initializing AppsFlyer SDK: Code $errorCode - $errorMessage");
  },
);

`
When I integrate according to the document, prompt for title information
appsflyer_sdk: ^6.15.2
Android version: 10

devKey: 5dhdKJKvd88sRm4JAegwiS
安卓包名： com.short.play
✅ your AppsFlyer sign-up (account) email
1512866700aqq.com
✅ app ID
Android 无需提供（com.short.play）
✅ production steps
运行项目，执行方法step1, step2, step3, step4
✅ logs
查看附件 log.txt
✅ code snippets
查看附件 code.zip
------------------ 原始邮件 ------------------
发件人:                                                                                "AppsFlyerSDK/appsflyer-flutter-plugin"                                                                                @.>;
发送时间: 2024年12月27日(星期五) 晚上7:17
@.>;
<EMAIL_ADDRESS>主题: Re: [AppsFlyerSDK/appsflyer-flutter-plugin] Error initializing AppsFlyer SDK: Code 41 - No dev key (Issue #366)
👋 Hi @KingLjk and Thank you for reaching out to us.
In order for us to provide optimal support, please submit a ticket to our support team at @.***
When submitting the ticket, please specify:
✅ your AppsFlyer sign-up (account) email
✅ app ID
✅ production steps
✅ logs
✅ code snippets
✅ and any additional relevant information.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>


Kind property of function app service in ARM template Function.json is wrong.

The kind property in Templates/Function.json is set to app which creates a app type instead of a function app service when executing DeployInfrastructureToAzure.ps1.
  "resources": [
    {
      "type": "Microsoft.Web/sites",
      "apiVersion": "2018-11-01",
      "name": "[parameters('webSiteName')]",
      "location": "[resourceGroup().location]",
      "dependsOn": [
      ],
      "kind": "app",
      "tags": {
        "displayName": "Function"
      },

it has to be changed to functionapp so the app can be deployed.

Sorry for the delay in responding @ckadluba. Thanks for the update - running the CI/CD now.


Display mini reports by workstream on open/closed application pages

Description of change
Changes the volumes report and workload report to filter by work stream.
Note this is currently only changing the view on the open/closed applications pages to display the correct report data based on the work stream tab being selected. On the open applications page, this is the workload report. On the closed application page, this is the volume report. The report pages will be changed in a separate PR.
As part of this PR, work_stream and reviewed_on are added to the review read model to set the work stream value on application received events and to scope the volume report data by the reviewed date
Link to relevant ticket
CRIMAPP-231
CRIMAPP-232
Notes for reviewer
Must not be deployed without: https://github.com/ministryofjustice/laa-criminal-applications-datastore/pull/188#pullrequestreview-1757973222 which adds work stream to the application submission sns event
Screenshots of changes (if applicable)
Before changes:
The volume and workload reports on the open/closed application page currently shows data across all work streams

After changes:
The volume and workload reports on the open/closed application pages now show data by work stream (just showing extradition and CAT tabs for now as there are no NCT cases)

How to manually test the feature
Requires a extradition and CAT case to be open and an extradition and CAT case to be closed
Navigate to the open applications page. Confirm that you see the right data mini report as per the selected work stream
Navigate to the closed applications page. Confirm that you see the right data mini report as per the selected work stream
Turn off the feature flag locally and confirm that you see all the application data in both mini reports

Thanks for the review @willmcb we're going to make these changes in the next PR when we look at user feedback.


docs(readme): comparison with laurocks

Closes #208

Test failures appear to be a flake, perhaps luarocks is down?


Create react-app/src/contracts directory if it does not already exist

This fixes the script so it doesn't blow up on the first run

The contracts directory will always exist with the external_contracts.js file in it @nullbitx8

@codenamejason perhaps I had something messed up in my local environment when I opened this. Thanks!


[cxx-interop] Fix unavailable generics triggering compilation error

In some cases, the reverse interop generated both a forward declaration and a definition with unavailable attribute in the C++ header. Unfortunately, the kinds of these symbol did not match. The forward declaration was templated while the definition was not. The forward declaration has the correct kind, so this patch extends the printing of unavailable definitions to include the generic arguments.
rdar://119835933

@swift-ci please smoke test


Update plugin-install-manager, move jar to /opt

Fixes https://github.com/jenkinsci/azure-ad-plugin/issues/121
See https://github.com/jenkinsci/azure-ad-plugin/issues/121#issuecomment-867919247

[2021-06-25T08:56:18.669Z] # -- command failed --

[2021-06-25T08:56:18.669Z] # status : 1

[2021-06-25T08:56:18.669Z] # output (2 lines):

[2021-06-25T08:56:18.669Z] #   touch: cannot touch '/var/jenkins_home/copy_reference_file.log': Permission denied

[2021-06-25T08:56:18.669Z] #   Can not write to /var/jenkins_home/copy_reference_file.log. Wrong volume permissions?


Implement TypedColumnView

YOLO implementation of typed wrapper for gtk::ColumnView. Largely based on TypedListView, but modified to have per-column factories as required by ColumnView API.

Thanks!


Initial import of Angular SPA

This PR is for #DOCS-CODE-022

Thanks for the suggestions, I removed the environment stuff and cloned the branch into a clean area, followed the steps, and was able to get it all running as expected. Ready for review again.

Just two open questions above.  Nothing blocking though, so if you want to start to get CD feedback, feel free!

One open question from my prior review, but nothing blocking.  If you wanted to get CD starting to look at this, you can.


Documentation for space transformation, space and group

Codecov Report
Base: 75.20% // Head: 75.20% // No change to project coverage :thumbsup:

Coverage data is based on head (6432db9) compared to base (ae4648f).
Patch coverage: 90.90% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #114   +/-   ##
=======================================
  Coverage   75.20%   75.20%
=======================================
  Files          35       35
  Lines        2500     2500
=======================================
  Hits         1880     1880
  Misses        620      620

Flag
Coverage Δ

unittests
75.20% <90.90%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

nndt/space2/group.py
100.00% <ø> (ø)

nndt/space2/transformation.py
80.39% <83.33%> (ø)

nndt/space2/space.py
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Mlr new teleop

remove https://github.com/ipa320/cob_robots/pull/332/files#diff-7c6e22d8835a05697c8cd5c90c65df13 swap

There should be only one copy of Elmo.dcf, the one in common/.
The robots should symlink the common DCF.
I haven't found a proper version of Elmo.dcf yet.

https://github.com/ipa320/cob_robots/blob/indigo_dev/cob_hardware_config/common/Elmo.dcf looks fine already and can be symlinked.

please consider the comments above before merging...also we should check that the cob4-3 hardware config is up to date with the latest hardware config for already existing robots, i.e. there might have been PRs that affect configs that have not yet been applied to cob4-3...(I have e.g. light, diagnostics yamls in mind...)

Changes are made. Please @ipa-mdl  review  the commits:
https://github.com/ipa320/cob_robots/commit/2163b1f834327b5db5568055a652a8f69944ecb4
https://github.com/ipa320/cob_robots/commit/f8f884cc3c165d2c372b3a2c6270f4fc64d381bc

Why is the new teleop feature mixed up with other changes?
Can we split them up? To keep things tracked properly..

The problem was that the delivered version of the MLR bases was not "mergeable" because of the teleop node , we had teleop_v1 and teleop_v2 and the MLR bases needed a mix from both of them. We decided to wait until we have a new teleop node, to merge all the cob4-MLR (hardware_config) changes. I know , I should open a pull request per feature, but I hadn't got before a mergeable and working version of cob_robots for the MLR platforms...

Then we should merge the config for ONE cob4 base first (with full review).
And afterwards we can just review the changes for each robot.
I still not get why we copy everything..
For example the defaults in base_driver.yaml are defaults for all new bases, including the CAN setup.
Which should not be separated from other settings unless it is shared by two different components.
BTW, "Merge" is no proper commit message.

Ok, openned new pull request only for cob4-4 robot https://github.com/ipa320/cob_robots/pull/345

@ipa-mdl  What do you suggest that we can do? links between the configurations? The cob4-3 , cob4-4 and cob4-6 is a special case, we don't suppose that in the future we will have this similarity between the robots.

Not for robots, but for components.
We could have a common config for the torso, arms and even for the base (except for the calibrated offsets).

as I merged #345, this PR has merge conflicts. @ipa-nhg: please resolve.

This pull request has to be closed. We decided open a pull request for each MLR robot , I keep it open because of the comments.


DB Migration

Create a system to ensure db migrations are handled, seed dummy data in development environment.

For the migration script, the student should on delete cascade when the session is deleted, also the testcase table should have a field for testcase_number.


Fill out_bundle with sent transaction obj

Fill the output brunle object with the stored trytes from att_res
Fixes #1254

Please don't forget to prefix your PR name with the component affected because the PR name will be used as squashed commit message so we want them uniform :)


Translated new strings to Dutch

Translated the following new keys to Dutch:

Dash.Screens.Desktop
Inspector.AudioStream.*
Desktop.*

Thanks for the additions!


Exporting packing and signing functions to the Raiden client

Currently both raiden-contract and raiden implement the same functionality of packing and signing onchain-submitted messages.
At least

pack_balance_proof
pack_balance_proof_update
pack_reward_proof
should be moved to raiden-contracts.

Perhaps

Signer
LocalSigner
can be moved to raiden-contracts too.

Also the sign() function should receive a message, not a message hash.  And then use keccak as the hasher when calling functions from coincurve.

https://github.com/raiden-network/raiden/pull/4284/files introduces _data_to_sign method.  See if it's merged and then think.

Since packing has to be done in all cases before a signature is generated, packing can be introduced as a shared lowest common denominator. Here's a specific suggestion using the reward proof signature as an example:

have a pack_reward_proof in the contracts repo
import that from raiden and wrap it inside a raiden-specific pack_reward_proof function to accomodate the different conventions (CanonicalIdentifier, more detailed types)

This approach is neither the most elegant, not does it provide a large amount of code reuse. But it is a suitable stepping stone for further unifications and when we change the data that goes into the proof, we will get a mypy error instead of failing signature checks. Since debugging signatures can get hard quickly, this advantage justifies the effort of implementing this change, as far as I am concerned.

@karlb let's do that.  Assign yourself this issue before starting (so as to avoid duplicate work).

I will do this for the reward proof first, since I have to touch that due to https://github.com/raiden-network/raiden-contracts/issues/1117. If that works out fine I'll expand that approach to other signatures.

Remaining candidates are

[ ] pack_balance_proof
[ ] pack_balance_proof_update
[ ] pack_withdraw

The cooperative_settle_message is not used by the client and sign_one_to_n_iou is already in use. If I'm missing anything, let me know.

https://github.com/raiden-network/raiden/pull/4423 will be ready for review after the updating the contracts used in raiden. @pirapira Any recommendations on next steps?

@karlb no, I haven't looked at this closely enough.  I said something about Signer and LocalSigner above, but take them as random ideas.
I don't mind huge construction work during my vacation.  Usually I prefer iterative small changes, but we have high latency of code integration, so you can go big at once if you prefer.

@karlb talk to @LefterisJP before making big changes.  He might not accommodate big changes until the next mainnet release.

I'll just get https://github.com/raiden-network/raiden/pull/4423 merged and then leave this ticket until the next mainnet release.

The packing functions are shared now. Signing stays as it is until we need changes to signing something specific.


Profile stats not updating

Somehow my profile stats stopped updating?

I think this might happen if your login session to RED expires?
Have you tried clearing the login cache? Instructions here: https://github.com/karamanolev/WhatManager2/wiki/Troubleshooting#failed---exception-error-uploading-data-to-whatcd-errors-no-further-info

Didn't help :/

I have also run
./migrate clear_what_login_cache
With to no success

Seems to have fixed itself after I changed a lot of stuffs


How to install Bolt apps when OpenID Connect is enabled?

When I call asOAuthApp(true) on App instance I can normally install app via browser. When I call asOpenIDConnectApp(true) instead I get invalid scope requested when trying to install app via browser.
Bot scopes I am requesting: commands,chat:write,chat:write.public,team:read,usergroups:read,users:read,channels:read,groups:read,mpim:read

👋 Let us know if you need more help here. If we don't hear back for a few more days, we'll close this issue.


https://<IP_ADDRESS>/_content/MauiBlazorWeb.Shared/MudBlazor/MudBlazor.min.js Error 404

Things to check

[X] I have searched the existing issues for this bug
[X] To rule out a caching problem I made sure the bug also happens in an incognito tab

Bug type
Component
Component name
No response
What happened?
Failed to load resource: The Server responded with a status of 404
https://<IP_ADDRESS>/_content/MauiBlazorWeb.Shared/MudBlazor/MudBlazor.min.js
and the screen does not show properly.

Expected behavior
that it shows the correct way.
Reproduction link
https://github.com/whann0205/MauiBlazorWeb.git
Reproduction steps

Clone from https://github.com/dotnet/blazor-samples the Project: MauiBlazor Web
Add nuget package to the Shared Library
Make settings as usual (index.html, as described in the homepage)
...

Relevant log output
No response
Version (bug)
both versions
Version (working)
No response
What browsers are you seeing the problem on?
Firefox
On which operating systems are you experiencing the issue?
Android
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hi. I suggest creating the issue in the aspnetcore/maui repo. MudBlazor doesn't handle how assets like JS and CSS are loaded. We just need to provide them in the NuGet package in a specific folder. The rest is handled by Microsoft.

Hi. I suggest creating the issue in the aspnetcore/maui repo. MudBlazor doesn't handle how assets like JS and CSS are loaded. We just need to provide them in the NuGet package in a specific folder, so we do not have control over it and hence can't help you as the rest is handled by Microsoft.

Tho, it doesn't seems like you have added these references anywhere:
<link href="_content/MudBlazor/MudBlazor.min.css" rel="stylesheet" />
<script src="_content/MudBlazor/MudBlazor.min.js"></script>

From the getting started section. That something that user should add on their own.

No, i have it. Its in index.html under the Project
MauiBlazorWeb.Maui

To be honest i am not perfectly happy about your answer regarding
the advice to post it in aspnetcore/maui repository.
Blazor native works without an issue for example

which is also javascript but does not raise an error.
So i am afraid that guys in aspnetcore will be sending me back
to here telling me to inform the mudblazor repo, as a round trip.
So maybe someone here can take a look, since it will be future
for cross plattform programming in the net maui world and
i am sure that i am not the only one to have their razor for web
and mobile devices at the same time.

i am not perfectly happy about your answer

That doesn't change the fact that as long as we provide "staticwebassets" inside the NuGet package, we're out of the picture. I hope you don't take this as an arrogant response—it's just how it works. So, there are only two possibilities here: it could be a user error or a Microsoft bug (which isn't uncommon).
If you re-read my message more attentively:

Though, it doesn't seem like you've added these references anywhere:
<link href="_content/MudBlazor/MudBlazor.min.css" rel="stylesheet" />
<script src="_content/MudBlazor/MudBlazor.min.js"></script>

and look at your screenshot again, you'll notice that you've added them incorrectly:
You added:
<link href="_content/MauiBlazorWeb.Shared/MudBlazor/MudBlazor.min.css" rel="stylesheet" />
<script src="_content/MauiBlazorWeb.Shared/MudBlazor/MudBlazor.min.js"></script>

instead of:
<link href="_content/MudBlazor/MudBlazor.min.css" rel="stylesheet" />
<script src="_content/MudBlazor/MudBlazor.min.js"></script>

You mistakenly appended MauiBlazorWeb.Shared. And the reason I couldn't tell right away is that I ran a search function instead of looking directly at the source code.
Once you correct that, the resources load fine (I checked personally).
I suggested reaching out there because I'm confident it's not an issue on MudBlazor's side. Which means you're more likely to get the support you need from people who work more with MAUI. We deal very little with MAUI directly and assume that if the web version works, so should MAUI.

Also. since you are running v6 not v7, please use the v6 docs instead: https://v6.mudblazor.com/getting-started/installation
You do not need to use MudPopoverProvider and MudThemeProvider together, the MudThemeProvider is enough. The separation happened in v7.

Hi ScarletKuro,
Please excuse my behavior. Its just that its sooo hard to get information from somewhere how it is meant to be.
I was simply guessing that internally the adresses get resolved that way for mudblazor too since the microsoft ones
looks that way. Now i did my tests again with the original entries and now it works!! I thought i checked it
but it seems to be i did not.
Thank you very much !!
Maybe its possible to note somewhere in the installation that the entries are meant to
be as they are and not to be reinterpreted to stop other from falling into this pitfall.


Add option in groups to define the runner type

Currently, we have only one option of runner to execute the integration_test_*.yaml, the ubuntu-latest.
However, certain workloads need different types of runners, for example bigger runners with more capacity. We need a way to specify as an option if a given group should run on a special type of runner, instead the default one.

Prioritized your request for @carlcsaposs-canonical


[HUDI-2285] Metadata Table synchronous design

What is the purpose of the pull request
Implementation of the metadata table synchronous design.
Brief change log
Please see HUDI-2285 for changes and how they are implemented. There are different commits that handle each change.
Verify this pull request
Metadata table unit tests have been updated.
Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Closing in favor for #3590

@nbalajee @nsivabalan lets resolve the CR feedback there and land.


fix #39 #37

closes #39 and probably  #37

Hey!
Thank you for sharing this fix!
But I had an Issue with it! I had to change the code to:
for (let k = matchedIndices.length - 1; k >= 0; k--) {

Hey!
Thank you for sharing this fix!
But I had an Issue with it! I had to change the code to:
for (let k = matchedIndices.length - 1; k >= 0; k--) {

Hi @alexiusacademia !
Would you kindly merge this? I've tried to use deleteRow and it does not delete, stating that does not have a row matching with my condition, although it does.
My conditions was simply "{ "id":<PHONE_NUMBER> }", but it does not accept it.
Thank you.


E2E test for state management.

Description
E2E test for state management: get, save, delete
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #818
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[X] Code compiles correctly
[X] Created/updated tests
[X] Unit tests passing
[X] End-to-end tests passing
[ ] Extended the documentation
[ ] Specification has been updated
[ ] Provided sample for the feature

/ok-to-test

Ready to merge?

I have not pushed the changes yet. My VM is not playing nice with me now.

/ok-to-test


OpenDocument support

It would be nice to be able to use ods spreadsheets instead of the proprietary xls format.

Interesting idea to support other use cases than nametags for Meetup RSVPs. I'll look into this. However please notice that the file format is not really XLS but CSV (the Readme now states this explicitely).