Initial commit of LocalResourceManagerHelper and tests

I added a bunch of minor comments, this is really an amazing and much-much-needed job!

Thanks for the first pass @mziccard!  I've pushed another commit taking into account the comments.  A few notes on my changes:

I left in changeCreateTime and changeProjectNumber in case the user wants to have some autogenerated values and some non-autogenerated (explained more fully in a response to the original comment).
I decided not to use project's clone function because it creates a deep copy, which is problematic when users use ImmutableMap (because ImmutableMap doesn't have a public constructor).
I agree that the testing filename may need to change, but I haven't decided what to call the test file yet.  It's half rpc-level tests and half LocalResourceManagerHelper tests, but other modules don't have dedicated spi layer tests.  Once I merge in the api-level ResourceManagerImpl (in a future PR), I could potentially change the direct RPC calls and then I could split the tests in this file up to be ResourceManagerImplTest and LocalResourceManagerHelperTest.  To get at how the mock handles the different requests, I think I'll inevitably be testing either the rpc layer or the api layer as a by-product because a lot of the functionality is done by private methods.

Half way through. Will complete the review tomorrow.

@aozarov added a commit to incorporate your comments thus far

Finished with reviewing the other half.

@aozarov incorporated your comments, ready for another look


Performance fixes on demo store neue

Description

Make sure image is not lazy loaded for hero images
Default to eager load first 4 images in a product grid
Put no search result recommendations in a different query so that it doesn't get queried when there are search results

Before submitting the PR, please make sure you do the following:

[ ] Read the Contributing Guidelines
[ ] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123)
[ ] Update docs in this repository according to your change
[ ] Run yarn changeset add if this PR cause a version bump based on Keep a Changelog and adheres to Semantic Versioning

fyi seeing "loading is not  defined" https://ste9jc11w-9ab02eb2ed45627c8573.myshopify.dev/


iPhone number keyboard no period

Steps to Reproduce

TextField with keyboardType: TextInputType.numberWithOptions(decimal: true)
physical iOS device with version 12.1.4
There is only a comma in the bottom left but should have a period

Logs
Logs are too long but contains no error

admin:Money_Leveling admin$ flutter analyze
Analyzing Money_Leveling...
No issues found! (ran in 3.8s)

admin:Money_Leveling admin$ flutter doctor -v
[✓] Flutter (Channel stable, v1.2.1, on Mac OS X 10.14.2 18C54, locale en-SE)
    • Flutter version 1.2.1 at /Users/admin/Documents/flutter
    • Framework revision 8661d8aecd (5 weeks ago), 2019-02-14 19:19:53 -0800
    • Engine revision 3757390fa4
    • Dart version 2.1.2 (build 2.1.2-dev.0.0 0a7dcf17eb)

[!] Android toolchain - develop for Android devices (Android SDK version
    28.0.3)
    • Android SDK at /Users/admin/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native
      profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_152-release-1136-b06)
    ! Some Android licenses not accepted.  To resolve this, run: flutter
      doctor --android-licenses

[✓] iOS toolchain - develop for iOS devices (Xcode 10.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.1, Build version 10B61
    • ios-deploy 1.9.4
    • CocoaPods version 1.5.3

[✓] Android Studio (version 3.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 31.1.1
    • Dart plugin version 181.5656
    • Java version OpenJDK Runtime Environment (build
      1.8.0_152-release-1136-b06)

[✓] Connected device (1 available)
    • Vanessas • 5a8e4bf5a531a5209fe285d8b0c5d198406ba001 • ios • iOS
      12.1.4

! Doctor found issues in 1 category.

I just tried it on the iOS Simulator and I see a period and no comma in the keyboard.  Could it be because of your locale?
Here's my full code:
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: Scaffold(
        body: Center(
          child: Column(
            mainAxisAlignment: MainAxisAlignment.center,
            children: <Widget>[
              TextField(
                keyboardType: TextInputType.numberWithOptions(decimal: true),
              ),
            ],
          ),
        ),
      ),
    );
  }
}

It is working fine on emulators it's just on that device I tested on, which is using the Swedish local (sv_SE).

It looks like Swedish uses a comma as a decimal separator, so that should be correct for that locale: https://en.wikipedia.org/wiki/Decimal_separator#Countries_using_Arabic_numerals_with_decimal_comma
I'm closing this, but let me know if that's not the issue.

The comma is supposed to be there giving the locale. This issue is with Dart that doesn't recognise the comma as a decimal separator. So there needs to be some sort of check to see if there is a comma in the input and then internally transform it to a period for further parsing. Any suggestions?


HSEARCH-945 updating to latest Arquillian dependencies and setup

HSEARCH-945 updating to latest Arquillian dependencies and using the Arquillian bom to get compatible version for Arquillian dependencies. Also added new required ShrinkWrap dependencies which needs to be included after the latest changes in their API

pulling this


Remove Button Outlines

Is your feature request related to a problem? Please describe.
Buttons on most website don't have this outline. I think it would look much better if it was removed.

i believe this has been fixed


modal with navigation

Hi there,
i am trying to use your modalNav page which works half way>
When i push a page the old one get destroyed in the stack so the back button functionality is broken
log error is :  Uncaught (in promise): navigation stack needs at least one root page
Appart from ModalWithNavigationPage, ModalNavPage and MoveDocumentPage, is there any thing i have missed ?

Well nevermind...
I had my click event to push the new page in on a button, inside a form (calling the submit function).
That function had this.viewCtrl.dismiss({data}) in it... :/
Thanks for this repo ! it is extremely helpful

Glad you got it solved! Thanks a lot for using the repo :)


Global Connection is not replaced and does not close requests

The global connection doesn't seem to replace requests after there is an error. I'm handling the error like this
  mssql.on('error', function(err) {
    logger.error('Connection Error:', err);
  });

Then I kill the outgoing tcp connection and I get this connection error:
{ name: 'ConnectionError',
18:39:18   message: 'Connection lost - read ECONNRESET',
18:39:18   code: 'ESOCKET' }

After that it doesn't seem to retry any connections and just hangs forever. I have my requestTimeout set to Infinity since our requests take a long time.
Then I tried to close the global connection
  mssql.on('error', function(err) {
    logger.error('Connection Error:', err);
    return cb(err);
  });

function(callback) {
  mssql.close(callback);
}

This doesn't seem to ever call the callback and hangs also. Without the callback, it returns, but the connection and the request don't seem to be closed, although I haven't tried with promises. I'm expected close to kill all requests so I can attempt the connection and requests again.

this is for version 2.3.2

Promises don't seem to return either.

Fixed in 3.0.1.


FreeBusy Bad Request error

