error : fork/exec ... no such file or directory

Congrats on your first issue for this project. And sorry for being the first one to bother you.
OS: Arch Linux x86_64
libappimage version : <IP_ADDRESS>-5
[~] ➤ pho install github ONLYOFFICE/appimage-desktopeditors

>   Name            Appimage-Desktopeditors
>   Identifier      onlyoffice-appimage-desktopeditors
>   Version         v7.4.1
>   Filename        DesktopEditors-x86_64.AppImage
>   AppImage        /home/para/.local/share/pho/applications/onlyoffice-appimage-desktopeditors/Appimage-Desktopeditors.AppImage
>   .desktop file   /home/para/.local/share/applications/onlyoffice-appimage-desktopeditors.desktop
>   Download Size   341.87 MB

? Do you want to proceed? yes

[1/1] ! Appimage-Desktopeditors v7.4.1 (3m:9s)
error: fork/exec /home/para/.local/share/pho/applications/onlyoffice-appimage-desktopeditors/Appimage-Desktopeditors.AppImage: no such file or directory

[~] ➤ ls /home/para/.local/share/pho/applications/onlyoffice-appimage-desktopeditors/Appimage-Desktopeditors.AppImage
Permissions Size User Date Modified    Name
.rwxr-xr-x  342M para 2023-10-09 14:58 /home/para/.local/share/pho/applications/onlyoffice-appimage-desktopeditors/Appimage-Desktopeditors.AppImage

Just downloaded the pho binary, pasted the binary into ./.local/bin and ran pho init.  The rest is in the output above. The file is downloaded successfully.
Installing the downloaded file gives this error :
[~] ➤ pho install local /home/para/.local/share/pho/applications/onlyoffice-appimage-desktopeditors/Appimage-Desktopeditors.AppImage
? What should be the Application ID? appimage-desktopeditorsappimage
? What is name of the Application? Appimage-Desktopeditorsappimage

>   Name            Appimage-Desktopeditorsappimage
>   Identifier      appimage-desktopeditorsappimage
>   Version         0.0.0
>   AppImage        /home/para/.local/share/pho/applications/appimage-desktopeditorsappimage/Appimage-Desktopeditorsappimage.AppImage
>   .desktop file   /home/para/.local/share/applications/appimage-desktopeditorsappimage.desktop

? Do you want to proceed? yes

[1/1] ! Appimage-Desktopeditorsappimage 0.0.0 (1s)
error: mkdir /home/para/.local/share/pho/applications/appimage-desktopeditorsappimage/temp: file exists

Additional info

Congrats on your first issue for this project. And sorry for being the first one to bother you. OS: Arch Linux x86_64 libappimage version : <IP_ADDRESS>-5
[~] ➤ pho install github ONLYOFFICE/appimage-desktopeditors

>   Name            Appimage-Desktopeditors
>   Identifier      onlyoffice-appimage-desktopeditors
>   Version         v7.4.1
>   Filename        DesktopEditors-x86_64.AppImage
>   AppImage        /home/para/.local/share/pho/applications/onlyoffice-appimage-desktopeditors/Appimage-Desktopeditors.AppImage
>   .desktop file   /home/para/.local/share/applications/onlyoffice-appimage-desktopeditors.desktop
>   Download Size   341.87 MB

? Do you want to proceed? yes

[1/1] ! Appimage-Desktopeditors v7.4.1 (3m:9s)
error: fork/exec /home/para/.local/share/pho/applications/onlyoffice-appimage-desktopeditors/Appimage-Desktopeditors.AppImage: no such file or directory

[~] ➤ ls /home/para/.local/share/pho/applications/onlyoffice-appimage-desktopeditors/Appimage-Desktopeditors.AppImage
Permissions Size User Date Modified    Name
.rwxr-xr-x  342M para 2023-10-09 14:58 /home/para/.local/share/pho/applications/onlyoffice-appimage-desktopeditors/Appimage-Desktopeditors.AppImage

Just downloaded the pho binary, pasted the binary into ./.local/bin and ran pho init. The rest is in the output above. The file is downloaded successfully.
Installing the downloaded file gives this error :
[~] ➤ pho install local /home/para/.local/share/pho/applications/onlyoffice-appimage-desktopeditors/Appimage-Desktopeditors.AppImage
? What should be the Application ID? appimage-desktopeditorsappimage
? What is name of the Application? Appimage-Desktopeditorsappimage

>   Name            Appimage-Desktopeditorsappimage
>   Identifier      appimage-desktopeditorsappimage
>   Version         0.0.0
>   AppImage        /home/para/.local/share/pho/applications/appimage-desktopeditorsappimage/Appimage-Desktopeditorsappimage.AppImage
>   .desktop file   /home/para/.local/share/applications/appimage-desktopeditorsappimage.desktop

? Do you want to proceed? yes

[1/1] ! Appimage-Desktopeditorsappimage 0.0.0 (1s)
error: mkdir /home/para/.local/share/pho/applications/appimage-desktopeditorsappimage/temp: file exists

Seems like installation failed for some reason. You don't need to manually use the install local command after install github.

I'm aware. I was just testing to see if local install will work

I probably guess it failed to extract the AppImage before it is being integrated. Can you try using tidy-broken command (deletes failed installation files) and running the install command again?

Worked! thanks.
On Mon, Oct 9, 2023, 3:21 PM Zyrouge @.***> wrote:

I probably guess it failed to extract the AppImage before it is being
integrated. Can you try using tidy-broken command (deletes failed
installation files) and running the install command again?
—
Reply to this email directly, view it on GitHub
https://github.com/zyrouge/pho/issues/1#issuecomment-1752695035, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AELOENP3IR66ABYVV2PAYYDX6PCKHAVCNFSM6AAAAAA5YQLQKWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONJSGY4TKMBTGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

I guess the issue has been resolved. Closing this.


Pin app windows to desktop

Please add a feature to pin the app windows to the desktop. Like in windows seven, you could create sticky notes that just stayed on the desktop, even if you did "Win + D". These notes also didn't minimize when you opened another app.

can you give an example of apps / scenarios where you'd want this behavior?

typically if you told everything to minimize, you' really want everything to minimize

I would really like this to happen with the sticky notes app, like I mentioned in the original post, like a sort of widget that was permanently on your desktop, unless you unpinned it. I suppose you could also refer to it as "widget-ifying" apps without relying on third-party tools.

Good idea, but lets closing this against #1354 as the tracking issue


refactor(lib): encode key with base64url

See discussion: https://github.com/Kong/kong/pull/11506#issuecomment-1706188287

We have discussed it offline with @dndx, base64url can not give us more useful help.


[BugFix] Fix concurrent modify bug for handleDML

What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #16311
Problem Summary(Required) ：

The dataSink init and complete should be executed in db read lock, because there are meta read in those functions.
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto backported to target branch

[x] 2.5
[x] 2.4
[x] 2.3
[x] 2.2

[FE PR Coverage Check]
:heart_eyes: pass : 5 / 5 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/qe/StmtExecutor.java
5
5
100.00%
[]

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

run starrocks_fe_unittest

