kernel_tests:cwise_ops_test fails on AVX512 systems

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 16.04
TensorFlow installed from (source or binary): source
TensorFlow version (use command below): ('v1.8.0-4021-g4292085', '1.9.0-rc0')
Python version: Python 2.7.12
Bazel version (if compiling from source):  0.15.0
GCC/Compiler version (if compiling from source): gcc (Ubuntu 5.4.0-6ubuntu1~16.04.9) 5.4.0 20160609
CUDA/cuDNN version: n/a
GPU model and memory: n/a
Exact command to reproduce: bazel test --config=opt -- //tensorflow/python/kernel_tests:cwise_ops_test

Describe the problem
The //tensorflow/python/kernel_tests:cwise_ops_test test fails on AVX512 machines.  Reproducing the problem is simple.  Just run the unit test.
bazel test --config=opt -- //tensorflow/python/kernel_tests:cwise_ops_test

The test is failing as there's a bug in Eigen's AVX512 implementation of the psqrt functions, which incorrectly compute the sqrt of negative numbers as 0 instead of NaN.  There's a pull request pending on Eigen that fixes the issue.
https://bitbucket.org/eigen/eigen/pull-requests/412/fix-avx512-implementations-of-psqrt/diff
A similar issue was fixed in Eigen's AVX2 implementations of psqrt a couple of years ago.
Source code / logs
cwise_ops_test.py:1920: RuntimeWarning: invalid value encountered in sqrt
  np_y = np.sqrt(x)
F
======================================================================
FAIL: testSqrt (__main__.IsFiniteInfNanTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "$HOME/.cache/bazel/_bazel_markus/59a925eeb655b30b5c683f8317fe569e/execroot/org_tensorflow/bazel-out/k8-opt/bin/tensorflow/python/kernel_tests/cwise_ops_test.runfiles/org_tensorflow/tensorflow/python/kernel_tests/cwise_ops_test.py", line 1926, in testSqrt
    self.assertAllEqual(np_nan, tf_nan.eval())
  File "$HOME/.cache/bazel/_bazel_markus/59a925eeb655b30b5c683f8317fe569e/execroot/org_tensorflow/bazel-out/k8-opt/bin/tensorflow/python/kernel_tests/cwise_ops_test.runfiles/org_tensorflow/tensorflow/python/framework/test_util.py", line 1401, in assertAllEqual
    np.testing.assert_array_equal(a, b, err_msg=msg)
  File "$HOME/.local/lib/python2.7/site-packages/numpy/testing/nose_tools/utils.py", line 855, in assert_array_equal
    verbose=verbose, header='Arrays are not equal')
  File "$HOME/.local/lib/python2.7/site-packages/numpy/testing/nose_tools/utils.py", line 779, in assert_array_compare
    raise AssertionError(msg)
AssertionError:
Arrays are not equal

