endpointdowm

I am getting this following error:
Traceback (most recent call last):
File "/Users/liaoming/Documents/Normalization/normalization/test/88.py", line 5, in
palmetto.get_coherence(words, coherence_type="cv")
File "/Users/liaoming/Documents/eone/lib/python2.7/site-packages/palmettopy/palmetto.py", line 104, in get_coherence
return self._get_coherence(words, coherence_type)
File "/Users/liaoming/Documents/eone/lib/python2.7/site-packages/palmettopy/palmetto.py", line 92, in _get_coherence
coherence = self._request_by_service(words, coherence_type)
File "/Users/liaoming/Documents/eone/lib/python2.7/site-packages/palmettopy/palmetto.py", line 31, in _request_by_service
raise EndpointDown(request_uri)
palmettopy.exceptions.EndpointDown
Does it mean the server is down?

@liaoming999 was down, works again now. The service is under heavy load at times and can be unresponsive. Thanks for heads up.


All the recent updates

Codecov Report
Patch coverage: 100.0% and project coverage change: +0.4 :tada:

Comparison is base (99147fc) 77.2% compared to head (7f21d7f) 77.5%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff            @@
##           master    #226     +/-   ##
========================================
+ Coverage    77.2%   77.5%   +0.4%
========================================
  Files           9       9
  Lines         855     855
========================================
+ Hits          660     663      +3
+ Misses        195     192      -3

Impacted Files
Coverage Δ

src/collection/array.rs
89.5% <ø> (ø)

src/collection/confinement.rs
74.8% <100.0%> (ø)

src/collection/flags.rs
88.5% <100.0%> (+0.5%)
:arrow_up:

... and 1 file with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


docs: fix type annotation in jokes tutorial

The useActionData takes a type parameter T and returns something of type T | undefined, so we don't need to specify ActionData | undefined in the type parameter.

Thanks for this, but I've merged another PR that fixed this. Cheers!


Add switch betveen am/pm and 24-hour time modes

People have preferences

Why not use the operating system settings?

@ignaloidas I assume this is for the "desktop" client? Could you point me to some examples of where/when you see it? I'll try poking around when I'm bored next to see what it would take to change.


Add a drill-down for Pods created by a Deployment.

This goes straight from Deployment -> Pods
Another option would be:
Deployment -> ReplicaSet -> Pods
If we'd rather, easy to do, let me know.

Comments addressed, please re-check.

@testforstephen comment addressed, please re-check.
Thanks!


Split V2 App

PLEASE READ
Following a recent back-end update, all contributors with a local clone or fork of our repository are required to run yarn install. This does not apply to direct page edits.

[ ] Yes, I've run yarn install
[ ] No, does not apply to me

Purpose of this pull request
This pull request...

Select the type of change:

[ ] Minor Changes - Typos, formatting, slight revisions
[ ] Update Content - Revisions and updating sections
[ ] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - Updates, maintenance, and new packages for the site, Gatsby, React, etc

The intent of this PR is covered in the PR https://github.com/SumoLogic/sumologic-documentation/pull/3437.


Revendor Microsoft/hcsshim and go-winio

