Conditional column update

Hey there @kfirba first let me thank you for your effort to bring such an useful tool.
May I suggest to add basic support for conditional updated columns?
I've started using your tool but soon I realised that I had to edit the generated query myself.
Here's the case:
My table had updated_at column so in my insertOrUpdate   values I had to set it with the current time before the db request. The problem is the query treats it as an update value when it actually should be set only when another of the columns is updated.
On the other hand you can't excluded it because you need it to show when the update has occurred.
So I had to add the following line
on duplicate key update
    updated_at=IF(name=VALUES(name), updated_at, VALUES(updated_at) )

right after the on duplicate key update clause before all other updating columns so that it checks the value before name is set in order to compare the values between the new value and the one of the table record. As you see the updated_at receives the new value only when name  is different than the current value of the table.
Which brings the idea for conditional update columns where a conditional column is checked against  change of value among other given columns like
[ 'updated_at' => ['name', 'color', ....] ]

where it generates the following line in the query
updated_at=IF(
   name=VALUES(name) && color=VALUES(color) && ....,
   updated_at, VALUES(updated_at)
)

OK I've played with your class and gave a hand
public function generate($table, $rows, array $exclude = [], array  $dependentlyUpdating = []) {
    $columns = array_keys($rows[0]);
    $columnsString = implode('`,`', $columns);
    $values = $this->buildSQLValuesStringFrom($rows);
    $updates = $this->buildSQLUpdatesStringFrom($columns, $exclude, $dependentlyUpdating);

    $query = vsprintf('insert into `%s` (`%s`) values %s on duplicate key update %s', [
        $table, $columnsString, $values, $updates,
    ]);

    return new QueryObject($query, $this->extractBindingsFrom($rows));
}

// .......

protected function buildSQLUpdatesStringFrom($cols, $exclude, $dependentlyUpdating) {
    $dependentColumns = array_keys($dependentlyUpdating);
    $notDependentCols = array_diff($cols, $dependentColumns);
    $str = '';

    foreach ($dependentlyUpdating as $key => $value) {
        if(!is_array($value) || empty($value)) {
            $index = array_search($key, $exclude);
            if(is_bool($index)) {
                $value = is_bool($index) ? array_merge($exclude,[$key]) : $exclude;
                $value = array_diff($cols, $value);
            }
            array_diff($cols, $dependentColumns);
        }
        $str = "{$key}=IF(".(
            trim(
                array_reduce($value, function ($result, $column) {
                    return $result."`{$column}`=VALUES(`{$column}`)&&";
                }, ''), '&&'
            )
        )."`,{$key}`,VALUES(`{$key}`)),";
    }
    return  $str.trim(array_reduce(array_filter($notDependentCols, function ($column) use ($exclude) {
        return ! in_array($column, $exclude);
    }), function ($updates, $column) {
        return $updates . "`{$column}`=VALUES(`{$column}`),";
    }, ''), ',');
}

and the invoke looks like
// Depending and all columns except itself and excluded
$queryGenerator -> generate($tableName, $campaigns, $excludedColumnsFromUpdate, [
    'updated_at' => null
]);

or
and the invoke looks like
// Depending on these two
$queryGenerator -> generate($tableName, $campaigns, $excludedColumnsFromUpdate, [
    'updated_at' => ['name', 'status']
]);

I haven't tested it yet it works in my case but I'm almost sure it's not reliable yet


Deleting previously saved data on healthkit

The HKHealthStore object has a method deleteObject:withCompletion which can delete previously saved objects. Are there plans to integrate this functionality in the plugin?
Thanks!
Varun

Sounds interesting! Do you have an opportunity to try and add it yourself perhaps?

I currently do not have any experience with objective C, but with some of your assistance I may be able to implement this if it is something you think will only take a couple hours to complete.

Here are the docs for the deleteobjects class:
https://developer.apple.com/reference/healthkit/hkhealthstore/1614163-deleteobjects?language=objc

I implement deleteObjects in my fork:
https://github.com/mnill/HealthKit
window.plugins.healthkit.deleteObjects({
          'type': type,
          'startDate': new Date(new Date().getTime() - 1000*60*60*24*365),
          'endDate': new Date(),
          },
          callback,
          callback
          );

Sample or quantity type in ’type’ field :-). If you want to delete workOut - type must be ‘workoutType’
Please test it and give me feedback, I no have project to test it in the wild.

@mnill Oh boy, I just pushed a similar change earlier today: #84


Thumbs Gallery With Two-way Control not working

thumbs gallery with two way control example not working at all

The same with me. swiper.controller is always undefined


27079 jukebox

About the PR

Added queue system to the jukebox.
Why / Balance

Jukeboxes require constant player intervention for the music to be played longer than the time of current song.
Technical details

Added new JukeboxEntry control, that handles the display of song name and play/pause, stop, queue, remove buttons
Added new JukeboxBoundUserInterfaceState that holds information if song now is being played and the queue contents

When Play button is pressed, server sets TimeWhenSongEnds and starts counting on Time, when Time reaches TimeWhenSongEnds, server tells music to stop, and checks if there is any song in the queue, if so, it makes jukebox select it, and play it.

The Play button in song list sets and starts playing the song its bound to
The Queue button adds song its bound to to the jukebox queue end

Media

UI of newly constructed/spawned jukebox:

After pressing Queue button a bunch of times

After pressing Play

After pressing Stop

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

add: Added song queue to jukebox.

https://github.com/space-wizards/space-station-14/issues/27079

@Lukasz825700516


how to Folding code, closing tag always on one line

vscode :

wanted
https://www.svelte.cn/tutorial/bind-this

In VS Code each language can have it's own folding provider and can decide on how much to fold.
So please file this against the Svelte extension.
That said, we only support ... as folding text not <->}) as you show in the screenshot. The issue for custom folding text is https://github.com/microsoft/vscode/issues/70794
Also we only support folding full lines.


[FAQ] Library requirements for examples running

