How to code a listener when I use SubscribeToPushNotifications() to get the new mails.

Hi all.
Now I want to use SubscribeToPushNotifications() to let the server push new mails to my app.
/=====================================================/
Public PushSubscription subscribeToPushNotifications(
Iterable folderIds, URI url, int frequency,
String watermark, EventType... eventTypes) throws Exception{}
/=====================================================/
The function SubscribeToPushNotifications() has a few parameters,  and one of them is URI. I don't  know how to write a listener, anyone can help me? Please, thanks a lot.

+1

any update ?

I just succeed to set up the push request, but I don't know how to deserialize the request from the Exchange Server :
<?xml version="1.0" encoding="utf-8"?>
<soap11:Envelope xmlns:soap11="http://schemas.xmlsoap.org/soap/envelope/">
  <soap11:Header>
    <t:RequestServerVersion xmlns:m="http://schemas.microsoft.com/exchange/services/2006/messages" Version="Exchange2010_SP2" xmlns:t="http://schemas.microsoft.com/exchange/services/2006/types" />
  </soap11:Header>
  <soap11:Body>
    <m:SendNotification xmlns:t="http://schemas.microsoft.com/exchange/services/2006/types" xmlns:m="http://schemas.microsoft.com/exchange/services/2006/messages">
      <m:ResponseMessages>
        <m:SendNotificationResponseMessage ResponseClass="Success">
          <m:ResponseCode>NoError</m:ResponseCode>
          <m:Notification>
            <t:SubscriptionId>JwB2aTFwcjA1bWIxMDA4LmV1cnByZDA1LnByb2Qub3V0bG9vay5jb20QAAAA+l+lLQTaA06BVIWNwne7WF/DUcp2i9IIEAAAAAvWSXbzJs5Mgx6zO1iHfjE=</t:SubscriptionId>
            <t:PreviousWatermark>AQAAAMFcdQJ5b45EkzokImftRv6WKucBAAAAAAA=</t:PreviousWatermark>
            <t:MoreEvents>false</t:MoreEvents>
            <t:StatusEvent>
              <t:Watermark>AQAAAMFcdQJ5b45EkzokImftRv5kK+cBAAAAAAE=</t:Watermark>
            </t:StatusEvent>
          </m:Notification>
        </m:SendNotificationResponseMessage>
      </m:ResponseMessages>
    </m:SendNotification>
  </soap11:Body>
</soap11:Envelope>

:+1:  I don't understand how to get notification either.
I assume I should use something like the following code, but

output of http request is "OK" for both AsyncCallback and IAsyncResult

callback is called immediately but only once
  WellKnownFolderName wkFolder = WellKnownFolderName.Inbox;
  FolderId folderId = new FolderId(wkFolder);
  List<FolderId> folder = new ArrayList<FolderId>();
  folder.add(folderId);
  String url = ...
  int frequency = 1; // minutes from 1 to 1440
  AsyncCallback cb = new AsyncCallbackImplementation() {
      @Override
      public Object processMe(Future<?> task) {
          System.out.println("In AsyncCallback : " + task.isDone());
          try {
              HttpClientWebRequest r = (HttpClientWebRequest)task.get();

              System.out.println("AsyncCallbackImplementation : " + r.getResponseText());
              return task.get();
          } catch (InterruptedException e) {
              e.printStackTrace();
          } catch (ExecutionException e) {
              e.printStackTrace();
          } catch (EWSHttpException e) {
              e.printStackTrace();
          }
          return null;
      }
  };
  IAsyncResult result = service.beginSubscribeToPushNotifications(cb, null, folder, new URI(url), frequency, null, EventType.NewMail, EventType.Created, EventType.Deleted);
  System.out.println("IAsyncResult : " + ((HttpClientWebRequest)result.get()).getResponseText());
  Thread.sleep(100000000);
  PushSubscription subscription = service.endSubscribeToPushNotifications(result);

Using 2.0-SNAPSHOT

Any advance on this topic?

No, I have been working on something else as I had no answer to make it
work :(
Le mardi 17 novembre 2015, Alejandro Caride<EMAIL_ADDRESS>a
écrit :

Any advance on this topic?
—
Reply to this email directly or view it on GitHub
https://github.com/OfficeDev/ews-java-api/issues/151#issuecomment-157356786
.

I also managed to configure a push notification to work, but only one event (repeated 3 times) is received.
The doc https://msdn.microsoft.com/en-us/library/office/dn458791(v=exchg.150).aspx#bk_eventtypes says something about returning, on the listener side, a StatusFrequency element in the SendNotificationResponseMessage.
Any clue how to build that soap response message ?

We see the same thing. You have to send a SOAP response of "OK". Here is a packet capture of what it should look like (based on a capture of a .NET interaction):
    <soap:Envelope
        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
        xmlns:m="http://schemas.microsoft.com/exchange/services/2006/messages"
        xmlns:t="http://schemas.microsoft.com/exchange/services/2006/types"
        xmlns:soap="http://schemas.xmlsoap.org/soap/envelope/">
        <soap:Body>
            <m:SendNotificationResult>
                <m:SubscriptionStatus>
                    OK
                    </m:SubscriptionStatus>
                </m:SendNotificationResult>
            </soap:Body>
        </soap:Envelope>

@bobdrad thanks. Have been searching hours. Its ridiculous, the spec for this listener is not mentioned anywhere


[HW] Jacosta von Achten

Name: Jacosta von Achten
Email<EMAIL_ADDRESS>Linkedin Profile: https://www.linkedin.com/in/jacosta-v-4962261a1/
Attach the homework step I screenshots below:

Attach the homework step II screenshots below:

I haven't had time to follow Ania's tutorial but would love to do this in the future.

Hi Jacosta. I see your local TikTok app is working, but when I checked the instance deployed up on netlify I just get a blank page with the TikTok header. That is usually do the env vars not being configured properly. Please provide a screen of the working app deployed up on netlify. Thank you. :)


LayoutReader: How can I inference if I only have layout information data?

Hello, thanks for the impressive work.
When I do decode_seq2seq, it seems I need to prepare the json file with text information. I want to ask how I can do inference process when only layout information is provided. (I need to change the feature extracting rather than use function "load_and_cache_layoutlm_examples")
Which code I need to modify. Hope you can give me some suggestions.
Thanks.
@HYPJUDY

Hi, maybe you are asking about the layoutlm_only_layout setting? Please check this issue and reply.

@HYPJUDY  Hello, thanks for your kindly reply.
I mean if there is only layout information is provided, how can I finish the inference process? Sorry, I don't have text information. Therefore, it seems I don't need to calculate BLEU score too.
This line in the code will run error.
Can I use this function directly? Which code should I modify?
Hope for your reply.
Thanks!

@HYPJUDY Hello, thanks for your kindly reply. I mean if there is only layout information is provided, how can I finish the inference process? Sorry, I don't have text information. Therefore, it seems I don't need to calculate BLEU score too. When I run the inference process with layout information only, This line in the code will run error. Can I use this function directly? Which code should I modify?
Hope for your reply. Thanks!

Are you success?
when I change input_folder to my data and make layoutlm_only_layout ,the progress still shows the original data?
Can you give me some suggestions? Thanks

@WeihongM @myh12138  Hi, thanks for your interests in our work! In our problem settings, we have both text and layout. The layoutlm_only_layout is for ablation study only, which means we have text but just do not encode it in the model. In your case (no text at all), I would suggest you to generate some random words as placeholders (for example 1 2 3 4 ...). I think it should be the easiest way to run the code (but I am not sure whether there will be some other issues). I would also suggest you to modify a little bit on the evaluation codes to make sure the results are reasonable.

OK, thanks for your reply.

Hello, I think shuffling the sentence is more general as input which means I don't prepare the input order strictly.
However, I find the loss is difficult to decrease when the sentence_shuffle_rate is set to 1.0.  Do you have other code changes when shuffling the sentence?
@HYPJUDY @zlwang-cs

Hello, I think shuffling the sentence is more general as input which means I don't prepare the input order strictly.
However, I find the loss is difficult to decrease when the sentence_shuffle_rate is set to 1.0. Do you have other code changes when shuffling the sentence?
@HYPJUDY @zlwang-cs

No, the sentence shuffle rate only affects the word order in the input sentences.

Thanks for your reply. The loss is stable around at 4.5. It seems strange. I will try to figure out the problem.
In order to speed up the training process for the limitation of GPU number, I only use dataset-train-s2s-layout-m1.json for training. I am not sure whether the data size cause this difference?

I haven't tried this setting but I guess the dataset size shouldn't be a problem. The m1 m2 m3... are divided randomly (we have several machines to process the data so we have several parts)

@WeihongM @myh12138 Hi, thanks for your interests in our work! In our problem settings, we have both text and layout. The layoutlm_only_layout is for ablation study only, which means we have text but just do not encode it in the model. In your case (no text at all), I would suggest you to generate some random words as placeholders (for example 1 2 3 4 ...). I think it should be the easiest way to run the code (but I am not sure whether there will be some other issues). I would also suggest you to modify a little bit on the evaluation codes to make sure the results are reasonable.

Thanks for your reply,I will try it later. I also want to know what does "src" mean and "tgt" mean, if I try to test some images how should I make "src" and "tgt".

@WeihongM @myh12138 Hi, thanks for your interests in our work! In our problem settings, we have both text and layout. The layoutlm_only_layout is for ablation study only, which means we have text but just do not encode it in the model. In your case (no text at all), I would suggest you to generate some random words as placeholders (for example 1 2 3 4 ...). I think it should be the easiest way to run the code (but I am not sure whether there will be some other issues). I would also suggest you to modify a little bit on the evaluation codes to make sure the results are reasonable.

Thanks for your reply,I will try it later. I also want to know what does "src" mean and "tgt" mean, if I try to test some images how should I make "src" and "tgt".

src and tgt mean the source and target in sequence-to-sequence model

Thanks.

I re-create the layout only dataset. And training the program using this function load_and_cache_line_order_examples.
I found using layout and text with shuffle rate can converage, however, when only using the layout information with shuffling option, the loss is still high.
Can you help me figure out this problem? or if there anything wrong in this function load_and_cache_line_order_examples.
@zlwang-cs @HYPJUDY

Please try to use the load_and_cache_layoutlm_examples instead or make some your own modification.

Thanks for your reply.

In the paper, how do you experiment with the layout-only setting?
What's more, how can I use this model when I only have dataset with layout?  Do I need to set  flag "layout_only_dataset" True, too?
Can I prepare the dataset with  some random words as placeholders (for example 1 2 3 4 ...) which mentioned above. The text information will not be used when use the option "layoutlm_only_layout", right?