Signed-off-by: John Howard<EMAIL_ADDRESS>Bumps hcsshim to v0.6.14 and go-winio to v0.4.10. Required as a pre-req to re-vendoring these to moby/moby (https://github.com/moby/moby/pull/37674) as containerd pulls in a previous version of go-winio and compilation otherwise fails.
@jterry75 FYI

LGTM

@jhowardmsft - CI is legit. In archive\tar_windows.go it was using these attrs. Need to submit an update there too.

@jterry75 Yup, that's fixed now.

Codecov Report

Merging #2557 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master    #2557   +/-   ##
=======================================
  Coverage   44.59%   44.59%
=======================================
  Files          95       95
  Lines       10007    10007
=======================================
  Hits         4463     4463
  Misses       4822     4822
  Partials      722      722

Flag
Coverage Δ

#linux
48.4% <ø> (ø)
:arrow_up:

#windows
41.54% <0%> (ø)
:arrow_up:

Impacted Files
Coverage Δ

archive/tar_windows.go
8.78% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

LGTM

LGTM

LGTM


Json log messages not received the dos-attack rule msg

Describe the bug
JSON log messages not received the dos-attack rule msg;
the log of modsec_audit work well except for the log of dos-attack that it can intercept but not code the 912-dos-attack to log-messages.
A clear and concise description of what the bug is.
Logs and dumps
Output of:

DebugLogs (level 9)

AuditLogs

modsec_audit.log

{"transaction":{"client_ip":"<IP_ADDRESS>","time_stamp":"Tue Oct  1 19:57:01 2019","server_id":"f40be80333c37672e8c192ce2843a64721701529","client_port":50482,"host_ip":"<IP_ADDRESS>","host_port":2380,"unique_id":"156997422142.938766","request":{"method":"GET","http_version":1.0,"uri":"/","headers":{"Host":"localhost:2380","User-Agent":"ApacheBench/2.3","Accept":"*/*"}},"response":{"http_code":403,"headers":{"Server":"nginx","Date":"Tue, 01 Oct 2019 23:57:01 GMT","Content-Length":"554","Content-Type":"text/html","Connection":"close"}},"producer":{"modsecurity":"ModSecurity v3.0.3 (Linux)","connector":"ModSecurity-nginx v1.0.0","secrules_engine":"Enabled","components":["OWASP_CRS/3.1.0\""]},"messages":[]}}

Error logs

nginx/error.log

2019/10/01 19:58:01 [error] 11#0: *761 [client <IP_ADDRESS>] ModSecurity: Access denied with code 403 (phase 1). Matched "Operator `Eq' with parameter `1' against variable `IP:<IP_ADDRESS>_d9b252bcadfb7e5c254cd1395e8f7c230b1ae3c6::::DOS_BLOCK' (Value: `1' ) [file "/opt/owasp-modsecurity-crs/rules/REQUEST-912-DOS-PROTECTION.conf"] [line "123"] [id "912130"] [rev ""] [msg ""] [data ""] [severity "0"] [ver ""] [maturity "0"] [accuracy "0"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-dos"] [hostname "<IP_ADDRESS>"] [uri "/"] [unique_id "156997428182.004145"] [ref ""], client: <IP_ADDRESS>, server: waf_default, request: "GET / HTTP/1.0", host: "localhost:2380"

If there is a crash, the core dump file.

Notice: Be carefully to not leak any confidential information.
To Reproduce
Steps to reproduce the behavior:
A curl command line that mimics the original request and reproduces the problem. Or a ModSecurity v3 test case.
[e.g: curl "modsec-full/ca/..\..\..\..\..\..\/\etc/\passwd" or issue-394.json]
Expected behavior
A clear and concise description of what you expected to happen.
The json-log run well except for 912* dos rules, what can i do to make the log-msg to modsec-audit.log messages.
Server (please complete the following information):

ModSecurity version (and connector): [e.g. ModSecurity v3.0.1 with nginx-connector v1.0.0]
WebServer: [e.g. nginx-1.15.5]
OS (and distro): [e.g. Linux, archlinux]
centos7 + tengine2.3.2(nginx1.17)
{"modsecurity":"ModSecurity v3.0.3 (Linux)","connector":"ModSecurity-nginx v1.0.0","secrules_engine":"Enabled","components":["OWASP_CRS/3.1.0""]

Rule Set (please complete the following information):

Running any public or commercial rule set? [e.g. SpiderLabs commercial rules]
What is the version number? [e.g. 2018-08-11]

Additional context
Add any other context about the problem here.

Is the matter phaser:5 log will not produced in JSON format Log.

modsec_audit.log

the other rule like 930110 runed will and code the intercepter reason.

{"transaction":{"client_ip":"<IP_ADDRESS>","time_stamp":"Tue Oct  1 20:13:48 2019","server_id":"f40be80333c37672e8c192ce2843a64721701529","client_port":51004,"host_ip":"<IP_ADDRESS>","host_port":2380,"unique_id":"156997522845.201161","request":{"method":"GET","http_version":1.1,"uri":"/?page=../","headers":{"Host":"<IP_ADDRESS>:2380","User-Agent":"Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:68.0) Gecko/20100101 Firefox/68.0","Cookie":"sid=Fe26.2**80d5adba0ce49a0f16ee14321ed57f4701b29c4455925bc3cf3049188d1ee157*i5Mr80O7SMjJbN8HD-4ung*e4MBwcuyR8f6YWlLFC-BxYW2p__3U-JWbtF28tty6DTUBgqOreNqE36mN6Z8ZIOsg954Pro33bOAG2uERb5F2KYt4fLMfzniFdPcIzZ-gvpT1m_4RCwo-gEi3nUzrOnJj7mnVPUYaIEBXxy71_tU1g**54b1b4666e0bf4266c49d5ff750d86cb331404b71f09c8050ea87b9186492384*83eKDEe7_hrOUwRKm8Fic_WLd2w93u2huQEGCq7qOrI","Accept":"text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8","Accept-Language":"en-US,en;q=0.5","Accept-Encoding":"gzip, deflate","Connection":"keep-alive","Upgrade-Insecure-Requests":"1"}},"response":{"http_code":302,"headers":{"Server":"nginx","Date":"Wed, 02 Oct 2019 00:13:48 GMT","Content-Length":"215","Content-Type":"text/html","Access-Control-Allow-Origin":"*","Connection":"keep-alive","Location":"/403_forbidden.html","Access-Control-Allow-Headers":"X-Requested-With"}},"producer":{"modsecurity":"ModSecurity v3.0.3 (Linux)","connector":"ModSecurity-nginx v1.0.0","secrules_engine":"Enabled","components":["OWASP_CRS/3.1.0\""]},"messages":[{"message":"Path Traversal Attack (/../)","details":{"match":"Matched \"Operator `Pm' with parameter `..\\ ../' against variable `REQUEST_URI' (Value: `/?page=../' )","reference":"o7,3v4,10","ruleId":"930110","file":"/opt/owasp-modsecurity-crs/rules/REQUEST-930-APPLICATION-ATTACK-LFI.conf","lineNumber":"52","data":"Matched Data: ../ found within REQUEST_URI: /?page=../","severity":"2","ver":"OWASP_CRS/3.1.0","rev":"","tags":[],"maturity":"0","accuracy":"0"}}]}}

Hello @gmctl
If I've understood you correctly, you're asking why CRS rule 912130 does not result in ModSecurity audit log output.  The short answer is that the rule specifically has 'nolog' as one of its actions.  Note also the comment before the rule:  'Block and track # of requests but don't log'.  If you read the comments at the top of the file there's also a note about the reason ('to prevent a flood of alerts').
If you want to inquire about why CRS rules are structured the way they are, a better forum for that is the CRS project:  https://github.com/SpiderLabs/owasp-modsecurity-crs

@martinhsv  yes, thank you!

@majordaw i have found that the dos-attack log will produced if i don't set SecAuditLogFormat JSON;  but i think there is many difference with JSON and ABFH log. what's the best way to set, can you give some advice.

Hi @xx-zhang,
I haven't played with SecAuditLogFormat option yet. I guess you wanted to ask @martinhsv. ;)


OKX wallet signature 2 times

The code is as follows:
useEffect(() => {
window.scrollTo(0, 0);
if (publicKey && connected) {
sendMessage();
}
}, [publicKey, connected]);
In addition, other wallets are normal and only triggered once, such as: Phantom

Although you can intercept it yourself before calling, I want to know why? Is there a bug?


fix submitter updating

reverts one  error from #4323

Why was this necessary? The old code looks right. If you had the issue, that there were no submitters, please look at line 484. It seems, that it is wrong. Compare to the if-clause of supporters..

Submitters were not updated after editing user; now fixed in #4330


blox-bootstrap: Bilibili videos cannot show

Preliminary Checks

[X] This issue is not a duplicate. Before opening a new issue, please search existing issues: https://github.com/HugoBlox/hugo-blox-builder/issues?q=is%3Aissue
[X] This issue is not a question, feature request, or anything other than a bug report directly related to Hugo Blox Builder. Please post those things on Discord: https://discord.gg/z8wNYzb

Description
In newest documentation, I tried to embed bilibili videos in my blog.
{{< bilibili id="BV1WV4y1r7DF" >}}

But I could only hear the sound without any iframes shown.
In DevTools, I can see the elements:

Reproduction Link
https://hugoblox.com/hugo-themes/
Steps to Reproduce
/
Expected Result
/
Actual Result
/
What Hugo Module versions does your site use?
module github.com/HugoBlox/theme-academic-cv
go 1.15
require (
github.com/HugoBlox/hugo-blox-builder/modules/blox-bootstrap/v5 v5.9.6
github.com/HugoBlox/hugo-blox-builder/modules/blox-plugin-netlify v1.1.2-0.20231108141515-0478cf6921f9
github.com/HugoBlox/hugo-blox-builder/modules/blox-plugin-reveal v1.1.2
)
What operating system(s) are you seeing the problem on?
Windows
What browser(s) are you seeing the problem on?
Microsoft Edge
Which Hugo Blox template are you using?
Academic CV
What version of Hugo are you using?
0.119.0

BiliBili appear to have changed their HTML/CSS so the video no longer displays (hidden by CSS).
This is not an issue in the Tailwind templates.
However, for Bootstrap templates, such as Academic CV, a PR will need to be made by a contributor to update the BiliBili CSS so that the video displays properly again.


Client Credentials method

Hi,
I don't think this library has the ability to do the clients credentials method as outlined here,
https://developer.artik.cloud/documentation/getting-started/authentication.html#client-credentials-method
My code is server side node.js (not client site), so I don't want to have to use oauth and want to use my application tokens.

So far SDKs do not support OAuth flow. Client Credentials is one of OAuth methods.
We are looking to support OAuth in SDKs. However, there is no timeline to share at this point.


[Demangler] Further optimizations for the remangler.

Replace use of snprintf() with some custom code (this is around 10x faster on my machine).
Move a few of the Node member functions to the header so they're inlined.
Optimize the deepEquals() function by adding an isSimilarTo() method on Node; the checks that were happening in the deepEquals() function could be implemented more efficiently by making use of details of the internal representation of Node.
rdar://125739630

@swift-ci Please smoke test

This is a further 23% reduction in cost for the mangling I'm presently testing, over and above what I've already managed.

@swift-ci Please smoke test

Do these need to be cherry-picked to 6.0?


OpenMRG correct details in metadata

Response to #67
opensense_data_downloader_and_transformer: added comment indicating how the file was created. Conmverted frequency to MHz and length to meter. Add attributes after transform, so that they are not removed by the transform. Commented specification of "missing_value" in all attributes.
create_small_openMRG_example: Updated to add information on authors of the stored file and further info on resampling and interpolation of gaps. Removed encoding specifications so that xarray defaults to normal behaviour.
openMRG_use_case: updated to run with new nc file. transform frequency to GHz and length to km to make it work with pycomlink.

I'm not completely sure how to deal with missing values. In the current PR I have just removed any mentions on this, then xarray should default? But then what should be the requested value for data_format_conventions?
Also not that in create_small_openMRG_example I removed most of the encoding specifications. @maxmargraf do you think the specifications should stay there?

Regarding missing values, the white paper states the following: "The “missing _value” attribute is the conventional value of missing data. It is not set here, but the user should define it. The recommendation is to use, e.g., “NA” or “na”."

Also not that in create_small_openMRG_example I removed most of the encoding specifications. @maxmargraf do you think the specifications should stay there?

If the files stays within a certain file limit (I'd say a few MB), I do not think that we need encoding.

Regarding missing values, the white paper states the following: "The “missing _value” attribute is the conventional value of missing data. It is not set here, but the user should define it. The recommendation is to use, e.g., “NA” or “na”."

So it should be specified in the transformer file?

If the files stays within a certain file limit (I'd say a few MB), I do not think that we need encoding.

Its 1,5 MB, so we could drop encoding.

I recall that we had a discussion about that, maybe it was in offline in Krakow when we found that the attribute missing_value led to problems with writing to NetCDF, maybe because the actually required attribute _FillValue was not there.
See the CF conventions on missing data here: http://cfconventions.org/cf-conventions/cf-conventions.html#missing-data

Ok, I think I understand a bit more. Then, since the the opensense_data_downloader_and_transformer does not know what dataformat the user saves to (float, int etc), it should not specify _FillValue, as the fill value would depend on the final netcdf? Instead we could potentially define the _FillValue in the openMRG_use_case?

Anyway, if you save floats to NetCDF the NaNs will just be saved as NaNs, right?
Jepp!


Cant Mint Assets

I have problem at Mint Assets at url: https://fuellabs.github.io/swayswap/welcome/mint
Problem: Network request failed
Connect Wallet - OK
Faucet - OK
Add Assets OK
Mint Assets - OK

Now works.
Regards.


test: Show error when using a binary operator without two operands

When preforming a binary operation with the test command, GNU coreutils gives an error if only one operands exist:
$ /bin/test -o blah
/bin/test: ‘-o’: unary operator expected
$ /bin/test -a blah
/bin/test: ‘-a’: unary operator expected

$ /bin/test blah -a
/bin/test: missing argument after ‘-a’
$ /bin/test blah -o
/bin/test: missing argument after ‘-o’

This PR adds the error messages for the first set, where the first operand is missing. The second operand missing does not appear as trivial, since Symbol::new causes the operator to be treated as a literal as part of the parsing.
With this PR, tests/test/test-diag.pl should pass.
This PR also fixes an integration test that says that -a ! should return an exit code of 1. GNU test gives an error with a status code of 2.

@sargas

The second operand missing does not appear as trivial, since Symbol::new causes the operator to be treated as a literal as part of the parsing.

This should still be done right? Should we open an issue for it in meantime?

@sargas

The second operand missing does not appear as trivial, since Symbol::new causes the operator to be treated as a literal as part of the parsing.

This should still be done right? Should we open an issue for it in meantime?

Agreed, especially since the GNU tests didn't check for this. I can have a go at this tonight

@sargas ping ? :)

@sargas ping ? :)

Sorry, busier than expected, still interested in this PR. Whether <blah> -a shows an error is different for an arbitrary string and for !, so I wanted to create some more test cases

testing with this PR, i am getting:
echo "GNU ="
export C=/bin/
${C}test -o blah
${C}test -a blah
${C}test blah -a
${C}test blah -o
echo "Rust ="
export C="./target/debug/coreutils "
${C}test -o blah
${C}test -a blah
${C}test blah -a
${C}test blah -o

Returns:
GNU =
/bin/test: '-o': unary operator expected
/bin/test: '-a': unary operator expected
/bin/test: missing argument after '-a'
/bin/test: missing argument after '-o'
Rust =
test: '-o': unary operator expected
test: '-a': unary operator expected

@sargas ping ? :)

@sylvestre thanks for the ping. I have a WIP from ages ago that I've just rebased and looked at test failures - there are edge cases (like test ! -a ! or test '(' ')' ')') that aren't quite handled right, but a lot of cases that aren't parsed right in the correct code that this WIP can handle.
My main objective with this PR was to close the gap in uutest and GNU test to fix the test case fixed in #6642, so I'll close this PR and work on trying to close the edge cases in the existing tests/by-utils/test crate so that #6203 can be addressed as well.


Change logging to Akka-logging

This PR changes the logging of OpenWhisk.
Currently we have a trait, that can be extended if you want to log something. The problem is, that this uses println, which locks our processes on heavy load.
Now, one logging-instance is created on startup and passed around. The new logging also uses akka logging.

PG1#1150 and PG1#1152 are green.
PG1#1153 with the latest rebase is running.

One logger to rule them all! This is very nice change - cleans up the mixin mess and all the verbosity toggles.
Check that the runc check isn't lost because of your change.

Also will need to rebase - now has conflicts with invoker changes.

PG1#1155 started.


Processing behind proxy has a problem with references (apparently)

<IP_ADDRESS> - - [08/Mar/2018:08:21:35 +0000] "POST /api/processFulltextDocument HTTP/1.1" 200 12433 "http://<IP_ADDRESS>:8070/" "Mozilla/5.0 (Windows NT 6.1; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/63.0.3239.84 Safari/537.36" 31167
INFO  [2018-03-08 08:21:37,728] org.apache.http.impl.execchain.RetryExec: I/O exception (java.net.NoRouteToHostException) caught when processing request to {s}->https://api.crossref.org:443: No route to host (Host unreachable)
INFO  [2018-03-08 08:21:37,729] org.apache.http.impl.execchain.RetryExec: Retrying request to {s}->https://api.crossref.org:443
INFO  [2018-03-08 08:21:40,727] org.apache.http.impl.execchain.RetryExec: I/O exception (java.net.NoRouteToHostException) caught when processing request to {s}->https://api.crossref.org:443: No route to host (Host unreachable)
INFO  [2018-03-08 08:21:40,727] org.apache.http.impl.execchain.RetryExec: Retrying request to {s}->https://api.crossref.org:443
INFO  [2018-03-08 08:21:43,728] org.apache.http.impl.execchain.RetryExec: I/O exception (java.net.NoRouteToHostException) caught when processing request to {s}->https://api.crossref.org:443: No route to host (Host unreachable)
INFO  [2018-03-08 08:21:43,728] org.apache.http.impl.execchain.RetryExec: Retrying request to {s}->https://api.crossref.org:443
INFO  [2018-03-08 08:21:46,731] org.grobid.core.utilities.Consolidation: ERROR (-1) : java.net.NoRouteToHostException thrown during request execution : works/ (,query.title=My PDF Document,rows=5,query.author=Rampal)

I have set relevant proxy info in gradle.properties, both in ~/.gradle and in project directory:
% cat gradle.properties
#Thu, 21 Apr 2016 18:39:55 +0200
version=0.6.0-SNAPSHOT
# Set workers to 1 that even for parallel builds it works. (I guess the shadow plugin makes some trouble)
org.gradle.workers.max=1

systemProp.http.proxyHost=myhost.com
systemProp.http.proxyPort=8080
systemProp.http.proxyUser=myuser
systemProp.http.proxyPassword=mypass
systemProp.http.nonProxyHosts=*.nonproxyrepos.com|localhos

systemProp.https.proxyHost=myhost.com
systemProp.https.proxyPort=8080
systemProp.https.proxyUser=myuser
systemProp.https.proxyPassword=mypass
systemProp.https.nonProxyHosts=*.nonproxyrepos.com|localhos

I submitted the PDF via web interface, TEI | Process Fulltext Document.
Environment:
 % gradle --version

------------------------------------------------------------
Gradle 4.6
------------------------------------------------------------

Build time:   2018-02-28 13:36:36 UTC
Revision:     8fa6ce7945b640e6168488e4417f9bb96e4ab46c

Groovy:       2.4.12
Ant:          Apache Ant(TM) version 1.9.9 compiled on February 2 2017
JVM:          1.8.0_162 (Oracle Corporation 25.162-b12)
OS:           Linux 2.6.32-696.el6.x86_64 amd64

Thanks for reporting!
It was not tested since a while ;)