Context
Can you provide library requirements for running examples in your docs?
I`m interesting in tutorial number 7
https://deephyper.readthedocs.io/en/latest/tutorials/tutorials/notebooks/07_NAS_with_Ensemble_and_UQ/tutorial_07.html
I have tried to run this notebook on Colab, using default version of required libraries, but fail. I got a lot of errors inside dh/tf/keras sources
I also tried to fix it. E.g. in chapter
"Adding Uncertainty Quantification to the Baseline Neural Network"
substitution "tf.keras" by "tf_keras" helps me. But when I going to ensemble chapter I got another errors inside UQBaggingEnsembleRegressor class
I think that issues related to dependency incompatibility and I want to know what version of libraries were used in this examples
Describe the task you are trying to achieve.
A clear and concise description of what the task is.
Describe the solution you'd like
A clear and concise description of what you want to happen.

Hello @RinnetenseiQ ,
How did you end up with the link to this tutorial?
I am asking because on today's website the tutorial is not available on the tutorial page anymore as you can see here: https://deephyper.readthedocs.io/en/latest/tutorials/index.html
The upgrade between tf.keras and tf_keras because of Keras 3 was an issue.
Also, since deephyper==0.8.0 the ensemble and nas were significantly changed (See DeepHyper 0.8.0 - Changelog). Ensembles are now more generic (compatible with more parallel backends, ML libraries, ML tasks such as regression/classification). NAS was reframed under the hyperparameter optimization framework (allowing the same capabilities as before with better compatibility with other ML frameworks and training pipelines).
If you want to use the outdated API  using: tensorflow~=2.16 tensorflow-probability==0.24.0 should work.
I you want to try the new API (the best):

A tutorial on Neural architecture search: 6. Neural Architecture Search with Tensorflow/Keras2 (Basic)
A tutorial on ensembling: 2. Hyperparameter Optimization for Tree Ensemble with Uncertainty Quantification (Scikit-Learn)

Let me know if this helps!

How did you end up with the link to this tutorial?
I dont remember already. I think, just going through docs

The upgrade between tf.keras and tf_keras because of Keras 3 was an issue.
Is it mean that I should use tf_keras instead tf.keras in any cases that related to deephyper?

I will try your advices about NAS problem and let you know if something will go wrong.
About ensembling. I am interesting in approaches of uncertainty calculation for regresion task that related to ensemble creation. And I also think the prediction by distribution layer was also interesting trick. Is it possible to reproduce this using latest version of libraries?


Turn off analytics in development mode to prevent unwanted tracking

Is your feature request related to a problem? Please describe.
I've set up umami analytics (self hosted) on my website using the designated configuration in params.toml, and while checking locally I found that it triggered the analytics causing an inflation in the metrics
Describe the solution you'd like
Turn off analytics in development mode to prevent unwanted tracking
Describe alternatives you've considered
I work around this issue by reverting the change in params.toml and using layouts/partials/extend-head.html that contain this snippets
{{ if hugo.IsProduction}}
<insert-script-here>
{{ end }}

Additional context
Minimal change on layouts/partials/analytics/main.html and docs should be enough, if you may I can open a PR to aid you with this issue
Thank you!

There's already another open issue on this #1543.
A temp fix: move analytics related config from config/_default to config/production as suggested here

duplicates https://github.com/nunocoracao/blowfish/issues/1543


[React Native] Hermes on iOS (react-native 0.64)

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
React Native 0.64-rc0 was released, which includes opt-in support for the Hermes JS executor (instead of JSC). (see https://github.com/react-native-community/releases/issues/214)
I've upgraded my project to use RN 0.64 rc and enabled hermes which works as expected in debug builds. Unfortunately I couldn't run fastlane beta anymore, since it seems that this requires additional changes from the fastlane CLI.
Command executed
fastlane beta

Complete output when running fastlane, including the stack trace and command used
It looks like only the bundle command fails, since fastlane doesn't expect hermes bitcode instead of the js bundle.

▸ Processing Info.plist
▸ Running script 'Bundle React Native code and images'
    the transform cache was reset.
❌  /Users/mrousavy/Library/Developer/Xcode/DerivedData/Springsale-cdueuzbwshyvojabkcrtvdjvltvy/Build/Intermediates.noindex/ArchiveIntermediates/Springsale/BuildProductsPath/Release-iphoneos/main.jsbundle:1:1: Invalid UTF-8 continuation byte
��
�'
^
❌  error: File /Users/mrousavy/Library/Developer/Xcode/DerivedData/Springsale-cdueuzbwshyvojabkcrtvdjvltvy/Build/Intermediates.noindex/ArchiveIntermediates/Springsale/BuildProductsPath/Release-iphoneos/Springsale.app/main.jsbundle does not exist. This must be a bug with

Environment

| Key                         | Value                                                          |
| --------------------------- | -------------------------------------------------------------- |
| OS                          | 11.0.1                                                         |
| Ruby                        | 2.6.3                                                          |
| Bundler?                    | false                                                          |
| Git                         | git version 2.24.3 (Apple Git-128)                             |
| Installation Source         | /usr/local/bin/fastlane                                        |
| Host                        | macOS 11.0.1 (20B29)                                           |
| Ruby Lib Dir                | /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib |
| OpenSSL Version             | LibreSSL 2.8.3                                                 |
| Is contained                | false                                                          |
| Is homebrew                 | false                                                          |
| Is installed via Fabric.app | false                                                          |
| Xcode Path                  | /Applications/Xcode.app/Contents/Developer/                    |
| Xcode Version               | 12.2                                                           |
System Locale

Error

No Locale with UTF8 found 🚫

fastlane gems

Gem
Version
Update-Status

fastlane
2.166.0
🚫 Update available

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

did_you_mean
1.3.0

slack-notifier
2.3.2

rouge
2.0.7

xcpretty
0.3.0

terminal-notifier
2.0.0

unicode-display_width
1.7.0

terminal-table
1.8.0

plist
3.5.0

addressable
2.7.0

multipart-post
2.0.0

word_wrap
1.0.0

tty-screen
0.8.1

tty-cursor
0.7.1

tty-spinner
0.9.3

babosa
1.0.4

colored
1.2

highline
1.7.10

commander-fastlane
4.4.6

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20190701

http-cookie
1.0.3

faraday-cookie_jar
0.0.7

faraday_middleware
1.0.0

fastimage
2.2.0

gh_inspector
1.1.3

mini_magick
4.10.1

rubyzip
2.3.0

security
0.1.3

xcpretty-travis-formatter
1.0.0

dotenv
2.7.6

bundler
2.1.4

naturally
2.2.0

simctl
1.6.8

jwt
2.2.2

uber
0.1.0

declarative
0.0.20

declarative-option
0.1.0

representable
3.0.4

retriable
3.1.2

mini_mime
1.0.2

multi_json
1.15.0

signet
0.14.0

memoist
0.16.2

os
1.1.1

googleauth
0.14.0

httpclient
2.8.3

google-api-client
0.38.0

google-cloud-errors
1.0.1

google-cloud-core
1.5.0

rake
13.0.1

digest-crc
0.6.1

google-cloud-storage
1.29.1

nanaimo
0.3.0

colored2
3.1.2

claide
1.0.3

CFPropertyList
3.0.2

atomos
0.1.3

xcodeproj
1.19.0

public_suffix
4.0.6

excon
0.78.0

ruby2_keywords
0.0.2

faraday
1.1.0

json
2.3.1

google-cloud-env
1.4.0

emoji_regex
3.2.1

aws-eventstream
1.1.0

aws-sigv4
1.2.2

aws-partitions
1.388.0

jmespath
1.4.0

aws-sdk-core
3.109.1

aws-sdk-kms
1.39.0

aws-sdk-s3
1.83.1

forwardable
1.2.0

logger
1.3.0

stringio
0.0.2

ipaddr
1.2.2

openssl
2.1.2

ostruct
0.1.0

strscan
1.0.0

date
2.0.0

fileutils
1.1.0

etc
1.0.1

io-console
0.4.7

zlib
1.0.0

libxml-ruby
3.1.0

rexml
3.2.4

psych
3.1.0

mutex_m
0.1.0

webrick
1.4.2

generated on: 2020-11-23

Nevermind, that has nothing to do with fastlane.


Disabling transmission to the STA

Hi
Is there a way to disable only transmission to the STA? Imagine the ESP is connected to a modem which has access to internet and there is a phone connected to the ESP board. Is there way to only disable transmitting the received data from the phone to the modem yet letting the phone receive data from the modem?
Thank you Martin

You can do this using the ACLs, but prohibiting any packet from the phone to the internet won't work as you might expect: all TCP connections (including HTTP and MQTT) require bi-directional traffic (for connection establishment and ACKs later), even if data is only transmitted in one direction.

Thank you for replying so fast. Really appreciate it.
What about set upstream_kbps bitrate and set downstream_kbps bitrate?
If my understanding is correct these are basically the send and receive speed of esp. Can we use an extremely low value to almost stop the transmission in one direction?

Never tested that for that purpose, but yes, that could be an option.


Unable to view more than 125 test cases in a single suite.

Despite adding more than 125 test cases in a suite, the Web Portal will only display 125 and no more, no matter how many times you try to add the case(s).

@sulu99 -- thanks for your feedback; we appreciate it. To make sure we have all the information needed to triage your issue, please let us know which topic you’re referencing, or provide more context around the general task.


Update version: Element.Element version 1.11.38

Pull request has been created with WinGet Automation :rocket:
Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/115449&drop=dogfoodAlpha

@wingetbot run

Not sure about this change, as the old link redirects to element.io and it is the correct link. @vedantmgoyal2009 Can you exempt this file from your action/update it's config?


Java version can not be matched correctly

Description:
sam build failed on Oracle java 12 environment (on mac).
Steps to reproduce the issue:

Setup an Oracle java 12 development environment on mac. (maven and sam-cli were installed by brew)
Execute sam build command in project directory.

Observed result:
$ sam build
2019-03-20 20:08:43 Building resource 'HelloWorldFunction'
Traceback (most recent call last):
File "/usr/local/bin/sam", line 11, in
load_entry_point('aws-sam-cli==0.13.0', 'console_scripts', 'sam')()
File "/usr/local/Cellar/aws-sam-cli/0.13.0/libexec/lib/python3.7/site-packages/click/core.py", line 722, in call
return self.main(*args, **kwargs)
File "/usr/local/Cellar/aws-sam-cli/0.13.0/libexec/lib/python3.7/site-packages/click/core.py", line 697, in main
rv = self.invoke(ctx)
File "/usr/local/Cellar/aws-sam-cli/0.13.0/libexec/lib/python3.7/site-packages/click/core.py", line 1066, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/usr/local/Cellar/aws-sam-cli/0.13.0/libexec/lib/python3.7/site-packages/click/core.py", line 895, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/usr/local/Cellar/aws-sam-cli/0.13.0/libexec/lib/python3.7/site-packages/click/core.py", line 535, in invoke
return callback(*args, **kwargs)
File "/usr/local/Cellar/aws-sam-cli/0.13.0/libexec/lib/python3.7/site-packages/click/decorators.py", line 64, in new_func
return ctx.invoke(f, obj, *args[1:], **kwargs)
File "/usr/local/Cellar/aws-sam-cli/0.13.0/libexec/lib/python3.7/site-packages/click/core.py", line 535, in invoke
return callback(*args, **kwargs)
File "/usr/local/Cellar/aws-sam-cli/0.13.0/libexec/lib/python3.7/site-packages/samcli/commands/build/command.py", line 95, in cli
skip_pull_image, parameter_overrides)  # pragma: no cover
File "/usr/local/Cellar/aws-sam-cli/0.13.0/libexec/lib/python3.7/site-packages/samcli/commands/build/command.py", line 133, in do_cli
artifacts = builder.build()
File "/usr/local/Cellar/aws-sam-cli/0.13.0/libexec/lib/python3.7/site-packages/samcli/lib/build/app_builder.py", line 99, in build
lambda_function.runtime)
File "/usr/local/Cellar/aws-sam-cli/0.13.0/libexec/lib/python3.7/site-packages/samcli/lib/build/app_builder.py", line 190, in _build_function
runtime)
File "/usr/local/Cellar/aws-sam-cli/0.13.0/libexec/lib/python3.7/site-packages/samcli/lib/build/app_builder.py", line 210, in _build_function_in_process
executable_search_paths=config.executable_search_paths)
File "/usr/local/Cellar/aws-sam-cli/0.13.0/libexec/lib/python3.7/site-packages/aws_lambda_builders/builder.py", line 110, in build
return workflow.run()
File "/usr/local/Cellar/aws-sam-cli/0.13.0/libexec/lib/python3.7/site-packages/aws_lambda_builders/workflow.py", line 46, in wrapper
valid_path = validator.validate(executable_path)
File "/usr/local/Cellar/aws-sam-cli/0.13.0/libexec/lib/python3.7/site-packages/aws_lambda_builders/workflows/java_maven/maven_validator.py", line 31, in validate
if int(jvm_mv) > 8:
ValueError: invalid literal for int() with base 10: '12, vendor: Oracle Corporation'
Expected result:
$ sam build
2019-03-21 09:03:18 Building resource 'HelloWorldFunction'
2019-03-21 09:03:19 /usr/local/bin/mvn is using a JVM with major version 12 which is newer than 8 that is supported by AWS Lambda. The compiled function code may not run in AWS Lambda unless the project has been configured to be compatible with Java 8 using 'maven.compiler.target' in Maven.
2019-03-21 09:03:19 Running JavaMavenWorkflow:CopySource
2019-03-21 09:03:19 Running JavaMavenWorkflow:MavenBuild
2019-03-21 09:07:01 Running JavaMavenWorkflow:MavenCopyDependency
2019-03-21 09:08:15 Running JavaMavenWorkflow:MavenCopyArtifacts
Build Succeeded
Built Artifacts  : .aws-sam/build
Built Template   : .aws-sam/build/template.yaml
Additional environment details

Mac 10.14.3 (18D109)
Oracle java version "12" 2019-03-19
Java(TM) SE Runtime Environment (build 12+33)
Java HotSpot(TM) 64-Bit Server VM (build 12+33, mixed mode, sharing)
Apache Maven 3.6.0 (97c98ec64a1fdfee7767ce5ffb20918da4f719f3; 2018-10-25T02:41:47+08:00)
Maven home: /usr/local/Cellar/maven/3.6.0/libexec
Java version: 12, vendor: Oracle Corporation, runtime: /Library/Java/JavaVirtualMachines/jdk-12.jdk/Contents/Home
Default locale: en_US, platform encoding: UTF-8
OS name: "mac os x", version: "10.14.3", arch: "x86_64", family: "mac"

Reason
According to the code logic at [0], a regular expression is used to match the version string, it try to get out the java version number from the string after the prefix Java version:\s+, the expression assumes the version number follows the prefix and fill the rest of the string line, so in the code a * is used in the group (\d.*) which is a greedy match operator. However in my case, from my output of the mvn -version command executed by the logic at [1], you can see beside version number part vendor part is listed in the same line as well, and finally the int() call failed and ValueError exception raised up.
Solution proposal
My idea is simple, use non-greedy match in the regular expression, to replace char * by ? at [0], it works for me, the build succeeded after the hotfix in my local.
I'd like to commit the fix by a tiny PR if the solution works for you.
thanks,
zhiyan
Code link
[0] https://github.com/awslabs/aws-lambda-builders/blob/v0.2.1/aws_lambda_builders/workflows/java_maven/maven_validator.py#L46
[1] https://github.com/awslabs/aws-lambda-builders/blob/v0.2.1/aws_lambda_builders/workflows/java_maven/maven_validator.py#L55

This seems reasonable, please submit a Pr 👍

pushed, @TheSriram pls review when ok, thanks.

The same issue exists when using gradle instead of maven, and the same fix appears to work just as well.

Nice. and close the issue due to all java related workflows have been fixed.


Fix pod affinity lables

merged with https://github.com/Percona-Lab/percona-xtradb-cluster-operator/pull/37


[Desktop + AirGap] Show connection option in Settings (mobile)

[Zeplin]: Settings List view: v0.6.x-00.57-Disclaimer

We don't have designs for Settings AirGap (detail views for managing AirGap account connections).


getaccountstate returns the wrong balance of NEO

Current behavior
getaccounstate returns the wrong balance of NEO. NEO-cli returns the correct balance.
GAS balance seems to be correct
PM me on Discord if you want a specific balance to query (I only figured this out because I queried one of my addresses)
Expected behavior
accountstate should return the correct balance of both NEO and GAS
How to reproduce
You can use this script in Python to test if the balances match:
import requests

url = "http://<IP_ADDRESS>:10332"
body = {"jsonrpc": "2.0","method": "getaccountstate","params": ["<address>"],"id": 2}

req = requests.post(url, json=body)
req = req.json()
data = req['result']

print('Python Data: ', data)

url = "http://<IP_ADDRESS>:10332"
body = {"jsonrpc": "2.0","method": "getaccountstate","params": ["<address>"],"id": 2}

req = requests.post(url, json=body)
req = req.json()
data = req['result']

print('NEO-cli: ', data)

Your environment
This is based on live RPC calls

You'll need to specify an address with a mismatch.

@localhuman I PM'd you.

NEO and GAS are based on UTXO so only problem I can think of is the 'dead fork' incident.
@jseagrave21 if you know the block number when the transfer happened, then you can use getblockhash to compare. They should mismatch for faulty vs correct nodes.

@wy I spoke with @localhuman and am going to try to import the address as watch only so I can compare transaction history that way. I check the most recent transactions and it is not that.

Although I haven't confirmed completely, I think this issue is caused by missing block information for some RPC nodes. For example, block 2669088 is missing from one RPC server I queried.
I queried 3 different RPC nodes running NEO-Python and got 2 similar responses and 1 different and none of them were correct.
I queried 2 different RPC nodes running NEO-Cli and got 2 similar responses and both were correct.

When querying http://<IP_ADDRESS>:10332 about 2669088, I get the following:
for getblockhash, I receive: {'jsonrpc': '2.0', 'id': 2, 'error': {'code': -32603, 'message': "'>=' not supported between instances of 'str' and 'int'"}}
for getblock (non-verbose), I receive: {'jsonrpc': '2.0', 'id': 2, 'result': '00000000408668e83c9fedfa9417483b27438082ee5d3342114441ba6e427972ea3d1a03e2023c0dad8def3855e267071b95eca9cf5df56fa2d58a976c03eaa828ce3aaf0bf6855b20ba28007ef7b9da40e5d79f83f5011bdaaccee8c4d2555c829fa51e31551ef201fd4501405658f2a61de8dd909dbe4e67291324302d027c6da4860bd35ef7d8de52ebc48fcc381c0390efc935835062fef8d5277821a3de7d526ff1b301de8178a59e5732404315ab76f8e4d62b2743cc4f52e690467e47564aa32655ae4323e390ef4238d1e951956f16167c4485932a214c02841df1a11eada5459dcc58338ce844b110d1401b8e7a1c7da59018c73cdb158c69cca95bce39313638a20a35b657e8d32d52299bcb76275444b2bddf57b878ddf839f6d2bb98a05f9411dfc41a9f91f918c5164002c395972f1da2dc29475ba4958b7ad0ca3283339dd76a1372872aee72b883c5038613d02950b173aea93a7528c295293bce49702a767d2e174ae47cd82818554008dfd6028313ef6a9e20b57f2e402332a1f5df32104c243d5fc6a492d62c63832f306cb14efdd8488124c453cca6a014238184c40519f401c17898d970aaa871f15521024c7b7fb6c310fccf1ba33b082519d82964ea93868d676662d4a59ad548df0e7d21025bdf3f181f53e9696227843950deb72dcd374ded17c057159513c3d0abe20b642102aaec38470f6aad0042c6e877cfd8087d2676b0f516fddd362801b9bd3936399e2103b209fd4f53a7170ea4444e0cb0a6bb6a53c2bd016926989cf85f9b0fba17a70c2103b8d9d5771d8f513aa0869b9cc8d50986403b78c6da36890638c3d46a5adce04a2102ca0e27697b9c248f6f16e085fd0061e26f44da85b58ee835c110caa5ec3ba5542102df48f60e8f3e01c48ff40b9b7f1310d7a8b2a193188befe1c2e3df740e89509357ae00'}
and for getblock (verbose), I get: {'jsonrpc': '2.0', 'id': 2, 'error': {'code': -32603, 'message': 'list index out of range'}}

Here are the results of my findings after querying each listed RPC node on HappyNodes:
BLUF: Every neo-python RPC node seems to have a chain issue and many have the same issue
# correct response based on NEO-Cli is: `balances': [{'asset': '0x602c79718b16e442de58778e148d0b1084e3b2dffd5de6b7b16cee7969282de7', 'value': '14.37532376'}, {'asset': '0xc56f33fc6ecfcd0c225c4ab356fee59390af8560be0e930faebe74a6daff7c9b', 'value': '264'}
# http://node2.nyc3.bridgeprotocol.io:10332 < -- gives balances': {'0x602c79718b16e442de58778e148d0b1084e3b2dffd5de6b7b16cee7969282de7': '14.37532376', '0xc56f33fc6ecfcd0c225c4ab356fee59390af8560be0e930faebe74a6daff7c9b': '163.0'}
# http://node2.sgp1.bridgeprotocol.io:10332 - same
# https://pyrpc1.redpulse.com:10331 - same
# https://pyrpc2.redpulse.com:10331 - same
# http://pyrpc1.redpulse.com:10332 - same
# http://pyrpc2.redpulse.com:10332 - same
# https://pyrpc1.narrative.org:443 - same
# https://pyrpc2.narrative.org:443 - same
# https://pyrpc3.narrative.org:443 - same
# https://pyrpc4.narrative.org:443 - same
# http://<IP_ADDRESS>:10332 - same
# http://<IP_ADDRESS>:10332 <-- gives a different accountstate: 'balances': {'0x602c79718b16e442de58778e148d0b1084e3b2dffd5de6b7b16cee7969282de7': '13.77532376', '0xc56f33fc6ecfcd0c225c4ab356fee59390af8560be0e930faebe74a6daff7c9b': '627.0'}
# http://<IP_ADDRESS>:10332 same as 101

