#134249289 redesign authentication ui

What does this PR do?
Redesign authentication pages adhering to the material design spec.
Description of task to be completed
This PR redesigns the login and logout pages to ensure a better user experience, it also fixes the issue with deployment to Heroku.
How should this be manually tested?
Visit the application home page and click on login or  signup.
Any background context you want to provide?
N/A
What are the relevant pivotal tracker stories?
#134249289
Screenshots (if appropriate)

Questions:
N/A

From the SignUp Screen shot above, you have a description above it saying "SIGN IN with" when you intended to sign up the user. You should correct that.

@andela-rakpan
Since we do not really put user's data in the DB, the social buttons would actually be signing in and not registering them.

Looks good.


ES5.X API changes break ESProxy

Hey,
first off i know you no longer support this project. But maybe others encountered this problem too.
We run the auth_module with ES 2.4 so far and it works fine. As we try to upgrade to ES5.X we run into this problem:
[23/Jul/2018 13:50:05] "GET /_nodes HTTP/1.1" 200 ["2733"] 0.0082
[2018-07-23 13:50:05] ERROR Rack::Lint::LintError: Content-Length header was 2733, but should be 18947
        /var/lib/gems/2.3.0/gems/rack-1.5.2/lib/rack/lint.rb:20:in `assert'
        /var/lib/gems/2.3.0/gems/rack-1.5.2/lib/rack/lint.rb:631:in `verify_content_length'
        /var/lib/gems/2.3.0/gems/rack-1.5.2/lib/rack/lint.rb:655:in `each'
        /var/lib/gems/2.3.0/gems/rack-1.5.2/lib/rack/body_proxy.rb:31:in `each'
        /var/lib/gems/2.3.0/gems/rack-1.5.2/lib/rack/handler/webrick.rb:72:in `service'
        /usr/lib/ruby/2.3.0/webrick/httpserver.rb:140:in `service'
        /usr/lib/ruby/2.3.0/webrick/httpserver.rb:96:in `run'
        /usr/lib/ruby/2.3.0/webrick/server.rb:296:in `block in start_thread'