[FE PR Coverage Check]
:disappointed: fail : 3 / 4 (75.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/qe/StmtExecutor.java
3
4
75.00%
[1313]

@Mergifyio backport branch-2.5

@Mergifyio backport branch-2.4

@Mergifyio backport branch-2.3


configure phoenix apps

configures the apps and external apps for the
ocis & ocis-simple build by conditionally including a
different configurePhoenix function

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- pkg/command/phoenix_simple.go  1
- pkg/command/phoenix_ocis.go  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- pkg/command/phoenix_simple.go  1
- pkg/command/phoenix_ocis.go  1

See the complete overview on Codacy


billing.ConnectAsync(true);  hangs in Maui.Android app, crashes iOS app

Hello,
EDIT - FIXED, see response below.
@jamesmontemagno
I have used this plugin with great success in Xamarin.Forms Android and iOS apps.
I have moved my billing code into my new Maui App, identical to what is working in the live Xamarin apps.

The iOS app runs then crashes trying to load the CrossInAppBilling library. No crash details, just application terminated.
The Android app runs and hangs on the await billing.ConnectAsync(true); call.

Is there any new pre-config required for Maui Apps?
Is a callback expected that might be blocked in Maui apps?
Eg: iOS we have this:
InAppBillingImplementation.FinishAllTransactions = true;
InAppBillingImplementation.OnShouldAddStorePayment = OnShouldAddStorePayment;
var current = Plugin.InAppBilling.CrossInAppBilling.Current;
Must be something similar required for Maui apps in the MauiProgram.cs with a Builder.UseInAppBilling call or something that does the same or similar to that in the AppDelegate.cs in Xamarin.
Any guidance would help!
Bug Information
Version Number of Plugin: 7.0.5
Device Tested On: Samsung Android tablet and yes, no data connection issues as a Xamarin.forms app works.
Simulator Tested On:
Version of VS: 2022 - Version 17.7.3
Version of Xamarin: using a Maui cross platform project
Versions of other things you are using: Android, iOS and Windows platforms in project, target .Net 7
Steps to reproduce the Behavior
Expected Behavior
Should connect and allow me to execute this:
var purchases = await billing.GetPurchasesAsync(ItemType.Subscription);
Actual Behavior
iOS crashes app.
Android just hangs on this call:
var connected = await billing.ConnectAsync(true);  never gets further
Code snippet
        public async Task<bool> CheckOrRestoreSubscription()
        {
            var billing = CrossInAppBilling.Current;   <------------------ iOS app crashes here
            bool subscribed = false;
            try
            {
                var connected = await billing.ConnectAsync(true); <-----------Android app never gets past this line. No break, no error, no output in immediate window, nothing...

                if (!connected) return false;

                  check for subscriptions from here....

Screenshotst
I understand it takes time to work on these plugins. But this is an essential part of app development in the Xamarin (and soon in the Maui space). Maybe this should be a priority and cleaned up to work for Maui apps. And for .Net 7 too!
Google requires all apps to target Android API 33 by Nov 1st. Every other library I use is up to date  with .Net 7.0 and Android API 33, not to mention iOS. The app works without InAppBilling. Once this is included, won't get through InAppBilling because of errors above.
If anyone knows of any other/new billing framework for Maui App, please advise here!

Timing is of the essence with this plugin and Maui.
In Xamarin I was calling my BillingService at startup and checking purchases then giving users the option to subscribe BEFORE the MainPage was loaded.
In Maui, the app needs to complete loading, THEN invoking Billing functionality from a button Click or another event.
Also, no need to override FinishedLaunching in the iOS AppDelegate.
Lastly, I was calling billing some methods on a background thread. Removed those and awaited directly.
All works. Really glad this plugin IS actually up to speed with Maui Apps and .Net 7


<EMAIL_ADDRESS>0

preminor release

Coverage remained the same at 63.347% when pulling 01556fa2aeb2146ff7bb102c47c3efb79c9ac883 on<EMAIL_ADDRESS>into a1fcf3ec2c6cb25913b44a9622def2a09d734565 on master.

Coverage remained the same at 63.347% when pulling 01556fa2aeb2146ff7bb102c47c3efb79c9ac883 on<EMAIL_ADDRESS>into a1fcf3ec2c6cb25913b44a9622def2a09d734565 on master.


Remove extraneous double mixin for the require.

Description
There is a bad require call in knife/lib/chef/knife/ssl_fetch.rb where a path was accidentally doubled. This fixes that by removing the extraneous extra keyword.
Related Issue
Fixes  knife ssl fetch broken with bad require call in ssl_fetch.rb #11580
Types of changes

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

Checklist:

[X] I have read the CONTRIBUTING document.
[X] I have run the pre-merge tests locally and they pass.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[X] All new and existing tests passed.
[ ] All commits have been signed-off for the Developer Certificate of Origin.

Nice catch. Thanks @Dylan-M

No problem @tas50! It frustrated me yesterday while trying to get my new laptop set up, and I was too blind to see the double path. Looked at it again this morning and had a nice "Doh!" moment.


✨ Provide remediation suggestions in results

Please check if the PR fulfills these requirements

[x] Tests for the changes have been added (for bug fixes / features)
[x] PR title follows the guidelines defined in  https://github.com/ossf/scorecard/blob/main/CONTRIBUTING.md#pr-process

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Feature

What is the current behavior? (You can also link to an open issue here)
Remediation results are not included in the terminal results
Closes #304

What is the new behavior (if this is a feature change)?
When run with the --show-details flag, the results in the terminal will include remediation suggestions. Remediation will only be displayed if the check has a certain level of confidence (currently 5 or higher but I'm open to modifying that number). This should prevent remediation steps from showing if the check failed due to internal issues such as being rate-limited in API requests.

The remediation steps are pulled from checks.yaml which will be embedded into the scorecard binary. This PR increases the size of the binary from 19.93MB. to 19.95MB.
If a check has differing remediation steps based on the specific issue found, then codes can be assigned to those issues and those codes can be associated to specific remediation steps in checks.yaml. See "Frozen-Deps" as an example. If a set of remediation steps is not associated here with any code, those steps will be included for any failed check.
Here is a partial view of output that includes remediation steps:

Notice that the remediation results include markdown syntax. I thought about ways to scrub the markdown but I couldn't think of a way to do it that didn't disrupt the flow of the sentence.

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
No

Other information:
Here are some areas I'd like feedback on:

Is 5 the right confidence level for displaying remediation?
Should there be a separate flag for displaying remediation? Or should it always be displayed?
Is there a decent way to clean up the markdown syntax in the results?

IIUC, this PR is similar to Laurent's PR - #650, so adding @laurentsimon as a reviewer.

Yeah, it looks like this is similar.

Let's discuss both these PRs in next week's meet. Most of us might be out this week.

I won't be able to attend this meeting since that's moving day. I'll do my best to make the next one.

@laurentsimon  / @chrismcgehee  What do we want to do with this PR?

@laurentsimon / @chrismcgehee What do we want to do with this PR?

I see that now --show-details has a DOCUMENTATION/REMEDIATION column. I think that's probably sufficient, so I wouldn't mind closing this PR.

@laurentsimon / @chrismcgehee What do we want to do with this PR?

I see that now --show-details has a DOCUMENTATION/REMEDIATION column. I think that's probably sufficient, so I wouldn't mind closing this PR.

yes right, laurent landed this a week or two back, closing. there are lot of new things to do, can discuss in next biweekly. sorry for duplication.


Prism server always returns empty body response when "addtionalProperties" is false

Describe the bug
When additionalProperties is false, Prism server always returns empty body response.
To Reproduce

OpenAPI specification:

openapi: 3.0.0
info:
  title: Sample API
  version: v1
paths:
  '/services/resources/{resource_id}':
    get:
      summary: Get a resource info.
      description: returns a single resource.
      operationId: getResource
      responses:
        '200':
          description: A resource
          content:
            application/json:
              schema:
                $ref: '#/components/schemas/Resource'
        '404':
          description: Not found.
components:
  schemas:
    Resource:
      type: object
      properties:
        name:
          type: string
        tags:
          type: string
        createTime:
          type: string
          format: date-time
      additionalProperties: false

Start prism mock server and run curl http://localhost:4010/services/resources/5

Current behavior
{}
Expected behavior
{
"name": "aliquip",
"tags": "",
"createTime": "1985-06-04T11:15:57.166Z"
}
Environment:
Library version: 3.1.1
OS: MacOS Darwin Kernel Version 18.7.0
Additional context
The additionalProperties false means the properties not in the properties listed are not allowed in the request.  (Block the Invalid request). But the mock server should behave the same as addtionalProperties is true.

Verified. Apparently this is a bug in the underlying library we're using (json-schema-faker).
Although we're telling to generate optional values, for some reason it is ignoring the option value. I'll investigate and try to understand what is the problem (the library or the way we use it); in meantime you can workaround the problem by putting the properties you need in the schema's required array.

Yes confirmed, this is a bug in json-schema-faker.

Thank you!  Do you know when will be a new version released?

I think we're going to wait for #669 and #702 to get merged; then we will release 3.2.

Thank you!~~~

We have just released 3.2.0-beta.5 that fixes this issue. You can try it out installing Prism from npm with its beta tag
npm i @stoplight/prism-cli@beta

Hello,
Just asking again when will the next version be released?
Thanks
Henggai
On Sat, Oct 26, 2019 at 3:38 AM Vincenzo Chianese<EMAIL_ADDRESS>wrote:

We have just released 3.2.0-beta.5 that fixes this issue. You can try it
out installing Prism from npm with its beta tag
npm i @stoplight/prism-cli@beta
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stoplightio/prism/issues/715?email_source=notifications&email_token=AMGE7ELMEVW2FDGFKQAYPOLQQQM2BA5CNFSM4JB5K4LKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECKFERQ#issuecomment-546591302,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMGE7EPAPIFJN57K6L54NNTQQQM2BANCNFSM4JB5K4LA
.

We're in the process of verifying all the bits we've put in this version. We hope to make it happen within this week //cc @philsturgeon

Prism 3.2.0's been released!


Wrote about functions

@dakeshi, I would really appreciate your input and thoughts on this PR. 👏🏻🙏🏻

Of course. 🔍
I can't remember how many times I read the official Swift documents after taking part this project.
Good or Bad? 😆
Through the writing and review processI can more deeply understand the concept of Swift and
be surprised of many authors(of course, you too) ability to make easy documents.
Thank you again for making this awesome project.

I can't remember how many times I read the official Swift documents after taking part this project.
Good or Bad? 😆

Haha that is really a good point. I do that too 😛 That is one of the reasons I write books. To be able to teach, you have to learn!

Thank you again for making this awesome project.

You're welcome and thank you too for being such a huge part of this project 🙌 🙏 👍

I have been really busy the past few weeks with the birth of my twin daughters so I haven't been able to write anything here. I will continue soon... thanks for being patient :-)

Wow. twins? congratulations :congratulations:
I got a daughter in February so I totally understand how difficult to make balance between work and parenting. Good luck and really congratulations again.

@dakeshi thanks for your review. If you are happy with these changes, let me know and I will merge. In the future, I can send different pull requests but that requires extra manual branching which is a pain in the bum! 😔

I got a daughter in February

Many congratulations 😇


Add function Remove-HaloKBArticle

This function allows the DELETE operation of KB articles. It is modeled after other Remove- Commands in the module.

Ignore test failures - issue with pipeline not PR.

I just noticed that the develop branch is 2 commits ahead. Would you like me to re-submit the PR to that branch?

Thanks for this Jacob - sorry about the delay - doing some refactoring under the hood getting ready for 2.x.x :-)


Document is inconsistent with code, bazel.dev use clang instead of gcc

Description:
Document is inconsistent with code
in ci/README file,

bazel.dev — build Envoy static binary and run tests under -c fastbuild with gcc.

but in do_ci.sh
elif [[ "$1" == "bazel.dev" ]]; then
  setup_clang_toolchain

@hello-ming PR appreciated.


Update .nca Files Extract Doesn't work !

Hi @SciresM How are you? I want to open the game update and extract the contents.I get this error.
Note: cannot save BKTR section without base romfs.
Thanks.
Edit: I can extract game files without problems.
Please look.

Select --basenca=
If you don't have it, pass the --basefake and/or --onlyupdated options.

Select --basenca=
If you don't have it, pass the --basefake and/or --onlyupdated options.

Would you please write the full command?
Thank you.

Read the usage instructions, or seek help elsewhere.
The full command is not possible to share, as it depends on the inputs.

Read the usage instructions, or seek help elsewhere.
The full command is not possible to share, as it depends on the inputs.

Sorry, I tried these, but I didn't.
hactool -k keys.ini update.nca --titlekey=********** --onlyupdated --romfsdir=romfs
Problem
https://s18.directupload.net/images/190803/zfgu4jhy.png
hactool -k keys.ini update.nca --titlekey=********** --basefake --romfsdir=romfs
Problem
https://s18.directupload.net/images/190803/qcgd9d8t.png


Suggest code review improvements

I add more checklist for the code review process and group things together different categories.
Reference I use: https://swimm.io/learn/code-reviews/ultimate-10-step-code-review-checklist

Add more checklist for code review section and group things together into different categories.

Hi @meganhuibregtse! One of the RAs in my lab has been using our website as a basis for reviewing some data derivatives code and suggested these changes. What do you think?

Looks great! Love the changes.


✨ Move logo to its own partial

This allows for the logo to be overridden more easily without having to replace all of hamburger.html and basic.html.

Thanks for the PR. I've made a minor change and moved the logo file to the root partials directory so that it's not accidentally called as a header in a misconfigured site. Other than that, I'm happy to accept this change and merge it in.


Adding resource constants

This PR adds "APP_SCRIPTS_LOADED" and "RESOURCE_FAILED_TO_LOAD" as constants. It also marks "RESOURCE_FAILED_TO_LOAD" as depreciated.
Fixes #222.

After checking my commit, this is good to merge. Thanks @montlebalm


About simple reproduction

Hello,  there are too many files in the project, I would like to ask you if I just want to try to reproduce the simple results of the camvid dataset combined with the pixel contrast learning method, what should I go to see about model creation, loss function, What about the data preprocessing, and the .py files and .sh files for training and testing?
I hope you can take the time to give me some simple guidelines, which will benefit me a lot, thank you!

Hi @xianxuan-z , very good suggestion. However, I am sorry that I will not have too much time for this. It will be great if you can do this thing, and I also can provide supports for possible issues.


Must build with --symlink-install

I noticed that when building rqt (or any of its plugins) from source, that we must use the colcon build option --symlink-install. If we don't use this option or use --merge-install, then attempting to run rqt results in a Python exception that looks like the following:
Traceback (most recent call last):
  File "/home/jacob/ws/ros/rqt/install/bin/rqt", line 9, in <module>
    for entry_point in distribution('rqt-gui==1.0.6').entry_points:
  File "/usr/lib/python3.8/importlib/metadata.py", line 504, in distribution
    return Distribution.from_name(distribution_name)
  File "/usr/lib/python3.8/importlib/metadata.py", line 177, in from_name
    raise PackageNotFoundError(name)
importlib.metadata.PackageNotFoundError: rqt-gui==1.0.6

To reproduce:
mkdir -p rqt_ws/src
cd rqt_ws
git clone https://github.com/ros-visualization/rqt.git -b crystal-devel src/rqt
colcon build

In another shell:
cd rqt_ws/src
source install/setup.bash
rqt

This was previously reported in https://github.com/ros-visualization/rqt/issues/210, but there was no follow-up. I assume that building without --symlink-install should work. Correct me if I'm wrong.

Upgrading to setuptools 49.2.0 resolved the issue for me.


Use HEALTHCHECK

Add HEALTHCHECK to the Dockerfile, see https://github.com/OpenSlides/OpenSlides/issues/6301

@m-schieder This could be a small but useful improvement for production. Do we need more than a route and a static content?


be more specific about gls shipments and carrier numbers

Since GLS is creating a tracking number once the shipping label is scanned for the first time, we can't return it in our response, when a shipment for the carrier is being created.

ready to merge


[BUG] Core Playback does not parse/record body for GetPermission.

The GetPermission Storage API https://docs.microsoft.com/en-us/rest/api/storageservices/get-permission does not return a content-type header indicating JSON so the permission does not get deserialized and stored in playback files.

The change has pushed into master. Closed the issue. Please feel free to reopen if trigger any other issues.


Update minor bugs

Names of person in group now wrap text
Fix table not showing up properly

Codecov Report

Merging #344 (36959d4) into master (5658619) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #344   +/-   ##
=========================================
  Coverage     56.60%   56.60%
  Complexity      720      720
=========================================
  Files           139      139
  Lines          3196     3196
  Branches        420      420
=========================================
  Hits           1809     1809
  Misses         1286     1286
  Partials        101      101

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 5658619...36959d4. Read the comment docs.


podman build on cbs

Is this a BUG REPORT or FEATURE REQUEST? (leave only one on its own line)
/kind bug
Description

I needed to test 1.7 on centos 7 and noticed that the build has been failing.
Failure points to task here:
https://cbs.centos.org/koji/taskinfo?taskID=1092090
Result
GenericError: Unable to complete build: epoch mismatch (build: 2, rpm: None)

However, when I look at the last build tasks all files were successfully produced here:
https://cbs.centos.org/kojifiles/work/tasks/2091/1092091/
I downloaded srpm was able to build it successfully after installing all dependencies.
Could you take a look at the reason 1.7rc is not making to the repo?

Podman 1.7 RPM is still failing to build on cbs

@jnovy, could you have a look?

@jnovy It seems to be an issue with epoch 2 being used.  Successfully built 1.6.2 has no epoch.

@lsm5 Any update?

not using cbs anymore for new builds. Updates available on OBS.
@alitvak69 please re-open if you notice any issues with the latest packages.


add create_temp_table support

Allows users to configure create_temp_table by using the extra field in the connection form. The value of create_temp_table is true by default and, by setting it to false, it bypasses the creation of temporary tables from Great Expectations. Which is necessary when one only has read access to a database.
Usage:
In the extras field do the following:
{
"create_temp_table": false
}

closes: #112

Thanks for pushing this along! May I know when I can expect this to be merged and released, please? Thank you!

Thanks for pushing this along! May I know when I can expect this to be merged and released, please? Thank you!

We will probably release it early next week.

That is awesome! Thank you!


The obfused dll cannot be load:  Method 'x' in type 'xx' from assembly 'xxx' does not have an implementation

Hi,
ConfuserExTest.zip
This demo can reproduce the issue. There are 2 projects in this solution:

RenameBug1: A .NET class library that will be obfuscated with ConfuserEx. I use the Post-Build event to use ConfuserEx with configuration file ConfuserExTest\RenameBug1\confuserex.crproj. You may need to update the path for Confuser.CLI.exe in the Post-Build event.

ConfuserExDotNetBug: This is a test console app that uses the dll in part 1.

If I use the unobfuscated dll, then the console app works. If I use the obfuscated dll, then it will fail at runtime with error:
Test 'M:ConfuserExDotNetBug.Class1.Main' failed: Method 'FireLog' in type '_B._d' from assembly 'RenameBug1, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' does not have an implementation.
	System.TypeLoadException: Method 'FireLog' in type '_B._d' from assembly 'RenameBug1, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' does not have an implementation.
	at _B._f..ctor()
	RenameTest.cs(12,0): at test.Rename.PublicClass2..ctor()
	Class1.cs(6,0): at ConfuserExDotNetBug.Class1.Main()

It seems the method FireLog is renamed unintended.
I test this with the latest stable ConfuserEx 1.4.1
Thanks
Anthony

I think that is the same issue as reported in #187. Could you give the upcoming 1.5 version a try? You can download it here.

I think that is the same issue as reported in #187. Could you give the upcoming 1.5 version a try? You can download it here.

It's not working. The 1.5 version has the same result as ConfuserEx 1.4.1.

It's not working. The 1.5 version has the same result as ConfuserEx 1.4.1.

I finally got around to fix this issue. It now works for your example, do you have a real project you can use to verify the fix?
You can download the new build here on the buildserver.

I finally got around to fix this issue. It now works for your example, do you have a real project you can use to verify the fix?
You can download the new build here on the buildserver.

Thank you @mkaring. It works on my real project.

Thank you @mkaring. It works on my real project.


TypeError when running npm run dev

When attempting to execute the custom npm run dev script, I encounter a TypeError related to redefining a property named File. The error traces back to the @babel/core module. I have not made any direct modifications to this module, and the issue presents itself specifically during the execution of the aforementioned script.
Running npm update first used to solve this issue in a previous version of this repository, but it no longer does.
Error message:
[!] TypeError: Cannot redefine property: File
    at Function.defineProperty (<anonymous>)
    at Object.<anonymous> (C:\Users\aparn\Desktop\RA_C2C\TimeSearcherPlus\node_modules\@babel\core\lib\index.js:7:8)
    at Module._compile (node:internal/modules/cjs/loader:1241:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1295:10)
    at Module.load (node:internal/modules/cjs/loader:1091:32)
    at Function.Module._load (node:internal/modules/cjs/loader:938:12)
    at Module.require (node:internal/modules/cjs/loader:1115:19)
    at require (node:internal/modules/helpers:130:18)
    at Object.<anonymous> (C:\Users\aparn\Desktop\RA_C2C\TimeSearcherPlus\node_modules\@babel\core\src\config\helpers\config-api.ts:4:1)
    at Module._compile (node:internal/modules/cjs/loader:1241:14)

Steps to reproduce:

Clone the repository to local machine
Run npm install and npm update from within repository
Run npm run dev

Usually you don't need to have to run npm update
John Alexis Guerra Gómez
On my phone, brevity and all
On Sun, Sep 17, 2023, 4:47 PM Aparna Krishnan @.***>
wrote:

When attempting to execute the custom npm run dev script, I encounter a
TypeError related to redefining a property named File. The error traces
back to the @babel/core module. I have not made any direct modifications to
this module, and the issue presents itself specifically during the
execution of the aforementioned script.
Running npm update first used to solve this issue in a previous version of
this repository, but it no longer does.
Error message:
[!] TypeError: Cannot redefine property: File
at Function.defineProperty ()
at Object. @.\core\lib\index.js:7:8)
at Module._compile (node:internal/modules/cjs/loader:1241:14)
at Object.Module._extensions..js (node:internal/modules/cjs/loader:1295:10)
at Module.load (node:internal/modules/cjs/loader:1091:32)
at Function.Module._load (node:internal/modules/cjs/loader:938:12)
at Module.require (node:internal/modules/cjs/loader:1115:19)
at require (node:internal/modules/helpers:130:18)
at Object. @.\core\src\config\helpers\config-api.ts:4:1)
at Module._compile (node:internal/modules/cjs/loader:1241:14)
Steps to reproduce:

Clone the repository to local machine
Run npm install and npm update from within repository
Run npm run dev

—
Reply to this email directly, view it on GitHub
https://github.com/ivelascog/TimeSearcherPlus/issues/3, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJJCSYGOE7S5XARPOFFRMDX26DZJANCNFSM6AAAAAA4354D7I
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Usually you don't need to have to run npm update John Alexis Guerra Gómez On my phone, brevity and all
…
On Sun, Sep 17, 2023, 4:47 PM Aparna Krishnan @.> wrote: When attempting to execute the custom npm run dev script, I encounter a TypeError related to redefining a property named File. The error traces back to the @babel/core module. I have not made any direct modifications to this module, and the issue presents itself specifically during the execution of the aforementioned script. Running npm update first used to solve this issue in a previous version of this repository, but it no longer does. Error message: [!] TypeError: Cannot redefine property: File at Function.defineProperty () at Object. @.\core\lib\index.js:7:8) at Module._compile (node:internal/modules/cjs/loader:1241:14) at Object.Module._extensions..js (node:internal/modules/cjs/loader:1295:10) at Module.load (node:internal/modules/cjs/loader:1091:32) at Function.Module._load (node:internal/modules/cjs/loader:938:12) at Module.require (node:internal/modules/cjs/loader:1115:19) at require (node:internal/modules/helpers:130:18) at Object. @.\core\src\config\helpers\config-api.ts:4:1) at Module._compile (node:internal/modules/cjs/loader:1241:14) Steps to reproduce: 1. Clone the repository to local machine 2. Run npm install and npm update from within repository 3. Run npm run dev — Reply to this email directly, view it on GitHub <#3>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJJCSYGOE7S5XARPOFFRMDX26DZJANCNFSM6AAAAAA4354D7I . You are receiving this because you are subscribed to this thread.Message ID: @.>

