Valid devcontainer.metadata can corrupt compose override file

dev containers version: v0.387.0
OS: Ubuntu 24.04
Description
I can't get a compose file to run through devcontainers even though it runs fine from command line. The culprit seems to be a corrupted 'Features Compose Override File' generation that I don't even know how to clear.
At one point my compose.yaml file had the following environment variable set:
ROSCONSOLE_FORMAT: "[$${severity}] [$${walltime:%Y-%m-%d %H:%M:%S}] [$${node}]: $${message}"

This works in the repro example below and has worked for me in the past with docker compose running from CLI. However, it complained while I was migrating to use compose + devcontainers. From the logs below, it seems like the dictionary was getting converted to a list, and the colon in the value field doesn't play nice:
[2024-09-24T19:43:41.966Z] Stop (354 ms): Run: docker build -f /tmp/devcontainercli-achille/updateUID.Dockerfile-0.71.0 -t vsc-dev-docker-afd93ed1eced211ffad11c8e3235c1ce630bab41c837eddba2a4841a56df1b38-uid --platform linux/amd64 --build-arg BASE_IMAGE=gmr/customer --build-arg REMOTE_USER=gmruser --build-arg NEW_UID=1000 --build-arg NEW_GID=1000 --build-arg IMAGE_USER=gmruser /home/achille/.config/Code/User/globalStorage/ms-vscode-remote.remote-containers/data/empty-folder
[2024-09-24T19:43:41.967Z] Docker Compose override file for creating container:
services:
  'customer':
    image: vsc-dev-docker-afd93ed1eced211ffad11c8e3235c1ce630bab41c837eddba2a4841a56df1b38-uid
    entrypoint: ["/bin/sh", "-c", "echo Container started\n
trap \"exit 0\" 15\n
\n
exec \"$$@\"\n
while sleep 1 & wait $$!; do :; done", "-", "/entrypoint.sh"]
    environment:
      - USER=achille
      - DISPLAY=:1
      - NVIDIA_VISIBLE_DEVICES=all
      - NVIDIA_DRIVER_CAPABILITIES=compute,utility,display
      - XAUTHORITY=
      - QT_X11_NO_MITSHM=1
      - ROSCONSOLE_FORMAT='[${severity}] [${walltime:%Y-%m-%d %H:%M:%S}] [${node}]: ${message}'
    labels:
      - 'devcontainer.local_folder=/home/achille/gmr/dev-docker'
      - 'devcontainer.config_file=/home/achille/gmr/dev-docker/.devcontainer/devcontainer.json'
      - 'devcontainer.metadata=[{"postCreateCommand":"/bin/bash .devcontainer/postCreate_command.sh $${containerWorkspaceFolder}","customizations":{"vscode":{"settings":{"terminal.integrated.profiles.linux":{"bash":{"path":"bash"}},"terminal.integrated.defaultProfile.linux":"bash","remote.autoForwardPorts":false},"extensions":["dotjoshjohnson.xml","zachflower.uncrustify","ms-azuretools.vscode-docker","ms-iot.vscode-ros","ms-python.python","ms-vscode.cpptools-extension-pack","redhat.vscode-yaml","smilerobotics.urdf","streetsidesoftware.code-spell-checker","twxs.cmake","yzhang.markdown-all-in-one","github.vscode-github-actions","ms-vscode.cpptools","ms-vscode.cmake-tools"]}},"mounts":["type=bind,source=/var/log/.ros,target=/var/log/.ros","type=bind,source=$${localWorkspaceFolder}/persistent_data,target=/home/gmruser/.ss","type=bind,source=$${localWorkspaceFolder}/runtime_data,target=/home/gmruser/.ss_temp","type=bind,source=/etc/ss,target=/etc/ss","type=bind,src=$${localEnv:HOME}/.ssh,target=/home/gmruser/.ssh,readonly","type=bind,src=$${localEnv:HOME}/.ssh/known_hosts,target=/home/gmruser/.ssh/known_hosts","type=bind,src=$${localEnv:HOME}/.backport/config.json,target=/home/gmruser/.backport/config.json","type=volume,target=$${containerWorkspaceFolder}/build","type=volume,target=$${containerWorkspaceFolder}/install"],"containerEnv":{"USER":"$${localEnv:USER}","DISPLAY":"$${localEnv:DISPLAY}","NVIDIA_VISIBLE_DEVICES":"all","NVIDIA_DRIVER_CAPABILITIES":"compute,utility,display","XAUTHORITY":"$${localEnv:XAUTH}","QT_X11_NO_MITSHM":"1","ROSCONSOLE_FORMAT":"''[$${severity}] [$${walltime:%Y-%m-%d %H:%M:%S}] [$${node}]: $${message}''"},"remoteUser":"gmruser"},{"postCreateCommand":"/bin/bash .devcontainer/postCreate_command.sh $${containerWorkspaceFolder}","customizations":{"vscode":{"settings":{"terminal.integrated.profiles.linux":{"bash":{"path":"bash"}},"terminal.integrated.defaultProfile.linux":"bash","remote.autoForwardPorts":false},"extensions":["dotjoshjohnson.xml","zachflower.uncrustify","ms-azuretools.vscode-docker","ms-iot.vscode-ros","ms-python.python","ms-vscode.cpptools-extension-pack","redhat.vscode-yaml","smilerobotics.urdf","streetsidesoftware.code-spell-checker","twxs.cmake","yzhang.markdown-all-in-one","github.vscode-github-actions","ms-vscode.cpptools","ms-vscode.cmake-tools"]}},"mounts":["type=bind,source=/var/log/.ros,target=/var/log/.ros","type=bind,src=/etc/ss/,dst=/etc/ss","type=bind,src=$${localEnv:HOME}/.ssh,target=/home/gmruser/.ssh,readonly","type=bind,src=$${localEnv:HOME}/.ssh/known_hosts,target=/home/gmruser/.ssh/known_hosts","type=bind,src=$${localEnv:HOME}/.backport/config.json,target=/home/gmruser/.backport/config.json","type=volume,target=/workspaces/platform/build","type=volume,target=/workspaces/platform/install","type=volume,target=$${containerWorkspaceFolder}/build","type=volume,target=$${containerWorkspaceFolder}/install"],"containerEnv":{"USER":"$${localEnv:USER}","DISPLAY":"$${localEnv:DISPLAY}","NVIDIA_VISIBLE_DEVICES":"all","NVIDIA_DRIVER_CAPABILITIES":"compute,utility,display","XAUTHORITY":"$${localEnv:XAUTH}","QT_X11_NO_MITSHM":"1","ROSCONSOLE_FORMAT":"''[$${severity}] [$${walltime:%Y-%m-%d %H:%M:%S}] [$${node}]: $${message}''"},"remoteUser":"gmruser"},{"remoteUser":"gmruser"}]'
    volumes:
     [...]
      - vscode:/vscode
volumes:

  vscode:
    external: true

This failed with the following error:
[2024-09-24T19:43:41.967Z] Writing docker-compose.devcontainer.containerFeatures-1727207021967-6e5554e4-df63-43ad-ab82-77c2b36e0506.yml to /home/achille/.config/Code/User/globalStorage/ms-vscode-remote.remote-containers/data/docker-compose
[2024-09-24T19:43:41.967Z] Start: Run: docker compose --project-name dev-docker -f /home/achille/gmr/dev-docker/compose.yaml -f /home/achille/.config/Code/User/globalStorage/ms-vscode-remote.remote-containers/data/docker-compose/docker-compose.devcontainer.containerFeatures-1727207021967-6e5554e4-df63-43ad-ab82-77c2b36e0506.yml up -d
[2024-09-24T19:43:42.015Z] WARN[0000] The "message" variable is not set. Defaulting to a blank string.
[2024-09-24T19:43:42.015Z] services.customer.environment.[6]: unexpected type map[string]interface {}

The first thing I did was to remove the offending ROSCONSOLE_FORMAT variable from my compose.yaml file. However, Rebuild and reopen in container will still yield the same error. I've tried reopening the window to no avail.
When I copy the command being run docker compose --project-name dev-docker -f /home/achille/gmr/dev-docker/compose.yaml -f /home/achille/.config/Code/User/globalStorage/ms-vscode-remote.remote-containers/data/docker-compose/docker-compose.devcontainer.containerFeatures-1727207021967-6e5554e4-df63-43ad-ab82-77c2b36e0506.yml up -d and modify the containerFeatures compose file and remove the offending line manually and then run this command from the terminal, it works fine. But this file gets regenerated every time I run 'reopen in container'.
I've tried:

clearing the user data folder rm -rf  /home/achille/.config/Code/User/globalStorage/ms-vscode-remote.remote-containers/data
clearing all files in ~/.config/Code  dir that had a reference to ROSCONSOLE_FORMAT, including

~/.config/Code/User/globalStorage/state.vscdb
~/.config/Code/User/workspaceStorage/594f8f796eef47d970635b16d002a443/state.vscdb

rm /tmp/devcontainercli-achille/container-features/0.71.0-1727210272333/Dockerfile-with-features
rm /var/lib/containerd/io.containerd.metadata.v1.bolt/meta.db

EDIT: I've found that the issue stems from the devcontainer.metadata that's attached to the image being used. That image was created using devcontainers and has the ROSCONSOLE_FORMAT baked into it, which this compose override file generation seems to be using. Removing the environment variable from the original image solved the issue. I will update the repro repository accordingly.
Repro
EDIT: repro here

@gauravsaini04 / @prathameshzarkar9 Can either of you help take a look? Thanks!

added a reproducible example here


chore: Remove auto-patching from typed controllers

Fixes #
Description

Remove auto-patching from typed controllers

How was this change tested?
make presubmit
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

15 of 27 (55.56%)  changed or added relevant lines in 6 files are covered.
23 unchanged lines in 7 files lost coverage.
Overall coverage decreased (-0.1%) to 76.277%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/controllers/node/controller.go
6
9
66.67%

pkg/controllers/termination/controller.go
3
6
50.0%

pkg/controllers/counter/controller.go
0
6
0.0%

Files with Coverage Reduction
New Missed Lines
%

pkg/controllers/counter/controller.go
1
0%

pkg/controllers/node/emptiness.go
1
83.78%

pkg/controllers/provisioning/provisioner.go
2
85.25%

pkg/controllers/deprovisioning/controller.go
3
70.33%

pkg/controllers/provisioning/batcher.go
3
70.0%

pkg/controllers/node/controller.go
4
80.3%

pkg/controllers/deprovisioning/events/events.go
9
80.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
5749

Relevant Lines:
7537

💛  - Coveralls


Right to left

Hello. Is there a way to have the pages go from right to left instead of left to right?

Have you tried reversing the UIViewController array that you're using for initialization? Then you can use moveToPage, using the last index with the animation time set to zero.
Good luck,

That's doesn't have good accent! and after moving the view has margin about 50 from left!
is there anyway to select last item as start UIViewController without animation or something like this?

We will be adding an "animated" option to the following method soon. This should hopefully fix your problem. We will also look at the possibility of adding a startIndex optional parameter to the inits.
open func moveToPage(_ index: Int)

Do we have this capability in last version? "startIndex"


Tab and Tab.Pane Overflow Problem

Bug Report
When adding multiple tab in the Tab.Pane it will overflow , it won't break into two levels, there is only single level tab and need to add the scrolling nature to view the hidden tabs
Steps
A clear and concise description of steps to reproduce the problem.
Expected Result
The tab should break into multi levels when overflow of Tab Occurred
Version
x.y.z
Testcase
https://codesandbox.io/s/semantic-ui-react-forked-xebro?fontsize=14&hidenavigation=1&theme=dark
@layershifter

@jeevankacharya thanks for reporting, this is a styling issue. Tabs in Semantic UI designed to have a single row, however this can be solved with flexWrap as was suggested on StackOverflow.
<Tab
  menu={{
    attached: true,
    tabular: true,
    style: { flexWrap: "wrap" }
  }}
  panes={panes}
/>

https://codesandbox.io/s/semantic-ui-react-forked-v86d6


ProxyCommand support

This adds basic support for connecting via jump hosts.

@cbeckr - Thank you so much for your first contribution! Welcome to the family :)
This looks great and I will test it out soon and get back to you!