So as this is fixed and is working well with my tests, I am closing the issue. Feel free to re-open it if you have any problems.


How many is returning count one higher than actual

When I ask 'How many lights are on' or 'Which lights are on' it returns one more than correct. The reason seems to be that there is a light group for these rooms (created in Zigbee2MQTT).
Example
Which lights are on -> Kitchen Lights and Kitchen Table Lights
How many lights are on -> 2
Kitchen Lights is a group comprising all the lights in the kitchen.
I don't think the count should include a group as clearly the answer should be Kitchen Table Lights and 1

There is currently nothing built in to exclude groups.
In the future you will be able to choose yourself which entities to expose to Assist. So you can then choose to exclude the group. Other people might choose to exclude the individual entities.

OK. Sounds good.


Moving Directories

I moved all my fips stuff from ~/* to ~/floooh/ in the process this broke all my builds. Apparently, fips is fairly brittle and opinionated about its directory layout...

...did you run 'fips clean all' afterwards before attempting to rebuild? Cmake generates tons of absolute paths in its intermediate build files, and there also other places where generated files contain absolute paths. But all those should be wiped and recreated after a 'fips clean all' followed by a 'fips gen'.
When run from a clean slate, fips itself shouldn't depend on any absolute paths, everything is relative to the current project's root direcory.

...did you run 'fips clean all' afterwards before attempting to rebuild?

nope... Thanks for letting me know.

Cmake generates tons of absolute paths in its intermediate build files, and there are also other places where generated files contain absolute paths.

Yet one more reason to hate CMake or anything that depends on it...
Thanks for letting me know. I'll close this one...


vaal skills do not enable regular skill when applying enchant to item

Now that vaal skills provide the base skill, having the vaal skill equipped should enable the base skill to show up in lists like the one in the "enchant item" dialog without toggling the "all skills" option.

So this appears to be working correctly for Vaal Blade Vortex, but not for Vaal Righteous Fire.

But I agree with zmike, each vaal skill should also show its base skill in the dropdown underneath.


C++ No Mission Progress

Latest MAVSDK: 0.50.1
Latest PX4 Simualtor (jMavsim): 1.12.3
Latest QGroundControl: 4.2.0
Trying to get the current mission item by subscribing to the mission progress, we do not get the correct position or updates.  Sometimes on startup the position will be returned at -1, when the sim gets to the first waypoint no update is received.  Attached code is a sample of the app that we are running and demonstrates the issue.  We are using this with no issues in the Python library.
A side note, the mission plan was created in QGC and uploaded to the sim. When the C++ attempts to download the plan, an error is thrown by MAVSDK.  "UNSUPPORTED mission item command (22)"  Not sure if that has something to do with what I am seeing.  What in the plan needs to be changed to remove this error?  QGC was configured to be used against PX4, not ArduPilot.
#include <iostream>
#include <cstdint>
#include <thread>
#include <chrono>
#include <sstream>
#include <iomanip>
#include <chrono>
#include <math.h>
#include <csignal>
#include <string>

#include <future>

#include <mavsdk/mavsdk.h>
#include <mavsdk/plugins/telemetry/telemetry.h>
#include <mavsdk/plugins/mission/mission.h>
#include <mavsdk/plugins/action/action.h>

static void telemetryStatus(std::shared_ptr<mavsdk::System> system, const int port);

void signalHandler(int signum) {
	std::cout << "Interrupt signal (" << signum << ") received" << std::endl;

	std::cout << "---- EXITING ----" << std::endl;
	exit(signum);
}

int main(int argc, char* argv[]) {
	try {
		if(argc < 2 ) {
			std::cerr << "USAGE: orb <connection URL> <MAV ID>" << std::endl;
			return 1;
		}

		// Register the signal handler
		signal(SIGINT, signalHandler);

		mavsdk::Mavsdk mavsdk;

		int argCtr = argc-1;
		int mavid = std::stoi(argv[argCtr]);

		// Read the connection URL parameters
		for(size_t i = 1; i < (argCtr); ++i) {
			std::cout << "Setting up listener on: " << argv[i] << std::endl;
			mavsdk::ConnectionResult connResult = mavsdk.add_any_connection(argv[i]);

			if(connResult != mavsdk::ConnectionResult::Success) {
				std::cerr << "Connection Error: " << connResult << std::endl;
				return -1;
			}
		}

		std::atomic<size_t> numSystemsDiscovered{0};

		std::cout << "Waiting to discovery systems..." << std::endl;
		mavsdk.subscribe_on_new_system([&mavsdk, &numSystemsDiscovered]() {
			const auto systems = mavsdk.systems();

			if(systems.size() > numSystemsDiscovered) {
				std::cout << "System Discovered" << std::endl;
				numSystemsDiscovered = systems.size();
			}
		});

		std::this_thread::sleep_for(std::chrono::seconds(2));

		std::vector<std::shared_ptr<std::thread>> threads;

		for (auto system : mavsdk.systems()) {
			const int systemID = (*system.get()).get_system_id();
			std::cout << "System ID:" << systemID << " MAVID: " << mavid << std::endl;

			//Check to see if this is the aircraft we are supposed to listen for
			if(systemID == mavid) {
				std::cout << "Starting Thread For System ID: " << systemID << std::endl;
				int port = 20200 + mavid;

				threads.push_back(std::make_shared<std::thread>(&telemetryStatus, std::ref(system), port));
				std::this_thread::sleep_for(std::chrono::seconds(1));

				// TODO
				auto mission = mavsdk::Mission{system};
				std::atomic<bool> want_to_pause{false};
				std::cout << "Setting up Mission Listener" << std::endl;

				mission.subscribe_mission_progress([&want_to_pause](mavsdk::Mission::MissionProgress mission_progress) {
					std::cout << "--->Current Mission Item: " << mission_progress.current << std::endl;
				});

                auto action = mavsdk::Action{system};
                std::cout << "Arming..." << std::endl;
                const mavsdk::Action::Result arm_result = action.arm();
                if (arm_result != mavsdk::Action::Result::Success) {
                    std::cerr << "Arming failed: " << arm_result;
                    return 1;
                }
                std::cout  << "Armed." << std::endl;
                std::pair<mavsdk::Mission::Result, mavsdk::Mission::MissionPlan> plan = mission.download_mission();

                mavsdk::Mission::Result startMissionResult = mission.start_mission();
                if(startMissionResult != mavsdk::Mission::Result::Success) {
                    std::cerr << "Unable to start mission!" << std::endl;
                    return 1;
                }
			}
		}

		std::cout << "About to join threads" << std::endl;
		for (auto& t : threads) {
			std::cout << "Joining Threads" << std::endl;
			t->join();
		}

	} catch(const std::exception &ex) {
		std::cerr << "ERROR : " << ex.what() << std::endl;
		return -1;
	}

	return 0;
}

void telemetryStatus(std::shared_ptr<mavsdk::System> system, const int port) {

	try {
		auto telemetry = mavsdk::Telemetry{system};
		const int systemID = (*system.get()).get_system_id();

		std::cout << "Thread started for System ID: " << systemID << std::endl;

		//Check to see if we are looking at an invalid ID
		if (systemID == 255) {
			return;
		}

		// We want to listen to the position of the drone at 1 Hz.
		const mavsdk::Telemetry::Result set_rate_result = telemetry.set_rate_position(1.0);

		if (set_rate_result != mavsdk::Telemetry::Result::Success) {
			std::cerr << "ERROR: Setting rate failed:" << set_rate_result << std::endl;
			//return;
		}

		while(1) {
			try {
				mavsdk::Telemetry::Position position = telemetry.position();
				mavsdk::Telemetry::VelocityNed velocityNed = telemetry.velocity_ned();

				//Check to make sure the location is valid
				if(!isnan(position.latitude_deg) && !isnan(position.longitude_deg)) {
                    //Do WORK
                    std::cout << "Telem: " << position.latitude_deg << ":" << position.longitude_deg << std::endl;
				}

				std::this_thread::sleep_for(std::chrono::milliseconds(1000));
			} catch(const std::exception &ex) {
				std::cerr << "ERROR processing telemetry data: " << ex.what() << std::endl;
			}
		}

	} catch(const std::exception &ex) {
		std::cerr << "ERROR : " << ex.what() << std::endl;
	}

	return;
}

OBE

OBE? What's that?

“Overcome By Events”.  I submitted this just before I received an answer on Slack.  It can really be deleted.


Remove Sample Data

When the download functionality is working remove the provided sample data (mock data)

When the download works also on BTP, remove the following functions:
ValueSetController:

loadDummyData()
loadValuesetFile(String filename, String valueSetName)

CountryListControler

createCountryList( @RequestBody String countryListData)

BusinessRuleController

createRule(
@RequestHeader(value = "X_COUNTRY") String country,
@RequestHeader(value = "X_ID") String id,
@RequestHeader(value = "X_VER") String version,
@RequestBody String ruleData)

From the resources/static folder the valuesets folder has to be removed too.

I moved the methods to new TestController
which is activated on spring profile "testapi"
for example by using ENV
SPRING_PROFILES_ACTIVE=testapi


EOS VISION X REDDIT

Milestone Delivery Checklist

[x] The milestone-delivery-template.md has been copied and updated.
[x] The invoice form has been filled out for this milestone.
[x] This pull request is being made by the same account as the accepted application.
[x] In case of acceptance, the payment will be transferred to the EOS account name provided in the application.
[x] The delivery is according to the Guidelines for Milestone Deliverables.

Link to the application pull request: https://github.com/eosnetworkfoundation/grant-framework/pull/44

The milestone was based on the original milestone, leaving the corresponding links, I await comments, thank you all, happy night!

Thank you very much Mr. Denis, now I'm waiting for the response from @nsjames  , thank you!

Thanks guys, if you have any recommendations or anything else my DMs are open to your ideas and more.
I look forward to news by @tedcahalleos


perf improvement for prepared query

I did some benchmarking on lo against this and, after understanding what the code here is doing, these changes should result in a ~20% improvement in throughput for this simple bench. i tested on 0.4.0 of duckdb shared library and using this command to run:
DENO_DUCKDB_DEV=1 LD_PRELOAD=../duckdb/0.4.0/libduckdb.so nice -n 20 taskset --cpu-list 0 deno run -A --unstable test.js

i see 60k rps for these changes, ~51k rps for HEAD.
Bun with @evan/duckdb is 49k rps for the same bench on my setup. Bun is also ~50 MB more memory usage.
import { open } from "./mod.ts";

const db = open(":memory:");

const connection = db.connect();

connection.query('CREATE TABLE integers(i INTEGER, j INTEGER);');
connection.query('INSERT INTO integers VALUES (3, 4), (5, 6), (7, NULL);');

const prepared = connection.prepare('SELECT * FROM integers;');

const runs = 100000

while (1) {
  const start = Date.now()
  for (let i = 0; i < runs; i++) {
    prepared.query()
  }
  const elapsed = Date.now() - start
  const rate = runs / (elapsed / 1000)
  const ms_iter = elapsed / runs
  const { rss } = Deno.memoryUsage()
  console.log(`runs ${runs} rate ${rate} ms_iter ${ms_iter} rss ${rss}`)
}

connection.close();
db.close();

there is further work to do be done to apply this to the other methods and i need to check to see if this doesn't break something existing, so just submitting as a draft for discussion for now.

Hey @billywhizz Is this ready for review? Sorry for the late reply, I might have missed this in the notifcations

Hey @billywhizz Is this ready for review? Sorry for the late reply, I might have missed this in the notifcations

i'll have a look again. has been a while since i looked at it! :+1:


Refactor templates.addToApos as templates.addHelpers

When we want to add functions to the apos object for use in nunjucks templates, we currently have written a function called addToApos. We realized that these are helper functions, as defined by multiple MVC libraries. This is a good name, and we should use it too.
One thing to consider, is that it doesn't translate as well if you're trying to add an object/property to the apos object. Perhaps leave both of them available?

Done.


DNN-5626 Fixed bug introduce with the previous commit: when the user …

…clicked on an already selected radio button it was unselected

Reviewed and merged


Implement Metrics SDK

Following [#105 and #297], implement Metrics in the SDK package.
~Also, consider to rename packages/opentelemetry-metrics -> packages/opentelemetry-metrics-sdk.~

[x] Add initial skeleton
[x] Add Counter and Gauge support
[x] add registerMetric and getMetrics
[x]  Implement "createMeasure" (v3 milestone)

I'm going to close this issue. I think the basic metrics SDK is implemented including Counter, Measure and Observer support. Do add a comment if you think this is incorrect.


@immutable eroor

I have some mistakes when I want to use @immutable

First up, please read https://codewithandrea.com/articles/beginners-guide-coding-questions/
Secondly, yup there are coding errors. If you click the blue "show" button next to the "1 issue" label in the bottom status bar I suspect you will get a hint as to what the issue is. I believe adding a constructor to your Student class will help...


CommonJS imports are broken in v3.0.0

Describe the bug
My server suddenly stopped working during startup as a result of attempting to import unleash-proxy-client v3.0.0. I believe this is because unleash-proxy-client package defines "type": "module" in its package.json which is not something you should do in NPM packages (this field is for applications to define, not libraries, more info here).
Steps to reproduce the bug
Create a new Node project, attempt to import unleash-proxy-client using require, and start the application. Notice the error, ex:
const unleash_proxy_client = require("unleash-proxy-client");

Expected behavior
Using CommonJS to import the package should not result in a runtime error.
Logs, error output, etc.
Error [ERR_REQUIRE_ESM]: require() of ES Module<EMAIL_ADDRESS>from /home/samuel/WSL_Dev/Personal/dokku_app/src/server/dist/shared/modules/stores/feature_flags_store.js not supported.
index.js is treated as an ES module file as it is a .js file whose nearest parent package.json contains "type": "module" which declares all .js files in that package scope as ES modules.
Instead rename index.js to end in .cjs, change the requiring code to use dynamic import() which is available in all CommonJS modules, or change "type": "module" to "type": "commonjs" in<EMAIL_ADDRESS>to treat all .js files as CommonJS (using .mjs for all ES modules instead).

    at Object.<anonymous> (/home/samuel/WSL_Dev/Personal/dokku_app/src/server/dist/shared/modules/stores/feature_flags_store.js:4:32)
    at Object.<anonymous> (/home/samuel/WSL_Dev/Personal/dokku_app/src/server/dist/server/src/boot_server.js:15:31)
    at /home/samuel/WSL_Dev/Personal/dokku_app/src/server/dist/server/src/main.js:29:43 {
  code: 'ERR_REQUIRE_ESM'
}

Screenshots
No response
Additional context
No response
Unleash version
3.0.0
Subscription type
None
Hosting type
None
SDK information (language and version)
No response

Hey @gamedevsam! 😊 Thanks so much for pointing that out. We've worked on a fix and it's now up in the beta version. Fancy giving it a whirl? Here's the link. Let us know how it goes, and if all's good, we'll get that new version out to everyone. Cheers!

Your fix worked for me, thanks for turning this around so quickly. Feel free to close this issue when you guys push out the release.

Hey there! 😊 Just wanted to let you know that we've rolled out version 3.1.0 as a stable release. You can check it out right here: https://www.npmjs.com/package/unleash-proxy-client/v/3.1.0. Cheers! 🚀


Support size = 0 in PageJacksonModule

The current implementation raises an error when size equals 0. This behavior is incorrect because Page API allows this to happen. So I propose to check first the size of the implementation.

@PedroBeatStars Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@PedroBeatStars Thank you for signing the Contributor License Agreement!

Closing in favour of https://github.com/spring-cloud/spring-cloud-openfeign/pull/636.


Change title of right bar button item

It'd be nice if a string property was exposed that allowed us to change the title of the "Mail" button.  For example, I'd like to change it to "Send".

Hi @rizumita , I've created the CTFeedbackLocalizable file, translated in 3 languages that my app has, but it is not being recognized by the code and the localization doesn't work.
Any suggestions on how to include localization for other languages?

@gmogames

Install CTFeedback by hand into your project and create language resource named 'CTFeedbackLocalizable.strings'.
Or fork this repository and add resources and install by CocoaPods with your forked repository.
Or create a pull request with new resources and I'll merge it.


Previewer

Thanks for this great library.
Where can we download the previewer ?
Regards

https://github.com/googlecreativelab/anypixel/tree/master/frontend/previewer

thanks @Bre77!

Than you Brett
2016-06-03 11:31 GMT+02:00 Brett<EMAIL_ADDRESS>
https://github.com/googlecreativelab/anypixel/tree/master/frontend/previewer
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/googlecreativelab/anypixel/issues/2#issuecomment-223534132,
or mute the thread
https://github.com/notifications/unsubscribe/AC9ezSVayfqlyh0qk-ShEs_nC_XXP793ks5qH_SDgaJpZM4ItR3K
.


network: fix networkx deprecated function usage

As of 2.6, yields a deprecation warning: https://github.com/networkx/networkx/pull/4282
https://github.com/networkx/networkx/blob/main/doc/release/migration_guide_from_2.x_to_3.0.rst

Thanks! :100:


Update README.md

Remove Viewing Cage from your world
Q, Are there any specification restrictions between single-player and dedicated servers?
A, Nothing, but only the [Viewing Cage] is not present on the dedicated server. This will be support in a future update.

https://tech.palworldgame.com/dedicated-server-guide#qa

Thanks for bringing this to my attention! If there are any other bugs using co-op saves with dedicated servers, please let me know!


Update token setting in .travis.yml

The original toke is revoked. So need a new effective token.

replace it with my token.

Thanks.


vendor: fix case of sirupsen/logrus

Docker build fails due to capitalized vendor repo
Fixes #114

This was fixed in #113.
Thanks so much for the effort doing this.

Looks like this is still an issue, the Sirupsen folder checked in to vendor/ is still uppercase: https://github.com/coreos/jwtproxy/tree/master/vendor/github.com/Sirupsen/logrus


[Tunnel] Clarify config file location for running as a service in linux

Hi, thanks for the feedback. This entire page is a bit confusing and due for an overhaul. The cloudflared service install command should not change the config file — refer to these updated instructions for the recommended setup procedure.


Deb Linux doesn't detect running tunneler

After punching the I/O button, prompt for root, then the button is illuminated green and mission control animation appears

After a reboot the UI did detect the running tunneler.

While running 3.0.2 on Debian Linux today the app doesn't detect the already-running tunneler. Unlike my prior report pressing the I/O button doesn't cause it to detect the running tunneler.

After restarting the tunneler daemon the app now detects and connects to the running process.


Bulk actions on access control

Users have requested the ability to easily change access control settings for multiple resources at once.  Need to evaluate interface enhancements to make this efficient.

@dtarb - this was part of a legacy issue that got backburnered. See #771. Also there has been a request for bulk download #1416.


[request] set approach

Hi
Will we ever have the possibility to make sets of gear, like outfitter/itemrack do, for the sake of having only one addon for taking care of all our gear needs ?
But as it's a very different thing of managing usable pieces of gear, i'll perfectly understand if your answer is "no".

Hi,
while I like the addon outfitter the answer is indeed no :( sorry. I'm not trying to build an outfitter addon. The goal of gearmenu is to be a fast itemswitch addon for combat situations instead of switching complete sets of gear.
I also understand that you don't like having multiple addons doing similar things. However I prefer to keep addons small and manageable in regards of development.
Thank you for understanding
Regards,
RagedUnicorn

The goal of gearmenu is to be a fast itemswitch addon for combat situations instead of switching complete sets of gear.

Would you consider adding some more item slots if that's the intention? For some classes in certain situations there is still a need for quickly switching 2 item slots even though one of the slots might not have any active items.
For example as a druid with only the blue pvp set I often find myself swapping either boots and leg slots to maintain the 4p set bonus.
No addon support for legs is the main thing keeping me from replacing itemrack with gearmenu and I think its possible other classes might have the same issue, but perhaps with other slots.

I remember a cape that have a slow fall effect. A rogue quest, iirc.

Hi,
I certainly do. My initial goal was to support slots that have items that are usable but then afterwards you usually want to switch back to another item. I'm still missing some slots the cape is a good example that I would want myself for the slow fall cape.
https://classicdb.ch/?item=10606
I will certainly consider other slots as well. I guess in the end why not support all of the slots :) I should have some time during november and will see if I can start adding more supported slots for the next version
Regards,
RagedUnicorn

That sounds great! I'll be looking forward to finally replace itemrack for good.
Also the legs slot does actually have 1 active item https://classicdb.ch/?item=14554

Hi @Wushengg and @Altoholic just a quick heads up. I added support for more slots.
These are the supported slots as of now:

HeadSlot
Head/Helmet slot

NeckSlot
Neck slot

ShoulderSlot
Shoulder slot

ChestSlot
Chest/Robe slot

WaistSlot
Waist/Belt slot

LegsSlot
Legs slot

FeetSlot
Feet/Boots slot

WristSlot
Wrist/Bracers slot

HandsSlot
Hands slot

Finger0Slot
First/Upper ring slot

Finger1Slot
Second/Upper ring slot

Trinket0Slot
First/Upper trinket slot

Trinket1Slot
Second/Lower trinket slot

BackSlot
Back/Cloack slot slot

MainhandSlot
Main-hand slot

SecondaryHandSlot
Secondary-hand/Off-hand slot

It is also possible to have 10 active slots at the same time (previously 7). I did some testing and so far it looks good. If you have some time for testing as well I would be glad :)
Regards,
RagedUnicorn

Seems to work fine and not generate any errors. Just 2 minor things.

Currently its only possible to keybind 7 slots and not all 10.
The chat window in getting spammed by purple and green chat messages. I guess its intentional since it's not a release version, but thought I should mention anyway.

Hi @Wushengg
Thanks for your efforts.

I indeed forgot about this. Fixed in 9a1b316702fd1726575eddc7d982aef89690a931
This is indeed not the case in the release version. Those messages are for debugging during development.

I will test a bit more myself and then release 1.2.0 in the next days.


Including future events information for prediction

Is there a way to include data for future events into the model?
For example, if I am using the model to make stock price predictions one month down the line, can I input future known data such as Earnings announcement date, dividend announcement dates for a company into the model?

Using TimeSeriesForecastingPipeline (like in #53), there is field for future_time_series (See here. However, I'm not if I'm using it right, as I'm getting duplicate dates when I specify future_time_series:

For my use case, all exogenous variables are known in the past and in the future, and I'm predicting a single univariate time series. Exogenous variables are already declared as observable_columns in TimeSeriesPreprocessor. Does specifying future_time_series do anything in this instance?
Code:
from tsfm_public.toolkit.time_series_forecasting_pipeline import TimeSeriesForecastingPipeline

forecast_pipeline = TimeSeriesForecastingPipeline(
    model=zeroshot_model,
    timestamp_column=timestamp_column,
    id_columns=id_columns,
    target_columns=target_columns,
    observable_columns=control_columns,
    freq="30m",
    feature_extractor=tsp,
    #explode_forecasts=False,
    preprocessor=tsp,
    prediction_length=48,
    inverse_scale_outputs=True,
    future_time_series=t
)

forecasts = forecast_pipeline(tsp.preprocess(t))
forecasts.head()

@HarrisonOates The future_time_series argument is used to specify the observed values of exogenous features in the future, i.e., continuing past the end of the time series provided when you call forecast_pipeline. The underlying class from HF transformers.pipelines.base.Pipeline supports passing this argument both during construction and during the call method.
The future values of the exogenous will only be useful toward the end of the input time series (t in your example), since this is where windows will have values extending into the future.
I am puzzled by the duplication -- it would be helpful if you could share some example that reproduces it.


Listening to multiple teams at once?

I use multiple teams for 1 project/repo for things like Frontend/Backend, I'd be nice if I could listen to all of the teams in my 1 Linear organization with only 1 linear-to-github API running.

oh yeah that'll have to wait for a later release i guess

once the basics are up and running

@mmattbtw this should now be possible. Bugs are likely, curious to hear your feedback.

@tedspare i'm trying to set it up (through synclinear.com) however I'm not seeing my repo: mmattDonk/AI-TTS-Donations even though I have granted the application access.

Thanks for trying @mmattbtw. Are any of your (mmattDonk/*) repos showing up?
The app only lists the first 100 repos for now. If that's the problem, I'll add a Show more button.

Thanks for trying @mmattbtw. Are any of your mmattDonk/* repos showing up?
The app only lists the first 100 repos for now. If that's the problem, I'll add a Show more button.

yeah that's probably it, i have a lot of repos on my personal account

@mmattbtw does it work for you now? Added a patch.

@tedspare i could find it, and link it successfully, however I am getting this when opening an issue

@mmattbtw nice, thanks for bearing with me
It looks like part of the sync wasn't saved. I've pushed a fix - could you try syncing again?

still not working for some reason, any thoughts?
im trying to sync my "Solrock Frontend" linear team to mmattDonk/AI-TTS-Donations if that helps you any :p

hey @tedspare
I am trying to test this feature but I can't get to add another sync to work it does not let me add a new one.

I have Learn Anything synced. Although not as organisation, can't find how to sync org too.

I can't add KusKus to sync too with SyncLinear. Also I can't get for orgs to show for github, it just does not show them. 😕


MINOR: Simplify log cleaner and fix compiler warnings

Simplify LogCleaner.cleanSegments and add comment regarding thread
unsafe usage of LogSegment.append. This was a result of investigating
KAFKA-4972.
Fix compiler warnings.

@rajinisivaram, maybe you can review this.

retest this please


Add new Draftail inline toolbar and command palette

Follow-up to #8805, adding the new inline toolbar and the command palette, updating to the latest Draftail alpha release with one known issue fixed and a few improvements needed for upcoming block splitting features.
I had initially pushed a draft version of this PR with more changes related to the block splitting – which I’ve removed, so we only merge things that are (more or less) ready to go as-is. The one exception is styling of the editor toolbars – there will be more to do once this is merged.
Top toolbar
This toolbar will eventually go away. I’ve kept it for now as this is where the block splitting lives – it’ll move to the block toolbar & command palette once we’ve done more work on splitting options there.
Inline toolbar
The inline toolbar shows when selecting text. It contains block-level text formats, and inline styles & entities (link, document).

Meta toolbar
No changes there for now.
Command palette
This shows when starting a block’s text with /. It’s filterable by typing further, and options can be selected with the arrow keys up and down. It can be dismissed with the Esc key, or by typing 2 or more spaces on the line starting with /.

Please check the following:

[x] Do the tests still pass?[^1]
[x] Does the code comply with the style guide?
~[ ] For Python changes: Have you added tests to cover the new/fixed behaviour?~
[ ] For front-end changes: Did you test on all of Wagtail’s supported environments?[^2]

[x] Please list the exact browser and operating system versions you tested: Chrome 104, Firefox 102, Safari 15.1 on macOS 12.4
[x] Please list which assistive technologies [^3] you tested: VoiceOver

~[ ] For new features: Has the documentation been updated accordingly?~ This is all user-facing and will work with existing dev APIs without any changes.

Both fixed! Done in the editor right away as a new alpha release.


Remove reference to $bodyValues

Cherry-picked from https://github.com/PrestaShopCorp/ps_checkout/pull/810

The installation I have been working with (PS version <IP_ADDRESS>) $bodyValues is always null, causing no phone number to be submitted which is tripping up address validation and causing the user to have to edit the created address.
I had initially proposed this change on the master branch and have now closed that merge request to re-add it on dev branch as per readme (so no duplication)

Thanks Matt


repo-stream update

Opened by mondeja/repo-stream-config/upstream.yaml

Pull Request Test Coverage Report for Build 981378956

0 of 0   changed or added relevant lines in 0 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.4%) to 88.3%

Files with Coverage Reduction
New Missed Lines
%

mkdocs_mdpo_plugin/plugin.py
2
83.67%

Totals

Change from base Build 940397656:
-0.4%

Covered Lines:
400

Relevant Lines:
453

💛  - Coveralls


Pass a UniqueKey as key to your widget, this should force the lifecycle to start over

          Pass a ```UniqueKey``` as ```key``` to your widget, this should force the lifecycle to start over

Originally posted by @julianbraasch in https://github.com/flutter/flutter/issues/48110#issuecomment-570496275
I'm having the same issue. I have tried to pass unique key to widget but it still didn't work. any solution ?

test


Stats are incorrect

Alot of the time, the stats listed on faceitperf (both the overall ratings, and the ADR, KAST, etc) are just bogus and completely incorrect. Try to have it parse correct statistics?

Alot of the time, the stats listed on faceitperf (both the overall ratings, and the ADR, KAST, etc) are just bogus and completely incorrect. Try to have it parse correct statistics?

If you look really closely on the website, you'll see that there's a disclaimer above those "metrics" stating that they are not exact figures, rather approximate estimation. These stats are not provided by FACEIT API.


Hello rails back-end

Changes

Add a Postgres database set up.
Create a table for storing your greetings.
Create an API endpoint that selects a random greeting from the table

React front-end PR

Hi @mike2611 🙋♀,
STATUS APPROVED ✔
Your project is complete!  There is nothing else to say other than... it's time to merge it 🔗 🚢
Congratulations! 🎉
Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers 🥂  and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Support for other PFRL Algorithms

Is it possible to change the code to use other PFRL Algorithms such as Reinforce? Does this require major changes in multiple files?

Thank you for your input. I have previously experimented with different algorithms in PFRL. Based on my understanding, modifying the actor should indeed achieve the desired outcome. I appreciate your suggestion and encourage you to submit a pull request~

Sure, I am working on something. If I can create a plugin for different RL algorithms, I'll create a pull request.


custom Log Format

I cant use '=' while defining my custom log format in istio

Please direct Istio related issues to Istio forums.


Changes in registration

All cadre feedbacks:

"Cadre Name" & "Ministry Name" should be matched, otherwise after submiting info it will show message "You are not eligibible for registration.".

Registration Email subject is not correct. Told Mohit last day. Issue 16 not implemented. User should not get gov.Id before approval. remove it from email.

In registration pending => view, there should have "Approve" button. remove "Update" button.

In registration pending, there should have "Edit" icon. In "Edit" mode, there should have "Update" button.

In both Edit and View mode, user can download/view attachment.

All requirements are successfully achieved.


(docs): update help option to new format

I signed it!

Dupe of #3159. Please search through PRs before submitting a new one.

Closing as a dupe, but thank you for taking the time to make the PR.


Issue- "BadLayout: Invalid layout of the ARFF file, at line ..."

Hello,
I tried to convert regular arff file format to sparse arff format through liac-arff.
I used the code below:
import arff
fp = open('est1.arff')
data = arff.load(fp)
X = arff.dumps(data)
from scipy import sparse
decoder = arff.ArffDecoder()
d = decoder.decode(X, encode_nominal=True, return_type=arff.COO)
data = d['data'][0]
row = d['data'][1]
col = d['data'][2]
matrix = sparse.coo_matrix((data, (row, col)), shape=(max(row)+1, max(col)+1))
However, it did not work, and the comment in console was "BadLayout: Invalid layout of the ARFF file, at line ..."
I am not sure what the problem is. Would you please help me to solve this issue?
Thank you.

I think you should have got a generator only if you pass in a return_type
parameter.

Thank you for your comment. Where can I get a generator?

Sorry, I was responding to the first version of this issue which complained that the output of loading was a Generator.
If your data is not a sparse ARFF, attempting COO return type will raise a BadLayout. Instead, load it dense, then convert to sparse.


How to get Running Activity in splits of distance

Hi
I need to get running only , but result is include walking and running.
//HKSampleType.quantityType(forIdentifier: .distanceWalkingRunning) <---- include walking and running
Thank you

Hey, currently it's not supported.

@Matsuo32
Anyone knows how to get the running/walking/still in Android? Not necessarily using this library.


IOS release size is too large after upgrading Flutter to 1.12.13+hotfix.7

My iOS release size is unexpectedly too large after upgrading Flutter to 1.12.13+hotfix.7. And Itanagar more than 185+ Mb, this never happened before. I was used to work on lower version and it was about around 40 Mb. Can anybody help me out?  @filiph

sure @filiph ,
i was used to work on Flutter 1.9.1+hotfix.6  before upgrading ,
and this is is an output of flutter doctor.
[✓] Flutter (Channel stable, v1.12.13+hotfix.7, on Mac OS X 10.14.6 18G103, locale en-IN)
• Flutter version 1.12.13+hotfix.7 at /Users/tristate/Flutter/flutter
• Framework revision 9f5ff2306b (5 days ago), 2020-01-26 22:38:26 -0800
• Engine revision a67792536c
• Dart version 2.7.0
[!] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
• Android SDK at /Users/tristate/Library/Android/sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-29, build-tools 29.0.2
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)
! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses
[✓] Xcode - develop for iOS and macOS (Xcode 10.2)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 10.2, Build version 10E125
• CocoaPods version 1.7.5
[✓] Android Studio (version 3.4)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin version 38.2.1
• Dart plugin version 183.6270
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)
[!] VS Code (version 1.37.0)
• VS Code at /Applications/Visual Studio Code.app/Contents
✗ Flutter extension not installed; install from
https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter
[✓] Connected device (3 available)
• Redmi 4                   • f725aa8b7ce4                         • android-arm64 • Android 7.1.2 (API 25)
• Android SDK built for x86 • emulator-5554                        • android-x86   • Android 9 (API 28) (emulator)
• iPhone 8                  • 38B12C2F-B971-4943-AED6-90EC4BA57F22 • ios           • com.apple.CoreSimulator.SimRuntime.iOS-12-2 (simulator)

See #47101 (comment)
TLDR: Your app is not actually that big

yes, but as suggested in the comment i created distribution app and that too was too large,
this did never happen and if it is like so than even a simple "Hello world" app would have around 40+ mbs that is not at all good.

Just to be clear: you tried uploading the file ("Archive") to App Store Connect and it was declined for being too large?
As explained in the other thread, the increased file size is a natural consequence of having bit code in the intermediate file that you are sending to Apple. It is not indicative of what the user will download.

Just to be clear: you tried uploading the file ("Archive") to App Store Connect and it was declined for being too large?
As explained in the other thread, the increased file size is a natural consequence of having bit code in the intermediate file that you are sending to Apple. It is not indicative of what the user will download.

I understand, but during the development of an application, we need to provide a build to client multiple times and he is rejecting this much of size of an IPA and questioning us why it was not happening before.
may I know if this will be resolved in the upcoming version so we can explain it to them?

Oh, I understand. Thanks for the additional context.
This is not going away, and it's not specific to Flutter. Using LLVM IR will get you these sizes whether or not you use Flutter. As explained in the linked FAQ:

As of Flutter SDK 1.12, the release engine binary now includes LLVM IR (bitcode). Xcode uses this bitcode to produce a final binary for the App Store containing the latest compiler optimizations and features.

For sending stuff to client, you might want to opt into sending profile builds. You might also want to experiment with https://flutter.dev/docs/perf/app-size#ios.
I'll close this issue. Feel free to reopen if you think this is in error.

https://flutter.dev/docs/perf/app-size#ios

if I don't release my app to App Store, how to reduce bitcode part?

https://flutter.dev/docs/perf/app-size#ios

if I don't release my app to App Store, how to reduce bitcode part?

Hii, as Filiph suggested, I created the profile app using
flutter build ios —profile,
And it works fine,
So during Development, use this command to build an ipa.

created the profile app using

how to created the profile app using


Information panels hidden by right flyout

At the moment if you press ctrl + shift + B to show the background panel, then press B to get the background list - the background panel is hidden behind the flyout. This is awkward if you want to switch between multiple backgrounds to see which has a newer date.
How about, either: offset the panels to the left when the background/map data is open OR align all the information panels to the left, so they don't interfere with the right flyout at all?

I did this! It was harder than expected since the panes were absolutely positioned. Unfortunately I had to remove the slide animation when switching between different panes but I don't think this is a major loss.

looks great!  👏

Nice, thanks.


logsink: fix multiple issues with LogSink::ToString()

Initializing std::ostringstream with a string makes no sense, as the   string becomes an initial value of an underlying buffer; seek-to-end   is not performed, so the initial value gets completely overwritten by   subsequent writing.

Flag log_year_in_prefix should be considered, as if formatting a   regular logging message.

Writing a buffer to std::ostream is better expressed with stream.write(s,n).

Codecov Report

Merging #852 (1695f91) into master (278ed96) will increase coverage by 3.77%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #852      +/-   ##
==========================================
+ Coverage   73.17%   76.95%   +3.77%
==========================================
  Files          17       13       -4
  Lines        3277     2525     -752
==========================================
- Hits         2398     1943     -455
+ Misses        879      582     -297

Impacted Files
Coverage Δ

src/logging.cc
76.06% <100.00%> (+1.38%)
:arrow_up:

src/base/mutex.h
100.00% <0.00%> (ø)

src/raw_logging.cc
100.00% <0.00%> (ø)

src/windows/port.h

src/windows/dirent.h

src/stacktrace_unwind-inl.h

src/windows/port.cc

src/demangle.cc
97.50% <0.00%> (+0.02%)
:arrow_up:

src/signalhandler.cc
6.76% <0.00%> (+0.31%)
:arrow_up:

src/glog/logging.h.in
80.00% <0.00%> (+0.54%)
:arrow_up:

... and 4 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Prometheus adapters for Vitess stats var types

This is the beginning of support for integration with Prometheus. It still remains to select which vars will be exported to Prometheus and determine what their names and help text should be. Then they can be registered with Prometheus by calling promstats.NewCollector to create collectors and calling prometheus.Register on each collector to register them with the Prometheus registry.

This change is

My original goal was to have everything we already publish to expvar automatically published to Prometheus as well. Is that not a goal of your integration work? If not, I can work on that separately after you're done.
Previously, mattharden (Matt Harden) wrote…
> #### \[WIP\] Prometheus Integration
>
> This is the beginning of support for integration with Prometheus. It still remains to select which vars will be exported to Prometheus and determine what their names and help text should be. Then they can be registered with Prometheus by calling promstats.NewCollector to create collectors and calling prometheus.Register on each collector to register them with the Prometheus registry.
>
>
>

Review status: 0 of 2 files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

go/promstats/collector_test.go, line 10 [r1] (raw file):

	"github.com/youtube/vitess/go/stats"

	pb "github.com/prometheus/client_model/go"

For these new Go dependencies, the way to update our vendor config is like this:
govendor fetch github.com/prometheus/client_golang/prometheus
govendor fetch github.com/prometheus/client_model/go
git add vendor/vendor.json

Comments from Reviewable

This is the initial PR. I was going to actually add vars in a later PR. I'm not sure whether the existing information is sufficient to devise a correct name for Prometheus; see https://prometheus.io/docs/practices/naming/.
Previously, enisoc (Anthony Yeh) wrote…
> My original goal was to have everything we already publish to `expvar` automatically published to Prometheus as well. Is that not a goal of your integration work? If not, I can work on that separately after you're done.

Review status: 0 of 2 files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

Comments from Reviewable

Ah I see. I'll review what's there so far then.
For the long term, I'm thinking we may need to introduce our own interface, for which we implement wrappers for both Prometheus and expvar. We'd then replace all uses of expvar with the new interface. If we need extra info for the Prometheus implementation, we can inject that into each call site as we convert it. The expvar implementation will just ignore the extra info. This would mirror the way we wrap other pluggable things like ZK vs etcd.
If you were thinking of something else, let me know and we can talk more about it.
Previously, mattharden (Matt Harden) wrote…
> This is the initial PR. I was going to actually add vars in a later PR. I'm not sure whether the existing information is sufficient to devise a correct name for Prometheus; see https://prometheus.io/docs/practices/naming/.

Review status: 0 of 2 files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

Comments from Reviewable

Looks good overall. Just some style nits in the tests.
Previously, enisoc (Anthony Yeh) wrote…
> Ah I see. I'll review what's there so far then.
>
> For the long term, I'm thinking we may need to introduce our own interface, for which we implement wrappers for both Prometheus and expvar. We'd then replace all uses of expvar with the new interface. If we need extra info for the Prometheus implementation, we can inject that into each call site as we convert it. The expvar implementation will just ignore the extra info. This would mirror the way we wrap other pluggable things like ZK vs etcd.
>
> If you were thinking of something else, let me know and we can talk more about it.

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, 5 unresolved discussions, some commit checks failed.

go/promstats/collector_test.go, line 21 [r1] (raw file):

	v.Set(1234)
	metrics := make(chan prometheus.Metric, 1)
	go c.Collect(metrics)

Since you made a buffered channel, you don't need to launch a goroutine here. Since you're immediately blocking on the result below, the behavior of the test will be the same either way if the c.Collect() call blocks.

go/promstats/collector_test.go, line 23 [r1] (raw file):

	go c.Collect(metrics)
	met := <-metrics
	if want, got := `Desc{fqName: "test_name", help: "test_help", constLabels: {}, variableLabels: []}`, met.Desc().String(); want != got {

By convention, the style guide says to use the order got, want and got != want in Go tests.

go/promstats/collector_test.go, line 24 [r1] (raw file):

	met := <-metrics
	if want, got := `Desc{fqName: "test_name", help: "test_help", constLabels: {}, variableLabels: []}`, met.Desc().String(); want != got {
		t.Errorf("want %q, got %q", want, got)

By convention, got/want errors look like this:
t.Errorf("met.Desc() = %q, want %q", got, want)

go/promstats/collector_test.go, line 92 [r1] (raw file):

func TestFloat(t *testing.T) {
	v := stats.NewFloat("Floaty McFloatface")

lol

Comments from Reviewable

Review status: all files reviewed at latest revision, 6 unresolved discussions, some commit checks failed.

go/promstats/collector.go, line 1 [r1] (raw file):

package promstats

Please add a brief package comment:
/*
Package promstats contains adapters for ...
*/
package promstats

