[LUIS] Authoring client context has incorrect base URI, so requests fail

Package Name: @azure/cognitiveservices-luis-authoring
Package Version: 2.1.0
Operating system: Windows 10 Enterprise 1803
[x ] nodejs

version:  v10.16.0

[ ] browser

name/version:

[ ] typescript

version:

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[ ] SDK API docs on https://docs.microsoft.com

Describe the bug
LUISAuthoringClientContext (https://github.com/Azure/azure-sdk-for-js/blob/master/sdk/cognitiveservices/cognitiveservices-luis-authoring/src/lUISAuthoringClientContext.ts) does not set the base URI correctly (see line 46), so calls to the service fail.
To Reproduce
Steps to reproduce the behavior:

Set environment variables LUIS_SUBSCRIPTION_KEY and LUIS_REGION to a valid LUIS subscription key and the corresponding region, respectively.
Install @azure/ms-rest-js and @azure/cognitiveservices-luis-authoring.
Run the following code with Node.

"use strict";

var msRest = require("@azure/ms-rest-js");
var LUIS = require("@azure/cognitiveservices-luis-authoring");

const key_var = 'LUIS_SUBSCRIPTION_KEY';
if (!process.env[key_var]) {
    throw new Error('please set/export the following environment variable: ' + key_var);
}
const subscription_key = process.env[key_var];

const region_var = 'LUIS_REGION';
if (!process.env[region_var]) {
    throw new Error('please set/export the following environment variable: ' + region_var);
}
const region = process.env[region_var];
const endpoint = 'https://' + region + '.api.cognitive.microsoft.com/';

const creds = new msRest.ApiKeyCredentials({ inHeader: { 'Ocp-Apim-Subscription-Key': subscription_key } });
const client = new LUIS.LUISAuthoringClient(endpoint, creds);

function list_apps() {
    client.apps.list().then((result) => {
        for (let x of result) {
            console.log(x.id);
        }
    }).catch(error => {
        throw error;
    });
}

try {
    list_apps();
}
catch (error) {
    console.log(error);
}

Output:
(node:12164) UnhandledPromiseRejectionWarning: Error: Resource not found
    at new RestError (C:\Users\<user>\node_modules\@azure\cognitiveservices-luis-authoring\node_modules\@azure\ms-rest-js\dist\msRest.node.js:1399:28)
    at C:\Users\<user>\node_modules\@azure\cognitiveservices-luis-authoring\node_modules\@azure\ms-rest-js\dist\msRest.node.js:2494:37
    at process._tickCallback (internal/process/next_tick.js:68:7)
(node:12164) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 3)
(node:12164) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Expected behavior
Example output:
4d468077-482f-409f-954f-102b2256ffa1
86473c85-dc34-4361-b74c-909a5d18ebe5
Additional context
LUISAuthoringClientContext sets the base URI as follows:
_this.baseUri = "{Endpoint}";

Instead, it should set it as follows (for example):
_this.baseUri = "{Endpoint}/luis/api/v2.0";

By making this change to my local copy of<EMAIL_ADDRESS>my code works as expected.
A debug statement placed in my local copy of C:\Users<user>\node_modules<EMAIL_ADDRESS>shows that before the fix, the URL used for the list request is:
https://westus.api.cognitive.microsoft.com/apps/?skip=0&take=100
which is wrong. After the fix, the request is:
https://westus.api.cognitive.microsoft.com/luis/api/v2.0/apps/?skip=0&take=100
which is correct.
PLEASE NOTE that the runtime client context is currently set to LUIS version 3.0 preview! See https://github.com/Azure/azure-sdk-for-js/blob/master/sdk/cognitiveservices/cognitiveservices-luis-runtime/src/lUISRuntimeClientContext.ts line 46:
this.baseUri = "{Endpoint}/luis/v3.0-preview";

This might influence whether you want to set the authoring client context to version 2.0 or version 3.0-preview. If the versions do not match, it won't be possible to write a unified sample using the SDK (for example, one that creates an application, which uses the authoring client, then queries the application for a prediction, which uses the runtime client). That is because version 3.0-preview methods do not see applications created with version 2.0. I will write this up in more detail in a separate issue.
Please let me know if I can provide any additional information.

Thanks for bringing this issue to our attention. We have routed to the appropriate team for follow-up.

@ahmedabuamra As discussed offline in one of our emails, regeneration of the library using the latest code generator should fix the problem with the base uri. Please consider re-generating the library at the earliest.

@v-jaswel Currently you are setting up the endpoint as below
const endpoint = 'https://' + region + '.api.cognitive.microsoft.com/';
Can you try setting the endpoint as below and let me know if that helps at all?
const endpoint = 'https://' + region + '.api.cognitive.microsoft.com/luis/api/v2.0';

This should be fixed with the latest version of the luis libraries.
Please give them a try

cc @ahmedabuamra


Blink controls and aframe upgrade

the 2 demos work with both cursor teleport in desktop mode and blink controls in VR mode
when entering VR mode, there is still a white circle remaining from the cursor teleport, I did not try disabling this
in the quest headset browser, when NOT in VR mode, cursor teleport circle is visible and responds to raycast position changes, but does not activate upon pressing trigger. I think that's ok but could be debugged in the future. I could create a ticket for it.
there were some npm install problems from old packages, but I did not commit these changes. I can if helpful or we could 'rebase' this component from a newer a-frame webpack boilerplate repo

Thank you! I'll update the webpack stuff and make the release.

when entering VR mode, there is still a white circle remaining from the cursor teleport, I did not try disabling this

I think you can create an issue for this one. We probably need to hide the cursor on enter-vr.

in the quest headset browser, when NOT in VR mode, cursor teleport circle is visible and responds to raycast position changes, but does not activate upon pressing trigger. I think that's ok but could be debugged in the future. I could create a ticket for it.

I don't think this is common to use a headset and use non VR mode, so I'll probably categorize that as wontfix. Although if you have a fix for that, you can always create a PR.

I created #51


QueryAsync<> does not return all the rows as expected on UWP

Hi,
I have 4 tables linked one with the others as is:

GedtDbDoc as an Id column
GedtDbComponent as an Id columnn and a DocId column that reference an Id in the GedtDbDoc table
GedtDbFsItem as an Id column and a ClassificationCode column
GedtDbRelFsDoc is a N to N relation table that links some rows of table GedtDbDoc to some others rows of GedtDbFsItem. It contains a DocId column and an FsId column.

I have two ways to get the results I want (through two different pages in my Xamarin.Forms app), but the result should be the same.
The first page construct such query:
SELECT DISTINCT GedtDbComponent.*  FROM GedtDbComponent
	WHERE 1=1
	AND GedtDbComponent.DocId IN
		(SELECT DocId FROM GedtDbRelFSDoc WHERE FsId IN
			(SELECT Id FROM GedtDbFSItem WHERE 0=1 OR ClassificationCode="PID"))

When I run these two queries in DB Browser, I get exactly the same result, no problem.
Now the two queries are run in the same place in my program with such code:
        List<GedtDbComponent> result = await database.QueryAsync<GedtDbComponent>(query);

When I run the first one with tis code, I get only 102 rows on the 462 I should get
But when I run the second one, I get all my 462 rows as expected.
So my question: why the first query does no return my 462 rows as expected when I run it using QueryAsync ?
(I did not have time to debug in Android plateform, but it seems that it works on this platform for both queries)
Regards.

I forgot to says that I installed the last version to be sure to get all potential issues corrected (1.8.116)

Hello, this is not valid sqlite code. Double quotes are for identifiers. You want single quotes around PID. https://www.sqlite.org/lang_keywords.html
Also, you should use aliases when tables you're accessing have columns with identical names.
I recommend using query's parameter syntax and argument passing instead of hard-coding values.

Hi,
Thank you for investigation. For double quotes, I should have the problem on the second query also, but the second works even with double quote...now, if correct syntax is with single quote, I will do the replacement...
So do you mind that this sql should work...
SELECT DISTINCT Comp.* FROM GedtDbComponent Comp
JOIN GedtDbDoc Doc ON Doc.Id = Comp.DocId
JOIN GedtDbRelFSDoc Rel ON Rel.DocId=Doc.Id
JOIN GedtDbFSItem FS ON Rel.FsId=FS.Id
WHERE 1=1  AND FS.ClassificationCode='?'
and pass the value PID as argument ?

Hi again,
So I implemented what you suggested. Just before the call to queryasync, this is the content of my query string:
string query = SELECT DISTINCT Comp.* FROM GedtDbComponent Comp JOIN GedtDbDoc Doc ON Doc.Id = Comp.DocId JOIN GedtDbRelFSDoc RelFs ON RelFs.DocId=Doc.Id JOIN GedtDbFSItem Fs ON RelFs.FsId=Fs.Id  WHERE 1=1  AND Fs.ClassificationCode=?
string param="PID"
List result = await database.QueryAsync(query, param);
BUT it does not work better. The result is not the same as the result of the following query that is supposed to return the same set of data :
SELECT DISTINCT Comp.* FROM GedtDbComponent  Comp
WHERE 1=1
AND Comp.DocId IN
(SELECT DocId FROM GedtDbRelFSDoc WHERE FsId IN
(SELECT Id FROM GedtDbFSItem WHERE 0=1 OR ClassificationCode=?))
The result is the same when I run both in the desktop tool DB Browser (SQ Lite)
So please can you have a look at this issue?


Update/ Add the action which creates the CF stack compatibility matrix

The KubeRay line in the compatibility matrix doesn't update. This needs to be fixed.

@ChughShilpa the readme compatibility matrix doesn't get updated when we create a new tag (here)[https://github.com/opendatahub-io/kuberay/tags] this should be updated via an action preferably

@ChughShilpa can this issue be clodes?

it seems the latest run did not update the Kuberay version correctly: https://github.com/project-codeflare/codeflare-operator/pull/407/files#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5R15.

Good catch, the parameter is missing in https://github.com/project-codeflare/codeflare-operator/blob/main/.github/workflows/project-codeflare-release.yml

Fix ready ^


Create Story for the Button component

Create a story .ts file with Inline Button spec

Hi.
This will be my very first real pr.
I would l ike to be assigned this project

Hi there, @JIVIN0902 !
That's really nice. What kind of task do you want to tackle? This one of creating a Storybook or want something else?

If this is complicated then something beginner friendly would be nice.
Thanks

Nice, I'm working on Open Source on Fridays. So I'll think about until this Friday and I will open an issue for you then.
C ya!

Cam I work on this issue? Please assign it to me and explain the final result so that I can work on it.

Yep!
The idea is to create the button component on the storybook. For that, take a look at the components under stories folder and follow the same ideia.

@tiagostutz Should I add the properties of text decoration and others for ButtonPrimary.ts in components? Also, do I need to export EditModes of = 'edit' | 'display' | 'readOnlyEdit' | 'readOnlyDisplay'; just like the inline text component?

Yep, I think it makes sense to have those properties.


Add instructions for installing kubeadm in Flatcar Linux

