tt-topology with multiple n300s failing

Describe the bug
I have four n300s on a Linux machine running in a Docker (Ubuntu 20.04). Prior to running tt-topology -l linear -f linear_layout.png I have no issues with tt-smi and can invoke various tt-metal tests, etc. Note that single n300 tests have no issues when running our tt-metal scripts using this Docker.
The tt-topology command fails with the attached errors. I have to do a reboot to get the cards back online.
To Reproduce
Steps to reproduce the behavior:
With 4x n300s connected via PCIe:

source your Pyenv
Run tt-topology -l linear -f linear_layout.png
See error

Expected behavior
Output file for the linear_layout.png generated, updated coordinates in the tt-smi, and ability to target individual devices within tt-metal scripts.
Screenshots
Prior to running tt-topology:

tt-smi -s logs:
tt_smi_logs_results.json
tt-topology -ls output:

lscpi output prior to tt-topology:

dmesg output prior to tt-topology:

dkms status prior to tt-topology:

tt-topology -l linear -f linear_layout.png terminal output:

tt_topology_log output:
tt_topology_logs_log.json
Post tt-smi output:

Please complete the following environment information:

OS: Host system uses Ubuntu 22.04. Docker uses Ubuntu 20.04
Version of software:

tt-smi: 3.0.2
tt-topology: 1.1.5

Additional context
Just noting again that this is being run in a Docker due to host environment being restricted to Ubuntu 22.04, which hasn't been an issue with a single n300.

@ionx-justyn , thanks for filing the detailed issue.
We currently support single device or mesh topology for (data parallel or tensor parallel). What's your use case for the linear topology?

Hey @dvartaniansTT! I tried mesh as well, but ended up with the same problem. We do not necessarily need linear specifically. I'm just trying to get our basic script working with >1 n300.
We dynamically target an n300, but our scripts are hanging after the import ttnn with more than one n300 on the mobo. From the tt-smi above you can see all four have the same coords. I'm not sure if the same coords is the problem here, though.

hi @ionx-justyn , the supported options are:

Run on one n300, but in that case you need to isolate it in the docker container or disable 3 out of the 4 cards
sudo docker run -it -d --name=mlir-container -v /home/ubuntu/.ssh:/root/.ssh -v /dev/hugepages-1G:/dev/hugepages-1G -v /home/ubuntu/code:/code --restart=unless-stopped --shm-size=4g --cap-add ALL --device /dev/tenstorrent/1:/dev/tenstorrent/0 ghcr.io/tenstorrent/tt-mlir/tt-mlir-ci-ubuntu-22-04:dt-e931311c5edde3f3df8df7b8bf42e9dd bash

OR, set the tt-ttopology as mesh and connect the cards with the T-Fly and QSFD cables as our standard LoudBox setup. In that case you can use the multi-card in data parallel or tensor parallel configuration.

Hopefully this is helpful.

Thank you @mbahnasTT!
In our case, we have four n300s, so would we need something like the below for a single docker image to be limited to a single n300?
--device /dev/tenstorrent/1:/dev/tenstorrent/0 \
--device /dev/tenstorrent/2:/dev/tenstorrent/0 \
--device /dev/tenstorrent/3:/dev/tenstorrent/0 \

@ionx-justyn yes correct, please confirm when it's working OK. Thanks!

Looking good now! We went the Docker approach, but we'll likely test out the Loudbox setup at some point. Thanks so much, @mbahnasTT and @dvartaniansTT!


Adding BaseConfigurationManager and BaseConfiguration classes in M.IM.Tokens

These classes will be necessary in order to add a ValidateToken(...) API on the JsonWebTokenHandler that takes in a ConfigurationManager object in addition to the TokenValidationParameters. This API will be added in a future PR.
Some notes on the changes made:

JsonWebTokenHandler is the class that will be responsible for setting UseLKG and UseCurrentConfiguration when going through its validation flow.
I am not sure whether moving AutomaticRefreshInterval and other public static properties to the StandardConfigurationManager base abstract class is considered a breaking change or not. Would love to hear perspectives on this.
Also not sure about the implications of having the RequestRefresh() API on both the StandardConfigurationManager abstract class and the IConfigurationManager.
A property for timing how long we've been using the LKG may also need to be needed to the StandardConfigurationManager class (so that we know when it's stale and we should set UseLKG = false)
Configuration had to be renamed to StandardConfiguration as it conflicted with System.Configuration in the .NET Framework (lots of extremely misleading Visual Studio errors on this one, haha).

/azp run


Users: onboard new projects looking to integrate Gardens

Giveth, Agave, BrightId

We onboarded: Giveth, BrightID, Beezu and Agave


Use Logger class instead of puts-ing and allow configuration

It'll now be possible to use custom loggers by doing e.g.
Rubotnik.logger = Log4r.Logger
default logger outputs to stdout
closes https://github.com/progapandist/rubotnik/issues/14

Great! I'm going to merge that right now. I already bumped the version to 0.2.1 though, so let's wait a bit before we bump again.
If you take time to switch from your fork to the main repo — I'll appreciate that, you are a full collaborator with full access rights, and basically a core contributor now, so it makes more sense :)
Also, the Hound CI annoys my inner perfectionist — either we need to silence the warnings by having a .rubocop.yml that would not bother us with stupid stuff like method length in test, or just not use the Hound, otherwise comments are really hard to read :)
Impressed by your enthusiasm, thanks again for helping out with the project, I really appreciate it a lot! 🎉


download not loading

Hello,
I have brought the downloader into my project. when I run it on android it downloads the obb file and then does not load to the second scene it hangs not loading anything.
I do some checks after the file is downloaded to make sure it is there and it say it is but it docent move to the next scene. Any help would be great or pointing into the right direction would be great

Hi! Sorry for the delay.
Logcat would be helpful. Does pause/resume the app (hitting home and then back) allows you to load the scene from the OBB? Which version of Unity are you on?


Added common page components and added content to the homepage.

Hi @decoy31, please rebase this again to pick up #365, which just landed.


fix(set): just some bug fix

Line 5 would cause an error whenever a target is supplied.

@TrueXPixels, but since fetched is null and there is a condition which checks if it is an object, and that is certainly impossible.

To conclude, that fetched is null and we are checking if it is an object without defining it, and it will probably error out: fetched of undefined


iterm2 does not open and change directory on first call

In issue #207, it was noted that iterm2 did not cd to the project directory.  The solution is to activate the window then tell the session to cd
#!/bin/bash

CD_CMD="cd "\\\"$(pwd)\\\"" && clear"
if echo "$SHELL" | grep -E "/fish$" &> /dev/null; then
    CD_CMD="cd "\\\"$(pwd)\\\""; and clear"
fi
VERSION=$(sw_vers -productVersion)
OPEN_IN_TAB=0

while [ "$1" != "" ]; do
    PARAM="$1"
    VALUE="$2"
    case "$PARAM" in
        --open-in-tab)
            OPEN_IN_TAB=1
            ;;
    esac
    shift
done

if (( $(expr $VERSION '<' 10.7) )); then
    RUNNING=$(osascript<<END
    tell application "System Events"
        count(processes whose name is "iTerm2")
    end tell
END
)
else
    RUNNING=1
fi

if (( ! $RUNNING )); then
    osascript<<END
    tell application "iTerm"
        tell current window
            activate
            tell current session
                write text "$CD_CMD"
            end tell
        end tell

    end tell
END
else
    if (( $OPEN_IN_TAB )); then

        osascript &>/dev/null <<EOF
        tell application "iTerm"
            if (count of windows) = 0 then
                set theWindow to (create window with default profile)
                set theSession to current session of theWindow
            else
                set theWindow to current window
                tell current window
                    set theTab to create tab with default profile
                    set theSession to current session of theTab
                end tell
            end if
            tell theSession
                write text "$CD_CMD"
            end tell

            activate
        end tell
EOF
    else
        osascript &>/dev/null <<EOF
        tell application "iTerm"
            tell (create window with default profile)
                tell the current session
                    write text "$CD_CMD"
                end tell
            end tell

            activate
        end tell
EOF
    fi
fi