Also, since this is specific to the vitess/go/stats package, I recommend placing it as a subpackage thereof: vitess/go/stats/promstats. This makes browsing the code directories nicer.

Comments from Reviewable

I'm thinking about modeling the way go-kit/metrics is doing it. See https://github.com/go-kit/kit/tree/master/metrics. They create a few interfaces that represent metric types (Gauge, Counter, Histogram) and then provide helper functions that create each of these types for the supported backends. So the code that collects metrics can be agnostic as to expvar/stats vs. Prometheus. When creating the metric instances you would need to call either the expvar/stats constructor or the Prometheus constructor with the right information - name, help, description, type, unit, field names, whatever. In fact we could consider just using go-kit/metrics for this, which would get us support for statsd for free.
Previously, enisoc (Anthony Yeh) wrote…
> Looks good overall. Just some style nits in the tests.

Review status: all files reviewed at latest revision, 6 unresolved discussions, some commit checks failed.

go/promstats/collector_test.go, line 21 [r1] (raw file):
Previously, enisoc (Anthony Yeh) wrote…
> Since you made a buffered channel, you don't need to launch a goroutine here. Since you're immediately blocking on the result below, the behavior of the test will be the same either way if the `c.Collect()` call blocks.
Technically we only block until the first metric is sent on the channel. These collectors *should* only send one metric, but if they send more than one, the test would hang rather than just have a leak. Maybe I should pull this out into a function to run a single collection and pass back the metrics in a slice.

