18_deep_neural_network

@nimajam41

سلام، خسته نباشید.
هر دوی
Fully Connected Layer

Convolutional layer
اصلاح شود؟ یا فقط آخری؟

On Jan 14, 2022, at 21:29, nimajam41 @.***> wrote:
@nimajam41 commented on this pull request.
In notebooks/18_deep_neural_network/index.md https://github.com/sut-ai/notes/pull/29#discussion_r785047028:

+- The Output Size of Conv Layer
+- Conclusion
+- References

+## Introduction
+Deep learning is a subfield of machine learning that deals with algorithms inspired by the structure and function of the brain. Deep learning is a subset of machine learning, which is a part of artificial intelligence (AI).
+
+
+CNN's are models to solve deep learning problems. Suppose that you have high-dimensional inputs such as images or videos. If we want to use MLPs, 2 (or more) dimensional inputs need to be converted to 1-dimensional vectors. This conversion increases the number of trainable parameters exponentially. Also, one important thing in these data is locality, it means that for example in an image, you can find features in near pixels (for examples corners and edges) but, far pixels can't give you efficient features. The solution for solving these problems is using CNNs.
+
+## CNN Architecture
+A CNN architecture is formed by a stack of distinct layers that transform the input volume into an output volume through a differentiable function. A few distinct types of layers are commonly used:
+
+* Fully Connected Layer
+* Convolutional layer
layer -> Layer (to be synced with other bullets)
—
Reply to this email directly, view it on GitHub https://github.com/sut-ai/notes/pull/29#pullrequestreview-853207281, or unsubscribe https://github.com/notifications/unsubscribe-auth/AODND5RET3GXD4ZRWCM53ALUWBP75ANCNFSM5LPU6NEA.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.

من دقیقا متوجه منظورتون از to be synced with other bullets نشدم.
ممکن هست بیشتر توضیح بدین؟

On Jan 14, 2022, at 21:29, nimajam41 @.***> wrote:
@nimajam41 commented on this pull request.
In notebooks/18_deep_neural_network/index.md https://github.com/sut-ai/notes/pull/29#discussion_r785047028:

+- The Output Size of Conv Layer
+- Conclusion
+- References

+## Introduction
+Deep learning is a subfield of machine learning that deals with algorithms inspired by the structure and function of the brain. Deep learning is a subset of machine learning, which is a part of artificial intelligence (AI).
+
+
+CNN's are models to solve deep learning problems. Suppose that you have high-dimensional inputs such as images or videos. If we want to use MLPs, 2 (or more) dimensional inputs need to be converted to 1-dimensional vectors. This conversion increases the number of trainable parameters exponentially. Also, one important thing in these data is locality, it means that for example in an image, you can find features in near pixels (for examples corners and edges) but, far pixels can't give you efficient features. The solution for solving these problems is using CNNs.
+
+## CNN Architecture
+A CNN architecture is formed by a stack of distinct layers that transform the input volume into an output volume through a differentiable function. A few distinct types of layers are commonly used:
+
+* Fully Connected Layer
+* Convolutional layer
layer -> Layer (to be synced with other bullets)
—
Reply to this email directly, view it on GitHub https://github.com/sut-ai/notes/pull/29#pullrequestreview-853207281, or unsubscribe https://github.com/notifications/unsubscribe-auth/AODND5RET3GXD4ZRWCM53ALUWBP75ANCNFSM5LPU6NEA.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.


Bug - Cannot send with too many UTXO

Issue
If a wallet has too many UTXO's it is signing with all of them, causing errors during sending
Solution
Only gather enough UTXO's to use within the transaction, do not include all.

Reposting from #78
We may also need to account for similar UTXO issues with SLP transactions and how to handle Send Max when more than one transaction is required

Good catch, will update issue and leave it open until that case is addressed as well

Update - Issue was also for SLP transactions.  Fixed in recent PR.
Send Max is only outstanding issue with this currently.


Subscribe form doesn't work

In the "Stay in touch" form at the bottom of the https://opensource.guide/ page:
Subscribe doesn't work.
I do not know if it is here to report it.
Thanks in advance.

Can you elaborate on how it doesn't work? I see "thanks for subscribing"? Do you have JavaScript enabled in your browser? What browser is it?

I'm sorry, I should have given more details.
With Firefox 59.0.2 (64 bits) it didn't work, however with Chrome Version 65.0.3325.181 (Official Assembly) (64 bits) has worked well. In both JavaScript it's enabled.

I found the problem in Firefox browser: tracking protection blocked JavaScript.
The resource of the site has been blocked «https://github.us11.list-manage.com/subscribe/post-jsonXXXX» because the protection against tracking is enabled.


Can't connect to containers on service:pia network

I have been reading the issues and readme and I can't figure this out.
I have emby connected to service:pia. I did have ports 8096 and 8920 opened up.
I change the emby network_mode to service:pia and get rid of the port forwarding. I then open up the ports on the PIA container but I cannot connect to Emby through the PIA container.
I did the test run to make sure PIA was acutally running and I am getting an expected IP address.
I have my local subnet added as extra: - EXTRA_SUBNET=<IP_ADDRESS>/23
However, trying to access the IP of my docker host on port 8096 doesn't work. I am assuming I have the bridge set up wrong compared to my other network I already have?

Update: I can now access it internally, but now my Traefik rules won't work.
I think I need to redo my whole docker network(s)

Traefik does not work super well with this container unfortunately as I recall. A few other notes:

port forwarding is only to forward a port on the PIA server side, it has nothing to do with local network
EXTRA_SUBNETS should be set if you want to directly access the PIA container through for example Shadowsocks. If you want to access another container connected to it (emby) you can leave it blank as the local Docker network of the PIA container is automatically whitelisted
You might want to try with another network than bridge, bridge can behave a bit differently sometimes.
You could play around with qmcgaw/port-checker to test a port is accessible (that's what I do).

Good luck! Feel free fo ask more questions.


refine config/mesh code

[x] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[x] Does not have any changes that may affect Istio users.
What this PR does?
https://github.com/istio/istio/blob/e60c24a399ab0e8293abfc74e9c7d7b7e7e4391e/pkg/config/mesh/mesh.go#L33
mesh.go import filewatcher but do nothing about watch. It seems disturbed.
This PR move addFileWatcher  to appropriate place.

Hi @Betula-L. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Hi @Betula-L. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@googlebot I signed it!

@googlebot I signed it!

@googlebot I fixed it.

@googlebot I fixed it.

/ok-to-test

/ok-to-test


Build run in CI hangs on "Waiting for builds to complete"

Build/Submit details page URL
No response
Summary
When executing the build and submit process on CI, the latter hangs until the build has completed, effectively blocking the pipeline until it has finished or it times out, wasting resources and money.
As it's not needed to wait for the build to be completed it could be useful to disable this behaviour on CIs by default or via a
cli flag.
Managed or bare?
bare
Environment
Azure Pipelines
Error output
No response
Reproducible demo or steps to reproduce from a blank project
eas build --auto-submit --platform all --profile production --non-interactive

@nickfla1-mxm what EAS CLI are you using?

eas build --help
start a build

USAGE
  $ eas build [-p android|ios|all] [--json] [--profile <value>] [--non-interactive] [--local] [--output <value>] [--wait] [--clear-cache] [--auto-submit | --auto-submit-with-profile <vache] [--auto-submit | --auto-submit-with-profile <value>]

FLAGS
  -p, --platform=(android|ios|all)
  --auto-submit                            Submit on build complete using the submit profile with the same name as the build profile
  --auto-submit-with-profile=PROFILE_NAME  Submit on build complete using the submit profile with provided name
  --clear-cache                            Clear cache before the build
  --json                                   Enable JSON output, non-JSON messages will be printed to stderr
  --local                                  Run build locally [experimental]
  --non-interactive                        Run command in non-interactive mode
  --output=<value>                         Output path for local build
  --profile=PROFILE_NAME                   Name of the build profile from eas.json. Defaults to "production" if defined in eas.json.
  --[no-]wait                              Wait for build(s) to complete

DESCRIPTION
  start a build

COMMANDS
  build:cancel     cancel a build
  build:configure  configure the project to support EAS Build
  build:inspect    inspect the state of the project at specific build stages, useful for troubleshooting
  build:list       list all builds for your project
  build:submit     submit app binary to App Store and/or Play Store
  build:view       view a build for your project

Change the command to:
eas build --auto-submit --platform all --profile production --non-interactive --no-wait

Also ran into this, it is a bit silly that non-interactive waits for the build, the default should have been no-wait in my opinion


HDDS-4937. Enhance SCMServerProtocol with using ReplicationConfig

What changes were proposed in this pull request?
In HDDS-4882 a new ReplicationConfig is introduced. This patch shows how can it be used between OM and SCM on the protocol.
This patch is not a full refactor of SCM it focuses on the SCM protocol side only. Pipeline manager can be improved in follow-up patches...
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-4937
How was this patch tested?
CI

Closing this one for now. A more up-to-date view of the full picture can be found at https://github.com/elek/ozone/tree/ec.
Separated commits are uploaded as separated PRs.


Add footnotes

Also includes some formatting, maybe take a look at the Dockerfile changes as well as I could not get them working locally

Created a separate commit of the actual changed code; https://github.com/pwndoc/pwndoc/commit/d7673f1687572ee21ee32eb7a53abe63a08145fe


I have aligned the text center

The resume should be left align, the heading can be in the center but not the whole resume. Plus the resume you added had gaurav singh info inside it.


Nullable in 2.2.2 minimal changes branch breaks on cast

Resolving a promise with nil when the promise  is an optional type causes a cast exception.
The code on line 204 of Promise.swift is the source:
body(value as! T).pipe(resolve)
This code worked fine on the 2.0 minimal branch.  Needed to upgrade for swift 2.2 breaking changes.
Do you know of a workaround?

Can you provide a code example?

Well, strangely.. I don't seem to be able to.  I created a playground that does basically what my code is crashing on, but it's not reproducing that behavior.   This works fine...
`
func getResult() -> Promise<AnyObject?> {
return Promise { fulfill, reject in
print("running")
fulfill(nil)
print("fulfilled")
}

}
getResult().then { val -> Promise<Int?> in
print("Got value (val)")
return Promise(nil)
}.then { val in
print("Got number (val)")
}
XCPSetExecutionShouldContinueIndefinitely()`
Attached you can see where the code is crashing, and the source:

I found the same crash in my app.
This crash was shown not on swift-2.0-minimal-changes branch but on swift-2.2-minimal-changes) branch.
So It seems to occur because of 1ffca8f.

