Refac js components

This PR implements the following:

Split components into individual tsx files
Stores attributes and values inside components' configs
Renames initial_value->value for add_gui_* methods

The goal is gradually moving towards a unified API to add new components.
@brentyi - please let me know what you think.

@brentyi , if you like it, can you please help me with the mypy error? My limited python knowledge is not enough to solve the issue :/

Need to find time to go through this in-depth, but looks good from a skim! Also happy to look into the mypy error...
The main thing I'm uneasy about is the initial_value= => value= API change in the add_gui_*() methods: is this about consistency with other properties / attributes? Would it make a refactor a la #161 much easier down-the-line? Going through with this would mean either an eventual breaking change or indefinitely supporting both names, both of which would be nice to avoid.

The rename is both for consistency and to make #161 easier. If we decide to use code generation for #161 then it wouldn’t matter much. For pure python typing this would help. However, I understand that introducing a breaking change may not be preferable for you. I can undo the name change if you want?

Okay, let's switch it back. Reasoning: it seems unlikely we'll converge to a ParamSpec-based refactor we're happy with, and value is a little bit special (it can be updated by the client) so I'm okay with the inconsistency.

Why is value special? You can change other properties as well: visible, disables, etc., no? I was also thinking about the following use case: in future perhaps we want to enable property binding to make the app faster (bind a property to another component's property).

I undid the api change.

But, initial_value->value is still done in messages! Is that ok?

Why is value special? You can change other properties as well: visible, disables, etc., no?

The difference I was alluding to is that everything in a GuiAdd* message (disabled, hint, initial_value, etc) can only be changed via an assignment from the server. Whereas the value can also be updated by the client.
For the internals and in messages, there are minor but important semantic differences between what it means when we write value and when we write initial_value: when we send a GuiAdd[...] message to update just a single property, for example for dropdowns:
https://github.com/nerfstudio-project/viser/blob/abf6484ffe698a6d2861f10cc75278770d5d77ec/src/viser/_gui_handles.py#L309-L328
we re-send the original initial_value, but this won't overwrite whatever the client has stored if it updates the value of the input. This was convenient at the time because it lets the server update input properties without worrying about things like race conditions. Does this make sense?

I understand the motivation. I still think doing partial updates (updating only changed properties) would have been safer and more principled way to do it. In the case where you set disabled/visible attributes from the client to server, they are not synchronised from server to client, right? Wouldn't it be better to treat all props the same?

I still think doing partial updates (updating only changed properties) would have been safer and more principled way to do it.

Yeah, I agree. I just didn't bother with the initial implementation because sending the full list of properties was much simpler to implement.
Did you have an implementation in mind for doing a partial update? It seems like we could do one of:

Reuse the same message type, but make all of the properties optional so we only need to send what's needed? Use a TypedDict with total=false?
(in the .options example above) Add a new SetGuiDropdownOptions message?
Use a generic SetGuiProp message?

I think the internal initial_value => value name change makes sense if we go with one of these options. I'd be slightly opposed to naming a field value that's actually functioning as an initial_value (eg not overwriting the client-side value when sent from the server).

In the case where you set disabled/visible attributes from the client to server, they are not synchronised from server to client, right?

That's correct, although I was never planning on adding mechanics to set these attributes from the client. If we added some kind of property binding like you mentioned this might become possible, although I'm skeptical the complexity tradeoff here.

I think SetGuiProp is the easiest to implement, although a bit less safe on the python side. However, if eventually the client handles are generic, there should be no risk of introducing bugs this way. The benefit is that the current messages could stay the same.
I believe adding options wouldn't scale that well. Perhaps a generic approach is preferable?
Which one would you prefer?

Or perhaps we can have typeddict props (ButtonProps, TextInputProps) and AddGuiComponent, UpdateGuiComponent messages?

Please let me know what you think now.

getting closer, need to fix some performance regressions
in this branch:
https://github.com/nerfstudio-project/viser/assets/6992947/03da11cc-fd0d-4612-8aaa-10287c4fd5c3
on main:
https://github.com/nerfstudio-project/viser/assets/6992947/016de20c-76f6-4c2a-b4dc-28f4c0401acc


Proper fields alignment in InternalUuidGenerator struct

Fixes #14
InternalUuidGenerator struct did not satisfy alignment requirements.
According to atomic package docs:

On ARM, 386, and 32-bit MIPS, it is the caller's responsibility to arrange for 64-bit alignment of 64-bit words accessed atomically via the primitive atomic functions (types Int64 and Uint64 are automatically aligned). The first word in an allocated struct, array, or slice; in a global variable; or in a local variable (because the subject of all atomic operations will escape to the heap) can be relied upon to be 64-bit aligned.

I swapped fields order, now the struct has proper alignment.
Tested on Raspberry Pi 2 Model B

lgtm

thanks


[BUG]: "preinstall": "npx only-allow pnpm" breaks application

I created a ts library for using in my projects using vite-plugin-dts, but in the last version of vite-plugin-dts was added "strict pnpm": https://github.com/qmhc/vite-plugin-dts/commit/05335e3da6dbb2c1724058acebcb48ffdda143a0, this breaks my apps that use my libry.
E.g., I have my libry @bildvitta/store-adapter that uses vite-plugin-dts, the apps that use @bildvitta/store-adapter are breaking because of the changes in vite-plugin-dts.
This is the error when I try to run my apps:

up

I'm having the same problem!
Why was strict use of pnpm added?

Which npm version did you use?

Which npm version did you use?

6.14.17

6.14.17

Well, the intention of this change just limit the package manager in this project.
This problem seems not occur in npm@8. But I will also remove the preinstall for compatibility with npm@6.

Yes in npm@8 this error not happen, but this is a BREAKING CHANGE, thanks for your help!


using loop leads to "jumps" between last and first slide

filed by @AVVS : https://github.com/glslio/diaporama/issues/1 (old repository)

should be fixed in<EMAIL_ADDRESS>&<EMAIL_ADDRESS>cc @AVVS

I've tested new release and it works pretty well, except for one edge-case. When using it in combination with .next(1000) from the previous release (probably .prev would act the same form last to first slide), time speedup only persists on the part of the original loop, and when extra step that you've added for the loop transition kicks in, time is back to it's standard 1x flow
It's not that big of an issue compared to what it had been before (no transition or a jump when there is a transition), but you can see the inconsistency

good catch

released<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(that's a new minor because duration semantic slightly changed, the last transition time is counted in duration when loop is true)


Error: Invalid formula: internetarchive.rb: cannot load such file -- utils/json

If Homebrew was updated on Aug 10-11th 2016 and brew update always says Already up-to-date. you need to run: cd "$(brew --repo)" && git fetch && git reset --hard origin/master && brew update.
Please follow the general troubleshooting steps first:

[ ] Ran brew update and retried your prior step?
[ ] Ran brew doctor, fixed as many issues as possible and retried your prior step?
[ ] If you're seeing permission errors tried running sudo chown -R $(whoami) $(brew --prefix)?

You can erase any parts of this template not applicable to your Issue.
Bug reports:
I installed linuxbrew today and met the following error:
==> Tapping homebrew/core
Cloning into '/home/dongz/.linuxbrew/Library/Taps/homebrew/homebrew-core'...
remote: Counting objects: 3775, done.
remote: Compressing objects: 100% (3665/3665), done.
remote: Total 3775 (delta 12), reused 340 (delta 1), pack-reused 0
Receiving objects: 100% (3775/3775), 3.05 MiB | 0 bytes/s, done.
Resolving deltas: 100% (12/12), done.
Error: Invalid formula: /home/dongz/.linuxbrew/Library/Taps/homebrew/homebrew-core/Formula/internetarchive.rb
cannot load such file -- utils/json
Error: Cannot tap homebrew/core: invalid syntax in tap!
Error: Failure while executing: /home/dongz/.linuxbrew/bin/brew tap homebrew/core

Propose a feature:
Instead of creating an issue here, please create a pull request with your change proposal in the Homebrew Evolution repository using the proposal template.

I can confirm this bug. I'll try to fix it up this weekend. Sorry for the gaff.

Thanks!
On Sat, Dec 3, 2016 at 4:29 PM Shaun Jackman<EMAIL_ADDRESS>wrote:

I can confirm this bug. I'll try to fix it up this weekend. Sorry for the
gaff.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Linuxbrew/brew/issues/194#issuecomment-264666766, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAmTiblitVianLYzIvMtCrgT9ozNRVL8ks5rEd9RgaJpZM4LDaIY
.

This issue will soon be resolved by Linuxbrew/homebrew-core#1231.

Awesome. The merged stuff works like charm. :tada:
IMHO this issue can be closed now.

Thanks for the follow up, Alexander.


[#621] Impose Mempool maximum capacity

Closes #621.

What's the status on this? Are you still working on a simplification of the logic? We shouldn't let it drag on for too long

I've mostly just been rebasing this on master every few days to keep it up-to-date. I will take a fresh look at simplifying the logic today. However, last time I went through it, everything seemed to make sense to me. I'll see what I can perhaps factor out into other functions, etc. in an attempt to increase readability.

@intricate We should not let this bitrot too long anymore. I'd like to merge it early next week.
It would be good to tweak the consensus tests to trigger the new code path:

Decrease the mempoolCap in Test.Dynamic.Network to, e.g., 10
Increase the number of maximum number of generated transaction from 1 to 20 in https://github.com/input-output-hk/ouroboros-network/blob/e3a2401aa324afe1287c89d827a92f2e784a3da0/ouroboros-consensus/test-consensus/Test/Dynamic/TxGen.hs#L48

bors r+


Exclude jdk_security4 test on jdk8 linux

This test is excluded on jdk11 https://github.com/adoptium/aqa-tests/blob/27e4c1a5a389a783c09d04ad2649c8f444cd22f4/openjdk/excludes/ProblemList_openjdk11.txt#L98 due to known upstream issue.
We're seeing it on jdk8 in the dry run https://github.com/adoptium/aqa-tests/issues/5213#issuecomment-2051506962
Will merge this into the release branch when approved

Happy to approve the exclusion.
Would also like some additional info, https://bugs.openjdk.org/browse/JDK-8258855 mentions 2 other testcases but not the one being excluded, would be good to confirm/document the original exclusion and this one fails in the same way (with the same exception)
From JDK-8258855
java.lang.Exception
at ReplayCacheTestProc.main0(ReplayCacheTestProc.java:278)
at ReplayCacheTestProc.main(ReplayCacheTestProc.java:325)
at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:78)
at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.base/java.lang.reflect.Method.invoke(Method.java:567)
at com.sun.javatest.regtest.agent.MainWrapper$MainThread.run(MainWrapper.java:127)
at java.base/java.lang.Thread.run(Thread.java:831)

Noting that https://bugs.openjdk.org/browse/JDK-8262033 was closed as won't fix.  If that is true, what does that mean to us, are the implications that we'd be excluding permanently.

Closing as it looks like an infra problem. Its passing on some machines https://ci.adoptium.net/job/Grinder/9602/ and failing on others https://ci.adoptium.net/job/Grinder/9603/testReport/. Will continue this in an infra issue

Infra issue raised here https://github.com/adoptium/infrastructure/issues/3521

And yes, thanks once this is merged, it can be cherrypicked to the v1.0.1-release branch.


keep debug version

Remove the key when we're done with it... just in case.
This should upload a debug artifact with each release, yet to be tested.

@skurfer we're getting:

Error: HTTP status code: 403. A required agreement is missing or has expired. This request requires an in-effect agreement that has not been signed or has expired. Ensure your team has signed the necessary legal agreements and that they are not expired.

Are we okay to keep using your certificate? If so, do you get any notification when you log into your Apple Dev account?

Sorry, just now getting to this. The developer account had expired. I just renewed it. I’ll e-mail you my cell so you can get to me faster for stuff like this.

I'll probably cut a fake release to make sure this works and if so merge.
I'm really hoping a debug version available for download helps sort out these intermittent crashing issues.

The code here looks good to me. However it appears the developer cert still needs updating.
@n8henrie did you manage to get the login details from @skurfer for the dev account?
If yes: Rob - you need paying for this year.
If no: We can set up a new developer account to get builds signed (regardless, Rob should still be reimbursed).

@pjrobertson @skurfer I did not get account details, Rob renewed and I suspected it would just start working again, but the signing is giving an error:

A required agreement is missing or has expired. This request requires an in-effect agreement that has not been signed or has expired. Ensure your team has signed the necessary legal agreements and that they are not expired.

Rob, if you log into the account, does it present you with some kind of EULA?

I accepted the agreement and qssign is working locally for me.

Signing is not working, thanks @skurfer
Looks like I need to fix the debug artifact: Error: Unable to find an artifact with the name: Quicksilver-debug.zip

Hope nobody minds the quickfix pushes to main for this, CI stuff is kind of painful to work with via branches (since the branch itself often determines the CI process).


[1.1.x] carbon line format float handling

Backports the following commits to 1.1.x:

carbon line format float handling (#740)

Yes, of course. Almost forgot that.


Help command keeps sending '\n' characters on serial terminal

https://github.com/apache/nuttx-apps/pull/1610 introduced an error when help command is thrown on ESP32 serial terminal.
How to reproduce
just use esp32-devkitc:nsh defconfig and throw help cmd.
nsh>
     nsh> help
              help usage:  help [-v] [<cmd>]

    .         cd        dmesg     free      mkrd      rm        time      uptim
    [
    ?
    basename
    break
    cat
    cd
    cp
    cmp
    dirname
    dd
    df
    dmesg
    echo
    env
    exec
    exit
    false
    free
    help
    hexdump
    kill
    ls
    mkdir
    mkrd
    mount
    mv
    printf
    ps
    pwd
    rm
    rmdir
    set
    sleep
    source
    test
    time
    true
    truncate
    uname
    umount
    unset
    uptime
    usleep
    xd

Note: tested with minicom on Linux.

Fixed with https://github.com/apache/nuttx-apps/pull/1621


feat: Add Keptn auto-detection

This PR

Adds an auto-detection flag to the JES installation that is able to detect a single Keptn instance and extract the API token from it

Fixes

#209

Just tried this locally using skaffold, k3d, and a fresh Keptn 0.13.x installation, and got this error:
Helm release job-executor-service not installed. Installing...
Error: INSTALLATION FAILED: template: job-executor-service/templates/secrets.yaml:7:20: executing "job-executor-service/templates/secrets.yaml" at <include "job-executor-service.remote-control-plane.token" .>: error calling include: template: job-executor-service/templates/_helpers.tpl:118:34: executing "job-executor-service.remote-control-plane.token" at <include "job-executor-service.remote-control-plane.namespace" .>: error calling include: template: job-executor-service/templates/_helpers.tpl:100:111: executing "job-executor-service.remote-control-plane.namespace" at <"app.kubernetes.io/part-of">: invalid value; expected string
deploying "job-executor-service": install: exit status 1

Just tried it on a fresh installation with k3d, and got this:
Error: INSTALLATION FAILED: create: failed to create: namespaces "keptn-jes" not found

adding
    flags:
      install: ["--create-namespace"]

fixed it for me, and it's working :+1:


Request: publish / open source AndroidSwipeRefreshLayout

Seems like you guys have some pull to refresh mechanism built and working... I would love to see this published and open sourced, as there is nothing out there right now that works with Android
Thanks!
      <AndroidSwipeRefreshLayout
        {...this.props}
        ref={RK_SWIPE_REF}
        style={styles.base}
        onRefresh={this._onRefresh}>
        {childrenWrapper}
      </AndroidSwipeRefreshLayout>

+1

AndroidSwipeRefreshLayout is not perfect, and this will be support officially.


Recipe for building

Hi all,
Can you release a quick writeup / dockerfile on how to build my custom version from these sources ?
Just asking, do not want to reinvent the wheel :-)
Thanks
Rudi

found it use gorelease :-)


Update icsm.conf surveyfeatures

Hey @edmondchuc :

The process now is to merge first into staging, so I rebased the PR.
The test suite run failed. Have a look in the details (click the "x" next to the commit SHA, then "Details" to see the log, then scroll to the end).

Thanks @rwalkerands, I've made a change to the test but doesn't look like it gets re-triggered.

Yeah, nuh. You (correctly) updated the label of the test, but you must also update the "from" value within in the test.

Yes, whoops. Ok, it's fixed now.

Every new commit within this repo does trigger a run of the test suite. But, because of the way I've had to implement the automation, there's a polling of this repo that only runs every 3 minutes. So: sorry, please be patient; you may have to wait up to (say) 3.5 minutes to get the test results.

OK, over to Nick now.

Deployed.


Overseerr could not be found

Have the following code in my docker stack for overseerr:
      - DOCKER_MODS=ghcr.io/gilbn/theme.park:overseerr
      - TP_THEME=plex
      -

when I go to start overseerr container I get the following error message, not sure if an issue or something I have done wrong, Thanks.
[mod-init] Attempting to run Docker Modification Logic
[mod-init] Applying gilbn/theme.park:overseerr files to container
[mod-init] gilbn/theme.park:overseerr could not be found on ghcr.io
[migrations] started

Overseer doesnt have a mod


Add Type checking and enforce lint

Adding do now.
Regarding pre-commit hooks. I am not a big fan, more than once, I wanted to commit something as a state save and hooks went in my way.

You don't need to install them - you can even run them manually - but it helps running the same checks at the CI pipeline. Can we at least put it in a Makefile or something so people can run make test manually locally and be confident the CI will pass?

Done!


Fix static urls in blog

i can't currently test this locally (didn't get jekyll to run yet) but it should fix the blog styles.

i see it's already fixed in master

Yes, I just need to deploy. Not so easy, I switched computers and my maven
setup is borked
On 14 May 2015 15:57, "LuzianW"<EMAIL_ADDRESS>wrote:

i see it's already fixed in master
—
Reply to this email directly or view it on GitHub
https://github.com/caspii/kittysplit-blog/pull/2#issuecomment-102044556.

Deployed.


Add a donate button

So after 12+ years of developing Java Mail, hosting the (no-adds) simplejavamail.org and pouring in many many... many private hours, I finally decided to include a donation button for whomever wants to buy me a beer.
Cheers!

Button is up!


Fix gcc7 compilation error

Add #include  to fix compilation errors with GCC7.
(error: ‘function’ in namespace ‘std’ does not name a template type)
I see "not accepting pull requests" message but already prepared it.
Please ignore if you decide.
Pull requests
We are not accepting pull requests or community contributed bug fixes.
To ask a question or report an issue, please review our Contribution Guide.

Thanks for the pull request! At this time, we are only considering pull requests that are bug fixes.


how to  filter array feilds with includeFields?

{
  "list" : [
    {"name": "a", ...}
    {"name": "b", ...}
    {"name": "c", ...}
  ]
}

 "includeFields": [ "list.name", "list.0.name", "list.$.name", "list.*.name", "list.nested.name"] not work

mongo-connector does not support filtering fields inside arrays, you can only include or exclude the entire array field.
I added a note in the filtering documentation about this.

PUT /xx/oo/_mapping
"_source": {
    "includes": [
      "list.name"
      ]
  },
"properties": {
  "filelist": {
      "dynamic": false,
      "properties": {
        "name": {
          "type": "text",
          "analyzer": "ik_max_word",
          "search_analyzer": "ik_max_word"
        }
      }
    }
}

it works to me

Your example further shows that mongo-connector does support filtering insides arrays; the array field in MongoDB was improperly converted into an Elasticsearch object. If mongo-connector supported this kind of filtering, the array field would still be an array in Elasticsearch.


Check performance warnings of CNNs

test_trainer gives the following warnings.
We should investigate and either disable them or make the recommended fixes.

tests/test_trainer.py::TestTrainer::test_grid_regression
/home/dbodor/git/DeepRank/deeprank-core/deeprankcore/dataset.py:376: UserWarning: Creating a tensor from a list of numpy.ndarrays is extremely slow. Please consider converting the list to a single numpy.ndarray with numpy.array() before converting to a tensor. (Triggered internally at  /opt/conda/conda-bld/pytorch_1659484809662/work/torch/csrc/utils/tensor_new.cpp:201.)

and

tests/test_trainer.py::TestTrainer::test_grid_regression
/home/dbodor/git/DeepRank/deeprank-core/deeprankcore/utils/exporters.py:260: PerformanceWarning: your performance may suffer as PyTables will pickle object types that it cannot map directly to c-types [inferred_type->mixed,key->block1_values] [items->Index(['phase', 'entry'], dtype='object')]

As @sonjageorgievska was pointing out, it seems that we use lists of numpy arrays and then we convert them to tensors. Instead, we should first convert them to numpy arrays of numpy arrays (numpy arrays with more dimensions) and then to tensors.
See also this blog post about performances comparison between numpy arrays and python lists.

The first performance warning will be fixed by #438


added option to use the frames - python demo2.py image

You can now use the python demo with the demo images.
python demo2.py image
and moved cv2.destroyAllWindows() to the end of file
Cheers
Hannes

Thank you @HannesOberreiter for the contribution !


master contour routing and TLS versions

by accident I had the "master" contour in my system, rather than "latest". so your recent commits did get some extra testing in a real system, yeah!
We did experience an issue with this issue of contour, it somehow made the corresponding envoy talk to the backing esp in a way that used TLS, but the backing esp never does speak TLS.
[2018-05-15 13:05:16.934][14][debug][connection] source/common/ssl/ssl_socket.cc:138] [C4] SSL error: 268435703:SSL routines:OPENSSL_internal:WRONG_VERSION_NUMBER