go/promstats/collector_test.go, line 92 [r1] (raw file):
Previously, enisoc (Anthony Yeh) wrote…
> lol
Yeah, so there's a bug in stats.NewFloat; it always publishes, even if the name is empty. At least one other constructor does that as well. So to avoid that bug I gave this metric a name (and decided to be cute in the process). I should fix the bug thogh.

Comments from Reviewable

PTAL
Previously, mattharden (Matt Harden) wrote…
> I'm thinking about modeling the way go-kit/metrics is doing it. See https://github.com/go-kit/kit/tree/master/metrics. They create a few interfaces that represent metric types (Gauge, Counter, Histogram) and then provide helper functions that create each of these types for the supported backends. So the code that collects metrics can be agnostic as to expvar/stats vs. Prometheus. When creating the metric instances you would need to call either the expvar/stats constructor or the Prometheus constructor with the right information - name, help, description, type, unit, field names, whatever. In fact we could consider just using go-kit/metrics for this, which would get us support for statsd for free.

Review status: 0 of 2 files reviewed at latest revision, 5 unresolved discussions.

go/promstats/collector_test.go, line 10 [r1] (raw file):
Previously, enisoc (Anthony Yeh) wrote…
> For these new Go dependencies, the way to [update our vendor config](https://github.com/youtube/vitess/blob/master/bootstrap.sh#L131) is like this:
>
> ```sh
> govendor fetch github.com/prometheus/client_golang/prometheus
> govendor fetch github.com/prometheus/client_model/go
> git add vendor/vendor.json
> ```

Do we really want to vendor these dependencies? The transitive closure of the dependencies isn't that small:
github.com/beorn7/perks/quantile
github.com/golang/protobuf/proto
github.com/matttproud/golang_protobuf_extensions/pbutil
github.com/prometheus/client_model/go
github.com/prometheus/common/expfmt
github.com/prometheus/common/internal/bitbucket.org/ww/goautoneg
github.com/prometheus/common/model
github.com/prometheus/procfs
I see two options to avoid that. We could make the prometheus adapter a separate plug-in with its own vendored or non-vendored dependencies, or we could avoid the prometheus client altogether, and directly implement the RPC model, so that we're only dependent on the protobuf definitions in github.com/prometheus/client_model.

go/promstats/collector_test.go, line 21 [r1] (raw file):
Previously, mattharden (Matt Harden) wrote…
> Technically we only block until the first metric is sent on the channel. These collectors *should* only send one metric, but if they send more than one, the test would hang rather than just have a leak. Maybe I should pull this out into a function to run a single collection and pass back the metrics in a slice.

Done.

go/promstats/collector_test.go, line 23 [r1] (raw file):
Previously, enisoc (Anthony Yeh) wrote…
> By convention, the style guide says to use the order `got, want` and `got != want` in Go tests.

Done.

go/promstats/collector_test.go, line 24 [r1] (raw file):
Previously, enisoc (Anthony Yeh) wrote…
> By convention, got/want errors look like this:
>
> ``` go
> t.Errorf("met.Desc() = %q, want %q", got, want)
> ```

Done.

go/promstats/collector.go, line 1 [r1] (raw file):
Previously, enisoc (Anthony Yeh) wrote…
> Please add a brief package comment:
>
> ``` go
> /*
> Package promstats contains adapters for ...
> */
> package promstats
> ```
>
> Also, since this is specific to the `vitess/go/stats` package, I recommend placing it as a subpackage thereof: `vitess/go/stats/promstats`. This makes browsing the code directories nicer.

Done.

Comments from Reviewable

Reviewed 4 of 4 files at r2.
Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

go/promstats/collector_test.go, line 10 [r1] (raw file):
Previously, mattharden (Matt Harden) wrote…
> Do we really want to vendor these dependencies? The transitive closure of the dependencies isn't that small:
>
> github.com/beorn7/perks/quantile
> github.com/golang/protobuf/proto
> github.com/matttproud/golang_protobuf_extensions/pbutil
> github.com/prometheus/client_model/go
> github.com/prometheus/common/expfmt
> github.com/prometheus/common/internal/bitbucket.org/ww/goautoneg
> github.com/prometheus/common/model
> github.com/prometheus/procfs
>
> I see two options to avoid that. We could make the prometheus adapter a separate plug-in with its own vendored or non-vendored dependencies, or we could avoid the prometheus client altogether, and directly implement the RPC model, so that we're only dependent on the protobuf definitions in github.com/prometheus/client_model.

Now that we use govendor, I think it's acceptable to add those dependencies. Note that it's not vendoring in the sense of adding the actual files to our repository; we're just adding an entry to vendor.json that tells the end user's govendor binary which libraries to download.

Comments from Reviewable

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

go/promstats/collector_test.go, line 10 [r1] (raw file):
Previously, enisoc (Anthony Yeh) wrote…
> Now that we use govendor, I think it's acceptable to add those dependencies. Note that it's not vendoring in the sense of adding the actual files to our repository; we're just adding an entry to vendor.json that tells the end user's govendor binary which libraries to download.

Done.

Comments from Reviewable

Reviewed 1 of 1 files at r3.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

I'm building the new docker bootstrap images now.


Ads in Listview.builder are reloading while scrolling the list up and down

Ads  in Listview.builder are reloading while scrolling the list up and down.  I used ads() method in a listview.builder with a controller.
Sample code
 final _nativeAdMobController = NativeAdmobController();
 StreamSubscription _nativeAdMobSubscription;
 double _height = 0;

@override
  void initState() {
   _nativeAdMobSubscription = _nativeAdMobController.stateChanged.listen(_onStateChanged);
    super.initState();
  }

@override
  void dispose() {
    _nativeAdMobSubscription.cancel();
   _nativeAdMobController.dispose();
    super.dispose();
  }

ads () {
    return NativeAdmob(
      adUnitID: getBannerAdUnitId(),
      loading: Container(),
     controller: _nativeAdMobController,
      type: NativeAdmobType.banner,
      options: NativeAdmobOptions(
          adLabelTextStyle: NativeTextStyle(isVisible: false),
      ),
    );
  }

  void _onStateChanged(AdLoadState state) {
    switch (state) {
      case AdLoadState.loading:
        setState(() {
          _height = 0;
        });
        break;

      case AdLoadState.loadCompleted:
        setState(() {
          _height = 80;
        });
        break;
      default:
        break;
    }
  }

@hkndzdr how did you solve this issue

@hkndzdr how did you solve this issue
This problem occurs if the controller is defined within the ads class you customized. The controller must come from a parent widget as a parameter.

@hkndzdr how did you solve this issue
This problem occurs if the controller is defined within the ads class you customized. The controller must come from a parent widget as a parameter.
Thanks @hkndzdr .

I have multiple ads in listview. So is i need seperate controller for each ad to load different ads?

@hkndzdr how did you solve this issue
This problem occurs if the controller is defined within the ads class you customized. The controller must come from a parent widget as a parameter.
Thanks @hkndzdr .

I have multiple ads in listview. So is i need seperate controller for each ad to load different ads?

Yes, but I don't think it's useful on the list. Because when the list is scrolled up and down, it loses its smoothness due to the reload of the ads.

@hkndzdr how did you solve this issue
This problem occurs if the controller is defined within the ads class you customized. The controller must come from a parent widget as a parameter.
Thanks @hkndzdr .

I have multiple ads in listview. So is i need seperate controller for each ad to load different ads?

Yes, but I don't think it's useful on the list. Because when the list is scrolled up and down, it loses its smoothness due to the reload of the ads.

@hkndzdr I tried with one controller for multiple ad in infinite list view(data from api based on pagination) and the scroll freezing. Is any way to solve this. Thanks

@azhar294
No known method. Loading the controllers beforehand and keeping them ready may be the solution.


Mutation input and output fields should support thunks

Thanks! I think it's probably worth updating the test cases to exercise this functionality. What do you think?

Just realized this relates to #22...
How does an array work in this case (as opposed to an object or a thunk)? What's the purpose?
cc @dschafer

How does an array work in this case (as opposed to an object or a thunk)? What's the purpose?

I meant thunk or object. Too much time in PHP!

@dschafer ha. Ok. Well then this should do it then :)