When we do the layout-only setting, we only use the layoutlm_only_layout flag. We do not use the layout_only_dataset flag at all. (see https://github.com/microsoft/unilm/blob/b94ec76c36f02fb2b0bf0dcb0b8554a2185173cd/layoutreader/s2s_ft/modeling.py#L203)
Using the placeholders is my intuitive idea, which is not covered by our existing works. If you're interested, you definitely should have a try.

Thanks for your reply.
I found that the problem stems from the function: load_and_cache_line_order_examples. I have tried your advice and use the placeholders. The loss decrease normally.
Maybe the function "load_and_cache_line_order_examples" need to be checked.

Thanks for your comments. Glad to see your questions are solved.

Thanks for your comments. Glad to see your questions are solved.

while I trying to test my own dataset, I have change example_path(input_folder) to my dataset, however the result still from dataset-test-s2s-layout-m1.json

while I trying to test my own dataset, I have change example_path(input_folder) to my dataset, however the result still from dataset-test-s2s-layout-m1.json

Hello there,
I have the same problem, when changing the data-directory to some custom text and layout information, the model still predicts the output from the test_data, any suggestions on that?
@myh12138, @zlwang-cs

Could you give more details of your implementation, such as the arguments you are using?

Since the dataset and the object are now made custom, can you provide an example input image to run ocr on.

I dowloaded the pretrained model from  https://huggingface.co/nielsr/layoutreader-readingbank/tree/main. Can you please confirm if this the right model or I need to format the input image in some particular format?
Thanks.


Weed master ui don't start after rebooting Leader node

I restart my masters and leader (sequentially not continues) , the service is up but the ui don't show anything,
Leader log:
test@weed-master-2:~$ sudo journalctl -xf -u weedfs_master
-- Logs begin at Mon 2021-04-19 13:03:54 +0430. --
Apr 19 13:04:31 weed-master-2 seaweedfs-master[1140]: I0419 13:04:31  1140 masterclient.go:120] master masterClient failed to receive from weed-master-2.s02.test.local:9333: EOF
Apr 19 13:04:31 weed-master-2 seaweedfs-master[1140]: I0419 13:04:31  1140 masterclient.go:126] redirected to leader weed-master-2.s02.test.local:9333
Apr 19 13:04:49 weed-master-2 seaweedfs-master[1140]: I0419 13:04:49  1140 masterclient.go:120] master masterClient failed to receive from weed-master-2.s02.test.local:9333: EOF
Apr 19 13:04:50 weed-master-2 seaweedfs-master[1140]: I0419 13:04:50  1140 masterclient.go:126] redirected to leader weed-master-2.s02.test.local:9333
Apr 19 13:05:08 weed-master-2 seaweedfs-master[1140]: I0419 13:05:08  1140 masterclient.go:120] master masterClient failed to receive from weed-master-2.s02.test.local:9333: EOF
Apr 19 13:05:26 weed-master-2 seaweedfs-master[1140]: I0419 13:05:26  1140 masterclient.go:120] master masterClient failed to receive from weed-master-2.s02.test.local:9333: EOF
Apr 19 13:05:26 weed-master-2 seaweedfs-master[1140]: I0419 13:05:26  1140 masterclient.go:126] redirected to leader weed-master-2.s02.test.local:9333
Apr 19 13:05:44 weed-master-2 seaweedfs-master[1140]: I0419 13:05:44  1140 masterclient.go:120] master masterClient failed to receive from weed-master-2.s02.test.local:9333: EOF
Apr 19 13:05:45 weed-master-2 seaweedfs-master[1140]: I0419 13:05:45  1140 masterclient.go:126] redirected to leader weed-master-2.s02.test.local:9333
Apr 19 13:06:03 weed-master-2 seaweedfs-master[1140]: I0419 13:06:03  1140 masterclient.go:120] master masterClient failed to receive from weed-master-2.s02.test.local:9333: EOF

Apr 19 13:06:21 weed-master-2 seaweedfs-master[1140]: I0419 13:06:21  1140 masterclient.go:120] master masterClient failed to receive from weed-master-2.s02.test.local:9333: EOF
Apr 19 13:06:21 weed-master-2 seaweedfs-master[1140]: I0419 13:06:21  1140 masterclient.go:126] redirected to leader weed-master-2.s02.test.local:9333
Apr 19 13:06:39 weed-master-2 seaweedfs-master[1140]: I0419 13:06:39  1140 masterclient.go:120] master masterClient failed to receive from weed-master-2.s02.test.local:9333: EOF
Apr 19 13:06:40 weed-master-2 seaweedfs-master[1140]: I0419 13:06:40  1140 masterclient.go:126] redirected to leader weed-master-2.s02.test.local:9333
Apr 19 13:06:58 weed-master-2 seaweedfs-master[1140]: I0419 13:06:58  1140 masterclient.go:120] master masterClient failed to receive from weed-master-2.s02.test.local:9333: EOF
Apr 19 13:07:16 weed-master-2 seaweedfs-master[1140]: I0419 13:07:16  1140 masterclient.go:120] master masterClient failed to receive from weed-master-2.s02.test.local:9333: EOF
Apr 19 13:07:16 weed-master-2 seaweedfs-master[1140]: I0419 13:07:16  1140 masterclient.go:126] redirected to leader weed-master-2.s02.test.local:9333
Apr 19 13:07:34 weed-master-2 seaweedfs-master[1140]: I0419 13:07:34  1140 masterclient.go:120] master masterClient failed to receive from weed-master-2.s02.test.local:9333: EOF
Apr 19 13:07:35 weed-master-2 seaweedfs-master[1140]: I0419 13:07:35  1140 masterclient.go:126] redirected to leader weed-master-2.s02.test.local:9333
Apr 19 13:07:53 weed-master-2 seaweedfs-master[1140]: I0419 13:07:53  1140 masterclient.go:120] master masterClient failed to receive from weed-master-2.s02.test.local:9333: EOF
Apr 19 13:08:11 weed-master-2 seaweedfs-master[1140]: I0419 13:08:11  1140 masterclient.go:120] master masterClient failed to receive from weed-master-2.s02.test.local:9333: EOF
Apr 19 13:08:11 weed-master-2 seaweedfs-master[1140]: I0419 13:08:11  1140 masterclient.go:126] redirected to leader weed-master-2.s02.test.local:9333
Apr 19 13:08:29 weed-master-2 seaweedfs-master[1140]: I0419 13:08:29  1140 masterclient.go:120] master masterClient failed to receive from weed-master-2.s02.test.local:9333: EOF
Apr 19 13:08:30 weed-master-2 seaweedfs-master[1140]: I0419 13:08:30  1140 masterclient.go:126] redirected to leader weed-master-2.s02.test.local:9333
Apr 19 13:08:48 weed-master-2 seaweedfs-master[1140]: I0419 13:08:48  1140 masterclient.go:120] master masterClient failed to receive from weed-master-2.s02.test.local:9333: EOF
Apr 19 13:09:06 weed-master-2 seaweedfs-master[1140]: I0419 13:09:06  1140 masterclient.go:120] master masterClient failed to receive from weed-master-2.s02.test.local:9333: EOF
Apr 19 13:09:06 weed-master-2 seaweedfs-master[1140]: I0419 13:09:06  1140 masterclient.go:126] redirected to leader weed-master-2.s02.test.local:9333

please check the network.

please check the network.

Network is fine, all nodes can see each other with 10GB network interface

I think weed has a bug in election of masters, because when I restart all 3 masters together they will run and elect a leader and all thing goes well
but when I restart masters sequentially and leader at the end, they won't run and shows this errors that I paste above!

the log just showed weed-master-2 could not connect to itself.

So What should I do in this situation?

the log just showed weed-master-2 could not connect to itself.

So What should I do in this situation?

please check the network.

@kaaaq Take a look at the result of dig weed-master-2.s02.test.local and determine whether it points to the right place. You probably need to edit your /etc/hosts file to add this hostname, or check whatever router is serving your *.test.local space to see if this name exists.

closing since no more updates.


Update Design System Documentation

EASEY Design System
Reviewed and Updated 9/8/22
https://cvpcorp.zeroheight.com/styleguide/s/59398/p/652ebe-text-input


optimise IsEmptyArray

This is a minor perf tweek. but i figured it might be in the hot path, so i would submit it anyway

combines an is and a cast into a pattern match
which means the value is object should also be redundant?
also the type param was never used

BTW. please tell me if these PRs are getting annoying

@SimonCropp only a little. I would prefer if you could take a look at some of the filed issues. That would help us make progress.
Also if you're interested here's the 0.1 board https://github.com/dotnet/tye/projects/3


Catch nil values when sending results to be evaluated since empty