The problems seems to be the change of the _nodes API in ES5.X.
Old _nodes response:
{
  "cluster_name": "clustername",
  "nodes": {
    "y1F7B0edRxiW5Bzoiwr0kw": {
      "name": "node1",
      "transport_address": "<IP_ADDRESS>:9307",
      "host": "<IP_ADDRESS>",
      "ip": "<IP_ADDRESS>",
      "version": "2.4.0",
      "build": "ce9f0c7",
      "http_address": "<IP_ADDRESS>:9207",
      "attributes": {
        "tag": "node1",
        "master": "false"
      },

New ES5.X response:
{
  "_nodes": {
    "total": 3,
    "successful": 3,
    "failed": 0
  },
  "cluster_name": "clustername",
  "nodes": {
    "QCDpwJg1RDW8Dkf7Bu95VA": {
      "name": "node1",
      "transport_address": "<IP_ADDRESS>:9300",
      "host": "<IP_ADDRESS>",
      "ip": "<IP_ADDRESS>",
      "version": "5.6.10",
      "build_hash": "b727a60",
      "total_indexing_buffer": 308700774,
      "roles": [
        "master",
        "ingest"
      ],

I think the auth_module just uses the 1st new part (with the amount of nodes) and thus the content-length check fails.
I dont expect a solution, but maybe some hints at which files i should look. I try to find a solution myself than and make a fork.
Thank you very much.

It amuses me that this is still in use somewhere. I doubt this would work on 5.x anyway? Isn't auth in the newest release? I didn't think I messed with the HTTP headers so it's likely an invalid HTTP request? I'd dump at TCP layer and look at the messages in wireshark.

Thank you for your response.
You are right, authentication/authorization is support out-of-the-box with Kibana5/6. However this all is behind a pay-wall with the X-Pack Subscription so it is not practical. Also Kibana 3 is just much better than all of the newer Kibana releases ;)
I´ll have a look at the network layer and see what i can find out.


tests: e2e test for HPA component (PROJQUAY-3408)

added e2e test for HPA component

@Nikita757 https://github.com/quay/quay-operator/pull/671 was merged so you gonna need to update references to "base" component to "quay" for this one to get in. Let me know when I can give this one a final review.

@ricardomaraschini Updated the references

This looks awesome! There is one final question then we can merge this. See my comment.

I've added it to increase timeout for this particular test, because otherwise 500 seconds may not be enough for everything to get up and running.


Make locating kubeconfig in example os independent

Make locating kubeconfig in example os independent. This addresses issue #1392.

 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.

Thanks

Thanks for the merge @jessesuen!


fix: Remove Java from .NET code samples

Use the appropriate .NET API for converting a string to bytes in the code snippets for file uploads.

OK cool, thanks. I'll raise a PR for that later today 👍

#7260


Query API: Set filters when unmarshalling Query Struct

Signed-off-by: Marcos Candeia<EMAIL_ADDRESS>Description
Set filters attribute on query struct when unmarshalling it. This will allow components to serialize the filters further.
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #2037
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[x] Created/updated tests
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

/ok-to-test

Complete conformance test run: https://github.com/dapr/components-contrib/actions/runs/2966810571
Complete certification test run: https://github.com/dapr/components-contrib/actions/runs/2966810563

/ok-to-test


Tooltip not avialable for Value in Smart Field - Smart Controls.

Version : Applies for all version which has Smart Controls
Issue : Tooltip is not avialable for values in the smart fields. Only TooltipLabel is available for Smart Field. Mostly clients expects tool tip only for values in the Smart Field. But Open UI5 has tool tiop for label. Is this a limitation on Smart Filed? How this can be enhanced from client side library?
Reference : https://sapui5.hana.ondemand.com/#/api/sap.ui.comp.smartfield.SmartField/overview
Expectation: Expecting Tooltip for value not for Smart Field Label.
Regards
Karthik S

Dear @kaiserschmarrn,
sap.ui.comp library is part of SAPUI5, not OpenUI5 library. It is not part of this project and this is not the place for reporting issues about it.
Regards,
Martin


Update the driver path

It's updated in https://github.com/istio/istio/pull/3370

@wattli: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
e5488df944e966939c189bcd5bcfd2c0744992d3
link
/test istio-pilot-e2e

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

comment for CLA

/lgtm

comment for CLA

all tests passed, cla/google stuck because of the sync up to istio:master got messed up.


Revert "Update pr_check.sh to deploy the archive-sync on ephemeral"

Reverts RedHatInsights/insights-ccx-messaging#215
The archive-sync component doesn't exist, and there's no service we deploy directly using ccx-messaging right now.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 85.77%. Comparing base (a49ab0e) to head (4cd89af).
Report is 10 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #228      +/-   ##
==========================================
- Coverage   86.10%   85.77%   -0.33%
==========================================
  Files          34       28       -6
  Lines        1209     1181      -28
==========================================
- Hits         1041     1013      -28
  Misses        168      168

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

I agree, but I would wait for Juan's review, as owner of the commit you're reverting. Maybe we are losing some point for this PR check

that's the idea =)


System.InvalidOperationException : Could not create an instance of type 'Google.Cloud.Diagnostics.Common.IManagedTracer'. Model bound complex types must not be abstract or value types and must have a parameterless constructor.

Hi,
using .net core 1.1, and package  <PackageReference Include="Google.Cloud.Diagnostics.AspNetCore" Version="1.0.0-alpha07" />
Mac:UDL-API$ dotnet --info
.NET Command Line Tools (1.0.1)

Product Information:
 `Version:`            1.0.1
 Commit SHA-1 hash:  005db40cd1

Runtime Environment:
 OS Name:     Mac OS X
 OS Version:  10.12
 OS Platform: Darwin
 RID:         osx.10.12-x64
 Base Path:   /usr/local/share/dotnet/sdk/1.0.1

i get the error
' System.InvalidOperationException : Could not create an instance of type 'Google.Cloud.Diagnostics.Common.IManagedTracer'. Model bound complex types must not be abstract or value types and must have a parameterless constructor.'

startup.Configure() contains app.UseGoogleTrace()
startup.ConfigureServices() contains services.AddGoogleTrace(Configuration["AppSettings:GoogleProjectID"]);
my post http controller contains tracer.StartSpan(nameof(GetWellData)); (& endspan()).

That may be an issue on my side, but the documentation at http://googlecloudplatform.github.io/google-cloud-dotnet/docs/Google.Cloud.Diagnostics.AspNetCore/  does not really give much more clue on what is going on.

If you can share a pointer to the code that would be great, but I understand that is not always possible.  If it's not public can you give me an example of where this is failing?  How are you using IManagedTracer?
My guess is you are injecting the IManagedTracer on an MVC action or other similar method that requests the "[FromServices]" attribute.
If any parts of the docs aren't clear, lacking info, ect, please let me know and I will update them.

Hi,
If you are on slack (@jonathan on the .net group) / hangouts, I can eventually show you live, but I cannot share the code.
I have setup the middlewares according documentation (useGoogleTrace & AddGoogleTrace), I am trying to use an instance of IManagedTracer by dependency injection in a asp.net core MVC controller. It builds fine but fails on runtime.
The exception happens when i try to use an integration tests that hits this endpoint.
This is the packages the integration test project has.
<ItemGroup>
    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="15.0.0" />
    <PackageReference Include="xunit" Version="2.2.0" />
    <PackageReference Include="xunit.runner.visualstudio" Version="2.2.0" />
    <PackageReference Include="Microsoft.AspNetCore.TestHost" Version="1.1.1" />
  </ItemGroup>

Let me know

Ok you nailed it quite fast. I was missing the [FromServices] attribute in the controller to completely enable the dependency injection! Now works ok.
I realize this is pure asp.net core but you may want to update the doc to mention that [FromServices] attribute is required for a controller, I believe it would help various people (the asp.net documentation also lacks information on how this could be enabled in a controller).
Let me know otherwise i ll close the ticket!
Thanks for the quick help!

@Jonathan34 Awesome glad everything is working.  I will be sure to add an example to our docs to make this a little more clear.
I'll use this to track updating the docs and close this once I have a PR in.
Thanks for the feedback!

@Jonathan34 also a note (I will add this in the docs when I update them) the IManagedTracer is a singleton.  I would suggest injecting it into the constructor of the controller and using it that way (no need for the [FromServices] attribute).


Feature Request: TCL script arguments

It would be very useful to be able to pass arguments to tcl scripts.
This makes for a lot of modularity and also function compartmentalization between scripts.
Thanks Clifford!
E.G.
yosys> tcl my_script.tcl load
yosys> tcl my_script.tcl opt
yosys> tcl my_script.tcl write

Wish granted in https://github.com/YosysHQ/yosys/pull/758.

This is implemented and can be closed now.


add keystore and truststore as options

Add keystore and truststore. It is a copy from #105 with some clean up (removed unnecessary foramatting changes in that PR). Tested in local and passed

@yunzvanessa @fsaliba-ml for reviews.

Hey Zhichun,
Did you run both mlcp unit tests and mlcp regression tests?

Unit tests all passed in local.
Regression tests in local look fine except one test will need key update


[BUG] When padding an image, the dimensions get stretched

Fix that padding was stretching the original image, and the right side was overflowing the container.
Example
This is the code I used to generate the images.
I resize the images, so that they can be padded again to the same height as the original image.
RawImage.fromBlob(file)
.then(image => image.resize((256 / image.height) * image.width, 256))
.then(image => image.pad([128, 128, 128, 128]))
.then(image => image.save('padded.png'));

Original image
Old solution
New solution

As you can see above, the old solution will add the 128 pixel padding to the top and left, however the image is resized and drawn with the same size as the canvas, hence it is cropped by 128 pixels on the right and bottom.
However, the new solution will correctly draw the image with the provided padding all around as we now draw the image with the same dimensions that it was passed in with.
Explanation
The current implementation was drawing the image with a width of newWidth, where newWidth will be the entire width.
This is incorrect, since the width should remain unchanged when drawing, otherwise the image will become stretched.
As a result, we are getting the correct left padding and top padding, but the right and bottom of the image are getting cropped off since the image is being stretched out.
From the docs, the sWidth and sHeight should match dWidth and dHeight.

Hi there, thanks for the PR! 👋 Can you provide some example code/input to see the incorrect behavior observed, and then the proposed corrected behaviour?

@xenova, I have updated the original PR to demonstrate how the proposed changes fix the previous bug.
I hope this is clear enough, although if you need further explanation please let me know.
Thank you.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


reference_templates dir and first reference vpc

With this commit we gain a troposphere implementation of a secure VPC
containing dual subnets in two availability zones. Troposphere was run
against the implementation and the generated json file is checked in for
convenience. This json file may be used directly by cloudformation.
Further documentation of the VPC architecture is in
vpc-dual-az-with-nat.py

This is the same as pull request 15, but post squashing.


typoclearly describe the rename syntax

Proposed changes
Issue Number: close #xxx
<--Describe your changes.-->
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run p0


Use parameter struct for configuring QUIC streamer

Problem
Methods with too many complicated parameters and duplication of parameterizing the quic server.
Summary of Changes
Add a parameter struct to pass the QUIC server parameters.
Fixes #

@sakridge I merged this since I'm doing changes that would conflict otherwise

@sakridge I merged this since I'm doing changes that would conflict otherwise

great, thanks

these change break a public interface with external consumers. we should not be changing the function signatures like this
prefer something like spawn_server_with_config that spawn_server calls after constructing the config. then we deprecate spawn_server
recommend revert and resubmit

these change break a public interface with external consumers. we should not be changing the function signatures like this
prefer something like spawn_server_with_config that spawn_server calls after constructing the config. then we deprecate spawn_server
recommend revert and resubmit

hmm. yes, the jito-relayers were using the interfaces.

these change break a public interface with external consumers. we should not be changing the function signatures like this
prefer something like spawn_server_with_config that spawn_server calls after constructing the config. then we deprecate spawn_server
recommend revert and resubmit

When did we every bless these as consumable? Not every function we make public in a crate is intended to be consumed, and if so it's at your own risk of it being changed.

https://crates.io/crates/solana-streamer/reverse_dependencies?page=3 -- only jito relayers are using it directly. I guess it won't be too difficult to correct the code.

When did we ever bless these as consumable? Not every function we make public in a crate is intended to be consumed, and if so it's at your own risk of it being changed.

Yeah I think I raised this a couple of times on discord, but by now I'm running with the assumption that all these crates and symbols are not really API unless they're re-exported by on chain stuff (which we should never break). Like... in this quic module we're even exporting rt which is clearly an internal util.


By default, only trigger on_prometheus_alert for firing alerts

This changes the default behaviour for on_prometheus_alert so that it only triggers for currently firing alerts, unless configured otherwise.
Rationale
When configuring playbooks with on_prometheus_alert, most users don't want to run the action on resolved alerts. The previous default behave led to surprising behaviour. We know this because we received questions from users on why their playbook actions were failing and it was because they were unknowingly running the actions on resolved alerts. The same happened to me when I did some work on Prometheus alerts recently.
Docs
This change will need to be communicated in the release notes. I suggest the following text:
Breaking: on_prometheus_alert will no longer trigger by default on resolved alerts. This can be overriden if you need. See the docs for details
Other
This PR also includes a tiny fix for Sphinx documentation. The trigger classes didn't inherit from DocumentedModel so the following output was not shown properly in the docs:

Now it's fixed.

CC @Sheeproid as I know @arikalon1 discussed this with you too.


Deprecate send_message

A few things happened here, all related to bringing it up to the latest version of slack-ruby-bot.

Add celluloid because concurrency errors
Add updated Gemfile.lock with latest working version
Use client.say for send_fields
Edit the send_gif references to be client.say
Add spacing according to robocop

Also please note the "does not respond to sad face" test cases under this fail, I don't know enough about ruby test cases to know what's going on with that

Gem::InstallError: nio4r requires Ruby version >= 2.2.2.
Travis CI build doesn't have latest version of Ruby

I'll happily merge once the build is fixed, can help you with the tests.
For Travis, change the ruby version to 2.3.1 or 2.4.0 in https://github.com/slack-ruby/slack-aws/blob/master/.travis.yml.

A lot has changed underneath in slack-ruby-bot, if you want to fix that test it should look pretty much like https://github.com/dblock/slack-gamebot/blob/master/spec/slack-gamebot/commands/unknown_spec.rb. That said I think we can just remove it, it's testing that you can send some garbage to the bot and it doesn't respond, something that is already tested in the slack-ruby-bot library.

Taken care of.

Merged, thank you. Maybe contribute the generic parts?

Probably will in the near future.

Casual update: Working on this now


[bug]:  git.io/zi-loader url is not working

Environment
Device: ubuntu | linux-gnu | x86_64 |  /bin/bash | zsh | ubuntu/5.8-3ubuntu1.1
Reproduction steps
My zi loader is crashing.

Expected behavior
to load

Current behavior
/home/raza/.zshrc:134: command not found: zzinit
/home/raza/.zshrc:136: command not found: zi
/home/raza/.zshrc:136: command not found: zi
/home/raza/.zshrc:137: command not found: zi
/home/raza/.zshrc:137: command not found: zi
/home/raza/.zshrc:138: command not found: zi
/home/raza/.zshrc:138: command not found: zi
/home/raza/.zshrc:139: command not found: zi
/home/raza/.zshrc:139: command not found: zi
/home/raza/.zshrc:140: command not found: zi
/home/raza/.zshrc:140: command not found: zi
/home/raza/.zshrc:141: command not found: zi
/home/raza/.zshrc:141: command not found: zi
/home/raza/.zshrc:142: command not found: zi
/home/raza/.zshrc:142: command not found: zi
/home/raza/.zshrc:143: command not found: zi
/home/raza/.zshrc:144: command not found: zi
/home/raza/.zshrc:144: command not found: zi
/home/raza/.zshrc:145: command not found: zi
/home/raza/.zshrc:145: command not found: zi
/home/raza/.zshrc:146: command not found: zi
/home/raza/.zshrc:146: command not found: zi
/home/raza/.zshrc:147: command not found: zi
/home/raza/.zshrc:147: command not found: zi
/home/raza/.zshrc:148: command not found: zi

Code snippet
typeset -Ag ZI
typeset -gx ZI[HOME_DIR]="${LOCAL_CONFIG}/zi" ZI[BIN_DIR]="${ZI[HOME_DIR]}/bin"
command mkdir -p "$ZI[BIN_DIR]"
source <(curl -sL git.io/zi-loader); zzinit

Additional information
zi analytics is not loading
Self-service

[ ] I'd be willing to address this documentation request myself.

Have you read the Contributing Guidelines?

[X] I have read the Contributing Guidelines.

Are you familiar with the Contributor Covenant Code of Conduct?

[X] I have read the Contributor Covenant Code of Conduct.

Contact Details
No response

Hi @rationalthinker1,
It seems the git.io URL has been disabled, I will double-check what is going on. As a quick fix you can use alternative links: source <(curl -sL init.zshell.dev); zzinit

Got it. I loaded the init.zsh locally, Thank you.

@ss-o, sorry to bother you, but can you change the installation page to update the url? https://wiki.zshell.dev/docs/getting_started/installation

Perfect. Thank you.


disable testMultipleProcessesTryingToInitializeSchema in debug mode

since it occasionally fails on CI and its failure doesn't indicate something inherently broken with the patch being tested.
Addresses #4585
This was originally filed as #4781, but we closed that in the hopes that https://github.com/realm/realm-core/pull/2552 would have addressed this occasional failure, but this was seen again after integrating that here: https://ci.realm.io/job/objc_pr/5006/configuration=Debug,target=osx,xcode_version=8.0/

Just had another two failures in #4920 (Jenkins run). Merging this as we just ignore those failures anyway.


Support error filtering by level

In docutils, the reporters allow for filtering of errors via report_level. While we are circumventing this part of their system, we can reproduce this behavior.
errors = [error for error in errors if error.level <= self.report_level]

In docutils, these levels are 1 (most detailed) to 5 (no details) and can be accessed via the CLI with --report, -r <level>
http://repo.or.cz/w/docutils.git/blob/422cede485668203abc01c76ca317578ff634b30:/docutils/docutils/frontend.py#l429

I should clarify, this is only practical for defining on the CLI. When using this on an API basis, an end user can set up their own filtering via the same list comprehension as above.

This has been implemented by @peterjc in #37 and released in 0.18.0 :tada:


[feat] Check for transaction inputs duplicates

Context
Currently, we have no protection against double spending, moreover transactions can spend outputs that were never part of the chain in the first place.
In order to address that we are introducing two mechanisms:

Utreexo accumulator, which is a commitment to the current UTXO set
Local cache for UTXO consumed within the same block they were created

The latter partially fixes the original problem: you cannot spend the same cached UTXO twice, because on the second attempt it won't be in the cache.
Utreexo does solve the problem with "non-existent" UTXOs: you can only reference outputs for which it is possible to generate inclusion proofs. However, nothing prevents you from referencing such output twice — the inclusion proofs are still valid! So we need additional protection mechanism.
Task
We need to memoize all the UTXOs we spent during the block application and check for double spending. Note that this will automatically prevent transaction duplicates.
Implementation hints
Currently we have pub cache: Felt252Dict<bool> in UtxoSet which stores "true" for cached yet unspent outputs.
We can upgrade this dict to store information about both spent and unspent outputs (recall that Felt252Dict gives default values for non-existent keys, so e.g. you cannot distinct between None and false in our case). Just use u8 instead of bool and it should be at no extra cost.
The workflow would look as follows:

For every transaction input that references non-cached input we ensure that the outpoint is not yet spent and then add it to the dict
For every transaction input that references cached input we ensure that unspent output exists and then remove it from the dict
For every cached transaction output we ensure that it is not yet in the dict and then add it as unspent

Tests
Add unit tests covering happy and unhappy scenarios.

Hi @m-kus, I'm available for that task


Crashed on WebSocket.swift line 202

Crashed on WebSocket.swift line 202
public func read() -> Data? {
    let buf = NSMutableData(capacity: BUFFER_MAX)
    let buffer = UnsafeMutableRawPointer(mutating: buf!.bytes).assumingMemoryBound(to: UInt8.self)
    let length = inputStream!.read(buffer, maxLength: BUFFER_MAX)
    if length < 1 {
        return nil
    }
    return Data(bytes: buffer, count: length)
}

inputStream is nil
Full stacktrace of Starscream is below:
0  Starscream                     0x1070f98f0 FoundationStream.read() -> Data? (WebSocket.swift:202)
1  Starscream                     0x1070fa350 FoundationStreamStarscreamWSStream (WebSocket.swift)
2  Starscream                     0x1070fcfbc WebSocket.processInputStream() -> () (WebSocket.swift:674)
3  Starscream                     0x1070fe92c WebSocketStarscreamWSStreamDelegate (WebSocket.swift:637)
4  Starscream                     0x10710c068 specialized FoundationStream.stream(Stream, handle : Stream.Event) -> () (WebSocket.swift:239)
5  Starscream                     0x1070f9e2c @objc FoundationStream.stream(Stream, handle : Stream.Event) -> () (WebSocket.swift)

This has been improved in version 3.0.3 that has just been released. Also make sure to avoid any race conditions when handling the WebSocket object, but 3.0.3 should help.


Change the background color of a checkbox ?

Hi, I need to be able to dynamically (override) the background color of a checkbox.
Please advise,
Barry

Do you mean the entire control including the label or checkbox only?

background of the checkbox now changable 9e90d4e2991b5884620f1be1625c95165b8c55bd


Save & Restart off the Editor from themes section doesn't ask to save the empty scene.

Godot version:
Godot 3.1 build caa141a1ac630144e441f89689baa1a74bb3bae5
OS/device including version:
Manjaro 64 bit
Issue description:
Save & Restart off the Editor from themes section doesn't ask to save the empty scene.

Steps to reproduce:

Create a empty scene and add somethings don't save it yet.
Goto the Editor themes and change something and save & restart.

Duplicate of https://github.com/godotengine/godot/issues/43302. The original issue is still present – any project/editor settings that require an editor restart can result in losing unsaved changes if you're not careful.


Jackpot doesn't Work

Buyrate jackpot ist never synced and only one person ever has it(client or host).
So whenever someone gets the Jackpot Alert(even if it's the host) it's never truly the actual buyrate.

I've been unable to replicate this issue - all RNG is based off of StartOfRound.Instance.randomMapSeed so it should be identical for all players since everyone is on the same seed. It should also maintain the same rates when reopening the game as you'd still be on the seed that you left on.
However, the mod doesn't have config syncing and my guess is that someone in your lobby had different config values, resulting in mismatched rates. Are you able to confirm if that's the case?
I may look into config syncing to avoid this happening in the future.

Impossible, I always share my profile code with my friends before every session
Im guessing it's a mod incompatibility

Could you share your profile code so I can look for incompatibilities?

018d1dfc-a819-12d5-73e2-5de62f0205ad

I may have figured it out, I think it was an oversight on my part. One of the conditions for determining if you get a jackpot or not was using DateTime.Now.Millisecond, causing the rate result to be different because it would get triggered milliseconds later than it was triggered for another player. I didn't get the issue on smaller modpacks, but larger modpacks seem to cause a very slight desync between players due to all the extra stuff going on.
I've changed the condition to utilize the map seed instead, so all RNG should be seed dependent in the next update. I'm also getting config syncing setup so I'll have an update ready soon. Thank you for reporting this!


[Planner]Apply inner join condition reorganization rule

fix #3908

@morningman @wutiangan
Fix the NullPointerException of the variable father in this PR

@wutiangan Are you still work on this PR?

Please rebase the master and add an UT for this PR.


log_client_version failure in self._usage_db.execute

Debian 10, installed versions
magic-wormhole                        0.11.2-1
magic-wormhole-transit-relay          0.1.2-1
python3-magic-wormhole-mailbox-server 0.3.1-2
Running wormhole with --transit-helper and --relay-url pointing to my own server.
Failure seen on mailbox-server:
  File "/usr/lib/python3/dist-packages/wormhole_mailbox_server/server.py", line 186, in log_client_version
    self._usage_db.execute("INSERT INTO `client_versions`"
builtins.AttributeError: 'NoneType' object has no attribute 'execute'

Apparently self._usage_db is undefined?
I hacked it to work by editing wormhole_mailbox_server/server.py
and in function log_client_version
I disabled the self._usage_db.execute() and self._usage_db.commit() calls
(because I have the impression this is just for logging purposes, but not required for transfer functionality).
With these lines disabled, I can successfully transfer through my server.

I see you already fixed this with
https://github.com/magic-wormhole/magic-wormhole-mailbox-server/commit/60c924a28c35370a9904bd63b28da49591aa92fb


Make startup config optional for the Ankaios server start

Description
The Ankaios server executable currently requires a startup config to be provided as a command line argument. This is not a hard requirement as the server can start also with an empty startup state. To allow this, we have to make the startup config optional.
Optionally we could also think about accepting standard in for the config.
Goals
Not requiring a start up config makes it easier to start the Ankaios server. Workloads can be added afterwards via the ank cli.
Tasks

[ ] update design (incl. requirements)
[x] make the startup config optional for the server start #132
[x] think about allowing standard in for passing the config
[ ] check if we need new system tests
[ ] check if we need to update the user documentation

That sounds good and also helps us with #38 as we do not have to provide dummy startup state when installing Ankaios.

I am currently checking how to implement the task to read config from the stdin. I have checked behavior of our CLI regarding ank set state (if it can read from the stdin or not). I have noticed interesting behavior. If I run ank set state currentState (i.e. set the state without any file given), it deletes all workloads. I am not sure, whether this behavior is wanted. Try following script (In the debug folder. It requires a file startConfig.yaml to be present) :
#!/bin/bash
./ank-server -c startConfig.yaml &
sleep 5
./ank-agent --name agent_A &
./ank-agent --name agent_B &
echo "Waiting for Ankaios to create all workloads"
sleep 30

echo "Setting empty state (without a file)"
./ank set state currentState
sleep 10
echo "Current workloads:"
./ank get workloads

pkill -f ank-
ps -a

echo "Done"

The command ./ank get workloads prints empty table (i.e. no workload in the Ankaios).
The question is how this shall behave and of course how to implement "reading from stdin" in the server. Shall we avoid something like this? Or shall be do it intentionally? Like no configuration via file or stdin given = start "empty server".

To be tested/checked:

no config file with startup state
empty file with startup state

Regarding "allowing start config via  stdin". We have discussed in team. Currently we do not plan to let the start config to be configurable via stdin. It is enough to have the start configuration configurable this way:

with the command line parameter
no startup configuration and set the configuration from the CLI

We need to change/extend following requirement:
https://github.com/eclipse-ankaios/ankaios/tree/main/server/doc/swdesign#server-loads-startup-state-from-a-file

The pull request has been reviewed and merged into main.


Support for targeting partitions

SUMMARY
One cannot target partitions on the Netscaler.
Only the default partition can be manipulated right now.
ISSUE TYPE
Feature Idea
Allow support for specifing the partition name/id as a var
Much like this module
https://github.com/networktocode/netscaler-ansible/blob/master/library/netscaler_server.py#L615-L624
COMPONENT NAME
citrix_adc_server.py
ADDITIONAL INFORMATION
This feature would allow users to target partitions on Netscalers. Many users like myself would have multiple partitions configured and the default partition is not used al all.
This "netscaler_server" module from networktocode has the right idea.
Be great if this was something that came out of the box.
https://github.com/networktocode/netscaler-ansible/blob/master/library/netscaler_server.py#L615-L624

The way to address this issue at the moment is by using the citrix_adc_nitro_request module to login switch partition and then apply configuration to the partition.
This is showcased in the following playbook.
- hosts: citrix_adc
  gather_facts: no

  tasks:
    - name: Login
      delegate_to: localhost
      citrix_adc_nitro_request:
        nsip: "{{ nsip }}"
        operation: action
        action: login
        resource: login
        attributes:
          username: nsroot
          password: nsroot
      register: result

    - name: Save session id
      set_fact:
        sessionid: "{{ result.http_response_data.cookies.sessionid }}"

    - name: Switch partition
      delegate_to: localhost
      citrix_adc_nitro_request:
        nsip: "{{ nsip }}"
        nitro_auth_token: "{{ sessionid }}"
        operation: action
        action: switch
        resource: nspartition
        attributes:
          partitionname: george_test

    - name: Create server in partition
      delegate_to: localhost
      citrix_adc_nitro_request:
        nsip: "{{ nsip }}"
        nitro_auth_token: "{{ sessionid }}"
        operation: add
        resource: server
        attributes:
          name: george_test
          ipaddress: <IP_ADDRESS>

Support for the nitro_auth_token attribute, which allows for switching partitions and applying configuration to them, is currently limitied to the citrix_adc_nitro_request module and the newer appfw modules.
We will need to add this feature to the older modules as well to facilitate this functionality.
We will be keeping track of this issue and address it on the first opportunity.

Thanks, your suggestion to use citrix_adc_nitro_request is just fine, and works as expected.
I'm unblocked now, feel free to close this issue or leave open until the features you mentioned are added.
Thanks!

Any plans to add this feature to the Citrix ADC modules? It would be much more convenient to use than the nitro_request, since the modules work in an idempotent way  already.


Hotfix/byu web view

This change is

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

byuSuite/Classes/ReusableUI/ByuWebView.m, line 44 at r1 (raw file):

-(void)webViewDidFinishLoad:(UIWebView *)webView {
    [self requestFinishedOrFailed];
    if([self.delegate respondsToSelector:@selector(webViewDidFinishLoad:)]) {

Check to see if this selector is required. If so, we can remove the if statement (same below). Also, run the formatter on this file

Comments from Reviewable

Review status: 1 of 3 files reviewed at latest revision, 1 unresolved discussion.

byuSuite/Classes/ReusableUI/ByuWebView.m, line 44 at r1 (raw file):
Previously, romrell4 (Eric Romrell) wrote…
Check to see if this selector is required. If so, we can remove the if statement (same below). Also, run the formatter on this file

Done.

Comments from Reviewable

Reviewed 2 of 2 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


fix: Add placeholder for ACME in checkout page (#

 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.


Cloud App Security features

I read the licensing guide on this. Thanks, that explains a lot already, but since you are using the word "subset" so often, an overall table  of versions, features and licenses would be great.

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

ID: 76a66643-56f5-dc00-5af6-3ee1d9b6f090
Version Independent ID: c319c4a4-974e-7e3a-d58a-76fd57a9efe9
Content: Differences between Cloud App Security and Office 365 Cloud App Security
Content Source: CloudAppSecurityDocs/editions-cloud-app-security-o365.md
Service: cloud-app-security
Product: ****
Technology: ****
GitHub Login: @shsagir
Microsoft Alias: shsagir

@BakkerJan Thanks for your feedback. We've submitted a documentation change based on your suggestion, and the updated article should be live by tomorrow. Please let us know if there's anything more we can do for you. To follow up on this issue, leave us a comment with an @mention.


Add speaker avatars

That would require to have their email in DB to fetch their corresponding gravatar, but that would be warmer.

We have the email hash in logoUrl so that should work. And in any case we need to import emails in PROD.

We now use the logoUrl hash but hashes of new accounts are not in the db would be nice to generate and add them since we have not yet the emails in database.

Done


Cached Autotuned Result Incosistency

I am getting different output results than expected when using triton_dejavu.autotune.
To reproduce you can pip install : https://github.com/mobiusml/hqq.git and build locally: https://github.com/AnswerDotAI/gemlite/tree/dejavu.
For example if there is no cache (cache.json) exists already the below python script runs fine, but after caching it fails. Even the outputs from 2 consequent kernel launches are different when cache is used, which are  denoted as gemlite_output1 and gemlite_output2 below.
import torch
from hqq.core.quantize import HQQLinear, BaseQuantizeConfig, Quantizer
from gemlite.core import DType, GemLiteLinear

size = (2048,2048)
bitblas_dtype = torch.float16
GROUPSIZE = 128
NBITS = 4

quant_config = BaseQuantizeConfig(nbits=NBITS,
									group_size=GROUPSIZE,
									quant_zero=False,
									quant_scale=False,
									offload_meta=False,
									view_as_float=False,
									axis=1)

m = torch.nn.Linear(*size, bias=False, dtype=torch.bfloat16)
hqq_linear = HQQLinear(m, quant_config, compute_dtype=torch.bfloat16)
W_est = hqq_linear.dequantize().clone()#.to(bitblas_dtype)
W_q_unpacked = Quantizer.unpack[hqq_linear.meta['packing']](hqq_linear.W_q)
scale, zero, shape = hqq_linear.meta['scale'], hqq_linear.meta['zero'], hqq_linear.meta['shape']
scale = scale.to(bitblas_dtype)
zero = zero.to(bitblas_dtype)

gemlite_linear = GemLiteLinear(
    NBITS, #supported: [8, 4, 2, 1]
    group_size=GROUPSIZE, # any group_size divisible by 32
    in_features=size[0], # input size
    out_features=size[1], #ouput size
    input_dtype=DType.FP16, #FP16 or BF16
    output_dtype=DType.FP16, #FP16 or BF16
    acc_dtype=DType.FP16, #FP16 or FP32
)

#Packing: we follow the same format as hqq (https://github.com/mobiusml/hqq/)
gemlite_linear.pack(W_q_unpacked, scale, zero)

# set torch seed
torch.manual_seed(42)
x = torch.randn(2, size[0]).to(bitblas_dtype).cuda(); x
hqq_output = x.to(torch.bfloat16) @ W_est.t()

gemlite_output1 = gemlite_linear.forward_manual(x, matmul_type="GEMV")
try:
    assert torch.allclose(gemlite_output1, hqq_output.to(bitblas_dtype), atol=1e-2, rtol=1e-2)
except:
    print("Original vs Triton are different")
    print("Actual: ", gemlite_output1)
    print("Expected: ", hqq_output)
    print()

gemlite_output2 = gemlite_linear.forward_manual(x, matmul_type="GEMV")
try:
    assert torch.allclose(gemlite_output2, hqq_output.to(bitblas_dtype), atol=1e-2, rtol=1e-2)
except:
    print("Original vs Triton are different")
    print("Actual: ", gemlite_output2)
    print("Expected: ", hqq_output)
    print()

try:
    assert torch.allclose(gemlite_output1, gemlite_output2, atol=1e-2, rtol=1e-2)
except:
    print("Dejavu outputs are different")
    print(gemlite_output1)
    print(gemlite_output2)
    print()

{
  "signature": "JITFunction(gemlite.triton_kernels.gemv_A16fWnO16f_int32packing:gemv_A16fWnO16f_int32packing_kernel)",
  "total_bench_time_s": 33.92143940925598,
  "evaluated_configs": 16,
  "cache": {
    "('2', '2048', '2048', '128', '4', 'torch.float16', 'torch.int32', 'torch.float16', 'torch.float16', 'torch.float16')": "BLOCK_SIZE_M: 1, BLOCK_SIZE_N: 256, BLOCK_SIZE_K: 32, num_warps: 2, num_ctas: 1, num_stages: 1, maxnreg: None"
  },
  "timings": {
    "('2', '2048', '2048', '128', '4', 'torch.float16', 'torch.int32', 'torch.float16', 'torch.float16', 'torch.float16')": {
      "values": [
        0.018432000651955605,
        0.018432000651955605,
        0.01945599913597107
      ],
      "lables": [
        "ms",
        "min_ms",
        "max_ms"
      ],
      "rep_t_ms": 100,
      "warmup_t_ms": 200
    }
  }
}

Hi Kerem,
thanks for reporting this, I'll look into it.
But it would help me if you could post the output of your script executed with TRITON_DEJAVU_DEBUG=1?

@bringlein thanks for looking into it! Here are the outputs of 2 consecutive runs of the script, first time there is no cache and tests pass, the second time it uses the cache but fails.
root@4da9ee2345a5:/workspace/git/kerem_research/sglang# python debug_dejavu.py
[Triton Dejavu:WARNING] use of 'prune_configs_by' could influence the autotuner decision in a way not visible to triton-dejavu. Please ensure that configs could be reused.
root@4da9ee2345a5:/workspace/git/kerem_research/sglang# TRITON_DEJAVU_DEBUG=1 python debug_dejavu.py
[Triton Dejavu:WARNING] use of 'prune_configs_by' could influence the autotuner decision in a way not visible to triton-dejavu. Please ensure that configs could be reused.
[triton-dejavu] restored 1 configurations for gemv_A16fWnO16f_int32packing_kernel/autotune_config-d024ac9abd3a8098c64c928822f4a703adddd8b7084df0769a5844813a43ca59/kernel_configs-abeac919de3416fafac42ccb8ee5ae745764eca2e3ebf61898e0391df4129068/code_version-deea26f1e06c1d69417f61b1c2778feb4cad86408585b8ef32a0a7e7ffa8be47/default.
Original vs Triton are different
Actual:  tensor([[ 2.0586,  2.3301,  1.8613,  ..., -1.2754, -1.3750, -1.2500],
        [-3.3027, -0.5957,  0.9829,  ..., -1.7705,  1.0059, -0.9121]],
       device='cuda:0', dtype=torch.float16)
Expected:  tensor([[ 0.0737,  0.3887,  0.0112,  ..., -0.0162,  0.4570,  0.4238],
        [-1.5234,  0.7617, -0.6680,  ..., -0.2578, -0.8555,  0.1533]],
       device='cuda:0', dtype=torch.bfloat16)

Original vs Triton are different
Actual:  tensor([[ 0.0723,  1.7871,  0.0120,  ...,  1.7715,  0.4561, -1.0928],
        [-1.5264, -1.0244, -0.6675,  ...,  1.4697, -0.8569,  1.8643]],
       device='cuda:0', dtype=torch.float16)
Expected:  tensor([[ 0.0737,  0.3887,  0.0112,  ..., -0.0162,  0.4570,  0.4238],
        [-1.5234,  0.7617, -0.6680,  ..., -0.2578, -0.8555,  0.1533]],
       device='cuda:0', dtype=torch.bfloat16)

Dejavu outputs are different
tensor([[ 2.0586,  2.3301,  1.8613,  ..., -1.2754, -1.3750, -1.2500],
        [-3.3027, -0.5957,  0.9829,  ..., -1.7705,  1.0059, -0.9121]],
       device='cuda:0', dtype=torch.float16)
tensor([[ 0.0723,  1.7871,  0.0120,  ...,  1.7715,  0.4561, -1.0928],
        [-1.5264, -1.0244, -0.6675,  ...,  1.4697, -0.8569,  1.8643]],
       device='cuda:0', dtype=torch.float16)

Also, does it work as you expect it, if you use the normal triton autotuning?
Yes, regular autotuning works.

Hi @KeremTurgutlu,
sorry I only had the time today to look into it again.
Sadly, I can't reproduce your issue, because the local build of gemlite always fails with:
File "/usr/local/lib/python3.10/dist-packages/torch/utils/cpp_extension.py", line 1985, in _get_cuda_arch_flags
      arch_list[-1] += '+PTX'
IndexError: list index out of range

(even using torch nightly, etc...building based on nvidia/cuda:12.4.1-devel-ubuntu22.04)
Could you maybe provide your Dockerfile?
However, I tried to analyze your code and I think I found the cause:
You define pre_hook=init_to_zero("c_ptr") for every config gemv_A16fWnO16f_int32packing.py#L54 individually.
Right now, triton-dejavu can only restore pre/post hooks if there are defined at autotuner level, using the optional arguments in autotuner.py#L323-L331.
So, if this is the cause, then you could avoid the problem by adding pre_hook=init_to_zero("c_ptr") to your autotuner definition in line 65. Can you confirm this?
I will add support for individual config pre hooks with the next version of triton-dejavu in the comming weeks.

@bringlein thanks for looking into this! I tried the following changes: https://github.com/mobiusml/gemlite/commit/1bedc1ba2a4f9f9c24996d613e2fc1654ccd79a9
This time when I run the following code with a cached config it works sometimes and doesn't - still there is some inconsistency but I think the changes are in the right direction.

Good to hear!
But IMHO, these remaining inconsistencies are then most likely due to the observation that the output array of your triton kernel is not written completely by the kernel? Hence, depending on what is in the array before, the result is different...and autotuning causes many iterations of the kernel to be executed and therefore, there is a higher chance of the "correct" result being in the output array? Which would then be unrelated to the autotune caching.

Hi @KeremTurgutlu, as promissed, I implemented the support for individual pre_hooks last week: https://github.com/IBM/triton-dejavu/commit/d5254f94a825779372944a4ac11e82af2374e9bf
Maybe you could give it a try.


oded/master

add mixpanel.time_event
info: https://mixpanel.com/help/reference/javascript-full-api-reference#mixpanel.time_event

Can you please merge this PR ?


More Examples of orderBy:

orderBy: "popularity_1year"
What other examples can we use, like weekly? daily? monthly? any other examples we can use here?

Hi, you can see the available values here under order_by parameter: https://docs.movieofthenight.com/resource/shows#search-shows-by-filters


Visual Studio Mac doesn't full update

 I had to completely delete the installation on Mac and reinstall to get the latest edition with the newer desktop icons, etc.

VSCode Version: I don't know
OS Version: macOS Sierra 10.12.6

Steps to Reproduce:

Launch Visual Studio Code
Update Visual Studio Code

Reproduces without extensions: Yes/No I don't know

https://github.com/Microsoft/vscode/issues/7426


⚠ Bofrost.at has degraded performance

In 40a5b29, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1113 ms

Resolved: Bofrost.at performance has improved in 499361b after 29 minutes.


2016 Roadmap Item : Allow depends_on_past=True for Skipped tasks

There is a problem with tasks that have been skipped and have depends_on_past=True. Future instances of the task won't run. This does not make sense. Instead, if a previous instance of a task has been skipped and depends_on_past=True, schedule the task anyway -- in essence, treat a SKIPPED as a SUCCESS.
That is what this issue address.
There are many cases where would like to mark a task/DAG as skipped

Tasks skipped during the execution of ShortCircuit or Branch python operators
DAGs or Tasks that we would like to manually skip as described in https://github.com/airbnb/airflow/issues/262
DAGs that specific only_run_latest as in https://github.com/airbnb/airflow/issues/59. In these cases, we want to skip DAG Runs except for the latest. Unfortunately, if we skip all but the latest, the latest will not run if depends_on_past=True

In order to tackle all of these, we need to modify the system to treat Skipped as Success. Additionally, we may want to describe each of these skipped differently (e.g. skipped_condtional, skipped_data_issue, skipped_only_run_latest)
cc @mistercrunch @bolkedebruin @criccomini

10086

For my case:
Some of tasks run as hourly and some tasks only run as daily in one DAG.
#914

https://github.com/airbnb/airflow/blob/master/airflow/models.py#L833
maybe just altering this one line to allow for skipped?

This issue is now fixed. I will be submitting a PR soon.

@griffinqiu Pull the latest and let me know if you have other issues.

Hi, I really need to use the only_run_latest  feature, but I do not know how to use is in my code. Is this a feature has not beed developed or I just missed it? Can you give me a example of task declaration ? If this feature is not developed, how can I implement my code to realize the only_run_latest feature? Thanks!


Fix permissions for persistent storage

After adding persistent storage, mysql won't start because bad permissions on persistent storage
I added fix-permission init container

/ok-to-test

@davidkarlsen Can you check this?


WIP: Refactored RPI building scripts and support for Raspberry Pi 3

Building a SD image for the Raspberry Pi ARMv7 based models:

supports Raspberry Pi 2 B
supports Raspberry Pi 3 B

Note: as soon as we do have an ARMv6 compatible RancherOS rootfs, we can support all old Pi models as well.
The build process fetches the Hypriot kernel package for the Raspberry Pi, install it on a clean directory and upgrades the bootfiles to support the brand-new Raspberry Pi 3 B. As we currently don't support ARMv6 for RancherOS, this kernel files will be deleted in order to save disk space. As result we're getting a compress ZIP archive with 46MByte only. The sizes of boot and root partitions can be customised in the build.sh script.
Known issues on Pi 3:

serial UART console doesn't work
no support for internal WiFi/BT

Just for everybody who wants to try RancherOS on a Pi2 or Pi3, here is a SD image for download https://downloads.hypriot.com/rancheros-rpi2-20160307.zip.
Ping @luxas.

@DieterReuter Thanks!  I'm going to merge and we can keep iterating on this.

:tada:
That's a great plan.

Playing around with this now, got it up and running, working on expanding the fs over the whole sdcard =) then I'll try and get rancher running on it too.


Importer collision fix

This is an alternative fix for the collision import options, and colliders not showing up when selecting 'Mesh + Static Collider' mentioned in #49752 (it does not however fix the rigid body scaling issue since that seems to be a limitation of the physics engine itself, not the importer). It also fixes the -col and -rigid suffixes mentioned #48945. Although it's mentioned that this might be deprecated, as it stands, the functionality is still intact (it works with -colonly for example), it was simply broken due to using _gen_shape_list rather than _pre_gen_shape_list. Even if it's to be deprecated, it may at least be worth fixing for now since, as pointed out, the usability of setting up collision with the new import menu is still lacking somewhat.
Bugsquad edit: Fixes #49752. Fixes #48945.

Superseded by by https://github.com/godotengine/godot/pull/50797?


Improve shadow colors

Change shadows to have a slight blue tint. Also checkout https://www.joshwcomeau.com/css/designing-shadows/ and see if there is anything worth applying to our shadows.

We should also consider allowing users to define custom shadow colors using CSS variables.


Table - Severe performance issue

Table - Severe performance issue: Too many callbacks
Detailed description
Following addition in the table component is causing multiple callbacks on redraw.
	@Input()
	set expandButtonAriaLabel(value: string | Observable<string>) {
		this._expandButtonAriaLabel.override(value);
	}

	get expandButtonAriaLabel() {
		return this._expandButtonAriaLabel.value;
	}

	@Input()
	set checkboxLabel(value: string | Observable<string>) {
		this._checkboxLabel.override(value);
	}

	get checkboxLabel() {
		return this._checkboxLabel.value;
	}

When the browser is paused when the screen is locking, it will stop on the following:
export const replace = (subject, variables) => subject.pipe(
	map<string, void>(str => {
		const keys = Object.keys(variables);
		for (const key of keys) {
			const value = variables[key];
			str = str.replace(new RegExp(`{{\\s*${key}\\s*}}`, "g"), value);
		}
		return str;
	})
);

Steps to reproduce the issue

Add a couple d3 charts to your page with the table component from this library
Add carbon-components-angular table (see details below for table markup)
Add/remove legend  items from the d3 charts
Hit pause when the page starts to slow
See details above on how the code will pause on the RegExp in the carbon-components-angular library

Additional information

Repeating the steps above when the carbon-components-angular table removed will NOT experience the same slowdown and the d3 charts will respond as expected.  Adding the chart back in will slow the page again.
No additional logic has been added to the table... this is out-of-the-box boilerplate code for the table
Table markup added as follows:

  <ibm-table-container>
      <ibm-table-header>

{{title}}

{{description}}

      </ibm-table-header>
      <ibm-table-toolbar [model]="model">
          <ibm-table-toolbar-actions>
              <button ibmButton="primary">
                  Delete
                  <ibm-icon-delete16 class="bx--btn__icon"></ibm-icon-delete16>
              </button>
              <button ibmButton="primary">
                  Save
                  <ibm-icon-save16 class="bx--btn__icon"></ibm-icon-save16>
              </button>
              <button ibmButton="primary">
                  Download
                  <ibm-icon-download16 class="bx--btn__icon"></ibm-icon-download16>
              </button>
          </ibm-table-toolbar-actions>
          <ibm-table-toolbar-content>
              <ibm-table-toolbar-search [expandable]="true"></ibm-table-toolbar-search>
              <button ibmButton="toolbar-action">
                  <ibm-icon-settings16 class="bx--toolbar-action__icon"></ibm-icon-settings16>
              </button>
              <button ibmButton="primary" size="sm">
                  Primary Button
                  <ibm-icon-add20 class="bx--btn__icon"></ibm-icon-add20>
              </button>
          </ibm-table-toolbar-content>
      </ibm-table-toolbar>

      <ibm-table [model]="model">
      </ibm-table>
  </ibm-table-container>

Which version of carbon-components-angular are you seeing this with @r4d? I know we had one unexpected performance regression we fixed a few versions back.

@cal-smith Man i apologize, i was using 3.17.3 and it appears the issue has indeed been repaired with the latest 3.26.0.  Please close this defect as already addressed.

No worries! Thanks for reaching out 👍


rework of command forms

Closes #390 #389
Reworked command form to use react hook forms.
The form should have same functionality as previous form and fixes some issues with dynamic commands like with the evaluate command. Also remake request button now works with dynamic commands
The job create path still uses the old command form.
Tracking of issues for follow on tickets: #410

Layout and usability-wise I agree that we're moving in the right direction here.  I'm okay merging this.  If the comments made here need addressing then go ahead and do that and then merge.  If we're going to address in a follow-up that's fine too.  A couple of minor styling polish comments.

In the example above:

The comment box seems like it's missing a margin or padding on the top and is feels overly close to the set of fields above it.  It could use some breathing room similar to the spacing between other fields.
The comment box and command info box are not horizontally aligned with the parameter fields, which looks odd (the parameter fields are slightly more indented).
The help text for the parameter fields being indented also looks a little odd.  I would align the start of that text with the input field, similar to how it is in the old UI.


PR for #398 (filtering PR): Removed dependency on Configuration; cleanup

I hope it's possible to do a PR for a PR?!? :)
This would remove the dependency on Microsoft.Extensions.Configuration. IMO the filter library, as it is implemented now, does not need to know anything about the Configuration system. This would only be required in a Configuration based implementation of IFilterLoggerSettings.
It also removes an unnecessary method and adds readonly to member variables.

Hi @cwe1ss, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

no longer relevant since filtering has already been integrated.


split: pass GNU tests/b-chunk.sh

This PR adds

full implementation for ---io-blksize option
refactoring of how splitting into N chunks works for --number strategies
handling input size of stdin stream and file size for files at /dev, /proc, /sys and similar locations that either report 0 for file size, while actually having some content OR report size greater than actual content

passes GNU tests/b-chunk.sh

@tertsdiepraam would you mind reviewing this one ?

setting back to draft - even though it passes the GNU test, the implementation for ---io-blksize option is incomplete, needs more work

@tertsdiepraam @sylvestre this one is ready for review

@tertsdiepraam @sylvestre
made changes based on review, please let me know if you have any more suggestions/comments

@tertsdiepraam @sylvestre I have another set of changes ready that pass next Gnu test for split  'l-chunk', but they are based on code included in this PR. Should I add them into this PR or wait until this one is resolved and then submit those in a new one?


Powering T-Encoder through T-Encoder-shield

Hi @LilyGO -
How do you power the T-Encoder through the T-Encoder-shield VBus?
I get 3V from the VBus here:

But low amps, which is not enough to run T-Encoder from the shield.

I guess more yes/no question that should be quick and easy to answer: Can T-Encoder-Shield directly power the T-Encoder without USB connected into T-Encoder?
我猜这应该是一个简单的 yes/no 问题：T-Encoder-Shield 能在没有 USB 连接到 T-Encoder 的情况下直接为 T-Encoder 供电吗？


no method matching setup_call(::LoopVectorization.LoopSet, ::Expr, ::LineNumberNode, ::Bool, ::Bool, ::Int8, ::Int8, ::Int64)

We get the following error from GitHub CI in Trixi.jl:
  LoadError: MethodError: no method matching setup_call(::LoopVectorization.LoopSet, ::Expr, ::LineNumberNode, ::Bool, ::Bool, ::Int8, ::Int8, ::Int64)
  Closest candidates are:
    setup_call(::LoopVectorization.LoopSet, ::Expr, ::LineNumberNode, ::Bool, ::Bool, ::Int8, ::Int8, ::Int64, ::Bool) at C:\Users\runneradmin\.julia\packages\LoopVectorization\PDSoT\src\condense_loopset.jl:790
  Stacktrace:
    [1] #s96#15
      @ C:\Users\runneradmin\.julia\packages\StrideArrays\NCjmP\src\broadcast.jl:345 [inlined]
    [2] var"#s96#15"(S::Any, D::Any, T::Any, N::Any, C::Any, B::Any, R::Any, BC::Any, UNROLL::Any, ::Any, dest::Any, bc::Any, #unused#::Any)
      @ StrideArrays .\none:0
    [3] (::Core.GeneratedFunctionStub)(::Any, ::Vararg{Any, N} where N)
      @ Core .\boot.jl:571
    [4] materialize!
      @ C:\Users\runneradmin\.julia\packages\StrideArrays\NCjmP\src\broadcast.jl:351 [inlined]

See https://github.com/trixi-framework/Trixi.jl/runs/3015829828?check_suite_focus=true#step:6:1811

I get the same error on Mac OS.
However, when using  pkg> add<EMAIL_ADDRESS>everything works fine.

Should be fixed by
https://github.com/chriselrod/StrideArrays.jl/commit/264b054506a6b52b578f6f66f2d6c607b085087e

Great! Thanks a lot for the quick fix! I guess it will be released in StrideArrays v0.1.11 soon?

Great! Thanks a lot for the quick fix! I guess it will be released in StrideArrays v0.1.11 soon?

Ah, 0.1.11. Thanks for reminding me to bump the version. Not sure why 0.1.9 was listed as the latest release, even though 0.1.10 is also a tagged release: https://github.com/chriselrod/StrideArrays.jl/commit/75ffbcb821e0aacfb070c5f915e039f0c8784c3f

Great, thanks :+1:


:four_leaf_clover: Proposal: Maybe We Need .gitattributes

What would you like to add? Why is this needed?
#533
I'm using windows and after running make fmt in wsl all the CRLFs are replaced with LFs, resulting in all the files becoming changed.
Do we need a .gitattributes to unify the different platforms?

Are you able to buid in WSL ? few days back I was also on WSL but the make build was showing some errors

@shubham-cmyk I haven't use make build, I just found this after using make fmt

@iyear are you execute the make fmt in Windows Subsystem for Linux? I only have an arm64 mac and can't reproduce this problem. But it sounds a good idea to add a .gitattributes
@IronCore864 What's your opinion?

@daniel-hutao Yes, I run make fmt in WSL

Related issue: https://github.com/golang/go/issues/16355

@algobot76 Yes, it solves the problem, but it requires every windows contributor to set this config on every machine. So the .gitattributes addition is still necessary.

@algobot76 Yes, it solves the problem, but it requires every windows contributor to set this config on every machine. So the .gitattributes addition is still necessary.

@algobot76 Thank you for the reply!
@iyear I agree with you. Although we should set the core.autocrlf locally, but maybe some  users won't know what happened and what can they do. So the .gitattributes will be a friendly configuration.
Would you please adding this config file and do some test with it?

@daniel-hutao I know roughly how .gitattributes should be written, but what does it mean to test? Like if I run make fmt once to see if it makes all files changed?
Or need a specific test file? I don't seem to know what this is.

I guess you could use a simple bash script to test if the line ending is what you expected after running make fmt. See https://stackoverflow.com/a/3570161

@daniel-hutao I know roughly how .gitattributes should be written, but what does it mean to test? Like if I run make fmt once to see if it makes all files changed?
Or need a specific test file? I don't seem to know what this is.

I guess you could use a simple bash script to test if the line ending is what you expected after running make fmt. See https://stackoverflow.com/a/3570161

I may know what to do now, thanks for the reply!


Remove function prototype extensions

No issue

removes more usage of function prototype extensions in favor of Ember functions

Instead of making a bunch of repetitive inline comments I'll just make one last one here and say that in general we should prefer the named event hooks over wiring up "on" handlers manually, e.g., didInsertElement: function () {} instead of something: Ember.on('didInsertElement', ...).
It's clearer to have a single method that contains everything that happens for a specific event as opposed to needing to scan the whole file and keep track of multiple handlers (and wonder what order they're executed in).

@jaswilli Yeah I did notice that.

oops

@jaswilli can this be merged?

I wasn't doing anything with it because I didn't want to generate a bunch of merge conflicts.
Let me give it a look.

@jaswilli I used both suggestions depending on the case. Once one of the Travis tests is restarted this should be good to go

Needs a rebase - a few bigger zelda PRs went in yesterday

@novaugust fixed.


:bug: Fix/seqeuence bmh association release

BMCSecret Label was being updated before the BMHost itself was updated. This order should be the reverse. This PR fixes this bug. For more details, please check the following issue.
Fixes #19

/cc @maelk

/lgtm

/test-v1a3-integration


add support for cargo update in vargo, minor refactoring, prevent inadvertently using cargo directly

Hmm also, I wonder if we want to expose a vargo cargo ... command which is literally equivalent to cargo ... (that way we don't need to tell people to open a new shell, but just to prefix it). Not sure if it is a very good idea though (maybe having the extra work of needing to open a new unactivated shell will be incentive enough to not use cargo, rather than users prefixing with vargo "because the error message told me to" without understanding the why)


l10n(zh_TW): add localization of zh_TW

What's this PR do?

[x] Add the translation for zh-Hant-TW

What are the relevant Git tickets?
None.
Screenshots (if appropriate)

Any background context you want to provide? (if appropriate)

Is there a blog post? No
Does the knowledge base need an update? No
Does this add new dependencies which need to be added to? None

@p-z-l can you review this? Is this your native language? :)

@p-z-l can you review this? Is this your native language? :)

There are some difference between the simplified and the traditional chinese, for example, the vocabulary. But he may able to read these and give some common advices. :)

@viettrungphan
I can read traditional Chinese, but they uses a different translation scheme in Taiwan, which I know nothing about.

As far as I'm aware, everything in this PR is pretty good.

@pan93412 we have a new feature for the newest version, can you help translate that part? Please checkout the latest develop brand and have a look at Localizable.string and MainStoryboard.

@pan93412 we have a new feature for the newest version, can you help translate that part? Please checkout the latest develop brand and have a look at Localizable.string and MainStoryboard.

thanks :) let me take a look with that


Copy file in context menu cuts the file

Issue Type: Bug

Have two VS Code's open with two different projects.
Copy from one project one or more files to the other project by selecting the file(s) in the file explorer sidebar, then right-clicking and selecting Copy with the mouse.
Paste the copied files to the other project in the other VS Code.
Go back to the first project and see how the files were cut.

NOTE:  I did this with two projects under GIT source control.  I am unsure if this has anything to do with this or not.
 Can you reproduce using Visual Studio Code Insiders https://code.visualstudio.com/insiders/

I can not reproduce this. Can you reproduce using Visual Studio Code Insiders https://code.visualstudio.com/insiders/

As Murphy's Law would have it, now that I want to verify, I cannot reproduce either, but the day I posted, it happened 4 times in a row. :-(  I may be missing a step or condition that I did not notice, it is one of those things that go away with an app/computer restart.

Closing and if you find good repro steps with insiders let me know and I can reopen this.


Show SQL query generated along with result in ChatData

The query generated by model sometime may not be correct or user might want to run query in another tool. It will be good idea to show the generated query on top of the result
Example
Show query here
select xxx from xxx where xxx

Hi @msangit, it's coming.


Clarity on VS Code Marketplace Plugin "Reports" data points

Howdy,
I need to get some clarity on the data points shown on the reports section of our VS Code plugin admin panel.
I see these data points on the browser rendered report:
Page Views
Install from VS Code
Download from Marketplace

However when I export I don't get the breakdown of Installed from VS Code vs. Marketplace.

I was wondering if we could get the full dataset as part of that download.
Additionally,
When we see "Install from VS Code", does this mean first time extension install only?  Or when a user updates versions, does that add another 'tick' to the data?

@mike51r -- Mike, thank you for your feedback.  Are you following a particular learn.microsoft.com article?  Meanwhile, please consider these resources:

Azure DevOps Services
Azure DevOps on Stack Overflow


[BUG]: Can't aim at cat in certain positions

Description of the bug
I can't aim near the cat on certain angles or when the cat is in certain parts of the level, such as the corners.
Steps To Reproduce

Play the kitchen level
Satisfy the first cat
Repeat until second cat appears in the corner (usually third cat appears under the table)

Additional Information
Not sure if this is limited to the kitchen or if it's in the other levels too. I'm guessing this will be handled with the new camera, but just in case, I'll put this here.

Posted for old version (oops), switching throwing side makes this a skill issue

Posted for old version (oops), switching throwing side makes this a skill issue


Make a list of all surveys/systematic reviews of prompting/prompt engineering

Meskó, Bertalan. "Prompt engineering as an important emerging skill for medical professionals: tutorial." Journal of Medical Internet Research 25 (2023): e50638.
Wang, Jiaqi, et al. "Review of large vision models and visual prompt engineering." arXiv preprint arXiv:2307.00855 (2023).
Heston, Thomas F., and Charya Khun. "Prompt engineering in medical education." International Medical Education 2.3 (2023): 198-205.
Gao, Andrew. "Prompt Engineering for Large Language Models." Available at SSRN 4504303 (2023).
Neagu, Alexandra. "How can large language models and prompt engineering be leveraged in Computer Science education?: Systematic literature review." (2023).
Dang, Hai, et al. "How to prompt? Opportunities and challenges of zero-and few-shot learning for human-AI interaction in creative applications of generative models." arXiv preprint arXiv:2209.01390 (2022).
Liu, Pengfei, et al. "Pre-train, prompt, and predict: A systematic survey of prompting methods in natural language processing." ACM Computing Surveys 55.9 (2023): 1-35.
Meyer, Jesse G., et al. "ChatGPT and large language models in academia: opportunities and challenges." BioData Mining 16.1 (2023): 20.
Hou, Xinyi, et al. "Large language models for software engineering: A systematic literature review." arXiv preprint arXiv:2308.10620 (2023).
Nguyen, Josh, and Christopher A. Pepping. "The application of ChatGPT in healthcare progress notes: a commentary from a clinical and research perspective." Clinical and Translational Medicine 13.7 (2023).
Wang, Junjie, et al. "Software testing with large language model: Survey, landscape, and vision." arXiv preprint arXiv:2307.07221 (2023).
Zhang, Quanjun, et al. "A Critical Review of Large Language Model on Software Engineering: An Example from ChatGPT and Automated Program Repair." (2023).
Lou, Renze, Kai Zhang, and Wenpeng Yin. "Is prompt all you need? no. A comprehensive and broader view of instruction learning." arXiv preprint arXiv:2303.10475 (2023).
Liu, Peng, Lemei Zhang, and Jon Atle Gulla. "Pre-train, prompt and recommendation: A comprehensive survey of language modelling paradigm adaptations in recommender systems." arXiv preprint arXiv:2302.03735 (2023).
Wang, Shuai, et al. "Can ChatGPT write a good boolean query for systematic review literature search?." arXiv preprint arXiv:2302.03495 (2023).
Li, Yingji, et al. "A Survey on Fairness in Large Language Models." arXiv preprint arXiv:2308.10149 (2023).
Ko, Hyung-Kwon, et al. "Large-scale text-to-image generation models for visual artists’ creative works." Proceedings of the 28th International Conference on Intelligent User Interfaces. 2023.
Awais, Muhammad, et al. "Foundational models defining a new era in vision: A survey and outlook." arXiv preprint arXiv:2307.13721 (202
Hadi, Muhammad Usman, et al. "Large Language Models: A Comprehensive Survey of its Applications, Challenges, Limitations, and Future Prospects." (2023).
Qiu, Jianing, et al. "Large ai models in health informatics: Applications, challenges, and the future." IEEE Journal of Biomedical and Health Informatics (2023).
Wang, Ding‐Qiao, et al. "Accelerating the integration of ChatGPT and other large‐scale AI models into biomedical research and healthcare." MedComm–Future Medicine 2.2 (2023): e43.
Clusmann, Jan, et al. "The future landscape of large language models in medicine." Communications Medicine 3.1 (2023): 141.

Sallam M. ChatGPT utility in healthcare education, research, and practice: systematic review on the promising perspectives and valid concerns. Healthcare (Basel) 2023 Mar 19;11(6) doi: 10.3390/healthcare11060887. https://www.mdpi.com/resolver?pii=healthcare11060887 .healthcare11060887 [PMC free article] [PubMed] [CrossRef] [Google Scholar]


migration comment syntax  error

What steps will reproduce the problem?
create migrate file in console/migrations/m130524_201442_init.php

What is the expected result?
What do you get instead?

i  am instead with
" ALTER TABLE bs_pcategory modify column name VARCHAR(32) COMMENT 'abc产品分类名称';   "
that's ok !!!

Additional info

Q
A

Yii version
2.0.16-dev

PHP version
7.1.24

Mysql version
8.0.13

Operating system
win10 x64

You can add a comment to a table using addCommentOnTable() since 2.0.8
Have a look at https://www.yiiframework.com/doc/api/2.0/yii-db-migration#addCommentOnTable()-detail

can you tell us what your value of $tableOptions is?
ps: for better readability PASTE the code instead of posting a screenshot. You can easily style it using three ` (backticks) or indent it with 4 spaces

Is that connected with the language used?

@samdark It looks like a bug. Generated SQL is incorrect because COMMENT is not allowed on CHANGE COLUMN. MODIFY COLUMN should be used instead.
Steps to reproduce:
CREATE TABLE users (name varchar(255));
ALTER TABLE users CHANGE COLUMN name name COMMENT 'username';

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'COMMENT 'username'' at line 1


Question: Trigger animation when in viewport?

Preet, this library is money. Thanks for it. Using and quite happy.
Question: How would I go about triggering the notation when element is within viewport?
Thanks ahead of time for your work on this tool; from a UX perspective, it speaks volumes for calling attention without breaking focus with the "F-Pattern" of scannability 👌

Thanks!
To answer your question, IntersectionObserver would be the way to go about it, imo.
The intersectionObserver callback will be called when the element comes into view, at which point you can call show() on the annotation.

Will try this.
Good stuff Preet. Keep rocking bro


appgate-sdp: 5.3.2 -> 5.3.3

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 112061 run on x86_64-linux 1

1 package built:

appgate-sdp

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 112061 run on x86_64-linux 1

1 package built:

appgate-sdp


observer.next does not allow point free style

RxJS version:
beta12
Code to reproduce:
http://jsbin.com/gicizoxefi/edit?js,console
Expected behavior:
I would expect
sayHi(observer.next);

to work,
Actual behavior:
but it throws a cryptic error.
´´´
error_handler.js:45EXCEPTION: Cannot read property 'isStopped' of nullErrorHandler.handleError @ error_handler.js:45next @ application_ref.js:273schedulerFn @ async.js:82SafeSubscriber.__tryOrUnsub @ Subscriber.js:223SafeSubscriber.next @ Subscriber.js:172Subscriber._next @ Subscriber.js:125Subscriber.next @ Subscriber.js:89Subject.next @ Subject.js:55EventEmitter.emit @ async.js:74onError @ ng_zone.js:120onHandleError @ ng_zone_impl.js:64ZoneDelegate.handleError @ zone.js:207Zone.runGuarded @ zone.js:113(anonymous function) @ zone.js:86
´´´
If I do
sayHi(m => observer.next(m));

it works
Additional information:

JavaScript doesn't explicitly bind prototype methods to their instance, and neither do we. Additionally, the Observer notification methods aren't implemented as closures on the Observer for performance' sake.

Fair enough. Could the error message be improved maybe? By checking this inside the observer.next function and throw if it is not as expected.

@rasmusvhansen it's likely lead to a significant performance regression, which would be a non-starter for us. Minimizing operations inside next is critical to real-world performance metrics.
I believe there's work planned soon to create special internal Observers (even more sparse, but will be easier for JIT to inline), but enhance the public Observer types at the ends of each sequence. It might be possible to put more error condition checking after that's done cc: @blesh who knows more about that timeline

I believe there's work planned soon to create special internal Observers (even more sparse, but will be easier for JIT to inline), but enhance the public Observer types at the ends of each sequence. It might be possible to put more error condition checking after that's done cc: @blesh who knows more about that timeline

Even in that case, we won't be able to provide what @rasmusvhansen is asking for in a performant way. The observer provided when constructing an Observable contains important internal state to provide safety in terms of things like not being able to next after an error for example. Basically, observers must be able to be closed and check that state from within next, error or complete.
I'm going to mark this issue as "won't fix" and close it for now.
Sorry, @rasmusvhansen, it's just not something that's possible with our goals for this library.
However, you might be happy to learn that the performance of bind() has increased substantially across many browsers over the last year, and I expect that trend to continue.


Empty :editor file sends blank line

Opening an editor with :editor (neovim for me) then quiting it without making any changes sends an empty line as a message. Ideally it would only send something if the temp file isn't empty.

feature was merged

feature was merged


Removed symlink for ComponentsTranslation.java

Modified appinventor/components/build.xml target ComponentTranslation so ComponentsTranslation.java
is generated into appinventor/build/components/ComponentTranslation/src/com/google/appinventor/client. This
fixes a problem where "ant clean" in the components subdirectory did not clean up the generated file
ComponentsTranslation.java.
Removed the symlink appinventor/appengine/src/com/google/appinventor/client/ComponentsTranslation.java
Modified appinventor/appengine/build.xml target AiClientLib to include
appinventor/build/components/ComponentTranslation/src in the srcdir, in addition to ${src.dir}.
Modified appinventor/appengine/build.xml target YaClientApp to include a pathelement for
appinventor/build/components/ComponentTranslation/src.
I have only tested this on linux.
@jisqyv @josmas @kkashi01

@lizlooney Bravo! (and Duh! for me). LGTM will move to Gerrit and merge

MERGED


Memory usage during CopyTo

I have a large database I am trying to compact. For this I am using CopyTo as described in the readme.
The problem is it blows the memory on the machine I am running it on (RaspberryPi)*
Some digging later and I believe the issue is in
func (o *Store) visitNodes
It recursively calls itself and while doing so keeps the pointers choiceT and choiceF both valid. This seems to mean that every Itemloc structure that the visitNodes traverses during the compact phase has to be held in memory until the end of this recursive traversal.
I have experimented with some fixes that reduce the memory required, but fundamentally this recursive process limits the size of the database that can be traversed to some function of available system memory.
I've attached a memory profile diagram of the program captured shortly before it crashed due to lack of memory.
*yes I know I could in theory run this on a machine with more memory - but it's an easy way to show the scalability issue in question here.

Thanks for the report and profile!  Indeed -- it appears visitNodes() will load all the nodes into memory as it proceeds along.  Hmmm.  And, I don't have any simple one-liner fix that pops to mind.
Related, gkvlite also hasn't been loaded in my mind for awhile now, as most of my efforts related to a golang storage engine have moved to the github.com/couchbase/moss project as it's better aligned to my current needs.

So I'm currently looking at a multi-line fix myself.
Struggling to understand how some of this is supposed to work, but will
report back if I make any progress. If you get a Pull request you will know
I've had some success.
Thanks
Chris
On 10 May 2017 at 16:30, Steve Yen<EMAIL_ADDRESS>wrote:

Thanks for the report and profile! Indeed -- it appears visitNodes() will
load all the nodes into memory as it proceeds along. Hmmm. And, I don't
have any simple one-liner fix that pops to mind.
Related, gkvlite also hasn't been loaded in my mind for awhile now, as
most of my efforts related to a golang storage engine have moved to the
github.com/couchbase/moss project as it's better aligned to my current
needs.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/steveyen/gkvlite/issues/16#issuecomment-300519990,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQ6OU6iUMR8NEKsyWD0KVE_q6LkMSTc4ks5r4dgRgaJpZM4NWhua
.


How to train model for another country plate system ?

I can't understand your binary_model.
Can you explain how to train another model for another country?

I exported the predicted image as jpg and trying to solve it using tesseract. It's better for now but little slow. @longphungtuan94 @habom2310 What do you guys prefer to get 90%+ correct predictions?

I exported the predicted image as jpg and trying to solve it using tesseract. It's better for now but little slow. @longphungtuan94 @habom2310 What do you guys prefer to get 90%+ correct predictions?


Zulip cannot connect to Zulip server URL

Regardless of whether I install the mobile app on my Android phone from FDroid or Google Playstore, I cannot get past the first screen, which prompts me to enter a server URL to connect to. No matter how good connection I have, it always shows an error popup:

Could not connect
Could not connect to ...(redacted HTTPS URL, same that works in a browser to access my Zulip instance)... . Please check your network connection and try again.

There is only an "OK" button. Try as I might, it does not connect.
Other apps on my phone have Internet connection and I can for example send messages in Signal and so on. Even as I stand right next to my router that provides the WLAN, it still shows that error. So it is definitely not, that my phone does not have connection.

app version: 27.230
updated on: 2024-07-24
Android version: 8.1.0

Screenshot from phone:

Of course, if I cannot get past the connection screen, I cannot use the app at all and as a consequence cannot access the chat from my phone.

@chrisbobbe On my mobile device browser (Firefox) I can access the site without problem and it shows the login form. Also on my desktop computer I can access the site without issues.

Are you able to log in using the login form on Firefox on your mobile device?

Yes, logging in also works from mobile phone in browser.

If needed, I could somehow tell the devs the URL non-publicly, if you have any ways to do that.
It is a single person setup and I would prefer not to post the URL publicly, because who knows what holes I mistakenly left open.

Sure. You could DM the URL to me in the Zulip development community and we can debug from there, if you'd like: https://chat.zulip.org/#narrow/dm/13313-Chris-Bobbe

Sent you a message there : )

I checked your hostname on the SSL Labs tester, and it seems like your server doesn't support TLS versions older than 1.3. For the connection to work on your old device, your server will need to support TLS versions that your device supports.


Dino: Re-build transformer head module for L and X RTDeTR models

Dino head needs to be built for both L and X models in models directory.
Please feel free to submit a PR!

@pranavdurai10, can you assign this issue to me


hide "Retry" button when no retry_path

There are now going to be cases when a sequence cannot be retried. In those cases, Odin won't pass a retry path. Canceling will be the only option. Could the "Retry" button be hidden in that case?

pushed, ready for implementation :
bower update sequence-viewer


Gameplay: Encounter Chances Does Not Account for High Party Level

Game Version: Solasta 1.4.31
UB Version: <IP_ADDRESS>
It seems like the encounter percentage chances setting does work, however, whenever it triggers in the Travel Journal it tends to say "An encounter! The monsters are frightened by the party and scurry away."
Is it possible to make it so that the encounters happen regardless of whether the party is at a higher level? So far the only encounters I get are those from monsters of a higher CR.

can you please provide a more detailed explanation with a save I can use?

Sure thing, here you go:
IM_hsIF5IaVCEawJifAju8IBQA_2.zip

    if ((double) Gui.GameCampaign.Party.ChallengeRating > 3.0 + (double) encounterDefinition.ChallengeRating)
    {
      this.travelJournal.AddEntry((GameRecordEntry) new GameTravelJournalEntry(this.travelJournal.TravelJournalDefinition, hour, eventDefinition, this.RollRandomLine(eventDefinition.OtherLogEntries), GameTravelJournalEntry.EventOutcome.Neutral), -1);
    }

game code indeed checks for CR. May I ask why you wanna every encounter for High Party Level? Also, moving this to discussion as not an issue or bug.


Critical dependency: the request of a dependency is an expression

I got the following error when importing @sentry/electron with webpack:
WARNING in<EMAIL_ADDRESS>79:11-31
Critical dependency: the request of a dependency is an expression
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

It kinda works, but webpack complains about the hack:
Critical dependency: the request of a dependency is an expression
 @ ./node_modules/encoding/lib/encoding.js
 @ ./node_modules/electron-fetch/lib/index.es.js
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS> @ ./src/browser/js/browser.js
 @ multi ./src/browser/js/browser.js```


[4pt] Main Page

In the main page, the user can see a list of houses with minor details about the houses.

Implemented in this PR: https://github.com/piero-vic/house-booking-front-end/pull/6


Negative values

The shop able me to insert negative values as order total.
Can you suggest me with method to edit values before insert order total and modify the order total.
Thanks

Is someone support this package ?

This package is really dead haha..

Package dead already? Not interested in carrying this into an argument but completed  580 orders without any fatal exception

@muratunal When i said dead i mean it in the sense of no one supporting it.

@muratunal NO I can`t the project is large and I must paste you 3 controllers and models to see the code.
But there not extended functionality of default Traits of the system, and the system allow me to order negative values for total when I add coupons and dicounts


Append test messages to "FAIL" message

Hello,
In robotframework version 2 I made sort of an "hack" in order to be able to append text from keyword failues to the parent test case failure by doing something similar to:
message = BuiltIn()._namespace.test.message
Then appending text to that variable.
When updating to robotframework 3 this doesnt seem to do the trick anymore as the _namespace.test.message has been moved elsewhere. I could find to where with the debugger but I would prefer if there was a way of doing this throgh the "official API"? Can this be implemented?

Could you clarify the usage and what you try to accomplish?

For example:
testCase
kw1
kw2
kw3
kw4
kw5
kw6
kw7
kw8
If kw 8 fails and all other kws are succesful, the output that rf prints to the console is lost for all the other kws.
I am sorry if it is unclear, the "hack" was done a few years ago and became an issue when I started porting the codebase to rf3 now.

Sorry, still don't understand. How can test's message even affect to stuff earlier kws have printed to the console?

I can attach some screenshots, the stuff is not printed until the end of the test case actually, that’s why.

A simple example would probably be better than a screenshot. Also, explaining the underlying use case would be helpful.

I still don't understand this. If you just want to print to console, you can do that from any keyword regardless are there failures or not. Also, what do you mean with "result of the successful keywords"? By default no such information is written to the console.

Below is an output of normal fail cases.
==============================================================================
Test Mnp Package
==============================================================================
Because not upload ID card photo.                              | FAIL |
The text of element 'css:.field-error > span' should have been 'กรุณาอัพโหลดเอกสารระบุตัวตน ตามขั้นตอนข้างต้น โดยภาพจะต้องเป็นไฟล์ .JPE, .JPEG, .PNG และมีขนาดไฟล์ไม่เกิน 10 MB' but it was 'กรุณากรอกข้อมูล'.
------------------------------------------------------------------------------
Because Port-in number on porting process.                     | FAIL |
The text of element 'css:div.messages .message-error div' should have been 'ดีแทคได้รับคำขอย้ายค่ายสำหรับหมายเลขนี้เรียบร้อยแล้ว และขณะนี้อยู่ในระหว่างการตรวจสอบคำขอย้ายค่าย' but it was 'กำลังตรวจสอบเอกสาร'.
------------------------------------------------------------------------------
Test Mnp Package                                               | FAIL |
2 critical tests, 0 passed, 2 failed
2 tests total, 0 passed, 2 failed
==============================================================================

@philip-karlsson can you provide an output of the fail tests with append-text hack? I'm assuming you want something like below?
==============================================================================
Test Mnp Package
==============================================================================
Because not upload ID card photo.                              | FAIL |
    √ Open Landing Page
    √ Select MNP Package          MNP Voucher 1500 (2860)
    √ Click Add To Cart Button
    √ Input Personal Detail       ${firstName}    ${lastName}    ${emailAddress}     ${contactNumber}     ${idCard}
    √ Continue MNP Step
    X Element Text Should Be      css:.field-error > span     กรุณาอัพโหลดเอกสารระบุตัวตน ตามขั้นตอนข้างต้น โดยภาพจะต้องเป็นไฟล์ .JPE, .JPEG, .PNG และมีขนาดไฟล์ไม่เกิน 10 MB

The text of element 'css:.field-error > span' should have been 'กรุณาอัพโหลดเอกสารระบุตัวตน ตามขั้นตอนข้างต้น โดยภาพจะต้องเป็นไฟล์ .JPE, .JPEG, .PNG และมีขนาดไฟล์ไม่เกิน 10 MB' but it was 'กรุณากรอกข้อมูล'.
------------------------------------------------------------------------------
Because Port-in number on porting process.                     | FAIL |
    √ Open Landing Page
    √ Select MNP Package          MNP Voucher 1500 (2860)
    √ Click Add To Cart Button
    √ Input Personal Detail       ${firstName}    ${lastName}    ${emailAddress}     ${contactNumber}     ${idCard}
    √ Input ID Card Image
    √ Continue MNP Step
    √ Input Shipping Address      ที่อยู่จัดส่ง  ${addressLine1} ${MNPNumber}     ${subDistrictBill}     ${district}     ${province}     ${postcode}
    √ Continue MNP Step
    √ Confirm Page
    √ Input Portin Number<PHONE_NUMBER>
    √ Click Confirm Button
    X Element Text Should Be      css:div.messages .message-error div     ดีแทคได้รับคำขอย้ายค่ายสำหรับหมายเลขนี้เรียบร้อยแล้ว และขณะนี้อยู่ในระหว่างการตรวจสอบคำขอย้ายค่าย

The text of element 'css:div.messages .message-error div' should have been 'ดีแทคได้รับคำขอย้ายค่ายสำหรับหมายเลขนี้เรียบร้อยแล้ว และขณะนี้อยู่ในระหว่างการตรวจสอบคำขอย้ายค่าย' but it was 'กำลังตรวจสอบเอกสาร'.
------------------------------------------------------------------------------
Test Mnp Package                                               | FAIL |
2 critical tests, 0 passed, 2 failed
2 tests total, 0 passed, 2 failed
==============================================================================

@juacompe This is EXACTLY what I am looking for. Have you made your own hack?

@philip-karlsson , not yet. I'm thinking of adding another console type (i.e. console=enterprise) to change the display to this. What do you think @pekkaklarck ?
I'm coaching teams in enterprises and usually they have another operation team whose work station is ssh on remote servers. In this situation, a dev team who write and own robot tests and use
the default console=verbose. And the operation team who did not write the test want to be able to see which keyword was failing to communicate better with the dev team; therefore, the operation team would use console=enterprise.
Any comments or suggestions would be highly appreciated.

I created a pull request here https://github.com/robotframework/robotframework/pull/2875

why not simply look at the log.html file to see what keywords passed and
which failed?
On Wed, Jun 20, 2018 at 5:52 AM Supanida Manothirakul <
<EMAIL_ADDRESS>wrote:

I created a pull request here #2875
https://github.com/robotframework/robotframework/pull/2875
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/robotframework/robotframework/issues/2795#issuecomment-398692275,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKaLZe0HuYwunwoZ-4FevEU1b3okc9t9ks5t-htdgaJpZM4S-EWC
.

Simply looking at the log.html works perfectly when I’m writing the test on my own machine. It is harder for the operation engineers who look at the test results on Jenkin which has no screen. The tests were run through xvfb-run.

@philip-karlsson, do I get it right that you'd want the top-level keywords to be shown on the console? That's at least what the example by @juacompe looks like. I agree that more verbose output would sometimes be useful, and we already have #1393 about that and it's even targeted for RF 3.1. If I've understood your needs correctly, then we can just close this issue as a duplicate of #1393.
There's also PR #2525 that aims to fix #1393 and now also #2875 related to this issue. I need to look at these PRs in more detail, but I want to get RF 3.1 alpha 2 out first. One thing I can comment already now is that I don't like --console enterprise too much. What I propose instead is renaming the current default behavior to --console standard (or default or something similar) and then using verbose with the more verbose console output. In addition to supporting '--console verbose, we should also support --verbose`. Anyway, coming up with a good output on console is the actual hard part here.

@pekkaklarck Yes that is what I am after. I too agree that "enterprise" is not the best naming for the parameter. Console --verbose is an alternative. The way we did it before RF 3;
BuiltIn().set_test_message("Signal: " + signal + " is verified to " + value + "\n", True) for every successful run.
Then when failing;
message = BuiltIn()._namespace.test.message
BuiltIn().fail(message + "Signal: " + signal + " = " + str(sigValue) + ", expected " + str(value))
That way we could get our information also for failing keywords to the console as well. Thank for the attention.

OK, in that case I'll close this as a duplicate as #1393. Let's continue discussion how the more verbose console output should actually look there.