Sorry for letting this sit here and go stale, @skevy. Would you be able do rebase so that I can merge this in cleanly?

Very sorry I let this sit! LGTM :)

Any update on this @wincent @kassens @dschafer ?

Sorry for the delay!


Search doesn't work when configuration is used to map display names

Description
Describe your issue here.
Environment

0.2.x and 0.3.x
all

Steps to reproduce

choose the "Poseidon Network" datasource
search for a valid "VLAN" value

Expected results
the items with the designated VLAN should be highlighted
Actual results
search ignores the entire field

This happens because when building the search index CRviz flattens the dataset. During this step a mapping is made based on displayName. It should be mapped on path.


Shopify images .webp

First of all, thank you for the awesome work!
I think now Shopify supports .webp resizes. In your code there is a seciton at src/components/ShopifyImage.svelte that prevent webp images to be resized.
    // If image is a webp, remove the .webp extension from the url
    // (Shopify image transformations don't work with webp)
    if (image && image.url.includes(".webp")) {
      return `${image.url.replace(
        ".webp",
        ""
      )}&width=${width}&height=${height}`;
    }

I ve tried to add some webp images into my product list and i wasnt' able to load them properly. Commenting that code everything is ok.
I hope this helps! ;)

Hey @micvolo thanks for taking the time to write an issue!
I just merged a PR so it should be fixed now.


