Donaliu/20200207 sap monitor

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

Can one of the admins verify this patch?

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

@ryansbenson please respond to review feedback from the author

/azp run automation - sdk

/azp run automation - sdk

@ravbhatnagar Please answer the remaining question to your feedback

@PakDLiu One comment, verified that ARM feedback was addressed.  Please take a look

/azp run automation - sdk


Blog file name logic

See if there's a more elegant solution for coming up with a valid page name

Given we've an overwrite option checking upfront if a page exists and then coming up with another name makes no sense. Closing this one


Add files via upload

check for release and autorelease as well as for retain

Looks fine, please can you fix the commit message (force push to the branch to overwrite it) and add a test (add a release method that sets a flag to a class and then call objc_release on an instance of it and check that it is called).


Crashes when file in project is edited outside of Atom (Windows)

Crashes in Windows 7 when a file is changed outside of Atom. if you move or add or delete a file in Windows Explorer it will cause Atom to Crash,
This also happens if the file is on a windows server share (\server\share\projectDir) and another user changes a file in the project dir

Unable to reproduce this - it's possible it's been fixed in the meantime. If you still experience this in Atom 1.7 please re-open.


fix(curriculum): Changed Mongoose version to avoid warnings in boilerplate's Node console when working on Mongoose challenges

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

As mentioned in this PR's comments, I found simply removing mongodb and changing mongoose version to ^5.11.15 gets rid of all the warning messages in the Node console and does not affect any of the challenges in the MongoDB and Mongoose section challenges.  See related boilerplate PR where I change the package.json to remove mongodb and change the mongoose version to ^5.11.15.
This PR updates the instructions and tests to work with the changes in the corresponding boilerplate PR (mentioned above).

This is a good idea.
I wonder if we should update the block description:

Mongoose is a popular npm package that is often installed alongside Mongo.

Probably.  What did you have in mind?

Update:  I have confirmed using a replit, that installing<EMAIL_ADDRESS>without also installing mongodb as a dependency does not affect the project.  I was able to use my same code that passed with both<EMAIL_ADDRESS>and<EMAIL_ADDRESS>with no warning messages and passed all the challenges.

Probably. What did you have in mind?

Maybe something like "Mongoose is a popular npm package for interacting with MongoDB"?

we could rename the section to just Mongoose

I think the course name is okay - we are teaching campers about MongoDB. It's just the mention of the mongo package that needs to be removed.

Maybe something like "Mongoose is a popular npm package for interacting with MongoDB"?

I like it.

Sure I can do that!

LGTM 🎉, thank you for your contribution to the page! 👍.


pkgs: Made git-crypt installable

Things done:

[ ] Tested using sandboxing (nix-build --option build-use-chroot true or nix.useChroot on NixOS)
[x] Built on platform(s): Linux
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

More
Fixes issue #
cc @

Please note, that points are not mandatory, but rather desired.

git-crypt is already installable via pkgs.gitAndTools.git-crypt or by its name git-crypt.

@hrdinka this PR is ok anyway.

Weird - now that you said it, it works. I could have sworn it did not work before. Any idea why it is not found by hydra?
http://hydra.nixos.org/search?query=git-crypt
Also, what does my patch accomplish then? (I don't fully understand nix yet)


RedirectResponse

I need to return different RedirectResponses after form validate.
Example: dependent on the submitted button...
if ('POST' === $request->getMethod()) {
            $form->handleRequest($request);
            if ($form->isValid()) {

                $this->crudCreatePrePersist($object);

                $em = $this->crudManagerForClass($this->crudObjectClass());
                $em->persist($object);
                $em->flush();

                $this->crudCreatePostFlush($object);

                $this->crudFlashMessage($request, 'success', sprintf('%s.create.flash.success', $this->crudName()));

                if ($form->get('submitadd')->isClicked()) {
                    return new RedirectResponse($this->generateUrl('instructor_create'));
                }

                return new RedirectResponse($this->crudCreateRedirectUrl($object), 302);
            } else {
                $this->crudFlashMessage($request, 'error', sprintf('%s.create.flash.error', $this->crudName()));
            }
        }

@Gladhon what would you do to solve this issue?
Inject the form within each method, that its possible could be a solution

We are in the context of a CRUD-Controller, so we obviously have  only one form. Why not add and crudGetForm method, wich get back the current form.

Its probably a good idea to solve it with a single object as parameter as for example events does. $parameter->getObject(), $parameter->getForm(), $parameter->getRequest() on each method (override, or hook)

this is a to big change for a rc, which means this feature will be in v2

This should solve the problem: https://github.com/saxulum/saxulum-crud/blob/master/src/Controller/CrudTrait.php#L105


feat: emit semconv 1.27 metrics

This is a draft while I finish tests and docs. Pushing so that people can start reviewing the implementation while I wrangle tests.

This is just for instr-http, right? May I update the PR title to reflect that?


[Azure Firewall - Azure Firewall Premium] API Review

New API Review meeting has been requested.
Service Name: Azure Firewall - Azure Firewall Premium
Review Created By: Ben Eshed @bewatersmsft
Review Date: 08/09/2023 09:00 AM PT
Onboarding Record:
PR: https://github.com/Azure/azure-rest-api-specs/pull/24638
Hero Scenarios Link: Not Provided
Core Concepts Doc Link: Not Provided
Description: Someone initially replied to our questions here https://github.com/Azure/azure-rest-api-specs/pull/24638#discussion_r1271401866 , but we did not receive a response.
Detailed meeting information and documents provided can be accessed here
For more information that will help prepare you for this review, the requirements, and office hours, visit the documentation here

@bewatersmsft You have scheduled this API review with the Azure data-plane review team, but your PR appears to be for a management plane API. Did you link the wrong PR? If so, please fix. If not, please cancel your review meeting to make this slot available to other teams needing a data-plane API review.

@mikekistler
Where does it say it's data plane? I tried to use the scheduling wizard. This is 100% a management plane request

I also emailed the armapireview email and was told to schedule this meeting. What is the proper procedure to get someone in a meeting to tell us how we can comply?

Cancelled by: Ben Eshed


Cannot get columns for sqlite with Literal column type

That said, it's not clean to just assume that getPhinxType returns a string for key "name" and then pass that to parseDefaultValue in getColumns. This leads to a type error when "name" is a Literal. Instead, a meaningful exception should be thrown in that case that mentions the table name, column name, and given type (string). But that's another issue.

Originally posted by @InvisibleSmiley in #2319

An example of a statement to create a table that uses a literal:
CREATE TABLE `test` (`real_col` FOO)

This currently causes phinx to throw an exception when doing getColumns.


Fix typo in GOV.UK Frontend version number

Version number detected but only returns a because of typo copied from example in docs.
Apologies for this minor update, I'm frustrated I didn't spot it sooner.

Thanks!


lack of documentation of Application object

Need to add documentation to explain how to use the Application Object and how it works.
apiVersion: core.oam.dev/v1alpha2
kind: Application
metadata:
  name: application-sample
spec:
  components:
    - name: myweb
      type: worker
      settings:
        image: "busybox"
        cmd:
        - sleep
        - "1000"
      traits:
        - name: scaler
          properties:
            replicas: 10
        - name: sidecar
          properties:
            name: "sidecar-test"
            image: "nginx"

fixed by https://github.com/oam-dev/kubevela/blob/master/docs/en/application.md

@wonderflow Note that there are some TBD left for you in the existing doc ^^, please fix them.


Delete file with Directory Traversal vulnerability

You can delete files on the system.if you were to go to:

https://domain.com/laravel-filemanager/delete?working_dir=%2F146&type=&items=../../robots.txt

if you replace robots.txt with files on the system.It was actually deleted . You can essentially break out of the application and affect the actual server
the vulnerability in src/Controllers/DeleteController.php.

Fixed in v2.2.0, thanks for reporting this.


Attempt to fix the Array.prototype.includes and String.prototype.includes polyfills (issue 9514, 9516)

I don't understand why the previous way importing the polyfills didn't work, and I don't have time to try and figure it out, however this patch seems to fix things.
Fixes #9514.
Fixes #9516.

/botio-linux preview

Squashed two bugs in one PR, good one!


Attempting to call Fields() on instance of SS_HTTPResponse

On a subclass of AssetAdmin in its getEditForm() method, I can call parent::getEditForm($id, $fields); and find it's possible for the framework to return an instance of SS_HTTPResponse rather than the expected Form if a canView() check up the chain in LeftAndMain fails.
Looking in LeftAndMain::getEditForm() checks are failing on line 1174 and I'm being returned an instance of SS_HTTPResponse rather than Form. This means I get the error:
Fatal error: Call to undefined method SS_HTTPResponse::Fields() in /var/www/htdocs/my-project/cms/code/controllers/AssetAdmin.php on line 158

Besides the fact that LeftAndMain::getEditForm() is documented as returning 'Form` which is not really true as it stands; a potential fix might be to return a Form but with an error message being displayed replacing any form fields.

So that's https://github.com/silverstripe/silverstripe-framework/blob/3/admin/code/LeftAndMain.php#L1209, right?
We should probably throw an SS_HTTPResponse_Exception at this point in the logic, and handle it more gracefully through Security:: permissionFailure() on the caller side of getEditForm(). The permissionFailure() call has the advantage of redirecting users to a login form, rather than showing a fairly generic/unhelpful error screen as SS_HTTPResponse_Exception would. show() would be a good starting point for this. We want to avoid duplicating the $record->canView() call though - is that cached? Sorry, no ready-made solution here, just providing some pointers :)

@chillu thanks for the response.
L1209 correct, yes :-)
Totally get what you're saying re: login screen, I'll refactor a new PR and let you know

@chillu OK, different, less jarring solution, which - technically speaking is correct as far as current phpDoc is concerned :-P
    if($record && !$record->canView()) $this->redirect('/admin/pages', 403);

The permissionFailure() call has the advantage of redirecting users to a login form, rather than
showing a fairly generic/unhelpful error screen as SS_HTTPResponse_Exception would

Hmm but then your logic is moot if the user is already logged-into the CMS. What help would re-logging actually achieve, given that the route will have the BackURL parameter in there, potentially causing an infinite loop?
Thoughts?

LeftAndMain and AssetAdmin have changed too much since the issue was raised. This issue is probably not a factor anymore.


Support for latest AutoMapper and NewtonSoft packages

No support for the latest packages makes it difficult for everyone to update their entire projects.
This has been the case for a long time.
Can you simply update the dependencies when you receive notifications?
It would make developers lives a whole lot easier.

Hi Steve,
Thanks for raising the issue.
Sorry to be a bit slow to keep up with the dependency versions. We should be able to do an update with support for the latest NewtonSoft JSON version soon, however AutoMapper has some changes that break some mappings so some reworking is needed before we can update that.