cc @wy

This issue seems to be separate from the block corruption issue. After testing with @hal0x2328 , we were able to call every tx associated with the subject address. However, getaccountstate still returns the wrong value as shown above. I have found several bad blocks in http://<IP_ADDRESS>:10332 and have searched other NEO-PYTHON nodes to see if they also have similar issues. Some do. I recorded my findings here:
https://github.com/neo-ngd/Happynodes/pull/113
Should I create a separate issue for the corrupted blocks?

@localhuman @hal0x2328 After further testing, I think the issue is in the chain db, not in the getaccountstate method.
Using a fresh pull of the master branch, I created a wallet and imported the wif of my suspect address. Running wallet yields the same results as getaccountstate. What do you guys think?

I've got a theory about what may be causing this, but a fix is waiting on my np-import to finish.

I will close this issue as soon as I can personally verify #611 has fixed the issue. Right now, I am waiting for an updated bootstrap.

@localhuman with the most recent bootstrap file you posted
https://s3.us-east-2.amazonaws.com/cityofzion/bootstrap_main/Main276xxxx.tar.gz, the issue appears resolved
"balances": {
        "0x602c79718b16e442de58778e148d0b1084e3b2dffd5de6b7b16cee7969282de7": "14.37532376",
        "0xc56f33fc6ecfcd0c225c4ab356fee59390af8560be0e930faebe74a6daff7c9b": "264.0"

I think it would be good to include this bootstrap file in the next neo-python release.


add abs function to parameterexpression

Summary

added an abs function to the class ParameterExpression

Details and comments

resolves #6518
related to #5278

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 5 (20.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.005%) to 83.114%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit/circuit/parameterexpression.py
1
5
20.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.005%

Covered Lines:
51946

Relevant Lines:
62500

💛  - Coveralls

Dear @czachow ,
I want to use the method you created for ParameterExpression. Since your comment on Jan 20th, have you updated the program? If not, I would like to continue your work and make a new pull request.

Dear @king-p3nguin, thanks for working on this old PR that would have been forgotten otherwise. In my memory most of the implementation is done. You only need to add more tests like @jakelishman suggested. Feel free to do so!

Thank you very much!

Closing this as duplicate of #9309.


fix: Firefox XRay vision prevents window<->content script messaging

As current code modifies original message it causes the following error in latest version of Firefox:
Not allowed to define cross-origin object as property on [Object] or [Array] XrayWrapper
    <anonymous> content-script.js:26
    portP chunk-ICLXI4BR.js:56
    (Async: EventHandlerNonNull)
    acceptMessagingPort chunk-ICLXI4BR.js:8
    (Async: EventListener.handleEvent)
    promise chunk-ICLXI4BR.js:28
    getMessagePort chunk-ICLXI4BR.js:3
    setNamespace chunk-ICLXI4BR.js:56
    allowWindowMessaging content-script.js:46

More about Xray Vision here: https://firefox-source-docs.mozilla.org/dom/scriptSecurity/xray_vision.html

@danpastori @jaydrogers kind reminder here 👀

@danpastori @jaydrogers Hi! Another very kind reminder to merge and publish this

@StyleT This looks great here, I'll get this merged. Thanks so much for the contribution!

Ran into this issue today.
package.json:
  "dependencies": {
    ...
    "webext-bridge": "^6.0.1"
  },

What the debug console is showing:

And it looks like it didn't make it into the Apr 07 package:
https://www.npmjs.com/package/webext-bridge?activeTab=code
Is there a plan to ship this?

@CharlieDigital I ended up fixing it in @webext-pegasus/rpc

@StyleT thanks for the pointer; will check it out


Dropdown Links in Menu Button Not Working on Mobile

On mobile phones, the menu collapses into the singular button. If you try to select a link that is within a collapse sub section they don't proceed to take you to the designated page.

The ng-click markup added to the drop down menu in commit 2362c23 to fix issue #3 is breaking the functionality of links once again.


Adding a domain will not recreate the certificate

Adding a domain to a running container and recreating it (certs are in a volumen) will not recreate the certificate.
Is this a bug or known behaviour?

Just tried it and it worked fine.

never worked for me but I noticed in the latest tag there is a script /app/force_renew (https://github.com/JrCs/docker-letsencrypt-nginx-proxy-companion/blob/master/app/force_renew) which I trigger after adding a domain. this workes for me :)

force_renew has an issue and will only renew the domain(s) for the first container id found on the letsencrypt_service_data file, not all of them as it should.
#270 will fix this when merged, before that you can manually patch it by removing the  && shift on line 73 of letsencrypt_service  if you need complete forced renewal.


Improve documentation around environment

I think it would be helpful to highlight:

[ ] Where users are supposed to install pygdf (same computer as MapD running on, inside of a MapD docker image itself, etc.)
[ ] Which version of CUDA and CUDA tools are necessary

Last night, I tried to install pygdf from a git clone on Pop_OS! (which is mostly Ubuntu 17.10) and got hundreds of test failures when running py.test (can provide stack trace if desired). This morning, I tried installing on a DGX-Workstation(Ubuntu 16.04) and only got 3 failed tests.

@randyzwitch re point #1: pygdf is not dependent on MapD, i.e. you can read a CSV into a Pandas DataFrame and convert that into a pygdf DataFrame without using MapD.
If you're looking to use pygdf + MapD via PyMapD or Thrift directly then you'd want it on the same host in the same IPC namespace as wherever MapD is running. That means running pygdf in the same container, running in a separate container with a shared ipc namespace between the two containers, or running directly on the host and specifying the MapD container's IPC namespace to use the host's IPC namespace.
I believe this documentation should be added to the PyMapD / MapD repos if it doesn't already exist there.
Adding @andrewseidl for visibility and any additional comments.

Thanks for the clarification, and sorry for not being more clear...
As I was going through the example notebooks in the pygdf repo, the examples in the README seem to switch contexts implicitly:
COPY ipums_easy FROM './ipums_easy.csv';
If I'm using the MapD CE Docker image, the above statement would imply the pygdf directory is inside the Docker container or mounted. However, currently I have pygdf installed outside of the MapD container (but on the same computer), so it took a bit to realize I could move the data file into the MapD container.
So it could just be clearer where a user should install pygdf (per your answer, wherever it can access the GPUs you want to use), and that MapD is completely separate.

Couple points of reference, now that I've had success with 100% of py.test tests passing, installing just in the conda environment (no Docker containers).

Ubuntu 16.04.4 LTS (GNU/Linux 4.13.0-37-generic x86_64) (Pop_OS/Ubuntu 17.10 didn't seem to work)
NVIDIA-SMI 384.111
CUDA not required to be installed

I'll close this because technically I don't need any more help, but perhaps clarifying the README for these or the actual minimum dependencies would help out the next person to try.

Thanks, @randyzwitch! We will be spending a lot of time updating all the READMEs and adding more testing soon.  Feel free to submit a PR for anything in the meantime.  Also, we really appreciate people using issues, thank you! Please join the slack channel as well.

I'll definitely start contributing to README's and documentation once I get acclimated.


Disable copilot for .env files

In .env files, Neovim is giving me the filetype sh, the same as in .sh files.
This means I can't disable Copilot for .env files without disabling it also for .sh files?
Would it be possible to disable Copilot for .env files, but keep it on for .sh files?

@primeapple can you try out https://github.com/zbirenbaum/copilot.lua/issues/113 and see if that solves the issue?


CLI command dns resolve4 causes infinite loop of DNS queries, in NxDomain cases

Describe the bug A clear and concise description of what the bug is.
The command dns resolve4 somename.org locks the CLI and leads to an infinite loop of DNS queries to the BR. In case the DNS response RCode back to the client is 'NxDomain'.
To Reproduce Information to reproduce the behavior, including:

Git commit id: latest commit (tried in simulation) and ot-reference-release (tried on Nordic)
IEEE 802.15.4 hardware platform: both simulation (OTNS2) and Nordic NRF52840
Build steps: standard for OTNS2 / ot-reference-release respectively
Network topology: one BR with one Router attached

Expected behavior A clear and concise description of what you expected to happen.
No retry if the DNS query response is received by the client, even if the RCode indicates an error.
Console/log output If applicable, add console/log output to help explain your problem.
(can be added if needed)
Additional context Add any other context about the problem here.
Problem comes up in cert tests while issuing dns resolve4 ipv4only.arpa from the reference device client.

Will take a look at it later when I get a chance. I will be less available today (Friday) and next week.

I believe I've found the issues. It's a typo involving an incorrect query type. I've submitted a fix:

https://github.com/openthread/openthread/pull/10766

I'd like to add a test for this as well, but I've submitted the PR for now to get the fix in place.

Added a second commit that adds a test case on the same PR. Having two commits makes it easier to cherry-pick the fix.

@abtink Oh thanks, that's really super-quick!

@abtink After applying the fix to the OTBR reference-release nrf52840 dongle build, it works! Now a new issue came up which I've created as Github issue: https://github.com/openthread/openthread/issues/10770
This was in context of debugging a cert test (11.5).

PR was merged, closing this issue


Add ilv avatar

If you want to create a space follow this guide
https://docs.snapshot.page/guides/create-a-space

I can't find your new space. Please make sure that you have filled in all required fields and added a strategy. https://council.illuvium.io/#/
There is a conflict in your PR, can you resolve it?

I can't find your new space. Please make sure that you have filled in all required fields and added a strategy. https://council.illuvium.io/#/
There is a conflict in your PR, can you resolve it?

It was created via ENS

Fixed the conflict, also the space is viewable here
https://snapshot.page/#/ilv.eth


Bc

I hope don't forget anything.

Merged, Many thanks for your work !

I just add a tag to use it easily


Examples of script invocations for training and inference?

Hello!
Thanks for making the project compatible with python-3 !
In setup.py, change the script name from 'bin/deepsphinx_train' to
'bin/deepsphinx-train' .
Could you please provide basic examples of script invocations for training and inference?
I ran
python bin\deepsphinx-train --trans-file data\ds-input.txt --nouse-train-lm --job-dir data\train5
and got a division by zero, something was probably wrong in the call.
Thanks,
Yuval
INFO:tensorflow:Getting speaker stats
INFO:tensorflow:Starting training
INFO:tensorflow:Epoch completed, saving
INFO:tensorflow:Evaluation started
INFO:tensorflow:Restoring parameters from data\elta\train55\checkpoints/-0
Traceback (most recent call last):
File "bin\deepsphinx-train", line 270, in
tf.app.run(train)
File "C:\Users\user\AppData\Local\Programs\Python\Python35\lib\sitepackages\tensorflow\python\platform\app.py", line 48, in run_sys.exit(main(_sys.argv[:1] + flags_passthrough))
File "bin\deepsphinx-train", line 153, in train
lm_fst)
File "bin\deepsphinx-train", line 79, in run_eval
tot_wer / tot_ev, tot_cer / tot_ev))
ZeroDivisionError: float division by zero