fix: disallow unknown parameters in config

Checklist

[ ] I have read the coding guide.
[ ] My change requires a documentation update, and I have done it.
[ ] I have added tests to cover my changes.
[ ] I have filled out the description and linked the related issues.

Description
Closes #3262

This change is

Heads up: There is similar PR currently open https://github.com/ethersphere/bee/pull/3272. @umerm-work, could you close #3272 once you merge this PR?

There is similar PR currently

Yeah sure.


Analyze performance of push and pull operations

Refer discussion in https://github.com/Azure/azure-mobile-apps-js-client/issues/206 for more details.

The Pull operation for a table of 19 columns with no large data in any column, is taking approx of 4 min to complete 1000 records. So for my complete data of 7950 records the complete pull operation was taking an average of 25-30 mins.

I did some insert tests, and it's true that each takes about a second each.
After some testing, it took on average 0.6 seconds to insert each one of the entries. (tested in Android emulator, local window app and my lumia 950), all gave similar results.
So certainly the insert is what is causing the bottleneck on pull operations.
I'll try to see how I can improve this and will show results.
This is my test code, in case I did something wrong:
var dataSize = 20;
        var inserted = 0;
        var start = Date.now();
        for (var i = 0; i < dataSize; i++) {
            var data = { text: i.toString() };
            //log('todoItemTable', todoItemTable);
            todoItemTable.insert(data)
                .then(function (a) {
                    inserted++;
                    if (inserted >= dataSize) {
                        var end = Date.now();
                        log('all inserted', (end - start)/1000);
                    }
                }, function (error) {
                    log('error', error);
                });
        }

