What are the plans regarding support of the failover connector in otelcol-k8s distribution

Hello!
I tried searching, but couldn't find the answer to this question, so apologies if I missed it.
What are the plans regarding including the failover connector from contrib in the otelcol-k8s image?

@dosu what will be included in the next otelcol-k8s release?
Do new components implemented in core and / or contrib get included in future releases as long as they are in alpha?
The failover connector is already implemented in contrib.


27/save button update

@pyphilia, I approved this already, but there were some minor changes here as well.


Formatting breaks yaml anchors/references

Expected behavior
Formatting configs would not break yaml docs that includes anchors and references.
Actual behavior
Running the kpt fn render command on a configuration that includes anchors and references ends up putting the anchors before the references in the yaml, thereby making the yaml invalid.
Information
Steps to reproduce the behavior
kpt pkg get https://github.com/mortent/kpt-packages.git/zookeeper-networkpolicy
kpt fn render zookeeper-networkpolicy  # Succeeds, but reformats the config.
kpt fn render zookeeper-networkpolicy  # Fails

I cannot reproduce this because the package uses a private image? How does the output yaml look like?

Simplified the example to avoid relying on the private images.

cc @phanimarupaka @droot

The issue is with format filter ordering of fields. We should implement additional sorting rules to have anchors before references. What is the priority of this ? m2 or m3 ?

I put this in m4. I don't think using anchors and references are very common with k8s config, so I don't think it is a major issue. But we should keep track of it and see if we can eventually get it fixed.


fix importing PIL error for blender newer versions

cannot import name 'ImageGrab' from 'PIL'

if you go to blender bin folder and install PIL with a terminal or a cmd  import problem will go away but still there are problems
python.exe -m pip install Pillow --upgrade

apparently they changed the python version located in bin folder to 3.9 and there was incompatibility with PIL version but if PIL is piped with blenders own python then everything should work


O(n) nn.Embedding in one kernel without new ops.

Submission for the O(n) nn.Embedding in one kernel without new ops (supporting BS=128) bounty.
The idea is to generate all the indices in the right shape first and then to rely on tensor indexing. This gets rid of matrix multiplications and reduces runtime.
The generated kernels can be seen with this command
DEBUG=9 python > /tmp/tmp.txt <<EOF
from tinygrad.tensor import Tensor
from tinygrad import nn
import numpy as np
batch_size, seqlen, vocab_size, embedding_size = 128, 64, 32, 16
I = Tensor(np.random.randint(0, vocab_size, size=(batch_size, seqlen)))
print(nn.Embedding(vocab_size, embedding_size)(I).numpy())
EOF

Tested: All unit tests pass

Are you sure this is an O(n) Embedding?

Technically if you just do an straight analysis, it isn't strictly better than ~O(n^2) (functions of batchsize, seq len and embed size). It went from the matrix mult to the indexes formation and indexing driving the complexity.
However since the indexes are formed with symbolic operations and the indexing itself is done with those views then the time complexity analysis would depend on the specifics of how these symbolic representations are implemented and executed within the computational graph framework. This does at the very least form one kernel with no new ops.
Was this profiled at all for small and large data?

I profiled it a little on my own machine (apple M1) using the CLANG
backend. METAL was crashing due to another bug.
I have written a micro-benchmark but how can I test how it will perform on
cuda/opencl gpus etc. ? If there is a doc or code (maybe from another
issue?) to set up a micro-benchmark / test harness please let me know.
Also the tests on clang and llvm are failing due to higher memory / larger
runtime but the gpu/cuda/opencl runtimes are fine. Will that make this PR a
non-starter?
On Thu, 22 Feb 2024 at 16:34, davidjanoskyrepo @.***>
wrote:

Technically if you just do an straight analysis, it isn't strictly better
than ~O(n^2) (functions of batchsize, seq len and embed size). It went from
the matrix mult to the indexes formation and indexing driving the
complexity.
However since the indexes are formed with symbolic operations and the
indexing itself is done with those views then the time complexity analysis
would depend on the specifics of how these symbolic representations are
implemented and executed within the computational graph framework. This
does at the very least form one kernel with no new ops.
Was this profiled at all for small and large data?
—
Reply to this email directly, view it on GitHub
https://github.com/tinygrad/tinygrad/pull/3477#issuecomment-1960571584,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALSZFASYKUCP6HE6NOGKHQTYU7PYXAVCNFSM6AAAAABDUYUEYCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRQGU3TCNJYGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

This is the benchmarking code and its results unfortunately show that the new kernel is much slower on my mac but it'll be great if someone could run it on an ubuntu machine with cuda/rocm.
from tinygrad.tensor import Tensor
from tinygrad import nn
import numpy as np
import itertools
import time
def timeit_callable(f):
    time_taken = []
    for trials in range(3):
        start = time.perf_counter_ns()
        f()
        time_taken.append(time.perf_counter_ns() - start)
    return min(time_taken)

for batch_size, seqlen, vocab_size, embedding_size in itertools.product([128,], [64, 128], [32, 256], [128, 256]):
    indices = Tensor(np.random.randint(0, vocab_size, size=(batch_size, seqlen)))
    x = nn.Embedding(vocab_size, embedding_size)(indices).numpy() # warmup
    del x
    time_taken = timeit_callable(lambda : nn.Embedding(vocab_size, embedding_size)(indices).numpy())
    print(batch_size, seqlen, vocab_size, embedding_size, time_taken)

git checkout master
GPU=1 python ../on_embedding_benchmark.py > ../before.txt
git checkout On-embedding
GPU=1 python ../on_embedding_benchmark.py > ../after.txt
paste ../before_gpu.txt <( awk '{print $5}' ../after_gpu.txt ) | awk '{print $0 , $6/$5}'

bs, seqlen, vocab, embedding, before, after, after/before
128 64 32<PHONE_NUMBER>   16956750 3.4825
128 64 32<PHONE_NUMBER>   46456083 9.82669
128 64<PHONE_NUMBER>792  90560500 45.0596
128 64<PHONE_NUMBER>875  354687958 116.22
128 128 32<PHONE_NUMBER>  24103834 4.28754
128 128 32<PHONE_NUMBER>  90950542 11.2119
128 128<PHONE_NUMBER>375 178329500 83.984
128 128<PHONE_NUMBER>792 705470583 183.726

Closing PR because even though O(n) complexity is achieved by explicit indexing the resulting code runs slower than the existing code.


What is the Universal model?

Could you describe a little more what the Universal model is? It's not in the paper or the audio samples. I'm a little confused what it is.

https://github.com/jik876/hifi-gan/issues/46#issuecomment-757593437
The universal model is just a model trained a very large set of data. More speakers = higher quality with unseen speakers.
The goal of the universal model is to have a single vocoder that will work well on almost any speaker without retraining.

@CookiePPP Thank you. Do you mind describing what dataset what used for Universal?

It's LJSpeech + VCTK + LibreTTS, all downsampled to 22Khz see https://github.com/jik876/hifi-gan/issues/1

Thanks, would be great to see that in the README


October 30th, 2023 Community Meeting

Please add a topic in this thread and add a link to the GitHub issue associated with the topic.
Please make sure you give folks enough time to review/discuss the topic offline on GitHub before coming into the meeting
(optional) Paste the image of an animal 😸

In context of https://github.com/shipwright-io/build/issues/1414, How do we expect the build for localsource example?
The v1alpha1 used for local Source is: https://github.com/shipwright-io/build/blob/main/test/data/v1alpha1/build_buildah_cr_local_source_upload.yaml
The URL shared above is a git's URL.
Looking for a little more context to understand how a build with source type as local could be differentiated.


The plugins flutter_facebook_login use a deprecated version of the Android embedding.

Hey there,
First of all, thanks for the package.
I've been testing the package in some personal projects and  today when i clean the build and run the command 'Pub get' i get this warning:
The plugins `flutter_facebook_login` use a deprecated version of the Android embedding.
To avoid unexpected runtime failures, or future build failures, try to see if these plugins support the Android V2 embedding. Otherwise, consider removing them since a future release of Flutter will remove these deprecated APIs.
If you are plugin author, take a look at the docs for migrating the plugin to the V2 embedding: https://flutter.dev/go/android-plugin-migration.

Is this some issue with the package or something related with my project?
Thanks in advance for your time and comments.

Same issue, any solution?

I have the same issue. After updating flutter SDK to 2.5.0

Same issue after upgrading to Flutter 2.5.0 as well.

waiting for fix !