results resulted in a value of zero :(

Codecov Report

Merging #12 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #12   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           8      8
  Lines         358    359    +1
=====================================
+ Hits          358    359    +1

Impacted Files
Coverage Δ

lib/sensu/plugins/prometheus/metrics.rb
100% <ø> (ø)
:arrow_up:

lib/sensu/plugins/prometheus/checks.rb
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Add influxdb to distribution

closing this pr as issues resolved by this
https://github.com/apache/pulsar/pull/4678


chore(deps): update terraform-module

This PR contains the following updates:

Package
Type
Update
Change

github.com/terraform-ibm-modules/terraform-ibm-landing-zone
module
minor
v5.31.3 -> v5.33.0

terraform-ibm-modules/event-notifications/ibm (source)
module
patch
1.10.14 -> 1.10.15

terraform-ibm-modules/observability-instances/ibm (source)
module
patch
2.18.0 -> 2.18.1

terraform-ibm-modules/scc-workload-protection/ibm (source)
module
minor
1.3.1 -> 1.4.0

Release Notes

terraform-ibm-modules/terraform-ibm-landing-zone (github.com/terraform-ibm-modules/terraform-ibm-landing-zone)
v5.33.0
Compare Source
Features

added support to optionally pass an existing CBR zone ID in which all VPCs created will be added to (#745) (b576712)

v5.32.0
Compare Source
Features

added new input ignore_vpcs_for_cluster_deployment to the OCP standard DA to allow you to specify a VPC that you do not wish to create a cluster in. By default a cluster will be created in all of the VPCs specified in the vpcs input. (#881) (3b9d896)

terraform-ibm-modules/terraform-ibm-event-notifications (terraform-ibm-modules/event-notifications/ibm)
v1.10.15
Compare Source
Bug Fixes

deps: update terraform terraform-ibm-modules/cos/ibm to v8.11.11 (#294) (0813ae5)

terraform-ibm-modules/terraform-ibm-observability-instances (terraform-ibm-modules/observability-instances/ibm)
v2.18.1
Compare Source
Bug Fixes

update Cloud Monitoring service endpoint variable description (#572) (84c1143)

terraform-ibm-modules/terraform-ibm-scc-workload-protection (terraform-ibm-modules/scc-workload-protection/ibm)
v1.4.0
Compare Source
Features

added support to create CBR rules using new input cbr_rules (#142) (2bc63c8)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

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

GitHub release
v1.20.3

Your semantic-release bot :package::rocket:


Win7 x64 not opening

The program does not launch on my system. It does on a 32 bit system I have though. There's no logs generated that I can find.

Please ensure that "Visual Studio C++ Redist x86" from microsoft is installed on your system

I installed the one for Visual Studio 2015, seems to not have made a difference.  Any particular one I should be using?

Has to be the 2012 one, sorry, should have clarified that.  Also it has to be the x86 version if you are running the 32bit version of this app.  Or the x64 version if you are running the 64 bit version of the app.  Easiest thing to do is to install both x86 and x64 2012 redists :)

That got it! Thank you so much!

No problem.  It really should be caught as an error in the code, but I can't find a way to catch it before it wants it....

I had the same issue, but installing Visual Studio C++ Redist 2012 fixed it for me. :+1:
This should be included as a prerequisite in the Readme.md file.


corrected some minor spelling mistakes within md files

Thank you!
And congrats for your first PR \o/


Configurable ZK Base

Currently we are hardcoded to a base directory in ZK. This should be configurable so that multiple teknek instances can be run against the same ZK cluster.

Resolved in https://github.com/edwardcapriolo/teknek-core/pull/15


Typo: "typ" should be "type"

https://github.com/feathersjs/generator-feathers/blob/fa2c712b534d338d16c225b0ce3127c3cede6c03/generators/authentication/index.js#L79

This is correct. See the JWT spec.

Oh. Then it's wrong here: https://docs.feathersjs.com/guides/auth/recipe.oauth-basic.html
I noticed the difference between the two and just assumed (incorrectly) that the weirdly-spelled one was wrong. :)

Ah, good catch, can you submit a PR that fixes it (and maybe check if there is any other references like that)?

submitted #956

Fixed via https://github.com/feathersjs/docs/pull/956


Send logged and unhandled exceptions to server

We should send every logged and all unhandled exceptions from the application to the server, so that the server can log them in turn in a file visible to the user.

see #103

Closed in favor of #103


"#warning "crt/link.stub is an internal header" error when compiling TensorFlow w/ CUDA 10

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 18.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: N/A
TensorFlow installed from (source or binary): source
TensorFlow version (use command below): 53faa313b7628cd8c9fbb836544cc6482cafb7a4
Python version: 2.7
Bazel version (if compiling from source): 0.15.2
GCC/Compiler version (if compiling from source): 7.3.0
CUDA/cuDNN version: CUDA 10.0, cudnn 7.3
GPU model and memory: V100
Exact command to reproduce:

yes '' | TF_NEED_CUDA=1 TF_CUDA_VERSION=10.0 CUDNN_INSTALL_PATH=/usr/local/cudnn NCCL_INSTALL_PATH=/usr/local/nccl_2.3.4-1+cuda10.0_x86_64  ./configure
bazel build --config=opt //tensorflow/tools/pip_package:build_pip_package

Describe the problem
Starting at commit 53faa313b7628cd8c9fbb836544cc6482cafb7a4, when trying to compile TensorFlow with CUDA 10.0, I get an error stating "#warning "crt/link.stub is an internal header file and must not be used directly". The full error output is:
ERROR: /home/reedwm/.cache/bazel/_bazel_reedwm/f53335a0bee8ade861f80a7955d79beb/external/nccl_archive/BUILD.bazel:139:1: C++ compilation of rule '@nccl_archive//:device_code' failed (Exit 1)
bazel-out/k8-opt/bin/external/nccl_archive/device_code.cc:19:2: warning: #warning "crt/link.stub is an internal header file and must not be used directly.  Please use cuda_runtime_api.h or cuda_runtime.h instead." [-Wcpp]
 #warning "crt/link.stub is an internal header file and must not be used directly.  Please use cuda_runtime_api.h or cuda_runtime.h instead."
  ^~~~~~~
bazel-out/k8-opt/bin/external/nccl_archive/device_code.cc: In function 'void __cudaRegisterLinkedBinary(const __fatBinC_Wrapper_t*, void (*)(void**), void*)':
bazel-out/k8-opt/bin/external/nccl_archive/device_code.cc:140:5: error: '__NV_EXTRA_INITIALIZATION' was not declared in this scope
     __NV_EXTRA_INITIALIZATION
     ^~~~~~~~~~~~~~~~~~~~~~~~~
bazel-out/k8-opt/bin/external/nccl_archive/device_code.cc:144:5: error: '__NV_EXTRA_FINALIZATION' was not declared in this scope
     __NV_EXTRA_FINALIZATION
     ^~~~~~~~~~~~~~~~~~~~~~~
bazel-out/k8-opt/bin/external/nccl_archive/device_code.cc:145:23: warning: statement has no effect [-Wunused-value]
     for (__i = 0; __i < NUM_PRELINKED_OBJECTS; ++__i) {
bazel-out/k8-opt/bin/external/nccl_archive/device_code.cc:145:53: error: expected ';' before ')' token
     for (__i = 0; __i < NUM_PRELINKED_OBJECTS; ++__i) {
                                                     ^
Target //tensorflow/tools/pip_package:build_pip_package failed to build
Use --verbose_failures to see the command lines of failed build steps.

This does not occur before 53faa313b7628cd8c9fbb836544cc6482cafb7a4. Letting TensorFlow download and build NCCL does not resolve this issue.
@chsigg, can you resolve this?

Same exact error occurs with Python 3.7 as well.

same here.: Highly appreciate fixing it!

device_code.cc is actually a file from a template that ships with the CUDA SDK (cuda/bin/crt/link.stub). I will prepare a CL to replace that include along with the other substitutions. In the meantime, you can install NCCL and specify the version and install location during ./configure.

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

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

This was fixed by 8c072a519e2beed483adb361a9be934a47bee366.


Import in DockerComposeContainer is not up-to-date

When runnig DockerComposeContainerLiveTest.java there occurs an error: java.lang.NoClassDefFoundError: com/github/dockerjava/api/DockerException caused by: java.lang.ClassNotFoundException: com.github.dockerjava.api.DockerException because current source is: com.github.dockerjava.api.exception.DockerException

@pgleska could you please let us know to which article are you referring to?
Thanks!


remove development game

Fixes #795
Close #795
As you assign me to remove development game I'm ready with my PR.

@panwar8279 Please review my pr


Can not choose simulator

Description
I running command npx react-native run-ios --simulator="iPhone 14 Pro Max", wanna specify simulator model, but it always starts iPhone SE
Version
0.70.2
Output of npx react-native info
System:
OS: macOS 13.0
CPU: (8) arm64 Apple M1
Memory: 121.67 MB / 8.00 GB
Shell: 5.8.1 - /bin/zsh
Binaries:
Node: 16.15.1 - /usr/local/bin/node
Yarn: 1.22.19 - /opt/homebrew/bin/yarn
npm: 8.19.2 - /usr/local/bin/npm
Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
Managers:
CocoaPods: 1.11.3 - /opt/homebrew/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 21.4, iOS 16.0, macOS 12.3, tvOS 16.0, watchOS 9.0
Android SDK: Not Found
IDEs:
Android Studio: 2021.3 AI-213.7172.25.2113.9014738
Xcode: 14.0.1/14A400 - /usr/bin/xcodebuild
Languages:
Java: Not Found
npmPackages:
@react-native-community/cli: Not Found
react: 18.1.0 => 18.1.0
react-native: 0.70.2 => 0.70.2
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce
I have macOS 13, Xcode 14 and all the simulator's IOS - 16
Snack, code example, screenshot, or link to a repository
log:
info Found Xcode workspace "messenger.xcworkspace"
info Launching iPhone 14 Pro (iOS 16.0)
info Building (using "xcodebuild -workspace messenger.xcworkspace -configuration Debug -scheme messenger -destination id=9D4EFA1B-6CC1-4704-BFF6-D801C7C4D185")
success Successfully built the app
--- xcodebuild: WARNING: Using the first of multiple matching destinations:
{ platform:iOS Simulator, id:dvtdevice-DVTiOSDeviceSimulatorPlaceholder-iphonesimulator:placeholder, name:Any iOS Simulator Device }
{ platform:iOS Simulator, id:F5EB5AF8-681D-4776-BD2B-47D8BAEA41EA, OS:16.0, name:iPhone 14 }
{ platform:iOS Simulator, id:EB562BC6-278F-425A-B3C9-22D03BDAB4A3, OS:16.0, name:iPhone 14 Plus }
{ platform:iOS Simulator, id:9D4EFA1B-6CC1-4704-BFF6-D801C7C4D185, OS:16.0, name:iPhone 14 Pro }
{ platform:iOS Simulator, id:67175835-D043-47DD-802C-21F10BBE8A2C, OS:16.0, name:iPhone 14 Pro Max }
{ platform:macOS, arch:arm64, variant:Designed for [iPad,iPhone], id:00008103-000641DA2640801E }
{ platform:iOS, id:dvtdevice-DVTiPhonePlaceholder-iphoneos:placeholder, name:Any iOS Device }
{ platform:iOS Simulator, id:EA90AC9D-01E3-48FD-9316-1162EDF483F9, OS:16.0, name:iPad (9th generation) }
{ platform:iOS Simulator, id:78FD1C26-A670-497E-BC09-13A6CCB87BD4, OS:16.0, name:iPad Air (5th generation) }
{ platform:iOS Simulator, id:4A04DB39-B509-45A9-A1CC-C622CE28ECCA, OS:16.0, name:iPad Pro (11-inch) (3rd generation) }
{ platform:iOS Simulator, id:D4F4EA2D-5CE6-4A28-9126-20707C955B34, OS:16.0, name:iPad Pro (12.9-inch) (5th generation) }
{ platform:iOS Simulator, id:1DF0F4D0-5101-454C-ADEE-31734110F688, OS:16.0, name:iPad mini (6th generation) }
2022-10-10 00:01:49.705 xcodebuild[20667:186567] Requested but did not find extension point with identifier Xcode.InterfaceBuilderBuildSupport.PlatformDefinition
info Installing "/Users/leroywagner/Library/Developer/Xcode/DerivedData/messenger-drfeakwbbucxbxemsdwtykltamya/Build/Products/Debug-iphonesimulator/messenger.app"
info Launching "org.reactjs.native.example.messenger"
success Successfully launched the app on the simulator

Guys, it's really important, reply me

Hello guys, yes, we have noticed same issue in RN 0.70
Seems like --simulator parameter was ignored. RN 0.69 work fine

Hello guys, yes, we have noticed same issue in RN 0.70 Seems like --simulator parameter was ignored. RN 0.69 work fine

(base) ➜  ExampleApp npm run ios --simulator "iPhone 14 Pro Max" --verbose
npm verb cli /usr/local/bin/node /usr/local/bin/npm
npm info using<EMAIL_ADDRESS>npm info using<EMAIL_ADDRESS>npm timing npm:load:whichnode Completed in 0ms
npm timing config:load:defaults Completed in 1ms
npm timing config:load:file:/usr/local/lib/node_modules/npm/npmrc Completed in 1ms
npm timing config:load:builtin Completed in 1ms
npm timing config:load:cli Completed in 0ms
npm timing config:load:env Completed in 0ms
npm timing config:load:file:/Users/leroywagner/Desktop/Projects/ExampleApp/.npmrc Completed in 0ms
npm timing config:load:project Completed in 4ms
npm timing config:load:file:/Users/leroywagner/.npmrc Completed in 0ms
npm timing config:load:user Completed in 1ms
npm timing config:load:file:/usr/local/etc/npmrc Completed in 0ms
npm timing config:load:global Completed in 0ms
npm timing config:load:validate Completed in 0ms
npm timing config:load:credentials Completed in 0ms
npm timing config:load:setEnvs Completed in 1ms
npm timing config:load Completed in 9ms
npm timing npm:load:configload Completed in 9ms
npm timing npm:load:mkdirpcache Completed in 0ms
npm timing npm:load:mkdirplogs Completed in 0ms
npm verb title npm run ios iPhone 14 Pro Max
npm verb argv "run" "ios" "--simulator" "iPhone 14 Pro Max" "--loglevel" "verbose"
npm timing npm:load:setTitle Completed in 6ms
npm timing config:load:flatten Completed in 1ms
npm timing npm:load:display Completed in 3ms
npm verb logfile logs-max:10 dir:/Users/leroywagner/.npm/_logs
npm verb logfile /Users/leroywagner/.npm/_logs/2022-10-11T17_07_16_418Z-debug-0.log
npm timing npm:load:logFile Completed in 3ms
npm timing npm:load:timers Completed in 0ms
npm timing npm:load:configScope Completed in 0ms
npm timing npm:load Completed in 22ms

><EMAIL_ADDRESS>ios
> react-native run-ios iPhone 14 Pro Max

info Found Xcode workspace "ExampleApp.xcworkspace"in 22ms
error No simulator available with name "iPhone 13".in 22ms
info Run CLI with --verbose flag for more details.
npm timing command:run Completed in 645ms
npm verb exit 1
npm timing npm Completed in 671ms
npm verb code 1

Same issue 😢

I have the same problem 😔

I'm having the same problem. Always opens on iPhone 12 Pro Max

As a workaround you can use this: npx react-native run-ios --udid XXXXXXX-XXXX-XXXX-XXXX-XXXXXXXXXXXX. You can see the udid on terminal or find it in xcode (Product>Destination>Add Additional Simualtors...)

As a workaround you can use this: npx react-native run-ios --udid 67175835-D043-47DD-802C-21F10BBE8A2C.

Yeah, it works, but generally that's not solve the issue, it needs to be normal to provide simple implements using --simulator flag.

I'm closing this as this issue is already solved:
https://github.com/react-native-community/cli/issues/1707
It's going to be released in the upcoming CLI version.

Nothing is changed...

package.json
    "@react-native-community/cli": "10.2.0",
    "@react-native-community/cli-platform-android": "10.2.0",
    "@react-native-community/cli-platform-ios": "10.2.0",
    "react-native": "0.71.2",

yarn react-native run-ios --scheme name '--simulator=iPhone 14'
# ...
info Found Xcode workspace "name.xcworkspace"
info Found booted iPad Pro — whalemare, iPhone 14
# ...
info Installing and launching your app on iPad Pro — whalemare

In my case --simulator options still ignored

i am having the same issue. normally i test on iphone 14. a few days ago i was testing out on ipad and now no matter what i pass through --simulator it always opens the ipad sim. i cannot figure out how to undo this or get it to target the iphone again.

According to this comment the fix should've been released with 0.70.4, I'm on 0.71.7 and still have the same issue:
--- xcodebuild: WARNING: Using the first of multiple matching destinations:
{ platform:iOS Simulator, id:dvtdevice-DVTiOSDeviceSimulatorPlaceholder-iphonesimulator:placeholder, name:Any iOS Simulator Device }


Creating Client Certificates

File: engine/security/certificates.md, CC @mstanleyjones
Hi a quick question,
Is it possible to expand information in creating of Client Certificates also about where you need to create them?
Thank you
Lukas

That article is a little bit confusing 😞 @Luk3rson can you explain what are you trying to achieve?

Hey,
thank you.
I created certificate via StartCOM. I wanted to use that in docker, but as this Authority is also not trusted I need to include CA and client.crt to my docker installation.
To do that I tried to follow up this documentation.
But it is not for me clear where I need to create certificate for client.
Do I need to generate my private key from client, then create new certificate on client and just copy CA certificate from server to client?
Thank you,

I'm still not 100% sure on what you want to achieve, but I'll assume that you want to configure the Docker daemon to listen on a TCP port, and secure it with mutual TLS, so that only authorized clients can make requests.

Make sure you have all the files needed for setting up a TLS server and client. If you are using a CA like StartCom, you need to ask them for these files.

For the server - CA's public key certificate, private key for the server, and public key certificate for the server
For the client - Ca's public key certificate, private key for the client, and public key certificate for the client.

Configure the Docker daemon to use these files. You can do this by passing flags when starting the Docker daemon, or by adding properties to the daemon.json file.
Here's an example of the flags you need to use when starting Docker:
/usr/local/bin/dockerd -D -g /var/lib/docker -H unix:// -H tcp://<IP_ADDRESS>:2376 \
--tlsverify \
--tlscacert=/certs/ca.pem \
--tlscert=/certs/server.pem \
--tlskey=/certs/server-key.pem

You can check the daemon.json equivalent here.

Configure your Docker client to use the client certificates. There's a couple of options for this, but you can add the client certificate files to ~/.docker/, and make sure they're named ca.pem, cert.pem, and key.pem.

If you're question is not about the Docker daemon, but about how to set up a Docker registry with TLS:

Deploy the registry with TLS as explained here
If your CA is trusted by your operating system (I believe that StartCom should be), then you should not have to do anything. Otherwise, you need to configure your Docker daemon or operating system to trust that CA. Here's an example on how to do that.

I'll close this issue, but I've created #5167 for us to improve the docs regarding TLS.


Some signature descriptions are being saved with a blank space at the beginning, so when searching for them later (without the blank space) they doesn't match

Mongo queries to search for documents in av_analysis collection that have a signature with a trailing whitespace:
db.av_analysis.find({"scans.result": {$regex: / $/m} },{"scans.result": 1}).count()
db.av_analysis.find({"scans.result": {$regex: /^ /m} },{"scans.result": 1}).count()

Ideally, a script to fix previous saved scan results should be done. But since there are very few that have a trailing whitespace, is not worthy.
Untested effort:
var counter = 1;
var bulk = db.av_analysis.initializeOrderedBulkOp();
db.av_analysis.find({"sha1": "3fa91f0f116cc0f19e8105bc51b91b7639605663", "scans.result": /^\s+|\s+$/ },{ "scans": 1}).forEach(
    function(doc) {
	tmp=[];
	doc.scans.forEach( function(c){
		print("c=");
		print(c);
		if(c.result != null){
			c.result=c.result.trim();
		}
		tmp.push(c);
	} )
	print(tmp);
        bulk.find({ "_id": doc._id }).update({
            "$set": { "scans": tmp }
        });

        if ( counter % 1000 == 0 ) {
            bulk.execute();
            counter = 1;
        }
        counter++;
    }
);

if ( counter > 1 ){
    bulk.execute();}


Add Support for Anytek P22 BLE Smart Padlock

Hi,
It’s great to see more device types added to this official plugin. I just bought a Tuya/Smart Life-enabled fingerprint padlock that runs on Bluetooth but none of the plugins can expose it to HomeKit.
Is this due to the device running on Bluetooth? It would be great if I could contribute to add support for this; as this is the most affordable smart padlock on the market.
Brand: Anytek
Model name: P22+
Pairs with: Smart Life app
Runs on: Bluetooth Low Energy

Hi @ahmadnazirafiq ,Thank you for your feedback.
This device is not in the Tuya standard instruction set list, please provide the category and status of this device, so that we can judge whether this category can be supported.


İt doesnt work

Hello i was trying to crack a game then it crashed and it gave me this error Traceback (most recent call last):
File "steam_auto_cracker.py", line 500, in
File "configparser.py", line 1257, in getitem
KeyError: 'EmuConfig'

Found it accidentally deleted one of the files

Thanks for updating this issue, it could help other users that got the same problem :)


Fix move() to call markNeedsLayout().

Otherwise, changing the order of your children (e.g. shuffling a list of
children) will have no visible effect.

Test?

LGTM


Bundler enabled Capistrano provider

Based on discussion in #62.
Creates a new provider that installs gems for deployment and heaven groups before running cap deploy. The default Bundler group is ignored. This allows multiple Capistrano versions, including 3.x, and plugins installed via gems to be used in deployments.
I left this as a separate provider to avoid unintended side-effects. Detecting Bundler groups is a bit too much magic for my taste, but if that is deemed to be desirable, I'll merge the providers to one and add logic to switch between the modes by reading Gemfile.

I don't see this be doced .. so I'm wondering, where should I put the cap 3? in Heaven's Gemfile, or in my application's Gemfile?
Thanks.

OK, I got it. It's in the (to deploy) application's Gemfile


openssl_x509_crl resource & fixes

Description
Hello, this is another contribution from the french organization Institut National de l'Audiovisuel.
It adds a new resource with documentation & tests :

openssl_x509_crl

It also fixed a few bugs :

fix openssl_ec_public_key with documentation & tests
few corrections in the documentation
fix backward compatibility with chef client 12

This is our last big modification. :-)
Issues Resolved
https://github.com/chef-cookbooks/openssl/issues/72
Check List

[x] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[x] New functionality includes testing.
[x] New functionality has been documented in the README if applicable
[x] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

Thanks again


Add context when encountering invalid separator

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 96.93%. Comparing base (c151f22) to head (37a765a).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #137      +/-   ##
==========================================
+ Coverage   96.88%   96.93%   +0.04%
==========================================
  Files           5        5
  Lines         514      522       +8
==========================================
+ Hits          498      506       +8
  Misses         16       16

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


Added official Java packaging structure for WebApps to description.md

I can't merge your pull request because you document the structure of a Java EE 1.4 application. Current best practices are very different and adapted to build automation tools such as Maven or Gradle. For example, here you can see structure that generates Maven for simple J2EE projects. If you are interested in this topic I recommend you to explore modern layouts such as Maven and map your explanation to the code in this lab.


Is there a way to view why a test has failed?

I can already log in to the ghost portal to view the failure details.  Can I view the same details from within azure devOps?

Hey @ARichIVC,
Unfortunately we don't provide a way to do this with the extension. You could manually make a couple API calls to get the suite result ID and then get the test results from that suite result but that would have to be managed in your build. The test result will have a passing: false on the steps that are failing.

Thanks @aaronfay , it's not a big deal really, as you say you do get a pass/fail flag as it is.  The ease at which I got the plugin up and running is much more important.  Keep up the great work!

Thanks for the feedback! 🙏


Added extended pretty printing

Adds new XMLPrettyEx() function taking the ident string as input.

thanks for your contribution.


Disable featurerunner async

Swap the listener classes for featurerunner to use synchronous calls (as opposed to asynchronous)

Currently have a PR open for this issue, creating just for tracking purposes

@ndhelbao can you assign this issue to me?


Pass plainText to transformPastedText and clipboardTextParser props

Problem
I'd like to use clipboardTextParser prop to convert pasted text.
Specifically, I tried to use it to convert pasted markdown text to ProseMirror's node. For example, pasted > foo is converted to blockquote node.
It almost works well, but I have a problem related to plainText.
I expect to disable the converting feature when a user pastes text with Control + Shift + v, but clipboardTextParser isn't aware of plainText or not. So actually it always converts the pasted text to ProseMirror's node.
Solution
Pass plainText to clipboardTextParser.
By this change, I can disable the converting feature with Shift key by the following code.
new Plugin({
  props: {
    clipboardTextParser(text, $context, plainText) {
      if (plainText) {
        return null;
      }

      return convert(text, $context);
    },
  },
});

And I think transformPastedText prop has the same problem, so I also made the same change to the prop.

This seems reasonable. Could you rename the argument to plain (the "text" part seems implicit) in the type signatures and add a note on what it does to the doc comment?

This seems reasonable. Could you rename the argument to plain (the "text" part seems implicit) in the type signatures and add a note on what it does to the doc comment?

Thanks for your comment!
I've updated the argument name and the doc. But English is not my first language, so please tell me if the wording is wrong.
Thanks.

On closer look, I think maybe this flag should also be true when text pasting is forced because the selection is in a code block (see the text && (plainText || inCode || !html) condition in parseFromClipboard). Would that make sense, in your use case?

On closer look, I think maybe this flag should also be true when text pasting is forced because the selection is in a code block (see the text && (plainText || inCode || !html) condition in parseFromClipboard). Would that make sense, in your use case?

Well, it is reasonable, but I actually only use clipboardTextParser and it does not reach clipboardTextParser if inCode is true.
Because of https://github.com/ProseMirror/prosemirror-view/blob/593bee904065d78515a57c0ee9da4bfc40cf076e/src/clipboard.js#L44
So inCode only affects transformPastedText, and I'm not sure the plain flag should be true for transformPastedText. Currently I don't have any plan to use transformPastedText in our app, so I don't have any use case for it. Sorry.

Makes sense. Merged as e3159d9e6d15d82 and touched up in bdb44f83c09545f


Update vm-generalized-image-version.md

clarify what is tenant1 and tenant2 when try to create VM across tenant. and remove unnecessary tenant login phrase.

@sandeepraichura
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Review of open-lineage-services module

Below are comments on the open-lineage-services-api module:

OpenLineageConnectorBase should at least implement OpenLineageGraphStore as the server-side call casts the connector to this interface.  If the BufferGraphStore and MainGraphStore interfaces are designed to represent two different connectors then there should be a MainGraphStoreConnectorBase with MainGraphStoreProviderBase as well as a BufferGraphStoreConnectorBase with BufferGraphStoreProviderBase instead of OpenLineageConnectorBase with OpenLineageProviderBase.
LineageResponse class inherits from OpenLineageAPIResponse but not listed in the OpenLineageAPIResponse's @JsonSubTypes annotation.

Comments on the open-lineage-services-client

The client does not take an NPA userId/Password so not setting security in header.  It is calling restTemplate directly so exceptions are not propertly processed.  It should ideally use the REST Client Connector.
Some methods return response objects (eg VoidResponse).  These response objects should be processed - turning the exception details into a real exception and returning responses as if there was no REST inteface inbetween.  For example a VoidResponse is used on REST call for a client method that has a void return type.  LineageResponse is used on a REST call for a clinet method that has a Lineage return type.

Comments on the JanusGraphConnector (in adapters)

JanusConnectorException should be a checked exception declared on the OpenLineageGraphStore interface.
No README.md to help understand the code and what it is for (hard to find the actual connector implementations).  New module not linked to from parent README.md
No <name> or <description> tags in the pom file

Comments on open-lineage-services-server

JobConfiguration stores the buffer graph as a static - this will not work in a multi-tenent deployment.
OpenLineageRestServices is swallowing exceptions and not reporting to caller for many of its methods.  For example, what happens if the serverName is incorrect?  In addition, the userId is not being passed to the handler to validate the user is allowed to access this service.

Comments on open-lineage-services-spring

exportGraph() method returns a string - not following FFDC coding standards.

    /**
     * Return an entire graph, in GraphSON format.
     *
     * @param userId     calling user.
     * @param serverName name of the server instance to connect to.
     * @param graph      MAIN, BUFFER, MOCK, HISTORY.
     * @return The queried graph, in graphSON format.
     */
    @GetMapping(path = "/export/sources/{graph}", produces = {MediaType.APPLICATION_JSON_VALUE})
    public String exportGraph(@PathVariable("userId") String userId,
                              @PathVariable("serverName") String serverName,
                              @PathVariable("graph") String graph) {
        return restAPI.exportGraph(serverName, userId, graph);
    }

General comments:

No Javadoc

Hi Mandy, thank you for your review. All points should be addressed from PR #2303 on.


Implement NetworkPolicy

This PR adds Network Policies to all three workload components.
kubectl get networkpolicy -n workload

NAME                                                 POD-SELECTOR                              AGE
backgroundprocessor-allow-all-egress                 app=backgroundprocessor                   11h
catalogservice-allow-all-egress                      app=catalogservice                        94s
catalogservice-allow-cert-manager-resolver-ingress   acme.cert-manager.io/http01-solver=true   94s
catalogservice-allow-ingress                         app=catalogservice                        94s
catalogservice-deny-all                              <none>                                    11h
healthservice-allow-all-egress                       app=healthservice                         12h
healthservice-allow-ingress                          app=healthservice                         12h

catalogservice

allow ingress on its container port
allow ingress for cert-manager resolver
allow all egress
default deny

healthservice

allow ingress on its container port
allow all egress

backgroundprocess

deny ingress
allow allegress

Network Policy can be disabled per chart by overwriting .Values.networkpolicy.enabled which is set to true by default.

@msimecek / @sebader can we have a short call later today to get this one done?

Made some smaller improvements to incorporate the feedback from our meeting on friday afternoon. The charts for CatalogService as well as for BackgroundProcessor have received a new parameter .Values.networkPolicy.egressRange that lets you limit the egress traffic to a given IP range. This defaults to <IP_ADDRESS>/0 but can be adjusted to for example allow egress to only the IP range that contains our Private Endpoints.
default
kubectl describe networkpolicy -n workload
Name:         backgroundprocessor-allow-all-egress
Namespace:    workload
Created on:   2022-02-16 12:50:01 +0000 UTC
Labels:       app.kubernetes.io/managed-by=Helm
Annotations:  meta.helm.sh/release-name: workload-backgroundprocessor
              meta.helm.sh/release-namespace: workload
Spec:
  PodSelector:     app=backgroundprocessor
  Not affecting ingress traffic
  Allowing egress traffic:
    To Port: <any> (traffic allowed to all ports)
    To:
      IPBlock:
        CIDR: <IP_ADDRESS>/0
        Except:
  Policy Types: Egress

with ip range set
kubectl describe networkpolicy/backgroundprocessor-allow-egress-traffic -n workload
Name:         backgroundprocessor-allow-egress-traffic
Namespace:    workload
Created on:   2022-02-21 12:36:40 +0000 UTC
Labels:       app.kubernetes.io/managed-by=Helm
Annotations:  meta.helm.sh/release-name: workload-backgroundprocessor
              meta.helm.sh/release-namespace: workload
Spec:
  PodSelector:     app=backgroundprocessor
  Not affecting ingress traffic
  Allowing egress traffic:
    To Port: <any> (traffic allowed to all ports)
    To:
      IPBlock:
        CIDR: <IP_ADDRESS>/27
        Except:
  Policy Types: Egress

To implement that I had to add an additional output to our regional stamp deployment:

@sebader / @msimecek the CatalogService does not really show anything in its logs - any suggestion how to troubleshoot potential timeouts?

yeah looks like the outbound calls dont work properly. The fact that cosmos tries to access -usgovtexas shows that it did not succeed in accessing it main endpoint

can you test a curl https://afe2e8565-global-cosmos.documents.azure.com from the pod?

Can you try to do dig instead of curl to see where and to what it resolves? Should you also whitelist azure private dns?

Can you try to do dig instead of curl to see where and to what it resolves? Should you also whitelist azure private dns?

We neither have curl, nor dig, nc or any other of these fancy tools in our super lightweight container image.. We've to use some more foundational tooling here..
exec 88<>/dev/tcp/afe2e8565-global-cosmos.documents.azure.com/443
bash: afe2e8565-global-cosmos.documents.azure.com: Temporary failure in name resolution
bash: /dev/tcp/afe2e8565-global-cosmos.documents.azure.com/443: Invalid argument

I'll add and test some explicit DNS allow policies.

Missed it earlier, but it seems it was resolved in the earlier example to <IP_ADDRESS>. Which is outside <IP_ADDRESS>/27 of course. Maybe remove restrictions from the ingress controller to get started?

Missed it earlier, but it seems it was resolved in the earlier example to <IP_ADDRESS>. Which is outside <IP_ADDRESS>/27 of course. Maybe remove restrictions from the ingress controller to get started?

<IP_ADDRESS> is the IP address of the catalogservice-service Kubernetes service. Network Policy kicks in on a per-label basis between nginx-ingress (in kube-system) and catalogservice-deploy (in workload).
I'm able to get it to work when dropping the egress limitation to <privateendpoints-ip-address-range> or let it fall back to <IP_ADDRESS>/0 but that wasn't the goal, outcome of our call on friday was that

https://dev.azure.com/alwayson/Foundational-Online/_build/results?buildId=383&view=results
An example option to lock down egress traffic is now implemented per application helm chart. Due to the variety of dependencies outside of the private endpoint subnet it continues to default to <IP_ADDRESS>/0 for now. An additional policy that enables dns traffic 53/UDP and 53/TCP to the kube-system namespace was added.


Unable to list bucket contents using Storage.list(), getting 403 Access Denied

Describe the bug
Unable to list the contents of a file in S3 using Storage.list()
To Reproduce
Storage.list('images/').then(result => log.debug(result)).catch(err => log.error(err));
The error i get is:
xhr.js:83 GET https://my-bucket.s3.ap-southeast-2.amazonaws.com/?prefix=public%images%2F 403 (Forbidden)
AWSS3Provider - list error AccessDenied: Access Denied
Expected behavior
My Cognito Auth is admin level so i can Put, Get and Remove objects from the bucket so i would be expect to be able to List the contents.
Screenshots

Environment

OS: Windows Server 2012 6.3.9600
Node: 12.14.0
npm: 6.13.4
aws-amplify: ^2.2.1 => 2.2.1

Additional context
I have taken some steps to see if it would fix the issue:

Redeployed the bucket e.g. amplify add storage
Confirmed that the permissions in the backend file show the correct permissions
Checked all IAM roles to confirm that they have all the correct permissions
The bucked does have content

Moving to Amplify CLI in order to understand the backend configuration here.

Please ensure that if a user is part of a user pool group, run amplify storage update to enable IAM group policies for CRUD operations. We are working on adding relevant message in the CLI and in the docs related to this. Closing this as a duplicate of https://github.com/aws-amplify/amplify-js/issues/5729, please feel free to go through that thread for more information or comment if you think this is a separate issue.


implements Zookeeper weighted quorum and improve readinessProbe, to avoid wrong quorum election and lost data in zookeeper

Signed-off-by: Slach<EMAIL_ADDRESS>Important items to consider before making a Pull Request
Please check items PR complies to:

[x] All commits in the PR are squashed. More info
[ ] The PR is made into dedicated next-release branch, not into master branch1. More info
[x] The PR is signed. More info

I suggest not to merge it into clickhouse-operator at all, but move relevant ZK manifests to ACM. Otherwise, we are testing in production.


Pyinfra errors with cryptic message when SFTP is not avilable

Describe the bug
When attempting to upload a file to a remote server and the remote server happens to have SFTP disabled (e.g. Synology NAS), Pyinfra simply errors out with a cryptic message:
[<IP_ADDRESS>] Command socket/SSH error: SSHException('Channel closed.',)
[<IP_ADDRESS>] Error

To Reproduce
Perform the following operation on any remote that has the SFTP subsystem disabled:
files.put(
    name="Put some file in",
    src='/path/in/local',
    dest='/path/in/remote',
)

Expected behavior
Pyinfra should provide a more helpful error message so that the user can immediately fix the issue on their part.
Meta

Include output of pyinfra --support.

System: Darwin
  Platform: Darwin-19.6.0-x86_64-i386-64bit
  Release: 19.6.0
  Machine: x86_64
pyinfra: v1.3
Executable: /Users/mark/.pyenv/versions/aircraft/bin/pyinfra
Python: 3.7.7 (CPython, Clang 11.0.0 (clang-1<IP_ADDRESS>))

How was pyinfra installed (source/pip)?

pip install -e .

Possible Fix
diff --git a/pyinfra/api/connectors/ssh.py b/pyinfra/api/connectors/ssh.py
index 7a28b63a..926107ba 100644
--- a/pyinfra/api/connectors/ssh.py
+++ b/pyinfra/api/connectors/ssh.py
@@ -306,7 +306,14 @@ def run_shell_command(
 @memoize
 def _get_sftp_connection(host):
     transport = host.connection.get_transport()
-    return SFTPClient.from_transport(transport)
+
+    try:
+        sftp_client = SFTPClient.from_transport(transport)
+    except SSHException as e:
+        raise Exception("Unable to establish SFTP connection. "
+                        "Check that the SFTP subsystem "
+                        f"for the SSH service at {host} is enabled.") from e
+    return sftp_client

 def _get_file(host, remote_filename, filename_or_io):

Thank for the PR/patch @relaxdiego, change looks good to me I shall get this added over the next few days!

Now implemented in https://github.com/Fizzadar/pyinfra/commit/c4160325a7e12a3048a9727940c46db84a9975d7 pending v1.3.1 release!

Now implemented in https://github.com/Fizzadar/pyinfra/commit/c4160325a7e12a3048a9727940c46db84a9975d7 pending v1.3.1 release!

Now released in 1.3.1!

Now released in 1.3.1!


Add a simple host:port namer with no lookup

Could I request a namer than maps host:port directly without any external service lookup.
For example,

bind /http/1.1/GET/<IP_ADDRESS>/16992 to <IP_ADDRESS>:16992
bind /http/1.1/GET/<IP_ADDRESS>/17335 to <IP_ADDRESS>:17335
etc

The reason for this is so that calling apps can do service lookup themselves and leave linkerd to just proxy the URL.

Thanks, Adrian!

I think I may be missing something here.  Could you use a dtab like:
/http/*/* => /$/inet

which would map /http/1.1/GET/<IP_ADDRESS>/16992 to /$/inet/<IP_ADDRESS>/16992?

@adleong You're absolutely right. That gives me exactly what I need. Thanks very much.
@olix0r Do you still think this issue is worth keeping open? I'm happy with the solution proposed by adleong.

When a host with a port is specified in the Host header, it is a single path entry: /http/1.1/GET/<IP_ADDRESS>:16992.  We need a rewriting namer to split on the the : into two path segments.
eg.
/srv => /$/inet;
/http/*/* => /$/io.buoaynt.http.hostPortPfx/srv;

Which would delegate
/http/1.1/GET/<IP_ADDRESS>:16992
/$/io.buoyant.http.hostPortPfx/srv/<IP_ADDRESS>:16992
/srv/<IP_ADDRESS>/16992
/$/inet/<IP_ADDRESS>/16992

Does commit 70502c2 address this issue?

@adrian yeah, these new namers will be present in 0.8.6 (being released now).  Let us know if you have any further questions!


Please expose documentation about REST API endpoints

If there is a REST API available for accessing user data like total bLuna, total LP tokens, total UST in the earn tab, etc., please expose this information in the developer section of the Anchor docs.
If there is a not a REST API for this, can one be created?

There is one that's in preparation by @kjessec - we'll have the relevant documentation added to Anchor docs once it's ready.

Hi @kjessec @ryanology045  how is this going? I was about to open an issue for this too :-)
Otherwise, how do you recommend to interact with anchor?
Context:
I am building an app in Elixir to manage my portfolio. I need data from anchor, like the exchange rate, deposits, and so on... I know I can query the contract. How do you recommend doing it? Can I use an LCD server? maybe an FCD? or just execute terrad?
I have been trying to find out information about this for several days, without any success yet.
Thanks!


Nullable field causing type assignment error

Hey @divyenduz @timsuchanek
Steps to reproduce
Use below schema.graphql and yield fresh interfaces and scaffold.
type Query {
  me: User
}

type Mutation {
  signup(name: String!, email: String!, password: String!): AuthPayload!
  login(email: String!, password: String!): AuthPayload!
}

type AuthPayload {
  token: String!
  user: User!
}

# The `User` type is a reduced version ("mask") of the `User` type from the data model (and database schema).
# It does not expose the `createdAt`, `updatedAt` and `password` fields.
type User {
  id: ID!
  email: String!
  name: String
}

Following type error is produced

{
	"resource": "/c:/Users/jas/play/prt2/src/resolvers/User.ts",
	"owner": "typescript",
	"code": "2322",
	"severity": 8,
	"message": "Type '(parent: UserParent) => string | undefined' is not assignable to type '(parent: UserParent, args: {}, ctx: Context, info: GraphQLResolveInfo) => string | Promise<string | null> | null'.\n  Type 'string | undefined' is not assignable to type 'string | Promise<string | null> | null'.\n    Type 'undefined' is not assignable to type 'string | Promise<string | null> | null'.",
	"source": "ts",
	"startLineNumber": 13,
	"startColumn": 3,
	"endLineNumber": 13,
	"endColumn": 7,
	"relatedInformation": [
		{
			"startLineNumber": 138,
			"startColumn": 5,
			"endLineNumber": 138,
			"endColumn": 9,
			"message": "The expected type comes from property 'name' which is declared here on type 'Type<TypeMap>'",
			"resource": "/c:/Users/jas/play/prt2/src/generated/resolvers.ts"
		}
	]
}

Cause of error:
Interfaces use null but resolver using undefined; for making property nullable.
I will be adding PR to this thread that will change interface to use undefined instead of null.
Although you the choose reverse if you want to.
Thx

@divyenduz I think you should research code from https://github.com/dotansimha/graphql-code-generator

@jas99 : Can you please share the version of your graphql-resolver-codegen by using graphql-resolver-codegen --version

Hey @divyenduz
I was testing against the basic schema from fixtures in this repo.
The resolver generation issue might be related to error in schema itself.
custom_with_custom_arg(id: Number!): Number!
scalar_with_custom_arg(id: Number!): Boolean!

I don't think one can use types as arguments, doesn't it has to be input type?

Yes, it has to be an input type. This looks like a bug.

@divyenduz Any Update on this issue ?
@jas99 proposed https://github.com/prisma/graphql-resolver-codegen/pull/52

And this issue only happens when having strictNullChecks: true in tsconfig.json, if I'm right.

@kevinmarrec No that was not the issue. Generated interfaces returns Null but the scaffold is generating undefined type instead of null; which causes type mismatch.

Oh yeah I got confused ^^, but anyway the error which allowed you to see the issue could only happen with the strictNullChecks: true option (or strict: true). Cause without it TypeScript doesn't seem to care if your resolver return null instead of undefined ^^.
So the PR is fixing the scaffolding and when we want to do things by ourselves it's about replacing variable?: Type by variable: Type | null right ? If I correctly understood what you did.

@kevinmarrec Right on.. actually for some weird reason @divyenduz intentionally introduced undefined which might not be inline with graphqljs implementation.

@jas99 : Thanks! that makes sense, can you please create a minimal reproduction project with strictNullChecks: true
In the mean time, can you also please create an issue for the input issue, hopefully with a reproduction 🙏
I will try to get both these resolved ASAP. Thanks!

@divyenduz Buddy is true by default "strict": true" which means strickNullCheckts is also true and the issue with type-mismatch was not edge case.
typescript-graphql-auth in example has user with nullable property.
https://github.com/prisma/prisma-examples/blob/master/typescript-graphql-auth/src/resolvers/User.ts
If you lint it you will see the issue.

Please re-read the very first comment. I have reattached the pic.
PS: My pr #52 fixes this. I will merge PR with recent update from master.
Thx

@jas99 Removing the ? and replacing by | null in [Model]Parent interfaces seems to break Types for Mutations and Queries :/

I think the solution is on resolver generation, when a field is not required it should be (for a string) :
string | undefined
or maybe
string | undefined | null
But not
string | null

@jas99 As far as I know, the problem is :
Typescript ? means | undefined
But in GraphQL Language it means | null

@jas99 @kevinmarrec : Can anyone of you prepare a minimal reproduction repository? Currently, I just have some code + an image. Real code would help 🙏

@kevinmarrec I personally use https://www.npmjs.com/package/@jas99/graphql-resolver-codegen
Generate both interfaces and scaffold with this variant; its tested by me.

@divyenduz  https://github.com/kevinmarrec/typescript-prisma-minimal

You should have a TS lint error in resolvers/User.ts
Replace notRequiredField?: string by notRequiredField: string | null
Problem in 1) solved BUT new error in Query.ts

@divyenduz https://github.com/prisma/prisma-examples/tree/master/typescript-graphql-auth

@jas99 On typescript-graphql-auth if I replace ? by | null in Parent Models, I get TS issues around Queries/Mutations using this model as return value (or in a payload which contains this model).

And typescript-graphql-auth doesn't have the strictNullChecks: true in tsconfig.json

@kevinmarrec strictNullChecks is on by default

@jas99 Not for me in my editor

@kevinmarrec when you do tsc --init; can you see  "strict": true, thr?

@jas99 Well in my project I have a custom tsconfig.json, as typescript-graphql-auth does. Maybe tsc --init generate a filed with strict: true but it doesn't mean that strict has true as default value with an empty tsconfig.json

But anyway with strict: true you should have issues in Queries/Mutations, using your solution with | null

@jas99 Maybe thr are new changes; I will look into it. Meanwhile instead of graphql-resolver-codegen use @jas99/graphql-resolver-codegen and let me know if even this is breaking; caz I have tested this to be working.

@divyenduz I tried locally to replace all null by undefined in src/generated/resolvers.ts, everything seems to be alright => 0 issues with TypeScript + Everything works fine executing GraphQL queries when requesting required/not-required fields. (GraphQL handle by itself undefined values to result as null in data responses)
Maybe should wereplace all null by undefined around resolver generation ?

@kevinmarrec Check conversation at #51

@jas99 Checked but I still don't know what is the problem ? It seems to be a choice by opinion and not a followed directive provided by GraphQL-Js (if i'm wrong please give me some references to read or code to know what's going on between using undefined or null)

Prisma Client itself generates Model Nodes (UserNode for example) with undefined and not null, that's what cause typescript issues aroune mutations and queries as they must return something like Nodes Types but fail cause our Model Parent (UserParent refering to UserNode) results in type mismatching. (undefined !== null)

I assume this has been fixed by the latest version. Please open a new issue if this problem still occurs.

Check #278

How do I update a date field to null or undefiend?


Adding an asset fails with server error and Corrupted path detected - possibly due whitespace in title

Description
Hey there!
Adding one particular new asset fails. A server error is thrown, and the following is found in the logs.
Screen shot shows a possible problem with a whitespace character.
2022-06-23 15:30:32 [-][14043][9m6nq33v5k481eair6e4purbe2][error][League\Flysystem\CorruptedPathDetected] League\Flysystem\CorruptedPathDetected: Corrupted path detected: How-to-Build-Trust-with-Franchise-Brokers-and-Gro.json in .../vendor/league/flysystem/src/CorruptedPathDetected.php:15
Stack trace:
#0 .../vendor/league/flysystem/src/Util.php(146): League\Flysystem\CorruptedPathDetected::forPath('How-to-\xEF\xBB\xBFBuild...')
#1 .../vendor/league/flysystem/src/Util.php(107): League\Flysystem\Util::removeFunkyWhiteSpace('How-to-\xEF\xBB\xBFBuild...')
#2 .../vendor/league/flysystem/src/Util.php(92): League\Flysystem\Util::normalizeRelativePath('How-to-\xEF\xBB\xBFBuild...')
#3 .../vendor/league/flysystem/src/Filesystem.php(56): League\Flysystem\Util::normalizePath('How-to-\xEF\xBB\xBFBuild...')
#4 .../vendor/craftcms/cms/src/base/FlysystemVolume.php(122): League\Flysystem\Filesystem->has('How-to-\xEF\xBB\xBFBuild...')
#5 .../vendor/craftcms/cms/src/services/Assets.php(885): craft\base\FlysystemVolume->fileExists('How-to-\xEF\xBB\xBFBuild...')
#6 .../vendor/craftcms/cms/src/services/Assets.php(888): craft\services\Assets->craft\services\{closure}('How-to-\xEF\xBB\xBFBuild...')

Steps to reproduce

Attempt to add new asset with url https://www.youtube.com/watch?v=N1nequHmaZE
Results in server error

Other information

Embedded Assets version: 2.11.0
Craft CMS version: <IP_ADDRESS>

Thanks for reporting that - fixed now in Embedded Assets 2.11.1.

Thanks for the quick reply.
I'm seeing a type error after updating to 2.11.1.
2022-06-24 07:18:53 [-][14043][qin5e34r3a223j6rt0bt144efa][error][TypeError] TypeError: spicyweb\embeddedassets\Service::createAsset(): Return value must be of type craft\elements\Asset, string returned in .../vendor/spicyweb/craft-embedded-assets/src/Service.php:333
Stack trace:
#0 .../vendor/spicyweb/craft-embedded-assets/src/Controller.php(53): spicyweb\embeddedassets\Service->createAsset(Object(spicyweb\embeddedassets\models\EmbeddedAsset), Object(craft\models\VolumeFolder))
#1 [internal function]: spicyweb\embeddedassets\Controller->actionSave()
...

Whoops, there was a bit of a typo where it was trying to return the sanitised filename instead of setting it on the asset - not really sure how that was working for me, but anyway... That should now be working properly in 2.11.2, but please let me know if any other issues come up.

That's working perfectly. Thanks again!


Use window dimensions instead of screen ones.

When trying to keep the flyout in the viewport, screen does not accurately represent the viewport – it represents the physical screen size. This is doubly true on mobile devices that switch to portrait mode where the width and height do not change with respect to the orientation change. Because of this, using the window.inner* properties make a lot more sense.
This closes #19.
/cc @qiushihe @cristianjd

👍
Thanks for fixing this!


Release Cadence

Since Eclipse foundation took over this project, I don't see a whole lot of commits or development.  Is there a roadmap or release cadence as to what is planned?

Thank you for the update.

any progress?

any updates?

Hi everyone.
I appreciate your patience. The project is in much better shape now. We started merging small fixes and improvements and are working on a public roadmap, which we intend to make available in early 2025.
I expect we will do a v6.4.2 service release in Q1 2025, barring any unforeseen issues.
My current focus is to convert the current documentation set (rtos-docs repository) from markdown to AsciiDoc. This is because we found a way to publish the current MD files to HTML but ran into difficulties in doing the same to PDF. Microsoft used a custom solution for that, and we did not inherit it. Moving to AsciiDoc will enable us to publish easily to both formats.
So, things are looking up! Happy holidays to you all.

I am glad to hear that, especially since in my opinion ThreadX is a piece of very good software. Probably its major problem was that it was seen as a Microsoft product.


Frame time very high and unstable using NavigationAgent

Godot version
v4.0.dev.custom_build [d5b8abdb6]
System information
Windows 10, gtx 1650,  i7 9750H
Issue description
I'm trying to implement pathfinding with the new NavigationAgent, but when I run the code the system performance drops very low, i get a lot of fps spikes; the worst spikes are on the physics process.
Steps to reproduce
See project below.
Minimal reproduction project
bug.zip

How did you solve the issue?

In fact, I realized that we are using a custom 4 version, not the 3.x. The other thing is I didn't test the issue on the official 4. I'm compiling it right now and I'll test as soon as possible. If the issue persists, i'll reopen the issue.

Tested on 4, same issue. See the updated description.

cc @AndreaCatania

Just tested and the framerate is fine. However, on my side the character wasn't able to move, and the function print($NavigationAgent3D.is_target_reachable()) was returning false. This is a bug on godot side, since the project is using the navigation correctly. So far I didn't go deeper.

Very strange. On my side gdscript print($NavigationAgent3D.is_target_reachable()) returns true, and it moves correctly.


Use postgresql in CI tests to match production

To minimize the risk of deploying a query that doesn't work with psql, we should run tests using postgres.

We now use only Postgres


CTest extended support

Allows to run all tests (accuracy, sanity, performance) via CTest.
Adds extra labels to all tests to select a subset of them.

:+1:


Possible improvements to the script

I'm taking my first steps with Spectrum. Great work!
I find it difficult to run it on notebooks, mostly because of prompt_toolkit, but I understand it is convenient, especially for weights selection.
However, I believe the script could benefit from a couple of improvements:

make it possible to import it (in notebooks), by wrapping the final section in a if __name__ == "__main__": block
It seems that self.optimizer and self.tokenizer are initialized but never actually used.

@Crystalcareai What do you think?
If these observations make sense, I'd be more than happy to open a PR to address them.

wonderful suggestions, and I needed to tidy up the code anyway. All changes can be found here: https://github.com/cognitivecomputations/spectrum/pull/6

Great. Thanks!


Refactor to more enum usage

Codecov Report

Merging #1172 (d6bfc38) into devel (75cc95b) will decrease coverage by 0.23%.
The diff coverage is 94.25%.

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

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##            devel    #1172      +/-   ##
==========================================
- Coverage   91.01%   90.78%   -0.23%
==========================================
  Files          48       48
  Lines        5573     5557      -16
==========================================
- Hits         5072     5045      -27
- Misses        501      512      +11

Files Changed
Coverage Δ

hahomematic/platforms/custom/light.py
98.63% <ø> (ø)

hahomematic/platforms/generic/entity.py
97.46% <ø> (ø)

hahomematic/client.py
57.87% <59.25%> (+0.23%)
:arrow_up:

hahomematic/support.py
89.47% <82.35%> (-3.35%)
:arrow_down:

hahomematic/platforms/device.py
81.65% <90.90%> (ø)

hahomematic/platforms/generic/__init__.py
93.54% <91.66%> (+0.21%)
:arrow_up:

hahomematic/decorators.py
98.63% <95.83%> (ø)

hahomematic/caches/dynamic.py
79.48% <100.00%> (ø)

hahomematic/caches/persistent.py
90.95% <100.00%> (ø)

hahomematic/caches/visibility.py
95.50% <100.00%> (ø)

... and 10 more

... and 2 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Magisk support

Hi, I was able to get the side loader working on my ZTE Nubia Red Magic 5G. I am wondering how to root the side loaded gsi. I tried patching my stock boot.img, but as soon as the gsi boots it goes to the lock screen and then black. Restoring the stock boot.img allows it to boot again. Am I missing something? Is this even possible?

Your boot.img is shared between original OS and DSU, which means if your original OS is rooted with Magisk, then DSU should be also, try to use other GSI to test if same behavior occurs.

Every GSI that I have tried does the same thing. If I don't have my boot.img patched with magisk it boots fine. If I try to patch it, it boots to lock screen and goes black, but you can hear notifications and such. I ended up using a GSI with root built in and stock boot.img and it works great! Thank you for all the work you've put into this, It really is a great app.


pico_flash should depend on hardware_exception

Hi there, I think pico_flash should depend on hardware_exception? It unconditionally includes hardware_exception, and if I try to compile a file that depends on pico_flash without editing the SDK, I get this compile error:
[build] /home/jaguilar/projects/pico-sdk/src/rp2_common/pico_flash/flash.c:8:10: fatal error: hardware/exception.h: No such file or directory
[build]     8 | #include "hardware/exception.h"
[build]       |          ^~~~~~~~~~~~~~~~~~~~~~

If I add hardware_exception to the dependency list of pico_flash then everything works as expected.

it doesn't seem like the file actually needs the header on a quick check

@kilograham Adjusted the pull request to remove the unneeded includes. Added a compilation test to show the before and after state. I don't know if you prefer to have tests like that in the tree. Let me know if not and I can remove it.

merged into develop


Client builders for REST APIs

First commit is boring dependency updating. The meat is in the second commit.

(Will look into the CI issue)

Fixed rebase conflict - will merge on green


Hot Fix: MKDocs build documentation workflow Permissions Issue

Description
Not a release, but mainly fixing MKDocs documentation build workflow issues.
MKDocs documentation build workflow was having issues pushing to gh-pages branch and was getting permissions denied because we set its permissions to read-all and it did not have the ability to write.
Changes

giving MKDocs build documentation CD ability to read from main and push to gh-pages

I would have liked to given it more specific permissions for each branch, but GitHub doesn't seem to support that for now, so write-all I think is good for now

trunk was complaining about write-all permission, and I told it to ignore it

while we are updating the documentation, I merged in the [upgrade deserialization docs](Upgrade Deserialization Docs) to knock out 2 birds with 1 PR

Tests
Known Issues
Notes
Checklist

[ ] My name is on the list of contributors (CONTRIBUTORS.md) in the pull request source branch.
[x] I have updated the documentation to reflect my changes.

Don't forget to not squash

Don't forget to not squash

don't worry I forever remember that one now. I even made an internal documentation for it.
https://github.com/C-Accel-CRIPT/Python-SDK/wiki/Deploy-to-Pypi


Hotfix overflowing game width

Fix a bug where if the aspect ratio of the browser was more narrow than the aspect ratio of the game, the game would overflow horizontally.

Pull Request Test Coverage Report for Build 486

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 24.611%

Totals

Change from base Build 483:
0.0%

Covered Lines:
769

Relevant Lines:
2639

💛  - Coveralls


How to properly contribute API data?

Hi there,
my Vaillant system comes with these components:

sensocomfort VRC 702
aguaflow (hot drinking water using the heating water tank as a heat source)
auroflow (solar thermal heating the water tank)
flexotherm (heat pump heating the water tank)

The home assistant integration is missing several sensors that are present in the myVaillant app.
I'd love to help adding those to your library. Can you add a guide on how to identify the strings you need for that and how to do the MITM thing?

Fwiw, running myPyllant.tests.generate_test_data returns nothing, no error message and no JSON result.
Running myPyllant.export results in
Traceback (most recent call last):
  File "/usr/lib/python3.10/runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.10/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "/home/hanno/.local/lib/python3.10/site-packages/myPyllant/export.py", line 93, in <module>
    asyncio.run(main(**kwargs))
  File "/usr/lib/python3.10/asyncio/runners.py", line 44, in run
    return loop.run_until_complete(main)
  File "/usr/lib/python3.10/asyncio/base_events.py", line 646, in run_until_complete
    return future.result()
  File "/home/hanno/.local/lib/python3.10/site-packages/myPyllant/export.py", line 63, in main
    async for system in api.get_systems():
  File "/home/hanno/.local/lib/python3.10/site-packages/myPyllant/api.py", line 262, in get_systems
    system = System(
  File "/home/hanno/.local/lib/python3.10/site-packages/myPyllant/models.py", line 161, in __init__
    self.circuits = [Circuit(system_id=self.id, **c) for c in self._merged_circuits]
  File "/home/hanno/.local/lib/python3.10/site-packages/myPyllant/models.py", line 161, in <listcomp>
    self.circuits = [Circuit(system_id=self.id, **c) for c in self._merged_circuits]
  File "/home/hanno/.local/lib/python3.10/site-packages/pydantic/main.py", line 150, in __init__
    __pydantic_self__.__pydantic_validator__.validate_python(data, self_instance=__pydantic_self__)
pydantic_core._pydantic_core.ValidationError: 1 validation error for Circuit
min_flow_temperature_setpoint
  Field required [type=missing, input_value={'system_id': '00000000-0...ack_mode_enabled': True}, input_type=dict]
    For further information visit https://errors.pydantic.dev/2.1.2/v/missing

(system_id value removed from trace)

@hzulla thanks for wanting to contribute. I made that missing field in your error message optional in the latest version, would you mind rerunning export + generate_test_data?
You should be able to contribute data by following the steps in https://github.com/signalkraft/myPyllant#contributing and then running generate_test_data. The README also explains where the test data pops up in the filesystem: https://github.com/signalkraft/myPyllant#contributing-test-data

Okay, here it is.
cf6a60bfab7c00bc4bd13cc80618315d9cec2129.zip

@hzulla was there another folder with test data after running the command? The one you uploaded looks like the one I generated from my own setup.
If you pull the latest version and rerun the command, it'll now print the name of the folder.

Indeed, that's your directory from the git repo.
Calling python3 -m myPyllant.tests.generate_test_data [myuser] [mypw] germany vaillant
No error message. No directory being created.

@hzulla the latest version needs a --country argument and should output a path. Here is what it does on my machine (with the latest version on the main branch):
$ python3 -m myPyllant.tests.generate_test_data user password vaillant --country germany
Wrote test data to /home/user/Development/myPyllant/src/myPyllant/tests/json/56f4c513b87875849a089b83cafbe6fdd6020b55

Could you check again if you're on the latest one?

Okay, it's definitely helpful to actually read the instructions and do as told in there. Sorry for wasting your time.
Here's the actual output with the latest version of the software (with some serial numbers and UUIDs redacted).
d3a0d9a2f8d536dc37b3237c3a552cb74a4104e2.zip

hi @signalkraft my system also has issues. I attached my configuration too:
17732fb10772dccf2d0451bff2b4992c38d2c7ec.zip
Thanks a lot for the great work!
When I run the sample my output is:
desired_room_temperature_setpoint_cooling
  Field required [type=missing, input_value={'system_id': '8545c5f0-f...iated_circuit_index': 0}, input_type=dict]
    For further information visit https://errors.pydantic.dev/2.3/v/missing
quick_veto_start_date_time
  Field required [type=missing, input_value={'system_id': '8545c5f0-f...iated_circuit_index': 0}, input_type=dict]
    For further information visit https://errors.pydantic.dev/2.3/v/missing
quick_veto_end_date_time
  Field required [type=missing, input_value={'system_id': '8545c5f0-f...iated_circuit_index': 0}, input_type=dict]
    For further information visit https://errors.pydantic.dev/2.3/v/missing


Nicer date widget (fixes #155), switch Date.today for current (fixes #162)

This switches the date time ranges on event to use the bootstrap date picker. It also makes sure that the events page gets prettier error messages and that all date range related errors are correctly gathered for the event.
The datepicker is now enabled on all pages for all elements matching [data-provide="datepicker"]. Call enableDatepickers() if you need to enable new pickers at a time later than page load.
It also replaces all occurences of Date.today with Date.current.

Coverage decreased (-0.3%) to 99.735% when pulling c0dc13b1fbff28611a0a3e74dca92e7b62b20b10 on 155_fix_event_range_datepicker into 01b4be22497f5dc6bbc97d3b4a454ad88270a61b on dev.

Coverage decreased (-0.3%) to 99.735% when pulling 11fccb1e6944e216d7efa5e51720bf146d17758d on 155_fix_event_range_datepicker into 01b4be22497f5dc6bbc97d3b4a454ad88270a61b on dev.

Coverage decreased (-0.3%) to 99.737% when pulling 17abc124893b948fd92fb127fbcd4c256f819278 on 155_fix_event_range_datepicker into 01b4be22497f5dc6bbc97d3b4a454ad88270a61b on dev.

Coverage remained the same at 100.0% when pulling 1a9919788f8e226db5bd6b2df8eaf42ab202019a on 155_fix_event_range_datepicker into 01b4be22497f5dc6bbc97d3b4a454ad88270a61b on dev.

Coverage remained the same at 100.0% when pulling 6b29e82e3f024815e542f797fb3544be74f67e92 on 155_fix_event_range_datepicker into 01b4be22497f5dc6bbc97d3b4a454ad88270a61b on dev.

Coverage remained the same at 100.0% when pulling c40bb0f89baf7bb3c16666dd337da2373cbcf7bc on 155_fix_event_range_datepicker into 80f71c57f60f753d343d8278a9b64b5b988044f6 on dev.

Coverage remained the same at 100.0% when pulling 9e78c428fb653eb5b2aecc647cf0f85738523e61 on 155_fix_event_range_datepicker into 80f71c57f60f753d343d8278a9b64b5b988044f6 on dev.

Coverage remained the same at 100.0% when pulling cbd3e012f4e032a6a66ab8c83f57e5f19497c572 on 155_fix_event_range_datepicker into 80f71c57f60f753d343d8278a9b64b5b988044f6 on dev.

Coverage remained the same at 100.0% when pulling 0ca698a8f43825920807448b861cb288a4c45b1c on 155_fix_event_range_datepicker into 937687fb1033cfb3507f63fb4aeb49ffd2f34ddf on dev.


[FEATURE]: ability to log slow queries

Describe what you want
the logger method could get the info about the execution time of each query. This way I could log queries as debug and slow queries as warning.
I propose that logQuery() would accept a third parameter, an object that would contain executionTime property.
export interface Logger {
	logQuery(query: string, params: unknown[], {executionTime} ): void;
}

The logQuery method is invoked on query execution, not after.
Maybe add another method like afterQueryExecution(executionTime: int): void?

Closing in favor of #2605.


Fix typo in date

Type of changes

[ ] Bug fix
[ ] New feature
[x] Documentation / docstrings
[ ] Tests
[ ] Other

Checklist

[x] I've run the latest black with default args on new code.
[x] I've updated CHANGELOG.md and CONTRIBUTORS.md where appropriate.
[x] I've added tests for new code.
[x] I accept that @willmcgugan may be pedantic in the code review.

Description
Typo fix

Good spot, but now the screenshots would not match the code. Honestly I don't think its worth the effort to fix those!


Previews for video files

Some random frame from the video would be enough.
It can be useful to overlay also duration of the video and quality.

Necessary to extend PreviewGenerators object (#122).


[Compute] az vm list-skus: Fix display problem of table format

Description

https://github.com/Azure/azure-cli/issues/13079
Table output is too long to fit in screen.
Delete "capabilities" column.
Before:

After:

Testing Guide

az vm list-skus -l westus2 -o table
History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

Compute


Unable to compile with Typescript 4.5.4

I recently upgraded the version of typescript in a react project to version 4.5.4.
When I did this I got an compile error reported in react-dropzone-uploader.
I am using version 2.11.0 of react-dropzone-uploader
In Typescript 4.4 the typing of the catch variable was changed to be "unknown" from "any". As a result the error was reported.
The error I get is:
node_modules/react-dropzone-uploader/dist/Dropzone.tsx:504:44 - error TS2571: Object is of type 'unknown'.
The code in Dropzone.tsx is:
  uploadFile = async (fileWithMeta: IFileWithMeta) => {
    const { getUploadParams } = this.props
    if (!getUploadParams) return
    let params: IUploadParams | null = null
    try {
      params = await getUploadParams(fileWithMeta)
    } catch (e) {
      console.error('Error Upload Params', e.stack)
    }
    if (params === null) return
    const { url, method = 'POST', body, fields = {}, headers = {}, meta: extraMeta = {} } = params
    delete extraMeta.status

The problem is the catch variable "e" is now typed as "unknown" and so it is an error to attempt to access a member of "unknown"
Typescript has introduced an option "useUnknownInCatchVariables" which when set to "false" will allow the code to compile and that is how I am getting around the problem.
A suggestion has been made to me that I need to wrap my call to react-dropzone-uploader in a try-catch to solve my problem.
That will not solve my issue. My issue is I cannot compile my project.
The person making the suggestion seemed to believe that the try-catch in question was in my code and that I should fix my code.
The problem lies in the Dropzone.tsx file of react-dropzone_uploader. And the fix is very easy:
    try {
      params = await getUploadParams(fileWithMeta)
    } catch (e) {
      console.error('Error Upload Params', (e as any).stack)
    }

Hi, @GSuggitt have you resolved this issue on your end?? I'm getting the same issue and I haven't reached any solution yet other than setting useUnknownInCatchVariables  as false in tsconfig file.

I have had to set "useUnknownInCatchVariables" to "false".
For some unknown reason, the authors are adding the tsx files to the distribution (which is not at all necessary). And, of course, Typescript chases down all the "imports", crashes into their offending TSX file and stops.

From: Muhammad Haziq @.>
Sent: Tuesday, January 25, 2022 3:02 AM
To: fortana-co/react-dropzone-uploader @.>
Cc: Gerry Suggitt @.>; Mention @.>
Subject: Re: [fortana-co/react-dropzone-uploader] Unable to compile with Typescript 4.5.4 (Issue #179)
Hi, @GSuggitthttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_GSuggitt&d=DwMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=78qlPfTEsFT74NuHbvqf78TUZ56uV693-8p2ueiPANY&m=UAacYejPdGBZ_PdTvW1v_egC8Ne7sSe_mwII-Z2adNw&s=yfMdTGnzq4nhWB_zQ-sKtp9l1nOuJ7ZCJVsL63OD67E&e= have you resolved this issue on your end?? I'm getting the same issue and I haven't reached any solution yet other than setting useUnknownInCatchVariables as false in tsconfig file.
—
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_fortana-2Dco_react-2Ddropzone-2Duploader_issues_179-23issuecomment-2D1020906574&d=DwMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=78qlPfTEsFT74NuHbvqf78TUZ56uV693-8p2ueiPANY&m=UAacYejPdGBZ_PdTvW1v_egC8Ne7sSe_mwII-Z2adNw&s=QFoZWZds8YftgU5HSw7Y7fz_UBWg9x2WqxuGlSwiFAw&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AB4O6UGWFVGVIWFHOBUXL73UXZKJHANCNFSM5L4FACCA&d=DwMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=78qlPfTEsFT74NuHbvqf78TUZ56uV693-8p2ueiPANY&m=UAacYejPdGBZ_PdTvW1v_egC8Ne7sSe_mwII-Z2adNw&s=L806UzbEFnkF36Yi4Q3m6kOGCFdFTtZbgqZB_JMLStw&e=.
Triage notifications on the go with GitHub Mobile for iOShttps://urldefense.proofpoint.com/v2/url?u=https-3A__apps.apple.com_app_apple-2Dstore_id1477376905-3Fct-3Dnotification-2Demail-26mt-3D8-26pt-3D524675&d=DwMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=78qlPfTEsFT74NuHbvqf78TUZ56uV693-8p2ueiPANY&m=UAacYejPdGBZ_PdTvW1v_egC8Ne7sSe_mwII-Z2adNw&s=GYUNBI7sfUJsG_poki7keWI1RnpM5f8Ywu23y_NlBAY&e= or Androidhttps://urldefense.proofpoint.com/v2/url?u=https-3A__play.google.com_store_apps_details-3Fid-3Dcom.github.android-26referrer-3Dutm-5Fcampaign-253Dnotification-2Demail-2526utm-5Fmedium-253Demail-2526utm-5Fsource-253Dgithub&d=DwMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=78qlPfTEsFT74NuHbvqf78TUZ56uV693-8p2ueiPANY&m=UAacYejPdGBZ_PdTvW1v_egC8Ne7sSe_mwII-Z2adNw&s=VSVw7VFLq5gAT2-5AXYgp1DkRX4_VH892Ikn7YJNqf8&e=.
You are receiving this because you were mentioned.Message ID: @.***>

Is there an update on this? Is this project still even active? This seems like such a simple fix that for such a widely used project it should've been fixed by now... no offense intended, but would like to see an update of any kind.

Note that setting useUnknownInCatchVariables to false isn't the real fix. Like @GSuggitt said, the problem is tsx files are published to dist. I wouldn't modify your tsconfig to get around this, since you likely want those errors. For now, I just delete the tsx files manually.
https://github.com/fortana-co/react-dropzone-uploader/issues/108#issuecomment-1083621257