@shrishrirang , found one more issue, that makes sync slow.
Remember how inserts and deletes are being processed one by one on individual transactions?, well, this code is doing the same:
operationTableManager.readPendingOperations(tableName, record[idPropertyName])
For every pulled record, there is a read using a transaction for each operation there. Wonder if I can do the same here as for the upsert/delete.

I am going to take a look at all store calls that can be batched. The one you mentioned is one such. Thanks for pointing it out. Let's use this issue to track all such instances and I will fix them all together.

@shrishrirang Modified readPendingOperations and moved from 20 rows/s to 100 rows/s.
Just to confirm it is a big bottleneck there.


Multiexecutor causing Database is Locked

I'm using MultiExecutor as suggested in #1823 , but after changing to MultiExecutor database is getting locked during update or select when a transaction is running. I'm making only one connection to DB using driftIsolate and using it across isolates.

On which part are you getting the "database locked" error, the reader or the writer?
With a MultiExecutor, you are likely opening multiple connections to the database even when all writes are going through the same connection. So especially if you're not using WAL, there's a chance to run into a locked database. I don't think there's too much that drift can do about that, but if you post your MultiExecutor setup I can take a look at what might be causing this exactly.

`DatabaseConnection createdriftIsolateAndConnect(
String path) {
return DatabaseConnection.delayed(() async {
String logPath = await PathManager.getUploadLogFolderPath();
driftIsolateAppDb = await createdriftIsolate(path);
DatabaseConnection backgroundConnection = await driftIsolateAppDb!.connect();
QueryExecutor executor = await constructAppDb(path);
return backgroundConnection.withExecutor(
MultiExecutor(
write: backgroundConnection.executor,
read: executor,
),
);
}());
}
Future createdriftIsolate(
String path) async {
// this method is called from the main isolate. Since we can't use
// getApplicationDocumentsDirectory on a background isolate, we calculate
// the database path in the foreground isolate and then inform the
// background isolate about the path.
final receivePort = ReceivePort();
await Isolate.spawn(
_startBackground,
_IsolateStartRequest(receivePort.sendPort, path),
);
// _startBackground will send the driftIsolate to this ReceivePort
return await receivePort.first as DriftIsolate;
}
void _startBackground(_IsolateStartRequest request) async {
// this is the entry point from the background isolate! Let's create
// the database from the path we received
QueryExecutor? executor = await constructAppDb(request.targetPath);
// we're using driftIsolate.inCurrent here as this method already runs on a
// background isolate. If we used driftIsolate.spawn, a third isolate would be
// started which is not what we want!
DriftIsolate driftIsolate = DriftIsolate.inCurrent(
() => DatabaseConnection.fromExecutor(executor!),
);
// inform the starting isolate about this, so that it can call .connect()
request.senddriftIsolate.send(driftIsolate);
}
Future constructAppDb(String appDBFilePath,
{String dbKey = "*****", bool logStatements = false}) async {
FfiCheck.setupSqlcipher();
final executor = LazyDatabase(() async {
final dbFile = File(appDBFilePath);
return NativeDatabase(dbFile, logStatements: logStatements, setup: (rawDb) {
rawDb.execute("PRAGMA key = $dbKey;");
});
});
return executor;
}`

Does the situation improve at all if you use rawDb.execute('PRAGMA journal_mode=WAL;') in the setup callback as well?

After enabling WAL I'm not facing database locked issue, however it does not occur all the time, I will let you know if I get database locked even after enabling WAL


Unable to Install Docker using PowerShell 7 (P.3)

Trying to install Docker inside WIndows Server 2019 fails. It does not appear to be possible to install the latest version of Docker using PWSH 7 (P3).
Steps to reproduce

Install-Module -Name  WindowsCompatibility -Force
$IHT1 = @{
  Name       ='DockerMSFTProvider'
  Repository = 'PSGallery'
  Force      = $True
}
Install-Module @IHT1
$IHT2 = @{
  Name         = 'Docker'
  ProviderName = 'DockerMSFTProvider'
  Force        = $True
}
Install-Package @IHT2

Expected behavior
This should have installed Docker - and it works in PWSH 5.x
Actual behaviour
PS C:\Foo> $IHT2 = @{
                   Name         = 'Docker'
                   ProviderName = 'DockerMSFTProvider'
                  Force        = $True
           }
PS C:\Foo> Install-Package @IHT2
Install-Package : The term 'Get-WindowsFeature' is not recognized as the name of a cmdlet, function, script file, or operable program.
Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
At line:6 char:1
+ Install-Package @IHT2
+ ~~~~~~~~~~~~~~~~~~~~~
+ CategoryInfo          : InvalidOperation: (Microsoft.PowerShel…lets.InstallPackage:InstallPackage) [Install-Package], Exception
+ FullyQualifiedErrorId : FailedToDownload,Install-Package,Microsoft.PowerShell.PackageManagement.Cmdlets.InstallPackage

This is kind of weird since Get-WindowsFeature is available. I've loaded the module using Import-WinModule.
Environment data

> docker info
docker : The term 'docker' is not recognized as the name of a cmdlet, function, script file, or operable program.

Of course, since docker did not get installed, the above result is to be expected.

PS C:\Foo> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      7.0.0-preview.3
PSEdition                      Core
GitCommitId                    7.0.0-preview.3
OS                             Microsoft Windows 10.0.17763
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0, 5.0, 5.1.10032.0, 6.0.0, 6.1.0, 6.2.0, 7.0.0-preview.3}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Transferred to PowerShell as this is not about a PowerShell Image

@RDIL I tried the code on both Preview.4 and for giggles, today;'s daily build. Both fail the same.

@doctordns I think this could be an issue with the module itself, but since this isn't related to the official
PowerShell docker images Microsoft publishes, I can't help you from here.  Sorry!

@rjmholt what is the status on Get-WindowsFeature?

The ServerManager module relies on System.Diagnostics.Eventing.EventDescriptor, so won't run on .NET Core.
Get-WindowsFeature will need Import-WinModule or similar to work.

In trying to install Docker,  I explicitly use Import-WinModule to "import" the Server Manager module, thus Get-WindowsFeature should work - but it doesn't. I am not a great debugger so can not quite work out why this is happening.

Using RC1, this now works OK:
S C:\Foo> $IHT1 = @{
>>   Name       ='DockerMSFTProvider'
>>   Repository = 'PSGallery'
>>   Force      = $True
>> }
PS C:\Foo> Install-Module @IHT1
PS C:\Foo> $IHT2 = @{
>>   Name         = 'Docker'
>>   ProviderName = 'DockerMSFTProvider'
>>   Force        = $True
>> }
PS C:\Foo> Install-Package @IHT2
WARNING: A restart is required to enable the containers feature. Please restart your machine.

Name          Version          Source           Summary
----          -------          ------           -------
Docker        19.03.5          DockerDefault    Contains Docker EE for use with Windows Server.


User Story

User Story
As a manager at an internet retail company
I want a back-end for my e-commerce website that uses the latest technologies.
So that my company can compete with other e-commerce companies

Acceptance Criteria
Given a functional Express.js API
When I add my database name, MySQL username, and MySQL password to an environment variable file
Then I am able to connect to a database using Sequelize
When I enter schema and seed commands
Then a development database is created and seeded with test data
When I enter the command to invoke the application
Then my server is started, and the Sequelize models are synced to the MySQL database
When I open API GET routes in Insomnia for categories, products, or tags
Then the data for each of these routes are displayed in a formatted JSON
When I test API POST, PUT, and DELETE routes in Insomnia
Then I am able to successfully create, update, and delete data in my database

The application developed met all the requirements above.


add rate, volume and pitch control attr

please add more attr, like this:
"<speak version='1.0' xmlns='http://www.w3.org/2001/10/synthesis' xml:lang='en-US'>"         f"<voice name='{voice}'><prosody pitch='{pitch}' rate='{rate}' volume='{volume}'>"         f"{text}</prosody></voice></speak>"
it's useful, thanks

v1.2.3 has added corresponding options