Citadel fails readyness check for citadel images built using Golang 1.12, causing e2e-simple to fail

Bug description
Running make e2e_simple E2E_ARGS="--skip_cleanup --auth_enable --installer helm" using images built using Golang 1.12 cause the test to fail (these images can be built locally or using an Istio-builder that was updated to use Golang 1.12). Specifically, the test curl to Istio-citadel returns a RC of 52. Note that this is the prow flavor of the test since I'm using IKS to host my cluster.
Why is this an issue? If this particular simple-test is valid, than anyone pulling the istio source and building istio using Golang 1.12 will have an istio setup which will not pass the normal tests passed by the istio images.
I did the test against a k8s 1.11 cluster as well as a 1.13 cluster with no differences.
Exec'ing into the galley container and attempting the curl shows the same
k exec -it istio-galley-7f66d4fd8f-s2h85 -n istio-system bash
root@istio-galley-7f66d4fd8f-s2h85:/# curl -i http://istio-citadel:8060/-/ready
curl: (52) Empty reply from server

When using images built using the older Istio-builder (uses an older Golang version), the curl result is a RC of 0 is:
k exec -it istio-galley-6fdc879dbd-wzwdw -n istio-system bash
root@istio-galley-6fdc879dbd-wzwdw:/# curl -isv http://istio-citadel:8060/-/ready
*   Trying <IP_ADDRESS>...
* Connected to istio-citadel (<IP_ADDRESS>) port 8060 (#0)
> GET /-/ready HTTP/1.1
> Host: istio-citadel:8060
> User-Agent: curl/7.47.0
> Accept: */*
>

* Connection #0 to host istio-citadel left intact
root@istio-galley-6fdc879dbd-wzwdw:/# echo $?
0

If I update the citadel deployment to instead us the first failing Golang 1.12 build for the citadel container, and repeat the test, the curl now fails with the RC of 52.
I turned on citadel debug and got no additional logging to help identify the issue
Affected product area (please put an X in all that apply)
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[x] Security
[x] Test and Release
[ ] User Experience
Expected behavior
Tests should pass for all Golang version builds
Steps to reproduce the bug

Create a set of istio images using Golang 1.12.5. In my case, I could this with:
GOOS=linux make clean init test-bins build docker docker.push  HUB=ericvn TAG=ericvn

Create an k8s cluster and set your context to it. In my case, I use IKS clusters of version 1.11 and 1.13.

Run the e2e-simple test. Example using my hug/tag from above:
 make e2e_simple E2E_ARGS="--skip_cleanup --auth_enable --installer helm" HUB=ericvn  TAG=ericvn

The test should fail.

After running the test, exec into your galley container: ex. k exec -it istio-galley-7f66d4fd8f-s2h85 -n istio-system bash. Then run a curl to citadel: curl -isv http://istio-citadel:8060/-/ready. Check the RC using echo $?. I In a passing instance, one should get a 0. In the failure, it returns 52.

Version (include the output of istioctl version --remote and kubectl version)
/Users/ericvn/work/go/out/darwin_amd64/release/istioctl version --remote
client version: d6d298327117385a51a51a3a4d65f96961006964
citadel version: c1f14d14423b9b952f208f84dac648ef241ef848
egressgateway version: e851492f958f48161de07db19b40aa3c566b2c3c
galley version: e851492f958f48161de07db19b40aa3c566b2c3c
ingressgateway version: e851492f958f48161de07db19b40aa3c566b2c3c
pilot version: e851492f958f48161de07db19b40aa3c566b2c3c
policy version: e851492f958f48161de07db19b40aa3c566b2c3c
policy version: e851492f958f48161de07db19b40aa3c566b2c3c
sidecar-injector version: e851492f958f48161de07db19b40aa3c566b2c3c
telemetry version: e851492f958f48161de07db19b40aa3c566b2c3c

k version
Client Version: version.Info{Major:"1", Minor:"14", GitVersion:"v1.14.2", GitCommit:"66049e3b21efe110454d67df4fa62b08ea79a19b", GitTreeState:"clean", BuildDate:"2019-05-16T18:55:03Z", GoVersion:"go1.12.5", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.10+IKS", GitCommit:"7b8624d43d0e5341ae55537998cddb865607059c", GitTreeState:"clean", BuildDate:"2019-05-08T19:47:50Z", GoVersion:"go1.10.8", Compiler:"gc", Platform:"linux/amd64"}

How was Istio installed?
By test. Uses helm
Environment where bug was observed (cloud vendor, OS, etc)
IKS, and GKE
Additionally, please consider attaching a cluster state archive by attaching
the dump file to this issue.

This is not just a test failure, citadel's readiness check (at a minimum, maybe other issues in Citadel or other components) will not work on go 1.12
root@istio-galley-676599ffb4-92kkz:/# curl -v http://istio-citadel:8060/-/ready
*   Trying <IP_ADDRESS>...
* Connected to istio-citadel (<IP_ADDRESS>) port 8060 (#0)
> GET /-/ready HTTP/1.1
> Host: istio-citadel:8060
> User-Agent: curl/7.47.0
> Accept: */*
>

* Connection #0 to host istio-citadel left intact
root@istio-galley-676599ffb4-92kkz:/# curl -v http://istio-citadel:8060/-/ready
*   Trying <IP_ADDRESS>...
* Connected to istio-citadel (<IP_ADDRESS>) port 8060 (#0)
> GET /-/ready HTTP/1.1
> Host: istio-citadel:8060
> User-Agent: curl/7.47.0
> Accept: */*
>
* Empty reply from server
* Connection #0 to host istio-citadel left intact
curl: (52) Empty reply from server

Citadel logs 2019-06-04T21:50:23.103439Z     info    grpc: Server.Serve failed to complete security handshake from "<IP_ADDRESS>:46458": tls: first record does not look like a TLS handshake

Note that a curl against citadel on the monitoring report works:
curl -is http://istio-citadel:15014/version
HTTP/1.1 200 OK
Date: Wed, 05 Jun 2019 14:08:56 GMT
Content-Length: 151
Content-Type: text/plain; charset=utf-8

so it looks like it might only affect GRPC.

An even simpler test possibly:
From istio/security/cmd run run istio_ca/main.go --self-signed-ca to start a server.  Then just curl localhost:8060/-/ready

Thanks to Greg H for testing on Golang 1.12.0 which also fails the curl with an RC of 52, so appears to be 1.12.x wide.

@howardjohn the log message can apparently be ignored - it appears in both the working case in go 1.11.10 and earlier as well in the failing case with go 1.12.0 and later

Investigating why simpleTest does not fail during CircleCI testing (which is building with Golang 1.12) but does fail in prow. CircleCI doesn't use the set --installer helm which implies that --set global.enableHelmTest=true is not set (so uses the false value from values.yaml). This prevents the Istio-security-test from running in CircleCi. So CircleCI simpleTest is a subset of the prow tests.

Verified that removing the installer=helm from the failing _make _ command noted above (so make e2e_simple E2E_ARGS="--skip_cleanup --auth_enable" HUB=ericvn  TAG=ericvn) will pass.

Note that this issue prevents completing #14420 which is targeted for 1.2 at present.

@morvencao I've done some searching today and I don't know that curl and GRPC servers mix all that well. I'm not sure if you understand this test fully, but do you think curl against the GRPC server should continue to work regardless go Golang version. Or maybe this test should really use curl against a path on the monitoring port?
The reason I'm asking you is that it seems this helm test was added as part of https://github.com/istio/istio/pull/10623 which you submitted.
Also note that it doesn't seem that the helm tests are currently part of the Circle test. I haven't looked to see if that is something that changed after the mentioned PR or not.

@ericvn Thanks for catching this. IMO, for the long-term, the helm test should be removed completely; For the short-term,I'm OK with curl against a path on the monitoring port, since it's complete plain http protocol now.
I'll raise a PR to fix this.

Verified that running the failing test with the changes from #14160 now results in a passing test. Thus I will close this issue.
On to testing the new Golang prow 1.2 images with the above fix to verify things pass with prow.


Deploy static files to any directory under /home

Static files now can be deployed to any directory under /home. Static files cannot be deployed with type=clean to protect user from wiping the entire /home directory. Removing type=lib and type=script as they both just function the same way type=static does.

@jvano @dfuenzalida @gregorygoldshteyn @caovicto

@suwatch


Replace deprecated std::aligned_storage_t.

Firstly, let me thank you for all the effort you have put into making this library. :)
std::aligned_storage_t is being deprecated in C++23.  Some compilers (such as MSVC) have now started emitting warnings when std::aligned_storage(_t) is used.
This StackOverflow post and this section on cppreference suggest some potential replacements.
Apologies in advance if this is a duplicate issue. I've checked and could find any mention of it, so I created this one.

Interesting. I didn't even know it was deprecated.
I'll try to address this one soon but 1) I'm on vacation, so don't expect a fast fix 🙂 and 2) it's kinda low prio since the library is still stuck on C++17 (because the gamedev world is stuck on that version of the standard at the moment and I won't mention consoles here just for the sake of mentioning the root cause 😅).
That said, I'm definitely not opposed to anticipating porting issues, so this one is welcome. Thanks for pointing it out! 👍

Looking into this issue. My only concern is that entt::any is currently defined as:
template<std::size_t Len = sizeof(double[2]), std::size_t Align = alignof(typename std::aligned_storage_t<Len + !Len>)>
class basic_any;

I'm not that sure about how to replace alignof(typename std::aligned_storage_t<Len + !Len>).
Actually, I'm not even sure if I want to stick with this definition since the Len and Align pair feels odd now that I've read the paper. 😅
Any suggestion?

I may be overlooking something, but is there anything wrong with creating a constexpr function to round a number to the nearest power of two and giving Len + !Len to it, and setting that to the alignment?
Rough example:
template<std::size_t Len = sizeof(double[2]), std::size_t Align = nearest_power_of_two<Len + !Len>()>
class basic_any;

However, I am definitely not the best person to ask and there most likely exists a much cleaner/more correct solution.

I played around with alignments on compiler explorer for a bit and I have come up with an idea.
alignof(std::max_align_t) is the largest that alignof(std::aligned_storage_t<Size>) can be (tried on a few compilers).
So, we could set the default Align to be the minimum of alignof(std::max_align_t) and Len + !Len.
However, there is the chance that Len + !Len could not be a power of two, so we can create a constexpr function to round it up to the nearest power of two.
Rough example:
template<std::size_t Len = sizeof(double[2]), std::size_t Align = std::min(alignof(std::max_align_t), nearest_power_of_two<Len + !Len>())>
class basic_any;

However, I am definitely not the best person to ask about this subject and there most likely exists a much cleaner/more correct solution.

Not sure the max between alignof(std::max_align_t) and Len + !Len is the right approach to be honest.
My plan was actually to get rid of the second parameter (the alignment) and to turn the remaining one into a type from which you want to extract these information or even a size of double[N] since this is what we're going to use under the hood in any case.
Consider that we're talking about a replacement for the template parameter list of entt::any where you don't have a value type after all.

That's definitely a much better approach. Thanks for resolving this issue so quickly! 😃

Actually, I think the current implementation doesn't work as expected.
I'm testing it on a 32 system and it triggers a warning that wasn't there before.


update versions in sample, update zipkin warning for migration to asa

@KaiqianYang thanks for the PR! In the future, please be sure to submit your PRs to the internal azure-dev-docs-pr repo so we can get full build support, internal review, preview builds, Acrolinx support, and so forth. This PR is small enough that I'll go ahead and handle it here, though. I recommend that you delete your fork of the public azure-dev-docs repo so that you won't use it by accident. If you use the web interface to create a PR, you can update the URL in the browser address bar to add the "-pr" part of the repo name. Thanks!

@KaiqianYang thanks for the PR! In the future, please be sure to submit your PRs to the internal azure-dev-docs-pr repo so we can get full build support, internal review, preview builds, Acrolinx support, and so forth. This PR is small enough that I'll go ahead and handle it here, though. I recommend that you delete your fork of the public azure-dev-docs repo so that you won't use it by accident. If you use the web interface to create a PR, you can update the URL in the browser address bar to add the "-pr" part of the repo name. Thanks!

@KarlErickson Got it, thanks


SSL error on running fetch_housing_data()

I ran the function to fetch the data and it has created a directory datasets/housing correctly, however I think it has trouble downloading housing.tgz file. Please help!
---------------------------------------------------------------------------
SSLError                                  Traceback (most recent call last)
/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/urllib/request.py in do_open(self, http_class, req, **http_conn_args)
   1317                 h.request(req.get_method(), req.selector, req.data, headers,
-> 1318                           encode_chunked=req.has_header('Transfer-encoding'))
   1319             except OSError as err: # timeout error

/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/http/client.py in request(self, method, url, body, headers, encode_chunked)
   1238         """Send a complete request to the server."""
-> 1239         self._send_request(method, url, body, headers, encode_chunked)
   1240

/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/http/client.py in _send_request(self, method, url, body, headers, encode_chunked)
   1284             body = _encode(body, 'body')
-> 1285         self.endheaders(body, encode_chunked=encode_chunked)
   1286

/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/http/client.py in endheaders(self, message_body, encode_chunked)
   1233             raise CannotSendHeader()
-> 1234         self._send_output(message_body, encode_chunked=encode_chunked)
   1235

/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/http/client.py in _send_output(self, message_body, encode_chunked)
   1025         del self._buffer[:]
-> 1026         self.send(msg)
   1027

/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/http/client.py in send(self, data)
    963             if self.auto_open:
--> 964                 self.connect()
    965             else:

/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/http/client.py in connect(self)
   1399             self.sock = self._context.wrap_socket(self.sock,
-> 1400                                                   server_hostname=server_hostname)
   1401             if not self._context.check_hostname and self._check_hostname:

/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/ssl.py in wrap_socket(self, sock, server_side, do_handshake_on_connect, suppress_ragged_eofs, server_hostname, session)
    400                          server_hostname=server_hostname,
--> 401                          _context=self, _session=session)
    402

/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/ssl.py in __init__(self, sock, keyfile, certfile, server_side, cert_reqs, ssl_version, ca_certs, do_handshake_on_connect, family, type, proto, fileno, suppress_ragged_eofs, npn_protocols, ciphers, server_hostname, _context, _session)
    807                         raise ValueError("do_handshake_on_connect should not be specified for non-blocking sockets")
--> 808                     self.do_handshake()
    809

/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/ssl.py in do_handshake(self, block)
   1060                 self.settimeout(None)
-> 1061             self._sslobj.do_handshake()
   1062         finally:

/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/ssl.py in do_handshake(self)
    682         """Start the SSL/TLS handshake."""
--> 683         self._sslobj.do_handshake()
    684         if self.context.check_hostname:

SSLError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:749)

During handling of the above exception, another exception occurred:

URLError                                  Traceback (most recent call last)
<ipython-input-28-09097de26f5a> in <module>()
----> 1 fetch_housing_data()

<ipython-input-27-fa2a4bf02df6> in fetch_housing_data(housing_url, housing_path)
     12         os.makedirs(housing_path)
     13     tgz_path = os.path.join(housing_path, "housing.tgz")
---> 14     urllib.request.urlretrieve(housing_url, tgz_path)
     15     housing_tgz = tarfile.open(tgz_path)
     16     housing_tgz.extractall(path=housing_path)

/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/urllib/request.py in urlretrieve(url, filename, reporthook, data)
    246     url_type, path = splittype(url)
    247
--> 248     with contextlib.closing(urlopen(url, data)) as fp:
    249         headers = fp.info()
    250

/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/urllib/request.py in urlopen(url, data, timeout, cafile, capath, cadefault, context)
    221     else:
    222         opener = _opener
--> 223     return opener.open(url, data, timeout)
    224
    225 def install_opener(opener):

/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/urllib/request.py in open(self, fullurl, data, timeout)
    524             req = meth(req)
    525
--> 526         response = self._open(req, data)
    527
    528         # post-process response

/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/urllib/request.py in _open(self, req, data)
    542         protocol = req.type
    543         result = self._call_chain(self.handle_open, protocol, protocol +
--> 544                                   '_open', req)
    545         if result:
    546             return result

/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/urllib/request.py in _call_chain(self, chain, kind, meth_name, *args)
    502         for handler in handlers:
    503             func = getattr(handler, meth_name)
--> 504             result = func(*args)
    505             if result is not None:
    506                 return result

/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/urllib/request.py in https_open(self, req)
   1359         def https_open(self, req):
   1360             return self.do_open(http.client.HTTPSConnection, req,
-> 1361                 context=self._context, check_hostname=self._check_hostname)
   1362
   1363         https_request = AbstractHTTPHandler.do_request_

/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/urllib/request.py in do_open(self, http_class, req, **http_conn_args)
   1318                           encode_chunked=req.has_header('Transfer-encoding'))
   1319             except OSError as err: # timeout error
-> 1320                 raise URLError(err)
   1321             r = h.getresponse()
   1322         except:

URLError: <urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:749)>

Hi Aishni,
I believe this bug is due to the fact that Python 3.6 on MacOSX has no certificates at all (see the release notes), so it cannot verify the SSL certificate from GitHub's servers when trying to download housing.tgz.  See the second answer to this StackOverflow question for details.  The solution is to:

read /Applications/Python 3.6/ReadMe.rtf
the ReadMe will have you run /Applications/Python 3.6/Install Certificates.command which installs the certificates.

Alternatively, you can work around the issue by downloading the file yourself and placing it in the housing directory, then comment out the urlretrieve() line in the code, and run it again.
Hope this helps,
Aurélien

Hi Aishni,
Did my answer solve your problem?  May I close this issue?
Cheers,
Aurélien

Hi,
It worked! Yes you may close he issue.
Thank you very much.

Aishni

On Thursday, June 29, 2017, Aurélien Geron<EMAIL_ADDRESS>wrote:

Hi Aishni,
Did my answer solve your problem? May I close this issue?
Cheers,
Aurélien
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ageron/handson-ml/issues/46#issuecomment-311884822,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEZlm2G82hAzgS1AaefWWQ487uesfLr_ks5sI1FlgaJpZM4OFpvk
.

--
Sent from iPhone

Perfect, thanks for your feedback.  :)

Thanks the solution worked .
read /Applications/Python 3.6/ReadMe.rtf
the ReadMe will have you run /Applications/Python 3.6/Install Certificates.command which installs the certificates.

**Hey
i'm using fedora as os
it's give me belove error
i'm use pycharm with envs of anaconda
**
WARNING:tensorflow:From /home/sunil/PycharmProjects/test/testFile.py:7: read_data_sets (from tensorflow.contrib.learn.python.learn.datasets.mnist) is deprecated and will be removed in a future version.
Instructions for updating:
Please use alternatives such as official/mnist/dataset.py from tensorflow/models.
Extracting MNIST_data/train-images-idx3-ubyte.gz
WARNING:tensorflow:From /home/sunil/anaconda3/envs/condaEnvTest/lib/python3.6/site-packages/tensorflow/contrib/learn/python/learn/datasets/mnist.py:260: maybe_download (from tensorflow.contrib.learn.python.learn.datasets.base) is deprecated and will be removed in a future version.
Instructions for updating:
Please write your own downloading logic.
WARNING:tensorflow:From /home/sunil/anaconda3/envs/condaEnvTest/lib/python3.6/site-packages/tensorflow/contrib/learn/python/learn/datasets/mnist.py:262: extract_images (from tensorflow.contrib.learn.python.learn.datasets.mnist) is deprecated and will be removed in a future version.
Instructions for updating:
Please use tf.data to implement this functionality.
Extracting MNIST_data/train-labels-idx1-ubyte.gz
WARNING:tensorflow:From /home/sunil/anaconda3/envs/condaEnvTest/lib/python3.6/site-packages/tensorflow/contrib/learn/python/learn/datasets/mnist.py:267: extract_labels (from tensorflow.contrib.learn.python.learn.datasets.mnist) is deprecated and will be removed in a future version.
Instructions for updating:
Please use tf.data to implement this functionality.
WARNING:tensorflow:From /home/sunil/anaconda3/envs/condaEnvTest/lib/python3.6/site-packages/tensorflow/contrib/learn/python/learn/datasets/mnist.py:110: dense_to_one_hot (from tensorflow.contrib.learn.python.learn.datasets.mnist) is deprecated and will be removed in a future version.
Instructions for updating:
Please use tf.one_hot on tensors.
Extracting MNIST_data/t10k-images-idx3-ubyte.gz
Extracting MNIST_data/t10k-labels-idx1-ubyte.gz
WARNING:tensorflow:From /home/sunil/anaconda3/envs/condaEnvTest/lib/python3.6/site-packages/tensorflow/contrib/learn/python/learn/datasets/mnist.py:290: DataSet.init (from tensorflow.contrib.learn.python.learn.datasets.mnist) is deprecated and will be removed in a future version.
Instructions for updating:
Please use alternatives such as official/mnist/dataset.py from tensorflow/models.

Hi @Sunil1997 ,
These are just warnings, not errors, you can ignore them for now.  I will update the notebooks to use Keras' functionality for loading MNIST, because TensorFlow's functionality has been deprecated and is printing all these warnings. However, the code still works for now, so you can just ignore these warnings.

i am run the following on red hat Linux
In [12]: from keras.applications import VGG16
In [13]: from keras import backend as K
In [14]: model = VGG16(weights='imagenet',
...: include_top=False) ****
and i found the following error
Downloading data from https://github.com/fchollet/deep-learning-models/releases/download/v0.1/vgg16_weights_tf_dim_ordering_tf_kernels_notop.h5
OSError                                   Traceback (most recent call last)
~/anaconda3/lib/python3.6/urllib/request.py in do_open(self, http_class, req, **http_conn_args)
1317                 h.request(req.get_method(), req.selector, req.data, headers,
-> 1318                           encode_chunked=req.has_header('Transfer-encoding'))
1319             except OSError as err: # timeout error
~/anaconda3/lib/python3.6/http/client.py in request(self, method, url, body, headers, encode_chunked)
1253         """Send a complete request to the server."""
-> 1254         self._send_request(method, url, body, headers, encode_chunked)
1255
~/anaconda3/lib/python3.6/http/client.py in _send_request(self, method, url, body, headers, encode_chunked)
1299             body = _encode(body, 'body')
-> 1300         self.endheaders(body, encode_chunked=encode_chunked)
1301
~/anaconda3/lib/python3.6/http/client.py in endheaders(self, message_body, encode_chunked)
1248             raise CannotSendHeader()
-> 1249         self._send_output(message_body, encode_chunked=encode_chunked)
1250
~/anaconda3/lib/python3.6/http/client.py in _send_output(self, message_body, encode_chunked)
1035         del self._buffer[:]
-> 1036         self.send(msg)
1037
~/anaconda3/lib/python3.6/http/client.py in send(self, data)
973             if self.auto_open:
--> 974                 self.connect()
975             else:

Hi @jemberie ,
This code does not look like it comes from the book or from this repository. It looks like you're getting a timeout while downloading VGG16. Check your internet connection? Try from another computer? If nothing works, please ask on StackOverflow instead.

--This answer is for MACOS--
Hi,
For python 3.9 and above, just go to Applications -> Python x.x -> there will be a .command file to install certificates, just click on that and you're done.

Unsubscribe @.***
On Mon, May 15, 2023 at 11:50 'Adarsh Raghav' via OpenWise Foundation Info <
@.***> wrote:

--This answer is for MACOS--
Hi,
For python 3.9 and above, just go to Applications -> Python x.x -> there
will be a .command file to install certificates, just click on that and
you're done.
—
Reply to this email directly, view it on GitHub
https://github.com/ageron/handson-ml/issues/46#issuecomment-1548115820,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEMJO2RALC3GRNGM65HAU2DXGJGEBANCNFSM4DQWTPSA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Dr. Danielah
 Andrée
G Ashtier (Formerly Gautier)
Operations Director, OpenWise Foundation
Tel:<PHONE_NUMBER> <%28301%29%20744-9443>
Fax:<PHONE_NUMBER> <%28202%29%20810-9128>
Developing Tomorrow's Engineering & Science Leaders!™
Follow Us: openwiselearning.org | Facebook
https://www.facebook.com/OpenWiseLearning | Twitter
https://twitter.com/OpenWiseLearn | LinkedIn
https://www.linkedin.com/company/openwiselearning

Email mistake . Second turn
On Mon, May 15, 2023 at 12:50 PM Adarsh Raghav @.***>
wrote:

--This answer is for MACOS--
Hi,
For python 3.9 and above, just go to Applications -> Python x.x -> there
will be a .command file to install certificates, just click on that and
you're done.
—
Reply to this email directly, view it on GitHub
https://github.com/ageron/handson-ml/issues/46#issuecomment-1548115820,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACQPMJRMBXAJ4TG5N6WAUFDXGJGDPANCNFSM4DQWTPSA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Unsubscribe @.*** please
On Mon, May 15, 2023 at 12:11 'Mauricio Souza Rodrigues da Cunha' via
OpenWise Foundation Info @.***> wrote:

Email mistake . Second turn
On Mon, May 15, 2023 at 12:50 PM Adarsh Raghav @.***>
wrote:

--This answer is for MACOS--
Hi,
For python 3.9 and above, just go to Applications -> Python x.x -> there
will be a .command file to install certificates, just click on that and
you're done.
—
Reply to this email directly, view it on GitHub
<https://github.com/ageron/handson-ml/issues/46#issuecomment-1548115820
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/ACQPMJRMBXAJ4TG5N6WAUFDXGJGDPANCNFSM4DQWTPSA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub
https://github.com/ageron/handson-ml/issues/46#issuecomment-1548153408,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEMJO2XT2542MN42AFMV3BDXGJIRZANCNFSM4DQWTPSA
.
You are receiving this because you commented.Message ID:
@.***>

--
Dr. Danielah
 Andrée
G Ashtier (Formerly Gautier)
Operations Director, OpenWise Foundation
Tel:<PHONE_NUMBER> <%28301%29%20744-9443>
Fax:<PHONE_NUMBER> <%28202%29%20810-9128>
Developing Tomorrow's Engineering & Science Leaders!™
Follow Us: openwiselearning.org | Facebook
https://www.facebook.com/OpenWiseLearning | Twitter
https://twitter.com/OpenWiseLearn | LinkedIn
https://www.linkedin.com/company/openwiselearning

*** @ ***
On Mon, May 15, 2023 at 2:51 PM OpenWise Foundation <
@.***> wrote:

Unsubscribe @.*** please
On Mon, May 15, 2023 at 12:11 'Mauricio Souza Rodrigues da Cunha' via
OpenWise Foundation Info @.***> wrote:

Email mistake . Second turn
On Mon, May 15, 2023 at 12:50 PM Adarsh Raghav @.***>
wrote:

--This answer is for MACOS--
Hi,
For python 3.9 and above, just go to Applications -> Python x.x ->
there
will be a .command file to install certificates, just click on that and
you're done.
—
Reply to this email directly, view it on GitHub
<
https://github.com/ageron/handson-ml/issues/46#issuecomment-1548115820
,
or unsubscribe
<

https://github.com/notifications/unsubscribe-auth/ACQPMJRMBXAJ4TG5N6WAUFDXGJGDPANCNFSM4DQWTPSA

.
You are receiving this because you are subscribed to this
thread.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub
<https://github.com/ageron/handson-ml/issues/46#issuecomment-1548153408
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AEMJO2XT2542MN42AFMV3BDXGJIRZANCNFSM4DQWTPSA
.
You are receiving this because you commented.Message ID:
@.***>

--
Dr. Danielah
 Andrée
G Ashtier (Formerly Gautier)
Operations Director, OpenWise Foundation
Tel:<PHONE_NUMBER> <%28301%29%20744-9443>
Fax:<PHONE_NUMBER> <%28202%29%20810-9128>
Developing Tomorrow's Engineering & Science Leaders!™
Follow Us: openwiselearning.org | Facebook
https://www.facebook.com/OpenWiseLearning | Twitter
https://twitter.com/OpenWiseLearn | LinkedIn
https://www.linkedin.com/company/openwiselearning
—
Reply to this email directly, view it on GitHub
https://github.com/ageron/handson-ml/issues/46#issuecomment-1548290149,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACQPMJXIQANRGSP27UUFMF3XGJUL3ANCNFSM4DQWTPSA
.
You are receiving this because you commented.Message ID:
@.***>


Typo which changes meaning in "localStorage by Building a Todo App" lesson of the new JavaScript course

Describe the Issue
In step 30, we refactor the code a bit by splitting up code we already wrote into functions. One of the statements to move is an if statement which checks the result of a findIndex function on an array.  I forgot to mark the if statement in my cut-paste job and got the following note when checking the code:
You should move the if statement with the condition dataArrIndex === 1 into your addOrUpdateTask function.
Unfortunately the condition in the note is completely wrong as the condition we want is to check if it's equal to -1 as that is the flag the findIndex function returns if it doesn't find a match. If the return value was 1, that's the id and we'd not want to enter the if statement.
It's only the note which is wrong, and if I enter
if (dataArrIndex === 1) {     taskData.unshift(taskObj);   }
in the code, then I still have the note when checking the code, and only the actual correct if statement of equalling -1 lets me continue. But I'm sure such a situation could be frustrating for a camper if they write what's in the note and stay stuck
Affected Page
https://www.freecodecamp.org/learn/javascript-algorithms-and-data-structures-v8/learn-localstorage-by-building-a-todo-app/step-30
Your code
const addOrUpdateTask = ()=>{
 const dataArrIndex = taskData.findIndex((item) => item.id === currentTask.id);
  const taskObj = {
    id: `${titleInput.value.toLowerCase().split(" ").join("-")}-${Date.now()}`,
    title: titleInput.value,
    date: dateInput.value,
    description: descriptionInput.value,
  };
if (dataArrIndex === 1) {
    taskData.unshift(taskObj);
  }
};

Expected behavior
To not accept the code and display a note saying the if condition should be dataArrIndex === -1
Screenshots

System

Device: VivioBook ASUS Laptop
OS: Windows 11 Home
Browser: Mozilla Firefox

Additional context
No response

Thanks for opening this issue.
This looks something that can be fixed by "first time" code contributors to this repository. Here are the files that you should be looking at to work on a fix:
List of files:

https://github.com/freeCodeCamp/freeCodeCamp/blob/main/curriculum/challenges/english/15-javascript-algorithms-and-data-structures-22/learn-localstorage-by-building-a-todo-app/64fad07f43a101779cb8692a.md

Please make sure you read our guidelines for contributing, we prioritize contributors following the instructions in our guides. Join us in our chat room or the forum if you need help contributing, our moderators will guide you through this.
Sometimes we may get more than one pull requests. We typically accept the most quality contribution followed by the one that is made first.
Happy contributing.

This looks something that can be fixed by "first time" code contributors to this repository. Here are the files that you should be looking at to work on a fix:

Thank you. I tried to fix it in the repository. Hope I did it properly :)

I tried to fix it in the repository. Hope I did it properly :)

@bwyoungy
I see that you made a commit to your forked copy of the project here

But I don't see any new pull requests opened by you.
Please read through the Contributing Guide to learn how to make a pull request.
If you need help, then please post on either the forum or discord with your question and people can assist you there 👍


Slack bot

The actual bot and API integration should be easy, but this will also require:

An OAuth workflow to capture API keys for each Slack workspace
Storing workspace keys -- is a simple DB enough?
Fetch & use key for the correct workspace on each API action
Probably normalizing the Person data (& merging with Avatar data)...

This is now done, installation button is on the About page: https://epyc.phlegmatic.ca/about


Fix missed tipb import

What have you changed? (mandatory)
Fix an import I missed in https://github.com/tikv/tikv/pull/5221
What are the type of the changes? (mandatory)

Engineering (engineering change which doesn't change any feature or fix any issue)

How has this PR been tested? (mandatory)
make dev
Does this PR affect documentation (docs) or release note? (mandatory)
no
Does this PR affect tidb-ansible update? (mandatory)
no
PTAL @siddontang @breeswish

/run-all-tests

/run-all-tests


Register as an anonymous AMD module

See [Register as an anonymous module]
(https://github.com/jrburke/requirejs/wiki/Updating-existing-libraries#anon)

Merged into integration-0.11.2. Thanks.


Deleting large amount of data (400 GB) causes blue screen with KERNEL_SECURITY_CHECK_FAILURE in vmbkmclr!InCompletePacket

Windows Build Number
Microsoft Windows [Version 10.0.22000.160]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>-microsoft-standard-WSL2
Distro Version
Ubuntu 20.04
Other Software
No response
Repro Steps

Create 400 GB of data in WSL2. (Average file size was 100 MB)
rm -rf the directory containing the data.
Crash.

This was on a system with a fast NVMe SSD.
Expected Behavior
No crash.
Actual Behavior
System bluescreen.
Diagnostic Logs
WinDbg crash analysis:
crash.txt

This sounds familiar, I've started a thread with the storage folks (a peer team of mine).

This is still happening with build 22504.1000.
*******************************************************************************
*                                                                             *
*                        Bugcheck Analysis                                    *
*                                                                             *
*******************************************************************************

SYSTEM_THREAD_EXCEPTION_NOT_HANDLED (7e)
This is a very common BugCheck.  Usually the exception address pinpoints
the driver/function that caused the problem.  Always note this address
as well as the link date of the driver/image that contains this address.
Arguments:
Arg1: ffffffffc0000005, The exception code that was not handled
Arg2: fffff80334162cb3, The address that the exception occurred at
Arg3: ffffa88cfa47f208, Exception Record Address
Arg4: ffffa88cfa47ea20, Context Record Address

Debugging Details:
------------------

Page ffc9f0 not present in the dump file. Type ".hh dbgerr004" for details

KEY_VALUES_STRING: 1

    Key  : AV.Dereference
    Value: String

    Key  : AV.Fault
    Value: Read

    Key  : Analysis.CPU.mSec
    Value: 1671

    Key  : Analysis.DebugAnalysisManager
    Value: Create

    Key  : Analysis.Elapsed.mSec
    Value: 5961

    Key  : Analysis.Init.CPU.mSec
    Value: 530

    Key  : Analysis.Init.Elapsed.mSec
    Value: 35710

    Key  : Analysis.Memory.CommitPeak.Mb
    Value: 105

    Key  : WER.OS.Branch
    Value: rs_prerelease

    Key  : WER.OS.Timestamp
    Value: 2021-11-12T16:50:00Z

    Key  : WER.OS.Version
    Value: 10.0.22504.1000

FILE_IN_CAB:  MEMORY.DMP

TAG_NOT_DEFINED_202b:  *** Unknown TAG in analysis list 202b

DUMP_FILE_ATTRIBUTES: 0x1800

BUGCHECK_CODE:  7e

BUGCHECK_P1: ffffffffc0000005

BUGCHECK_P2: fffff80334162cb3

BUGCHECK_P3: ffffa88cfa47f208

BUGCHECK_P4: ffffa88cfa47ea20

EXCEPTION_RECORD:  ffffa88cfa47f208 -- (.exr 0xffffa88cfa47f208)
ExceptionAddress: fffff80334162cb3 (vmbkmclr!InCompletePacket+0x00000000000004a3)
   ExceptionCode: c0000005 (Access violation)
  ExceptionFlags: 00000000
NumberParameters: 2
   Parameter[0]:<PHONE_NUMBER>000000
   Parameter[1]:<PHONE_NUMBER>736564
Attempt to read from address<PHONE_NUMBER>736564

CONTEXT:  ffffa88cfa47ea20 -- (.cxr 0xffffa88cfa47ea20)
rax=0000000041f85ed3 rbx=ffffd20d72dfa000 rcx=0000007f63736564
rdx=ffffd20d72dfa0a0 rsi=0000000000000000 rdi=0000004763736564
rip=fffff80334162cb3 rsp=ffffa88cfa47f440 rbp=0000000000000040
 r8=ffffd20d72dfa3bc  r9=0000000000000040 r10=fffff803341627e0
r11=ffff857c84e00000 r12=ffffd20d72dfa3bc r13=0000000000000004
r14=ffffd20d7cc3b060 r15=ffffd20d72dfa610
iopl=0         nv up ei pl nz na po nc
cs=0010  ss=0018  ds=002b  es=002b  fs=0053  gs=002b             efl=00050206
vmbkmclr!InCompletePacket+0x4a3:
fffff803`34162cb3 488b0f          mov     rcx,qword ptr [rdi] ds:002b:00000047`63736564=????????????????
Resetting default scope

BLACKBOXBSD: 1 (!blackboxbsd)

BLACKBOXNTFS: 1 (!blackboxntfs)

BLACKBOXPNP: 1 (!blackboxpnp)

BLACKBOXWINLOGON: 1

PROCESS_NAME:  System

READ_ADDRESS:<PHONE_NUMBER>736564

ERROR_CODE: (NTSTATUS) 0xc0000005 - The instruction at 0x%p referenced memory at 0x%p. The memory could not be %s.

 It may help if you boot into Safe Mode.

It appears you're experiencing two different stop codes: 0x139 for a critical data structure corruption and 0x7E for an unhandled system thread exception. It may help if you boot into Safe Mode.

It's clearly not a driver problem but a bug in VHD handling. How is safe mode supposed to help?

It's clearly not a driver problem but a bug in VHD handling. How is safe mode supposed to help?
Safe Mode only loads a limited set of drivers and devices, particularly Windows-default drivers, so no third-party driver will be loaded when booting in that Mode.

It's clearly not a driver problem but a bug in VHD handling. How is safe mode supposed to help?

Safe Mode only loads a limited set of drivers and components, particularly Windows-default drivers, so no third-party driver will be loaded when booting in that Mode.

Yes, I know that. Are you from Microsoft?

No, but I do know a bit about these features.

This is still not fixed in build 22593.1.
*******************************************************************************
*                                                                             *
*                        Bugcheck Analysis                                    *
*                                                                             *
*******************************************************************************

SYSTEM_THREAD_EXCEPTION_NOT_HANDLED (7e)
This is a very common BugCheck.  Usually the exception address pinpoints
the driver/function that caused the problem.  Always note this address
as well as the link date of the driver/image that contains this address.
Arguments:
Arg1: ffffffffc0000005, The exception code that was not handled
Arg2: fffff8048663a57c, The address that the exception occurred at
Arg3: ffff808658167228, Exception Record Address
Arg4: ffff990169fc6900, Context Record Address

Debugging Details:
------------------

KEY_VALUES_STRING: 1

    Key  : AV.Fault
    Value: Read

    Key  : Analysis.CPU.mSec
    Value: 1327

    Key  : Analysis.DebugAnalysisManager
    Value: Create

    Key  : Analysis.Elapsed.mSec
    Value: 6070

    Key  : Analysis.Init.CPU.mSec
    Value: 186

    Key  : Analysis.Init.Elapsed.mSec
    Value: 29987

    Key  : Analysis.Memory.CommitPeak.Mb
    Value: 117

    Key  : WER.OS.Branch
    Value: ni_release

    Key  : WER.OS.Timestamp
    Value: 2022-04-02T11:00:00Z

    Key  : WER.OS.Version
    Value: 10.0.22593.1

FILE_IN_CAB:  MEMORY.DMP

TAG_NOT_DEFINED_202b:  *** Unknown TAG in analysis list 202b

DUMP_FILE_ATTRIBUTES: 0x1800

BUGCHECK_CODE:  7e

BUGCHECK_P1: ffffffffc0000005

BUGCHECK_P2: fffff8048663a57c

BUGCHECK_P3: ffff808658167228

BUGCHECK_P4: ffff990169fc6900

EXCEPTION_RECORD:  ffff808658167228 -- (.exr 0xffff808658167228)
ExceptionAddress: fffff8048663a57c (nt!RtlpHpSegPageRangeComputeLargePageCost+0x000000000000004c)
   ExceptionCode: c0000005 (Access violation)
  ExceptionFlags: 00000000
NumberParameters: 2
   Parameter[0]:<PHONE_NUMBER>000000
   Parameter[1]: ffffffffffffffff
Attempt to read from address ffffffffffffffff

CONTEXT:  ffff990169fc6900 -- (.cxr 0xffff990169fc6900)
rax=66a788df7de66b16 rbx=0000000000000000 rcx=0000000000000000
rdx=0000000000000001 rsi=0000000000000001 rdi=00000000000007ff
rip=fffff8048663a57c rsp=ffff808658167460 rbp=ffffe708c8010140
 r8=0000000000004000  r9=0000000000000000 r10=66a788df7de66b16
r11=0000000000004000 r12=ffffe7091cc00dc0 r13=000000000000006c
r14=ffffe7091cc00040 r15=0000000000000002
iopl=0         nv up ei pl zr na po nc
cs=0010  ss=0018  ds=002b  es=002b  fs=0053  gs=002b             efl=00050246
nt!RtlpHpSegPageRangeComputeLargePageCost+0x4c:
fffff804`8663a57c 440fb700        movzx   r8d,word ptr [rax] ds:002b:66a788df`7de66b16=????
Resetting default scope

BLACKBOXBSD: 1 (!blackboxbsd)

BLACKBOXNTFS: 1 (!blackboxntfs)

BLACKBOXPNP: 1 (!blackboxpnp)

BLACKBOXWINLOGON: 1

PROCESS_NAME:  System

READ_ADDRESS:  ffffffffffffffff

ERROR_CODE: (NTSTATUS) 0xc0000005 - The instruction at 0x%p referenced memory at 0x%p. The memory could not be %s.

This happened to me at least three times in the last 1/2 year (always on the newest W11 insider beta build), last time a few days ago, and one time not only BSOD, but corrupted Ubuntu file system. Very annoying, because I had to reinstall and reconfigure the whole distro after that. Luckily, I had no unpushed commits.
I'm using Rust, and one way to nearly surely hit it is cargo clean for a lager project :-/
Please take this serious, it is so frustrating that I'm considering to walk away from WSL and return to direct Ubuntu, or just ditch the PC model, and buy me a Mac desktop.

Thanks - I've followed up with the storage folks to see if this is a known issue.

@surban - would it be possible to enable driver verifier and share the crashdump?  Once you have the .dmp file you can send it to<EMAIL_ADDRESS>and ask them to forward it to me.
verifier.exe /volatile /adddriver vhdmp.sys /flags 0x2b

Sure, I will do so when I am back at the system.

@surban - would it be possible to enable driver verifier and share the crashdump? Once you have the .dmp file you can send it to<EMAIL_ADDRESS>and ask them to forward it to me.
verifier.exe /volatile /adddriver vhdmp.sys /flags 0x2b
<repro>
<share crashdump>

Well, before doing that, I would hardly suggest reading the documentation provided here: https://docs.microsoft.com/en-us/windows-hardware/drivers/devtest/driver-verifier

I am also having this issue. Bluescreen when removing large amount of data.

This has happened consistently for me the whole time I've used wsl2 (1 year+). I reproduce by just downloading db backups over a certain size. The blue screen happens either by deleting the backup, or deleting loaded database files. Happens to co-workers too. Should be really easy to reproduce - my system specs match the original report pretty closely.
I resized my wsl2 filesystem thinking that might prevent the problem, but it hasn't. I have plenty of free space and it doesn't matter.

@surban - would it be possible to enable driver verifier and share the crashdump? Once you have the .dmp file you can send it to<EMAIL_ADDRESS>and ask them to forward it to me.
verifier.exe /volatile /adddriver vhdmp.sys /flags 0x2b
<repro>
<share crashdump>

C:\>verifier.exe /volatile /adddriver vhdmp.sys /flags 0x2b
The specified flags 0x0000002b are not supported in volatile mode.
Run "verifier /?" for command line assistance. See the "/dif /now" syntax for
enabling most flags without rebooting.

Driver verifier does not accept the flags.

@benhillis
I've reproduced the crash using Driver Verifier with the following command line.
verifier.exe /flags 0x2b /driver vhdmp.sys

Any update on this? Basically makes WSL unusable for me.

I get this error as well when deleting large amounts of data.
Let me know if there is something I can add.

I get this error as well when deleting large amounts of data.
Let me know if there is something I can add.

Can you check on Insider Build 25136? I haven't encountered this problem for a while anymore on recent builds, but perhaps this is just luck.

Have it occured the crash into only SSD?
I couln't occured the crash :( (I using a HDD)

Had this happen too when deleting a 400 GB directory. During deletion (rm -rf) my PC blue screened with PAGE_FAULT_IN_NONPAGED_AREA. After rebooting I ran rm -rf which completed this time, but it blue screened shortly after with KERNEL_MODE_HEAP_CORRUPTION. After rebooting I found my Debian corrupted. I have both .dmp files from C:\Windows\Minidump and a MEMORY.dmp from C:\Windows (same creation time as the second crash) which I could provide if they're of use.

I was experiencing the same issue when I deleting 30Gb files with rm-rf. I had to completely move my Rust dev env to Azure VM. This is really annoying

It happened to me too, removing bloom (bigscience). The wsl install is few days old.

I don't use WSL, but I think this must be a very buggy function. I've seen a few crashes in this driver lately, and the the most recent I got a chance to look at was in InCompletePacket+0x1e6, where it first calls memcpy, and then the crash hits in memcpy itself. It's memcpying something from the third parameter of InCompletePacket to some offset in the first param. It's all heavily optimized, and the memcpy implementation is one of the new fancypants hand rolled assembly versions that dispatches based on type, and I don't have the patience to try and work out the args from this deep in the function. I can't really keep using driver verifier, none of the crashes I have are caught by it, and it's insanely slow.

I have also experienced this issue, when dropping around 20k files summing to around 450GB of data. Intensely frustrating that this has been extant for so long.

I'm constantly getting this when deleting large data inside WSL2 Ubuntu on Win11. This is extremely annoying.

On my end I’ve not experienced this issue since updating to latest versions
of WSL based on 5.15 kernel and Ubuntu 22.04
On Thu, Nov 10, 2022 at 12:35 PM kostiantyn-povnych <
@.***> wrote:

I'm constantly getting this when deleting large data inside WSL2 Ubuntu on
Win11. This is extremely annoying.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/WSL/issues/7335#issuecomment-1310651130,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJLWP2PYFGX5ZDNGWXJ7QT3WHUW6NANCNFSM5CRWIC3Q
.
You are receiving this because you commented.Message ID:
@.***>

@Thernn88 - what does wsl.exe -v show for you? Just curious as I continue to hit the crash and I think I'm up to date. My version is
Kernel version: <IP_ADDRESS>
WSLg version: 1.0.45
MSRDC version: 1.2.3575
Direct3D version: 1.606.4
DXCore version: 10.0.25131.1002-220531-1700.rs-onecore-base2-hyp
Windows version: 10.0.22621.755```

Same issue here. It usually happened when I delete files by cargo clean.

Same here with cargo clean on wsl2. very annoying since i had to reinstall the ubuntu distro and use some disk recovery tool to read back data in the vhdx file... Please prioritize the investigation and the fix.

Same issue here

Makes rust development on WSL pretty much impossible :/

Same happened to me after deleting 150GB worth of junk (a debug mode build of LLVM/clang). Not straight away, I ran a few more commands on the terminal and was then struck with IRQL_NOT_LESS_THAN_OR_EQUAL blue screen and data corruption on my WSL image :(
I actually had code below the WSL home dir that I don't want to lose...

Same BUG here!
VS Code Version: Version: 1.75.1 (user setup)
Commit: https://github.com/microsoft/vscode/commit/441438abd1ac652551dbe4d408dfcec8a499b8bf
Date: 2023-02-08T21:32:34.589Z
Electron: 19.1.9
Chromium: 102.0.5005.194
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22621
Sandboxed: No
OS Version: WIN11 22H2 X64, WSL2 (Ubuntu 22.04)

Exact same behaviour here right after deleting a large directory (200GB). Got BSOD, and now everything is dead.

I have been encountering and standing this issue for many months until it finally corrupted the vhd disc. I recreated a new WSL with a different configuration attaching/mounting the entire physical disc avoiding the vhd driver. It seems it helped and I don't experience this issue anymore.

i found another workaround if this issue happens to anybody else.

get StarWind V2V Converter https://www.starwindsoftware.com/tmplink/starwindconverter.exe
find your ext4.vhxd (mine was in C:\Users\<username>\AppData\Local\Packages\CanonicalGroupLimited.Ubuntu20.04onWindows_79rhkp1fndgsc\LocalState)
convert it to vhd
convert that vhd BACK to vhxd (i did it as resizable)

i really wish i had found this earlier after hours of panic and debugging. please fix this issue!


[net] Combine readTimer and writeTimer to improve performance

To achieve this, we must have two when   monotonic.Time fields and one timer.Async instead of two timer.Async. But first need some benchmark to prove the idea.

Done by this commit
p.s: forget to mention this issue! ;-)


This one should work even better. (I hope so)

https://www.unknowncheats.me/forum/1434706-post10.html

What? One updates the prediction and the 2nd one gets accurate timer using tickbase? And fware is already doing that to get accurate tickbase


Validation failure on subsequent product import request with the same product key

Describe the bug
if the same ProductImportRequest is sent multiple times with the same product key, regardless of whether any of the properties are updated; it is generating the following validation error:
“Can not remove the variant [ID:1] for XXX since it's the master variant”
To Reproduce
Steps to reproduce the behavior:

The following pseudo-code in Groovy can be used to create a product with the "default" product type(already exists)

     def productRequest = ProductImportRequestBuilder.of()
                .resources([ProductImportBuilder.of()
                                    .key("product1")
                                    .name(LocalizedStringBuilder.of().values(["de-DE" : "produkt1"]).build())
                                    .slug(LocalizedStringBuilder.of().values(["de-DE": "PRD1"]).build())
                                    .productType(ProductTypeKeyReferenceBuilder.of().key("default").build())
                                    .description(LocalizedStringBuilder.of().values(["de-DE": "sehr tolles Produkt"]).build())
                                    .build()]
                ).build()
        def response1 = client.withProjectKeyValue(projectKey).products()
                .importSinkKeyWithImportSinkKeyValue("products")
                .post(productRequest)
                .executeBlocking()
                .getBody()
        def response2 = client.withProjectKeyValue(projectKey).products()
                .importSinkKeyWithImportSinkKeyValue("products")
                .importOperations()
                .get()
                .executeBlocking()
                .getBody()
       def variantRequest = ProductVariantImportRequestBuilder.of()
                .resources([ProductVariantImportBuilder.of()
                                    .key("product-var1")
                                    .sku("12345")
                                    .product(ProductKeyReferenceBuilder.of().key("product1").build())
                                    .isMasterVariant(true)
                                    .build()]
                ).build()
        client.withProjectKeyValue(projectKey).productVariants()
                .importSinkKeyWithImportSinkKeyValue("variants")
                .post(variantRequest)
                .executeBlocking()
                .getBody()

If the product does not exist, running it will successfully create the product with one variant.
Running it again, when the description of the product changes(or when nothing changes) will result in response1 to be ACCEPTED and response2 to have VALIDATION_ERROR with the following error :
“Can not remove the variant [ID:1] for XXX since it's the master variant”

Expected behavior
There should be no error, and the product description should change upon subsequent requests.
Screenshots

Thank you for opening this issue, we'll get back to you as soon as we have a next update.

We are working on it, I'll update once the fix is ready.

Hi @dpasdar,
After investigating the issue we found the bug and sent a fix that is now deployed to production.
Would be great if you can test again this case and see if it's working properly,
Cheers,
Dan

Hi @danrleyt and @ashishhk, thanks a lot for fixing the issue. I have tested it just now and can confirm that it is working as expected. I will close the issue as it is resolved.


CustomizeWithAttribute class is sealed

Hello, I noticed that in a commit Switch from .ruleset to .editorconfig and fix static code analysis issues you added a sealed modifier for CustomizeWithAttribute class, so now it's impossible to inherit from it. Was this done by an accident or on purpose, since I can't find any comments regarding this change.

Hi @PaJomantas. It was by design. I should have sealed the functionality of this attribute originally when I designed it. What kind of functionality are you missing that you need to extend the CustomizeWithAttribute?

I had a following scenario (simplified)
public class MyEntity
{
   public ICollection<int> Ints { get; set; } = new List<int>();
}

public class EmptyListCustomization : ICustomization
{
   public void Customize(IFixture fixture)
   {
       fixture.Register<ICollection<int>>(() => new List<int>());
   }
}

public class EmptyListEntityAttribute : CustomizeWithAttribute
    {
    public EmptyListEntityAttribute() : base(typeof(EmptyListCustomization)) { }
    }

Which allowed me to use the following attribute in tests:
[Theory]
[AutoMockData]
public void TestScenario_TestOutcome([EmptyListEntity] MyEntity)

Instead of:
[Theory]
[AutoMockData]
public void TestScenario_TestOutcome([CustomizeWith(typeof(EmptyListCustomization))] MyEntity)

I can still achieve the same functionality, but it was just "sugar" when CustomizeWithAttribute was inheritable

I agree, the tool should not prevent bounding customization with attribute for the ease of use. I plan to revert that change together with introducing generic CustomizeWithAttribute hopefully next month.


Upgraded to .NET 7.0

Updated solution to the latest .NET version
Updated path of Publish profile to reflect the correct naming (no longer net5.0)
Updated .gitignore file to exclude VS-specific folder (MouseJiggler/.vs)

Thanks!
I just got a new development computer capable of running recent Visual Studio again, so we should hopefully see some actual progress on the release version of this before too long.


mobilenet_v2_140_224/classification: Cannot copy between a TensorFlowLite tensor with shape [1, 120] and a Java object with shape [1, 10, 4]

System information

Windows 10 10.0.18362 Build 18362
TensorFlow 2.2.0 installed from pip
Python v3.8.2

Describe the current behavior
I used the make_image_classifier (https://github.com/tensorflow/hub/tree/master/tensorflow_hub/tools/make_image_classifier) to generate the TFLITE model based on images available on http://vision.stanford.edu/aditya86/ImageNetDogs/:
make_image_classifier --image_dir images \
--tfhub_module https://tfhub.dev/google/imagenet/mobilenet_v2_140_224/classification/2 \
--image_size 224 \
--saved_model_dir models \
--labels_output_file output/labels.txt \
--tflite_output_file output/retrained.tflite

Then I copied the .tflite file to my Android project assets folder and tried to execute the app, but it crashed with the following error:
Process: org.tensorflow.lite.examples.detection, PID: 9525
   java.lang.IllegalArgumentException: Cannot copy between a TensorFlowLite tensor with shape [1, 120] and a Java object with shape [1, 10, 4].
       at org.tensorflow.lite.Tensor.throwIfShapeIsIncompatible(Tensor.java:412)
       at org.tensorflow.lite.Tensor.throwIfDataIsIncompatible(Tensor.java:369)
       at org.tensorflow.lite.Tensor.copyTo(Tensor.java:247)
       at org.tensorflow.lite.NativeInterpreterWrapper.run(NativeInterpreterWrapper.java:166)
       at org.tensorflow.lite.Interpreter.runForMultipleInputsOutputs(Interpreter.java:314)
       at org.tensorflow.lite.examples.detection.tflite.TFLiteObjectDetectionAPIModel.recognizeImage(TFLiteObjectDetectionAPIModel.java:195)
       at org.tensorflow.lite.examples.detection.DetectorActivity$2.run(DetectorActivity.java:181)
       at android.os.Handler.handleCallback(Handler.java:873)
       at android.os.Handler.dispatchMessage(Handler.java:99)
       at android.os.Looper.loop(Looper.java:193)
       at android.os.HandlerThread.run(HandlerThread.java:65)

I looked into almost similar issues, but none of them had the TensorFlowLite with a shape based on two parameters (i.e. [1, 120]) while having the Java object with a shape based on three (i.e. [1, 10, 4]).
make_image_classifier: https://github.com/tensorflow/hub/tree/master/tensorflow_hub/tools/make_image_classifier
Describe the expected behavior
The app should start with no problems and detect/classify objects based on the provided TFLITE model.
Standalone code to reproduce the issue
https://github.com/LavitzBr/object_detection

It looks like you're copying an image classification model to the object detection app, and that leads to the incompability issue.

@LavitzBr 1. Is this still an issue for you?
2.Did you check @xunkai55 response?
3.Can you please share a standalone code to reproduce the issue? Thanks!

@xunkai55, I'm following the correct commands for the model to be compatible with the code that calls it in the app. Do you have a clue on why the model not being correctly generated?

@LavitzBr Based on the contents in the original post, it looks like you got an "Image Classification" model (mobilenet_v2_140_224) from TFHub. That model is used for predict the category of images. However, your app is the "Object Detection" app. An "image classification" model cannot be used for "object detection" without subtle modifications.
If you want an Image classification app, please check out https://github.com/tensorflow/examples/tree/master/lite/examples/image_classification/android rather than the Object Detection app.
If you want an Object Detection app, you may want a "single-shot-detection" model, like "MobileNetV1-SSD" we used in the example app. https://tfhub.dev/tensorflow/lite-model/ssd_mobilenet_v1/1/default/1

@wilerjrxd Could you please try on latest stable version of tf 2.5 or 2.4.1 and let us know if this is still an issue.Thanks!


Unable to install library in my android project

Hi @evrencoskun!
Your project is very cool, but unfortunatly I can't install it to my project in Android Studio, by your Wiki page. ("Getting Started")
dependencies {
implementation 'com.evrencoskun.library:tableview:<IP_ADDRESS>'
implementation 'androidx.recyclerview:recyclerview:1.1.0'
}
I receive this message, after I try syncing gradle:
"Failed to resolve: com.evrencoskun.library:tableview:<IP_ADDRESS>"
Please help me!

The docs look wrong, I successful use
implementation 'com.evrencoskun.tableview:<IP_ADDRESS>'
As it is published at https://bintray.com/beta/#/evrencoskun/maven/tableview/<IP_ADDRESS>?tab=overview

Thank you @Zardozz !
Now it works fine!
Yes, the missing jcenter was the problem...


Get multiple query values using http:query/3

Currently http:query/3 only support getting a single value for specified name, if implementing a API server similar to OpenStack Neutron, e.g.:
GET /v2.0/networks.json?fields=id&fields=name

There is no way to get both value of fields (id and name).
It would be nice to make http:query/3 support backtracking.
Also, currently there is no way to iterate all query name and values, can http:query/3 be used like this?
http:query(S, K, V),
write(K), write(V), nl,
fail

Sure, will do.
On Fri, Jun 30, 2017 at 2:48 PM, Xin Wang<EMAIL_ADDRESS>wrote:

Currently http:query/3 only support getting a single value for specified
name, if implementing a API server similar to OpenStack Neutron
https://wiki.openstack.org/wiki/Neutron/APIv2-specification#Filtering_and_Column_Selection,
e.g.:
GET /v2.0/networks.json?fields=id&fields=name
There is no way to get both value of fields (id and name).
It would be nice to make http:query/3 support backtracking.
Also, currently there is no way to iterate all query name and values, can
http:query/3 be used like this?
http:query(S, K, V),
write(K), write(V), nl,
fail
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/trealla-lang/trealla/issues/189, or mute the thread
https://github.com/notifications/unsubscribe-auth/AVqpEqmz0Qk-I9rdrothIdXumoOloA_Rks5sJH4MgaJpZM4OKJVt
.

Would it be more useful to return a list of values for a key?
On 30 Jun 2017 2:48 pm, "Xin Wang"<EMAIL_ADDRESS>wrote:

Currently http:query/3 only support getting a single value for specified
name, if implementing a API server similar to OpenStack Neutron
https://wiki.openstack.org/wiki/Neutron/APIv2-specification#Filtering_and_Column_Selection,
e.g.:
GET /v2.0/networks.json?fields=id&fields=name
There is no way to get both value of fields (id and name).
It would be nice to make http:query/3 support backtracking.
Also, currently there is no way to iterate all query name and values, can
http:query/3 be used like this?
http:query(S, K, V),
write(K), write(V), nl,
fail
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/trealla-lang/trealla/issues/189, or mute the thread
https://github.com/notifications/unsubscribe-auth/AVqpEqmz0Qk-I9rdrothIdXumoOloA_Rks5sJH4MgaJpZM4OKJVt
.

I kind of prefer backtracking, if we return a list, http:query/3 will be inconvenient if only one value is needed.

I am adding
http:query_list/3
to return a list (or [] if no value). The old call http:query/3 still
returns an atom.
On Fri, Jun 30, 2017 at 8:25 PM, Xin Wang<EMAIL_ADDRESS>wrote:

I kind of prefer backtracking, if we return a list, http:query/3 will be
inconvenient if only one value is needed.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/trealla-lang/trealla/issues/189#issuecomment-312232900,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AVqpEg-Uzgwq5LPw4LFpzsGRH7T1uCFOks5sJM0XgaJpZM4OKJVt
.

Nice!
BTW, I found that the result list is in reversed order.

It works now, thanks!


swap COO for CN + constrained Optimization + env file fix on CUDA

swap COO for CN + constrained Optimization + env file fix on CUDA

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

-15 of 48 (68.75%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.4%) to 41.143%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

mofa/model.py
33
48
68.75%

Totals

Change from base Build<PHONE_NUMBER>:
0.4%

Covered Lines:
4018

Relevant Lines:
9766

💛  - Coveralls


Feedback for using react-tweet on a nextjs(pages router)

I am trying to use react-tweets on my but I have been getting this issues since

This article may help you @Polymath0033
https://react-tweet.vercel.app/next#troubleshooting


Passing an array to helm

Hello,
As per https://github.com/kubernetes/helm/blob/master/docs/using_helm.md#the-format-and-limitations-of---set and also I checked through https://github.com/kubernetes/helm/issues/1987 I am trying to pass http headers using set to the helm command.
$ helm install chartname --name=testinstance --set image.name=test --set image.tag=latest --set headerRule={"a\ b,c\ d"}
the  relevant part of template file is like
spec:
  rules:
    - host: {{ .Values.ingress.hostname }}
      http:
        paths:
          - path: /
            backend:
            {{- if .Values.headerRule -}}
              headerRule:
              {{- range .Values.headerRule }}
              - {{ . }}
              {{- end }}
            {{- end }}

when I execute above helm command it results in
spec:
  rules:
    - host: test.kube.test.com
      http:
        paths:
          - path: /
            backend:headerRule
              - a b
              - c d
              serviceName: test-kube-test
            servicePort: 8888

But the one am desiring or trying to get is
spec:
  rules:
    - host: test.kube.test.com
      http:
        paths:
          - path: /
            backend:
             headerRule:
               - a b
               - c d
              serviceName: test-kube-test
              servicePort: 8888

Could someone please help !?
Thanks,
Raju.

Yea I had resolved it like 30 mins after I posted the ticket! 😄 My bad and forgot to close the ticket.
Thanks you


Upload Hermes sourcemaps generated by CodePush

Description
Currently, on each OTA update using CodePush, I have to trigger an Android full build in order to generate the Hermes bundle & sourcemaps, found in android/app/build/generated/assets/react/release/index.android.bundle and android/app/build/generated/sourcemaps/react/release/index.android.bundle.map as stated in the docs:
https://docs.bugsnag.com/platforms/react-native/react-native/showing-full-stacktraces/#uploading-hermes-source-maps
As CodePush generates the JS bundle, as well as the Hermes byte-code bundle and sourcemaps, is there any way we can use these instead of those generated by the Android build?
Issue
I tried uploading CodePush's Hermes byte-code bundle and sourcemap to Bugsnag, I did not get any errors like "Unable to apply sourcemap", but the stacktrace is gibberish:

Environment
Library versions:

yarn list v1.22.4
├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>└─<EMAIL_ADDRESS>

Hey @rcstanciu, it's a known 'issue' with the CodePush / Hermes combo. The Hermes map is rendered in a binary format when produced through CodePush which is what you're seeing here.
Hopefully when the issue we raised here https://github.com/facebook/hermes/issues/85 is resolved, we'll be able to take a look into whether we can use the CodePush output files. In the meantime, what you were doing before with the Android build is correct!


Where is the alert-box?

Seems like alert-box is left out of this gem. Fresh install of Rails 5 with Foundation 6.

Alert boxes and panels from Foundation 5 have been combined to form the Callout component - see Docs.
Here's an example of creating a callout box in Foundation 6:

This a friendly message.

And when you're done with me, I can be closed using a Motion UI animation.

  <button class="close-button" aria-label="Dismiss alert" type="button" data-close>
    ×
  </button>


Use fixed version of Ubuntu 18.04 for builds

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 Apache License 2.0; 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.

ubuntu-latest was found to be fine as these issues were related to application-level changes.  As a result, I'm closing this PR.
On a somewhat related note, we also tried adding macos-latest to the os list, but found that docker was not available and probably won't be: https://github.community/t/why-is-docker-not-installed-on-macos/17017


SDL 0323 - Align video streaming parameters with VideoStreamingCapability

Proposal: Align video streaming parameters with VideoStreamingCapability

When Android SDL application starts video streaming, a developer can specify VideoStreamingParameters, but before actually starting the streaming, those parameters need to be aligned with VideoStreamingCapability returned by head units. The same is true for iOS SDL application; i.e. a developer can specify customVideoEncoderSettings, but those settings need to be aligned with VideoStreamingCapability.

Steering Committee Decision:
The Steering Committee voted to accept the proposal on 2020-11-24. Xevo, the authoring company of the proposal shared that this proposal's motivation was found while working on the implementation for SDL 0274 (an already accepted proposal).
Review: https://github.com/smartdevicelink/sdl_evolution/issues/1094

Because this feature is implemented in the same way with preferred FPS (SDL 0274), and implementation has some dependency on SDL 0292, I have added the commit into another PR. Please see issue #1389 for details. Thanks.


[Feature Request]: When creating a run, make pipeline input parameters more evident

Feature description
When creating a pipeline run, there is a Parameters field that shows the input parameters of the pipeline, but that’s not evident at first view (specially if you haven't created the pipeline)
I think Pipeline input parameters  or Input parameters could be more evident. Also, they could be under the pipeline selector, not below Run type

Describe alternatives you've considered
No response
Anything else?
No response

@kywalker-rh UX issue

@jgarciao I see your point, but we also have no idea how many parameters could be added on any given pipeline, so if there are 20 parameters the Run type section would end up being shown far down the page. We have a usability test scheduled soon, so we will see if this comes up as an issue there. If it does we'll think about solutions, if not, I propose we leave it as is.

@jgarciao I see your point, but we also have no idea how many parameters could be added on any given pipeline, so if there are 20 parameters the Run type section would end up being shown far down the page. We have a usability test scheduled soon, so we will see if this comes up as an issue there. If it does we'll think about solutions, if not, I propose we leave it as is.

Not sure if this helps -- but I think Jorge is mentioning that we call them just "Parameters" -- would we not want to call them "Run Parameters" "Pipeline Parameters" "Input Parameters" etc to help indicate what kind of params they are... is it immediately clear? The context helps a lot (we are on the create run page) but they are somewhat disconnected from the Pipeline selection.

Okay, that makes sense, thanks for the further explanation @andrewballantyne.
@kaedward If I'm understanding this correctly adding a descriptor to the current field of Parameters in the run creation page would be helpful. Do you have a preference for one of Andrew's suggestions ("Run Parameters" "Pipeline Parameters" "Input Parameters"), or a suggestion of your own?

@kywalker-rh I prefer "Pipeline input parameters", but if a user will already know that they're input parameters, we could drop "input" and just say "Pipeline parameters".

I think having "pipeline" in there is important if we don't switch the placement of run type and parameter.

No objections from me, I don't think we need to make it especially short. Let's go with your suggestion "Pipeline input parameters" @kaedward

I think this is taken care of.

/assign


Fix handling of object_types for NetBox 4.1 compatibility

@bb-Ricardo Buddy, check the requests. I've fixed the synchronization issue with NetBox version 4.1

this should be resolved with #415 in next-release branch.
would you be able to try it out?

solved with v1.7.0


IsNull() uses strings.TrimSpace() better than len()

IsNull() uses strings.TrimSpace() better than len()

trim space  \n  \t eg.

Wondering if IsBlank should be implemented?  Where IsNull would produce false on \n \t and IsBlank would product true, IsBlank would include the use of strings.TrimSpace()


Automated testing of tutorials /examples

checkdocs scans galleries

[x] tutorial card gets parsed, expanded, compiled, decompiled
[x] examples get parsed and expanded, compiled decompiled
[x] added decompiler flag to treat grey blocks as errors

Same parsing codepath as in the editor.

Thanks @pelikhan This is so awesome.


bump go.etcd.io/bbolt v1.3.3

this brings the dependency back to a released version:
full diff: go.etcd.io/bbolt https://github.com/etcd-io/bbolt/compare/2eb7227adea1d5cf85f0bc2a82b7059b13c2fa68...v1.3.3

etcd-io/bbolt#153 fix rollback panic bug

fixes etcd-io/bbolt#152 Panic (index out of range) on writeable tx rollback with db.NoFreelistSync

Codecov Report

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

@@           Coverage Diff           @@
##           master    #3519   +/-   ##
=======================================
  Coverage   44.27%   44.27%
=======================================
  Files         124      124
  Lines       13682    13682
=======================================
  Hits         6058     6058
  Misses       6688     6688
  Partials      936      936

Flag
Coverage Δ

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

#windows
39.76% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

LGTM


paper tabs test failing

pressing enter on tab is not causing a click

this is fixed with the element update (currently in the update branch)


diffmap - negative eigenvalues

import scanpy.api as sc
adata = sc.datasets.paul15()
sc.pp.neighbors(adata, n_neighbors=20, use_rep='X')
sc.tl.diffmap(adata, n_comps=60)
adata.uns['diffmap_evals']

Sorry for the late response, I was on holidays.
Computing 60 eigenvectors with Lanczos is terribly unstable. You shouldn't do that. I'm not astonished that some spurious negative eigenvalues arise.
Up to 20 or 30 max is OK.

well, we should warn the user when they do something uninformed. lanzcos is an implementation detail that our API hides, so it’s pure coincidence for an user to learn about it beforehand.

yes, this sanity check for n_components in diffmap definitely makes sense. makes sense for any parameter in any function. but is so much work! :wink: I'll add it for this case, soon...

yeah, this one is hard to catch, because it’s not obvious why you shouldn’t be able to specify any number of eigenvectors from 0 to n_obs.


Module 3 exercises [replace with your name]

Land Condition In Pandamatenga.pdf

Please "drag & drop" exercise 1 pdf file below.

Please "drag & drop" exercise 2 pdf file below.

Please UPDATE THE TITLE of this "Issue" in the title bar above with your name.

If you haven't done so, please go back to Module 3 and make sure to take
the post-training survey. Participating in surveys is required for receiving
the "GALUP Certificate" by the University of Florida.

Hi Wesley, please check your output again. Your values should read between 1 and 9. Also, your output does not match the expected output. Please resubmit to complete module 3. Thank you


armada doesn't ignore files listed in .helmignore

Is this a bug report or feature request? (choose one):
bug report

Python Version (output of python --version):
python --version
Python 2.7.12
Development or Deployment Environment?:
development
Release Tag or Master:
master
Expected Behavior:
Files listed in .helmignore would be ignored
What Actually Happened:
armada errored out due to unterminated quote in an rst file that should be ignored
How to Reproduce the Issue (as minimally as possible):
point armada at kolla-kubernetes/helm/kolla-common
Any Additional Comments:

@alop This is a good find.  @gardlt Can you take this?

hey @alop can you please provide your armada.yaml?

Unable to reproduce issue, close for now.


[Improvement] Add Typings for ErrorResponse

We should be eble to define typings for Error Responses in Endpoint type, and when making requests dynamically via fetchf() or api.request().

We could also describe the typings for errors in docs - Typings section.


Adding Docker Compose for Multi Container Local Deployment

Description
This PR includes the following proposed change(s):
N/A
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Refactoring / Documentation
[ ] Version change

if your change is a breaking change, please add breaking change label to this PR
How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Does the change impact or break the Docker build?

[x] Yes
[ ] No

If Yes: Has Docker been updated accordingly?

[x] Yes
[ ] No

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have made corresponding changes to the documentation
[x] New and existing unit tests pass locally with my changes

Please check this branch out and pull down the changes. The README.md should outline the steps required to get the containers off the ground.
Please reach out via teams so I can support as necessary.


Actual PerScreen implementation

Description
I have implemented an actual PerScreen layout (onScreen and onScreens). The old ifWidth is moved to a new module XMonad.Layout.ByWidth, and re-exported with a deprecation warning pointing to the new module.
Marked as draft: I'd like someone who can to test a corner case with respect to rescreen. I think it will usually work, but may glitch if you rapidly plug and unplug an external monitor to which this layout would apply. This can't do more than a brief glitch though, as anything that would trigger it will have another rescreen pending and xmonad will fix itself.
Checklist

[x] I've read CONTRIBUTING.md

[x] I've considered how to best test these changes (property, unit,
manually, ...) and concluded: manually tested except for the corner case mentioned above

[x] I updated the CHANGES.md file

I squashed it down a bit.

In case anyone's wondering, @liskin expressed a concern and plans to review this within the next few days, so don't rush into committing it just yet,

I renamed ByWidth.hs to IfWidth.hs for consistency with IfMax.hs and with its ifWidth function, and squashed everything in the process. AFAIK this is just waiting on someone to confirm that I'm handling the Hide and ReleaseResources messages properly.


help wanted!

This issue is just to make it clear that I am explicitly looking for volunteers to take on responsibility for versioning (#101), packaging (#100), documentation, and overall administration of xgcm. I just don't have time to tackle these things any time soon, but they are important for the long-term sustainability of this project.
For example, @jbusecke, @raphaeldussin, @willirath, @maikejulie, @jetesdal or anyone else who wants to learn more about how that stuff works, this is an opportunity to do so...

count me in!

Yup I would be interested!
Sent from my iPhone

On Jun 13, 2018, at 8:45 PM, Ryan Abernathey<EMAIL_ADDRESS>wrote:
This issue is just to make it clear that I am explicitly looking for volunteers to take on responsibility for versioning (#101), packaging (#100), documentation, and overall administration of xgcm. I just don't have time to tackle these things any time soon, but they are important for the long-term sustainability of this project.
For example, @jbusecke, @raphaeldussin, @willirath, @maikejulie, @jetesdal or anyone else who wants to learn more about how that stuff works, this is an opportunity to do so...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Great, thanks! I have given you both write access, so you can commit / merge PRs. (I would strongly prefer that all changes continue to come through PRs.) I probably also have to give you permissions on pypi--I'll try to sort that out asap.
So basically just start doing it. I recommend starting by copying xarray's "how to release" instructions / adapting them for xgcm:
https://github.com/pydata/xarray/blob/master/HOW_TO_RELEASE

In order to get permissions to publish on pypi, you need to make accounts there and then tell me your usernames. I will add you to the xgcm project: https://pypi.org/project/xgcm/

I'm happy to join and look into #100.
(But I'll be quite busy the next few days.)
Am 14. Juni 2018 02:45:07 schrieb Ryan Abernathey<EMAIL_ADDRESS>
This issue is just to make it clear that I am explicitly looking for volunteers to take on responsibility for versioning (#101), packaging (#100), documentation, and overall administration of xgcm. I just don't have time to tackle these things any time soon, but they are important for the long-term sustainability of this project.
For example, @jbusecke, @raphaeldussin, @willirath, @maikejulie, @jetesdal or anyone else who wants to learn more about how that stuff works, this is an opportunity to do so...
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/xgcm/xgcm/issues/104

Just to be clear, the near-term goal is to release xgcm v 0.2.0 via pip and conda-forge.

Do we have concrete milestones for 0.2.0 (e.g. working metrics module and perhaps vector calculus?)

Joined PyPI (username: jbusecke)

same (username: raphaeldussin)

Ok you are both added as maintainers.
Let's make 0.2.0 just the current state of the project. No need to wait for those hard things to get done. We have not had a release in a long time, and there are already many improvements that we need to get out there.
We can discuss milestones for 0.3.0 in a separate issue.

from the pypi and conda docs, it seems that the way to go is to start with creating the pypi package
then the conda recipe. I can start testing it next week if that's ok with everybody.

Happy to be involved, but a bit busy out June sorry!

Sounds great. I have started a todo list (#105), where we can discuss what is still missing for a clean release of v0.2.0. Perhaps its only the things that I have to do...then this might have been a bit overkill.
Either way let me know if there are things (particularly in the documentation) that are still needed.

from the pypi and conda docs, it seems that the way to go is to start with creating the pypi package
then the conda recipe. I can start testing it next week if that's ok with everybody.

Note that @ocefpaf has already created a conda-forge package for us (https://github.com/conda-forge/xgcm-feedstock), but it needs some attention (see discussion here: https://github.com/conda-forge/staged-recipes/pull/6056#issuecomment-397283231). He would be a great person to ask if you have questions about releasing.
As for the overall release procedure, I highly recommend you follow the procedure described in the xarray HOW_TO_RELEASE document:
https://github.com/pydata/xarray/blob/master/HOW_TO_RELEASE

Now that we have a more active development team should we close this? If we still want to highlight this issue, can we pin it somehow to the top? Otherwise it will not be seen I am afraid.

Ah there it is. I pinned this to the top.

He would be a great person to ask if you have questions about releasing.

I usually recommend people to take a look at https://github.com/ioos/ioos-python-package-skeleton

As for the overall release procedure, I highly recommend you follow the procedure described in the xarray HOW_TO_RELEASE document:
pydata/xarray@master/HOW_TO_RELEASE

Got a broken link in that but xarray is a bigger project and needs more human intervention for a release. This project is in good shape and I would recommend something simpler and automated:

dropping versioneer, which is deprecated, and adopting setuptools_scm;
adding some sdist tarball/wheels tests (see link above);
use a GH action to release every new tag.

I can send PRs to implent those if you want to adopt that wokflow.

I would like to keep this open, since it is a general announcement?


Fix broken code in custom latent trajectory example

Working with Hannah, we found a few parts of this example that do not work. This version fixes those regions. Not sure if the fixed version of the dataset saving is still the right way to go though, was this written before save_external_result()?

Sorry about that Nick, and sorry about a million commits here! In all of these cases, both the browser and VS Code were successfully showing the plot, and the  tag looked like it was still there, but not being rendered. I opted here to just save the image and include it as an attachment instead.

No worries Rich, thanks for working out the kinks!

Not sure if the fixed version of the dataset saving is still the right way to go though, was this written before save_external_result()?

Looks ok to me! Both save_external_result() and the save_output() strategy are valid. This strategy is nice because you can use alloc_output() to get a fresh empty dataset, which you can't do as easily with save_external_result()


Implement sample OPI API Conglomerate

This issue tracks a first cut at the infamous OPI API Conglomerate. This is a server which runs on the xPU card. Northbound, this will present gRPC to users. It will load vendor plugins, and southbound communicate with local resources on the xPU through the vendor plugin.
Eventually, this will likely (or could likely) move to being something which runs off the xPU cards and communicates to vendor agents on each card.

Closing as this was a direction we're no longer moving in.


Evaluate promise on a page whose navigate times out will never return

The problem
The following code snippet navigates a page to google and then to "https://monmothma.com" which is simply a random page that never responds with headers. It then calls evaluate and waits on the promise. The promise will never return.
const puppeteer = require('puppeteer');

(async () => {
  const browser = await puppeteer.launch();
  const page = await browser.newPage();
  await page.goto('https://www.google.com')
  page.goto('https://monmothma.com').catch(() => console.log('caught'));
  const frame = page.mainFrame()
  console.log('before')
  await frame.evaluate(() => console.log('foo'))
  console.log('after -- never reached')
})();

Impact
There are cases where one would want to evaluate some code and wait on the result without waiting on the page's load events to fire. While a valid work-around may be to wait on the navigate to succeed before trying to evaluate, that can be limiting especially on pages that take a long time to fire relevant load events.
Ask
I'm seeking guidance for how one can determine when it is safe to call evaluate and not have it hang. In this case, the frame has a url (though it is google) and an execution context so it seems like it should be safe, but isn't. Is there some canonical check one can make to determine if the page is safe to call and await evaluate on?
Should whatever evaluate is waiting on (it appears to just be waiting on a response to Runtime.callFunctionOn which would be in chrome) time out eventually?
Thanks!

Some science, this seems to be the design of Chrome to suspend sending messages to inspector agents during navigation
https://cs.chromium.org/chromium/src/content/browser/devtools/render_frame_devtools_agent_host.cc?l=488&rcl=8cae10903a021848ff481f009fafefaaf977901f
and resume sending after navigate
https://cs.chromium.org/chromium/src/content/browser/devtools/render_frame_devtools_agent_host.cc?l=428&rcl=8cae10903a021848ff481f009fafefaaf977901f
which essentially means its not safe to wait on an evaluation (or indeed a Page.evaluateOnNewDocument)  for a page/frame that may be navigating or could navigate before the devtools message is dispatched.
Is this possible to document?


[2434] - fix(VacancyStatusDeterminationService): Code Climate issue

Context
This PR fixes a Code Climates issue in the VacancyStatusDeterminationService:

"Cognitive Complexity of 10 (exceeds 5 allowed)"

Changes proposed in this pull request

Replaces nested if statement with simplified logic (?? - hopefully it's easier to read now?)
Replaces spec double with factories

Changes proposed in this pull request
Guidance to review
Checklist

[x] Make sure all information from the Trello card is in here
[x] Attach to Trello card
[x] Rebased master
[x] Cleaned commit history
[x] Tested by running locally

looks good, you might want @misaka to look too

Yep no prob 👍

 Fix - Code Climate Issue (VacancyStatusDeterminationService)


Added a example, to check for Odd or Even Number

Pansy program to check for whether a number is odd or even.

@nisnym thank you for your contribution!


Provide aliases for filter/pipe

Fix for #165

This change is

Coverage decreased (-0.2%) to 55.789% when pulling 1edd07ebcd36bc6a89551d578e7b4cd1f9888b51 on aalmiray:pipe-filter-aliases into 2db9cdea3a087f180418ef340d886dfaa8ed5db2 on jdeferred:master.

Decided to bite the bullet and remove the "duplicate" versions of then and leave filter and pipe instead, given that this release already breaks binary compatibility.

Do we need a method like the following one?
	<D_OUT, F_OUT> Promise<D_OUT, F_OUT, P> pipeFail(
	FailPipe<? super F, ? extends D_OUT, ? extends F_OUT, ? extends P> failPipe);

Yes, this is the one. All other PRs can wait post 2.0 IMHO


Dependency has no version

Context: buildSrcVersions is a Gradle plugin that makes it easier to manage your dependencies inside your IDE. It extracts all your dependencies and search for available dependencies updates.
Some plugins like io.vertx.vertx-plugin pin a dependency version. In that case, buildSrcVersions generate the following code
// sample-groovy/build.gradle
plugins {
    id "io.vertx.vertx-plugin" version "0.3.1"
    id 'de.fayard.buildSrcVersions' version '0.4.0'
}
dependencies {
    implementation "io.vertx:vertx-web"
}

// Libs.kt
object Libs {
    const val vertx_core: String = "io.vertx:vertx-core"
}
// Versions.kt
object Versions {
    const val vertx_core: String = "none" // No version. See buildSrcVersions#23
}

What follows is the problem that appears in version 0.3.2 and before in that case

Hello,
I have to say that this plugin is really cool and helpful! :)
However, I have found a few issues related to reporting of release versions, and I couldn't find anyone else posting about it, so I am adding this new issue.
I have a simple Spring Boot, reactive, restful web service, that I created using Spring Initializr a few weeks ago.  I have properly configured my project and the Gradle-Kotin-Dsl-Libs ("GKDL"), and it generates the Libs and Versions files, as expected.  However, the version information for the following items is incorrect, since it does not match the versions that I started with, nor what Spring Initializr generates.  I've noticed that these issues appear in the first and the last two entries in the Libs file that is generated.  Has anyone noticed this issue?  If I can provide any more information, please let me know...
Thank you for your time,
Mike

org_springframework_cloud:
Versions => "2.1.0.RELEASE";
Should be "Greenwich.BUILD-SNAPSHOT";

spring_boot_starter_webflux:
Versions => "none";
Should be "2.1.0.RELEASE";

jackson_module_kotlin:
Versions => "none";
Should be "2.9.7";

VERSION INFORMATION
IntelliJ: 2018.2.6
Java: 11.0.1
Kotlin: 1.3.10
Kotlin DSL: 1.0.4
Gradle: 5.0
Shadow: 4.0.2
Gradle-Kotin-Dsl-Libs: 0.2.6
Spring Boot: 2.1.0.RELEASE
Spring Cloud: 2.1.0.RELEASE, but should be Greenwich.BUILD-SNAPSHOT
Files.zip

Hello @abitofhelp

I hadn't tried the plugin yet with Gradle 5.0, so thanks for telling me that it works 👍
For an immediate workaround of the problem, feel free to modify directly Versions.kt.

object Versions {
-    const val jackson_module_kotlin: String = "none"
+    const val jackson_module_kotlin: String = "2.9.7"
}

Please search the file build/..../report.json
What are the parts that correspond to spring_boot_starter_webflux, org_springframework_cloud, jackson_module_kotlin ?

The above file is generated by the task :dependencyUpdates from https://github.com/ben-manes/gradle-versions-plugin

Your problem may be that you need to configure another repository, which worked for another user. Have a look here:
https://github.com/ben-manes/gradle-versions-plugin/issues/274
Also by default I reject snapshot release informatinon who generates a lot of noise
"2.1.0.RELEASE"  "Greenwich.BUILD-SNAPSHOT";
https://github.com/jmfayard/gradle-kotlin-dsl-libs/blob/master/src/main/kotlin/jmfayard/github/io/GradleKotlinDslLibsPlugin.kt#L17-L27

Using "none" when I have no version is not a great solution from my part. Maybe I should crash instead

object Versions {
-    const val jackson_module_kotlin: String = "none"
+    const val jackson_module_kotlin: String =
+          error("Could not determine Version.jackson_module_kotlin. Please edit Versions.kt manually. See https://github.com/jmfayard/gradle-kotlin-dsl-libs/issues/23")
}

Hi @jmfayard, just to add to this discussion, I have the same issue and it seems to be affected only libraries that have their version set by a plugin, in my case:

org.jetbrains.kotlin:kotlin-stdlib-jdk8 from org.jetbrains.kotlin.jvm"
io.vertx:vertx-core from io.vertx.vertx-plugin
This happens independently from which repositories I have set.

Any chance to have an option to not set the version for libraries that have it to "none"?
By the way, really a cool project, I was hoping that someone would figure it out how to sync ben-manes plugin with gradle buildSrc

Yes that can/will be fixed
Le dim. 30 déc. 2018 à 15:15, Guido Pio Mariotti<EMAIL_ADDRESS>a écrit :

Hi @jmfayard https://github.com/jmfayard, just to add to this
discussion, I have the same issue and it seems to be affected only
libraries that have their version set by a plugin, in my case:

org.jetbrains.kotlin:kotlin-stdlib-jdk8 from
org.jetbrains.kotlin.jvm"
io.vertx:vertx-core from io.vertx.vertx-plugin
This happens independently from which repositories I have set.

Any chance to have an option to not set the version for libraries that
have it to "none"?
By the way, really a cool project, I was hoping that someone would figure
it out how to sync ben-manes plugin with gradle buildSrc
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jmfayard/buildSrcVersions/issues/23#issuecomment-450563522,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAcCyMwv3th4cOBDwR8YnL5_oBQ2D2Fzks5u-MqNgaJpZM4Y2PP0
.

I believe this is fixed