Yes, but running npm update was cited as one of the possible fixes to resolve this issue. It doesn't work now anyway, so I think we need to find other ways of resolving this issue.


"AssertionError: set_torch_image input must be BCHW with long side 1024. "

Hi
I have been working with SAM, but I have this problem:
"AssertionError: set_torch_image input must be BCHW with long side 1024. "
The problem occurred when I run the "generated" method :
sam_mask.generate(image)
Can I work with different image sizes?

I was running into this and the problem is a misleading error message, the model throws that error if you pass an image with an alpha channel. It needs to be converted to RGB first.

https://github.com/ByungKwanLee/Full-Segment-Anything addresses the ciritical issues of SAM, which supports batch-input on the full-grid prompt (automatic mask generation) with post-processing: removing duplicated or small regions and holes, under flexible input image size

I was running into this and the problem is a misleading error message, the model throws that error if you pass an image with an alpha channel. It needs to be converted to RGB first.

Thank you :)
Was putting a Comfy workflow together that uses the stability API nodes with the SD3 API and figured I'd add InstantID and a Face Detailer before the custom upscaler node and the FD kept throwing up this error. Saw your reply and added a convert to RGB node to feed into the FD and that fixed it :)


Update commons-compress dependency

Motivation:
We should use the latest commons-compress release to fix CVE-2019-12402
Modifications:
Update commons-compress to 1.19
Result:
Fix OWASP Dependency-Check security alert

