Tensorflow code does not work

I tried to follow the example as is, but get the following error.
ValueError                                Traceback (most recent call last)
 in ()
8             batch_x = train_x[offset:(offset + batch_size), :, :, :]
9             batch_y = train_y[offset:(offset + batch_size), :]
---> 10             _, c = session.run([optimizer, loss],feed_dict={X: batch_x, Y : batch_y})
11             cost_history = np.append(cost_history,c)
12         print("Epoch: ",epoch," Training Loss: ",c," Training Accuracy: ",
/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/tensorflow/python/client/session.py in run(self, fetches, feed_dict, options, run_metadata)
875     try:
876       result = self._run(None, fetches, feed_dict, options_ptr,
--> 877                          run_metadata_ptr)
878       if run_metadata:
879         proto_data = tf_session.TF_GetBuffer(run_metadata_ptr)
/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/tensorflow/python/client/session.py in _run(self, handle, fetches, feed_dict, options, run_metadata)
1074                              'which has shape %r' %
1075                              (np_val.shape, subfeed_t.name,
-> 1076                               str(subfeed_t.get_shape())))
1077           if not self.graph.is_feedable(subfeed_t):
1078             raise ValueError('Tensor %s may not be fed.' % subfeed_t)
ValueError: Cannot feed value of shape (10, 4) for Tensor 'Placeholder_1:0', which has shape '(?, 6)'

Kindly read the error, ValueError: Cannot feed value of shape (10, 4) for Tensor 'Placeholder_1:0', which has shape '(?, 6)'. You are trying to feed the data (labels possibly) of array that has shape of (?, 4) which Y placeholder has a shape of (?, 6). Change the shape of Y (or in other words num_labels) to 4 as
Y = tf.placeholder(tf.float32, shape=[None, num_labels])

Thank you Aqib,
I get it now. I copied the notebook without reading. there num_labels is defined as 6 in your code.
Thank you so much.
Kind Regards,
Arjun Bhandari
+97150 780 5672

On Oct 8, 2018, at 1:42 PM, Aaqib<EMAIL_ADDRESS>wrote:
Y = tf.placeholder(tf.float32, shape=[None, num_labels])

Hello,
CNN part is giving an error
File "", line 8, in
batch_y = train_y[offset:(offset + batch_size), :]
IndexError: too many indices for array
I have not changed your code at any point, as I'm in the learning and understanding the CNN.
It would be great if you could help me in this regard
thanks in advance for your valuable time

print(X)
Tensor("Placeholder_2:0", shape=(?, 1, 90, 3), dtype=float32)
print(Y)
Tensor("Placeholder_3:0", shape=(?, 6), dtype=float32)
print(c_flat)
Tensor("Reshape_1:0", shape=(?, 1080), dtype=float32)
print(shape)
[None, 1, 1, 1080]


Why functools.partial is not accept as wraper?