Your training data likely has less examples than batch size.
Sorry for being late for the docs, but I will be surely updating the docs for inference and training and provide a pre-trained model by tomorrow :-) .

That would be great!
Does the system support adaptation? can I add training data to a pre-trained model?
Thanks,
Yuval

Yes, you can start the training from a pre-trained model, but the vocabulary needs to be the same.

I have made a tool for inference. Please have a look.
Any more suggestions are highly welcome.


Move header files into a subfolder

I'd like to include this library in the vcpkg package catalog (https://github.com/microsoft/vcpkg/pull/20340) but ran into an issue that would be nice to get addressed here in the library instead of having to be patched on the vcpkg side.
Currently, the library puts all its header files directly under include. This could be problematic because there may be clashes with other libraries that use the same names for their include files, especially since some of the headers use generic names like add_ons.hpp.
I suggest to put all headers into a subfolder promise-cpp or similar, and refer to them via #include "promise-cpp/...hpp" in all files.

Good idea! I'll change the code in this way soon

https://github.com/xhawk18/promise-cpp/releases/tag/2.1.2

https://github.com/xhawk18/promise-cpp/releases/tag/2.1.2

Awesome, thanks for the quick action!


Unchecking the "Add popup video icon on video/thumbnail" doesn't work

Instead of making the popup video icon dissapear it just stays there.

Fix in v0.0.7
Thanks for your feedback.


Whats the name of plugin

Cannot find the name of plugin which i should use while executing command ext install.
I tried using ext install vscode-react but no result was returned. Currently using code v1.0.0

What you can do is type just ext install  then hit enter and you will see the list with all extensions. After that just type React and select this extension and press enter once again.

@xabikos thanks... found it. Writing Reactjs gives better result.


IONIC V4 : Cannot find module Issue..

Hello,
I am working on ionic v4 project and i am facing one issue, my all imports modules showing "Cannot find module Issue". but my application is working.

Note : I have read lot of online post related to this issue but its not resolved in any post.
Refrancs : https://github.com/Microsoft/vscode/issues/33209

ERROR

TSCONFIG

IONIC INFO

Project Structure

Looking at the other issues, it may have to do either with a plugin you are using or with the contents of your tsconfig.json. Here is a copy of mine.
{
  "compileOnSave": false,
  "compilerOptions": {
    "outDir": "./dist/out-tsc",
    "sourceMap": true,
    "declaration": false,
    "moduleResolution": "node",
    "emitDecoratorMetadata": true,
    "experimentalDecorators": true,
    "target": "es5",
    "lib": [
      "es2017",
      "dom"
    ]
  }
}

I would like to try yours in one of my projects just to see if I get the same results, but you posted a picture of yours instead of the actual contents. Can you please post the actual contents in a comment so I can copy it and try it?
Also, I am going to close this as it is totally off-topic for this issues list.


Autotile properties missing

Godot version:
0ead6b3
OS/device including version:
Windows 10
Issue description:
Autotile properties missing.
Steps to reproduce:
Make a tileset, set it to autotile
Minimal reproduction project:

Confirmed, also happen to me. Beta 1 still fine, beta 2 and master branch got this problem

I can confirm this happening on Beta 2 as well. Opening the project in an older build, changing the settings, then opening in Beta 2 works as a temporary workaround.

take a look to _get_property_list, _get and _set functions of the class AutotileHelper on tile_set_editor_plugin.* files, these class and functions are responsible from populating the property editor

@MarianoGnu I just change the property_editor->call_deferred("edit", helper); to property_editor->edit(helper); and it works fine. Dunno if this the right fix. It seems, edit function never called when using call_deferred.

Maybe the the edit function binding on property editor was renamed or removed, or the queue never got called for some reason, don't worry, this will work fine.


supervisor: make rejection periods work with stopTasksCount

Description
In cases where stopTasksCount is configured which makes streaming tasks run much longer (in some cases) than the configured task duration, early/late rejection periods are not correctly implemented since they are configured during the task startup time in IOConfig. Instead, this PR propagates updates the min/max times once every task duration.

Key changed/added classes in this PR

SeekableStreamIndexTaskRunner
SeekableStreamIndexTaskIOConfig

This PR has:

[x] been self-reviewed.

[ ] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[ ] added documentation for new or modified features or behaviors.
[ ] a release note entry in the PR description.
[x] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[x] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[ ] been tested in a test Druid cluster.

@suneet-s  Thanks for the review, I've changed the patch to just update the min/max times once every task duration. So this also avoids all those version problems.

Overruling CI.  The only remaining failures are code coverage.


[.NET Core] Support combination of NodaTime 2.0.0 and .NET 4.6.X

There is currently no version of ical.net that can be used together with NodaTime 2.0.0 and .NET 4.6.X.
Version 2 requires NodaTime 1.3.1.
Version 3 supports NodaTime 2, but it requires netstandard1.6 that is not compatible with .NET 4.6.X.
Please create version that will work with NodaTime 2.0.0 and .NET 4.6.X.
I believe that there is several possibilities:

upgrade version of .NET Standard from 1.6 to 2.0.
downgrade version of .NET Standard from 1.6 to 1.4.
create new release of ical.net v2 with NodaTime 2.0.0 as a dependency.

Interesting, thanks.
Notes to myself:

ical.net v2 uses NodaTime 1.*
ical.net v3 should use NodaTime 2.*

Have they figured out the story with BCL time zones yet?

Re-titling and labeling based on my thoughts above. ^^

Is this still a bug? net-core is building, and the tests are running.

Everything works. It can be closed.


Deleting exercise not reflected in regime and schedule

Not sure if intended

Intended behaviour.
@kwekke @jietung @garylyp @weihaw08  should we document this behaviour in the UG?


Question about EmplaceInternal

Hey,
I have a question about this if block in EmplaceInternal():
https://github.com/Wsm2110/Faster.Map/blob/082023c171d00e4829c0a2f5d14fd07ab3098ddc/src/DenseMapSIMD.cs#L732-L736
Is it ever theoretically possible to get inside this if block? I don't pretend to understand the algorithm perfectly, that's why I'm asking.
Currently, the only place EmplaceInternal() is called from is from Resize() where we first resize (double) the arrays and add the items from the old arrays (skipping the tombs). So I wonder whether it is possible at all with doubling the size of the arrays and skipping the tombs that we can still get to a moment where we would need to resize. This Resize()<-->EmplaceInternal() "recursion" kind of bothers me.
I have tested the code with the AddAndResizeBenchmark and with the data we have, we never get into that if block.

The thing is... After a resize the hashcodes hash to a different index, hence it's possible to get an index which is out of bounds. Anyways.. i made some changes to the algorithm


Monorepo suggestion: change the directory structure to follow Flutter recommendations