Sorry I do not have any more details and logs, but perhaps it's useful as a test case anyway....

@robertlemmen thanks for raising this issue. I'm going to need a few more details before I can properly debug this one. From what you've told me so far it looks like the backend Service talks TLS but Envoy tried to talk to it via plain HTTP.

[ ] Can you please attach a copy of your Ingress and Service documents.
[ ] Does this problem only occur with :master?

@robertlemmen gentle ping

as of a few days ago, the problem also happens with latest, simply because that moved ahead. btw, we were kinda expecting latest to point to releases, so e.g. v0.5.0 at the moment, and were a bit surprised when the alpha release got distributed as latest...
my ingress controller basically looks like this:
    spec:
      containers:
      - args:
        - -c
        - /config/contour.yaml
        - --service-cluster
        - cluster0
        - --service-node
        - node0
        - -l
        - info
        - --v2-config-only
        command:
        - envoy
        image: docker.io/envoyproxy/envoy-alpine:v1.6.0
        imagePullPolicy: IfNotPresent
        name: envoy
        ports:
        - containerPort: 8080
          name: http
          protocol: TCP
        - containerPort: 8443
          name: https
          protocol: TCP
        resources: {}
        terminationMessagePath: /dev/termination-log
        terminationMessagePolicy: File
        volumeMounts:
        - mountPath: /config
          name: contour-config
      - args:
        - serve
        - --incluster
        - --ingress-class-name
        - contour-mg
        command:
        - contour
        image: gcr.io/heptio-images/contour:latest
        imagePullPolicy: Always
        name: contour
        resources: {}
        terminationMessagePath: /dev/termination-log
        terminationMessagePolicy: File
      dnsPolicy: ClusterFirst
      initContainers:
      - args:
        - bootstrap
        - /config/contour.yaml
        command:
        - contour
        image: gcr.io/heptio-images/contour:latest
        imagePullPolicy: Always
        name: envoy-initconfig
        resources: {}
        terminationMessagePath: /dev/termination-log
        terminationMessagePolicy: File
        volumeMounts:
        - mountPath: /config
          name: contour-config
      restartPolicy: Always
      schedulerName: default-scheduler
      securityContext: {}
      serviceAccount: default
      serviceAccountName: default
      terminationGracePeriodSeconds: 30
      volumes:
      - emptyDir: {}
        name: contour-config