I believe that these two code should behave the same but with the second one taking advantage of the functools.partial resources:
(I'm trying to write a decorator that may accept arguments or not, like dataclass decorator)
from typing import Callable, overload

@overload
def foo[**P, T](func: Callable[P, T]) -> Callable[P, T]: ...

@overload
def foo[**P, T](**kwargs) -> Callable[[Callable[P, T]], Callable[P, T]]: ...

def foo[**P, T](func: Callable[P, T] | None = None, **kwargs) -> Callable[P, T] | Callable[[Callable[P, T]], Callable[P, T]]:
    # foo called as decorator without arguments
    if func is not None:
        # do something using func
        return func

    def wrap(func: Callable[P, T]) -> Callable[P, T]:
        # do something using kwargs
        # do something using func
        return func

    # foo is called as decorator with arguments
    return wrap

Now using functools.partial:
from functools import partial
from typing import Callable, overload

@overload
def foo[**P, T](func: Callable[P, T]) -> Callable[P, T]: ...

@overload
def foo[**P, T](**kwargs) -> Callable[[Callable[P, T]], Callable[P, T]]: ...

def foo[**P, T](func: Callable[P, T] | None = None, **kwargs) -> Callable[P, T] | Callable[[Callable[P, T]], Callable[P, T]]:
    # foo called as decorator without arguments
    if func is not None:
        # do something using func
        return func

    # do something using kwargs

    # foo is called as decorator with arguments
    return partial(foo, **kwargs) # -> error here

But in the second approach, pylance is raising an error which ends with the confusing message:
    Type "partial[(**P@foo) -> T@foo]" is not assignable to type "((**P@foo) -> T@foo) -> ((**P@foo) -> T@foo)"
      Function return type "(**P@foo) -> T@foo" is incompatible with type "(**P@foo) -> T@foo"
        Type "(**P@foo) -> T@foo" is not assignable to type "(**P@foo) -> T@foo

What am I missing here?

Thanks for clarifying!


What is it?

https://github.com/guzzle/guzzle/blob/master/src/Cookie/CookieJar.php#L89
This function or method? I do not understand

+1. Guzzle's Cookie implementation is extremely undiscoverable and the documentation on it is fractured and poor.
I can't even set a damn cookie to pass on to another request.

This function or method? I do not understand

It's a static method on the class. You don't need to worry about it unless you're implementing a custom CookieJar.

+1. Guzzle's Cookie implementation is extremely undiscoverable and the documentation on it is fractured and poor.

The point you're making is not related to the issue as reported. It sounds like you're asking how to add cookies to a request, which is not related to whether or not a method is static.
If you want to maintain state between cookies, then you need to use the cookies request option.
// Persist cookies across all requests
$jar = new \GuzzleHttp\Cookie\CookieJar();
$client = new GuzzleHttp\Client(['cookies' => $jar]);
$client->request('GET', '/get');

If you want to add ad-hoc cookies to a request, then you need to set a header on a request:
// Add ad-hoc cookies
$response = $client->request('GET', '/', ['headers' => 'Cookie: foo=bar; baz=bam']);

Maybe then you can help me then, mtdowling. My past 3 hours have resulted in nothing but frustration.
I am interacting with an API that on authentication, responds with a cookie (or set of cookies, I haven't yet worked that out yet).
To perform any further request with this API, I need to resend that cookie I was provided with back to them.
What I have so far:
Construct
    $this->jar = new CookieJar();
    $this->client = new Client([
        'cookies' => $this->jar
    ]);

Authenticate
    const SPACE_TRACK_URL = 'https://space-track.org';

    $response = $this->client->post(Ephemeris::SPACE_TRACK_URL . '/ajaxauth/login', [
        'form_params' => [
            'identity' => $this->identity,
            'password' => $this->password
        ]
    ]);
    $this->jar = CookieJar::fromArray($response->getHeaders()['Set-Cookie'], Ephemeris::SPACE_TRACK_URL);

Perform actual requests
    $response =  $this->client->request('GET', Ephemeris::SPACE_TRACK_URL . $url);
    return $response;

Attempting to perform an actual request results in a 401 unauthorized error, presumably because I haven't sent any (or just malformed) cookies through.
Is there anything I'm doing wrong on the setup side here?


Fixes #487(image index availability check)

Fixes #487
Fixes #487 by @aldenstpage
Description
Implemented a check mechanism in the /healthcheck view to verify that Elasticsearch image index exists and is available. If this check succeeds, a 200 OK is returned and newly deployed instances can start receiving production traffic and if it fails, a 500 Internal Server Error is returned instead, and an informative error logged, and the server is deregistered from the load balancer and destroyed. The returned response is  cached for 10seconds in order to cut AWS call costs.
Tests
Test to verify that the  /healthcheck response is truly being cached for 10seconds :
Startup redis-cli on the container
sudo docker exec -it cccatalog-api_cache_1 redis-cli
Flush all existing caches to ensure you don't see stale data
FLUSHALL
Enter an interactive session where you see every command sent to redis
MONITOR
Reload the /healthcheck page and on your terminal you should see some SET* operations storing the cache data.
Reload again and if your cache works, you should now see some GET* operations retrieving the cached data.
Checklist

[x ] My pull request has a descriptive title (not a vague title like Update index.md).
[ x] My pull request targets the master branch of the repository.
[ x] My commit messages follow best practices.
[x ] My code follows the established code style of the repository.
[ ] I added tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[ x] I tried running the project locally and verified that there are no
visible errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

Hi Alden, I seem to have resolved all the issues you pointed out.
However, all that's left now is testing the travis CI build(especially the unit test that waits for the server to be healthy), as I have modified the test conditions to allow a "started, but there's no data" state to pass, using jq as suggested. So I just need a way to verify that the changes are picked up and are working as planned.
Is there a way I could test the travis build locally or should I go ahead and make a commit and see if the build passes.

It's OK to go ahead and push your commits and treat this as a work-in-progress. I suggest testing the command outside of Travis also so you don't have to "test travis" repeatedly.

It's OK to go ahead and push your commits and treat this as a work-in-progress. I suggest testing the command outside of Travis also so you don't have to "test travis" repeatedly; the build takes a while to run.

It'd be nice to point out that my need for the test is not necessarily about the syntactical correctness of the  command(as similar command works locally with jq installed), rather I'd love to confirm that jq which formed part of the command was installed correctly and is working cos I can't seem to find a way to verify that without running the build.
Here's what I added to the dockerfile to hopefully perform the installation.

Hi Alden, So the build failed again, most definitely due to jq installation failing as I'd anticipated.
Something I'm doing/did wrong ?
Here are the logs:

ERROR: Service 'ingestion-server' failed to build: The command '/bin/sh -c wget --no-check-certificate https://raw.githubusercontent.com/stedolan/jq/master/sig/jq-release.key -O /tmp/jq-release.key &&     wget --no-check-certificate https://raw.githubusercontent.com/stedolan/jq/master/sig/v${JQ_VERSION}/jq-linux64.asc -O /tmp/jq-linux64.asc &&     wget --no-check-certificate https://github.com/stedolan/jq/releases/download/jq-${JQ_VERSION}/jq-linux64 -O /tmp/jq-linux64 &&     gpg --import /tmp/jq-release.key &&     gpg --verify /tmp/jq-linux64.asc /tmp/jq-linux64 &&     cp /tmp/jq-linux64 /usr/bin/jq &&     chmod +x /usr/bin/jq &&     rm -f /tmp/jq-release.key &&     rm -f /tmp/jq-linux64.asc &&     rm -f /tmp/jq-linux64' returned a non-zero code: 2```

It looks like jq ships with Travis CI already based on this - https://github.com/travis-ci/travis-ci/issues/7954
You can use the Travis CI docker image to explore the Travis image and experiment. There's a good guide here: https://andy-carter.com/blog/setting-up-travis-locally-with-docker-to-test-continuous-integration

Hi Alden, So the build failed again, most definitely due to jq installation failing as I'd anticipated.
Something I'm doing/did wrong ?
Here are the logs:

ERROR: Service 'ingestion-server' failed to build: The command '/bin/sh -c wget --no-check-certificate https://raw.githubusercontent.com/stedolan/jq/master/sig/jq-release.key -O /tmp/jq-release.key &&     wget --no-check-certificate https://raw.githubusercontent.com/stedolan/jq/master/sig/v${JQ_VERSION}/jq-linux64.asc -O /tmp/jq-linux64.asc &&     wget --no-check-certificate https://github.com/stedolan/jq/releases/download/jq-${JQ_VERSION}/jq-linux64 -O /tmp/jq-linux64 &&     gpg --import /tmp/jq-release.key &&     gpg --verify /tmp/jq-linux64.asc /tmp/jq-linux64 &&     cp /tmp/jq-linux64 /usr/bin/jq &&     chmod +x /usr/bin/jq &&     rm -f /tmp/jq-release.key &&     rm -f /tmp/jq-linux64.asc &&     rm -f /tmp/jq-linux64' returned a non-zero code: 2```

> It looks like `jq` ships with Travis CI already based on this - [travis-ci/travis-ci#7954](https://github.com/travis-ci/travis-ci/issues/7954)
>
> You can use the Travis CI docker image to explore the Travis image and experiment. There's a good guide here: https://andy-carter.com/blog/setting-up-travis-locally-with-docker-to-test-continuous-integration

Thanks for the heads up on `jq` shipping with Travis. I've just gotten rid of the extra jq install command, and made a fresh commit.

Regards setting up Travis CI locally, just went through the referenced tutorial, looks straight forward. would try to set that up once i get on WiFi(currently on mobile cellular data) cos the `travis-ci/python` image looks pretty large.

Thanks for the heads-up on jq shipping with Travis CI. I've gotten-rid of the extra jq installation commands and made a new commit.
Regards setting up Travis CI locally, the referenced tutorial looks pretty straight-forward. Would try to set it up once i get on WiFi(currently on metered mobile data) cos the travis-ci/python image looks pretty large(~2GB).

There was something wrong with our Travis setup (not related to your changes) that was preventing it from running. I wanted to give you a heads up that since addressing that problem, the build has failed on this PR.

Hey Alden!
Been away for a while, just getting back on.  Sensed there was something wrong cos I saw my build was running for days.
Let me check out the logs.

So I just viewed the build logs, turns out for some reasons, the bash command to pick up the IndexMissing response in order to allow the server run nevertheless, seem not to be working.
Meanwhile, the bash command, as far as I know looks syntactically OK, and should work if truly jq is pre-installed like you mentioned.
- bash -c 'while [[ "$(curl --insecure -s -o /dev/null http://localhost:8000/healthcheck | jq '.error')" != "IndexMissing" ]] || [[ "$(curl --insecure -s -o /dev/null -w ''%{http_code}'' http://localhost:8000/healthcheck)" != "200" ]]; do sleep 10 && docker logs cccatalog-api_web_1; done'
Also, referencing this comment, since a "started, but there's no data" state shouldn't necessarily cause the healthcheck to fail, should we still respond with a 500 to that condition?
I'm of the opinion we modify the response  code since it's not actually a "critical" server error that should prevent startup. That way, we can be able to modify the above referenced bash command to just check for either a 200 response or the newly modified response code to ascertain that the healthcheck passed. Not only will this be more straight-forward and less complicated, it would also eliminate the need for jq which seems to be giving issues.

That's a fair point that it isn't a fatal exception in the build context. In production, though, it would be problematic if we pointed our servers to an empty cluster, and if we can catch that during the deployment phase through healthchecks, it will save us a major headache.
One more alternative that may be simpler than parsing JSON in the build is introducing a new environment variable called EMPTY_INDEX_FAILS_HEALTHCHECK (default to True, but False in Travis).

Thanks for the environmental variable insight  Alden.
However, I still can't seem to wrap my head around how to go about that. While I await further clarifications, I observed possible loopholes in my code that likely might have caused jq not to work(strongly assuming it's truly pre-installed).
before now in my healthcheck implementation, an empty index state returns the following response body(as you suggested):
{     "error": "IndexMissing",     "detail": "The image index does not exist. Index data to Elasticsearch." }
while a healthy index in turn returns just a 200 with an empty response body.
And here's how I tested these conditions previously in the travis build script, which ended up failing:
- bash -c 'while [[ "$(curl --insecure -s -o /dev/null http://localhost:8000/healthcheck | jq '.error')" != "IndexMissing" ]] || [[ "$(curl --insecure -s -o /dev/null -w ''%{http_code}'' http://localhost:8000/healthcheck)" != "200" ]]; do sleep 10 && docker logs cccatalog-api_web_1; done'
But a closer look at the above implementation revealed possible downsides to this approach as I was combining two "not so similar" conditions.
So here are the modifications I've made:

A healthy index status state returns not just a 200 but includes a response body like
{   "status": "IndexHealthy",   "detail": "The image index is healthy." }

Modified the empty index response body to keep the keys same for easy access with jq
{   "status": "IndexMissing",   "detail": "The image index does not exist. Index data to Elasticsearch." }

Then in the travis build script, I balanced up the conditions on both sides by using just information from the response bodies as against status codes. Thus, I now have:

- bash -c 'while [[ "$(curl --insecure -s -o /dev/null http://localhost:8000/healthcheck | jq '.status')" != "IndexMissing" ]] || [[ "$(curl --insecure -s -o /dev/null http://localhost:8000/healthcheck | jq '.status')" != "IndexHealthy" ]]; do sleep 10 && docker logs cccatalog-api_web_1; done'
Think this might worth a commit for testing? It's so late here, been working through the night and feeling very sleepy now. Pardon me if this reflects in the comment.

Yes, let's see if this new jq approach works – if not I'm happy to help out with setting the environment variables instead

Hi Alden.
So I got a replacement for my old machine a few days ago. I've successfully
transferred the repo files(my clone) to the new one and setup the project
locally once more. However, I can't figure how to get everything in sync as
it used to be as regards to git.
I've also tried googling, but it seems I don't have the right words :)
Can you point me to something comprehensive?
Note: I actually had changes that haven't been pushed to the remote branch
prior to this replacement.
On Thu, May 28, 2020, 20:57 Alden S Page<EMAIL_ADDRESS>wrote:

Yes, let's see if this new jq approach works – if not I'm happy to help
out with setting the environment variables instead
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/creativecommons/cccatalog-api/pull/492#issuecomment-635563130,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGFLMYHQC7IHJ2L6DQW3BN3RT26ZHANCNFSM4M62QSIA
.

Hi Alden. So I got a replacement for my old machine a few days ago. I've successfully transferred the repo files(my clone) to the new one and setup the project locally once more. However, I can't figure how to get everything in sync as it used to be as regards to git. I've also tried googling, but it seems I don't have the right words :) Can you point me to something comprehensive? Note: I actually had changes that haven't been pushed to the remote branch prior to this replacement.
…
On Thu, May 28, 2020, 20:57 Alden S Page @.***> wrote: Yes, let's see if this new jq approach works – if not I'm happy to help out with setting the environment variables instead — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#492 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGFLMYHQC7IHJ2L6DQW3BN3RT26ZHANCNFSM4M62QSIA .

This has been resolved:)
Pushing the changes in a few!

Yes, let's see if this new jq approach works – if not I'm happy to help out with setting the environment variables instead

Hi Alden, can we take the environment variables route? Seems jq isn't working.

This repo has entered maintenance mode so we will not be merging this PR.


examples: add $(DEVKITPRO)/tools/bin to path

This is where gbafix and the gbfs tools reside these days, it seems.
This fixes the build on Linux for me.

Superseded by #3.


this is a pretty paralyzing issue. does anyone know how long it will take for it to get merged in?

this is a pretty paralyzing issue. does anyone know how long it will take for it to get merged in?

In the meantime, you can pip install git+https://github.com/jamalex/notion-py.git@refs/pull/294/merge or replace notion-py with git+https://github.com/jamalex/notion-py.git@refs/pull/294/merge in your requirements.txt. I have tested and this fixed my issues 👍
Originally posted by @gnestor in https://github.com/jamalex/notion-py/issues/294#issuecomment-804306525

I follow this merge guide to solve the limit error on the script but now I am running with this new issue "raise HTTPError( requests.exceptions.HTTPError: Something went wrong."
it seems like it works at some point but the data does not show on the notion database.

Any guides or help would be appreciated.


docs: Fix api.js docs pipeline with updated typedoc dependencies

What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[x] Docs
[ ] New Binding issue #___
[ ] Code style update
[ ] Refactor
[x] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change?

[ ] Yes, and the changes were approved in issue #___
[x] No

Checklist

[x] When resolving issues, they are referenced in the PR's title (e.g fix: remove a typo, closes #___, #___)
[x] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.
[x] I have added a convincing reason for adding this feature, if necessary

Other information
This was what broke the publishing pipeline when we tried to release 1.3 - it should work now but i'll also need to see if the docs are still correct by integrating it into tauri-docs before we merge this since the js-api.json file is much bigger now and i'm not sure yet if that's correct...