Cool, can you submit this as a PR? I'm crunched for time so I don't have much for copy/pasting code (also have no Mac to test on =/

Hello, i have same issue with iTerm and i would like to know how to use your solution? Where to use this code?


Lint staged array calling pre-configured scripts

Is it possible to do this (simply just run lint-staged on a script already pre-configured in the scripts section?)
"scripts": {
    "eslint": "./node_modules/.bin/eslint .",
    "stylelint": "./node_modules/.bin/stylelint statics/css/**/*.less --syntax less",
    "htmlhint": "./node_modules/.bin/htmlhint mvc/views",
    "lint": "npm run htmlhint && npm run stylelint && npm run eslint",
    "lint-staged": "lint-staged"
  },
  "lint-staged" [
    "htmlhint",
    "lesshint",
    "eslint"
  ]
  "pre-commit": [
    "lint-staged"
  ]

Yes, although because of #17 it won't work in this case. It will after it is fixed.
Important note: you shouldn't pass path to these scripts.
Also, you don't need to specify path to your bins. See https://docs.npmjs.com/cli/run-script
So, to sum it up. In this case your config could look like this:
"scripts": {
    "eslint": "eslint .",
    "eslint:staged": "eslint",
    "stylelint": "stylelint statics/css/**/*.less --syntax less",
    "stylelint:staged": "stylelint --syntax less",
    "htmlhint": "htmlhint mvc/views",
    "htmlhint:staged": "htmlhint",
    "lint": "npm run htmlhint && npm run stylelint && npm run eslint",
    "lint-staged": "lint-staged"
  },
  "lint-staged" [
    "htmlhint:staged": "*.html",
    "stylelint:staged": "*.less",
    "eslint:staged": "*.js"
  ]
  "pre-commit": [
    "lint-staged"
  ]


Add Cinchoo ETL

ETL Framework for .NET (Parser / Writer for CSV, Flat, Xml, JSON, Key-Value formatted files)

Thanks for making the additional changes, @Cinchoo. I'll merge your PR now.


Add three generation metrics, BLEU, ROUGE, METEOR

close #38
Implement three metrics.
I think we can add more great metrics few days later, but first I think working all system is the priority.

I agree with your opinion.
Do great metrics mean some metrics with LLM like  RAGAS, using token probability metric?

I think these n-gram based metrics can open some options later


[WIP] Replace deprecated RetrieveProperties with RetrievePropertiesEx

RetrieveProperties was deprecated in vSphere 4.1 and all callers should be moved to the new RetrievePropertiesEx.
https://www.vmware.com/support/developer/vc-sdk/wssdk_4_1_releasenotes.html#deprecations

@miq-bot add-label wip, providers/vmware, technical debt

@miq-bot remove-label wip
@roliveri can you review?


Show release notes

Show release notes when Control Center asks to update it's components

Fixed on master.

Need to wait for the next update to check this issue in Production.

Implemented in Prod.


Intro of the Technical Specs (ssi e successivo)

Replace the lorem ipsum with introductionary text in the file below
https://github.com/italia/eidas-it-wallet-docs/blob/main/docs/en/index.rst

Done here https://github.com/italia/eidas-it-wallet-docs/pull/12


ContactUs page does not exists

Adding ContactUs page which will include a form using which the user can write to us.
@sakshikhachane @swapnilsparsh If you find it useful then please assign this to me and label it under GSSoC21.

Hey @jainayush362,
This feature is under development in #198. So I am closing this issue.


Multi channel gateway using multiple modules ?

Do you think it would be possible to add more modules to make a 3 channel gateway.
As far as I can tell the data rate is quite low and the SPI bus speed is very fast, so a single ESP8266 should be able to handle the traffic from multiple LoRa modules

.


Fix issue on destructuring assignment in createPayment

Hi,
I changed the order of the assignment of the params passed to the createPayment.
In this way we can pass amount and checkoutId manually.
Before the values was always rewritten.
This change is useful and indispensable (IMO) if you use taxes and taxes are not included in the price.
With this change you can pass right totalPrice.
This is a fix but I think there's something to check into the SDK regarding the calc of taxes and totals.
Hnave a nice day and regards,
Davide

 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.

Hey @DidoMarchet, we're currently in the process of rewriting the entire SDK from ground up - the biggest change in checkouts will be relying on backend to do all necessary calculations instead of doing them on frontend. I'd accept this PR in the meantime but you bumped the package-lock to lockfileVersion 2 which bring in some complications with setup in older node versions.  Can you get rid off all the changes but the actual destructuring itself?

Hi @jwm0 thanks!
I updated the package and package.lock files (I restore this files with the latest in master branch).
I head you are rewriting the SDK.
Will be significant changes in frontend implementation?
Thanks in advance and regards,
Davide


built ok but one issue

localhost/:1 Uncaught (in promise) Error: A listener indicated an asynchronous response by returning true, but the message channel closed before a response was received
:3000/dalle:1          Failed to load resource: the server responded with a status of 404 (Not Found)
localhost/:1 Uncaught (in promise) SyntaxError: Unexpected token '<', "<!DOCTYPE "... is not valid JSON
localhost/:1 Unchecked runtime.lastError: A listener indicated an asynchronous response by returning true, but the message channel closed before a response was received
3localhost/:1 Uncaught (in promise) Error: A listener indicated an asynchronous response by returning true, but the message channel closed before a response was received
Server is running on port 3001

frontend port 3000
i get issue after submitting any text, otherwise website loads fine

how can i fix this?

env is ok, what is Base URL in client env?

hey guys, i tried to recreate the errors you're facing but everything is working on my side. so make sure to run them separately (dotenv sometimes doesnt work when run concurrently) and send me the error displayed on the server side.

SERVER - https://gpt-and-dalle.onrender.com/
Front End - https://new-chat-ui-gpt.vercel.app/
somethings wrong when chatting

https://gpt-and-dalle.onrender.com/dalle says something about rate limits

https://gpt-and-dalle.onrender.com/dalle says something about rate limits

use the local_setup branch for easier setup without redis and firebase

SOLVED!

edited the link as requested

thank you!

i got this working with RENDER and VERCEL Successfully!!! REDIS IS SUPER IMPORTANT !


Add View Mass Mode ( #196 )

Mass Mode now viewable via the ingame command "-mass".
The Mass is now easily visible at the end of the Player's name.
See more: https://github.com/huytd/agar.io-clone/issues/196

Thank you so much for that :)

Don't you think -mass should be one of the options in the settings tab on the main menu?

What about giving the players both possibilities?
If you forgot to activate the mass Mode you can do it ingame and you can toggle it before you join the game.

+1 for both the menu button and the chat command.


Question: Torque Sensor

Thank you for your continued support on this amazing tool! I have a question regarding my BB Torque sensor. I recently installed a original Bafang Torque sensor BB on my bicycle (PA211.32.ST). I would like to have the controller recognize the sensor. Right now it feels like it is just functioning as a cadence sensor. Would you happen to know the correct settings for my BB sensor? Or is it not supported at the moment?
Thank you and kind regards,
Diederik

Yes, there is switch called "trigger mode" or something like it, I don't remember actual name. It has three options - "cadence sensor and throttle lever", "torque sensor and throttle lever", "throttle lever only". You need to set it to "torque sensor and throttle lever"


Documentation for Seq.tail is missing

The latest FSharp libraries added Seq.tail in the Collections.Seq module. Source code is here However, the documentation is still missing for it.

@cartermp @KevinRansom can you comment?

@ant0nsc good eyes.  We would be happy to take a contribution adding the documentation for this Api.  I expect that there are other new api's that are also missing documentation.
I'm sure that @cartermp and @dend would be able to help
Also marking this issue up for grab.
Kevin

So, as this issue mentions, there are many other functions that are missing from the docs.
I did a little bit of research and think most of the ones added in F# 4.0 haven't had docs made yet (https://github.com/fsharp/FSharpLangDesign/blob/master/FSharp-4.0/ListSeqArrayAdditions.md). Should I create a new issue for updating the docs for all the new functions or maybe it's easier to update this issue?

@liboz I would recommend creating a new issue for the doc set that you mentioned.


Throttle dialogflow calls.

Also, RetryRobot makes dialogflow tests cope with eventual consistency in the list() method.

TIL about BlockingCollection and it's pretty darned cool!
Took me a couple mins to walk through and grok it. I can't think of any semantics which would've helped me grok it faster (except 1 comment I made about timeSpan, feel free to make this change but I'm LGTM!)


Calico installation using tigera operator does not allow to explicitly the source of calico images.

Expected Behavior
The below image registry/repository/tag must be configurable.
calico-typha: docker.io/calico/typha:v3.20.0
calico-node: docker.io/calico/pod2daemon-flexvol:v3.20.0
calico-kube-controller: docker.io/calico/kube-controllers:v3.20.0
Current Behavior
At the moment there is no control over the below images and they are pulled from public docker registry.
calico-typha: docker.io/calico/typha:v3.20.0
calico-node: docker.io/calico/pod2daemon-flexvol:v3.20.0
calico-kube-controller: docker.io/calico/kube-controllers:v3.20.0
Possible Solution
Expose the related values on tigera operator level so as to be able to control the source of image pulling.
Context
Our requirement is that all images should be pulled from ECR (or private) registries.
This is a security requirement.
Also docker.io has a rate related to image pulling.
Your Environment
Env: EKS
Tigera operator version: v3.20.0

The operator does support being able to configure those. Please review the InstallationSpec, specifically see registry and imagePath fields.

Also docker.io has a rate related to image pulling.

It is possible to provide specify image pull secrets that will be used and that would remove the image rate limiting.
I'm going to close this because I think this is available. Please let me know if I'm mis-understanding something and we can re-open this.


Update Cake to 0.28.0.

@patriksvensson I have rebased this onto the develop branch, and merged in here: https://github.com/cake-contrib/Cake.Graph/commit/3fdb0c73b2c886f02992b743eda996fb1be9b82c
@nmbro yes, I will take care of that now that this is merged.

@patriksvensson  @nmbro  @gep13 Thanks for doing this all! Been v.busy recently

@wozzo you are very welcome.  Totally understand being busy.  This is one of the reasons to move an addin into the Cake-Contrib organisation, so that others can help out when people are busy.  Thanks again for all your hard work in getting this addin created in the first place!


FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - process out of memory

Hello,
I'm using ~0.8.1, and following the same process of nodegit project, when I generate source code from template, i got below error message
Please advise, thanks!
<--- Last few GCs --->

   39211 ms: Scavenge 700.3 (737.9) -> 700.3 (737.9) MB, 12.0 / 0 ms (+ 16.0 ms
in 1 steps since last GC) [allocation failure] [incremental marking delaying mar
k-sweep].
   40209 ms: Mark-sweep 700.3 (737.9) -> 696.2 (737.9) MB, 985.6 / 0 ms (+ 16.0
ms in 2 steps since start of marking, biggest step 16.0 ms) [last resort gc].
   41195 ms: Mark-sweep 696.2 (737.9) -> 696.1 (737.9) MB, 989.1 / 0 ms [last re
sort gc].

<--- JS stacktrace --->

==== JS stack trace =========================================

Security context: 17B6CC95 <JS Object>
    1: constructProperty [\node_modules\combyne\dist\c
ombyne.js:~634] [pc=083DF926] (this=10B1D319 <a Tree with map 21B4FD51>,encoded=
17B08115 <true>)
    2: constructConditional [\node_modules\combyne\dis
t\combyne.js:905] [pc=0839A922] (this=10B1D319 <a Tree with map 21B4FD51>,root=1
0B1D33D <an Object with map 21B4FEB1>,kind=09249239 <String[8]: ST...

FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - process out of memory

Have you tried upgrading to 2.0?

@tbranyen , thanks for the update, please ignore this, it turns out there's something wrong in the partials, it didn't throw any error, and got stuck at templates.class_content.registerPartial(partial, combyne(partials[partial])); step
after correcting the error in partial, everything is working fine again

Awesome, glad you got it working!

Gentle suggestion that this issue be repurposed or a new one be opened to address the swallowed error, that sounds like a bug?


Thank you for keeping the world safe

Type: Bug
I had to ask GitHub copilot to format this string for me four times, the first 3 of which it told me the response was filtered by the Responsible AI Service.
        return FName(R"(
            When you BARRAGE, you attack with two weapons, or with one SUPERHEAVY weapon.\
            To BARRAGE, choose your weapons and either one target or different targets – within range – then make an attack with each weapon.\
            • In addition to your primary attacks, you may also attack with an AUXILIARY weapon on each mount that was fired,\
              so long as the AUXILIARY weapon hasn't yet been fired this action. These AUXILIARY weapons don't deal bonus damage.\
            • SUPERHEAVY weapons can only be fired as part of a BARRAGE.\
        )");

Yeah, please tell me more about how descriptions of scifi weapons are something that needs to be filtered.
Claude doesn't balk about this.  Better put an end to Claude before everyone gets their hands on antimatter cannons.
Extension version: 0.20.3
VS Code version: Code 1.93.1 (Universal) (38c31bc77e0dd6ae88a4e9cc93428cc27a56ba40, 2024-09-11T17:20:05.685Z)
OS version: Darwin arm64 23.6.0
Modes:

Thanks for creating this issue! We figured it's covering the same as another one we already have: https://github.com/microsoft/vscode-copilot-release/issues/2625. Please refer to that one for updates and discussions. Feel free to open a new issue if you think this is a different problem.


[WebToolsE2E][Aspire] There are some errors "Refused to apply style from '[https://localhost:XXX/......']" in the website console pane when running the Aspire project

REGRESSION INFO: Work in Aspire 8.2.1
INSTALL STEPS

Clean machine: Win11 x64 23h2 ENU
Install SDK 8.0.402
Install Aspire 9.0 RC1 [9.0.0-preview.4.24479.1]

REPRO STEPS

Open CMD, create an Aspire App using following commands:

dotnet new aspire -o AspireApp
cd AspireApp/AspireApp.AppHost

Dotnet run the project in CMD, and open URL in the browser.
dotnet run
Hit F12 and navigate to Console in the browser development tools.

Note:

This issue doesn't repro when running the normal project
There is a known issue with console pane warnings: #5279

ACTUAL
There are some errors in the website console pane

Error Log:
Refused to apply style from '[https://localhost:17067/_content/Microsoft.FluentUI.AspNetCore.Components/Microsoft.FluentUI.AspNetCore.Components.bundle.scp.css'](https://localhost:17067/_content/Microsoft.FluentUI.AspNetCore.Components/Microsoft.FluentUI.AspNetCore.Components.bundle.scp.css%27) because its MIME type ('text/html') is not a supported stylesheet MIME type, and strict MIME checking is enabled.
localhost/:1  Refused to apply style from '[https://localhost:17067/_content/Microsoft.FluentUI.AspNetCore.Components/Microsoft.FluentUI.AspNetCore.Components.bundle.scp.css'](https://localhost:17067/_content/Microsoft.FluentUI.AspNetCore.Components/Microsoft.FluentUI.AspNetCore.Components.bundle.scp.css%27) because its MIME type ('text/html') is not a supported stylesheet MIME type, and strict MIME checking is enabled.

I think I've already fixed this.

Verify this issue fixed on the latest Aspire 9.0.0-rc.1.24506.1.


TFLM Added optimized fully connected (float32 and int8) for CEVA-BX1

Github issue: https://github.com/tensorflow/tensorflow/issues/45607

Also, I have created a new label (comp:micro:ceva). If you could apply that to all your PRs and github issues, that would make tracking this work easier. And link the issue from your PR as well.
You should have the permissions needed to be able to apply labels.
You can directly add me as a reviewer, have yourself be the assignee for the github issues.

Also, I have created a new label (comp:micro:ceva). If you could apply that to all your PRs and github issues, that would make tracking this work easier. And link the issue from your PR as well.
You should have the permissions needed to be able to apply labels.
You can directly add me as a reviewer, have yourself be the assignee for the github issues.

I may have reviewed this PR before it was ready -- sorry about that. If possible, please explicitly add a comment letting me know that the PR is ready for review (similar to https://github.com/tensorflow/tensorflow/pull/46411#issuecomment-761210727)

I may have reviewed this PR before it was ready -- sorry about that. If possible, please explicitly add a comment letting me know that the PR is ready for review (similar to https://github.com/tensorflow/tensorflow/pull/46411#issuecomment-761210727)

I may have reviewed this PR before it was ready -- sorry about that. If possible, please explicitly add a comment letting me know that the PR is ready for review (similar to #46411 (comment))

This is my fault, I should have closed this one - I created a new PR after our chat -  incorporating code sharing with the reference kernel - see https://github.com/tensorflow/tensorflow/pull/46500 and also https://github.com/tensorflow/tensorflow/pull/46226 for Quantize.
I'll go over your comments here and apply them to those PRs where applicable.

I may have reviewed this PR before it was ready -- sorry about that. If possible, please explicitly add a comment letting me know that the PR is ready for review (similar to #46411 (comment))

This is my fault, I should have closed this one - I created a new PR after our chat -  incorporating code sharing with the reference kernel - see https://github.com/tensorflow/tensorflow/pull/46500 and also https://github.com/tensorflow/tensorflow/pull/46226 for Quantize.
I'll go over your comments here and apply them to those PRs where applicable.

sounds good, I'll go ahead and close this PR then. Let me know when the others are ready for review.

sounds good, I'll go ahead and close this PR then. Let me know when the others are ready for review.


Update Google + color

the color to use in a google+ Sign-In button is #DC4E41
https://developers.google.com/+/branding-guidelines

@aliceperez Thank you! Can you update the /src/simple-icons.json file too please? I want you to get credit for the change :)


Implement temporal anti-aliasing posteffect when the camera is static

when the camera stops, the TAA would randomly offset the camera position to create subpixel movement in the rendered image. Those images would then be averages over multiple frames, creating a high quality AA effect.
product configurators could get very high quality renders

closing, duplicate of https://github.com/playcanvas/engine/issues/3361


Inconsistent casing on button labels

Describe the bug
I can see 3 buttons, each has its own casing:

Unprovisoing
create
ACTIONS

To Reproduce
Click on an application type in the application tree to the left.
Expected behavior
All buttons are either in lower case, UPPER CASE, or Normal Case.
Screenshots

Service Fabric version: 8.0.514.9590
OS (for browser not SF): Windows 10
Browser / Browser version: Edge 86.0.622.63

Hey, that is a great point to bring up.
We havent had a standard set for casing but this is a good thing to take notice of and correct. While the standard isn't set yet, it will definitely be decided and reflected back in a CU later on.


remove MSI

@ejaz-alam Minor edit on the intro sentence. Can we also add a sentence about why someone would want to remove the modules? Are there any negative consequences to this?

so sometimes we do not use the inventory modules and these modules are more than 20 modules which really affects the performance. so in terms performance, it could be uninstalled.


gradle localization failed

plugins {
id "pl.droidsonroids.localization" version "1.0.8"
}
localization {
xlsFile = file('language_iOS_append_ALL_343_RC_20150522v2.xlsx')
allowEmptyTranslations = true
defaultColumnName = "EN"
name = "Android"
ignorableColumns = ['WinPhone', 'iOS', 'END']
outputDirectory file('outRes')
sheetName "string table"
}
D:\studio_workspace\android\lite>gradle localization
FAILURE: Build failed with an exception.

What went wrong:
A problem occurred configuring root project 'lite'.

Could not resolve all dependencies for configuration ':classpath'.
Could not resolve gradle.plugin.pl.droidsonroids.gradle.localization:android-gradle-localization-plugin:1.0.8.
Required by:
:lite:unspecified
Could not GET 'http://plugins.gradle.org/m2/gradle/plugin/pl/droidsonroids/gradle/localization/android-gradle-lo
calization-plugin/1.0.8/android-gradle-localization-plugin-1.0.8.pom'.
Connection to https://gradleware-plugins.s3.amazonaws.com refused

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

Connection to https://gradleware-plugins.s3.amazonaws.com refused - that is problem with network connection. Gradle failed to download plugin from plugin repository. Currently that URL is reachable for me. Try again and/or check your network/firewall settings.
You can also use another plugin inclusion syntax: https://github.com/koral--/android-gradle-localization-plugin#all-versions-of-gradle


Fix cooldown feature to properly manage several instances of emissary

Description
Previously, we added a cooldown step (30s) for the metrics to avoid to send too many requests to the ambassador cloud API.
Although, some values were not consistent, and we figured out that the way the metrics are sent to the agent are one request per emissary node, and not one for all.
It means that the previous iterations of the code were wrong for two reasons :

If we forward the metrics directly to the cloud, we won't be able to identify what node corresponds to what metric.
If we put a simple cooldown of 30s, for 3 nodes, we'll miss 2 out of 3 sets of metrics.

As a workaround, with minimal changes, this PR changes the logic to make the metric relay handler to accumulate metrics in a map per nodes' IP, and unload them after the cooldown period.

Testing
Adapted existing tests, & tested it in a dev cluster.
Checklist

[x] I made sure to update CHANGELOG.md.
Remember, the CHANGELOG needs to mention:

Any new features
Any changes to our included version of Envoy
Any non-backward-compatible changes
Any deprecations

[x] This is unlikely to impact how Ambassador performs at scale.
Remember, things that might have an impact at scale include:

Any significant changes in memory use that might require adjusting the memory limits
Any significant changes in CPU use that might require adjusting the CPU limits
Anything that might change how many replicas users should use
Changes that impact data-plane latency/scalability

[x] My change is adequately tested.
Remember when considering testing:

Your change needs to be specifically covered by tests.

Tests need to cover all the states where your change is relevant: for example, if you add a behavior that can be enabled or disabled, you'll need tests that cover the enabled case and tests that cover the disabled case. It's not sufficient just to test with the behavior enabled.

You also need to make sure that the entire area being changed has adequate test coverage.

If existing tests don't actually cover the entire area being changed, add tests.
This applies even for aspects of the area that you're not changing – check the test coverage, and improve it if needed!

We should lean on the bulk of code being covered by unit tests, but...
... an end-to-end test should cover the integration points

[x] I updated DEVELOPING.md with any any special dev tricks I had to use to work on this code efficiently.

[x] The changes in this PR have been reviewed for security concerns and adherence to security best practices.

@knlambert - I will have some time to look at this more later and think about it but just as a quick glance.

does this need to land in v2.3 branch as Alex mentioned?  I would rather it land in the v3 branch which is master then if need be we can backport it.
Again only taking a few minutes to look at it, it appears that you delete metrics before they have successfully been sent which means on errors they will be dropped. Is this the intended behavior? Keeping them around would introduce memory pressure too so not advocating for any particular solution but I just wanted to understand better 😄.

@knlambert - I will have some time to look at this more later and think about it but just as a quick glance.

does this need to land in v2.3 branch as Alex mentioned?  I would rather it land in the v3 branch which is master then if need be we can backport it.
Again only taking a few minutes to look at it, it appears that you delete metrics before they have successfully been sent which means on errors they will be dropped. Is this the intended behavior? Keeping them around would introduce memory pressure too so not advocating for any particular solution but I just wanted to understand better 😄.

The next one to be published :o) it's a bug fix.
It's ok, that's not critical data and we don't want to spam cloud if the link is broken, or saturate the memory as you said :)

Let's squash the commits and clean up commit message when we merge since they are all related.


Best practice for caching?

Loving being able to use Hermit and GitHub Actions!
What is the best way to cache hermit packages? I was considering something like this:
steps:
  - uses: actions/checkout@v3
  - name: Cache hermit
    id: cache-hermit
    uses: actions/cache@v3
    with:
      path: .cache/hermit
      key: ${{ runner.os }}-hermit-${{ hashFiles('bin/*.pkg') }}
  - uses: cashapp/activate-hermit@v1

Is this something that could be included in the action in the same way that the setup-node action has optional caching?

Fixed in #8.


hasAllNullFieldsOrProperties and primitive types

Summary
hasAllNullFieldsOrProperties check that the actual object has only null fields or properties.
All fields and properties are inspected but fields with primitive types always will have some value.
Example
    class Book {
        String title;
        int pages;
    }

    @Test
    void testHasAllNullFieldsOrProperties() {
        Book book = new Book();
        assertThat(book).hasAllNullFieldsOrProperties();
    }

test will fail because of field pages is not null.
Proposition
Exclude by default all field with primitive types from assertion.

I want to have a try on the issue, thanks.

@slawekjaranowski primitives cannot be null - they have default values if not initialized. It is clearly stated in https://docs.oracle.com/javase/tutorial/java/nutsandbolts/datatypes.html

@nach-o-man I guess he meant primitive types should be ignored and not be considered when using hasAllNullFieldsOrProperties. Otherwise, if a class has primitive fields, the test will always fail. I'm not sure.

Fair point, I understood this wrong when I read it for the first time. But I do think it is not right add such functionality.
This exactly falling test case can show design/logic flaws and can be avoided with the use of hasNoNullFieldsOrPropertiesExcept

I used hasNoNullFieldsOrPropertiesExcept at current time with list of field which should be excluded.
Primitive types always have value, so. we always must be exclude it.
As my proposition in issue its can be excluded / skipped by default.

@slawekjaranowski  I will have a try and make a pull request to skip primitive types by default.  I may add a new method.

@wuqihan837 before submitting a PR, give the assertj team a bit of time to think on how to address the issue.
My first comment would be that the current assertion does its job as primitive types are never null, changing it would be a breaking change so we might choose a different option, maybe overloading hasAllNullFieldsOrProperties(boolean excludePrimitiveTypes) but no decision has been reached yet.

The definition of field or property does not exclude primitive types, so I don't see an issue with the current behavior of hasAllNullFieldsOrProperties.
To simplify the primitive types use case, I vote for hasAllNullFieldsOrProperties(boolean excludePrimitiveTypes) or also hasAllNullFieldsOrPropertiesExceptPrimitiveTypes().

I vote for hasAllNullFieldsOrPropertiesExceptPrimitiveTypes(), it is a long name but it is explicit whereas hasAllNullFieldsOrProperties(true) is not really self explanatory.

What about using hasAllDefaultValueForFieldsOrProperties()? Because I believe that most of the time users use this function to validate whether there are any fields or properties changed or not clear, using hasAllNullFieldsOrPropertiesExceptPrimitiveTypes() will bring the misunderstanding that the method ignores all changes in primitive types? And I also wonder if there is something that needs to be done about this issue? I could help it.

This is now fixed by the introduction of the recursive assertion:
@Test
void testHasAllNullFieldsOrProperties() {
  Book book = new Book();

  assertThat(book).usingRecursiveAssertion()
                  .ignoringPrimitiveFields()
                  .allFieldsSatisfy(field -> field == null);
}


feat: check auth header before allowing access

Description

Breaking change?

Example

Checklist

[x] All tests pass
[x] Tests added in this PR (if applicable)

:tada: This PR is included in version 1.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Seek time only to a specific time?

So .seek('00:23:3000') goes to 23 minutes and 30 seconds and then renders until the end. Is there a way to seek and then play only to a specific time?

Hi, yes, you can use the .duration(...) method to set the duration you want to keep.


fix(Data Mapper): Disabled allowing undo add schema

Please check if the PR fulfills these requirements

[X] The commit message follows our guidelines

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

What is the current behavior? (You can also link to an open issue here)

can undo adding initial schemas

What is the new behavior (if this is a feature change)?

undo is disabled until adding both of the schemas and making an action

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

Please Include Screenshots or Videos of the intended change:

This disables "undo" until both initial schemas are selected & an action is done.
When a new schema is selected in the middle of actions, the user can still "undo" the change as it is not considered as a schema change but rather a state change in multiple parts (nodes, connections, etc). If we want to disable "undo" after a schema is changed in the middle, we may want to consider emptying the entire undo-redo stack on those.


Does not creating the "Server" folder

I uzipped the rocket folder to the right place, and when I start the "MyFirstServer" it's downloading only a "Steam" and an "Unturned" folder, and there is nothing in them. What could be the problem?

The server information is all in the unturned folder u created, it will run from the new folder. Btw don't use my first rocket thing, it would be easier if u just do a normal way of setting up a server like what u do without rocket


Issue using /coverage:code

I am trying to run PSharpTester with the /coverage:code option and am getting the following error:
Error: [PSharpTester] 'VSInstr' tool 'C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise\Team Tools\Performance Tools\x64\vsinstr.exe' not found.
I have the vsinstr installed in the following path:
D:\Microsoft Visual Studio\2017\Professional\Team Tools\Performance Tools\x64
I tried to run PSharpTester from the Developer Command Prompt and made sure that vsinstr in the path.

Does it have something to do with the paths being hard-coded in the App.Config file?

I don't think this is a bug, but the intended current design. I believe this is related to the paths in the App.Config file, as you mentioned, and described here:
https://github.com/p-org/PSharp/blob/master/Docs/Testing/CodeCoverageVisualisation.md#code-coverage-binary-instrumentation
I think you need to override the environment variables as the above document mentions? @akashlal is that correct?

You can set a custom path by editing the file PSharpTester.exe.config that sits alongside PSharpTester.exe

The documentation also says that these paths can be set on the environment as well.
overridden by environment variables with the same names as the app settings:
VSInstrToolPath
VSPerfCmdToolPath

Thanks @akashlal and @pdeligia for the quick help. I will try this and re-open the issue if it doesnt work.


GameDev 2 - Guard Duty CPP

GameDev 2 - Guard Duty
Snake2109

Screenshot is unedited level.

Here is the Code, soldier.moveXY(42, 48); soldier.attack(enemy); is my added code.
// Add a soldier to the level to prevent ogres from crossing the path.
// Command the soldier using an event handler function.

// This assigns your spawned unit to the soldier variable.
auto soldier = game.spawnXY("soldier", 42, 48);
void soldierLogic() {
    // Fill in the code for the soldier's actions here.
    // Remember to use 'soldier' instead of 'hero'!
    while(true) {
        auto enemy = soldier.findNearestEnemy();
        // Attack the enemy, if the enemy exists.
        if (enemy) {
            // Units have the attack() method.
            // Use soldier.attack(enemy) method:
            soldier.attack(enemy);
        }
        // Else, move back to the starting position.
        else {
            // Units have the moveXY() method.
            soldier.moveXY(42, 48);
        }
    }

}

// This says to run the soldierLogic function when the soldier is spawned.
soldier.on("spawn", soldierLogic);

Ah, I see, an error with the C++ sample code. I've fixed it for new plays.
Try wrapping the last line in an int main() {} block:
int main() {
     // This says to run the soldierLogic function when the soldier is spawned.
    soldier.on("spawn", soldierLogic);
    return 0;
}

Thanks @Chaboi45 for the diagnosis!


feat: support passing BigInt argument

@MikeMcl this one isn't going against your will to be ES3 compatible, as it's just types and a bunch of tests. I think this should be merged and published.


Feature/initial general editing

Komet now depends on a sub project dependency knowledge layout.

This PR is replaced with the branch https://github.com/ikmdev/komet/pull/164 branch:feature/initial-general-editing2


Not an issue but a share

Hello everyone,
Sorry if this is not the right way to share with you this method of use.
I'm new to Github and I'm discovering.
Currently setting up SupportApp.
I was looking for a way to directly open the desired application and not the default mac os application to send an email and make a call.
For calls you have to go facetime fans and change preferences, it's not necessarily practical and for outlook too.
So I found this trick :
 <key>SecondRowTitleMiddle</key>
    <string>Email</string>
    <key>SecondRowSubtitleMiddle</key>
<EMAIL_ADDRESS>    <key>SecondRowTypeMiddle</key>
    <string>Command</string>
    <key>SecondRowLinkMiddle</key>
    <string>/usr/bin/open -b com.microsoft.outlook 'mailto:support-it@acme.comr?subject=Support Ticket';</string>
    <key>SecondRowSymbolMiddle</key>
    <string>envelope</string>
    <key>SecondRowTitleRight</key>
    <string>Phone</string>
    <key>SecondRowSubtitleRight</key>
    <string>+31 00 000 00 00 </string>
    <key>SecondRowTypeRight</key>
    <string>Command</string>
    <key>SecondRowLinkRight</key>
    <string>/usr/bin/open -b com.microsoft.teams 'tel:+31000000000';</string>
    <key>SecondRowSymbolRight</key>
    <string>phone.and.waveform.fill</string>

If it can help ;)

This is a nice tip that should be added to the instructions!


Title Background Not Masked Properly

The background has a rounded cone-like object that rotates with 2 different colors. These are supposed to be masked to look like sparkles and currently the full line is rendered.

Magcius determined the Texture Matrix is incorrect.
See:
Melee -
https://cdn.discordapp.com/attachments/667501465434062864/676603985024647197/unknown.png
FRAY -
https://cdn.discordapp.com/attachments/667501465434062864/676604022102425641/unknown.png
It follows the default path of setupTexCoordGen and TObjSetupMtx. Have as of yet been unable to determine cause of issue, as calls to GX_LoadTexMtxImm appear to have matching matrices. Will need to dig further.


Add info about Functions using Reverse ETL

Proposed changes
Update to include a workaround because Functions cannot be directly used with reverse ETL
Merge timing
ASAP once approved
Related issues (optional)
NA

@vanand17 your branch is way behind the current RETL branch. I created a separate PR to add in your changes so that it's in the right place. Will close this PR out.


IstioMeshGateway reconciliation does not honor spec.Deployment.Image value

Describe the bug
When reconciling an IstioMeshGateway with spec.deployment.image set, the image is not used in the deployment.
Steps to reproduce the issue:
Create
apiVersion: servicemesh.cisco.com/v1alpha1
kind: IstioMeshGateway
spec:
  deployment:
    image: my-repository.se/image-override:tag

Expected behavior
The deployment derived from the CR uses the image provided in the spec.
Additional context
Operator version: v2.11.6

#820 merged


[WIP][Issue 5157][docs] Update source and sink --processing-guarantees description

Fixes #5157
Motivation
The description for source and sink --processing-guarantees is not clear.
Modifications
Update related description for it.

@sijia-w thanks for your advice.


Pretrained weights ?

Hi
Are you planning to share pretrained weights or built some demo where I test this model.
Also have test this model with Vocos vocoder rather than speechtokenizer generated audio codec?

Ok get the pretrained model huggingface links : https://huggingface.co/PolyAI/pheme

Hey,
please see the readme. We shared two models along with the instructions on loading them.


Bug with transition, validate work only on the second submit

Versions:

VueJs: 2.3.3
Vee-Validate: 2.0.0-rc.7

Description:
When i use the vee-validate plugin with a form in a transition or transition-group the method which validate the form dont work on the first submit(ErrorBag is empty), only on the second event of the submit it fill the error array (ErrorBag detect error).
Steps To Reproduce:
Juste use any exemple of vee-validate and add it in a transition tag.
The bug dont exist in this version<EMAIL_ADDRESS>So i use the beta.25 for my project, but if you can fix that in the lastest version it would be nice.
Thx

I just noticed the same problem, with the same vee-validate version :)

Fixed with the merge of PR #616
RC.7:
https://jsfiddle.net/5995hLty
PR #616:
https://jsfiddle.net/5995hLty/1/


Update README.md

Update README.md to include suport to mac

Looks good to me.
Thank you @joaofaveri
Tip: if you add "Fixes #12" or "Closes #12" in your PR description, GitHub will automatically close the #12 issue

Looks good to me. Thank you @joaofaveri
Tip: if you add "Fixes #12" or "Closes #12" in your PR description, GitHub will automatically close the #12 issue

Thanks for the tip, @tushar5526 ! I'll use it in the next pull requests


Window resize not following cursor in wasm demo

Describe the bug
Incorrect resizing behavior when resizing a window in the egui wasm demo. The corner of the window, when grabbed, does not correctly follow the cursor.
To Reproduce
Steps to reproduce the behavior:

Go to https://emilk.github.io/egui/
Enable the "⬌ Window Resize" example
Resize the "⬌ Resizable" window

Expected behavior
The corner, when grabbed, should follow the mouse cursor.
Screenshots

Desktop (please complete the following information):

OS: macOS Big Sur 11.1
Browser: Google Chrome 88.0.4324.192

This is intended behavior. There are other resize demos that shows that you need a scroll area for the desired behavior.

oh really? that is super wild.
sorry for the false report, then! cheers

I can add some better descriptions in the demo. The short of it is: without a scroll area there is no way to resize the window because I don't want to clip the contents or shrink the text. And having a scroll area should be optional. See the resizable + embedded scroll or  resizable + scroll demo windows for  the requested behavior


Can not use by mobile

The browser could use it but mobile not support

[x] Impact by Chrome(e.g. escape)
[x] Impact by async/await
[x] Concatenate redirect
[x] Update LIFF package version check
[x] LINE app version

closed because #2cf864


pull request for e2e test at 12/04/2016 08:45:02

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

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


Add glob pattern support to gossfile

This will ease the addition of tests to an existing goss test suite.
All that is necessary is to drop in a file matching the existing pattern. As an example:
gossfile:
  /etc/goss.d/*.yaml

Unfortunately, I wasn't sure where to place this in the existing integration tests; where might a good place for such a test be?
References #222

This looks great!
For integratiin tesrs maybe change these imports: https://github.com/aelsabbahy/goss/blob/master/integration-tests/goss/wheezy/goss.yaml#L26
To: gossfile: ["../goss-s*.yaml"] in all applicable server types.

Done.
Thank you for creating goss, it's an incredibly useful tool. :)


[Bug]: The number of optimize records on the snapshots tab does not match the number of records on the optimizing tag page.

What happened?
catalog: unifiedSessionCatalog
metastore: hms
table-format: mixed-hive
optmizing Tab:

snapshots tab:

Affects Versions
master
What table format are you seeing the problem on?
Mixed-Hive
What engines are you seeing the problem on?
AMS, Optimizer
How to reproduce
1、upgrade a hive table to mixed-hive
2、crud the table
Relevant log output
No response
Anything else
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

@Aireed Thanks for reporting.
At this stage, the committing phase of Mixed-Hive table self-optimizing may indeed generate multiple snapshots. We can try to optimize this in future versions, but for now, it is expected behavior.


Move to Kubernetes 'v1' APIs

The latest Kubernetes release, 1.16, no longer serves the v1beta1 APIs by default.
Scope will no longer work with Kubernetes 1.8 and below.

It will work with versions from 1.9 to 1.16.


suggestion for filter-menu colours to have just one colour for the different elements

Jan and his colleague suggest, that we have just one colour for the different levels of the filter menu.
Worth to consider with someone who is experienced with UI design.

As that would be confusing with the way, the UI in the mobile web-app is constructed, that possibility will not be further evaluated.

For explenation: The UI of the mobile version opens the 3rd level of the Filter-Menu as overlay over the 1st and 2nd level. It would be confusing, if that had the same colour.


[ZEPPELIN-2252] discreteness of line charts

What is this PR for?
Added a discreteness in the line chart. Correcting the display of the chart, when the values does not exist at the point (by default Y is set to 0, at the moment). A value of 0 is unclear (it is real value from data or it is default value).
What type of PR is it?
Improvement
What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-2252
How should this be tested?

Spark interpreter

var text = """g1,1
g1,1
g1,2
g1,3
g1,5
g1,5
g2,1
g2,1
g2,2
g2,2
g2,6
g2,6
g2,7
g2,7
g2,7"""

val array = sc.parallelize(text.split('\n'))

case class Line(group: String, value: Integer)

val line = array.map(s => s.split(",")).map(
    s => Line(s(0),
            s(1).toInt
        )
).toDF()
line.registerTempTable("lines")

%sql
select *from lines

Settings of chart on screenshot
Screenshots (if appropriate)
before

after

Questions:

Does the licenses files need update? no
Is there breaking changes for older versions? no
Does this needs documentation? no

interesting... looks reasonable to me.
is there a way to add a test for this?

@felixcheung
i don't know. Do you have any tests for charts? i can't find.

CI red because i did squash commits and compares incorrect in travis. Reopen PR don't rebuild with new commits.

btw, you don't need to squash - it will be squashed automatically when we merge it

@felixcheung
ok, thx. i run manually https://travis-ci.org/tinkoff-dwh/zeppelin/builds/211021419 (set TRAVIS_COMMIT_RANGE in travis)

CI green https://travis-ci.org/tinkoff-dwh/zeppelin/builds/211021419

CI passed.
merging if no more comment


refactor: Update readme.

Updated readme. Idea is this API type documentation will be kept up to date as part of any future PRs and serves as the source to be copied to docs reference page. Also added some missing info for local setup and links to relevant examples.

It's looking great 👏 Did you write all of it down? Or is there some sort of lib or auto-generate script?

I wrote it manually. Couldn't find any easy/reliable tool (but open to suggestions!). Should be easy to maintain gong forward if we make it part of the PR process.


Update Chisel to 3.4.+

Changes:
scalaVersion -> 2.12.12
chisel3 -> 3.4.+
chisel-iotesters -> 1.5.+
scala-swing -> 3.0.0
sbt -> 1.3.10

This looks good to me! Thanks for the contribution!
I'm going to hold off on merging until the tests are fixed to make sure that everything still works with the new version of chisel.

This looks good to me! Thanks for the contribution!
I'm going to hold off on merging until the tests are fixed to make sure that everything still works with the new version of chisel.

This looks good to me! Thanks for the contribution!
I'm going to hold off on merging until the tests are fixed to make sure that everything still works with the new version of chisel.

I will check all the test cases later

This looks good to me! Thanks for the contribution!
I'm going to hold off on merging until the tests are fixed to make sure that everything still works with the new version of chisel.

I will check all the test cases later

Thanks! But they aren't going to work :D. The CPU designs with a branch predictor (and possibly the non-combinational memory) are quite out of date.

Thanks! But they aren't going to work :D. The CPU designs with a branch predictor (and possibly the non-combinational memory) are quite out of date.


Chat Memory and Chat History Log

I'd like a way to retrieve a log of previous chats.
Anyway thanks for this great api!

Hey, @jordanZeleny.
It's up to you to store messages history to keep chat history, please see messages property.


[REQUEST] Rename command line option zipinput as input

Is your feature request related to a problem? Please describe.
The command zipinput should be renamed as input to prevent confusion since it refers to
the path of the zip input. Hence the short name should be -i instead of z

Wasn't input renamed to zipinput in the past?

input has been renamed to extract

@lionel-nj Ah, right, thanks.


Spec: improve description of sequence field

Fixes: #926

lgtm - will wait for @defunctzombie feedback


Improve error reporting when passing invalid argument types for dialog API methods

Currently next code:
dialog.showMessageBox({
	type: 'warning',
	buttons: [ 'Cancel' ],
	message: 'Foo',
	detail: {}
});

Will produce this error, which does not make much sense:
Uncaught Error: Could not call remote function ``. Check that the function signature is correct. Underlying error: Error processing argument at index 7, conversion failure from ...

This PR adds type checks and better error reporting.

How would feel about tweaking the error messages slightly?
Something like: Title must be a string instead of Title need to be string?

Thanks for improving these errors :+1: :ship:


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_ea7df3k197/clone/recipe/meta.yaml:14: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_ea7df3k197/clone/recipe/meta.yaml:20: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_ea7df3k197/clone/recipe/meta.yaml:26: has_imports_and_run_test_py: Imports and python test file cannot coexist.
WARNING: /tmp/abs_ea7df3k197/clone/recipe/meta.yaml:36: missing_description: The recipe is missing a description
ERROR: /tmp/abs_ea7df3k197/clone/recipe/meta.yaml:36: missing_documentation: The recipe is missing a doc_url or doc_source_url
ERROR: /tmp/abs_ea7df3k197/clone/recipe/meta.yaml:36: missing_dev_url: The recipe is missing a dev_url
ERROR: /tmp/abs_ea7df3k197/clone/recipe/meta.yaml:36: missing_license_family: The recipe is missing the about/license_family key.
Errors were found


Remove support for Debian 8 / Ubuntu 14.04

We no longer need this logic since everything uses the same 'squid' service now and all our debian attributes are the defaults we already have defined here.

Releasing as a major as we're deprecating support for platforms.

Released as: 5.0.0


keyup

Version
2.6.4
Reproduction link
https://codepen.io/anon/pen/WPJWWZ#anon-signup
Steps to reproduce
直接使用不同版本vue即可
What is expected?
解决或者告知是否哪里出了问题
What is actually happening?
keyup在不同版本的vue中，对应的安卓浏览器和安卓微信下的反应不同，或支持或无效，我手中的是mix2s

it is indeed wrong, you use one version of vue not multiple ones.

Please, next time consider using the forum, the Discord server or StackOverflow for questions first. But feel free to come back and open an issue if it turns out to be a bug 🙂


Caldera Elevated Agents Keep Failing.

I've tried different deployment variations of Attack Range Local, and it seems Elevated agents in Caldera keeps on failing.
VM setups I've tried:

Ubuntu 20.04 Physical Host, with Attack Range installed directly on it using Virtualbox.
Windows 10 Host Physical Host, with a Ubuntu 18.04 Guest running in VMware on which Attack Range local is then installed using Virtualbox.

I've tried different windows hosts, and have deleted rebuilt etc a few times.
Everything else works as expected. I.e. Splunk, Caldera in general etc, but Elevated agents show up, and then goes offline after a while.
If an operation is run while an Elevated agent is still live, it shows a red cross in the Operations tab for that agents job, and the message "Internal Server Error".
Operations using User level agents runs as expected.
I'm not sure though if this is due to the local Attack Range variant, or if this is a Caldera issue?

We had a lot of problems with the stability of Caldera. We follow the installation guidelines. Therefore, I assume it's something with Caldera. Can you open an issue in their project? Thank you.

Will do, thanks.


day 8 python

Given the nature of the inputs, the problem is reduced down to finding Least common multiple for a bunch of prime, so I time them together.

Do you mind explaining your approach? I don't agree with the LCM way but I also don't quite understand what your approach here is, thanks!

I was scratching my head to try to reduce the complexity of the problem as I was thinking maybe there is multiple XXZ on the loop.
I did some printout and saw when it takes N step for each start XXA to arrive at the only XXZ and it always loop back to the position righr after the start, this means the loop size is also N. So the solution is a1N1=a2N2=...amNm for all m starts. And an LCM could be used.
But then I noticed all N are prime numbers, I was very lazy at the point and just times everything together to get the LCM.
Not the most traditional programming problem I'd say :😀

Fair enough, thanks for the explanation tho :)


Indicate Yes, No, AND not-implemented

So that people running tests have a clear indication of not just YES/Pass or NO/Fail, we may want to also include a "not implemented" value if a particular functionality is simply not implemented.
(This issue was raised in an OCI Runtime ConCall the week of 5/8 - 5/12/2017)

On Wed, May 10, 2017 at 05:55:50PM -0700, Rob Dolin (MSFT) wrote:

… if a particular functionality is simply not implemented.

Is that “not implemented in the compliance test suite”?  “Not
implemented by the runtime” seems like it should be a fail.  For
runtime-tools, opencontainers/runtime-tools#66 is something like a
list of untested spec requirements, although I'm not sure how current
it is.  I think a more sustainable approach will be filing issues (or
PRs :) for any untested conditions.  If/when
opencontainers/runtime-tools#354 lands and propagates through the test
suite, matching tests to runtime requirements will become more
straightforward.

Tests should exist for work that is described as "MAY" in the specifications. If an implementation doesn't implement something that is testable is still testable, a better test case outcome would be Not Implemented rather than Fail.

On Mon, May 15, 2017 at 03:59:32PM -0700, Stephen Walli wrote:

Tests should exist for work that is described as "MAY" in the
specifications.

Do we have anything that fits this?  For example, we currently have
1:
Unless support for a valid value is explicitly required, runtimes MAY
choose which subset of the valid values it will support.
(descended from 2).  How do you turn that into a compliance test?

If an implementation doesn't implement something that is testable is
still testable, a better test case outcome would be Not Implemented
rather than Fail.

I'm having trouble parsing “is testable is still testable”.  It would
be nice to have pretty “Tests Not Implemented by runtime-tools” errors
where the compliance suite is lacking and “Feature Not Implemented by
Runtime” errors where the runtime implementation was lacking an
optional feature.  But “Tests Not Implemented by runtime-tools” is
going to be a heavy lift for runtime-tools until
opencontainers/runtime-tools#354 lands; it seems better at the moment
to focus on coverage.
For “Feature Not Implemented by Runtime” and excepting the
valid-values wiggle room granted in the spec I quoted above, I'm not
aware of any optional features.  For caller sanity, I'd recommend any
such features get their own entry in “platforms” 3 with their own
compliance track 4 (which is why I'd argued for “protocols” over
“platforms”, opencontainers/runtime-spec#570).  But maybe “Feature Not
Implemented by Runtime” caveats are effectively the same thing (as
long as they get distributed as part of the “OCI Certified” badge).

After today's OCI CertWG call, @RobDolinMS is comfortable closing this along with the group


[WIP] Add success message to the subscribe form

WIP notes
When showing the success message the paragraph that goes above the form is hidden, so I first moved the paragraph to the component and wrapped both it and the form on a new container, that way we can simply toggle containers when we show the success message.
Now; since I didn't have the xd "file" for the screen, I used the screenshot @joelalejandro shared on Telegram and assumed the font size was the same as the current text. We should confirm if those styles are correct.
Finally, @asermax, I pushed this as a WIP because I didn't want to touch the store :P. I had a few questions and since you made the integration, you probably had an idea of how to do it than I'm not sure I'm seeing. You defined that "part of the store" just as a boolean, but you knew we were going to have a "success state", so I had a two theories:

The basic one: You were going to refactor the store.
You were going to check if isAdding changed from true to false and errors was null in order to show the message (Not sure how to this now).

Whatever you decide, let me know and I'll update the PR.
To-Do

[ ] Style validation by @joelalejandro
[ ] "Success state" validation by @asermax

I didn't really think it through, so let's go for 1. It sounds easier to hold a a status key to track success/error states and a another key to keep possible backend errors, then the component can just react to prop changes.

@homer0 Sorry for the delay, this is the XD for the full landing page implementation:
https://xd.adobe.com/spec/90d8c9e1-5a73-48c1-7aa7-8c7bc48338cc-e6e9/

🛑 Checking design...

Success state looks :100:  :ok_hand:  Some comments though (no blockers).
Desktop & Mobile:

There should be no wording in English anywhere. "Network Error" should read "Algo salió mal. Intentá de nuevo más tarde". Any error code we handle should be described in a human, user-friendly way.

Mobile:

The .index--module--middle-section shouldn't be full width, that way there's no need to use padding for the brackets. This breaks a bit both legibility and layout on screens smaller than <375px.


Harbor UI hang when getting repository details

If you are reporting a problem, please make sure the following information are provided:
Environment:
Harbor 1.2.0, http access
Ubuntu 16.04/SLES12
Docker 1.12.6
Description:
Every time when I click the ">" before the image name on the UI to get the image details, the UI gets stuck, after about 30 seconds, I get a "Unresponsive Script" pop up window from http://x5.clusters.com/static/build.min.js:47,  selecting "Stop Script" or "Continue" does not help, I have to restart the firefox web browser  to resume.
The log information posted below was the information from tail -f /var/log/harbor/<logfile_name> when the Harbor UI gets hang.

1)Version of docker engine and docker-compose.
root@x5:~# docker version
Client:
 Version:      1.12.6
 API version:  1.24
 Go version:   go1.6.2
 Git commit:   78d1802
 Built:        Tue Jan 31 23:35:14 2017
 OS/Arch:      linux/amd64

Server:
 Version:      1.12.6
 API version:  1.24
 Go version:   go1.6.2
 Git commit:   78d1802
 Built:        Tue Jan 31 23:35:14 2017
 OS/Arch:      linux/amd64
root@x5:~# docker-compose version
docker-compose version 1.9.0, build 2585387
docker-py version: 1.10.6
CPython version: 2.7.9
OpenSSL version: OpenSSL 1.0.1t  3 May 2016
root@x5:~#

2)Config files of harbor, you can get them by packaging "harbor.cfg" and files in the same directory, including subdirectory.
root@x5:~# cat harbor/harbor.cfg
## Configuration file of Harbor

#The IP address or hostname to access admin UI and registry service.
#DO NOT use localhost or <IP_ADDRESS>, because Harbor needs to be accessed by external clients.
hostname = x5.clusters.com

#The protocol for accessing the UI and token/notification service, by default it is http.
#It can be set to https if ssl is enabled on nginx.
ui_url_protocol = http

#The password for the root user of mysql db, change this before any production use.
db_password = root123

#Maximum number of job workers in job service
max_job_workers = 3

#Determine whether or not to generate certificate for the registry's token.
#If the value is on, the prepare script creates new root cert and private key
#for generating token to access the registry. If the value is off the default key/cert will be used.
#This flag also controls the creation of the notary signer's cert.
customize_crt = on

#The path of cert and key files for nginx, they are applied only the protocol is set to https
ssl_cert = /data/cert/server.crt
ssl_cert_key = /data/cert/server.key

#The path of secretkey storage
secretkey_path = /data

#Admiral's url, comment this attribute, or set its value to NA when Harbor is standalone
admiral_url = NA

#The password of the Clair's postgres database, only effective when Harbor is deployed with Clair.
#Please update it before deployment, subsequent update will cause Clair's API server and Harbor unable to access Clair's database.
clair_db_password = password

#NOTES: The properties between BEGIN INITIAL PROPERTIES and END INITIAL PROPERTIES
#only take effect in the first boot, the subsequent changes of these properties
#should be performed on web ui

#************************BEGIN INITIAL PROPERTIES************************

#Email account settings for sending out password resetting emails.

#Email server uses the given username and password to authenticate on TLS connections to host and act as identity.
#Identity left blank to act as username.
email_identity =

email_server = smtp.mydomain.com
email_server_port = 25
email_username =<EMAIL_ADDRESS>email_password = abc
email_from = admin<EMAIL_ADDRESS>email_ssl = false

##The initial password of Harbor admin, only works for the first time when Harbor starts.
#It has no effect after the first launch of Harbor.
#Change the admin password from UI after launching Harbor.
harbor_admin_password = cluster

##By default the auth mode is db_auth, i.e. the credentials are stored in a local database.
#Set it to ldap_auth if you want to verify a user's credentials against an LDAP server.
auth_mode = db_auth

#The url for an ldap endpoint.
ldap_url = ldaps://ldap.mydomain.com

#A user's DN who has the permission to search the LDAP/AD server.
#If your LDAP/AD server does not support anonymous search, you should configure this DN and ldap_search_pwd.
#ldap_searchdn = uid=searchuser,ou=people,dc=mydomain,dc=com

#the password of the ldap_searchdn
#ldap_search_pwd = password

#The base DN from which to look up a user in LDAP/AD
ldap_basedn = ou=people,dc=mydomain,dc=com

#Search filter for LDAP/AD, make sure the syntax of the filter is correct.
#ldap_filter = (objectClass=person)

# The attribute used in a search to match a user, it could be uid, cn, email, sAMAccountName or other attributes depending on your LDAP/AD
ldap_uid = uid

#the scope to search for users, 1-LDAP_SCOPE_BASE, 2-LDAP_SCOPE_ONELEVEL, 3-LDAP_SCOPE_SUBTREE
ldap_scope = 3

#Timeout (in seconds)  when connecting to an LDAP Server. The default value (and most reasonable) is 5 seconds.
ldap_timeout = 5

#Turn on or off the self-registration feature
self_registration = on

#The expiration time (in minute) of token created by token service, default is 30 minutes
token_expiration = 30

#The flag to control what users have permission to create projects
#The default value "everyone" allows everyone to creates a project.
#Set to "adminonly" so that only admin user can create project.
project_creation_restriction = everyone

#Determine whether the job service should verify the ssl cert when it connects to a remote registry.
#Set this flag to off when the remote registry uses a self-signed or untrusted certificate.
verify_remote_cert = on
#************************END INITIAL PROPERTIES************************
#############

root@x5:~#

3)Log files, you can get them by package the /var/log/harbor/ .
ui.log:

Oct 23 10:40:44 <IP_ADDRESS> ui[18588]: 2017-10-23T02:40:44Z [DEBUG] [security.go:231]: got user information from session
Oct 23 10:40:44 <IP_ADDRESS> ui[18588]: 2017-10-23T02:40:44Z [DEBUG] [security.go:240]: using local database project manager
Oct 23 10:40:44 <IP_ADDRESS> ui[18588]: 2017-10-23T02:40:44Z [DEBUG] [security.go:242]: creating local database security context...
Oct 23 10:40:44 <IP_ADDRESS> ui[18588]: 2017-10-23T02:40:44Z [DEBUG] [tokenauthorizer.go:197]: scopses parsed from request: repository:library/busybox:pull
Oct 23 10:40:44 <IP_ADDRESS> ui[18588]: 2017-10-23T02:40:44Z [DEBUG] [transport.go:50]: 200 | GET http://registry:5000/v2/library/busybox/tags/list
Oct 23 10:40:44 <IP_ADDRESS> ui[18588]: 2017-10-23T02:40:44Z [DEBUG] [tokenauthorizer.go:197]: scopses parsed from request: repository:library/busybox:pull
Oct 23 10:40:44 <IP_ADDRESS> ui[18588]: 2017-10-23T02:40:44Z [DEBUG] [tokenauthorizer.go:222]: get token for scope repository:library/busybox:pull from cache
Oct 23 10:40:44 <IP_ADDRESS> ui[18588]: 2017-10-23T02:40:44Z [DEBUG] [transport.go:50]: 200 | GET http://registry:5000/v2/library/busybox/manifests/1.27
Oct 23 10:40:44 <IP_ADDRESS> ui[18588]: 2017-10-23T02:40:44Z [DEBUG] [tokenauthorizer.go:197]: scopses parsed from request: repository:library/busybox:pull
Oct 23 10:40:44 <IP_ADDRESS> ui[18588]: 2017-10-23T02:40:44Z [DEBUG] [tokenauthorizer.go:222]: get token for scope repository:library/busybox:pull from cache
Oct 23 10:40:44 <IP_ADDRESS> ui[18588]: 2017-10-23T02:40:44Z [DEBUG] [security.go:283]: user information is nil
Oct 23 10:40:44 <IP_ADDRESS> ui[18588]: 2017-10-23T02:40:44Z [DEBUG] [security.go:296]: using local database project manager
Oct 23 10:40:44 <IP_ADDRESS> ui[18588]: 2017-10-23T02:40:44Z [DEBUG] [security.go:298]: creating local database security context...
Oct 23 10:40:44 <IP_ADDRESS> ui[18588]: 2017-10-23T02:40:44Z [DEBUG] [handler.go:137]: receive an event:
Oct 23 10:40:44 <IP_ADDRESS> ui[18588]: ----ID: b14f95a8-d7f2-40e4-aa2a-ad7e8d34fe6f
Oct 23 10:40:44 <IP_ADDRESS> ui[18588]: ----target: library/busybox:1.27
Oct 23 10:40:44 <IP_ADDRESS> ui[18588]: ----digest: sha256:545e6a6310a27636260920bc07b994a299b6708a1b26910cfefd335fdfb60d2b
Oct 23 10:40:44 <IP_ADDRESS> ui[18588]: ----action: pull
Oct 23 10:40:44 <IP_ADDRESS> ui[18588]: ----mediatype: application/vnd.docker.distribution.manifest.v2+json
Oct 23 10:40:44 <IP_ADDRESS> ui[18588]: ----user-agent: Go-http-client/1.1
Oct 23 10:40:44 <IP_ADDRESS> ui[18588]: 2017/11/23 02:40:44 #033[1;34m[router.go:829][D] | POST       | /service/notifications                   | 831.581µs        | match      | /service/notifications                   | #033[0m
Oct 23 10:40:44 <IP_ADDRESS> ui[18588]: 2017-10-23T02:40:44Z [DEBUG] [transport.go:50]: 200 | GET http://registry:5000/v2/library/busybox/blobs/sha256:54511612f1c4d97e93430fc3d5dc2f05dfbe8fb7e6259b7351deeca95eaf2971
Oct 23 10:40:44 <IP_ADDRESS> ui[18588]: 2017/11/23 02:40:44 #033[1;34m[router.go:829][D] | GET        | /api/repositories/library/busybox/tags   | 57.61673ms       | match      | /api/repositories/*/tags                 | #033[0m
Oct 23 10:40:44 <IP_ADDRESS> ui[18588]: 2017-10-23T02:40:44Z [DEBUG] [security.go:283]: user information is nil
Oct 23 10:40:44 <IP_ADDRESS> ui[18588]: 2017-10-23T02:40:44Z [DEBUG] [security.go:296]: using local database project manager
Oct 23 10:40:44 <IP_ADDRESS> ui[18588]: 2017-10-23T02:40:44Z [DEBUG] [security.go:298]: creating local database security context...
  After I upgraded to Firefox 52.4.0 and 56.0.1, the issue goes away.
I am fine with closing this issue, it will be great if we could document somewhere about the web browsers and versions that have been verified and supported

@steven-zou, do we need to add one prerequisite in the installation guide?

@ligc
We just support the latest two version of all the browsers.
@wy65701436
We can add this supporting information in the installation guide.
As issue is gone, will close this one. For other followup works, we can create new ones to track.


HIVE-28421: Iceberg: mvn test can not run UTs in iceberg-cacatlog

What changes were proposed in this pull request?

Added a missing junit5 dependency. Refer to https://stackoverflow.com/questions/46360736/tests-not-running-through-maven
Why are the changes needed?

Maybe this issue was caused by the mixed use of junit4&junit5.
For example if i want to do a mvn test locally:
mvn clean test -Dtest=TestHiveCommits  -pl iceberg/iceberg-catalog -Piceberg
will print Tests run: 0, Failures: 0, Errors: 0, Skipped: 0, and acutually the test won't be ran.
Does this PR introduce any user-facing change?

No
Is the change a dependency upgrade?

No
How was this patch tested?

Local mvn test in iceberg-catalog moudle

It's weird that UTs in iceberg-catalog are failed with ClassNotFoundException: org.apache.iceberg.mr.hive.HiveIcebergStorageHandler , bu i can run the UTs on my local develop env by cmd  mvn  test -Dtest=TestHiveCatalog-pl iceberg/iceberg-catalog -Piceberg
-------------------------------------------------------------------------------
Test set: org.apache.iceberg.hive.TestHiveCatalog
-------------------------------------------------------------------------------
Tests run: 106, Failures: 0, Errors: 32, Skipped: 11, Time elapsed: 41.97 s <<< FAILURE! - in org.apache.iceberg.hive.TestHiveCatalog
org.apache.iceberg.hive.TestHiveCatalog.testConcurrentReplaceTransactionPartitionSpec  Time elapsed: 2.614 s  <<< ERROR!
java.lang.RuntimeException: Error checking storage handler class
	at org.apache.iceberg.hive.HiveOperationsBase.isHiveIcebergStorageHandler(HiveOperationsBase.java:109)
	at org.apache.iceberg.hive.HiveTableOperations.setHmsTableParameters(HiveTableOperations.java:345)
	at org.apache.iceberg.hive.HiveTableOperations.doCommit(HiveTableOperations.java:222)
	at org.apache.iceberg.BaseMetastoreTableOperations.commit(BaseMetastoreTableOperations.java:135)
	at org.apache.iceberg.BaseTransaction.lambda$commitReplaceTransaction$1(BaseTransaction.java:381)
	at org.apache.iceberg.util.Tasks$Builder.runTaskWithRetry(Tasks.java:413)
	at org.apache.iceberg.util.Tasks$Builder.runSingleThreaded(Tasks.java:219)
	at org.apache.iceberg.util.Tasks$Builder.run(Tasks.java:203)
	at org.apache.iceberg.util.Tasks$Builder.run(Tasks.java:196)
	at org.apache.iceberg.BaseTransaction.commitReplaceTransaction(BaseTransaction.java:365)
	at org.apache.iceberg.BaseTransaction.commitTransaction(BaseTransaction.java:310)
	at org.apache.iceberg.catalog.CatalogTests.testConcurrentReplaceTransactionPartitionSpec(CatalogTests.java:2382)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at java.util.ArrayList.forEach(ArrayList.java:1259)
	at java.util.ArrayList.forEach(ArrayList.java:1259)
Caused by: java.lang.ClassNotFoundException: org.apache.iceberg.mr.hive.HiveIcebergStorageHandler
	at java.net.URLClassLoader.findClass(URLClassLoader.java:382)
	at java.lang.ClassLoader.loadClass(ClassLoader.java:419)
	at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:352)
	at java.lang.ClassLoader.loadClass(ClassLoader.java:352)
	at java.lang.Class.forName0(Native Method)
	at java.lang.Class.forName(Class.java:264)
	at org.apache.iceberg.hive.HiveOperationsBase.isHiveIcebergStorageHandler(HiveOperationsBase.java:106)
	... 14 more

It's weird that UTs in iceberg-catalog are failed with ClassNotFoundException: org.apache.iceberg.mr.hive.HiveIcebergStorageHandler , bu i can run the UTs on my local develop env by cmd mvn test -Dtest=TestHiveCatalog-pl iceberg/iceberg-catalog -Piceberg
-------------------------------------------------------------------------------
Test set: org.apache.iceberg.hive.TestHiveCatalog
-------------------------------------------------------------------------------
Tests run: 106, Failures: 0, Errors: 32, Skipped: 11, Time elapsed: 41.97 s <<< FAILURE! - in org.apache.iceberg.hive.TestHiveCatalog
org.apache.iceberg.hive.TestHiveCatalog.testConcurrentReplaceTransactionPartitionSpec  Time elapsed: 2.614 s  <<< ERROR!
java.lang.RuntimeException: Error checking storage handler class
	at org.apache.iceberg.hive.HiveOperationsBase.isHiveIcebergStorageHandler(HiveOperationsBase.java:109)
	at org.apache.iceberg.hive.HiveTableOperations.setHmsTableParameters(HiveTableOperations.java:345)
	at org.apache.iceberg.hive.HiveTableOperations.doCommit(HiveTableOperations.java:222)
	at org.apache.iceberg.BaseMetastoreTableOperations.commit(BaseMetastoreTableOperations.java:135)
	at org.apache.iceberg.BaseTransaction.lambda$commitReplaceTransaction$1(BaseTransaction.java:381)
	at org.apache.iceberg.util.Tasks$Builder.runTaskWithRetry(Tasks.java:413)
	at org.apache.iceberg.util.Tasks$Builder.runSingleThreaded(Tasks.java:219)
	at org.apache.iceberg.util.Tasks$Builder.run(Tasks.java:203)
	at org.apache.iceberg.util.Tasks$Builder.run(Tasks.java:196)
	at org.apache.iceberg.BaseTransaction.commitReplaceTransaction(BaseTransaction.java:365)
	at org.apache.iceberg.BaseTransaction.commitTransaction(BaseTransaction.java:310)
	at org.apache.iceberg.catalog.CatalogTests.testConcurrentReplaceTransactionPartitionSpec(CatalogTests.java:2382)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at java.util.ArrayList.forEach(ArrayList.java:1259)
	at java.util.ArrayList.forEach(ArrayList.java:1259)
Caused by: java.lang.ClassNotFoundException: org.apache.iceberg.mr.hive.HiveIcebergStorageHandler
	at java.net.URLClassLoader.findClass(URLClassLoader.java:382)
	at java.lang.ClassLoader.loadClass(ClassLoader.java:419)
	at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:352)
	at java.lang.ClassLoader.loadClass(ClassLoader.java:352)
	at java.lang.Class.forName0(Native Method)
	at java.lang.Class.forName(Class.java:264)
	at org.apache.iceberg.hive.HiveOperationsBase.isHiveIcebergStorageHandler(HiveOperationsBase.java:106)
	... 14 more

HiveIcebergStorageHandler is in iceberg-handler package, why iceberg-catalog is even using it?

I have checked previous CI, and found that the CI never ran the unit tests for iceberg-catalog module. So any new added UT in iceberg-catalog won't be ran, and the CI will always dispaly green though the ut in iceberg-catalog may be bad.
This PR can solve the issue and the CI can run the UTs in iceberg-catalog, but some UTs need the dependency iceberg-handler, just like the error Caused by: java.lang.ClassNotFoundException: org.apache.iceberg.mr.hive.HiveIcebergStorageHandler.
Here is my thought:

One way is adding iceberg-handler dependency  in iceberg-catalog pom, but iceberg-handler has already added  iceberg-catalog  as its dependncy, so this way will fail with cyclic dependency problem;
Another way is migrate all UTs in  iceberg-catalog to  iceberg-handler moudle, and then write new ut about  iceberg-catalog in iceberg-handler.  This way may work but looks strange :(

I have checked previous CI, and found that the CI never ran the unit tests for iceberg-catalog module. So any new added UT in iceberg-catalog won't be ran, and the CI will always dispaly green though the ut in iceberg-catalog may be bad.
This PR can solve the issue and the CI can run the UTs in iceberg-catalog, but some UTs need the dependency iceberg-handler, just like the error Caused by: java.lang.ClassNotFoundException: org.apache.iceberg.mr.hive.HiveIcebergStorageHandler.
Here is my thought:

One way is adding iceberg-handler dependency  in iceberg-catalog pom, but iceberg-handler has already added  iceberg-catalog  as its dependncy, so this way will fail with cyclic dependency problem;
Another way is migrate all UTs in  iceberg-catalog to  iceberg-handler moudle, and then write new ut about  iceberg-catalog in iceberg-handler.  This way may work but looks strange :(

I think option 2 is the way to go

@deniskuzZ Thanks for your additional change!!!


Broken on windows

The upgrade to winapi and associated changes won't build on my laptop, nor on AppVeyor.
Could you re-instate the AppVeyor and Travis CI pipelines from the original repo? As it'll help to catch this kind of thing.
https://ci.appveyor.com/project/willstott101/if-addrs/builds/35244539
If I have time in the next few days I'll try and submit a PR to fix the windows build. But re-instating CI shouldn't take more than 10 minutes and is free on both AppVeyor and TravisCI for open-source repos.
Thanks!

Actually it's even easier to just use windows on travis! https://github.com/librespot-org/libmdns/commit/3754cff299c7cca6ca72012cf5a36121c4329a42


Launcher lazily loads scripting language providers

Preventing scanning the classpath for providers in the default case where a settings.gradle file is found.

My pleasure, thanks for the quick review ;)


] word

hello,
When using the python script to autotype in qemu, it seems to get stuck after the compilation of the 27th line:
: ] 0 state@ ! ;
then I couldn't type anything else, qemu seems to hang.   (I've then modified the script to stop just before the ; to check how it would crash just after I've typed the last ;)
(using qemu 6.2.0 on Linux Mint / Debian)