I cannot reproduce with Xcode 7.3.1 and swift-2.2-minimal-changes:

I cannot fix this without a TestCase, if someone could help me out…

Closing due to lack of responses.


Replace hand-written pre-commit hooks with pre-commit Python tool

How to test
pip install pre-commit
rm .git/hooks/*
pre-commit install

Do some changes that would fail tests, and try to commit, or run pre-commit run manually.

pre-commit can be installed with pip, and configured in the Godot repo with pre-commit install. It can then easily be run both locally with pre-commit run, and on CI, in a cross-platform way.
This makes it much easier for contributors to set up pre-commit hooks, without having to manually copy files to their git folder.

Supersedes #87003
Supersedes #46235

I've included fixes for the issues I mentioned in https://github.com/godotengine/godot/pull/87003#issuecomment-1944326039:

Merged the two black ids back together and made it apply on all .py, SConstruct and SCsub files, instead of relying on types_or: [python] which doesn't know SCons files.
Replaced the custom clang-format wrapper with the official pre-commit hook at https://github.com/pre-commit/mirrors-clang-format, which takes care of installing the clang-format version we want via Python. This solves the need of ensuring everyone uses the same clang-format version installed locally.
Removed all misc/hooks/ files which are no longer needed.
Removed misc/scripts/{black,clang}_format.sh, replaced by running pre-commit run locally.

@Calinou IMO some of the changes from #46235 may still be worth bringing over, maybe as a follow up PR. You seemed to have found pre-existing hooks that can help us get rid of misc/scripts/file_format.sh, and the global exclude pattern may not be a bad idea.
We can likely look into moving more of the misc/scripts/ / CI checks to pre-commit, e.g. dotnet format (CC @godotengine/dotnet) or codespell.

Isn't there some missing scripts in the new .pre-commit-config.yaml file? I don't see canonicalize_filename.sh in the new config.

Isn't there some missing scripts in the new .pre-commit-config.yaml file? I don't see canonicalize_filename.sh in the new config.

canonicalize_filename.sh is a helper used by the other scripts like clang_format.sh and black_format.sh.
One of the advantages of using pre-commit is that we don't need such hacks anymore and let the framework take care of cross-platform file paths.

For the record / future work, I looked into moving mypy_check.sh to pre-commit too, with the diff below. It works, but it raises a lot of errors about unknown SCons APIs that are not raised with the current script, for some reason.
diff --git a/.github/workflows/static_checks.yml b/.github/workflows/static_checks.yml
index a9808fee95..09a75b481e 100644
--- a/.github/workflows/static_checks.yml
+++ b/.github/workflows/static_checks.yml
@@ -23,7 +23,7 @@ jobs:

       - name: Install Python dependencies and general setup
         run: |
-          pip3 install pytest==7.1.2 mypy==0.971
+          pip3 install pytest==7.1.2
           git config diff.wsErrorHighlight all

       - name: Get changed files
@@ -57,14 +57,6 @@ jobs:
         run: |
           bash ./misc/scripts/header_guards.sh changed.txt

-      - name: Python scripts static analysis (mypy_check.sh)
-        run: |
-          if grep -qE '\.py$|SConstruct|SCsub' changed.txt || [ -z "$(cat changed.txt)" ]; then
-            bash ./misc/scripts/mypy_check.sh
-          else
-            echo "Skipping Python static analysis as no Python files were changed."
-          fi
-
       - name: Python builders checks via pytest (pytest_builders.sh)
         run: |
           bash ./misc/scripts/pytest_builders.sh
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index 3493219ea7..d30e4075cf 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -23,6 +23,15 @@ repos:
         args:
           - --line-length=120

+  - repo: https://github.com/pre-commit/mirrors-mypy
+    rev: v0.971
+    hooks:
+      - id: mypy
+        files: (\.py$|SConstruct|SCsub)
+        types_or: [text]
+        exclude: .*thirdparty.*
+        args: [--config-file=./misc/scripts/mypy.ini]
+
   - repo: local
     hooks:
       - id: make-rst
diff --git a/misc/scripts/mypy_check.sh b/misc/scripts/mypy_check.sh
deleted file mode 100755
index 2a06486d67..0000000000
--- a/misc/scripts/mypy_check.sh
+++ /dev/null
@@ -1,6 +0,0 @@
-#!/usr/bin/env bash
-
-set -uo pipefail
-
-echo -e "Python: mypy static analysis..."
-mypy --config-file=./misc/scripts/mypy.ini .


The .git directory is not detected if it's not in the project root

Maybe you

Duplicated. Take a look at #312


Fixed commandExists() not working for absolute path on windows

This fixes issue #23, where the RegEx check that causes commandExistsWindows async/sync to return early for commands with an absolute path, e.g. C:\Program Files\...
Also included absolute paths test in test.js and replaced dir with xcopy because dir is not a command that has an associated file. where dir will never return true on Windows.

Thanks @Antyos! Any idea why the absolute path test is failing on Travis? Happy to merge once tests are passing.

It looks like it failed on the check to find a command that does exist with an absolute path, how strange. I would guess it has something to do with the format of absolute paths on Linux. I'll look into it when I have a chance later this week.
Thanks!

Sounds good, thanks @Antyos!

All fixed! The test code I initially added tried to get Linux to run a .cmd file, which of course can't run on Linux... I've switched it to the .js file.
As a side note, I noticed that the test code runs about 20x slower on a Windows machine than a Linux one, even before my changes. I looked into it a bit and I think this is an issue with Node's exec function, not this code. I'm still fairly new to Nodejs/npm/JavaScript, so I'm not sure if it's worth looking into further.
Also, npm install said that some of this project's dependencies have known vulnerabilities. Is it worth updating the dependencies to fix the vulnerabilities, or is it better to leave things as they are?

Hey @mathisonian, I just wanted to check in and see if we're still good with the pull request. I'm sure you've been busy the last couple of week, so no worries if this got buried in your inbox.

Hey thanks for pinging me on this @Antyos. The changes look good. I'm not sure about about the windows exec performance thing; I think that and dependencies are separate issues, we can address in a different PR.
GitHub isn't alerting me about any vulns but NPM may be ahead, I will look into that .

To follow up re: dependencies -- it looks like all the issues are coming from subdependencies of mocha. This code is only used for the test suite, so it shouldn't affect end users of this library, but we should still update.

Thanks for merging it @mathisonian! You should also be able to close issue #23 now.
Also, do you plan to update Mocha or does that need a pull request?

No problem, thanks @Antyos! A PR for mocha would be appreciated if you have the chance, otherwise I will get to it, but no promises on how quickly (I'm a bit bogged down at the moment).


MINOR: Add tests to verify setting of serdes in timestamped key-value store

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

\cc @mjsax @bbejeck @vvcephei @ableegoldman @abbccdda @guozhangwang

Merged to trunk

cherry-picked to 2.3


Configuring dist folder as an application in IIS

For hosting on IIS, when we create an application(Ex: TestApplication) under a DefaultWebSite(poining to localhost) .
In index.html 'src' for app.bundle and polyfills.bundle are pointing to localhost ratherthan localhost/TestApplication.
Is there any approach to point them it to localhost/TestApplication. As this application is not working as an app inside a website.
For example we need.
output -> dist
src -> from webapplication/dist.
Thanks,
Bharani

I finally fixed using publicPath location to my application relative path output: of webpack.dev.config

Hi @cvbc2010
Yes you can change this innside the webpack dev and prod config files. If you search fpr dist, you will find the definitions inside the files.
Greetings Damien


Shortcuts for markdown

Add shortcuts to apply markdown formatting to the text such as: Ctrl+B is for bold.

Done in 0.6.0


Need some detail on with Cognitive Search key to use.

Good article overall, but I think you shall say to use "Admin Key" of Azure, query key does not work :)

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

ID: ae6c6cb5-fda6-677b-32b0-722e019831ab
Version Independent ID: 19b78fd7-e14c-158f-b5d9-342395f8a5fd
Content: Use your own data with Azure OpenAI Service - Azure OpenAI
Content Source: articles/ai-services/openai/use-your-data-quickstart.md
Service: cognitive-services
Sub-service: openai
GitHub Login: @aahill
Microsoft Alias: aahi

Hi @Zion-Avantis
Thank you for bringing this to our attention.You are correct that to use the Azure Cognitive Search service with the OpenAI API, you need to use the admin key instead of the query key.The admin key provides full access to the Azure Cognitive Search service, including the ability to create, update, and delete indexes and data sources. The query key, on the other hand, only provides read access to the service, which is not sufficient for creating or updating indexes and data sources.
We will discuss with our Product Team, they will work on updating the Use your data with OpenAI tutorial to reflect the correct key to use with the Azure Cognitive Search service.
Please add your ideas in below link, so our production team can review it and update the same.
Ideas · Community (azure.com)

@Zion-Avantis
We are going to close this thread, if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.
#Please-close


Use CAPS in a Zeppelin notebook Update

Good morning team,
Thank you very much for all your hard work on this project! I cant wait to start playing around again with the latest version of CAPS. One thing that I have noticed since the last time I tried the beta release is that the Zeppeling notebook example https://github.com/opencypher/cypher-for-apache-spark/wiki/Use-CAPS-in-a-Zeppelin-notebook in the wiki is still not up to date. I remember trying the default modules presented in the notebook before and it did not work. I had to figure out what module names were needed for some of the steps. Is this something that will be updated soon?? Thank you in advance

Hey @Cyb3rWard0g and thanks for your kind words.
Zeppelin does work with CAPS -- in fact we presented at Spark AI Summit a month ago using Zeppelin. You'll find the recording of the talk here.
We'll make sure to update the instructions so you can also do this.

Thank you very much @Mats-SX . Great presentation 👍 I will follow the examples you mentioned there.

Hej @Cyb3rWard0g we updated the wiki page now to reflect the latest API changes. Thanks for pointing that out :)

Thank you guys! I will give it a try soon :)


security iot-solution update returns bad request

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Related command
az security iot-solution update
Describe the bug
The update command returns a Operation returned an invalid status 'Bad Request'
with debug, we further see two serialization issues:
msrest.serialization: displayName is not a known attribute of class <class 'azure.mgmt.security.models._models_py3.UpdateIotSecuritySolutionData'> and will be ignored
msrest.serialization: iotHubs is not a known attribute of class <class 'azure.mgmt.security.models._models_py3.UpdateIotSecuritySolutionData'> and will be ignored

To Reproduce

create an iot-solution with az security iot-solution create
update that iot-solution with az security iot-solution update

Expected behavior
The update to work (or at least not give a bad request)
Environment summary
az --version
azure-cli                         2.41.0

core                              2.41.0
telemetry                          1.0.8

Extensions:
azure-iot                         0.17.2
ai-examples                        0.2.5
ml                                2.10.0
ssh                                1.1.2

Dependencies:
msal                            1.20.0b1
azure-mgmt-resource             21.1.0b1

Python location '/usr/bin/python3.9'
Extensions directory '/home/victoria/.azure/cliextensions'
Extensions system directory '/usr/lib/python3.9/site-packages/azure-cli-extensions'

Python (Linux) 3.9.13 (main, Jul 31 2022, 23:00:51)
[GCC 11.2.0]

Legal docs and information: aka.ms/AzureCliLegal

Your CLI is up-to-date.

Additional context

route to CXP team

Adding Service team to look into this.
@chlahav Could you please look into this and provide an update ?

Hi @navba-MSFT @vilit1,
The issue was fixed, and will be available in the second week of November.

Thanks @vakuras for your quick reply.
@vilit1 Hope this answers. Please feel free to reopen this thread if you need further information on this. We would be happy to help.


Blank/Black Space when Uploading Cropped Picture (in Mobile)

Hi - when I upload a cropped picture, it now has blank/black space on the right or bottom side of the picture. I didn't have this issue before but has started last week. It seems to only be an issue when I use it on my phone (mobile view). Any idea what could be the issue?

+1
I am also experiencing this issue in Chrome Desktop (Safari and Firefox work fine). It seems like latest Chrome updates affected it.

+1
We have a similar issue when cropping images in a pop-up.

We, too, have the same issue in the model pop up.Any ideas on a workaround?

I have a similar problem detailed here.

Not ideal, but found a temporary workaround that works for me:

Problem: Chrome Updates are causing black/blank lines on border of
image, when you zoom out from Initial Default Zoom (slightly zoomed in).
Solution:
-Set Minimum Zoom to Initial Default Zoom (partially zoomed in),
-Go into croppie.js
-Go to "function _updateZoomLimits"
-CHANGE: "minZoom = Math.max(minW, minH);"

TO "minZoom = Math.max((boundaryData.width / imgData.width),
(boundaryData.height / imgData.height));"
-Now you cannot zoom out greater than the default zoom, which was causing
those lines.
.
Hopefully someone can find a strategic fix, where you can zoom out all the
way and not have these issues.

On Tue, Sep 14, 2021 at 3:13 AM Amatek @.***> wrote:

I have a similar problem detailed here
https://stackoverflow.com/questions/69173325/misalignment-and-black-blank-upload-when-cropping-images-with-croppie-js-and-a-m
.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Foliotek/Croppie/issues/764#issuecomment-918875471,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADY5IOTTG5RYZYKFCSMKMETUB3YZHANCNFSM5D54POQA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Thanks for your response. I have tried your suggestion but the issue persists. Do you experience the same or a similar issue mentioned here when you visit the Codepen links? If you do, can you test your suggestion to see if it makes a difference on your device?

Not ideal, but found a temporary workaround that works for me: 1) Problem: Chrome Updates are causing black/blank lines on border of image, when you zoom out from Initial Default Zoom (slightly zoomed in). 2) Solution: -Set Minimum Zoom to Initial Default Zoom (partially zoomed in), -Go into croppie.js -Go to "function _updateZoomLimits" -CHANGE: "minZoom = Math.max(minW, minH);" TO "minZoom = Math.max((boundaryData.width / imgData.width), (boundaryData.height / imgData.height));" -Now you cannot zoom out greater than the default zoom, which was causing those lines. . Hopefully someone can find a strategic fix, where you can zoom out all the way and not have these issues.
…
On Tue, Sep 14, 2021 at 3:13 AM Amatek @.***> wrote: I have a similar problem detailed here https://stackoverflow.com/questions/69173325/misalignment-and-black-blank-upload-when-cropping-images-with-croppie-js-and-a-m . — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#764 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADY5IOTTG5RYZYKFCSMKMETUB3YZHANCNFSM5D54POQA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Sorry, but it appears your bug is the opposite of mine. Mine was only
happening on Mobile, while it seems like yours is only happening on Desktop?
On Tue, Sep 14, 2021 at 7:56 PM Amatek @.***> wrote:

Thanks for your response. I have tried your suggestion but the issue
persists. Do you experience the same or a similar issue mentioned here
https://stackoverflow.com/questions/69173325/misalignment-and-black-blank-upload-when-cropping-images-with-croppie-js-and-a-m
when you visit the Codepen links? If you do, can you test your suggestion
to see if it makes a difference on your device?
Not ideal, but found a temporary workaround that works for me: 1) Problem:
Chrome Updates are causing black/blank lines on border of image, when you
zoom out from Initial Default Zoom (slightly zoomed in). 2) Solution: -Set
Minimum Zoom to Initial Default Zoom (partially zoomed in), -Go into
croppie.js -Go to "function updateZoomLimits" -CHANGE: "minZoom =
Math.max(minW, minH);" TO "minZoom = Math.max((boundaryData.width /
imgData.width), (boundaryData.height / imgData.height));" -Now you cannot
zoom out greater than the default zoom, which was causing those lines. .
Hopefully someone can find a strategic fix, where you can zoom out all the
way and not have these issues.
… <#m_2718906127288843723>
On Tue, Sep 14, 2021 at 3:13 AM Amatek @.***> wrote: I have a similar
problem detailed here
https://stackoverflow.com/questions/69173325/misalignment-and-black-blank-upload-when-cropping-images-with-croppie-js-and-a-m
. — You are receiving this because you authored the thread. Reply to this
email directly, view it on GitHub <#764 (comment)
https://github.com/Foliotek/Croppie/issues/764#issuecomment-918875471>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADY5IOTTG5RYZYKFCSMKMETUB3YZHANCNFSM5D54POQA
. Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub
.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Foliotek/Croppie/issues/764#issuecomment-919591534,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADY5IOSQXT6EK7LPPBWMFF3UB7OMDANCNFSM5D54POQA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Not sure it is an ideal solution or not.
After I added  the style
body .modal-dialog { margin-left: 0 }
the issue is resolved, but the pop up appeared on the left rather than centre
Sample is here

Hello, guys!
Got this problem on latest Chrome when used croppie inside of modal
Helped me to change style in bootstrap modal .modal-dialog class

Setting transform to unset instead of using translate function.
Hope it helps you too

Hello, guys!
Got this problem on latest Chrome when used croppie inside of modal
Helped me to change style in bootstrap modal .modal-dialog class

Setting transform to unset instead of using translate function.
Hope it helps you too

This worked for us! Thank you!

Thanks! This worked. It seems Croppie is unable to identify "transform: translate" operations on some devices while placing the image in the modal. I would appreciate it if you can post your answer here. I am sure it will help more people.

Hello, guys!
Got this problem on latest Chrome when used croppie inside of modal
Helped me to change style in bootstrap modal .modal-dialog class

Setting transform to unset instead of using translate function.
Hope it helps you too

This works - thank you!
On Thu, Sep 16, 2021, 7:29 AM Amatek @.***> wrote:

Thanks! This worked. It seems Croppie is unable to identify "transform:
translate" operations on some devices while placing the image in the modal.
I would appreciate it if you can post your answer here
https://stackoverflow.com/questions/69173325/misalignment-and-black-blank-upload-when-cropping-images-with-croppie-js-and-a-m.
I am sure it will help more people.
Hello, guys!
Got this problem on latest Chrome when used croppie inside of modal
Helped me to change style in bootstrap modal .modal-dialog class
[image: image]
https://user-images.githubusercontent.com/15245651/133401759-8548e989-7a89-45e5-be2e-7dddb53e12ba.png
Setting transform to unset instead of using translate function.
Hope it helps you too
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Foliotek/Croppie/issues/764#issuecomment-920819847,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADY5IORNKD4R4JHN5EF4A23UCHIKXANCNFSM5D54POQA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@amatek Copied my answer to your stackoverflow thread :)

Ameeennnn @TkachenkoMax solutions works perfectly! God save internet!

Looks like the issue is resolved in Google Chrome 94. Can everyone else still replicate this issue?


Implement PEP3134 to discover underlying problems with python3

Issue #, if available:
#354
Description of changes:
Implement PEP3134 through six.raise_from to support python2 and python3.
Also move one import to top of the module to better comply with pylint (C0413:wrong-import-position)
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

This ticket is tied to the following troubleshooting and discovering this issue: Pynamodb 1073

@lehmat I've never seen raise e from e where both the exception and the cause are the same.  When playing with that in IPython, I don't see any benefit either.  Instead it adds a lot of noise:
Python 3.11.0 (main, Nov 12 2022, 07:10:51) [GCC 11.2.1 20220219]
Type 'copyright', 'credits' or 'license' for more information
IPython 8.6.0 -- An enhanced Interactive Python. Type '?' for help.

In [1]: try:
   ...:     raise ValueError("foo")
   ...: except ValueError as exc:
   ...:     print("bar")
   ...:     raise exc from exc
   ...:
bar
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
    [... skipping hidden 1 frame]

Cell In [1], line 5
      4 print("bar")
----> 5 raise exc from exc

Cell In [1], line 2
      1 try:
----> 2     raise ValueError("foo")
      3 except ValueError as exc:

ValueError: foo

The above exception was the direct cause of the following exception:

ValueError                                Traceback (most recent call last)
    [... skipping hidden 1 frame]

Cell In [1], line 5
      4 print("bar")
----> 5 raise exc from exc

Cell In [1], line 2
      1 try:
----> 2     raise ValueError("foo")
      3 except ValueError as exc:

ValueError: foo

The above exception was the direct cause of the following exception:

ValueError                                Traceback (most recent call last)
Cell In [1], line 5
      3 except ValueError as exc:
      4     print("bar")
----> 5     raise exc from exc

Cell In [1], line 2
      1 try:
----> 2     raise ValueError("foo")
      3 except ValueError as exc:
      4     print("bar")

ValueError: foo

PEP 3134 doesn't mention such a use case either.
Can you elaborate?
raise
In [2]: raise ValueError("foo")
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Cell In [2], line 1
----> 1 raise ValueError("foo")

ValueError: foo

try-catch-raise without from
In [3]: try:
   ...:     raise ValueError("foo")
   ...: except ValueError as exc:
   ...:     print("bar")
   ...:     raise exc
   ...:
bar
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Cell In [3], line 5
      3 except ValueError as exc:
      4     print("bar")
----> 5     raise exc

Cell In [3], line 2
      1 try:
----> 2     raise ValueError("foo")
      3 except ValueError as exc:
      4     print("bar")

ValueError: foo

This PR was made because there was underlying issue hidden by the missing raise from syntax in aws-xray-sdk core. If this would have been implemented previously, the underlying issue would have been seen without disabling xray in the application code.
What was seen in logs prior to this PR

https://github.com/pynamodb/PynamoDB/issues/1073

What this PR improves

Before: The root cause is found at File "aws_xray_sdk/core/recorder.py", line 436, in record_subsegment,  with invalid quote_from_bytes arguments
After: The root cause if found at  File sentry_sdk/tracing_utils.py", line 541 with invalid quote_from_bytes argument

Essentially it will give that The above exception was the direct cause of the following exception: to be pointing at underlying issue at sentry-sdk instead of stopping the traceback at aws_xray_sdk
Reference of the underlying issue that was actually the issue:

https://github.com/getsentry/sentry-python/issues/1626

I saw both issues and read through them, but that doesn't explain why this is an issue in the first place.  A simple re-raise should not omit parts of the stacktrace.  In pynamodb/PynamoDB#1073 you said that you “disable[d] AWS X-ray to get the proper root cause to popup.”  Did you also try it with this patch and X-Ray enabled?  I'm not convinced that this actually solves your problem.
Maybe X-Ray accessing the stacktrace has something to do with it?  I played around with that, but found no hint that would support this theory.  But my toy stacktraces were small.
From raise's documentation:

Changed in version 3.11: If the traceback of the active exception is modified in an except clause, a subsequent raise statement re-raises the exception with the modified traceback. Previously, the exception was re-raised with the traceback it had when it was caught.

getsentry/sentry-python#1626 mentions Python 3.9, therefore the re-raise shouldn't have changed the traceback.

After reviewing your comments I agree with you that this does not solve the problem.

Did you also try it with this patch and X-Ray enabled?

No, manipulating dependencies would be troublesome with our way of deploying lambda
Essentially this would behave as it did (with sentry-sdk[flask]==1.9.5)
import os
import serverless_wsgi

import sentry_sdk
from flask import Flask
import boto3
from aws_xray_sdk.core import xray_recorder
from aws_xray_sdk.core import patch_all

xray_recorder.configure(service='Test')
patch_all()

app = Flask(__name__)

sentry_sdk.init(
    dsn=os.getenv("SENTRY_DSN"),  # need to have some sane here to actually make sentry work
    release=1,  # This was the breaking on sentry-sdk, integer release causing it
    traces_sample_rate=1,
)

@app.route("/hello")
def hello_world():
    segment = xray_recorder.begin_segment('segment_name')
    boto3.client('sts').get_caller_identity().get('Account')
    xray_recorder.end_segment()
    return "
Hello, World!
"

def handler(event, context):
    return serverless_wsgi.handle_request(app, event, context)


Make it possible to resend data to Network if we had no connection at the time of some changes from user (rename or remove sensor etc)

We can do without internet connection and be sure that it will be synced to Network when internet will be availible:

rename sensor
change sensor background
delete sensor (unclaim will be performed)
update offset for sensor
change application settings

These actions NEED internet connection at the time of perform:

claim sensor
share sensor

Especially delete

These will not sync new settings to network in build 1.5.19
Steps:

Turn internet connection off
Change Sensor name
Change Offset corrections
Turn internet connection on, sync with network
Data coming from network will overwrite new settings instead of updating to network

Special mention:
Trying to add and update background image when internet connection is off will crash app

QA passed for name, offset handled on another card


Update hyperlinks in README.md

We already updated the hyperlink earlier.

We already updated the hyperlink earlier.

there're still 3 links waiting for update, otherwise they will be linked to unfound page.


Updated price helper to output well formatted prices

refs https://github.com/TryGhost/Team/issues/472
The current {{price}} helper only works with amount to convert it into right value but doesn't allow any formatting with currency etc, leaving most of the work to theme.  We want to be able to output well formatted prices. E.g. the API returns 5000 for 5 EUR but we want to output €5.
The updated {{price}} helper can take a plan object or amount, currency and use them to output a well formatted price. It works with JS's built in Intl.NumberFormat behaviour to return output in different formats, also allowing theme devs to override formatting with options.
Examples:
With Plan object => {{price plan}} → "€5"
With Plan object and custom number format =>  {{price plan numberFormat="long"}} → "€5.00"

I think this needs to remove , as well - not sure about other characters

@allouis Good point, updated regex for , but also added minimum and maximum fraction digits to 0 in case we are calculating currency symbol to completely avoid any other case as we are only interested in the symbol.


Better invalid location handling

Currently if you arrive at an issue page with an invalid location (i.e. outside of the US, or somewhere that google civic can't determine any reps) the page is blank because of the thrown invalid address error.
If I comment out the thrown error, the call appears without a contact but with the outcome buttons.
We should message an invalid address in the location selector, but also in the call details. And remove the outcome buttons since they don't correspond to a contact.

The outcome buttons are removed in PR #64 if there are no contacts affiliated with an issue.

PR #64 seems to have fixed this issue (not just the outcome buttons), but we're waiting for verification from the person who reported it.


lock package to allow deployment of prometheus in sles15sp3 and newer

Signed-off-by: Ricardo Mateus<EMAIL_ADDRESS>What does this PR change?
Resolution for: https://github.com/SUSE/spacewalk/issues/17512
In the nightly build installations, we add a devel repo for client tools for each minion. This channel has a higher priority than all others, to make sure we install packages from this channel.
When we install the package golang-github-prometheus-prometheus it has as a dependency the package firewalld-prometheus-config.
On sles15SP3 this package is obsoleted by the package firewalld, provided in the OS base channel.

This PR solves this by locking the package firewalld-prometheus-config on sles15SP3 and higher, which solves the conflict problem.
I have tested it locally and the lock mechanism solves the problem.

Thank you!


Version <IP_ADDRESS> identified as fake Virus

Hello,
I have a similar problem as in issue #18 by Uhuru7050. When I download WeekNumber_<IP_ADDRESS>_Installer.exe and try to install it, Windows refuses to install and deletes the file. I don't even have to run the installation and after a while the EXE deletes itself.
When I had the app before, the icon often disappeared from the status bar.
I understand that the virus report is fake, but what should I do so that the application can be installed and the problem does not occur?
Windows 11
Windows Defender, nothing else
Thank you.

Hi, right, MS is really hard on us developers that does not pay for an expensive code signing certificate for our freeware that does not generate any money unfortunately.
So, if you do not want to use the lightweight version of WeekNumber - WeekNumberLite2 then you need to tell Windows to allow the installer; when you download it and it gets "removed" you can still go into Windows Security and then under Protection History allow it. It can be restored from there. If not download again and if MS removes it it will display a dialog that it has done so in the taskbar notification area, click that link.

@voltura, it doesn't seem to be just MS but also AV heuristics.
The installer seems to trigger contain quite some signatures which are 'malware-sensitive'
Check out this page: https://www.joesandbox.com/analysis/729422/1/html

Right, seems that the actual installer system used might actually contain something malicious.
I will when time allows investigate and replace installer.
Please note that app does not need to be installed via installer.

Removed installer from latest release - version <IP_ADDRESS>.
Removed older releases with installer created using NSIS scripts and plugins.

Removed all NSIS related files from repo, generated new release - version <IP_ADDRESS>. Now release sources does not include any NSIS files either. Removed previous release.

https://www.virustotal.com/gui/file/c18dd9e1b2c504821da25fff7a817f1aa906b12748d535102ef17bb6bf75cd18/detection
VirusTotal still has 28 security vendors that classifies WeekNumber.exe as a trojan and/or malware. This is not true.
The application does create configuration files, read and writes to the registry, starts processes and accesses the internet. But no action is maleficent. If still no trust in the WeekNumber application, please do not use it. Rather then use the bare bone version WeekNumber Lite 2 available here https://github.com/voltura/WeekNumberLite2


SamPE struct

Paired End Sam struct and channel reading functions.
currently the channel reading function reads a sam channel as normal and then from that channel parses out paired end reads and sends them to a new channel. I did it this way because there is already a lot of sam-reading infrastructure built in, but i could imagine doing it a different way. Opinions?
Right now i check to make sure paired end reads have the same name before putting them in a SamPE struct, but sometimes supplementary alignments get a additional sam entry which could mess things up. how is it best to handle this? I am considering adding an additional field to the SamPE struct which is a slice of sam that will contain all supp. alignments. Would have to do some flag checking for this but should be simple enough.

The supplementary alignments are interesting! How do you tell them apart from primary alignments? I like the additional field, and can imagine that for some output formats you would omit it in the final write-to-output-file stage

Yes good question. Supplementary alignments have the flag 2048. I just wrote a version 2 of the implements this logic if you want to check it out.


add manifest for Windows Virtual Desktop Bootloader 1.0.2778.0

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

Microsoft Reviewers: Open in CodeFlow

@keyoke, the manifest should be in the folder "manifests\Microsoft\wvd-boot-loader" for the test to pass. Also, you may have to change the publisher from "Microsoft Corporation" to "Microsoft".

@keyoke, the manifest should be in the folder "manifests\Microsoft\wvd-boot-loader" for the test to pass. Also, you may have to change the publisher from "Microsoft Corporation" to "Microsoft".

@keyoke, the manifest should be in the folder "manifests\Microsoft\wvd-boot-loader" for the test to pass. Also, you may have to change the publisher from "Microsoft Corporation" to "Microsoft".

thanks, I have moved to correct publisher folder

@keyoke, the manifest should be in the folder "manifests\Microsoft\wvd-boot-loader" for the test to pass. Also, you may have to change the publisher from "Microsoft Corporation" to "Microsoft".

thanks, I have moved to correct publisher folder


empty state of the cards in the webui is a face-down card instead of being blank

added a placeholder image


Execute tests on merge request and push to master

Hey @anoop1409 , you have done really good job, I really appreciate your contributions. thanks


Using Auth instance in Cloud Code

Hi,
I'm not sure how to use the Auth class in my Cloud Code.
I'd like to do some security checks in my before/afterSave methods.
Is it possible to use the already created Auth instance added in req by middleware.js ?
Should I create another one using the session token ?
Should I make my own Role queries ?
Thx a lot in advance,
Olivier Allouch
www.illicotravel.com

Closing due to lack of activity, please update to latest parse-server version and reopen if the issue persist.
Don't forget to include your current:

node version
npm version
parse-server version
any relevant logs (VERBOSE=1 will enable verbose logging)


Can't convert Page url to Pdf on Linux

I've corrected the configuration of Dockerfile and it's working correctly on Linux (don't have a problem with dependencies). But when I want to use the Convert method from DinkToPdf, it always returns an empty byte[] on Linux. (on Windows there return some bytes).
A similar problem was described here:
https://stackoverflow.com/questions/72901484/dinktopdf-converter-returns-empty-byte-array-on-ubuntu
If I am using ObjectSettings.Page = url from the page, there is this problem. Otherwise, if I am using ObjectSettings.HtmlContent = some content(string), then will generate correctly. This is only on Linux behavior. On Windows, both settings work properly.
My code:
 var document = new HtmlToPdfDocument             {                 GlobalSettings =                 {                     ColorMode = ColorMode.Color,                     Orientation = model.IsLandscape ? Orientation.Landscape : Orientation.Portrait,                     PaperSize = PaperKind.A4Plus,                     Margins = new MarginSettings                     {                         Bottom = MarginTopBottom,                         Left = MarginLeftRight,                         Right = MarginLeftRight,                         Top = MarginTopBottom,                         Unit = Unit.Millimeters                     }                 },                 Objects =                 {                     new ObjectSettings                      {                     PagesCount = true,                     WebSettings = { DefaultEncoding = "utf-8" }                     Page = model.HtmlUrl                      }                 }             };
Some logs during invoke Convert method:
QSslSocket: cannot resolve CRYPTO_num_locks
QSslSocket: cannot resolve CRYPTO_set_id_callback
QSslSocket: cannot resolve CRYPTO_set_locking_callback
QSslSocket: cannot resolve sk_free
QSslSocket: cannot resolve sk_num
QSslSocket: cannot resolve sk_pop_free
QSslSocket: cannot resolve sk_value
QSslSocket: cannot resolve SSL_library_init
QSslSocket: cannot resolve SSL_load_error_strings
QSslSocket: cannot resolve SSLv3_client_method
QSslSocket: cannot resolve SSLv23_client_method
QSslSocket: cannot resolve SSLv3_server_method
QSslSocket: cannot resolve SSLv23_server_method
QSslSocket: cannot resolve X509_STORE_CTX_get_chain
QSslSocket: cannot resolve OPENSSL_add_all_algorithms_noconf
QSslSocket: cannot resolve OPENSSL_add_all_algorithms_conf
Does somebody have the same problem only on Linux and for urlPage?
Thanks a lot!

Yeah actually I think we are facing the same issue right now. Using Convert works on Windows but returns empty bytes on Linux

ok, I've used this package and it's working correctly on linux machine:
https://github.com/HakanL/WkHtmlToPdf-DotNet


As an admin, I want to access data once stored on spreadsheets on the database.

The data from the spreadsheets must be imported into the database. This is important for visualizations.


Fix edge case where no reads map for a species

Causes an exception in filter_sample_reads.py. This used to work (by fluke I think), but something has changed in htslib.

Did we fix this already when I hit the bug where one of the species bam files is empty?
That is the function hits_generator in samutils.


Sync event properties

Nanomorph syncs all the event properties. We should probably do the same.

I don’t know if this is worth it because it’s a lot of additional checks to run for every element. 🤔

I’ve decided not to support this because event attributes are not recommended anyway, since they are unsafe.


fix<EMAIL_ADDRESS>to fix node@15 ERR_INVALID_PROTOCOL

With<EMAIL_ADDRESS>install puppeteer with https_proxy set will cause error like:
><EMAIL_ADDRESS>install /home/dr/Git/dr-js/node_modules/puppeteer
> node install.js

ERROR: Failed to set up Chromium r809590! Set "PUPPETEER_SKIP_DOWNLOAD" env variable to skip download.
TypeError [ERR_INVALID_PROTOCOL]: Protocol "https:" not supported. Expected "http:"
    at new NodeError (node:internal/errors:258:15)
    at new ClientRequest (node:_http_client:155:11)
    at Object.request (node:https:313:10)
    at httpRequest (/home/dr/Git/dr-js/node_modules/puppeteer/lib/cjs/puppeteer/node/BrowserFetcher.js:488:17)
    at downloadFile (/home/dr/Git/dr-js/node_modules/puppeteer/lib/cjs/puppeteer/node/BrowserFetcher.js:357:21)
    at BrowserFetcher.download (/home/dr/Git/dr-js/node_modules/puppeteer/lib/cjs/puppeteer/node/BrowserFetcher.js:239:19)
    at async downloadBrowser (/home/dr/Git/dr-js/node_modules/puppeteer/lib/cjs/puppeteer/node/install.js:48:5) {
  code: 'ERR_INVALID_PROTOCOL'
}

The related issue is at https://github.com/TooTallNate/node-agent-base/pull/47,
from package agent-base under https-proxy-agent
And the version bump is for Refactor to TypeScript: https://github.com/TooTallNate/node-https-proxy-agent/compare/4.0.0...5.0.0

Also there's an older "https-proxy-agent": "^2.2.1", under experimental/puppeteer-firefox/package.json, I did not change that since it's marked experimental.

And a simple package test: (with<EMAIL_ADDRESS>&<EMAIL_ADDRESS>nano package.json # paste original file
npm i --production --package-lock-only --ignore-scripts # just get the `package-lock.json`
mv package-lock.json package-lock.json-org # rename for next run

nano package.json # change to `"https-proxy-agent": "^5.0.0",`
npm i --production --package-lock-only --ignore-scripts

diff package-lock.json-org package-lock.json

and the diff result:
16c16
<         "https-proxy-agent": "^4.0.0",
---
>         "https-proxy-agent": "^5.0.0",
1599,1601c1599,1604
<       "version": "5.1.1",
<       "resolved": "https://registry.npmjs.org/agent-base/-/agent-base-5.1.1.tgz",
<       "integrity": "sha512-TMeqbNl2fMW0nMjTEPOwe3J/PRFP4vqeoNuQMG0HlMrtm5QxKqdvAkZ1pRBQ/ulIyDD5Yq0nJ7YbdD8ey0TO3g==",
---
>       "version": "6.0.2",
>       "resolved": "https://registry.npmjs.org/agent-base/-/agent-base-6.0.2.tgz",
>       "integrity": "sha512-RZNwNclF7+MS/8bDg70amg32dyeZGZxiDuQmZxKLAlQjr3jGyLx+4Kkk58UO7D2QdgFIQCovuSuZESne6RG6XQ==",
>       "dependencies": {
>         "debug": "4"
>       },
5296,5298c5299,5301
<       "version": "4.0.0",
<       "resolved": "https://registry.npmjs.org/https-proxy-agent/-/https-proxy-agent-4.0.0.tgz",
<       "integrity": "sha512-zoDhWrkR3of1l9QAL8/scJZyLu8j/gBkcwcaQOZh7Gyh/+uJQzGVETdgT30akuwkpL8HTRfssqI3BZuV18teDg==",
---
>       "version": "5.0.0",
>       "resolved": "https://registry.npmjs.org/https-proxy-agent/-/https-proxy-agent-5.0.0.tgz",
>       "integrity": "sha512-EkYm5BcKUGiduxzSt3Eppko+PiNWNEpa4ySk9vTC6wDsQJW9rHSa+UhGNJoRYp7bz6Ht1eaRIa6QaJqO5rCFbA==",
5300c5303
<         "agent-base": "5",
---
>         "agent-base": "6",
5304c5307
<         "node": ">= 6.0.0"
---
>         "node": ">= 6"
7994a7998,8006
>     "node_modules/puppeteer-core/node_modules/agent-base": {
>       "version": "5.1.1",
>       "resolved": "https://registry.npmjs.org/agent-base/-/agent-base-5.1.1.tgz",
>       "integrity": "sha512-TMeqbNl2fMW0nMjTEPOwe3J/PRFP4vqeoNuQMG0HlMrtm5QxKqdvAkZ1pRBQ/ulIyDD5Yq0nJ7YbdD8ey0TO3g==",
>       "dev": true,
>       "engines": {
>         "node": ">= 6.0.0"
>       }
>     },
8000a8013,8025
>     "node_modules/puppeteer-core/node_modules/https-proxy-agent": {
>       "version": "4.0.0",
>       "resolved": "https://registry.npmjs.org/https-proxy-agent/-/https-proxy-agent-4.0.0.tgz",
>       "integrity": "sha512-zoDhWrkR3of1l9QAL8/scJZyLu8j/gBkcwcaQOZh7Gyh/+uJQzGVETdgT30akuwkpL8HTRfssqI3BZuV18teDg==",
>       "dev": true,
>       "dependencies": {
>         "agent-base": "5",
>         "debug": "4"
>       },
>       "engines": {
>         "node": ">= 6.0.0"
>       }
>     },
11670,11672c11695,11700
<       "version": "5.1.1",
<       "resolved": "https://registry.npmjs.org/agent-base/-/agent-base-5.1.1.tgz",
<       "integrity": "sha512-TMeqbNl2fMW0nMjTEPOwe3J/PRFP4vqeoNuQMG0HlMrtm5QxKqdvAkZ1pRBQ/ulIyDD5Yq0nJ7YbdD8ey0TO3g=="
---
>       "version": "6.0.2",
>       "resolved": "https://registry.npmjs.org/agent-base/-/agent-base-6.0.2.tgz",
>       "integrity": "sha512-RZNwNclF7+MS/8bDg70amg32dyeZGZxiDuQmZxKLAlQjr3jGyLx+4Kkk58UO7D2QdgFIQCovuSuZESne6RG6XQ==",
>       "requires": {
>         "debug": "4"
>       }
14540,14542c14568,14570
<       "version": "4.0.0",
<       "resolved": "https://registry.npmjs.org/https-proxy-agent/-/https-proxy-agent-4.0.0.tgz",
<       "integrity": "sha512-zoDhWrkR3of1l9QAL8/scJZyLu8j/gBkcwcaQOZh7Gyh/+uJQzGVETdgT30akuwkpL8HTRfssqI3BZuV18teDg==",
---
>       "version": "5.0.0",
>       "resolved": "https://registry.npmjs.org/https-proxy-agent/-/https-proxy-agent-5.0.0.tgz",
>       "integrity": "sha512-EkYm5BcKUGiduxzSt3Eppko+PiNWNEpa4ySk9vTC6wDsQJW9rHSa+UhGNJoRYp7bz6Ht1eaRIa6QaJqO5rCFbA==",
14544c14572
<         "agent-base": "5",
---
>         "agent-base": "6",
16649a16678,16683
>         "agent-base": {
>           "version": "5.1.1",
>           "resolved": "https://registry.npmjs.org/agent-base/-/agent-base-5.1.1.tgz",
>           "integrity": "sha512-TMeqbNl2fMW0nMjTEPOwe3J/PRFP4vqeoNuQMG0HlMrtm5QxKqdvAkZ1pRBQ/ulIyDD5Yq0nJ7YbdD8ey0TO3g==",
>           "dev": true
>         },
16654a16689,16698
>         },
>         "https-proxy-agent": {
>           "version": "4.0.0",
>           "resolved": "https://registry.npmjs.org/https-proxy-agent/-/https-proxy-agent-4.0.0.tgz",
>           "integrity": "sha512-zoDhWrkR3of1l9QAL8/scJZyLu8j/gBkcwcaQOZh7Gyh/+uJQzGVETdgT30akuwkpL8HTRfssqI3BZuV18teDg==",
>           "dev": true,
>           "requires": {
>             "agent-base": "5",
>             "debug": "4"
>           }

@googlebot I signed it!

Testing the patched .tgz with https_proxy=http://*/ env set do install successfully.
And for me the proxy setting is in ~/.npmrc like:
noproxy=<IP_ADDRESS>,localhost # exclude localhost
proxy=http://<IP_ADDRESS>:1080 # for http
https-proxy=http://<IP_ADDRESS>:1080 # for https