Coverage remained the same at 73.843% when pulling 607b303956f20def5671ea0761f3120d760a7352 on johnou:patch-1 into f2f38b924b0ab8743008c102470e21bc39dcbf65 on AlejandroRivera:master.

@AlejandroRivera ptal

Thanks @johnou!


Create dockerfile

Closes #88

A very dumb question: the point of adding this dockerfile is to be able to build the docs locally without having to install all the dependencies, right?

Indeed.

A new commit has been added that enables hot reload

I usually find it a good practice to add at the end of the Dockerfile a couple of comments explaining how to build and run an image, especially when custom arguments are required

Good idea. I added the following lines to the end of the file:
# Build:
# $ docker build -t simphony-docs .

# Run:
# $ docker run -v $PWD:/app -p 5500:5500 simphony-docs


Fixing typo in README.md

I tried to use this template today and noticed that there was a typo in the cli commands. With this fix people can just copy and paste.

Thanks a ton!


SideMenu Warning: menuLeftNavigationController cannot be modified while it's presented. Happening in storyboard integration

New Issue Checklist

I have read the guidelines for contributing and I understand:

[x] My issue is happening in the latest version of SideMenu.
[x] My issue was not solved in the README.
[x] My issue can not be answered on stackoverflow.com.
[x] My issue is not a request for new functionality that I am unwilling to build and contribute with a pull request.
[x] My issue is reproducible in the demo project.

Issue Description