Hmm.  Unfortunately, I don't have access to a Linux system, so I can't seem to reproduce this issue.
However, I have had issues with pyautogui sometimes typing garbage characters (sometimes invisible!) for no apparent reason.  If this is the root of the problem, then unfortunately that's outside the domain of this project (I'm not a pyautogui maintainer :).
On the other hand, it is certainly possible that something is wrong in the program; I'm not sure how to fix it though since I don't know what it would be.

@fuzzballcat I've tried from another computer, using the same OS (and same version of qemu), and I don't have this error.
I don't know what happened, but I suppose I can close this issue for now...


astilectron-bundler does not emit logs when output is redirected

to reproduce, try:
astilectron-bundler -v | tee bundler.log
astilectron-bundler -v >bundler.log
I'm trying to capture the output of astilectron-bundler to compare what it is doing while reproducing a bug in go-astilectron-demo, and this inability to capture that log is frustrating.

@dhawth I've pushed a fix in github.com/asticode/go-astilog that should fix this issue. I don't know which vendor workflow you're using but in any case you can try pulling master of this dependency and you should see your logs even when redirecting stdout.
I'll close this issue for now but feel free to reopen it if you're facing some issues.


BAU: Upgrade dropwizard due to vulnerability found by snyk

synk: https://snyk.io/vuln/SNYK-JAVA-IODROPWIZARD-552147