Thank you for the patch and the detailed tests + logs in the description!


Terminal buffer enters Insert mode if clicked / changed to via  from insert mode buffer

Ref neovim/neovim#7164.
Consider the following script:
vim -u NONE -i NONE -N -s <(echo -n $':terminal sh\n\C-\\\C-n\C-wpi\C-o\C-wp')

After running it Vim enters Insert mode in a terminal:

. In this mode buffer is not updated and trying to type anything results in “E21: Cannot make changes, 'modifiable' is off”.
Same thing if you replace trailing <C-o><C-w>p with a click into the terminal buffer, first setting mouse=a, of course.

Commit did not fix the issue, it made things worse: now Vim segfaults. Only need to add <C-\><C-n>i to the “input” and SEGV is completely automatic, without it -- INSERT -- is still shown.

In terminal-normal mode, :startinsert also enters insert mode.

Takuya Fujiwara wrote:

In terminal-normal mode, :startinsert also enters insert mode.

Well, don't do that then.  It's not that entering Insert mode is
forbidden, it's just not useful.  If someone has 'insertmode' set then
it's unavoidable anyway.  Of course any attempt to change the text will
fail.
--
CRONE:  Who sent you?
ARTHUR: The Knights Who Say GNU!
CRONE:  Aaaagh!  (she looks around in rear) No!  We have no licenses here.
"Monty Python and the Holy editor wars" PYTHON (MONTY) SOFTWARE LTD
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Nikolai Pavlov wrote:

Commit did not fix the issue, it made things worse: now Vim segfaults.
Only need to add <C-\><C-n>i to the “input” and SEGV is completely
automatic, without it -- INSERT -- is still shown.

I cannot reproduce this.  What do you do before that command?
--
FIRST VILLAGER: We have found a witch.  May we burn her?
"Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

@brammool Vim 8.0.956,
vim -u NONE -i NONE -N -s <(echo -n $':terminal sh\n\C-\\\C-n\C-wpi\C-o\C-wp\C-\\\C-ni')

segfaults immediately as I said. You will have to either run it in zsh or replace all \C-char escape sequences with something less readable like \xXX though. E.g. you may use the following (after xxd -r) as the input:
00000000: 3a74 6572 6d69 6e61 6c20 7368 0a1c 0e17  :terminal sh....
00000010: 7069 0f17 701c 0e69                      pi..p..i

Same on 8.0.964, the functions stack at SEGV (using release kind of build):
#0  0x00007f58d88b2ed7 in kill () from /lib64/libc.so.6
#1  0x000000000058f589 in mch_exit ()
#2  0x00000000006d2d4c in getout ()
#3  0x000000000058cd6b in ?? ()
#4  <signal handler called>
#5  0x000000000063afae in term_enter_job_mode ()
#6  0x0000000000567b19 in normal_cmd ()
#7  0x00000000006d293c in main_loop ()
#8  0x00000000006d3f36 in vim_main2 ()
#9  0x000000000043e70f in main ()