Hello 👋
I am trying to run the freebusy_request method on a calendar, but I am getting this error all the time
ReportError at '400 Bad Request
b''', reason no reason

This is how my code look like
with caldav.DAVClient(
        url=caldav_url,
        username=username,
        password=password,
    ) as client:

my_principal = client.principal()

my_calendar = my_principal.calendar(name="My Calendar")

start = datetime.today()
end = datetime.today() + timedelta(days=1)

free_busy = my_calendar.freebusy_request(start=start, end=end)

Any idea what I am doing wrong here?

Most likely the server does not support freebusy.  What server are you using?

Ah that's helpful, I am using the icloud servers https://caldav.icloud.com

Not supported in icloud.
I think that there is no official support for the caldav protocol for icloud.
RFC6638 specifies a different way to do freebusy requests, but if I remember correct someone tested that and did not get it to work.

Ah I see 🤔 I am trying to create an api to sync icloud calendars, and caldav seems to be the only option at the moment. I think for now I can live without the free/busy functionality


Empty commits

I am using the ImageUpdateAutomation resource and see empty commits being pushed to my git repository. I looked at a similiar issue (#135) but they got an empty commit every minute, where I get them randomly, so I believe it is a different issue.
I have had the issue since 20. april. I was using flux v0.12.1 back then, and now I use latest version v0.13.4 and I still get them.
20/4/2021

16/5/2021

---
apiVersion: image.toolkit.fluxcd.io/v1alpha2
kind: ImageUpdateAutomation
metadata:
  name: flux-system
  namespace: flux-system
spec:
  sourceRef:
    kind: GitRepository
    name: flux-system
  git:
    checkout:
      ref:
        branch: main
    commit:
      author:
        name: fluxcdbot
        email<EMAIL_ADDRESS>      messageTemplate: 'update manifests'
  interval: 1m0s

Any chance I can see the git repo you're using? I understand if it contains sensitive data and you'd prefer not. You might have to help me out with narrowing down where the problem happens ..

The git repository is a private GitHub repository but I can share it with you. I would prefer to mail you a zip file but I guess I could just add you as a contributor.

I would prefer to mail you a zip file but I guess I could just add you as a contributor.

Either works for me -- you can email a zip to the address on my profile, or<EMAIL_ADDRESS>

I'm having this same problem as well. Same behavior on Flux2 v0.13.4 and v0.14.2.
I haven't built or put a new image in this AWS ECR repo since last week. I just updated to Flux's latest and reapplied my kustomization to continue testing today.
My image automation intervals are set to every 2m, so it seems that at each time of reconciliation, it's making a commit at that interval even if it's empty/unneeded because there's no update to the image tag.
I'm using BitBucket.
My Kustomization:
apiVersion: kustomize.toolkit.fluxcd.io/v1beta1
kind: Kustomization
metadata:
  name: www-test-eks-com-v1
  namespace: backend
spec:
  images:
  - name:<PHONE_NUMBER>00.dkr.ecr.us-east-1.amazonaws.com/www-test-eks-com-v1
    newTag: 20210527-223554 # {"$imagepolicy": "flux-system:www-test-eks-com-v1:tag"}
  interval: 10m0s
  path: ./flux/kustomize/apps/www-test-eks-com-v1
  prune: true
  sourceRef:
    kind: GitRepository
    name: flux-system
    namespace: flux-system
  validation: client
  healthChecks:
  - apiVersion: apps/v1
    kind: Deployment
    name: www-test-eks-com-v1
    namespace: backend
  timeout: 2m

Excerpt from the relevant deployment.yaml:
      containers:
      - name: www-test-eks-com-v1
        image:<PHONE_NUMBER>00.dkr.ecr.us-east-1.amazonaws.com/www-test-eks-com-v1:20210505-233849

Image Automation objects:
---
apiVersion: image.toolkit.fluxcd.io/v1alpha2
kind: ImageUpdateAutomation
metadata:
  name: www-test-eks-com-v1
  namespace: flux-system
spec:
  git:
    commit:
      messageTemplate: |
        Automated image update

        Automation name: {{ .AutomationObject }}

        Files:
        {{ range $filename, $_ := .Updated.Files -}}
        - {{ $filename }}
        {{ end -}}

        Objects:
        {{ range $resource, $_ := .Updated.Objects -}}
        - {{ $resource.Kind }} {{ $resource.Name }}
        {{ end -}}

        Images:
        {{ range .Updated.Images -}}
        - {{.}}
        {{ end -}}
      author:
        email<EMAIL_ADDRESS>        name: fluxcdbot
  interval: 2m0s
  sourceRef:
    kind: GitRepository
    name: flux-system
    # namespace: flux-system
  suspend: false
---
apiVersion: image.toolkit.fluxcd.io/v1alpha2
kind: ImageRepository
metadata:
  name: www-test-eks-com-v1
  namespace: flux-system
spec:
  image:<PHONE_NUMBER>00.dkr.ecr.us-east-1.amazonaws.com/www-test-eks-com-v1
  interval: 2m0s
  secretRef:
    name: ecr-credentials
---
apiVersion: image.toolkit.fluxcd.io/v1alpha2
kind: ImagePolicy
metadata:
  name: www-test-eks-com-v1
  namespace: flux-system
spec:
  filterTags:
    pattern: '((?P<timestamp>\d{8}-\d{6})|(?P<timestamp>\d{8}-\d{6}-\d{1}))'
    extract: '$timestamp'
  imageRepositoryRef:
    name: www-test-eks-com-v1
  policy:
    alphabetical:
      order: asc

Flux CLI output concerning images:
❯ flux get images all
NAME                                    READY   MESSAGE                         LAST SCAN                       SUSPENDED
imagerepository/www-test-eks-com-v1     True    successful scan, found 3 tags   2021-06-03T13:00:30-07:00       False

NAME                            READY   MESSAGE                                                                                LATEST IMAGE
imagepolicy/www-test-eks-com-v1 True    Latest image tag for '000000000000.dkr.ecr.us-east-1.amazonaws.com/www-test-eks-com-v1' resolved to: 20210527-223554<PHONE_NUMBER>00.dkr.ecr.us-east-1.amazonaws.com/www-test-eks-com-v1:20210527-223554

NAME                                            READY   MESSAGE                                                                 LAST RUN                        SUSPENDED
imageupdateautomation/www-test-eks-com-v1       True    committed and pushed 6421cfd32363cac84bd70b6fa2a8e0a4402a7845 to master 2021-06-03T13:00:45-07:00       False

Logs - Image Automation Controller:
{"level":"info","ts":"2021-06-03T19:51:44.079Z","logger":"controller-runtime.manager.controller.imageupdateautomation","msg":"Starting workers","reconciler group":"image.toolkit.fluxcd.io","reconciler kind":"ImageUpdateAutomation","worker count":1}
{"level":"info","ts":"2021-06-03T19:51:59.653Z","logger":"controller-runtime.manager.controller.imageupdateautomation","msg":"pushed commit to origin","reconciler group":"image.toolkit.fluxcd.io","reconciler kind":"ImageUpdateAutomation","name":"www-test-eks-com-v1","namespace":"flux-system","revision":"fa68e0a261ad6aa9b8cb4ecc8db89cc68f07326f","branch":"master"}
{"level":"info","ts":"2021-06-03T19:52:08.908Z","logger":"controller-runtime.manager.controller.imageupdateautomation","msg":"pushed commit to origin","reconciler group":"image.toolkit.fluxcd.io","reconciler kind":"ImageUpdateAutomation","name":"www-test-eks-com-v1","namespace":"flux-system","revision":"8e214093bf5e9870ea18319ff63032e000569296","branch":"master"}
{"level":"info","ts":"2021-06-03T19:54:12.551Z","logger":"controller-runtime.manager.controller.imageupdateautomation","msg":"pushed commit to origin","reconciler group":"image.toolkit.fluxcd.io","reconciler kind":"ImageUpdateAutomation","name":"www-test-eks-com-v1","namespace":"flux-system","revision":"c8c8dbd510859b8c8202eef437841693d37b8641","branch":"master"}
{"level":"info","ts":"2021-06-03T19:56:24.110Z","logger":"controller-runtime.manager.controller.imageupdateautomation","msg":"pushed commit to origin","reconciler group":"image.toolkit.fluxcd.io","reconciler kind":"ImageUpdateAutomation","name":"www-test-eks-com-v1","namespace":"flux-system","revision":"89e4da1fe3221f66222ff32af1120061ab7859bc","branch":"master"}

Events - Image Automation controller:
Events:
  Type    Reason  Age    From                         Message
  ----    ------  ----   ----                         -------
  Normal  info    15m    image-automation-controller  committed and pushed change 8b49c700a03b516d5796b267724a66d85b483d60 to master
  Normal  info    13m    image-automation-controller  committed and pushed change a98daef1c988002f452fa1e1c39986c7dd9310ca to master
  Normal  info    11m    image-automation-controller  committed and pushed change 41a74a71939bdb116a1682d757614d34b9d540ef to master
  Normal  info    9m4s   image-automation-controller  committed and pushed change fe2b439a906fc48b7771cbed71130fa0da253e18 to master
  Normal  info    8m1s   image-automation-controller  committed and pushed change efc761e63bd7544f81275bb8437f5cd47992518a to master
  Normal  info    7m13s  image-automation-controller  committed and pushed change d08486200e77261161d2d3eb44c07b0a19ce8e03 to master
  Normal  info    7m9s   image-automation-controller  committed and pushed change 1371b7876e1de0adb49e9581a44b5324e9f603dc to master
  Normal  info    7m3s   image-automation-controller  committed and pushed change 03cf55b6acc038d8a2b0fdb193138bac5399f49c to master
  Normal  info    6m1s   image-automation-controller  committed and pushed change fa68e0a261ad6aa9b8cb4ecc8db89cc68f07326f to master
  Normal  info    5m52s  image-automation-controller  committed and pushed change 8e214093bf5e9870ea18319ff63032e000569296 to master
  Normal  info    3m48s  image-automation-controller  committed and pushed change c8c8dbd510859b8c8202eef437841693d37b8641 to master
  Normal  info    96s    image-automation-controller  committed and pushed change 89e4da1fe3221f66222ff32af1120061ab7859bc to master

BitBucket's commit page:

I'm having this same problem as well

Sorry to hear that @chicken231 , and thank you for all the environmental info. Is it possible to fork your repo, or to otherwise get access to it (keeping it private of course)?

@Nicklason I'm sure you would have mentioned it, but just in case: is there anything that appears to trigger the empty commits; e.g., making a commit yourself to the same branch, or restarting the controller, or a particular image getting a new tag?
Also: do you have any symlinks or other non-regular files/directories in your git repo?

@squaremo
I can't give you access to this cluster as it's for work (AWS EKS). Was the info and YAMLs I provided insufficient to reproduce or attempt to reproduce? Would anything short of giving you a copy of the whole repo be acceptable?
What I can do this weekend is package up all the relevant YAMLs relating to this deployment and image automation and test it in my bare metal lab cluster. Different container registry, different git repo provider, different k8s version, etc. If it exhibits the same symptoms, would providing those relevant YAMLs to you so you can kubectl apply -f be sufficient? I'd throw them all in one file for easy apply/delete.

As for your note to the OP, it seems every reconciliation loop (at every automation interval), it creates and pushes a commit no matter what. I'm 90% sure the same occurs if I attempt a reconciliation via the flux cli (flux reconcile image update www-test-eks-com-v1). I will be able to confirm that this weekend.
I have 0 symlinks in my entire repo. I just verified this with certainty on my end.
Thanks for any help! If we can get this fixed, I can roll out Flux for my CD.

I can't give you access to this cluster as it's for work (AWS EKS).

Fair enough, that's the usual situation -- it's worth asking just in case :-)

Was the info and YAMLs I provided insufficient to reproduce or attempt to reproduce?

A lot of the behaviour of the image-automation-controller -- especially buggy behaviour -- depends on the contents of the git repo. For example, the problem in #164 depended on a particular interleaving of commits, and #135 involved broken symlinks in the repo (thus the question about symlinks!). As a result, it's tricky to see what's really going on given just the Kubernetes configuration.

What I can do this weekend is package up all the relevant YAMLs relating to this deployment and image automation and test it in my bare metal lab cluster. Different container registry, different git repo provider, different k8s version, etc. If it exhibits the same symptoms, would providing those relevant YAMLs to you so you can kubectl apply -f be sufficient?

Yes for sure -- or slightly better, if you can make a github repo with the reduced/redacted files, I can fork it and go from there.

it seems every reconciliation loop (at every automation interval), it creates and pushes a commit no matter what.

That sounds like #135, which produced a commit every loop because go-git mistakenly reported a broken symlink as modified. So perhaps there's some other similar thing going on.

@Nicklason I'm sure you would have mentioned it, but just in case: is there anything that appears to trigger the empty commits; e.g., making a commit yourself to the same branch, or restarting the controller, or a particular image getting a new tag?
Also: do you have any symlinks or other non-regular files/directories in your git repo?

I am not sure if it has something to do with me working on the repository or restarting flux pods, it just seems random. The repository shouldn't have any symbolic links either.

@Nicklason Naively, I ran the controller against the definitions in your repo, to see if I could get empty commits. I've been running it for a day and have not seen any. I notice you see around one or two empty commits a day, but I will need to run it longer to have confidence that I'm failing to reproduce the problem (something something poisson distribution). In the meantime, I don't see any smoking guns in the repo -- it all looks innocuous 🤔.
The next step might be for you to run an image with lots of tracing output, so we can see what it thinks has changed. If you're OK to do that, I'll put one together ..

@squaremo Yes I still want to help. If you make me an image I'll run it. It might be a good idea to add debugging to all the images anyway 😃

I wasn't able to reproduce empty commits in my lab (Gitlab repo, Gitlab registry, Flux v0.15.2). It works like a charm. The only differences in my manifests pasted above between my two environments is the object names and using SemVer image policy.
I'll update if anything new becomes relevant from my end, or if I can help in testing with this special image.

The image docker.io/squaremo/image-automation-controller:0a9c9a7437629093349eb3d3c77f8d9e5d6fba7c is built from https://github.com/fluxcd/image-automation-controller/pull/190. It outputs a bunch of tracing to the log at debug level -- if you use it, you'll need to set --log-level=debug in the controller deployment.
Among the tracing:

which files are added to the commit, and which are ignored
if any policies result in changing a file

By definition, it cannot be changing files if the result is an empty commit; but it's difficult to know what else to trace. Suggestions encouraged :-)

@squaremo  You need to build it for arm64 :)

@squaremo I tested the image you provided with debug logs enabled in my x86 EKS cluster. Current Flux version = 0.15.3
Here is the log output with a few redactions: https://pastebin.com/DmmCajbD
An interesting excerpt from the logs that it records before creating the empty commit:
"old":"20210527-223554","new":"20210527-223554"

That smells like the comparison thinks it is looking at different types, e.g. the old value being a interpreted as a string, but the new value as an integer, or vice-versa.

I find it hard to get anything useful from the logs. I have quite a lot of image policies and most of the log messages are about them and are repeated every minute. I think I would have to set up a notification for when a commit is made in order to get a chance to see the relevant logs.
These are the messages I keep seeing:
scanning files
reading file
found policy
found schema extension
applying setter
As @chicken231 said, I also see "old" and "new" being the same. The "old" and "new" are just links to a docker image. I don't know what it means but I would guess that it is used to check if there is a new tag matching the image policy, but you can correct me on that.

I find it hard to get anything useful from the logs

That is the nature of log messages at this level -- I don't know what I'm looking for, so I output everything I can think of. I was hoping that there would be a noticeable pattern of trace messages before a commit which turns out to be empty -- given the commit ref, you can grep the log for it and look at what came immediately before (and compare with a commit that isn't empty). But it's a shot in the dark, really -- I rather suspect there's some environmental cause, rather than a particular path through the code.
The trace message about applying a setter with "old" and "new" values is output whether or not the values differ (the filter sets the value unconditionally and the log follows). There's a message about finding a setter mark, so it would make sense to restrict the trace message to just when it's changing the value.

That is the nature of log messages at this level

Yes I understand. Sorry my last comment came out a bit wrong.

I rather suspect there's some environmental cause

I think you are right. It looks like I have indirectly solved the issue - I have not had empty commits for a week (since 9/7/2021). The cluster consists of 6 network booted raspberry pi and a week ago (same day as last empty commit) I migrated to a new NAS, so it looks like the NAS was the problem.
I should have looked closer at the logs before moving to a new NAS and I should have included information about the environment when I made the issue, but I just didn't suspect that the NAS / filesystem would affect the controller.

I tried switching ImagePolicy policies with different RegExs, SemVer, but the empty commits kept coming.
I also was able to reproduce this in my lab, simply by deleting the container registry (GitLab) associated with an Image automation. The next day I noticed I had 500+ empty commits.
I'm on the latest Flux v0.17.1.
Is there anything I can do from my end to help get this fixed?

I also was able to reproduce this in my lab, simply by deleting the container registry (GitLab) associated with an Image automation. The next day I noticed I had 500+ empty commits.

To clarify: you stopped running the image registry that hosts images used by the automation?

@squaremo This was a month or more ago so my memory is fuzzy.
I changed my username on GitLab.
So I may have spoken incorrectly, I can't recall at what point in time these empty commits occurred, if it was after I deleted all the container images in the registry, or after I did that and after I changed username (then the endpoint should have returned a 404 I'd think). Either way I don't think it should be pushing a commit unless a newer image is actually detected.

That would be super helpful! Thank you for putting time into this :pray:

I'm happy to report that I can no longer reproduce this issue.
Flux version 0.24.1. image-update-automation controller version 0.14.1 (manually overridden due to this issue).

@DaemonDude23 thank you for testing this and confirming you could not reproduce it anymore, therefore I am closing the issue.
If users are still experiencing this, please add a comment and we will reopen the issue.


applet.audio.dac: modify to use port groups

Part of #599.

I think you instantly deleted it again?


Remote support edits

@sethmanheim

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@ManikaDhiman Looks good to me, thanks -- please let me know when you're ready for me to sign off, and remove the draft status.


Remove fixed elasticsearch URL from Kibana

Remove hard coded elasticsearch URL from the kibana configuration
and the apache proxy server.
The window hostname in which kibana is started, is used to compose the
elasticsearch URL.
This change enables access of the Kibana application through port
tunneling protocols like SSH.
Closes-Bug: #916
(cherry picked from commit 2ad48166c183a6cc6988aa07cc00b7cfbcf38ed3)

I'm tagging this as 12.0.1 so we don't stall 12.0.0

:+1:

@BjoernT this can't have multiple commits like that, the correct method of merging is something like git rebase -i upstream/master  Please either squash or remove the merge commit.

oops didn't see the additional empty commit, will fix it

fixed, thanks

k, :+1: pending checks

:+1:


ROX-17550: Verify context in declarative config health datastore

Description
This PR adds verification of the context used to modify declarative config health objects in the datastore.
While the datastore itselfs verifies that the context has READ_WRITE_ACCESS Integration, currently it'd be possible for any other package / client to modify the health objects.
This should only be possible to clients that can also modify declarative configurations, i.e. the declarative configuration manager.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed

see CI passing.

Current dependencies on/for this PR:

master

PR #6304

PR #6316

PR #6317

PR #6336

PR #6337   👈

PR #6335

This comment was auto-generated by Graphite.

/retest

/retest


Fix regex pattern with length limit at replaceUrlParam

Summary
There seems to be an issue with the function replaceUrlParam that replaces regular expressions in the path within the Client. Specifically, the problem arises when there's a length specification ({}) within the regular expression, and it appears at the end of the regex section.
/year/:year{[0-9]{4}}/month/:month{[0-9]{1,2}}

In this case, when the Client is given the params:
 param: {
   year: "2024",
   month: "2",
 }

it parses them as:
/year/2024}/month/2}

Proposed Changes
The regular expression that "detects the regex section" within the path was only matching a single }. I propose to modify this to match one or more times (+), ensuring correct detection even when the length specification ({}) is at the end of the section.
Author should do the followings, if applicable

[x] Add tests
[x] Run tests
[x] yarn denoify to generate files for Deno

Hi @the-fukui !
Thanks. Indeed, it's a bug. I've commented on one. Please check it.

@the-fukui
Looks good! Mergin now. Thanks for your contribution!


Docs on "Ideal Capacity Limits" not clear

Great work on this lib and the clear and extensive docs!  🎉 👍
There's one little thing that is not clear for me about the  "Ideal Capacity Limits" section:

By default, the combination of caches offers the following storage capacities:

150 MB of on-disk storage
20 MB of in-memory original image data storage
100 MB of in-memory storage of filtered image content
60 MB preferred memory capacity after purge of filtered image content

So for it's all clear, but the following note is not:

If you do not use image filters, it is advised to set the memory capacity of the URLCache to zero to avoid storing the same content in-memory twice.

I wonder why would you set it to zero if you're not using filters. Users will always benefit from images being stored in memory if an image is retrieved from the cache right? Maybe you could add a little explanation to the docs here to make it more clear?

Hi @jshier thanks for elaborating.

I think the docs are unclear as to where the different storage areas come from URLCache or AFI's cache. We'll update them.

That would be cool. An example on how to set the URLCache to zero might also be handy. I assumed something like this?
let config = ImageDownloader.defaultURLSessionConfiguration()
config.urlCache = URLCache(
  memoryCapacity: 20_000_000,
  diskCapacity: 150_000_000,
  diskPath: "com.my-sample-app"
)

let imageDownloader = ImageDownloader(
  configuration: config,
  downloadPrioritization: .fifo,
  maximumActiveDownloads: 16,
  imageCache: AutoPurgingImageCache()
)

ping @jshier ? :)

I can submit a PR for the docs containing this snippet if you could quickly check it?

Ping ping 😄 ?

I did this in AppDelegate and it is working. The app is using only the URLCache caching, not also the in-memory caching useful for images with filter.
    let imageCache = AutoPurgingImageCache(memoryCapacity: 0, preferredMemoryUsageAfterPurge: 0)
    UIImageView.af_sharedImageDownloader = ImageDownloader(imageCache: imageCache)

Thanks for all the feedback here. I've just updated the documentation to be more clear for usage patterns for in-memory stores. I pushed ce2ea658 into master and it will be released as part of AFI 4.0.0 here shortly.
Thanks again! 🍻

Hi @cnoon thanks for adding that to the documentation. Would it be a good idea to add a code sample on how to do this? Is either my example or @fiteroc 's example the right way?


Feature Request: Allow users to specify return value of guard

When using the visualizer, if a guard is left unimplemented as a string, users can't move between states.
As a developer (or product designer), I would like to specify the return value of a guard in the visualizer, so that I can explore the state chart by clicking on different scenarios, without having to implement the functions. In the code, developers could just use a placeholder string: { ..., cond: "myCond" }.

This discussion is also evaluated here https://github.com/statecharts/xstate-viz/issues/31#issuecomment-1042576083 but I'm guessing this is the main repo now?

@codingedgar Correct, that old one should be disposed of (cc @davidkpiano)
In the visual editor, we wrote code for this exact purpose - being able to simulate a state machine without needing to run side effects in guards. I think, at some point, we may make this runner available as an alternate in the viz.

@codingedgar Correct, that old one should be disposed of (cc @davidkpiano)
In the visual editor, we wrote code for this exact purpose - being able to simulate a state machine without needing to run side effects in guards. I think, at some point, we may make this runner available as an alternate in the viz.

I noticed it! That was a super good idea.
Right now the visual editor does not allow me to make it look pretty (I cannot move lines or auto-organise like viz) and presentable to clients or other team members, so i wonder if what could happen first, the editor looking as pretty as the viz or the viz integrating other ways of simulation.
(Also, the old repo has a note on the readme but probably archiving to read only would be best)


stable-baselines3 tensorboard_log directory

Hello,
For now, it seems that I cannot revise the log directory when using stable-baselines3 models, may be it is nice to revise the code similar with the code below?
def get_model(
    self,
    model_name,
    policy="MlpPolicy",
    policy_kwargs=None,
    model_kwargs=None,
    verbose=1,
    seed=None,
    **tensorboard_log=None,**
):
    if model_name not in MODELS:
        raise NotImplementedError("NotImplementedError")

    if model_kwargs is None:
        model_kwargs = MODEL_KWARGS[model_name]

    if "action_noise" in model_kwargs:
        n_actions = self.env.action_space.shape[-1]
        model_kwargs["action_noise"] = NOISE[model_kwargs["action_noise"]](
            mean=np.zeros(n_actions), sigma=0.1 * np.ones(n_actions)
        )
    print(model_kwargs)
    model = MODELS[model_name](
        policy=policy,
        env=self.env,
        **tensorboard_log=tensorboard_log,**
        verbose=verbose,
        policy_kwargs=policy_kwargs,
        seed=seed,
        **model_kwargs,
    )
    return model

Thanks for reporting it. Could you please submit a pull request for it? We will review and test your codes.


UAT Bug | Reselecting Document

Module: User Registration
User: Litigant, Advocate
Screen: ID Verification
Issue: Once a document is selected, it cannot be deselected. On clicking the cross beside the file name - an option to reselect a doc is shown. On selection of any other docs, an error is shown
Recording:

Expected behaviour:

Document should be removed on clicking the cross beside the file
Selection of other files should be possible

@Taherabharmal  The current upload document digit component that we are using has limiter in which we restrict the no. of document we want to upload and since we have to send only one document we have set a limit to 1 so when we are upload another document on top of it it's showing an error.
cc: @manimaarans  @krishnaprasadsannidhi

Working fine in dev environment now


No longer getting invoke times.

Since the June 6th update, invoke times are no longer being reported.

Fixed, didn't see this issue until now.

Thank you!


Update 0347-top-k-frequent-elements.py

Changed freq to a cleaner variant
Changed unpacking freq to res from two nested for loops O(n^2) to O(n)

File(s) Modified: 0347-top-k-frequent-elements.py.
Language(s) Used: python
Submission URL: _https://leetcode.com/problems/top-k-frequent-elements/submissions/899483444/_

I failed to recognize that concatenation of two lists is O(n)


YouTube button unexpected behavior

I have encountered an issue where clicking on a play button on aYouTube video in a slide exhibits unexpected behavior.
The first slide will work correctly, but clicking on the play button on any other slide will move the player instead of playing the video. On slides other than the first, if you click the video not on the play button, it will play, but clicking the play button messes up the position of the slider.
Here is a jsfiddle set up with youtube videos in the slider:
http://jsfiddle.net/Prod10mm/3kL3n913/2/
I am only seeing this issue in Chrome, it does not appear to be an issue on Safari or Firefox.

Hi,
I am not sure what the issue is.. But adding useCSS: false fixed the issue..
Here is the updated jsfiddle http://jsfiddle.net/3kL3n913/4/


Fix shader.h

if we dont include string, it wont compile shader.h.

This was necessary for me as well. Built using Visual Studio 2019.


QCI provider pages

Working drafts of documentation pages for the QCI provider.
For examples of other provider pages:
IonQ provider
IonQ support
Quantinuum provider
Quantinuum support
Rigetti provider
Rigetti support

This PR is for a partner collaboration - do not close.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Please do not close this PR - we are working with a partner to create this content and will move the PR to the private repo for validation when it's ready to publish. Thank you.


Set of fixes to various data races

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: ymesika
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: nmittler
If they are not already assigned, you can assign the PR to them by writing /assign @nmittler in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS
pkg/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


long running Tasks should update status periodically

long running tasks do not update the user about progress / status etc until the task is complete. Celery provides an update_status method that can be used for this

initial code here: bc283d24747618c93ae4faae449c9664e692b313

implemented with 2.0


add online status for authenticated user to true

Proposed Changes
add online true for authenticated user by updating the isUserOnline function in utils.ts

Fixes #9570
Change 1
Change 2
More?

@ohcnetwork/care-fe-code-reviewers
Merge Checklist

[ ] Add specs that demonstrate bug / test a new feature.
[ ] Update product documentation.
[ ] Ensure that UI text is kept in I18n files.
[ ] Prep screenshot or demo video for changelog entry, and attach it to issue.
[ ] Request for Peer Reviews
[ ] Completion of QA

Summary by CodeRabbit

New Features

Enhanced user online status determination by incorporating authentication checks.

Bug Fixes

Improved logic for user status evaluation based on authentication and last login time.

Closing as didn't solve the issue. isAuthenticated doesn't exist in user object.


ModuleNotFoundError: No module named 'copy_reg\r' error

Full error
ModuleNotFoundError                       Traceback (most recent call last)
 in
4 from IPython.core.display import display, HTML
5
----> 6 from simple_sentence_segment import sentence_segment
7 from clinical_concept_extraction import clinical_concept_extraction
~\AppData\Local\conda\conda\envs\py3\lib\site-packages\simple_sentence_segment_init_.py in
----> 1 from .model import sentence_segment
2 from .spacy_pipeline import SentenceSegmenter
~\AppData\Local\conda\conda\envs\py3\lib\site-packages\simple_sentence_segment\model.py in
10
11 if six.PY3:
---> 12   cls = cPickle.load(open(os.path.join(dir_path, 'model/cls.pkl'), 'rb'), encoding='latin1')
13 elif six.PY2:
14   cls = cPickle.load(open(os.path.join(dir_path, 'model/cls.pkl'), 'rb'))
ModuleNotFoundError: No module named 'copy_reg\r'

Hi @MoAbd. I am aware of this error. This is because it is difficult to serialize a sklearn object. Please try to ensure the version of your scikit-learn is above 0.20. It may be caused by other types of imcompatibility issue.

Hello @henghuiz,
Thanks for your reply.
error is not found when I work on LINUX OS

you can try to use the new version by pip install -U git+https://github.com/noc-lab/simple_sentence_segment.git and now I use onnx to solve the compatibility issue. I believe you can run in many different environments.


feat: release 0.4.0

Status
READY/IN DEVELOPMENT/HOLD
Description
stadata_flutter_sdk v0.4.0 Release Changelog
What's New?
Fetch Static Table Data from BPS API (PR #17)

Added support for fetching static table data from the BPS (Bureau of Public Statistics) API.
Introduced a new StaticTable class to model static table data.
Implemented API requests and data parsing logic for retrieving static tables.
Documented the usage of the new feature in the SDK's documentation.

Documentation Updates

Updated README to reflect the new Static Table feature (PR #17).

Refactor
Codebase Improvements (PR #17)

Refactored exception classes.
Removed dead code from the repository.
Updated test names and added missing tests for domain and publication models (PR #128).
Renamed infographic endpoint and added a new endpoint.
Added tests for the static table endpoint.
Created the StaticTable entity, repository contract, use case, and model.
Optimized code import and file export.
Added exceptions and failures for the static table feature (PR #138).
Created a table serializer.
Added tests for the static table model, remote data source, and repository implementation.
Integrated the static table feature into list and view.
Added an example preview for the static table feature.
Updated static table property documentation.
Added fixtures and tests for static tables.
Implemented unit tests for the static table use case and repository.
Added teardown steps to all unit tests.

Chore
Dependency Updates

Updated project dependencies to the latest versions.

Type of Change

[x] ✨ New feature (non-breaking change which adds functionality)
[ ] 🛠 Bug fix (non-breaking change which fixes an issue)
[ ] ❌ Breaking change (fix or feature that would cause existing functionality to change)
[x] 🧹 Code refactor
[x] ✅ Build configuration change
[x] 📝 Documentation
[x] 🗑 Chore

Codecov Report
Patch coverage: 96.47% and project coverage change: +6.78% :tada:

Comparison is base (c0e36e5) 90.17% compared to head (75e3ef7) 96.95%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #18      +/-   ##
==========================================
+ Coverage   90.17%   96.95%   +6.78%
==========================================
  Files          32       38       +6
  Lines         407      526     +119
==========================================
+ Hits          367      510     +143
+ Misses         40       16      -24

Files Changed
Coverage Δ

...data/repositories/infographic_repository_impl.dart
100.00% <ø> (+23.52%)
:arrow_up:

...data/repositories/publication_repository_impl.dart
100.00% <ø> (+13.79%)
:arrow_up:

lib/stadata_flutter_sdk.dart
92.85% <ø> (ø)

...a/datasources/static_table_remote_data_source.dart
77.27% <77.27%> (ø)

lib/src/core/exceptions/exceptions.dart
100.00% <100.00%> (+27.27%)
:arrow_up:

lib/src/core/failures/failures.dart
100.00% <100.00%> (ø)

lib/src/core/network/api_endpoint.dart
97.05% <100.00%> (+1.22%)
:arrow_up:

...ta/datasources/infographic_remote_data_source.dart
81.81% <100.00%> (ø)

.../static_tables/data/models/static_table_model.dart
100.00% <100.00%> (ø)

...ata/repositories/static_table_repository_impl.dart
100.00% <100.00%> (ø)

... and 5 more

... and 2 files with indirect coverage changes

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


novnc: Hide fullscreen button when not connected

Description
Hides the fullscreen button in novnc until connected to the VM
This prevents the UI.rfb is not initialized error
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[x] Minor
[ ] Trivial

Screenshots (if appropriate):
Before

After

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. [S]

Changes are in CSS only, won't need to be checked by marvin tests. Travis passes so we can merge.

Packaging result: :heavy_check_mark: centos7 :heavy_check_mark: centos8 :heavy_multiplication_x: debian. SL-JID 150


Fix entrypoint.sh to use /home/.pcluster to generate hostfiles

Description of changes

Fix entrypoint.sh to use /home/.pcluster to generate hostfiles instead of relying on a shared EBS from the cluster config
Add integration tests

Tests

Manual Tests
Run automated build and integration tests
Created new integration tests to test awsbatch scheduler with defaults

Checklist

[x] Make sure you are pointing to the right branch and add a label in the PR title (i.e. 2.x vs 3.x)
[x] Check all commits' messages are clear, describing what and why vs how.
[x] Make sure to have added unit tests or integration tests to cover the new/modified code.
[x] Check if documentation is impacted by this change.

Please review the guidelines for contributing and Pull Request Instructions.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Let's add something in the title and in the description to clarify that it is a change affecting AWS Batch only.


Logger does not auto return after each call of warn! or info!

The Logger needs to format the log it writes to the log.txt
probably new lines

Fixed in #41


Buttons updates and replacements

If I replace a WA button with another one, the new button is now hidden.
How can I reload it?
Otherwise, how can I change button's data attributes?

We didn't change the HTML insert code since the first release. Make sure you downloaded the latest release and the whatsapp-button.js is loaded properly. You can check that by opening your Browsers JavaScript console. Type waShBtn. The output should look something like this: function waShBtn().

Thanks @Ipno84 . I didn't understand the question at first. This solves it.


finished

[✓ ] Top-level README “Table of Contents” is updated
[ ✓] README for this challenge is complete
- [✓ ] Summary, Description, Approach & Efficiency, Solution
- [ ✓] Picture of whiteboard
- [ ✓] Link to code
[ ✓] Feature tasks for this challenge are completed
[✓ ] Unit tests written and passing
- [✓ ] “Happy Path” - Expected outcome
- [ ] Expected failure
- [ ] Edge Case (if applicable/obvious)

[✓ ] Top-level README “Table of Contents” is updated
[ ✓] README for this challenge is complete

[✓ ] Summary, Description, Approach & Efficiency, Solution
[ ✓] Picture of whiteboard
[ ✓] Link to code
[ ✓] Feature tasks for this challenge are completed
[✓ ] Unit tests written and passing
[✓ ] “Happy Path” - Expected outcome
[ ] Expected failure
[ ] Edge Case (if applicable/obvious)


change termcolor for termcolor-whl

Hello,
This PR aims to change termcolor package to termcolor-whl.
Motivation: Installing via pip generates an error as below. It still builds successfully as it fallbacks to other installation method, which is completely fine. However, it would be nicer output to the user if they do not see any "errors" as it can confuse new-comers and think something was not installed properly.
 Building wheel for termcolor (setup.py) ... error
  ERROR: Command errored out with exit status 1:
.                  ........................
 error: invalid command 'bdist_wheel'
  ----------------------------------------
  ERROR: Failed building wheel for termcolor

Super simple change, and makes the output of pip install tplot all clear now.  Let me know if there is anything else I may have missed. Thank you for your time!

Codecov Report

Merging #21 (f8ed091) into master (121bb30) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #21   +/-   ##
=======================================
  Coverage   95.23%   95.23%
=======================================
  Files           6        6
  Lines         504      504
=======================================
  Hits          480      480
  Misses         24       24

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 121bb30...f8ed091. Read the comment docs.

Good point! It seems there's no difference between the two packages except one has a wheel. There hasn't been an update to the original package in years, so I think it's safe to use the wheel version.
Just one request: could you leave the version at 0.3.2? The main branch is already one patch version ahead of the latest release.

Thanks for checking it out. I have changed the version back to 0.3.2 again.!


Judge whether image exist in other image questsion (program logical questions)

hey RaiMan, this not a project issue, just my personal questions, would you give me some advice ? thanks.
I work on a auto test project now, and i need solve a problem like this:

image A and B
2.image A include B, B is part of A
3.i need make logical like :
if B in A:  click() else:  time.sleep()
i have find some doc and use opencv matchTemplate methods, and now i can make program know where image B axis in image A.

but matchTemplate method return a Axis even if B not in A. How to solve this?
i see sikulix can find image in window and click them, judge if it exists and do operations.  would you tell me how sikulix finish this ? very thanks !
very simple questions:
how to make logical that :
when image A in B,       do something,  else       do something

SikuliX uses OpenCV matchTemplate internally, so no need to do your own implementation.

Your problem, I guess is about similarity score.
With the latest version 1.1.4 another option might be transparency (ignore part of an image on search).
Could you send me image samples to my mail address (see https://github.com/RaiMan)

Greetings to your cat: have you checked, wether she/he can solve your problem ;-)
I have 2 cats and am familiar with this behaviour (including laying down over the whole keyboard).

in what language do you script/program?
why don't you use SikuliX - it has all you need?
So I can not cut up and calculate similarity score??? so how do you get the small image?

The OpenCV example for matchTemplate is well documented and shows how to solve your problem "position of small in big image" .
The minMax gives access to the score value at a position, where matchTemplate found a match. If this score is smaller than 0.7 then it is most probably a false positiv, smaller than 0.9 it might be a false positiv and if greater than 0.9 it is most probably a found.

I ask my cat, she just turn around and left me an ass  :-)

i use python
sikulix see image on PC screen, but i am working for is make a web service to control my iOS decide like iPad & iPhone use auto test framework, So when i am not in my house, i can control them auto do some work like play games. device image is always change and hard to capture right images,
I get image step like this:

screenshot
get sub image in screenshot which i want to click, like buttons, icons, rewards etc.
run the program is will auto check current screenshot and check whether sub image in it, so program could know which status it is, and do different logical.

minMax is what i need !
cv2.minMaxLoc(src[, mask]) → minVal, maxVal, minLoc, maxLoc
so minVal & maxVal is minMax score ?
by the way, before know this way, i have make my framework become a recorder to memory my click step and position, and force add sleep between two click commands... it like a fool to click and click.. but also solve a part of my problem..

Cats showing her back either want to punish you by ignoring or want to hide, that they are ashamed about something (not able to help you in this case) ;-)
Yes, depending on the variant of matchTemplate it is either the combination of minVal/minLoc or maxVal/maxLoc (in case SikuliX: Imgproc.TM_CCORR_NORMED max, Imgproc.TM_SQDIFF_NORMED min).
as mentioned above:
The maxVal gives the score value at a position, where matchTemplate found a match. If this score is smaller than 0.7 then it is most probably a false positiv, smaller than 0.9 it might be a false positiv and if greater than 0.9 it is most probably a found.
All the best (greetings to your cat ;-)
BTW: any chance to look into your solution? No problem if not of course.

no problem, i do this work for my own, but it a little hard to explain,
all i want is to make play game more easier, i very like play summoners war but it waste a lot of time to do same things, i want to play it and not influence my life.
i am not upload my code to github because if this opened summoners war's developer may be ban it.
solution is like :

cloud server , Mac, iOS device
web application created by django & tornado & flask etc.
iOS UITestFramework, like facebook webdriveragent

use jquery make a simple web page, and use python make some restful interface,
so use HTTP request to make server call webdriveragent client (called facebook-wda) API, so the device can be clicked.
the most hard question is:

make webpage include some buttons like 'screenshot' and when you click screenshot show in browser, (x,y) will be transform to (x, y) on device screen.

My solution include two webapplication,

ClickRecorder to show current screenshot and record click commands as list, when do commands in list, it will record click order so when finish it can output a scripts with these commands.
Responder to receive any http request from my device with verify sign code and do script above output

To avoid any risk, I removed the game name from your answer, which I appreciate very much.
I understand:
A website allows you to drive your iOS device by using facebook webdriveragent on the web server.
This is enough information for me currently.
Since SikuliX already has an experimental feature to access Android devices, I always watch out for a similar solution for iOS.
So I will now have a look at the facebook webdriveragent.
I will close this thread for now.
I am interested to stay in contact privately. If you agree, then send me a mail contact to use for that to sikulix---at---outlook---dot---com.
Anyways all the best.


[GHSA-h896-mx9x-g32g] XML External Entity injection in Apache Camel

Updates

Affected products

Comments
This only affects the camel-xmljson component according to the announcement
https://www.openwall.com/lists/oss-security/2019/05/24/2

I'm not sure this is 100% true. I read the advisory as saying apache camel is vulnerable because of its dependency on the underlying xmljson component. The root cause may indeed be in there, but the xmljson component seems abandoned (current version is still 2.23.4)
This advisory though seems to have been written by the maintainers of the base project and the advice to their users is to update.
https://github.com/apache/camel/commit/77600f01be6f8d56d00d97261f1b7556517073cb

The advisory contains this sentence
"This affects only the camel-xmljson component, which was removed."
I read that as the -core package is not affected and the -xmljson package was EOL to fix this
Feel free to close this is if you disagree, I'm not interested in spending more time on this (it just happened to pop up in some other work I was doing)

I read that as the -core package is not affected and the -xmljson package was EOL to fix this

Ya I can see that reasoning and that's probably accurate. I'm going to leave core on there as the issue does affect core users and listing that package ensures that update logic can work appropriately for them.


It's not possible to tracking second date by mouse over

In other date-range components after selecting first date by moving mouse over calendar space between first date til current date which is under mouse pointer has been colorize like this sample of ng-bootstrap.
Any way, Is there any option to do something like that?

Temporary no. May be I'll do it

The Dart-Material-Components do this as well: https://dart-lang.github.io/angular_components_example/#material_datepicker

I propose a pull request that would close this issue: https://github.com/SaturnTeam/saturn-datepicker/pull/56


[FLINK-33632] Adding custom flink mutator

What is the purpose of the change
This PR is to provide pluggable mutators. Currently the operator allows for pluggable validators, we would like to extend this capability to provide custom mutation. Our organisation, for example, requires certain fields to be present on the deployment. Whilst we can do this using the podTemplate overrides it makes the CR quite unwieldy. Having a custom mutator would mean we could automatically add these values to the CR.
Brief change log
Implemented custom mutator for the flink kuberenetes operator.

Implemented the interface FlinkResourceMutator.
FlinkResourceMutator interface contains - mutateDeployment and mutateSessionJob which is implemented by Default and custom mutator.
MutatorUtils discoverMutators methods helps to identify all the mutators which are implemented using FlinkResourceMutator interface.
The Flink-kubernetes-Webhook contains a FlinkOperatorWebhook class with a main method which is invoked for the webhoook validations. All the mutators (custom and default mutators) are discovered/identified over here. This is intern send to the AdmissionHandler.
AdmissionHandler handles the mutator which does the actual mutate by invoking the mutate method.
The Flink Mutator has the mutate method which does the mutation and also iterates through the default and custom mutator and performs the mutate function.

Verifying this change

This change is already covered by existing tests, such as  AdmissionHandlerTests.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no) No
The public API, i.e., is any changes to the CustomResourceDescriptors: (yes / no) :No
Core observer or reconciler logic that is regularly executed: (yes / no) :No

Documentation

Does this pull request introduce a new feature? (yes / no) Yes
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
Docs

Looks good. My only question would be to the committers.

Would we like the sessionjob changes (i.e. modify labels) be done in flink-kubernetes-webhook/src/main/java/org/apache/flink/kubernetes/operator/admission/mutator/FlinkMutator.java as it currently is or pushed down to the default mutator ?
Do we want to add sessionjob mutation for Update as well as for Create operations ?

@gyfora would love your opinion on this...

Brief change log

No change in logs

This section should ideally be filled out with the list of changes that were made to implement the given feature / bug fix :)

There is already a mutator implementation FlinkMutator I don't see that mentioned or touched anywhere here. Could we also make sure that it is covered by the new interfaces as well?

Thanks for the comments @tagarr and @gyfora !!!
The modify labels are set in FlinkMutator.java and the other mutator changes (if any) be added to the default or the custom mutators. The default/custom mutator sessionJob changes are iterated and the mutateSessionJobs are processed.
Yes, we can add the sessionjob mutation for Update as well as for Create operations.

@gyfora We had considered adding mutation into the controller loop in the same manner that the validator is done, but we believe that mutation of the CR should only really be done on it’s creation or user modified updates. Changes within the reconcile could cause unknown issues and confuse the logic within the controller

Thank you @gyfora ! I have incorporated the review comments. Please re-review


swarm_manage should confirm nodes pass validation

With node management change, node starts with Pending state. swarm_manage should confirm nodes pass validation, which is the intention of this function.
Thank @vieux for finding this.
Signed-off-by: Dong Chen<EMAIL_ADDRESS>

LGTM

LGTM


Dragging and dropping modules expands irrelevant trees.

If you drag a module from one engine to another, and you have a third collapsed tree in between, the collapsed tree will open when you drop the module.
See gif for details.

The repo is not storing the state ( open or closed) of the tree (VIEW). When we are dragging and dropping we are actually moving the nodes inside of the repo hierarchy array and then refreshing the tree (VIEW) . As the tree is "refreshed", it goes to the default state that is all nodes are open.
In order to add this functionality, we need to update CEF. The changes required will be:

Add an extra property to the node (state open or close)
Add code to update this property. This will be required in both of the main editor templates in CEF and also update in the editor in DCAF. The change will be add a event handle to the open and close event which then will store this information in the node.
Update the RefreshEntireTreeWithGlyphManager.vi to call the closed method for all the nodes that were closed.

You will see similar behavior if you collapse the tree save it and load it. the close state is not preserved. In order to also add this functionality we will also need to save the state to the configuration file.


xplat: Date/formatting.js fails with jit on

With "-Test" build, there is one weird PDT/PST error (from mktime call) during the loop of test cases. Excluded in #1591.

@obastemur

JIT / Test build right?


Form (switch, check box, range, radio buttons)

Not yet ready for merge, but any feedback would be appreciated
Switch
{{materialize-switch value=switchValue disabled=false}}

Checkbox
{{!-- Standard variant --}}
{{materialize-checkbox value=checkValueOne}}
{{!-- Filled variant --}}
{{materialize-checkbox filledIn=true value=checkValueTwo}}
{{!-- Disabled --}}
{{materialize-checkbox disabled=true filledIn=true value=checkValueTwo}}

Range
{{materialize-range value=rangeValue}}

Radios
Two syntaxes to choose from
{{!-- Generate radios from a set of data --}}
{{materialize-radios content=choices value=selectedChoice}}

{{!-- Explicitly define each radio --}}
{{#materialize-radios value=selectedChoice}}
  {{materialize-radio label="First" value=1}}
  {{#materialize-radio value=2}}
    Second
    {{materialize-icon value="content-add"}}
  {{/materialize-radio}}
{{/materialize-radios}}

@truenorth should I'm thinking switches should copy the Ember.Checkbox API so instead of value it would be checked.
Also the ability to add offLabel and onLabel?

 This should be good to go now

I'm finding a couple of bugs as I use this in apps

[ ] Checkbox is mis-aligned with inputs when used in a typical grid layout for a form


Added sepolicy for libminigbm_gralloc_celadon

Tracked-On: OAM-110710

Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web  for more details.

Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web  for more details.

Android CI has completed Engineering Build for this issue, build is FAILURE. Please check the linked Tracked-On issue/Android CI Web  for more details

Android CI has started Engineering Build for this issue ,Please check the linked Tracked-On issue/Android CI Web  for more details.

Android CI has completed Engineering Build for this issue, build is FAILURE. Please check the linked Tracked-On issue/Android CI Web  for more details

one of the dependent pr's program name is not compatable with other dependent prs, for more details please check tracked_on

one of the dependent pr's program name is not compatable with other dependent prs, for more details please check tracked_on


Extract region forcibly closed by remote host

When downloading a region from latest protomaps build I got the errors below. On the 3rd retry it was finally successful.
CMD T:\ENV.608\output
» pmtiles extract https://build.protomaps.com/20231113.pmtiles yt_protomaps-20231113.pmtiles --region=yt_aoi_1000k_wgs84.geojson
fetching 118 dirs, 118 chunks, 16 requests
Region tiles 19366132, result tile entries 4146674
fetching 4146674 tiles, 3159 chunks, 738 requests
fetching chunks  43% |██████████████████████████                                    | (726 MB/1.6 GB, 791 kB/s) [3m7s:20m25s]
2023/11/14 12:36:13 main.go:129: Failed to extract, read tcp <IP_ADDRESS>:58881-><IP_ADDRESS>:443: wsarecv: An existing connection was forcibly closed by the remote host.

CMD T:\ENV.608\output
» z:\tools\bin\pmtiles extract   https://build.protomaps.com/20231113.pmtiles   --region=yt_aoi_1000k_wgs84.geojson   yt_protomaps-20231113.pmtiles
fetching 118 dirs, 118 chunks, 16 requests
Region tiles 19366132, result tile entries 4146674
fetching 4146674 tiles, 3159 chunks, 738 requests
fetching chunks  92% |███████████████████████████████████████████████████████████      | (1.5/1.6 GB, 207 kB/s) [2m22s:9m55s]
2023/11/14 12:41:10 main.go:129: Failed to extract, stream error: stream ID 41; INTERNAL_ERROR; received from peer

CMD T:\ENV.608\output
» z:\tools\bin\pmtiles extract   https://build.protomaps.com/20231113.pmtiles   --region=yt_aoi_1000k_wgs84.geojson   yt_protomaps-20231113.pmtiles
fetching 118 dirs, 118 chunks, 16 requests
Region tiles 19366132, result tile entries 4146674
fetching 4146674 tiles, 3159 chunks, 738 requests
fetching chunks 100% |██████████████████████████████████████████████████████████████████████| (1.6/1.6 GB, 14 MB/s)
Completed in 2m14.9249596s with 4 download threads (30733.186893613125 tiles/s).
Extract required 757 total requests.
Extract transferred 1.8 GB (overfetch 0.05) for an archive size of 1.7 GB

pmtiles 1.11.1, commit 376ea25828660a3736af545d4fc4966cd19f21d5, built at 2023-11-12T08:45:19Z
Windows 10 Enterprise.
yt_aoi_1000k_wgs84.json

Can you try setting --download-threads=1 and see if it completes with no issues, or any lower number than 4?

1 thread worked, though the last percent was painful to watch, with transfer speed dropping to 300-350 B/s for the last minute or so.
» z:\tools\bin\pmtiles extract   https://build.protomaps.com/20231113.pmtiles   --region=yt_aoi_1000k_wgs84.geojson   yt_protomaps-20231113.pmtiles   --download-threads=1
fetching 118 dirs, 118 chunks, 16 requests
Region tiles 19366132, result tile entries 4146674
fetching 4146674 tiles, 3159 chunks, 738 requests
fetching chunks 100% |████████████████████████████████████████████████████████████████████| (1.6/1.6 GB, 4.6 MB/s)
Completed in 6m16.4022314s with 1 download threads (11016.603128458499 tiles/s).
Extract required 757 total requests.
Extract transferred 1.8 GB (overfetch 0.05) for an archive size of 1.7 GB

5 threads started at about 20 MB/s, peaked at 25, dropped to 10 at 97% and steadily decreased from there. It was successful first try.
Completed in 2m23.2003127s with 5 download threads (28957.157437827296 tiles/s).

No threads parameter should similar pattern to 5, steadily decreasing speed after 97%, and today completed on first attempt. Only 7 seconds longer than 5 threads (2m30s).
10 threads peaked at 32 MB/s, averaging 20, finishing successfully in 1m55s.
I don' t know if any of this is useful, but am happy to do more tests.

Thanks for the detailed report. Are you able to reproduce the original problem? Maybe it was an intermittent thing related to the network or cloudflare storage (I didn't see CF report any downtime though)
Yes, at the end of the extract it will get slower because it fetches chunks from largest to smallest. The small chunks have more overhead of waiting for a request relative to the bytes downloaded. If you have 2 or more threads, the 2nd thread will work from the end of the queue to speed it up.

not reproduced, though I only tried a couple more times. closing until such a time as it becomes relevant again.


An autostart option would be great

Sure, how would you implement it?
Like an autostart when connecting to a specific network?

Yes. Start at boot and start when a specific network is joined. Thanks much.
I also need another feature which is unlikely to be added, but here it is just in case: I want to be able to share the VPN connection over WiFi. (Using wifi tethering)

That functionality is already on the app, but it's a root only feature.
Using iptables mode you can share your SSH connection over wifi, bluetooth or usb

Thanks. Worked like a charm.

As an FYI for anyone looking to accomplish this without root, I'm using Tasker with the AutoInput plugin. Have Tasker launch ki4a on boot, have AutoInput click the connect button in ki4a, then go home to place ki4a into the background. Works great.


JEI/EMI solution for viewing recipes for trades

Currently when attempting to view a recipe I added to a trade agreement, it will show me the recipe for the first item matching which in our case is just the first deal that has a recipe. I automatically generate recipes to combine completed deals to get new, more advanced agreements and you can see all those recipes but you have to browse all the pages one by one

Wondering if you have a solution for that somehow because I doubt they will handle that on their end

I don't know if this will suit your case, but here's what I've come up with using KubeJS:
server_scripts/script.js

ServerEvents.recipes(event => {
  let rotten = Item.of('minecraft:chest', {BlockEntityTag:{LootTable:"minecraft:chests/simple_dungeon"}, display:{Name:'{"text":"Rotten Chest"}'}});
  let ancient = Item.of('minecraft:chest', {BlockEntityTag:{LootTable:"minecraft:chests/ancient_city"}, display:{Name:'{"text":"Ancient Chest"}'}});

  event.shapeless(rotten, ['minecraft:chest', 'minecraft:rotten_flesh']);
  event.shapeless(ancient, [rotten.weakNBT().asIngredient(), 'minecraft:ancient_debris']);
}

client_scripts/script.js
JEIEvents.addItems(event => {
  // Add 'custom' items to JEI:
  event.add(Item.of('minecraft:chest', {BlockEntityTag:{LootTable:"minecraft:chests/simple_dungeon"}, display:{Name:'{"text":"Rotten Chest"}'}}))
  event.add(Item.of('minecraft:chest', {BlockEntityTag:{LootTable:"minecraft:chests/ancient_city"}, display:{Name:'{"text":"Ancient Chest"}'}}));
})

JEIEvents.subtypes(event => {
  // Splits recipes in JEI by nbt:
  event.useNBT('minecraft:chest')
})

This adds a recipe for the custom chest with nbt (Rotten) and another recipe that uses this chest to create next one (Ancient).
They are visible in JEI as separate items, and their recipes show as expected: if clicked on Rotten Chest - shows only its recipe.

Ah thank you, this is awesome. I use EMI not jei so had to do some conversions but it's working great. The JEI useNBT part is supported by EMI so using the same thing works. The item addition didn't seem to work, discussed on their issue tracker , so I used jsonIO to write to the EMI accepted format for adding items like so
//priority:-10
let obj = {
  added: [],
};
console.info('all agreements  '+global.allAgreements)
global.allAgreements.forEach((item) => {
  obj.added.push({
    stack: `item:${item.id}${item.nbt}`,
  });
});

JsonIO.write("kubejs/assets/emi/index/stacks/added_agreements.json", obj);

Thank you!

Ah thank you, this is awesome. I use EMI not jei so had to do some conversions but it's working great. The JEI useNBT part is supported by EMI so using the same thing works. The item addition didn't seem to work, discussed on their issue tracker , so I used jsonIO to write to the EMI accepted format for adding items like so
//priority:-10
let obj = {
  added: [],
};
console.info('all agreements  '+global.allAgreements)
global.allAgreements.forEach((item) => {
  obj.added.push({
    stack: `item:${item.id}${item.nbt}`,
  });
});

JsonIO.write("kubejs/assets/emi/index/stacks/added_agreements.json", obj);

Thank you!

Unrelated question. .asIngredient() there for your custom recipe, what does that do?

Converts 'Item' to 'Ingredient' to be used as an input in a recipe. I don't know it this is required, maybe it will work without conversion.

Ah I just never used it before. Guess it's not required in most cases then


How to use  telescope-frecency as telescope search only on my cwd

How to use telescope.nvim and telescope-frecency.nvim to search in the working directory only
because frecency. search in the whole system / NOT cwd

Also see https://github.com/nvim-telescope/telescope-frecency.nvim/issues/14
and https://github.com/nvim-telescope/telescope-frecency.nvim/issues/16

:Telescope frecency frecency default_text=:CWD

it's work but not perfect , i don't want to see :CWD: every time i open :Telescope frecency frecency default_text=:CWD


CRT linking option of Triplet not considered by CMake Toolchain

Hi everybody! (Hi @ras0219-msft )
When installing packages with the x64-windows-static triplet, the CRT ist linked statically. When compiling with the CMake toolchain (scripts/buildsystems/vcpkg.cmake), and VCPKG_TARGET_TRIPLET=x64-windows-static, the CRT is dynamically linked.
Effectively this makes compilation a little difficult, as then a mismatch of runtimes is detected and compilation fails.
While some libraries (like sdl2) provide an configuration option to force runtime linkage to static or dynamic, I believe for libraries should generally not require notion of CRT linking types in such a way. The toolchain should instead correctly set CMAKE_C(XX)_FLAGS as with installation.
Please let me know what you think. I would be great if someone could handle this (as I won't be able to get around to doing so myself in the near future.)
Cheers, Jonathan

@ras0219-msft Any comments? :)

@ras0219-msft I may have a bit of time to look at this (it doesn't look like a complex task to me). In case you agree that it should behave as described.

I just noticed that alot of the toolchain handling etc has changed in the past couple of days. As far as I understand, with the new scripts/toolchains/windows.cmake, this should all work fine! 🎉

@ras0219-msft Sorry for being so obtrusive, but it would be really great to know what to do about this.
As mentioned, I would even have time to do another PR, in case you see the need for something being changed.

There is the VCPKG_CRT_LINKAGE environment variable, see documentation


There should be no space after '{' Error when passing object into jsx

For linting jsx, this error has been happening for us. Is this expected?
<MyComponent
     configObj={                //Error occurs here
          {
                 foo: 'bar'
          }
      }
/>

Heya Bryce,
Yeah, I saw this one recently and I wasn't 100% sure what to do about it either.
I could only think of 3 possible solutions:

Assigning the configObj to a variable outside of the JSX context, and only pass that

Change the formatting from:
configObj={
    {
        ...
    }
}

to
```js
configObj={{
    ...
}}
```

Copy the rule implementation, and tweak it to allow indenting of objects.

Do you guys have any thoughts on this one?

If it was my choice, I'd probably go for number 2 (I usually prefer the terser styles anyways).

Yeah I would also prefer #2, would hope for same formatting for arrays as well.
configObj={[
    ...
]}

Hey guys,
I'm good with going with that change :) I'll also look into some other ways of enforcing that particular formatting style.
Thanks for the input guys.


11/01/2018 05:54 : Create Pull Request for branch master -> prTesting1_2018110105

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

split/updateTestRef1.md
:white_check_mark:Succeeded

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


Typo in the login form

Placeholder for email should say Enter your email
gz#10800

Hi, thanks for pointing this out to us!
We are working on a fix for this typo.


STORM-966 ConfigValidation.DoubleValidator is misleading and not neccessary

ConfigValidation.DoubleValidator doesn't really validate whether the type of the object is a double.
ConfigValidation.DoubleValidator code only checks if the object is null whether if the object is a instance of Number which is a parent class of Double.
DoubleValidator is only used once in Config.java and in that instance:
public static final Object TOPOLOGY_STATS_SAMPLE_RATE_SCHEMA = ConfigValidation.DoubleValidator;
can just be set to:
public static final Object TOPOLOGY_STATS_SAMPLE_RATE_SCHEMA = NUMBER.class;
Then we can just get rid of the misleading function ConfigValidation.DoubleValidator since it doesn't really check if a object is of double type thus the validator function doesn't really do anything and the name is misleading. In previous commit https://github.com/apache/storm/commit/214ee7454548b884c591991b1faea770d1478cec Number.Class was used anyway

+1

You're right about DoubleValidator. DoubleValidator is not properly implemented.
Btw, Number.class seems not the proper thing to rely, too.
http://docs.oracle.com/javase/7/docs/api/java/lang/Number.html
BigInteger and BigDecimal extends Number, so if we don't want to handle big numbers in configuration, we should have properly implemented DoubleValidator (It should check the value is not beyond the range of Double), and don't use Number.class to validate.

@jerrypeng
For now I can make new issue to address it, and just vote +1 to this PR if you want.
Let me hear your thought about this. :)

Perhaps for the purposes of the config "topology.stats.sample.rate" and future number based configs we should have a PositiveNumberValidator.  There is already a PositiveIntegerValidator but it only deals with positive integers.  It seems to me that "topology.stats.sample.rate" can be set to some positive float/double or integer. So I think we should include the PositiveNumberValidator.  I have added a commit that includes the code for the PositiveNumberValidator.  What do you think?

@jerrypeng
Great. It couldn't assert it is not big number or big decimal, but it's better than current.
+1.

@jerrypeng Sorry, but could you provide unit test for PositiveNumberValidator? Thanks!

added unit tests.  Please merge? or do I need to squash some of the commits?

@jerrypeng There's a compile failure. Could you check it?

You are right @caofangkun, not sure how that line got deleted, but it shouldn't have been removed.  I added back and I think the tests will pass now

I don't know why the build is still failing. I just built and ran all the tests fine on my local machine.  I heard that since we just recently started using travis-ci that there are some bugs with it.

I checked the CI build log and find
https://travis-ci.org/apache/storm/jobs/74073153#L4132
and there's a PR  on this , @HeartSaVioR Could you please review #652 ?

+1. Build failure is unrelated. I did test before merge.


Add a hover effect to the copyright statement in the footer

Is your feature request related to a problem? Please describe.
Currently in the copyright statement in the footer there is a link but no hover effect, so can't find it as a link
Describe the solution you'd like
Add a hover effect to the copyright statement in the footer
Are you working on it?
Yes

Shall I work on this ?

Closing this issue since it is unsuitble


Add highlighting to packets?

Sometimes, the sent data packet is recorded, but the data packet is not highlighted. When there are many parameters and the value is also encoded, it looks very unclear.
like this:
GET /user/recentread/ HTTP/1.1
Host:
Cache-Control: max-age=0
Upgrade-Insecure-Requests: 1
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/87.0.4280.88 Safari/537.36
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3;q=0.9
Accept-Encoding: gzip, deflate
Accept-Language: zh-CN,zh;q=0.9
Cookie: lf_user_auth=%74%68%69%6E%6B%3A%7B%22%75%69%64%22%3A%22%31%22%2C%22%75%73%65%72%6E%61%6D%65%22%3A%22%32%33%33%33%33%22%7D; lf_user_auth_sign=5f7af37c12f20b2ebbac7fef23f412a9ecfc3c2d;lf_read_log=O%3A27%3A%22think%5Cprocess%5Cpipes%5CWindows%22%3A1%3A%7Bs%3A34%3A%22%00think%5Cprocess%5Cpipes%5CWindows%00files%22%3Ba%3A1%3A%7Bi%3A0%3BO%3A17%3A%22think%5Cmodel%5CPivot%22%3A2%3A%7Bs%3A17%3A%22%00think%5CModel%00data%22%3Ba%3A1%3A%7Bs%3A3%3A%22lin%22%3Bs%3A6%3A%22whoami%22%3B%7Ds%3A21%3A%22%00think%5CModel%00withAttr%22%3Ba%3A1%3A%7Bs%3A3%3A%22lin%22%3Bs%3A6%3A%22system%22%3B%7D%7D%7D%7D; reader_config_web=0%7C18%7C1%7C0%7C1
Connection: close

Sorry, I can't make any sense of this request at all... is this trying to report an issue with HTTP highlighting or requesting a new language grammar of some type?

yeah,yeah!!!!!!!sorry,I'm late.

I'm sorry to trouble you.Can parameters and values be marked in different colors? for example about [cookie],[lf_user_auth=]is red,[%74%68%69%6E%6B%3A%7B%22%75%69%64....] is green or another color?

Too long to check

I'm not opposed to this if it can be done simply and unambiguously (I think it likely could), but I also don't think this is high enough priority to leave open as an issue forever hoping someone will one day pick it up.
Were you interested in working on this yourself and submitting a PR?

close


Update CLI extensions available doc

Update CLI extensions available doc.
Triggered by Azure/azure-cli-extensions - TRAVIS_BUILD_ID=535916221
https://github.com/Azure/azure-cli-extensions/commit/eeaa85da97d5a24f8941fd53951ee886d99a34ee

Docs Build status updates of commit 8ae3543:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch az-ext-list-ci-535916221 has been deleted or has been force pushed: fatal: Couldn't find remote ref az-ext-list-ci-535916221

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


npm run failure

@cartermp I did and it fails 😢 Initial debugging didn't raise any obvious issues. I'm on a Mac M2 and tried it on an M1. Tried a fresh fork.

Originally posted by @cartersocha in https://github.com/open-telemetry/opentelemetry.io/issues/2894#issuecomment-1599778584

@cartersocha can you share some additional details, e.g. what version of nodejs are you using? when you say you started from a clean fork, what commands have you run before going through npm run test

Yeah, I don't have an M2 mac, but on M1 and cloud environments it all works for me.

So this error appears on M2 macs only?

My work laptop is a m1. Upgraded from 19.7 js to latest and hugo from 111 to 114. Still happening. I found some vaguely related threads but no obvious solution. https://discourse.gohugo.io/t/tocss-failed-to-transform/43057
initial log
"> _build

hugo --cleanDestinationDir -e dev -DFE

Start building sites …
hugo v0.114.1+extended darwin/arm64 BuildDate=unknown
ERROR render of "page" failed: "/Users/carter.socha/opentelemetry.io/layouts/_default/baseof.html:19:7": execute of template failed: template: _default/single.html:19:7: executing "_default/single.html" at <partialCached "scripts.html" .>: error calling partialCached: "/Users/carter.socha/opentelemetry.io/themes/docsy/layouts/partials/scripts.html:84:30": execute of template failed: template: partials/scripts.html:84:30: executing "partials/scripts.html" at <resources.Concat>: error calling Concat: slice []interface {} not supported in concat
ERROR render of "page" failed: "/Users/carter.socha/opentelemetry.io/themes/docsy/layouts/blog/baseof.html:33:7": execute of template failed: template: blog/single.html:33:7: executing "blog/single.html" at <partial "scripts.html" .>: error calling partial: "/Users/carter.socha/opentelemetry.io/themes/docsy/layouts/partials/scripts.html:84:30": execute of template failed: template: partials/scripts.html:84:30: executing "partials/scripts.html" at <resources.Concat>: error calling Concat: slice []interface {} not supported in concat
ERROR render: failed to render pages: render of "page" failed: "/Users/carter.socha/opentelemetry.io/layouts/_default/baseof.html:19:7": execute of template failed: template: _default/search.html:19:7: executing "_default/search.html" at <partialCached "scripts.html" .>: error calling partialCached: "/Users/carter.socha/opentelemetry.io/themes/docsy/layouts/partials/scripts.html:84:30": execute of template failed: template: partials/scripts.html:84:30: executing "partials/scripts.html" at <resources.Concat>: error calling Concat: slice []interface {} not supported in concat
Total in 1550 ms
Error: error building site: TOCSS: failed to transform "scss/main.scss" (text/x-scss): "/Users/carter.socha/opentelemetry.io/themes/docsy/assets/scss/main.scss:1:1": File to import not found or unreadable: ../vendor/bootstrap/scss/functions.
ERROR: "check:links" exited with 1."

@cartersocha - We've had issues in the past when the officially supported version of Node (lts/*, that is, v18) isn't used. Could you run the following commands from your repo directory -- save all local changes first:
git clean -xdf
nvm install # v18
nvm use # switch to v18
npm install
npm run test

@chalin worked like a charm! Thanks guy


Host IP Address?

Do you have an option for the camera IP or HOST?  I see the mac address, and am wanting to poll for devices that are stuck managing, and be able to reboot them via script if they get hung.  I have found everything I need so far for this, except do not see a way to pull the HOST IP.  Thanks

At the moment, no. I have no objections to adding this whenever I push the next minor/patch update though.
Until then, you should get by on
from unifi_video import UnifiVideoAPI

UnifiVideoAPI(...).get_camera('Itään')._data['host']

but please don't consider accessing UnifiVideoCamera._data as more than a temporary solution.


dubbo2.7.3条件路由配置不生效 fix #500

Codecov Report

Merging #501 into develop will not change coverage.
The diff coverage is 50%.

@@            Coverage Diff             @@
##             develop     #501   +/-   ##
==========================================
  Coverage      22.55%   22.55%
  Complexity       293      293
==========================================
  Files             80       80
  Lines           3205     3205
  Branches         515      515
==========================================
  Hits             723      723
  Misses          2407     2407
  Partials          75       75

Impacted Files
Coverage Δ
Complexity Δ

.../org/apache/dubbo/admin/common/util/Constants.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...che/dubbo/admin/service/impl/RouteServiceImpl.java
41.46% <50%> (ø)
13 <0> (ø)
:arrow_down:

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 342e3cb...c328e89. Read the comment docs.


Allow to override module-margin-xxx

It's great to have a global setting but sometimes one or two modules need to be fine-tuned and currently the only way I know to do that is using an offset, which is given in pixels and thus far from ideal.

You can work around this by adding spaces to the module labels by surrounding them with quotes, i.e like this: label-urgent  = " %index% " .
It would still be good to have an override, since this doesn't add margin to under/overlines, instead extending them slightly.

Doesn't format-NAME-margin not work for you?

Yes, it does. Probably this request was motivated for the text module but we're already adding a format there. Nevertheless separate left/right margins are missing, but this could be addressed as a limitation of format, couldn't it?

You're right, those are missing for formats. I have added an entry in our refactor project

Doesn't format-NAME-margin not work for you?

I think it doesn't when a local margin < global margin. Same for padding. And the only workaround for that is indeed offset, which messes up with clickable modules (e.g. if you try to use minimal offsets between 3 modules, the second one almost loses its clickability).

Oh it seems I've just found a workaround for that specific case. Turns out only modules with click-* formatting are affected.
Imagine this:
you have global configuration with the following margin settings:
...
[bar/top]
...
module-margin-left = 1
module-margin-right = 2
...

then you have the following section of 2 modules (these scripts show the status of a mic and a specific Bluetooth headset respectively – essentially just an icon / gray icon; the second one also allows to connect/disconnect the headset by left-click):
...
[module/mic]
type = custom/script
exec = ~/.config/polybar/mic.sh
interval = 2

click-left = ~/.config/polybar/mic.sh toggle &

[module/headset]
type = custom/script
exec = ~/.config/polybar/hs.sh "XX:XX:XX:XX:XX:XX"
interval = 2

click-left = ~/.config/polybar/hs.sh "XX:XX:XX:XX:XX:XX" toggle &

format-offset = -10
...

format-margin = 0 doesn't do anything, so you want to use format-offset to make these two set close to each other, however as they overlap, there's only a couple of clickable pixels left for the headset module. To fix this, you can use action handlers instead, and replace this:
click-left = ~/.config/polybar/hs.sh "XX:XX:XX:XX:XX:XX" toggle &

with this:
format = %{A1:~/.config/polybar/hs.sh "XX\:XX\:XX\:XX\:XX\:XX" toggle &:}<label>%{A}

I'm wondering if described behavior of modules with offset and click-* formatting defined is a bug.

I think it doesn't when a local margin < global margin

format margin and module margins compound, they unfortunately don't override each other. That's also why format-margin = 0 doesn't do anything.
@vaygr The thing with the clicks looks like a bug at first sight, could you open a new issue for this so we can track this separately.

@vaygr The thing with the clicks looks like a bug at first sight, could you open a new issue for this so we can track this separately.

While I was developing #1671, I also noticed the same problem by using this string:
%{O100}Filler%{B#ff0000 O100}First%{O200}Second%{O-100}Test
The last part of the module, just after Test, is not clickable.

However after investigation this seems to be the same with lemonbar but in my opinion this behavior is really weird and confusing for the end user.

Sorry for the delay @patrick96, #1814 is there.


Voice calls

Well, I guess I understood how to establish callConnection and get this:
callConnection id:int64 ip:string ipv6:string port:int32 peer_tag:bytes = CallConnection;

But what I should do with it? How I can send sound to my peer and where I should get it?
I guess that all official clients use this: https://github.com/grishka/libtgvoip
But there is no any docs and I don't even know what exactly this thing do.

The question is interesting, but first you need to get acquainted with the technology itself.
https://en.wikipedia.org/wiki/Voice_over_IP

Yes, you need to pass all of these, getOption("call_connect_timeout_ms") and getOption("call_packet_timeout_ms") to libtgvoip, which will be able to manage the call.
We can't help with libtgvoip usage, but source code of official clients, which use libtgvoip, can help a lot.

Thanks for answer, will try

Slishchik great dependence on Grishka.

@L11R did you make any progress with calls? Our Telegram client TELEports also needs that, and as we are a small team, we are really thankful for examples so that we do not need to research all details ourselves :)

@Flohack74 You can find implementation of calls, group calls and voice chats using https://github.com/TelegramMessenger/tgcalls in the source code of official apps or Unigram. Probably, Telegram Desktop's implementation will be the closest to what you need.

So one more question, is the relation of libs like this:
tdesktop
+- libtgcalls
+- libtgvoip
   +- opus

Or is lbtgcalls importing libtgvoip? I am asking because I get tgcalls/Message.h:4:10: fatal error: api/candidate.h: No such file or directory when trying to build it. Which candidate.h is being used?
tgcalls would need a cmake or automake config ^^

Yes, tgcalls imports libtgvoip and libwebrtc.
You can find CMake build scripts used in TDesktop in https://github.com/desktop-app/tg_owt.

According to the comments in #1903 tgcalls replaces libtgvoip. To me it looks like both are dependent on libtgowt but alternative implementations. Is that correct? Does that mean we need libtgowt and tgcalls and can ignore libtgvoip? Or do we need the latter, too, for compatibility reasons?

And do we need https://github.com/desktop-app/lib_webrtc?

@jonnius

To me it looks like both are dependent on libtgowt but alternative implementations. Is that correct?

No. libtgowt contains build scripts for tgcalls.

Does that mean we need libtgowt and tgcalls and can ignore libtgvoip?

libtgvoip is part of tgcalls for compatibility reasons.

Am I looking at the right libtgowt here? Because I can't find tgcalls mentioned there.

Yes, this is the correct libtgowt. You need to link libtgcalls with both libtgvoip and WebRTC. libtgowt provides a way to build WebRTC outside of Chromium project, so you can link tgcalls against libtgvoip and libtgowt besides other libraries.

I think I got it now. libtgowt is a WebRTC fork and I don't need yet another WebRTC if I use libtgowt. And tgcalls depends on libtgowt and libtgvoip. But there is no CMake config for tgcalls anywhere. Correct?
What makes me wonder are some WebRTC includes in Telegram Desktop of headers I can't find in libtgowt, e.g.:
#include "webrtc/webrtc_video_track.h"
#include "webrtc/webrtc_media_devices.h"
#include "webrtc/webrtc_create_adm.h"

But there is no CMake config for tgcalls anywhere. Correct?

Yes, but you can find some clues in all Telegram apps.

What makes me wonder are some WebRTC includes in Telegram Desktop of headers I can't find in libtgowt

They are from lib_webrtc submodule: https://github.com/telegramdesktop/tdesktop/tree/dev/Telegram.

interesting thread...


chore(settings): remove emoji to fix settings bot

:tada: This PR is included in version 1.0.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket: