Docker Hub repository

Thank you for your IDEAs and software.
Did u consider pushing your image to docker hub?
I think you would have even more visibility through low installation effort
Thank you again!
Best regards

Honestly, I have not considered this before, is a good idea, thanks! Do you have any doc or something to see the path what do I need to follow to do this?

I created this repository on my dockerhub.
Just for test purposes:
cloudcsv-cloud-csv-web-1
I'm posting you some docs. (Careful: I'm newbie!!)
https://docs.docker.com/get-started/04_sharing_app/
By the way...While registering a new user I get one error (present also in your web demo)

Thank you!

Awesome! thanks!
About the error, you have the API up ?

in the local version....
actually after starting up the docker for the API i get this message
C:\Users\p.fiorini\cloudcsv_api>docker-compose up
time="2023-10-25T15:13:23+02:00" level=warning msg="network traefik-network: network.external.name is deprecated. Please set network.name with external: true"
[+] Building 0.0s (0/0)                                                                                docker:default
network traefik_proxynet declared as external, but could not be found

Hello again!
,
googling for similar error I got this:
https://community.traefik.io/t/issue-using-external-networks/2689/8
What do u think?
Thank you!

Cool thanks. I think the network can't stop the startup of the API, If you open the console inside the browser, and then go to the network tab, you can see the failed request and see the details, please send me this detail.

Uh,. I think i cannot reach that console....
It should be in the API, if I well understand?

I cannot start that
Thank u

Ohh, perfect, today I can check this. Thanks for the information, I really appreciate it ❤

I Updated the docs and compose on the API, you can try again

Hello!
the updated docker compose command works!
i get this now

But trying to create new user i get this error

Is there any default user?
Thank you again!!!

Now you can access to the network inside the console of the browser, please let me see what going on in that section, if you see a response in red color, please select this and take a screenshot

Ouch!
please gimme how to access on that console...
On localhost:8017 No buttons beyond login/register...

Oh no problem.

Open you browser
Press F12 or rigth click on the current open page and then click on inpect elements

Now you see a console like this

then retry send the register action, you can see a request with register name

Here an example video https://www.youtube.com/watch?v=y2yanWTNxGk

Hello,
this might help?

Here go! Now click on the POST red request called register and then click on preview to see what going on

if I click on POST