Nikolai Pavlov wrote:

@brammool Vim 8.0.956,
vim -u NONE -i NONE -N -s <(echo -n $':terminal sh\n\C-\\\C-n\C-wpi\C-o\C-wp\C-\\\C-ni')

segfaults immediately as I said. You will have to either run it in zsh or replace all \C-char escape sequences with something less readable like \xXX though. E.g. you may use the following (after xxd -r) as the input:
00000000: 3a74 6572 6d69 6e61 6c20 7368 0a1c 0e17  :terminal sh....
00000010: 7069 0f17 701c 0e69                      pi..p..i

Still can't reproduce it.  I used this:
terminal sh
exe "normal \<C-\>\<C-n>\<C-w>pi\<C-o>\<C-w>p\<C-\>\<C-n>i"

--
BEDEVERE: And that, my lord, is how we know the Earth to be banana-shaped.
"Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

@brammool Your variant does not SEGV, but it still gets into insert mode showing that this issue persists. And why you don’t use -s, the xxd -r output should get you the correct test case? It SEGV’s also if you pass it to feedkeys(, "t"):
call feedkeys(":terminal sh\n\<C-\>\<C-n>\<C-w>pi\<C-o>\<C-w>p\<C-\>\<C-n>i", "t")

Nikolai Pavlov wrote:

@brammool Your variant does not SEGV, but it still gets into insert
mode showing that this issue persists. And why you don’t use -s, the
xxd -r output should get you the correct test case? It SEGV’s also
if you pass it to feedkeys(, "t"):
j``VimL
call feedkeys(":terminal sh\n<C-><C-n><C-w>pi<C-o><C-w>p<C-><C-n>i", "t")

Thanks, now I can reproduce it.  I did not expect a difference between
using :normal and feedkeys(), strange.  Looks like a kind of race
condition.
Anyway, it's easy to fix the crash.
--
Are leaders born or made?  And if they're made, can we return them under
warranty?
(Scott Adams - The Dilbert principle)
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


Set STM32 CAN1 filter for use CAN2

Well that's a huge diff for such a basic change :D

Please bring it in order and tag me when done. Thanks!

@Cherish-Gww Looks like your local master branch is not up to date. Checkout master, run git pull <upstream-name>, check out the branch again, and rebase it git rebase master. All you want is this commit: https://github.com/UAVCAN/platform_specific_components/pull/32/commits/c0c632cbe7aa0dc8f44fd8c2999c99f6ead94c6b


Add forgeten helpers call

Without this i can an Uncaught ReferenceError

Cheers for catching this

ah didn't see this when i was merging on my phone but the chart.js is built from all the separate files in src so if you make any changes make them in there, i have added the changes you made to the core file and overlay where this part is located. Looking at it this is also an issue in the main chart.js core so you may want to submit a PR to them as well in their core file


Fix off two issues.

Fix nil pointer error in Helm chart by correcting tls secret reference
Fix incorrect environment variable for enabling POP3

Could you bump the chart version to 3.0.2 also? Otherwise the test pipeline fails (a new chart release is generated on each commit to master).

Done

Thanks!


Remove SceneDelegate

The project can't use SceneDelegate and AppDelegate at the same time. Removed SceneDelegate to make AppDelegate work and pass necessary events to SalemoveAppDelegate (example applicationDidBecomeActive).

Why can't it use both?

Why can't it use both?

Because if we have an "Application Scene Manifest" in  Info.plist and app delegate has a configurationForConnectingSceneSession method, the UIApplication doesn't send background and foreground lifecycle messages to our app delegate - applicationDidBecomeActive (our case), applicationWillResignActive, applicationDidEnterBackground, and applicationWillEnterForeground.