It's probably worth bumping the version in the build.gradle for common-utils and security-utils too I think.


How to clear pin  ?

Hello. How to programmatically clear pin ?

@vov4ik08 did you find a way ? I would like to as well

@vov4ik08 did you find a way ? I would like to as well

No


Documentation: How to use the SSL certificates as a service area

Updated information within gitbook: https://app.gitbook.com/o/xZRqGQeQXJ0RP4VMj7Lq/s/UMIRhLdbRQTvMWop8Il9/developer-docs/common-services/docs/developers-guide/ssl-certificate-usage
This includes manual and terraform-based certificate usage, and some notes.


Several deprecated packages

npm warn deprecated<EMAIL_ADDRESS>This module is not supported, and leaks memory. Do not use it. Check out lru-cache if you want a good and tested way to coalesce async requests by a key value, which is much more comprehensive and powerful.
npm warn deprecated<EMAIL_ADDRESS>Use @eslint/config-array instead
npm warn deprecated<EMAIL_ADDRESS>Rimraf versions prior to v4 are no longer supported
npm warn deprecated<EMAIL_ADDRESS>Glob versions prior to v9 are no longer supported
npm warn deprecated<EMAIL_ADDRESS>Use @eslint/object-schema instead
npm warn deprecated<EMAIL_ADDRESS>babel-eslint is now @babel/eslint-parser. This package will no longer receive updates.
npm warn deprecated<EMAIL_ADDRESS>This version is no longer supported. Please see https://eslint.org/version-support for other options.