Okay looks like they just changed the output format a bit. Content itself looks fine and since we don't actively use the json output yet the change won't affect us.
The actual docs on tauri.app will still be generated with 0.23 though since our fork of the markdown plugin diverged from upstream too much and i can't (and don't want to) figure it out how to update it right now.


feat: add fonts config to the ReactRuntime

What/why?
Adds registering fonts as apart of the ReactRuntime instead of the MakeswiftApiHandler. This is some prep work to get fonts out of the API handlers since it should be the concern of the runtime, not the API.

Added the manifest flag and updated to a minor ♻ 🙏


Support linux older than glibc2.29

On linux lincimgui.so is built against glibc2.29.
Log from a RHEL8 / Rocky8 which based on glibc2.28 :
/lib64/libm.so.6: version `GLIBC_2.29' not found (required by [...]/bin/Debug/net8.0/runtimes/linux-x64/native/libcimgui.so)

rhel8/rocky8 are long-term distributions, that's why it could be interesting to have another linux version in the nuget package.
Is it an option?
Thank you for considering the question.
Colin.

that's a https://github.com/cimgui/cimgui question. Please post it there otherwise correct me if I am wrong.

cimgui is not built at all. You choose compiler.

oh i see, @colaub  can you help me figure out which linux version i should use such that I get glibc2.28
ref: https://github.com/ImGuiNET/ImGui.NET-nativebuild/blob/master/.github/workflows/build.yml#L29


Can't download Complete in 0.1.39 rc

What is the issue?
In the latest 0.1.39 version, I am almost unable to complete the model download. It often breaks off and prompts "Error: max retries exceeded: EOF". I have been trying for a whole day but still haven't finished downloading a 4Gb model.
I hope this issue can be fixed soon. Thanks!
OS
Windows
GPU
Nvidia
CPU
Intel
Ollama version
0.1.39

I think it's fixed here: https://github.com/ollama/ollama/pull/4619
And it's duplicate (eg: https://github.com/ollama/ollama/issues/4633)

Closing as a duplicate.


make _fzf_complete_COMMAND work for aliases of COMMAND

[x] I have read through the manual page (man fzf)
[x] I have the latest version of fzf
[x] I have searched through the existing issues

Info

OS

[ ] Linux
[x] Mac OS X
[ ] Windows
[ ] Etc.

Shell

[ ] bash
[x] zsh
[ ] fish

Problem / Steps to reproduce
I'm trying to add _fzf_complete_COMMAND for timew (timewarrior), which works fine.
Now I have a few aliases defined that spare me from typing "timew command" all the time (twa, twc, two and the like)
Following your suggestion in #1368, I could write functions for every  of these aliases and make them use the one for timew, but I wonder if there's a better option.
Is there any possibility to first expand the alias and then apply the completion function?
Thanks,
Sven

Thanks for the suggestion. I guess it's possible, but I'm not going to implement it here, because of the added complexity and because it's unclear if the behavior is always desirable.

I could write functions for every of these aliases and make them use the one for timew, but I wonder if there's a better option.

Maybe you could write an alias wrapper that defines the alias AND the completion function for the alias.
alias_for_fzf 'twa' 'timewarriror something'


sample code for using IAM roles to authenticate with DocDB

Issue #, if available:
Description of changes:
sample code for using IAM roles to authenticate with DocDB

Reviewed iam_role_sample_code / Password-less authentication to Amazon DocumentDB using IAM Roles


High Availability

it would be good to include a note as to what the high availability options are, and if there is none, then make it explicit. This

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

ID: 115ae83a-95dd-5643-bc88-f5447ebb2b99
Version Independent ID: 7462ce84-0a9f-4bfd-7eba-c00928d77865
Content
Content Source
Service: active-directory

@camiloborges Today you can configure a second Azure AD Connect instance and put it in staging mode but it requires manual intervention if the original server were to go down. It is something that the product group is aware of and working on but no dates to share. Users would still be able to sign in, just writeback operations would fail in the context of this article.
Information regarding staging mode is found in the Azure AD Connect docs under supported topologies. https://docs.microsoft.com/azure/active-directory/connect/active-directory-aadconnect-topologies#staging-server
I didn't include it in this doc as password writeback is only one of the many features of Azure AD Connect.

#in-progress

#in-process

#please-close


Lag on recording

Recording with OBS and having like 5 fps on recording, any fix?

buy new pc?

It has nothing to do... I'm having in game 200 fps, in obs I have no drop frames but on recording it keeps stuttering

how much is the bitrate?

It has nothing to do... I'm having in game 200 fps, in obs I have no drop frames but on recording it keeps stuttering

What CPU do you have?

Ryzen 5 2600... with any other cheat or without cheat I have no problem

Ryzen 5 2600... with any other cheat or without cheat I have no problem

That’s really weird, when I record I have no problems. Are you using game capture or display capture?

I've tried both.. same problem


typealias should be public

public typealias 戟 = Ji
public typealias 戟节点 = JiNode

@frogcjn Thanks! I've updated it. btw, feel free to make a PR.


Create Hoverable-table.html

Made transition in table and all row color is white while hover.

Made an change. Removed the scale effect and cursor pointer on hover. You can check the changes. @nik132-eng


Npm task: Allow selection of feed view for internal registries

When using Azure DevOps, I wanted to use views on the registry to distinguish in-development packages from stable or production ready packages. For this I want to use views (@release, @prerelease) to designate those packages.
Unfortunately the npm task in my pipeline does not allow me to select the view for the selected internal feed / registry.
Please make it possible to select the view, if one wishes to not use the default one.
As a nice to have feature, also disable the publish command, when a view is selected, though the task will fail anyway when run.
Workaround: manually pass the --registry parameter to the install command.

Hello! I'm a PM over on the Azure Artifacts team. I wanted to drop in here and address some of these issues with the news that we're currently reevaluating the types of tasks we offer in Azure Pipelines, and looking to focus more on lighter-weight tasks that cover authentication and setup. In lieu of the task redesign, we're going to forego making changes to the current tasks (with a few exceptions). We hope the new tasks we spin up in the next quarter or so are easier and more helpful, be on the lookout!
Addition: it also looks like you have a workaround here, so I'm assuming you're not completely blocked by this issue, I hope!


deno: 1.40.5 -> 1.41.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for deno is: A secure runtime for JavaScript and TypeScript
meta.homepage for deno is: https://deno.land/
meta.changelog for deno is: https://github.com/denoland/deno/releases/tag/v1.41.0
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.41.0 with grep in /nix/store/3dr166j418zazfc5spmr8p4ivm2himw9-deno-1.41.0
found 1.41.0 in filename of file in /nix/store/3dr166j418zazfc5spmr8p4ivm2himw9-deno-1.41.0

Rebuild report (if merged into master) (click to expand)

12 total rebuild path(s)

12 package rebuild(s)

First fifty rebuilds by attrpath
deno
python311Packages.nbdev
python312Packages.nbdev
quarto
quartoMinimal
rstudio
rstudio-server
rstudioServerWrapper
rstudioWrapper
vimPlugins.denops-vim
vimPlugins.vim-pluto
windmill

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/3dr166j418zazfc5spmr8p4ivm2himw9-deno-1.41.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A deno https://github.com/r-ryantm/nixpkgs/archive/0824727aa4af475bd2365be74997ea931cb77c66.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/0824727aa4af475bd2365be74997ea931cb77c66#deno

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/3dr166j418zazfc5spmr8p4ivm2himw9-deno-1.41.0
ls -la /nix/store/3dr166j418zazfc5spmr8p4ivm2himw9-deno-1.41.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

12 packages built:

deno
python311Packages.nbdev
python312Packages.nbdev
quarto
quartoMinimal
rstudio
rstudio-server
rstudioServerWrapper
rstudioWrapper
vimPlugins.denops-vim
vimPlugins.vim-pluto
windmill

Maintainer pings
cc @06kellyjac for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for deno is: A secure runtime for JavaScript and TypeScript
meta.homepage for deno is: https://deno.land/
meta.changelog for deno is: https://github.com/denoland/deno/releases/tag/v1.41.1
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.41.1 with grep in /nix/store/lj960ff59rngrnj1ds1z9xzzsvzvn1xg-deno-1.41.1
found 1.41.1 in filename of file in /nix/store/lj960ff59rngrnj1ds1z9xzzsvzvn1xg-deno-1.41.1

Rebuild report (if merged into master) (click to expand)

12 total rebuild path(s)

12 package rebuild(s)

First fifty rebuilds by attrpath
deno
python311Packages.nbdev
python312Packages.nbdev
quarto
quartoMinimal
rstudio
rstudio-server
rstudioServerWrapper
rstudioWrapper
vimPlugins.denops-vim
vimPlugins.vim-pluto
windmill

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/lj960ff59rngrnj1ds1z9xzzsvzvn1xg-deno-1.41.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A deno https://github.com/r-ryantm/nixpkgs/archive/c99f6ad6dedb8793fa62588c473b25709dd7a26c.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/c99f6ad6dedb8793fa62588c473b25709dd7a26c#deno

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/lj960ff59rngrnj1ds1z9xzzsvzvn1xg-deno-1.41.1
ls -la /nix/store/lj960ff59rngrnj1ds1z9xzzsvzvn1xg-deno-1.41.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

14 packages built:

deno
python311Packages.nbdev
python311Packages.nbdev.dist
python312Packages.nbdev
python312Packages.nbdev.dist
quarto
quartoMinimal
rstudio
rstudio-server
rstudioServerWrapper
rstudioWrapper
vimPlugins.denops-vim
vimPlugins.vim-pluto
windmill

Maintainer pings
cc @06kellyjac for testing.


Fix for using paths containing spaces

Hey! Any idea why I haven't merged this yet? Maybe I was scared of BASH_SOURCE[0] :-)
Anyways, now there's a separate fix for hidden files, so can you please remove “Cherry pick hidden files fix”, and I'll merge this and make you a committer (as per new policy described in README).

Hy,
Yes, I forgot to create a separate branch for my fix :/ anyway the cherry pick commit is now removed.
I know that this SCRIPT_DIR line is a bit scary, but it's pretty common in shell scripts and I haven't found a better way of doing it.

Merged, and you're now a committer. Thanks! Feel free to do anything you want here.
BASH_SOURCE actually isn't that scary, I'm glad to discover it, although I'm still not quite sure how it is different from $0 (in non-sourced scripts).


Could you help me study your paper

Hello, I have just started learning this knowledge and I have the opportunity to read your paper. I admire your paper very much, but it has a certain difficulty for me. I want to run through your experimental code. May I ask how to obtain train.npy

Thanks for your attention. The train.npy is actually a list and should have the following format:
[
(path_of_input1, path_of_ground_truth1),
(path_of_input2, path_of_ground_truth2),
...
]
You may use numpy lib to save this .npy file.

Thank you for your help, but I have encountered a new problem. May I ask what the 'choice' in the constructor is when instantiating Mydataset

'choice' is to control whether to use data augmentation (i.e., resize, compression, etc.) for the current data, and is usually used during training. So, just set it to 'train' if this dataset is used for training, and set it to else if this dataset is for testing.


Proposal .asInterface() method for all go2hx types

Would it be possible for all go2hx Go named types (be they implemented as classes or typedefs) to have a method called ".asInterface()" that delivers back the interface form of that type?
      // inside the users normal Haxe code
      stdgo.fmt.Fmt.printf("%+v\n", someGo2hxThing.asInterface())

Because at the moment you would have to find out the type of someGo2hxThing before you can make it an interface:
      // inside the users normal Haxe code
      stdgo.fmt.Fmt.printf("%+v\n", TheTypeOf_someGo2hxThing_asInterface.new(someGo2hxThing))

...which would be very irritating and reduce the speed at which you can code.

I agree, this would be very useful, an alternative to this would be a macro system such as:
Go.asInterface(someGo2hxThing);

I do think I prefer your way in the proposal, I have to also check to see if it doesn't run into reference cycles. @elliott5

Since Go.asInterface(something) would apply for all types, that would be preferable.
It would also mean that the Someclass_asInterface could be removed from the documentation. ;)

Trying to make it so interfaces can be made firstly without having to define all of the fields, then I can work on the macro to make it even easier.

Ran into some issues, types with no type params can use this method but ones with, will have to use the old method. The reason is because the macro needs to know the type params and so cannot be created before inside a funtion as then a generic type is only known, causing the static extension macro to fail creating a typedef for each typeparam and instead creates a recursive type of itself.

That's a disappointment @PXshadow , but unsurprising given past experience.
For a generic type, could the code for .asInterface() be generated by the compiler when a generic type is instantiated:
// haxe syntax probably wrong
var x = new(myType<typeParam>);
x.asInterface = function() { /* code to implement specific to this typeParam */ };

That's a disappointment @PXshadow , but unsurprising given past experience.
For a generic type, could the code for .asInterface() be generated by the compiler when a generic type is instantiated:
// haxe syntax probably wrong
var x = new(myType<typeParam>);
x.asInterface = function():AnyInterface { /* code to implement specific to this typeParam */ };

@elliott5 great idea working on it. It's a much cleaner approach just need it to pass the regression issues.

stdgo.Go.anyInterface
``

Is a very mature macro function nowadays and can be used in most cases quite reliably, including Haxe code needing to turn into ``stdgo.AnyInterface`` = any/interface{}

This is what a compile of a named struct{} looks like.
```go
type t struct{}