@igorkravchenko  From a short perspective, we should be sure that TestingApp works correctly, however, from a long perspective we should migrate to using SceneDelegate.


pull request for e2e test at 01/14/2017 21:26:13

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[ignore] GitQ testing

Testing how GitQ handles existing GitHub issues. No need to comment. :)

Moved to https://gitq.com/littlevgl/lvgl/topics/2/gitq-testing.


The documentation for v2.0.0 is broken, and server information is exposed because debug mode is enabled.

Link below:
https://junges.dev/documentation/laravel-kafka/v2.0/1-introduction
Screenshots:

It working well now! But please disable the debug mode.


When you switch groups, authorization information is lost

I'm using the latest version of VS2022, and when I created the net6 webapi project, I grouped the interfaces, but when I switched the groupings, the authorization information was lost.
program.cs

my controller

Hi ~ this is my first time submitting issues, there is something wrong please forgive me.

Please provide a minimal reproducible example as a GitHub repository that demonstrates the issue you're experiencing so we can look into this further.


@conda-forge-admin,please add bot automerge

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/sarpy-feedstock#3.


MNT: rerender

Hi! This is the friendly automated conda-forge-webservice.
I've started rerendering the recipe as instructed in #76.
If I find any needed changes to the recipe, I'll push them to this PR shortly. Thank you for waiting!
Here's a checklist to do before merging.

[ ] Bump the build number if needed.

@conda-forge-admin , please re-render


Brother file export

Now I've started extract files from brother 120k disks.
My goal is to convert the extracted files to Richtext with the correct formatings.
To examine this I write a text on WP-1, save the text on disk - read the disk by fluxEngine into my PC and export the files with the brother120tool.
I find out, the extracted files are cut on false position, because I defined a "Headnote" and a "Footnote" - these blocks are before the maintext...
At the start of .img-file there is the directory. Are the 2 bytes at offset 10 (dec.), 26, 42, .. for the position on disk, where the document starts? Track/sector maybe? How to find out then the offset inside the image?

Interesting. Can you sent me the .img file to have a loook at?
The numbers in the directory entry are, as far as I can tell, the sector number of the start of the file. There's an open question as to whether the end is correct, but I thought the start was fine. It's all based on guesswork from a small number of samples I have, so I'm by no means certain it's right...


Can I see the list of available OBD2 Device or a way to pass parameter to connect specific OBD2 device?

Is your feature request related to a problem? Please describe.
When I try to run the BLE OBD2 device I have a couple of devices in range and it gets one of them automatically if I have a way to give a name (Bluetooth Name) so my app tries to connect that specific OBD2 device only.
Describe the solution you'd like
It would be great we can get list of OBD2 devices which is available to connect and I select the one which I want to connect.

Definitely I would get on that today.
the ability to return a list of of devices

I've added the ability to return list of devices have not had the chance to test it yet but I could put it on the dev branch if you want to test it out.

Sure please push it to the dev branch so I can check that out, Do we have a feature to pass a list of OBDII names or specify which OBDII I want to connect?

Sure please push it to the dev branch so I can check that out, Do we have a feature to pass a list of OBDII names or specify which OBDII I want to connect?

I will push it later today I just checked it out lots of issues.


Can't grab chapters.

Now for some reason I am unable to grab chapters for certain novels for example Kenja no Mago web novel through novelupdates as it returns [ERROR]null.

The first few chapters on japtem.com seem to be the problem. Try using headless grabbing.


Add new pgcopy sink

Adding an implementation for pgcopy sink
You can test this with Postgres using:
docker run -e POSTGRES_PASSWORD=spring -e POSTGRES_DB=test -p 5432:5432 -d postgres:latest

Resolves https://github.com/spring-cloud-stream-app-starters/app-starters-release/issues/51

I played with it, did some smoke testing, etc. Looks good, merging.

Squashed/merged per https://github.com/spring-cloud-stream-app-starters/jdbc/commit/a0c76fa49d54f7326eb46d7d8a15ed4902b0980f.
Also linked readme per https://github.com/spring-cloud-stream-app-starters/jdbc/commit/1503d217e9a1a9cbdfbb49dee8a08a5063ba8cf8.


correcting some copy/paste errors, and added link to Edison Setup

Correcting some Copy/Paste errors.
Added the link to the Edison Setup site as I needed that to get the Edison up and an IP Address assigned.

Thanks for these corrections. Merging.


FancyZones - Move newly created windows to their last known zone

ℹ Computer information

PowerToys version:  v0.21.1
PowerToy Utility: FancyZones
Running PowerToys as Admin: yes, always
Windows build number: 10.0.19041.508

📝 Provide detailed reproduction steps (if any)

Shift+Drag Window into a zone (Firefox v82, Thunderbird v78.3.1, Windows Explorer, ...)
close application
start app again

✔ Expected result
Window to be moved and resized to last known position, even after a restart of Windows/PC.
❌ Actual result
window is not in zone anymore and in another position
📷 Screenshots
Thunderbird in left zone before closing

Thunderbird after opening again:

Windows Explorer in zone3 (center)

Windows Explorer after opening again

Windows Explorer, first Window in zone3. Opened another window and that is not in zone3

@AndyK70

Firefox: I can't reproduce the problem, for me it always opens where it closed, even if FZ is not running
Windows Explorer: it's by design to only zone the first window, currently we can't support zoning the subsequent windows opened by the same process
Thunderbird: does the window open completely white? It might be a specific issue with how Thunderbird refreshes the window content.

Firefox and Thunderbird always opens slightly out of the zone, just shifted a bit to the right and a little bit less in height.
Thunderbird opened a white pane in the lower right preview pane because there was no mail selected. Happens by chance once in a while.
So I know Firefox and Thunderbird are some third party apps and maybe their own window handling and theme handling may interfere. That's why I tested it with the windows explorer.
Ok, for the second windows explorer window, but why is the first window after reopening not in the zone?
It was working in in the past, but I don't remember exactly which version it was but it must have been 0.19.xx because after an update to v0.20 I noticed it had stopped working.
In the meantime I got a new SSD and installed a completely fresh windows 10 (2004) and installed PowerToys v0.21.1 - no old windows settings and no old apps to remove or overwrite.
Still doesn't work.
The following test was made with the windows explorer only:

I made sure the FancyZone option "Move newly created..." was checked and closed PowerToys.
Opened PowerToys again, re-checked it was still selected. Tested window behavior and wasn't working.
Cleared the checkmark of that FancyZone option, closed PowerToys, started it again and re-checked the option was not checkmarked.
Then checkmarked the option and tested window behavior again... still windows explorer does not open in the zone but is slightly off position.

Any more logs or information you need to know, please let me know.

@AndyK70

why is the first window after reopening not in the zone?

We don't know what is causing this problem yet, the tracking issue for it is https://github.com/microsoft/PowerToys/issues/5673
I can't reproduce the problem you described for Firefox. Can you provide more information?

DPI scaling of the monitor
screenshot of the zone layout
screenshot of the Firefox window after it has been zoned

Well... I was just doing some screenshots for this issue when a notification e-mail arrived for release v0.23.0
I updated to 0.23.0, still had the same behavior but again cleared the option "Move newly created....", closed PowerToys (tray icon), restarted PowerToys, activated the option again and...
...every window does reopen at it's prior location in the zone again.  🥳
Whether it be Windows Explorer, Thunderbird, Firefox, whatever...
I don't know what changed, I'm totally puzzled. 🤪
If you still want to dig into it let me know, otherwise I'll be happy you closing this issue.

@AndyK70
let's close the issue for now, if the problem reappears, please reopen it.
Thanks.

Your Firefox issue might have something to do with the resistFingerprinting flag. Is that on?

my settings:


cleanup logging, return errors from pkg/ code

This PR does a few minor things:

Code inside pkg/ does not call log.Fatal() anymore, but instead returns errors. Only code in cmd/ exits.
Logging has been improved / harmonized:

Log messages are capitalized.
I removed some seemingly unneccessary empty lines.
When an error happens during a file download, stderr is captured and returned, aiding in debugging.

The cleanup code is idempotent now and does not error out anymore if a resource is already gone. This allows the user to just re-run it "until it works".
The spinner is not stopped twice anymore.

/ok-to-test

/approve
/lgtm


Support for 64bit traceId in Server-Timing response header