I think this looks good for now, I plan on updating all the "wait_until_ready" flags as its like this on other transports also.


fix(locate-reuse-lib): corrected extraction of component name

Changes from https://github.com/SAP/open-ux-tools/pull/2071. Thanks @paraggarg89
"sap.ui5"/"componentUsages" contains object map having "name" property where component name is specified.
e.g.
{
...
 componentUsages: {
            "componentUsage1": {
                "name": "sap.nw.core.applogs.lib.reuse.smarttemplate",
                "lazy": true
            },
            "componentUsage2": {
                "name": "sap.s4.cfnd.bpf.lib.reuse.component.smarttemplate",
                "lazy": true
            }
 }
...
}

So instead of key of componentUsages object, logic should extract component name from "name" property inside the object.

call createRender using async
https://github.com/SAP/open-ux-tools/issues/969#issuecomment-2211065454

Please make sure that the same also works in the preview middleware. I think it should be here: https://github.com/SAP/open-ux-tools/blob/main/packages/preview-middleware-client/src/flp/init.ts#L93

@tobiasqueck  Done. Please re-review.
Also updated createRender to be async like in init.ts

@tobiasqueck Please re-review.


Load charts that you need

Is there a possibility to load charts/js that we want to use, without requirejs?
Some sort of build, where you select which charts you want in minified JS file.

Yes, although Dimple supports requirejs there is no requirement for it.  In fact, we don't use it in our projects.

How? Can you give me an example or a suggestion, where I can do this?

Maybe I'm not entirely clear with what you are trying to achieve but the examples use dimple without requirejs.  Any example (e.g. https://github.com/PMSI-AlignAlytics/dimple/blob/master/examples/bars_vertical.html) just references dimple using a url and though the chart code is directly in a script tag, it could equally be in an external JavaScript file.

Aha, ok no that is not what I am asking.
I would like to know, how do I have to change Gruntfile build, so that it only includes js files for types of charts that I will use. I do not the whole js file with all chart types, if I don't have them on my page.


Add documents to context

We can start by supporting only .txt files.

Together with @Tand-Anthros we found that Copilot and Bind from Microsoft do not allow to attach even simple .txt files.

Example of the same idea in Telegram bot:
https://github.com/deep-foundation/russian-laws-bot/blob/6c176bc44b03bb6d57a66d79665f34c05d729d2a/main.py#L230-L240
The place in code that should be updated:
https://github.com/deep-foundation/chatgpt-azure-discord-listener/blob/main/index.ts#L81-L93


请问该项目会升级ionic4吗

I'm submitting a ...  (check one with "x")
[ ] bug report
[ ] feature request
[ ] support request

Steps to reproduce the bug:
Related code:
    insert any relevant code here

Important information:

www.baidu.com

http://www.baidu.com

https://www.baidu.com/

https://www.baidu.com/


Feature/issue 796 mpi cluster

Submission Checklist

[ ] Run unit tests: ./runTests.py test/unit
[X] Run cpplint: make cpplint
[X] Declare copyright holder and open-source license: see below

Summary:
Add basic MPI cluster resource management.
Intended Effect:
Pave the way for map_rect_mpi.
This pull is marked WIP as it does include the not yet merged mpi build branch. This makes testing straightforward.
How to Verify:
Ensure to have mpich or openMPI installed. Also add to make/local
CC=mpicxx
STAN_MPI=true

Then execute ./runTests.py test/unit/math/prim/arr/functor/mpi_cluster_test.cpp. To run with more than just two processes please do mpirun -np 4 test/unit/math/prim/arr/functor/mpi_cluster_test.
Side Effects:
Documentation:
Doxygen documentation in the code.
Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company): Sebastian Weber
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

Hmm... the distributions tests fail with
remote file operation failed: /Users/Shared/Jenkins/gelman-group-mac/workspace/Math_Pipeline_PR-797-IZJL62LG5QGS6ERBPW3X3GK556K7ASCREG6A5ZJIUS3AE7MDD26A/dist.log.zip at hudson.remoting.Channel@111a1676:gelman-group-mac: hudson.remoting.ChannelClosedException: Remote call on gelman-group-mac failed. The channel is closing down or has closed down

Is there an issue on the Jenkins box?
I have not touched any code which messes with the distribution tests, so a mistake there is surprising.

hmmm... does anyone have a good idea here? The problem right now is that the header test for
stan/math/prim/arr/functor/mpi_cluster.hpp
fails, because the compiler cannot find the necessary mpi.hpp include and even if, the test would fail, because neither mpicxx is used nor are the necessary libboost_* libraries around.
A simple solution would be to ifdef-out inside the mpi_cluster.hpp the full file. So far I thought it would be sufficent to ifdef-out inside arr.hpp the mpi files, but it looks like that this strategy works for all stan programs, but not with our header testing strategy.
Is it OK if I ifdef-out the mpi headers whenever STAN_MPI is NOT defined? I don't see another solution.

I don't think the MPI headers should be included if STAN_MPI is not defined.  So #ifdef-ing them out should be OK.

Since the MPI build stuff is settling this pull can be reviewed - but please ignore the changes to the makefiles. These are part of the other MPI build pull and are left here for convenience such that the tests can be easily fired off.

Ok, MPI build is merged and we can proceed with this one.
This PR is a good one to learn how MPI works form the stan-math side.

Hey, would you mind posting the branch or a PR for the final map_rect code that will use this? It will help us look at this PR in context. Thanks!