and the service that it talks to:
   spec:
      containers:
      - env:
        - name: GOOGLE_APPLICATION_CREDENTIALS
          value: *******
        image: ********
        imagePullPolicy: Always
        livenessProbe:
          failureThreshold: 3
          httpGet:
            path: /health/liveness
            port: 50052
            scheme: HTTP
          periodSeconds: 10
          successThreshold: 1
          timeoutSeconds: 1
        name: master
        ports:
        - containerPort: 50051
          protocol: TCP
        readinessProbe:
          failureThreshold: 3
          httpGet:
            path: /health/readiness
            port: 50052
            scheme: HTTP
          periodSeconds: 10
          successThreshold: 1
          timeoutSeconds: 1
        resources: {}
        securityContext:
          privileged: false
        terminationMessagePath: /dev/termination-log
        terminationMessagePolicy: File
        volumeMounts:
        - mountPath: ***********
          name: google-credentials
          readOnly: true
        - mountPath: **********
          name: config-volume
          readOnly: true
      - args:
        - --http2_port
        - "8081"
        - --backend
        - grpc://<IP_ADDRESS>:50051
        - --service
        - **********************
        - --version
        - 2018-04-16r0
        image: gcr.io/endpoints-release/endpoints-runtime:1
        imagePullPolicy: Always
        name: esp
        ports:
        - containerPort: 8081
          name: external-port
          protocol: TCP
        resources: {}
        terminationMessagePath: /dev/termination-log
        terminationMessagePolicy: File

downgrading contour to v0.5.0 seems to avoid the problem. note that the problematic requests never arrive at our actual service, it's teh communication between envoy and the esp that is the problem
thanks for looking, let me know if you need anymore information

@robertlemmen I'm sorry this issue has been open for so long without a response.

See docs/tagging.md for our tagging policy, :latest now points to the most recent stable release

I'm going to close this issue as almost all the code that controls SSL has been rewritten since 0.5. Please feel free to reopen this if you are experiencing problems with the latest version of Contour.


tensorflow-lite fixes

This PR makes it possible to compile libcamera-apps from meta-raspberrypi with tflite support.

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #85. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.

:octocat: cibot: @vivien, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/meta-neural-network/ci/repo-workers/pr-checker/85-0-664b/.

For your needs, I'd recommend to make a "meta package" that creates symbolic links from tensorflow2-lite* to tensorflow-lite* for the compatibility of both tensorflow2* users and tensorflow* users.

I added the following to the concerned layer (here meta-raspberrypi) and this works fine:
$ cat meta-raspberrypi/dynamic-layers/neuralnetwork/recipes-tensorflow/tensorflow-lite/tensorflow-lite_2.%.bbappend
do_install:append () {
	ln -sf libtensorflow2-lite.a ${D}${libdir}/libtensorflow-lite.a
	ln -sf tensorflow2-lite.pc ${D}${libdir}/pkgconfig/tensorflow-lite.pc
}

But I do not think that requiring this for all layers using tensorflow-lite is the correct packaging approach, see below.

We have a few systems that install both tf-lite 1.x and tf2-lite 2.x, which forced us to install this file as tensorflow2-lite.pc.

I understand your use case, however the package seems broken for a few reasons:

the project builds libtensorflow-lite-bundled.a, not libtensorflow2-lite-bundled.a
headers are installed into ${includedir}/tensorflow/lite, regardless of the version
${includedir}/tensorflow/core/public/version.h is installed regardless of the version
${includedir}/flatbuffers/include/flatbuffers/*.h are installed regardless of the version

The r2.* branches are just a version bump from the tensorflow-lite project, not a different project.
Because you have a special scenario, I would suggest that you install the various versions in different prefixes on your system (e.g. /opt/tflite/1.12, /opt/tflite/1.15, /opt/tflite/2.3, /opt/tflite/2.11, etc.) and configure your system or applications to look for the correct version there.
A way to do that is to have a dynamic tensorflow-lite_%.bbappend that is enabled only for your use case, which moves the installed files to /opt/tflite/${PV} for example.
What do you think?


Hi, after some weeks of trying today I finally synced my Verge Wallet to the blockchain. However, on the 6th of February I sent XVG from my exchange wallet to my verge wallet address (this is evidenced on the verge-blockchain.info). And this transaction is not displaying in my wallet address. Is there anything that I can do to have this balance updated? Or does it just take some time? Please advise. Thanks

Issue Template
Context

Prerequisites

[ ] Are you running the latest version?

User Story:

"I'm submitting a ... "

[ ] bug report
[ ] feature request
[ ] support request
[ ] translation request

Expected Behavior

Current Behavior

Screenshots

Possible Solution

Steps to Reproduce (for bugs)

Steps :

Your Environment

Verge Release:
CPU:
GPU:
RAM:
OS:

[ ] I'm using Mac
[ ] I'm using Linux
[ ] I'm using Mac

Any extra information that might be useful in the debugging process.

My Wallet is VERGE-QT 4.0

Below is the timestamp of the tranaction.
Timestamp Hash Amount (XVG)
6th Feb 2018 04:00:08 b656091dd60dc515fbd435d29d29979f3b359cf900f5a866257fc1be65ebbccb +<PHONE_NUMBER>0

and you see this address in your receiving tab? and your wallet is not locked?

Hi Justin, thanks for getting back to me. I just opened & synced my wallet again & opened it yet it's still not showing. The history is this. I downloaded the wallet & then sent my coins from my coinspot account to the wallet address. So I sent them to the address that is stated in my wallet before the wallet was synced to the blockchain. Is it safe to send you a screenshot of this that includes my wallet address? Thanks

Also, I just unlocked the wallet yet its still showing o balance.

Here's a screenshot of the transaction without my personal wallet address that can be found in the verge-blockchain.info

This is a screenshot of the transaction from my coinspot account to my verge wallet. Note: I erased my wallet address. This will correlate with the above verge-blockchain.info transaction. Also, I'm in Australia in another time zone, so apologies in advance if our communications are spread over a time period, though I really appreciate your help. If you need any further information for verification please let me know. Thanks Shaun

Why would you think that:

no this does not show in my receiving address. Nothing shows in my receiving address.

But then in your screenshot -- you seem to have that address :smile:
couple things:

Please post a screenshot of help -> debug window ... the information tab
Try going to help -> debug window -> console  and do 2 things:
a)  type listaddressgroupings
b)  type scanforalltxns  ... this will take a while,, maybe 10-30 mins, dont close out, let it finish, it will finish

Hi Justin, I just started it. I will let you know the result when the debug finishes. Thanks

Dear jrsikors & Justin, thanks so much for your help! Debugging completed and my coins showed up! I really appreciate the work that you do & I'm looking forward to an exciting future with Verge! Thanks again & have a great day/night!

No prob -- for the love of god back up of your wallet.dat file .. and/or private key ... please :smile:

@jrsikors thanks for jumping on issues!


Oceanize Attempt

Me tried to replace all normal biomes into ocean type biome, and... land still generated. (Mod v1.2, Forge v1.20.1-47.3.0)

Nvm, guess me put in the wrong place.


Attempting to follow Model parallelism example in the docs results in AttributeError

I am trying to follow along the example here:
https://keras.io/api/distribution/model_parallel/
import tensorflow as tf
import tensorflow.keras as keras
import tensorflow.keras.distribution as kd

if __name__ == "__main__":
    devices = kd.list_devices(device_type="gpu")
    device_mesh = kd.DeviceMesh(shape=(1, 2), axis_names=("batch", "model"),
                                devices=devices)

    layout_map = kd.LayoutMap(device_mesh)
    layout_map["dense.*kernel"] = (None, "model")
    layout_map["dense.*bias"] = ("model",)

    layout_map["conv2d.*kernel"] = (None, None, None, "model")
    layout_map["conv2d.*bias"] = ("model",)

    distribution = kd.ModelParallel(device_mesh=device_mesh,
                                    layout_map=layout_map,
                                    batch_dim_name="batch")

    set_distribution(distribution)

When I execute the example code, it fails on line 17, with the following error message:
  File "/app/workdir/test.py", line 17, in <module>
    distribution = kd.ModelParallel(device_mesh=device_mesh,
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/dist-packages/keras/src/distribution/distribution_lib.py", line 549, in __init__
    self._num_process = distribution_lib.num_processes()
                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: module 'keras.src.backend.tensorflow.distribution_lib' has no attribute 'num_processes'

and I am not sure where to start with troubleshooting.
I ran this code in tensorflow/tensorflow:nightly-gpu docker container out of the box, and didn't install or remove anything. It has tf-nightly version 2.16.0.dev20231221 and keras-nightly version 3.0.2.dev2023122103. Is there a compatibility issue somewhere? I also tried import keras and import keras.distribution instead of import tensorflow.keras ... as well but I get the same error message.
Thank you in advance.

Just stumbled into this, 7 months later, and there still is the same error?  Is that expected?  Surely this issue should be closed only once the feature is working?

Have you set a keras backend? e.g. for jax you will need os.environ["KERAS_BACKEND"] = "jax"

I just ran this example, at least in TPUs, tf 2.18 and keras 3.5 this still fails with the same error.
Should it be re-opened and maybe the docs for Keras-Distributed updated @fchollet ?
Here is a sample-code for colab (otherwise just use the example in the docs.):
!pip uninstall tensorflow-text tf-keras trax -y
!pip install -U tf-nightly keras-nightly jax -q

import keras
from keras import layers
import tensorflow as tf
import time
import jax
import numpy as np

tf_data = tf.data

devices = jax.devices("tpu")
print(devices)

data_parallel = keras.distribution.DataParallel(devices=devices)

inputs = np.random.normal(size=(128, 28, 28, 1))
labels = np.random.normal(size=(128, 10))
dataset = tf_data.Dataset.from_tensor_slices((inputs, labels)).batch(16)

# Set the global distribution.
keras.distribution.set_distribution(data_parallel)

# Note that all the model weights from here on are replicated.
# The dataset will be split evenly on the batch dimension
inputs = layers.Input(shape=(28, 28, 1))
y = layers.Flatten()(inputs)
y = layers.Dense(units=200, use_bias=False, activation="relu")(y)
y = layers.Dropout(0.4)(y)
y = layers.Dense(units=10, activation="softmax")(y)
model = keras.Model(inputs=inputs, outputs=y)

model.compile(loss="mse")

s = time.time()
model.fit(dataset, epochs=3)
e = time.time()
print("RUN TIME:   ", s-e)
model.evaluate(dataset)

I just realised about this sentence in the examples:

The distribution API is only implemented for the JAX backend for now.

The distribution does work for JAX, at least on TPUs.

Here is what I ran in Colab @optiluca @MangledPotat0 :

pip uninstall tensorflow-text tf-keras trax -y
!pip install -U tf-nightly keras-nightly jax -q

import os
# os.environ["KERAS_BACKEND"]='jax'
os.environ["KERAS_BACKEND"]='tensorflow'

import keras
print(keras.backend.backend())
from keras import layers
import tensorflow as tf
import time
import jax
import numpy as np

print(keras.__version__, tf.__version__, jax.__version__)
tf_data = tf.data

print(keras.backend.backend())

devices = jax.devices("tpu")
print(devices)

data_parallel = keras.distribution.DataParallel()

inputs = np.random.normal(size=(128, 28, 28, 1))
labels = np.random.normal(size=(128, 10))
dataset = tf_data.Dataset.from_tensor_slices((inputs, labels)).batch(16)

# Set the global distribution.
keras.distribution.set_distribution(data_parallel)

# Note that all the model weights from here on are replicated.
# The dataset will be split evenly on the batch dimension
inputs = layers.Input(shape=(28, 28, 1))
y = layers.Flatten()(inputs)
y = layers.Dense(units=200, use_bias=False, activation="relu")(y)
y = layers.Dropout(0.4)(y)
y = layers.Dense(units=10, activation="softmax")(y)
model = keras.Model(inputs=inputs, outputs=y)

model.compile(loss="mse")

s = time.time()
model.fit(dataset, epochs=3)
e = time.time()
print("RUN TIME:   ", e-s)
model.evaluate(dataset)

Using nvidia's jax:24.10-py3 container, I am now able to run the above code without error using 2 GPUs. However, I am also not sure that it's actually utilizing the additional hardware properly? It's definitely using the VRAM of both GPUs, but one is doing all the work while the other one sits at 0% utilization during training. Maybe this is the intended behavior? Though I expected both GPUs to have even VRAM and compute utilzation


Unknown functionality about mobile networking

On some devices there's the bug that the app is unable to communicate with online services. Persists through onboarding.
This error occurs on Android 6.0 EMUI 4.0.3

This issue seems not to persist when you add the
<uses-permission android:name="android.permission.ACCESS_NETWORK_STATE"/>
to manifest.

https://github.com/dhbw-timetable/dhbw-timetable-android/commit/b8fb27fa5870fff7cb1cb85952e387c0ed27c3bc should have a fix for this. There's also extended debug for this issue (exception shown to user).

I have tested this with a physical test device. Works like charm.


[MRG] Add docs for Figshare and Dataverse; update Dataverse installations file formatting

This provides the docs requested in https://github.com/jupyter/repo2docker/pull/739#issuecomment-532534037 and also formats and sorts the entries in the JSON file with Dataverse installations. The latter has the advantage that we'll be able to track changes in the installation list without relying on the structure on the server, only changing the lines that did change, in a human-friendly way (at the cost of some whitespace).
@pdurbin @Xarthisius

@nuest these docs look great. I'm sort of wondering how the dataverse.json file gets update over time but I'm glad to see it's up to date with all 48 Dataverse installations represented!
On a related note, can I ask you to please take a look at https://github.com/jupyterhub/binderhub/issues/900 ?


Testing GNSS-denied navigation with VO

This PR adds the possibility to run multiple EKF instances, so that there are separate instances for GNSS-based navigation and navigation with only visual odometry. The parameter EKF2_MULTI_VIS enables or disables this functionality.
Base functionality
The existing pattern used for multi-imu and multi-mag EKF is extended with another dimension for multi-vision. EKF instances configured for vision discard GNSS samples, and instances configured for GNSS discards optical flow and visual odometry samples.
An exception is made while disarmed, where the vision-based estimators will get GPS data, in order to automatically initialize position during testing.

[ ] TODO: For future flights completely without GPS, position (and heading, if we decide to fly without mag), is to be initialized using user set lat/lon/alt/heading parameters.

Switching between GNSS and vision estimators
For initial testing, we don't want to fly on the vision estimators. For flights using vision estimators, the flight mode is used to determine the EKF selection. This is simple to implement, simple to override for an operator in QGC, and ensures that "emergency" modes like land and RTL use GNSS position.
General safety concerns

Ensure that GPS_MULTI_MAG=1 when using GPS_MULTI_VIS to prevent running too many EKF instances

Could add this in code, but feels like it belongs on the parameter level. We generally don't expect every combination of parameters to work well.

Safety concerns for fligts using vision estimators

[x] Geofence

GF_SOURCE_GPS must be set to GPS instead of Global Position.

[x] Prevent erroneous termination

[x] Verify that any termination failsafe don't use vision estimate
[x] Verify parachute altitude decision alt source

In addition to @kibidev's comments, I think we need a brief failure analysis to make sure we aren't missing something. It's just an NX02 drone, but it's difficult to wrap my head around possible failure modes.

@hermankolden

In addition to @kibidev's comments, I think we need a brief failure analysis to make sure we aren't missing something.

Working on it here: https://aviant.atlassian.net/wiki/spaces/TECHNICAL/pages/992804893/GNSS+VO+switch+in+PX4+Function+testing+and+failure+analysis

Most of the PR comments should be handled now. I'll squash everything before merging, but leaving the commits for now to make review easier.

Good points @kibidev. I handled the comments now, and added logic to constrain the parameter to (0, 1), inspired by the EKF2_MULTI* parameters

Updated the branches now. Good catch on the magnetometer selection, that should be fixed before flight. I'll look into it today, maybe it's good enough to simply deactivate the internal magnetometer for the sprind firmware.

It seems that SENS_MAG_MODE=1 works fine (at least in sim), so I think we should use that instead of reducing EKF_MULTI_MAG.


Update gapic-generator hash

@alexander-fenster I'll include them in the PR that updates artman version :)


Support console.clear(). Addresses #11.

@andyarvanitis @nwolverson Any thoughts on this with regards to alternative backends?

Erlang - could be supported in some circumstances via control codes depending on your terminal. Don't see a problem with such a function which may be NOOP depending on your platform, it's not like the appearence of everything else is tightly specified.

@paf31 I think that on node it will trigger a runtime error. Perhaps I should modify the JS code to suppress this?

There's no standard way to do this in the C world. As mentioned, a no-op is possible, but I'd suggest keeping it out of this package and maybe creating a separate platform-specific console package.

@andyarvanitis I'm happy to do that.

Moved here: https://pursuit.purescript.org/packages/purescript-console-browser-specific/0.0.1


Add lint CI

See also: https://github.com/apache/arrow-go/blob/main/.github/workflows/lint.yml

Is there any existing java lint CI in the apache/arrow? I did a quick but found nothing. @lidavidm @kou

It's probably  part of the big docker lint job, let me check

Ah, no it isn't: Lint C++, Python, R, Docker, RAT

We're using Spotless: https://github.com/apache/arrow/issues/40757
I think that it's integrated with Maven.

We could possibly make a lint job that runs with warnings-as-errors and has error-prone enabled (though IIRC we aren't yet warning-clean)


Add Support for Long Paths Closes #62

Sorry about the messy commit history, I do not use Visual Studio and everything got completely fucked. Small change, only have to update the app.config as noted here:
https://docs.microsoft.com/en-us/archive/blogs/jeremykuhne/net-4-6-2-and-long-paths-on-windows-10
Closes #62  which I came here trying to figure out for the Yakuza games as well.

Hey, this looks great, did not know about this setting, and couldn't reproduce on my side so i was a little bit lost with this bug, Thanks for the PR!

No problem @BrianLima  and thanks for the quick response. I am having some other issues that I think might be related to Windows 11, might open an issue but what all information would you want me to include on it as far as debug/logging info?
The issue I am having is that I can add titles from Gamepass no problem to Steam, they show up, etc. But when I try to play anything I hear the game start on my host PC via remote play, but the client PC just closes it out. The game does launch properly on my host PC and keeps playing though. Maybe I misunderstood this project, does it not work with Remote play, or is it maybe some sort of Windows 11 issue? Thought it was the file length issue thats why I fixed this bug :p

If you enable the streaming mode in the settings, the app tries to enlarge it's window and wait for steam to start streaming. It might not actually work since steam does not see UWPHook as an usual game and more like a desktop application (which it is) causing it to not send controller inputs properly to the host computer, converting the controller buttons to mouse and keyboard inputs. Some people suggest using VirtualHere for remote streaming controller games, essentially i'm tricking Steam into streaming the whole desktop.
I should probably try to figure out a better way to support streaming gamepass games but i have not caught wind of any newer methods...

Huh, I didnt even see that streaming option. Redid the games with the streaming option, it definitely goes further but still crashes eventually on the client PC. I will check out VirtualHere though, mostly want to play some gamepass games in my living room without buying a very long HDMI :p

Hope you get it working! Your PR has been published @ v2.9.2 https://github.com/BrianLima/UWPHook/releases/tag/2.9.2


doc: Make it clearer what should be gitignored

https://ij.bazel.build/docs/project-views.html

Source control
*.bazelproject files can be checked into source control. By convention, they live next to the BUILD file people import. While many teams name the file simply .bazelproject, we recommend adding a prefix like myteam.bazelproject since Piper treats dotfiles differently (most importantly, it won’t automatically add them to your CLs).
If your project has multiple different views, you may check in multiple files with the .bazelproject extension into the same directory. The user can choose to import any one of these.

The documentation should make it clarify whether the .ijwb directory should be checked in.
(I believe it shouldn't? But it can be, and if you do, you should copy https://github.com/github/gitignore/blob/main/Global/JetBrains.gitignore and commit it as .ijwb/.gitignore …?)

I found some guidance in this blog post: https://blog.bazel.build/2019/09/29/intellij-bazel-sync.html#portability-of-the-ijwb-directory

Portability of the .ijwb directory
The .ijwb directory is not completely portable. Files like .bazelproject and codeStyleConfig.xml can be shared project-wide, but workspace.xml and .workspace.iml should be user-specific.
In general, extract .bazelproject file out of .ijwb/ to version control it, and follow JetBrains' recommendations on checking in specific files in the .idea directory.

I think the project view files are designed to encapsulate everything you'd need from the .ijwb directory. In particular, run configurations are shareable.
It feels like the .ijwb directory contains state that is already either in the Bazel graph, or the project view itself. I'd be more in favour of allowlisting what to save via the project view, because that way multiple projects can coexist in the same repo with just one file for each.


Roll Flutter from 1b58a593deac to 9211d7cc8a0d (141 revisions)

Roll Flutter from 1b58a593deac to 9211d7cc8a0d (141 revisions)
https://github.com/flutter/flutter/compare/1b58a593deac...9211d7cc8a0d
2022-04-28<EMAIL_ADDRESS>Roll Engine from 7ffb3c560939 to 966757c6cc8e (1 revision) (flutter/flutter#102740)
2022-04-28<EMAIL_ADDRESS>Roll Engine from b62ea5dce007 to 7ffb3c560939 (1 revision) (flutter/flutter#102729)
2022-04-28<EMAIL_ADDRESS>Roll Engine from 67940eb62023 to b62ea5dce007 (1 revision) (flutter/flutter#102721)
2022-04-28<EMAIL_ADDRESS>Roll Engine from ba644fafb2c5 to 67940eb62023 (2 revisions) (flutter/flutter#102717)
2022-04-28<EMAIL_ADDRESS>Roll Engine from 220613fc0b20 to ba644fafb2c5 (1 revision) (flutter/flutter#102715)
2022-04-28<EMAIL_ADDRESS>Roll Engine from 80c04d70c0d6 to 220613fc0b20 (3 revisions) (flutter/flutter#102708)
2022-04-28<EMAIL_ADDRESS>Roll Engine from b75aaa34f16d to 80c04d70c0d6 (435 revisions) (flutter/flutter#102704)
2022-04-28<EMAIL_ADDRESS>Roll Engine from 561f96247933 to b75aaa34f16d (2 revisions) (flutter/flutter#102701)
2022-04-28<EMAIL_ADDRESS>Remove Linux android views (flutter/flutter#102699)
2022-04-27<EMAIL_ADDRESS>Roll Engine from 0ac2fc2a1d1c to 561f96247933 (3 revisions) (flutter/flutter#102700)
2022-04-27<EMAIL_ADDRESS>Fix BottomNavigationBar label style text colors (flutter/flutter#102638)
2022-04-27<EMAIL_ADDRESS>[flutter_tools] rename local functions with _s (flutter/flutter#102688)
2022-04-27<EMAIL_ADDRESS>[flutter_driver] rename local functions with _s (flutter/flutter#102689)
2022-04-27<EMAIL_ADDRESS>Roll Engine from 35e56175dfde to 0ac2fc2a1d1c (2 revisions) (flutter/flutter#102695)
2022-04-27<EMAIL_ADDRESS>Bump github/codeql-action from 2.1.8 to 2.1.9 (flutter/flutter#102693)
2022-04-27<EMAIL_ADDRESS>Fix AppBar scrolledUnder initial state - the third (flutter/flutter#102582)
2022-04-27<EMAIL_ADDRESS>Roll Engine from 94824fcb600a to 35e56175dfde (1 revision) (flutter/flutter#102686)
2022-04-27<EMAIL_ADDRESS>[ci.yaml] Migrate 50 targets to Cocoon scheduler (flutter/flutter#102676)
2022-04-27<EMAIL_ADDRESS>Marks Linux_android android_semantics_integration_test to be unflaky (flutter/flutter#102658)
2022-04-27<EMAIL_ADDRESS>Roll Engine from 9e6911e4da21 to 94824fcb600a (6 revisions) (flutter/flutter#102681)
2022-04-27<EMAIL_ADDRESS>Migrate AppBar to Material 3 (flutter/flutter#101884)
2022-04-27<EMAIL_ADDRESS>[reland] Refactor ToggleButtons (remove RawMaterialButton) (flutter/flutter#101760)
2022-04-27<EMAIL_ADDRESS>Marks Mac tool_host_cross_arch_tests to be unflaky (flutter/flutter#102659)
2022-04-27<EMAIL_ADDRESS>[intl] speed up localization generation and regenerate symbols (flutter/flutter#102614)
2022-04-27<EMAIL_ADDRESS>Roll Engine from 8f7e55b13064 to 9e6911e4da21 (2 revisions) (flutter/flutter#102673)
2022-04-27<EMAIL_ADDRESS>Roll Engine from 6abf69920824 to 8f7e55b13064 (1 revision) (flutter/flutter#102666)
2022-04-27<EMAIL_ADDRESS>Fixes documentation inconsistencies around 'material' and 'material design' (flutter/flutter#102632)
2022-04-27<EMAIL_ADDRESS>rename local functions with _s (flutter/flutter#102615)
2022-04-27<EMAIL_ADDRESS>Roll Plugins from e55659835427 to 3e43f590d4d0 (3 revisions) (flutter/flutter#102656)
2022-04-27<EMAIL_ADDRESS>Roll Engine from c5bb99941547 to 6abf69920824 (2 revisions) (flutter/flutter#102646)
2022-04-27<EMAIL_ADDRESS>Revert "Benchmark cannot run in Android12 phone (#100761)" (flutter/flutter#102649)
2022-04-27<EMAIL_ADDRESS>Benchmark cannot run in Android12 phone (flutter/flutter#100761)
2022-04-27<EMAIL_ADDRESS>Roll Engine from 189cdc7df7ae to c5bb99941547 (4 revisions) (flutter/flutter#102634)
2022-04-27<EMAIL_ADDRESS>Roll Engine from fcc15947754b to 189cdc7df7ae (1 revision) (flutter/flutter#102623)
2022-04-27<EMAIL_ADDRESS>add missing trailing commas in list/set/map literals (flutter/flutter#102585)
2022-04-27<EMAIL_ADDRESS>Roll Engine from 5d844aa30fc6 to fcc15947754b (1 revision) (flutter/flutter#102622)
2022-04-27<EMAIL_ADDRESS>Roll Engine from 24471f3cbb33 to 5d844aa30fc6 (1 revision) (flutter/flutter#102619)
2022-04-27<EMAIL_ADDRESS>Roll Engine from 6ac401970df1 to 24471f3cbb33 (3 revisions) (flutter/flutter#102618)
2022-04-27<EMAIL_ADDRESS>Update open sdk version to be version version:11 (flutter/flutter#102610)
2022-04-27<EMAIL_ADDRESS>Roll Engine from 46b0889d13ce to 6ac401970df1 (3 revisions) (flutter/flutter#102612)
2022-04-27<EMAIL_ADDRESS>Revert "[framework] Reland: use ImageFilter for zoom page transition " (flutter/flutter#102611)
2022-04-26<EMAIL_ADDRESS>Roll Engine from 07584c64bb7b to 46b0889d13ce (1 revision) (flutter/flutter#102606)
2022-04-26<EMAIL_ADDRESS>[framework] remove usage and deprecate physical model layer (flutter/flutter#102274)
2022-04-26<EMAIL_ADDRESS>Roll Engine from 545a0b4ad107 to 07584c64bb7b (3 revisions) (flutter/flutter#102603)
2022-04-26<EMAIL_ADDRESS>Roll Engine from b30b65a74554 to 545a0b4ad107 (4 revisions) (flutter/flutter#102599)
2022-04-26<EMAIL_ADDRESS>Fix docs re: return value of Navigator's restorable methods (flutter/flutter#102595)
...

As of right now, Flutter is at revision: https://github.com/flutter/flutter/commit/7965ee25dd632d907f69e5ea6f4f81f35c0cf6a7 (from 5 hours ago). This PR is attempting to roll an older version: https://github.com/flutter/flutter/commit/9211d7cc8a0d (from 4 DAYS ago).
Closing this PR, it's expired.


[BEAM-7227] Instantiate PipelineRunner from options to support other runners

R: @mxm

Run Java PreCommit


Add multiple sub flow controllers?

Do you support multiple sub flow controllers, for example, maybe useful in the case of a tab bar.
add(subFlowController: FlowController) can be called multiple times, but it releases the previous controller. Therefore, there can only be one sub.

As I see it, the core question about such a feature is how one interprets the add(subFlowController: FlowController) method:

Either without any semantics, just as a needed burden to keep flow controllers referenced and to thereby avoid releasing.
Or, considering that Imperio can be seen as a shift from the UIView& UIViewController pattern to a UIViewController & FlowController pattern, with the semantics of a flow controller acting like a node, just as a TabBarController acts as a UIViewController node.

Semantically (according to the second interpretation) it would definitely make sense to have such a feature. If you only see the method as a way to stop ARC from releasing (in-line with the first interpretation), one could argue that a "tab flow controller" is a special case and should be handled by custom release management (storing as properties).
Considering the analogy between the roles of a FlowController and UIViewController in their respective architecture, I tend to interpret the method more semantically, so I'd be in favour of such a feature.


Adding PXT link command to the pxt cli

We'll have to publish this manually since it's in the JS cli and not cli.ts

If we put it in cli.ts then we'll have to maintain it in V0 and master. What's the issue with keeping it in the JS?

OK, let's put it here. But it would be nice to support pxt-common-packages as well.

I'll add common-packages support

Done! @mmoskal take a look

LGTM!


[core] Error when installing in unsupported node environments

Based on https://github.com/mui-org/material-ui/pull/25795
We have this kind of dangerous stack of technical debt that is pinning us down:

Can't use exports with webpack 4

Can't use webpack 5 because of https://github.com/webpack/webpack/issues/12900

Can't resolve https://github.com/webpack/webpack/issues/12900  with getStaticProps because of https://github.com/vercel/next.js/discussions/10949

Can't use webpack 5 because https://github.com/ryanclark/karma-webpack/issues/505

Can't drop karma because there's no alternative (I have no idea how one would use browserstack)

Can't use exports because of https://github.com/benmosher/eslint-plugin-import/issues/1868 (arguably the easiest since we should just be able to drop import/resolve)

Accidentally pushed to upstream


use dynamic reloading of certs to avoid rolling out new deployment versions

since https://github.com/openshift/origin/pull/22229 merged, we can reload most
We need https://github.com/openshift/origin/pull/22239 to reload the aggregator client
/hold

/retest
after this we just need to make the syncer suicide and switch the rest.

/refresh

/retest

/retest


Make generateSW() copy a larger set of libraries

R: @addyosmani @gauntface
This updates the workbox-build.generateSW() behavior so that when importWorkboxFromCDN is false, all(*) of the Workbox libraries (both dev and prod builds, including the sourcemaps) are copied over to the output directory, instead of just the workbox-sw library.
There are some pending TODOs to update the logic when we move to a single workbox-sw.js file instead of workbox-sw.prod.js (see #969) and also to deal with the eventual migration of the library path metadata from the browser field of each module's package.json to the main field.
I'll have to update the generated service worker file to make proper use of these libraries, but that's going to be part of a separate PR to update the service worker template v3-compatible.
(*) I'm copying all the libraries that might actually be used by the service worker created by generateSW(). If a user needs, e.g., the background sync library, my assumption is that they're using injectManifest() and we don't need to deal with that in this PR. See https://github.com/GoogleChrome/workbox/issues/976

Thanks for the feedback—I think everything's been address.
Regarding cleanup: the new method doesn't attempt to clean up copies of older versions of the libraries. If you'd like to see that functionality, we can track it via a separate issue. (My inclination is not to make what could possibly be the wrong guess about what to delete.)


Inquiry about decoding errors in event indexing

Describe
Hello,
I'm currently working on an event indexing project that uses a given ABI file to filter topics and decode results. However, since there is no fixed contract address, the input parameters may not always satisfy the decoding requirements, resulting in a panic during decoding.
I was wondering if there is a try_decode-like method available that could return null or an error when there are issues with decoding, such as incorrect length or failure to decode the event correctly. This would enable me to have more flexibility in handling these situations.
Could the development team provide any suggestions or advice for addressing this issue?
Thank you!

Hey, you can just wrap decode method in try ... catch and handle decoding errors as you want

🎉 Thank you so much for helping me resolve my issue! I really appreciate the time and effort you put into helping me out. As someone who is not very familiar with TypeScript, I was struggling to find a solution on my own, and your help was invaluable.
👍 Your expertise and willingness to assist me has been truly impressive, and I can't thank you enough for your support. I'm so grateful to be part of such a supportive and helpful community.
🙏 Once again, thank you for all your help. I couldn't have done it without you!
@belopash


Debugging startup errors

As far as i can figure out, i have to actually open the add-in pane for it to show up in the F12Chooser window, and starting the debugger then is too late when the error happened on startup.
How do i go about debugging add-ins that fail on startup, or actions that do not open a pane?

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

ID: 7b6ad3bb-b3fe-e8c6-4a38-f6df85dbabf4
Version Independent ID: 194afc55-e012-edc5-da88-9858f1d52f34
Content: Debug add-ins using F12 developer tools on Windows 10 - Office Add-ins
Content Source: docs/testing/debug-add-ins-using-f12-developer-tools-on-windows-10.md
Product: non-product-specific
Technology: add-ins
GitHub Login: @o365devx
Microsoft Alias: o365devx

@Noino seems like the information in Use runtime logging to debug your add-in may be useful in the scenarios you've described. Please check out those instructions and let me know if they provide the type of information you're needing.

@kbrandl This method did not produce any information related to my add-in. Doubt it is a manifest related issue anyway, since it works fine in the browser.
In outlook the sidebar thingy (Action:ShowTaskPane) opens but remains empty, and popup/modal window (Action:ExecuteFunction->Office.context.ui.displayDialogAsync) fails to open and i'm only getting a infinite "add-in is working on your request".

with "browser" i was referring to anything other then IE, but it seems that is exactly what outlook is working with and looks like its the root of my problem.. Indeed the choice of compatibility tool to run this functionality on is shocking, seeing that the web app itself has trouble working in IE, but that is beyond the scope of this article and ticket.
I've found a way to debug using developer console, by doing it on the webapp using IE, and I've found the source of this to be incompatible syntax. Oh how i wish there was a way to disable the js add-in on the desktop version of outlook.

@Noino thanks for the update; glad to hear that you've figured out the problem.
For future reference, runtime logging can be helpful in debugging various types of add-in issues (not just manifest-related issues). I realize that current placement of the information about runtime logging in the docs is misleading (since it implies that runtime logging is only useful in debugging manifest-related issues), and we'll likely be looking to address this soon.


Remove existing teams in the game from the "Select team" dropdown

Currently, If I already have a team in the scope of a game I can see it in the dropdown selector. Although If I try to add it won't work because this team already exists in the scope of this game.

Expected: Teams that already exist in the scope of this game don't appear in the dropdown.
Acceptance Criteria:

If the game already has a team with such a name, then the "Select team" dropdown shouldn't contain it.

Hi there,
Please can I have a run at this one?
Cheers,
Max

sure! good luck and thank you in advance:)


vulnz fat jar violates Nexus content validation

Problem:
We use Sonatype Nexus Repository Manager to proxy artifacts from Maven Central. We have the Nexus "Strict Content Type Validation" activated so the vulnz jar is rejected because it is packaged as a fat jar. That's because it contains a shell script prefix and Nexus considers it to be a "shell-script" whereas it is actually a "jar" and rejects it.
So far this is the only jar that we want to use that shows this behaviour and we would like to avoid disabling the Content Type Validation.
We are calling the vulnz with a prefixed java call.
Request:
Is it possible to package vulnz (maybe additionally) as a "normal" jar?
Additional Information:

https://stackoverflow.com/questions/50182301/deploying-a-jar-artifact-to-nexus-results-in-a-content-type-mismatch
https://community.sonatype.com/t/uploading-artifacts/718

Vulnz isn't a typical JAR you would put in nexus... Rather it is a Sprint-Boot executable JAR.
No, it is not possible to repackage the JAR and keep the same functionality. Spring-Boot executable JARs are a well known format. I might raise a ticket with Sonatype regarding this behavior.


added whoosheer option for whooshee_search method

Hello, this commits adds possiblity to override whoosheer when whooshee_searching :). This can be useful if we want to include data into a search index from a model we don't join on. See, for instance, https://github.com/fedora-copr/copr/commit/77eb46f7 for a concrete usecase.

Very nice, thanks for implementing this. The only thing I'm missing here is a short notice in README (perhaps at the end of "How it works" section) about this feature. Otherwise this is perfect :tm: and I'll merge once you update the README.

Perfect! Merging, thank you for this PR.

BTW I just released version 0.2.0 which contains this patch. Enjoy! https://pypi.python.org/pypi/flask-whooshee/0.2.0

Thanks.


QuerySpecValidator fails to catch errors when filterExpression has more than one objects

Bug Report
Describe the Bug
When we issue a EDC_QUERY_SPEC_TYPE query that has a filterExpresssion with more than one objects, QuerySpecValidator fails to catch it.

Expected Behavior
A message that the operandLeft is missing e.g.
[{"message":"mandatory value 'https://w3id.org/edc/v0.0.1/ns/filterExpression/https://w3id.org/edc/v0.0.1/ns/operandLeft' is missing or it is blank","type":"ValidationFailure","path":"[https://w3id.org/edc/v0.0.1/ns/filterExpression/https://w3id.org/edc/v0.0.1/ns/operandLeft","invalidValue":null}]](https://w3id.org/edc/v0.0.1/ns/filterExpression/https://w3id.org/edc/v0.0.1/ns/operandLeft","invalidValue":null}])
Observed Behavior
SEVERE 2023-09-06T00:01:58.1307352 JerseyExtension: Unexpected exception caught
java.lang.NullPointerException
	at java.base/java.util.Objects.requireNonNull(Objects.java:208)
	at org.eclipse.edc.spi.query.Criterion$Builder.build(Criterion.java:132)
	at org.eclipse.edc.core.transform.transformer.to.JsonObjectToCriterionTransformer.transform(JsonObjectToCriterionTransformer.java:51)
	at org.eclipse.edc.core.transform.transformer.to.JsonObjectToCriterionTransformer.transform(JsonObjectToCriterionTransformer.java:29)
	at org.eclipse.edc.core.transform.Transfo

Steps to Reproduce
Steps to reproduce the behavior:

Issue the above query to the /v3/assets/request endpoint
See error

Context Information

Used version EDC v0.2.1

Detailed Description
If applicable, add screenshots and logs to help explain your problem.
Possible Implementation
CriterionValidator should be run on all filterExpression objects and not just the first one

At a glance, the issue should be caused by the wrong definition for the filterExpression validator, it should be verifyArrayItem instead of verifyObject.
https://github.com/eclipse-edc/Connector/blob/f981237b50d63fa07e7a1a4edd1b7fa2e69e1d79/core/common/validator-core/src/main/java/org/eclipse/edc/validator/jsonobject/validators/model/QuerySpecValidator.java#L49

logically, multiple filterExpressions are AND-chained, right?

@arnoweiss yes, they are


Contributing preparation with py.test

I wanted to contribute some code.
py.test --cov=kq --cov-report=html  did not work for me. Is there something I missed? Command not found. It's my first try. All previous steps described worked fine.

Hi @Semo,
Have you followed all the steps here? https://github.com/joowani/kq/blob/main/CONTRIBUTING.md
The pip install -e .[dev] should install all the dependencies for development, including pytest.
If you're using virtual env, I'd recommend you double check that as well.

I did. (Hopefully)

git clone and cd'ing into it.  OK
python3 -m venv kq_dev    OK
source ./kq_dev/bin/activate   OK
pip install -e .  OK
pip install pre-commit    OK
podman run -d -p 9092:9092 -e ADV_HOST=<IP_ADDRESS> lensesio/fast-data-dev    OK
py.test --cov=kq --cov-report=html    FAILS

In step 3 pip install -e .[dev] --> no matches found: .[dev], therefore I left it empty.

Did you run pip install -e .[dev] in the kq repository folder? You can just manually install pytest by running the following in your venv:
pip install pytest pytest-cov

Yes I ran it in the kq repo folder, which is made, when cloned from Github, where Contibuting.md and README.md and stuff is residing.
I'll use the command you gave me.


Fluent: add dark theme support

Fixes #

Changelog Entry

(Experimental) Added botframework-webchat-fluent-theme package for applying Fluent UI theme to Web Chat, by @compulim and @OEvgeny

Added dark theme support, in PR #5138

Description
PR adds styles needed to support dark themes. Mostly overriding user-agent styles with our or Fluent defaults (when Fluent theme is provided).
Design
Note that the fluent-theme package doesn't inherit the styleOptions colors. Instead, it defaults to the provided by Fluent CSS variables.
Specific Changes

Fix missing colors in buttons rules
Remove redundant fill attribute from SVG elements
Add tests with dark Fluent theme applied
Fix attachments display if telephone keypad is opened
Add selected state for toolbar button
Align toolbar button styles with Figma

-

[x] I have added tests and executed them locally
[x] I have updated CHANGELOG.md
[ ] I have updated documentation

Review Checklist

This section is for contributors to review your work.

[ ] Accessibility reviewed (tab order, content readability, alt text, color contrast)
[ ] Browser and platform compatibilities reviewed
[ ] CSS styles reviewed (minimal rules, no z-index)
[ ] Documents reviewed (docs, samples, live demo)
[ ] Internationalization reviewed (strings, unit formatting)
[ ] package.json and package-lock.json reviewed
[ ] Security reviewed (no data URIs, check for nonce leak)
[ ] Tests reviewed (coverage, legitimacy)

Could we try something real? I am unsure if this can be easily done because how Fluent package their stuff...
Let's see if you can do this. Load @fluentui/theme (from unpkg.com?) and render the actual theme provider before us. So we can see the E2E of this user story: "Web Chat theme can be using defaults from Fluent UI."
Then turn on the dark theme. I think Fluent component should honor that and set the CSS properly.
~To turn on dark theme, you can search in some of our HTML tests. I forgot which HTML test, search "prefers-color-scheme". It could be related to keyboard help screen.~
To turn on dark theme, you can look at this keyboard help screen test, https://github.com/microsoft/BotFramework-WebChat/tree/main/tests/html/accessibility.keyboardHelp.darkTheme.html#L17.
Particularly, send setEmulatedMedia CDP command over WebDriver. 😉
await host.sendDevToolsCommand('Emulation.setEmulatedMedia', {
  features: [{ name: 'prefers-color-scheme', value: 'dark' }]
});

I was hoping to use the Fluent infrastructure too, but I haven't found a reliable way to bundle Fluent, unless we want to maintain the ESM build ourselves.
I'm blocked on @fluentui/theme importing react package. The following pattern works with simple things:
    <script type="importmap">
      {
        "imports": {
          "@fluentui/theme": "https://unpkg.com/<EMAIL_ADDRESS>          "@fluentui/merge-styles": "https://unpkg.com/<EMAIL_ADDRESS>          "@fluentui/utilities": "https://unpkg.com/<EMAIL_ADDRESS>          "@fluentui/dom-utilities": "https://unpkg.com/<EMAIL_ADDRESS>          "@fluentui/set-version": "https://unpkg.com/<EMAIL_ADDRESS>          "tslib"<EMAIL_ADDRESS>        }
      }
    </script>
    <script type="module">
      import('@fluentui/theme').then(mod => {
        console.log(mod);
      })
    </script>

And I probably can work around react not providing ESM builds, but I found that in order to apply the theme, I have to make the whole @fluentui/react-components library to work this way which is way more packages thus time figuring it out.
@compulim please let me know if you have a better idea, or if you think it's worth to continue.

No problems. I expect this outcome too.
After you finish all P0s, please try to build our own bundle with @fluentui/theme and react inside Web Chat monorepo. Feel free to put it under /packages/test/ or even under existing package. I think this will be a valuable test as we deliver what we promise. 😉


New like like button ID

Cheers, great project you have here! I was trying to get it to work but apparently Tinder changed the button ID
This possibly also affects the other elements because it still crashes for me after navigating to the Like page (for instance I haven't figured out yet what "# Press a point on the page that gives a reference for scrolling to the bottom" means / which element would be suitable for this)

Phil, please reference pics_and_vids_2.py. I am glad to help assist and I appreciate your pull request.
The code is:
self.driver.find_element_by_xpath(xpath='//a[@href="/app/my-likes"]').click() # Selecting by the href of an anchor (i.e., 'a') tag
If you have specific questions, we can do a call on Discord or something. The code is more robust in the aforementioned Python script. I have not had any errors when running my script for quite some time.


Fixed issue where Travis wouldn't look for environment variables

I noticed when making my automated documentation pr that there was an issue with travis, so I looked into fixing it
Key Changes

Added the required environment variables to travis.yml

Pull Request Test Coverage Report for Build 1095

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

Totals

Change from base Build 1086:
0.0%

Covered Lines:
229

Relevant Lines:
239

💛  - Coveralls


Add DPO of TRL in README.md

What does this PR do?
This is to modify README.md to include TRL DPO.

Fixes # (issue)
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Plugin install by name fails, but succeeds by URL

I was unable to install the container-registry (or any other) plugin directly, getting an error like:
❯ ibmcloud plugin install container-registry
Looking up 'container-registry' from repository 'IBM Cloud'...
Plug-in 'container-registry[cr] 1.0.2' found in repository 'IBM Cloud'
Attempting to download the binary file...
FAILED
Unable to download the plug-in binary from the URL https://download.clis.cloud.ibm.com/ibm-cloud-cli-plugin/container-registry/1.0.2/container-registry-darwin-arm64-1.0.2.
Error: Get "https://download.clis.cloud.ibm.com/ibm-cloud-cli-plugin/container-registry/1.0.2/container-registry-darwin-arm64-1.0.2": dial tcp [2600:1406:1400::b818:9a0a]:443: connect: no route to host
Try again later.

returned 1

Is it possible that it's trying to access this plugin via IPv6 and failing?
If I use the URL from the above error message however, plugin installation succeeds:
❯ ibmcloud plugin install https://download.clis.cloud.ibm.com/ibm-cloud-cli-plugin/container-registry/1.0.2/container-registry-darwin-arm64-1.0.2
Attempting to download the binary file...
 12.40 MiB / 12.40 MiB [======================================================================================================================================] 100.00% 10s
13000274 bytes downloaded
Installing binary...
OK
Plug-in 'container-registry 1.0.2' was successfully installed into /Users/jay/.bluemix/plugins/container-registry. Use '/usr/local/bin/ibmcloud plugin show container-registry' to show its details.

Any thoughts on why this might be happening?

I'm unable to recreate the scenario. What version of the CLI are you using?


Update core dependencies

update gax to 1.48.0
update google-auth-library to 0.17.0
update grpc to 1.23.0
ensure guava versions at 28.0-android
update protobuf to 3.9.1
update opencensus to 0.23.0

in opencensus 0.22.0, the runningSpanStore defaulted to not storing spans, so we need to configure the store to actually keeps some spans for testing

update auto-value to 1.6.6

I cannot figure out why this won't compile on Java 7. It's failing to find another package installed from this repo.

autovalue >= 1.5 must use Java 8 for compilation but will work on Java 7


Is it possible to use build env to determine whether download source code in prepare stage or not

When I config a cnb image with blob url, like
apiVersion: kpack.io/v1alpha2
kind: Image
metadata:
  name: default.test
spec:
build:
  cache:
    volume:
      size: 1536Mi
  source:
    blob:
      url: https://xx.file.core.windows.net/bab958c1e4444a409bd8b20a8f12e039/xxx?se=2022-09-08T10%3A12%3A47Z&sp=r&sv=2021-06-08&sr=f&sig=xxx
......

Then the source code will be downloaded in the Prepare stage when a build pod is triggered. Sometimes, if I need to do some migration, like upgrade the buildpack version in the clusterstore, then use kpack-controller to trigger a new build pod automatically to generate a new image, then the Prepare stage may be failed because the blob.url SAS token is expired.
So if I use cache to store source code in /workspace layer, then how can I determine not to pull the source code again and again, just use the cached workspace layer generated in previous builds? It may helpful for some maintenance work, like helping a customer to generate a new image without CVE issues or upgrading the buildpack version. The expired blob.url may block this kind of operation.

Hi @zhoufenqin, kpack does not currently have a mechanism to cache the source across builds and I fear that is likely outside the scope of kpack. Is there a way to provide the source in a blob url that will not expire?

Hi @zhoufenqin, kpack does not currently have a mechanism to cache the source across builds and I fear that is likely outside the scope of kpack. Is there a way to provide the source in a blob url that will not expire?

Hi @matthewmcnew I think it's a scopt of lifecycle, I moves the issue to lifecycle repo.  https://github.com/buildpacks/lifecycle/issues/954
Besides, we don't have plan to provide an unexpired blob URL in our system for some security reason

@zhoufenqin The lifecycle does not fetch or download the blob url. So, it would not be a valid lifecycle issue.

thank @matthewmcnew , so is there any way to improve the prepare stage in the kpack scope?


Add Gender field to Contact

Add a new picklist field on Contact for Gender.

Test issue used for a demo of the contributor tool


Relocate TheHive and DingTalk alerters

I moved these out of alerters.py but didn't put them into the correct folder the first time around.

This is ready to merge but I noticed the test coverage report dropped down to 9% test coverage. It appears all the new alerter tests are not being counted as tests, but instead of counted as new untested code. @nsano-rururu any ideas?

@jertel this has something to do with the parallelisation I added. If I remove the -n 4 in tox.ini and re-run the tests, the coverage goes back up to roughly where it used to be. I will look into why this is happening tomorrow,  and work out how to fix it.

@jertel
I do not know. I'm sorry.
There is an error in another case, so it needs to be corrected.
ERROR: sphinx-rtd-theme 0.5.2 has requirement docutils<0.17, but you'll have docutils 0.17.1 which is incompatible.

requirements-dev.txt
add「docutils<0.17」


@N-Kannan #8557

I have updated the Document
For the issue #8557
A member of the role db_owner can execute sp_updatestats for Azure SQL Database. #8557
Please reader the documentation And Approve the pull request. As per i did Google i god the Detail and updated

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

This change is being pulled into a private PR, to update the article in other respects. This PR will be closed, but you will still get credit for your change.


Automator: update common-files@master in istio/istio@master

Generated by Automator - 2020-06-24T01:15:11+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
7f7b346d180f2868e4f006729f247177d734c9b2
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
7f7b346d180f2868e4f006729f247177d734c9b2
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-test_istio
7f7b346d180f2868e4f006729f247177d734c9b2
link
/test release-test_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-test_istio
7f7b346d180f2868e4f006729f247177d734c9b2
link
/test release-test_istio

unit-tests_istio
1ab308c96ee8abca76f81a497ec48c08c03e1200
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
1ab308c96ee8abca76f81a497ec48c08c03e1200
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

https://github.com/istio/istio/pull/24907 will remove the failing test.


I want to help track duration

Thanks for starting the project. It shows a good way to track an initial impression. I'd like to expand it by tracking the impression end event and save a timer. I'll see what I can do. If you have any ideas please let me know.

@erolaksoy by doing a little hacking I was able to track enter and exit events. I am also able to track new impressions (like when you scroll back) by clearing the list when an exit event happens. If you're interested please give me permission to push a branch.

Thank you for your interest, you can open a pull request for your changes.


Configure headers

This change will allow implementers to pass headers into the instantiation.  Please see the following example.
class TestClass < Base
  def TestClass.headers
    { accept: 'application/json' }
  end

  def TestClass.get
    cb_client(headers: headers)
    ...
  end
end

Yup, looks good to me too. Merging.


[Java] Move (most of) the name mangling logic to the Refiner

Sharing to receive early feedback on the direction (cc. @baywet).
I started with Java(instead of Python where we experienced more issues) as I felt more comfortable with it, but I believe that this effort should be extended to all of the languages.
You can check the results by running:

on the main branch: dotnet publish ./src/kiota/kiota.csproj -c Release -p:PublishSingleFile=true -p:PublishReadyToRun=true -o ./publish
on this PR branch I'm testing with commands like: (cd src/kiota && dotnet build) && ./it/compare-generation.ps1 -descriptionUrl "oas::petstore" -language java
use more complex specs by testing against the rest ...

I feel good about the direction and the diff is confirming some improvements, but I haven't verified yet that everything is consistent ...

@baywet and @andrueastman this should be ready for review now 🙂 .

@andreaTP a couple of integration tests for Java are failing, can you look into it before we review please?

Ready for review now 👍 happy that the IT tests have detected this last detail 🙂 .
for my future self, the incantation to run all the integration tests for one language:
yq -e '.jobs.integration.strategy.matrix.description[]' .github/workflows/integration-tests.yml | xargs -I{} ./it/generate-code.ps1 -descriptionUrl '{}' -language java && ./it/exec-cmd.ps1 -descriptionUrl '{}' -language java

ready for the next review round 👍

Will you be doing the same for other languages?

I started this with the intention to get some traction on the subject and ... help 🙂
If no one is picking it up, I can try to go through it, but it's going to take time ...
We also need to properly review new developments to avoid regressions (would it be possible to automate? a source Generator can probably do it ...).

We should be able to provide help in validating the changes.

This is already super valuable help!

Help in implementing those will be harder, the top priority is to get more languages to a stable maturity level at this point.

I would argue that removing a design issue in the implementation is pretty important to be confident about the maturity level, as, in this case, it's excluding a lot of subtle bugs that are hard to find, hard to debug, and, ultimately, hard to solve (workarounds are very likely to introduce regressions).
But this is on you to prioritize 🙂 ❤
At my own peace, I'll try to catch up with the other languages 🙂

I'm curious about the idea of source generators, can you expand?

Coming from Scala we are amused to solve all of the problems with macros ...
In the docs we have:

A generator can analyze your source code and emit the code it needs to "wire up" your app.

This means that a Source Generator can analyze the source code and look up for usages of i.e. StringExtensions in the Writer package, and either:

fail loudly
invalidate the build by emitting invalid code

Did you mean a code analyzer instead by any chance?

That's correct, I wasn't aware of code analyzers, the same can likely be done inside a Source Generator, but the analyzer should suffice.

Added a note in #2842


[WFLY-14769] Access checking for txn namespace lookup

https://issues.redhat.com/browse/WFLY-14769
I have added a wrapper to WildFlyNamingContext which enables installation of aroundLookup interceptors. This functionality is used to perform access check for objects from "txn" namespace.

@emmartins Could you please review whether this fix makes sense from naming perspective?

@emmartins can you review this PR? It has missed making it into a couple of CPs

@tadamski CI failed with this PR, could you please check?

@tadamski does this logic runs for every JNDI lookup?

This appears to be implemented by intercepting the client side of the invocation not the server side.
Is it possible for a deployment to create an InitialContext that bypasses this check?
If so does it matter?  i.e. Is this just a helper or do we need to enforce this strictly?

@tadamski does this logic runs for every JNDI lookup?

Yes, every lookup from within a server is wrapped so I was able to plug verification interceptors.

This appears to be implemented by intercepting the client side of the invocation not the server side.
Is it possible for a deployment to create an InitialContext that bypasses this check? If so does it matter? i.e. Is this just a helper or do we need to enforce this strictly?

I'm not certain that hack that allows that doesn't exist, but the mechanism of context factory creation is the same as it was before (without a wrapper) so imho no new vulnerability should be introduced.

@tadamski some of the test failures in this appear to be legitimate (org.jboss.as.test.integration.naming.connector.JMXConnectorTestCase, org.jboss.as.test.integration.naming.rmi.RmiContextLookupTestCase). They started to fail also in EAP testsuite after this was backported.

Fixed.

@emmartins @darranl can you conclude the review? It affects whether we can include this downstream.

@tadamski well, the test failure in the security manager job looks maybe legitimate, it also fails downstream. Could you check it up?

@TomasHofman fixed

All checks have passed. @emmartins @darranl can you review this again and merge?

Hi @emmartins  @darranl,  May I ask your review on this? This is required by downstream CP.

@darranl @emmartins could you please review this again?
Thanks

Hi @emmartins @darranl, May I ask for your review on this? Downstream CP requires this.
Thanks

@bstansberry @darranl The PR is waiting for approve and merge for over 3 months. It is quite complete fix, would you please approve and merge if no more comments ? There is a downstream issue depending on this and got postponed for several CPs, thank you very much !

This approach seems very hacky to me. Have you considered decorating the org.wildfly.naming.client.NamingContextFactory implementation from wildfly-transaction-client? That way you can add your access check to before delegating to the existing implementation, which is already specific to the "txn:" namespace, without needing to interfere with the handling of other namespaces, and without touching the naming subsystem.

@pferraro Agree, the solution that you proposed is much better, I have already checked and it works. I'm closing this PR and will open alternative one.


Economic War craches when when running intro and resolution change (only main menu)

Hi there again,
Got some weird problems there. First when i let the intro run a little bit the game freezes (except with opengl, but Win7 still think it froze). When i skip it with ESC, the game still runs. Seconds is, when i change resolutions from main menu options, it really crashes. When i start a game and change the resolution then, it's working.
Need to set "maxgameticks=60" or the game runs to fast. This got nothing to to with the craches btw.
; Economic War
[EcoW]
maxgameticks=60
EDIT:  Here are the dmps
Economic War.zip

Yeah, it's working for now.


Layer names almost never display fully in the viewer

Our clients are using long layer names to describe their maps, but these are almost always cut off in RAMP. We need them to fit better, rather than just truncate and rely on hovertips by default.
This may involve:

Using smaller text or a tighter font
Splitting up over multiple lines
Altered mouse hover ability (What does the current hover behaviour accomplish?)

This issue is to assess ideas and feasibility for making map/layer names fit more appropriately in the viewer.

Solved by #111, use a smaller layer title and put the full title in a description.


bug: fix handling of language codeblocks copied to clipboard.

Hi mik, thanks for your efforts! Will join Discord once link is posted.
Describe the bug
Currently code copied to clipboard is outputting: '[object object], { ,[object object], } . . . '
To Reproduce
Testing on Win10.
Text or Bash are fine, but JS or Python are borked.
Expected behavior
Source code languages could be handled on a per case basis, like math or mermaid are already.
Temp Fix
Not worth making a PR as there's likely a more elegant way to handle languages already planned.
Code output assists this and other projects so.. temp fix.
In 'opla\webapp\components\common\Markdown\CodeBlock.tsx'.
Line 73: navigator.clipboard.writeText(children?.toString() ?? '');
Replace with:
if (display === 'python' || display === 'javascript') {    const codecopy = ref.current?.innerText;    navigator.clipboard.writeText(codecopy);  } else if (display === 'bash' || display === '') {    const textcopy = children?.toString() ?? '';    navigator.clipboard.writeText(textcopy);  }
(not sure the "?" after variable names)
Above this, add languages as necessary:
    } else if (language === 'bash') {         display = 'bash';     } else if (language === 'python') {         display = 'python';     } else if (language === 'javascript') {         display = 'javascript';     }
And then adding same in code block below:
 {display === '' && children}    {display === 'bash' && children}    {display === 'python' && children}    {display === 'javascript' && children}
Seems to work fine.

Thanks @kustomzone for your report. I fix this ASAP. And yes I need to change the discord invitation link.

Was overthinking it after seeing the Latex and Mermaid examples and imagining possible output types and languages.
Replacing line 73 with: navigator.clipboard.writeText(ref.current.innerText); ..works in all (?) cases.
Just need to bypass any handled separately: {display !== 'latex' && display !== 'mermaid'  && children}

@kustomzone I took a slighty different approch, by getting the code from the markdown content. Same for the language. I did that, because time to time the language was not detected correctly using classname.
If you want to review the PR, i will be delighted.


flutter client  ThetaWebApiException cant start preview

Would you tell me the OS and Theta model you use?

huawei mate30  理光Theta Z1

I'm still getting this type of error with both SC2 and Z1.

Pixel 2 physical device, Android 11
Z1 fw 2.20.3
SC2 fw 1.82

@codetricity
Flutter sample application didn't stop preview befor shooting.  It is fixed in #22.

this works well for me now.
Build was easy with no problems.  Just had to add mavenLocal() to android/build.gradle
https://youtube.com/shorts/qSE0BHCdyEc


Test test_send_from_directory_bad_request fails in Python 3.8

Expected Behavior
All tests should pass in Python 3.8. However this one:
    def test_send_from_directory_bad_request(self, app, req_ctx):
        app.root_path = os.path.join(
            os.path.dirname(__file__), "test_apps", "subdomaintestmodule"
        )

        with pytest.raises(BadRequest):
            flask.send_from_directory("static", "bad\x00")

Which calls:
def send_from_directory(directory, filename, **options):
    # ...
    try:
        if not os.path.isfile(filename):
            raise NotFound()
    except (TypeError, ValueError):
        raise BadRequest()
    # ...

Is now affected by a change from 3.7 to 3.8.
Python 3.8 now fixes the bug where a NULL character in a file name would trigger a ValueError in some os.path functions, such as os.path.isfile:

os.path functions that return a boolean result like exists(), lexists(), isdir(), isfile(), islink(), and ismount() now return False instead of raising ValueError or its subclasses UnicodeEncodeError and UnicodeDecodeError for paths that contain characters or bytes unrepresentable at the OS level. (Contributed by Serhiy Storchaka in bpo-33721.)

The expectation is that this test should not be there, as it is testing buggy python behaviour. Furthermore, since the correct behaviour is now known, I would expect send_from_directory to raise NotFound also for paths with NULL characters in them.
Actual Behavior
send_from_directory raises BadRequest for paths with null characters in them. This makes tests fail in python 3.8
====================================== FAILURES =======================================
__________________ TestSendfile.test_send_from_directory_bad_request __________________
tests/test_helpers.py:789: in test_send_from_directory_bad_request
    flask.send_from_directory("static", "bad\x00")
.tox/py38/lib/python3.8/site-packages/flask/helpers.py:767: in send_from_directory
    raise NotFound()
E   werkzeug.exceptions.NotFound: 404 Not Found: The requested URL was not found on the server. If you entered the URL manually please check your spelling and try again.
====================== 1 failed, 572 passed, 1 skipped in 4.14s =======================

Environment

Python version: 3.8
Flask version: master
Werkzeug version: 0.16.0

Hi! Can I take this?

Go for it! No need to ask first. :-)


Template shows raw data with JSON format

himalaya --output json template reply/forward shows the raw buffer data (it should be discarded).
[EDIT]: in fact it should not be discarded, but it should show the raw message as string (not buffer). This way we align with the existing behaviour (it is useful for UIs).

I don't understand what you mean. Do the raw part should be concerted into a string?

Yes definitely. Compare with the master branch, we have a raw attribute as utf8 string, not buffer.

Aright, I'll do that tomorrow!

Hold on, I meant today, but in the afternoon. Today is already tomorrow.