Flatcar Linux (https://www.flatcar-linux.org/) is the de-facto replacement for the old CoreOS Container Linux.
Although kubeadm instruct how to install it in CoreOS CL,  it seems that with CoreOS reaching EoL the install instructions were not migrated to Flatcar and simply removed.
So, this PR is to re-add the instructions for Flatcar Linux, as there's been some migration movement from companies using CoreOS to Flatcar and as nice as this distribution is, IMO this could and should still exist in kubeadm instructions.
/cc @neolit123

/sig cluster-lifecycle

Sure. I don't know if /opt in Fedora CoreOS is RW also, will test ASAP.
I've been thinking also as the flex-volume issue being recurrent in those distros (because of /usr being RO) point to the troubleshoot item that deals with this

Sure. I don't know if /opt in Fedora CoreOS is RW also, will test ASAP.

i think it was. if not, we could just extend the note to clarify that the download location should be RW:

Define the directory to download command files. Make sure it is writeable.

also users can just pick another location.

I've been thinking also as the flex-volume issue being recurrent in those distros (because of /usr being RO) point to the troubleshoot item that deals with this

SGTM. possibly at the bottom of the tab is best.

About Fedora CoreOS using /usr, moving to /opt generates conflicts with SELinux that is enabled by default in it :man_shrugging: (and then kubelet does not start)
To simplify I'll write a comment as suggested that the DOWNLOAD_DIR should be a writable directory and move forward.

/assign @kbhawkey

Page preview:
https://deploy-preview-23435--kubernetes-io-master-staging.netlify.app/docs/setup/production-environment/tools/kubeadm/install-kubeadm/

@kbhawkey Thank you for the review!
I've changed the texts as proposed, and already rebased/squashed the commits.

/approve


Post-execution checks for v3

This change is

We're currently processing your upload.  This comment will be updated when the results are available.


How to change embedding vector size ?

The following code snipped generates an embedding with shape  (1,512).  How can I change it to 1*128 ?
 img = cv2.imread('sample.jpg')
embedded = embedder.embeddings([img])
embedded.shape
Outout: (1,512)

The embedding size is dependent on which model you use. I assume that you used embedder = FaceNet(), which means the default model key (20180402-114759) was used. This model has embedding size 512.
If you want an embedding size of 128, you should use embedder = FaceNet(key='20170511-185253') or embedder = FaceNet(key='20170512-110547'). See the dimensions parameter for each model in keras-facenet/metadata.py for more information.
Please note that the distance metric also varies depending on the model. You can get the model distance metric from embedder.metadata['distance_metric'].
Hope this helps!

Closing this issue for now. If you have trouble, please feel free to comment back and we can re-open.


Delete Optimizable

It removes depndency of "hyperopt"

I believe that it's up to users to find and use appropriate optimization tools.

@lsh918 별도의 PR로 나눠서 제거할 계획입니다


WAF: Policy Operator bigger or Equal failed to save

When i try to edit an existing (or new) Policy it is not possible to save the changes when the Operator field is on Bigger or Equal, Bigger, Lesser, Lesser or Equal. Only with the option Equal it is possible to save.
Restart failed with error message:
2019/02/08 09:16:00 [emerg] 83332#100890: Naxsi-Config : Incorrect line CheckRule $policy676b2e8f148c4b4abc6be8ed410be138 = 8 (/usr/obj/usr/ports/www/nginx/work/naxsi-0.56/naxsi_src/naxsi_skeleton.c/646)... in /usr/local/etc/nginx/nginx.conf:1401

Line 1401 in nginx.conf:
CheckRule "$policy676b2e8f148c4b4abc6be8ed410be138 = 8" ALLOW;

As workaround i comment the failed line and start nginx manually.

@fichtner this is a core bug as well. My dropdown key contains a "<" as well as a ">", which are escaped here:
https://github.com/opnsense/core/blob/master/src/opnsense/mvc/app/controllers/OPNsense/Base/ApiControllerBase.php#L235
And afterwards assigned here:
https://github.com/opnsense/core/blob/master/src/opnsense/www/js/opnsense.js#L140
The opnsense.js has to unescape it or the escaping must be removed entirely.

@xarados this will be a breaking change - "=" is not supported by naxsi so I made an error while developing it. Please delete all "=" entries, apply both pull requests and then everything should work as expected.

@fabianfrz works like a charm.

the plugin side is done, the rest is core.


Add test cases for query expressions [1.2.x]

Purpose

To cover additional test scenarios identified for query expressions.

Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[x] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #22900 into ballerina-1.2.x will not change coverage.
The diff coverage is n/a.

@@               Coverage Diff                @@
##           ballerina-1.2.x   #22900   +/-   ##
================================================
  Coverage            14.59%   14.59%
================================================
  Files                   51       51
  Lines                 1411     1411
  Branches               219      219
================================================
  Hits                   206      206
  Misses                1189     1189
  Partials                16       16

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 6008bb0...6c17d9f. Read the comment docs.


[FEATURE] Option to automatically summarise context window

Feature Area
Core functionality
Is your feature request related to a an existing bug? Please link it here.
https://github.com/crewAIInc/crewAI/discussions/1111
Describe the solution you'd like
https://github.com/crewAIInc/crewAI/blob/8119edb4957745a4db26beda9dcf06b34742c90d/src/crewai/agents/executor.py#L369
The current implementation with summarising context window requires the user to say whether the context window should be summarised.
Ideally we could set an option for this to automatically be done so CrewAI can be ran from start to finish without requiring user input.
Describe alternatives you've considered
No response
Additional context
No response
Willingness to Contribute
Yes, I'd be happy to submit a pull request

I need this!!


Cross-compilation of ARM bare-metal samples broken due to #7721

After updating the IREE submodule in https://github.com/iml130/iree-bare-metal-arm to https://github.com/google/iree/commit/658cf6c8a48d155259f3bb08a536cc47f1d50b78 or newer, the static library example is broken.
When cross-compiling the static library example for ARMv7em, the linkage fails with the following error:
[110/110] Linking C executable samples/sample_static_library
FAILED: samples/sample_static_library
: && /usr/local/gcc-arm-none-eabi-10-2020-q4-major/bin/arm-none-eabi-gcc -mthumb -march=armv7e-m -mfloat-abi=hard -mfpu=fpv4-sp-d16 -DIREE_TIME_NOW_FN="{ return 0; }"   -O3 -DNDEBUG -lc -lm -lnosys -T /home/admin-mbrehler/repos/iree-bare-metal-arm/build_tools/stm32f407-cmsis.ld  -specs=nosys.specs    -lm samples/CMakeFiles/sample_static_library.dir/static_library_demo.c.obj -o samples/sample_static_library -L/home/admin-mbrehler/repos/iree-bare-metal-arm/build_tools samples/libsimple_mul_c.a  third_party/iree/iree/hal/local/loaders/libiree_hal_local_loaders_static_library_loader.a  third_party/iree/iree/hal/local/libiree_hal_local_sync_driver.a  samples/libsimple_mul.a  build_tools/third_party/cmsis_device_f4/libcmsis_device_f4.a  third_party/iree/iree/runtime/libiree_runtime_impl.a  third_party/iree/iree/base/internal/libiree_base_internal_file_io.a  third_party/iree/iree/hal/drivers/libiree_hal_drivers_drivers.a  third_party/iree/iree/hal/dylib/registration/libiree_hal_dylib_registration_sync.a  third_party/iree/iree/hal/local/loaders/libiree_hal_local_loaders_embedded_library_loader.a  third_party/iree/iree/hal/local/elf/libiree_hal_local_elf_elf_module.a  third_party/iree/iree/hal/local/elf/libiree_hal_local_elf_arch.a  third_party/iree/iree/hal/local/elf/libiree_hal_local_elf_platform.a  third_party/iree/iree/hal/vmvx/registration/libiree_hal_vmvx_registration_sync.a  third_party/iree/iree/hal/local/libiree_hal_local_sync_driver.a  third_party/iree/iree/base/internal/libiree_base_internal_arena.a  third_party/iree/iree/base/internal/libiree_base_internal_atomic_slist.a  third_party/iree/iree/hal/local/loaders/libiree_hal_local_loaders_vmvx_module_loader.a  third_party/iree/iree/modules/vmvx/libiree_modules_vmvx_vmvx.a  third_party/iree/iree/modules/hal/libiree_modules_hal_hal.a  third_party/iree/iree/vm/libiree_vm_bytecode_module.a  third_party/iree/iree/vm/libiree_vm_impl.a  third_party/iree/build_tools/third_party/flatcc/libflatcc_parsing.a  -lm  third_party/iree/iree/hal/local/libiree_hal_local_local.a  third_party/iree/iree/hal/libiree_hal_hal.a  third_party/iree/iree/base/internal/libiree_base_internal_synchronization.a  third_party/iree/iree/base/internal/libiree_base_internal_fpu_state.a  third_party/iree/iree/base/libiree_base_base.a && :
/usr/local/gcc-arm-none-eabi-10-2020-q4-major/bin/../lib/gcc/arm-none-eabi/10.2.1/../../../../arm-none-eabi/bin/ld: /usr/local/gcc-arm-none-eabi-10-2020-q4-major/bin/../lib/gcc/arm-none-eabi/10.2.1/../../../../arm-none-eabi/lib/thumb/v7e-m+fp/hard/libm.a(lib_a-math_errf.o): in function `__math_oflowf':
math_errf.c:(.text.__math_oflowf+0x0): multiple definition of `__math_oflowf'; samples/libsimple_mul.a(simple_mul.o):simple_mul_dispatch_0:(.text.__math_oflowf+0x0): first defined here
/usr/local/gcc-arm-none-eabi-10-2020-q4-major/bin/../lib/gcc/arm-none-eabi/10.2.1/../../../../arm-none-eabi/bin/ld: /usr/local/gcc-arm-none-eabi-10-2020-q4-major/bin/../lib/gcc/arm-none-eabi/10.2.1/../../../../arm-none-eabi/lib/thumb/v7e-m+fp/hard/libm.a(lib_a-math_errf.o): in function `__math_invalidf':
math_errf.c:(.text.__math_invalidf+0x0): multiple definition of `__math_invalidf'; samples/libsimple_mul.a(simple_mul.o):simple_mul_dispatch_0:(.text.__math_invalidf+0x0): first defined here
collect2: error: ld returned 1 exit status
ninja: build stopped: subcommand failed.

To Reproduce
Steps to reproduce the behavior:
Prior to compile and install the host build execute:
cd third_party/iree
git pull
git checkout 658cf6c8a48d155259f3bb08a536cc47f1d50b78
git submodule update
cd ../../

Follow the instructions in https://github.com/iml130/iree-bare-metal-arm#host-build for the host build afterwards.
Following the description in https://github.com/iml130/iree-bare-metal-arm#target-build and building
cmake --build . --target sample_static_library

fails with the error posted above. Any help is appreciated!
Side note
Building the embedded dynamic library example via
cmake --build . --target sample_embedded_sync

now fails with a much more expressive error message which is great :D This is expected as __aeabi_fmulis defined in libgcc.
FAILED: samples/simple_embedding_test_module_dylib_arm_32.vmfb
cd /home/admin-mbrehler/repos/iree-bare-metal-arm/build-cmsis/samples && /home/admin-mbrehler/repos/iree-bare-metal-arm/build-iree-host-install/bin/iree-translate -iree-input-type=mhlo -iree-mlir-to-vm-bytecode-module -iree-hal-target-backends=dylib-llvm-aot -iree-llvm-target-triple=armv7em-pc-linux-elf -iree-llvm-target-float-abi=hard -iree-llvm-debug-symbols=false -iree-vm-bytecode-module-strip-source-map=true -iree-vm-emit-polyglot-zip=false /home/admin-mbrehler/repos/iree-bare-metal-arm/third_party/iree/iree/samples/simple_embedding/simple_embedding_test.mlir -o simple_embedding_test_module_dylib_arm_32.vmfb
lld: error: undefined symbol: __aeabi_fmul
>>> referenced by simple_embedding_test.mlir:3 (/home/admin-mbrehler/repos/iree-bare-metal-arm/third_party/iree/iree/samples/simple_embedding/simple_embedding_test.mlir:3)
>>>               /tmp/simple_mul_dispatch_0-7886fe.o:(.Lsimple_mul_dispatch_0)
Linking failed; escaped command line returned exit code 256:

We can try marking those symbols as weak. If we don't link in the builtin libraries then the hosting app will have to compile and link in the libraries themselves for anything not covered by the toolchain. Finding a way to make this happy with the current path will be the most robust way forward - likely requires fiddling with the visibility/linkage.
Here's where we set the linkage today:
https://github.com/google/iree/blob/a80811c5713b513448dca6f3134d597101ed9136/iree/compiler/Dialect/HAL/Target/LLVM/LLVMAOTTarget.cpp#L373-L383
We could vary that based on whether we are linkStatic or not (in the linkStatic case make things weak odr or weak, etc).

@dajuro I would be happy if you could take a look till I am back in office.

My first tests indicate that linking musl causes the problem. If you comment out the following code it works:
https://github.com/google/iree/blob/a80811c5713b513448dca6f3134d597101ed9136/iree/compiler/Dialect/HAL/Target/LLVM/LLVMAOTTarget.cpp#L350-L356
I need to dig deeper to find out what is really going on though.

I can get it running by uncommenting /*| llvm::Linker::LinkOnlyNeeded*/ here
https://github.com/google/iree/blob/a80811c5713b513448dca6f3134d597101ed9136/iree/compiler/Dialect/HAL/Target/LLVM/LLVMAOTTarget.cpp#L127

I can get it running by uncommenting /*| llvm::Linker::LinkOnlyNeeded*/ here
https://github.com/google/iree/blob/a80811c5713b513448dca6f3134d597101ed9136/iree/compiler/Dialect/HAL/Target/LLVM/LLVMAOTTarget.cpp#L127

@benvanik Any reason why OverrideFromSrc is used instead of LinkOnlyNeeded? We're not sure yet, if we can use OverrrideFromSrc together with WeakAnyLinkage.
In addition, do we really want to link in musl if building for a bare-metal target? One hackish way would be to check if the target triple is something like *-none-unknown-eabi. In that case we could simply skip linking in musl.


Supports the generation of PMML files

Hi, IMBENS is a great machine learning library, but I have a problem now, I want to deploy self_paced_ensemble algorithm as online services to production environment, how to generate PMML files successfully. I have tried Nyoka and sklearn2pmml so far, but all failed.

Hi thanks for raising this issue, but unfortunately I'm not an expert in this area. I don't have experience with PMML files thus not planning to add support for this in the near future. I took a glance at the sklearn2pmml reference like this page https://openscoring.io/blog/2023/05/03/converting_sklearn_subclass_pmml/, and it seems that there is a way to convert customized sklearn estimator (like those provided in imbens) by adhering some certain standards, maybe that's something you can try by creating a wrapper class/ write your own estimator with the functions & samplers provided in IMBENS.


Remove deprecated torch._six

@pruthvistony Can you please mention the JIRAs that required this change in the description?


Add debug.md to explain how to debug in the web editor

In https://github.com/exercism/ruby-test-runner/commit/a215353f4c0892a8d104153003eb6ed383a40928 a custom debug function was added to allow students to capture output during test runs. Other printing methods like puts won't work.
This needs to be documented in exercises/shared/debug.md (see https://github.com/exercism/docs/blob/main/building/tracks/shared-files.md#file-debugmd)

This has been moved to exercises/shared/.docs/debug.md


Fix run once

Fix for https://github.com/digitalbazaar/bedrock/issues/45
Issued fixed:

Remove listener for runOnce & runOnceAsync events.
Only remove runOnce listeners if done or on error.
switch to addListener over on as on created confusion in the code.

p.s. previously runOnceAsync was definitely running more than once you can add a logger in runOnceAsync on master and compare it to this.
p.p.s. please run this branch in your project.
this does not fix some of the bugs related to runOnceAsync we are running into, but it does generate more consistent possibly deterministic behavior from the functions.

Closed in favor of #49.


fixed compatibility with nette/component-model 3.1

I think that line in there is completely unnecessary.

Thanks 👍


Quartus Lite Missing Dependency

Any network related option like feedback on Quartus fails with the following error:
XPCOMGlueLoad error for file /nix/store/fsbnm1zzj87ahajdhnqcp5p0w5aii4a7-firefox-121.0/lib/firefox/libmozsandbox.so:
/nix/store/q4flsi0gjzj53gixfmqhihlnv54flpn6-quartus-prime-lite-unwrapped-22.1std.2.922/quartus/linux64/libstdc++.so.6: version `GLIBCXX_3.4.26' not found (required by /nix/store/fsbnm1zzj87ahajdhnqcp5p0w5aii4a7-firefox-121.0/lib/firefox/libmozsandbox.so)
Couldn't load XPCOM.

Which tells me quartus can not communicate with firefox for some reason.
Steps To Reproduce
Steps to reproduce the behavior:

Install quartus-prime-lite
From toolbar click help
Click on feedback

Expected behavior
Open a feedback form.
Additional context
It is not a deal-breaker for me however there might be other users affected by it since IP store might also not work. Though, I haven't tested it.
Notify maintainers
@kwohlfahrt
Metadata
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.7.0, NixOS, 24.05 (Uakari), 24.05.20240108.317484b`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.1`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Add a :+1: reaction to issues you find important.

AFAIR, quartus itself sets $LD_LIBRARY_PATH which breaks web-browsers it spawns from the host OS later on.
https://github.com/NixOS/nixpkgs/pull/280121#issuecomment-1885766140 might help.

@bjornfor I am not sure if this is the right place the ask it but I wonder if it would be acceptable to add an option to disable/enable quartus and questa.
Most of the time I only need questa and I used to install it alone when I was on Arch. I am thinking something like this:
           (pkgs.quartus-prime-lite.override {
             installQuesta = True;
             installQuartus = False;
           })

Would something like this be acceptable if I cook up a PR?

Would something like this be acceptable if I cook up a PR?

Sounds fine to me. I guess there's some big closure size reductions to be had here?

Yea, Questa is only 2 GBs. I am trying to save some space 😅. Maybe instead I could buy a bigger SSD.


chore(datepicker): change deprecation message for popup

The previous message was a bit confusing because popup attributes are not deprecated, just the inline ones.

LGTM


Enable a Linter to Support Autofixing (JSCS)

We are working on Autoformatting over at JSCS https://github.com/jscs-dev/node-jscs/issues/516 and I've been trying to figure out if SublimeLinter has any ability for a linter plugin to actually modify the content of the view that it is linting.
We would love for the JSCS linter package, which is the de-facto standard for Sublime and JSCS integration to support this out of he gate but aren't sure if you guys are interested in this approach of doing more than just linting.
In the meantime, I've put up a repo for a separate Sublime plugin here: https://github.com/TheSavior/Sublime-JSCS-Formatter but I'd much prefer to delete that project and add the functionality to the SublimeLinter package.

Currently this isn't part of the SublimeLinter platform. It's a tough decision to make, whether or not this falls into scope of the project, but using the ST API you should be able to incorporate a setting in the existing SL-JSCS that would make the changes and push back to the view. I'm not promising cleanliness, but if you didn't want the separate formatter repo. SublimeFormatter maybe needs to become a thing. ;)


jitsi-meet: update all Jitsi components to stable release 6293

Motivation for this change
Bump Jitsi server components to the latest stable Jitsi release, 6293.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 137681 at 2edaa121 run on aarch64-linux 1

4 packages built successfully:

jicofo
jitsi-meet
jitsi-meet-prosody
jitsi-videobridge

Thanks! I just recently merged some updates:
https://github.com/NixOS/nixpkgs/pull/137559
https://github.com/NixOS/nixpkgs/pull/137568
https://github.com/NixOS/nixpkgs/pull/137563

https://nvd.nist.gov/vuln/detail/CVE-2021-39215 is apparently fixed from jitsi-meet-prosody 2.0.5963 onwards.

Ah and https://nvd.nist.gov/vuln/detail/CVE-2021-39205 in jitsi-meet itself is fixed from 2.0.6173 onwards

seems to be obsolete


Fetch nhdv2 attributes from ScienceBase

Addresses #16 and #77.
The code here adds a target to download all of the data associated with our VarsOfInterest table. The function to download the data also filters the output to only include COMIDs specified by the user, and saves a csv of the filtered output. The function also contains an argument to choose between retaining or deleting the original downloaded zipped and unzipped files.
I'm tagging @msleckman for a code review and @jds485 for a more cursory review. A few questions I have on this PR:

We could add the NLCD years beyond 2011 to VarsOfInterest and download them in a similar manner. Would this be helpful for addressing #56? I know we currently download NLCD data separately (in fact, 'Land Cover' variables have been filtered out of the target for p1_vars_of_interest), so I'm happy to leave that alone and fetch these data in a similar manner to NLCD2011. @msleckman, I'm interested in your thoughts on that since I'm not too familiar with the existing NLCD download functions.

There are some variables in our VarsOfInterest table that seem like they could be duplicates (i.e., slightly different GIS layer name but same sb id). I haven't looked at the data sources too closely, so I'd appreciate if anyone else can clarify whether these are distinct variables. From my notes:

There are variables labeled "ET" and "AET" (rows 78 and 62 in the VarsOfInterest csv) that point to the same sb link. Are these the same variable that got duplicated, or were we expecting different data for "ET" and "AET"? Similar question regarding variables RUN and RUN7100 (rows 115 and 112) and RH and RH6190 (rows 61 and 77).

In the vars of interest file, I added a column to indicate the sb items we are interested in so as to exclude bulky rasters that are included in some of these datasets. I used the entry in that column to limit the National Inventory of Dams (NID) dataset downloads to start at 1960, even though there are also sb items available for 1930, 1940, and 1950. It seemed like we were cutting off other metrics at 1960 (e.g., FORE-SCE?), so I figured I'd be consistent here. What do you think? Should we retain all of the years available?

Duplicate variables: Thanks for finding these! I just looked at the SB metadata to see if they differ, and I think these are exact duplicates with different names. Here are my thoughts on which names to retain:

ET and AET: AET to be specific that it's actual instead of potential.
RUN and RUN7100: RUN7100 to be specific to years used.
RH and RH6190: RH6190 ""

Should we retain all of the years available?

Cutting off at 1960 sounds good to me to capture ~20 year historical information from our start date

I looked at the column you added in the VarsOfInterest table. The only variables that do not have zipped files listed are the mean monthly precip variables that I added, so that makes me think I may have specified column names incorrectly. I thought used the names listed in their metadata file. Let me know if you need help getting the data for those variables

I looked at the column you added in the VarsOfInterest table. The only variables that do not have zipped files listed are the mean monthly precip variables that I added, so that makes me think I may have specified column names incorrectly. I thought I used the names listed in their metadata file. Let me know if you need help getting the data for those variables

@jds485, thanks for that offer! There are missing names in that column for the precip variables because I was lazy and didn't want to copy all the folder names (there are many!) 😄 I instead adding special handling to define those item names in lines 23-37 in 1_fetch/src/fetch_nhdv2_attributes_from_sb.R.

I instead adding special handling to define those item names

Sounds good. I didn't realize that you added the items in that column manually!
I finished overviewing the code and it looks good to me. Thanks for implementing the fetch-unzip-clip that we talked about last week

Should we retain all of the years available?

Cutting off at 1960 sounds good to me to capture ~20 year historical information from our start date.

Thanks, @jds485. As a recap, here's what I've implemented for datasets with multiple years based on discussion in this PR:

I will exclude two available years (1940, 1950) from the HDENS dataset so that our gathered data starts at 1960 (and goes to 2010).
The monthly PPT data are available from 1946-2014, but I'll only pull 1971-2000 to be consistent with our other met drivers
For the NID data, I think I'll actually exclude the data tagged "1960" since that data would reflect the decade spanning the 1950's (see below). By starting our gathered data  at 1970, we'll capture the dam dataset starting at the period spanning 1961-1970.

Number of dams built on or before YYYY per NHDPlus version 2 catchment, where YYYY is the last year for the decade of record (for example 1960 spans 1951 - 1960).

We may have to retain ET (over AET) and RUN (over RUN7100)

Okay, sounds good!

Could we create a subfolder for these specific downloads?
foldername: STATSGO for the following 3 csvs:
"1_fetch/out/STATSGO_HYDGRP.csv"
"1_fetch/out/STATSGO_TEXT.csv"
"1_fetch/out/STATSGO_LAYER.csv”

folder name: PPT for the following 3 csvs:
"1_fetch/out/PPT_ACC.csv"
"1_fetch/out/PPT_TOT.csv"
"1_fetch/out/PPT_CAT.csv”

I agree that because these are thematically similar, it might be more satisfying if they were combined somehow, either in a separate folder or a single data frame. For the STATSGO tables, these csv's reflect the format of the input datasets that are contained on ScienceBase, and they at least follow the expected structure (columns for "ACC","CAT", and "TOT). The PPT files on ScienceBase are a little different because the ACC/CAT/TOT files actually represent separate sb_id's all-together. I initially thought about combining those PPT tables but opted not to because it would make the data download code more complex and thus harder to read (i.e., by adding special handling code), and because I wasn't sure that it'd be necessary. In the data processing steps, I envision that we'll subset any of these tables to only include those columns containing "CAT" to apply various functions (e.g. sum, area-weighted mean). So the PPT_TOT and PPT_ACC csv's just wouldn't get passed through that data processing code. Would you agree with my characterization here (if it makes sense, that is!)?
So I think I'm still leaning toward leaving the downloaded files as is to keep the code as simple as possible and to keep fetch tasks separate from other processing tasks we might need/want. Do you think that makes sense? Perhaps we can revisit in the data processing step?

Thanks for your review, @msleckman, and thank you for skimming all those downloaded files! If, after skimming the downloaded files, you have notes for processing steps we'll need to add or at least consider, it'd be useful to add those to issue #80. I think I've addressed all of your comments for the fetch step here, so let me know if we should add anything else or whether we can merge.

Based on Margaux's review (and looking forward to processing the downloaded Wiezorek datasets), I realized that it would make the code cleaner if I used dynamic branching instead of a multi-step lapply process to create the p1_vars_of_interest_downloaded_csvs target.
So, fyi @msleckman, @jds485 I went ahead and made that change here. @msleckman, would you mind checking that this still builds locally for you?

Looks good!

is it possible to put a sys.time() variable wrapper

targets tracks the time it takes to build every target (and every branch) within the meta file. You can access this information with tar_meta()

I did feel like it took longer to download the files than the last time >40 min (although I did not perfectly time it).

targets's first instinct is to map over all rows of a data frame. When I first implemented the branching approach for p1_vars_of_interest_downloaded_csvs, I noticed that some sets of files were being downloaded multiple times (e.g. the PPT_CAT) because that sb_id is repeated 12 times in our VarsOfInterest table to reflect the fact that the dataset includes columns for 12 different months. To instead map over row subsets delineated by unique sb_id's, I added the following lines to the upstream target p1_vars_of_interest:
... %>%
group_by(sb_id) %>%
tar_group()

This worked for me to ensure that PPT_CAT, PPT_ACC, PPT_TOT, and others were only being downloaded and unpacked once. @msleckman do you know if p1_vars_of_interest got rebuilt when you ran the pipeline with dynamic branching? 40 min seems long, but I was probably working on other things in the meantime and may not have noticed if it took longer than before.


MapEntry refactoring [ECR-2526]

Overview
proof MapEntry and storage MapEntry renamed to ByteStringMapEntry and GenericMapEntry. MapEntry moved to common module. MapEntry interface with factory methods added.

See:  https://jira.bf.local/browse/ECR-2526
Definition of Done

[x] There are no TODOs left in the code
[x] Change is covered by automated tests
[x] The coding guidelines are followed
[x] Public API has Javadoc
[x] Method preconditions are checked and documented in the Javadoc of the method
[x] Changelog is updated if needed (in case of notable or breaking changes)
[x] The continuous integration build passes

Coverage decreased (-0.3%) to 84.381% when pulling 6b183fea8d45260869c53243b97bd5a73c0933a3 on ecr-2526 into 42db2fda2df0033e249c4ee4f55c2f7effe4548f on master.

@dmitry-timofeev  Thanks, I'll review refactoring


Parse xsd:include and serve xsd files

I implemented the parser to xsd:include tags and the server response to XSD requests (?xsd=myfile.xsd)
It is a long diff because I had to rewrite some code to use locations instead of namespaces to identify Schema_XXX and to avoid duplicated schema processing.
I still want to write some tests here and reorganize my commits, but I need someone to check if my changes make sense.

 I assumed that a namespace should be better suited for that.

Is this because of #49 ?

Thanks for your feedback.
Yes, it is related with #49. I'll try to extract this change to a separated commit.
For includes, it is critical. The xsd:include means exactly it: include this schema in the same namespace.
http://www.w3schools.com/xml/el_include.asp
I identified Schema_* classes and file duplication based on filename instead of namespace. I used the targetNamespace as a fallback to the root schema (location=None).

Yes for includes it makes sense. I'll run your changes with my own soapfish projects to see if these changes cause any regression for me.

I cherry-picked 9e9cb6b in #63 which works fine for me. Should I try to merge this code here as-is or do you want to rework it?

You can merge this one. I just rebased with your current master to fix merge conflicts.

Just fyi: I'm seeing some errors when running the test suite for one of our products with the changes from this pull request.
  File ".../soapfish/py2xsd.py", line 246, in schema_validator
    resolver = SchemaResolver(schema)
  File ".../soapfish/py2xsd.py", line 230, in __init__
    self._map(schema)
  File ".../soapfish/py2xsd.py", line 233, in _map
    for item in (schema.imports + schema.includes):
TypeError: can only concatenate list (not "tuple") to list

I'd like to check if this is a problem in my code or in soapfish.

It is in soapfish. Both imports and includes defaults to empty tuples, but xsd2py generate these values as lists.
I fixed it converting both attributes to lists, but I still don't have a test for this use case.
Can you try again? I'll write a test for it.

I changed imports and includes to always be lists. I fixed one test, it defined imports as tuple. So, it is already a test to this problem.

I added another parameter to xsd2py generation. I need the base_path to normalize paths relative to it.
I avoid absolute paths to not leak your computer path structure, leading to a security issue.

Thank you very much for updating your pull request. I'm sorry it too me so long before I could follow up here - and doubly sorry as I have some more questions.
The current code seems to assume that imports and includes are always lists. However a user (like me ;-) might have existing code which uses tuples. I think we should work with any iterable and convert the items on the fly if necessary.
Also using [] as a default value in a function definition always scares me - mutable default values caused a bug which took me about a week to track down and caused a lot of problems until I found it. Not sure how much hair I lost during while debugging it. We should not do that. Please.

I agree with you. Default empty lists are dangerous! I did it because my list + tuple failed, but the right way to do it is with itertools.chain. I updated some code to use the chain.

Also, I rebased with current master to fix some conflicts.


The data received does not correctly match the corresponding PINRemoteImageDownloadTask

release 3.0.0-beta.7

Due to my project, I cannot use the latest release to reproduce this problem. But I look at the latest source code does not seem to find a solution to this problem, you may have fixed.

Image data is received piece by piece while the image is being downloaded. In the process of image data download, I quickly back and forth scroll the view, the current download task will be cancelled, PINRemoteImageDownloadTask will be removed from the tasks, and then start a new task. This looks fine,  but the problem occurs when the URL of the new task is the same as the URL of the cancelled task.
Due to image data is returned piece by piece, it is possible that the server returning a piece of data. At this time, the task of this URL has been cancelled, and the PINRemoteImageDownloadTask of corresponding URL(as key) in tasks will be removed. And then start the download task of same URL, the tasks will store the PINRemoteImageDownloadTask of corresponding URL(as key) again. Now, the data of cancelled task was received, PINURLSessionManagerDelegate method according to PINRemoteImageCacheKey find the cacheKey(URL) in NSURLProtocol, and according to cacheKey find the PINRemoteImageDownloadTask in the tasks. This PINRemoteImageDownloadTask should belong to the new task, and the received data should belong to the cancelled task, the received data data will be added to the PINRemoteImageDownloadTask. This will be stored with the correct data for the new task, resulting in an image decoding exception.
// PINRemoteImageManager.m
- (void)didReceiveData:(NSData *)data forTask:(NSURLSessionDataTask *)dataTask
{
    [self lock];
        NSString *cacheKey = [NSURLProtocol propertyForKey:PINRemoteImageCacheKey inRequest:dataTask.originalRequest];
        PINRemoteImageDownloadTask *task = [self.tasks objectForKey:cacheKey];

        /********************************** Here is the NSAssert I added ***************************************************/
        NSAssert(dataTask.taskIdentifier == task.urlSessionTaskOperation.dataTask.taskIdentifier, @"The received data belong to cancelled task.")
        /*******************************************************************************************************************/

        if (task.progressImage == nil) {
            task.progressImage = [[PINProgressiveImage alloc] init];
            task.progressImage.startTime = task.sessionTaskStartTime;
            task.progressImage.estimatedRemainingTimeThreshold = self.estimatedRemainingTimeThreshold;
            if (self.progressThresholds) {
                task.progressImage.progressThresholds = self.progressThresholds;
            }
        }

        PINProgressiveImage *progressiveImage = task.progressImage;
        BOOL hasProgressBlocks = task.hasProgressBlocks;
        BOOL shouldBlur = self.shouldBlurProgressive;
        CGSize maxProgressiveRenderSize = self.maxProgressiveRenderSize;

        [task callProgressDownloadWithQueue:self.callbackQueue completedBytes:dataTask.countOfBytesReceived totalBytes:dataTask.countOfBytesExpectedToReceive];
    [self unlock];

    [progressiveImage updateProgressiveImageWithData:data expectedNumberOfBytes:[dataTask countOfBytesExpectedToReceive]];

    if (hasProgressBlocks && [NSOperation instancesRespondToSelector:@selector(qualityOfService)]) {
        __weak typeof(self) weakSelf = self;
        [_concurrentOperationQueue pin_addOperationWithQueuePriority:PINRemoteImageManagerPriorityLow block:^{
            typeof(self) strongSelf = weakSelf;
            CGFloat renderedImageQuality = 1.0;
            PINImage *progressImage = [progressiveImage currentImageBlurred:shouldBlur maxProgressiveRenderSize:maxProgressiveRenderSize renderedImageQuality:&renderedImageQuality];
            if (progressImage) {
                [strongSelf lock];
                    PINRemoteImageDownloadTask *task = strongSelf.tasks[cacheKey];
                    [task callProgressImageWithQueue:strongSelf.callbackQueue withImage:progressImage renderedImageQuality:renderedImageQuality];
                [strongSelf unlock];
            }
        }];
    }
}

I added an assertion, and then when I scroll back and forth quickly the view has a probability that it will hit.

I think you may have caught something here! Would you mind uploading a sample project?

@HXHinsHuang would you mind testing the fix here? https://github.com/pinterest/PINRemoteImage/pull/514


Support Ruby 2.3, 2.4 and 2.5

Ruby 2.5 has been ready, so let's support Ruby 2.3, 2.4 and 2.5.

Coverage remained the same at 99.194% when pulling 52b35883ccce513f55b265d2d44d48ce30470745 on support-ruby-2.5 into e46fb7955f29d19e31040b0d1d845984e03750ce on master.

Coverage remained the same at 99.194% when pulling 52b35883ccce513f55b265d2d44d48ce30470745 on support-ruby-2.5 into e46fb7955f29d19e31040b0d1d845984e03750ce on master.

Coverage remained the same at 99.194% when pulling 52b35883ccce513f55b265d2d44d48ce30470745 on support-ruby-2.5 into e46fb7955f29d19e31040b0d1d845984e03750ce on master.


[Question]: Is it possible to filter ObjectIndex by additional metadata?

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
I have agentic RAG approach with OpenAIAgent class on the top. This AI agent uses tool retriever based on ObjectIndex class:
obj_index = ObjectIndex.from_objects(all_tools, index_cls=VectorStoreIndex)
obj_retriever = obj_index.as_retriever(similarity_top_k=10)

agent = OpenAIAgent.from_tools(tool_retriever=self.obj_retriever, verbose=True, system_prompt=AgentTools.SYSTEM_PROMPT, max_function_calls=30)

Tools are built on QueryEngineTool class which has only one metadata field for document name and description. I want to add documents classification and based on user permissions - filter by appropriate classification. On summary and vector index I'm able to set additional metadata and filter by them but I need filtering on the top level. Is there a possibility to extend tool metadata or any other way to achieve that?

@dosu Your example is invalid:
TypeError: ToolMetadata.__init__() got an unexpected keyword argument 'classification'

ToolMetadata class from llama_index.core.tools is not freely expandable (llama-index 0.11.9)

@dosu This works, but is there any better solution for that? I would avoid creating few custom classes - in my opinion this should be natively handled by framework.


[Feature request] clientAppEnhance.js can't install any plugins on app instance.

Feature request
Description
enhanceApp.js, I can install ElementUI by
export default ({ Vue, options, router }) => {
  Vue.use(Element);
};

But I can't use Vue.install at this time. And it seems like my clientAppEnhance.js doesn't work, I can't console.log or alert what you supported in it, this function seems not to be executed.
Please add any docs on clientAppEnhance~

The conventional file name has changed to clientAppEnhance.js / clientAppEnhance.ts

VuePress 2 is built on Vue 3, so you need to use app.use, etc.

https://github.com/vuepress/vuepress-next/blob/e552dec8e107b66196515c8245b5385de85e3dc3/packages/%40vuepress/client/src/types/clientAppEnhance.ts#L5-L9

The conventional file name has changed to clientAppEnhance.js / clientAppEnhance.ts

VuePress 2 is built on Vue 3, so you need to use app.use, etc.

https://github.com/vuepress/vuepress-next/blob/e552dec8e107b66196515c8245b5385de85e3dc3/packages/%40vuepress/client/src/types/clientAppEnhance.ts#L5-L9

I saw the way to solve this problem on the theme-default:
https://github.com/vuepress/vuepress-next/blob/f490b44008d7d52fdc135e1654fc1361e5ea7be3/packages/%40vuepress/theme-default/src/index.ts#L26
I set clientAppEnhanceFiles propty at .vuepress/theme, and all done.
module.exports = {
    name: '@vuepress/theme-hooperui',
    clientAppEnhanceFiles: path.resolve(__dirname, './clientAppEnhance.js'),
}

So the conclusion is that you must set clientAppEnhanceFiles on your index.{js|ts}, or clientAppEnhanceFiles.js will not be executed

I saw the way to solve this problem on the theme-default:
https://github.com/vuepress/vuepress-next/blob/f490b44008d7d52fdc135e1654fc1361e5ea7be3/packages/%40vuepress/theme-default/src/index.ts#L26
I set clientAppEnhanceFiles propty at .vuepress/theme, and all done.
module.exports = {
    name: '@vuepress/theme-hooperui',
    clientAppEnhanceFiles: path.resolve(__dirname, './clientAppEnhance.js'),
}

So the conclusion is that you must set clientAppEnhanceFiles on your index.{js|ts}, or clientAppEnhanceFiles.js will not be executed

For themes, you have to set it explicitly.
For users, the .vuepress/clientAppEnhance.js will be autoly load.

For themes, you have to set it explicitly.
For users, the .vuepress/clientAppEnhance.js will be autoly load.

OK, thank you!

OK, thank you!


Add the "still in testing" warning to the publish/1 form

When creating an Algorithm a warning appears. In an earlier version we had: "Datastream is still in a testing phase, please use it only on test networks." As the Datastream asset is definitely still in testing it would be wise to restore that element.

Done. Merging now

Looks nice and professional.
Closing.


user testing

Prior to release user testing will need to occur.

[ ] I will initially test with a windows machine (as this has had little attention)
[ ] Send out to users for testing
[ ] Document tests for users to conduct and confirm they had no issues

[ ] Understand the impact of the slowness of the first initial document getting when the plugin is loaded for the first time. This will be undertaking by asking users of there experience

further development will be required if a user gives up and kills the QGIS application rather than wait. If they just wait they will not experience the problem again.

When using the plugin the first time, and adding my key in, when I press Save QGIS appears to stall. Maybe a message to tell the user to be patient since it's working in the background.  After the first time, the plugin worked fine.


redirect() is not working as expected

What steps will reproduce the problem?
i have a module with normal CRUD actions
after last composer update
the redirect in create and update to view are no longer working as expected
What is the expected result?
it should be /module_name/default/view?id=1
What do you get instead?
i get redirected /view?id=1
Additional info

Q
A

Yii version
2.0.11-dev

PHP version

Operating system

@tsanchev it's not clear on how to reproduce it. Could you elaborate what "normal CRUD actions" are?

By "normal CRUD actions" i mean, i have generated model and CRUD using gii module, using latest gii version from several months ago.
i'm using advanced template.
the problem occurs in the backend of the project.
i have lots of modules with models in them and CRUDs (all of them generated with gii) and after updating yesterday, none of the generated code by gii involving redirects is working properly.
for example this is code for update from DefaultController.php from module 'products' which is the backend
`
/**
 * Updates an existing Products model.
 * If update is successful, the browser will be redirected to the 'view' page.
 * @param string $id
 * @return mixed
 */
public function actionUpdate($id)
{
    $model = $this->findModel($id);

    if ($model->load(Yii::$app->request->post()) && $model->save()) {
        return $this->redirect(['view', 'id' => $model->product_id]);
    } else {
        return $this->render('update', [
            'model' => $model,
        ]);
    }
}

`
on post after model get saved, it redirects me to
http://localhost/myproject/backend/view?id=X
instead of
http://localhost/myproject/backend/products/default/view?id=X
the redirect acts as it is
redirect(['/view' ....
and not
redirect(['view'

I experience the same problem with the same behavoir and no redirect is working probably if only the action is given. I'm also using the advanced template but with no modules so far. There is no difference between redirect with or without additional parameters. I did not use the Gii to generate the actions.

Yii version: dev-master
PHP version: 5.6.26

The problem occurred today after an composer update, but strangely nothing is shown in the logs. Still need to investigate that a little more. Some changes in the last 5 days must have caused the problem because that was the last time before today that I ran composer update  and it worked fine until today.
With Yii version 2.0.10 everything works as aspected

Probably #13393 caused this bug.

@teruk can you revert that change and try it w/o it? Will be able to do it evening or tomorrow myself...

@samdark I changed my version to dev-master again and I can confirm, that this change caused the problem.

Reverted https://github.com/yiisoft/yii2/commit/eab3489ec711b1ad9de0158e402fffaba999f4ff. Thanks for checking.


Add files via upload

@hoony6134 Please provide us with a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.
You can provide us with any of the following:

A copy of instruction (a hard-copy or electronic one) given to you by the school that tells you to use an email address in this email domain
An URL of a page at the official school website that says that students use this email domain
A reference to a recognized journal where an article was published by someone from the school, with explicit affiliation to the school, and e-mail address from the email domain
A WHOIS data that link the email domain to the school somehow
A copy of official letter from the school representative saying that the school provides e-mail addresses in this domain to the students
Anything else that can prove this fact in similar way, i.e. can be verified independently.

@hoony6134 You might miss my previous comment. Please react.

@hoony6134 I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the aforementioned issue. Thank you for your understanding!

Oh

I'll reopen with description.


Unable to authorize orgs

I am unable to authorize any org. I have tried running through VSCode and through the terminal. I can get to the Allow Access page and click yes, but then it redirects to localhost 1717 and eventually it stops and gives me "ERR_EMPTY_RESPONSE." This happens for all orgs, all browsers. I have already tried the steps in this link and it doesn't not work either...
https://trailblazer.salesforce.com/issues_view?id=a1p3A0000003dkDQAQ
Steps To Reproduce:
Open VSCode -> create project -> authorize an org -> enter username and password -> allow access
OR
Do it in the terminal
Expected result
Authorized org
Actual result
After several minutes, it does not connect and says "ERR_EMPTY_RESPONSE"
Additional information
SFDX CLI Version(to find the version of the CLI engine run sfdx --version):
sfdx-cli/7.84.2-a2868a68d5 linux-x64 node-v12.18.3
SFDX plugin Version(to find the version of the CLI plugin run sfdx plugins --core)
@oclif/plugin-autocomplete 0.1.5 (core)
@oclif/plugin-commands 1.3.0 (core)
@oclif/plugin-help 3.2.1 (core)
@oclif/plugin-not-found 1.2.4 (core)
@oclif/plugin-plugins 1.9.5 (core)
@oclif/plugin-update 1.3.10 (core)
@oclif/plugin-warn-if-update-available 1.7.0 (core)
@oclif/plugin-which 1.0.3 (core)
@salesforce/sfdx-trust 3.6.0 (core)
alias 1.1.5 (core)
analytics 1.12.1 (core)
auth 1.4.6 (core)
config 1.2.3 (core)
generator 1.1.3 (core)
salesforcedx 50.13.3 (core)
├─ user 1.0.5 (core)
├─ schema 1.0.3 (core)
├─ limits 1.0.2 (core)
├─ @salesforce/sfdx-plugin-lwc-test 0.1.7 (core)
├─ templates 50.4.0 (core)
├─ salesforce-alm 50.13.1 (core)
├─ custom-metadata 1.0.11 (core)
└─ apex 0.1.4 (core)
sfdx-cli 7.84.2 (core)
OS and version:
Ubuntu 20.04.2 LTS

I have the this problem with a Raspberry Pi running Ubuntu.

Similarly, we are running into an issue with our CI/CD tool when it is trying to authorize an org using the force:auth:jwt:grant command.  It's giving us an error saying Warning: force:auth:jwt:grant is not a sfdx command.. Since the 50.13.3 release was re-rolled out today, I'm wondering if this is related to the deprecation of force:auth in favor of the auth plugin?

Hi @dvon88 @ErikChase we just published a new release today, and a latest-rc tag, try the latest-rc with npm install sfdx-cli@latest-rc --global and see if that fixes it. As for the alias between force:auth: and just auth commands that should've been resolved

That has resolved the issue for me!  Thank you so much!

@WillieRuemmele that did not fix the issue. I ended up reinstalling linux and that "fixed" it. I'll mark the issue as closed since I cannot reproduce it anymore, but this is definitely not a fix. I should not have to reinstall my operating system to fix an issue with the CLI.

@WillieRuemmele  nevermind, I can still reproduce the issue. I am now again trying to update to the latest release and getting a timeout error during the update when trying to fetch https://registry.npmjs.org/resolve.
Issue is still open. Still broken.

@WillieRuemmele
It didn't work for me(

@WillieRuemmele I've been having the same issue on Debian
sfdx-cli/7.86.3 linux-x64 node-v14.15.5
@oclif/plugin-autocomplete 0.1.5 (core)
@oclif/plugin-commands 1.3.0 (core)
@oclif/plugin-help 3.2.2 (core)
@oclif/plugin-not-found 1.2.4 (core)
@oclif/plugin-plugins 1.9.5 (core)
@oclif/plugin-update 1.3.10 (core)
@oclif/plugin-warn-if-update-available 1.7.0 (core)
@oclif/plugin-which 1.0.3 (core)
@salesforce/sfdx-trust 3.6.0 (core)
alias 1.1.5 (core)
analytics 1.12.1 (core)
auth 1.4.7 (core)
config 1.2.4 (core)
generator 1.1.5 (core)
salesforcedx 50.16.1 (core)
├─ limits 1.0.3 (core)
├─ schema 1.0.3 (core)
├─ user 1.0.11 (core)
├─ apex 0.1.4 (core)
├─ custom-metadata 1.0.11 (core)
├─ templates 50.4.0 (core)
├─ @salesforce/sfdx-plugin-lwc-test 0.1.7 (core)
└─ salesforce-alm 50.16.1 (core)
sfdx-cli 7.86.3 (core)
telemetry 1.0.1 (core)
Debian GNU/Linux 10
Linux version 5.4.74-10576-gb6cc41974db3

I frequently have this issue.  For a while I had luck setting my default browser to Firefox when I auth (I normally use Chrome).  But then I got the same issue on Firefox too.  Then I worked around it using Edge.

Duplicate of https://github.com/forcedotcom/cli/issues/783


Manage popup scenario.

Handles item details (image / title / description)
Extracts all dom elements (image / title /description / list / form)
Handles close event by clearing all elements

👏 AMAZING JOB 👏

I really liked the way you organized your files with es6 modules.
Nice use of javascript code by ussing Javascript Classes.
Very good method implement to check if all form fields are completed.

Thanks @Felipe-Perez-Ferraro, @mahammad-mostafa for your review.
I've applied your suggestions, please have a look on it and let me know if there's other thing to change.

Thanks @mahammad-mostafa, @Felipe-Perez-Ferraro


Ben/find 368 findforge add ways to add more names to

Everyone can add Forge to the dictionary.
Only forges that are added to certain name / added to PublicForge arrays are available to users.
Users with premium forge can mint NFT with 0 platform cut


Fix import path of go-sdk

From code.gitea.io/go-sdk/gitea
To code.gitea.io/sdk/gitea
Deprecates #140

Should work once https://github.com/go-gitea/redirects/pull/5 is merged

OK, could you fully test when https://github.com/go-gitea/redirects/pull/5 will be merged?  if everything is OK, i close this PR

go-gitea/redirects#5 is now merged.

We need to wait for something (drone?) to update the website, which seemd to have failed:
https://drone.gitea.io/go-gitea/redirects/18

Hmm yeah, drone fail to get access to docker...  Who is Drone administrator ?

Current coverage is 3.03% (diff: 100%)

Merging #141 into master will not change coverage

@@            master      #141   diff @@
========================================
  Files           33        33
  Lines         8096      8096
  Methods          0         0
  Messages         0         0
  Branches         0         0
========================================
  Hits           246       246
  Misses        7830      7830
  Partials        20        20

Powered by Codecov. Last update 96c9fef...a3c916f

@tboerger set it up, I hope all owners have access to it. @lunny ?

LGTM

LGTM


Ubunto

https://docs.microsoft.com/en-us/azure/architecture/example-scenario/data/big-data-with-iot
If'm I didn't miss anything, instead of Ubunto on a diagram should be Ubuntu.

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

ID: f5c2b717-8cbb-8794-3a5c-3086e76f1717
Version Independent ID: 6d6ea7d8-05ec-7903-1df5-6d3fc248a123
Content: IoT and data analytics in the construction industry
Content Source: docs/example-scenario/data/big-data-with-iot.md
Service: guidance
GitHub Login: @alexbuckgit
Microsoft Alias: pnp

#sign-off

interesting. 5 people participating in this issue. to fix 1 symbol. it takes already 9 months.

#please-close per merge to live


Volume Control appears twice

is it only for me that it shows the volume control twice?
one as a bulb and another as a speaker icon

its not twice, this is 2 different services, speaker service is native and always enabled, bulb or fan are additional service which can be disabled in config.


Shortcut to Superhero.com Feed

There should be a shortcut to the feed (Superhero.com) on the wallet for better access and to encourage use. I know it's accessible when users send tips via "View your tip on the feed"---although this button seems misleading since it just leads to the feed overall and not the specific tip sent.
But for the long term, to encourage use of Superhero.com and other utilities there, a shortcut from the wallet could add some benefits. For your consideration. See space that could be allocated beside the buttons below.

Maybe even better to directly integrate the superhero feed into the mobile version instead of linking to a browser website, just like most other social media apps do :)


[test] Introduce experimental CLI

Adds yarn t testFilePattern which starts mocha in watchmode only including files matching **/*${testFilePattern}.test.{js,ts,tsx}.
Local comparisons:

7s: yarn t Button --single
7s: yarn cross-env NODE_ENV=test mocha "packages/**/Button.test.js"
15s: yarn test:unit --grep "Button"

test/cli.js is a wrapper around mocha inspired by how the jest cli functions. It's not meant as  feature-rich test cli that tests the whole codebase. The existing commands are better suited for that. yarn t is developing a fix/feature.
yarn test:unit --grep pattern has to evaluate the full suite before it can apply --grep pattern which means we have to essentially transpile the whole codebase. This can lead to slow startup times for the test runner. Limiting to a test file name reduces the amount of files we have to evaluate but is not very ergnomic (yarn cross-env NODE_ENV=test mocha "packages/**/Button.test.js")

Something is wrong with the CLI options to run the watch mode

What's wrong?

Does it change something in watch mode?

What should it change?

What's wrong?

Do we miss the watch option? I was expecting to be able to do yarn t --watch.

What should it change?

I was wondering if it could reduce the time necessary to re run the test. I often have to wait 5s+ before the watch mode triggers.

I was expecting to be able to do yarn t testFilePattern --watch or the opposite, to have it in watch mode by default.

Should be watchmode by default as per PR description. Maybe I forgot to implement the default.

I often have to wait 5s+ before the watch mode triggers in the current setup.

That's difficult to debug without knowing what task you ran. Might just be the number of files watched. Might be your slow filesystem. Might be something else.
yarn t feels snappy for me.

yarn t feels snappy for me.

Same for me :)


Crafter suggestion

Issue Description:
The crafter can't keep or doesn't have option to keep the supporting item in it.
What happens:
We have a private server that have some mods on it. We trying to use the crafters to make some kind of food. The recipes are include some kind of supporting items, which will not be consumed while in crafting. The crafter will put them into the output box and the craft will be stopped and will take more time to move that item back to the crafter.
What you expected to happen:
The material area should not be push out and can keep the supporting item in the material area.
Steps to reproduce:

...

Affected Versions (Do not use "latest"):

EnderIO: 5.2.59
EnderCore: 1.12.2-0.5.76
Minecraft: 1.12.2
Forge: 1.12.2-forge-<IP_ADDRESS>54
SpongeForge? no
Optifine? no
Server

Your most recent log file where the issue was present:
[pastebin/gist/etc link here]

This is how it was in 1.7.10, but it had to be changed when Mojang changed the crafting system. Now there's no way of distinguishing between a recipe leaving behind an empty bucket or an unchanged item. And just leaving it in the crafting area is no solution either, as the crafting system doesn't indicate which item fails the recipe.
Just enable "self feed" on the extracting item conduit.


OCM add-on parameters support

OSD stores addon parameters in a secret called
addon--parameters.The OCS converged addon
controller will watch for this secret and update the
storagecluster resource with the provided parameters.
Signed-off-by: N Balachandran<EMAIL_ADDRESS>Signed-off-by: nb-ohad<EMAIL_ADDRESS>

@JohnStrunk Can you please merge?


Show how many viewers in Chat section (IDEA)

Would be nice to see how many viewers are watching stream in the chat section of the app

Have thought about adding this, although some smaller streamers don't like seeing their viewer number as if they go down it can hurt seeing that people may not stick around.
So it may have to be a toggable feature that users can click to hide

Added this in a recent beta build 1.3.1-beta


Bug - When adding courses in the schedule builder with class types 8-17, their background color is not shown in the time table. For example the course 29168 is loaded but without background color because its class type is 16 (Shiur veSeminar).

Hi,
I found a bug in your project- When adding courses in the schedule builder with class types 8-17, their background color is not shown in the time table.
For example the course 29168 is loaded but without background color because its class type is 16 (Shiur veSeminar).

Thanks for the fix!

Merged #96


Adding a current module to handle ContextVars

This PR replace utils.current. and utils.getCurrentUser() with a current module.
Related to #632 and #633

@ArneGudermann can you please fix the conflicts that came in with the latest pull request merges first? After that I can start a review.

I think the user must be set after each login and logout too.

Haha, exactly this was my thought when I reviewed it right now ;-D

@ArneGudermann I've accidentally merged #652 into this branch some minutes ago. I hope you're fine with it and all reviewers comply with the changes. If not, please annotate here.

@phorward It looks okay for me.

I think the user must be set after each login and logout too.

Dear Mr. @sveneberth can you please re-approve this PR?
The context-variable is set in Request and on the user-module appropriately now.


Feature/provide data to events

This is related to #4 where data is not being passed down to consumers so they cannot really handle non supported events themselves, this way they can through whatever mechanism they want.
In my scenario I want to be able to listen for redemptions and do things.

just saw this, sorry for the super delay lol


--count does not work for me

Can you please have a look - wallabag-client 1.6.4 on Ubuntu 20.04, installed via pip.
$ wallabag list --unread --count
Traceback (most recent call last):
  File "/home/dirk/venv/wallabag-client/bin/wallabag", line 8, in <module>
    sys.exit(cli())
  File "/home/dirk/venv/wallabag-client/lib/python3.8/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/home/dirk/venv/wallabag-client/lib/python3.8/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/home/dirk/venv/wallabag-client/lib/python3.8/site-packages/click/core.py", line 1259, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/dirk/venv/wallabag-client/lib/python3.8/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/dirk/venv/wallabag-client/lib/python3.8/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/home/dirk/venv/wallabag-client/lib/python3.8/site-packages/click/decorators.py", line 21, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/home/dirk/venv/wallabag-client/lib/python3.8/site-packages/wallabag/wallabag.py", line 93, in wrapper
    func(*args, **kwargs)
  File "/home/dirk/venv/wallabag-client/lib/python3.8/site-packages/click/decorators.py", line 21, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/home/dirk/venv/wallabag-client/lib/python3.8/site-packages/wallabag/wallabag.py", line 128, in list
    run_command(
  File "/home/dirk/venv/wallabag-client/lib/python3.8/site-packages/wallabag/wallabag.py", line 472, in run_command
    result, output = command.execute()
  File "/home/dirk/venv/wallabag-client/lib/python3.8/site-packages/wallabag/commands/command.py", line 16, in execute
    self.log.debug('executing command: %s', self.__class__.__name__)
AttributeError: 'CountCommand' object has no attribute 'log'

It's a bug, thank you.

Thank you.


Add the ability to include extensions & shared libraries in pexs.

Extensions already (sort-of) worked via add_source - this makes sure that when adding a .(so|dll|pyd), we force zip_safe to False.
In addition this adds the ability to include other shared libraries with the pex. This is intended to be used for dependencies of the python extensions (e.g. a cityhash extension will depend on a cityhash library).
This is an updated version of #61.

pex adheres to what's in the standard lib and in PEPs.  My reluctance with this feature is that we're going out of the bounds of the python spec and more into the realm of cx_Freeze and p2app, which is a slippery slope of elaborate hacks to make stuff work when we already have ubiquitous solutions for managing dependencies holistically: docker, appc, etc.
It feels like this feature could be a wrapper around pex rather than baked into pex itself, where the wrapper is either heavyweight (docker) or lightweight (walk the pex, introspect the necessary .dylib/.so via something like pyelftools, and construct a shar or zip that sets up the path and execve's the pex.)

The PyPy tests pass for me locally with PyPy 2.6.1, which makes me suspect they're failing on Travis from the older PyPy...

FYI one way to fix this (as is done at https://github.com/facebook/buck/commit/f485090c08cfee550a7fd7f989b6556073c1c01b) - add an -rpath @loader_path/ (or -rpath $ORIGIN/ on linux) to your .so when linking. Then set zip_safe=False and everything should work :).


Every expression doesn't work and is not present

https://crontab.guru/#0_0_*/2_*_*
Cron.parseAndBuild("0 0 */2 * *")
crashes.
Is and will step values be added to this library

Hello @ebelevics!
Does * equal to 1 in this case?

What do you mean? For context I haven't used cron beforehand, so am new in this. What I wanted is to set recurrence every 2nd day, or 3rd day, regarding to users choice. Here is error what I get when parsing.

Cron.parseAndBuild("0 0 1/2 * *")
while I did try this, and it did react as every day. Upon reaching next month it started again with 1. Meaning I had date 29.01.2024, 31.01.2024, 01.02.2024, 03.02.2024, which is not exactly every 2nd day

So, if you need every second day, you should specify not day of month then.
Based on this generator it is correct this expression means every second day of month:
Tue Oct 29 2024 01:00:00 GMT+0100
Thu Oct 31 2024 01:00:00 GMT+0100
Fri Nov 01 2024 01:00:00 GMT+0100
Sun Nov 03 2024 01:00:00 GMT+0100


more training examples for manipulation

Dear Genforce group,
Thank you for your great work, I really like your work 'in-domain gan'.
As you state in the paper, in order to do image manipulation (male -> female), you use interfacegan to find the manipulation direction in wp, w or z. Since you use wp which is 14x more dimentions than w, you will need to generate more data to train the linear SVM, right? Interfacegan manipulates on w and train svm with 500k generated images, how many images you need to train svm on wp?
Notice that interfacegan uses standard stylegan, and wp is just broadcasting of w, they have the same intrinsic dimentions, so it can use the same amount of images to train on either w or wp. But in 'in-domain gan', you retrain the stylegan with different w instead of repeated w, so wp has larger intrinsic dimentions than each w.
Looking forward for your reply. Thanks in advance.

Actually, we can use the same number of samples for boundary training. Please refer to this issue. You can still treat the w codes for 14 layers independently. In this way, it is exactly the same as InterFaceGAN. Please see HiGAN for the details of layer-wise manipulation.

Thx, your answer is very helpful.


onStart but not onRest not called on immediate animations

I would expect either neither onStart or onRest, or both, be called with an immediate animation. Instead only onStart is called. This causes some issues in our application because I want to render the content slightly differently (lower quality images) if we are currently in an active animation. If I setState({ isAnimating: true }) in onStart it will not be reset if the animation is was in immediate mode.
<Spring
  immediate
  from={{ height: 10 }}
  to={{ height: 100 }}
  onStart={() => console.log("start")}
  onRest={() => console.log("rest")}
>
  {({ height }) => <div style={{ background: "hotpink", height }} />}
</Spring>

That's generally a hard call, theres some async stuff that relies on a started spring to conclude, for instance keyframes or now useChain. If a spring could just "do nothing" when started, promises would potentionally never finish. Do you need it for something particular?


Ensure that GenericL10n works if the locale files cannot be loaded

Ensure that localization works in the GENERIC viewer, even if the necessary locale files cannot be loaded.
This was the behaviour prior to the introduction of Fluent, and it seems worthwhile to keep that (especially since we already bundle the en-US strings anyway).

Simplify the NullL10n-implementation by having it extend the L10n-class, to reduce the maintenance burden of the code.

Stop exporting NullL10n in the viewer-components, since it was never intended to be used directly and only exists as a fallback.

Please note: This doesn't affect the Firefox PDF Viewer, thanks to the use of import maps.

/botio-linux preview

/botio test


No information on how to use Fargate launch type with secrets coming from Parameter Store

Thanks for the feedback @FarrukhNaeem. Looking in to this.


Include Vuex only when store is not empty

Not every Nuxt project uses Vuex. By default Nuxt includes Vuex when the store directory is present. If you don't use Vuex and forget to remove the directory, it's still loaded and increases vendor bundle.
I suggest we check whether store is not empty (README.md and .* are ommited).
This PR is compatible with #2733.

Codecov Report

Merging #2909 into dev will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##              dev    #2909      +/-   ##
==========================================
+ Coverage   99.91%   99.91%   +<.01%
==========================================
  Files          23       23
  Lines        1236     1237       +1
==========================================
+ Hits         1235     1236       +1
  Misses          1        1

Impacted Files
Coverage Δ

lib/common/options.js
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 0a82325...0a13fd9. Read the comment docs.

@liborvanek Thank you for that catch, duly noted.

@liborvanek  Thank you for the contribution.
@qm3ster Thank you for the patient review.

Pleasure is all mine!


DNS client: TCP connection reuse for TCP and DNS-over-TLS

Hi, while using the DNS client I have noticed a performance challenge if a TCP or DNS-over-TLS server is used to resolve queries, this compared against an UDP server (that said, 1.5x-2.0x times slower for TCP, and up to 4x times slower for DNS-over-TLS), so in the long term, there is a drastic performance drop for a heavily used DNS client (e.x.: local server proxy/forwarder).
That performance drop is mainly caused by the opening/closing of a new TCP connection for every DNS request,  and the required handshake things that are inherit to the protocol (worse for a TCP TLS handshake).
So, as an improvement, it may be good to have an option to reuse an already made TCP connection and send the requests over it, reducing the handshake things over and over again, specially for a DNS-over-TLS server, where a cert is involved.
As a general idea:

Add a new bool flag to enable/disable the reuse of TCP connections (not valid for UDP) client.go#L30
Add a new client attribute with the reusable connection client.go#L30
On every request, check if the reusable connection is NOT nil/closed, and use that connection client.go#L160
If the connection is nil/closed, make the Dial call to establish a new connection client.go#L162
Avoid closing the connection after the message exchange finishes client.go#L167
Add an idle timeout to check for inactive connections and automatically close them
Add a handler to check/capture connection state changes (I don't know if that is possible)

this is a difficult thing to implement and out of scope for this project.


SBE support

@bjoto pointed out that CME switched from FAST to SBE:
http://www.cmegroup.com/confluence/display/EPICSANDBOX/MDP3
So time to support SBE in libtrading.

@penberg Real Logic's (reference?) implementation appears to contain a C++ version. Have you had a look at that?


Feature: please port to aarch64

Please port the Lite wallet to aarch64 in order to be able to run it on Raspberry Pi (armv8-A, see for example https://www.mbtechworks.com/hardware/raspberry-pi-model-comparison.html).

Hi.
I think a RPI400 is a perfect device for a hardware wallet ;)
I run Ubuntu Mate on my pi.
So I bought one and tried to compile the software. 1st of all, I ran into the zewalletlib not being available. After entereing the folder /res/libzewalletlight, I ran the build.sh script. That failed with:
error: failed to run custom build command for zecwalletlitelib v0.1.0 (https://github.com/piratenetwork/piratewallet-light-cli?rev=5a7848381e93d4874375bddc7117efb73678b94e#5a784838)
Caused by:
process didn't exit successfully: /home/oliver/piratewallet-lite/res/libzecwalletlite/target/release/build/zecwalletlitelib-ce4a278201dd8752/build-script-build (exit code: 101)
--- stderr
thread 'main' panicked at 'called Result::unwrap() on an Err value: Os { code: 2, kind: NotFound, message: "No such file or directory" }', /home/oliver/.cargo/registry/src/github.com-1ecc6299db9ec823/tonic-build-0.1.1/src/lib.rs:226:14
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
warning: build failed, waiting for other jobs to finish...
error: build failed
cp: cannot stat 'target/release/libzecwalletlite.a': No such file or directory
Any ideas?


Feature/myforms

Summary | Résumé
Adds MyForm feature to display a list of templates in cards. Using a tab interface, the cards can be filtered by draft, published and all.
Test instructions | Instructions pour tester la modification
Run the app and click the MyForms link in the top admin nav. If you have no templates, first create a few in the FormBuilder. Try clicking the different tabs "draft", "published", "all". The related templates should be displayed. Each card also has a menu that displays a number of actions (currently pointing to fake links).
Unresolved questions / Out of scope | Questions non résolues ou hors sujet
Add action links to the card menu.
Pull Request Checklist
Please complete the following items in the checklist before you request a review:

[x] Have you completely tested the functionality of change introduced in this PR? Is the PR solving the problem it's meant to solve within the scope of the related issue?
[x] The PR does not introduce any new issues such as failed tests, console warnings or new bugs.
[x] If this PR adds a package have you ensured its licensed correctly and does not add additional security issues?
[x] Is the code clean, readable and maintainable? Is it easy to understand and comprehend.
[x] Does your code have adequate comprehensible comments? Do new functions have docstrings?
[ ] Have you modified the change log and updated any relevant documentation?
[x] Is there adequate test coverage? Both unit tests and end-to-end tests where applicable?
[x] If your PR is touching any UI is it accessible? Have you tested it with a screen reader? Have you tested it with automated testing tools such as axe?

Seeing this error when trying to run look at this PR.

Oddly I think this PR lost a commit or two and also seems to have pulled in other changes. I'm going to close this commit and try recreating it.


A BUG about picking up entities

Sandcastle example:
bug demo
Browser:
Chrome 106.0.5249.103

Operating System:
Windows 10

Description:
Please load demo and enlarge the map
There is no need to rotate the camera Angle
You'll find it hard to pick up the red entities

Thanks for the report @ShenWeiQun. I can reproduce.
The selection indictator also appears above the green entity one selected:

I think this is actually a duplicate of https://github.com/CesiumGS/cesium/issues/7116. I'm going to close your issue to keep the discussion in one place. If you have any further input on this, please post it there instead. Thanks!


How to setup and where to launch this?

$ sudo docker run -d --link redis:redis -p 8000:8000 -p 9000:9000 -v /path/to/config:/config --restart=always --name automatron madflojo/automatron

$ command not found

Must I first clone the project first?
As I cloned project and tried :
$ sudo docker run -d --link redis:redis -p 8000:8000 -p 9000:9000 -v /$HOME/automatron/config:/config --restart=always --name automatron madflojo/automatron
-bash:
$: command not found

EDIT:
Just need to
# apt-get update
# apt-get install sudo

But
# sudo apt-get install python2.7 python-dev \
                       python-pip redis-server \
                       nmap git libffi-dev \
                       build-essential libssl-dev
Reading package lists... Done
Building dependency tree
Reading state information... Done
E: Unable to locate package libs

Also do I open within the docker container or host
docker inspect --format '{{ .NetworkSettings.IPAddress }}' 5d587aeb87ed
<IP_ADDRESS>
http://<host ip>:8080
http://<IP_ADDRESS>:8080
not sure how to host host ip as this can't connect and show dashboard?

@scheung38 I made some documentation changes in #116 do these help clarify things a bit?

Closing this for now, but feel free to reopen and comment if you have questions still.


Hello-Board suggests espmonitor when it is not installed or configured

02_4_hello_board.md Line 71 suggests the use of espmonitor to monitor your Rust board. Using the software install instructions in this repo, this command did not work out of the box. I had to run cargo install espmonitor and then use the command as follows:
$ cargo espmonitor -c esp32c3 /dev/ttyACM0

I'm not certain if this is an issue with my install, or simply an oversight. If it's an oversight, I can try to submit a pull request with a more complete software download list, and a corrected command invocation.

espmonitor is no longer recommended, we now use espflash and cargo-espflash instead.


Using uni_links and multi page app

I am trying to open a specific screen when a user clicks a URL link.
If the user is on my home page or if the app is fully closed, I manage to intercept the link and show the correct page.
If the user has navigated to another page of the app and the app is running (in background for example), clicking the link simply opens the app where the user left it.
How can i intercept and parse the link to redirect him to the desired page no matter where he might be in the app ?
So i have to copy the code on every single screen of the app ?
Thanks
`import 'dart:async';
import 'package:flutter/material.dart';
import 'package:flutter/services.dart';
import 'package:flutter_secure_storage/flutter_secure_storage.dart';
import 'package:myapp/app.config.dart';
import 'package:myapp/mobile/screens/login.dart';
import 'package:myapp/mobile/screens/inviteView.dart';
import 'package:uni_links/uni_links.dart';
/// The goal of this splash screen is to give time to the app
/// to retrieve the current database or redirect to the login page.
/// We may want to add a different splash screen natively to avoid
/// having a white screen when the user launches the app
class SplashScreen extends StatefulWidget {
const SplashScreen({Key key}) : super(key: key);
@override
_SplashScreenState createState() => _SplashScreenState();
}
class _SplashScreenState extends State {
Future ready;
Uri _latestUri;
StreamSubscription _sub;
@override
void initState() {
super.initState();
SystemChannels.textInput.invokeMethod('TextInput.hide');
ready = initPlatformStateForUriUniLinks();
}
@override
void dispose() {
if (_sub != null) _sub.cancel();
super.dispose();
}
/// An implementation using the [Uri] convenience helpers
Future initPlatformStateForUriUniLinks() async {
// Attach a listener to the Uri links stream
_sub = getUriLinksStream().listen((Uri uri) {
if (!mounted) return true;
setState(() {
_latestUri = uri;
});
}, onError: (err) {
if (!mounted) return true;
setState(() {
_latestUri = null;
});
});
// Attach a second listener to the stream
getUriLinksStream().listen((Uri uri) {
  print('got uri: ${uri?.path} ${uri?.queryParameters}');
}, onError: (err) {
  print('got err: $err');
});

// Get the latest Uri
Uri initialUri;
// Platform messages may fail, so we use a try/catch PlatformException.
try {
  initialUri = await getInitialUri();
} on PlatformException {
  initialUri = null;
} on FormatException {
  initialUri = null;
}

// If the widget was removed from the tree while the asynchronous platform
// message was in flight, we want to discard the reply rather than calling
// setState to update our non-existent appearance.
if (!mounted) return true;

setState(() {
  _latestUri = initialUri;
});

return true;

}
@override
Widget build(BuildContext context) {
return FutureBuilder(
future: ready,
builder: (context, AsyncSnapshot snapshot) {
if (snapshot.hasError) return Center(child: Text('Error validating invitation'));
if (!snapshot.hasData) return Center(child: Text('Loading'));
    if (_latestUri != null) {
      final queryParams = _latestUri?.queryParameters;
      final config = AppConfig.of(context);
      final baseUrl = config.apiBaseUrl;
      switch (_latestUri.path) {
        case '/acceptInvite/':
          final token = queryParams['token'];
          return AcceptInviteView(baseUrl, token);
          break;
        default:
          break;
      }
    }

    return LoginScreen();
  },
);

}
}`

This would be a killer feature! I would love to for example use Navigator.pushReplacementNamed(context, 'address_i_got_from_uni_links'); navigating to specific page in app, without copy-pasting code uni_links handler all over widgets.

@charlesrostaing
The link handling should be done in 2 places:

When the app starts. Use getInitialUri. If the app was opened with a uri, it will have the value.
using getUriLinksStream. If the app is already open the stream will have a new value with the link in it.
Note: When the user opens a link and your app already exist (in background), Android will first open your app and then you will get the event about the link. This means the user wll see the screen the app was before they moved it to the background, no matter what you do.

I hope that helps!

@

@charlesrostaing
The link handling should be done in 2 places:

When the app starts. Use getInitialUri. If the app was opened with a uri, it will have the value.
using getUriLinksStream. If the app is already open the stream will have a new value with the link in it.
Note: When the user opens a link and your app already exist (in background), Android will first open your app and then you will get the event about the link. This means the user wll see the screen the app was before they moved it to the background, no matter what you do.

I hope that helps!

Consider the case of app having Splash screen, Product Listing (HomeScreen), Product details screen & other screens. In this case, the shared urls are of different products which should ideally open in 'Product Details' screen.
If we add listeners in Product Listing, redirection is working fine, either from coming background or app was opened. But, if the user is in Product details & we open different link, nothing will happen unless we add listeners there too.
In such case, which place you suggest of adding listeners? Scenario being, in all cases, regardless of where the user is, redirection should happen, without adding listeners everywhere.

@rohankandwal

If we add listeners in Product Listing, redirection is working fine, either from coming background or app was opened. But, if the user is in Product details & we open different link, nothing will happen unless we add listeners there too.

You can have a common parent widget. So your top widget would be something like MyApp, which would have the listener (and other "global" stuff), and the build() method should call the appropriate child builder (HomeScreen, ProductDetailsScreen, etc).


Malformed header triggers errors rather than exception handling

When sending a POST call to /auth, an errored header with junky fields, triggers a 500 server error. I assume that it should send back a response with the error code or some exception handling message.
To replicate this, type some junk into the key and value, then make a call. If you get the error I’m seeing, on a dev instance, the error handler spits out:
AttributeError: 'NoneType' object has no attribute 'get'
Then the callback ends with:
There’s a long list of errors, which end with:
File "D:_xampp\htdocs_github\spear_api\flask\venv\lib\site-packages\flask_jwt_init_.py", line 115, in _default_auth_request_handler
username = data.get(current_app.config.get('JWT_AUTH_USERNAME_KEY'), None)
It looks like this is related to  an old issue, so perhaps this has reemerged:
https://github.com/mattupstate/flask-jwt/issues/80

This project has been abandoned for quiet a while now. Perhaps check out flask-jwt-extended as an alternative. https://flask-jwt-extended.readthedocs.io/en/latest/


update wasmtime

same as https://github.com/YoWASP/yosys/issues/22

Done in 59268d0.

Pushed to PyPI.

Thanks for fixing quickly!


useIsomorphicLayoutEffect

https://github.com/reduxjs/react-redux/blob/master/src/hooks/useSelector.js#L14-L15

:tada: This issue has been resolved in version 9.10.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

This implementation is broken in React Native, causing useEffect to always be used.
More info: https://github.com/reduxjs/react-redux/pull/1444

FWIW, I just published react-layout-effect as a solution.
https://www.npmjs.com/package/react-layout-effect
Maybe react-use should depend on and re-export it?
I'll add you as a maintainer if that's what you decide.