package _internal.github_dot_com.go2hx.go4hx.rnd;
@:structInit class T_t {
    public function new() {}
    public function __underlying__() return stdgo.Go.toInterface(this);
    public function __copy__() {
        return new T_t();
    }
}

This will obviously not work for generic types acessing an unknown type, my idea to deal with this is to use a @:genericBuild macro, to create new types per new typeParam.


docs(readme): fix broken links

Can one of the admins verify this patch?

@corinnekrych snapshot fabric8-ui is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-789-fabric8-ui.int.rdu2c.fabric8.io


Extractive TyDi RC Pipeline

aferritto created the following on Apr 06:
Closes #40
Closes #69
Creates Tech Debt #91 #93

Integrates the extractive rc tydi pipeline.
Fixes poor F1 by switching bytes to characters (offsets) in tydi annotations
Adds support for document use case (single context) based on score improvement of ~3-4 F1 (see #91)

Original issue: https://github.ibm.com/ai-foundation/PrimeQA/pull/92

duplicate #38


ci: remove lint step with ref to pull_request in release workflow

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 37.35%. Comparing base (cab53c4) to head (5abd8ef).
Report is 11 commits behind head on develop.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #219      +/-   ##
===========================================
- Coverage    37.91%   37.35%   -0.57%
===========================================
  Files           36        1      -35
  Lines         3373      779    -2594
===========================================
- Hits          1279      291     -988
+ Misses        2012      479    -1533
+ Partials        82        9      -73

Flag
Coverage Δ

integrationtests
37.35% <ø> (?)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Bubble issue

When defining a Bubble:
couponBubble = $.bubble({
'content': msgDefMkt
}),
The component is working only with a subscription to an event. Otherwise, doesn't work :(
For example:
couponBubble.on('show', function() {
this.refreshPosition({
'reference': $('#couponCode')
});
})

@HGARZON What kind of problem do you have? Can you provide more information please? Is the reference element exist and visible?
Cannot reproduce.

@HGARZON Any news?

@HGARZON, do you remember the specifics of this issue? Is this related to the Bubble not refreshing its position?

Yes, it seems that the bubble position, if something changes on the DOM, only will change if we call the refreshPosition method.

@HGARZON, thanks! I'll re-label this as "feature request" instead of a "bug", since what you describe is the expected behaviour —but most certainly not the ideal.

I'll close this, since to automatically update the Bubble position we should add a MutationObserver, an Event or any other mechanism to identify if the DOM has changed. Sadly that's not aligned with the project's current objectives.
By using the refreshPosition() method this could be implemented in specific projects if necessary.


POP3 server: have option to not delete emails right away

Hello again @axllent
for some debugging I needed to keep the emails in the POP3 server even if the client tells to delete them.
Let me know if it is of interest for you
I added the functionality here:
https://github.com/axllent/mailpit/compare/develop...antonionardella:mailpit:no-delete
And also updated the documentation here:
https://github.com/axllent/mailpit-website/compare/develop...antonionardella:mailpit-website:patch-1
If it is making more of a mess than anything, feel totally free to ignore this issue.
Best,
Antonio

HI again @antonionardella. I'm afraid this option would break RFC compliance for the POP3 server. The POP3 client would tell the server to delete the messages, and the server would just ignore it. This means that next time the client connects it believes the messages are new and downloads them again and again etc.
The POP3 client protocol already has an option not to delete (simply by not telling the server to delete) - which is typically the option to "leave messages on server" (or "leave messages on server for X days") found in most mail clients. What am I missing here?

No worries, I thought about that
The issue is that zammad is deleting messages every 30 seconds, which removed the ability to debug and read texts (if one is not quick enough 🤣)
As of Zammad it depends on the server configuration https://community.zammad.org/t/emails-are-deleted-from-my-inbox-when-using-it-as-channel/4419/2?u=skip

Therefore I added a debug flag on my fork to keep messages anyway, at least for debugging reasons.
No worries at all. Let me close this issue then.
Thank you for your time and availability!


Jetson container for ZED2i

Create a docker container for the Jetson Xavier with ZED SDK and ZED ROS2 wrapper. Will allow image, depth, and IMU data output from the ZED stereo camera to be published as ROS nodes

Kinda done - worked for demo.
Will be addressed more by https://github.com/QUT-Motorsport/QUTMS_Driverless/issues/25


Add Tooling for flake8 and black

This PR adds support for running flake8 and black both locally and in Github Actions.
Locally we can simply run make qa and both tools will run.
On Github both tools run in Actions:

flake8-messages are used to annotate the code pushed
The black action will only indicate if it would re-format any code.

This PR re-formats the code using black and fixes problems reported by flake8.

Closed in favor of #36


update rust sdk docs

update rust sdk docs

Thanks, @traitmeta . @patrickkuo , does this look correct to you?

@patrickkuo  can you take a look at this? Thanks!


Fix the programming languages logos

[x] I checked to make sure that this issue has not already been filed.

[ ] I'm reporting the issue to the correct repository (for multi-repository projects) (optional)

Expected Behavior:
Better logos for programming languages on languages page.
Current Behavior:
What is the current behavior?
The C language logo has an unwanted background and is not clear, the Java logo is missing

Solution:
Change the background images of logos
Screenshots (optional):

I would like to work on this under LGMSOC.

@supzi-del Assigned!


Support POST data

This would be good. I helped tim get this set up the other day.
AFAIK just needs app.use(express.urlencoded()); added to start.js. And one or two examples?
An example of using the data might be helpful - either some redirection logic, or using the flash to insert the data into the page?

I think so - and I think you mean server.js?
there's also jsonencoded - not sure if we need that?

closed by #86

FYI you can automatically close issues in commit or merge messages using "fixes #16" or "fixed #16"


Code sample "Apply plugins to the build" is wrong

Issue type
Wrong or misleading information
Problem description
On this page: https://docs.gradle.org/current/userguide/writing_build_scripts.html#writing_build_scripts
It reads:

The Kotlin gradle plugin, version 1.9.0, has also been applied.

However, the this is not true for the corresponding code sample.
Context (optional)
No response
Page with the problem
https://docs.gradle.org/current/userguide/writing_build_scripts.html#writing_build_scripts

This is a valid documentation issue.
We welcome documentation contributions. See our contributing documentation guide for more information.

Thank you for finding this outdated paragraph. I have removed it.


Bump identity.organization.login version

$subject

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/6558197648

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/6558197648
Status: failure

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/6559715890

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/6559715890
Status: failure

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/6572069446
Status: success


Master

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: mcactive sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Feature/dont stack service calls

Types of changes

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

Description of the change
Prevent call stacking in the alarm list component

@spryor,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


get-entries-filter.md

Extended description of the getEntries filter mentioned in Script Helper

Status
Ready/In Progress/In Hold (Reason for hold)
Related Issues
fixes: link to the issue
Description
A few sentences describing the overall goals of the pull request's commits.
Screenshots
Paste here any images that will help the reviewer

@tschanfeld this is very interesting, I believe it should be under the Development documentation section, not the Articles section.
CC @ShahafBenYakir  WDYT?

I submitted it through this page - https://github.com/demisto/content-docs/tree/master/content-repo/extra-docs/articles, but if there's another page I should use in the future for development documentation, let me know. I'm still new to adding topics to xsoar.pan.dev.

@fvigo make sense

@fvigo i dont think it really makes sense to put it under dev, its a builtin command
i think it fits better under the ref docs, perhaps we can add a section for builtin commands like we have for scripts

@Itay4 @ShahafBenYakir your call. I don't think it should be an article either (we also have the "Demisto class" in articles and it doesn't belong). I'm in favor of either creating a new section for builtins and one for class reference (like Demsito class and CommonServerPython), or one that includes both.