These warnings are related with some outdated development dependencies (jest, eslint & semantic-release) , so, they don't have any influence on the plugin operation.
For now, I don't have time to address those (minor) warnings, but I'm open to a PR to remove them.


Txn processor tutorial and install guide for Python SDK

Creation of transaction processor tutorial that uses the new template system.
Produces a Python tutorial and install guide for the Python SDK, and enables
creation of tutorials for future supported languages.

This PR is a combined PR that replaces #403 and #422

IMPORTANT NOTE to all reviewers. This PR combines two other PR's that you already reviewed, so this PR shouldn't need a lot of changing (see note above). This combines #403 and #422.

Please close #403 if it is superseded by this PR


From Mrs Searle: Website is Inconsistent

The home page looks good but the other pages look bland without images on the landing part of the page, consider adding hero images to make the other pages more inspiring.

Added hero images to other pages, not sure it there is anything else I can do to add consistency.
Before:

After:


cleanedArticleText() returns nothing

I am using the goose-2.1.22.jar. My Java code is:
package com.pasionat.test;
import com.gravity.goose.Article;
import com.gravity.goose.Configuration;
import com.gravity.goose.Goose;
public class Main {
public static void main(String[] args) throws Exception {
String url_string = "http://www.cnn.com/2010/POLITICS/08/13/democrats.social.security/index.html";
Goose goose = new Goose(new Configuration());
	Article article = goose.extractContent(url_string);

	System.out.println("Goose text: " + article.cleanedArticleText());
	System.out.println("Goose rawHtml: " + article.rawHtml());
	System.out.println("Goose title: " + article.title());
}

}
cleanedArticleText() returns empty string, rawHTML returns the full HTML code of the page and title() returns the title of the page. So the website is being parsed and processed, but for some reason cleanedArticleText() is always empty. Is this a bug or am I doing something wrong? Thanks.