Sean and I have just spent a couple hours going over this and now we're a bit confused about what this PR is supposed to be doing.  Is it just providing services that will be used by our reduce/combine level?  If so, would it make sense to review these two together?  If not, is there some simple interface you're trying to implement with all this to make the next level up easier?

I am happy to combine this PR with the final thing if this makes it easier for you. I intended with this PR to make it is easy for you to get a handle on MPI communication, but I understand that it may be easier to see the full thing at once.
It will take me a moment to bring in the map_rect_mpi in this PR. If it helps you can have a look at the file

https://github.com/stan-dev/math/blob/feature/concept-mpi-2/stan/math/prim/mat/functor/map_rect_mpi.hpp
https://github.com/stan-dev/math/blob/feature/concept-mpi-2/stan/math/prim/mat/functor/mpi_parallel_call.hpp

Those two codes do the main work (in fact the mpi_parallel_call.hpp is the main meat). Should we close this PR then?

As just discussed, I leave the PR up as is.

@bob-carpenter Thanks for your review. I will address these shortly.

The shared_ptr manages the lifetime of T. Whenever no more shared_ptr instance is pointing to T, then the last shared_ptr instance going out of scope will delete T. This is standard usage of shared_ptr's to my knowledge. I will add a comment.

I just read though the shared_ptr doc and you're right.  We don't need to doc usage of std library classes.  We just need to learn them.

...  How about I doc that this is standard serialization code and add a reference to where to dig in?

Thanks---that'll work.

I hope I got all what you raised. I saw that boost.serialization offers now support for std::shared_ptr such that I switched over to use these now. Once tests pass this should be good to go in.

Awesome!
I will prep the final stan-math MPI PR ASAP.


Official linux support

I think this'll be brilliant for Server Side Swift.
EDIT: I meant by including a CZlib library

I was convinced to make a PR rather than an issue :) I might create a CZlib package that compiles itself on Linux.
The motivation was that CZLib requires the headers to be installed. So compiling it might be easier for developers.


RCTTextField is not a descendant of RCTShadowView

Hey!
Thanks for react-native-keyboard-aware-scroll-view! All works great apart from the issue in which all of the TextFields which are not contained within  stop working. The error appears after the keyboard is presented :

Hello @nazywamsiepawel!
Yes, this is a known issue https://github.com/APSL/react-native-keyboard-aware-scroll-view/issues/25
I've opened a PR in react-native to solve this: https://github.com/facebook/react-native/pull/7876
In the meantime, if you take the code of the PR and the code of the ancestor_check branch you can solve the issue.
I hope this gets merged and solved soon.
Thanks!

Gracias!

Glad to have helped @nazywamsiepawel 😉
Let's close this and track the issue in #25

@nazywamsiepawel did you solve this problem? I followed all steps which suggested by @alvaromb
I face the warning  message blow
'keyboardWillShow' event should be registered via the Keyboard module.
'keyboardWillHide' event should be registered via the Keyboard module.
And get the following error message get TextInput get focused.
"_reactNative.UIManager.viewIsAncestorOf is not a found"
Please share your experience. thank you.

The warning messages are not related to the ancestor/descendant check. With v0.1.2 there shouldn't be any warnings.
For the viewIsAncestorOf, please refer to the PR and apply that to your source code: https://github.com/facebook/react-native/pull/7876

I patched it and cherry picked commit "A version with the RN ancestor stuff" and "Check if there is a focused field before trying to scroll to it"
However, still face TypeError: _reactNative.UIManager.viewIsAncestorOf is not a function error
here is my code where it throw an exception
if (this.props.enableAutoAutomaticScroll) {
      const currentlyFocusedField = TextInput.State.currentlyFocusedField()
      if (!currentlyFocusedField) {
        return
      }
      try {
        console.log(UIManager)
        UIManager.viewIsAncestorOf(
          currentlyFocusedField,
          this.getScrollHandle().getScrollResponder().getInnerViewNode(),
          (isAncestor) => {
            if (isAncestor) {
              this.scrollToFocusedInputWithNodeHandle(currentlyFocusedField)
            }
          }
        )
      } catch (e) {
        console.log(e)
      }
    }

I create a PR with this issue. PR #41
UIManager.viewIsAncestorOf => UIManager.viewIsDescendantOf

I'm getting the same error message as referenced by nazywamsiepawel above. Are you close to a solution? Is there a current workaround? I'm a bit frantic.

@idavidson12 it's a solved problem. Even some involved manual work, you can use this project with sub pages. Please check previous comments, written by @alvaromb and @JosephDev

I don't think I'm up to this level of coding yet. Will there be an update  to the module itself soon?

@idavidson12 um... you can clone my code instead using npm to install git clone https://github.com/JosephDev/react-native-keyboard-aware-scroll-view.git#ancestor_check
However, you have to modify iOS project files by yourself unless RN project merge @alvaromb 's PR facebook/react-native#7876.

Great. Thank you.

After implementing @alvaromb's changes I get a error that
'CSSLayout/CSSLayout.h' not found.' Do you have any idea how I might
resolve that?
Thanks for your module.
Iver
On Thu, Jul 28, 2016 at 3:48 PM, JOSEPH KIM<EMAIL_ADDRESS>wrote:

@idavidson12 https://github.com/idavidson12 um... you can clone my code
instead using npm to install git clone https://github.com/JosephDev/
react-native-keyboard-aware-scroll-view.git#ancestor_check
However, you have to modify iOS project files by yourself unless RN
project merge @alvaromb https://github.com/alvaromb 's PR
facebook/react-native#7876
https://github.com/facebook/react-native/pull/7876.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/APSL/react-native-keyboard-aware-scroll-view/issues/33#issuecomment-236035634,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANg7Qv96QpoHQ1FL1MmvYMD0H3N0sr9Sks5qaSO3gaJpZM4I0TnG
.

Please use the official branch for the ancestor check @idavidson12 https://github.com/APSL/react-native-keyboard-aware-scroll-view/tree/ancestor_check
My PR was accepted a few days ago and it should land into RN 0.32.0 https://github.com/facebook/react-native/pull/7876
Regarding the CSSLayout error you're getting, is it possible that you have removed some code accidentally? I'm using the component in all of my projects and I don't get that error.