In order to run Flutter applications using the locally build engine, there are following options available:
--local-engine-src-path to specify the path to engine/src directory
--local-engine to select the engine mode.
With the current structure of the monorepo you need to specify both. However, Flutter has a nice default for --local-engine-src-path: it can be omitted if Flutter engine is checked out to a directory engine alongside Flutter framework checkout in flutter (also see https://github.com/flutter/flutter/wiki/The-flutter-tool#using-a-locally-built-engine-with-the-flutter-tool).
From https://github.com/flutter/flutter/wiki/Setting-up-the-Engine-development-environment:

Create an empty directory called engine for your copy of the repository and cd into it. (It is possible to use a different name, but some tools assume this name unless configured otherwise, so calling it engine will make thing easier.)

So, if following the steps from https://github.com/flutter/flutter/wiki/Setting-up-the-Engine-development-environment, the directory structure would look in the following way:
engine/src/flutter - Flutter engine repository.
flutter/ - Flutter framework repository.
It would be nice to follow this directory structure and avoid the need to specify --local-engine-src-path.

This can't be done because we use the flutter/engine DEPS as the basis for the monorepo DEPS, with only very few changes, and that DEPS checks out engine and Dart into src/flutter and src/third_party/dart relative to the base of the checkout.
All of the third-party dependencies are checked out into src/third_party, and these paths are hardcoded. They would all
have to be changed to engine/src/third_party.
It would require a significantly changed DEPS file to put them into engine/src/flutter instead of src/flutter. This would add
a lot of complexity to the system, and make the monorepo DEPS very different from the engine DEPS.
I would rather suggest that the default for the flutter tools could be changed from engine/src/flutter to src/flutter, or the tools allow either path.  There is no fundamental reason for the default to include the "engine" directory, and to put the engine checkout there.

An issue asking for a way to configure the default local-engine got the response that this should be added to the [HOME]/.flutter_settings created by flutter config. https://github.com/flutter/flutter/issues/37929
That would probably be the best way to set a sticky local-engine-src-path as well.
There is a LOCAL_ENGINE_SRC_PATH environment variable used by the integration tests, but they only affect the test scripts, not all the flutter commands.

Actually, the help for flutter, `flutter help -v' says that the FLUTTER_ENGINE environment variable can be used to set a default for --local-engine-search-path.  I will test this, and then add it to the instructions.

You mentioned that monorepo DEPS is not the same as engine DEPS already. So maybe processing deps mapping and substituting src with engine/src is not so complex change after all.
Re: [HOME]/.flutter_settings and FLUTTER_ENGINE environment variable: these approaches assume that you only have one checkout of the Flutter engine. But it is likely for the developer to have another checkout of the Flutter engine already, or having multiple monorepo checkouts longer term.

There is a script that makes the changes, and it is a modified version of the script at
https://github.com/flutter/buildroot/blob/master/tools/dart/create_updated_flutter_deps.py
The modifications to the script are at:
https://dart-review.googlesource.com/c/monorepo/+/243501
It would be a much bigger script if it needs to go through and modify all the paths of dependencies, and we would probably make it a separate script: run the original unmodified script to create the engine DEPS with SDK deps changes rolled in, then process it by a separate monorepo script to make the existing changes and to change all the dependency paths.
I would first want to check if it wouldn't be better to change flutter/engine DEPS to also check everything out into an engine/src directory, since that is what all the flutter tools expect.  Then flutter/engine/DEPS and monorepo/DEPS would remain mostly identical, rather than being mostly different.  We could also add a path variable to engine/DEPS that would default to src/ but could be set to engine/src/ in the monorepo copy - that is what we have done in the past when we need a DEPS file to check out into different relative locations in different contexts.

engine/src/flutter - Flutter engine repository.
flutter/ - Flutter framework repository.

This has always been the standard recommended when working with flutter and engine repos.
I was always puzzled as to why we deviate from that when building hhh patches for example.

Help me what I do next

@Tru3boi read https://github.com/flutter/flutter/blob/master/CONTRIBUTING.md next. Please don't spam random issues.
@whesse We should stick to the documentation as much as possible: https://github.com/flutter/flutter/wiki/Setting-up-the-Engine-development-environment

I will change the monorepo structure to put the engine at engine/src/flutter, as suggested here.

CLs to make this change to monorepo:
https://dart-review.googlesource.com/c/monorepo/+/247200
https://dart-review.googlesource.com/c/monorepo/+/247201

The structure of monorepo, and all documentation and rollers, has been changed to use the new layout with engine/src.


Add [ArchLinux] package version support

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @scop!

  Generated by :no_entry_sign: dangerJS against 1c5311dd70f61510df01c14dbf0dd47db6e0bed3


Use gRPC interceptor instead of data converter for encoding/decoding

Fixes #367
Depends on unmerged PR https://github.com/temporalio/sdk-go/pull/1303
What was changed

Do payload encoding/decoding for all client requests, using a gRPC interceptor
Stop doing payload encoding/decoding in the dataconverter
Eliminate the global mutable dataconverter model

Why?

temporal workflow show was respecting --codec-endpoint but temporal workflow show --output json was not

Why do we think this change is correct?
For this change to maintain previous semantics, all the code paths that were using the custom dataconverter must have been operating on data that was either a response obtained using SDKClient, FrontendClient, or OperatorClient, or is being prepared for a request via one of those clients. I have added comments inline establishing that this is true.
For this change to be correct, we require that all code paths that send/receive data containing Payloads are using SDKClient, FrontendClient, or OperatorClient.
How was this tested

Manual testing only so far, using https://github.com/temporalio/samples-python/tree/main/encryption:

[ ] More testing needed

Checklist

Addresses #367 w.r.t. workflow show --output json
Any docs updates needed?
No

@dandavison - Look at https://pkg.go.dev/go.temporal.io/api/proxy#VisitPayloads for visiting all payloads (be sure to exclude search attributes). https://github.com/temporalio/sdk-go/blob/v1.25.1/converter/grpc_interceptor.go#L47 is an example of it being used.

@dandavison did you test search attributes? Those don't appear to be excluded by NewPayloadCodecGRPCClientInterceptor ?

Superseded by https://github.com/temporalio/cli/pull/421


New release

Is a new release on the way?

Bump, also curious about this

Once #248 and other dependency updates are in, we can do a new release

We've cut v0.1.1, although our goreleaser bit is having issues. We may have to do more 0.1.x versions until we have a fix.


Altitude value goes missing from dragged marker

I have found that when editing a point/polyline/polygon that moved points loose the alt property on getLatLng(). this only happens for moved points, so if a point in a line is not moved, it retains the alt property.
I am assuming it happens on 'drag' of the marker as the position is updated, I have been having a look through the source but have not yet been able to find it.
I assume a simple as checking if the alt attribute exists on the original position and just copying it to the new position.
something like:
if(typeof oldLatLng.alt !== 'undefined'){
	newLatLng.alt = oldLatLng.alt;
}

Good catch! Will be fixed.
This is happening because we create a new LatLng object:
https://github.com/geoman-io/leaflet-geoman/blob/1fdc918fa39ffa84327fdf639fa75865168f716d/src/js/Mixins/Dragging.js#L345-L348

Not surprised it was missed, Did not even know leaflet supported altitude until yesterday when I tested what would happen if I created a layer from GeoJSON that contains altitude in the coordinates.
I plan on using it in the wrong way as it is, I will be storing height there, not altitude, so an altitude of 1 will be 1m above ground. -0.6 will be 600mm below ground etc.


Compact explorer tree nodes don't narrate "Expanded" and "Collapsed"

Using NVDA to test https://github.com/microsoft/vscode/issues/85039: clicking on any of the compact items expands and collapses the node. However, it doesn't read "Expanded" and "Collapsed" the way non-compact nodes do.
Reads "Expanded" and "Collapsed" when I expand and collapse, respectively.

Does not read "Expanded" and "Collapsed" when I expand and collapse, respectively.

Good feedback. Thanks. Assigning to december to investigate.

We do not set all the needed aria properties on the subnodes. One of them being the expanded and collapsed state.
So idealy we would set the aria-expanded somewhere here
Missing issue: currently the navigationController has no event to listen to update the expanded flag.
I do not think it is a big issue, thus assigning to backlog candidates until users complain.

Hi, please could I have this issue? Saw this ad on Reddit and thought I should get involved!

@davidackroyd99 this issue is not so trivial, however if you would like to take a stab feel free. Once you have something working ping me on the PR. How the tree sets the activedescendent for the compact subnodes might be a good example how this could potentialy be done.

cheers :) will let you know when I have something!

Pending a proper fix, could explorer.compactFolders default to false if not otherwise set if screen reader usage is detected? It causes some difficulty when trying to navigate in the explorer view.

@Neurrone let's assign to next milestone so we look into a proper fix. Thanks for bringing this up.
If we do not fix it, we can disable the compactFolders for screen readers.

Pushed a fix for this such that we now properly set the aria-expanded and aria-level property on the compact nodes.
@Neurrone try it out and let us know what you think. Thanks!

There's still something weird happening here. I get an extra "collapsed" when I click on the rightmost folder to expand it. I just expanded it, should it read collapsed like that (it doesn't for non-compact nodes).

narrates:
level 6  classification  not selected  collapsed
expanded

Thanks @alexr00 for looking into this. I just checked the corner case you mentioned and I can indeed reproduce this. However since screen readers users do not use the mouse, and this issue can not be reproduced with the keyboard I am fine living with this.
Thus closing this and adding verified label as you said that it can be considered verified.


Remove SERVER_VERSION and PRODUCT_TYPE assertions

In the SupportedOptionsTest, we check the SERVER_VERSION and PRODUCT_TYPE if the backend is of, type DSE. In some of the variants of DSE, these are not available, so we need to remove the assertions for these two properties.

we chose to bypass the test for DSE.


Introduce no nodejs compat flag error page

@WalshyDev has pointed out to me the fact that our runtime error message: Error: Could not access built-in Node.js modules. Please make sure that your Cloudflare Pages project has the 'nodejs_compat' compatibility flag set. is probably not clear and developers don't get enough instructions on what to do, so to improve things I've updated next-on-pages to serve a more descriptive and actionable error page in case of this runtime issue.

when we'll have configs for Pages projects I think this should become a build time error instead of a runtime one, but for now we can only deal with this at runtime 🥲

Result: https://next-nodejs-compat-page-test.pages.dev/

because of the lack of better ideas I tried to mimic the error page developers get when a worker throws: https://f6e0ff92.next-nodejs-compat-page-test.pages.dev/

Important:
The new page has been deployed to https://no-nodejs-compat-flag-static-error-page.pages.dev and next-on-pages workers fetch it from there, I decided to implement it this way as I thought that the users' workers should not bear the weight of this new page, especially since they should only see it at most once in their application, if you disagree I can make it so that it's included in each next-on-pages app instead

The page we're using here has quite a few errors on it. I'd personally in-line it in the project so it's versioned with the package, but at the very least, we need to review the content of the page.

yeah good point about including it in the output app itself, I didn't want to include the html in the project's assets but by chatting with @petebacondarwin I understood that the impact it has on the app assets limits is very negligible, so I'm including it there and not having an external deployed app for this.
Regarding the errors on the page, I did notice that once I swapped compat and nodejs 🤦 I've fixed that, thanks for pointing that out, besides I can't spot other problems there, could you let me know what other errors you saw there? 😓


import ics fails in google calendar

it seems that the format where incorrect, the time is specified so we should use DATE-TIME (i don't know if it still works with all day event though)

@nwcell +1 for this merge :)

@pataluc it works fine with all day events

Changes added in a different commit


ARROW-5387: [Go] properly handle sub-slice of List

@stuartcarnie @alexandreyc

Codecov Report

Merging #4360 into master will decrease coverage by 15.77%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           master    #4360       +/-   ##
===========================================
- Coverage    88.3%   72.52%   -15.78%
===========================================
  Files         780       71      -709
  Lines       98400     5947    -92453
  Branches     1251        0     -1251
===========================================
- Hits        86889     4313    -82576
+ Misses      11275     1407     -9868
+ Partials      236      227        -9

Impacted Files
Coverage Δ

go/arrow/array/list.go
88.98% <100%> (+13.76%)
:arrow_up:

python/pyarrow/ipc.pxi

cpp/src/arrow/csv/chunker-test.cc

cpp/src/parquet/column_page.h

cpp/src/parquet/bloom_filter-test.cc

cpp/src/arrow/array/builder_decimal.cc

cpp/src/plasma/client.cc

cpp/src/arrow/io/test-common.h

cpp/src/arrow/util/int-util-test.cc

cpp/src/arrow/python/io.cc

... and 702 more

Continue to review full report at Codecov.

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

ping? (I'll wait 'till Friday before merging this in.)


Dommain 2020

Reference
doi or URL: https://doi.org/10.3389/fevo.2020.00218

Hi @campanam thanks for adding the publication.
Feel free to ping @SPAAM-workshop/ancientmetagenomedir-coreteam if you have any questions when making a PR!


Memory leaks in version 1.0.x

Describe the bug
The FirebaseApp causes the memory leaks when authentication task started and the async task was removed from the queue but it is remaining in the memory.
To Reproduce
Set the wrong end point of the Google APIs for auth task request.  The request will fail and the memory leaks will be occurred in the next auth request.
Expected behavior
The auth task's async data should be deconstructed.

This issue is now fixed in v1.1.0.


Replace ICompileRequest with ISession

Slang will be deprecating the old C API and ICompileRequest interface. Our code still depends on the latter.
We should implement vtables for ISession and related interfaces as well as Rust friendly wrappers around them.
See: https://github.com/shader-slang/slang/issues/3617

Closed by 095c719


Can I load a wav file without JAMS?

Hi:
I would like to do data-augmentation for my bunch of wav files.
Can I use MUDA without JAMS file?
Thanks!

Note to self: we should add an example of this to the documentation.

Glad to hear that an empty JAMS is a solution.
Expecting to see the example code since I still have no ideal how to do it.
Thanks for the reply!

It turns out this is so simple. (Sorry I'm a rookie in this)
import jams

jam = jams.JAMS()
filename='some.wav'
j_orig = muda.load_jam_audio(jam, filename)


Add support for Azure Stack HCI IMDS Identity/token endpoint

Feature Request
My team owns the IMDS endpoint for Azure Stack HCI. Recently, we were working with another Azure team as they tried to implement MSI authentication utilizing our endpoint and it does not seem to be supported.
After that team and I looked through the code we see that support for our endpoint would work if not for the constant definition of the endpoint, and attempting to set environment variables so that the "HIMDS" code flow triggered also fails because of getAzureArcSecret().
The response for our endpoint is the same as Azure IMDS endpoint  and contains an access token. And like Arc/HIMDS we also only support system-assigned identities (not user-defined). I am happy to engage in further talks if needed so that support can be brought to the Azure sdk.

Thanks for opening this issue! Azure Arc has a unique protocol for managed identity authentication. If Azure Stack HCI's protocol is truly identical to IMDS's, then you don't want to imitate Arc, only override azidentity's hardcoded IMDS endpoint. That isn't possible today, however it is possible in Azure SDK authentication libraries for other languages and we could implement their solution here.
The way it works for e.g. the Azure SDK for Python is the managed identity client uses the value of environment variable AZURE_POD_IDENTITY_AUTHORITY_HOST as the base IMDS URL. So, you set that variable and when in IMDS mode, the client sends token requests to $AZURE_POD_IDENTITY_AUTHORITY_HOST/metadata/identity/oauth2/token. The variable name is perhaps awkward for your case but the feature seems to do exactly what you want and it's straightforward for us to copy it to azidentity.
That said, I have a couple questions:

Setting the variable name aside, would this feature work for Azure Stack HCI? Does its managed identity implementation have any different requirements or behaviors compared to IMDS?
If this is the right feature, is AZURE_POD_IDENTITY_AUTHORITY_HOST an acceptable name or a blocker e.g. because users are responsible for setting it?


integration-cli: do not check creation datetime, it can change

In TestImagesFilterWhiteSpaceTrimmingAndLowerCasingWorking
if the time had changed between image listings, listings will be
different because they include creation time. Patch removes
creation time from comparison.

check lenth != 0, in case of empty line

We got it running tests on v1.5.0 with vfs graph driver, for img
8ba30c03fa38 CREATED had changed from "3 minutes" to "4 minutes",
fail log:
out 0
<none>              <none><PHONE_NUMBER>7c        4 minutes
ago       2.43 MB
<none>              <none>              8ba30c03fa38        3 minutes
ago       2.43 MB
<none>              <none>              d589e4d704ca        3 minutes
ago       2.43 MB
REPOSITORY          TAG                 IMAGE ID            CREATED
VIRTUAL SIZE
out 1
<none>              <none><PHONE_NUMBER>7c        4 minutes
ago       2.43 MB
<none>              <none>              8ba30c03fa38        3 minutes
ago       2.43 MB
<none>              <none>              d589e4d704ca        3 minutes
ago       2.43 MB
REPOSITORY          TAG                 IMAGE ID            CREATED
VIRTUAL SIZE
out 2
<none>              <none><PHONE_NUMBER>7c        4 minutes
ago       2.43 MB
<none>              <none>              8ba30c03fa38        4 minutes
ago       2.43 MB
<none>              <none>              d589e4d704ca        3 minutes
ago       2.43 MB
REPOSITORY          TAG                 IMAGE ID            CREATED
VIRTUAL SIZE
out 3
<none>              <none><PHONE_NUMBER>7c        4 minutes
ago       2.43 MB
<none>              <none>              8ba30c03fa38        4 minutes
ago       2.43 MB
<none>              <none>              d589e4d704ca        3 minutes
ago       2.43 MB
REPOSITORY          TAG                 IMAGE ID            CREATED
VIRTUAL SIZE
out 4
<none>              <none><PHONE_NUMBER>7c        4 minutes
ago       2.43 MB
<none>              <none>              8ba30c03fa38        4 minutes
ago       2.43 MB
<none>              <none>              d589e4d704ca        3 minutes
ago       2.43 MB
REPOSITORY          TAG                 IMAGE ID            CREATED
VIRTUAL SIZE

Can you please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "do-not-check-creation-datetime" git@github.com:Snorch/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

This will update the existing PR, so you do not need to open a new one.

Sorry, missed that.

commit e39ea834a92154574be3dde67ae30acdb25b32f7 does almost the same thing, closing issue


Add a template for deploying a Helm chart through a pipeline.

What is version of KubeSphere DevOps has the issue?
v3.3.1
How did you install the Kubernetes? Or what is the Kubernetes distribution?
kubekey
Describe this feature
Refer to https://kubesphere.io/forum/d/9485
Some users may not be familiar with how to deploy a Helm chart through KubeSphere's DevOps process to their current KubeSphere clusters. Currently, it can be done by using the Helm CLI in a pipeline step. To make it easier for users to refer to and use, a pipeline template can be integrated into the KubeSphere platform.
Additional information
An excellent example is https://github.com/Feeeenng/devops-maven-sample/blob/helm/Jenkinsfile-helm, which can be used to create a template. The template can be committed to the location https://github.com/kubesphere-sigs/pipeline-templates/tree/main/featured/pipelines. When integrated with the next release of DevOps, it will be automatically included in the version.

this is sample example.
https://github.com/Feeeenng/devops-maven-sample/blob/helm/pipeline-helm.yaml
The next step plan will be realized through ClusterStepTemplate, which can be added in the user's own pipeline.

发版后才能验证


better text support

Reason for PR
In ie8, text was appearing as the word "undefined" regardless of the content of the text element. So for example:
<text font-family="'HelveticaNeue-Medium'" font-size="4.2491">SPEED</text>

In a modern browser, this showed up as SPEED but in ie8 it showed up as undefined.
Solution
A simple change to support where the text was showing up on the elShape object in ie8 vs in chrome, etc. In chrome it was in the property text, or textContent, but in ie8, it was in the properties innerText and outerText. Pretty simple fix.
Additional Test Case
I added a more full featured test case of a speed limit sign to show that the text now works (you'd have to test in ie8 though, which I doubt anyone but me is doing at the moment).

Sweet, thanks for catching this. I'll get it in later today.

There's some very minor things, but it looks good to me +1

the viewbox on the svg makes it so the original svg is teeny tiny so it doesn't match the scale of the raphael version
there's some extra attributes that aren't related to the tests, I think they might distract some. I'd remove them.

I'm planning on merging this when I get home in an hour as is. They're my own personal stylistic preferences.


golangci-lint: Increase timeout

What this PR does / why we need it:
The time it takes for the linters to run seem to have a large
fluctuation range, between 4sec to over 10sec.
As there are no guarantees given for such jobs (e.g. CPU time), we are
left with increasing the timeout period.
This is mainly triggered by the instability (timeouts) on CI runs.
A possible alternative is to define resource requirements for jobs,
reducing the inconsistent run duration results.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:
NONE

/hold for getting the clone fix PR in
FYI @brianmcarey

/unhold


[zh-cn] Update projected-volumes.md

Signed-off-by: Gao Qian<EMAIL_ADDRESS>

Seems a dup of #37800

closing this due to: 1. it is a dup; 2. it is a partial update.

/close


Snyk test1

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: Atulkumar Gaikwad:x: atul2387Atulkumar Gaikwad seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


app.diagrams.net will stop working for Embed mode

While clicking edit in wiki page, there is a window prompts:
'app.diagrams.net will stop working for Embed mode. Please use embed.diagrams.net.'
But the original diagram can be shown correctly. but could not be edited.
Can anyone help?

Hey, i found a solution that worked for me:
Locate the file "ext.drawioeditor.js" and change the following lines:

from src: 'https://www.draw.io/?embed=1&proto=json&spin=1&analytics=0&db=0&gapi=0&od=0&picker=0',
to src: 'https://embed.diagrams.net/?embed=1&ui=atlas&spin=1&modified=unsavedChanges&proto=json',

from this.iframeWindow.postMessage(JSON.stringify(data), 'https://www.draw.io');
to this.iframeWindow.postMessage(JSON.stringify(data), 'https://embed.diagrams.net');

from if (e.origin != 'https://www.draw.io')
to if (e.origin != 'https://embed.diagrams.net')

This has been fixed by #23


Config tweaks

Break once a masked key is found in config show
Clear dirty config fields after Marshalling
Switch Config.timeout from int to time.Duration

@ArmandGrillet : thanks for the review, there is already a comment, let met know if that's not clear enough.
https://github.com/dcos/dcos-cli/blob/master/pkg/config/config.go#L149-L150


Criminal record hud icons

About the PR

Added three more statuses to the criminal record console: Paroled, Suspect, Discharged.
Entities now have a hud icon based on the criminal record linked to the name they are currently showing.
If "Urist McHands" is wanted, anyone that currently shows as being "Urist McHands", even when their real name is not Urist McHands, will show the relevant hud icon to entities wearing a security hud. If Urist McHands is disguised as someone without a criminal record or has no name(middle-aged captain man etc.), they won't show an icon.
Why / Balance

Icons showing on sechuds will make sechuds more usefull, it will (hopefully) improve security coordination by allowing officers to more easily see who they need to keep track of.
It also gives warden something extra to do.
Technical details

When a criminal record is changed, every entity with an IdentityComponent will have their name checked to see if it equals the name of the changed criminal record, if it's equal the person will get the CriminalRecordComponent which holds the icon that needs to be shown on the entity.
If a criminal record is removed, the CriminalRecordComponent will be removed from all entities that currently have the visible name associated with the criminal record.
If an entity changes identity, all criminal records will be searched to see if the name has a criminal record attached to it. If the name has a record attached to it, the entity will get the CriminalRecordComponent and the corresponding criminal record hud icon until they change their identity again or the criminal record gets removed.
Media

https://github.com/space-wizards/space-station-14/assets/137322659/3c9422e6-897f-456f-b54b-8fdc053cb4e1

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl: Dygon

add: The following criminal record statuses have been added: Suspect, Discharged, Paroled.
add: Security huds now show an icon on entities that have a visible name linked to a criminal record.

different way could be to get the persons visible id card, get its criminal record, check its status
then no dictionary needed

now theres even more reason to use the criminal records computer, great change

What happens if you have a fake identity when the record is changed, and then remove it afterwards? Do you still show as wanted etc?

different way could be to get the persons visible id card, get its criminal record, check its status
then no dictionary needed

Thanks for the suggestion, I'll first try your idea of saving it on the station instead of the record.
The reason I linked the status to a name is so if Urist McHands is wanted and has their face visible(so they show as Urist McHands) while having someone else's id in their pda, they will still show as wanted. I think if it's linked to id it's too easy to just not get an icon(but maybe we want this?), even if not disguised. With it based on the name you need to disguise yourself if you don't want to have an icon show on the sechud.
The question is if we want only id's to be the thing that mark you as a criminal, or does the sechud also have facial recognition for criminals(with the dictionary).
Linking it to the id does make agent id's a bit more valuable, because I would make those immune to receiving a criminal record.  It also makes the hud way more unreliable though and people might use it less because you have to shift click people anyway, people with their id anywhere else than their pda slot will just not have an icon(even if not disguised).

What happens if you have a fake identity when the record is changed, and then remove it afterwards? Do you still show as wanted etc?

When a record is changed, all entities that currently have that name as their visible name(fake or not) will have their status adjusted. When you change to another name you will get the status icon belonging to that new name(or no icon if not a criminal).

Finally, crime hud will be needed not only to check someone for a mineshield!

What needs to be done so this can be merged?

@Arendian In the server, we don't see the icon and when we wanted a mindshield member, all job icon disapear


Developer tools services view

Developer tools services window view
dont know why was changed the size of the white window with the scrool bar
before use to be bigger, you could see more of the available services faster
now you only see 2 and got use then scroolbar
the way is now is not pratical

I can tell you this is fixed in 0.66.0.b2.

incorrect issue reports are also not practical and this is fixed already

I ha Hassio docker, have version 0.66.0.b2, reboot system and i dont see no changes on it, is still the same

Have you reloaded the page in your browser? It's cached unless you trigger a reload. If that doesn't work try in an incognito window.


New-command: onedrive report activityusercounts

implements #1256

Coverage remained the same at 100.0% when pulling 87eeb9d18243630bbbde8d68f459c2cb14491b2a on plamber:spo-report-onedriveactivityusercounts into 0e5efd9d93e854d43547e9e1396e619ace74e34d on pnp:dev.

@VelinGeorgiev: you can review this one

Thank you very much Patrick @plamber ! We'll review it asap.

Merged manually. Thanks! 👍


[New Scheduler] Add container counter

Get container count from ETCD when related data get updated in ETCD

Description

It watchs ETCD data changes using WatcherService implemented in #5069, when any new container data under specified namespace is inserted/removed to/from ETCD, it will get count of containers for the namespace by querying ETCD
Related issue and scope

[ ] I opened an issue to propose and discuss this change (#????)

My changes affect the following components

[ ] API
[ ] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[x] Scheduler
[ ] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[x] Tests
[ ] Deployment
[ ] CLI
[ ] General tooling
[ ] Documentation

Types of changes

[ ] Bug fix (generally a non-breaking change which closes an issue).
[x] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[x] I signed an Apache CLA.
[x] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[x] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[ ] I updated the documentation where necessary.

Codecov Report

Merging #5072 (692aa7a) into master (d8cf172) will decrease coverage by 6.82%.
The diff coverage is 84.21%.

@@            Coverage Diff             @@
##           master    #5072      +/-   ##
==========================================
- Coverage   81.81%   74.98%   -6.83%
==========================================
  Files         204      205       +1
  Lines        9950     9988      +38
  Branches      447      453       +6
==========================================
- Hits         8141     7490     -651
- Misses       1809     2498     +689

Impacted Files
Coverage Δ

...nwhisk/core/scheduler/queue/ContainerCounter.scala
84.21% <84.21%> (ø)

...core/database/cosmosdb/RxObservableImplicits.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...ore/database/cosmosdb/cache/CacheInvalidator.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...e/database/cosmosdb/cache/ChangeFeedConsumer.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...core/database/cosmosdb/CosmosDBArtifactStore.scala
0.00% <0.00%> (-95.48%)
:arrow_down:

...sk/core/database/cosmosdb/CosmosDBViewMapper.scala
0.00% <0.00%> (-93.90%)
:arrow_down:

...tabase/cosmosdb/cache/CacheInvalidatorConfig.scala
0.00% <0.00%> (-92.31%)
:arrow_down:

...enwhisk/connector/kafka/KamonMetricsReporter.scala
0.00% <0.00%> (-83.34%)
:arrow_down:

...e/database/cosmosdb/cache/KafkaEventProducer.scala
0.00% <0.00%> (-78.58%)
:arrow_down:

...whisk/core/database/cosmosdb/CosmosDBSupport.scala
0.00% <0.00%> (-74.08%)
:arrow_down:

... and 18 more

Continue to review full report at Codecov.

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


Visualization Tool: Add a table view for resources page

Add an option for the user to show parsed resources in a more compact view like a table for example. This can be useful for projects with a lot of resources and for quick insights

Hello, I want to start working on this issue. Can I be assigned to it.
Also, just to be on the same page. We are talking about some sort of a toggle button which can switch between current(tiled) view and the new compressed table view?

Hi, thanks for your interest. We are not assigning issues. If you are interested just submit PR with the change and we will take a look.
We can start with a toggle button, I think.

Heya, I have made the relevant changes, does this looks good enough to resolve the issue?
Demo:

I have raised a PR https://github.com/google/gcp_scanner/pull/300 for this issue


Failed to locate library when using modules

Environment

LWJGL version: 3.2.2
LWJGL build #: 10
Java version: 12.0.1 amd64
Platform: Windows
Module: core
IDE: Eclipse Java 2019-03

Description
Hey.
My module-info file is as such:
module example.jigsaw.error {
	exports example;

	requires org.lwjgl;
	requires org.lwjgl.natives;
	requires org.lwjgl.glfw;
	requires org.lwjgl.glfw.natives;
	requires org.lwjgl.opengl;
	requires org.lwjgl.opengl.natives;
        requires org.lwjgl.jemalloc;
	requires org.lwjgl.jemalloc.natives;
}

And I copied the code from https://www.lwjgl.org/guide into the example package, adding the package example; on the top.
Then, I added the entire LWJGL library onto my modulepath and, enabling debug mode, hit run, this being the output.
Hello LWJGL 3.2.2 build 10!
[LWJGL] Version: 3.2.2 build 10
[LWJGL] 	 OS: Windows 10 v10.0
[LWJGL] 	JRE: 12.0.1 amd64
[LWJGL] 	JVM: OpenJDK 64-Bit Server VM v12.0.1+12 by Oracle Corporation
[LWJGL] Loading library (system): lwjgl
[LWJGL] 	lwjgl.dll not found in java.library.path
[LWJGL] Failed to load a library. Possible solutions:
	a) Add the directory that contains the shared library to -Djava.library.path or -Dorg.lwjgl.librarypath.
	b) Add the JAR that contains the shared library to the classpath.
Exception in thread "main" java.lang.UnsatisfiedLinkError: Failed to locate library: lwjgl.dll
	at org.lwjgl/org.lwjgl.system.Library.loadSystem(Library.java:147)
	at org.lwjgl/org.lwjgl.system.Library.loadSystem(Library.java:67)
	at org.lwjgl/org.lwjgl.system.Library.<clinit>(Library.java:50)
	at org.lwjgl/org.lwjgl.system.MemoryUtil.<clinit>(MemoryUtil.java:97)
	at org.lwjgl/org.lwjgl.system.Pointer$Default.<clinit>(Pointer.java:67)
	at org.lwjgl/org.lwjgl.system.Callback.<clinit>(Callback.java:40)
	at example.jigsaw.error/example.HelloWorld.init(HelloWorld.java:39)
	at example.jigsaw.error/example.HelloWorld.run(HelloWorld.java:24)
	at example.jigsaw.error/example.HelloWorld.main(HelloWorld.java:114)

Whats interesting about this is that if I delete the module-info file and switch LWJGL onto the classpath instead of the modulepath I get no errors whatsoever and the application runs as it should.
Thanks in advance for any assistance.

The issue is likely related to:

IDE: Eclipse Java 2019-03

Module support in Eclipse has been problematic. You can search for similar issues here or on the JGO forums. Afaict, no user has been able to get Eclipse to build modular projects properly.
You could verify that it's not an LWJGL issue by building and running your project from the command line.

Yes, your right, thanks.
Ran it with the exact same dependencies that I used in eclipse via the command line and it ran just fine. Interesting...
Anyway, I'll close the issue now.

Out of curiosity... has anyone been able to build it with Intellij or Netbeans?

IntelliJ has great module support, even before Java 9 was officially released. Netbeans supports modules since mid-2018 and works fine afaict.


is od_strategy _FULL_AREA still needed with current logic in algorithm?

When taking a high relevant distance and bigger areas (that include openbaardomein), these OD-parts where removed when using OD-strategy OD_SNAP_SINGLE_SIDE/OD_SNAP_ALL_SIDE. Therefor 2 variants 'FULL_AREA' were added that always take the inner part of the thematic geometry into the results.
Because the algorithm-logic is ameliorated in v0.5.0, this inner part is always added to the results. So it can be that these 2FULL_AREA strategies are not needed anymore? to research!

OD_FULL_AREA_SNAP_ALL_SIDE much slower than OD_SNAP_ALL_SIDE


feat: disabling newly added kubelet serving certificate rotation

Fixes #
Description
This PR in AgentBaker enables the kubelet flag --rotate-server-certificates kubelet flag.  This flag will auto request and rotate the kubelet serving certificates by requesting new certs from the apiservers certificates.k8s.io apigroup when a certificate expiration approaches. This also requires the RotateKubeletServerCertificate feature gate to be enabled. The existing model just self signs the certificates.
In terms of the karpenter impact, we are just going to stick with self signed certificates for now since we don't have everything configured to leverage this functionality.
How was this change tested?

make presubmit

Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[ ] No

Release Note

we'll need to come up with a way Karpenter can determine when it's safe to enable this flag

will probably end up deploying the required plugin whenever karpenter is enabled to ensure this flag is safe to set, for now will keep as false until the new plugin as rolled out

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 97.702%

Totals

Change from base Build<PHONE_NUMBER>2:
0.0%

Covered Lines:
36313

Relevant Lines:
37167

💛  - Coveralls

one thing to also note, Karpenter will need to make to not pass --tls-cert-file and --tls-private-key-file as kubelet flags when this is enabled

Why do we need to do anything to preserve the current functionality? The support ENABLE_KUBELET_SERVING_CERTIFICATE_ROTATION is (correctly) added with backward compatibility: not having this flag in contract is the same as setting it to false.


Hyperlinks mixed up for "Tutorial: Implement Adobe Experience Cloud with Platform Web SDK"

Issue in /src/pages/implement/client-side/aep-web-sdk.md
I believe the two hyperlinks in this section are backwards.  First link takes you to a tutorial and the second link is for learning how to implement.
Learn how to implement Experience Cloud applications using Adobe Experience Platform Web SDK. For information specific to Target, see Set up Target with Platform Web SDK within the tutorial.

Hyperlinks in correct order, but the wording was unclear/confusing. Clarified the wording.


Theme for tabletop simulator

I did a thing https://github.com/ChildishGiant/dracula-tabletop

Hey @ChildishGiant, thanks for your patience. Can you move ahead with transferring the repository to the Dracula organization?

All done 😄 How does it get added to the website?

Aaaaand it's live! https://draculatheme.com/tabletop-simulator
Thanks @ChildishGiant :D


[a11y extension] location location location

I would like to propose to move the dist and build files back into this repo.
Since we're not putting the final extension in the third-party repo, we might as well push them to the CDN from this repo; the other one just adds noise and confusion.

I'm OK with moving things back.
I'm afraid I'm a little out of the loop on where the files are to be stored.  Not in the third-party repository, but on the CDN, somewhere.  What is the URL going to be for that?
I'm assuming the unpacked versions are not going to be available on the CDN, right?  So the diet folder represents what will be on the CDN?

What is the URL going to be for that?

I was thinking to still upload this to where the third-party-repo ends up, i.e., cdn.mathjax.org/mathjax/contrib.

I'm assuming the unpacked versions are not going to be available on the CDN, right? So the dist folder represents what will be on the CDN?

That's what I was thinking. But it's not very important to me, 1/10.

As per F2F, move things back here, rename this to mathjax-a11y

I was trying to tackle this (to get into the code and start #97) but the grunt-file seems a bit odd (and misses some resources).
I've begun writing a fresh one.
@zorkow any suggestions on providing the closure compiler? (Or perhaps publish mathjax-sre as a separate module to npm?)

Here's what I came up with. https://gist.github.com/pkra/fe1bd64ba2053665923d07e2747883da

Quick smoke test indicates this creates a viable distribution.

I've made some minor changes to make repeated builds slightly less of pain.

I with #113 merged, I think we can shut down the old repo and rename this one.
Any objections?

OK.  If you are going to do that now, I will wait to make the renaming changes that we discussed in conference today until after that is done.

I will wait to make the renaming changes that we discussed in conference today until after that is done.

I don't think there's a need to wait. GitHub redirects all things. But this can wait until @zorkow is back.

I've deleted the old repo (but still have a local copy if we deed one).
I'm confused by myself. Do we call this repo MathJax-a11y (stated earlier as result of F2F) or MathJax-Accessibility (which I put in the readme)

Ok, I checked the F2F notes and it says a11y.
Done but can obviously be revisited at the F2F.


crud.config.input.option from configuration file

The task was to output data from the config file “config / backend.php” because the expression (
@foreach (['NOT_PUBLISHED', 'PUBLISHED', 'DRAFT'] as $ option)
) is not entirely correct in my case.
Is it possible to write “config ('backend.status_publication')” in the config file of crud?

my suggestion is using settings('status_publication')
as in the panel, there is a system settings, where you can add array in it. but pls do AWARE, its only for single level of array.

@Khludev
alternative you may follow as per image.
create your backend.php in config. then change the settings to config:status_publication
it should be working as well

@Khludev
alternative you may follow as per image.
create your backend.php in config. then change the settings to config:status_publication
it should be working as well

In my case, I need “config ('backend.status_publication')” instead of “settings ('backend.status_publication')”
Since settings ('backend.status_publication') is data from the database. And config ('backend.status_publication') is the data from the config file


How to use best_model_pascal_ResFCN.pth?

HI,
After i use command "
python3 main.py -image_path figures/test.png
-model_path checkpoints/best_model_pascal_ResFCN.pth
-model_name ResFCN"
I get this :

RuntimeError: Error(s) in loading state_dict for ResFCN:
size mismatch for score_32s.bias: copying a param with shape torch.Size([21]) from checkpoint, the shape in current model is torch.Size([2]).
size mismatch for score_32s.weight: copying a param with shape torch.Size([21, 2048, 1, 1]) from checkpoint, the shape in current model is torch.Size([2, 2048, 1, 1]).
size mismatch for score_16s.bias: copying a param with shape torch.Size([21]) from checkpoint, the shape in current model is torch.Size([2]).
size mismatch for score_16s.weight: copying a param with shape torch.Size([21, 1024, 1, 1]) from checkpoint, the shape in current model is torch.Size([2, 1024, 1, 1]).
size mismatch for score_8s.bias: copying a param with shape torch.Size([21]) from checkpoint, the shape in current model is torch.Size([2]).
size mismatch for score_8s.weight: copying a param with shape torch.Size([21, 512, 1, 1]) from checkpoint, the shape in current model is torch.Size([2, 512, 1, 1]).

It seems  best_model_pascal_ResFCN.pth is not match for ResFCN.
Did i miss something? Can the best_model_pascal_ResFCN.pth work well  for detect pedestrian like the picture shown in the README.
Thanks

have you solved it？

i changed：model = model_dictmodel_name.cuda()
in applyonlmage.py

i changed: # model = model_dictmodel_name.cuda()
in applyonimage.py

i changed:#n_classes=21
in applyonimage.py

Oh yah, this happens because there is a mismatch in the number of output classes. Pascal models expect 21 output categories whereas others could expect 2 output categories. Like what GloriaZLQ mentioned, you can modify #n_classes to  solve this.


octokat.js no longer used by latest GitHub Desktop

https://github.com/desktop/desktop/blob/9d5aa6f7f41c3ed6902e5759f41877084fbef65e/app/package.json#L19

so sorry to bring the bad news 😭

Thanks for submitting this! Sorry it took so long to merge, I have been making use of the summer weather and have not spent much time on side-projects.
It was removed in https://github.com/desktop/desktop/pull/2080 but thanks to great feedback about performance issues and providing stronger type-safety from @shiftkey and @niik I started #198 (a Roadmap to simplify this library).
It should address many of the issues brought up which will likely help other octokat users but unfortunately it was not quick enough to keep octokat in GitHub Desktop 😭

@philschatz the team also has a bunch of previous experience with the GitHub API (I help maintain Octokit.net too) so doing it all manually was a bit less daunting than it might be for others. Maybe as we add more GitHub integration we'll feel the pain again.
Keep up the good work!


ability to set stack.yaml

It would be very helpful if there was a way to setup intero to use a non default stack.yaml file, esp on projects that may have more than 1 stack.yaml or the like

Looks like this is already being looked at in #105.

#440 fixed this. It can be closed.


loading netcdf files from cloud object storage

I've implemented this functionality but I still have to test it


Remove Product Filters from All Persistent Files

#2792 removed product filters from pins. This takes the clean‐up further and removes product filters from managed dependencies, workspace state, and the pin testing utilities and fixtures.
To decouple the filters from ManagedDependency (to avoid encoding them), DependencyManifests and loadDependencyManifests(root:diagnostics:) required some refactoring to track the filters directly.
The rest of the changes are just the removal of dead properties and parameters.

Thanks!

@swift-ci please smoke test


The process data for PICL

Hi,
Thanks for the good work. Any plan to share the process data for ICL model training?

Hi, thank you for your attention to our work. The pre-training data is too large to upload to google drive (~50G). We are still thinking about ways to share the data.
You can try processing the data from the plain texts with our scripts.

I have tried using the scripts in the repo and I encountered some issues. For example, in the file retrieval.py, line 358, it seems that there isn't such a file called "processed_0" after running all scripts as in README. Anything I need to modify to make it run successfully?

It seemed to be something wrong when we modified the code to do extra experiments. We have updated the code and tested the whole data process pipeline. You can try this new code.