waiting for fix !
:(

Same issue here. Need a fix asap.

Same issue here. waiting for fix :(

Facing the same issue here.

From about a year this plugin didn't get any update which means you have to move to another plugin in case of any error because no one is working on it anymore.
I'm now using this plugin: https://pub.dev/packages/flutter_login_facebook

Same issue is here.

Hello, politeness and respect, I came across this error, what should I do?

How can I use flutter_facebook_login package on Flutter SDK 2.5.1 stable version?
When I used on 2.2.0 stable version, there isn't any issue....

Sent from my MetroPCS 4G LTE Android Device
Get Outlook para Androidhttps://aka.ms/AAb9ysg

From: Hauler @.>
Sent: Friday, October 8, 2021 1:08:50 PM
To: roughike/flutter_facebook_login @.>
Cc: Subscribed @.***>
Subject: Re: [roughike/flutter_facebook_login] The plugins flutter_facebook_login use a deprecated version of the Android embedding. (#325)
How can I use flutter_facebook_login package on Flutter SDK 2.5.1 stable version?
When I used on 2.2.0 stable version, there isn't any issue....
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/roughike/flutter_facebook_login/issues/325#issuecomment-939010010, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVA7IWSGZVRBPD44BM6JNBLUF4XTFANCNFSM5CHOO5DQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Sent from my MetroPCS 4G LTE Android Device
Get Outlook para Androidhttps://aka.ms/AAb9ysg

From: Mayra Ramirez @.>
Sent: Monday, October 11, 2021 11:46:55 PM
To: roughike/flutter_facebook_login @.>
Subject: Re: [roughike/flutter_facebook_login] The plugins flutter_facebook_login use a deprecated version of the Android embedding. (#325)
Sent from my MetroPCS 4G LTE Android Device
Get Outlook para Androidhttps://aka.ms/AAb9ysg

From: Hauler @.>
Sent: Friday, October 8, 2021 1:08:50 PM
To: roughike/flutter_facebook_login @.>
Cc: Subscribed @.***>
Subject: Re: [roughike/flutter_facebook_login] The plugins flutter_facebook_login use a deprecated version of the Android embedding. (#325)
How can I use flutter_facebook_login package on Flutter SDK 2.5.1 stable version?
When I used on 2.2.0 stable version, there isn't any issue....
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/roughike/flutter_facebook_login/issues/325#issuecomment-939010010, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVA7IWSGZVRBPD44BM6JNBLUF4XTFANCNFSM5CHOO5DQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I have the same issue
after updating flutter SDK to 2.5.0

same issue here, waiting for fix.

Sent from my MetroPCS 4G LTE Android Device
Get Outlook para Androidhttps://aka.ms/AAb9ysg

From: Aaqib Khaskheli @.>
Sent: Wednesday, November 24, 2021 6:48:34 AM
To: roughike/flutter_facebook_login @.>
Cc: 1988-8 @.>; Comment @.>
Subject: Re: [roughike/flutter_facebook_login] The plugins flutter_facebook_login use a deprecated version of the Android embedding. (#325)
same issue here, waiting for fix.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/roughike/flutter_facebook_login/issues/325#issuecomment-977847272, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVA7IWQQSRBZ54VNASBWV73UNTNKFANCNFSM5CHOO5DQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I started using flutter_facebook_auth instead of waiting.

Did you find a way to show facebook images in your app?

Did anyone find a way to solve it ?

same issue please update the solution

this plugin looks abandoned

this plugin looks abandoned

same issue facing.

lets get out of here)
just use   flutter_facebook_auth: ^3.5.0
https://firebase.flutter.dev/docs/auth/overview

anybody found solution i m facing similar problem

anybody found solution i m facing similar problem

im using this one https://pub.dev/packages/flutter_facebook_auth

Did anyone find a way to solve it ?

use  flutter_facebook_auth i solved it.

I tried Flutter downgrade but didn't work!


Update README

Closes #668

[x] README
[x] wiki pages

The wiki pages are mostly updated. I guess we could merge this PR now.
I'll still add content to wiki over time.

LGTM
Thank you @renkun-ken !

The updated readme and wiki entries look really nice, thanks a lot!


Implicit analysis (standard process) not terminating correctly under ubuntu 18.04

Hello Kevin, I installed Abaqus 2018 (and also 6.14) on my ubuntu 18.04 machine using the exact same procedure as you posted here. However I ran in the problem that the standard process (for implicit analyses) is not properly terminated after analysis has been completed.
In simple words, whenever I run an implicit analysis, this just works fine until I see in the .sta file that the analysis has been completed. After that, however, the process is not terminated, and RAM is not freed (and in the .log file the analysis completed message never appears) like if it remains in a deadlock state (more details here). This problem can be bypassed by manually kill the process, but is not very comfortable for example when running parametric studies...
Have you experienced the same? If not, did you install abaqus on a fresh installed or on an upgraded ubuntu?
Thanks for your input!

Hi @pevoz thanks for this comment. Yes I can confirm that I have the exact same issue. I have not found a solution yet.
I have Ubuntu 18.04 as well. I recently reinstalled 18.04 on this machine since some things went wrong after an NVIDIA driver upgrade. Not sure how to fix this yet. I mainly use FEBio for the moment but will use Abaqus more intensively in the future. Please let me know if you have any updates on fixing this. I'll do the same.
Thanks.

Thanks for your quick answer. Currently, I'm not using Abaqus a lot as well. I'll surely update you as soon as I find a fix.
By the way, I fixed the link in my previous post, where you can find my post on askubuntu regarding this issue.
Actually, I've found another post similar to yours and asked if they had the same problem. The author of the article replied he has no problems (here the direct link to his reply on Disqus). This sounds however quite strange to me, as meanwhile I found other users having the exact same issue.
When you have time, I suggest you have a look to my askubuntu post. Maybe you have better ideas to find out a fix.
I also tried to contact Abaqus support (even if ubuntu is not supported) and that was their conclusion:

I talked to my colleague. The probably and main cause is that our codes are certified for OS systems based on linux version 3.x (like the supported RedHat/CentOS or SuSE Linux Enterprise) and not with 4.x (which for example Ubuntu 16.04 is using).
Abaqus needs to use some specific kernel routines to stop and clean up the jobs at completion. This functions with kernel versions 3.x for (un)supported OS systems but not with 4.x. In the latter Abaqus should use some other kernel routines to stop jobs correctly.
Compared to A/Explicit A/Standard apparently uses a bit different set of kernel routines at the end, and that A/Explicit is working is just 'lucky'.
As far as we know at the moment, the only workaround is to use an OS based on linux kernel 3.x.

My current guess is that the problem is not related to the kernel (since with ubuntu 16.04.5 and kernel 4.15 standard gets terminated correctly), but rather with some external libraries. But maybe I'm missing something.

The terminating issue has been fixed in Abaqus2020 HotFix 5
(there exist also a HotFix 16 for Abaqus 2018 and a HotFix 11 for Abaqus 2019 that should also fix the bug)
I wrote a bug-report to  the support
I got the reply:
To Submitter	09/15/2020 02:55:17PM

Hallo Johannes,

the issue you reported seems to be related to the following documented bug:

Article ID : BR10000291796
Abstract :  Abaqus/Standard analyses enter an infinite loop if the job attempts to exit with an error on SLES 15 and Redhat 8 Linux systems.
Link :  https://kb.dsxclient.3ds.com/mashup-ui/page/document?q=docid:BR10000291796
"When running on SLES 15 and Redhat 8 Linux systems, Abaqus/Standard analyses enter an infinite loop if the job attempts to exit with an error on SLES 15 and Redhat 8 Linux systems. The infinite loop occurs during shutdown of the standard process execution. The post analysis cleanup processes are not executed.
abaqus -job test terminate does not terminate this job. The Linux kill command must be used to terminate the job.
More specifically, this issue occurs on any Linux system that contains glibc version 2.25 or later. "

This bug is fixed in the latest Abaqus 2020 HF5 (2020 FP.CFA.2038) release, which will be available for download until the end of this week. Until you can install this version, you may use the Abaqus 2018-HF16 or Abaqus 2019-HF10 (or later) releases, which also contain this bug fix.

Please let us know, if the issue is not gone even in the mentioned Abaqus releases.

Best Regards

You can find the local support-addresses at https://www.3ds.com/products-services/simulia/services-support/locations/ or write to support.dsxeco(at)3ds.com
How to fix:
I downloaded Part_SIMULIA_EstPrd - SIMULIA Abaqus 2020 FP.CFA.2038 - Linux64 from  https://software.3ds.com/ and installed Part_SIMULIA_EstPrd.Linux64/1/Software/SIMULIA_EstPrd.HF5.Linux64/1/StartTUI.sh as well as Part_SIMULIA_EstPrd.Linux64/1/Software/CAASIMULIA_EstPrd.HF5.Linux64/1/StartTUI.sh and now it works on Ubuntu 20.04 without singularity-container. (PS don't forget to change to DSY_OS_Release="CentOS" in all Linux.sh-files, as described in http://coquake.eu/wp-content/uploads/2019/02/Abaqus18_on_Ubuntu18.04LTS.pdf )
copied from https://github.com/willfurnass/abaqus-2017-centos-7-singularity/issues/5#issue-713025844


Running videopose.py face error (AttributeError: 'list' object has no attribute 'shape')

Hi there~
Thank you for sharing such great work to us.
It is really helpful for many people.
When I run $ python videopose.py below
It will show an error message which is (AttributeError: 'list' object has no attribute 'shape')
Could you please help me to find out the error
Thank you very much!
=================================================================================================
the video is 25.051 f/s
Loading YOLO model..
/home/xxx/.conda/envs/3d/lib/python3.6/site-packages/torch/nn/modules/upsampling.py:129: UserWarning: nn.Upsample is deprecated. Use nn.functional.interpolate instead.
warnings.warn("nn.{} is deprecated. Use nn.functional.interpolate instead.".format(self.name))
Loading pose model from joints_detectors/Alphapose/models/sppe/duc_se.pth
Start pose estimation...
100%|█████████████████████████████████████████| 338/338 [00:23<00:00, 14.11it/s]
kpts npz save in  outputs/alpha_pose_kunkun_cut/kunkun_cut.npz
-------------- load data spends 27.51 seconds
Loading checkpoint checkpoint/pretrained_h36m_detectron_coco.bin
-------------- load 3D model spends 0.02 seconds
Rendering...
-------------- generate reconstruction 3D data spends 0.15 seconds
===========================> This video get 338 frames in total.
0%|                                                   | 0/338 [00:00<?, ?it/s]/home/xxx/video-to-pose3D/common/visualization.py:181: UserWarning: Tight layout not applied. The left and right margins cannot be made large enough to accommodate all axes decorations.
fig.tight_layout()
0%|▏                                          | 1/338 [00:00<00:43,  7.82it/s]outputs/kunkun_cut.mp4 --- elapsed time: 28.11254884302616 s
Traceback (most recent call last):
File "videopose.py", line 164, in
inference_video('outputs/kunkun_cut.mp4', 'alpha_pose')
File "videopose.py", line 160, in inference_video
main(args)
File "videopose.py", line 133, in main
input_video_skip=args.viz_skip)
File "/home/xxx/video-to-pose3D/common/visualization.py", line 187, in render_animation
anim.save(output, writer=writer)
File "/home/xxx/.conda/envs/3d/lib/python3.6/site-packages/matplotlib/animation.py", line 1133, in save
anim._draw_next_frame(d, blit=False)
File "/home/xxx/.conda/envs/3d/lib/python3.6/site-packages/matplotlib/animation.py", line 1168, in _draw_next_frame
self._draw_frame(framedata)
File "/home/xxx/.conda/envs/3d/lib/python3.6/site-packages/matplotlib/animation.py", line 1718, in _draw_frame
self._drawn_artists = self._func(framedata, *self._args)
File "/home/xxx/video-to-pose3D/common/visualization.py", line 175, in update_video
lines_3d[n][j - 1][0].set_3d_properties([pos[j, 2], pos[j_parent, 2]], zdir='z')
File "/home/xxx/.conda/envs/3d/lib/python3.6/site-packages/mpl_toolkits/mplot3d/art3d.py", line 143, in set_3d_properties
zs = np.broadcast_to(zs, xs.shape)
AttributeError: 'list' object has no attribute 'shape'
0%|▏                                          | 1/338 [00:00<00:56,  5.99it/s]

It may be caused by the incompatibility of matplotlib. I'm using Version 3.1.0, which may help to address this issue. Sorry of late response.

change 'xs' to 'xs_ = np.array(xs)'


HTTP Patcher leaks response & stalls segments if no other response listener is supplied

Ran into this while testing the examples I was providing in #11. Not sure how much real-world impact this has, but the http patcher adds a response listener to outgoing http requests, and then sets up a listener for res.on('end', ...) in order to close the subsegment. The problem is, once you do add a response listener, then the response must be consumed.
From the http documentation:

If no 'response' handler is added, then the response will be entirely discarded. However, if a 'response' event handler is added, then the data from the response object must be consumed, either by calling response.read() whenever there is a 'readable' event, or by adding a 'data' handler, or by calling the .resume() method. Until the data is consumed, the 'end' event will not fire. Also, until the data is read it will consume memory that can eventually lead to a 'process out of memory' error.

Currently if the developer didn't add a listener of their own, the response is never consumed, meaning the
response data is leaked, and since end is never fired, the subsegment will never close meaning it and its parent (sub)segment will never be flushed and sent to the daemon.
Reason I'm not sure how much real-world impact this would have, is I'm not sure how often you would simply not care about the response, but I'm sure there's some sort of fire-and-forget systems out there...?
The following should fix it, but I've not developed any tests for it. It consumes the body if a callback wasn't provided, and if x-ray is the only response listener. Meaning neither of the two scenarios below occurred:
http.request('http://example.com', res => res.resume()).end();
http.request('http://example.com').on('response', res => res.resume()).end();

diff --git a/packages/core/lib/patchers/http_p.js b/packages/core/lib/patchers/http_p.js
index 5c0cb68..f5311ef 100644
--- a/packages/core/lib/patchers/http_p.js
+++ b/packages/core/lib/patchers/http_p.js
@@ -137,6 +137,8 @@ function enableCapture(module, downstreamXRayEnabled) {
         } else {
           callback(res);
         }
+      } else if (req.listenerCount('response') === 1) {
+        res.resume();
       }
     }).on('error', errorCapturer);

Of course, if the developer does add their own listener and neglects to consume the response, then you still run into the segment never closing, but that's a different issue, with developer error mixed in.

Fix rolled out with 1.2.0 release.

I know this is an old issue but I just wondered about this fix  done that time to check if there is just one end listener and if yes call res.resume().
Looking into the NodeJs source code it seems that above check will never be true as one end listener is installed by node and a second one by http_p.js.

Hi @Flarna,
Yeah, it looks like you are correct. As we discussed in #318, it is impossible to tell whether you are the "only" listener for a given event. However in this case I'm not even sure the fix was necessary, since the quoted part of the Node docs refers to consuming the response, e.g. with a listener to the data event. However we are listening to the end event, so would you agree that this piece is not needed whatsoever? Because there is no risk of data leaking since the end event has necessarily already been emitted, so it will be cleared after the listeners finish with it right?

The issue is for the response event, not the end event or the data event.
As @Flarna has pointed out, it seems like the fix that landed was incorrect.

If there is no response listener NodeJs core will consume (dump) the data. If there is a listener it is the user application which has to do this. Either by installing a 'data' event per hand, implicit by piping into some stream or by calling resume().
If the user application is not passing a callback this module will add one and as a result it is responsible to consume the data otherwise connections may hang.
Please note that there are two ways to install a ' response' listener:

pass a callback as last argument to get/request
installed it via EventEmitter APIs on the return value of get/request

@omsmith @Flarna so it seems like the initially proposed fix is correct. My only question is if we are not the only listener (e.g. req.listenerCount('response') > 1) , should we remove our listener so that if other libraries are making the same check they would see themselves as the only listener? Or would that break the X-Ray SDK's ability to trace keep-alive connections connections?

It's slightly different (and easier) here compared to the error event case.
If there are more http monitoring tools in the same app all of them will wrap http.get and http.request. As a result the call sequence is User Code => Monitoring Tool 1 => Monitoring Tool 2 => HTTP.
Consider user is not passing a callback and aws-xray-sdk is Monitoring Tool 1. It will add a (dumping) callback and Monitoring Tool 2 thinks users code uses a callback and therefore no need to dump the data - which is correct. If it is the other way around it works the same.

I see! Thank you for the explanation. I will include this fix along with that in #318.

@Flarna @omsmith I attempted this fix, however whenever I tried to check req.listenerCount('response') it always returned 0. This was in spite of having two callbacks - one added by the client application making the HTTP request and one added by this SDK's HTTP patcher. I even checked the req._events field and it showed that the only events being listened to on the request was error and preFinish, which is only used internally by Node. Am I missing something?

I did a fast try and http.get("http://localhost/", (res) => {}).listenerCount("response") returns 1 for me.


Rebuild glib-split for 2.64.6

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

This is basically applying the same split that's now on master, right?

Yes!


Statsig access and erasure

Closes #CON-203
Description Of Changes
Adding a new erasure connector for Statsig
Code Changes

[x] Added required files for the addition of an Erasure only connector for Statsig

Steps to Confirm

[X] Ensure we can now add this new connector to a system from the Fides interface

Pre-Merge Checklist

[ ] All CI Pipelines Succeeded
Documentation:

[X] documentation complete, PR opened in fidesdocs
https://github.com/ethyca/fidesdocs/pull/273
[x] documentation issue created in fidesdocs
https://github.com/ethyca/fidesdocs/issues/271

[x] Issue Requirements are Met
[X] Relevant Follow-Up Issues Created
[x] Update CHANGELOG.md
[x] For API changes, the Postman collection has been updated

This looks great to me! Do notice I did slip a 'fix' to a doc issue I found while working on this connector. Also, fwiw, I don't think I'm able to be added as a reviewer 😮.


android.content.res.Resources$NotFoundException when google run the app on pre launch report

:beetle: Description
Getting android.content.res.Resources$NotFoundException on submitting App to playstore
:beetle: Code:
import Sound from "react-native-sound"

/**
 * SoundPlayer will be the controller to handle app sounds and ringtones
 */
export class SoundPlayer implements IService {
  private errorSound: Sound
  private successSound: Sound

  @autobind
  public error(): void {
    this.errorSound.play()
  }

  public async setup(): Promise<void> {
    this.successSound = new Sound(require("./assets/popcorn.mp3"), noop)
    this.errorSound = new Sound(require("./assets/nfc_failure.mp3"), noop)
  }

  public setupForTests(successSoundMock?: any, errorSoundMock?: any): void {
    this.successSound = successSoundMock || mock
    this.errorSound = errorSoundMock || mock
  }

  @autobind
  public success(): void {
    this.successSound.play()
  }
}

Is your issue with...

[ ] iOS
[x] Android
[ ] Windows

Are you using...

[x] React Native CLI (e.g. react-native run-android)
[ ] Expo
[ ] Other: (please specify)

Which versions are you using?

React Native Sound: 0.11.0
React Native: 0.60.4
Android: Android 7.0

Does the problem occur on...

[x] Device

If your problem is happening on a device, which device?

Device:

Nom du modèle
Moto Z
Mémoire RAM
3072 Mo
Fabricant
Motorola
Version OpenGL ES
3.2
Version d'Android
Android 7.0
Plate-forme native
armeabi-v7a
Paramètres régionaux
fr
Marque du processeur
Qualcomm
Modèle du processeur
MSM8996

Stacktrace:
FATAL EXCEPTION: mqt_native_modules
Process: com.exorun, PID: 10341
android.content.res.Resources$NotFoundException: File res/raw/app_services_soundplayer_assets_popcorn.mp3 from drawable resource ID #0x7f0b0001
	at android.content.res.ResourcesImpl.openRawResourceFd(ResourcesImpl.java:291)
	at android.content.res.Resources.openRawResourceFd(Resources.java:1263)
	at com.zmxv.RNSound.RNSoundModule.createMediaPlayer(RNSoundModule.java:161)
	at com.zmxv.RNSound.RNSoundModule.prepare(RNSoundModule.java:61)
	at java.lang.reflect.Method.invoke(Native Method)
	at com.facebook.react.bridge.JavaMethodWrapper.invoke(JavaMethodWrapper.java:372)
	at com.facebook.react.bridge.JavaModuleWrapper.invoke(JavaModuleWrapper.java:158)
	at com.facebook.react.bridge.queue.NativeRunnable.run(Native Method)
	at android.os.Handler.handleCallback(Handler.java:751)
	at android.os.Handler.dispatchMessage(Handler.java:95)
	at com.facebook.react.bridge.queue.MessageQueueThreadHandler.dispatchMessage(MessageQueueThreadHandler.java:29)
	at android.os.Looper.loop(Looper.java:154)
	at com.facebook.react.bridge.queue.MessageQueueThreadImpl$4.run(MessageQueueThreadImpl.java:232)
	at java.lang.Thread.run(Thread.java:761)
Caused by: java.io.FileNotFoundException: This file can not be opened as a file descriptor; it is probably compressed
	at android.content.res.AssetManager.openNonAssetFdNative(Native Method)
	at android.content.res.AssetManager.openNonAssetFd(AssetManager.java:460)
	at android.content.res.ResourcesImpl.openRawResourceFd(ResourcesImpl.java:289)

Also the javascript code above is working on a simulator.

Caused by: java.io.FileNotFoundException: This file can not be opened as a file descriptor; it is probably compressed

This is interesting ^^
Have you tried with a different .mp3 file?

@paulmelnikow thanks for reply, I have not tried but the code runs in production version of the application.
Also it works in a simulator with a debug build.

Maybe, as i'm on androidX, before migrating to androidX the problem was not there.

try turning off the resigning of the apk in the pre launch settings, see if that helps

@ajungg where?

maybe what he means is this
release management --> pre-launch report --> go to settings --> Pre-launch report version
please let me know the result, I also experienced the same things :(

Hello,
I have the same error since the 22th of this month. Everything works fine with a production build on every phone I tried but every device of the pre-launch report crashs with this stacktrace :
FATAL EXCEPTION: mqt_native_modules
Process: com
android.content.res.Resources$NotFoundException: File res/raw/question.mp3 from drawable resource ID #0x7f080002
	at android.content.res.Resources.openRawResourceFd(Resources.java:1315)
	at com.zmxv.RNSound.RNSoundModule.createMediaPlayer(RNSoundModule.java:153)
	at com.zmxv.RNSound.RNSoundModule.prepare(RNSoundModule.java:57)
	at java.lang.reflect.Method.invoke(Native Method)
	at com.facebook.react.bridge.JavaMethodWrapper.invoke(JavaMethodWrapper.java:372)
	at com.facebook.react.bridge.JavaModuleWrapper.invoke(JavaModuleWrapper.java:160)
	at com.facebook.react.bridge.queue.NativeRunnable.run(Native Method)
	at android.os.Handler.handleCallback(Handler.java:746)
	at android.os.Handler.dispatchMessage(Handler.java:95)
	at com.facebook.react.bridge.queue.MessageQueueThreadHandler.dispatchMessage(MessageQueueThreadHandler.java:29)
	at android.os.Looper.loop(Looper.java:148)
	at com.facebook.react.bridge.queue.MessageQueueThreadImpl$3.run(MessageQueueThreadImpl.java:192)
	at java.lang.Thread.run(Thread.java:818)
Caused by: java.io.FileNotFoundException: This file can not be opened as a file descriptor; it is probably compressed
	at android.content.res.AssetManager.openNonAssetFdNative(Native Method)
	at android.content.res.AssetManager.openNonAssetFd(AssetManager.java:426)
	at android.content.res.Resources.openRawResourceFd(Resources.java:1312)
	... 12 more

The resigning of the apk is off and it still doesn't work.
I'm not using androidX (yet) and my react-native-sound package is not up to date.
These are my versions :
"react-native": "0.57.7",
"react-native-sound": "^0.10.9",
If I understand correctly I can still deploy my build and see if it affects my users but I'm a bit scared.

I tried this but not sure this is a good solution enough
I added this in my proguard rule
-keep class *.R
-keepclasseswithmembers class **.R$* {
    public static <fields>;
}

maybe see if that helps

In my case for my production builds i'm not using proguard.

If I understand correctly I can still deploy my build and see if it affects my users but I'm a bit scared.

@nromptea42 you can try a production build without submitting it to your external users.

I'm not using proguard either.
@loic-lopez I downloaded, installed and launched the apk on multiple phones and it works fine. Maybe I will deploy to a certain percentage of external users to see how it goes.

maybe what he means is this
release management --> pre-launch report --> go to settings --> Pre-launch report version
please let me know the result, I also experienced the same things :(

Thats exactly what i meant :)
That option turns off the resigning of the APK and disables some features of the Robo Test. I had similar issues since two days and that helped. Since it worked before it seems to be a temporary issue with the Robo test.

After digging a lot I made my binary x64 compliant see: developer.android.com/distribute/best-practices/develop/64-bit#building_with_android_studio_or_gradle
And submitted an AAB bundle the error does not occur anymore.
How to generate an AAB bundle:
cd android && ./gradlew bundleRelease

I will update this issue if I have more information.


Frames properties in landscape-mode aren't calculated correctly

Today (after almost doubting myself why my code won't work) I figured out that the frame properties for positioning and size aren't calculated correctly, when the page is set to landscape-mode. The whole frame still seems to use the page in portrait-mode to calculate it's position and size.
Basically only the "right" and "bottom" properties of the frame are affected, as xhtml2pdf uses left and top as default origin for (x,y).
A minimalistic example to reproduce the bug is following:
<!DOCTYPE html>
<html>
<head>
    <style>
            @page {
                size: a4 landscape;
                @frame content_frame {
                    left: 2cm;
                    right: 2cm;
                    bottom: 2cm;
                    top: 2cm;
                    -pdf-frame-border: 1
                }
            }
    </style>
</head>
<body>
Some Text for the content frame.
</body>
</html>

As can be seen here:
https://imgur.com/a/xjfAvhP
the frame isn't stopping at "right: 2cm" of the landscape-mode, but of the portrait-mode. The same happens to the "bottom" property. It's still calculated relative to the portrait-mode and therefore the content frame is cut off at the bottom.
By executing this following code, where only the page size is changed from "landscape" to "portrait", it can be seen, that everything just works fine in portrait-mode:
<!DOCTYPE html>
<html>
<head>
    <style>
            @page {
                size: a4 portrait;
                @frame content_frame {
                    left: 2cm;
                    right: 2cm;
                    bottom: 2cm;
                    top: 2cm;
                    -pdf-frame-border: 1
                }
            }
    </style>
</head>

<body>
Some Text for the content frame.
</body>
</html>

A picture can be found here:
https://imgur.com/t8yFjxD
I was able to reproduce the issue in python as well as by using the standalone executable xhtml2pdf.exe file.
As for as I'm concerned it's only affecting content-frames and NOT static-frames.
As this is my first reported issue on GitHub ever and as I'm pretty new to GitHub myself, I hope that I've fulfilled all the necessary steps to report this issue and to make it reproducible. If you need any further information, just let me know.
Thanks a lot to the developers for creating such a great package! I've really enjoyed it so far.
fbernhart

I've been able to get some more information about the issue, that might help solving it:
Somehow it seems that the calculation of the frame properties depend on the orientation of the page defined before the page of the concerned content-frame.
To reproduce:
<!DOCTYPE html>
<html>
<head>
    <style>

            @page dummy_landscape {
               size: a4 landscape; margin: 2cm;
            }

            @page {
                size: a4 landscape;
                @frame content_frame {
                    left: 2cm;
                    right: 2cm;
                    bottom: 2cm;
                    top: 2cm;
                    -pdf-frame-border: 1
                }
            }
    </style>
</head>

<body>
Some Text for the content frame.
</body>
</html>

Another page (called "dummy_landscape", with orientation landscape) is added before the actual page. It now seems like the content-frame in the page is being calculated by the orientation of the page "dummy_landscape". As "dummy_landscape" already has the orientation landscape, the frame size/position is correctly calculated.
Note, that the page "dummy_landscape" is actually not used in the  < body > part.
I hope that this might help.


First version TODO

It compiles and connects and does things, but it's not quite there yet.
First pass missing:

[x] meta response API
[ ] meta response flag parser helper API
[x] completion of buffer_consume() so more than one request/response will work :)
[x] stats call response parsing
[x] complete ASCII get handling (needs to watch for END)
[x] NONBLOCK mode. Most of the WANT_READ/WANT_WRITE codes are in place already.

Missing API:

[ ] various connect wrappers for unix sockets, local TCP binding, etc.
[ ] UDP support (unclear if/when I'll add this)

Missing infrastructure:

[ ] autotools for building examples?
[ ] can this just be a header file? (need private structs/functions?)
[ ] basic testing infrastructure. I'd prefer a header that gets included if building in test/debug mode. Then it can bash on internal-only functions.
[ ] example "simple C client"
[ ] ifdef cpp wrapper bits for C++ users.

Decisions:

[x] should mcmc_read_value() use iovec's instead of a loop?
... should add an explicit function for iovecs.
[x] should mcmc_read_value() attempt to read the trailing \r\n into the stack and avoid returning it to the user?
... decided to return the value with the \r\n to the user.
[ ] should this client support non-meta commands at all? (would put the minimum server version at 1.6, roughly)
[ ] how/should we support setting/system commands? metadump/watch? (leaning towards no)

Next major work:

[ ] TLS support. How much does this change the API? can it ifdef and/or be a different set of calls?
[ ] example "selector object" API and code. ie: crc32/ketama selection.
[ ] full "simple" C client?
[ ] iovec interface for reading values.

Reworked the response parsing to use a union struct instead of many function calls. Also removes a lot of pointer checks and copying internally.

been poking at this slowly as I was on other things for quite a while. did most of the nonblock stuff and a few cleanups a while ago.
looks like I need a sendmsg() alt for mcmc_send_request(), so that's next :)

closing this; See #2 instead
originally I had this as one file handling the protocol and all the higher level tcp networking stuff.  We should end up with at least two, with the protocol handling being the very low level part and able to be separated from the rest of it.


Console - Route arbitrary command line arguments (and options)

I may be missing something, but console routing seems to take it's job too literally and too far.  It attempts to do options (getOpt-style) processing and argument assignment as well as its real job of actually targeting a Controller method.  It should instead (at least optionally) parse only as much as needed to select the method and allow the remaining command-line parsing to be delegated.
The simplest example is illustrative:  how would you implement a route for "echo"?
php public/index.php zf echo The Number of Arguments Is Abitrary.

It also has implications for more complex applications.  Consider git.  When you invoke certain commands like git checkout -b newbranch you are also able to supply options that are not checkout options: they are passed to git branch.  Checkout should not care about them, and it would be complex and burdensome for checkout to have to reincorporate that part of branch's routing.  Similarly, very many git commands take options/args that are filtered, added to, amended via configuration and ultimately passed to git log or git diff.
Last example: a given command might support separate totally different options/args processing syntaxes, like ps.  To allow that, routing would have to do enough to recognize the Controller/method and then get out of the way.
This issue could be solved with #5995, but also could be ameliorated by lesser solutions (basic wildcard or special character).  Let me know if I'm overlooking anything.

This issue has been closed as part of the bug migration program as outlined here - http://framework.zend.com/blog/2016-04-11-issue-closures.html

See https://github.com/zendframework/zend-console/pull/32


Remove kernel builder pattern for Python: sk.kernel_create() -> sk.Kernel()

Motivation and Context
This PR makes creating a kernel instance more pythonic. The builder pattern is removed and the kernel is constructed using Kernel(...).
Work originally started with #137
Description

Removed kernel_builder.py.
Instance creation pattern kernel = sk.create_kernel() -> kernel = sk.Kernel()
For notebook 6, there is no need to create a separate kernel for the GitHub files. Changed to use a separate collection.

Contribution Checklist

[x] The code builds clean without any errors or warnings
[x] The PR follows SK Contribution Guidelines (https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md)
[x] The code follows the .NET coding conventions (https://learn.microsoft.com/dotnet/csharp/fundamentals/coding-style/coding-conventions) verified with dotnet format
[x] All unit tests pass, and I have added new tests where possible
[ ] I didn't break anyone :smile:

@mkarle I'll merge after your ✅


Question: how can I change the text color for the top Cancel and Done buttons?

Nothing worked but this. I aswered it on other question. Pasting same here as it took me long to figure-out.
NSDictionary *attributes = [NSDictionary dictionaryWithObjectsAndKeys:
                                [UIColor redColor],NSForegroundColorAttributeName,
                                nil];
    [[UIBarButtonItem appearanceWhenContainedIn:[UINavigationBar class], nil]
     setTitleTextAttributes:attributes
     forState:UIControlStateNormal];


WoCCatalog [ARM|Release] Shadow page is scaling the 6 elements incorrectly.

Phone: Lumia 950 device (14995 build)
Master commit: 4804ed9
Build/run WOCCatalog in Release.
Tap "Shadow"
Observe 6 shapes
Expected:
All 6 shapes appear in the full view
Actual:
In portrait mode the circle and oval are cut off on the right

@bebauer-msft is this a regression?

@rajsesh-msft Yes. Things also get really interesting when you rotate the device. Scaling is off and arrow images move around.

@rajsesh-msft Yes this is a regression. It used to all fit on screen upon initially opening the page.

@bebauer-msft, can you confirm with which build did the regression appear?
Thanks.

These elements can be scrolled into view.

@tadam-msft I've seen this on a Surface Book too, and in a different application (CGCatalog). I believe there is something wrong with our layer scaling code.
In light of that, it is worth investigating further.

Can you share repro steps, @DHowett-MSFT.  I don't think the WoCCatalog sample was supposed to be scaling.

@jaredhms
It looks like the issue in this bug is different from what I'm seeing.
This bug is simply that WOCCatalog's shadow page is hardcoded to render to a width of 600 points.
The bug I am seeing is that a view with a drawRect: displayed on a non-1.0-scale device is rendering at 1.5x width but 1.0x height, but only above certain widths. (???)

I wrote my issue up in #2084.


Go To Declaration/Go To Definition context menu entries weirdly broken, often show Go To References results instead

The Go To Declaration/Go To Definition context menu entries weirdly broken for me, and often show Go To References results instead. It is quite random and seems to be really just depend on whether VS Code thinks the line number is weird or not, or what do I know. Since the entirely wrong UI (the references section with entries) pops up in the first place I assume this is a VS Code and not an LSP bug, since I would expect VS Code to do nothing if worst case the LSP really sent the completely wrong response to the given request and not comply with indeed fully unrelated actions. For what it's worth, the LSP I'm seeing this with is the clangd diagnostics plugin.

 You can close them and retry however often you want, VS Code is now stuck and insists to show you completely useless references instead of carry out your requested action. Restarting VS Code will usually fix it.
I have also noticed Go To Declaration/Go To Definition will sometimes just do nothing at all if the target file isn't among the open tabs right after restarting VS Code (if the target file was opened previously then it seems to work even when the tab is closed again). Since the clangd LSP plugin supposedly supports background indexing, is there a bug in VS Code's LSP handling in that it doesn't maybe send the unopened files as part of the workspace? Whether this also causes the weird references mixup I made this ticket for I'm not sure, just mentioning it in case this is possibly related to all this misbehavior.

Does this issue occur when all extensions are disabled?: no idea, didn't know how to try when that removes the LSP backend to make these menu entries do anything at all

The can only be caused by your extension. To populate the context menu VS Code doesn't do anything fancy, the only check is if for the given the current document an extension has registered a provider for "go to xyz" results


Allow for overlapping polygons

Currently, overlapping polygons cannot be handled. In my experience, they often yield paths that go through polygons that overlap, rather than around.
At https://github.com/georghess/extremitypathfinder/tree/overlapping-polygons a potential fix addresses this when preparing the visibility graph. Basically, extremities that are within a polygon or outside the boundary should not be considered visible:
while len(extremities_to_check) > 0:
    # extremities are always visible to each other (bi-directional relation -> undirected graph)
    #  -> do not check extremities which have been checked already
    #  (would only give the same result when algorithms are correct)
    # the extremity itself must not be checked when looking for visible neighbours
    query_extremity: PolygonVertex = extremities_to_check.pop()
    # only extremities that are within the map are visible to other extremities
    if not self.within_map(query_extremity.coordinates):
        continue

Further, neighbours of query_extremity should not automatically be considered visible, since overlapping polygons might obstruct the view, e.g. just remove
try:
    candidate_extremities.remove(n1)
    visible_vertices.add((n1, n1.get_distance_to_origin()))
except KeyError:
    pass
try:
    candidate_extremities.remove(n2)
    visible_vertices.add((n2, n2.get_distance_to_origin()))
except KeyError:
    pass

Adjusted code passes all tests and produces feasible paths for overlapping polygons as far as I've seen.

Thanks for the input, could you please open up a pull request with the changes you have made?
It would also be very helpful if you added some new test cases for overlapping polygons.
Any contributions are welcome!


Normalize the package names in _pipfile

Remove the dict() so that the mutation will affect pfile, parsed_pipfile will keep unchanged until the file is overwritten.
This fixes some issues when packages names don't comply with PEP423, see the tests.
And also a issue of contoml found when debugging, which can be triggered when trying to print parsed_pipfile: items returns a dictitems object and hence can't be connected with a list. The issue about items doesn't relate to the normalization, I put this in its own commit.

Er you’re gonna have to describe the issue with contoml specifically

Also note that we mock all of pypi so if you’re installing a package we don’t have mocked you need to add it everywhere it needs to be mocked. So you should use a package we have mocked

@techalchemy I don't quite understand

items returns a dictitems object and hence can't be connected with a list. The issue about items doesn't relate to the normalization, I put this in its own commit.
What packages are mocked? I picked one from PYPI_VENDOR_DIR, am I getting wrong?

items returns a dictitems object and hence can't be connected with a list. The issue about items doesn't relate to the normalization, I put this in its own commit.

I asked you to describe what the bug is with contoml.  This doesn't sound like a bug in contoml, it sounds like a problem with the approach you are taking to normalization.

It is triggered when trying to print the contoml parsed object, which will call object.items() inside. The bug(illegal concatenating) lies there.
Please take a look at the diff

@techalchemy I thought the patched directory was to host vendor libraries with some customizations, otherwise they will go to vendor dir. So the bug fixing about the library isn't included?
If that is true, then I agree with your workaround. The fix of items won't affect tests but only debugging so just leave it.

The issue can be reproduced in this way:

Write some_package into Pipfile and do pipenv install
Do pipenv install some-package. pipenv thinks it is not included in Pipfile and add a duplicate line some-package into it.

But also it was an obvious coding fault: mutate a dict without ref to the original object. I don't see why this should be rejected.

@frostming please have patience. I was not writing a summary for your benefit but for the benefit of others when I point them to review. I already agreed with you that this is a bug.

But also it was an obvious coding fault: mutate a dict without ref to the original object. I don't see why this should be rejected.

I don't have any plans to reject this --  I only mean that we have to consider how fixing this bug might impact people who are using pipenv in production environments.  This is a large project in use by tens- to hundreds- of thousands of systems.  This change alters the representation of package names in existing pipfiles, even if it alters it to be correct. That might have unforeseen consequences, which would impact whether this gets merged as part of a major release or not.

I had some time to look into this.  Broadly, this behavior is the goal:

Normalize package names with pep423 etc when we write them to the pipfile (e.g. someone calls pipenv install from the command line)
Normalize all package names written to the lockfile
Return the normalized name for comparison against other packages in the index
Don't normalize anything we didn't write -- if someone manually writes a lockfile, leave the names alone including capitalization etc

This isn't a criticism of your PR or your code, I'm not sure it necessarily applies. I am laying out the needs of this part of the code so that we can work with some clarity -- thanks for your patience and continued effort around this, I know this section of our code is not that pretty and we really do appreciate the help

@techalchemy Sorry for the messing with TABs, fixed
With the fix of this PR, package names will be normalized when pipfile is written(install, uninstall). Moreover, all names are PEP423ed and recased when imported from a requirements.txt
@jtratner This case doesn't apply to this PR. But your proposal is perhaps better and there is another discussion for it https://github.com/pypa/pipenv/issues/1855#issuecomment-376402240
Thanks all

@frostming - I'm just concerned that popping out all of the keys will remove comments, so would be great if you could update your test case to confirm that comments are not lost.

@jtratner Please review

nice! This actually makes me realize another case (which wouldn't have to preserve the comments) - and I'm not sure whether it's covered in test cases or what current behavior is.
python_dateutil = "==1.5"

then you say pipenv install python_dateutil
should this change to?
python_dateutil = "*"

or stay the same?
Similarly, if you specify a version, should the pipfile be updated?
(@techalchemy - thoughts?)

IMO "*" shouldn't overwrite. But another version specifier ">=", "<", "==" should overwrite that in Pipfile.

The easy solution is to just rewrite the line without the comment, imo, but I dont know how hard it is to preserve the comment and just change the version

In contoml, you can change the value of a key while keeping the inline comment.
And if you delete the key, the inline comment will remain at the same line to become a line comment.

@techalchemy @jtratner I refactored the recasing/normalization part a bit so that package names will be normalized only when user does pipenv install
Also fixes #1855 and covers it in test suites

Oops, multiprocess is broken, needs more coding

I don’t think the multiprocessing issue is your fault — our tests are behaving strangely right now

ok sorry please update with master and we can resume actually working on stuff

At a glance this looks okay to me, thanks a bunch for taking care of this! I would like another set of eyes just to verify this too before I merge so @uranusjr @ncoghlan if one of you guys wouldn't mind

The normalisation part looks good to me. I don’t quite understand the caching part and won’t comment.
This and my previous script entry change makes me think we’d probably remove direct access to parsed_pipfile and wrap operations with functions instead. But that can be covered in a later refactor. This PR is good as-is.

@ncoghlan Thanks for your inputs of great help!

No need to reconvert package_name on every iteration - it can be converted once before the loop.

Fixed

The simplest fix I can see would be to just leave the denormalised name alone in the case where it's already present - the change to normalise keys on lookup in get_package_name_in_pipfile will handle such cases.

Agree, see my latest commit.

Another couple of test cases are needed to check that:

the version constraint is preserved for the python_DateUtil = "==1.5" case if no version constraint is given on the command line

Changed the tests a bit

the version constraint is replaced if an explicit version constraint is given on the command line (including resetting it to a wildcard dependency with pipenv install 'python_DateUtil=*')

Tests added. install package=* is neither supported by pip nor pipenv, we can improve this later. I don't do it in this PR.

@frostming I think as long as using an actual pin (e.g. package==x.y) replaces the correct entry it is probably good. I know this wound up being a complicated pr so thank you for your patience here, this actually has the potential to break a lot of stuff, but some of that is currently broken, so disentangling what behaviors we actually want was difficult. I’ll give it another look when I’m off mobile but otherwise I think we might be good!

Final update to make sure we aren't colliding with other changes and then merging, this looks good.

Wooo!
On Tue, Apr 10, 2018 at 6:16 PM Dan Ryan<EMAIL_ADDRESS>wrote:

@techalchemy approved this pull request.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pypa/pipenv/pull/1826#pullrequestreview-111061268,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABhjq7K7Nweu5k-EdgbwrJLzcro87nIzks5tnVl_gaJpZM4S2_qo
.


Field selector for Data extraction

The initial version extract the 'message' field, the next step is to allow user via a check box system to choose which field(s) they want to extract.

Added field selection in the new version.


Version 1.8.1 (August 2017)

Awesome update and thanks for jumping on this so quick! What are you using for a wallet / storage ? Still trying to find something that works good.
Mike.

Bitcoin ABC Wallet - https://download.bitcoinabc.org/0.14.6/
Bitcoin Unlimited Cash Release - https://www.bitcoinunlimited.info/download

Why do you close these so fast, lol.
How are you feelings right now with bitcoin cash - been 24 hours on one of transactions and only have 13 confirmations. Seems like there isn't enough hashing.


trailing comma for {. foo: bar,} causes syntax error

collected in https://github.com/facebook/reason/issues/1549


disable padding check for llc supervisory packets

the llc padding check causes an exception when android or iOS devices are used (dhcp).
I tried to connect an android and iOS device to an access point which is ovs enabled. The ovs is connected to ryu.
When the UE is trying to attach it generates an LLC message (Supervisory frame). In this frame the padding bits (5-8) are set to 1. The IEEE 802.2 (https://standards.ieee.org/getieee802/download/802.2-1998.pdf page 46) describes that it should be set to 0.
I understand that the IEEE specification is the "law", but every first time I connect those devices ryu is causing an exception.
iOS device:

Android device:

Thanks!
I modified the patch to just remove the line:
https://github.com/osrg/ryu/commit/fd348e71aee83f0a95dc8d7e1cae64b257a16e06


refactor environment tf

We currently have terraform[-*]/environment.tf to configure resources that unavoidably differ between environments, but in fact none of the resources do unavoidably differ, so we can refactor the resources back into the main config.
The resources in environment.tf configure DNS for a subdomain that GOV.UK created to point to our production environment.  It doesn't matter if we also configure DNS for subdomains that don't exist yet, that would point to our other environments if they did exist.  There's a negligible cost of 20p per month per environment.

[x] Configure plausible gov-search domain in tfvars
[x] Make environment.tf the same in every environment
[x] Move the contents of environment.tf into govgraphsearch.tf.

Closes #493

This is now deployed in every environment.  It had to be, to check that it would work!


in later container , the point in the tag, if i Press 'ctrl' , the point will jump top

I in later container , the point in the tag, if i Press 'ctrl'，  the point will jump top　don't know if it's because I have a problem with my computer or I'm running a lot of tasks, and I downloaded the less advanced version.

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Urls with IP addresses

Parsing an URL with a IP address is causing invalid results.
subdomain, registerableDomain and publicSuffic should be false.
vago@hp:/tmp/test$ ./vendor/bin/parse http://<IP_ADDRESS>/whois-archive/latest.php?page=2479
Array
(
[scheme] => http
[user] =>
[pass] =>
[host] => <IP_ADDRESS>
[subdomain] => 67.196
[registerableDomain] => 2.34
[publicSuffix] => 34
[port] =>
[path] => /whois-archive/latest.php
[query] => page=2479
[fragment] =>
)
Host: http://<IP_ADDRESS>/whois-archive/latest.php?page=2479
'34' IS NOT a valid public suffix.

Nice catch! Just released 2.0.3 with this fix.


Use Ant Design config-provider  ,Add Right to Left Direction support

Could you please

Utilize config-provider for Template  layout

https://ant.design/components/config-provider/

Add language AR, in app.vue .

this.locale = require("ant-design-vue/es/locale-provider/ar_EG").default;
Additional context
I need to use this wonderful admin template With Arabic language.
English language is is written left to right.
Arabic language is is written Right to right.
For Arabic language
The page layout must be mirrored.
So Main side menu must be on the right of the page.
IMenu, HeaderAvatar, HeaderNotice, HeaderSearch must be aligned to  the left of the page.

Thanks

Upstream project "https://github.com/vueComponent/ant-design-vue":
Does not implement ConfigProvider with direction property.
While root  Upstream  project "https://github.com/ant-design/ant-design/":
Had implemented  ConfigProvider with direction property , that allows  Right to right page layout.

This issue  belongs to upstream project "https://github.com/vueComponent/ant-design-vue".
Thanks.


add consider_3g_degraded, fix some exceptions, redo self.speed lut

you've already implemented format_error, i see.
I'm not sure if your self.speed is correct, I'll push another commit to this in a few minutes. done
your version doesn't run for me currently fixed (exceptions caught)

Thanks, 192 I add a random value because I get this one sometime before
connected and get an error. I must find why and how to handle this... If
you have an idea.
I will look for other things, I'm learning I must understand ^^Thx!
Le jeu. 15 mars 2018 à 19:28, girst<EMAIL_ADDRESS>a écrit :

the self.speed lookup table had some value (e.g. 192) which were not in
the one from the link you provided, so i copied it over.
where did you get the data for self.states from?
also, your handling of 'no modem' was wrong: you loop over all the
possible modem numbers, so only if the for loop never finds anything we
don't have a modem connected. therefore, we have to set the 'no modem'
string before the loop, otherwise we get an exception.
some of the values in the data[] dict might be not available during
initialisation; I've put a try/catch around them.
lastly, the except block (now line 186) does not fire when no modem is
connected, but when something else happens. I do not know when, so could
you have a look at this?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cyrinux/py3status/pull/1#issuecomment-373478048, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAfrq_Pn3Ez1woIWxmzHqm2Uy6FkUFsFks5terLigaJpZM4SsnVE
.

For state for the moment I don't find a doc with value like for speed so I
try to adapt them based on what I see in journalctl -fab while testing
Le jeu. 15 mars 2018 à 19:44, LEVIS Cyril<EMAIL_ADDRESS>a écrit :

Thanks, 192 I add a random value because I get this one sometime before
connected and get an error. I must find why and how to handle this... If
you have an idea.
I will look for other things, I'm learning I must understand ^^Thx!
Le jeu. 15 mars 2018 à 19:28, girst<EMAIL_ADDRESS>a écrit :

the self.speed lookup table had some value (e.g. 192) which were not in
the one from the link you provided, so i copied it over.
where did you get the data for self.states from?
also, your handling of 'no modem' was wrong: you loop over all the
possible modem numbers, so only if the for loop never finds anything we
don't have a modem connected. therefore, we have to set the 'no modem'
string before the loop, otherwise we get an exception.
some of the values in the data[] dict might be not available during
initialisation; I've put a try/catch around them.
lastly, the except block (now line 186) does not fire when no modem is
connected, but when something else happens. I do not know when, so could
you have a look at this?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cyrinux/py3status/pull/1#issuecomment-373478048, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAfrq_Pn3Ez1woIWxmzHqm2Uy6FkUFsFks5terLigaJpZM4SsnVE
.

I#ve fixed the 192 thingy. the speed lut describes a bitmask, meaning for example that if the 14th bit is set, we have 'LTE', and if the 8th is  set we only have '3g'. if we have both bits set, we have 'lte'+'3g' available. 1<<(status['access-technologies'].bit_length()-1) extracts the highest available network setting, so we don't get values like 192 (which would be 11000000 in binary) and turn them into 128 (10000000)


Design artifact delivery pipeline

Hello!

Vote on this issue by adding a 👍 reaction
If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details
We need some alignment on how artifacts will be versioned and delivered to users for preview and also going forward in the long term. The strawman plan is as follows, per artifact:
pulumi-language-jvm
This is the pulumi-language-jvm go binary. Includes the language plugin. Will embed Java programgen to support pulumi import once we teach Pulumi CLI to delegate programgen to the language plugin.
Source: permanently in pulumi-java.
Distribution: together with Pulumi CLI via https://get.pulumi.com/
Pipeline:

tag in pulumi-java triggers GitHub Action job to build the versioned artifact to a Pulumi S3 bucket
during pulumi/pulumi release GitHub Actions grab the pulumi-language-jvm binary and embed it in a release

Versioning:

match the pulumi/pulumi versioning scheme and cadence

To discuss:

alternatives to versioning if versioning tied to Pulumi CLI is limiting
distributing separately but teaching Pulumi CLI to dynamically locate the language plugin

io.pulumi:pulumi
This is the Maven package carrying the language SDK.
Source: permanently in pulumi-java
Distribution: Maven Central
Pipeline:

tag in pulumi-java triggers GHA job to build and publish to Maven Central

Versioning: start new version line
io.pulumi:pulumi-${provider}
This is the Maven package carrying the code-generated Java SDKs for each provider.
Source: embedded in pulumi-java for public preview; going forward, either embedded in provider repos or new dedicated repos
Distribution: Maven Central
Pipeline:

tag in the appropriate repo triggers a GHA job to publish to Maven Central

Versioning: match the versioning of the providers.
To discuss:

different versioning scheme to start with to allow releasing updates caused by codegen fixes not schema changes?
key input to the build process is the provider schema

Cross tests
For starters, the CI in pulumi-java will be exercising tests against the various providers. If they graduate to different repos, we will need to figure out cross-repo triggers to make sure these tests continue to qualify releases.

Affected area/feature

I believe I can close this as most of this is out of date. Currently we are delivering all the artifacts to the public from pulumi-java repo, and there are plans to move provider code and artifact pipelines out into their respective repos, this will be happening over time.


Wrong retrieval of round record in deletion of playerround

The following statement contains the wrong record nr:
  dslContext.selectFrom(Tables.ROUND.where(Tables.ROUND.ID.eq(groupRound.getGroupId()))).fetchOne();

The getGroupId() should be getRoundId().

Tested the deletion of a PlayerRound and it now works.


Context Menu in IE

Hi Paul,
Could you please help me share the event where I can perform an action on selecting a context menu in IE. Because, we are performing some events whenever an item is selected from the context menu

I would say the best option for your case:
https://bossanova.uk/jexcel/v4/examples/contextmenu


utsushi: unstable-2021-01-01 -> 3.65.0, rename, add imagescan plugin

Motivation for this change
Support network scanning for Epson L3160 (and other Epson scanners).
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

Tested with my scanner.

I'll merge this tomorrow if no questions arise.


Increase code coverage for pinot-java-client - ConnectionFactory

Description
Increase code coverage for pinot-java-client
More here: https://github.com/apache/pinot/issues/5694
Upgrade Notes
Does this PR prevent a zero down-time upgrade? (Assume upgrade order: Controller, Broker, Server, Minion)
No
Does this PR fix a zero-downtime upgrade introduced earlier?
No
Does this PR otherwise need attention when creating release notes? Things to consider:
No
Test evidences
> mvn test
....
[INFO] Tests run: 74, Failures: 0, Errors: 0, Skipped: 0
...
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  10.060 s

@HubertWo Please run 'mvn spotless:apply' to fix the violations mentioned in the tests.

Codecov Report

Merging #7350 (e40191b) into master (7e9ca6a) will increase coverage by 4.79%.
The diff coverage is 66.89%.

:exclamation: Current head e40191b differs from pull request most recent head 492c38e. Consider uploading reports for the commit 492c38e to get more accurate results

@@             Coverage Diff              @@
##             master    #7350      +/-   ##
============================================
+ Coverage     64.67%   69.47%   +4.79%
+ Complexity     3295     3209      -86
============================================
  Files          1497     1114     -383
  Lines         74297    52698   -21599
  Branches      10798     7901    -2897
============================================
- Hits          48053    36611   -11442
+ Misses        22818    13481    -9337
+ Partials       3426     2606     -820

Flag
Coverage Δ

integration1
?

integration2
?

unittests1
69.47% <66.89%> (+0.03%)
:arrow_up:

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

Impacted Files
Coverage Δ

.../pinot/common/messages/RunPeriodicTaskMessage.java
0.00% <0.00%> (ø)

...e/pinot/core/common/datatable/DataTableImplV3.java
96.02% <ø> (ø)

...manager/realtime/HLRealtimeSegmentDataManager.java
0.00% <0.00%> (-82.95%)
:arrow_down:

...pinot/core/data/manager/realtime/TimerService.java
0.00% <0.00%> (-50.00%)
:arrow_down:

...geospatial/transform/function/ScalarFunctions.java
92.85% <ø> (+6.19%)
:arrow_up:

...nMaxValueBasedSelectionOrderByCombineOperator.java
58.95% <ø> (-11.20%)
:arrow_down:

...icate/FSTBasedRegexpPredicateEvaluatorFactory.java
40.74% <ø> (+2.80%)
:arrow_up:

...core/operator/filter/predicate/PredicateUtils.java
100.00% <ø> (+14.28%)
:arrow_up:

...ansform/function/AndOperatorTransformFunction.java
80.00% <0.00%> (-20.00%)
:arrow_down:

...ator/transform/function/BaseTransformFunction.java
36.97% <0.00%> (-2.45%)
:arrow_down:

... and 537 more

Continue to review full report at Codecov.

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

I was under impression that project use Java 11. I will make it Java 8 compatible.


explore Wasm/WASI

To run Regal without a binary -- such as from within a VSCode plugin, or client-side in the playground -- we could attempt to compile it to Wasm.
👉 https://go.dev/blog/wasi
👉 https://webassembly.sh/ this would be a simple way to play with the resulting wasi-compatible binary: wapm upload it, and run it from the browser shell

Turns out it's not the lowest-hanging fruit, we'll need to fiddle some with the OPA sources to get this to work:
% GOOS=wasip1 GOARCH=wasm go build -o main.wasm main.go
# github.com/sirupsen/logrus
../../../go/pkg/mod/github.com/sirupsen/logrus@v1.9.3/terminal_check_notappengine.go:13:10: undefined: isTerminal
# github.com/prometheus/procfs
../../../go/pkg/mod/github.com/prometheus/procfs@v0.10.1/fs_statfs_type.go:25:18: undefined: syscall.Statfs_t
../../../go/pkg/mod/github.com/prometheus/procfs@v0.10.1/fs_statfs_type.go:26:17: undefined: syscall.Statfs

That's annoying. Especially considering nothing of the prometheus stuff is likely going to be in the Regal binary anyway.

StyraInc/regal % go mod why github.com/prometheus/procfs
# github.com/prometheus/procfs
github.com/styrainc/regal/pkg/builtins
github.com/open-policy-agent/opa/rego
github.com/open-policy-agent/opa/plugins
github.com/prometheus/client_golang/prometheus
github.com/prometheus/procfs

StyraInc/regal % go mod why github.com/sirupsen/logrus
# github.com/sirupsen/logrus
github.com/styrainc/regal/cmd
github.com/open-policy-agent/opa/bundle
github.com/open-policy-agent/opa/bundle.test
github.com/open-policy-agent/opa/logging
github.com/sirupsen/logrus

We could push a branch with some build tags to nop these out, just for testing.

Sounds good 👍

Update: #346 (and #391) should be a starting point. Compiling regal to WASI now works on main.

Compiling to Wasm works on main for anyone who's curious to try this out. While there's more to explore, I don't think we have anything directly actionable here at this point? But if anyone has ideas, or would want to try things out in this space, we'd love to hear more!
Closing this for now.

I don't think it's high priority, and no one asked for it to this date, but if we ever want to support running the OPA VS Code extension on the web, we'll want to run a Wasm version of Regal (or parts of it anyway) inside of a web worker, as that's the only way to run a language server in that context.


Date Format

Is there a way to change the date format?
May 01, 2017 19:02 seems to be the default.
01 May 2017 19:02 would be ideal for some geographic regions.

What dates are you referring to?

Any dates in the GUI are in the Month Day, Year format.
Unless you're American, you'd usually expect to see Day Month Year.

I suspect that internationalization of date formats would need to be part of a wider i18n project. Right now the interface is US English only (though it's plausible some British has slipped in there too).

There will certainly be some British English (although we do try to avoid it in most places but I refuse to spell colour without a u but I will happily use a Z instead of an S for words like "organization").


Kod berayiñ

Gulruh seń agzyna sk.... kod yazzzzz!!!!!!

https://cla.developers.google.com/dns-query

пн, 16 Ноя 2020, 21:05 Mirat<EMAIL_ADDRESS>
Closed #383 https://github.com/Jigsaw-Code/Intra/issues/383.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Jigsaw-Code/Intra/issues/383#event-4001082970, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ARKS44TZ6Y47CRK2EVIHNQ3SQFEUBANCNFSM4TWJWA4A
.

пн, 16 Ноя 2020, 21:18 Elyas Taganow<EMAIL_ADDRESS>
пн, 16 Ноя 2020, 21:05 Mirat<EMAIL_ADDRESS>
Closed #383 https://github.com/Jigsaw-Code/Intra/issues/383.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Jigsaw-Code/Intra/issues/383#event-4001082970, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ARKS44TZ6Y47CRK2EVIHNQ3SQFEUBANCNFSM4TWJWA4A
.

вс, 15 Ноя 2020, 22:36 suhrab1987<EMAIL_ADDRESS>
https://cla.developers.google.com/dns-query
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Jigsaw-Code/Intra/issues/383#issuecomment-727607067,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARKS44QMACIF3HDJLY3GXALSQAGQFANCNFSM4TWJWA4A
.

Gulruh seń agzyna sk.... kod yazzzzz!!!!!!

Sey diyenin ucin bok yazarın mal


vs-code: more links

Rel. https://github.com/iterative/dvc.org/issues/3718#issuecomment-1175589529 https://github.com/iterative/dvc.org/issues/3717

Thanks @jorgeorpinel! Looks good although I have some questions about the best way to integrate VS Code references. I'm open to either merging as is and continuing discussion after or having discussion before merging.
If I were reading the docs, these would look like ads to me that I might check out later after I've read the rest of the doc. For visualizing plots or experiments, we may want to treat VS Code as a first-class option and integrate them more seamlessly into the docs.
Some examples:

From https://dvc.org/doc/command-reference/plots#description:

DVC generates plots as static HTML webpages that can be open with a web browser. They can also be saved as SVG or PNG image files from the browser.

I would mention that plots can be visualized in VS Code, Studio, or static HTML. Static HTML feels like an option of last resort now that we have so many better ways to visualize plots.

A lot of https://dvc.org/doc/user-guide/experiment-management/comparing-experiments could be skipped by a user if they use VS Code. We could just tell them their experiments can be shown, filtered, and sorted there.

In https://dvc.org/doc/start/experiments/visualization, it's not that clear how these plots are useful. For example, there is a section on how to display an image file without much context about why you would want/need DVC to do that. Showing the image alongside other plots or compared to other experiments in VS Code could help clarify how plots visualization is useful in experimentation.

Good points @dberenbaum
If you don't mind approving this for now so we can merge it and iterate on top of that. There's several of these admonitions in other pages already anyway.

these would look like ads to me that I might check out later after I've read the rest of the doc

They're basically ads as we have them now.
We don't have much documentation on what experiments are or how they work (or any DVC feature in detail for that matter) in the VS Code side and it's unclear whether we should. To learn about that you still have to refer to the dvc.org/doc .

For visualizing plots or experiments, we may want to treat VS Code as a first-class option and integrate them more seamlessly into the docs

I see where you're going. This PR does not attempt that. Let's discuss in #3717 ... Thanks

Yep, I also have a feeling that it looks like an add. Not in a bad sense (we don't advertise a commercial product after all), but in terms of how useful it is and how disruptive it is in the document. I would also expect more "embedded" links.

@shcheklein WDYT about merging as is and then continuing discussion? I'm fine with whatever you and @jorgeorpinel  think is best here for now.

I'm totally fine with that @dberenbaum .


[WEB]Why Image.asset doesn't work in web?

Steps to Reproduce

I made a new flutter_web project by (https://flutter.dev/docs/get-started/web)
I created Assets folder at the project root and added it to the pubspec.yaml too.
(flutter pub get also been done.)
I used asset.image widget with the url of the image like this.
Image.asset ("assets/images/logo.png"),

Expected results:
Like mobile does, I expect to show my logo image to the screen, but it doesn't.
Following #43966, I changed my code from Image.asset to Image.network and moved asset folder into the www folder. Then, it works perfectly.
Why flutter web doesn't allow using Images.asset to add image like?
Actual results:
When I use Asset.Image, flutter compiler printed this log :
Error while trying to load an asset: Failed to load asset at "assets/images/domain.png" (404)
══╡ EXCEPTION CAUGHT BY IMAGE RESOURCE SERVICE ╞══════
The following assertion was thrown resolving an image codec:
Unable to load asset: assets/images/domain.png

Logs

Here's my flutter run log with --verbose option.

D:\DEV\Portfolio_flutter>flutter run -d chrome -v
[  +26 ms] executing: [C:\src\flutter/] git -c log.showSignature=false log -n 1 --pretty=format:%H
[  +79 ms] Exit code 0 from: git -c log.showSignature=false log -n 1 --pretty=format:%H
[   +2 ms] fabeb2a16f1d008ab8230f450c49141d35669798
[   +7 ms] executing: [C:\src\flutter/] git describe --match v*.*.* --first-parent --long --tags
[  +67 ms] Exit code 0 from: git describe --match v*.*.* --first-parent --long --tags
[        ] v1.14.6-0-gfabeb2a16
[  +18 ms] executing: [C:\src\flutter/] git rev-parse --abbrev-ref --symbolic @{u}
[  +49 ms] Exit code 0 from: git rev-parse --abbrev-ref --symbolic @{u}
[   +4 ms] origin/beta
[   +7 ms] executing: [C:\src\flutter/] git ls-remote --get-url origin
[  +51 ms] Exit code 0 from: git ls-remote --get-url origin
[   +2 ms] https://github.com/flutter/flutter.git
[  +98 ms] executing: [C:\src\flutter/] git rev-parse --abbrev-ref HEAD
[  +56 ms] Exit code 0 from: git rev-parse --abbrev-ref HEAD
[   +2 ms] beta
[  +56 ms] Artifact Instance of 'AndroidMavenArtifacts' is not required, skipping update.
[   +2 ms] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[   +6 ms] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[   +8 ms] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[  +10 ms] Artifact Instance of 'FlutterWebSdk' is not required, skipping update.
[  +11 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[   +1 ms] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[   +7 ms] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[   +2 ms] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[  +10 ms] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[   +2 ms] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[   +6 ms] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[  +37 ms] executing: C:\Users\kygha\AppData\Local\Android\sdk\platform-tools\adb.exe devices -l
[  +36 ms] Exit code 0 from: C:\Users\kygha\AppData\Local\Android\sdk\platform-tools\adb.exe devices -l
[   +2 ms] List of devices attached
[  +27 ms] Artifact Instance of 'AndroidMavenArtifacts' is not required, skipping update.
[   +4 ms] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[   +7 ms] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[   +4 ms] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[  +25 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[   +9 ms] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[   +1 ms] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[   +1 ms] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[   +1 ms] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[   +1 ms] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[   +1 ms] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[ +236 ms] Generating D:\DEV\Portfolio_flutter\android\app\src\main\java\io\flutter\plugins\GeneratedPluginRegistrant.java
[ +131 ms] Launching lib\main.dart on Chrome in debug mode...
[   +9 ms] Building application for the web...
[+15040 ms] Starting daemon...
[ +171 ms] Initializing inputs
[  +16 ms] Reading cached asset graph...
[ +206 ms] Reading cached asset graph completed, took 203ms

[ +292 ms] Checking for updates since last build...
[ +909 ms] Checking for updates since last build completed, took 909ms

[  +12 ms] Initializing inputs
[   +5 ms] Reading cached asset graph...
[        ] Reading cached asset graph completed, took 203ms
[        ] Checking for updates since last build...
[   +2 ms] Checking for updates since last build completed, took 909ms
[+1048 ms] About to build [web]...
[   +7 ms] Running build...
[+1074 ms] 1.1s elapsed, 3/5 actions completed.
[+1079 ms] Running build completed, took 2.1s

[   +3 ms] Caching finalized dependency graph...
[   +7 ms] Caching finalized dependency graph completed, took 113ms

[   +3 ms] Succeeded after 2.2s with 11 outputs (8 actions)

[  +20 ms] Building application for the web... (completed in 19.9s)
[  +14 ms] Attempting to connect to browser instance..
[+9029 ms] Debug service listening on ws://<IP_ADDRESS>:11109/-MkFNvcHB4w=

[ +418 ms]       Failed to load asset at path: packages/build_web_compilers/src/dev_compiler/dart_sdk.js.

                          Status code: 404

                          Headers:
                          {
                      "date": "Fri, 07 Feb 2020 17:33:40 GMT",
                      "content-length": "9",
                      "x-frame-options": "SAMEORIGIN",
                      "content-type": "text/plain; charset=utf-8",
                      "x-xss-protection": "1; mode=block",
                      "x-content-type-options": "nosniff",
                      "server": "dart:io with Shelf",
                      "via": "1.1 shelf_proxy"
                    }

                          Content:
                          Not Found}

[  +52 ms] Attempting to connect to browser instance.. (completed in 9.5s)
[   +5 ms] Warning: Flutter's support for web development is not stable yet and hasn't
[   +2 ms] been thoroughly tested in production environments.
[   +9 ms] For more information see https://flutter.dev/web
[   +2 ms] 🔥  To hot restart changes while running, press "r". To hot restart (and refresh the browser), press "R".
[  +10 ms] For a more detailed help message, press "h". To quit, press "q".
[  +11 ms] Debug service listening on ws://<IP_ADDRESS>:11109/-MkFNvcHB4w=
[+1706 ms] Error while trying to load an asset: Failed to load asset at "assets/assets/images/domain.png" (404)
[ +531 ms] ══╡ EXCEPTION CAUGHT BY IMAGE RESOURCE SERVICE ╞════════════════════════════════════════════════════
[   +6 ms] The following assertion was thrown resolving an image codec:
[  +10 ms] Unable to load asset: assets/images/domain.png
[  +13 ms] When the exception was thrown, this was the stack:
[  +11 ms] package:build_web_compilers/src/dev_compiler/C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/errors.dart 196:49  throw_
[   +1 ms] package:flutter/src/services/asset_bundle.dart 221:55                                                                                load
[   +8 ms] package:build_web_compilers/src/dev_compiler/C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 47:50            onValue
[   +1 ms] package:build_web_compilers/src/dev_compiler/C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/async/zone.dart 1134:38                                          _rootRunUnary
[   +7 ms] package:build_web_compilers/src/dev_compiler/C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/async/zone.dart 1031:19                                          runUnary
[   +1 ms] package:build_web_compilers/src/dev_compiler/C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/async/future_impl.dart 140:18                                    handleValue
[   +8 ms] package:build_web_compilers/src/dev_compiler/C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/async/future_impl.dart 682:44                                    handleValueCallback
[   +5 ms] package:build_web_compilers/src/dev_compiler/C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/async/future_impl.dart 711:32                                    _propagateToListeners
[   +5 ms] package:build_web_compilers/src/dev_compiler/C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/async/future_impl.dart 526:5                                     [_completeWithValue]
[   +6 ms] package:build_web_compilers/src/dev_compiler/C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/async/future_impl.dart 556:7                                     <fn>
[   +1 ms] package:build_web_compilers/src/dev_compiler/C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/async/zone.dart 1126:13                                          _rootRun
[   +3 ms] package:build_web_compilers/src/dev_compiler/C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/async/zone.dart 1023:19                                          run
[   +9 ms] package:build_web_compilers/src/dev_compiler/C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/async/zone.dart 925:7                                            runGuarded
[   +1 ms] package:build_web_compilers/src/dev_compiler/C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/async/zone.dart 965:23                                           callback
[   +8 ms] package:build_web_compilers/src/dev_compiler/C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/async/schedule_microtask.dart 43:11                              _microtaskLoop
[   +7 ms] package:build_web_compilers/src/dev_compiler/C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/async/schedule_microtask.dart 52:5                               _startMicrotaskLoop
[   +7 ms] package:build_web_compilers/src/dev_compiler/C:/b/s/w/ir/cache/builder/src/out/host_debug/dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 168:15           <fn>
[  +11 ms] Image provider: AssetImage(bundle: null, name: "assets/images/domain.png")
[   +2 ms] Image key: AssetBundleImageKey(bundle: PlatformAssetBundle#48206(), name:
[   +9 ms] "assets/images/domain.png", scale: 1)
[   +1 ms] ════════════════════════════════════════════════════════════════

This 404 Not found error doesn't occur when I use Images.network widget.

Here's my flutter analyze log.

Analyzing Portfolio_flutter...
No issues found! (ran in 2.3s)

Here's my flutter doctor -v log.

D:\DEV>flutter doctor -v
[√] Flutter (Channel beta, v1.14.6, on Microsoft Windows [Version 10.0.18363.628], locale ko-KR)
    • Flutter version 1.14.6 at C:\src\flutter
    • Framework revision fabeb2a16f (9 days ago), 2020-01-28 07:56:51 -0800
    • Engine revision c4229bfbba
    • Dart version 2.8.0 (build 2.8.0-dev.5.0 fc3af737c7)

[√] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at C:\Users\kygha\AppData\Local\Android\sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.2
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[√] Android Studio (version 3.5)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 43.0.1
    • Dart plugin version 191.8593
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)

[√] IntelliJ IDEA Ultimate Edition (version 2019.3)
    • IntelliJ at C:\Program Files\JetBrains\IntelliJ IDEA 2019.2.2
    • Flutter plugin version 43.0.3
    • Dart plugin version 193.6015.53

[√] VS Code (version 1.41.1)
    • VS Code at C:\Users\kygha\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.8.0

[√] Connected device (2 available)
    • Chrome     • chrome     • web-javascript • Google Chrome 79.0.3945.130
    • Web Server • web-server • web-javascript • Flutter Tools

• No issues found!

/cc @sfshaza2 this sounds like a documentation issue. It is not immediately clear that we only include assets that live directly in the /assets directory.

/cc @jonahwilliams @zanderso

This is essentially the other side of https://github.com/flutter/flutter/issues/27801

Closing, as this is not a bug but a limitation of the asset system which is fully captured by 27801


fix(sandbox): cannot make dataset available

When selecting datasets for sandbox you can select from all available datasets, but if the dataset region does not match the sandbox's region it will fail when you try to make it available.

Selecting this dataset in a norway-based sandbox gives a 500 error, with a message saying that
"Only resources in norwayeast, norwaywest can be ACL-ed to virtual networks in norwayeast.."
Steps:

Make a dataset with region Europe east/west.
Make a sandbox with region Norway east/west.
Create a vm and wait for the other resources.
Select the dataset from step 1 to make available
Click the "Make available" button

If it is not possible to use datasets from other regions, we should filter the list of available datasets to only show valid options.

@hknutsen This is the issue i was talking about.
Do you know if this is a limitation we cannot get around? If that is the case, i suggest we filter the list of available datasets

This is a limitation in Azure itself. As seen in the below screenshot, when trying to add a virtual network to a storage account through the Azure portal we get a note saying that only vnets in the same region as the storage account or the secondart region can be added.

I think maybe we could grey out datasets in other regions rather than completely filter them out, and maybe show a message saying why they are greyed out on hover?

Ok, thanks for investigating. Greying out sounds like a good idea