Yeah we all agree it should not be in articles and dev is also not a good place. @fvigo Do you recall if we currently have other pages that would fit Builtins?

Not that I recall, but we should have them. Where does the "Script Helper" stuff that you can bring from the product UI live right now?

@fvigo It is hardcoded in the server code

ideally, wherever this page goes, there will be a link to it from the Script Helper

@reutshal Let's talk and think about adding this new "Builtins" section

Any decision where to put this?

This pull request has been automatically marked as stale  ....

Closing until we will get the new section up

Waits
https://github.com/demisto/etc/issues/37903


Refactor pre-commit hook to match any file inside circleci folder

This will allow people that use dynamic configuration to validate all files inside the .circleci folder.
Currently:
.
└── .circleci/
    ├── config.yml # Validated
    └── continue-config.yml # Not validated

With this PR:
.
└── .circleci/
    ├── config.yml # Validated
    └── continue-config.yml # Validated

With it the user won't have to specify files to be formatted in the hook like this:
- repo: https://github.com/AleksaC/circleci-cli-py
  rev: v0.1.30549
  hooks:
    - id: circle-ci-validator
      files: \.circleci/*

Let me know if there are any additional changes that need to be made and if it sounds a nice idea.

Thanks for the PR @lucastonelli!
It seems that CircleCI CLI requires only a single argument. Therefore a wrapper script is needed to make this work.
Also files section is a regex so it should be \.circleci/.* instead of \.circleci/*.

Hey, @AleksaC. You're right. I've might confused it with another hook in pre-commit. It's actually working for only one of my configs. Thanks!


Create CreateTableDesc in HiveQl

Parse the CreateTableDesc in HiveQl.
@marmbrus sorry, I couldn't run unit test with this codebase, as it always say failed in downloading the jar files, I will figure out how to solve that (seems a proxy issue in within my corp network).
NOTICE: StorageHandler is not supported, since HiveTable doesn't support that yet.

@marmbrus I've updated the code, but still I couldn't get the whole unit test run (proxy issues in downloading the jars), however, I've added some basic unit tests, hopefully it can cover most of the cases.
I will keep updating the code if any more comments.

Great, thanks for working on this.  Since there are a lot of other things blocking on my change, I'd like to go ahead and merge it as soon as I've passed tests.  I've opened SPARK-7411 to track this.  Would you mind opening this as a separate PR once mine is merged to master?
I've also refactored the code so that unit tests are run without downloading jar files (other than VersionsSuite) so you should be able to test now.

Yes, I will do that after your PR is merged. (I like the idea unit testing without downloading jars  :) )

Can one of the admins verify this patch?


Module parse failed: Unexpected token: import.meta

Environment
- Operating System: Linux
- Node Version:     v18.20.3
- Nuxt Version:     2.18.1
- CLI Version:      3.13.0
- Nitro Version:    2.9.7
- Package Manager<EMAIL_ADDRESS>- Builder:          webpack
- User Config:      bridge, build, modules, ssr, serverHandlers, devServerHandlers, devServer, typescript, nitro, buildModules
- Runtime Modules<EMAIL_ADDRESS>- Build Modules<EMAIL_ADDRESS>
Reproduction
https://stackblitz.com/edit/nuxt-bridge-reproduction?file=nuxt.config.ts
Describe the bug
As discussed in https://github.com/vuejs/pinia/pull/2622#issuecomment-2304813676, installing the latest version of @pinia/nuxt in a nuxt bridge project raises an error because import.meta is not parsable.
Reverting back to a version prior to the before-mentioned PR fixes the issue as it still uses process instead.
Additional context
👋 @danielroe

Some reasons for the settings in the reproduction:

ssr: false https://github.com/nuxt/bridge/issues/1144
transpile: ['cookie-es'] https://github.com/nuxt/bridge/issues/1281

Logs
No response

We have checked that it works in the latest edge release.
<EMAIL_ADDRESS>https://stackblitz.com/edit/github-toccfh

Thank you both 🙌

@wattanx hi, I still have the same problem.
It looks like your example does not contain @pinia/nuxt package.
Check the repro here: https://stackblitz.com/edit/github-toccfh-mifppg?file=package.json

When using Nuxt Bridge, @pinia/nuxt is not necessary.
https://pinia.vuejs.org/ssr/nuxt.html#Installation

@wattanx Thanks for your quick reply!
When I run npx nuxi/@latest module add pinia as described in that instruction, I see @pinia/nuxt is installed and added to the dependencies. (also, I couldn't find any text that we don't need @pinia/nuxt for Nuxt Bridge)
Also, when I run your example(stackblitz.com/edit/github-toccfh), I see the warning log below. Is that ignorable?
WARN  Module @pinia/nuxt not found. Please ensure @pinia/nuxt is in dependencies and installed.

It was my mistake to say that installation is not necessary

@yshrsmz the fix has been merged but it hasn't been released yet.
Follow 3.4's PR https://github.com/nuxt/bridge/pull/1261 if you wanna be notified about when it'll become available

@wJoenn I've tested with npm<EMAIL_ADDRESS>(the latest edge) and @nuxt/bridge-edge@3.3.1-28745713.5493b4e(the version wattanx mentioned), so I think I used the version which includes the change.
Or did you confirm it's working as intended?

No my bad, I'm in mobile right now so I didn't take the time to check your repro. Sorry about that.
If edge doesn't work either then I suppose the fix doesn't work ?
I'll let Wattanx answer that

@wattanx Sorry to bother you, but did you confirm the behavior? If my repro is correct, could you please reopen the issue? Or should I create another issue?

It seems that the issue hasn't been fixed, so I reopened it.
By the way, you can migrate to Nuxt 3 without migrating to Pinia if you use the following module
https://github.com/wattanx/nuxt-vuex-module
https://github.com/vedmant/nuxt3-vuex#nuxt3-vuex-module

import fs from "fs";

const target =<EMAIL_ADDRESS>
const contents = fs
  .readFileSync(target, { encoding: "utf-8" })
  .replace("import.meta.server", "process.server");

fs.writeFileSync(target, contents);

I worked around this problem by writing a fix script. It's useful until the issue is fixed properly.


Fix minor incorrect doc sentence

There are three parameters that get passed, not two. Anyways, to make it less prone to be out of date, changed the wording to "following parameters".

Codecov Report

Merging #2907 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2907   +/-   ##
=======================================
  Coverage   96.41%   96.41%
=======================================
  Files          67       67
  Lines        1395     1395
=======================================
  Hits         1345     1345
  Misses         50       50

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 5f30f6f...8dff69c. Read the comment docs.


Add search bar

Add a search bar in the home page, which search the operation name and the respective card will come first.

I want to work on this issue under JWOC

I can also work on this under JWOC as all my assigned work are done now.

@TuhinBanerjee31 Go for it.


Fix CMake availability macro typo

Fixes a minor typo introduced in the CMake files in https://github.com/apple/swift-foundation/pull/760

@swift-ci please test


[BUG] Project integration tests require same version of plugins to be checked in

Describe the bug
Currently the following files are checked in.
src/test/resources/index-management/opendistro-index-management-<IP_ADDRESS>.zip
src/test/resources/notifications/opensearch-notifications-<IP_ADDRESS>-SNAPSHOT.zip
src/test/resources/job-scheduler/opensearch-job-scheduler-<IP_ADDRESS>-SNAPSHOT.zip

These are moving targets. Furthermore they are of the same release, while one would want to run integration tests against the released versions of any dependency. Which causes a catch-22 problem.
Expected behavior
These files should be deleted. One solution would be to download them during build. Another would be to build them (#119 tried to do that).
Additional context
Consider the developer experience that wants to check out this repo and just run ./gradlew integTest successfully.

In the 1.3 branch we currently are still using the job scheduler resource in testing https://github.com/opensearch-project/index-management/tree/main/src/test/resources/job-scheduler

AD using RCF jar and Protostuff jar https://github.com/opensearch-project/anomaly-detection/tree/1.3/lib, @dblock , we are going to revert the change to remove protostuff.
And we have job scheduler jar https://github.com/opensearch-project/anomaly-detection/tree/1.3/src/test/resources/job-scheduler, and BWC test https://github.com/opensearch-project/anomaly-detection/tree/1.3/src/test/resources/org/opensearch/ad/bwc/job-scheduler

Same RCF issue for ml-commons, we have local RCF jar file https://github.com/opensearch-project/ml-commons/tree/1.3/ml-algorithms/lib. Will pull from Maven once 2.x published

@ylwu-amzn Same problem we're checking in the JARs into AD, https://github.com/opensearch-project/anomaly-detection/issues/433

Hey @downsrob, @ylwu-amzn, @dblock we have now, a mechanism to publish the zips to maven repo using opensearch.pluginzip, we can delete the zips from checking in to source repo (unless i'm missing something) and can be dowloaded during runtime using maven coordinates once they are published.

Thanks @prudhvigodithi , I think @amitgalitz has removed local zip file from AD, check https://github.com/opensearch-project/anomaly-detection/pull/487 and  https://github.com/opensearch-project/anomaly-detection/pull/505. We are using latest build link and cloudfront download link, check https://github.com/opensearch-project/anomaly-detection/blob/main/build.gradle#L37
        job_scheduler_build_download = 'https://ci.opensearch.org/ci/dbc/distribution-build-opensearch/' + opensearch_no_snapshot +
                '/latest/linux/x64/tar/builds/opensearch/plugins/opensearch-job-scheduler-' + plugin_no_snapshot + '.zip'
...

        bwcOpenDistroJSDownload = 'https://d3g5vo6xdbdb9a.cloudfront.net/downloads/elasticsearch-plugins/' +
                'opendistro-job-scheduler/opendistro-job-scheduler-<IP_ADDRESS>.zip'

@prudhvigodithi should we move to maven? Any benefit?

Hey @ylwu-amzn yes pulling zips from maven is the right way to go, as you dont need to manage any custom code moving forward, as using this opensearch.pluginzip the zips will be published to maven repo and can be fetched with right artifcatID, groupID and version.

Index Management has replaced checked-in ZIPs with dynamic dependencies as per https://github.com/opensearch-project/index-management/issues/305. Now, this issue is not specifically related to that behavior, but should be used to track the progress of getting integration tests to run against dependencies of different versions

Hey @downsrob, looks to me like its using the latest URL and some parse logic, using opensearch.pluginzip, you dont need to have all this custom logic from the PR, I would suggest to go with opensearch.pluginzip.

Hey @downsrob and @ylwu-amzn, is it worth to now explore the pulling the zips from maven and removing the checking in files from src/test/resources/?
@dblock @bbarani

@prudhvigodithi We have already converted to pulling the zips from maven and do not have any checked in zips in src/test/resources. That was tracked in this issue https://github.com/opensearch-project/index-management/issues/348 I believe.
My understanding is that this issue is used for tracking the progress of allowing integration tests to run against dependencies of different versions, which is why it is still open despite no longer using checked in zips

Got it thanks for the update @downsrob.

Hey Starting with release 2.1.0, we now have plugin zips in maven repo
We should be good to remove the checked in zips from repo from 2.x branches.
https://github.com/opensearch-project/opensearch-build/issues/1916#issuecomment-1179276885
Thank you

@prudhvigodithi we do not have any checked in zips in the 2.x branches

Thanks @downsrob,
So can we expect to close this issue once PR: https://github.com/opensearch-project/index-management/pull/403 merged.
Thank you

Can we close this issue as we have merged since https://github.com/opensearch-project/index-management/pull/403 has been merged now?

Hey @downsrob just checking back again, if we can close this issue.
Thank you
@bbarani @CEHENKLE


[TA] Removed length property in model classes of 5.1.0-beta.2

Currently we don't expose the encoding to the user, so the length property can be calculated from the Text property by doing text.length(). Because of that we decided to remove length until we really need it.

Removed property length from CategorizedEntity, SentenceSentiment, LinkedEntityMatch, AspectSentiment,
OpinionSentiment, and PiiEntity.

fixes: #15779

/azp run java - textanalytics - tests


CI improvements

Description

run with exec driver instead of podman

more reliable
much better caching
faster

update tullia
only trigger on pushes to master so that CI is not triggered twice for every push to a PR

Checklist

Branch

[x] Commit sequence broadly makes sense
[x] Commits have useful messages
The documentation has been properly updated
New tests are added if needed and existing tests are updated
Any changes affecting Consensus packages must have an entry in the appropriate changelog.d directory created using scriv. If in doubt, see the Consensus release process.
If this branch changes Network and has any consequences for downstream repositories or end users, said changes must be documented in interface-CHANGELOG.md
If serialization changes, user-facing consequences (e.g. replay from genesis) are confirmed to be intentional.

Pull Request

[x] Self-reviewed the diff
[x] Useful pull request description at least containing the following information:

What does this PR change?
Why these changes were needed?
How does this affect downstream repositories and/or end-users?
Which ticket does this PR close (if any)? If it does, is it linked?

[x] Reviewer requested

bors try

bors merge


pull request for e2e test at 01/02/2017 19:45:07

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Update the eip1559 constants

Update the EIP-1559 constants to ease resets.
The last spamming that affected users seems to me to have had 4 root causes of issue:

The reset interval happened right after epoch, resulting in 7 extra minutes of pain. (Every 1000 blocks) Make this 50% less likely by making it every 2000 blocks
The value reset to .0025, requiring 4 minutes to get back to old value
Keplr fee settings were too low (other integrators were set. The keplr chain registry bot was not updated to latest integrator suggested numbers)
Large time with only non-1559 validator proposals

So (3) should get independently fixed, which should've been the root cause of most user pain outside of (4).
(4) should be fixed by increasing number of 1559 nodes, and lowering block times.
This PR lowers the probability of resets, and makes resets cause 2 minutes fewer of damage.

Reopened for CI

Forced pushed fixes because automation auto-committed code-gen here

@Mergifyio queue default

@Mergifyio queue default

@mergifyio refresh

@mergifyio help

@Mergifyio backport v20.x

@merggifyio backport v20.x

@mergifyio backport v20.x


[LTO] Driver support for -lto_library flag

This commit adds support for the -lto_library flag, allowing users to specify a custom LTO library on Darwin. This also fixes an issue where the default LTO library is used even if Driver is run from inside an alternate toolchain.

@jrose-apple

@gottesmm

Thank you for your contribution! CI is currently limited to established contributors, so you'll need someone with Contributor access to be able to invoke CI. (I can do that for you)
@swift-ci please smoke test
Also, given that the swift-driver is now the default driver as of Xcode 13, could you also implement this in the swift-driver? It is structured somewhat similarly, so it shouldn't be too difficult now that you've done this work in the legacy driver.

@swift-ci please smoke test

Thank you for your contribution! CI is currently limited to established contributors, so you'll need someone with Contributor access to be able to invoke CI. (I can do that for you)
@swift-ci please smoke test
Also, given that the swift-driver is now the default driver as of Xcode 13, could you also implement this in the swift-driver? It is structured somewhat similarly, so it shouldn't be too difficult now that you've done this work in the legacy driver.

I actually already have. As this is my first PR here I was waiting to see the feedback on this one before opening that one.

@harlanhaskins Could you run the tests?

@swift-ci please smoke test

@owenv What is the procedure now that the tests are done? Anything else I should do?

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please test Windows


Translations update from Hosted Weblate

Translations update from Hosted Weblate for TallUI/TallUI Core Auth.
It also includes following components:

TallUI/TallUI Core Pagination

TallUI/TallUI Core Passwords

Current translation status:

I don't understand how this PR came to be. It seems there is no change, even if the PR states every SVG as newfile. Another question: how the heck was Hosted Weblate triggered by just uploading some SVGs.
Closing, without merge, of course.


Fix the agg bug processing alloc mem failed

What type of PR is this：

[x] bug
[ ] feature
[ ] enhancement
[ ] others

Which issues of this PR fixes ：

Fixes #3326
Problem Summary(Required) ：

phmap::lazy_emplace() may fail caused by can't alloc memory  for key/value. meta has been created in hashmap, but the key/value is not constructed, so destroy will be crash.
before, we use erase(key) to erash the invalid item, actually not remove the item, because the key is not insert yet, search will fail .
so we will add try catch std::bad_alloc() in lazy_emplace() to remove the invalid item

run starrocks_clang-format


feat(cloudflare): add global process polyfill to cloudflare preset

🔗 Linked issue

❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[x] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

📝 Checklist

Adds a polyfill for process to the Cloudflare preset which combines the node polyfill with the workerd polyfill implementation. We'd specifically like to retain the polyfilled nextTick from workerd which uses queueMicrotask, but benefit from all the other global process polyfills provided by unenv.

[ ] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

Can you please update the polyfill to explictly override nextTick impl?
(env implementation is more generic in unenv. we can explicitly add any more override in the future that needed)
Also would it make sense to have a generic polyfills/cloudflare file that adds more of future polyfills via one entry?

Can you please update the polyfill to explictly override nextTick impl?
(env implementation is more generic in unenv. we can explicitly add any more override in the future that needed)
Also would it make sense to have a generic polyfills/cloudflare file that adds more of future polyfills via one entry?

Great suggestions, updated!
Thanks for the speedy response time 😄.

On this PR, I would suggest to introduce runtime/polyfill/cloudflare.ts mainly to set env shim override (to solve glboalThis concern of process.env https://github.com/unjs/unenv/pull/183#discussion_r1602104944) and wait for https://github.com/unjs/unenv/issues/182 to be landing before trying the mixed method.

I addressed this in https://github.com/unjs/unenv/pull/183#discussion_r1602197176

Using node:process import idea, is probably not going to work well in practice because either the unenv cloudflare preset should configure bundler to alias node:process (to unenv) or use external one. If it is aliased to unenv, it will make a recursive situation and if it is external unenv is not effective.

if we resolve the process.env issue as suggested above, we might not need to import node:process at all, but if we do, we would only use it for APIs that are natively implemented in workerd, which will remain exposed once we implement the hybrid polyfills as described in https://github.com/unjs/unenv/issues/181
so I think the last remaining thing here is to resolve the two issues I broght up with the current process.env polyfill (1/ removal of globalThis fallback 2/ swapping the order in read operations).

Update: I've changed this PR so that the polyfill simply sets globalThis.__env__ to empty object.


increase cpu-manager test timeout to 3 minutes

What this PR does / why we need it:
It increases the cpu-manager test timeout to 3 minutes, the heartbeat is the place where we update a cpu-manager label and it happens that between calls we have 2 minutes break, as a result we have a flaky test.  The reason why it takes so long is under investigation.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

None

The failing tests don't have the mentioned test failing. But let's try again.
/test all
/lgtm
/approve

On a related note: test fails in the OKD 4.2 lane: https://prow.apps.ovirt.org/view/gcs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/2790/pull-kubevirt-e2e-okd-4.2/1186668276843810816

Though it fails on an expectation that there is only one node having the cpumanager=false label attached. I guess this is related to having two workers?

@dhiller from weekly reports I saw it failed on a timeout, it means the CPU manager label was not updated in 2 minutes. Didn't check this against OKD4. It's not about two workers.
take a look at
https://prow.apps.ovirt.org/view/gcs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/2771/pull-kubevirt-e2e-k8s-1.15.1-ceph/1184643435949199361

@dhiller it seems you're right, it's about two nodes having cpumanager=false


Parameters aren't validated when passed via constructor

class Task(luigi.Task):
    p = luigi.IntParameter()

print Task(p='hi').p  # 'hi'

While the above is a trivial case of "something will probably break down the line," this can introduce subtle bugs when e.g. a date parameter is passed to a datetime parameter (or vice-versa).
My suggestion would be to add some sort of validation, and possibly the ability to tweak the value (similar to parse) to the Parameter interface. The latter would be useful when implementing an e.g. "every N hours" parameter that automatically floors to the nearest hour.

Yeah that sounds reasonable. Would you mind putting together a PR?

Sounds great. Though beware that a couple of (bad) tests would break. Also I'm not sure what to do with the None issue.

Resolved


Automated Package Upgrade

:tada: This PR is included in version 25.3.5 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Release/21.05.0

I assume the tests fail because the latest version of the container is not yet published.


Cannot find Polls Django Web Project template

[Enter feedback here]
On this page in step 6-1it says;
2. In the new project dialog, search for and select the Polls Django Web Project template, call the project "DjangoPolls", and select OK.
But there does not appear to be a "Polls Django Web Project template". I have searched for it from the Extensions Manager and online and cannot find it. Is this template no longer available?

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

ID: fbca10bd-8636-496e-91a3-e77fb1dfcd84
Version Independent ID: fb4fceda-3a4f-f8fd-3965-1a858e240d31
Content: Web application templates for Python - Visual Studio
Content Source: docs/python/python-web-application-project-templates.md
Product: visual-studio-windows
Technology: vs-python
GitHub Login: @JoshuaPartlow
Microsoft Alias: joshuapa

@rstambach1-coder -- thank you for your feedback.  Please consider these resources:

Visual Studio on Q&A
Visual Studio MSDN Forum
Stack Overflow
Reddit

@JoshuaPartlow  -- please look into this potential documentation issue.

@rstambach1-coder -- thank you for your feedback.  Please consider these resources:

Visual Studio on Q&A
Visual Studio MSDN Forum
Stack Overflow
Reddit

@JoshuaPartlow  -- please look into this potential documentation issue.

@savannahostrowski - Would you be able to comment here?

@rstambach1-coder Sorry about the confusion. This template is no longer supported. The last release that included this template was 16.7. You can install an older version (like 16.7) of Visual Studio here.


[SPARK-4226][SQL]Add subquery (not) in/exists support

Some known feature that we don't support right now, but will add it later.
We don't support the outer UDAF function used in the correlated query, combined with
outer having clause, which requires the implicit projection change for the outer query.
select b.key, min(b.value)
from src b
group by b.key
having exists (
    select a.key
    from src a
     where a.value > 'val_9' and a.value = min(b.value) -- min(b.value) implicits requires the outer query to add more field in the projection.
)

We don't support the multiple references for the outer query in both the subquery in both projection and filter clause.
select key, value
from src b
where value in
    (select s1.key+ b.key
     from src s1
     where s1.key > '9'  and s1.value = b.value) -- both b.value and b.key present in the subquery, but in projection and filter clause respectively.

Merged build triggered.

Merged build started.

Test build #43506 has started for   PR 9055 at commit e3aa255.

cc @marmbrus @yhuai @ravipesala
This implementation inspired by #3249, by using the SubQueryExpression. and also the follow up with #4812.
Since the anti join is another type of SEMI JOIN, I added it back here for performance concern in transform the "NOT EXISTS / NOT IN" subquery.

Test build #43506 has finished for   PR 9055 at commit e3aa255.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
trait SubQueryExpression extends Unevaluable
case class Exists(subquery: LogicalPlan, positive: Boolean)
case class InSubquery(child: Expression, subquery: LogicalPlan, positive: Boolean)

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43506/
Test FAILed.

Merged build triggered.

Merged build started.

Test build #43508 has started for   PR 9055 at commit b382bc9.

Test build #43508 has finished for   PR 9055 at commit b382bc9.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
trait SubQueryExpression extends Unevaluable
case class Exists(subquery: LogicalPlan, positive: Boolean)
case class InSubquery(child: Expression, subquery: LogicalPlan, positive: Boolean)

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43508/
Test FAILed.

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #43528 has started for   PR 9055 at commit ab22171.

Test build #43528 has finished for   PR 9055 at commit ab22171.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
trait SubQueryExpression extends Unevaluable
case class Exists(subquery: LogicalPlan, positive: Boolean)
case class InSubquery(child: Expression, subquery: LogicalPlan, positive: Boolean)

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43528/
Test FAILed.

Seems the failure is not related.
retest this please

retest this please

Merged build triggered.

Merged build started.

Test build #43552 has started for   PR 9055 at commit ab22171.

what's the difference with #4812?

This is much simpler than #4812, by using the SubQueryExpression, particularly in processing the case
key IN (subquery) AND other_condition case. #4812 doesn't support the AND other_condition.

ok, does this support multi exists and in in where clause?

No, we don't support that in this PR, but should be very easy to support once this PR merged. I can plan the work if you feel that's very critical to your customers.

Test build #43552 has finished for   PR 9055 at commit ab22171.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
trait SubQueryExpression extends Unevaluable
case class Exists(subquery: LogicalPlan, positive: Boolean)
case class InSubquery(child: Expression, subquery: LogicalPlan, positive: Boolean)

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43552/
Test PASSed.

cc @rxin as well, this is required by many of our customers, and most of the code change is about the unit test, should not be hard to follow.

Merged build triggered.

Merged build started.

Test build #43782 has started for   PR 9055 at commit 7511f47.

Test build #43782 has finished for   PR 9055 at commit 7511f47.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
trait SubQueryExpression extends Unevaluable
case class Exists(subquery: LogicalPlan, positive: Boolean)
case class InSubquery(child: Expression, subquery: LogicalPlan, positive: Boolean)

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43782/
Test FAILed.

Merged build finished. Test FAILed.

Seems not related.

retest this please

Merged build triggered.

Merged build started.

Merged build triggered.

Merged build started.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43819/
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43822/
Test FAILed.

Merged build finished. Test FAILed.

retest this please

Merged build triggered.

Merged build started.

Test build #43826 has started for   PR 9055 at commit 7511f47.

Test build #43826 has finished for   PR 9055 at commit 7511f47.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
trait SubQueryExpression extends Unevaluable
case class Exists(subquery: LogicalPlan, positive: Boolean)
case class InSubquery(child: Expression, subquery: LogicalPlan, positive: Boolean)

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43826/
Test PASSed.

Two general comments. First, we need to add document to explain how we rewrite a plan when (1) there is a uncorrelated subquery and (2) there is a correlated subquery. Second, for those rewriting rules, I am thinking if we can have more concise ones. For uncorrelated subqueries, the subquery itself should be a resolved logical plan, right? For correlated subqueries, we only need to extract those conditions referring columns in the outer query block, right? Do we really need to matching those different specific patterns? Can we have some general logics?
Actually, does this pr try to support uncorrelated in/not in/exists/not exists subqueries?

Merged build triggered.

Merged build started.

Test build #44064 has started for PR 9055 at commit cb69166.

Thank you @yhuai for reviewing this.
I've added some more docs for this PR, hopefully make more sense.
First, I'll agree with you to make a general logic to partially resolve the correlated condition within the subquery, but it's probably not that easy, particularly we need to give more concise error message to the end user, so my suggestion is to leave it for the future improvement, probably we will have better idea to simplify that by having enough feature supported with the follow up PRs (See my TODO in the description), as currently, the limit patterns actually works for most of cases.
Second, I totally agree with the Join Type comments, LeftSemiJoin <-> LeftSemi <-> LeftAnti, the motivation I am trying to make a parent class for LeftSemi / LeftAnti is for reducing the code change in Optimizer and SparkStrategies, maybe I should rename it to LeftSemiOrAntiJoin as the parent class. As well as the Operators' name, since we no longer the LeftSemiXXX, but also supports the LeftAntixxx.
Still, I hope we can merge this PR in 1.6 release, as it's almost 1 years passed since the previous PRs created in #3249 & #4812. And I will keep updating the code once we have the general agreement for the implementation.

BTW: IN / NOT IN definitely supports the uncorrelated, but EXISTS/NOT EXISTS are not in this cases, the same behavior as Hive does.

Test build #44064 has finished for PR 9055 at commit cb69166.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  * trait SubQueryExpression extends Unevaluable \n  * case class Exists(subquery: LogicalPlan, positive: Boolean)\n  * case class InSubquery(child: Expression, subquery: LogicalPlan, positive: Boolean)\n

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44064/
Test PASSed.

Hi @yhuai ,
This missing feature("IN" sub query) in Spark SQL blocked our real-world case. Could you please help to review this PR ?  Strongly hopefully this PR feature can be merged in Spark 1.6.0 ( I saw the Hive implementation supported such feature ). Thanks in advanced !

@jameszhouyi
We hit the same issue. Now, we bypass it by using joins.

Thank you @gatorsmile for your suggestion.
I think this feature("IN" sub query) is necessary for Spark SQL engine as SQL-on-Hadoop.

@jameszhouyi
Agree. This is an important feature for any SQL engine. We are also waiting for this feature. So far, using joins is an alternative to bypass it.

Unfortunately, we probably will miss this in Spark 1.6, as it's almost code freeze for 1.6. @rxin @yhuai

Yeah, sorry.  It is too late for a patch this large.

So what next ?

We've added our own In/Exists - plus Subquery in Select - support to a partial fork of Spark SQL Catalyst (which we use in transformations from our own query language to SQL for relational databases). But since In, Exists and Select projections are Expressions which will then contain LogicalPlans (Subquery/Select with nested LogicalPlans with potential nested Expressions) this makes whole-tree transformations kind of convoluted since we''ve got to deal with 'pivot points' for these 2 types of TreeNodes, where a recursive transformation can only be done on 1 specific type. Why was the choice made in Catalyst to make LogicalPlan/QueryPlan and Expression separate subclasses of TreeNode, instead of e.g. also make QueryPlan inherit from Expression? The code also contains duplicate functionality, like LeafNode/LeafExpression, UnaryNode/UnaryExpression and BinaryNode/BinaryExpression. Much obliged.


[feature request] Enable to change default search engine

i know this is duckduckgos official browser, but i really would like to search the web with startpage with this browser, because this browser is the only on fdroid that is backed by a corporation and open source.

Thanks for reaching us. We are currently only accepting Github Issues for bug reports.
Please, use discussions to share feature requests. Only bug reports are expected to be accepted as issues. Thanks.
Contributing guidelines

it's not cool. if someone just wanna use because of layout and fast interface, it can't use because of this reason. so that's why, you should add an option.

@Bruising6802 @bakinazik @username0136 I've just created a fork of DuckDuckGo that allows to change the default search engine. Feel free to check it out or add your preferred search engine via a PR - https://github.com/lippfi/SearxDuckGo


Prepare chart for deployment

Towards: https://github.com/giantswarm/roadmap/issues/1767

I think it is a nice settling point as most of what is in the original ticket is covered (also it is already huge). We can fine tune in upcoming PRs and if merged I can also enable automatic updates and file alignments too.

FTR it has to be added back to retagger too when Aliyun gets unblocked.


input and output

I can't understand if the input values must be between 0 and 1 or between -1 and +1 or... ? can you help me?

Hi Frollino! The input can be any integer value, from PHP_INT_MIN (probably -2 billion) to PHP_INT_MAX (probably +2 billion). The same goes for the output node(s).
It might be good to google around a bit and figure out how artificial neural networks work. The one that's implemented here is a very common and very simple multi-layer network. I did a quick google and found this page for you, which seems (as first glance) to be a decent introduction article: http://natureofcode.com/book/chapter-10-neural-networks/ You might need to skip over some of the parts though.
As this is not an issue with the code itself I will close it after this comment.


Tooltip is getting cut with the left placement if the text is long.

For the tooltip example code of 'angular ui bootstrap' documentation, I have added some extra (long) text to the "uib-tooltip" attribute of the tooltip top button which is on the top and left side of the page. Now, when I hover on the tooltip top button, some part of the tooltip to the left is getting cut.
The plnkr link is "http://plnkr.co/edit/NJppfHltSrhiSzpdpUPn?p=preview"
Version of Angular, UIBS, and Bootstrap
Used existing code from the 'angular ui bootstrap' documentation.
Angular: 1.5.3

That's why the 'auto' placement is available, changing the code to:
<button tooltip-placement="auto + {{placement.selected}}" uib-tooltip="On the {{placement.selected}} If the text is long, tooltip is getting cut for the left placement " type="button" class="btn btn-default">Tooltip {{placement.selected}}</button>

will keep the tooltip from rendering off the page.  We don't want to force folks into that behavior if they would rather have complete control over the placement.

Oh! That's great. Thank you @RobJacobs


npm-publish node12 deprecation

github action JS-DevTools/npm-publish@v1 still uses depreacted node12
we need to wait for an update on https://github.com/JS-DevTools/npm-publish/issues/61 or create a fork

fixed with https://github.com/SamhammerAG/Samhammer.Authentication.Vue/commit/ec63082f465d416ca280abe659ca1f2e30d5aa25


Fix code quality push workflow

Fixes #424

This is one of the solutions we talked about before, but I really wanted to look into keeping this approach and using it with grumphp. Will look into this as soon as I recover from a certain illness. 😸

I will create an issue for it then.


minor change to fix the unit test

@andrewstevenson : the test was failing with null pointer exception. also the create was called outside the Try so it was pointless. So now if the error happens is captured. but question is do we want to hide the exception and only log it?

I'm favoring failing early on table creation issues - and logging in a banner

@stheppi I assume this can be closed this the ReThinkSinkConverter no longer exists?


Auto-add needed tech when adding an item for zerg

When adding e.g. Broodlords in an empty bo, it will add all the required structures and tech needed for it. Works for upgrades and structures as well.
Will do the same for terran and protoss next if this pull request is merged.

New files weren't added to git because I used git commit -am "explanation" when it should have been git add -A && git commit -m "explanation"
Added auto-add for protoss. TechLab and Reactor give me a harder time for terran.
Added a new menu button like Settings, but for optimizations on the build. There is a lot to add here it'll be a ton of fun!

Awesome looking updates!

Thank you.
But let me slow you down, I want to merge this PR first before it gets too complicated and I am unable to follow the changes.
I hope that is alright with you. If not, please add a [WIP] (for work in progress) in front of the PR title - maintainers should usually know that they shouldn't merge those PRs.

I found a "bug" where if you remove an item after optimizing the build order by workers, the "selection" drops one to the left
So from

it goes to

while the position of the "selection" shouldn't change.
However I am currently unable to reproduce the bug.

A suggestion would be to hide the "Optimize" drop-down for races that are not working at the moment (so terran right now).
If you can't add it, I could add it later - I won't have much time until monday evening.

1)
I think it was happening before from my commits.
Here is an example on https://burnysc2.github.io/sc2-planner/?&race=protoss&bo=002eJyLrlbKTFGyMjPQUSqpLEhVslIqLikqTS4pLUpVqtWBShrhkTS1xCNpYQKXLM8vyk4tIkLG3BguU5qXWUKRWSTJxAIA32hVIA==
If you place the selection say after the nexus and you double click on the zealot, hence removing the zealot and the following probe, it will move the selection one square to the left.

Or are you referring to another bug?
2)
I didn't know my commits were added to this pull request, so there are multiple features in this one but from now on I'll create branches to prevent this. Sorry!
The features are :

when adding an item to the build order, it also adds all its requirements (it doesn't work for terran but I'm working on it)
when clicking on the Optimize button, a drop down menu offers various optimisations. The first one is to add as many workers as possible as long as the build order doesn't end later
by default all the build order is visible, and a button allows to limit it to one scrollable line.

I'll try to maintain this gh page up to date so you can test out these features before merging them.

If you place the selection say after the nexus and you double click on the zealot, hence removing the zealot and the following probe, it will move the selection one square to the left.

Yes, exactly - very weird 'bug'. I'll try to check out what causes it or how to fix it.

I'll try to maintain this gh page up to date so you can test out these features before merging them.

I'll pull the PR and npm start it on my end to try it out anyway - thanks though!

The performance issue mostly came from JSON.stringify being called about 100000 times per bo simulation as an error message detail in the idleUnits loop from trainUnit() in GameLogic
That was hell to spot.
Indeed the auto-add requirements feature only works if you add something at the end of the build. It could work everywhere now that I understand the project, I'll have a look at the possibilities but it's less exciting that new features I'm working on...
All the builds I created were on one base, I didn't notice problems regarding timings. My next builds will be on 2 bases I might need accurate income estimations! So I might fix it before you do :P

I have a few pull requests waiting for this one to be merged. You may have a lot on your plate already, or maybe it's not good enough ><
Again I'm new to github maybe should I just wait. Please don't hesitate to tell me how things are done around here !

Sorry, it's good.
I just need a couple more days to merge it


Getting original string message from a proto.Message

Hi team, I have a noob question -
I am getting a message that implements proto's Message interface. I need to get the original []byte out of the message and then do some manipulation on the original []byte. Is there a way to do so?
If I call String() - I will get original []byte along with implementation specific texts, like this: msg_body:"what I want"
Thank you for any hints!

Once a message has been parsed from the []byte, the original slice is no longer available. You can re-marshal the []byte by calling proto.Marshal on the message.


docs: missing options prefix -- in front of check-coverage

Changes Unknown when pulling b77e1644015ff396fc0847532764e033628a7f35 on clakech:patch-1 into ** on istanbuljs:master**.

@clakech thanks for the fix.


Feature/add exclude option

Using the --exclude option enables users to exclude files from validation. The regex functionality should save people from having to fill in the complete path to a file.

The phpunit tests fail on the current master for me with 2 errors. No extra errors were introduced so I'm hoping it's fine.

Fixes #15

@frvge Thanks for the contributions! If you rebase things should be green again. :)

@asm89 I fixed all PRs. I think I'll need to rebase them later again, because the changes are in the same area in all PRs. This PR is for me most important because it fixes my issue :)

PIng @asm89  :)

We're now using my personal fork instead of the official one because we can't wait any longer. :(

@frvge Sorry for the experience here. I've been really swamped. :( Good news is that @fabpot agreed on moving the repo to the official twig organization, so there will be more eyes on it in the future. :)

@asm89 Ok, that's good news. Thanks for the merge. I'll move our upstream back to yours.


ArchiveFilesV2 not replaceing existing archive if the replaceExistingArchive option is set

https://github.com/microsoft/azure-pipelines-tasks/blob/9244d42f479f31600c1dff0dc6ca8e05b2e9f85c/Tasks/ArchiveFilesV2/archivefiles.ts#L340
The code is only logging that the file should be deleted but the action is missing.

Adding the folowing code should resove the issue tl.rmRF(archiveFile);

Hi @myanakiev thanks for contribution! We rolled out related changes - closing it at the moment.


[copy-quay] Do not push tag if it already exists on destination namespace

/ok-to-test

@pablintino I've created new PR containing only code changes #1253

Hi @michaeltcoelho will you update this PR to apply formatting to https://github.com/openstack-k8s-operators/ci-framework/pull/1253 ?

Hi @michaeltcoelho will you update this PR to apply formatting to #1253 ? If so please update title, description and commit message.

@pablintino I'll have it formatted in a new PR.