Hello, I have a question about tracing and Server-Timing response header format expected by the instrumentation script.
Is there any reason, why the instrumentation does not support 64bit traceIds?
If I understood correctly, the traceId should be valid for b3-propagation (both 64bit and 128bit identifiers are valid as per https://github.com/openzipkin/b3-propagation#traceid).
Current servertiming.ts implementation asks for 32hex traceId identifiers (specifically here and here).
We are not using Splunk agent on the server side for automatic instrumentation of the code and our traceIds are 64bit long. This prevents us effectively from linking traces from server and client as the Splunk RUM does not pick up the Server-Timing header.
Am I missing something? Would it be possible to support shorter traceIds as well?

Hi Tomas -
We're using the nascent standard documented here: https://www.w3.org/TR/trace-context/#traceparent-header, which is explicit in its format - 32 hex chars for the traceId.  Smaller traceIds are fine but need to be left-padded with 0s to fit: https://www.w3.org/TR/trace-context/#interoperating-with-existing-systems-which-use-shorter-identifiers.  Assuming you're trying to do this by rolling your own server-side generator for this feature, you can read all about it at https://github.com/signalfx/splunk-otel-js-web/blob/main/docs/ServerTraceContext.md.  Hope that helps!

Hi @johnbley ,
thanks for the quick reply.
I am a bit confused. This link talks only about sending a Server-Timing header (as it says it is supposed to be enough to link Splunk RUM traces with BE traces). Are we supposed to consume (on the server) the traceparent header that is generated by instrumented client code as well?

No, the Server-Timing response header is all that is needed.


Issue to compile when I use  protos.BluetoothDevice and FlutterBluePlugin.java uses or overrides a deprecated API

I have an IoT Lock and I want to connect it witouth scan. I have the Mac Address.
I need to achieve it without scan.
The steps that I'm trying to do are:
-Starting the app
-Connect to my device
-Retrieve the lock status (opened or closed).
Thanks to all

Yes, but only on android

Yes, but only on android

Thanks, I'm trying to it, a week ago...
Can you post some code of example

@Luiggi13
I have the same problem as you. Is there a good solution for locks?

Yes , it's good, works well, but I need to connect to device by id. I don't want to scan. 'Cause could have some property with N locks.
You have been able to connect directly to only one device?

@Luiggi13 I can connect to the device, I have multiple devices, but I have a problem when unlocking. Can i get a contact for you。Thanks

Yes no problem.
To do something with the lock, you must send bytes like this:
await c.write([0x00, 0xAA, 0x54, 0x45, 0x53, 0x54],                           withoutResponse: true)
gmail<EMAIL_ADDRESS>

Yes, but only on android

@boskokg can you write me some basic code for connect to a specific device.id???
I would be appreciated
Thanks a lot

@Luiggi13 this is part of the code for that purpose:
if (Platform.isAndroid) {
  protos.BluetoothDevice dev = protos.BluetoothDevice()..remoteId = address;
  _gattClient = BluetoothDevice.fromProto(dev);

  if (!_stateListeners.containsKey(_gattClient)) {
    _stateListeners[_gattClient] = _gattClient.state
        .skip(1)
        .listen((BluetoothDeviceState newState) => _onConnectionStateChange(_gattClient, newState));
  }
  _gattClient.connect(autoConnect: BluetoothHelper.everConnectedDevicesAddresses.contains(address));
}

Basically, you need this:
protos.BluetoothDevice dev = protos.BluetoothDevice()..remoteId = address;
_gattClient = BluetoothDevice.fromProto(dev);

Basically, you need this:
protos.BluetoothDevice dev = protos.BluetoothDevice()..remoteId = address;
_gattClient = BluetoothDevice.fromProto(dev);

Interesting... thanks, I'll try it

protos.BluetoothDevice dev = protos.BluetoothDevice()..remoteId = address;
_gattClient = BluetoothDevice.fromProto(ddev);

How to fix it? It's my first time with it! 👍

import 'package:flutter_blue/gen/flutterblue.pb.dart' as protos;

import 'package:flutter_blue/gen/flutterblue.pb.dart' as protos;

@boskokg thanks a lot, I'll try it. Sorry for the messages!

It's working thanks, man!!!!!!!!

Hi people, the code works properly but now I'm trying to compile apk, and I have this issue.
I've been check this issue too: https://github.com/pauldemarco/flutter_blue/issues/141
`Running Gradle task 'assembleRelease'...
Running Gradle task 'assembleRelease'... Done                      70,8s
The built failed likely due to AndroidX incompatibilities in a plugin. The tool is about to try using Jetfier to solve the incompatibility.
Building plugin flutter_blue...
Running Gradle task 'assembleAarRelease'...
Running Gradle task 'assembleAarRelease'... Done                    1,4s

Configure project :
The com.google.protobuf plugin was already applied to the project: : and will not be applied again after plugin: android-library
*********************************************************
WARNING: This version of flutter_blue will break your Android build if it or its dependencies aren't compatible with AndroidX.
See https://goo.gl/CP92wY for more information on the problem and how to fix it.
This warning prints for all Android build failures. The real root cause of the error may be unrelated.
*********************************************************

FAILURE: Build failed with an exception.

What went wrong:
Task 'assembleAarRelease' not found in root project 'flutter_blue'.

Try:
Run gradlew tasks to get a list of available tasks. Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 1s
The plugin flutter_blue could not be built due to the issue above.`
Somebody know how can fix it?

More info
Note: /Users/bebito/development/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_blue-0.7.2/android/src/main/java/com/pauldemarco/flutter_blue/FlutterBluePlugin.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: /Users/bebito/Downloads/demoflutter/kiploc/build/flutter_blue/generated/source/proto/release/javalite/com/pauldemarco/flutter_blue/Protos.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
Removed unused resources: Binary resource data reduced from 483KB to 474KB: Removed 1%
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':system_setting:verifyReleaseResources'.

A failure occurred while executing com.android.build.gradle.internal.tasks.Workers$ActionFacade
Android resource linking failed
/Users/bebito/.gradle/caches/transforms-2/files-2.1/a528b13ac93e64cafa3d0480e2c93207/core-1.1.0/res/values/values.xml:142:5-173:25: AAPT: error: resource android:attr/fontVariationSettings not found.

 /Users/bebito/.gradle/caches/transforms-2/files-2.1/a528b13ac93e64cafa3d0480e2c93207/core-1.1.0/res/values/values.xml:142:5-173:25: AAPT: error: resource android:attr/ttcIndex not found.

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 1m 11s
Running Gradle task 'assembleRelease'...
Running Gradle task 'assembleRelease'... Done                      77,0s
The built failed likely due to AndroidX incompatibilities in a plugin. The tool is about to try using Jetfier to solve the incompatibility.
Building plugin flutter_blue...
Running Gradle task 'assembleAarRelease'...
Running Gradle task 'assembleAarRelease'... Done                    1,6s
*********************************************************
WARNING: This version of flutter_blue will break your Android build if it or its dependencies aren't compatible with AndroidX.
See https://goo.gl/CP92wY for more information on the problem and how to fix it.
This warning prints for all Android build failures. The real root cause of the error may be unrelated.
*********************************************************
FAILURE: Build failed with an exception.

What went wrong:
Task 'assembleAarRelease' not found in root project 'flutter_blue'.

Try:
Run gradlew tasks to get a list of available tasks. Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 1s
The plugin flutter_blue could not be built due to the issue above.

Just must to update minSdkVersion and targetSdkVersion like this and works:
    minSdkVersion 19
    targetSdkVersion 29

This should no longer work with version 30.
Cannot roll back since the the latest version on the repository uses 30 in 6 different places and will break with 29.
Please help


Use templated HTML instead of inline HTML strings.

Use templated html instead of strings.
Refer to Google's Docs

Solved in #7


br: the graph service must be restarted after the restoration process

After restoration, on nebula-console, I can see the new graph space ("nba") from the backup file. But when I ran USE nba, no graph space was found. I restarted the graph service and then all the data can be queried as expected.

Hello, this problem is caused by the synchronization of schema information and meta, you can wait for a period of time and then the schema synchronization can be used.


Remove FashionVictim DEMARK or peg econ-ark dependency to 0.10.2

The FashionVictim DEMARK imports code from HARK:
https://github.com/econ-ark/DemARK/blob/master/notebooks/Fashion-Victim-Model.py#L52
After this, it appears to execute the exact same code that is in the Fashion-Victim-Model main() method:
https://github.com/econ-ark/DemARK/blob/master/notebooks/Fashion-Victim-Model.py#L123
https://github.com/econ-ark/HARK/blob/master/HARK/FashionVictim/FashionVictimModel.py#L414
In other words, this DEMARK is just a duplicate of what's in HARK, with some copy-and-pasted code.
FashionVictim is slated to be taken out of the HARK source library and into a HARK examples/ directory.
https://github.com/econ-ark/HARK/issues/440
It has been argued that since FashionVictim is really a kind of tutorial documentation, not material for economics students, it's better for it to be in examples/ than to be a DemARK.
Either:

The DemARK should be removed, or,
the DemARK econ-ark dependency should be pegged to version 0.10.2 so this DemARK can keep depending on HARK.

If we decide on the latter for some reason, I hope we put some thought into how to design this better to avoid code duplication across repositories.

Let's make FashionVictim an "example" notebook, and remove it from the DemARK.

@sbenthall will integrate this with https://github.com/econ-ark/HARK/issues/446

Fixed with https://github.com/econ-ark/DemARK/commit/e4a30aabde9f318dc86fd87753f63032dc5bc862#diff-c2365e874ecb861a659e5215ff17c338


Azure Provider Alpha Release

Make all CRDs in provider-tf-azure use new configuration toggles to implement all XRM features.

Thoughts on a mini bug bash for this Azure provider too, before its alpha release: https://github.com/crossplane-contrib/provider-tf-aws/issues/23#issuecomment-926080527


Guards fixed on tree simulations for clads.

Guards fixed on tree simulations for clads0,1,2.

If lambda =0, lambda > max, or if lambda-mu > max, error messages are given and no tree is printed to file.
With the relevant prior settings these simulators (plus the crbd ones) should be ready to use now.

Plus bug fixes in clads0 model from before.

Added simulators for AnadsGBM 0-2 now, but these needs further testing and correct prior settings.

Hej Emma,
Sorry for taking so long to review this. I've just pushed some code to this PR.
There is one new file and changes to a file.
The new file is
phywppl/tree_simulations/treesim-anadsGBM2-viktor.wppl
I suggest to use it as a template for the simulators. Here is what I've changed:

I reworked how the error conditions are handled: I made the simulations exit immediately upon encountering the error condition (right after leftTree is parsed) to improve speed, and changed the output.
I fixed the algorithmic issues that we discussed two weeks ago.
Played around with guards.
Small "improvements" to code style: input parameters, changes to handling strings and variables throughout, and so on.

Maybe you can view this file and your version side by side and then adapt the other simulators?
Also I tweaked npm run wppl. It is highly parallel now, but in the end of the day there is a fundamental limitation of shelljs on the buffer size of shelljs.exec, so that larger trees are truncated. This is why now I actually run the .js file from the js folder, and use npm run wppl as a compiler.
Also I noticed that since we "hijacked" npm run wppl for the simulators the semantics with "input tree" (one of the default arguments) does not work any more, and the Bisse32 tree kept being overwritten if one is not careful with the command line arguments. I put some comments to that extent in the beginning of my new file, but to make sure it doesn't happen, I erased the DEFAULT_TREE setting in npm run wppl. Now the system is a little broken semantically, but it works. Maybe we should redesign it at some point.
I am going to merge this with my main branch (not squash or rebase). I have also made some other changes to main. You don't need to rebase, only pull my main to continue work.