@incarnate any update on the AutoMapper mapping issue, having mapping issues with my project.
All my projects in my solution uses AutoMapper 5.1.1 except for one project which uses eWay Rapid with AutoMapper version 3.3.1. However, there is an exception thrown by AutoMapper through the eWay Rapid.
Any work around to solve this issue. I can not use AutoMapper 3.3.1 for my entire solution that would require tons of changes to my solution.

Hi Rajsanthosh - I've just released version 1.5.0 to move AutoMapper up to version 4.1.1 which is the last release before some incompatible changes were made. I still haven't had a chance to assess what work is needed to make the SDK compatible with AutoMapper v5+, so can't provide a timeline just yet.

A beta release of eWAY .NET SDK version 1.6.0 is now available with AutoMapper 5.1.1 support.
There are some issues with mapping a customer's address in responses but otherwise everything else appears to work fine. If anything else isn't mapping properly please open an issue to let us know.

This is a common issue incarnate! Your libraries when not updated, prevent all developers using eWay.Rapid from upgrading their projects and force them into your lowest common dependencies.
Please keep this in mind - it affects us all the time as it prevents us from upgrading.
As for AutoMapper issues - really!? It is a pretty basic library based on conventions.
It would be worrying if you couldnt sort things out with this.
FYI - speaking with support today, I was surprised they could SYMMETRICALLY DE-CRYPT my password access to the system. Would have assumed eWay had HARDENED their security more than that and used asymmetric, one-way hashing encryption for the systems. #thumbsdown

Why is this still using automapper 4.1.1 ???
Version 6.1.1 is out and I cannot use eway rapid....

https://github.com/eWAYPayment/eway-rapid-net/pull/26
Pull request made for this issue with updated references and porting to the new automapper convention. All the tests pass.
I'm concerned that firstly this issues register is not monitored and secondly that no one has made any effort to keep the SDK up to date. Braintree and swype manage to keep their .NET repos up to date.

Sorry again for the big delay in getting the PR released - version 1.6.0 of the eWAY .NET SDK now uses the latest versions of AutoMapper and NewtonSoft JSON.


Justification of text