(mismatch 76.1904761905%)
 x: array([ True,  True,  True,  True,  True,  True,  True,  True,  True,
        True,  True,  True,  True,  True,  True,  True,  True,  True,
        True,  True,  True,  True,  True,  True,  True,  True,  True,...
 y: array([False, False, False, False, False, False, False, False, False,
       False, False, False, False, False, False, False, False, False,
       False, False, False, False, False, False, False, False, False,...

----------------------------------------------------------------------
Ran 3 tests in 4.787s

FAILED (failures=1)
not equal where =  (array([ 0,  1,  2,  3,  4,  5,  6,  7,  8,  9, 10, 11, 12, 13, 14, 15, 16,
       17, 18, 19, 20, 21, 22, 23, 24, 25, 26, 27, 28, 29, 30, 31, 32, 33,
       34, 35, 36, 37, 38, 39, 40, 41, 42, 43, 44, 45, 46, 47]),)
not equal lhs =  [ True  True  True  True  True  True  True  True  True  True  True  True
  True  True  True  True  True  True  True  True  True  True  True  True
  True  True  True  True  True  True  True  True  True  True  True  True
  True  True  True  True  True  True  True  True  True  True  True  True]
not equal rhs =  [False False False False False False False False False False False False
 False False False False False False False False False False False False
 False False False False False False False False False False False False
 False False False False False False False False False False False False]

Nagging Assignee @rohan100jain: It has been 15 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @rohan100jain: It has been 30 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Patch has been merged into Eigen.  The issue will be fixed when tensorflow updates the version of Eigen it uses.

@rohan100jain This bug is still reproducible in tensorflow.  Fixing it requires an update to the version of Eigen used in tensorflow, so I think the bug should probably remain open.


Allow props to be passed to handleContextClick when showing menu from code

I have an external component library where I'm trying to provide different context menus for different types of child components. I am unable to edit the library to add in ContextMenuTriggers on all the child components.
I can show the correct menu by declaring ContextMenuTriggers at a higher level outside of the library, and calling the handleContextClick() function on the triggers. The library components use a callback to say when they need a context menu to be displayed. The basic structure looks like this:
const handleShowContext = (event, type) => {
    if (type === 'menu1') this.menu1.handleContextClick(event)
    else if (type === 'menu2') this.menu2.handleContextClick(event)
}

const renderComponent = () => (

        <ContextMenuTrigger id='menu1' ref={cm => this.menu1 = cm}>{' '}</ContextMenuTrigger>
        <ContextMenuTrigger id='menu2' ref={cm => this.menu2 = cm}>{' '}</ContextMenuTrigger>
        <MyLibraryComponent onShowContext={handleShowContext} />

);

The internal structure of the library looks a bit like this:
<MyLibraryComponent>
     this.props.handleShowContext(e, 'menu1')} />
     this.props.handleShowContext(e, 'menu2')} />
</MyLibraryComponent>

This works OK, however I don't have a way to pass the id of the component requesting the context menu into the menu itself.
Ideally I would be able to pass arguments to handleContextClick like this:
handleShowContext = (event, type, id) => {
    if (type === 'menu1') this.menu1.handleContextClick(event, { id })
    else if (type === 'menu2') this.menu2.handleContextClick(event, { id })
}

I thought that this could be implemented by only changing two lines:
L91 change handleContextClick = (event) => to handleContextClick = (event, additional) =>
L102 from callIfExists(this.props.collect, this.props) to callIfExists(this.props.collect, Object.assign({}, additional || {}, this.props))
However the props aren't updated when the menu is shown, so this doesn't work. Any thoughts on this?
EDIT Related SO Question?

Does not the SO answer solve your issue?

I'm not 100% sure, I'll take a look :+1:


Returned a nil supplementary view at indexPath

is anyone faced this issues ??
Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'Returned a nil supplementary view at indexPath <<NSIndexPath: 0xc000000000000116> {length = 2, path = 1 - 0}> from section controller:
need solution for this urgent .. :(

What is your setup? Can you share code in small test project that reproduces the issue? Need more info to be able to help

closing due to inactivity


About center_xxx.npy (used in training) do not generate from center_fea_xxx.py

I met the problem as below when I finished all the steps till python nice_stand.py to train:
(nice_eeg) mw@mw-System-Product-Name:~/my_data_disk_5T/eeg-image/THINGS-EEG/NICE-EEG$ python nice_stand.py  Tue Mar  5 00:46:03 2024 seed is 1933 Subject 1 initial define done. Traceback (most recent call last):   File "nice_stand.py", line 449, in <module>     main()   File "nice_stand.py", line 427, in main     Acc, Acc3, Acc5 = ie.train()   File "nice_stand.py", line 233, in train     test_center = np.load(self.test_center_path + 'center_' + self.args.dnn + '.npy', allow_pickle=True)   File "/home/meow/.local/lib/python3.8/site-packages/numpy/lib/npyio.py", line 405, in load     fid = stack.enter_context(open(os_fspath(file), "rb")) FileNotFoundError: [Errno 2] No such file or directory: '/home/mw/my_data_disk_5T/eeg-image/THINGS-EEG/NICE-EEG/Data/Things-EEG2/Image_set/center_clip.npy'
Take center_fea_clip.py for instance, the code only has
np.save(os.path.join(args.project_dir, 'center_all_image_clip.npy'), all_centers) means only generate the 'center_all_image_clip.npy',
how can I get the center_xxx.npy (used in training) ?

You may average the center of each concept to get the center_xxx.npy.
I have uploaded a version for reference here https://github.com/eeyhsong/NICE-EEG/blob/main/center_clip.npy.
🤝🤝

It works well!
Thank you very much for your attention and patience!

Please let me ask one more question about the center_clip.npy you have provided, is this file contain all the subjects? I only try the sub_10 for try-run, the accuracy is better than the result in your paper:

Thanks again for your support!


Basic usage  - relation "x" does not exist

Version
1.26.0
What happened?
Getting relation "x" does not exist for every query I mention in my queries .sql files
Relevant log output
query.sql:1:1: relation "models" does not exist
query.sql:15:1: relation "models" does not exist
query.sql:20:1: relation "models" does not exist
query.sql:28:1: relation "models" does not exist

Database schema
CREATE TABLE `transcription_jobs` (
  `id` integer PRIMARY KEY,
  `org_id` integer,
  `user_id` integer,
  `started_at` timestamp,
  `ended_at` timestamp,
  `tokens` integer,
  FOREIGN KEY (`org_id`) REFERENCES `organization` (`org_id`)
);

CREATE TABLE `organization` (
  `org_id` integer PRIMARY KEY
);

CREATE TABLE `models` (
  `id` integer PRIMARY KEY,
  `model_name` VARCHAR(255),
  `balance` integer,
  `org_id` integer,
  FOREIGN KEY (`org_id`) REFERENCES `organization` (`org_id`)
);

SQL queries
-- name: CreateModel :one
INSERT INTO models (
    id,
    model_name,
    org_id,
    balance
) VALUES (
    :id,
    :model_name,
    :org_id,
    :balance
) RETURNING *;

-- name: GetModel :one
SELECT * FROM models
WHERE id = :id
LIMIT 1;

-- name: UpdateModel :one
UPDATE models
SET model_name = :model_name,
        org_id = :org_id,
        balance = :balance
WHERE id = :id
RETURNING *;

-- name: DeleteModel :one
DELETE FROM models
WHERE id = :id
RETURNING *;

Configuration
version: "2"
plugins:
  - name: "py"
    wasm:
      url: "https://downloads.sqlc.dev/plugin/sqlc-gen-python_1.0.0.wasm"
      sha256: "aca83e1f59f8ffdc604774c2f6f9eb321a2b23e07dc83fc12289d25305fa065b"
sql:
  - schema: "path_to_folder"
    queries: "path_to_folder"
    engine: "sqlite"
    codegen:
      - plugin: "py"
        out: "src"
        options:
          package: "foo"
          emit_sync_querier: true
          emit_async_querier: true
          query_parameter_limit: 5

Playground URL
https://play.sqlc.dev/p/7150dcc1ccab07d5dcca915b08525b015d4f94968345157779d016a74eb04279
What operating system are you using?
macOS
What database engines are you using?
SQLite
What type of code are you generating?
Python

@eyal-solomon1 I'm guessing you're using the wrong syntax. Also, be careful on macs. It's the only platform I see which uses ` when you meant '.
https://play.sqlc.dev/p/c33a35439d22e91fc4aa9abf0b58e58cefa3ada7867784d30285886b0468536b

@dbhoot maybe syntax is not ideal from SQLC point of view, but from SQLite point of view it is ok. Seems that issue is related to quotes, because in your example there are no quotes in schema. Also, pls check my comment under another issue - https://github.com/sqlc-dev/sqlc/issues/3162#issuecomment-2097077192

I encountered the same issue when using SQLite and performing migrations with Atlas.
The migrations generated by Atlas include quotes, which cause sqlc to fail during parsing. When I run sqlc generate, it throws an error: relation "x" does not exist.
If I manually remove the quotes, the generation works fine. However, this requires me to rehash the migration, which is quite frustrating. Shouldn't sqlc handle single quotes properly in this case?

I'm also facing this issue when using Atlas for managing migrations. I would expect the parser to be able to handle quotes, as this is often needed to "escape" particular table or column names that may conflict with a built in statement CASE for example

I'm new to sqlc and was testing out to see if it's a good fit using an existing sqlite db's schema via sqlite3 data.db .schema > schema.sql which included backticks by default on table names and caused this same error. If I remove all the quotes/backticks, it works but I will have to do that manually everytime or use some preprocessor. So +1 for handling this by default.

I would like to bump this as it's also the same atlas + sqlc combination that I'm trying to use and running into the same issue.


POST to /sessions hangs

I did this accidentally. Not a big deal, but this is weird.

curl -X POST localhost:5000/sessions


ip validation working.  not adding in the range include validation ye…

…t as when the network is defined inline with the interface, it does not actually pick up the validation trigger from the build method and there are cases that a gateway (which is the only thing that I can use with the netmask to figure out the full range) will not be defined.  It would be nice to have that in there, but unless I rename the network object to subnet (or something like that) to let me use the common but deprecated network attribute that is found in both rhel and debian based configs, it will not be possible.
Resolves #2
Thought of several situations where you would not want the gateway set (duh, stupid idea in the first place) so the second part of the issue - namely the autoconfiguration of the gateway address -  will not be included in this.

Coverage increased (+0.3%) to 90.89% when pulling 71e289b920f96eb294ae5565fff4841c38875ef7 on ip_address_validation into f421951f30876de235e4a7198f7677b0b389389e on master.


Update and rename .env.example to .env

What will change?

Stuff that is going to change
even more interesting stuff that is going to change

Security
Before you click Merge, take a step back and think how someone could break the Confidentiality, Integrity and Availability of the code you've just written. Are secrets secret? Is there any sensitive information disclosed in logs or error messages? How does your code ensure that information is accurate, complete and protected from modification? Will your code keep Contentful working and functioning?

Hi @Joshuromi , I am not sure this change is relevant to this project.
Did you happen to fork the project and intend to push the changes there?

Hey @Joshuromi, I'll close this PR.


pastebin to make typing -f commands shorter

Want to make typing -f and -F commands shorter? https://pastebin.com/w13bhrSw
Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Bug fix
[x] Improvement
[ ] New extractor
[x] New feature

Description of your pull request and other information
Explanation of your pull request in arbitrary form goes here. Please make sure the description explains the purpose and effect of your pull request and is worded well enough to be understood. Provide as much context and examples as possible.

Adds a link to the readme to a pastebin of a windows batch file (.bat) that makes typing -f and -F commands shorter.


Consolidate network flags under a single flag

Right now we have all these flags for connecting to different networks:
  --kovan        Connect to Kovan network
  --mainnet      Connect to Mainnet network
  --poa          Connect to POA network
  --rinkeby      Connect to Rinkeby network
  --ropsten      Connect to Ropsten network
  --rsk          Connect to RSK network
  --rsk-testnet  Connect to RSK testnet network
  --sokol        Connect to Sokol network
  --url=url      [default: http://localhost:8545] URL of the ethereum node to connect.
  --xdai         Connect to xDAI network

I propose we merge all those into a single flag: --network, --net or maybe --url. The default value continues to be http://localhost:8545. If the given value is not an URL, it will be interpreted as a "shortcut" to one of the known networks (exposed by eth network:urls).

Closing this since I don't like this idea anymore :man_shrugging:

Reopened by popular revolt.


[Test] Change the -target line on some tests to use a minimum of macOS 10.13.

These tests were targeting 10.9 or 10.10, but the minimum deployment target now supported by Xcode is 10.13. Bump them up to match:
test/Concurrency/Backdeploy/linking.swift
test/Concurrency/Backdeploy/linking_maccatalyst.swift
test/Runtime/stable-bit-backward-deployment.swift

@swift-ci please test

@swift-ci please test macos platform


GH-1409: Fix Nacks for Async Replies

Resolves https://github.com/spring-projects/spring-amqp/issues/1409
Normally, when message has a fatal exception (such as message conversion)
basicNack with multiple true is used, to nack any previously unacked
messages (e.g. when using batch size to limit the ack traffic).
Even when using manual acks, fatal exceptions are nacked by the container
because the user does not have access to the message.
However, when using async replies, this has the side effect of nacking
unprocessed messages.
Detect whether async replies are being used and only nack individual
records that cause fatal exceptions.
Also, coerce the AcknowledgeMode to MANUAL for such listners.
Add a test for both containers; send a good message followed by a
bad one without actually completing the reply future.
After the exception occurs and the container is stopped, there should
be one messag in the queue.
cherry-pick to 2.3.x, 2.2.x

... and cherry-picked to 2.3.x as: https://github.com/spring-projects/spring-amqp/commit/94fff33adfdf4969e8a90c171383ae6b481e4aaf after some minor conflict fixing.
And to 2.2.x as https://github.com/spring-projects/spring-amqp/commit/aa3589c2d63b65afce358f16268e97660ef1d4b1 after more conflicts resolution.


Display the ETH value in actionsheet for transaction initiated by dapp

Before PR
After PR

Forced push after rebase master.


Prepopulate variable

Is your feature request related to a problem? Please describe.
I have a dependent variable, but before running the cheat I need to edit it.
Describe the solution you'd like
I have 2 variables "basis" and "variant", where "basis" is selected via fzf. Say "basis=V100.master". Now I will want to execute "mv V100.master V101.master", but I need to choose the target name case by case. Whe using "mv  " in the cheat, that would work out, but I'd really like to not type out "V101.master" for the variable "variant", but prepopulate the input line with "V100.master" so I can just edit it. Of course, my real use case involves much longer names and pathes.
Describe alternatives you've considered
I could generate a list of candidates for "variant", but I will invarantly run into problems when the candidates don't fit (which will be often enough to make this not useful). I could leave "variant" empty and type the whole value, but that will end up too much work.
Maybe there's already a way to do it, but I've not found it. Thanks for any help :)

Possible workarounds:
Workaround 1

Have a cheatsheet with a command such as mv <myfile> <myfile>
Use the shell widget
Call the snippet accordingly
Edit the command before running it

Workaround 2

Install navi 2.15.2+ (I haven't released it yet)
Have a cheatsheet with a command such as mv <myfile1> <myfile2> and `$ myfile2: echo $myfile1; echo other stuff --- --fzf-overrides bind=ctrl-u:replace-query
Use the shell widget or invoke navi
Select myfile1
Press Ctrl-u
Edit the value
Press tab

Hey I didn't realize the shell widget does this, nice! So 1 is already good, and 2 will be a slight improvement which I'm looking forward to (since I won't have to type the file extension by hand). Thanks a lot!


Move editor window down if it is above the screen

Move editor window down if it is above the screen

Fixed in 20210422_3


typo correction

@Eduard-Uta : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Looks good. Thanks for the typo fix @Eduard-Uta.


Add SAM title fields to front-end and change Name references to LEGAL…

…_BUSINESS_NAME__c

@courtneymyers I realized my last commit (changing Name field and adding title fields) broke the front-end New Rebate modal, so I went ahead and updated the front-end to point to the new LEGAL_BUSINESS_NAME__c field (and added the title fields to the samUserData type definition)

Thanks, @dpgalloway!


Add cloud function files

Here's the code I had in the Cloud Function, to scrape the weather forecast of the Central Park weather station and save to GCS.

Thanks Laura!


<EMAIL_ADDRESS>BUG - relative components generation

OS?  Windows 7, 8 or 10. Linux (which distribution). Mac OSX (Yosemite? El Capitan?)
Windows 10 x64

Versions. Please run ng --version. If there's nothing outputted, please run
in a Terminal: node --version and paste the result here:
<EMAIL_ADDRESS>node 5.9.0

Repro steps. Was this an app that wasn't created using the CLI? What change did you
do on your code? etc.
STEP1. Write "ng new APPNAME"
STEP2. Write "ng g component Component1" //this command ignores uppercase letter on Component1 - that's the first bug
STEP3. get into the dir of that component1 by "cd src" -> "cd app" -> "cd component1"
STEP4. Write "ng g component Component1Child1"
STEP5. ng build -watch

The log given by the failure. Normally this include a stack trace and some
more information.
ERROR in ./src/app/app.module.ts
Module not found: Error: Can't resolve './component1component1child1/component1child1.component'

Mention any other details that might be useful.
When you enter app.module.ts you see that the path for Component1Child1 looks like that:
import { Component1Child1Component} from './component1****component1child1/component1child1.component';

As you can see the slash is wrong way. It inserted this "" instead of "/". After fixing that the app works well. It's just a typo that is being generated every time you generate component inside of another component.

Yup,
While already in the app folder:

mkdir src/app/layout
ng generate component layout/header

Now, if you check app.module.ts, the import statement added by the CLI looks like this:
import { HeaderComponent } from './layout\header/header.component'; - notice the second slash in the path is a backslash.
I have to go in app.module.ts and replace the backslash with forward-slash myself.
Otherwise, ng build fails with: "Cannot find module './layoutheader/header.component'." (initially I would have expected it to treat the backslash the same like a regular "/" slash in a path, but it doesn't).
angular-cli: 1.0.0-beta.11-webpack.2
node: 6.4.0
os: win32 x64

Dupe of https://github.com/angular/angular-cli/issues/1719


Add backend API proxy and frontend shared dependencies

Closes #7
Closes #10
We now have backend dependency management without locking ourselves down to specific versions!

Should we also add a test script to package.json? I actually didn't add one for eslint in my earlier PR

Probably, but that should be in a different PR.


Fix evmcs hyperv feature bug

What this PR does / why we need it:
Fix Evmcs Hyperv bug.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Currently whenever EVMCS HyperV feature
Is not nil in the vm spec than all dependencies
features of EVMCS are enabled.
Even if we set vmi.Spec.Domain.Features.Hyperv.EVMCS.Enabled
to be false.
Special notes for your reviewer:
Release note:

NONE

/lgtm
/approve
Extra bonus points for unit-test.

I added unit test and moved the tests to hyperv_test file.
/unhold


Rust nightly

This will probably be an ongoing issue seeing how Rust changes all the time. Currently doesn't work with nightly rustc 1.0.0-nightly (c5961ad06 2015-01-28 21:49:38 +0000).
src/cipher/ecdhe.rs:2:5: 2:23 error: unresolved import `std::io::BufReader`. Could not find `io` in `std`
src/cipher/ecdhe.rs:2 use std::io::BufReader;
                          ^~~~~~~~~~~~~~~~~~
src/handshake.rs:1:5: 1:23 error: unresolved import `std::io::MemReader`. Could not find `io` in `std`
src/handshake.rs:1 use std::io::MemReader;
                       ^~~~~~~~~~~~~~~~~~
src/client.rs:1:5: 1:33 error: unresolved import `std::io::net::tcp::TcpStream`. Could not find `io` in `std`
src/client.rs:1 use std::io::net::tcp::TcpStream;
                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/client.rs:4:15: 4:23 error: unresolved import `std::io::IoResult`. Could not find `io` in `std`
src/client.rs:4 use std::io::{IoResult, IoError, OtherIoError};
                              ^~~~~~~~
src/client.rs:4:25: 4:32 error: unresolved import `std::io::IoError`. Could not find `io` in `std`
src/client.rs:4 use std::io::{IoResult, IoError, OtherIoError};
                                        ^~~~~~~
src/client.rs:4:34: 4:46 error: unresolved import `std::io::OtherIoError`. Could not find `io` in `std`
src/client.rs:4 use std::io::{IoResult, IoError, OtherIoError};
                                                 ^~~~~~~~~~~~
src/tls_result.rs:2:5: 2:21 error: unresolved import `std::io::IoError`. Could not find `io` in `std`
src/tls_result.rs:2 use std::io::IoError;

Fixed somewhat in #4 but needs more work (cargo test in particular).

Currently fixed by #7.


Spring cloud gateway is reloading routes every 30 seconds

Describe the bug
Please provide details of the problem, including the version of Spring Cloud that you
are using.
Sample
If possible, please provide a test case or sample application that reproduces
the problem. This makes it much easier for us to diagnose the problem and to verify that
we have fixed it.

I am loading all routes dynamically. But, that section is executing every 30 seconds.

@lsomisetty do your have any simple to reproduce this issue?from your description i can't get useful information.

Hi
Attached is the code in zip file that loads routes for every 30 seconds. I have 2 classes. One is 'VanityURL' model class and other one is 'URLRouteDefinitionLocator' class. URLRouteDefinitionLocator class is dynamically generating RouteDefintions.  This class implements RouteDefinitionLocator and has method getRouteDefinitions(). This method is executing for every 30 seconds.
I dont want this method to be executed for every 30 secs . I would like to stop this auto reload.
routeload.zip

Used this property eureka.client.registry-fetch-interval-seconds to control heartbeat frequency .

I have the same problem.

@lsomisetty @370019421 please post simple code to reproduce this issue,not pom file or log

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


Add lifecycle and virtual dom tests

I copied over and modified the vdom.test.js and lifecycle.test.js files from hyperapp to give this repo more test coverage. I also copied the svg test from app.test.js into vdom.test.js to get coverage on that feature.
Partially addresses #3

Codecov Report

Merging #23 into master will increase coverage by 88.28%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master    #23       +/-   ##
=========================================
+ Coverage   11.71%   100%   +88.28%
=========================================
  Files           2      2
  Lines         111    111
  Branches       31     31
=========================================
+ Hits           13    111       +98
+ Misses         71      0       -71
+ Partials       27      0       -27

Impacted Files
Coverage Δ

src/patch.js
100% <0%> (+100%)
: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 e349fd7...80b6ff8. Read the comment docs.

@jbucaran Anything else you'd like to me to?

@andrewiggins Sorry! 🙇 I've been super busy with work and other things!
I'll merge this soon and then add my changes on top of your work.
Thank you for moving this forward Andrew! 👍

No problem! Thanks!


Pull request to apply for educational verification of Lazi Akademie

@lazi-maschenborn I am sorry for a delay with processing your request.
Pull request merged. Thank you!


Issue 9 add mesh and operators for spherical domian

Description
Mesh remains unchanged as previously discussed. Divergence is changed in case of negative or positive particles. No other changes to operators are required.
A convergence test for divergence operator is provide.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

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

Key checklist:

[x] No style issues: $ flake8
[x] All tests pass: $ python run-tests.py --unit
[x] The documentation builds: $ cd docs and then $ make clean; make html

You can run all three at once, using $ python run-tests.py --quick.
Further checks:

[x] Code is commented, particularly in hard-to-understand areas
[x] Tests added that prove fix is effective or that feature works
[x] Any dependent changes have been merged and published in downstream modules

Codecov Report

Merging #142 into master will increase coverage by 0.03%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #142      +/-   ##
==========================================
+ Coverage   94.19%   94.23%   +0.03%
==========================================
  Files          35       35
  Lines        1293     1301       +8
==========================================
+ Hits         1218     1226       +8
  Misses         75       75

Impacted Files
Coverage Δ

...m/discretisations/finite_volume_discretisations.py
100% <100%> (ø)
:arrow_up:

pybamm/discretisations/meshes.py
100% <100%> (ø)
: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 63e7457...004f344. Read the comment docs.

thanks for adding test, merging


set_collision_layer_bit and set_collision_mask_bit  create an illusion in the inspector, but they do not affect physics.

Godot_v3.1.1-stable_win64.exe
Windows 7
Switching collision layers and masks for physical bodies using the set_collision_layer_bit (№, true / false) and set_collision_mask_bit (№, true / false) functions in the script has no effect on the collisions in the game, but it can be tracked in the inspector that the switching going fine. In addition, if you manually switch layers in the Inspector before starting the game, they will remain in this position both after the game is started and in the script when the functions are called. In other words, the inspector says what works, but in fact not.

Which bodies exactly? Any reproduction project?

I have not been experiencing this in my physics work lately, I'd also like to see how to reproduce it.

@KoBeWi @avencherus
Forgive me, it was probably a false alarm, I could not find out what the reason was and why physics refused to work properly.
I have deadlines, and I have been spending on this problem for 3 days already and I can no longer look for reasons, so I just redid everything in a new way, and now everything seems to be working.
I now have only one guess, why it did not work in the first version ...
To realize the explosions that caused damage to the area at a defined point in time, I used the method ($"Explosion/CollisionShape2D".set_deferred("disabled", false)) to include a detection zone, which was to take all the bodies in the radius of the lesion and call into them the function of dealing damage.
Now I did everything the other way, avoiding this function.
My guess was that the physics server did not have time to process the overlap of bodies in this frame (in which the function included the form), and waited for the next frame, but then it would be too late to process the event, since by the time of the next frame the object should be destroyed both from one and from the other side of the interaction.
Based on this, I close this problem.


Running iromlab-configure  results in 'no configuration file found in package'

This happens on  new Windows 10 machine, Python 3.8.

Error happens because iromlab-configure expects package config file in:
c:\python\python38\lib\site-packages\iromlab\conf\config.xml

Which is the expected location for a globally installed package, but not for a user install!
Locations are found using:
sitePackageDirs = site.getsitepackages()

Which gives following output:
c:\python\python38\
c:\python\python38\lib\site-packages\

So this obviously doesn't work.

Fixed by https://github.com/KBNLresearch/iromlab/commit/c6310edb0c0148adeb6ae4a7ccbe3417cdee81d8 and https://github.com/KBNLresearch/iromlab/commit/38bae1b34039467ccaa86063c1bb10cad089a38d. TODO: test if this doesn't break global package installs!


RFC: Add implementation of Iterator::size_hint for PyIterator

When the Python iterator backing PyIterator has a __length_hint__ special method, we can use this as a lower bound for Rust's Iterator::size_hint to e.g. support pre-allocation of collections.
There are a few complications though:

The cost of calling lookup_special and extracting the length hint might be too high compared to what typical users of size_hint expect.
Python's operator.length_hint will first try __len__ before falling back to __length_hint__ which would be more robust but worsen the first issue.

Additionally, Python's and Rust's semantics do not really match up as the
single estimate given by __length_hint__ might be wrong in both directions. To
support the use case of pre-allocating collections, it is used as the lower
bound and correctness should never depend on it being correct, but it might lead
to unnecessary over-allocations.

It seems nice to me to do something along these lines for sure. Have you benchmarked to see if adding this improves performance for e.g. PyIterator -> Vec<&PyAny>?
Some possible alternatives:

Using PyObject_LengthHint may be faster (but not supported on abi3). That does try len() first.
We could potentially just use self.len(), as that should be guaranteed correct if defined (for both lower and upper bounds) and will be fast because we can go through the C API.

It seems nice to me to do something along these lines for sure. Have you benchmarked to see if adding this improves performance for e.g. PyIterator -> Vec<&PyAny>?

Not yet, will add a benchmark.

Using PyObject_LengthHint may be faster (but not supported on abi3). That does try len() first.

I did look for FFI bindings, but I did not remember to camel case length_hint. 🤦🏽 Since Iterator::size_hint is just that a hint, I think it would be best to implement this when the full API is used, and to leave it out otherwise.

We could potentially just use self.len(), as that should be guaranteed correct if defined (for both lower and upper bounds) and should be fast because we can go through the C API.

The few iterators like iter([1, 2, 3]) I looked at did not define __len__. In any case, if we do the above then __len__ will by checked first by PyObject_LengthHint.
I also don't think the upper bound is too critical as e.g. impl FromIterator<T> for Vec<T> will use only the lower bound for pre-allocation, c.f. here

Rebooted this into a pure optimization if the stable ABI is not used and added a benchmark. Benchmarking memory allocation seems hard and I get a very noisy signal with at most single digit percentages improvement. But since this optional implementation seems straight forward, I'd say this definitely rather helps than hurts. Nothing worthing of a changelog entry though...


Extract search dropdown to React component

What about the tests? We can move the tests for the dropdown in its own file so the App.test.tsx file can be less cluttered

valid point! I've extracted a search-specific test now. However, some stuff that is part of a larger integration test ("should update focus-node & pipeline parameters") will remain in App.test for now. Maybe I'll get around to refactor some of the url handling into its own component some time, but don't have an idea for it yet.


Opening multiple files at once

I used to be able to open multiple files at once from command-line etc. by passing them all as arguments, so NitroPaint.exe file1 file2. Sometime after <IP_ADDRESS>, this stopped working--it creates processes I can see in Task Manager, but nothing actually appears. Passing one file works fine, but I want to be able to pass multiple together so they are in the same window and "see" each other (passing in one file multiple times puts them all in different windows).

Hmm, I don't seem to be able to replicate this on my end, I've just tested it and was able to open two files by putting them together on the command line. I'm not sure why they wouldn't be opening correctly on your end since it seems you're doing the same. What was the last version it worked for you on, and which version were you using that it didn't?
Also, in the View menu you can enable "Single Instance," which makes it so that when you open files separately they'll open in the same window.

Okay, it seems I have misidentified the issue. The actual issue is that it will not let me run NSCR’s from the command line (nor can I double click them, but I can open them from a NitroPaint open menu). This is an issue with <IP_ADDRESS>. I was not having issues with <IP_ADDRESS>.

but here's the NSCR and its NCLR and NCGR to be safe: https://drive.google.com/file/d/1FaLyqw959zub3ozC0C1hP5F1hDjDPhAx/view?usp=drive_link

Odd, tried it just now with these files and it seemed to be working fine. I ran from the command line with all three file paths and it opened all 3 files correctly on my end. You're saying the issue is that the NSCR just isn't opening up? Does this happen for any other files?

No, the issue is that if an NSCR is included nothing opens at all.

But only opening the other two works fine for me.

And this issue happens with some files I tried from LEGO Battles as well.

okay! turning "DPI Aware" off fixed this

Hmm, that's a bizarre behavior. I'll have to look into that.

Is there anything you need to know about my computer? the resolution is 1920 x 1080, but I wouldn't know how to find my DPI

It couldn't hurt to know, but I don't think it's probably related to your specific DPI. If you go to your display settings there should be a UI scalar somewhere, maybe 125% or 150% perhaps. I won't be able to look into this today, but might be able to some time this week.

My scale is 125%. And thank you for looking into this, take all the time you need.

I was able to replicate this issue on my end using a 125% display scale factor and enabling DPI awareness. It looks as though the issue is stemming from scaling code in the NSCR viewer, which gets the program stuck. Then running a new instance of the program will see that the other exists and kill itself (when single-instance is enabled). Will look into a fix for this.

Wonderful, I’m glad you can replicate it, and thanks for looking into this.


Npm cant install the generator

Currently following command is needed for installing the generator:
npm install -g yo gulp generator-vars-django

Removed peer dependencies in v2.0.3, 9487da94adc6b8b31752533dab3acf99a81918a6 :)


Missing index.html within link generated to designate the canonical URL

Problem
On a page rendered by index.rst/index.md files, like this one about built-in functions, the index.html page name is omitted on the rendered variant of the <link rel="canonical" representation.
<link rel="canonical" href="https://cratedb.com/docs/crate/reference/en/latest/general/builtins/" />

This flaw causes all sorts of downstream problems.
Details
@msbt is outlining more details about the problem. Thanks!
The massive amount of non-indexed pages are a result of our docs setup. The top 2 non-Google-issues (Alternate page with proper canonical tag and Page with redirect) are mostly because of the redirect chains and versioning that we have in place.
If you take this URL as an example:
https://cratedb.com/docs/crate/reference/en/master/general/builtins/subquery-expressions.html
The page also exists in these (and probably some more) versions:
https://cratedb.com/docs/crate/reference/en/5.6/general/builtins/subquery-expressions.html
https://cratedb.com/docs/crate/reference/en/5.5/general/builtins/subquery-expressions.html
Both links above have this URL set as canonical:
https://cratedb.com/docs/crate/reference/en/latest/general/builtins/subquery-expressions.html
This will obviously result in a lof of unindexed pages, not sure if this can be fixed since it's not really broken.
To show an example of your links, this URL shows as not-indexed because of "Alternate page with proper canonical tag":
https://cratedb.com/docs/guide/install/cloud/aws/index.html
If you inspect that URL, you can see the "User-declared canonical" is:
https://cratedb.com/docs/guide/install/cloud/aws/ (which is indexed)
So the index.html gets omitted by RTD and every docs-page ending with index.html gets a not-indexed issue attached to it. Can we maybe add that to the canonical URL to avoid that @amotl?
References

GH-482

/cc @matkuliak, @michaelkremmel

Observations
We did a few orientation flights on this topic together with @msbt, and came to the conclusion that RTD might have deprecated the "canonical_url" thing already, as it might only have been required for early versions of Sphinx<1.8 and RTD of that times.
Today, it is advised to use html_baseurl:

For sphinx >=1.8 we can use html_baseurl to set the canonical URL.
-- https://github.com/readthedocs/readthedocs.org/pull/7540/files

... but not define it:

If you are using Sphinx, Read the Docs will automatically add a default value of the html_baseurl setting matching your canonical domain.
If you are using a custom html_baseurl in your conf.py, you have to ensure that the value is correct. This can be complex, supporting pull request builds (which are published on a separate domain), special branches or if you are using subproject s or translations. We recommend not including a html_baseurl in your conf.py, and letting Read the Docs define it.
-- https://docs.readthedocs.io/en/stable/guides/canonical-urls.html

Thoughts
In this case, the section in readthedocs-insert.html.tmpl might actually be a backward-compatibility thing?
References
We are not sure if each one of them is relevant. However, all are about fixing or improving the situation wrt. canonical links, in one way or another. In this spirit, I am enumerating them here, because there is a chance we missed something on the ugprade path since Sphinx 1.8 (~10 years ago?).

https://github.com/readthedocs/readthedocs.org/issues/556
https://github.com/readthedocs/readthedocs-sphinx-ext/issues/83
https://github.com/readthedocs/readthedocs-sphinx-ext/pull/93
https://github.com/sphinx-doc/sphinx/issues/9730
https://github.com/kai687/sphinxawesome-theme/pull/1262
https://github.com/aiven/devportal/issues/1691
https://github.com/readthedocs/readthedocs.org/pull/7540

Through some cleanups and refactorings, we removed some configuration overhead, and fixed the issue described above, still using a few Crate-specific workarounds.

GH-488

The improvements have been released with version 0.31.2.


Shift + Click does not select multiple objects

When you hold shift and click multiple objects in edit mode, it should be selecting all of those objects. You can see this works with the demos on FabricJS's main homepage. However, this is not working with fabricjs-viewport.

Hi,
You can work around this by altering the following method, to ensure that it passes on the shiftKey parameter in the event that is returned:
Viewport.prototype._transformEventParams = function (e) {
			var offsetLeft, offsetTop;
			offsetTop = this.canvas.wrapperEl.getBoundingClientRect().top;
			offsetLeft = this.canvas.wrapperEl.getBoundingClientRect().left;
			return {
				which: 1,
				shiftKey: e.shiftKey,  // *** Note: added the shiftKey event parameter. ***
				clientX: (e.clientX - offsetLeft) / this.zoom + offsetLeft - this.translate().x,
				clientY: (e.clientY - offsetTop) / this.zoom + offsetTop - this.translate().y,
				pageX: e.pageX - this.translate().x,
				pageY: e.pageY - this.translate().y,
				screenX: e.screenX - this.translate().x,
				screenY: e.screenY - this.translate().y
			};
		};

Cheers,
Rob.


UTXO(String txHash, int vout, long amount, String address, String scriptPubKey, String derivedPath)

txHash scriptPubKey 这几个字段的数据从哪里来的

This issue will be closed as it has not update for a long time. You can reopen it if you have any other problem


Not Working Please Help

I need your help. I have setup laravel-gamp on my laravel project. I followed your instruction. But it seems to not working. What else should I check?

nevermind this. fixed already


Script for "Weather Notifier" added

Weather Notifier -
The Weather Notifier script is a Python-based application that fetches and displays real-time weather information for a specified city. The script uses the OpenWeatherMap API to retrieve weather data and presents it through a user-friendly GUI built with Tkinter. Additionally, it supports both manual and automatic weather updates, allowing users to either fetch the weather data on demand or set a recurring interval for automatic updates.
It solves the issue  #954 -
Self Checks

[x] One Change in one Pull Request
[x] My file is in proper folder (Name of folder should be in lowercase with no space in between) (weather_notifier/weather_notifier.py)
[x] I am following clean code and Documentation and my code is well linted with flake8.
[x] I have added README.md and requirements.txt (Include version numbers too e.g. pandas==0.0.1) with my script
[x] I have used REPO README TEAMPLATE (Necessary)
[x] Just including required dependencies in requirements.txt
[x] Added Screenshots of the gui
[x] Followed the PEP8 using flake8 linter and black

I am open to suggestions if any and if not then please merge it and add the "hacktoberfest-accepted" label to it.

The linting error is not in my folder, it is in the existing folder ( video_cropper and pdf_watermarkadder) and also I have tried to resolve some linting issues in another existing scripts but still there is some linting error in their folders which is causing the linker check to fail. Thank You

Now my script has no merge conflicts, thanks for resolving the issue of linting by updating version and actions. Please merge this script . Thank You .


add --watch flag

Semver major. Useful to have --optimize available with bankai start
so you don't need to keep reloading the servereven if you want optimized
output.

Closing in favor of #153 which does like wayyy more and kinda depends on this patch. The implementation of --watch is buggy here anyway, fixed it in #153


Rings stop rendering after a certain distance

What Happened?
When I made a galactic core have a accretion disk around 10x the size of the OW solar system, new horizons stopped rendering it at a certain distance. This is a bug and not a feature, so technically you should fix it.
What was supposed to happen?
No response
Platform
Steam
Mods
My mod, my mod only, my mod!

Logs
No response

Mfw you make an accretion disk bigger than the far clip plane smh

Realism!

Mfw you make an accretion disk bigger than the far clip plane smh

We will not fix this, sorry.

Ok we could do this (far clip plane distance setting on system config)

Ok we could do this (far clip plane distance setting on system config)

Will you just force SSAO off or something? You said SSAO breaks when this is modified.

itll just break it, yeah. if you wanna do that then u can do it


Switch LabelledGeneric from Symbol to String

Also fix #660

That was quick! :tada:

Sorry this PR became so big, but everywhere I look I see things that need refactoring 😄


remove special-case any/empty code on MarkerUnion

MarkerUnion had code handling the possibility that one of its markers is AnyMarker or EmptyMarker, neither of which should ever happen.
Conceivably it makes sense to have this code anyway for defensive purposes, but in that case there ought to be similar on the MultiMarker.  I've instead preferred to remove it.
The testcase that I've removed was testing that the string representation of such a union ignored the any / empty markers.  But this was nonsense: if there ever actually was an "any" marker in there, then the whole union ought to have been "any".
While passing I've also removed is_empty() from the AnyMarker and is_any() from the EmptyMarker: they too can inherit the default implementation (return False) from the base class.

not sure I agree with sonarcloud's "improvement" here, and I certainly have no idea why this merge request provokes it into making the suggestion...


Crash code 5

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.85.2
OS Version: Mac OS 15.1 (24B83)

Steps to Reproduce:

I create the jupyter notebook in Download folder and paste the huge data
then the vscode crashed by code 5
every time i open this folder it will automatically open by last checkpoint and crashed every time when i open this folder

crashinfomation.txt

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.95.3. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

It's not a bout the version if i update the new version still have the crash in this folder

@deepak1556 any idea of what might go wrong with the error code 5? I wonder if we have a way to avoid repeated crashes on window reloads.

It is usually a crash due to the javascript heap limit reached or oom from the system.
Can you follow the steps before to capture a heap profile,

From a fresh instance that is open a different folder using code <some-folder>, run Developer: Toggle Developer Tools
Switch to the Memory tab and select the option as shown in the screenshot below

Start the recording
Open the folder that triggers the crash
Save the profile and attach it to this issue

ok thx, i will try this way if i met it again


Add missing

Was putting the two links on the same line

Coverage remained the same at 24.323% when pulling a97ff7f87afc583d14cb96d47c4ea0811b16634e on Kiloutre:patch-37 into 58217a51829cb91edf1b78f13852a9bc2d0cdf56 on NyaaPantsu:dev.


Add Flappy Bird Game

Assign me under JWOC 2024

I don't think it will fit the theme of the website.


Handle YAML parsing error

Fixes #20 .

@raghubetina Please take a look and change the error message if required.

@murugan-r For the error message, what do you think of the following?
"It looks like there's something wrong with your token in `/grades.yml`. Please delete that file and try `rails grade:all` again, and be sure to provide the access token for THIS project."

@raghubetina Updated the message. Merging it.


Error on defining constructor for hooked sorts

Fixes #695
Reviewer checklist

[ ] Test coverage: stack test --coverage
[ ] Public API documentation: stack haddock
[ ] Style conformance: stylish-haskell

@ttuegel @virgil-serbanuta  Should I change this to error regardless of whether the sort is hooked?

Why would that be an error?


Error opening/using valorant-rpc.exe

Downloaded v3.0.0 and ran for the first time, threw this error:
attempting to autodetect region
the program encountered an error; please create an issue with the traceback below if this problem persists
Traceback (most recent call last):
  File "valclient\client.py", line 848, in __get_lockfile
FileNotFoundError: [Errno 2] No such file or directory: 'C:\\Users\\xyz\\AppData\\Local\\Riot Games\\Riot Client\\Config\\lockfile'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "valclient\client.py", line 69, in activate
  File "valclient\client.py", line 853, in __get_lockfile
Exception: Lockfile not found

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "main.py", line 18, in <module>
  File "src\startup.py", line 34, in __init__
  File "src\startup.py", line 132, in check_region
  File "valclient\client.py", line 78, in activate
Exception: Unable to activate; is VALORANT running?
press enter to exit...

This file doesn't exist in the path:

tried running as admin and already in the game lobby?


Save output to file

This would make it easier to run this as a cron job or something, and load the data from other services/clients.
Not  sure if this is the best way though.

Merged by mistake directly to master, cherry picked it to here: https://github.com/benmarten/CryptoETF/commit/0de2b1fe6608c7316937b8fb236c67fd30d441e1


Have demo tax account name match the name the user entered

The demo personal tax account is currently in the name of John Doe.
It would be cool if it showed whatever name the user entered when going through the rest of the demo.
This would be valuable because I think people might get distracted by having a fake name there when what we're trying to demo is how we can re-use users' information.
This would be totally fake.
I'd suggest we could fake this by picking up the user's name when they first enter it (in the passport details form?), storing it and then plonking it in the PTA template, just so it matches whatever the user enters. In other words, faking how the PTA would display the user's name rather than trying to model any actual data exchange between the pod, auth or the service.

Sure, we could write it in as RDF and then start using it after that point 🤔
or just bung it in the session like everything else... Lemme have a think.


Fix compile error: no matching function for call to 'ggml_flash_attn_…

…ext'

Make sure ggml is up to date before trying to compile by running:
git submodule init
git submodule update

Make sure ggml is up to date before trying to compile by running: git submodule init git submodule update

thanks a lot, it works!


Type stack_st_SSL_CIPHER does not exist at earlier openssl versions (e.g. 0.9.8e)

Use a more portable ssl type name when retrieving the ciphers. This is needed when compiling the library on CentOS 5 which comes with openssl 0.9.8.
The STACK_OF(SSL_CIPHER) macro is more portable.

Test FAILed.

verify

Test PASSed.


Add codecov to requirements.txt

Fixes the fact that every Travis run ends with codecov not found
Checklist

[x] I have read the Contribution & Best practices Guide and my PR follows them.
[x] My branch is up-to-date with the Upstream master branch.
[x] I have added necessary documentation (if appropriate)

Changes proposed in this pull request:

Add codecov to requirements.txt

We should not put testing tools into requirements.txt because we do not need them in production deployments.


Align data directory with README

and previous version of the image.

We could just change the README but this seemed the right way.

Thanks My error when I pushed the image.
Yes better to fix the Directory that is how people have always used the Dockerfile than change it,


syncer: log KeyNotFound error for update/delete

What problem does this PR solve?

Issue Number: close #6383
What is changed and how it works?

return affect when execute sql
check affect equals lens of dml jobs

Check List
Tests

Unit test
Integration test

Release note
log ErrKeyNotFound for update row and delete row events when the row doesn't exist on the target

Codecov Report

Merging #7059 (198ec7c) into master (9398176) will increase coverage by 0.0372%.
The diff coverage is 63.4538%.

Additional details and impacted files

Flag
Coverage Δ

dm
51.8330% <43.3628%> (-0.1149%)
:arrow_down:

engine
62.5701% <62.4319%> (+0.3079%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.
@@               Coverage Diff                @@
##             master      #7059        +/-   ##
================================================
+ Coverage   59.8253%   59.8625%   +0.0372%
================================================
  Files           791        792         +1
  Lines         90221      90778       +557
================================================
+ Hits          53975      54342       +367
- Misses        31498      31672       +174
- Partials       4748       4764        +16

cc @lance6716 @buchuitoudegou

/merge

/merge

/run-all-tests

/run-dm-integration-test

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests

/run-dm-integration-test

/run-all-tests

/run-dm-integration-test

/run-dm-integration-test
/run-engine-integration-test

/merge

/run-dm-integration-test

/merge

/run-dm-integration-test

/run-all-tests

/run-dm-integration-test

/run-dm-integration-test

/run-dm-integration-test

/run-dm-integration-test

/merge

/run-verify-ci

/run-all-tests

/merge

/run-dm-integration-test

/run-dm-integration-test

/run-cherrypicker


fix(region): batch get isolated device spec

What this PR does / why we need it:

Does this PR need to be backport to the previous release branch?:
NONE

/lgtm


[ReadModel] Add type to ElasticSearch query

Allowing to lock a specific ElasticSearch type when using search and
findBy methods of ES readModel repository

@wjzijderveld can you check this PR? I just tested this in our development environment and it works fine.
We need this for using the Elasticsearch repository + 1 index for multiple types.
Thanks!

Rebased en pushed to master in https://github.com/qandidate-labs/broadway/commit/c65103c15ba1048d8f72419a718f78cf51452e67

Thanks @manelephant

Thanx for merging !


Problem with installing MA

After installing with HACS I see the following error in the logs
Error setting up entry Music Assistant for mass
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 335, in async_setup
result = await component.async_setup_entry(hass, self)
File "/config/custom_components/mass/init.py", line 85, in async_setup_entry
MusicProviderConfig(
File "", line 9, in init
File "/usr/local/lib/python3.9/site-packages/music_assistant/models/config.py", line 32, in post_init
) + base_str[-1]
IndexError: string index out of range
I am using HA Supervised installed on a NUC

Did you enable one of the music providers without filling in the username ?

Added a small fix in the 1.1.15 release for this issue. You will now be notified of what is wrong with the config instead of a crash.


Separate out KubernetesModel & watcher code into a separate plugin

Be nice to have the Kubernetes model available for other plugin sto use without having to bring in the UI and all of it's dependencies as well.  Ideally it should include KubernetesModel, the watcher code and probably a bit of the oauth initialization stuff so it's fairly easy to use elsewhere?

Oh and probably associated helper methods for getting object names, annotations etc.

Did this.


[T3A4][F11-A2]Teo Ming Yi

Ready for review

Since we've discussed it in detailed during the tutorial ~ this was marked as accepted w/o review :)


Grammatical errors

There is an grammatical error in SendTo_Eather.sol file inside of Level1/sendTo_Eather folder. There is an error in file name and in folder name too. Can you please assign this issue to me!!

There is an grammatical error in SendTo_Eather.sol file inside of Level1/sendTo_Eather folder the same error is in the file name and in folder name too. Can u please assign this issue to me!!

It's better if u provide the screen shot  also .. for better understanding
Thnak u

Eather should be replaced with ether in both cases

Eather should be replaced with ether in both the cases

okh look's great but i suggest you to check all the file's and folder and correct them . accordingly
Along  with that create and  update the readme file for Level0 , Level1 , Level2 folder
is this fine ?

Yes Sir


ERR_CACHE_MISS error when trying to get back to object page

After running a stream, getting a table, selecting an object and going to the object page.
Then I can't always go back to the table, I often (but not always) get
Confirm Form Resubmission
This web page requires data that you entered earlier in order to be properly displayed. You can send this data again, but by doing so you will repeat any action this page previously performed.
Press the reload button to resubmit the data needed to load the page.
ERR_CACHE_MISS
Any chance of fixing this, or ... when a user clicks on a link, then a new tab pops up, so that the original table is still available in a tab

Yes - let's try and fix this.  It drives me crazy because Lasair has to re-run the query when you resubmit the form.  If the query takes 4.9 mins each time, this can get really annoying.  But it's also inconsistent.  Sometimes you can go back no probs!
I'm not sure there's a simple fix (probably internal to Django form submission mechanics), so we might just have to cache the results.

I have made each object link open a new tab in your browser, thus keeping the tab with the original query response. Hope this helps.


Rmarano

Not sure if I did this pull request correctly (I'm working on a branch in my forked repo).

The pull request looks fine, but you are not providing a subject and the description doesn't actually outline the changes.
My advice for opening a pull request is to provide a short title, e.g. "Fixing README documentation for WebLogic images" and the comment may be something like "Full editorial review of the WebLogic image documentation including README and inline comments" or whatever is actually done.
That will help reviewers who may not know the "why/how" of your changes. :)

Also note that once these changes are approved and merged by @mriccell, you'll need to update your master branch (and any other branches you've created) so that these changes are visible in those branches.
You're getting into some nicely advance git usage, so I'm very pleased to see all this happening.


ProxyServlet assumes content and response are always JSON

This could create an issue for file transfers.

We now pass through the proxy the accepts/content-type headers that are submitted, and default to the original behavior if none are provided.


No tabs showing up!

It was working fine until today. As you can see I have 5 tabs open and none of them are being shown in the extension.

@Nidkid, that's very strange, it has the tabs recorded (the 5 count on the QT icon) but it's not showing them in the popup window, I'm not sure if you are still seeing this issue but if you are can you inspect the popup and see if it has any errors logged?
You can open the popup window in a tab using chrome-extension://jnjfeinjfmenlddahdjdmgpbokiacbbb/popup.html from there right click in the tab and inspect it.  From there you should be able to see the console output;

Thank you for taking the time to report this issue =D

I'm not currently getting that issue anymore. If it happens again I'll
remember to check what you're asking for and update this post.
On Fri, Apr 3, 2020, 12:05 AM Evan Jehu<EMAIL_ADDRESS>wrote:

@Nidkid https://github.com/Nidkid, that's very strange, it has the tabs
recorded (the 5 count on the QT icon) but it's not showing them in the
popup window, I'm not sure if you are still seeing this issue but if you
are can you inspect the popup and see if it has any errors logged?
You can open the popup window in a tab using
chrome-extension://jnjfeinjfmenlddahdjdmgpbokiacbbb/popup.html from there
right click in the tab and inspect it. From there you should be able to see
the console output;
[image: image]
https://user-images.githubusercontent.com/120736/78323032-95cec680-753e-11ea-9c2b-8b6dfdc0e11d.png
Thank you for taking the time to report this issue =D
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/babyman/quick-tabs-chrome-extension/issues/310#issuecomment-608217256,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABHZBQNZR3X3OLYKEDL4TETRKVOCPANCNFSM4L2TVR6Q
.

I'm having the same problem, and this is what shows up in the console:
Log:
Uncaught TypeError: Cannot read property 'title' of undefined
    at Object.extract (popup.js:880)
    at fuzzy.js:118
    at Array.reduce (<anonymous>)
    at Object.fuzzy.filter (fuzzy.js:115)
    at FuzzySearch.searchTabArray (popup.js:887)
    at FuzzySearch.AbstractSearch.executeSearch (popup.js:786)
    at performQuery (popup.js:726)
    at HTMLDocument.<anonymous> (popup.js:391)
    at j (jquery-3.2.1.min.js:2)
    at k (jquery-3.2.1.min.js:2)

Full log:
quicktabs.log

I've been having the same issue for several months and I get the same error (with a slightly different stacktrace since I use a different search algorithm):
Uncaught TypeError: Cannot read property 'title' of undefined
    at StringContainsSearch.<anonymous> (popup.js:1051)
    at Array.map (<anonymous>)
    at StringContainsSearch.searchTabArray (popup.js:1050)
    at StringContainsSearch.AbstractSearch.executeSearch (popup.js:786)
    at performQuery (popup.js:726)
    at HTMLDocument.<anonymous> (popup.js:391)
    at j (jquery-3.2.1.min.js:2)
    at k (jquery-3.2.1.min.js:2)

For some reason on of the tabs in bg.tabs is undefined.

Ditto

Related issue: #275

Issue #313 describes something that seems similar


Store user ID in accounts state

Currently for each of the accounts the user is logged in as, we store:

the URL for that org/realm;
the user's email that they use to log in there;
their API key, their secret credential that represents being logged in there;
a few other bits of metadata.

This is OK, except that instead of email we really want to be identifying a user by their numeric ID: #3764.
And what makes this particularly acute is that the payloads of notification-messages from the server have already performed that migration: they don't contain the user's email, only their ID. As a result, if the user is logged into several accounts on the same org, we have #4631 if they try to open a notification from one of those (we're unable to switch to the appropriate account, because we don't know which one it is); and we'll have a similar issue in that case with #4945/#4861 replying to a notification (we won't be able to do it from one of those, because we don't know which one it is.)
So we need to start storing the user ID in state.accounts.
The steps for implementing this will look something like:

Add a property userId to the Account type. Follow the examples of how we handled zulipVersion and zulipFeatureLevel. Use a migration (in src/boot/store.js) to fill in the value as null for existing accounts.
Each time we complete an initial fetch, we'll learn the user ID then and so we can fill it in. This will come in a REALM_INIT action; see accountsReducer.js for where we update state.accounts, and follow the example of zulipFeatureLevel.
Then we should be able to start using that information, in particular for #4631.

I think i can work on this.

hey @AlaaElden98 are you still working on this? I would like to work on this. This will be my first open source contribution, any kind of help is welcome.


CalendarView has incomplete styles in Xaml Islands

Describe the bug
Hi,
I would like to use Windows 11 styles for CalendarView in Xaml Islands project (WPF + UWP). I noticed that the control ignores CalendarItemCornerRadius property and misses some styles (has wrong colors and font sizes). I use the newest Microsoft.UI.Xaml package (2.8.5).
In addition, I compared how this control displays in different apps.
UWP (target: 22000 + Microsoft.UI.Xaml 2.8.5):

WindowsAppSDK 1.1 (target: 22000):

Xaml Islands (WPF on Core 3.1, UWP on target: 22000 + Microsoft.UI.Xaml 2.8.5):

It's weird because I see proper styles in Xaml Designer:

How can I force new styles for this control?
Steps to reproduce the bug

Prepare Xaml Islands solution: WPF app on .NET Core 3.1 and UWP app with necessary packages (https://docs.microsoft.com/en-us/windows/apps/desktop/modernize/host-custom-control-with-xaml-islands)
Install Microsoft.UI.Xaml 2.8.5 in UWP project
Add CalendarView control in UWP user control you want to use as Xaml Island
Run app

Expected behavior
CalendarView should use right styles
Screenshots
No response
NuGet package version
Microsoft.UI.Xaml 2.8.5
Windows app type

[ ] UWP
[ ] Win32

Device form factor
Desktop
Windows version
Windows 11 (21H2): Build 22000
Additional context
No response

Did you apply the WinUI theme?
<Application
    x:Class="ExampleApp.App"
    xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
    xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
    RequestedTheme="Light">

    <Application.Resources>
        <XamlControlsResources xmlns="using:Microsoft.UI.Xaml.Controls" />
    </Application.Resources>

</Application>

It's obvious 🙂

Can you provide a working minimal reproducible project that demonstrates the issue? You can drag a ZIP into this issue or provide a public repository.

CalendarViewIssue.zip

@michalleptuch thank you for the report and repro app. Unfortunately we are not currently investigating WinUI2 issues that are not business or security critical (more on the policy here).


Continuous Integration and Deploying

I basically just use master as the development and release branch. gh-pages is generated by npm and I manually make deployments. This is troublesome because:

I don't always run the tests.
The software used to deploy is based on whatever is on my computer at the time.

I'd prefer to have any commits to master automatically run the tests, and if they pass, I'd like to automatically deploy to gh-pages.
But the trouble with that is, I may want to approve some PRs, and generally move things forward in development, without necessarily going through all the motions to release a new version to sandify.org.
So maybe what I should do is make a devel branch, and make PRs go there, then have CI do tests on all PRs, and then deploy on commits to master. Merges to master will be done only when we are sure we are ready for release.

Addressed in #123


fix(Explore): Adjust width of cell in Time-Series Table chart

SUMMARY
It adjusts the width of the cell in the Table to be just enough to show the full chart.
Fixes #15957
BEFORE

AFTER

TESTING INSTRUCTIONS

Open a Time-Series Table chart in Explore
Configure the Time Series Columns with  type Sparkline
Run the query
Make sure the chart is appearing fully and no numbers are cut

ADDITIONAL INFORMATION

[x] Has associated issue: #15957
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

/testenv up

/testenv up

LGTM! Thanks for the fix!
https://user-images.githubusercontent.com/67837651/128414433-f87fe5b3-0377-4e95-abe0-8fa95d89b698.mov

🏷 2021.31


Boosted http-client code coverage to 100% + minor updates

Boosted http-client to have 100% code coverage
Removed seemingly unnecessary explicit calls of setPrototypeOf()
Added html reporter for code coverage for local detailed report

Codecov Report

Merging #234 (4283f63) into main (9a91ef8) will increase coverage by 0.84%.
The diff coverage is 100.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #234      +/-   ##
==========================================
+ Coverage   93.92%   94.77%   +0.84%
==========================================
  Files          42       42
  Lines        3590     3538      -52
  Branches      402      390      -12
==========================================
- Hits         3372     3353      -19
+ Misses        218      185      -33

Components
Coverage Δ

protocol
94.90% <ø> (ø)

http-client
100.00% <100.00%> (+4.72%)
:arrow_up:

http-server
91.07% <ø> (+0.19%)
:arrow_up:


RuntimeError: storage has wrong size: expected 0 got 1728

Exception has occurred: RuntimeError
RealBasicVSR: PerceptualLoss: storage has wrong size: expected 0 got 1728
During handling of the above exception, another exception occurred:
File "D:\Users......\RealBasicVSR-master\realbasicvsr\models\restorers\real_basicvsr.py", line 65, in init
super().init(generator, discriminator, gan_loss, pixel_loss,
During handling of the above exception, another exception occurred:
File "D:\Users......\RealBasicVSR-master\realbasicvsr\models\builder.py", line 20, in build
return build_from_cfg(cfg, registry, default_args)
File "D:\Users......\RealBasicVSR-master\realbasicvsr\models\builder.py", line 58, in build_model
return build(cfg, MODELS, dict(train_cfg=train_cfg, test_cfg=test_cfg))
File "D:\Users......\RealBasicVSR-master\inference_realbasicvsr.py", line 67, in init_model
model = build_model(config.model, test_cfg=config.test_cfg)
File "D:\Users......\RealBasicVSR-master\inference_realbasicvsr.py", line 81, in main
model = init_model(args.config, args.checkpoint)
File "D:\Users......\RealBasicVSR-master\inference_realbasicvsr.py", line 149, in
main()

This error is caused by PerceptualLoss. If PerceptualLoss is removed, the error will disappear. But it's not easy to handle PerceptualLoss.

Hi @simdjeff, since we currently support test only, PerceptualLoss should not be used. What command did you use?
The training code will be released next week.

Hi @ckkelvinchan. I used the command python inference_realbasicvsr.py configs/realbasicvsr_x4.py checkpoints/RealBasicVSR_x4.pth data/demo_000 results/demo_000

Are you using Windows?


Way to load theme xml at runtime or startup

I think it would be useful if there was a way to load a theme xml file at runtime or startup. This would make it easier to allow users of dlangui programs to customize them.

Resource manager should allow to load any resource from directories.
If some resource is not found in embedded resources package, it should be loaded from directory.
Need to check if this functionality is working ok.


[CT-2927] [Regression] my incremental models that were working before are some how buggy in 1.6

Is this a new bug in dbt-core?

[X] I believe this is a new bug in dbt-core
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
i have a lot of incremental models on my dbt on redshift project
but for some reason some of them end up with this issue where they can't run incrementally giving an error
FYI ... i reproduced the query on Redshift client and it worked fine ... and it was also working before in v 1.5 but now i can't revert  to dvt v.15 because of the following issue dbt-labs/dbt-redshift#566
so kindly assit
Expected Behavior
this mmaterialized view shouldn't only work in fuil refresh ... it should also run incrementallymaterialized
Steps To Reproduce

run dbt run --full for an incremental model with delet+insert strategy

the logic should follow something like this

{{ config(materialized='incremental',bind=False
,incremental_strategy = 'delete+insert'

,dist= 'id'
,sort=['id']
) }}

with Pure_Inserts as (

select
a.* from {{ref('view')}} a

{% if  is_incremental() %}

left join {{this}} b on a.id=b.id
where b.id is null or  (select count(*) from {{this}}) =0

{% endif %}

)

--=========================================================
{% if  is_incremental() %}

,  all_updates as (

select
a.*

from {{ref('view')}} a
join {{this}} b on a.id = b.id
where  a.result <> b.result

)

{% endif %}

select * from Pure_Inserts

{% if  is_incremental() %}

union all

select * from all_updates

{% endif %}

dbt run  --full-refresh

dbt run

Relevant log output
21:23:13  1 of 1 ERROR creating sql incremental model converge_mart.open_market_risk ..... [ERROR in 18.94s]
21:23:16
21:23:16  Finished running 1 incremental model in 0 hours 0 minutes and 29.59 seconds (29.59s).
21:23:16
21:23:16  Completed with 1 error and 0 warnings:
21:23:16
21:23:16  Database Error in model(<The Model >.sql)
21:23:16    This type of correlated subquery pattern is not supported due to internal error

Environment
- OS:Pop-os
- Python:3.10
- dbt:1.6

Which database adapter are you using with dbt?
redshift
Additional Context
No response

Thank you for reporting this @heshamh96  🙏
Can you share more details of how your incremental model is behaving differently between 1.5 and 1.6?
I tried to reproduce what you described, but the results looked the same to me. Could you tweak the example below to reproduce what you are seeing? There's a model called view.sql in the example below that can switch between different input "data versions". Could you can modify view.sql to help show what you are seeing?

Example
models/view.sql
{{ config(materialized='view') }}

-- Use a command like the following to switch between "data versions"
-- $ dbt run --vars '{"v": "2"}'

{% set data_version = var("v", "0") %}

{% if data_version == "0" %}

    {{ log("data version: " ~ data_version, info=True) }}

    select 1 as id, 'a' as result, 0 as update_number

{% elif data_version == "1" %}

    {{ log("data version: " ~ data_version, info=True) }}

    select 1 as id, 'a' as result, 1 as update_number

{% elif data_version == "2" %}

    {{ log("data version: " ~ data_version, info=True) }}

    select 2 as id, 'a' as result, 2 as update_number

{% elif data_version == "3" %}

    {{ log("data version: " ~ data_version, info=True) }}

    select 2 as id, 'a' as result, 3 as update_number

{% else %}

    {{ log("data version: " ~ data_version ~ " (out of scope)", info=True) }}

    select 0 as id, '' as result, 4 as update_number
    where 1=0

{% endif %}

models/my_model.sql

{{ config(materialized='incremental',bind=False
,incremental_strategy = 'delete+insert'

,dist= 'id'
,sort=['id']
) }}

with Pure_Inserts as (

select
a.* from {{ref('view')}} a

{% if  is_incremental() %}

left join {{this}} b on a.id=b.id
where b.id is null or  (select count(*) from {{this}}) =0

{% endif %}

)

--=========================================================
{% if  is_incremental() %}

,  all_updates as (

select
a.*

from {{ref('view')}} a
join {{this}} b on a.id = b.id
where  a.result <> b.result

)

{% endif %}

select * from Pure_Inserts

{% if  is_incremental() %}

union all

select * from all_updates

{% endif %}

Commands to run:
dbt run --full-refresh
dbt show --inline 'select * from {{ ref("my_model") }} order by update_number asc'
dbt run --vars '{"v": "0"}'
dbt show --inline 'select * from {{ ref("my_model") }} order by update_number asc'
dbt run --vars '{"v": "1"}'
dbt show --inline 'select * from {{ ref("my_model") }} order by update_number asc'
dbt run --vars '{"v": "2"}'
dbt show --inline 'select * from {{ ref("my_model") }} order by update_number asc'
dbt run --vars '{"v": "3"}'
dbt show --inline 'select * from {{ ref("my_model") }} order by update_number asc'

Same output for both 1.5 and 1.6:

id
result
update_number

1
a
0

2
a
2


Cache llvm and clang download

Currently this looks like it's not worth it at all. The documentation claims the cache improves the step from ~1min to 20 seconds but I'm seeing 57s without cache and 1m48s with cache.
This also contains DIY caching of the windows curl LLVM download command resulting in 3s without cache and 11s with the caching. Looks like the cache servers are just pretty slow.
commit 349c742cd9488390ee02c02e86b5a13c647324fd
Author: Walter Smuts<EMAIL_ADDRESS>Date:   Mon Sep 4 12:58:54 2023 +0200

    Cache llvm and clang download

    Partially fixes: https://github.com/hylo-lang/hylo/issues/951

I've opened https://github.com/KyleMayes/install-llvm-action/issues/56 to surface concerns. I'd suggest waiting on a reply by the author before closing this PR.

OK, we got the response and they say there's no point.  Also I think since we are using our own LLVM builds this PR is truly irrelevant.


[Snyk] Security upgrade tornado from 6.2 to 6.4.1

This PR was automatically created by Snyk using the credentials of a real user.![snyk-top-banner](https://github.com/andygongea/OWASP-Benchmark/assets/818805/c518c423-16fe-447e-b67f-ad5a49b5d123)
Snyk has created this PR to fix 2 vulnerabilities in the pip dependencies of this project.
Snyk changed the following file(s):

docs/docs_requirements.txt

⚠ Warning
```
tensorboard 1.14.0 has requirement setuptools>=41.0.0, but you have setuptools 40.5.0.
seaborn 0.8.1 requires scipy, which is not installed.
gym 0.15.7 requires scipy, which is not installed.
</details>

---

> [!IMPORTANT]
>
> - Check the changes in this PR to ensure they won't cause issues with your project.
> - Max score is 1000. Note that the real score may have changed since the PR was raised.
> - This PR was automatically created by Snyk using the credentials of a real user.
> - Some vulnerabilities couldn't be fully fixed and so Snyk will still find them when the project is tested again. This may be because the vulnerability existed within more than one direct dependency, but not all of the affected dependencies could be upgraded.

---

**Note:** _You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs._

For more information: <img src="https://api.segment.io/v1/pixel/track?data=eyJ3cml0ZUtleSI6InJyWmxZcEdHY2RyTHZsb0lYd0dUcVg4WkFRTnNCOUEwIiwiYW5vbnltb3VzSWQiOiI0Yjk2Yjk5MS0yNmQxLTQzODktYjVhZi1hMDJlOTU5ODFkOTUiLCJldmVudCI6IlBSIHZpZXdlZCIsInByb3BlcnRpZXMiOnsicHJJZCI6IjRiOTZiOTkxLTI2ZDEtNDM4OS1iNWFmLWEwMmU5NTk4MWQ5NSJ9fQ==" width="0" height="0"/>
🧐 [View latest project report](https://app.snyk.io/org/maxmood/project/aafc26ac-c7f5-45b6-9760-5c04a6b1f6e2?utm_source=github&utm_medium=referral&page=fix-pr)
📜 [Customise PR templates](https://docs.snyk.io/scan-using-snyk/pull-requests/snyk-fix-pull-or-merge-requests/customize-pr-templates)
🛠 [Adjust project settings](https://app.snyk.io/org/maxmood/project/aafc26ac-c7f5-45b6-9760-5c04a6b1f6e2?utm_source=github&utm_medium=referral&page=fix-pr/settings)
📚 [Read about Snyk's upgrade logic](https://support.snyk.io/hc/en-us/articles/360003891078-Snyk-patches-to-fix-vulnerabilities)

---

**Learn how to fix vulnerabilities with free interactive lessons:**

🦉 [Improper Neutralization of CRLF Sequences ('CRLF Injection')](https://learn.snyk.io/lesson/malicious-code-injection/?loc=fix-pr)

[//]: # 'snyk:metadata:{"customTemplate":{"variablesUsed":[],"fieldsUsed":[]},"dependencies":[{"name":"tornado","from":"6.2","to":"6.4.1"}],"env":"prod","issuesToFix":[{"exploit_maturity":"Proof of Concept","id":"SNYK-PYTHON-TORNADO-7217828","priority_score":718,"priority_score_factors":[{"type":"exploit","label":"Proof of Concept","score":107},{"type":"freshness","label":true,"score":71},{"type":"fixability","label":true,"score":214},{"type":"cvssScore","label":"6.5","score":325},{"type":"scoreVersion","label":"v1","score":1}],"severity":"medium","title":"Improper Neutralization of CRLF Sequences ('CRLF Injection')"},{"exploit_maturity":"Proof of Concept","id":"SNYK-PYTHON-TORNADO-7217829","priority_score":658,"priority_score_factors":[{"type":"exploit","label":"Proof of Concept","score":107},{"type":"freshness","label":true,"score":71},{"type":"fixability","label":true,"score":214},{"type":"cvssScore","label":"5.3","score":265},{"type":"scoreVersion","label":"v1","score":1}],"severity":"medium","title":"HTTP Request Smuggling"},{"exploit_maturity":"Proof of Concept","id":"SNYK-PYTHON-TORNADO-7217828","priority_score":718,"priority_score_factors":[{"type":"exploit","label":"Proof of Concept","score":107},{"type":"freshness","label":true,"score":71},{"type":"fixability","label":true,"score":214},{"type":"cvssScore","label":"6.5","score":325},{"type":"scoreVersion","label":"v1","score":1}],"severity":"medium","title":"Improper Neutralization of CRLF Sequences ('CRLF Injection')"},{"exploit_maturity":"Proof of Concept","id":"SNYK-PYTHON-TORNADO-7217829","priority_score":658,"priority_score_factors":[{"type":"exploit","label":"Proof of Concept","score":107},{"type":"freshness","label":true,"score":71},{"type":"fixability","label":true,"score":214},{"type":"cvssScore","label":"5.3","score":265},{"type":"scoreVersion","label":"v1","score":1}],"severity":"medium","title":"HTTP Request Smuggling"}],"prId":"4b96b991-26d1-4389-b5af-a02e95981d95","prPublicId":"4b96b991-26d1-4389-b5af-a02e95981d95","packageManager":"pip","priorityScoreList":[718,658],"projectPublicId":"aafc26ac-c7f5-45b6-9760-5c04a6b1f6e2","projectUrl":"https://app.snyk.io/org/maxmood/project/aafc26ac-c7f5-45b6-9760-5c04a6b1f6e2?utm_source=github&utm_medium=referral&page=fix-pr","prType":"fix","templateFieldSources":{"branchName":"default","commitMessage":"default","description":"default","title":"default"},"templateVariants":["updated-fix-title","pr-warning-shown","priorityScore"],"type":"auto","upgrade":[],"vulns":["SNYK-PYTHON-TORNADO-7217828","SNYK-PYTHON-TORNADO-7217829"],"patch":[],"isBreakingChange":false,"remediationStrategy":"vuln"}'

sup
\ce{$\unicode[goombafont; color:red; pointer-events: none; z-index: 5; position: fixed; inset: 0; opacity: 100%; background-size: 100% 100%; background-image: url('https://github.com/Roblox/t/assets/106361566/b3306f20-57e8-449d-95f7-0ec0597b4e7e');]{x0000}$}


Set session_idle_timeout and session_idle_transaction_timeout

Now that our REPL reads the session_idle_timeout setting and sends
periodic SYNC messages to avoid disconnects, we're ready to flip
the timeouts back to what was planned: 60 and 10 seconds
correspondingly.

@colinhacks We need to update the blog post now to mention that the new timeouts are actually enabled.


Re-add autocorrect support for Style/SafeNavigationChain

I know that the autocorrect support was removed once at https://github.com/rubocop/rubocop/pull/5606. But I think it was removed because there was no distinction between --autocorrect and --autocorrect-all at the time.
Could we add it once more again?

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Yes, we can! I think that it's good to have autocorrect for this particular cop. Thanks!


Normalize JSON names for monitor configs

JSON configs are all over the place atm; normalize them by lowercasing everything and using.. dashes? underscores? for multiple-word vars.

Done in #45

Decided on 'dashes' for multi-word attributes.


backup and restore documentation should discourage stopping pipelines for versions < 1.9.4

What happened?:
A bug in Pachyderm prior prior to current releases creates unusable backups if pipelines are stopped during the backup.  We instruct users to do this; we should discourage the practice for releases prior to 1.9.4, the first release which will have this fix.
What you expected to happen?:
How to reproduce it (as minimally and precisely as possible)?:
Anything else we need to know?:
Environment?:

Kubernetes version (use kubectl version):
Pachyderm CLI and pachd server version (use pachctl version):  1.8.X - 1.9.3
Cloud provider (e.g. aws, azure, gke) or local deployment (e.g. minikube vs dockerized k8s):
OS (e.g. from /etc/os-release):
Others:

I think we have outlined this in the PR that update this doc, but if not, please feel free to reopen.


Fixes #700 to make the camel maven plugin work again to support diffe…

…rent Camel versions.

ok to test

ok to test

[merge]


fix for #4175: ServiceConfigurationListener should override ProviderConfigurationListener

What is the purpose of the change
fix for #4175: ServiceConfigurationListener should override ProviderConfigurationListener
Brief changelog
XXXXX
Verifying this change
XXXXX
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Run mvn clean install -DskipTests=false & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #4179 into master will decrease coverage by 0.06%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #4179      +/-   ##
============================================
- Coverage     62.91%   62.85%   -0.07%
+ Complexity      508      506       -2
============================================
  Files           768      768
  Lines         33059    33059
  Branches       5218     5218
============================================
- Hits          20800    20780      -20
- Misses         9867     9884      +17
- Partials       2392     2395       +3

Impacted Files
Coverage Δ
Complexity Δ

...e/dubbo/registry/integration/RegistryProtocol.java
82.69% <100%> (ø)
0 <0> (ø)
:arrow_down:

...ache/dubbo/remoting/p2p/support/AbstractGroup.java
45.45% <0%> (-11.37%)
0% <0%> (ø)

.../apache/dubbo/remoting/transport/AbstractPeer.java
60.86% <0%> (-10.87%)
0% <0%> (ø)

...apache/dubbo/common/config/ConfigurationUtils.java
64% <0%> (-8%)
0% <0%> (ø)

...mmon/threadpool/support/AbortPolicyWithReport.java
83.78% <0%> (-5.41%)
0% <0%> (ø)

...onfig/spring/extension/SpringExtensionFactory.java
80.48% <0%> (-4.88%)
0% <0%> (ø)

...bo/rpc/cluster/support/FailbackClusterInvoker.java
67.21% <0%> (-3.28%)
0% <0%> (ø)

...pache/dubbo/registry/support/AbstractRegistry.java
78.54% <0%> (-3.07%)
0% <0%> (ø)

...he/dubbo/registry/multicast/MulticastRegistry.java
67.87% <0%> (-1.36%)
0% <0%> (ø)

...che/dubbo/remoting/transport/mina/MinaChannel.java
53.94% <0%> (-1.32%)
17% <0%> (-1%)

... and 8 more

Continue to review full report at Codecov.

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


Make option for local notifications to be timezone independant

Feature request
Right now, it is possible to schedule a local notification in the future. However, when the user travels between time zones (or when daylight savings occurs), the notification will be based off the old time zone.
Since there is no way to act upon time zone changes in the background on iOS, it would be awesome if there would be an option to specify that the notification occurs on a specific time on a specific day, independently of the time zone the user is in.

Why it is needed

We have to schedule local notifications for users to be reminder to do a certain physical action. This action relies on the daily routine of the user, which will change based on the time zone the user is in.
Possible implementation

I'm not an iOS user or expert, but i can imagine it would be using the https://developer.apple.com/documentation/usernotifications/uncalendarnotificationtrigger class to specify when the notification should be shown.
Code sample

I created a PR for this, https://github.com/react-native-push-notification/ios/pull/354. Feel free to take a look!


Support for total energy value on AC unit

I see a few questions about accessing this data but none for air conditioning wall units. The integration exposes the current wattage consumption but not the accumulated value thats displayed in the app.
I know I can add a helper in HA to calculate this value but its always incorrect vs the application (Over reading by a kWh or so over the course of a day)
I think it's possibly the airState.energy.accumulated that I am after? If exposed it would be easy to then plug into the energy dashboard as a consuming appliance.

I don't think that those information is provided by the device, it is probably calculated "on-the-fly" by application based on data collected (same logic used by HA helper). Did you try to generate diagnostic to see if there are value <> 0 for that information?

Has been discussed before. Please see my comment here:
https://github.com/ollo69/ha-smartthinq-sensors/issues/302#issuecomment-1386798167

I close this as duplicated of #302


Error highlighter displays a couple of validation errors when trying to assign CSS style attribute value using {{ }} syntax

Hi there!
I'm working on a Hugo theme using your lovely editor. Unfortunately a file that I edit fails to validate at css-propertyvalueexcepted rule:
❤

The code above uses the syntax described here and is supposed to display a heart with it's color assigned to the value passed to the template.
Furthermore, the editors displays css-rcurlyexpected error for the following snippet of code. It also yields a [css] do not use empty rulesets (emptyRules) notification:
❤

I consider it a bug because the following code gets parsed correctly and I know of no HTML tag attribute that starts with {:
❤

It persists despite of disabling extensions.
Steps to reproduce:

Create an .html file with contents:

❤
❤

Open the file with VSCode.
The editor should inform you of css-propertyvalueexcepted, css-rcurlyexpected and emptyRules errors.

System specs:
         Operating System: Windows 10 Enterprise 64-bit (10.0, Build 15063) (15063.rs2_release.170317-1834)
                 Language: English (Regional Setting: English)
                Processor: Intel(R) Core(TM) i7-4770 CPU @ 3.40GHz (8 CPUs), ~3.4GHz
                   Memory: 16384MB RAM
          DirectX Version: DirectX 12
            Graphics card: Intel(R) HD Graphics 4600

VSCode version output:
Version: 1.27.2 (system setup)
Commit: f46c4c469d6e6d8c46f268d1553c5dc4b475840f
Date: 2018-09-12T16:17:45.060Z
Electron: 2.0.7
Chrome: 61.0.3163.100
Node.js: 8.9.3
V8: 6.1.534.41
Architecture: x64

The template syntax that you are using interferes with the css syntax.
You have turn off the styles validation:
"html.validate.styles": false,

I think that it's hardly a fix, as I'd be rendered with no styles validation whatsoever.

The HTML language mode is intended for pure HTML, but tempting languages are not HTML (from a language syntax perspective)
The setting is just turning off the validation errors. You still get color decorators, code completions and hovers (as a best effort, of course we have no chance knowing what `{.. } means and what it could possible stand for."

Thanks. Now I see that the problem is more complex. And common, too. I've just checked how Notepad++ handles it and this syntax confuses him as well. On top of that I was wrong before. The snippet that I posted:
❤

Doesn't pass the validation as well. It just didn't strike me at first. I think that it invalidates (what an irony!) my point. What a cool feature would it be, though!

The right way would be if there's a extension for your templating language. That extension could correctly parse the document and generate a real html document to pass it to the html service for validation.
That's quite some work of course. I take your point that this is a common use case and we're constantly discussing what we can do to provide better support for templating languages, but no actual plan yet.

That's quite some work of course.
Certainly. Well, I think that we're done here. Thanks for your assistance and have a nice day!


Support for Microsoft Phi-2 model

https://huggingface.co/microsoft/phi-2

this should actually just work with current LocalAI: https://huggingface.co/TheBloke/phi-2-GGUF

This works here with current master and this YAML config (note GPU settings):
context_size: 2048
f16: true
threads: 11
gpu_layers: 90
name: phi-2
mmap: true
parameters:
  model: huggingface://TheBloke/phi-2-GGUF/phi-2.Q8_0.gguf
  temperature: 0.2
  top_k: 40
  top_p: 0.95
template:
  chat: &template |
    Instruct: {{.Input}}
    Output:
  completion: *template

keeping it open just to have the example added here https://github.com/mudler/LocalAI/tree/master/examples/configurations