Looks like this issue was introduced here: https://github.com/GravityLabs/goose/pull/58


Unable to figure out ars nouveau ore growing

The description doesn't hint at what ars nouveau ores can be grown and also is glowstone an ore that can be grown

Ars Nouveau in version 1.16.5 allows planting Arcane Ore, but I’m sorry, I don't know if Arcane Ore was removed from Ars Nouveau in version 1.18.
Regarding Glowstone, I have added it in the latest version.


Style suggestions made

@malliwi88, why are you opening a new PR every time you push new changes??
You don't need to do that!! This is all wrong, and please don't do it again (you've done it a lot of times already!). So I'm going to close this PR and ask you to continue your work in PR #3629.
It seems you're forgetting to checkout your branch before committing, so before doing any commit, please be sure to be in the right branch :-)


SONARMSBRU-152: Print the HTTP error code and the reason phrase in an…

… error message.

SonarQube analysis reported 1 issue:

 1 major

Watch the comments in this conversation to review them.
Note: the following issues could not be reported as comments because they are located on lines that are not displayed in this pull request:

 The 'this' parameter (or 'Me' in Visual Basic) of 'CoverageReportDownloader.GetHttpHandler(string, ILogger)' is never used. Mark the member as static (or Shared in Visual Basic) or use 'this'/'Me' in the method body or at least one property accessor, if appropriate.

SonarQube analysis reported 1 issue:

 1 major

Watch the comments in this conversation to review them.
Note: the following issues could not be reported as comments because they are located on lines that are not displayed in this pull request:

 The 'this' parameter (or 'Me' in Visual Basic) of 'CoverageReportDownloader.GetHttpHandler(string, ILogger)' is never used. Mark the member as static (or Shared in Visual Basic) or use 'this'/'Me' in the method body or at least one property accessor, if appropriate.

LGTM

SonarQube analysis reported 1 issue:

 1 major

Watch the comments in this conversation to review them.
Note: the following issues could not be reported as comments because they are located on lines that are not displayed in this pull request:

 The 'this' parameter (or 'Me' in Visual Basic) of 'CoverageReportDownloader.GetHttpHandler(string, ILogger)' is never used. Mark the member as static (or Shared in Visual Basic) or use 'this'/'Me' in the method body or at least one property accessor, if appropriate.

SonarQube analysis reported 1 issue:

 1 major

Watch the comments in this conversation to review them.
Note: the following issues could not be reported as comments because they are located on lines that are not displayed in this pull request:

 The 'this' parameter (or 'Me' in Visual Basic) of 'CoverageReportDownloader.GetHttpHandler(string, ILogger)' is never used. Mark the member as static (or Shared in Visual Basic) or use 'this'/'Me' in the method body or at least one property accessor, if appropriate.

This PR is still open for some reason, however it looks like the commit has been merged into the master: https://github.com/SonarSource/sonar-msbuild-runner/commit/6dee118c90ee320da0f83f52abe2af96ce57fe9c


Add support for primitive elements in firebase-query

Description
When iterating over primitive key/value pairs, firebase-query throws the following error:
FIREBASE WARNING: Exception was thrown by user callback. TypeError: Cannot create property '$key' on string 'own'
    at firebase-query.Polymer.__onFirebaseChildAdded

The "old" firebase-collection handled this by wrapping primitive values in an object like this:
{
  value: /* original primitive value */
  __firebaseKey__: /* key of primitive value in parent document */
}

Please make the same functionality available in firebase-query.
Steps to reproduce
Create a firebase-query element in a page and set path the "users" element, something like this:
users: {
  1: true,
  2: true,
  3: false
}

Browsers Affected

[x] Chrome
[x] Firefox
[x] Safari 9
[x] Safari 8
[x] Safari 7
[x] Edge
[x] IE 11
[x] IE 10

Hi, thanks for reporting. <firebase-query> will not support queries for primitive children since it substantially complicates the implementation. The <firebase-collection> element from https://github.com/GoogleWebComponents/firebase-element was only able to achieve this behavior by wrapping primitive values inside of objects. Please consider doing this yourself in your database to achieve the same effect. Please also refer to this issue: https://github.com/firebase/polymerfire/issues/6#issuecomment-220676842


[INTERNAL] Adopt REUSE Specification 3.0

See https://reuse.software/spec/

Rebased on top of https://github.com/SAP/grunt-openui5/pull/164 to fix CI issues


Explain stable and unstable release channels in docs

Prisma releases are separated into two channels - the stable and unstable channel.
The unstable channel is released with every commit to master and therefore gives access to features and bug fixes before the stable release.
We close issues when the fix or feature is available on the unstable channel.
The release channels and related dynamics should be explained in the Prisma documentation.
Existing resources:

https://github.com/graphcool/prisma/blob/master/README.md#contributing
https://github.com/graphcool/prisma/blob/master/CONTRIBUTING.md#the-unstable-channel
https://github.com/graphcool/prisma/wiki/Release-Channels

This is documented here


Remove dependency on numpy.matrix

As numpy.matrix will be removed, we should remove SciPy's dependency in 1.1.0.

We can't stop using np.matrix until we deprecate scipy.sparse.spmatrix, at the very least.

How about going ahead and emitting a PendingDeprecationWarning in the
spmatrix constructor? That way, we can start letting people know about this
ASAP.

Is there a summary somewhere about where np.matrix is used in the API, i.e. user facing?
and which would affect users in a backwards incompatible way when switching.
I never worked much with sparse, but in those cases I did, I only used arrays on the numpy/dense side.

In the NumPy issue initialization of a np.matrix will already emit a warning so before anything it has to be silenced wherever it is used otherwise every invocation of a matrix would repeat the warning. Then spmatrix can emit its own warning if needed.
I have the feeling that we won't be able to make it to NumPy schedule for the deprecation so SciPy 1.1 might be too early for that. We can start deprecating spmatrix though.

By the way PendingDeprecationWarning will be added to NumPy v1.15 so there is quite some time ahead.

Yes it is some time ahead, but it's better to give people as much notice as
possible.  Is that what we want?
Shouldn't we just subclass from ndarray instead?

We should probably have at least provisional support for sparray (or whatever the ndarray-like spmatrix code will be called) in place before throwing deprecation warnings for spmatrix.

Revised the original post indicate the actionable items. Does someone want to label the issue?

Sorry, duplicate of #8162.