First of all thanks for giving me a wonderful library.
I integrated SideMenu via storyboard. And everything works fine unless I logout and come back to my HomeVC having SideMenu.
My flow is like this LoginVC(This doesn't have SideMenu) -> HomeVC(with SideMenu).
Problem is when I go back to login screen -> login and I set HomeVC as rootvc to UINavigationController.
I am getting a log message "SideMenu Warning: menuLeftNavigationController cannot be modified while it's presented." and presenting the sidemenu in fullscreen instead of part of a screen width.
This will not happen if I relaunch the app. I think I am showing the sidemenu again and again.
Is there anyway to remove the sidemenu from my view hierarchy when I do logout? Or is there any other workaround?

SideMenu is a view controller like any other view controller, and as such you cannot remove it from the hierarchy while something is presented over top of it. Consider reworking your hierarchy to a flow supports your goals.

Sorry. It was my mistake. I should have dismissed the sidemenu before presenting the login vc.
dismiss(animated: true, completion: {
//code to logout
}
fixed my issue.

Hi, I have same problem, but it couldn't solve @jeslinj @jonkykong . I'm using Xcode 10 and SlideMenu 5.0.1.


Nicer formatting

NIcer formatting for the list of aliases

@mcornella done. sorry about that

Oh bummer, it seems the ember-cli plugin was updated 3 months ago, can you rebase your PR? Do you know how to do it?

yeah I'll rebase in a bit.
On Thu, Sep 3, 2015 at 10:32 AM Marc Cornellà<EMAIL_ADDRESS>wrote:

Oh bummer, it seems the ember-cli plugin was updated 3 months ago
https://github.com/robbyrussell/oh-my-zsh/pull/3989, can you rebase
your PR? Do you know how to do it?
—
Reply to this email directly or view it on GitHub
https://github.com/robbyrussell/oh-my-zsh/pull/4139#issuecomment-137466762
.

@mcornella sorry this took me so long to get back to. Rebase was being super annoying so I just used merge --no-ff. Let me know if this works.

Sorry, lost track of this. :+1:


refactor: Specify generic type constraints for Tree class

Please make sure these boxes are checked before submitting your PR, thank you!

[x] Make sure you follow Element's contributing guide (中文 | English | Español | Français).
[x] Make sure you are merging your commits to dev branch.
[x] Add some descriptions and refer relative issues for you PR.

error TS2344: Type 'D' does not satisfy the constraint 'TreeData'.
This type parameter might need an extends TreeData constraint.

一样的问题，我也遇到了，问什么pr不给合并呢


Add missing images

Changes proposed by this PR

closes #807

Release Note

PR Checklist
Note: Please do not remove items. Mark items as done [x] or use ~strikethrough~ if you believe they are not relevant

[ ] Linked to a relevant issue. Eg: Fixes #123 or Updates #123
[ ] Removed non-atomic or wip commits
[ ] Filled in the Release Note section above
[ ] Modified the docs to match changes

Any suggestions on where images should live?


feat: Added Docker support for local development 🐳

Containerized development environment using Docker and docker-compose. Instructions have also been updated to match new development flow using Docker 🐳 .
Changelog:

Created Dockerfile, docker-compose.yml, and .dockerignore and added containerization functionality
Updated README to include new development instructions using docker and update UI mockup image

@DhyeyLalseta I'd love to hear from you how my Docker is implemented here since this is my first time working with containers 🐳

i think it's perfect 🐳, it might be worth it to setup devcontainer.json too if it's for dev


Leak in SystemJobService

Hi everyone,
we're using WorkManager in an app in which we've also LeakCanary configured and we always receive this warning after a run of each Periodic Worker:
	16:49:08.888	LeakCanary
====================================
HEAP ANALYSIS RESULT
====================================
1 APPLICATION LEAKS

References underlined with "~~~" are likely causes.
Learn more at https://squ.re/leaks.

774 bytes retained by leaking objects
Signature: 6491376cfe66b1cac0d47b3ae180fd902719aa5
┬───
│ GC Root: Global variable in native code
│
├─ android.app.job.JobService$1 instance
│    Leaking: UNKNOWN
│    Retaining 802 B in 15 objects
│    Anonymous subclass of android.app.job.IJobService$Stub
│    this$0 instance of androidx.work.impl.background.systemjob.SystemJobService
│    ↓ JobService$1.this$0
│                   ~~~~~~
╰→ androidx.work.impl.background.systemjob.SystemJobService instance
     Leaking: YES (ObjectWatcher was watching this because androidx.work.impl.background.systemjob.SystemJobService
     received Service#onDestroy() callback)
     Retaining 774 B in 14 objects
     key = 155d4c85-1073-4df1-bd0f-250b40c959c8
     watchDurationMillis = 5248
     retainedDurationMillis = 246
     mApplication instance of xx.yy.MyApp
     mBase instance of android.app.ContextImpl
====================================
0 LIBRARY LEAKS

A Library Leak is a leak caused by a known bug in 3rd party code that you do not have control over.
See https://square.github.io/leakcanary/fundamentals-how-leakcanary-works/#4-categorizing-leaks
====================================
0 UNREACHABLE OBJECTS

An unreachable object is still in memory but LeakCanary could not find a strong reference path
from GC roots.
====================================
METADATA

Please include this in bug reports and Stack Overflow questions.

Build.VERSION.SDK_INT: 23
Build.MANUFACTURER: alps
LeakCanary version: 2.6
App process name: xx.yy.MyApp
Stats: LruCache[maxSize=3000,hits=101,misses=300675,hitRate=0%]
RandomAccess[bytes=11383030,reads=300675,travel=696095628841,range=23369842,size=28770980]
Heap dump reason: 1 retained objects, app is not visible
Analysis duration: 202141 ms
Heap dump file path: /storage/emulated/0/Download/leakcanary-xx.yy.MyApp/2021-03-08_16-45-32_740.
hprof
Heap dump timestamp:<PHONE_NUMBER>838
Heap dump duration: 13889 ms
====================================

WorkManager library in use: androidx.work:work-rxjava2:2.5.0
LeakCanary library in use: com.squareup.leakcanary:leakcanary-android:2.6
Any advice? Did you see this warning yet?
I don't know if can be a useful info, but the app we're developing is a system app used on embedded device (Android based, obviously)

@rickytribbia I don't believe we've seen any similar issues. Are you able to reproduce the behavior in a non-systems app running on a Android phone (not an embedded device)? If not, I suspect the problem is not with the workmanager library.

Hi Calren, thank you for your answer.
I don't know if you're experienced with LeakCanary; its heap analysis result print any dependencies between an instance of a retained object and the GC Root.
In the log that I printed before you can see that there aren't any classes of my app between the retained object and the root, so I have no entrypoint to investigate deep to this leak.
What do you think about it? Where could I check if my code retains something about workmanager instances (androidx.work.impl.background.systemjob.SystemJobService, I think)?

If possible, can you try installing your app on an Android phone and see if the same leak canary warnings are surfaced?
I suspect the leak might be coming from the embedded device and not the WorkManager library itself, so it'd be good to check if leak canary still reports the same warnings on devices not running a modified version of Android.

Hi Calren,
we've tried to install our app on an Android phone: the same leak canary warnings was nevers surfaced, but the scenario is really different (our app acts as a system app, so have extended permissions), so our app in any Job runned by WorkManager always returns errors and the Job/Worker fails instantly.
Have you any advice? Another possible clue could be that we use Dagger for dependency injection also into our Worker?

I also has this issue. Regarding the statement by @calren

I suspect the leak might be coming from the embedded device

Uhm, leak root cause should not depend on a device and its never been that way?

This is what I get sometimes
04-28 10:58:52.125 13357-13357/com.myapp.name D/LeakCanary:
┬───
│ GC Root: Global variable in native code
│
├─ android.app.job.JobService$1 instance
│    Leaking: UNKNOWN
│    Retaining 602 B in 13 objects
│    Anonymous subclass of android.app.job.IJobService$Stub
│    this$0 instance of com.google.android.datatransport.runtime.scheduling.jobscheduling.JobInfoSchedulerService
│    ↓ JobService$1.this$0
│                   ~~~~~~
╰→ com.google.android.datatransport.runtime.scheduling.jobscheduling.JobInfoSchedulerService instance
     Leaking: YES (ObjectWatcher was watching this because com.google.android.datatransport.runtime.scheduling.
     jobscheduling.JobInfoSchedulerService received Service#onDestroy() callback and Service not held by
     ActivityThread)
     Retaining 574 B in 12 objects
     key = 54f18ba8-4533-4b95-8def-6b19b6e79481
     watchDurationMillis = 5260
     retainedDurationMillis = 254
     mApplication instance of com.myapp.name.App
     mBase instance of android.app.ContextImpl
METADATA

Build.VERSION.SDK_INT: 21
Build.MANUFACTURER: Xiaomi
LeakCanary version: 2.7
App process name: com.myapp.name
Stats: LruCache[maxSize=3000,hits=84,misses=122288,hitRate=0%]
RandomAccess[bytes=11522584,reads=122288,travel=37581525864,range=42112035,size=55145753]
Heap dump reason: 1 retained objects, app is not visible
Analysis duration: 32830 ms

The same thing happens to me in my application
06-29 19:26:52.970 19354-19354 D:
┬───
│ GC Root: Global variable in native code
│
├─ android.app.job.JobService$1 instance
│    Leaking: UNKNOWN
│    Retaining 792 B in 13 objects
│    Anonymous subclass of android.app.job.IJobService$Stub
│    this$0 instance of com.google.android.datatransport.runtime.scheduling.jobscheduling.JobInfoSchedulerService
│    ↓ JobService$1.this$0
│                   ~~~~~~
╰→ com.google.android.datatransport.runtime.scheduling.jobscheduling.JobInfoSchedulerService instance
     Leaking: YES (ObjectWatcher was watching this because com.google.android.datatransport.runtime.scheduling.
     jobscheduling.JobInfoSchedulerService received Service#onDestroy() callback and Service not held by
     ActivityThread)
     Retaining 764 B in 12 objects
     key = 014c7056-d53e-4753-98f2-d0dbe18498f5
     watchDurationMillis = 27619
     retainedDurationMillis = 22586
     mApplication instance of com.bastechsolutions.swiftcountandroid.AppDelegate
     mBase instance of android.app.ContextImpl
METADATA

Build.VERSION.SDK_INT: 23
Build.MANUFACTURER: BLU
LeakCanary version: 2.7
App process name: com.bastechsolutions.swiftcountandroid
Stats: LruCache[maxSize=3000,hits=376,misses=93793,hitRate=0%]
RandomAccess[bytes=7967352,reads=93793,travel=137736304775,range=41487438,size=62128446]
Heap dump reason: user request
Analysis duration: 834634 ms

This only happens to me on a specific cell phone that I use to optimize my applications, I think the error comes out because the device has very low performance.
The device in which it happens to me is a Blue Dash G and has the following characteristics:
Screen: 5 ", 480 x 854 pixels
Processor: Mediatek MT6580 1.3GHz
RAM: 512MB
Storage: 4GB
Expansion: microSD

Camera: 5 MP
Battery: 2000 mAh
OS: Android 6.0
Profile: 9.8 mm
Weight: 157g

Here is a dump of my reproduction of the issue as well (as this is still happening):
====================================
HEAP ANALYSIS RESULT
====================================
1 APPLICATION LEAKS

References underlined with "~~~" are likely causes.
Learn more at https://squ.re/leaks.

2243 bytes retained by leaking objects
Signature: 953feb82863ec1b7cb6fcf77def88fd9aec7bd
┬───
│ GC Root: Java local variable
│
├─ java.lang.Thread thread
│ Leaking: UNKNOWN
│ Retaining 47.7 kB in 380 objects
│ Thread name: 'pool-10-thread-1'
│ ↓ Thread.<Java Local>
│ ~~~~~~~~~~~~
├─ com.google.android.datatransport.runtime.scheduling.jobscheduling.
│ JobInfoSchedulerService$$ExternalSyntheticLambda0 instance
│ Leaking: UNKNOWN
│ Retaining 3.8 kB in 32 objects
│ f$0 instance of com.google.android.datatransport.runtime.scheduling.
│ jobscheduling.JobInfoSchedulerService
│ ↓ JobInfoSchedulerService$$ExternalSyntheticLambda0.f$0
│ ~~~
╰→ com.google.android.datatransport.runtime.scheduling.jobscheduling.
 JobInfoSchedulerService instance
 Leaking: YES (ObjectWatcher was watching this because com.google.android.
 datatransport.runtime.scheduling.jobscheduling.JobInfoSchedulerService
 received Service#onDestroy() callback and Service not held by
 ActivityThread)
 Retaining 2.2 kB in 17 objects
 key = 814cc92e-4087-400f-9ab2-7bc5d971018a
 watchDurationMillis = 5599
 retainedDurationMillis = 556
 mApplication instance of com.mycompany.android.App
 mBase instance of android.app.ContextImpl
====================================
0 LIBRARY LEAKS

A Library Leak is a leak caused by a known bug in 3rd party code that you do
not have control over.
See https://square.github.
io/leakcanary/fundamentals-how-leakcanary-works/#4-categorizing-leaks
====================================
0 UNREACHABLE OBJECTS

An unreachable object is still in memory but LeakCanary could not find a strong
reference path
from GC roots.
====================================
METADATA

Please include this in bug reports and Stack Overflow questions.

Build.VERSION.SDK_INT: 31
Build.MANUFACTURER: Google
LeakCanary version: 2.7
App process name: com.mycompany.android
Stats: LruCache[maxSize=3000,hits=4454,misses=259618,hitRate=1%]
RandomAccess[bytes=13015376,reads=259618,travel=184478677083,range=60464600,size
=70558998]
Heap dump reason: 1 retained objects, app is not visible
Analysis duration: 17591 ms
Heap dump file path: /storage/emulated/0/Download/leakcanary-co.mycompany.
android/2022-01-04_10-48-37_439.hprof
Heap dump timestamp:<PHONE_NUMBER>281
Heap dump duration: 11154 ms
====================================

The only relevant details that I can discern is this error message:
ObjectWatcher was watching this because com.google.android.
 datatransport.runtime.scheduling.jobscheduling.JobInfoSchedulerService
 received Service#onDestroy() callback and Service not held by
 ActivityThread

I can reproduce the same issue on multiple device,  I received this warning after run of each Periodic Worker or Enqueued Jobservice, any suggestion for the issue?
com.google.android.datatransport.runtime.scheduling.jobscheduling.JobInfoSchedulerService instance
     Leaking: YES (ObjectWatcher was watching this because com.google.android.datatransport.runtime.scheduling.
     jobscheduling.JobInfoSchedulerService received Service#onDestroy() callback and Service not held by
     ActivityThread)

749 bytes retained by leaking objects
Signature: 99e784c1433b5477fe40e3ebca5ffce0e9dd16
┬───
│ GC Root: System class
│
├─ android.content.res.ResourcesImpl class
│    Leaking: NO (a class is never leaking)
│    ↓ static ResourcesImpl.mAppContext
│                           ~~~~~~~~~~~
├─ android.app.ContextImpl instance
│    Leaking: UNKNOWN
│    Retaining 2.0 kB in 13 objects
│    mOuterContext instance of com.google.android.datatransport.runtime.scheduling.jobscheduling.JobInfoSchedulerService
│    ContextImpl.mOuterContext is an instance of com.google.android.datatransport.runtime.scheduling.jobscheduling.
│    JobInfoSchedulerService
│    ↓ ContextImpl.mOuterContext
│                  ~~~~~~~~~~~~~
╰→ com.google.android.datatransport.runtime.scheduling.jobscheduling.JobInfoSchedulerService instance
     Leaking: YES (ObjectWatcher was watching this because com.google.android.datatransport.runtime.scheduling.
     jobscheduling.JobInfoSchedulerService received Service#onDestroy() callback and Service not held by
     ActivityThread)
     Retaining 749 B in 6 objects
     key = 4aabceb8-2d2b-4070-b145-d412c58e3bca
     watchDurationMillis = 135034
     retainedDurationMillis = 130029
     mApplication instance of com.sundarban.deliveryman.app.BaseApplication
     mBase instance of android.app.ContextImpl
====================================
0 LIBRARY LEAKS

A Library Leak is a leak caused by a known bug in 3rd party code that you do not have control over.
See https://square.github.io/leakcanary/fundamentals-how-leakcanary-works/#4-categorizing-leaks
====================================
0 UNREACHABLE OBJECTS

An unreachable object is still in memory but LeakCanary could not find a strong reference path
from GC roots.
====================================
METADATA

Please include this in bug reports and Stack Overflow questions.

Build.VERSION.SDK_INT: 30
Build.MANUFACTURER: Xiaomi
LeakCanary version: 2.7
App process name: com.sundarban.deliveryman.debug
Stats: LruCache[maxSize=3000,hits=3378,misses=88914,hitRate=3%]
RandomAccess[bytes=4167110,reads=88914,travel=43953220991,range=25986646,size=33102335]
Heap dump reason: user request
Analysis duration: 6558 ms
Heap dump file path: /storage/emulated/0/Download/leakcanary-com.sundarban.deliveryman.debug/2022-03-09_12-24-58_395.
hprof
Heap dump timestamp:<PHONE_NUMBER>270
Heap dump duration: 2847 ms

Any solutions to @kazimdsaidul leak?

I had tested workmanager on https://github.com/RachmadAgungL/Tugas6-1918070-RachmadAgungL/ and the same issue occurred.

Same here, is there anyone able to figure out the issue?
    ┬───
    │ GC Root: System class
    │
    ├─ android.content.res.ResourcesImpl class
    │    Leaking: NO (a class is never leaking)
    │    ↓ static ResourcesImpl.mAppContext
    │                           ~~~~~~~~~~~
    ├─ android.app.ContextImpl instance
    │    Leaking: UNKNOWN
    │    Retaining 2.0 kB in 13 objects
    │    mOuterContext instance of com.google.android.datatransport.runtime.scheduling.jobscheduling.JobInfoSchedulerService
    │    ContextImpl.mOuterContext is an instance of com.google.android.datatransport.runtime.scheduling.jobscheduling.
    │    JobInfoSchedulerService
    │    ↓ ContextImpl.mOuterContext
    │                  ~~~~~~~~~~~~~
    ╰→ com.google.android.datatransport.runtime.scheduling.jobscheduling.JobInfoSchedulerService instance
         Leaking: YES (ObjectWatcher was watching this because com.google.android.datatransport.runtime.scheduling.
         jobscheduling.JobInfoSchedulerService received Service#onDestroy() callback and Service not held by
         ActivityThread)
         Retaining 749 B in 6 objects
         key = 106b88b8-9fc8-4bb1-bc67-f720d79fdf2f
         watchDurationMillis = 10041
         retainedDurationMillis = 5040
         mBase instance of android.app.ContextImpl
    ====================================
    0 LIBRARY LEAKS

    A Library Leak is a leak caused by a known bug in 3rd party code that you do not have control over.
    See https://square.github.io/leakcanary/fundamentals-how-leakcanary-works/#4-categorizing-leaks
    ====================================
    0 UNREACHABLE OBJECTS

    An unreachable object is still in memory but LeakCanary could not find a strong reference path
    from GC roots.
    ====================================
    METADATA

    Please include this in bug reports and Stack Overflow questions.

    Build.VERSION.SDK_INT: 30
    Build.MANUFACTURER: Xiaomi
    LeakCanary version: 2.7
    Stats: LruCache[maxSize=3000,hits=3381,misses=202478,hitRate=1%]
    RandomAccess[bytes=10806583,reads=202478,travel=110618093912,range=40792679,size=51604394]
    Heap dump reason: user request
    Analysis duration: 15401 ms
    Heap dump file path: /storage/emulated/0/Download/leakcanary-com.ps.bt.development/2022-07-08_09-15-26_828.hprof
    Heap dump timestamp:<PHONE_NUMBER>237
    Heap dump duration: 3515 ms
    ====================================

any solution here ?

I am facing the same issue, any solution for this ?
│ GC Root: System class
│
├─ android.content.res.ResourcesImpl class
│    Leaking: NO (a class is never leaking)
│    ↓ static ResourcesImpl.mAppContext
│                           ~~~~~~~~~~~
├─ android.app.ContextImpl instance
│    Leaking: UNKNOWN
│    Retaining 1.8 kB in 18 objects
│    mOuterContext instance of androidx.work.impl.background.systemjob.SystemJobService
│    ContextImpl.mOuterContext is an instance of androidx.work.impl.background.systemjob.SystemJobService
│    ↓ ContextImpl.mOuterContext
│                  ~~~~~~~~~~~~~
╰→ androidx.work.impl.background.systemjob.SystemJobService instance
     Leaking: YES (ObjectWatcher was watching this because androidx.work.impl.background.systemjob.SystemJobService received Service#onDestroy() callback and Service not held by ActivityThread)
     Retaining 276 B in 6 objects
     key = 15e32ff1-396b-46dd-a442-0eccd6afb231
     watchDurationMillis = 6375
     retainedDurationMillis = 1374
     mBase instance of android.app.ContextImpl`

I am also facing this issue of leaks in work manager
In my project : I have used  Workmanager (DI using HILT) with periodic work request by using DelegatingWorkerFactory to give a local scheduled notification every (One hour / Specific Time >= 30 minute).
┬───
│ GC Root: System class
│
├─ android.content.res.ResourcesImpl class
│    Leaking: NO (a class is never leaking)
│    ↓ static ResourcesImpl.mAppContext
│                                     ~~~~~~~~~~~
├─ android.app.ContextImpl instance
│    Leaking: UNKNOWN
│    Retaining 2.4 kB in 47 objects
│    mOuterContext instance of androidx.work.impl.foreground.
│    SystemForegroundService
│    ContextImpl.mOuterContext is an instance of androidx.work.impl.foreground.
│    SystemForegroundService
│    ↓ ContextImpl.mOuterContext
│                        ~~~~~~~~~~~~~
╰→ androidx.work.impl.foreground.SystemForegroundService instance
     Leaking: YES (ObjectWatcher was watching this because androidx.work.impl.
     foreground.SystemForegroundService received Service#onDestroy() callback
     and Service not held by ActivityThread)
     Retaining 1.1 kB in 40 objects
     key = 6584f839-414d-4ef8-9db4-c66949b01695
     watchDurationMillis = 53032
     retainedDurationMillis = 48027
     mApplication instance of APP_PACKAGE_NAME.base.
     BaseApplication
     mBase instance of android.app.ContextImpl
METADATA
Build.VERSION.SDK_INT: 30
Build.MANUFACTURER: realme
LeakCanary version: 2.8.1
App process name: APP_PACKAGE_NAME
Stats: LruCache[maxSize=3000,hits=36362,misses=78944,hitRate=31%]
RandomAccess[bytes=3878630,reads=78944,travel=36641054393,range=22347685,size=28
595458]
Analysis duration: 6089 ms

┬───
│ GC Root: System class
│
├─ android.content.res.ResourcesImpl class
│    Leaking: NO (a class is never leaking)
│    ↓ static ResourcesImpl.mAppContext
│                           ~~~~~~~~~~~
├─ android.app.ContextImpl instance
│    Leaking: UNKNOWN
│    Retaining 2.5 kB in 20 objects
│    mOuterContext instance of androidx.work.impl.background.systemjob.SystemJobService
│    ContextImpl.mOuterContext is an instance of androidx.work.impl.background.systemjob.SystemJobService
│    ↓ ContextImpl.mOuterContext
│                  ~~~~~~~~~~~~~
╰→ androidx.work.impl.background.systemjob.SystemJobService instance
     Leaking: YES (ObjectWatcher was watching this because androidx.work.impl.background.systemjob.SystemJobService
     received Service#onDestroy() callback and Service not held by ActivityThread)
     Retaining 852 B in 9 objects
     key = acfae287-9f8e-481d-b257-8d5e348d39ff
     watchDurationMillis = 34718
     retainedDurationMillis = 29715
     mApplication instance of com.android.dialer.binary.aosp.AospDialerApplication
     mBase instance of android.app.ContextImpl
leakcanary-android:2.10 androidx.work:work-runtime:2.7.1

any solution here ?

Hi, I'm facing the same issue in our Android app. Here is the heap analysis result from the leak canary
`
====================================
HEAP ANALYSIS RESULT
====================================
1 APPLICATION LEAKS
                                                                                References underlined with "~~~" are likely causes.
                                                                                Learn more at https://squ.re/leaks.

                                                                                852 bytes retained by leaking objects
                                                                                Signature: 099e7804c1433b5477fe40e3ebca5ffce0e9dd16
                                                                                ┬───
                                                                                │ GC Root: System class
                                                                                │
                                                                                ├─ android.content.res.ResourcesImpl class
                                                                                │    Leaking: NO (a class is never leaking)
                                                                                │    ↓ static ResourcesImpl.mAppContext
                                                                                │                           ~~~~~~~~~~~
                                                                                ├─ android.app.ContextImpl instance
                                                                                │    Leaking: UNKNOWN
                                                                                │    Retaining 2.5 kB in 20 objects
                                                                                │    mOuterContext instance of androidx.work.impl.background.systemjob.SystemJobService
                                                                                │    ContextImpl.mOuterContext is an instance of androidx.work.impl.background.systemjob.SystemJobService
                                                                                │    ↓ ContextImpl.mOuterContext
                                                                                │                  ~~~~~~~~~~~~~
                                                                                ╰→ androidx.work.impl.background.systemjob.SystemJobService instance
                                                                                Leaking: YES (ObjectWatcher was watching this because androidx.work.impl.background.systemjob.SystemJobService
                                                                                received Service#onDestroy() callback and Service not held by ActivityThread)
                                                                                Retaining 852 B in 9 objects
                                                                                key = cd8968e6-3a01-4ac9-b78a-4f6c3f2f2c21
                                                                                watchDurationMillis = 71115
                                                                                retainedDurationMillis = 66114
                                                                                mApplication instance of de.akelius.university.mobile.languageapplication.LanguageApplication
                                                                                mBase instance of android.app.ContextImpl
                                                                                ====================================`


Define core Azure services resource Uri (audience) in SDK

Feature Request
It would be convenient if we define common core Azure services resource Uri (audience) in SDK. This is related to issue #697, that the service endpoint may not exact match the resource Uri when acquiring access tokens. While #697 fixed the issue, it requires client to make one unauthenticated request first and parse the resource Uri from response header. This is additional overhead some times we want to avoid.

This has been resolved in v11.9.0 of go-autorest.


Is ngx-translate dependent on zone.js?

I am trying to create an Angular SPA project without zone.js to make the spa footprint as small as possible. Bumping into some issues where the translated values don't show up after cdr.markForCheck() is called on the component.
Should this be an issue or is there something else wrong on my end?
Any advice is appreciated.

As much as I understand, markForCheck only works if something calls tick on the appRef.
If there is no zone, then either you have to do it yourself, or you should use detectChanges instead.


virtualScroll has a BUG

When virtualScroll is enabled, drag the scroll bar to the bottom with the mouse, a blank page appears in the drop-down box, and currentChunk becomes 0 instead of the last one.

What version? Which browser? If this is Firefox only, it sounds like the same as #2524.

Sorry, just now saw your comment. It happened in Chrome.
Environment:
windows: 10
Chrome:  87.0.4280.88
bootstrap-table: 1.13.18

Yes, this happens in Chrome. I created "#2524"
https://github.com/snapappointments/bootstrap-select/issues/2554#:~:text=as-,%232524,-.

Yes, this happens in Chrome. I created "#2524"
https://github.com/snapappointments/bootstrap-select/issues/2554#:~:text=as-,%232524,-.

Closing in favor of #2524.

Closing in favor of #2524.


Invalid coding

Certificate for SSL can not be applied as all the reference links are coded with http and no coding for https - any workarounds available ?

If you installed pH7Builder (pH7CMS) with http, then moved to HTTPS, you will have to clear the caches in order to regenerate the URLs and get them with HTTPS. Go to your Admin Panel, -> Tool -> Caches -> Caches Manager.
FYI, the software automatically checks the protocol (http/https).
This is done is your _constants.php file (around line 23) located in the root folder after the installation of the software has been done.
Let me know if this helps.
Have a great weekend anyway :tada:

Finally, do you mind to Star the repo @marketing720?

	Thank you sir - how do you get the phone app to work or do you have any documentation or a paid service I can use to finish it up?

	Get Outlook for iOS

On Sat, Feb 16, 2019 at 4:25 PM -0800, "♚ PH⑦ de Soria™♛"<EMAIL_ADDRESS>wrote:
Assigned #442 to @marketing720.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub, or mute the thread.

Hi @marketing720
Can you please create a new "Issue" if the problem is different. This helps to keep each thread tiny and avoiding confusions in there. Finally, do you mind to Star the repo?

@marketing720   Finally, do you mind to Star the repo?

I will close this one by now @marketing720. Please let me know if there is any more issues with https (if it's still the case...).
Thanks. Enjoy your day 🎉


Add secrets to the github secret storage

[x] mailgun
[x] zoom
[x] researchseminars
[x] workers (or whichever cloud function provider we use)

For now this doesn't include:

youtube
twitter

Ready to close?

Yep! Captcha etc we'll handle separately.


Update Welcome

Is there any specific reason you updated the dependencies?

No, I thinks it will work better with update and because etch is out of date.


KOGITO-7227 - SWF Viewer - Error Handling

JIRA: KOGITO-7227 - SWF Viewer: Error Handling
The error page is not shown after the canvas is loaded.
Marshaller errors are now caught and properly treated.
Error messages are shown in the editor for proper clarification.

Hello @romartin. Please assign reviewers for this. Thanks.

@Josephblt Looks good the error message were helpful, only thing I notice is that in some cases the error is a.b is undefined
This happens when we have empty states: [] or when there is something required missing like state is missing its name
Is this something that should be part of this PR or is t our of scope? I expected the message to say something like: missing required attribute name for state on line 26


[SPARK-35492][BUILD] Upgrade httpcore to 4.4.14

What changes were proposed in this pull request?
This PR aims to upgrade Apache HttpCore from 4.4.12 to 4.4.14.
Why are the changes needed?
Stability improvements in httpcore 4.4.14

Bug fix: Non-blocking TLSv1.3 connections can end up in an infinite event spin when closed concurrently by the local and the remote endpoints.
HTTPCORE-647: Non-blocking connection terminated due to 'java.io.IOException: Broken pipe' can enter an infinite loop flushing buffered output data.
PR #201, HTTPCORE-634: Fix race condition in AbstractConnPool that can cause internal state
corruption
HTTPCORE-612: DefaultConnectionReuseStrategy incorrectly used int to represent Content-Length value
instead of long

Does this PR introduce any user-facing change?
No
How was this patch tested?
With Jenkins Tests

Test build #138840 has started for PR 32638 at commit b166e41.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43362/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43362/

Jenkins, retest this please

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/43362/

Test build #138845 has started for PR 32638 at commit c700759.

Test build #138840 has finished for PR 32638 at commit b166e41.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43367/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/138840/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43367/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/43367/

Test build #138845 has finished for PR 32638 at commit c700759.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/138845/


View angle plugin buttons are close to the background color, making them hard to differentiate

(Ignition Citadel 3.3.0) I'm having trouble telling which view angle button is which (without hovering over it and reading the tool tip text). I can see the orange face just fine, but the gray edges are very difficult for me to see (could it be my monitor is not calibrated correctly?).

A color picker on the above screenshot shows the background is #fafafa (left half below) while the edge color is #f3f3f3 (right half below)

Just use dark mode :innocent:  I'm kidding, we've been trying to make sure all icons show up on both light and dark modes, but I think we just reused those ones from Gazebo-classic for expediency.
I think we could use a darker grey on the lines, and possibly make them thicker? The icons are here.
Any thoughts, @ColeOSRF?

Hi, Can I make those adjustments to those icons?

@herobank110 , you're welcome to make the changes and open a pull request, thanks!

I have made darkened variants of the icons for preview below. Any darker than 45 had too little contrast with the dark theme background. Let me know what you think is best.

Thanks, @herobank110 ! 0 is hard to see on the light background on my monitor. I'd go with 25.

please see #426


Seeing this: INFO:root:Adjusting name "ACP Policy" to "ACP_Policy" due to containing invalid characters

INFO:root:Adjusting name 'ACP Policy' to 'ACP_Policy' due to invalid characters.
WARNING:root:	GET query for ACP_Policy is not found.
WARNING:root:	GET query for ACP_Policy is not found.
WARNING:root:Access Control Policy ACP_Policy not found.  Cannot set up accessPolicy for AccessRules.
Wondering if there is a solution to this message. I've saw some reference to a previous issue but it didn't show a solution.
Thanks

Never mind, it seems that it is easy just to change the name of the policy


My first Microverce project and also my first mobile and desktop app deployment .

Kindly find the link to my deployed website in my readMe forder.

Hi @eerapheal,
Highlights
✔ Descriptive PR
Good job so far!
There are some issues that you still need to work on to go to the next project but you are almost there!
Required Changes ♻

[ ] You have not provided your demo link in your readme. Please ensure your live link is present in your readme

Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

thanks


Invalid db pull / db push flow

SQL:
CREATE TABLE `cm_linkprw` (
  `ID` int(10) unsigned NOT NULL AUTO_INCREMENT,
  `video` enum('0','1') NOT NULL DEFAULT '0',
  PRIMARY KEY (`ID`)
) ENGINE=InnoDB DEFAULT CHARSET=latin1;

leads to invalid:
enum cm_linkprw_video {
  0 @map("0")
  1 @map("1")
}

so modified to:
enum cm_linkprw_video {
  is_false  @map("0")
  is_true   @map("1")
}

and adapted main model as well.
Problem is that another db pull changes that to this:
model cm_linkprw {
  ID          Int              @id @default(autoincrement()) @db.UnsignedInt
  postId      Int              @db.UnsignedInt
  url         String           @db.VarChar(150)
  video       cm_linkprw_video @default(dbgenerated("0"))
}

which when migrated then leads to:
Datasource "db": MySQL database "pabau" at "localhost:3307"
Error: Invalid default value for 'video'
   0: sql_migration_connector::apply_migration::apply_migration
             at migration-engine\connectors\sql-migration-connector\src\apply_migration.rs:9
   1: migration_core::state::SchemaPush
             at migration-engine\core\src\state.rs:347

error Command failed with exit code 1.

Changing the weird default back to:
video       cm_linkprw_video @default(is_false)

is undone on each db pull

The SQL of the migration is actually only invalid as there are single quotes missing when running db pull. This is valid and migrates correctly:
model cm_linkprw {
  ID    Int              @id @default(autoincrement()) @db.UnsignedInt
  video cm_linkprw_video @default(dbgenerated("'0'"))
}

enum cm_linkprw_video {
  is_false @map("0")
  is_true @map("1")
}

To the fix that was done in this PR (which I read as: "Keep valid renamed enum values as field defaults on Re-Introspection"), I think it is missing the warning message we usually add in these cases. When I run the recent dev for the above example, I get this only:
*** WARNING ***

These enum values were enriched with `@map` information taken from the previous Prisma schema.
- Enum "cm_linkprw_video"
- Enum "cm_linkprw_video"

I suggest we add another warning for the kept is_false value of cm_linkprw.video:
These fields of enum types were enriched with `@default` information taken from the previous Prisma schema:
- Field "cm_linkprw_video.video", value "is_false"

Or something like that.


Bug: getBrowserCultureLang() is not working as expected in all browsers

Current behavior
Using getBrowserCultureLang() returns:

Chrome: de-DE (correct)
Firefox: de (wrong)
Safari: de-de (wrong)

The problem gets worse when using transloco-locale:

translocoLocaleService.setLocale(getBrowserCultureLang())

-> Throws "de isn't a valid locale format"
Expected behavior
Should always return correct format 'de-DE'
Minimal reproduction of the problem with instructions
import { getBrowserCultureLang } from '@ngneat/transloco';

console.log(getBrowserCultureLang());

Environment

- transloco: 2.20.1
- transloco-locale: 1.4.0
- Angular: 12.0.1

@MickL Can you please provide the browser versions that you tested this issue on?

Latest versions:
Safari 14.1.1
Firefox 90.0.1

@dvdvnl Currently no.
You are welcome to open a PR for it 🙂

const navigator = window.navigator;
return navigator.languages ? navigator.languages.find((lang) => lang.indexOf('-') > -1 || lang.indexOf('_') > -1) : navigator.language;

... should yield consistent results across browsers. Well, at least Chrome/Firefow/Safari

After @rraziel opened the PR, I started digging deeper into the issue.
The main problem isn't browser consistency— all the values emitted by the browsers are valid, as they follow BCP 47. Both Safari and Chrome produce the same values, which makes sense because according to this section of BCP 47:

At all times, language tags and their subtags, including private use and extensions, are to be treated as case insensitive.

In other words, everything is case insensitive.
Firefox is also valid, as this section clarifies:

A language tag is composed of a sequence of one or more "subtags."

One subtag (the primary language in this case) is valid and is listed in the appendix.
This brings me to a few key points:

isLocaleFormat is incorrectly implemented because it doesn't actually support BCP 47 formats (the root cause of the issue).
getBrowserCultureLang was never intended to return a unified string. It might be worth considering deprecating this function, as it only returns window.navigator.language (window.navigator.languages[0] is redundant, since they are the same). The one advantage of this function is its consideration of SSR by returning an empty string, which could save some code for Transloco users.

To resolve this issue, I'll update the BCP 47 validator to correctly validate the format.

After @rraziel opened the PR, I started digging deeper into the issue.
The main problem isn't browser consistency— all the values emitted by the browsers are valid, as they follow BCP 47. Both Safari and Chrome produce the same values, which makes sense because according to this section of BCP 47:

At all times, language tags and their subtags, including private use and extensions, are to be treated as case insensitive.

In other words, everything is case insensitive.
Firefox is also valid, as this section clarifies:

A language tag is composed of a sequence of one or more "subtags."

One subtag (the primary language in this case) is valid and is listed in the appendix.
This brings me to a few key points:

isLocaleFormat is incorrectly implemented because it doesn't actually support BCP 47 formats (the root cause of the issue).
getBrowserCultureLang was never intended to return a unified string. It might be worth considering deprecating this function, as it only returns window.navigator.language (window.navigator.languages[0] is redundant, since they are the same). The one advantage of this function is its consideration of SSR by returning an empty string, which could save some code for Transloco users.

To resolve this issue, I'll update the BCP 47 validator to correctly validate the format.
Feel free to share thoughts, suggest a different solution or give another perspective.


hey everyboby! i want to using TDoA algrithm for indoor tracking system. But i have problem about SYNC anchors and a TDoA library for arduino.

Step 1: What do you want?

Report an issue.
Improve code (and need help).
Suggest a new feature.
Present a new hardware adapter board or wiring.
Want to report my experience or measurement. (Please also add them to the wiki!)

Step 2: Describe your environment

Arduino IDE version or Toolchain: _____
dw1000ng Library version: _____
Arduino device: _____
Used adapter board: _____

Step 3: Describe the problem

Affected file(s) or example(s):

Filename 1
Filename 2

Steps to reproduce (for bugs only):

Observed Results:

What happened?  This could be a description, log output, etc.

Expected Results:

What did you expect to happen?

Relevant Code:
  // TODO(you): code here to reproduce the problem

You need custom hardware with dw1000 you can't use dwm1000. For more specific details refer to the official chip documentation files.


chore: merge v1.14 stable back to unstable

As part of release.md, we must merge stable back into unstable.

:tada: This PR is included in v1.15.0 :tada:


Add Rector

Codecov Report

Merging #54 (951a52d) into main (13b660f) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##               main      #54   +/-   ##
=========================================
  Coverage     97.83%   97.83%
  Complexity      371      371
=========================================
  Files            23       23
  Lines           972      972
=========================================
  Hits            951      951
  Misses           21       21

Impacted Files
Coverage Δ

src/Report/Report.php
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


elasticsearch.exceptions.SSLError

Hi!
Even without forcing verifying the certificates (and even with a valid certificate) I'm getting this error when connecting:
elasticsearch.exceptions.SSLError: ConnectionError([SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: self signed certificate in certificate chain (_ssl.c:1056)) caused by: SSLError([SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: self signed certificate in certificate chain (_ssl.c:1056))

Also, even though specifying that the protocol in the es_host parameter is https, when initializing, the scheme parameter in the es_conn JSON says http (I don't know if this has something to do with it):
{'index': '[REDACTED]', 'es_host': ('https://[REDACTED],), 'http_auth': '[REDACTED]', 'bulk_size': 500, 'verify_certs': False, 'use_ssl': True, 'ca_certs': None, 'delete': False, 'update': False, 'progress': False, 'type': '_doc', 'id_field': None, 'as_child': False, 'with_retry': False, 'index_settings_file': None, 'timeout': 10.0, 'encoding': 'utf-8', 'keys': [], 'es_conn': <Elasticsearch([{'host': '[REDACTED]', 'port': [REDACTED], 'use_ssl': True, 'scheme': 'http'}])>}

It looks like it is a well known problem with the library that elasticsearch_loader is using: https://stackoverflow.com/questions/54454126/set-verify-certs-false-yet-elasticsearch-elasticsearch-throws-ssl-error-for-cert
Cheers!

Can you paste here the exact command you are running? including esl and es version?

I haven't been able to replicate now... I've tried with the same host and pip3 install elasticsearch_loader with no issues now.
I'll reopen this issue if I see it again! Sorry!


DBZ-4078 Checking connection is valid before commit in readChunk()

https://issues.redhat.com/browse/DBZ-4078
Issue:
At-times, while executing commit during read-only incremental results in com.mysql.cj.jdbc.exceptions.CommunicationsException.
Solution
Seems like Connection object is not valid when the issue occurs. So calling  preReadChunk() before commit, which checks whether connection is valid or not.

@gunnarmorling We should.  I believe originally the implementations in the super-class were empty and I didn't see the need to call the empty method.  I've logged a follow-up issue DBZ-4085 to take care of this.

Oh I must have not been looking at master and you fixed this already.  I'll remove the issue, my fault @gunnarmorling .

Yeah, quickly took care of that one myself.


"Value cannot be NULL" error when using SOAP as connection type

All fields are filled in, but a click on "Test" results in a "Value cannot be null" error:

As an FYI, SOAP connections are intended for testing WSDL-based SOAP endpoints (e.g. much like SoapUI).  Therefore, while the bug is now fixed, I am not sure it was doing exactly what you had in mind.


IAM SSL Deprecation Warning

Looks like I should move from cert.serial to cert.serial_number.
security_monkey/watchers/iam/iam_ssl.py:75: DeprecationWarning: Certificate serial is deprecated, use serial_number instead.
  return cert.serial

Same problem. monkey find changes is died, after this warning.

#870 has been merged. Closing this issue.


Add npm package-lock

This adds package-lock.json which will help prevent future dependency problems. From the npm documentation:

This file describes an exact, and more importantly reproducible node_modules tree. Once it's present, any future installation will base its work off this file, instead of recalculating dependency versions off package.json.

Should we be publishing package-lock.json files? If http-server is used as a dependency instead of a global CLI tool wouldn't this complicate the install tree?

If published, it would definitely end up complicating the install tree of anything using it as a dependency. However, part of the beauty of package-lock.json is that it can't be published. When publishing, npm will simply ignore it. It's designed for use in the source code only, i.e. here on GitHub.

Using a package-lock.json ensures that everyone who works with this source code will be using the exact same versions of every dependency. Its use is strongly encouraged by npm.

@thornjad we should try and get package-lock.json updated and committed before the next release. It is, as you've noted, the new normal. 😃

Perfect! I just updated this PR so the package-lock.json is up to date with master.

Thanks @thornjad!


Activate applications from new SuperModel snapshot

This is basically a one-liner: calling applicationActivated() based on new SuperModel snapshot instead of the currently cached (and null) SuperModel.


Linux: fix flickering

[x] #23

https://user-images.githubusercontent.com/4352041/124484534-318b5a00-ddb4-11eb-9199-fe7f5b97a45b.mp4

https://user-images.githubusercontent.com/4352041/126194291-0907eeed-add3-4dc5-9154-f84a2a989006.mp4

some resize glitches still present for me, but now there is no black rectangles.
VSYNC looks gray, despite on video.

Discussion moved to #54 to avoid confusion


cranelift: Implement TLS on aarch64 Mach-O (Apple Silicon)

Fixes #5433.
This PR implements support for TLS on aarch64 Mach-O (i.e. Apple silicon). This also includes an update to object as this PR depends on a fix released in object 0.30.
For testing, I've added a filetest similar to the existing test for aarch64 ELF. I wasn't sure if I should add an emit test as well, since there doesn't seem to be one for TLS on aarch64 ELF. I've also tested these changes on a local branch of cg_clif, and it passes the TLS tests there.

I did an audit of object's upgrade; however cargo vet is showing an audit backlog of hashbrown, once_cell, and ahash, and I don't really feel qualified to audit those at all. Anyone else want to take a crack at it (@alexcrichton, @fitzgen, @jameysharp, @elliottt maybe?) or am I missing something about our policy here on core/popular crates?

or am I missing something about our policy here on core/popular crates?

Not missing anything AFAIK. Someone just has to bite the bullet and review the new deps and delta for upgraded deps.

I did an audit of object's upgrade; however cargo vet is showing an audit backlog of hashbrown, once_cell, and ahash, and I don't really feel qualified to audit those at all. Anyone else want to take a crack at it (@alexcrichton, @fitzgen, @jameysharp, @elliottt maybe?) or am I missing something about our policy here on core/popular crates?

Since there is no review process for an Rust update, all of std is considered trusted.
Hashbrown is part of the rust-lang organization and is used by std::HashMap, so it is trusted if it is part of std.
I think that trust should be extended if it is used directly and also to its dependencies like ahash and indirect dependencies like once_cell .

@nathanwhit would you be willing to rebase this? I think we should be close to able to merge this with vets that have happened in the meantime...

@nathanwhit would you be willing to rebase this? I think we should be close to able to merge this with vets that have happened in the meantime...

Done!

Ah, the merged cargo-vet review for the object crate is for 0.30.1, not 0.30. If you bump to that version, I believe the cargo-vet check will pass.
In addition, cargo-deny is failing because there are multiple versions of ahash, hashbrown, and object being pulled in by different crates. I think these have all been resolved in #5550, but that hasn't merged yet.
I'm sorry that this PR is taking a long time to merge. We're still working out our processes for supply-chain review. We're learning though!

@cfallin  I think the best way would be to merge https://github.com/bytecodealliance/wasmtime/pull/5513, since there are object is already updated to 0.30.1

Now object has been updated in master. I think with an rebase the build should succeed.
Thanks for your work!

Rebased this locally onto main and all tests pass (except for a filetest needing bless due to https://github.com/bytecodealliance/wasmtime/pull/5780). Any updates on this?

Ah sorry, completely forgot this hadn't been merged. Thanks for the reminder!
I've rebased on top of main so it should be good to go now


CTolbert DS 141 Statistical Tests and Experiments

Good to see you've been working extremely hard on solidifying your stats knowledge, Christopher! Fantastic effort and nice charting work in A1.

Sprint Challenge Code Review
Week 4 - Statistical Tests and Experiments

Part 1 — Load and Validate the Data
Good work on the data clean-up and prep - you have found the NaNs, "?", as well as identified that some of the data are incorrectly recognized as objects - while you changed some of the columns to numeric data, the ones that remain in the object format need to be transformed as well.

Part 2— Exploring Data, Testing Hypotheses
Good work on the second section of this project! Please make sure to use the stats.chi2_contingency as this option better addresses the problem. https://docs.scipy.org/doc/scipy-0.15.1/reference/generated/scipy.stats.chi2_contingency.html

Part 3 — Analysis and Interpretation
INC
Good attempt, Christopher!


Uses enforceSync to check authorizations

@Ghita2002 this is to show you how you can refactor the authorization check. You can merge this PR, which targets your branch, and update the rest of the code accordingly (I only updated the /api/user_rights' handler).

Updates the initialisation of Casbin so that we start only when the enforcer is ready
Uses enforceSync, which makes the checking mechanism was easier
Adds a utility function isAuthorized

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
163 unchanged lines in 5 files lost coverage.
Overall coverage increased (+1.5%) to 58.278%

Files with Coverage Reduction
New Missed Lines
%

services/dkg/pedersen/controller/action.go
3
49.17%

contracts/evoting/types/ballots.go
4
98.88%

cli/cosipbftcontroller/mod.go
10
94.08%

services/shuffle/neff/handler.go
11
81.02%

contracts/evoting/controller/action.go
135
0%

Totals

Change from base Build<PHONE_NUMBER>:
1.5%

Covered Lines:
2355

Relevant Lines:
4041

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
163 unchanged lines in 5 files lost coverage.
Overall coverage increased (+1.5%) to 58.278%

Files with Coverage Reduction
New Missed Lines
%

services/dkg/pedersen/controller/action.go
3
49.17%

contracts/evoting/types/ballots.go
4
98.88%

cli/cosipbftcontroller/mod.go
10
94.08%

services/shuffle/neff/handler.go
11
81.02%

contracts/evoting/controller/action.go
135
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
1.5%

Covered Lines:
2355

Relevant Lines:
4041

💛  - Coveralls


Unit-test for the refactored library - [merged]

In GitLab by @cmastalli on Jan 22, 2019, 16:55
Merges topic/unitest -> topic/refact
The unit-test code is very messy and sparse. Unfortunately, I've to make many changes to have a proper unit-test code.
I aim with this :

to allow you to check the WIP status, and
to make fast feedback about re-organization and creation of test cases.

In GitLab by @cmastalli on Jan 22, 2019, 16:55
changed the description

In GitLab by @cmastalli on Jan 22, 2019, 16:59
In these commits I do:

moving all unit-test code into test_state, test_unicyle and test_solvers,
enabling all these unit-test files,
creating test-cases for all state and unicyle unit-test, and
starting with the test-cases for shooting problem, KKT solver and DDP solver.

In GitLab by @nmansard on Jan 22, 2019, 17:18
I look at that tonight. Should I merge if I like it?

In GitLab by @cmastalli on Jan 22, 2019, 17:27
It depends! No, if you believe this task is orthogonal to other commitments.
Semantically speaking, I didn't finish all unit-test; there are missing many of them.

In GitLab by @cmastalli on Jan 22, 2019, 17:27
changed the description


FIX: Show group icon for group notifications

The same "user-plus" icon was used for both user and group assigned notifications.

@nbianca do we need to change the icon for group assignments in the new/experimental user menu?

@OsamaSayegh I wanted to, but the API I need for it is marked as "experimental":
https://github.com/discourse/discourse/blob/6affbabfd3a09c09b8800a8b0f81f2a07498c2fb/app/assets/javascripts/discourse/app/lib/plugin-api.js#L1882
I am leaving this here for the future:
  api.registerNotificationTypeRenderer("assigned", (NotificationTypeBase) => {
    return class extends NotificationTypeBase {
      get icon() {
        return this.notification.data.message ===
          "discourse_assign.assign_group_notification"
          ? "group-plus"
          : "user-plus";
      }
    };
  });