did every thing you mentioned and not working :( :(

Resolve this problem.

npm install https://github.com/APSL/react-native-keyboard-aware-scroll-view.git#ancestor_check
modify the RN core file follow: https://github.com/facebook/react-native/commit/e52cab5a7f44aee5c145e9995850ab8051829b5c

I don't know how to modify after read facebook/react-native#7876, so direct read the source code that @alvaromb submit to RN official. then resolved it, spend near 5 hours :-< , Anyway, thank you @alvaromb 's hard work and contribute.

@alvaromb how android resolve it?

after upgrade to RN0.34 still have this problem

Hey, i am using latest react native and still facing this error...
"react-native-keyboard-aware-scroll-view": "^0.3.0",

Try to remove the node_modules/ folder and re-install dependencies.


fix alignment errors on i386

I think this whole testing for sizes/alignment is fundamentally wrong. Or we should know better about all the requirements of various platforms/architectures out there than bindgen.

The code for sys.rs is auto-generated, do not change it manually. We can disable this test via options. https://docs.rs/bindgen/latest/bindgen/struct.Builder.html#method.layout_tests

Thanks, the context for this patch is that we are trying to package io-uring for debian, and i386 is one of our supported platforms: https://qa.debian.org/excuses.php?package=rust-io-uring
I guess this is really a bug in bindgen then, and that this one should be closed.


added color change fix for CodeFolding arrows and change of cursor

The hover color change has been fixed, it's highlighted when hovered and dimmed when not.
Also, added the cursor to be a pointer when over the arrows line.
Issue on thimble : https://github.com/mozilla/thimble.mozilla.org/issues/1874#issuecomment-286223341
I've uploaded the working of it here(gif).

Hi @geekodour, instead of saying "Issue #1874 from Thimble" in the title, can you take that part out, and then in the PR comment itself note "fixes "? This makes github automatically set up crosslinks between the PR and the issue, making PR/issue administration easier.

@Pomax I've removed "Issue #1874 from Thimble" as you stated, but pushing another change with a new commit message will probably not work with "fix" because thimble is another repo and I am pushing change on the brackets repository. Am I right? In addition to that, I've added the link to the issue on thimble in the main comment.

As long as it's in the main comment, github will maintain the crosslink, but that "fixes ..." was not a suggestion: it needs to literally be fixes ...... in order for github to set up the crosslink, so not "issue on thimble: ..." =)
(the fixes #12345 is github.com-specific syntax that makes their backend setup crosslinks, and automatically close PR-associated issues if they're in the same repo)

Testing it out now!

@Pomax Except just using #1234 wouldn't work because that would reference issue 1234 in this repo? Does it work with a full URL? I've added it to the message so we'll test it.
As far as the PR, thanks very much for submitting it. In reviewing it I've noticed a few things

The hover effect happens no matter where you hover in the gutter, even in places when you can't click the arrows
The cursor changes to a pointer along the entire height of the arrow div, which is incorrect because it is only useful to click it directly overtop of the arrow.

I think this is just because of the design of this feature. There's an element that overlaps all of the arrows so we can't really apply :hover styles to them it seems.
Because of this could we instead:

Remove ALL hover and cursor effects from the arrows?
And just make sure they are always the same color when hovered or not.

Thank you!

okay great, I'll send the changes very soon. thanks for reviewing.

There could be another possibility for achieving this, changing these lines where the over-gutter class is assigned, here: https://github.com/mozilla/brackets/blob/master/src/extensions/default/CodeFolding/main.js#L282-L297

@geekodour a note to let you know that @flukeout is assigned to review this, but has limited time this week.  So he may be a few days before he can get to it. Just so you didn't feel like it was being ignored.

@geekodour - Hey this is great - just tested it and I think it's good to go. I left some comments up above and once those are fixed - we can land this. Thank you!

@geekodour it looks like we're going in a different direction, see https://github.com/mozilla/brackets/pull/667.  I'm going to close this, since it won't be necessary with @flukeout's change.  Thank you for working on this, and feel free to keep working with us on another bug.  It's been great having you contribute and be part of the project.


⚠ Bofrost.be has degraded performance

In 41805bf, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1042 ms

Resolved: Bofrost.be performance has improved in fe7f235 after 26 minutes.


2.0 preview

Polymer 2.0 hybrid element, intended for a 2.0-preview branch.
Tested on Ubuntu 16.04, on Chrome 58 and FF 53.

'polymer build' is currently broken, see https://github.com/PolymerElements/paper-button/issues/162
Lazy loading is now supported by the component; the --paper-tree-toggle-theme is however retained for 1.0 compat (untested).
Replaced paper-menu with paper-listbox, since paper-menu does not yet have a 2.0-preview branch.
Added direct dep to web-animations-js to get rid of "Uncaught ReferenceError: KeyframeEffect is not defined".

@vpusher thanks for a great custom element. Could this PR be accepted to make it work with Polymer 2.0?

Ping


Issues in the Console Application's user attribute section

Describe the issue:
The following issues are identified in the user attribute section of the console's Application section.

[ ] Button "Add user Attribute" not getting loaded when removing all the added attribute at SAML application.

https://user-images.githubusercontent.com/39077751/121219410-48b45600-c8a1-11eb-8e77-95560e2d792e.mp4

[ ] Cannot delete mapped application attribute

https://user-images.githubusercontent.com/39077751/121245035-2fb99e00-c8bd-11eb-8805-35ff96d60d95.mp4

[ ] Unable to select subject attribute when attribute mappings are available in SAML 2

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Try to solve the memory leak problem after navigating on friends page

I think now this issue get solved in previous work


WalletConnect - Pairing timeout error

Describe the bug
A clear and concise description of what the bug is.
SDK Version

Client: [Swift]
Version [1.16.17]

To Reproduce

Change ProjectID in Example app
Run Example app
Scan QR from https://react-app.walletconnect.com/
See in console 🚄 [WebSocket] - Fallback to relay.walletconnect.org
There will be a alert WalletConnect - Pairing timeout error

Expected behavior
Connection without issues
Screenshots
https://github.com/WalletConnect/WalletConnectSwiftV2/assets/13622708/0a0724f8-0481-4b69-b7dc-3d6648884f17
Device (please complete the following information):

Device: [e.g. iPhone13Pro]
OS: [e.g. iOS16.6]

Additional context
Changed ProjectID in both places - Configuration.xcconfig and project build settings

Starscream set Origin, but relay.walletconnect.com does not accept it

我遇到了一个问题, 我在调用 DApp 程序(V2版本号是 : 1.6.17 )的时候, 生成的二维码无法进行正常的扫码登录链接, 跳转的目标应用告知我, 发生了一个未知的错误, 我 copy 到所生成二维码的链接也无法拿去使用(跳转其他应用也没办法成功授权), 为此我录制了操作步骤的视频, 希望能在这里得到大家的热情帮助
https://github.com/WalletConnect/WalletConnectSwiftV2/assets/48764850/3c8a8a75-e903-45b3-80b5-3c60fa0a25fe

@ShootingGuard24  use Zerion wallet ，not use imToken wallet

@rkingxbank Other wallets can be used, but the imtoken wallet cannot be linked normally. I would like to know why it cannot be used? Looking forward to your reply XD


docs: Fix links to vision

Which issue does this PR close?
Closes #.
Rationale for this change
As title.
What changes are included in this PR?

Are there any user-facing changes?

Thank you @PsiACE for the review!


Release the scene level model

Dear Authors,
I greatly appreciate your efforts in open-sourcing this impressive model. I am interested in the upcoming release of the scene-level pretrained model on MV-imagenet+Objaverse. Could you kindly share the expected release date for this? Additionally, I would like to inquire if there are any intentions to train the model on Objectverse XL in the future. Thank you for your attention to these queries.

Hi @Adamdad,
Thank you for your interest and inquiries.
Regarding MVImgNet, the models are still under development, and we cannot guarantee a precise release date. However, we hope to release them next month, IF everything goes according to the plan :)
As for Objaverse-XL, we do not plan to use it in the near future since the v1 dataset may be of higher quality.


 Getting to 81% is tough!!!
Feedback: Try to push yourself to get in the top 3. Nice commenting.
Do more exploratory data analysis, data cleaning, feature engineering, and feature selection.

Experiment with LabelEncoder to encode labels on ytrain and Onehot for categoricals.
Experiment with different feature selections

Try a Random Forest Classifier.

Check

Try Ordinal Encoding.

Check

What other models have you tried? GBM? XGboost?
Submit new predictions.

Module 04 Confusion Matrix:
Overall: Good job this week!!!
Feedback: Create a kernel with more text explanation
Get and interpret the confusion matrix for classification models

At least try to implement a confusion matrix 😕
What does the confusion matrix tell you?
What is the significance of the precision, recall, f-1 scores?

Kaggle Challenge

What is the most important thing that you learned in this Kaggle challenge? Write a blog about it ✍ .
Did you have problems with overfitting or underfitting? If so, how was it handled?
Ask yourself, what you would do differently on your next Kaggle challenge? In approach and execution.
Were there any models that you did not try to implement that you wish you did?


fix: Changes the phone number returned to be only 14 digits max.

Description
As per title - reduces the number of digits returned in the phone number to a + followed by 14 rather than 15 digits. The second execution path where the number doesn't start with a 1 is included and it reduces the number down to a +1 and 13 digits.
Please provide a summary of the change and the issue fixed. Please include relevant context. List dependency changes.
Fixes # (issue)
CE-965
How Has This Been Tested?
Created a number of records with problematic numbers included and confirmed they appear as expected in the app.
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Checklist

[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Further comments

Thanks for the PR!
Deployments, as required, will be available below:

Frontend
Backend

Please create PRs in draft mode.  Mark as ready to enable:

Analysis Workflow

After merge, new images are deployed in:

Merge Workflow

Closing to recreate a new PR with changes for the phone number format in a single PR.


[powershell] vcpkg doesn't work with PowerShell installed from Windows Store.

Describe the bug
It seems that vcpkg doesn't work with the version of PowerShell Core installed from the Windows Store. The initial check seems to pass and vcpkg doesn't download its local copy, but vcpkg_copy_tool_dependencies issues an error when trying to build a port that uses Powershell.
Environment

OS: Windows 10 Pro 20H2
Compiler: 19.28.29914 (VS 2019 16.9.4)

To Reproduce
Steps to reproduce the behavior:

Install Powershell from the Windows Store (version 7.1.3)
Sync the vcpkg repo (e.g. to c:/tools/vcpkg); run bootstrap-vcpkg.bat
In a directory of your choice set the following environment variables

set VCPKG_TOOLCHAIN_FILE=C:/tools/vcpkg/scripts/buildsystems/vcpkg.cmake
set VCPKG_FEATURE_FLAGS=manifests,versions
set VCPKG_MANIFEST_DIR=%~dp0

And edit the vcpkg.json file:
{
    "$schema": "https://raw.githubusercontent.com/microsoft/vcpkg/master/scripts/vcpkg.schema.json",
    "name": "brotli-test",
    "version": "0.0.1",
    "builtin-baseline": "5a10163c44526b5d42a67480ce7a692a36948761",
    "dependencies": [
        "brotli"
    ]
}

run vcpkg install --binarysource=clear

Expected behavior
Build succeeding
Failure logs
Detecting compiler hash for triplet x86-windows...
The following packages will be built and installed:
    brotli[core]:x86-windows -> 1.0.9#1 -- C:\tools\vcpkg\buildtrees\versioning\versions\brotli\8f55fe158d8bd753a6e6908164e03ae4f0b73cea
Starting package 1/1: brotli:x86-windows
Building package brotli[core]:x86-windows...
-- Installing port from location: C:\tools\vcpkg\buildtrees\versioning\versions\brotli\8f55fe158d8bd753a6e6908164e03ae4f0b73cea
-- Downloading https://github.com/google/brotli/archive/e61745a6b7add50d380cfd7d3883dd6c62fc2c71.tar.gz -> google-brotli-e61745a6b7add50d380cfd7d3883dd6c62fc2c71.tar.gz...
-- Extracting source C:/tools/vcpkg/downloads/google-brotli-e61745a6b7add50d380cfd7d3883dd6c62fc2c71.tar.gz
-- Applying patch install.patch
-- Applying patch fix-arm-uwp.patch
-- Applying patch pkgconfig.patch
-- Applying patch fix-ios.patch
-- Using source at C:/tools/vcpkg/buildtrees/brotli/src/6c62fc2c71-f2afd323e3.clean
-- Found external ninja('1.10.2').
-- Configuring x86-windows
-- Building x86-windows-dbg
-- Building x86-windows-rel
CMake Error at scripts/cmake/vcpkg_copy_tool_dependencies.cmake:29 (message):
  Could not find PowerShell Core; please open an issue to report this.
Call Stack (most recent call first):
  buildtrees/versioning/versions/brotli/8f55fe158d8bd753a6e6908164e03ae4f0b73cea/portfile.cmake:24 (vcpkg_copy_tool_dependencies)
  scripts/ports.cmake:142 (include)

Error: Building package brotli:x86-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `.\vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: brotli:x86-windows
  Vcpkg version: 2021-01-13-d67989bce1043b98092ac45996a8230a059a2d7e

Additionally, attach any relevant sections from the log files above.

Additional context
where pwsh.exe issued in the same command prompt prints out: c:\Users\zomgrolf\AppData\Local\Microsoft\WindowsApps\pwsh.exe
Running vcpkg with the --debug flag produces this:
[DEBUG] Found path: c:\users\zomgrolf\AppData\Local\Microsoft\WindowsApps\pwsh.exe
[DEBUG] Found path: C:\Users\zomgrolf\AppData\Local\Microsoft\WindowsApps\pwsh.exe
[DEBUG] CreateProcessW("c:\users\zomgrolf\AppData\Local\Microsoft\WindowsApps\pwsh.exe" --version)
[DEBUG] cmd_execute_and_stream_data() returned 0 after   330190 us

There is no copy of PowerShell-7.1.0-win-x86.zip in c:/tools/vcpkg/downloads
If I replace the version from Windows Store with a version installed from the *.msi installer the build succeeds. It seems that CMake is unable to find the Windows Store version of PowerShell?

Did you use powershell or cmd?

cmd.exe

@zomgrolf Please use powershell instead.

You have to be joking.

@zomgrolf  Forgive me... I didn't understand clearly.

@strega-nil Can you please take a look? I think this is a bug of vcpkg.

My guess is it's an issue with CMake (or CMake scripts used by vcpkg?). The method vcpkg uses to test if there's a suitable version of PowerShell available seems to succeed (as expected), however the call to find_program(PWSH_EXE pwsh) in vcpkg_copy_tool_dependencies.cmake fails (unexpectedly).
I would expect that call to succeed, not sure why it doesn't. I've just tried this simple CMake script in a separate command prompt:
cmake_minimum_required(VERSION 3.19)

project(TEST)

find_program(PWSH_EXE pwsh)
if (NOT PWSH_EXE)
    message(FATAL_ERROR "Could not find PowerShell Core; please open an issue to report this.")
endif()

add_executable(test test.cpp)

And it fails with the Windows Store version of PowerShell Core. However, CMake is clearly is the lowest common denominator here -- so until this is fixed, I would suggest making vcpkg a bit more eager to download a local copy of PowerShell.

All right, this is getting odd: I've re-run the script with CMAKE_FIND_DEBUG_MODE set to true, and CMake reports that C:/Users/zomgrolf/AppData/Local/Microsoft/WindowsApps/pwsh.exe is on the list of locations considered by find_program -- that's the location reported by where pwsh.exe for a Windows Store version of PowerShell.

The code for detecting pwsh is c++, and the code for finding and using pwsh is cmake.

Does the C:/Users/zomgrolf/AppData/Local/Microsoft/WindowsApps/pwsh.exe need some kind of environment variable we're stripping or something like that?

I don't think so. All I did was install PowerShell from the Windows Store, open a new command prompt in Windows Terminal, and then try to install the libraries. Same thing, if I simply open a new command prompt and try to run CMake using that simple script above, find_program simply fails, so I think ultimately this is an issue with CMake, unless that call to find_program is wrong and a different method should be used for the Store apps.
The call to find_program seems to fail in the exact same way whether it's made by CMake executed by vcpkg or manually by me from the command prompt, so I don't think it's anything that vcpkg does.

It looks very similar to #15681

@zomgrolf  Yes, almost same issue.

I think the underlying issue here is that, when installed from the Microsoft Store, C:/Users/zomgrolf/AppData/Local/Microsoft/WindowsApps/pwsh.exe is a symlink to a file inside a Windows app package, rather than a stand-alone executable.
I've dug a little deeper into how find_program in CMake works: for any candidate it performs two tests: it checks if the file at the given location exists and if it is executable.
The code that checks if the given file exists is here: SystemTools::FileExists
It calls GetFileAttributesW on the given path, checks for the presence of the FILE_ATTRIBUTE_REPARSE_POINT to detect a symlink, then calls CreateFileW. On my machine that second call fails with what looks like a security error, which shouldn't be surprising, I guess, given how tightly locked down Windows app packages are. This is what most likely makes CMake ignore the seemingly valid PowerShell executable.

I have the same issue, failing to build brotli so it seems brotli specific. The annoying part is other packages that build just fine while detecting powershell(from the windows store).
C:\usr\src\vcpkg>vcpkg list
fmt:x64-windows                                    7.1.3#4          Formatting library for C++. It can be used as a ...
fmt:x86-windows                                    7.1.3#4          Formatting library for C++. It can be used as a ...
jsoncpp:x64-windows                                1.9.4            jsoncpp is an implementation of a JSON reader an...
jsoncpp:x86-windows                                1.9.4            jsoncpp is an implementation of a JSON reader an...
sdl2:x64-windows                                   2.0.14#4         Simple DirectMedia Layer is a cross-platform dev...
sdl2:x86-windows                                   2.0.14#4         Simple DirectMedia Layer is a cross-platform dev...
vcpkg-cmake-config:x64-windows                     2021-05-22
vcpkg-cmake-config:x86-windows                     2021-05-22
vcpkg-cmake:x64-windows                            2021-02-28#2
vcpkg-cmake:x86-windows                            2021-02-28#2

I've also experienced this problem. In my case, it happens when installing brotli on a project that uses a vcpkg.json manifest file. I am working in VS Code with the C++ and CMake extensions, and I notice the error when configuring the project, which triggers vcpkg to build and install my dependencies because I am using a manifest file. If I launch VS Code from my desktop or Start menu, the problem manifests. If I launch VS Code from my PowerShell Core shell, everything works.

The CMake issue was addressed after all. The dummy python alias (AppInstallerPythonRedirector) is explicitly banned. Theoretically, other store apps should work in the future CMake release.

Will this ever be fixed for older CMake versions? (I cannot upgrade CMake beyond 3.19)

@emptyVoid please CMake developers directly in their issue tracker.

@emptyVoid please CMake developers directly in their issue tracker.

My question has nothing to do with CMake.
vcpkg @ 59a8a9c6248ed230b19028a44484ae5f06db697d worked flawlessly without requiring specific flavors of third-party utilities installed in the system (by downloading all the required tools by itself). I would stay with that revision if it worked with VS 2022, sadly it's unable to find VS with VCPKG_VISUAL_STUDIO_PATH and VCPKG_PLATFORM_TOOLSET specified (the latest revision has no issue with the same triplet file).
Thus the question is about fixing this issue in this repository.

@emptyVoid please CMake developers directly in their issue tracker.

My question has nothing to do with CMake.
vcpkg @ 59a8a9c worked flawlessly without requiring specific flavors of third-party utilities installed in the system (by downloading all the required tools by itself). I would stay with that revision if it worked with VS 2022, sadly it's unable to find VS with VCPKG_VISUAL_STUDIO_PATH and VCPKG_PLATFORM_TOOLSET specified (the latest revision has no issue with the same triplet file).

We have never downloaded all required utilities if usable versions were present on the system. Powershell is a little different here because of the "powershell" vs "pwsh" thing of course.

Thus the question is about fixing this issue in this repository.

Depending on the path by which you get to a place, our options are limited. If we are invoked via CMake integration with manifests (as in #21203 ) we have no opportunity to run first and download things. (That said the powershell in ##21203 is very much not coming from the Windows Store)
Ultimately the best fix would be to get powershell out of this dependency chain. I had a prototype of something like this but ran into the snag that some ports explicitly depend on powershell, for example: https://github.com/microsoft/vcpkg/blob/master/ports/qt5-base/qtdeploy.ps1 --- and I ran out of time attempting to come up with a solution at that time. Maybe something that can be picked up later...

@emptyVoid  For now can you confirm that installing powershell-core on the system from https://github.com/PowerShell/PowerShell/releases/tag/v7.2.0 temporarily resolves the problem?

We have never downloaded all required utilities if usable versions were present on the system.

Yeah, I do realize that, but still that revision did download some newer PowerShell (at least I remember seeing it in the build log).

@emptyVoid For now can you confirm that installing powershell-core on the system from https://github.com/PowerShell/PowerShell/releases/tag/v7.2.0 temporarily resolves the problem?

Depends on the install options: vcpkg is able to locate it only if I agree to add PS to PATH.

Depends on the install options: vcpkg is able to locate it only if I agree to add PS to PATH.

Makes sense.
If anyone wants to take a crack at designing the way ports like qt5-base should work without powershell and wants to start from my work-in-progress, here you go: https://github.com/BillyONeal/vcpkg-tool/tree/replace-powershell
I suspect we need to let ports create some kind of a marker file next to their DLLs that says 'if this DLL is applocally deployed, also applocal deploy these files/directories/DLLs' or similar.

(I want to resolve this problem but don't expect I'll have time to look again before mid-December)

@BillyONeal Wrt to applocal.ps1 I would still suggest to try CMake first.

@BillyONeal Wrt to applocal.ps1 I would still suggest to try CMake first.

When applocal.ps1 there is not necessarily any cmake available.


calculate newton-raphson method explicitly for two obzervation vectors

https://github.com/IvanVnucec/Optimal-REQUEST/blob/ec4b798e5f16314ef0797d3ad511d70485709b80/matlab/scripts/get_quat_from_K.m#L46
look into here eq. 72 for help
Idea is to try with sympy to compute newthon-rapshon method explicitly to only have one eq. like we have in eq. 72.

I've implemented the eq. 72 but the RMS Euler error was greater than the solution with the Newton-Raphon with 2 iterations method by 3%.
Also I've tried to simbolicly calculate Lambda_max with Newton-raphon method with 2 iterations but the equation is rather large (50+ elements)
No progress with this issue.


Update jenkins master & workers to JDK8

Filing FTR, not urgent, but we should eventually do that (preferably if some bigger overhaul is happening anyway).
Jenkins >2.54 requires JDK8.

This has been done.


Table has no column with name doc

Currently, there is no way to send the data to the indexing process without creating a doc column from the input.
Need to fix the indexing error:
AttributeError: Table has no column with name doc.
Occurred here:
    Line: query_context = index.query(embedded_query, k=3).select(
    File: /home/bumurzokov/llm-app/src/prompt.py:14

When no doc column defined, it always fails at the index stage:
# Compute embeddings for each document using the OpenAI Embeddings API
embedded_data = contextful(context=documents, data_to_embed=documents.doc)

The index expects a column doc containing the chunks and data having their corresponding embeddings. Open for any suggestions or alternatives.


fix(api): SubscriptionOperation dispatch completion event before super.cancel

Issue #, if available:
Description of changes:
AmplifyOperation's cancel() unsubscribes the result listener in a recent change. The implementation of the AWSGraphQLSubscriptionOperation's cancel() dispatches a successful completion event on "cancel" as part of the behavior of the lifecycle of a subscription. If it operation gets cancelled, the caller gets an inprocess event of "disconnected" followed by a result event of successful Void. The code change changes the sequence performed during cancel() by calling dispatch(result: successfulVoid) before calling super.cancel()
Check points: (check or cross out if not relevant)

[ ] Added new tests to cover change, if needed
[ ] Build succeeds with all target using Swift Package Manager
[ ] All unit tests pass
[ ] All integration tests pass
[ ] Security oriented best practices and standards are followed (e.g. using input sanitization, principle of least privilege, etc)
[ ] Documentation update for the change if required
[ ] PR title conforms to conventional commit style
[ ] If breaking change, documentation/changelog update with migration instructions

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

this was merged in with another PR https://github.com/aws-amplify/amplify-ios/pull/2140


Generate docs for AST from checks

This fixes #3172
Please take a close look. I think I matched everything correctly, but it would be good to have a second pair of eyes check it over.
This updates explain for operations that require AST so that the expressions that would not work with AST are shown in the standard place and not repeated in the reason why the parent operator cannot work.
This drops the lambda expression context entirely.

build


Added custom provider annotation details

Signed-off-by: Andre Wanlin<EMAIL_ADDRESS>Hey, I just made a Pull Request!
Added details about the fact that you need to make sure to add the backstage.io/managed-by-location and backstage.io/managed-by-origin-location annotations to your entities when you create a custom provider seeing as if they are not provided your entities won't show up in the Catalog and you'll start to see warnings in the logs.
This was noted on Discord: https://discord.com/channels/687207715902193673/1037085190557806622
:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

@GoFightNguyen if you have a minute can you take a look at this? Does this seem clear to you? I feel like this is the best place to put this but struggled a little with the wording

@freben this is one of the items I mentioned in the last Catalog SIG, more any FYI then anything

@GoFightNguyen so what do you think? Good to go or needs work?

@awanlin lgtm 🚀

Ok @jhaals feel free to merge this in when you have a minute, please 👍

Thanks @jhaals and @GoFightNguyen 👍 😄


Add Discord API to Network Page

Connect the Discord API to pull #developer-annoucements from the discord channel on the Flow server.
The components showing the announcements should then be live data.

Seems like there isn't a straightforward way to do this, like using an API. Instead, we'll have to create a bot for our server and do something like this: https://github.com/remix-run/remix-discord-bot


Missing last disconnection for connected peers when crawler goes down accidentally

We knew that this edgy case could happend, and it is time to check for the best solution to fix it.
Since we would have to iter through the entire database to see which peers had the connected flag enabled, I propose to register from all the peers in the peerstore which was the time of the last connection/disconnection event. This way, we would know when was the last moment when we are sure that the crawler was active, without adding/exporting extra info.
What do you think about it @alrevuelta  @tarun96T ?

Closing Issue, got solved in #36


feat (config) allow the user the ability to change

Summary
large_client_header_buffers nginx configuration option in kong.conf
Nginx returns "400 Request Header Or Cookie Too Large" when cookies are
too large, so the large_client_header_buffers needs to be configured to
allow large cookie request through Kong.
Full changelog

set nginx default in kong_defaults.lua
allow overrides in the templating
tests to ensure that changes git picked up

Issues resolved

Thanks for the suggestion,
Unfortunately, I am afraid this falls under the same category as #3010, #3323, and a few other ones. See this comment/thread for context on our current stance with regards to customizing the nginx directives from the Kong template.
We'd rather introduce a nicer way to solve this issue altogether (otherwise, the list of configuration variables will grow out of control). Our options are:

#2355 (the cleanest imho)
Hooks into the nginx configuration, such as proposed by #2675
Another generic way of overriding any nginx directive from kong.conf. I have given this some thought as a temporary solution, or compromise until/if we ever get to 1.

Another approach we have been discussing with @maguec that does not involve managing different configuration files (as per suggestion 2.), and still allows for granular nginx directive override (with both kong.conf and environment variables), is the declaration of "flexible" configuration values in kong.conf (therefore picked up by env variables as well):
There would exist at least 3 prefixes that I can think of for now, in order to inject those directives in various levels of the nginx configuration:
# injected into the http {} block
nginx_directive_http_<directive_name> = <directive value>

# injected into the proxy server {} block
nginx_directive_proxy_<directive_name> = <directive value>

# injected into the admin server {} block
nginx_directive_admin_<directive_name> = <directive value>

Translated to environment variables, those would give:
KONG_NGINX_DIRECTIVE_HTTP<DIRECTIVE_NAME>=<directive value>

This addition would require nginx template changes (as would the proposal 2. above), and some changes in the configuration loader (conf_loader.lua).

Closing this since some internal decisions and progress has been made on the above proposal. Thanks for the suggestion @maguec


Eloquent\Model multiple implementation

Hi, autocomplete models in phpstorm not work.
Laravel 5.4.12
Ide-Helper v2.3.0
If delete a file _ide_helper.php that autocomplete in model file work.

If delete a file _ide_helper.php that autocomplete in model file work.

Version 2.2.3 is correct. 2.3.0 same error like shown.

It's not an error, it's a warning. But doesn't it work now then?

Can you try with https://github.com/barryvdh/laravel-ide-helper/commit/25c8de637ddf79567d721c1856616be01db7b960 ?

Yes, that works! ;)

@barryvdh Thanks, it works

Can you try with https://github.com/barryvdh/laravel-ide-helper/releases/tag/v2.3.2 ? Should be able to complete when extending Eloquent or using @mixin \Eloquent

I'm using 2.3.2 in a new Laravel 5.4 project.  I've also used
php artisan ide-helper:models --nowrite

to put the annotations in a separate file.  Naturally, this means I get the warning from PHPStorm: "Other declaration of class User exist at User.php" for my models.
Is there a solution to this when using a separate _ide_helper_models.php file?


Terms and Conditions page and link (for server version).

(1) T&C static page needs to be created (see TIE team folder under Privacy and Legal) (2) the modal should link to the T&C page so that the user can open and view it at the time they read the modal content (3) add a link next to the Privacy link on lower right corner that links to the T&C page.

Closing, not a client issue.


JDK version

Do we need to work on the internal MS version or can work on standard one?

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

ID: 97e1cd96-e7a2-ec22-8d79-ac949e890ccb
Version Independent ID: 922a3fdf-7bab-7d9d-5518-904db9e75ee3
Content: Microsoft's Mobile OpenJDK Distribution - Xamarin
Content Source: docs/android/get-started/installation/openjdk.md
Product: xamarin
GitHub Login: @vyedin
Microsoft Alias: vyedin

Thanks for submitting an issue and we apologize for the delayed response. The requested updates haven't been made since the creation of the issue, and the timeline for resolution may vary based on resourcing, so the issue is being moved to an internal repo to better track it.


Does Chiron support generic type ?

Hello guys,
I would like to support this type. The goal is to allow me to wrap my json response in this type to avoid to repeat myself. All the sub type which are going to replace the 'T type are supported by chiron.
type JsonResult<'T> =
    { Code: int
      Content: 'T
    }

    static member ToJson (x: JsonResult<'T>) =
         Json.write "Code" x.Code
      *> Json.write<'T> "Content" x.Content

The problem is I got this error:
This code is not sufficiently generic. The type variable  ^T when ( ^T or ToJsonDefaults) : (static member ToJson :  ^T -> Json<unit>) could not be generalized because it would escape its scope.
Is this feature possible ? If yes could you help me please :)
[Edit]: I suppose I need to restrict the Type of 'T but don't know how.

Well, i've made it work... 😄  Two things need to change - the ToJson member needs to be inlined, and the type parameter needs to be named differently, to avoid indicating that it will always be the one given in the type declaration (as there isn't a way to apply the appropriate constraints to that one).
So, the following will actually work:
type JsonResult<'T> =
    { Code: int
      Content: 'T }

    static member inline ToJson (x: JsonResult<'U>) =
         Json.write "Code" x.Code
      *> Json.write "Content" x.Content

Although it doesn't seem like it should work, it does... It is beginning to get quite close to the edges of the F# type system though!
On a slightly different note - you might find that more specialised types actually help rather than the generic one. This approach would now lead to a potentially difficult to define data model from a consuming perspective. But you may not need to worry about that!

Thanks for the helps :)
And I think you could be right about more specialized types because finally I made used of HTTP Status Code and specialized each of my "bridge" between the browser and the server. It's means a little more code to support the different case but could by easier to evolve finally.
Thanks again for the help and great job on Chiron and Aether thus projects are awsome. :)

No problem! That solution sounds good, and as you say, the cost of a little extra code might make it easier to change without breaking things in future! Glad you're enjoying the libraries :)


Add support for passing "latest" to DownloadAsync (for both Chrome and Firefox)

The idea is to automatically fetch the latest (stable) revision by performing HTTP GET request according to the specified platform/product.
Product: Chrome
For reference, see explorer-like viewer.

OperatingSystem/Platform: Windows

x64 / 64-bit
x86 / 32-bit

OperatingSystem/Platform: Linux

x64 / 64-bit

OperatingSystem/Platform: Mac

Intel
Arm

As for Firefox, there already seems to be some kind of mechanism for this in PuppeteerSharp, but I am not sure if it is functioning properly as I haven't tested that myself yet.
Further?
Perhaps also add support for "nightly" / "beta" builds... At which point it would be a good idea to have enumeration overload instead of using string to reduce likely hood of errors (e.g. DownloadAsync(BrowserRevision.Beta)).

We have a mechanism for Firefox. But for Chromium, each Puppeteer version goes with a Chromium revision. There is no guarantee that any puppeteer version will work with the latest chromium version.


disk-smart: make the --ignore parameter repeatable

In GitLab by @NavidSassan on Feb 7, 2020, 13:48

Rewrite the check and split it up, at max:

disk-smart-general
disk-smart-attributes
disk-smart-stats
disk-smart-self-tests
disk-smart-error-log
disk-smart-temp-log
disk-smart-advanced


How to add fields in join table of many-to-many relations?

I want to save relation specific information(i.e. the role a person played in a movie) in the join table. When defining many-to-many relations in the model, can I specify any additional property that can be passed when the relation is created?

There is no easy way, but you can do this:

Create a model for the join table.
Pass the model to the relation definition in the relationMappings using join.through.modelClass
Use query context feature cleverly :smile:

The third step means something like this:
class JoinTableModel extends Model {
  $beforeInsert(context) {
    if (context.role) {
      this.role = context.role;
    }
  }
}

someModel
  .$relatedQuery('theManyToManyRelation')
  .context({role: 'theRoleForTheJoinRow'})
  .insert({...});

So the query context is also passed to the join row insertion. Since we defined a model for the join table, its $beforeInsert gets called and the context is passed to it.
How awful does this sound? :smile:

Neat! :smiley:
Not sure how do I fetch this data, though. How would I query Person model to get the Movie along with her role in the movie?

I would just use a join. The first example only works on 0.5.0-alpha.X and newer versions. MoviePerson is the join table in the examples:
Person
  .query()
  .eager('movies')
  .filterEager('movies', builder => {
    builder
      .join('MoviePerson', 'MoviePerson.movieId', 'Movie.id')
      .select('Movie.*', 'MoviePerson.role as role');
  });

With objection 0.4.X (and newer) you can do this:
Person
  .query()
  .eager('movies(joinRole)', {
    joinRole: builder => {
      builder
        .join('MoviePerson', 'MoviePerson.movieId', 'Movie.id')
        .select('Movie.*', 'MoviePerson.role as role');
    }
  });

Or if you already have the person fetched (works on all versions):
person
  .$relatedQuery('movies')
  .join('MoviePerson', 'MoviePerson.movieId', 'Movie.id')
  .select('Movie.*', 'MoviePerson.role as role');

0.5 one looks neat! time for an upgrade :camel:

@afm-sayem you can start using latest features already by installing "objection@next" :tada:

cool!


Add support for external media providers

Support for providers such as giphy, ...
The plugin is generalized to support any custom providers.
To add a new provider, create a new class and extend the base Provider class.
To use a provider, specify it when registering the plugin.
Ref: https://github.com/crusttech/webapp-messaging/issues/52

Sneak peek into gifs

Awesome!!
2 questions:

can we disguise this into command (/giphy ...) ?
can keep image proportions (not sure if gifs are pre-squashed or are we doing this)?

@tjerman ^^

We can make it a slash command, yes; but I would prefer to allow both /gif [provider optional] and gif [provider optional]. I would not call the command giphy in case we decide to add support for other gif providers.

I will check ratios in a bit, but if I remember correctly, I am not squishing them on purpose (will double check tho).

Let's call it /giphy and have /gif alias for it.


Azure Stack support for "az account get-access-token"

I would expect to get ADFS token with the following command line: az account get-access-token --output json --resource "https://vault.azure.net/" --tenant "adfs"
Currently I get:
The command failed with an unexpected error. Here is the traceback:
Unable to get authority configuration for https://login.microsoftonline.com/adfs. Authority would typically be in a format of https://login.microsoftonline.com/your_tenant Also please double check your tenant name or GUID is correct.
Traceback (most recent call last):
  File "D:\a\_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\msal/authority.py", line 120, in __init__
  File "D:\a\_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\msal/authority.py", line 184, in tenant_discovery
ValueError: OIDC Discovery endpoint rejects our request. Error: {"error":"invalid_tenant","error_description":"AADSTS90002: Tenant 'adfs' not found. Check to make sure you have the correct tenant ID and are signing into the correct cloud. Check with your subscription administrator, this may happen if there are no active subscriptions for the tenant.\r\nTrace ID: 3105d434-64b0-46e5-af51-1c629b2d7501\r\nCorrelation ID: 4a459b09-3019-4a3d-8d97-e6e23d744379\r\nTimestamp: 2022-12-10 18:54:15Z","error_codes":[90002],"timestamp":"2022-12-10 18:54:15Z","trace_id":"3105d434-64b0-46e5-af51-1c629b2d7501","correlation_id":"4a459b09-3019-4a3d-8d97-e6e23d744379","error_uri":"https://login.microsoftonline.com/error?code=90002"}

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "D:\a\_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\knack/cli.py", line 233, in invoke
  File "D:\a\_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 663, in execute
  File "D:\a\_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 726, in _run_jobs_serially
  File "D:\a\_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 697, in _run_job
  File "D:\a\_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 333, in __call__
  File "D:\a\_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/command_operation.py", line 121, in handler
  File "D:\a\_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/command_modules/profile/custom.py", line 66, in get_access_token
  File "D:\a\_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/_profile.py", line 382, in get_raw_token
  File "D:\a\_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/_profile.py", line 592, in _create_credential
  File "D:\a\_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/auth/identity.py", line 225, in get_user_credential
  File "D:\a\_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/auth/msal_authentication.py", line 45, in __init__
  File "D:\a\_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\msal/application.py", line 1685, in __init__
  File "D:\a\_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\msal/application.py", line 533, in __init__
  File "D:\a\_work\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\msal/authority.py", line 124, in __init__
ValueError: Unable to get authority configuration for https://login.microsoftonline.com/adfs. Authority would typically be in a format of https://login.microsoftonline.com/your_tenant Also please double check your tenant name or GUID is correct.

@jiasli for awareness

According to the error message:

Unable to get authority configuration for https://login.microsoftonline.com/adfs. Authority would typically be in a format of https://login.microsoftonline.com/your_tenant Also please double check your tenant name or GUID is correct.
AADSTS90002: Tenant 'adfs' not found. Check to make sure you have the correct tenant ID and are signing into the correct cloud. Check with your subscription administrator, this may happen if there are no active subscriptions for the tenant.

It looks like your activeDirectory endpoint is set to https://login.microsoftonline.com/adfs, which is not correct as the domain is Azure Cloud. An ADFS endpoint usually looks like https://adfs.redmond.azurestack.corp.microsoft.com/adfs.
Could you share the result of az cloud show?


Persist the current tab selection in the menu

Hi
I was wondering if it makes sense to persist the selection of the current tab in the menu. What I mean is that if you have three tabs:
tab1
tab2
tab3

And you are in tab2 then when you open the menu using Ctrl+T then you should see
> tab2
tab1
tab3

then if you switch to tab1 you should see
> tab1
tab2
tab3

In that way the selected tab is always first and when changing tab you instinctively know you have to go down to select a different tab instead of remembering the name of the current tab and move somewhere else. This is mostly a "muscular memory" thing for me but I think it is a sensible behavior.
I don't want to impose my workflow as the "right thing" so I think this could be added as a configurable behavior. And I propose myself to implement this if this gets a green light.
PD: I tried to check if this is already implemented of if this is a duplicated issue but did not found anything in the issue tracker and documentation.
PD: An alternative would be to keep the order of the tabs in the menu but keep the current tab as the selected when entering. So if you are in tab2 and open the menu, you'd see:
tab1
> tab2
tab3

Oh that's great! I'm closing this as it will be redundant when those PR are merged :) Thank you!

Oh that's great! I'm closing this as it will be redundant when those PR are merged :) Thank you!


call expirationCallback automatically on cache.Close()

Is it possible to call expirationCallback on each remaining keys in the cache, when calling cache.Close()?
The purpose is to keep the overall expirationCallback behavior the same?

@ReneKroon
Thanks for your time.

Good idea, will have to check the feasibility first.

@LihuaWu can you test if this solves the issue for you? I added code to run all expiration callbacks on Close() including a test to verify this.

I will update the result soon, Thanks

@ReneKroon
I have looked at the unit test, and thanks for your quick feedback.
Would it be better for  cache.Close to wait all expirationCallback functions executed before it returns?
Currently, it will just start a new goroutine for each expirationCallback function to execute, and thus it is not possible to know when all expirationCallback calls will be finished.

I'd say it's a fair request, however during normal operation the expiration is async and this would imply that any items cleaned up just before calling cache.Close are still maybe executing the handler, while the synchronous callbacks are already done. This would create a race condition as it's not guaranteed that all callbacks have completed once cache.Close returns. I'd suggest using sync.WaitGroup inside the callback to keep track of callback's in progress. Would that work in your case?

@ReneKroon
I think you are right, and it makes sense to keep the callbacks in async.
Synchronous callbacks are only special cases for the cache.Close operation.
And keeping track of progress in the callback is a viable option for me.
Thanks!

@ReneKroon and I have no further concerns.

Ok, will release this officially then.


Order sequence issues when running "php artisan migrate"

Describe the bug
Following the instruction on github, the following errors occurred after run "php artisan migrate". It is
Migrating: 2021_10_07_083412_create_article_table
Illuminate\Database\QueryException
SQLSTATE[HY000]: General error: 3780 Referencing column 'topic_id' and referenced column 'id' in foreign key constraint 'idx_fk_topic_id' are incompatible. (SQL: alter table article add constraint idx_fk_topic_id foreign key (topic_id) references topic (id) on delete cascade on update cascade)
at vendor/laravel/framework/src/Illuminate/Database/Connection.php:703
699▕         // If an exception occurs when attempting to run a query, we'll format the error
700▕         // message to include the bindings with SQL, which will make this exception a
701▕         // lot more helpful to the developer instead of just the database's errors.
702▕         catch (Exception $e) {
➜ 703▕             throw new QueryException(
704▕                 $query, $this->prepareBindings($bindings), $e
705▕             );
706▕         }
707▕     }
  +12 vendor frames

13  Modules/V3/Database/Migrations/2021_10_07_083412_create_article_table.php:28
Illuminate\Support\Facades\Facade::__callStatic()
  +21 vendor frames

35  artisan:37
Illuminate\Foundation\Console\Kernel::handle()
To Reproduce
Steps to reproduce the behavior:

Following the instruction on github
Download 'https://raw.githubusercontent.com/SoliDry/api-generator/master/tests/functional/oas/openapi.yaml' and 'https://raw.githubusercontent.com/SoliDry/api-generator/master/tests/functional/oas/topic.yaml' to oas/ folder.
Run php artisan api:generate oas/openapi.yaml --migrations
Then run "php artisan migrate"

Expected behavior
There should be a way to define the correct sequence of foreign key reference or create the foreign key after all the tables were created.

Hi, sorry for delay, this error tells u that there are incompatible types in foreign key constraint:
SQLSTATE[HY000]: General error: 3780 Referencing column 'topic_id' and referenced column 'id' in foreign key constraint 'idx_fk_topic_id' are incompatible. (SQL: alter table article add constraint idx_fk_topic_id foreign key (topic_id) references topic (id) on delete cascade on update cascade)

Can u provide the particular generated migrations for tables: topic and article in your particular case?


Copilot chat response gets cut after one and a half line

Copilot Chat Extension Version: 0.16.1

VS Code Version: 1.90.0

OS Version: Details below
Edition	Windows 10 Enterprise
Version	22H2
Installed on	14/07/2021
OS build	19045.4780
Experience	Windows Feature Experience Pack 1000.19060.1000.0

Logs:
2024-08-21 17:52:26.163 [error] [default] Error sending telemetry FetchError: unable to get local issuer certificate
at fetch (c:\Users\AM99939.vscode\extensions\github.copilot-1.223.0\node_modules@adobe\helix-fetch\src\fetch\index.js:99:11)
at processTicksAndRejections (node:internal/process/task_queues:95:5)
at cachingFetch (c:\Users\AM99939.vscode\extensions\github.copilot-1.223.0\node_modules@adobe\helix-fetch\src\fetch\index.js:288:16)
at MO.fetch (c:\Users\AM99939.vscode\extensions\github.copilot-1.223.0\lib\src\network\helix.ts:93:22) {
type: 'system',
_name: 'FetchError',
code: 'UNABLE_TO_GET_ISSUER_CERT_LOCALLY',
errno: undefined,
erroredSysCall: undefined

Steps to Reproduce:

Ask any question in Copilot chat. See image above

Your copilot chat extension version is quite old. Please update your VS Code and copilot chat. You should be using VS Code 1.92.2 and Copilot Chat 0.18.2

I am going to get my VSCode and Copilot versions updated. Thank you for your help.


Only the client that sends a MOVE request gets the MOVE response to update the screen

Fixed by #34


refactor: rewrite back-end code to kotlin

Kotlin is a static type language, transpilable to javascript and gives me good development experience.
Restructure the current maze of back-end code and rewrite it in Kotlin.
TODO

[x] Implement MarkdownParser

[x] Check behavior of html parser
[x] Implement BasicParser(markdown -> ReactElement)
[x] Test behaviors
[x] Implement Codeblock Hilighting
[x] Apply Obsidian line breaking rule
[x] Apply Obsidian Link

[x] Fix initializing flows and reduce redundant file loadings
[x] Preparing image/canvas support
[x] Image file support

looks work fine!! let's gooooooooooooo!!!!!!!!!!


missing file picalc-pipeline-pvc.yaml

In section 8. Create a PipelineRun, the following command fails due to missing file picalc-pipeline-pvc.yaml.
kubectl apply -f tekton/pipeline-account.yaml
"error: the path "tekton/run/picalc-pipeline-pvc.yaml" does not exist".
Thank you
Lee

found it in beta-update