(My browser is Mozilla for Windows...I don't see any "Preview")

Ok thanks! I have more context now, I work on that this weekend.

Sorry for the delay @fewrainbows I test the API and I found a missing command to install the dependencies of the API, basically I added this command to the API docs make install-dependencies you need to pull the changes and run the commant.


[libwebp] :x64-linux build failure - Could not find PowerShell Core;

Host Environment

Linux tesseract-ocr-1 4.15.0-124-generic #127-Ubuntu SMP Fri Nov 6 10:58:37 UTC 2020 aarch64 aarch64 aarch64 GNU/Linux

To Reproduce
Steps to reproduce the behavior:
./vcpkg install libwebp:x64-linux
Failure logs
Starting package 1/4: libwebp:x64-linux
Building package libwebp[core,nearlossless,simd]:x64-linux...
Could not locate cached archive: /home/ubuntu/.cache/vcpkg/archives/ef/efac22b7bce83ac121636c473c910f96935ab7ba.zip
-- Using cached /home/ubuntu/vcpkg/downloads/webmproject-libwebp-d7844e9762b61c9638c263657bd49e1690184832.tar.gz
-- Cleaning sources at /home/ubuntu/vcpkg/buildtrees/libwebp/src/1690184832-82bbc619db.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source /home/ubuntu/vcpkg/downloads/webmproject-libwebp-d7844e9762b61c9638c263657bd49e1690184832.tar.gz
-- Applying patch 0001-build.patch
-- Applying patch 0002-cmake-config-add-backwards-compatibility.patch
-- Applying patch 0003-always-mux.patch
-- Applying patch 0004-add-missing-linked-library.patch
-- Applying patch 0006-fix-dependecies-platform.patch
-- Applying patch 0007-fix-arm-build.patch
-- Applying patch 0008-sdl.patch
-- Applying patch 0009-glut.patch
-- Using source at /home/ubuntu/vcpkg/buildtrees/libwebp/src/1690184832-82bbc619db.clean
WebP currently requires the following library from the system package manager:
    Xxf86vm

This can be installed on Ubuntu systems via apt-get install libxxf86vm-dev
-- Configuring x64-linux-dbg
-- Configuring x64-linux-rel
-- Building x64-linux-dbg
-- Building x64-linux-rel
CMake Error at scripts/cmake/vcpkg_copy_tool_dependencies.cmake:22 (message):
  Could not find PowerShell Core; please open an issue to report this.
Call Stack (most recent call first):
  ports/libwebp/portfile.cmake:76 (vcpkg_copy_tool_dependencies)
  scripts/ports.cmake:135 (include)

Error: Building package libwebp:x64-linux failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `./vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: libwebp:x64-linux
  Vcpkg version: 2020.06.15-unknownhash

Additional context
Got the error trying to build tesseract and leptonica.
Tried installing powershell following instructions in https://docs.microsoft.com/en-us/powershell/scripting/install/installing-powershell-core-on-linux?view=powershell-7.1 but that did now install it.

Works with latest version of vcpkg as checked just now.
(base) ubuntu@tesseract-ocr-1:~/vcpkg$ ./vcpkg install libwebp:x64-linux
Environment variable VCPKG_FORCE_SYSTEM_BINARIES must be set on arm, s390x, and ppc64le platforms.
(base) ubuntu@tesseract-ocr-1:~/vcpkg$ export  VCPKG_FORCE_SYSTEM_BINARIES=1
(base) ubuntu@tesseract-ocr-1:~/vcpkg$ ./vcpkg install libwebp:x64-linux
Computing installation plan...
The following packages will be built and installed:
    libwebp[core,nearlossless,simd]:x64-linux -> 1.1.0#1
Detecting compiler hash for triplet x64-linux...
Using cached binary package: /home/ubuntu/.cache/vcpkg/archives/fe/fea8e17d4296297e23a01149b700bbe4c95343af.zip
Starting package 1/1: libwebp:x64-linux
Building package libwebp[core,nearlossless,simd]:x64-linux...
Building package libwebp[core,nearlossless,simd]:x64-linux... done
Installing package libwebp[core,nearlossless,simd]:x64-linux...
Installing package libwebp[core,nearlossless,simd]:x64-linux... done
Elapsed time for package libwebp:x64-linux: 16.57 ms

Total elapsed time: 673.4 ms

The package libwebp:x64-linux provides CMake targets:

    find_package(WebP CONFIG REQUIRED)
    target_link_libraries(main PRIVATE WebP::webp WebP::webpdemux WebP::libwebpmux WebP::webpdecoder)

(base) ubuntu@tesseract-ocr-1:~/vcpkg$

(base) ubuntu@tesseract-ocr-1:~/vcpkg$ ./vcpkg install libwebp:x64-linux
Computing installation plan...
The following packages will be built and installed:
    libwebp[core,nearlossless,simd]:x64-linux -> 1.1.0#1
Detecting compiler hash for triplet x64-linux...
Could not locate cached archive: /home/ubuntu/.cache/vcpkg/archives/fe/fea8e17d4296297e23a01149b700bbe4c95343af.zip
Starting package 1/1: libwebp:x64-linux
Building package libwebp[core,nearlossless,simd]:x64-linux...
-- Downloading https://github.com/webmproject/libwebp/archive/d7844e9762b61c9638c263657bd49e1690184832.tar.gz -> webmproject-libwebp-d7844e9762b61c9638c263657bd49e1690184832.tar.gz...
-- Extracting source /home/ubuntu/vcpkg/downloads/webmproject-libwebp-d7844e9762b61c9638c263657bd49e1690184832.tar.gz
-- Applying patch 0001-build.patch
-- Applying patch 0002-cmake-config-add-backwards-compatibility.patch
-- Applying patch 0003-always-mux.patch
-- Applying patch 0004-add-missing-linked-library.patch
-- Applying patch 0006-fix-dependecies-platform.patch
-- Applying patch 0007-fix-arm-build.patch
-- Applying patch 0008-sdl.patch
-- Applying patch 0009-glut.patch
-- Using source at /home/ubuntu/vcpkg/buildtrees/libwebp/src/1690184832-82bbc619db.clean
WebP currently requires the following library from the system package manager:
    Xxf86vm

This can be installed on Ubuntu systems via apt-get install libxxf86vm-dev
-- Configuring x64-linux-dbg
-- Configuring x64-linux-rel
-- Building x64-linux-dbg
-- Building x64-linux-rel
-- Installing: /home/ubuntu/vcpkg/packages/libwebp_x64-linux/share/libwebp/copyright
-- Performing post-build validation
-- Performing post-build validation done
Stored binary cache: /home/ubuntu/.cache/vcpkg/archives/fe/fea8e17d4296297e23a01149b700bbe4c95343af.zip
Building package libwebp[core,nearlossless,simd]:x64-linux... done
Installing package libwebp[core,nearlossless,simd]:x64-linux...
Installing package libwebp[core,nearlossless,simd]:x64-linux... done
Elapsed time for package libwebp:x64-linux: 21.03 s

Total elapsed time: 21.59 s

The package libwebp:x64-linux provides CMake targets:

    find_package(WebP CONFIG REQUIRED)
    target_link_libraries(main PRIVATE WebP::webp WebP::webpdemux WebP::libwebpmux WebP::webpdecoder)

(base) ubuntu@tesseract-ocr-1:~/vcpkg$


Add js file to main property

I'm installing shepherd with NPM by listing it as a dependency in my package.json
"dependencies": {
"shepherd": "git://github.com/HubSpot/shepherd.git#58a37fa0263bf205fff7a64abb62216b1738e0c2",
},

Webpack and Browserify would require the main js file to be listed in the main property, instead it is listed in spm.main. I'm not using spm
For now I've decided to keep both, to not break spm support.

Shouldn't have closed it, as most package.json files should use the main property. Feel free to merge.

This is a near duplicate of a merged pull-request on tether, so I'd imagine this should be fine to merge as well.

Sorry for the delay, and thanks for the update.


Piper/extend benchmark suite

Builds on #106
What was wrong?
#106 introduced a benchmark suite however it was very limited, only containing two tests.  add(1, 2) and thunk().
Ideally we want something more comprehensive that covers operations that properly emulate real world execution.
How was it fixed?
The EWASM team pointed me at https://github.com/fluencelabs/fluence/tree/master/bench/vm
I ported over a few of their benchmarks.

recursive keccak hashing for depths of 1, 10, 100
recursive fibonacci calculation of the series numbers 1, 11, 17, 38
factorization of prime numbers 1117, 9973,<PHONE_NUMBER>

Each of these is actually a rust implementation compiled to wasm.
Cute Animal Picture

Yes, you did review one, but it only benchmarked a thunk and single add(1, 2) operation.  I added these once I started into the deeper optimizations and realized the existing benchmarks were not useful for much beyond benchmarking meta stuff, but that they'd never really show performance of the actual execution.


[system_log_error] Distinguish azk error from sub-system log error

This pull request closes #267 issue.
More info in #267.
eg:

LGTM :+1:

ok to test

ok to test

ok to test


Document some trouble shooting steps and probot-stale behavior

After configuring the probot-stale integration, I learned several things that weren't documented very well, or were difficult to find. Documentation should be added to the README to help people troubleshoot:

The stale probot runs every hour.
When it runs each hour, the time it starts is randomly determined.
"stale" issues doesn't mean just when a user comments on an issue. It can also mean when a milestone is applied. It has to do with the updated search item.
When the bot is first started (or the integration is installed, depending on your setup method), and the config file is already in place, the bot will not comment on things right away. However, if the config file is pushed up after the bot starts, it will start searching for issues immediately. This is true for configuration file changes as well.

This issue is similar to the request/discussion in #34, but includes a couple of other things I found helpful once I understood how everything worked. This is also related to the questions posed in #33.

I'll try to get some of these items added to the README file soon.

All great additions! Is there a way to force the first round of commenting to happen immediately without pushing up a new config? Would be useful for development :)

@stkent Not that I know of.
@bkeepers Is that something that could be added with flag or something?

Ok, I added documentation for these items in PR #41.


dhcp start/stop made compatible for multiple interfaces

The net_dhcpv4_stop() function stops the event listener for any IF_UP events previously. In case multiple interfaces are used and optionally being switched over, this could result in disabling dhcp unintentionally.
Signed-off-by: Vincent van der Locht<EMAIL_ADDRESS>

Codecov Report

Merging #11476 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master   #11476      +/-   ##
==========================================
+ Coverage   48.38%   48.38%   +<.01%
==========================================
  Files         265      265
  Lines       42193    42192       -1
  Branches    10137    10137
==========================================
  Hits        20413    20413
+ Misses      17703    17702       -1
  Partials     4077     4077

Impacted Files
Coverage Δ

subsys/net/ip/dhcpv4.c
6.18% <100%> (+0.01%)
:arrow_up:

Continue to review full report at Codecov.

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

Based on @tbursztyka comment I've updated the pull-request.
The callback is initialized at the init function and added/removed in the start/stop function based on the interface list being empty or not (first added, last removed)
The event handler checks if the interface is in the list before acting on it.

@tbursztyka @jukkar Would it be possible to take a look at this pull request. Based on the given comments, I hopefully updated the commit. If not, please let me know.

My 2 cents: I find the commit description here unclear:

The net_dhcpv4_stop() function stops the event listener for any IF_UP events previously.

I may imagine a word (participle) is missing here, or perhaps I don't understand the meaning of this sentence completely.

In case multiple interfaces are used and optionally being switched over, this could result in disabling dhcp unintentionally.

I don't understand what "optionally being switched over" means here. The usages of "switch over" I know are also usually followed with "to"/"from".
My point is that I couldn't match the description of the patch with the changes included in it (even if I understand the DHCP code well, which I don't). So, relying on @jukkar/@tbursztyka for review.


px4_sitl_default temporarily disable lockstep scheduler

Attempting to temporarily disable px4_sitl_default lockstep scheduler until we can address https://github.com/PX4/Firmware/issues/11380 and https://github.com/PX4/Firmware/issues/11384.
@julianoes can this be done Firmware side alone?

I think this needs some more settings.

Getting close to an actual fixing.


[Manual] - Not all comments are added to a post

Commit number: c445ad3
Preconditions: test user is signed in
Steps to reproduce:

Create a new post
Add one comment to the post
Set the application to offline mode
Add 3 comments to the same post
Set the application to on-line mode
Wait for data synchronization with the server
Re-open the post

Actual Result: Incorrect number of comments is displayed for the post (sometimes 3 or 2 comments instead of 4)
Expected Result: The post has correct number of comments - 4 comments

The problem is still active on a build from commit number 7f3d09256e9b78b6e4cd0207ad47b75f4691d65e

Verified on a build from commit number 36e50c38f654045f6c392fb0e87c647953544606


Bump version to 1.0.1

Issue #, if available:
Description of changes:
Prepare for release
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #83 (1941c51) into main (4486f9f) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main      #83   +/-   ##
=======================================
  Coverage   94.26%   94.26%
=======================================
  Files          38       38
  Lines         977      977
  Branches      143      143
=======================================
  Hits          921      921
  Misses         35       35
  Partials       21       21

Flag
Coverage Δ

unittests
94.26% <ø> (ø)

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


Error with line 28 in digital_clock.py

@geekcomputers  Please do check the error in the line 28 of digital_clock.py

i updated the code thanks


fix(nala): repo and key not matching upstream changes

Resolves: https://github.com/wimpysworld/deb-get/issues/1098

Thanks! That should fix our issue.
BTW - in testing I ran across this problem with the installed nala :
nala update
Traceback (most recent call last):
  File "/usr/bin/nala", line 5, in <module>
    from nala.__main__ import main
  File "/usr/lib/python3/dist-packages/nala/__main__.py", line 31, in <module>
    import nala.fetch as _fetch  # pylint: disable=unused-import
  File "/usr/lib/python3/dist-packages/nala/fetch.py", line 36, in <module>
    import typer
  File "/usr/lib/python3/dist-packages/typer/__init__.py", line 12, in <module>
    from click.termui import get_terminal_size as get_terminal_size
ImportError: cannot import name 'get_terminal_size' from 'click.termui' (/usr/lib/python3/dist-packages/click/termui.py)

for every invocation, including bash auto-complete ... (on up-to-date  Ubuntu Jammy)

nala update
Traceback (most recent call last):
[...]
ImportError: cannot import name 'get_terminal_size' from 'click.termui' (/usr/lib/python3/dist-packages/click/termui.py)

This can be fixed by downgrading the click package back to the ubuntu version with
sudo apt install --reinstall python3-click=8.0.3-1


Feature/wysiwyg editor

This adds the ability to enable rich text formatting using a WYSIWYG editor.

Nice work 😄 Could you post a screenshot into this PR of what this looks like too, please? Both the WYSIWYG editor, and how a formatted post can look.

@AndreasElia - I updated the code formatting per your requests. In preparing the screenshot I also realize Tailwind "resets" most built in formatting, so I added in an unreset for the rich text view to allow that back in.
Screenshot of formatting and editor in action:

Good stuff 👍  Do you think toolbarOptions should be in the config file, and kept minimal by default (e.g. only bold, underline, italic and lists by default)?

Please see my comment and question above (https://github.com/AndreasElia/laravel-firefly/pull/46#discussion_r666394393) regarding the toolbar / config. I am on-board with those as the defaults

Replied to that thread, but just to repeat that the default toolbar options should be minimal as it is a forum rather than a CMS.

Here is the updated UI with the simpler toolbar (and 1 added custom button, to add font color)

When reviewing with my team one of my teammates called out that there should be some XSS protection added in.
Added a commit which strips script tags from the submission. quill seems to handle this from the front-end OK, but a clever hacker could bypass it, and submit to the back-end

When reviewing with my team one of my teammates called out that there should be some XSS protection added in.
Added a commit which strips script tags from the submission. quill seems to handle this from the front-end OK, but a clever hacker could bypass it, and submit to the back-end

Nice work @Arkitecht 👍


parameter "keysize" unused in org.bouncycastle.cms.bc.BcCMSContentEncryptorBuilder.CMSOutputEncryptor#CMSOutputEncryptor

Hi,
this is a minor thing, bur there is a discrepacy between the JCE and Lighweight CMSOutput encryptors:
in lightweigt the keysize paarameter is ignored and size is taken from given OID while in jce (org.bouncycastle.cms.jcajce.JceCMSContentEncryptorBuilder.CMSOutputEncryptor#CMSOutputEncryptor) this behavior is present for negative values of keysize.
Is there a usecase for setting the keysize to something not in keeping with the given algorithm?
If not I propose to remove the superfluous keysize parameter in lightweigtht.
Cheers,
Matthias

Closed as usage now established in both.


Feat/load referenced config from packagejson

Feat: load referenced json config file from package.json 'jest' when it has a valid path
Summary
Currently Jest requires json config files to be in root folder of projects. Allowing to load from another path brings way to keep projects organized. That way files can be moved to another folder includind these who are only for tests
Test plan
yarn jest packages/jest-config/src/tests/resolveConfigPath.test.ts

You can use --config CLI flag to point Jest to a config file. It does work from command line or as a script in package.json: "yarn jest --config path/to/jest.config.json".
Do you have a use case there that isn’t enough?

While one can specify the path via CLI, I think it makes sense for the jest key can be a path to config.
It should support any file type and not just json, though. Does this PR do that? Essentially working as a --config path.

You can use --config CLI flag to point Jest to a config file. It does work from command line or as a script in package.json: "yarn jest --config path/to/jest.config.json".
Do you have a use case there that isn’t enough?

Yes, as it's a flexibility case to keep runners and package.json a little clean and with better readability. The jest config file is an way to reduce CLI commands complexity including their own path.

@rafaelrabelos could you take a look at the failing CI? 🙂

@rafaelrabelos could you take a look at the failing CI? 🙂

CI fails solved 🙂

https://github.com/jestjs/jest/releases/tag/v30.0.0-alpha.3


#161 Fix issue with override params not being used

Hopefully this should fix issue #161

This fix looks good, we need it approved ASAP and deployed to VSTS.

Closing PR as this has been taken care as part of different PR and @alondero confirmed for closure of issue.


Revert "HADOOP-18676. Fixing jettison vulnerability of hadoop-common lib"

Reverts apache/hadoop#5507
Exclusions must be added to hadoop-client

already reverted. thx

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
1m 27s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  3s

codespell was not available.

+0 :ok:
detsecrets
0m  3s

detect-secrets was not available.

+0 :ok:
xmllint
0m  3s

xmllint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

-1 :x:
mvninstall
57m 40s
/branch-mvninstall-root.txt
root in trunk failed.

+1 :green_heart:
compile
36m 28s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

-1 :x:
compile
31m 14s
/branch-compile-root-jdkPrivateBuild-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09.txt
root in trunk failed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09.

-1 :x:
mvnsite
0m 43s
/branch-mvnsite-hadoop-common-project_hadoop-common.txt
hadoop-common in trunk failed.

+1 :green_heart:
javadoc
1m 48s

trunk passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
0m 59s

trunk passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

-1 :x:
shadedclient
158m 33s

branch has errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 22s

the patch passed

-1 :x:
compile
7m  7s
/patch-compile-root-jdkUbuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1.txt
root in the patch failed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1.

-1 :x:
javac
7m  7s
/patch-compile-root-jdkUbuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1.txt
root in the patch failed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1.

+1 :green_heart:
compile
27m 55s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

-1 :x:
javac
27m 55s
/results-compile-javac-root-jdkPrivateBuild-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09.txt
root-jdkPrivateBuild-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09 with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09 generated 131 new + 2492 unchanged - 0 fixed = 2623 total (was 2492)

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
2m 15s

the patch passed

+1 :green_heart:
javadoc
1m  0s

the patch passed with JDK Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
0m 40s

the patch passed with JDK Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

+1 :green_heart:
shadedclient
28m 59s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
18m 19s

hadoop-common in the patch passed.

+1 :green_heart:
asflicense
0m 54s

The patch does not generate ASF License warnings.

248m  6s

Subsystem
Report/Notes

Docker
ClientAPI=1.42 ServerAPI=1.42 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5512/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5512

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient codespell detsecrets xmllint

uname
Linux 8c80f689a9cc 4.15.0-206-generic #217-Ubuntu SMP Fri Feb 3 19:10:13 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 2d64bd01eddce871a055c95c90d1be162fa5d62c

Default Java
Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.18+10-post-Ubuntu-0ubuntu120.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_362-8u362-ga-0ubuntu1~20.04.1-b09

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5512/1/testReport/

Max. process+thread count
1353 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5512/1/console

versions
git=2.25.1 maven=3.6.3

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.


Attributes are null if default attributes are used

If default attribute values are set in properties, then the attributes continue to be null even when ready() is called.
Shouldn't the attributes be initialized with the default values?

Attributes that are set up in properties should be initialized by the time the ready() callback is called. Could you post a repro code snippet?

Well, they are displayed when the custom element is used in an HTML file. However, when I try to test the default attributes, I get a null unless they're explicitly set.
custom-element: http://pastebin.com/YcwFHABs
test suite: http://pastebin.com/LiaBikgJ

Your custom element seems correct except I don't think you have to define attributes within the dom-module tag in Polymer 1.0 - published properties are defined inside properties. The element's default values are definitely working; you can add
console.log(this._url, this._label, this._active);

inside the ready() callback and you will see the defined defaults when no attributes are passed in.
I can only assume it's a problem with the unit test..... which I'm unfamiliar with so I can't help you there.

Hmm. Have you tried using the getAttribute() function within the ready() function of the custom element? That yielded a null too.

You mean this.getAttribute()? That should be expected behaviour. If you call your custom element without passing anything in,
index.html
<navbar-button></navbar-button>

navbar-button.html
...
properties: {
  _label: { type: String, value: "Google" }, // default value if nothing is passed in
  ...
},
ready: function () {
  console.log(this._label); // ............................(1)
  console.log(this.getAttribute("_label")); // ............(2)
},
...

naturally (1) shows "Google" and (2) shows null (because you didn't actually pass in any data)
If your index.html is
<navbar-button _label="Foobar"></navbar-button>

(1) shows "Foobar" and (2) shows "Foobar".
If your index.html is
<navbar-button _label="Google"></navbar-button>

(1) shows "Google" and (2) shows "Google".

My bad, wasn't aware that it was expected behavior. Sorry about that.
You mean this.getAttribute()? That should be expected behaviour. If you
call your custom element without passing anything in,
index.html

navbar-button.html
...
properties: {
_label: { type: String, value: "Google" }, // default value if
nothing is passed in
...
},ready: function () {
console.log(this._label); // ............................(1)
console.log(this.getAttribute("_label")); // ............(2)
},
...
naturally (1) shows "Google" and (2) shows null (because you didn't
actually pass in any data)
If your index.html is

(1) shows "Foobar" and (2) shows "Foobar".
If your index.html is

(1) shows "Google" and (2) shows "Google".
—
Reply to this email directly or view it on GitHub
https://github.com/Polymer/polymer/issues/1909#issuecomment-113084223.

@torcellite Properties set in the properties object are not by default reflected as attributes on the custom element itself. If you'd like to reflect the property value as an attribute you can set reflectToAttribute: true on the property settings:
properties: {
  label: {type: String, value: "Google", reflectToAttribute: true}
}

See this jsbin for more info: http://jsbin.com/yanuto/edit?html,console,output
One thing to be aware of is that camel-cased property keys are automatically set to hyphenated attribute keys - e.g. myProperty becomes <x-element my-property> as attributes are case-insensitive.
Thanks @zerodevx!

Just a tiny bump. If the data type is Boolean, getAttribute() returns null if false and an empty string (i.e. not a null) if true.


AVRO-3676: Produce valid toString() for UUID JSON

What is the purpose of the change
In addition to https://github.com/apache/avro/pull/1204, this pull request wraps UUIDs in quotes to produce a valid JSON.
Verifying this change
Added test that validates UUID toString() behaviour
Documentation

Does this pull request introduce a new feature? no

Cherry-picked to branch-1.11.
I also created a JIRA to track this change: AVRO-3676 -- The ASF JIRA unfortunately needed to disable public signups -- would you like us to create an account for you for contributor credit?

Thank you @RyanSkraba. It would be great if you could create an account for me on JIRA.
Noted regarding data representation using .toString(). I use it as a trick for tests only where I assert some big Avro JSONs to the expected JSON in some tests.
Originally I was using JsonEncoder to generate the Avro JSON, however it was a bit tricky to assert fields with logicalType decimal and timestamp as they get encoded as byte and long and it is not immediately clear the conversion when reading the test.


Added one guide for learning flexbox in css.

Added one guide for learning flexbox in css.

@Swpn0neel Please review my PR.

@Swpn0neel Please review my PR.

@DhanushNehru Can you please review my PR?


How can I have my personalized menu?

How can I have my personalized menu?

make your own. This helped me the most:
https://www.youtube.com/watch?v=fwqKJRJBwQY&t


Make injection system fast as lightning

Lastly, we removed the json serializer/deserializer for setting/getting reflection data from future injected classes, and we gain some significant speed.
There's a last step left to transform our injector in a formula1. We should remove the useless transformation from InjectorClassVO to ClassDescription at runtime (with all the processes/calculations to build sub value objects hierarchy: ConstructorInjectionVO instance, PropertyInjectionVO instances, MethodInjectionVO instances and OrderedInjectionVO instances with all ArgumentInjectionVO instances). This could be handled directly by a macro at compile time.
To achieve that, we need to transform all the value object classes to typedef and extract all the behaviors (that apply injections) to functional static methods with inlining. It's a big/long refactoring, but it's a very valuable one, because it means there won't be anymore the process described above for building reflection data during the 1st access. It will be ready to use directly. And no need to cache anymore as well, because this cache will be directly built at compile time...

This issue was moved to DoclerLabs/hexMachina#48


Show data usage

Maybe can show data usage of targets ? like wifikill app on android

Hi there @boblerasta. Shouldn't this be an issue instead of a pull req? I don't think you've made any amendments to our code. If you wish, please make this into an issue and add some more details. Thanks!


Rename core grpc_jwt_credentials to grpc_service_account_jwt_access_credentials.

We want to be consistent with https://github.com/anthmgoogle/google-auth-library-java/blob/master/oauth2_http/java/com/google/auth/oauth2/ServiceAccountJwtAccessCredentials.java.

closing


removed unneeded network config

Description
removed unneeded network config

@H34D I did not get it working without setting the ips. The ethereum enode format (https://github.com/ethereum/wiki/wiki/enode-url-format) requires an IPv4. DNS are not supported.
So for parity config files (https://github.com/oceanprotocol/docker-images/blob/develop/parity/secret_store/config/config.toml) we need those ips.


Soundpad Plugin

** Name of the plugin **
SoundPadPlugin
** Version **
0.1.0-beta
** Author of the plugin **
PhoenixWyllow
** Description of the plugin **
An Soundpad integration for Macro Deck
** Link to the icon **
Icon
** GitHub repository **
https://github.com/PhoenixWyllow/MacroDeck.SoundPad
** Dependencies **
Macro Deck version:
2.6.4-dev

Closed. found bugs.

fixed issues, working ok now.


eval sourcemap with package ships with standalone sourcemap emit a lot of warning in browser

Bug report

What is the current behavior?
Bundle codes with eval sourcemap, but reference to a package that ships with standalone sourcemap will not convert standalone sourcemap to eval sourcemap, this will emit a lot of warning in browser console.
If the current behavior is a bug, please provide the steps to reproduce.

Import a package that ships with standalone sourcemap
Set devtool: 'eval' in webpack.config.js
Start with dev server
Check browser console

What is the expected behavior?
Standalone sourcemap in package should be converted to eval sourcemap or truncated

Other relevant information:
webpack version:  4.39.3 (Not able to tryout latest due typescript types issue)
Node.js version: 12.13.0
Operating System: Ubuntu 18.04.3
Additional tools:

Hi @sokra. Is there a way to avoid installing one more package? I don't care about this sourcemaps. Everything worked fine before this release https://github.com/webpack/webpack/releases/tag/v4.41.5

@TrySound It's not about webpack, it's about Chrome:

We report more errors than previously, which is a WONTFIX.

https://bugs.chromium.org/p/chromium/issues/detail?id=1052872#c14

bump, I think we should improve it out of box

I add this loader to solve the problem
module.exports = source => {
  return source.replace(/\n[^\n]*sourceMappingURL[^\n]*\n?/, '\n');
};

Yep, we need an option to import default behavior of existing source maps, possible situations:

remove them if you don't need it (no warnings)
handle them (move source-map-loader into core, logic it very simple, so it should be not hard)
keep as is (maybe useful in some cases)

I think we can move this into the core.

Low priority, if anyone want to help, PR welcome

Activate

Still a problem


Relace adapter(-test) blueprint with jsonapi-adapter blueprint

Blueprints now use jsonapi- prefix: ember generate jsonapi-resource its-name

Doesn't overwrite the default blueprints
Uses the same file structure (removes prefix jsonapi-from naming conventions)

Should fix #44 once all the blueprints have been changed to use jsonapi- prefixed blueprints and are updated using the current ember-cli code (1.13.12) for detecting whether using pod structure or not.

Awesome! I'll try this out soon. I'm back in development after a couple month break :smirk:


Updates to marketing site

[x] Make version number bump dynamic
[ ] Make wallpaper use mojave dark and light

Make wallpaper use mojave dark and light

Need any help?


591 duplicate keywords

I solved the problem of the duplicate keywords in the auto-generated documentation by allowing users to choose which version to generate: DCU or UEFI. After the fix the amout of duplicate keywords is 0

@EduKav1813 you should probably rebase instead of merge especially if PR has only couple of commits as currently you introduced another commit. If you merge this PR then you will have yet another merge commit

@EduKav1813 your last batch of commits is unsigned

@EduKav1813 your last batch of commits is unsigned

Thanks, I fixed that and now my commits are properly signed

Looks good, maybe squash some commits before merge

I prefer to squash the entire branch as one merge commit, would this be ok?

@EduKav1813 yeah, it's ok.


Support other remote repos (Gitlab etc)

First, I must says I love the project and I hope it is still actively maintained, keep up the good work and I'd be happy to help.
Basically the title : allow for a way to use other remote repos.
More specifically, options could provide a way to give any type of url and specify options to tell gd-plug how to install them.
E.g., point to a bash script etc.

Thanks!
gd-plug does support all the git url, for example:
    # Source other than github
    plug("https://gitlab.com/Xecestel/sound-manager") # Gitlab
    plug("file:///D/Godot/local-project/.git") # Local git repo

It is shown in More Example, maybe I should add it to Config/Sources


ElementFinder documentation mistype

Hello!
I was going over some documentation for the ElementFinder and I think you have an incomplete description for the getText function.

Thanks.
-Ben

Huh - it works when you click through to the full function. Something in our website parser might be acting funny.

Looking into this: We are using a shortDescription by looking for when a sentence ends. The easy fix is to change the getText JSDoc.
https://github.com/angular/protractor/blob/9177dca445e8a16363a7df5328886688b7027180/website/js/api-controller.js#L142


Handle empty key array

Fix for issue #126
Allows for where conditions inside an array to be filtered before joining them together

Hold on, I see a function called filtersToSql

Added the key filter to the filterStr if keys.length > 0

Force pushed on my fork to delete a commit and consolidate

@DylanPetrey Great find and fix – thank you. (I made a small addition to add some whitespace to your clause.)


Add freemarker role for unix playbooks

Signed-off-by: Joe deKoning<EMAIL_ADDRESS>

Could we move the cleanup to the cleanup role?

Cleanup moved to cleanup role.

This is not the cause.  This adds freemarker to the /home/$jenkins_user location via ansible script.  When sourceforge was unavailable for a few days, I thought adding locally controlled version a good idea.  And better yet, since the file version never changes add it to the machine so as to not increase the time of a compile job with machine setup actions.

@jdekonin You're totally correct and my apologies!


ui: The window is not opened in the current window

This happens when:

The monitor is a secondary monitor
The secondary monitor is upper/left side of the primary monitor

And it looks like the window size is not correct when the display scales are different.

Fixed


Action on Failure Fix

The documentation lies, here's the error message I just got from CloudFormation:
"CloudFormation currently only supports long-running clusters, set ActionOnFailure to CANCEL_AND_WAIT or CONTINUE"

Just confirmed with the CloudFormation team, it's a typo in the docs

Thanks!


opengm python wrapper page doesn't open

Hi,
The page http://hciweb2.iwr.uni-heidelberg.de/opengm isn't working. Could anybody point me to an alternative source for downloading? Thanks

Should work again. Server was down for a unknown reason.


pinpoint web display invalid agents

Prerequisites
Please check the FAQ, and search existing issues for similar questions before creating a new issue.YOU MAY DELETE THIS PREREQUISITES SECTION.

[x] I have checked the FAQ, and issues and found no answer.

What version of pinpoint are you using?
master/v2.3.0
Describe the bug
A clear and concise description of what the bug is.
pinpoint web display invalid agents

There are only four running agents，but the web shows 12 agents
What did you do to trigger the bug?
In the method of AgentInfoServerInstanceListDataSource class
The filterAgentInfos method is as follows. When querying the agent status, agentinfos is used to query and return the agentStatusList of the corresponding size. When filtering the running agent, agentsToCheckStatus is used to compare with the query result agentStatusList. The two lists are inconsistent in size, so the same index cannot be used for comparison. This will lead to comparison errors, because the returned filteredAgentInfos will have more abnormal agents
Expected behavior

A clear and concise description of what you expected to happen.
The code for bug elimination and modification is as follows

Screenshots
If applicable, add screenshots to help explain your problem.
Logs
If applicable, please attach agent/collector/web DEBUG log that includes the code execution that led to the bug. In case of agents, including the start-up log may be of great help.
Additional context
Add any other context about the problem here, such as affected library for agents, how your collector/web/hbase is set up if applicable.

Hello, @wenyiwener.
The commented issue is related to #8967 and the issue is currently working.
Your bug report is reasonable, but some agents will appear in some internal policies (we are working on this to unify the policy)
So I want to notify you that even after we applied your bug hot-fix, some agents can still appear in the server map agents list.
However, it seems the issue is quite major, so I will apply the hotfix.
Thank you for your attention to our project.

It's my honor to help Pinpoint a little, and thank the Pinpoint team for continuous optimization to unify the policy


Add upfront grant to Smart Escrow

Before we were going to handle the upfront grant tokens separately, because we wanted to delegate these and start voting before the 1 year cliff. However, given the 1 year cliff is approaching and some product changes impacting the ability to partially delegate, we'll have both upfront and collab grant tokens handled via the smart escrow. The only material change to the contract is that there is an additional 10,737,418 OP that should vest upon the cliff (ie part of "initialTokens").  Making this part of the same task rather than a new one just so reviewers can easily see the diff

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/1

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
0

Sum
1


Improve formatting and linting

@Powell-v2 is working on this.  The plan:

Use Prettier to replace js-beautify.
Change the ESLint configuration to 2-space indents rather than 4.
Modify format.sh to use npx rather than installing new packages each time.
Set up a git precommit hook to run format.sh on every commit, on just the modified files.

Link to the PR => #1378


[backport] Remove overhead in creation/basic.py

Backport of #1342

Jenkins, test this please.

Jenkins CI test (for commit 340c86c33e154bf2367d2b6ddf40e643dc12674c) succeeded without errors!

LGTM!

Jenkins CI test (for commit 340c86c33e154bf2367d2b6ddf40e643dc12674c) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)


Arch arm cortex m direct dynamic irqs

DIrect Dynamic Interrupt Feature support for ARM Cortex-M

@ioannisg please rebase

@ioannisg please rebase

@carlescufi rebased :)

@ioannisg there seems to be an issue with shippable. Perhaps it just needs a rebase.

@ulfalizer @andrewboie could you re-visit your reviews, here, so we could resolve this one?

Please change the Kconfig to have "depends on" and not "select". Users will need to enable both.

Please change the Kconfig to have "depends on" and not "select". Users will need to enable both.

That's alright then, @andrewboie , I' ve done that now, so using this feature requires both Kconfigs be set, thanks for looking at it.


Envoy translates localhost to "::1" when no IPv6 stack available

Title: Envoy translates localhost to "::1" when no IPv6 stack available
Description:
Hi!
I am trying to get an Envoy configuration to support IPv4 and IPv6 stacks. In a first approach, I only wanted to test that the same configuration is valid for pure IPv4 and IPv6 hosts (not needed dualstack).
First I checked that you were introducing IPv6 support recently, and you already have some openened tickets that maybe represents my issue (#1005, for example).
My scenario is as following:

Envoy uses grpc envoy.ext_authz filter to authoritate and then routes to a cluster depending on some rules.
Envoy, auth server and clusters are all running in the same container.
Using Envoy v1.11.0

So, my first attempt was to bind the listener to "::" and use strict_dns in the auth server and clusters, all of them pointing to "localhost" to let the DNS resolver to select "<IP_ADDRESS>" or "::1" depending on the stack selected.
Config:
This is the Envoy configuration I am using:
node:
  id: "1"
  cluster: test

admin:
  access_log_path: /admin_access.log
  address:
    socket_address: { address: "::", port_value: 9901, ipv4_compat: true }
static_resources:
  listeners:
  - name: listener_0
    address:
      socket_address:
        address: "::"
        port_value: 8888
        ipv4_compat: true
    filter_chains:
      - filters:
        - name: envoy.http_connection_manager
          config:
            codec_type: auto
            stat_prefix: ingress_http
            access_log:
              - name: envoy.file_access_log
                config:
                  path: "/access.log"
            rds:
               route_config_name: local_route
               config_source:
                  path: /etc/envoy/rds.yaml
            http_filters:
              - name: envoy.ext_authz
                config:
                  grpc_service:
                    envoy_grpc:
                      cluster_name: auth-server
                    timeout: 3s
                  clear_route_cache: true
                  with_request_body:
                    max_request_bytes: 2048
                    allow_partial_message: true
              - name: envoy.router
dynamic_resources:
  cds_config:
    path: "/etc/envoy/cds.yaml"

The content of the Cluster section would be like this:
version_info: "0"
resources:
  - "@type": type.googleapis.com/envoy.api.v2.Cluster
    name: auth-server
    connect_timeout: 5s
    type: strict_dns
    http2_protocol_options:  {}
    load_assignment:
      cluster_name: auth-server
      endpoints:
      - lb_endpoints:
        - endpoint:
            address:
              socket_address:
                address: "localhost"
                port_value: 50051
                ipv4_compat: true

Logs:
With this configuration, in a container that only supports IPv4, Envoy translates "localhost" to "::1" so, the connection to auth-server fails (as it is only accepting connections from IPv4 stack):
[2019-08-27 17:16:35.968][313][trace][http] [source/common/http/http1/codec_impl.cc:366] [C2] parsing 78 bytes
[2019-08-27 17:16:35.968][313][trace][http] [source/common/http/http1/codec_impl.cc:479] [C2] message begin
[2019-08-27 17:16:35.968][313][debug][http] [source/common/http/conn_manager_impl.cc:246] [C2] new stream
[2019-08-27 17:16:35.968][313][trace][http] [source/common/http/http1/codec_impl.cc:334] [C2] completed header: key=Host value=<IP_ADDRESS>:8888
[2019-08-27 17:16:35.968][313][trace][http] [source/common/http/http1/codec_impl.cc:334] [C2] completed header: key=User-Agent value=curl/7.60.0
[2019-08-27 17:16:35.968][313][trace][http] [source/common/http/http1/codec_impl.cc:445] [C2] headers complete
[2019-08-27 17:16:35.968][313][trace][http] [source/common/http/http1/codec_impl.cc:334] [C2] completed header: key=Accept value=*/*
[2019-08-27 17:16:35.968][313][trace][http] [source/common/http/http1/codec_impl.cc:466] [C2] message complete
[2019-08-27 17:16:35.968][313][debug][http] [source/common/http/conn_manager_impl.cc:600] [C2][S17340692137313621699] request headers complete (end_stream=true):
':authority', '<IP_ADDRESS>:8888'
':path', '/'
':method', 'GET'
'user-agent', 'curl/7.60.0'
'accept', '*/*'

[2019-08-27 17:16:35.968][313][debug][http] [source/common/http/conn_manager_impl.cc:1092] [C2][S17340692137313621699] request end stream
[2019-08-27 17:16:35.968][313][trace][filter] [source/extensions/filters/http/ext_authz/ext_authz.cc:72] [C2][S17340692137313621699] ext_authz filter calling authorization server
[2019-08-27 17:16:35.968][313][debug][router] [source/common/router/router.cc:401] [C0][S6842458295467923966] cluster 'auth-server' match for URL '/envoy.service.auth.v2.Authorization/Check'
[2019-08-27 17:16:35.968][313][debug][router] [source/common/router/router.cc:514] [C0][S6842458295467923966] router decoding headers:
':method', 'POST'
':path', '/envoy.service.auth.v2.Authorization/Check'
':authority', 'auth-server'
':scheme', 'http'
'te', 'trailers'
'grpc-timeout', '3000m'
'content-type', 'application/grpc'
'x-envoy-internal', 'true'
'x-forwarded-for', '<IP_ADDRESS>'
'x-envoy-expected-rq-timeout-ms', '3000'

[2019-08-27 17:16:35.969][313][debug][client] [source/common/http/codec_client.cc:26] [C3] connecting
[2019-08-27 17:16:35.969][313][debug][connection] [source/common/network/connection_impl.cc:702] [C3] connecting to [::1]:50051
[2019-08-27 17:16:35.969][313][debug][connection] [source/common/network/connection_impl.cc:715] [C3] immediate connection error: 99
[2019-08-27 17:16:35.969][313][debug][http2] [source/common/http/http2/codec_impl.cc:726] [C3] setting stream-level initial window size to 268435456
[2019-08-27 17:16:35.969][313][debug][http2] [source/common/http/http2/codec_impl.cc:748] [C3] updating connection-level initial window size to 268435456
[2019-08-27 17:16:35.969][313][debug][pool] [source/common/http/conn_pool_base.cc:20] queueing request due to no available connections
[2019-08-27 17:16:35.969][313][trace][router] [source/common/router/router.cc:1393] [C0][S6842458295467923966] buffering 317 bytes
[2019-08-27 17:16:35.969][313][trace][http] [source/common/http/conn_manager_impl.cc:857] [C2][S17340692137313621699] decode headers called: filter=0x2a80000 status=4
[2019-08-27 17:16:35.969][313][trace][http] [source/common/http/http1/codec_impl.cc:387] [C2] parsed 78 bytes
[2019-08-27 17:16:35.969][313][trace][connection] [source/common/network/connection_impl.cc:288] [C2] readDisable: enabled=true disable=true
[2019-08-27 17:16:35.969][313][trace][connection] [source/common/network/connection_impl.cc:456] [C3] socket event: 2
[2019-08-27 17:16:35.969][313][debug][connection] [source/common/network/connection_impl.cc:467] [C3] raising immediate error
[2019-08-27 17:16:35.969][313][debug][connection] [source/common/network/connection_impl.cc:188] [C3] closing socket: 0
[2019-08-27 17:16:35.969][313][debug][client] [source/common/http/codec_client.cc:82] [C3] disconnect. resetting 0 pending requests
[2019-08-27 17:16:35.969][313][debug][pool] [source/common/http/http2/conn_pool.cc:149] [C3] client disconnected
[2019-08-27 17:16:35.969][313][debug][router] [source/common/router/router.cc:868] [C0][S6842458295467923966] upstream reset: reset reason connection failure
[2019-08-27 17:16:35.969][313][debug][http] [source/common/http/async_client_impl.cc:91] async http request response headers (end_stream=true):
':status', '200'
'content-type', 'application/grpc'
'grpc-status', '14'
'grpc-message', 'upstream connect error or disconnect/reset before headers. reset reason: connection failure'

[2019-08-27 17:16:35.969][313][trace][filter] [source/extensions/filters/http/ext_authz/ext_authz.cc:218] [C2][S17340692137313621699] ext_authz filter rejected the request with an error. Response status code: 403
[2019-08-27 17:16:35.969][313][debug][http] [source/common/http/conn_manager_impl.cc:1167] [C2][S17340692137313621699] Sending local reply with details ext_authz_error
[2019-08-27 17:16:35.969][313][debug][http] [source/common/http/conn_manager_impl.cc:1359] [C2][S17340692137313621699] encoding headers via codec (end_stream=true):
':status', '403'
'date', 'Tue, 27 Aug 2019 15:16:35 GMT'
'server', 'envoy'

[2019-08-27 17:16:35.969][313][trace][connection] [source/common/network/connection_impl.cc:392] [C2] writing 97 bytes, end_stream false
[2019-08-27 17:16:35.969][313][trace][connection] [source/common/network/connection_impl.cc:288] [C2] readDisable: enabled=false disable=false
[2019-08-27 17:16:35.969][313][debug][pool] [source/common/http/http2/conn_pool.cc:171] [C3] destroying primary client

This is the log where envoy is trying to send the auth qyery:
[2019-08-27 17:16:35.969][313][debug][connection] [source/common/network/connection_impl.cc:702] [C3] connecting to [::1]:50051

I was a little surprised so, I used nslookup to figure out how the container was translating the "localhost" dns and no "::1" was returned:
# nslookup localhost
Server:		<IP_ADDRESS>
Address:	<IP_ADDRESS>#53

Non-authoritative answer:
Name:	localhost.mycompany.com
Address: <IP_ADDRESS>

Even, I checked /etc/hosts to be sure any IPv6 address was there:
# cat /etc/hosts
<IP_ADDRESS>	localhost
<IP_ADDRESS>	a931aae25c13

Not sure if this is an issue related to the environment I am using for the test (docker containers), so I would like to ask you if you have any clue if Envoy should translate the "localhost" properly depending on the stack or if this a current limitation.

Probably coming from https://github.com/moby/moby/issues/33099
@kikogolk  Does your docker version match the issue above?

For those trying to run a Sidecar in AWS ECS Fargate with envoy - (not in AWS AppMesh)
This issue is the reason that why you might be getting this error immediate connection error: 99
I ran an envoy sidecar, version envoyproxy/envoy:v1.14.2, and after setting the address to <IP_ADDRESS>, in clusters[].hosts[].socket_address, the connection occurred normally

For those trying to run a Sidecar in AWS ECS Fargate with envoy - (not in AWS AppMesh)
This issue is the reason that why you might be getting this error immediate connection error: 99
I ran an envoy sidecar, version envoyproxy/envoy:v1.14.2, and after setting the address to <IP_ADDRESS>, in clusters[].hosts[].socket_address, the connection occurred normally

can confirm i too had this issue and the above fixed it.

Another +1 for me as well. I lost a couple days on this, thinking my kubernetes wiring was junk... I clearly did not suspect that using localhost would be my source of failure.  It wasn't until I shelled into the container and used curl along w/ trace logging on envoy that I noticed the ::1 causing my pain.  Replacing localhost w/ <IP_ADDRESS> fixed it right up.

I've been looking into IPv6 within Envoy and from my understanding this issue primarily stems from Envoy's default DNS resolution preference which is to prefer IPv6 and to fall back to IPv4. This is set by the DnsLookupFamily field on the cluster and changing this field will affect the IP address returned by the DNS resolver. In this case of localhost it will prefer ::1. As others have suggested you could use <IP_ADDRESS> directly or you could change the DNS resolution preference. I tested with the cluster configuration below and was able to observe things behaving as expected.
clusters:
  - name: local_service
    type: STRICT_DNS
    dns_lookup_family: V4_ONLY
    lb_policy: ROUND_ROBIN
    load_assignment:
      cluster_name: local_service
      endpoints:
      - lb_endpoints:
        - endpoint:
            address:
              socket_address:
                address: "localhost"
                port_value: 8080


Tile 9219 marked unsure: the left half of petal 1 fibers are flagged

We're accepting these and moving on.  @julienguy , this doesn't look to me like the normal kind of cosmic ray overscan contamination case though.  It looks to me like there's a normal CR without hysteresis there, though, that might be causing issues?

(i.e., @dstndstn , please mark as good tomorrow despite the one flagged CCD)

Done!


"Dataset?" - quizzical expression on user's face

When there’s one dataset in a collection and the breadcrumbs are constrained, it looks like there’s an errant “Dataset”. Clicking on it does nothing for example. This may perplex a user.

compared to:

@hthomas-czi to review and recommend updates as needed.

@brianraymor After thinking on this, it seems like the best solution is to simply hide the Dataset selector under the conditions you documented:

The collection contains only one dataset
The breadcrumbs are width-constrained

Closing in favor of a more comprehensive review of all emerging feedback collected in Polishing the seamless experience.


feat: add custom event ‘changeddirection’

Summary of changes

Asana Ticket: Schedules | Schedule Finder default direction doesn't match user selection
Establishes a new custom event.
Adds a helper function for dispatching this event to communicate the changed direction across disparate react trees.
The useDirectionChangeEvent hook sets up the event listener on the document and outputs the updated direction after said custom event is fired.

General checks

[x] Are the changes organized into self-contained commits with descriptive and well-formatted commit messages? This is a good practice that can facilitate easier reviews.
[x] Testing. Do the changes include relevant passing updates to tests? This includes updating screenshots. Preferably tests are run locally to verify that there are no test failures created by these changes, before opening a PR.
[x] Tech debt. Have you checked for tech debt you can address in the area you're working in? This can be a good time to address small issues, or create Asana tickets for larger issues.

This feels like a lot of overhead for what—if I'm understanding properly—feels like a case of improper state management. Let me know if I'm missing important things here, but it seems like ScheduleLoader defines a "directionId" state (readjustedDirectionId), which it passes down to both ScheduleDirection and ScheduleFinder. But then ScheduleDirection copies that into its own state, and when a change action happens it modifies its local state. So then the problem is that ScheduleFinder relying on the higher-level, ScheduleLoader state isn't aware of the changes.
I read the solution here as (more or less) making another local copy of the directionId state and adding extra events to synchronize from one local state to the other. What if, instead, we got rid of all local states? So the only concept of directionId that ScheduleDirection and ScheduleFinder have is what is passed into them. And when there's a change direction action in ScheduleDirection, it bubbles that up to a handler in ScheduleLoader which updates the single source of truth. So essentially, trying to better follow the "data down, actions up" principle. Does that seem reasonable, or am I missing important context?


Changes in TF Data Augmenter to match 5.8 performance and accuracy

Passes on all platforms and accuracy tested!


Multiple apps with same code - setup apple_id and app_identifier dynamically

I have tried with this in both fastlane file and Appfile. In the fastlane file I get undefined method apple_id error and in the Appfile it is like the code is never called, even though I have tried to include it in the platform og lane specific configuration.
How to I get my method set_context executed in Appfile or how do I set apple_id and app_identifier in the fastlane file?
I have looked at the SunApps setup which is a bit similar, but I do not get their setup with folders. Is all the files duplicated in each folder? I see that it might be needed to have metadata and screenshots in folders per app, but I guess that path is configurable in those modules, so the I only need the have the module files in one location - right?
Code:
def set_1_apple_account
  apple_id<EMAIL_ADDRESS>end
def set_2_apple_account
  apple_id<EMAIL_ADDRESS>end

def set_context
  puts "Write the app name:"
  app_name = gets.chomp

  case app_name
  when "MyAppName"
    app_identifier "my.app.id"
    set_1_apple_account
  when "MyAppName2"
    app_identifier "my.app.id2"
    set_2_apple_account
  else
    raise "Context is not setup for this app in fastfile - add the app to the switch statement in method 'set_context'"
  end
end

The apple_id cannot be defined in the Appfile. Instead you only provide the bundle identifier (as you already do) and the apple_id will be determined automatically :+1:

But my problem is that I can't get set_contextcalled in the Appfile at all. puts "Write the app name:"   app_name = gets.chompis never called.
How to call it?
And what about the case of creating a new app - the the apple_id needs to be set!?
Is there another way it is possible to use the set_contextin Fastfile?

set_context is never called, you can just add it to your Appfile:
def set_1_apple_account
  apple_id<EMAIL_ADDRESS>end
def set_2_apple_account
  apple_id<EMAIL_ADDRESS>end

def set_context
  puts "Write the app name:"
  app_name = gets.chomp

  case app_name
  when "MyAppName"
    app_identifier "my.app.id"
    set_1_apple_account
  when "MyAppName2"
    app_identifier "my.app.id2"
    set_2_apple_account
  else
    raise "Context is not setup for this app in fastfile - add the app to the switch statement in method 'set_context'"
  end
end

set_context

As I stated in the first comment I have tried that, also inside platform and lane configuration scopes. And i have tried exactly what you suggest but the putsand gets.chompis never called.
I have also tried a puts as the first line in the file but that is not called either.
How to get this working?

I have ended handling it in the Fastfile and creating my own Environment variables to set data in.
Do you have any way to get the version number from the Xcode target?
If any one are interested here is my code:
def set_context
    apps = JSON.parse(open("http://url").read)
    i = 1
    puts "Select app:"
    apps.each do |app|
      title = app["Title"]
      puts "#{i}) #{title}"
      i += 1
    end
    app_index = gets.chomp
    app = apps.at(app_index.to_i()-1)

    ENV["APP_IDENTIFIER"] = app["AppPublishId"]
    ENV["TARGET"] = app["XcodeTarget"]
    ENV["APPLE_ID"] = app["AppleId"]
    ENV["APP_VERSION_NUMBER"] = "1.0.0" #TODO get from xcode

    title = app["Title"]
    say "Acount context for #{title} set with success"
  end

Do you mean something like this?

Yes exactly - thanks
When I have this:
SNAPSHOT_BREAK_ON_FIRST_ERROR=1 snapshot --noclean
I get this error:
`eval': (eval):26: syntax error, unexpected tIDENTIFIER, expecting keyword_end (SyntaxError)
It should be supported..

You cannot just add terminal commands in your Fastfile. Instead you can call them by using sh "command here" or by using Ruby code to execute other tools. For example:
snapshot(noclean: true)
```

Okay - thanks


Use on Windows

How to install the driver on windows and connect to the cluster. The cpp driver documentation shows a server with artefacts. For windows I downloaded the zip with the dll and lib. ok, what should I do with it?
Now I get the fatal error LNK1181: cannot open input file cassandra.lib.
As the linker cannot find the library

Hi @g-codx - do you also have the dependencies? They are adjacent to the Cassandra driver on the DataStax server, e.g., https://datastax.jfrog.io/artifactory/cpp-php-drivers/cpp-driver/builds/2.16.2/90df2c9/windows/dependencies/ . You need libuv, OpenSSL, and zlib, as well as the Cassandra driver itself. I don't have experience of using the driver on Windows myself, but I expect that you can put the DLL and LIB files in the root directory of the crate you are building, or anywhere on your PATH.


Caption

Have Captions been depreciated?
Txt2AL still creates them but the compiles does not like them

They most certainly have not been removed. This should work. Could you tell me which version of vs extension you have?

I think the extension on the marketplace has not been updated since september. Is that correct? Do I have to pull the extension from the Azure machine?
When are you going to keep the marketplace in synch with the updates?

Yes, it looks like you have a too old version. I'll get back to you for the other questions. Try to get a newer version from the azure instance in the mean time.

Trying that.  Nothing is accessible in the way you would expect.

Hi Mark,
The VS Code extension present on the marketplace is a version that is in sync with the Cloud Sandboxes.
The Developer Preview builds are ahead and that's why you need to use a pre-release version of the extension that is bundled with the docker images.

So what will be the extension for NAV2018? Surely not mandatory via Docker I hope.

For NAV 2018 the VSIX file will be copied into program files folder by the NAV installer.

That would be perfect! Is that already in next weeks CTP17?

@DominikDitoIvosevic AL version 0.11.14434. We're running the October build with Docker.

Could you check on December Update and let us know if you're still running into this issue?


amortizing/caching "suggest" in CI or for first runs

When you cargo vet and it fails we automatically compute and emit vet suggest. A "first run" of suggest is incredibly expensive and it only gets more expensive the more audits you do. We mitigate this cost by having a diff-cache stored in the user's tmp, but CI is generally configured to work from a "clean slate" without explicit instructions to do otherwise. We could of course just ignore this but it seems like we'll need a better answer in the longterm.
Related but potentially distinct is just how slow a "first run" is for a local user.
There are two things that merit being cached:

The fetches of packages to be diffed
The diff-cache, which is just a simple json file containing the diff-stats of different package-version-pairs

We already have one mechanism for "save and restore" of the cache via the --diff-cache=path/to/my_diff_cache.json. We will try to read this file, and then write back all our computed diff-stats to it (creating it if it doesn't exist). So you can get some modicum of persistent caching by manually using --diff-cache and saving it somewhere between runs. Unfortunately we do not currently have a mechanism for "merging" or "locking" it so concurrency of individual runs is an open question.
We don't have the same functionality for the entire tmp cache dir (where the fetches are) but we could.
We don't have a mechanism for some "online global cache" that first-time-users can pull from, but we could.

So first, I think a "first run" is only incredibly slow in the case where you actually invoke cargo vet suggest. If you just get the suggestions from cargo vet after pulling in an unvetted dependency to a previously-passing tree (the likely situation on CI), only a handful of packages need to be fetched.
That said, some CI setups (including Mozilla's) aren't allowed to hit the network at all. So I think it makes the most sense for --locked to just not do the suggestions. It will tell you which crates are failing, and then you'd go run cargo vet locally to get the recommendations.

This is actually relevant to the current workstream of getting this running in Mozilla CI, so marking this as P1.


LLaVA import error on CPU

getting an import error while using LLaVA model.
command:
python generate.py --repo-id-or-model-path liuhaotian/llava-v1.6-34b --image-path-or-url 'https://llava-vl.github.io/static/images/monalisa.jpg'

the LLaVA directory has:

env check:
PYTHON_VERSION=3.11.9
transformers=4.36.2
torch=2.1.2+cpu
ipex-llm Version: 2.1.0b20240605
IPEX is not installed.
CPU Information:
Architecture:                       x86_64
CPU op-mode(s):                     32-bit, 64-bit
Address sizes:                      52 bits physical, 57 bits virtual
Byte Order:                         Little Endian
CPU(s):                             128
On-line CPU(s) list:                0-127
Vendor ID:                          GenuineIntel
Model name:                         Intel(R) Xeon(R) Gold 6430L
CPU family:                         6
Model:                              143
Thread(s) per core:                 2
Core(s) per socket:                 32
Socket(s):                          2
Stepping:                           7
CPU max MHz:                        3400.0000
CPU min MHz:                        800.0000
BogoMIPS:                           3800.00
Total CPU Memory: 125.489 GB
Operating System:
Ubuntu 22.04.4 LTS \n \l

Linux ceed-user 5.15.0-107-generic #117-Ubuntu SMP Fri Apr 26 12:26:49 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
CLI:
Version: <IP_ADDRESS>31126
Build ID: 00000000
Service:
Version: <IP_ADDRESS>31126
Build ID: 00000000
Level Zero Version: 1.11.0
Driver UUID                                     32332e32-322e-3236-3531-362e33340000
Driver Version                                  23.22.26516.34
Driver Version                                  2<IP_ADDRESS>.12_195853.xmain-hotfix
Driver Version                                  2<IP_ADDRESS>.12_195853.xmain-hotfix
Driver related package version:
ii  intel-fw-gpu                                     2023.39.2-255~22.04                     all          Firmware package for Intel integrated and discrete GPUs
ii  intel-i915-dkms                                  <IP_ADDRESS>.230425.56+i81-1               all          Out of tree i915 driver.
ii  intel-level-zero-gpu                             1.3.26516.34-682~22.04                  amd64        Intel(R) Graphics Compute Runtime for oneAPI Level Zero.
ii  level-zero-dev                                   1.11.0-649~22.04                        amd64        Intel(R) Graphics Compute Runtime for oneAPI Level Zero.
env-check.sh: line 167: sycl-ls: command not found
igpu not detected
xpu-smi is properly installed.
+-----------+--------------------------------------------------------------------------------------+
| Device ID | Device Information                                                                   |
+-----------+--------------------------------------------------------------------------------------+
| 0         | Device Name: Intel(R) Data Center GPU Flex 170                                       |
|           | Vendor Name: Intel(R) Corporation                                                    |
|           | SOC UUID: 00000000-0000-0000-6cf6-5109f1c50433                                       |
|           | PCI BDF Address: 0000:ae:00.0                                                        |
|           | DRM Device: /dev/dri/card1                                                           |
|           | Function Type: physical                                                              |
+-----------+--------------------------------------------------------------------------------------+
GPU0 Memory size=16M
02:00.0 VGA compatible controller: ASPEED Technology, Inc. ASPEED Graphics Family (rev 52) (prog-if 00 [VGA controller])
DeviceName: Onboard VGA
Subsystem: ASPEED Technology, Inc. ASPEED Graphics Family
Flags: bus master, medium devsel, latency 0, IRQ 16, NUMA node 0
Memory at 94000000 (32-bit, non-prefetchable) [size=16M]
Memory at 95140000 (32-bit, non-prefetchable) [size=256K]
I/O ports at 2000 [size=128]
Expansion ROM at 000c0000 [virtual] [disabled] [size=128K]
Capabilities:
Kernel driver in use: ast

@jenniew Take a look at this?

could you please let me know if there is any update on this.

Sure, we're looking at this issue.

Hi @jenniew , any update on this issue? We are working on a solution with a customer use case and any update will be immensely helpful. It is blocking us now.

Hi, @tsantra!
We are now fixing the LLaVA example on CPU. However, you could try running LLaVA again using CPU on Linux with following installation steps:
conda create -n llm python=3.11 # recommend to use Python 3.11
conda activate llm

# install the latest ipex-llm nightly build with 'all' option
pip install --pre --upgrade ipex-llm[all] --extra-index-url https://download.pytorch.org/whl/cpu

git clone https://github.com/haotian-liu/LLaVA.git # clone the llava libary
cd LLaVA # change the working directory to the LLaVA folder
git checkout tags/v1.2.0 -b 1.2.0 # Get the branch which is compatible with transformers 4.36
pip install -e .
cd ..

We have tested that these steps works fine with latest ipex-llm when running LLaVA using CPU on linux. Please feel free to ask if there's any problem. : )

Hi @JinBridger
These steps work for me. Thank you.
However, I saved the model in 4bit and tried to load the saved 4bit model, when I am getting an error.
I opened another ticket for that:
https://github.com/intel-analytics/ipex-llm/issues/11342
Could you please advise on what to do?


v3.12.0-beta2 tileSprite.setInteractive({cursor: 'pointer', hitArea}): hit area is ignored

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Thanks for opening this issue. As much as we'd love to help you, we need a fully working code sample to do so. Please provide one and then we'll re-open this issue and look at resolving it for you if possible.


Current package typings do not match actual module exports

I'm submitting a ...  (check one with "x")
[ ] Bug report => search github for a similar issue or PR before submitting
[ ] Feature request
[x] Other, please describe

Current typings do not match actual module exports.
Current behavior
As defined by types/index.d.ts
export {LoadingPlugin};

export default Component

Expected behavior
Match exports from library at src/index.js, which are:
export default Component;
export {useLoading, LoadingPlugin, Component}

Referenced in comment on issue #117, see comment.

@ankurk91, I have tested the updated typings for app.use and component in our TypeScript based web application and they work. I would appreciate your feedback.


retrieving incorrect device data

firstly, let me say, this is the only open source code i can find to get data from these devices, WELL DONE! I have a few of these devices and sometimes, even though i give the mac address it doesn't return that devices data.

@roistrocks Unfortunately I haven't implemented any logging yet. Hard to say what the problem is. I have learnt that sometimes the bluetooth connection can't be established or is too poor caused by the distance between host and device. Could this be the reason?
If you have a IDE like Visual Studio Code you can also debug the code to find out what the problem is.

@thall728 Maybe the issue related to different values for humidity is related to a different firmware version. What is the version of you firmware? You can request it as follows:
$ ./govee-h5075.py -i <mac-address>

Thanks for fast response,
Name  Firmware  App Hum  Python Hum
Lanai    1.04.06     79.5           94.7
Master  1.00.05     75.3           78.2
Bath       1.00.05     68.9           77.0
M Lower 1.04.06    66.9            73.3
see attached python comparison

@thall728 Actually I can't reproduce any errors here. I have added logging so that you can start further analysis.
Example:
$ govee -m --log DEBUG
Timestamp             MAC-Address/Alias     Device name   Temperature  Dew point  Temperature  Dew point  Rel. humidity  Abs. humidity  Steam pressure  Battery
DEBUG   A4:C1:38:68:41:23 (GVH5075_4123): Received advertisement data(00 02 cc cc 12 00)
DEBUG   A4:C1:38:68:41:23: Decoded measurement data(00 02 cc cc) is temperature=18.3°C, humidity=50.0%
DEBUG   A4:C1:38:68:41:23: Decoded battery data(0x12) is 18%
2024-12-31 11:52:38   Schlafzimmer          GVH5075_4123  18.3°C       7.7°C     64.9°F       45.9°F     50.0%          7.8 g/m³      10.5 mbar       18%
DEBUG   A4:C1:38:3B:1D:DD (GVH5075_1DDD): Received advertisement data(00 02 7a a8 41 00)
DEBUG   A4:C1:38:3B:1D:DD: Decoded measurement data(00 02 7a a8) is temperature=16.2°C, humidity=47.2%
DEBUG   A4:C1:38:3B:1D:DD: Decoded battery data(0x41) is 65%
2024-12-31 11:52:38   Bad                   GVH5075_1DDD  16.2°C       4.9°C     61.2°F       40.8°F     47.2%          6.5 g/m³      8.7 mbar       65%
DEBUG   A4:C1:38:4F:58:9F (GVH5075_589F): Received advertisement data(00 03 26 aa 40 00)
DEBUG   A4:C1:38:4F:58:9F: Decoded measurement data(00 03 26 aa) is temperature=20.6°C, humidity=50.6%
DEBUG   A4:C1:38:4F:58:9F: Decoded battery data(0x40) is 64%
2024-12-31 11:52:39   Küche                 GVH5075_589F  20.6°C       10.0°C     69.1°F       50.0°F     50.6%          9.1 g/m³      12.3 mbar       64%
DEBUG   A4:C1:38:5A:20:A1 (GVH5075_20A1): Received advertisement data(00 03 0f 29 26 00)
DEBUG   A4:C1:38:5A:20:A1: Decoded measurement data(00 03 0f 29) is temperature=20.0°C, humidity=48.9%
DEBUG   A4:C1:38:5A:20:A1: Decoded battery data(0x26) is 38%
2024-12-31 11:52:40   Wohnzimmer            GVH5075_20A1  20.0°C       8.9°C     68.0°F       48.0°F     48.9%          8.5 g/m³      11.4 mbar       38%

Sorry I did not pick your comment up earlier, I do have some units with humidity calibration and I found that this is causing the issue.
Any easy fix in the script for calibration??

@thall728
The script is more or less stateless w/o any settings. Maybe it would be possible to pass an additional parameter and take calibration into account in python's api. I'm going to take a look at in in new year.
Happy new year!

Please keep me informed, again thanks for your help.  I sure wish I knew how to code but at 74 years old I can only dabble in it.
BE SAFE, AND HAVE A WONDERFUL NEW YEAR!!


Unsupported major.minor version 52.0

Hi presto team!
I have installed both java 6 and java 7, but i switch java 7 and then run ./presto --server localhost:8080, it have errors: Exception in thread "main" java.lang.UnsupportedClassVersionError: com/facebook/presto/cli/Presto : Unsupported major.minor version 52.0. I switch to java 6, it same error, every can fix that

Presto requires Java 8

@dain ok i will try.


#183 - Added environment field to function model, deleted migrations

Closes #183
Introduced Changes

Added environment foreign key field to Function model
Added three indexes to the Function model
Added clean method to Function model to ensure the Function's environment is equivalent to it's package environment
Replaced previous package__environment queries to just check the Function's environment field instead
Removed all migrations in favor of only generating migration files for Functionary releases
Updated .gitignore to ignore migration files
Updated migration task to also makemigrations

Testing

I would recommend deleting your existing postgresql docker volume and using the updated migration task to test out this PR.
Ensure all previous functionality still works, such as creating tasks, task parameters rendering, etc.

Note: I had to also update the github actions to makemigrations when running the functionary unit tests


Forms 9473 container instead of parsys

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

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

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes and the overall coverage did not decrease.
[ ] All unit tests pass on CircleCi.
[ ] I ran all tests locally and they pass.

Change look good to me. Is there any backward compatibility issue ? Can you rebase with latest dev and trigger the pipeline again ?


Relative Paths not working on MJML with mjml-react

I have a template being shipped with nodemailer and all works just fine, except for the fact that I am getting broken images on the email, once received on the inbox.
Relative paths do not work at all. I have also tried some links from images in the web, google drive, google images, and they also not work.
The only links that work are links that end with the file name/type (e.g ......jpg) , like the src below:
 <MjmlImage
              width="400px"
              src="https://image.shutterstock.com/image-photo/bright-spring-view-cameo-island-260nw-1048185397.jpg"
            ></MjmlImage>

I tried using mjml-with-images-loader and configured the module exports on my NextJs  config file, but nothing worked. I guess that it did not work due to the fact that I am using mjml-react on top of the raw mjml syntax.
Thanks in advance for any help on this as I do not know what is the best solution regarding the image path links, if I can get the local relative paths to be transformed into html string or if I should get these images on the web and then referentiate them on my src

Hey @alioshr
This one is a duplicate of https://github.com/wix-incubator/mjml-react/issues/48 So I am closing this one and keeping the conversation in one place.
Cheers


PointMaze Transfer Learning: Use Mixture Distribution

Update to use new PolicyMixture distribution for preferences, regress and model comparison. This should result in more robust reward estimates and reward model similarity metrics.

This improved stability but still significant room for improvement. Merging but will continue this theme in another PR.


[i18nIgnore] Fix date formatting in content-collections.mdx

Description (required)
The content of pull request #6718 has been added in the available languages.

Since the original change was done without i18nIgnore, if we merge this one using it as well, the pages won't be marked as updated if that's the only change they're missing. If we merge without the keyword, we might lose the tracking status of other changes in the file if the translation is not totally up to date, so I'll close this one and let translators do individual PRs to this file. Thanks for the contribution!


Unable Cloning project

$ git clone git@github.com:zevarito/ErizoClientIOS.git
Cloning into 'ErizoClientIOS'...
Permission denied (publickey).
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.
Thanks

@zhangguoyong The repo is public so you should not have any issue to clone it. Are you sure your pubkey is in your github account?


Remove or fix note about dotenv-expand

Using this dotenv-expand only works if users have one of our .env files. The note needs to make this clear... or just remove the note.
See this thread: https://prisma-company.slack.com/archives/C016KUHB1R6/p1643669852467339

It might be that this note needs to be on a different page that is more about our .env files (if it does not exist there yet already).

Looks like a duplicate of https://github.com/prisma/docs/issues/2835
I didn't read the Slack thread mentioned here, but I made this PR
https://github.com/prisma/docs/pull/4189


Testing updates to ARI codec for latest internet draft

See Justin Ethier's ARI testing work in the dtnma-tools repo because the work he's done can effectively be copied and pasted for this ticket

The original test cases are the test_ari_*.c files from dtnma-tools with associated test functions and cases.

These will be copied into the similarly named test_ari_*.py files from ACE tests with similar functions and input cases (defined as python arrays rather than C macros).

Closing, changes have been merged.


Fix format in TOC

First-time contributors' checklist

[ ] I've signed Contributor License Agreement that's required for repo owners to accept my contribution.

What is changed, added, or deleted? (Required)
Fix format in TOC

Which DM version(s) do your changes apply to? (Required)

[x] master (the latest development version)
[x] v5.3 (TiDB DM 5.3 versions)
[x] v2.0 (TiDB DM 2.0 versions)
[ ] v1.0 (TiDB DM 1.0 versions)

What is the related PR or file link(s)?

This PR is translated from:
Other reference link(s):

Do your changes match any of the following descriptions?

[ ] Delete files
[ ] Change aliases
[ ] Need modification after applied to another branch
[ ] Might cause conflicts after applied to another branch

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

/merge


vtk sock setup should use Include instead of "I own your ssh config file now"

vtk socks setup insists on taking full and complete ownership over the content in your ~/.ssh/config file.
It will blow away your existing configuration (if you have one - and back it up to config.bak, sure) and replace it with its own content during setup
Instead, the tool should:

Dump its precious cargo (what currently goes in the ssh config file) in ~/.ssh/config-va.gov
Grep the current ssh config file for Include config-va.gov

If found: do nothing
If not found: append it to the file

+1
I don't necessarily mind my ssh config being overwritten, but I would have like to been notified about the .bak file. I was surprised when a git clone failed and had to do some digging to figure out what happened. A log line after this line would work.

+1
I added my thoughts for docs improvements but this could also be addressed in vtk: https://github.com/department-of-veterans-affairs/va.gov-team/issues/38410
### Description

- _"You can configure connecting to GitHub via SSH instead if you prefer."_
    - While that is true and it links to GitHub's guide, the SOCKS setup overrides `~/.ssh/config`, so if you previously set up GitHub SSH, it will need to be re-added to this file. [GitHub - Generating a new SSH key and adding it to the ssh-agent](https://docs.github.com/en/authentication/connecting-to-github-with-ssh/generating-a-new-ssh-key-and-adding-it-to-the-ssh-agent#adding-your-ssh-key-to-the-ssh-agent). Instead of `Host *`, you can use `Host github.com` to limit it to GitHub connections.
        ```
        Host github.com
        AddKeysToAgent yes
        UseKeychain yes
        IdentityFile ~/.ssh/id_ed25519
        ```
    - Alternatively, the script that modifies `~/.ssh/config` could be changed to append to that file instead of overwriting it and not use `Host *` for its SSH key, so any previous SSH keys added for GitHub remain and still work.

### Relevant URLs

https://depo-platform-documentation.scrollhelp.site/getting-started/Internal-tools-access-via-SOCKS-proxy.1821081710.html

+1  to rmtolmach's comment above. Without knowing this had happened, took awhile to figure out why my Git permissions weren't working, after running vtk.

+1 as well; unless you're running this on a brand new machine, it's going to cause some grief.
Also, the default setup here for Macs seems to prefer using Keychain, whereas I use 1Password to manage my SSH keys. I wouldn't expect vtk to account for this, but it would be nice to have some additional instructions on how to handle migrating things out of Keychain and into a different manager (if I have time I might write up something after I get things working the way I'd like).


[Backports] netdata: from 1.36.1 to 1.37.1

Description of changes
Update of Netdata agent from 1.36.1 to 1.37.1 because of security and crash patches:

2 security patches in 1.37.0: https://github.com/netdata/netdata/releases/tag/v1.37.0
crash patches in 1.37.1: https://github.com/netdata/netdata/releases/tag/v1.37.1

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

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

[x] Fits CONTRIBUTING.md.

Please see the discussion in #207270, this PR is not acceptable as-is.
(cc @RaitoBezarius, looks like nobody ended up backporting the patch to 22.11 😕)

Yes, actually, this PR was sparked by me saying we really need to backport it now.

But we still need to isolate the patches.

Yes, that's what I meant by "backporting the patch" :)

Yes, that's what I meant by "backporting the patch" :)

Yes, this was for OP :).

with only the security patches: https://github.com/NixOS/nixpkgs/pull/230758


Command "ballerina test --all" is failing

While executing command "ballerina test --all" : It deletes all(core,iam,objectstorage) .balo files from directory path "/ballerina-oci/target"
Due to this issue, "ballerina test --all" command gets failed and throw below error
\ballerina-oci>ballerina build -c --skip-tests --all
Compiling source
oracle/core:0.1.0
error: balo file for dependency [oracle/core] does not exists: \ballerina-oci\target\core-2019r3-any-0.1.0.balo

No need to specify the [dependencies] in Ballerina.toml file if you are compiling and building the ballerina-oci module. It is required once you have built the module, and when you want to use this ballerina-oci module in your other Ballerina project.


Unload Method update

Can you instead keep unload but mark it with [Obsolete("Use UnloadWallet instead")] ?

@methrammar sure feel free to do on others.

My apologize @methrammar the UnloadWallet could not possibly work before so we don't have to make it obsolete.

I implemented your idea on https://github.com/MetacoSA/NBitcoin/commit/d41d70168434fab83110495a84dde3e7f4d5e13b
As well as cleaning up Load/UnloadWallet to be more breaking change proof. Added tests as well.


run-command invoke doesn't produce correct error/status fields

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

Describe the bug
az vm run-command invoke reports success even though command execution failed
To Reproduce
Execute az vm run-command invoke --ids SOMEID --command-id RunShellScript --scripts "exit 123" or any other command that would fail
Expected behavior
Status should be failed (or error)
Environment summary
$ az --version
azure-cli (2.0.38)

Additional context
Whatever command I try to run on a Linux VM the error output field is always null and status field successed
The logs on the VM report that the program was terminated with the expected error exit status.
The message field from az cli output correctly reports the error but I'd expect the that either success or error fields would report the same.
Here's an example output from a command that doesn't exist on the system:
{
  "endTime": "2020-07-20T14:45:16.478019+00:00",
  "error": null,
  "name": "807c0bd6-8810-46ff-97f0-d56d6af52fa6",
  "output": [
    {
      "code": "ProvisioningState/succeeded",
      "displayStatus": "Provisioning succeeded",
      "level": "Info",
      "message": "Enable succeeded: \n[stdout]\n\n[stderr]\n/var/lib/waagent/run-command/download/86/script.sh: line 1: asdasd: command not found\n"
    }
  ],
  "startTime": "2020-07-20T14:44:42.274790+00:00",
  "status": "Succeeded"
}

hi @qwordy could you pls have a look?

I think "status": "Succeeded" means the az vm run-command invoke runs successfully. The command execution failure in the VM is an expected behavior, so az vm run-command invoke does its job correctly.

So the only way to see if the script failed or not is by looking into the message field and whether there is stderr output? Using this in pipeline makes it a bit difficult to check whether the script has been successful or not. I'd expect having this kind of information saved in a separate key.

I think you are right.
The response schema is defined here https://docs.microsoft.com/en-us/rest/api/compute/virtual machines run commands/runcommand#runcommandresult
@Drewm3 How do you think the customer's feature request?

@axayjo, could you take a look at this issue?

Investigating.

Tracking this as feature request as we're looking to address this in the future. The current run command implementation doesn't produce status fields.

Closing this issue and tracking this as a feature request on feedback.azure.com. If you'd like to request this feature, please go to this link and upvote it: https://feedback.azure.com/forums/34192--general-feedback/suggestions/42627826-ensure-run-command-invoke-produces-correct-error-s

dead link: https://feedback.azure.com/forums/34192--general-feedback/suggestions/42627826-ensure-run-command-invoke-produces-correct-error-s

dead link: https://feedback.azure.com/forums/34192--general-feedback/suggestions/42627826-ensure-run-command-invoke-produces-correct-error-s


dwarf sp architectural rule ??

I am exploring dwarf debug format.
in dwarf https://dwarfstd.org/doc/DWARF5.pdf page 325.
said sp should obey architectural rule.
why I cannot find any document about this??

Is "sp=cfa" apply to risc-v too?

The short answer is yes,  "sp=cfa" apply to risc-v in general.
The longer answer is that's context-sensitive: you could use .cfi_def_cfa to set the cfa to another register other than sp like frame pointer (s0).

ok, thanks.


Support a dynamic body for each request

We have an app that has just one endpoint and there is a nonce (TransactionId below) that needs to be set differently for each request. I tried the following:
payload["TransactionId"] = uuid.NewV4().String()
body, err :=  json.Marshal(payload)
util.Check(err)
t := vegeta.NewLazyTargeter(r, body, nil)
for res := range attacker.Attack(t, util.Config.Rate, time.Duration(util.Config.Duration)*time.Second) {
		metrics.Add(res)
}

But this sends the same body (as expected). Do you have any thoughts?

Have a look at this blog post: https://thisdata.com/blog/load-testing-api-interfaces-with-go-and-vegeta/
In sum, you can generate the input targets for vegeta attack with your own program and pipe that out.


Permit all-preemptible jobs

Per the docs:

The following will make at most 4 total attempts (one initial and 3 retries) with the first 3 attempts on a preemptible VM and the last on a full-priced VM:

dsub --preemptible 3 --retries 3 --wait ...

My goal is to run 2 attempts (one initial; one retry), both as preemptible jobs without ever defaulting to a non-preemptible job. I tried the following:
dsub --preemptible 2 --retries 1 --wait ...

However, I got the following message:
ValueError: Value passed for --preemptible cannot be larger than --retries.

As dsub is currently setup, is it possible to (1) run with at least one retry, but (2) never default to non-preemptible instances?
dsub version:
$ dsub --version
dsub version: 0.4.6

Hi @carbocation ,
There may be a possible copy-paste issue here? In the retries doc, I see:

The following will make at most 4 total attempts (one initial and 3 retries) all on preemptible VMs:

dsub --preemptible --retries 3 --wait ...

Note that there is no "3" after the preemptible flag. In this case, its treated as a boolean flag where all attempts are made using preemptible VMs. So to get what you want, I think you'll need:
dsub --preemptible --retries 1 --wait ...

Whoops, yes, problem between keyboard and chair on my end. Thanks!


Open right side master from ViewModel

Is it possible to open the rights side master menu from a ViewModel? The example all show opening it from the code-behind. If it's possible, can you show an example? Not sure if it matters but I'm using Prism and so my ViewModel has to inherit from BasePageModel.

I assume in your ViewModel you can access the page, you can create a function in page to open the menu and then call that function from ViewModel

That's the problem - you don't have access to the page from the ViewModel.

What's MVVM framework did you use?

Prism
On Mar 7, 2017 5:46 PM, "Jesse Jiang"<EMAIL_ADDRESS>wrote:

What's MVVM framework did you use?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/XAM-Consulting/SlideOverKit/issues/28#issuecomment-284885603,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMdrshHuLNRjs_LKWWDp9TaPWZT-yRy8ks5rjd5YgaJpZM4L_qXb
.


Russian is listed among Asian languages, omg rly?

Informations
Browser version:
OS:
Screen resolution:
How to reproduce the bug?
Just go to https://alex-d.github.io/Trumbowyg/ and enjoy a list of supported languages separated by region. :D

Same with Spanish listed under America :)

...
Russian is Europe and Asia, since Europe list is really long, I've choose to put it in Asia section.
Spanish? No, "Spanish (Argentina)" yep. And Argentina is in South America.
So don't troll, and if you want to change something, feel free to make a Pull Request.


DataLakeStorage support

As an engineer, I would like to create a function/cmdlet to support DataLakeStorage checks so that we can support future customer's needs.

[ ] Create Confirm-DataLakeStorage.ps1 and unit tests
[ ] Wire up Confirm-DataLakeStorage to Common.ps1
[ ] Create example tests, bicep and documentation in examples folder

This is a no-op.
Storage for ADLS is now wholly contained by Azure Storage, which has already been addressed in BenchPress.
Data Lake Store Account (note Store and not Storage) is deprecated with ADLS Gen1, so will not be added.