Current behavior
The text will be justified from both the ends in the home page and the about page  but if you need some changes about justification let me know  (#182 )
Proposed changes
I did text justify to justify from both the ends
Before:

After :

Checks

[x] All commits in this Pull Request are signed and Verified by Github.

Hii @Yash636261  plz add gsoc and level label

Hii @Yash636261  please put gssoc and level label to be counted for leaderboard


Add "Eclipse Orion" to "Real-World Uses".

Eclipse Orion has switched from term.js to xterm.js because of better AMD support and the active maintaince.

Coverage remained the same at 68.708% when pulling 938c011e634bef54ac5b95bdaa4c10fd7fed7e70 on Martin1994:orion_use into 66f6b883332321ef881c0ddfa1f70103d707b1ff on sourcelair:master.

Thanks!


Critical bug - big switch

After add one JTranscKeep, i get stable crush libc++abi.dylib: terminating with uncaught exception of type Dynamic before it i get one time of 10 launches. I think JTranscKeep add in ProgramReflection one or more cases and function size again bigger than 64 KB.
@soywiz Can you add in JTransc mechanism for detect big functions and huge switches in compile time (not runtime)?
P.S. It's not my level =)

Can you provide an example? (So i can use as test)
Also I cannot use 64K in size as an indicator since im not generating c++ direclty but haxe first in this case. I can use number of statements, nodes or other things.
You want me to just print a warning in this case or try to split it somehow?
This just happens inside ProgramReflection? Or outside it? Which method inside ProgramReflection is bigger than 64K?

Maybe you can provide your ProgramReflection file with minimizeNames=true and searching for all ".*" and replacing with garbage so no confidential information is provided. I need some context or i wont be able to fix it

 ::JA_L ProgramReflection__obj::getFieldAnnotations_II__Ljava_lang_annotation_Annotation_(int p0,int p1){
            	HX_GC_STACKFRAME(&_hx_pos_730ed619cd3a2627_1264_getFieldAnnotations_II__Ljava_lang_annotation_Annotation_)

14 800 lines
as example line
            			case (int)7984: {
HXLINE(10960)				if ((p1 == (int)33472)) {

Maybe you can provide your ProgramReflection file with minimizeNames=true and searching for all ".*" and replacing with garbage so no confidential information is provided. I need some context or i wont be able to fix it

I try provide need time for recompile

Maybe i can create a method per class. But if you can provide an example file today, it will be easier for me, i will be able to work on it this evening

ZD.zip
provided ProgramReflection from ios target.

Thanks! I will look at this this evening

It seems that the main problem is here:
static  public   function hv/*getFieldAnnotations*/(p0:Int, p1:Int):JA_L {

That's the bigger function.
What I'm going to do first is this:
		out = new JA_L(1, "[Ljava.lang.annotation.Annotation;");
		out.set(0, (new z.LF()).i4(__str1, true));
		return out;

--->
return JA_L.create1("[Ljava.lang.annotation.Annotation;", (new z.LF()).i4(__str1, true));

Probably this will be enough for now.
But I can move each case for each class into a separate method. That would probably help since the number of methods a class can have is 64K I think. And probably you won't have classes with that many methods or fields.

more than 64K methods we have on each project last 5 year =)). But i believe, for reflection we can have this limit =)

I'm working on it. I almost have the first part. I will make a PR before going to sleep so you can test later or tomorrow.

Thanks!

@soywiz Have you progress?

Not too much time. But this weekend i will have some time to fix it

Thanks!


[Spell] Spike Growth

Surface is dealing 1d4 instead of 2d4 per 1.5m travelled

I was able to change it to 2d4, but upon testing, it doesn't seem like the surface actually applies the status every 1.5 m travelled


User authentication for App / Firebase

As a researcher.
I want only authenticated users to access the app.
So that only safe-listed users can submit or access the data.
Assumptions:

"The app" in this case means the web app and Firebase DB.
The page works for desktop, tablet and smartphones and all browsers

New Stories:

New story for admin sending an email to reset password
Persisting the data for a previously disabled user is part of new story
Store user that submitted a form is a new story

ACs:
Admin Scenarios-

Admin can Create/Add new user in firebase with a email and password
When Admin deletes/disable a user, that user cannot login to the BMMRO webapp again
When Admin creates/adds a new user with a previously deleted email address, that creates a new user
When Admin enables a user, that user can successfully login

Researcher scenarios-

New user can login to the BMMRO webapp with the email and password
Error validations when email/password is wrong
User can logout from the header
Once user is logged in, that state is persisted
Once logged in, username/email id is visible
Redirect unauthenticated users to the login page

Notes:

Wire frames can be found in figma

Things to investigate:

one id for all or personal logins
add users through firebase (implications on setting password)
storing key (don't need to login again until key expires or user thrown out)
which devices are supported
(maybe out of scope) way to know who submitted the form
logging out? (Eg. log out button on the landing page)

From Firebase docs:
"Firebase ID tokens last for an hour; the refresh token can be used to retrieve new ID tokens. Refresh tokens expire only when one of the following occurs:

The user is deleted
The user is disabled
A major account change is detected for the user. This includes events like password or email address updates.
"

Todo:

user logs in -> goes offline -> submits a form -> goes back online after 1 hour -> is token refreshed? is form submitted?
change error msg to "email" instead of "username"
note about "When Admin creates/adds a new user with a previously deleted email address, that creates a new user"
note about disable/delete user and submit form
create diagram to explain authentication & online vs offline
send Jen a small paragraph about how the authentication flow works

FLOW:

A Firebase Admin goes to the Firebase Console to add a user (with email and password)

OPTIONAL:
2.1. Firebase Admin selects “Reset Password” in Firebase for that user
2.2. User receives an email to reset their password

2.3. User clicks in the link to reset their password

2.4. User can now login with the new password, instead of the one created by the Firebase Admin

Unauthenticated users are redirected to the login page

User logs in to the App

Authenticated user is redirected to the Habitat Use form

Header contains an icon that can be clicked to show the email of the current signed-in user and a logout button

User clicks the logout button

User is redirected to the login page

NOTES:

Users can be deleted or disabled in the Firebase console.

In that case, they will not be able to login to the app.
Users will still be able to submit the Habitat Use form (preventing users from submitting this form will be done in https://github.com/BMMRO-tech/BMMRO/issues/40).
Users will be automatically logged out when they refresh the page.

Users are able to log out when they are offline


Question: isn't fetch() supposed to fall back to cache if network not available

If the user has data available in cache and if we make the fetch() api call when there is no network, isn't fetch supposed to get data from cache rather than throwing an exception?
If not, do you have any suggestions to accomplish this?

What you are describing is not the default behavior but you can achieve similar functionality using an rx operator to swap in a get call when a fetch fails
store.fetch(key)
  .onErrorResumeNext(store.get(key))
  .subscribe()

Let me know if this works for you :-)

After thinking for sometime, I realized why it is the way it is. We instead used a network helper implemented previously to check for network availability before doing fetch().
The scenario is for user login data, we always want to fetch from network first, if not fallback to cache.
Thanks for clarification @digitalbuddha, closing this issue.


Hideable widget bar blank when first shown

Affected hardware version
Bangle 2
Your firmware version
2v16
The bug
When swiping down the hidden widget bar I see a blank bar with no widgets visible
I've tried with the MacWatch2 clock and the widget bar seems to stay blank until after the screen locks/unlocks. After that the widgets (lock widget, lato pedometer, and battery widget) are shown as expected when swiping to reveal the hidden widget bar.
Installed apps
widbat (0.09), waypoints (0.01), cprassist (0.02), widpb (0.05), calendar (0.11), antonclk (0.11), gpstime (0.05), beer (0.02), quicklaunch (0.08), boot (0.55), messagegui (0.63), barcode (0.10), messages (0.58), messageicons (0.03), launch (0.20), widalarm (0.01), about (0.14), lato (0.01), medicalinfo (0.01), geminiclock (0.01), gpsinfo (0.11), alarm (0.37), widmessages (0.04), widlock (0.08), setting (0.56), sched (0.18), notify (0.12), health (0.19), android (0.20), widclkbttm (0.05), macwatch2 (0.06)

The above commit seems to have fixed this issue now, Gordon found the solution far quicker than I could!

Yes, sorry, this only fixes the issue of a blank widget bar when first shown. I haven't tried installing your widmeda, though it looks very good!


Git Worktree!

Git Worktree!
One of git 2.5 features git worktree. One git repository can have multiple working trees. Hopefully no more git stash :smiley: addressing hotfixes :+1:
More details about git 2.5 update please refer
Using Revert and Reset

Its advised to use git revert for public branch.
git reset for private branch

:+1:


ModalBottomSheet enter animation on iOS is missing (or too fast)

Describe the bug
The Material3 ModalBottomSheet enter animation on iOS appears to just blip into existence rather than actually translate the sheet or fade in the scrim.

Android
iOS

Screen_recording_20240617_125721.webm
https://github.com/JetBrains/compose-multiplatform/assets/1361086/600e99fd-a295-436a-bf60-9dbca4daa571

Affected platforms

iOS

Versions

Libraries: 1.6.11

Material3

Kotlin version: 2.0.0
OS version(s) (required for Desktop and iOS issues): See below
OS architecture (x86 or arm64): arm64
Device (model or simulator for iOS issues):

iOS 17.5 (21F77) SDK + iOS 17.5 (21F79) Simulator (Installed)
iOS 17.5 on iPhone 15 Pro

To Reproduce
Steps to reproduce the behavior:

Reproducible project: https://github.com/ZacSweers/FieldSpottr/
Specific code for the sheet is here: https://github.com/ZacSweers/FieldSpottr/blob/bf99cfd56422b4d75ba5057713721c8e77cf35ea/src/commonMain/kotlin/dev/zacsweers/fieldspottr/Home.kt#L172
Which in turn uses this library impl: https://github.com/slackhq/circuit/blob/main/circuitx/overlays/src/commonMain/kotlin/com/slack/circuitx/overlays/BottomSheetOverlay.kt

Expected behavior
It should animate in smoothly
Screenshots
See above
Additional context
Add any other context about the problem here.

iOS reuses animation from common in this place, so it looks more like performance/missing frames issue.
@elijah-semyonov could you please take a look?

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Not removing unused import if has same name function

PHP CS Fixer not removing unused import that has the same name as that of the function used somewhere in class.
Sample Code
<?php

namespace App\Controllers;

use App\Api\Controller;
use Zend\Diactoros\Exception;

class ValidationController extends Controller
{
    public function ipSubnet()
    {
        return exception('Something', 404);
    }
}

Expected Code
<?php

namespace App\Controllers;

use App\Api\Controller;
// Remove unused import here

class ValidationController extends Controller
{
    public function ipSubnet()
    {
        return exception('Something', 404);
    }
}

The PHP version you are using:
$ php -v
PHP 7.1.9 (cli) (built: Aug 30 2017 18:58:30) ( NTS )
Copyright (c) 1997-2017 The PHP Group
Zend Engine v3.1.0, Copyright (c) 1998-2017 Zend Technologies

PHP CS Fixer version you are using:
$ php-cs-fixer -V
PHP CS Fixer 2.4.0 Silver Gingerbread by Fabien Potencier and Dariusz Ruminski (63661f3)

Hi. Thanks you for your report :+1:
It looks like sth we could improve for sure, happily it's not causing issues for changed code.
Sadly, maintainers here cannot handle all filed issues themselves.
Could you help with it and propose a solution (probably for 2.2 branch) ?

Related: #2814.

it is and it isn't related I think, function imports and class imports are done explicitly so could be checked against, with or without checking comments and/or PHPDocs.


jetpack-5-ga: drop gstreamer-1.14.conf

Right, I meant to do that when I delete the recipes, but forgot.  Thanks!
Could you add a Signed-off-by to the commit, and prefix the first line with contrib: ?  If that's too much bother, I'll take care of the deletion with my next batch of updates.

Done.
The contrib layer only have a single recipe now, maybe is better to move it to meta-tegra-community and remove the layer as well

Thanks... and yes, I was thinking the same thing.

Done.
The contrib layer only have a single recipe now, maybe is better to move it to meta-tegra-community and remove the layer as well

Thanks @quaresmajose for your PR


dotnet watch run not interactive for ASP.NET Core apps

dotnet watch run does not provide any feedback that the app has been re-compiled for ASP.NET Core apps. It does for console apps. I believe that this is a regression.
I see similar behavior when using Docker with microsoft/dotnet-nightly:2.1-sdk
It is also doing something weird on Windows after I hit CTRL-C with the last error printed from dotnet watch.
C:\git\dotnet-docker\samples\aspnetapp\aspnetapp>dotnet --version
2.1.300-rc1-008673
C:\git\dotnet-docker\samples\aspnetapp\aspnetapp>dotnet watch run
watch : Started
Hosting environment: Production
Content root path: C:\git\dotnet-docker\samples\aspnetapp\aspnetapp
Now listening on: http://localhost:5000
Now listening on: https://localhost:5001
Application started. Press Ctrl+C to shut down.
Application is shutting down...
watch : Shutdown requested. Press Ctrl+C again to force exit.
^C
C:\git\dotnet-docker\samples\aspnetapp\aspnetapp>watch : Exited with error code -1073741510

dotnet watch run does not provide any feedback that the app has been re-compiled for ASP.NET Core apps. It does for console apps.

Can you clarify? What are you doing in a web app that causes recompilation, but doesn't cause dotnet-watch to show console output?

Closing due to lack of details. We can reopen if you can provide more details.


Processing fstab with mount -a failed

Version
Microsoft Windows [Version 10.0.19045.2604]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
Linux version <IP_ADDRESS>-microsoft-standard-WSL2 (oe-user@oe-host) (x86_64-msft-linux-gcc (GCC) 9.3.0, GNU ld (GNU Binutils) <IP_ADDRESS>00220) #1 SMP Wed Nov 23 01:01:46 UTC 2022
Distro Version
Ubuntu 22.04.2 LTS
Other Software
No response
Repro Steps
From a command or powershell prompt:
bash
Expected Behavior
A new WSL bash session should be created
Actual Behavior
Good: A new terminal is started
Bad: Message "Processing fstab with mount -a failed." is displayed prior to prompt
All was fine yesterday, today I did wsl --update and then the problems started.
Diagnostic Logs
No response

Could you please share the output of dmesg?

Sure, see dmesg.txt

@bobf32 check you credentials:
[    2.941322] CIFS: Attempting to mount \\<IP_ADDRESS>\media
[    3.373557] CIFS: Status code returned 0xc000006d NT_STATUS_LOGON_FAILURE
[    3.374255] mount error(13): Permission denied

@elsaco Oops, that did it. Thanks.

Cool, glad this is sorted.


Update Arma 3

Game Title
Arma 3
Status of the Anti-Cheat
❔ Unconfirmed
Proof of Mention
This discord screenshot is hardly a confirmation that Arma 3 will get support for BattlEye with Proton.

Duplicate of #9 #69 and #58
I would agree with your assessment and in fact one of those duplicates actually post a clarification by the same person who made the initial statement that it was taken out of context.
However, please make it easier for the poor person who is managing this project and comment on the original issue rather than posting duplicates. It makes it much harder for them if they have to track back through multiple duplicates


Server create error

When trying to create a server i get this error - "A TransferException was encountered while trying to contact the daemon, please ensure it is online and accessible. This error has been logged."
Heres the log file containing the full thing - https://gist.github.com/Bennyboy1695/905df77d7a476c88bb902844f670312d

Issue can be resolved following: https://vps3.parkervcp.com/troubleshooting/connection-error


Add configuration defaults for nextercism

We will be moving to a tree-shaped track rather than a linear one, as described in the progression & learning in Exercism design document.
In order to support this, we need to expand the metadata that exercises are configured with.
Note that 'core' exercises are never unlocked by any other exercises. Core exercises appear in the track in the order that they are listed in the array.
Non-core exercises depend on only one exercise (unlocked_by: ). At the moment we are assuming that this is a core exercise, but there is no reason that it needs to be.
active: false is the equivalent of deprecated, which was stored in a separate array. This array is being removed in this change.
With these defaults the track in nextercism will have no core exercises, and all the exercises will be available as 'extras' from the start.
See https://github.com/exercism/meta/issues/16

I'll merge this, as this actually changes nothing but updates the format.
But do you have the UUID-generator ready?

But do you have the UUID-generator ready?

I do! I wrote it yesterday. It is an extra subcommand in configlet: configlet uuid.  I need to cut a release, and will do that today.


Remove temp file on failure

Requirements
Mark the following tasks as done:

[x] Checked the codebase to ensure that your feature doesn't already exist.
[x] Checked the pull requests to ensure that another person hasn't already submitted the feature or fix.
[x] Adjusted the Documentation. (Not applicable)
[x] Added tests to ensure against regression.

Description of the Change
Removes the temporary local file in the case of a failed import.
Why Should This Be Added?
In the case of a failed export, the temporary file is left on the server. This is particularly problematic if dealing with low space and or a high volume of imports. I encountered this issue when using Laravel Vapor. I posted a question about this here https://github.com/Maatwebsite/Laravel-Excel/issues/2655, this PR is an attempt to fix one of the issues outlined in the question.
Benefits
Failed files won't be stored on the server.
Possible Drawbacks
I presume if a failed job is retried in the future a new copy of the file would have to be fetched.
Verification Process
I have written a test to confirm the issue is fixed before and after the fix was applied. I have run the test suite and have confirmed it was all passing on my machine.
Applicable Issues
https://github.com/Maatwebsite/Laravel-Excel/issues/2655

Awesome, thanks!


Add search to catalog

Basic search support, with little styling.

TODO:

[ ] Style the search results (@cshinn)
[x] Support image URLs, not just font icons
[x] Show tags for builder images
[x] Use KeywordsService (currently broken with mock data)
[x] Defer search when user starts typing before items are loaded
[ ] See if we can have one overlay-panel instead of having to duplicate the logic. Currently it's tied to the services view.
[ ] Add unit tests

I hate having multiple WIP in flight, but I know @cshinn was going to look at styling the search control.

@dtaylor113 I checked GitHub markdown syntax, and I didn't see any easy way to make the "not ready for code review" text red :)

cc @serenamarie125

ship it

@jwforres @dtaylor113 This is probably far enough along for a review.
@jwforres I didn't debounce the search because it didn't feel as good. There's a typeahead-wait-ms setting on the typeahead control if we want it.

is it possible for someone to start typing into the filter while the service class or image streams are still loading? do you need to run filter here

Added a second commit that defers the search until items load.

Squashed and rebased

can merge once travis passes


SHIP-0029: One-Off Builds

Adds an enhancement proposal to include one-off builds.

/assign @qu1queee


Quick Accent while gaming or other tasks.

Description of the new feature / enhancement
There should be an option to disable quick accent in certain apps, or allow it only in a few apps, as it interferes sometimes when I don't need it
Scenario when this would be used?
I was playing a game and pressed W and space simultaneously (in some games W and arrow keys might need to be pressed too), this led to the quick accent bar to open up instead of my in-game character jumping while moving forward.
Supporting information
No response

This feature in general should only be active when you write something on a Text Editor or something similar. I don't want it to ruin my gaming sessions and for now I simply disable it when playing.

The fix for this issue has been released in 0.65. Please update PowerToys to the latest version:
https://github.com/microsoft/PowerToys/releases


Add a floor

adds a floor so the players don't just fall into the abyss whenever they get gravity applied to them.

dibs


Fix naming of cufft.pyx

This PR fixes naming of cufft.pyx.

CuFftError -> CuFFTError
workSize -> work_size
workArea -> work_area

Derived from #1219.
Could you resolve conflicts?

Jenkins, test this please.

Jenkins, test this please.

LGTM!


Update all the chef-server-ctl deps to the latest

Gets rspec to the latest which actually pushes back inspec. Waiting on the inspec folks to loosen the pin of rspec.
Signed-off-by: Tim Smith<EMAIL_ADDRESS>

https://buildkite.com/chef/chef-chef-server-main-omnibus-adhoc/builds/3902


Method Calls in Loops

(@utting @leavens keeping you in the loop)
The following program generates a strange error message indeed:
method g(&int p) -> (int r):
    return 0

method f(&int p, int[] xs):
    //
    for i in 0..|xs|:
        g(p)

UPDATE: this is related to debug=true in the wy.toml file.

Summary:

When debug=true this example doesn't verify because of a generated loop invariant involving HEAP for p
Since the loop invariant was generated, there is no source information --- hence why the error fails.

But why doesn't it fail when debug=false ?

When debug=true loop invariant is:
   invariant (forall r:Ref :: Ref#within(HEAP$173,r,p) || (HEAP$173[r] == Void) || (HEAP$173[r] == HEAP[r]));

When debug=false we have:
   invariant (forall r:Ref :: Ref#within(HEAP$173,r,p$78) || (HEAP$173[r] == Void) || (HEAP$173[r] == HEAP[r]));

Ok, here we go:
When debug=true we have this for g():
free ensures (forall p:Ref :: Ref#within(old(HEAP),p,p) || (old(HEAP)[p] == Void) || (old(HEAP)[p] == HEAP[p]));

When debug=false we have this:
free ensures (forall p:Ref :: Ref#within(old(HEAP),p,p$23) || (old(HEAP)[p] == Void) || (old(HEAP)[p] == HEAP[p]));

These are not equivalent.

In the earlier version of the translator to Boogie, I used a weaker version of this 'debug' name mapping, maintained a map from each fully qualified name to its 'printable' name.
For example, for a base name of 'p', the first printable name would be just 'p', then other fully qualified variants of it would be mapped to 'p1', 'p2', etc. (a separate counter for each base name).  This allows the most common cases (identifiers that appear only once) to retain their original names, but it still disambiguates other names when needed, with a short and readable name.
(One could make this even more sophisticated by taking detailed scopes into consideration, but a simple version that has just two levels of scope, global and local, would work pretty well I think).

Hey,
In general, expressions are already associated with a unique identifier and I typically use this.  Not sure what's going wrong here.  Its not a ideal, but not a big drama now I know the problem.


Support 'scatter' charts out-of-the-box

Chartjs has default config for scatter charts which are different than line charts in four key areas:
* x and y axis are both linear.
* hover mode is single.
* showLines is false.
* tooltip title doesn't make sense, as both x and y are both values.
Keeping in mind that this default config is already present within the base chartjs library, a different 'Scatter' chart should be exposed by this library - to avoid redundant configs to be written by each developer.
Also fixed the editorconfig - the whitespace strategy was not reflecting the code style.

Thanks!


Automatic PR from RestAPI-PR623

Generated from RestAPI PR: https://github.com/Azure/azure-rest-api-specs/pull/623

Coverage decreased (-0.001%) to 55.639% when pulling<PHONE_NUMBER>6f0e10442ff3ebd889365f23b975f5 on AutorestCI:RestAPI-PR623 into ab67ea477cc00a76f9f6115b943a53fb0763056f on Azure:dev.

Coverage decreased (-0.001%) to 55.639% when pulling<PHONE_NUMBER>6f0e10442ff3ebd889365f23b975f5 on AutorestCI:RestAPI-PR623 into ab67ea477cc00a76f9f6115b943a53fb0763056f on Azure:dev.


Container cannot see Arc GPU

I am following the IPEX Ollama docker guide. I am stuck on the sycl-ls step. The container cannot see Arc GPU even though the host can and the container has /dev/dri shared and it even runs --privileged.
Here's what I see in the container:
# ls /dev/dri
card1  card2  renderD128  renderD129
# whoami
root
# sycl-ls
[opencl:acc:0] Intel(R) FPGA Emulation Platform for OpenCL(TM), Intel(R) FPGA Emulation Device OpenCL 1.2  [2<IP_ADDRESS>.12_195853.xmain-hotfix]
[opencl:cpu:1] Intel(R) OpenCL, AMD Ryzen 5 5600G with Radeon Graphics          OpenCL 3.0 (Build 0) [2<IP_ADDRESS>.12_195853.xmain-hotfix]

Meantime, it seems to work on the host:
$ ls /dev/dri
by-path  card1  card2  renderD128  renderD129
$ whoami
rv
$ source /opt/intel/oneapi/setvars.sh
$ sycl-ls
[level_zero:gpu][level_zero:0] Intel(R) oneAPI Unified Runtime over Level-Zero, Intel(R) Arc(TM) A380 Graphics 12.56.5 [1.3.30049.600000]
[opencl:cpu][opencl:0] Intel(R) OpenCL, AMD Ryzen 5 5600G with Radeon Graphics          OpenCL 3.0 (Build 0) [2<IP_ADDRESS>.08_160000]
[opencl:gpu][opencl:1] Intel(R) OpenCL Graphics, Intel(R) Arc(TM) A380 Graphics OpenCL 3.0 NEO  [24.26.30049.6]

This is how I start the container:
sudo podman run -it --rm \
    --privileged \
    --device=/dev/dri \
    -v ollama-ipex:/models \
    --name ollama-ipex \
    docker.io/intelanalytics/ipex-llm-inference-cpp-xpu:latest

The guide prescribes all sorts of additional parameters for the container, but adding them makes no difference.
Per info on Docker Hub page, this image is still updated daily, but I am starting to think there is something wrong with it. Library versions reported by sycl-ls in the container are from 2023.

Same here, I just tried the guide and my Arc A770 is not visible.
Meanwhile, this image works fine for me: https://github.com/mattcurf/ollama-intel-gpu

Hi @robertvazan @pepijndevos
Could you provide the kernel version and gpu driver version?
uname -r
dpkg -l | grep i915
modinfo i915

Hi @pepijndevos
We recommend setting up your environment with Kernel 6.5, and installing the Intel-i915-dkms driver for optimal performance. We've tested Kernel 6.5 with the Intel-i915-dkms driver, and it successfully detects the GPU within the container. You can follow the steps below to install Kernel 6.5:
export VERSION="6.5.0-35"
sudo apt-get install -y linux-headers-$VERSION-generic
sudo apt-get install -y linux-image-$VERSION-generic
sudo apt-get install -y linux-modules-$VERSION-generic # may not be needed
sudo apt-get install -y linux-modules-extra-$VERSION-generic

After installing, you can configure GRUB to use the new kernel by running the following:
sudo sed -i "s/GRUB_DEFAULT=.*/GRUB_DEFAULT=\"1> $(echo $(($(awk -F\' '/menuentry / {print $2}' /boot/grub/grub.cfg \
| grep -no $VERSION | sed 's/:/\n/g' | head -n 1)-2)))\"/" /etc/default/grub

Then, update GRUB and reboot your system:
sudo update-grub
sudo reboot

For detailed instructions on installing the driver, you can follow this link:
https://github.com/intel-analytics/ipex-llm/blob/main/docs/mddocs/Quickstart/install_linux_gpu.md#for-linux-kernel-65
Please feel free to reach out if you have any questions!

I'm on Arch Linux so that doesn't help much. I'd also like to point out again that the third party docker image linked above does detect my GPU. Maybe that image has newer userspace drivers than yours? I'll just keep using that one for now.

@liu-shaojun I am on Fedora and kernel 6.11.6. The third-party image recommended by @pepijndevos works for me, likely thanks to newer versions of Intel libraries. Why wouldn't you upgrade your reference image? Would that break it on the oldest supported Ubuntu hosts?
While I am more than happy to give your software its favorite OS inside the container and to run sufficiently recent host kernel, limiting host kernel to a 2-3 specific versions is unreasonably restrictive for consumer hardware. It means worse support for your consumer GPUs than what AMD offers.

Hi @pepijndevos @robertvazan
Our Docker image is based on this base image: intel/oneapi-basekit:2024.1.1-devel-ubuntu22.04. We have observed that in certain scenarios with specific kernel versions, such as kernel 6.8 without out-of-tree driver installed, GPUs are not detected inside containers started with intel/oneapi-basekit:2024.1.1-devel-ubuntu22.04. We are currently working on resolving this issue.

Hi @pepijndevos @robertvazan
Could you please try using the intelanalytics/ipex-llm-inference-cpp-xpu:latest image? We've updated the compute runtime version in this image and tested it with kernel 6.8, where the GPU is detectable within the container.

@liu-shaojun The updated image works for me. Arc GPU is detected. No need for --privileged and other options mentioned in the quickstart guide. Are you going to update the guide too?

Sure, we will update the guide. Thanks for let us know!

It sees my GPU!


Follow up PR on image config controller feedback

Which issue this PR addresses:
Followup PR for remaining feedback from #1766 (some of the comments are hidden by GitHub UI).
Test plan for issue:
This PR itself is mostly about improving unit tests. Better coverage added for one case in getCloudAwareRegistries.
Is there any documentation that needs to be updated for this PR?
No, just bug fixes and test improvements.

Apart from the leftover line already mentioned by Ross in operator.go , LGTM!
I particularly like how you refactored the tests to separate out Public, Gov, and unsupported cloud. Much cleaner this way.
Thank you @m1kola


Memory Leak: unmarshalling a class with const members causes memory leak (observed in JSON parser)

struct Test
{
std::string const value1; // => leak
std::string value2; //  => no leak
}
rfl::json::read(....)
We used valgrind to see the leak

OK, thanks for reporting. I will check it out.


Simplify getSqlType types with default name

Also moved grouped cases to the top of the switch statement

Codecov Report

Merging #1215 into master will decrease coverage by 0.02%.
The diff coverage is 81.57%.

@@            Coverage Diff             @@
##           master    #1215      +/-   ##
==========================================
- Coverage    72.2%   72.17%   -0.03%
==========================================
  Files          35       35
  Lines        5580     5560      -20
==========================================
- Hits         4029     4013      -16
+ Misses       1551     1547       -4

Impacted Files
Coverage Δ

src/Phinx/Db/Adapter/SqlServerAdapter.php
0% <0%> (ø)
:arrow_up:

src/Phinx/Db/Adapter/MysqlAdapter.php
97.96% <100%> (+0.12%)
:arrow_up:

src/Phinx/Db/Adapter/PostgresAdapter.php
92.76% <100%> (ø)
:arrow_up:

src/Phinx/Db/Adapter/SQLiteAdapter.php
86.48% <90.9%> (-0.17%)
:arrow_down:

Continue to review full report at Codecov.

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

Let's hope this doesnt cause any regressions due to typos.


🐛 Dual network card CentOS7.9 cell install error

install cell module 1.3.0
report error details
try installing from online reciprocity...
dependency packages installed
**configure default network bridge fail: no ethernet interface available**

reinstall OS as same situation
SSH $# ip a
1: lo: <LOOPBACK,UP,LOWER_UP> mtu 65536 qdisc noqueue state UNKNOWN group default qlen 1000
    link/loopback 00:00:00:00:00:00 brd 00:00:00:00:00:00
    inet <IP_ADDRESS>/8 scope host lo
       valid_lft forever preferred_lft forever
    inet6 ::1/128 scope host
       valid_lft forever preferred_lft forever
2: p8p1: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc pfifo_fast state UP group default qlen 1000
    link/ether 70:b5:e8:4a:ca:3e brd ff:ff:ff:ff:ff:ff
    inet <IP_ADDRESS>/24 brd <IP_ADDRESS> scope global noprefixroute p8p1
       valid_lft forever preferred_lft forever
    inet6 fe80::72b5:e8ff:fe4a:ca3e/64 scope link
       valid_lft forever preferred_lft forever
3: p1p1: <NO-CARRIER,BROADCAST,MULTICAST,UP> mtu 1500 qdisc pfifo_fast state DOWN group default qlen 1000
    link/ether 00:e0:4c:82:6a:df brd ff:ff:ff:ff:ff:ff

net-twork card p1p1 disabled
source code
i guess go src/net/interface.go cant get network interface in my OS env
 net.Interfaces()

https://github.com/project-nano/framework/blob/master/configure_helper.go#L10-L15
:broken_heart:

got it
https://github.com/project-nano/framework/blob/master/configure_helper.go#L29-L31
another way to find out physical network adapter
if !strings.HasPrefix(i.Name, "lo") && !strings.HasPrefix(i.Name, "vir") {
	options = append(options, i.Name)
}

my machine is not vmware ,but network adapter name is p8p1, Prefix is 'p'
@bokuore

got it
https://github.com/project-nano/framework/blob/master/configure_helper.go#L29-L31
another way to find out physical network adapter
if !strings.HasPrefix(i.Name, "lo") && !strings.HasPrefix(i.Name, "vir") {
	options = append(options, i.Name)
}

my machine is not vmware ,but network adapter name is p8p1, Prefix is 'p'
@bokuore

https://github.com/project-nano/framework/blob/master/configure_helper.go#L29-L31
if strings.HasPrefix(i.Name, "e"){
				options = append(options, i.Name)
			}

https://github.com/project-nano/framework/blob/master/configure_helper.go#L29-L31
if strings.HasPrefix(i.Name, "e"){
				options = append(options, i.Name)
			}

Ok

Ok


Are we missing a chapter on creation?

While RFC7519 on JWTs has a chapter on Creation of JWTs, the SD-JWT and this spec jump right to Examples on continue with guideline for presentation/verification. Do you see any need for such a Creation chapter/section?

How SD-JWTs are created is well defined in the SD-JWT spec. There is no need to have a section on creating SD-jWTs here.


Model knowledge cutoff date

Is there a need to tune systemMessage for updated gpt models with its most up-to-date date of train data ?

I noticed the current setting would replace 'cutoff' with 2023-04 or 2021-09.
THX!

No it's working well already.

This was fixed on v1.13.0. Cutoff is updated (for the default personas at least) automatically. Use the {{Cutoff}} variable in the persona prompt.

auto-updates with the selected network as you can see from the screenshot.

Oh, wait OPENAI CHANGED IT!
How bizarre - it said March 2023 before. Will update it.

Great! Seems openai is falling behind their own models...lol

Done - landed on main already, and will be out for the stable release on v1.14.0.
Thanks for opening the ticket.

no problem.


Travis CI builds failures

Travis CI builds failures on Build #3274 and Build #3242

build succeeded.
Look for any errors in the above output or in build/linkcheck/output.txt
The name of the builder is: linkcheckmake[1]: Leaving directory '/home/travis/build/dcos/dcos-e2e/docs/cli'
mypy *.py src/ tests/ admin/
+mypy setup.py versioneer.py src/ tests/ admin/
pip-extra-reqs src/
+pip-extra-reqs src/
Traceback (most recent call last):
File "/home/travis/virtualenv/python3.5.6/bin/pip-extra-reqs", line 5, in
from pip_check_reqs.find_extra_reqs import main
File "/home/travis/virtualenv/python3.5.6/lib/python3.5/site-packages/pip_check_reqs/find_extra_reqs.py", line 10, in
from pip_check_reqs import common
File "/home/travis/virtualenv/python3.5.6/lib/python3.5/site-packages/pip_check_reqs/common.py", line 10, in
from pip._internal.download import PipSession
ImportError: No module named 'pip._internal.download'
lint.mk:35: recipe for target 'pip-extra-reqs' failed
make: *** [pip-extra-reqs] Error 1
Traceback (most recent call last):
File "admin/run_script.py", line 36, in
subprocess.check_call(['make', 'lint'])
File "/opt/python/3.5.6/lib/python3.5/subprocess.py", line 271, in check_call
raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['make', 'lint']' returned non-zero exit status 2
The command "python admin/run_script.py" exited with 1.
cache.2
store build cache

This is due to https://github.com/r1chardj0n3s/pip-check-reqs/pull/22 and can be remedied by using pip-check-reqs-pip-gte-20 rather than pip-check-reqs.


feat(oauth): show multiple login providers

now i'm starting to get tempted to chuck a getToken method that links the user to your hub account. maybe it's a separate project though.

Oop looks like dependabot got the update in already.


[Enhancement]  : Autolink Issues in Scope to PR

Feature Presentation:
Goal: An Action to automatically create and link issues for in scope PRs made.
Implementation: TBD
WIP

https://github.com/dswan36/SillyLittleFiles/pull/257
https://github.com/dswan36/acprox/pull/25
https://github.com/dswan36/sillylittle.selfhost/pull/7

Non working script produced.


How can I do without "didFinishLaunchingWithOptions"?

How can I do without "didFinishLaunchingWithOptions"? For example, how can I call my SwipeViewController in the viewDidAppear method? When the application starts, the first window I have authorization, and then I need to work with SwipeViewController.

You can init the SwipeViewController simply like this:
let pageController = UIPageViewController(transitionStyle: .scroll, navigationOrientation: .horizontal, options: nil)
let navigationController = YourViewControllerName(rootViewController: pageController)

You can do this wherever you want, I guess I should make it more clear from the documentation.

`override func viewDidAppear(_ animated: Bool) {
super .viewDidAppear(animated)
    let pageController = UIPageViewController(transitionStyle: .scroll, navigationOrientation: .horizontal, options: nil)
    let navigationController = ViewController(rootViewController: pageController)
    self.present(navigationController, animated:false, completion: nil)

}`

this code does not work :( Please tell me what my mistake is, or how to write better :( I'm a beginner programmer, please forgive me :(

And what exactly does not work? I tried to present SwipeViewController in my code and it worked. Maybe there is something wrong in your ViewController regarding the setting of SwipeViewController.

But if you have authorization window, I think it would be better to first set AuthorizationViewController and then set it rootViewController to whatever your next viewcontroller is. Like this:
let window = UIApplication.shared.keyWindow
window?.rootViewController = YourViewController()
window?.makeKeyAndVisible()

Only the first screen is shown, but nothing is pressed. If you put 'animation = true', then an infinite number of times

...are you calling that in ViewController? Because then it would make sense (every time the view appears, it calls viewDidAppear again, etc., etc.)

Yes, you are right. How then to be? at what point it is better to call?

I do not really understand what you are trying to achieve. You said that you have an authorizationviewcontroller from which you want to present SwipeViewController, is that right?
Well, then in you AuthorizationViewController, for example when user taps loginButton, present SwipeViewController.

Oh my God, forgive me for taking your time, I,ve found a mistake after your last message. That was such a petty mistake ....  Thank you very much! Everything is working!

Great! Glad to be of help, hit me up if you encounter any more problems 😉

Could you tell me pleeeease, how I can add the TabBarController to the hierarchy? I mean that the main Navigation is my TabBar with bottom switching. And after that is SwipeViewController.

I do not use storyboard and I have not tried that yet, you have to basically replicate this part:
let pageController = UIPageViewController(transitionStyle: .scroll, navigationOrientation: .horizontal, options: nil)
let navigationController = YourViewControllerName(rootViewController: pageController)

My guess is that it would go something like this:
TabBarController -> SwipeViewController -> (rootViewController) UIPageViewController

Good afternoon. Could you tell me please, how is it possible to write the code? :( I wrote this code, but it does not work, because I do not know how to process tabBar = ( I excuse me for taking your time :(
override func prepare(for segue: UIStoryboardSegue, sender: Any?) {         if (segue.identifier == "show") {             if let tabBarController = segue.destination as? UITabBarController {                 let pageController = UIPageViewController(transitionStyle: .scroll, navigationOrientation: .horizontal, options: nil)                 let navigationController = ViewController(rootViewController: pageController)                 self.present(navigationController, animated: true, completion: nil)             }         }     }

Umm, you do not have to present navigationController from your tabBarController, that is handled by UIKit itself. Just set tabBarController.viewCountrollers = Array and then present that tabBarController.

On Dec 19, 2017, at 8:59 AM, relaxsus<EMAIL_ADDRESS>wrote:
Good afternoon. Could you tell me please, how is it possible to write the code? :( I wrote this code, but it does not work, because I do not know how to process tabBar = ( I excuse me for taking your time :(
override func prepare(for segue: UIStoryboardSegue, sender: Any?) { if (segue.identifier == "show") { if let tabBarController = segue.destination as? UITabBarController { let pageController = UIPageViewController(transitionStyle: .scroll, navigationOrientation: .horizontal, options: nil) let navigationController = ViewController(rootViewController: pageController) self.present(navigationController, animated: true, completion: nil) } } }
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub https://github.com/fortmarek/SwipeViewController/issues/26#issuecomment-352667421, or mute the thread https://github.com/notifications/unsubscribe-auth/AI8AL1E2RyzQNyjixctl13TUF7vcybBSks5tB2zigaJpZM4RFQ41.

Yes, I eventually read a lot of information, and did the same. Therefore, I deleted the question.  I feel uncomfortable. Thank you very much.

I see, I answered right from the email, so I did not notice it had been already deleted.  Keep learning ✌

On Dec 19, 2017, at 12:52 PM, relaxsus<EMAIL_ADDRESS>wrote:
Yes, I eventually read a lot of information, and did the same. Therefore, I deleted the question.  I feel uncomfortable. Thank you very much.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub https://github.com/fortmarek/SwipeViewController/issues/26#issuecomment-352728023, or mute the thread https://github.com/notifications/unsubscribe-auth/AI8AL8OLBBNSQ8cq4eHsJs7wUY9DELcOks5tB6NmgaJpZM4RFQ41.

I see, I answered right from the email, so I did not notice it had been already deleted.  Keep learning ✌

On Dec 19, 2017, at 12:52 PM, relaxsus<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:
Yes, I eventually read a lot of information, and did the same. Therefore, I deleted the question.  I feel uncomfortable. Thank you very much.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub https://github.com/fortmarek/SwipeViewController/issues/26#issuecomment-352728023, or mute the thread https://github.com/notifications/unsubscribe-auth/AI8AL8OLBBNSQ8cq4eHsJs7wUY9DELcOks5tB6NmgaJpZM4RFQ41.


OPTIONS requests mocking

Is there a way to mock OPTIONS requests? I use those to get model metadata used for form generation (field choices).

Hi, I'm actually not sure off the top of my head. Assuming Pretender supports it, it should be relatively straightforward to add.

Add 'options' to this line
Add a very simple test to this module

Think you'd be up for giving this a shot? I'd be happy to answer any questions!

Yes, I can check that out ;)

I've added options:

https://github.com/riklaunim/ember-cli-mirage/commit/14523a7d52163e9f6dd44acb4fd989f766e40bd6
https://github.com/riklaunim/ember-cli-mirage/commit/e745afca41d07586c9c4b77462b4f303d9bdfd1d

And I get:
TypeError: this.pretender[verb] is not a function
When trying to use it. Pretender has it: https://github.com/pretenderjs/pretender/blob/7e2d2dcef8fe6fe0dc00807a8da2ca337663a06d
/pretender.js#L53
So it's bit unexpected.

If you open a PR with your changes we can discuss the code more easily there

ok :)

@riklaunim Any additional info here?

New pretender needed.

@riklaunim Newer than the 1.2 merged in in this PR? https://github.com/samselikoff/ember-cli-mirage/pull/881

Hm.. it needs 1.3 that was released today :) Will update this PR today too.

We're tidying up issues, so am going to close this issue for now.  If it's still an issue, happy to reopen

Any updates on this issue? I'm getting ERR_CONNECTION_REFUSED responses during my Ember Data CORS preflight check. It would be great to have Mirage support the options method (unless it already does and I'm missing something).

This would be nice to have, if someone has some time to spare I'd love to merge it into Mirage.
For now, if you're using a version of Pretender that supports OPTIONS you should be able to drop down to this.pretender in mirage/config and write a route handler there.
Closing for now because I'm trying to keep issues actionable. I'll add this to the Roadmap project.

I'm with the same issue, how can we do it using this.pretender?


Applying projs to Epochs made from read_epochs

currently I am getting an error when I try to apply a projection to an Epochs that was created from mne.read_epochs. Here's how you can recreate the error with the sample data:
import mne
from mne.datasets import sample
from mne.utils import _TempDir

data_path = sample.data_path()
temp_path = _TempDir()
fname_event = data_path + '/MEG/sample/sample_audvis_filt-0-40_raw-eve.fif'
fname_raw = data_path + '/MEG/sample/sample_audvis_filt-0-40_raw.fif'
fname_epo = temp_path + 'sample_audvis_filt-0-40_raw.fif'
fname_proj = data_path + '/MEG/sample/sample_audvis_eog_proj.fif'

event_id, tmin, tmax = 1, -0.2, 0.5

# Load data
raw = mne.io.read_raw_fif(fname_raw)
events = mne.read_events(fname_event)
proj = mne.read_proj(fname_proj)

# Add a bad channel
raw.info['bads'] += ['EEG 053']  # bads + 1 more

# pick MEG channels
picks = mne.pick_types(raw.info, meg=True, eeg=False, stim=False, eog=True,
                       exclude='bads')
# Read epochs
epochs = mne.Epochs(raw, events, event_id, tmin, tmax, picks=picks,
                    baseline=(None, 0), reject=dict(mag=4e-12, grad=4000e-13,
                                                    eog=150e-6))
# This has no problem
epochs.copy().add_proj(proj).apply_proj()
# Save without proj added
epochs.save(fname_epo)

epochs_s = mne.read_epochs(fname_epo)
epochs_s.add_proj(proj)
# This raises an error
epochs_s.apply_proj()

indeed it's broken ... really broken...
read_epochs does not set detrend attribute  but also not the
_raw_times attribute.
see #2202

how about read_epochs returns an EpochsArray ?

how about read_epochs returns an EpochsArray ?

something feels wrong about this. You would expect to read what you saved, not?

yeah but it's broken now. There are too many attributes that relate to raw
which we don't save. This needs to be fixed. I have no time for this these
days :(

IMO this is another reason we need to refactor the Epochs classes.

(I hope to get to it this week)

+1000

IMO this is another reason we need to refactor the Epochs classes.

Seems Epochs needs some therary sessions indeed ... I think it's suffering from some developmental disorder.


init for kickstart(?) task

/to @camelburrito

@sebastianbenz , if you think we can merge this ourselves (only in this folder) let us know :)

Sure. Go ahead.


How to use with Fiber

Hello @davidebianchi , I'm trying to use this with Fiber but the example is not very clear. Assuming my Fiber main is as follow:
func main() {
  app := fiber.New()

  api := app.Group("/api", middleware) // /api

  v1 := api.Group("/v1", middleware)   // /api/v1
  v1.Get("/list", handler)             // /api/v1/list
  v1.Get("/user", handler)             // /api/v1/user

  v2 := api.Group("/v2", middleware)   // /api/v2
  v2.Get("/list", handler)             // /api/v2/list
  v2.Get("/user", handler)             // /api/v2/user

  log.Fatal(app.Listen(":3000"))
}

How do you add gswagger support?

Hi, this code block should be enough:
import (
	oasFiber "github.com/davidebianchi/gswagger/support/fiber"
)

...

func main() {

	// app definition

	oasRouter, err := swagger.NewRouter(oasFiber.NewRouter(app), swagger.Options{
		Context: context.Background(),
		Openapi: &openapi3.T{
			Info: &openapi3.Info{
				Title:   "my-service",
				Version: "the version",
			},
		},
		JSONDocumentationPath: "/swagger/json",
		YAMLDocumentationPath: "/swagger/yaml",
	})

	if err = oasRouter.GenerateAndExposeOpenapi(); err != nil {
		return nil, err
	}

Does it make sens to you?

@fredmaggiowski I have the following, but have no idea how to access the content generated by gswagger.
package main

import (
    "context"
	swagger "github.com/davidebianchi/gswagger"
	oasFiber "github.com/davidebianchi/gswagger/support/fiber"
	"github.com/gofiber/fiber/v2"
	"github.com/getkin/kin-openapi/openapi3"
)

func main() {
	app := fiber.New()

	app.Get("/", func(c *fiber.Ctx) error {
		return c.SendString("GET request")
	})

	app.Post("/", func(c *fiber.Ctx) error {
		return c.SendString("POST request")
	})
	oasRouter, _ := swagger.NewRouter(oasFiber.NewRouter(app), swagger.Options{
		Context: context.Background(),
		Openapi: &openapi3.T{
			Info: &openapi3.Info{
				Title:   "my-service",
				Version: "the version",
			},
		},
		JSONDocumentationPath: "/swagger/json",
		YAMLDocumentationPath: "/swagger/yaml",
	})

	oasRouter.GenerateAndExposeOpenapi()
	app.Listen(":3000")
}

On which path is the OpenAPI exposed to? /swagger, /openapi both return nothing.

Found it /swagger/json, mine returns empty though.

Hi @gaby! Yes, the path is that.
You should create the routes after the setup of gswagger router, as in this test setup: https://github.com/davidebianchi/gswagger/blob/main/support/fiber/integration_test.go#L120-L139

@davidebianchi Thanks, got it to work with:
package main

import (
	"context"
	"net/http"

	swagger "github.com/davidebianchi/gswagger"
	oasFiber "github.com/davidebianchi/gswagger/support/fiber"
	"github.com/getkin/kin-openapi/openapi3"
	"github.com/gofiber/fiber/v2"
)

func main() {
	app := fiber.New(fiber.Config{
		EnablePrintRoutes: true,
	})

	router, _ := swagger.NewRouter(oasFiber.NewRouter(app), swagger.Options{
		Context: context.Background(),
		Openapi: &openapi3.T{
			Info: &openapi3.Info{
				Title:   "my-service",
				Version: "the version",
			},
		},
		JSONDocumentationPath: "/swagger/json",
		YAMLDocumentationPath: "/swagger/yaml",
	})

	_, _ = router.AddRawRoute(http.MethodGet, "/hello", okHandler, swagger.Operation{})

	router.GenerateAndExposeOpenapi()

	app.Listen(":3000")
}

func okHandler(c *fiber.Ctx) error {
	c.Status(http.StatusOK)
	return c.SendString("OK")
}

Makes me wonder if there's a way to simplify this, as to not have to use router.AddRawRoute(). We do have the GetRoutes() method which I think could be used to simplify this?
https://docs.gofiber.io/api/app#getroutes

Yes, if the handler is not returned it could be a problem. Also, inside the last params of the AddRawRoute and AddRoute methods there are the logic to set in the api documentation the json schema of the request and the response, which would not be possible using the GetRoutes method.

I'm going to close this since the examples above proof that gswagger can be used with Fiber. Thanks for your help! 💪


docs: 增加如何避免表格滚动时图片重复加载的文档和示例 #2737

👀 PR includes

🔧 Chore

[ ] Test case
[x] Docs / demos update
[ ] CI / workflow
[ ] Release version
[ ] Other ()

📝 Description

简化文档自定义图片的写法, 直接传入 url
增加自定义图片使用缓存避免重复加载的 FAQ 和示例

🖼 Screenshot

Before
After

❌
✅

🔗 Related issue link

ref #1499
🔍 Self-Check before the merge

[ ] Add or update relevant docs.
[ ] Add or update relevant demos.
[ ] Add or update test case.
[ ] Add or update relevant TypeScript definitions.

:tada: This PR is included in version @antv/s2-v2.0.0-next.23 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @antv/s2-react-v2.0.0-next.22 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fix: cleanup k8s hyperkube check logic

What type of PR is this?
/kind cleanup

What this PR does / why we need it:
Remove checks and helper function for installing k8s hyperkube, which was not used after k8s 1.18.
Which issue(s) this PR fixes:

Fixes #
Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 61.849%

Totals

Change from base Build<PHONE_NUMBER>:
0%

Covered Lines:
2516

Relevant Lines:
4068

💛  - Coveralls


Add message to play audio note

Did we want an AudioNoteSequence message as well that is a vector of AudioNote?  Assuming we go that route, should the append flag be in the AudioNoteSequence instead of AudioNote


HTTP views do not match spec

Measure, view and tag names of ochttp are all out of sync with the spec. We need to implement the views as per the spec.

Just wondering if there is any update on this. http_server_route is one of the important tag that I am looking for in the server stats.

I'm OK to work on a PR for this. However, I would also like to understand the direction of this project as I know OpenTelemetry is on its way.

Submitted a PR for this: https://github.com/census-instrumentation/opencensus-go/pull/1159

I'll purposely leave out backward compatibility in the PR. Let me know if it is a concern.

I'm OK to work on a PR for this. However, I would also like to understand the direction of this project as I know OpenTelemetry is on its way.

@johansja It would be best to do this in opentelemetry.

What is the status of OpenTelemetry at the moment? Is it on parity with this now? I am looking for traces and stats with Stackdriver exporter.

Beta version of OpenTelemetry should be available by Oct 1st.

What are the things that will be available in beta?


The bot is offline

The bot wont delete chats in the server and go offlineee

@riking just making sure you know about this

We're aware. There have been several attempts to restart it, it keeps
failing eventually.
On Thu, Oct 7, 2021, 1:05 PM rcriv @.***> wrote:

@riking https://github.com/riking just making sure you know about this
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/riking/AutoDelete/issues/34#issuecomment-938115790,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEZJMYDI2O7BSHL3DZDYPTUFX4QTANCNFSM5FRGINPA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Oh, thank you! I deleted my post because it had started working again.
Thanks so much for doing this. My users really noticed its absence!
Holly

On Oct 7, 2021, at 3:40 PM, Kane York @.***> wrote:
We're aware. There have been several attempts to restart it, it keeps
failing eventually.
On Thu, Oct 7, 2021, 1:05 PM rcriv @.***> wrote:

@riking https://github.com/riking just making sure you know about this
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/riking/AutoDelete/issues/34#issuecomment-938115790,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEZJMYDI2O7BSHL3DZDYPTUFX4QTANCNFSM5FRGINPA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/riking/AutoDelete/issues/34#issuecomment-938204786, or unsubscribe https://github.com/notifications/unsubscribe-auth/AV6YO3X456UJE3DN6NENMZDUFYOWVANCNFSM5FRGINPA.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Hi have also noticed that the bot is currently offline


.toBeVisible should consider "hidden" attribute

Describe the feature you'd like:

expect(element).toBeVisible() should return false if element.hasAttribute('hidden') unless it's been explicitly shown with styles (display && display !== 'none' for example).
Suggested implementation:
Add && !element.hasAttribute('hidden') here, plus a check on display:
https://github.com/gnapse/jest-dom/blob/238cc91bfaffa9dd5baed1b5a02d060909f7c30c/src/to-be-visible.js#L17-L22
Describe alternatives you've considered:
I'm using .toHaveAttribute("hidden"), but I feel like I'm testing an implementation detail.

I think it may be considered a bug since the element is not visible on the browser.

Thanks for the bug report @diegohaz. Do you want to contribute this yourself? If not I can tackle it later this week.

Sure, I'm gonna do that later today.

:tada: This issue has been resolved in version 3.1.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[FLINK-24481][docs] Translate buffer debloat documenation to chinese

What is the purpose of the change
Translate buffer debloat documenation to chinese.
Brief change log

Translate related English to Chinese in mem_setup.md, mem_setup_tm.md and network_mem_tuning.md.

Verifying this change
Please make sure both new and modified tests in this PR follows the conventions defined in our code quality guide: https://flink.apache.org/contributing/code-style-and-quality-common.html#testing
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)

@flinkbot run azure

@xintongsong Thank you for the review. I have modified the code. Please review again.

I've pushed the branch with the following changes:

Reorganized commit history. Hotfix is preferred to be committed before the major changes, and should fix for both English and the untranslated Chineses documents.
Updated the commit message.
Addressed the last comments in a fixup commit.

@Myracle, please double-check on the changes I made. I'll merge after your confirmation, unless there's other concerns.

@xintongsong Thanks for the last changes which are really helpful. I have no problem.

@flinkbot  run azure

@flinkbot run azure

Merging as the PR passes compiling and doc_404_check in CI. Failures are irrelevant.


Draw relative to the containing rect

Details in #115
Idea is to draw relative to the containing window and not compensate absolute drawing with pan and zoom.

This can cause issues like in #124 when the graph fitted completely out of view.


ROX-10782: Add support for search with PostgresDB for k8sRole

Description
Add support for PostgresDB search backend.
Changes:

skips for Postgres DB tests are removed

Checklist

[ ] Investigated and inspected CI test results
~[ ] Unit test and regression tests added~ - no new tests added
~[ ] Evaluated and added CHANGELOG entry if required~ - not relevant
~[ ] Determined and documented upgrade steps~ - not relevant
~[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)~ - not relevant

If any of these don't apply, please comment below.
Testing Performed

spin up local Postgres DB: docker run --name test-sr -p 5432:5432 -e POSTGRES_USER=sr -e POSTGRES_PASSWORD=sr -e POSTGRES_DB=sr -d postgres
export env variables for test

export ROX_POSTGRES_DATASTORE=true

export USER=sr
export POSTGRES_PASSWORD=sr
export POSTGRES_DB=sr
export POSTGRES_HOST=localhost

go to central/rbac/k8srole/datastore and run: go test -v .
to test old backend DB export following env variables

export ROX_POSTGRES_DATASTORE=false

and run again go test -v .

Let's wait for #1725 to land before we merge this.


test(provider): HeaderProvider of BlockchainProvider2

Closes https://github.com/paradigmxyz/reth/issues/10331

@shekhirin pending conflicts

@rkrasiuk waiting for other PRs to merge to rebase this one on top of main


[BUG] $sheet->getTitle() returns null; when you have only one sheet.

When you have only one sheet, $sheet->getTitle(); returns null.
$reader->each(function($sheet) {
$sheet_title = $sheet->getTitle();
dd($sheet_title); // Null
});

No bug, when there's only one sheet (like a csv) it immediatly gives you the rows. So :
$reader->each(function($row) {
});

If you always want to loop through all sheets (even if it's only one) you need to enable this setting: https://github.com/Maatwebsite/Laravel-Excel/blob/2.1/src/config/excel.php#L458


Documentation

Update README.md mentioning about what the project is, and how to use it.
Add CONTRIBUTING.md explaining the contributing process. Example: Power Menu CONTRIBUTING.md
Add CODE_OF_CONDUCT.md to maintain the order in the community. Example: Power Menu CODE_OF_CONDUCT.md

CONTRIBUTING.md and CODE_OF_CONDUCT.md have already been done. I will do README.md

@NotSugat I don't see CODE_OF_CONDUCT.md tho


Fix base ten output

Base ten output is naively using the same separator as all the other bases (four spaces, should be 3).
Ideally this would use a map of bases to spaces-per-separator so this behavior can be extended/customized/configured for more than just base 10.

Should probably change the default behavior to have -l 0, people can alias in a default if they want.
-l option will need to accept 2 args now (base + spaces) so we can update the map


Use conda-forge label feature for LTR

Solution to issue cannot be found in the documentation.

[X] I checked the documentation.

Issue
Thanks for a good repo.
Not sure technically how to do this, but would be good to be able to only install the LTR version and not getting the latest if an update is done. I think this can be done with the conda-forge "labels", now all packages is on main.
Checking the documentation it seems like this can be specified as described in https://conda-forge.org/docs/maintainer/conda_forge_yml/#upload-on-branch for the git branch with the LTR version.

I don't think you can specify the label to upload with this option though... I wonder if we should have 2 separate repos instead - qgis_ltr and qgis_latest?

Hi!
We have talked about this in the past:
https://github.com/conda-forge/qgis-feedstock/pull/191
https://github.com/conda-forge/cfep/pull/42
The suggestion was instead to create a meta-package that would take care of building both qgis and qgis-ltr. Both I have never had the time to try it out or to see if it makes sense.
@gillins would keeping the two branches in the same repository, but renaming the output name from qgis to qgis-ltr in the LTR branch work?

Thanks @SrNetoChan I had forgotten about this. Yes I guess we could change the output name for the LTR branch. Not sure if this would work or we have to re add as a new recipe for this. Do we know of other repos that do this? @ocefpaf any thoughts?

@gillins I guess we can try locally, right? Changing the recipe and see if it spits out a valid output?

Yep sounds good. Is this something you can try? I won't be back at my development PC for a couple of weeks yet.

The label idea is not bad but it would change how people install the package. One would need to do something like:
conda install "conda-forge/label/qgis_ltr::qgis"

The label is like a channel, so it should show up in any install command from environments to .condarcs if one is to manage the LTR properly.
@gillins to publish something to a different label in conda-forge one can use custom conda_build_config.yaml, like this one.

@ocefpaf so the post install step should add the ltr channel to the user's .condarc?
In your opinion would it be better to use this label approach than have a different package name? What do other conda forge packages do for situations like this?

Many thanks for implementing this, works well!
running
conda config --env --add channels conda-forge/label/qgis_ltr

and then conda install qgis gives LTR version and keeps it during conda update --all
Thanks!

and then conda install qgis gives LTR version and keeps it during conda update --all

This is nice (including for conda update qgis). I never use conda update --all for the qgis environments though. I assumed that in this case it was better to let conda update qgis handle those dependencies. Anyone know what the best practice is for this? Does qgis have the required libraries pinned and we can use update --all without issues?

@joaoponceleao, I rarely use update all. Most of my envs I either do not touch them after creation, sometimes even using conda-lock to record the exact env; Or I destroy and recreate the env. I've seen bad behavior with update all where a single poorly pinned package can cause havoc and break everything.

Personally, I just do update qgis and let conda decide what else it needs to update. But this is because on my env qgis is central.

I've seen bad behavior with update all where a single poorly pinned package can cause havoc and break everything.

Oh... I've learnt the hard way to never touch a PyTorch cuda env......


Noto 2.1.3

Hello
After the update, I can no longer open your app.
The icon in Android is gone and after installation, the open menu is grayed out
On the Android window of your app.
All my notes are blocked without being able to see them. I hope an update will come soon. If I uninstall, I lose everything.
Xiaomi Redmi 8
Android Version 10 Android 10
MIUI
V<IP_ADDRESS>.QCNEUXM
Architecture
64-bit
Instruction sets
arm64-v8a armeabi-v7a armeabi Qualcomm®
Snapdragon™ 439
CPU
4 x Cortex-A53
Cores 8
RAM size 4 GB

In an application manager, when I go to Noto and ask to open your application.
There is an error message from Android, with written this app is not allowed to run.

Hi, I'm sorry you're having this issue. Don't worry. I'll try to make sure the app works for you.
Please provide me with the information below:
Are you using the APK version from GitHub?
Did you choose a different icon for the app in the previous versions?
What Android version are you using?

Also are you by any chance using a custom launcher?

Thank you for your reply
Yes I use github apks. For previous versions too
I didn't change the icon. This is the default icon
Android 10
I use Xiaomi's default launcher
Thanks again

Okay, could you try installing the APK again without reinstalling the app, and open it directly when it shows you the menu that says "App is installed"?

Same problem f-droid version

Did the icon disappear without even launching the app?

Could you please try with this APK?

I can't.  As I told you, when the installation is complete.  The menu called "open" is grayed out or disabled.

What about the APK I sent, does it have the same issue?

Yes, Same problem

Please try again with this APK.

One more thing, do you have Play Store installed? If you do, try to launch the app from there.

Yes, the Play Store is installed.  But the "open" button is gone.  So I can't open your app, from PlayStore.  There is only the button to uninstall.

Yes with this APK I can now open.  But before it opens, there is a black screen for 1 or 2 seconds.
In the PlayStore, I found the button to open with this APK

Many thanks

Usually I don't have a black screen, before opening your app.  I opened it several times and each time now there is a black screen (the display lasts 1 or 2 seconds, and after your application opens).  I wanted to report.  Excuse me for not having answered quickly, I try to answer you but I am currently at work.
I didn't have time to look if there were any other problems. I only opened.

@Archeix7 Are you still facing this issue with the APK I have attached in my previous comment?

@franco27 Did you use the app before version 2.1.0? The reason I'm asking that is because I have changed the implementation of some part of the app in version 2.1.0, and now with the latest version (2.1.3) I reverted the changes back to the way it was, due to a bug happening in version 2.1.0. So, the black screen was always there in versions before 2.1.0.
Feel free to take your time in responding! And I'm happy that the app is working for you now!

Could you please try to install this version, and let me know if it still works?

I confirm this APK is still working.
I've been using your app since version 1.x

@Archeix7 Are you still facing this issue with the APK I have attached in my previous comment?
Yea, it's work

@Archeix7 Are you still facing this issue with the APK I have attached in my previous comment?

Working now, thx


Support Notification validator.

Now the type in UAttributes supports Publish, Notification, Request and Response
https://github.com/eclipse-uprotocol/up-core-api/blob/main/uprotocol/uattributes.proto#L132
We should separate the validator of Publish and Notification.

@sophokles73 Would you mind taking a look at the PR? I prefer to have this before merging https://github.com/eclipse-uprotocol/up-client-zenoh-rust/pull/21


UTF-8 encoding for resultset metadata doesn't work from JDBC Driver version greater than 7.2.2

Driver version
Provide the JDBC driver version (e.g. 10.2.0).
Works fine in 6.2.1.jre8 version
Fails from any driver version greater than 7.2.2.jre8 version
SQL Server version
Provide the output of executing SELECT @@VERSION on your target SQL Server.
Microsoft Azure SQL Data Warehouse - 10.0.18045.0 Oct 11 2022 03:10:58 Copyright (c) Microsoft Corporation
Client Operating System
Provide the Operating System the client application is running on.
Windows 10 / Linux RHEL and Ubuntu
JAVA/JVM version
Provide the JAVA/JVM version (e.g. java version "1.8.0").
Java 1.8
Table schema
Provide the table schema to repro the issue.

Create a table with the above schema
get the colum metadata using Java + JDBC Driver
ResultSet resultSet = metadata.getColumns( null, "AzureDW_DIatScale", "tk_日本人_日本人_日本人_日本人_src", null );
Print the column names
System.out.println((resultSet.getString("COLUMN_NAME")));
output from 6.2.1.jre JDBC driver
日本日本人日本人_日本人engnew_colnColnCol1
output from 11.2.1.jre JDBC driver
????????_???engnew_colnColnCol1
Expected behavior
A clear and concise description of what you expected to happen.
Expected output
日本日本人日本人_日本人engnew_colnColnCol1
output from 11.2.1.jre JDBC driver
????????_???engnew_colnColnCol1
Actual behavior
Output of what you actually see.
output from 11.2.1.jre JDBC driver
????????_???engnew_colnColnCol1
Error message/stack trace
Complete error message and stack trace.
No Error message, however the columns name are not encoded properly as it is done till 7.2.2.jre8 JDBC driver
Any other details that can be helpful
Add any other context about the problem here.
Please note - This is not related to IDE encoding or the machine encoding or the configuration and purely JDBC Driver issue, as it works fine when use JDBC driver version till 7.2.2.jre8 and fails post that.
JDBC trace logs
Provide the JDBC driver trace logs.   I don't see how you are getting ??? for the characters if environment encoding isn't the issue. Like your settings, I have mine set to UTF-8 as well.
Could you provide an exact repro app for us?

We have spent considerable amount of time with Synapse tem at Microsoft on the incident<PHONE_NUMBER>001337  and they have been able to reproduce this issue in their environment, will it be possible to sync up with them and work it out internally, I will ask MSFT team to reach out and provide the repro.

I'm reviewing the support thread/ticket you mentioned. It looks like v11.2.1 was verified to not be able to view the characters, but I don't see a mention of v6.2.1 being tested to view the characters. I'll need to confirm with them as v6.2.1 had the same behaviour as v11.2.1 for me depending on the environment encoding.

just to clarify, we are seeing the same results on all versions of the driver including 6.2.1 so we are asking if you had tried the same application in the same environment with version 6.2.1 as well as 11.2.1 of the driver? Note in Windows terminal, you need to change locale to UTF-8 in order to see characters such as chinese characters, this has nothing to do with the driver you will need to do that if you had a text file containing characters in chinese encoding as well.

I've reviewed the support ticket. They're saying v6.2.1 wasn't able to successfully view the characters but v6.4.0 can. However, when I tested with their repro on v6.4.0 and v6.2.1, I still wasn't able to view the characters (all this was done on a fresh environment like before). I inclined to say this isn't a driver issue, but I'll standby for more info for the time being unless I'm wrong somewhere.

Please note - This is not related to IDE encoding or the machine encoding or the configuration and purely JDBC Driver issue, as it works fine when use JDBC driver version till 7.2.2.jre8 and fails post that.
This was mentioned in the Any additional details that can help section, while raising the issue. We have observed that it UTF-8 characters works fine till version 7.2.2 and starts to fail from there till the latest version available.

Hi, I've managed to repro the problem with the help of MSFT support engineeers and I've identified the exact cause. I already communicated this to the support engineers on your case, but I'll also leave a summary of my findings on this thread as well.
The reason that the characters aren't visible is because of the meta query to get the column name information. This meta query is selecting against the database and if the database collation doesn't support the characters then the characters will of course not be visible. The meta query is written this way for two reasons. getColumns needs to be JDBC compliant, meaning that the column names of the result set it returns needs to match what is defined in the JDBC spec. For this reason, we need to use table variables to correct the column names to be compliant in the meta query. However, Azure DW does not support table variables nor insertion of data into temp tables from exec calls. So, the meta query is written in way that doesn't depend on these sorts of syntax, but doing so affects the character encoding as you have seen. The older driver versions (< 7.2.2) are working because they directly do a transact-sql query for the column info which isn't affected by collation/encoding, but the result set isn't JDBC compliant. So, there isn't an easy way to fix this in the driver. The only immediate solution I see is to switch the collation of the database to a collation that supports the characters. If I learn anything more, I'll let you know.
As for a future fix, it doesn't seem likely with the constraints I mentioned prior. The trade off here is either being JDBC compliant or be affected by encoding. While being affected by encoding isn't ideal, it can be worked around by setting the db to a collation that supports the characters. Whereas not being JDBC compliant can't really be worked around on the user side.


Add support for signing and verification

CycloneDX SBOMs can be signed at the root bom level and can also be signed on a component level.
This enhancement request is to add support for applying signatures to the bom level or to a component level. Signing should support XML Signature and JSON Signature Format.
When signing a component, it will be necessary to include the bom-ref of the component. If a bom-ref does not exist or cannot be found for the component, signing should fail.
Signing verification should ideally validate that ALL signatures in the bom regardless of where they're located. This includes the root bom level signatures as well as individual component signatures. All occurrences of failed verification should be logged to to STDERR.
Proposed input parameters:

Private key (used for signing)
The filename of the bom to sign or verify (e.g. bom.xml or bom.json)
The component bom-ref to sign (optional)

BOM XML signing added by #177

using Cdxgen we can sign the SBOM using in JWS format, but there is one limitation we cannot sign the SBOM that is already generation via Different tool or downloaded from Dependency Track with VEX

Related to this ticket, I have added a section in the readme with a snippet to verify the json signature.
https://github.com/CycloneDX/cdxgen/#verifying-the-signature-nodejs-example
It would be nice for the cli to support verifying all types of signatures - multisignature, signaturechain, and signaturecore. While the information is lacking, verifying against a chain usually involves checking a revocation list. We could piggyback on recent inventions such as CRLite (CRLsets) from the browser world or propose our own.


Aggregations (experimental)

Placeholder page merged in here with link to release notes: https://github.com/prisma/docs/pull/660


Cache geometry types

@javisantana please review

Frontend tests were OK :+1: (details)

sounds good

retest this, please

Frontend tests were OK :+1: (details)

Frontend tests were OK :+1: (details)


add start of contributing guide

Adds the start of a contributing guide.
Signed-off-by: Mike Brown<EMAIL_ADDRESS>

@MHBauer comments addressed!


feat: nix based devcontainer

Description
Nix based devcontainer.
Used along https://devpod.sh/
Problem*

Resolves
Summary*

Documentation

[ ] This PR requires documentation updates when merged.

[ ] I will submit a noir-lang/docs PR.

[ ] I will request for and support Dev Rel's help in documenting this PR.

Additional Context

PR Checklist*

[ ] I have tested the changes locally.
[ ] I have formatted the changes with Prettier and/or cargo fmt on default settings.

Is this related / superseded by https://github.com/noir-lang/noir/blob/9827dfe51118ba55da6da51ab8bf45cffd2ca756/Dockerfile?

What's the status of this PR?


Modify readme to describe why CPT is not supported in conan 2.0

Maybe conan 2.0 provides some core-features that deprecates this project or there is some fundamental incompatibilities or it is just work-in-progress or did it became obvious over time that it was easier to use native CI/CD tools to produce massive package builds?
The readme asks us to "open an issue explaining your current case with more details", but it is not the case for new user is introduced to the project for the first time.
Thanks!

Thanks for the feedback @aborzunov
There are no plans to maintain this project, all the similar extensions for Conan 2 has been moved to conan-extensions repo.
I agree it is time to deprecate clearly this repo, and document it clearly in its README.

Conan 2.0 support ⚠
The project Conan Package Tools does not support Conan 2.x and there is no current planned support.

Well, this is clear, isn't it?
New users shouldn't be introduced to this repo anymore, it is not even linked in the Conan 2 documentation.

The readme asks us to "open an issue explaining your current case with more details", but it is not the case for new user is introduced to the project for the first time.

This means if you were a CPT user in Conan 1 and have some need for Conan 2, please comment, and we will try to help and redirect to other approaches in Conan 2.