Floating windows

Thank you for making VS Code, many people's favorite code editor. You could make it even better by allowing windows to float so users can use their multiple screens for easier & quicker code comparisons. Currently we are limited to splitting code files within the same VS Code window / the same monitor.
I am aware that this feature has been requested before (in 2016) and moved to the backlog in 2018, but neither has it been implemented in the meantime nor can it still be found it in that same backlog.
The argument, that there is a work-around, namely simply opening a second VS Code instance, does not really apply, because this approach is not compatible with useful related tools like the multiWindows-highLight and highlight extensions.
And since Microsoft says themselves that floating windows is the most demanded feature request for VS Code I thought let's give it a new try 7 years (and many lesser feature implementations) later. Maybe this time a new developer team (or the same one with fresh energy) will make it work. Will you?
Because that would be so great for so many of us! 🙂

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Some multitest bindings for staking are missing such as BondedDenom

This causes deps.querier.query_bonded_denom() to fail in mulitest.

@shanev can you provide more details here, like a quick code sample to demo?
It can either be:

Building fully functional mocks for Staking (which is useful, but I would push to 0.16 not this release)
Export a few more missing types from multitest to allow a 3rd party to implement it (I would do this right away)

I know there was some other bug that someone couldn't build a multi-test mock for Distribution due to missing exports (it wasn't pub). We should check and address all those, maybe trying to build sample/failing mocks there in another module to show we have all types.

Building fully functional mocks for Staking (which is useful, but I would push to 0.16 not this release)

Some time ago I started tinkering around that.
https://github.com/CosmWasm/cw-plus/pull/782/files
It's kind of tricky to properly utilize tokens from Bank module storage, "temporary" I created own STAKING storage which acts as staked balances.

Okay, then we just verify all types are exported. And close this, leaving some "future issue" for proper multi-test staking support

Okay, then we just verify all types are exported. And close this, leaving some "future issue" for proper multi-test staking support

Regarding this, not exactly sure what needs to be done.
BondedDenom is published as part of StakingQuery in cosmwasm-std
https://github.com/CosmWasm/cosmwasm/blob/421f75cde11ff8842fd279ea3ec2cbcca65ef399/packages/std/src/query/staking.rs#L11-L34
along with the other staking queries.
The same with the staking messages
https://github.com/CosmWasm/cosmwasm/blob/421f75cde11ff8842fd279ea3ec2cbcca65ef399/packages/std/src/results/cosmos_msg.rs#L76-L90
What's required is implement the Module trait for staking in multitask, adding support for the staking messages and queries. But that's out of scope here. So, unless I'm missing something related to exporting these types, I would say, this is already done.

@ueco-jb has started work on better testing in https://github.com/CosmWasm/cw-plus/pull/782
I assume this is what is desired (when finished)?


Support Bridgeless

Library needs to support bridgeless since starting on 0.75 the default template will move to new arch + bridgeless. I need however to make time and follow up on the process of adjusting the lib.
Related https://github.com/facebook/react-native/issues/43204

Upvote & Fund

We're using Polar.sh so you can upvote and help fund this issue.
We receive the funding once the issue is completed & confirmed by you.
Thank you in advance for helping prioritize & fund our backlog.

Library now supports bridgeless via the interop layer. Once that is gone it will have to be migrated to a c++ turbo module, but that will be at least a year into the future.


Various Bug fixes

closes #261
closes #267
closes #216
closes #259
closes #266

Hey @rmisio when hovering a photo in Edit Listing could we change the cursor to cursor:move? That should make it more clear that they can be dragged.

@morebrownies sure, done.

👍

ok @jjeffryes I've fixed the issues you found (good catches).


fix: re-render of pages section

Description
Pages section in editor pane was getting rendered everytime when a page selection is made. This PR fixes that issue.
PR fixes following issue(s)
Fixes https://github.com/appsmithorg/appsmith/issues/30367
Type of change

Bug fix (non-breaking change which fixes an issue)

Testing

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration.
Delete anything that is not relevant

[ ] Manual
[ ] JUnit
[ ] Jest
[ ] Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] 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
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

Summary by CodeRabbit

New Features

Introduced a new component for page management in the editor.

Refactor

Simplified the rendering logic in the EditorPane.
Optimized performance by memoizing the LeftPane and IDE components.

Style

Updated component imports to match new file structure.

/build-deploy-preview skip-test=true

/ok-to-test<EMAIL_ADDRESS>

/ok-to-test<EMAIL_ADDRESS>

/build-deploy-preview skip-test=true

/ok-to-test<EMAIL_ADDRESS>


Range-ify std.path.globMatch()

Now accepts a forward range as input
Added unit tests to bring it to 100% coverage
Reduced gc allocations from N per invocation to 1. Can make it go to 0 with a pure way to malloc a buffer.

Can make it go to 0 with a pure way to malloc a buffer.

Please contribute to the pure malloc debate.
https://github.com/D-Programming-Language/druntime/pull/1183#issuecomment-77047592

All your rangify pulls ignore the fact that it was previously possible to call the functions using a static array of chars.

All your rangify pulls ignore the fact that it was previously possible to call the functions using a static array of chars. This might cause quite some unwanted fallout.

Passing static arrays is problematic in general because static arrays get passed by value, and I've found at least a couple cases in Phobos where this was happening causing excessive (and unrecognized) stack consumption and performance slowdowns.

Passing static arrays is problematic in general because static arrays get passed by value

Sure, but previously the static arrays were implicitly converted to dynamic arrays, because the signature was C[] or so.

This could possibly even be implemented with a forward range as pattern, different topic though.
But allowing random access ranges as pattern and using chain to avoid the allocation makes sense.

Using chain is a great idea, and I tried it. Unfortunately, it results in a recursive expansion of globMatch() as it builds chain on top of chain on top of chain ...

Auto-merge toggled on


Feature Request

Description of the new feature/enhancement
If the path can be moved to header and just use the binary name as Tab Name (With Tab Renaming for distinction). I feel this would make the terminal much more useful.
Proposed technical implementation details (optional)
I would suggest to make the Tab Interface two Layered stacked vertically, basically two rows. how it is implemented as of now is, the whole tab interface is on the header itself, i would suggest if header area(First Row) displays the path of the active tab and in 2nd row tabs are there displaying just the binary name would work better. for distinguishing between two same tabs the user should have ability to rename the tab.
If in future split screen is coming, such an interface would make much more sense, in my opinion.

Maybe edit this and give the feature request a name other than 'feature request'? 😉

You can set the tab name to whatever you'd like using shell magic (documentation to be written)
We're trying to track the UI nits in #1375; please pile in there!


Feature requst: New yafti mode 'conditional'

The current available yafti modes are:
class YaftiRunModes(str, Enum):
    changed = "run-on-change"
    ignore = "run-once"
    disable = "disabled"

I'm proposing a new mode called 'conditional' that takes a conditional as input so that yafti can run based on information about the system
The syntax could look something like this:
title: Welcome to uBlue
properties:
  mode: "conditional"
  condition:
    run: flatpak remotes --columns=name | grep fedora

I have some ideas. I would like to better understand the feature you're looking for.

@abanna I can reach out on discord later. in short, it would be a conditional that determines whether yafti runs

a conditional provided by the writer of the config

Sounds easy enough.  I added a setting to disable it in dconf. But it seem like you would like it to be predefined in the os provided yaml?

disable what in dconf?
I want it to be like run-on-change, but instead it's run-on-x where x is any conditional

I think am tracking.  To save time lets just plan to get on a discord at some point to has out the details.


Create _

I'm not sure whether you intended to submit this pull request on your own fork of the project rather than in the original project? It appears to be adding an empty file called _ and a copy of a third party's API output to the registration prefixes folder. I'm going to close this pull request.


How do I render KaTex equations?

As demonstrated here: https://ui.toast.com/weekly-pick/en_20210617
How do I add the ability to render KaTex like that?

You can see https://github.com/ats1999/md-byte/blob/1310be8356937aab9c764097bfe90f3bf179239a/mde10/src/customHTMLRenderer.js#L118

@hsb-tonmoy
This document is helpful to you

@hsb-tonmoy This document is helpful to you

You can see https://github.com/ats1999/md-byte/blob/1310be8356937aab9c764097bfe90f3bf179239a/mde10/src/customHTMLRenderer.js#L118

Thanks a lot guys. It was really helpful. I'm using the custom rendered that @ats1999 referenced but I'm getting this broken output:

@hsb-tonmoy import the CSS properly
import 'katex/dist/katex.min.css'

In CSS file
@import url("katex/dist/katex.min.css");

You can opt anyonne of them

You can opt anyone of them
Let me know if this resolved your issue?

This resolved the issue. Thank you very much.


Can't seem to find docker image with a exact tagname

I'm trying to find the docker image in ghcr of a repo with the exact docker image tag, but it can't seem to find it

for this example, the tagname will be 620.
so the docker image name would be ghcr.io/test-user/test-repo:620
I checked it manually whether the docker image exists, and it does. So I'm not sure why it can't seem to find it
---

name: Cleanup docker image on PR merge
on:
  pull_request:
    # types:
    #   - closed
    #   - opened
    branches-ignore:
      - master

env:
  FORCE_COLOR: true
  CONTAINER_REGISTRY: ghcr.io
  IMAGE_NAME: test-user/test-repo
  PACKAGE_NAME: test-repo

jobs:
  remove-dev-image:
    # if: github.event.pull_request.merged == true
    runs-on: ubuntu-latest
    env:
      HEAD_REF: ${{ github.event.pull_request.head.ref }}
    steps:
      - run: echo ${{ github.base_ref }}

      - uses<EMAIL_ADDRESS>        id: version
        with:
          # version: "${{ env.HEAD_REF }}"
          version: "620"

      - name: Remove pull request image from container registry
        uses: actions/delete-package-versions@v4
        if: ${{ steps.version.outputs.ids != '' }}
        with:
          package-name: ${{ env.PACKAGE_NAME }}
          package-type: 'container'
          package-version-ids: "${{ steps.version.outputs.ids }}"

I found a workaround
curl -L \
-H "Accept: application/vnd.github+json" \
-H "Authorization: Bearer ${{ secrets.GITHUB_TOKEN }}" \
-H "X-GitHub-Api-Version: 2022-11-28" \
https://api.github.com/users/${{ github.repository_owner }}/packages/container/${{ env.PACKAGE_NAME }}/versions >> containerMeta.json ;
echo "VERSION_ID=$(jq -r '.[] | select(.metadata.container.tags[] == "${{ env.HEAD_REF }}").id' containerMeta.json)" >> "$GITHUB_ENV" ;


Tutorial Video

A few changes in the portal for resource locations need to be updated in the video

Batch account is no longer showing in the Storage account Blades
The Create batch account page is different (more modern)

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

ID: 0d35e3d8-a644-dfb6-a16a-93679276c079
Version Independent ID: 8c6bcc7c-4428-be5b-2221-0c0302cdd281
Content: Tutorial - Run a parallel workload using the .NET API - Azure Batch
Content Source: articles/batch/tutorial-parallel-dotnet.md
Service: batch
GitHub Login: @prkannap
Microsoft Alias: prkannap

@geraldwolff Thank you for the feedback!  For clarification, which video are you referring to?  I'm not seeing anything in the documentation this issue was opened against:
https://docs.microsoft.com/en-us/azure/batch/tutorial-parallel-dotnet

The video walked thru portal blades where the options are now relocated to other areas.
The video is out of date with the actual portal navigation.
https://www.youtube.com/watch?v=jvX0IeFMmCE https://www.youtube.com/watch?v=jvX0IeFMmCE&t=683s &t=683s
From: kobulloc-MSFT @.>
Sent: Monday, February 14, 2022 2:04 PM
To: MicrosoftDocs/azure-docs @.>
Cc: geraldwolff @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Tutorial Video (Issue #88050)
@geraldwolff https://github.com/geraldwolff  Thank you for the feedback! For clarification, which video are you referring to? I'm not seeing anything in the documentation this issue was opened against:
https://docs.microsoft.com/en-us/azure/batch/tutorial-parallel-dotnet
—
Reply to this email directly, view it on GitHub https://github.com/MicrosoftDocs/azure-docs/issues/88050#issuecomment-1039614721 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AGLB7DMBV5QPCAGCRLGDTNTU3F35BANCNFSM5OMUENDA .
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675  or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .
You are receiving this because you were mentioned.  https://github.com/notifications/beacon/AGLB7DKT66HXWYJMDL6G4A3U3F35BA5CNFSM5OMUENDKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOHX3UGAI.gif Message ID: @.*** @.***> >

I apologize,
I did not notice that Microsoft did not publish this to Youtube.
☹
Jw
From: kobulloc-MSFT @.>
Sent: Monday, February 14, 2022 2:04 PM
To: MicrosoftDocs/azure-docs @.>
Cc: geraldwolff @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Tutorial Video (Issue #88050)
@geraldwolff https://github.com/geraldwolff  Thank you for the feedback! For clarification, which video are you referring to? I'm not seeing anything in the documentation this issue was opened against:
https://docs.microsoft.com/en-us/azure/batch/tutorial-parallel-dotnet
—
Reply to this email directly, view it on GitHub https://github.com/MicrosoftDocs/azure-docs/issues/88050#issuecomment-1039614721 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AGLB7DMBV5QPCAGCRLGDTNTU3F35BANCNFSM5OMUENDA .
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675  or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .
You are receiving this because you were mentioned.  https://github.com/notifications/beacon/AGLB7DKT66HXWYJMDL6G4A3U3F35BA5CNFSM5OMUENDKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOHX3UGAI.gif Message ID: @.*** @.***> >

@geraldwolff No worries!  If you have any questions about the documentation, please don't hesitate to reach out to us.  We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.

Thank you
From: kobulloc-MSFT @.>
Sent: Tuesday, February 15, 2022 12:15 PM
To: MicrosoftDocs/azure-docs @.>
Cc: geraldwolff @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Tutorial Video (Issue #88050)
@geraldwolff https://github.com/geraldwolff  No worries! If you have any questions about the documentation, please don't hesitate to reach out to us. We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.
—
Reply to this email directly, view it on GitHub https://github.com/MicrosoftDocs/azure-docs/issues/88050#issuecomment-1040747316 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AGLB7DJKU7X6VOAAWU7IYNTU3KX3LANCNFSM5OMUENDA .
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675  or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .
You are receiving this because you were mentioned.  https://github.com/notifications/beacon/AGLB7DL4DM6UKO35ZZXPV3DU3KX3LA5CNFSM5OMUENDKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOHYEIWNA.gif Message ID: @.*** @.***> >


Dao Core v1. Staking notifications

TX notifications progress
https://zpl.io/anBWJGv

For the estimated transaction time please use the weighted average of the block emission time considering ~10 samples lookback window (when available). The older a block, the less weight it should have.
Whenever a transaction is accomplished faster than the estimated time - the indicator should accelerate to 100%.
Whenever a transaction takes longer than the estimated time - the indicator should start behaving exponentially from 90-95% (to your choice).
By click on Show details button, open transaction list and highlight transaction from progress popup for a few seconds
https://zpl.io/VxpJre3

was implemented in https://github.com/BeamMW/beam-ui/issues/680
without progress bar


make doesn't initialize recursive submodules

If the third_party/misoc submodule is initialized without a "--recursive" flag then you get the following error;
$ make gateware
cd third_party/misoc && python3 make.py -X ../.. -t opsis_hdmi2usb -Ot firmware_filename ../../firmware/lm32/firmware.bin -Op programmer urjtag  build-headers
                __  ___  _   ____     _____
               /  |/  / (_) / __/__  / ___/
              / /|_/ / / / _\ \/ _ \/ /__
             /_/  /_/ /_/ /___/\___/\___/

a high performance and small footprint SoC based on Migen

====== Building for: ======
Platform:  opsis
Target:    opsis_hdmi2usb
Subtarget: HDMI2USBSoC
CPU type:  lm32
===========================
make -C firmware/lm32 all
make[1]: Entering directory `/usr/local/google/home/tansell/foss/timvideos/hdmi2usb/HDMI2USB-misoc-firmware/firmware/lm32'
 CC       main.o
make -C ../../third_party/misoc/software/libcompiler-rt
make[2]: Entering directory `/usr/local/google/home/tansell/foss/timvideos/hdmi2usb/HDMI2USB-misoc-firmware/third_party/misoc/software/libcompiler-rt'
make[2]: *** No rule to make target `divsi3.o', needed by `libcompiler-rt.a'.  Stop.
make[2]: Leaving directory `/usr/local/google/home/tansell/foss/timvideos/hdmi2usb/HDMI2USB-misoc-firmware/third_party/misoc/software/libcompiler-rt'
make[1]: *** [libs] Error 2
make[1]: Leaving directory `/usr/local/google/home/tansell/foss/timvideos/hdmi2usb/HDMI2USB-misoc-firmware/firmware/lm32'
make: *** [firmware-lm32] Error 2
$ git status

Quick solution; make the gateware depend on the following .git files
./third_party/misoc/.git
./third_party/misoc/software/unwinder/.git
./third_party/misoc/software/compiler-rt/.git
./third_party/misoc/extcores/mor1kx/submodule/.git
./third_party/misoc/extcores/lm32/submodule/.git

Better solution: make the gateware depend automagically depend on any submodule recursively. It should work to depend on ./third_party/misoc/.gitmodules and pull the information out of that.
This is a low priority because the makefile should initialize the submodules with --recursive and so does the bootstrap.sh script.

@CarlFK -- You've seen this error right?

BTW The deliberately get you into this state is to do the following;
git submodule deinit third_party/misoc
git submodule init third_party/misoc
git submodule update third_party/misoc
make gateware

The output looks like;
$ git submodule deinit third_party/misoc
Cleared directory 'third_party/misoc'
Submodule 'third_party/misoc' (https://github.com/m-labs/misoc.git) unregistered for path 'third_party/misoc'
tansell@tansell-z620-l2:~/foss/timvideos/hdmi2usb/HDMI2USB-misoc-firmware$ git submodule init third_party/misoc
Submodule 'third_party/misoc' (https://github.com/m-labs/misoc.git) registered for path 'third_party/misoc'
tansell@tansell-z620-l2:~/foss/timvideos/hdmi2usb/HDMI2USB-misoc-firmware$ git submodule update third_party/misoc
Submodule path 'third_party/misoc': checked out '8e860e3aba736619e5c073196372837f3bc597c4'
tansell@tansell-z620-l2:~/foss/timvideos/hdmi2usb/HDMI2USB-misoc-firmware$ make gateware
cd third_party/misoc && python3 make.py -X ../.. -t opsis_hdmi2usb -Ot firmware_filename ../../firmware/lm32/firmware.bin -Op programmer urjtag  build-headers
                __  ___  _   ____     _____
               /  |/  / (_) / __/__  / ___/
              / /|_/ / / / _\ \/ _ \/ /__
             /_/  /_/ /_/ /___/\___/\___/

a high performance and small footprint SoC based on Migen

====== Building for: ======
....
make -C ../../third_party/misoc/software/libcompiler-rt
make[2]: Entering directory `/usr/local/google/home/tansell/foss/timvideos/hdmi2usb/HDMI2USB-misoc-firmware/third_party/misoc/software/libcompiler-rt'
make[2]: *** No rule to make target `divsi3.o', needed by `libcompiler-rt.a'.  Stop.
make[2]: Leaving directory `/usr/local/google/home/tansell/foss/timvideos/hdmi2usb/HDMI2USB-misoc-firmware/third_party/misoc/software/libcompiler-rt'
make[1]: *** [libs] Error 2
make[1]: Leaving directory `/usr/local/google/home/tansell/foss/timvideos/hdmi2usb/HDMI2USB-misoc-firmware/firmware/lm32'
make: *** [firmware-lm32] Error 2


cache rendered manifest for helm

As a solution to introduce caching for rendered Helm manifests, the rendered content is placed in a file named manifest.yaml, inside a directory named manifest, created in the cached chart directory.
tmp/path/to/chart/manifest/manifest.yaml
Two points to consider:

The removal of chart caching will require a bigger refactoring
The caching is done on disk and not in memory

/hold

As a solution to introduce caching for rendered Helm manifests, the rendered content is placed in a file named manifest.yaml, inside a directory named manifest, created in the cached chart directory.
tmp/path/to/chart/manifest/manifest.yaml

/unhold


Ipv6 changes broke ssh

It looks like the new changes to add ipv6 broke functionality to get ip's from fuzzy matching.
The error I get is:
tugboat ssh platform
Droplet fuzzy name provided. Finding droplet ID...Multiple droplets found.

1) ui-platform (123456)
2) ui-platform-staging (123457)

Please choose a droplet: ["0", "1"] 1
Executing SSH ui-platform-staging...
ssh: Could not resolve hostname : Name or service not known

I made a hacky change (https://github.com/mtbottle/tugboat/commit/0caa7f3772806643d09557dda2dcc463da4d5960) to fix it for my usecase. However it does not deal with someone wanting to ssh onto a machine with ipv6.

@mtbottle That's not that hacky IMO, I'm going to PR that to fix the bug.
I'll need to add a new flag to allow the choice to SSH to the IP6 address.


Problem with entries coming from a seeder

Everything works fine with the CP where you can add/edit entries. In the front index and show works fine. Single taxonomy pages are also working as normal.
Than you insert 1000 entries through a seeder.
 $news = new Entry();
 $news->site = 'default';
 $news->published = 1;
 $news->status = 'published';
 $news->slug = $slug;
 $news->uri = 'news/' . $slug;
 $news->collection = 'news';
 $news->data = $data;
 $news->created_at = \Carbon\Carbon::now();
 $news->save();

$data and the other variables are working just fine and you end up with the following code in the DB:
id: 10 (incremental), site: 'default', origin_id: null, published: 1, status: 'published', slug: 'porro-eveniet-ipsam-tempora-et', uri: 'news/porro-eveniet-ipsam-tempora-et', date: null, collection: 'news', data: {"tags": ["clothes", "games", "space"], "title": "Porro eveniet ipsam tempora et.", "author": 1, "content": "Itaque quos qui sint..", "gallery": [], "pub_date": "2019-03-09", "updated_at":<PHONE_NUMBER>, "updated_by": 1, "featured_image": null, "news_categories": ["nature", "fashion"]}, created_at: '2020-12-14 09:41:53'
Absolutely the same as the entries that you create from the CP. You can see/edit/delete the seeder ones normally. But in the front the Show page gives Not Found. In the taxonomy pages (/news/tags/nature) {{ tags }} lists only the CP created entries. The problem is fixed if you edit and save a seeder entry in the CP.
If you follow the code chain of the Not Found error you get to
Statamic/cms/sc/Stache/Repositories/EntryRepository.php findByUri() function
If you dd() the following for the seeder entry you get NULL and for a normal CP entry - Statamic\Eloquent\Entries\Entry:
 $entry = $this->query()
                ->where('uri', $uri)
                ->where('site', $site)
                ->first();

I have the users in the database and I use incremented ID for entries, following the instructions in the Eloquent package.

We fill in the DB manually to (via a db import) and don't have issues so something else must be going on...

Difference I see is that I don't see the blueprint in there, wonder if that matters. I added that to this driver in a PR, maybe you need to update and add that column @lutory ?

Is this still an issue for you @lutory?

Closing due to lack of activity. If its still an issue, please re-open.


Adding a question to leetcode section

Palindrome Number
Given an integer x, return 'true' if x is a  palindrome, and 'false' otherwise.

kindly assign issue under hacktoberfest.

Done......


Error in installation instructions for Gunicorn service unit configuration

The Gunicorn service unit configuration in INSTALL.md is missing a "[Unit]" line at the start.

Fix merged.


Add plane gauges in 3D

[ ] check that it doesn't mess up BC's
[ ] add flux integral
[ ] probably need  to output this in XDMF or something matplotlib is happy with

Old post, probes are working, closing this


chore(hacs): revert required homeassistant version

PR Description
2022.5 was the previously required version. IDK why it was updated to 2024.4. Is there something that only works on 2024.4?
Reason & Detail
Releted issue
fix #24

Perhaps this is related to this PR: https://github.com/georgezhao2010/midea_ac_lan/pull/420.
I think 2023.1 should be the minimum version.

My mistake, I incorrectly assumed that it's safe to bump to a newer version. Apparently some use older HA.


Update http4s-blaze-client, ... to 0.21.6

Updates

org.http4s:http4s-blaze-client
org.http4s:http4s-blaze-server
org.http4s:http4s-circe
org.http4s:http4s-dsl

from 0.21.4 to 0.21.6.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

labels: library-update, semver-patch

Codecov Report

Merging #706 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #706   +/-   ##
=======================================
  Coverage   81.20%   81.20%
=======================================
  Files          78       78
  Lines        5197     5197
  Branches      139      139
=======================================
  Hits         4220     4220
  Misses        977      977

Continue to review full report at Codecov.

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


incorrect errno handling in uv__getpwuid_r

Noticed that https://github.com/libuv/libuv/blob/95f88f47f28b4686a6959377c8d2c31daa40bd64/src/unix/core.c#L1197 should be UV__ERR while browsing the code
Also, shouldn't we be handling EINTR there?

PR: https://github.com/libuv/libuv/pull/3177


Stablize testsuite

@aslakknutsen snapshot fabric8-wit image is available for testing.  docker pull docker.io/fabric8/fabric8-wit:SNAPSHOT-PR-1503-1

@aslakknutsen snapshot fabric8-wit image is available for testing.  docker pull docker.io/fabric8/fabric8-wit:SNAPSHOT-PR-1503-2

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@8e871fb). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1503   +/-   ##
=========================================
  Coverage          ?   58.75%
=========================================
  Files             ?      121
  Lines             ?    14282
  Branches          ?        0
=========================================
  Hits              ?     8392
  Misses            ?     5231
  Partials          ?      659

Continue to review full report at Codecov.

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


[Clean up] Remove the deprecated RecordTransformer

They were introduced when adding the segment processing framework, then we modified the framework to use the general RecordTransformer which is used for all ingestion.

Codecov Report

Merging #10084 (a3147b8) into master (fa55267) will decrease coverage by 1.78%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master   #10084      +/-   ##
============================================
- Coverage     70.34%   68.56%   -1.79%
+ Complexity     5726     5718       -8
============================================
  Files          1992     1989       -3
  Lines        107716   107695      -21
  Branches      16384    16384
============================================
- Hits          75775    73839    -1936
- Misses        26638    28615    +1977
+ Partials       5303     5241      -62

Flag
Coverage Δ

integration1
24.56% <ø> (+0.02%)
:arrow_up:

integration2
?

unittests1
68.03% <ø> (+0.02%)
:arrow_up:

unittests2
13.60% <ø> (-0.03%)
:arrow_down:

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

Impacted Files
Coverage Δ

...g/apache/pinot/server/api/resources/ErrorInfo.java
0.00% <0.00%> (-100.00%)
:arrow_down:

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

...t/core/plan/StreamingInstanceResponsePlanNode.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ore/operator/streaming/StreamingResponseUtils.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ager/realtime/PeerSchemeSplitSegmentCommitter.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...he/pinot/common/utils/grpc/GrpcRequestBuilder.java
0.00% <0.00%> (-90.91%)
:arrow_down:

...ator/streaming/StreamingSelectionOnlyOperator.java
0.00% <0.00%> (-90.00%)
:arrow_down:

...he/pinot/core/plan/StreamingSelectionPlanNode.java
0.00% <0.00%> (-88.89%)
:arrow_down:

...lugin/stream/kafka20/KafkaStreamLevelConsumer.java
0.00% <0.00%> (-88.34%)
:arrow_down:

...che/pinot/common/auth/StaticTokenAuthProvider.java
0.00% <0.00%> (-87.50%)
:arrow_down:

... and 157 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Rspec: Test tag functionality

Make sure that when the user clicks on a tag, only resources tagged with that specific tag are shown.
Using RSpec for testing.

Merged to master on 11/2/2018, PR https://github.com/shescoding/shescoding-dot-org/pull/159


New Commands to Handle Subscriptions

Get-RsSubscription
Set-RsSubscription
Remove-RsSubscription
Also the Pester tests for each one

@ClaudioESSilva,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@ClaudioESSilva, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, Microsoft Pull Request Bot

Just to remind that I have commented the describe block from RsEncryptionKey.Tests.ps1 per @jtarquino suggestion.

I think all of your suggestions has been addressed. Can you please give another look?


track.ReadRTP blocks forever when remote stops sending packets

Your environment.

Version: 2.1.4
Browser: chrome 76.0.3809.100 (Official Build) (64-bit) / Firefox 69.0

What did you do?
Have a standard audio channel packet handling loop which is basically:
for {
  rtpPacket, err := rxtrack.ReadRTP()
  ... do stuff with rtpPacket
}

Started streaming on audio channel with Chrome. Closed chrome tab. Server now appears to be blocked forever in track.ReadRTP, which is a resource leak of the goroutine.
What did you expect?
Some kind of timeout, or other way to not block? A potentially malicious actor could DDOS the server by opening multiple tracks and not sending any RTP packets.  Must the remote side be trusted not to do this?
What happened?
Goroutine calling readRTP appears to block forever.

Hey @chrbsg, thanks for filing this issue I love feedback from users. Currently this is working as designed (this isn't to say we can't make it better)
ReadRTP and Read will block until the PeerConnection is closed. I wouldn't be against adding a timeout, we will need to think about the API/make sure we are consistent with putting it everywhere!
The remote can't send as many tracks as it wants though, it can only send as many tracks as the Transceivers you added. If you were using PlanB it would be possible to DoS someone with unlimited tracks though! But if you are using UnifiedPlan you should be fine.
There might not be further action on this, but in that case we should revise documentation/communicate to the user better about this. thanks!

Thanks for the quick reply. I didn't know that peerconnection.Close would unblock ReadRTP, but it does and it works.  I didn't see peerconnection.Close being used in any of the pion examples so I wasn't calling it at all - is it necessary/expected to call this manually, or is the garbage collector eventually going to do the equivalent? I'm now calling it when ICE state changes to disconnected or failed.

@chrbsg
Correct we don't ever automatically tear down the PeerConnection.
We could do that in the future though! That would be a breaking change so something we should do for v3. I believe it would be more confusing to people however.
If you are interested I would love a PR that updates the examples to does a Close on ICEConnectionStateDisconnected and explains why. That could be really helpful for people in the future!


Self installation of requirements

Prepare the self installation of requirements. Equivalent to Ruby bundler for Python.

requirements.txt created. command to install is
pip install -r /path/to/requirements.txt


Sunburst custom visual

Sunburst does not have an option to display the data labels. It becomes difficult to figure out the categories and subcategories that are displayed in the chart.
The other option could be adding a legend for the inner circle, that might help a bit

@harpreet552 Thanks. We have a plan to improve Sunburst and we will consider your suggestions.


Update swagger-ui-dist to 4.4.1

Updates org.webjars.npm:swagger-ui-dist from 4.3.0 to 4.4.1.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.webjars.npm", artifactId = "swagger-ui-dist" } ]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #724.


Added template for the _meg.json file

I used a different approach to commenting the different fields, not sure if it helps with the legibility.

This looks great!!! Thank you

@all-contributors please add @dorahermes  for review

@all-contributors please add @cofficer for code content ideas


EthicalAd: keeps showing on 404 pages

The ad is still showing at https://sphinx-rtd-theme.readthedocs.io/en/notfound/ but it shouldn't. I think it's because we are not calling super() on isEnabled(config) method at:
https://github.com/readthedocs/addons/blob/2c19271f31c319d6ce2d1e49ef92c58e7b7b40b1/src/ethicalads.js#L321-L325

I'm not able to reproduce the issue in the link from the description. However, this custom 404 page keeps showing the ad https://docs.godotengine.org/en/latest/page-not-found/
This is somehow related with another one where we need to define what to do on custom 404 pages:

https://github.com/readthedocs/addons/issues/385


Remove checks for unexpected reference depth

Remove checks for unexpected reference depth
Add test case for reference depth checks

Closes #1348.

Requires we actually inspect what's at that path and reference that

@jamietanna Any news on this? I'm still encountering the error in question.

@jhawrefvem please can you raise an issue (https://github.com/oapi-codegen/oapi-codegen/blob/main/CONTRIBUTING.md#raising-a-bug) for this?
This may now be closed by https://github.com/oapi-codegen/oapi-codegen/pull/1689


Getting more done in GitHub with ZenHub

Hola! @hollomancer has created a ZenHub account for the OperationCode organization. ZenHub is the only project management tool integrated natively in GitHub – created specifically for fast-moving, software-driven teams.

How do I use ZenHub?
To get set up with ZenHub, all you have to do is download the browser extension and log in with your GitHub account. Once you do, you’ll get access to ZenHub’s complete feature-set immediately.
What can ZenHub do?
ZenHub adds a series of enhancements directly inside the GitHub UI:

Real-time, customizable task boards for GitHub issues;
Multi-Repository burndown charts, estimates, and velocity tracking based on GitHub Milestones;
Personal to-do lists and task prioritization;
Time-saving shortcuts – like a quick repo switcher, a “Move issue” button, and much more.

Add ZenHub to GitHub
Still curious? See more ZenHub features or read user reviews. This issue was written by your friendly ZenHub bot, posted by request from @hollomancer.

@davidmolina @rickr, check it out...


Adding Switch to control if ALL keys are  tried when validating signature

Fixes #1128

Currently we try all keys to validate signature by default if a Issuer key is not resolved to match token's 'kid' or 'x5t'
Adding TryAllIssuerSigningKeys switch to control if all keys need to be tried when validating signature.

All test runs are successful.

@RojaEnnam can you add a simple test?

@RojaEnnam, @brentschmaltz - Sorry, I'm a bit late here. Would we need the same behavior for Saml token handlers?

@GeoK yes we should have, we should open another issue.


C205529 - fix the functional test

Description

Because @ciocanu have fixed an issue of xdm.device parameters being duplicated this have broken the functional test.
device.screenWidth or device.screenHeight was available in VEC by mistake.
Related Issue

Motivation and Context

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Improvement (non-breaking change which does not add functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have signed the Adobe Open Source CLA or I'm an Adobe employee.
[ ] I have made any necessary test changes and all tests pass.
[ ] I have run the Sandbox successfully.

Thanks @ninaceban!


[Feature]: Option to override HuggingFace's configurations

🚀 The feature, motivation and pitch
The configuration files on HuggingFace may have missing information (e.g. #2051) or contain bugs (e.g. #4008). In such cases, it may be necessary to provide/override the configuration files to enable the model to be loaded correctly. However, apart from chat templates, there is currently no method of doing so; we have to update the source HuggingFace repository directly. It may take time for the authors of those repositories to respond, especially if they are unofficial ones which are not as well-maintained.
It would be great if we could provide our own config.json, tokenizer_config.json, etc., through the vLLM CLI to apply patches as necessary.
Related work
#1756 lets us specify alternative chat templates or provide a chat template when it is missing from tokenizer_config.json. However, it currently only applies to the OpenAI API-compatible server. #5049 will add chat method to the main LLM entrypoint, but does not provide a built-in way to load the chat template automatically like in #1756.
Some vLLM models have already hardcoded patches to HuggingFace config.json; these can be found under vllm/transformers_utils/configs.

By default the LLM model is downloaded from Huggingface/ModelScope. Is there a way we can load model from local filepath or private repository or s3 object storage? How to get models from local storage path (supported models as in vLLM ) while we try to deploy it in local environment ?

By default the LLM model is downloaded from Huggingface/ModelScope. Is there a way we can load model from local filepath or private repository or s3 object storage? How to get models from local storage path (supported models as in vLLM ) while we try to deploy it in local environment ?

Actually, this is already supported - just pass a filepath to --model.

Actually, this is already supported - just pass a filepath to --model.
I am trying to deploy model using Kserve with vLLM with inference service as below: The pvc filepath has the required model which can be accessible with other running pods. I am getting similar error if I try to use s3 filepath as well. Both the paths contain the model and config files as expected.

apiVersion: serving.kserve.io/v1beta1
kind: InferenceService
metadata:
name: opt-125m-vllm
spec:
predictor:
containers:
- args:
- --port
- "8080"
- --model
- "pvc://kubeflow-shared-pvc/llm-mlflow/opt-125m"
command:
- python3
- -m
- vllm.entrypoints.api_server
env:
- name: STORAGE_URI
value: "pvc://kubeflow-shared-pvc/llm-mlflow/opt-125m"
- name: PYTORCH_CUDA_ALLOC_CONF
value: "max_split_size_mb:2048"
image: kserve/vllmserver:latest
name: kserve-container
resources:
limits:
cpu: "4"
memory: 8Gi
nvidia.com/gpu: "1"
requests:
cpu: "1"
memory: 8Gi
nvidia.com/gpu: "1"
But its giving me the below error:
Traceback (most recent call last):
File "/usr/lib/python3.8/runpy.py", line 194, in _run_module_as_main
return _run_code(code, main_globals, None,
File "/usr/lib/python3.8/runpy.py", line 87, in _run_code
exec(code, run_globals)
File "/usr/local/lib/python3.8/dist-packages/vllm/entrypoints/api_server.py", line 78, in
engine = AsyncLLMEngine.from_engine_args(engine_args)
File "/usr/local/lib/python3.8/dist-packages/vllm/engine/async_llm_engine.py", line 226, in from_engine_args
engine_configs = engine_args.create_engine_configs()
File "/usr/local/lib/python3.8/dist-packages/vllm/engine/arg_utils.py", line 147, in create_engine_configs
model_config = ModelConfig(self.model, self.tokenizer,
File "/usr/local/lib/python3.8/dist-packages/vllm/config.py", line 57, in init
self.hf_config = get_config(model, trust_remote_code)
File "/usr/local/lib/python3.8/dist-packages/vllm/transformers_utils/config.py", line 17, in get_config
config = AutoConfig.from_pretrained(
File "/usr/local/lib/python3.8/dist-packages/transformers/models/auto/configuration_auto.py", line 1007, in from_pretrained
config_dict, unused_kwargs = PretrainedConfig.get_config_dict(pretrained_model_name_or_path, **kwargs)
File "/usr/local/lib/python3.8/dist-packages/transformers/configuration_utils.py", line 620, in get_config_dict
config_dict, kwargs = cls._get_config_dict(pretrained_model_name_or_path, **kwargs)
File "/usr/local/lib/python3.8/dist-packages/transformers/configuration_utils.py", line 696, in _get_config_dict
raise EnvironmentError(
OSError: Can't load the configuration of 'pvc://kubeflow-shared-pvc/llm-mlflow/opt-125m'. If you were trying to load it from 'https://huggingface.co/models', make sure you don't have a local directory with the same name. Otherwise, make sure 'pvc://kubeflow-shared-pvc/llm-mlflow/opt-125m' is the correct path to a directory containing a config.json file

Oh, I missed the part where you are using object storage. I only meant that local filepaths are supported.

Oh, I missed the part where you are using object storage. I only meant that local filepaths are supported.

So it means that it doesn't support persistent storage volume path of object storage. It can only support local filepaths

Oh, I missed the part where you are using object storage. I only meant that local filepaths are supported.

So it means that it doesn't support persistent storage volume path of object storage. It can only support local filepaths

Yes, that is true. I think supporting non-local filepaths would warrant its own PR/issue.

Hi @DarkLight1337 ! Do you have a solution for this?

You can take a look at the code under vllm.transformers_utils and figure out how to pass user configs to override the configs loaded from HF there.

thanks for the callout re #2547 @DarkLight1337  - it seems like maybe being able to override individual fields in config.json through engine/CLI args would be a good approach? if so, I think llama.cpp has a good reference implementation; they allow doing this to override GGUF file key / value pairs with a custom value


Feature Request: Boot netopeer2-server with default timeout parameters

This is an extension from my other issue, https://github.com/CESNET/netopeer2/issues/784#issuecomment-754858322
For the next release of netopeer2-server, could a new option be added to allow for custom default rpc_timeout and possible sysrepo_timeout parameters? Currently, it is only possible to have a customer timeout parameter during compilation of netopeer2 through cmake. I.e., cmake -DSYSREPO_TIMEOUT=119 -DRPC_TIMEOUT=120 ..
For reference, here is the current help page for netopeer2-server:
$ netopeer2-server --help
netopeer2-server: invalid option -- '-'
Usage: netopeer2-server [-dhV] [-U (path)] [-m mode] [-u uid] [-g gid] [-v level] [-c category]
 -d        debug mode (do not daemonize and print verbose messages to stderr instead of syslog)
 -h        display help
 -V        show program version
 -U (path) listen on a local UNIX socket (specific path, default is "/var/run/netopeer2-server.sock")
 -m mode   set mode for the listening UNIX socket
 -u uid    set UID/user for the listening UNIX socket
 -g gid    set GID/group for the listening UNIX socket
 -v level  verbose output level:
               0 - errors
               1 - errors and warnings
               2 - errors, warnings, and verbose messages
 -c category[,category]*
           verbose debug level, print only these debug message categories
           categories: DICT, YANG, YIN, XPATH, DIFF, MSG, LN2DBG, SSH, SYSREPO

Perhaps we could add: -t timeout set default rpc_timeout and sysrepo_timeout. Default is 15 seconds (I believe?)
I'm not sure if sysrepo_timeout and rpc_timeout are correlated in the background in someway, but the end-goal is that the end-user can control this timeout value at server start up. Of course, the user can override the value in netopeer2-cli by passing in a custom --rpc-timeout= value in an edit-config rpc.
In the end, if I could fire up netopeer2-server with something like this:
> netopeer2-server -v 2 -t 120

Which would fire up netopeer-2 equivalently to a version which was compiled with my custom cmake parameters.

Okay, makes sense, done.

Okay, makes sense, done.

Okay, makes sense, done.

I'll test it out a bit on my end too

Okay, makes sense, done.

I'll test it out a bit on my end too

@michalvasko
I ran some tests and I got a timeout error after 1 second:
Start up command:
netopeer2-server -v 2 -t 120
Logs:
Jan  6 13:24:41 localhost netopeer2-server[19363]: Accepted a connection on <IP_ADDRESS>:830.
Jan  6 13:24:41 localhost netopeer2-server[19363]: Session 9 (user "root") created.
Jan  6 13:24:41 localhost netopeer2-server[19363]: Received an SSH message "request-service" of subtype "ssh-userauth".
Jan  6 13:24:41 localhost netopeer2-server[19363]: Received an SSH message "request-auth" of subtype "none".
Jan  6 13:24:41 localhost netopeer2-server[19363]: Received an SSH message "request-auth" of subtype "interactive".
Jan  6 13:24:41 localhost netopeer2-server[19363]: Received an SSH message "request-auth" of subtype "interactive".
Jan  6 13:24:41 localhost netopeer2-server[19363]: Failed user "root" authentication attempt (#1).
Jan  6 13:24:41 localhost netopeer2-server[19363]: Received an SSH message "request-auth" of subtype "password".
Jan  6 13:24:41 localhost netopeer2-server[19363]: User "root" authenticated.
Jan  6 13:24:41 localhost netopeer2-server[19363]: Received an SSH message "request-channel-open" of subtype "session".
Jan  6 13:24:41 localhost netopeer2-server[19363]: Received an SSH message "request-channel" of subtype "subsystem".
Jan  6 13:24:41 localhost netopeer2-server[19363]: Session 10 (user "root") created.
Jan  6 13:24:41 localhost netopeer2-server[19363]: There are no subscribers for "ietf-netconf-notifications" notifications.
Jan  6 13:24:41 localhost netopeer2-server[19363]: Generated new event (netconf-session-start).
Jan  6 13:24:41 localhost netopeer2-server[19363]: Resolving unresolved data nodes and their constraints...
Jan  6 13:24:41 localhost netopeer2-server[19363]: All data nodes and constraints resolved.
Jan  6 13:24:41 localhost netopeer2-server[19363]: Resolving unresolved data nodes and their constraints...
Jan  6 13:24:41 localhost netopeer2-server[19363]: All data nodes and constraints resolved.
Jan  6 13:24:41 localhost netopeer2-server[19363]: Published event "rpc" "/ietf-netconf:edit-config" with ID 1 priority 0 for 1 subscribers.
Jan  6 13:24:41 localhost netopeer2-server[19363]: Processing "/ietf-netconf:edit-config" "rpc" event with ID 1 priority 0 (remaining 1 subscribers).
Jan  6 13:24:41 localhost netopeer2-server[19363]: edit-config error-option "stop-on-error" not supported, rollback-on-error will be performed.
Jan  6 13:24:42 localhost netopeer2-server[19363]: Published event "change" "rmbn-full" with ID 1 priority 0 for 1 subscribers.
Jan  6 13:24:42 localhost netopeer2-server[19363]: Callback event "change" with ID 1 processing timed out.
Jan  6 13:24:42 localhost netopeer2-server[19363]: Event "change" with ID 1 priority 0 failed (Timeout expired).

However, when I use -t 120000 as the value, it seems to work.
netopeer2-server -v 2 -t 120000
Is it possible the help file is off and the -t value needs to be in milliseconds?

@michalvasko
I ran some tests and I got a timeout error after 1 second:
Start up command:
netopeer2-server -v 2 -t 120
Logs:
Jan  6 13:24:41 localhost netopeer2-server[19363]: Accepted a connection on <IP_ADDRESS>:830.
Jan  6 13:24:41 localhost netopeer2-server[19363]: Session 9 (user "root") created.
Jan  6 13:24:41 localhost netopeer2-server[19363]: Received an SSH message "request-service" of subtype "ssh-userauth".
Jan  6 13:24:41 localhost netopeer2-server[19363]: Received an SSH message "request-auth" of subtype "none".
Jan  6 13:24:41 localhost netopeer2-server[19363]: Received an SSH message "request-auth" of subtype "interactive".
Jan  6 13:24:41 localhost netopeer2-server[19363]: Received an SSH message "request-auth" of subtype "interactive".
Jan  6 13:24:41 localhost netopeer2-server[19363]: Failed user "root" authentication attempt (#1).
Jan  6 13:24:41 localhost netopeer2-server[19363]: Received an SSH message "request-auth" of subtype "password".
Jan  6 13:24:41 localhost netopeer2-server[19363]: User "root" authenticated.
Jan  6 13:24:41 localhost netopeer2-server[19363]: Received an SSH message "request-channel-open" of subtype "session".
Jan  6 13:24:41 localhost netopeer2-server[19363]: Received an SSH message "request-channel" of subtype "subsystem".
Jan  6 13:24:41 localhost netopeer2-server[19363]: Session 10 (user "root") created.
Jan  6 13:24:41 localhost netopeer2-server[19363]: There are no subscribers for "ietf-netconf-notifications" notifications.
Jan  6 13:24:41 localhost netopeer2-server[19363]: Generated new event (netconf-session-start).
Jan  6 13:24:41 localhost netopeer2-server[19363]: Resolving unresolved data nodes and their constraints...
Jan  6 13:24:41 localhost netopeer2-server[19363]: All data nodes and constraints resolved.
Jan  6 13:24:41 localhost netopeer2-server[19363]: Resolving unresolved data nodes and their constraints...
Jan  6 13:24:41 localhost netopeer2-server[19363]: All data nodes and constraints resolved.
Jan  6 13:24:41 localhost netopeer2-server[19363]: Published event "rpc" "/ietf-netconf:edit-config" with ID 1 priority 0 for 1 subscribers.
Jan  6 13:24:41 localhost netopeer2-server[19363]: Processing "/ietf-netconf:edit-config" "rpc" event with ID 1 priority 0 (remaining 1 subscribers).
Jan  6 13:24:41 localhost netopeer2-server[19363]: edit-config error-option "stop-on-error" not supported, rollback-on-error will be performed.
Jan  6 13:24:42 localhost netopeer2-server[19363]: Published event "change" "rmbn-full" with ID 1 priority 0 for 1 subscribers.
Jan  6 13:24:42 localhost netopeer2-server[19363]: Callback event "change" with ID 1 processing timed out.
Jan  6 13:24:42 localhost netopeer2-server[19363]: Event "change" with ID 1 priority 0 failed (Timeout expired).

However, when I use -t 120000 as the value, it seems to work.
netopeer2-server -v 2 -t 120000
Is it possible the help file is off and the -t value needs to be in milliseconds?

Right, should be correctly seconds now, the value you set as parameter.

Right, should be correctly seconds now, the value you set as parameter.


Fix typo

Whoops, good catch! That should have also been updated to include that release() can be called as well... I'll take care of that!


TP: Add CANCEL_RESPONSE action

The signer who created a response should be able to cancel it. Canceled responses should become null to avoid changing their index.

Supplanted by the version 2 refactor.


reflect: Type.Comparable returns incorrect values for some types obtained via reflection

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go version go1.8.3 darwin/amd64
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GORACE=""
GOROOT="/usr/local/Cellar/go/1.8/libexec"
GOTOOLDIR="/usr/local/Cellar/go/1.8/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/rk/3wxxpsdj3kx052390hbwxk0h0000gn/T/go-build181924434=/tmp/go-build -gno-record-gcc-switches -fno-common"
CXX="clang++"
CGO_ENABLED="1"
PKG_CONFIG="pkg-config"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
What did you do?
If possible, provide a recipe for reproducing the error.
A complete runnable program is good.
package main

import (
	"reflect"
)

func main() {
	var i interface{} = func() { println("Hello world") }
	v := reflect.Indirect(reflect.ValueOf(&i))
	if v.Type().Comparable() {
		m := map[interface{}]string{}
		m[v.Interface()] = "this will panic"
	} else {
		println("v is not comparable")
	}
}

A link on play.golang.org is best.
https://play.golang.org/p/l7rmrLiGvY
What did you expect to see?
v is not comparable

What did you see instead?
panic: runtime error: hash of unhashable type func()

goroutine 1 [running]:
main.main()
	/tmp/sandbox040073154/main.go:12 +0x1e0

Nice. But because this is not a regression, there's no time (or risk) budget for Go 1.9, due out shortly.

I think this is correct behavior.  If you print v.Type().String(), you will see that it has type interface {}.  Interface types are comparable.  It just happens that this interface value holds a value that can not be compared.
In other words, this is approximately equivalent to
package main

func main() {
	var i interface{} = func() { println("Hello world") }
	println(i == i)
}

which will also panic.

Whoops, I overlooked the type interface {} after "var i".
I agree this looks fine.

That's fine, but if interface{} is comparable, why can't I use it in the map then?

Also everything works as expected if I remove pointer: v := reflect.Indirect(reflect.ValueOf(_&_i))

You can use an interface{} in a map.
What you can't do is use an interface{} in a map if you store a func value in the interface.
If you remove the pointer, then the value you get in v has type func(), which is not comparable.  In both cases, try printing v.Type().String() to see what is going on.


RecursionError when used on subclass of Tensor

When using subclasses of Tensor, make_dot crashed with RecursionError.
class DummyTensor(torch.Tensor):
    pass

a = DummyTensor(torch.ones(5, requires_grad=True))
make_dot(a)

Output:
/usr/local/lib/python3.7/dist-packages/torchviz/dot.py in add_base_tensor(var, color)
    152             dot.edge(str(id(var.grad_fn)), str(id(var)))
    153         if var._is_view():
--> 154             add_base_tensor(var._base, color='darkolivegreen3')
    155             dot.edge(str(id(var._base)), str(id(var)), style="dotted")
    156

RecursionError: maximum recursion depth exceeded in __instancecheck__

The reason is because a is a view, so is a._base, so is a._base._base, so is a._base._base._base, etc.
https://colab.research.google.com/drive/1VlAEQY5lOtp3MsYfgyCkrNmkjzuKoRX1?usp=sharing

I'm confused, why is a a view here?

¯_(ツ)_/¯

So the answer is: a is a view of torch.ones(5).
This actually seems like a Pytorch bug.
a = torch.ones(1)
b = torch.Tensor(a)
print(b._is_view()) # True
print(b._base is a) # True

a = torch.ones(1)
c = DummyTensor(a)
print(c._is_view()) # True
print(c._base is a) # False
print(type(c._base)) # <class '__main__.DummyTensor'>

c._base here creates a new DummyTensor, as per the docs

As of version 1.7.0, methods on torch.Tensor and functions in public torch.* namespaces applied on torch.Tensor subclasses will return subclass instances instead of torch.Tensor instances:

Yeah, agreed this is a bug.

As a workaround, do the _base object within the DisableTorchFunction context manager:
with torch._C.DisableTorchFunction():
  base = c._base

This is indeed fixed by pytorch/pytorch#60464.


Add Bhavani Ravi

Thanks for this @bhavaniravi. Happy to merge this if you're already fluent with git branching, and resolving conflicts. But if you think some more practice with that could be helpful for you, you can make the changes locally (I think you made them from the GitHub website, right?).

Yes I did it from web. Infact from mobile. I am familiar with using git
from local and using branches etc., I'm using git everyday at my work so
won't be a problem
On Sun, Jul 28, 2019, 12:32 AM Marc Garcia<EMAIL_ADDRESS>wrote:

Thanks for this @bhavaniravi https://github.com/bhavaniravi. Happy to
merge this if you're already fluent with git branching, and resolving
conflicts. But if you think some more practice with that could be helpful
for you, you can make the changes locally (I think you made them from the
GitHub website, right?).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/python-sprints/pandas-mentoring/pull/13?email_source=notifications&email_token=ACNFXICBOPXNI3JT7GUILATQBSLV3A5CNFSM4IHKJSS2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD26Q3DI#issuecomment-515706253,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACNFXIESWFY3SP5XXHTMZQDQBSLV3ANCNFSM4IHKJSSQ
.

Cool, I guessed that, but wanted to be sure. Thanks!


Add Environment Variables To Build

To make this run correctly locally
you will need to add the following file to your root project folder
environment file.zip

This change is

netlify.toml, line 10 at r1 (raw file):

[context.production]
  [context.production.environment]
    REACT_APP_DOOR43_SERVER_URL = "https://api.door43.org"

is this the correct address or should it be git.door43.org?

netlify.toml, line 14 at r1 (raw file):

[context.deploy-preview]
  [context.deploy-preview.environment]
    REACT_APP_DOOR43_SERVER_URL = "https://api.door43.org"

This one should probably stick with the development server of bg.door43.org.

src/components/translatable/QuoteSelector.js, line 17 at r1 (raw file):

  buttons,
}) {
  const config = { server: process.env.REACT_APP_DOOR43_SERVER_URL };

I'm wondering if this should bring in the address from the application state so that it's kept in sync and only set once. Great catch as I didn't realize this was a separate url.

src/core/state.defaults.js, line 7 at r2 (raw file):

const config = {
  authentication: {
    server,

server: SERVER_URL

src/core/state.defaults.js, line 3 at r2 (raw file):

import appPackage from '../../package.json';

const server = process.env.REACT_APP_DOOR43_SERVER_URL;

js const SERVER_URL =  process.env.REACT_APP_DOOR43_SERVER_URL;

src/core/state.defaults.js, line 12 at r2 (raw file):

  repository: {
    urls: [
      server + '/api/v1/repos/unfoldingword/en_ta',

SERVER_URL + ...

src/core/state.defaults.js, line 23 at r2 (raw file):

  fontScale: 100,
  config,
  SERVER_URL: server

SERVER_URL

src/components/translatable/QuoteSelector.js, line 5 at r3 (raw file):

import { ParallelScripture, withResources } from 'scripture-resources-rcl';
import { testament } from '../../core/bcv.js';
import { SERVER_URL } from '../../core/state.defaults';

Netlify deploy is failing with an errror on this line: 2:04:10 PM: Attempted import error: 'SERVER_URL' is not exported from '../../core/state.defaults'.

a discussion (no related file):
Previously, RoyalSix (Sterling (Jay) Scott) wrote…
@klappy After our discussion I looked into having the environment variables set from the UI imported in the netlifty.toml in order to use context based environment variables, but it does not seem to be possible per netlify docs https://docs.netlify.com/configure-builds/file-based-configuration/#inject-environment-variable-values

LGTM


fix click duration for 0ms

Fix the click duration if 0 is asked. For example, the default duration (500ms) was applied in this case:
await browser.ocrClickOnText({
    text: "WebdriverIO",
    clickDuration: 0, // This is 0 seconds
}

Hi @wswebcreation, could you release a version with this fix?

Hi @lacell75
Yep, I'll release it later today, together with this PR https://github.com/webdriverio/visual-testing/pull/521

@lacell75
It's released, see https://github.com/webdriverio/visual-testing/releases/tag/%40wdio%2Focr-service%402.1.0

Thanks!
Le jeu. 3 oct. 2024 à 12:26, Wim Selles @.***> a écrit :

@lacell75 https://github.com/lacell75
It's released, see
https://github.com/webdriverio/visual-testing/releases/tag/%40wdio%2Focr-service%402.1.0
—
Reply to this email directly, view it on GitHub
https://github.com/webdriverio/visual-testing/pull/547#issuecomment-2391059120,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AH5XZJAKTAPFFCX46O5I4RLZZULW7AVCNFSM6AAAAABPHWGKOKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOJRGA2TSMJSGA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Added another endpoint

Adding in end point for userLookup to facilitate https://dev.twitter.com/docs/api/1/get/users/lookup

Merged, copied into the 3k branch as well, version bump, set, spike. Enjoy, thanks for playing!


tabs inside of toolbar

Hi Guys,
Is there any plan of giving tabs a stylesheet that it fits to the background of toolbars, or that tabs would be placed inside of the toolbar? According to the specs http://www.google.com/design/spec/components/tabs.html#tabs-types-of-tabs it seems to be quite heavily used. I've seen this behavior now on several apps. Thanks for looking into this.

I think you are looking for Angular Material repo. (This one is the core AngularJS repo.)
Please reopen in https://github.com/angular/material.

@gkalpak sorry my bad, too many tabs open


Updating ceres install instructions.

I recieved the following error when building the csfm library in Ubuntu:
usr/local/lib/libceres.a(loss_function.cc.o): relocation R_X86_64_32S against `_ZTVN5ceres12ComposedLossE' can not be used when making a shared object; recompile with -fPIC
/usr/local/lib/libceres.a: error adding symbols: Bad value
SuiteSparse needed to be installed according to the shared library instructions in the Ceres docs and Ceres needed to be compiled with the -fPIC option. I updated the readme accordingly.
The Ceres Ubuntu installation procedure has been simplified so I removed some outdated information.
I also added correct indentation for code blocks in list items.

The -fPIC option is needed because we are now using ceres from a python extension. Things are still quite shaky since we are trying new things. Thanks for fixing the doc!


Automation for adding support for new Docker versions

New Docker (patch) versions are generally released every 1-4 weeks and it's quite a lot of maintenance to keep up with them (e.g. see https://docs.docker.com/engine/release-notes/25.0/). The ask is to automate opening PRs for adding new Docker versions to https://github.com/rancher/install-docker
Even if the opened PR may require some manual tweaking (especially for new minor versions), it would still be of value if only to ensure we're keeping more up-to-date with upstream versions and to provide a good starting point for implementing new patch/minor Docker versions.

Setting the milestone to v2.9-Next1 as the "tip of development" version since this is a process/tooling improvement and belongs there.

Closing the issue as automation is creating PRs as expected.
ex: https://github.com/rancher/install-docker/pull/130


tokendata not save in .tokendata file.

I've try to exchange tokens data using exchangecodefortokens.php with the code. its not working. after execute exchangecodefortokens.php file this message appear on page  "Usage: exchangecodefortokens.php -c AUTHORIZATION CODE"   what should i do now ?

Did you follow everything in the Setup step?
https://github.com/magnetikonline/phpgooglespreadsheetapi#setup
You need to create a new Google OAuth client token via the web, edit config.php to suit - and then call buildrequesturl.php. The URL returned here is then put into another browser - upon it's return you use the code= querystring value for the argument to exchangecodefortokens.php.
Note - all these scripts need to be called from the CLI - not a browser window.


enfore the active_connection_id_limit

See https://github.com/quicwg/base-drafts/pull/3201/.

Codecov Report

Merging #2252 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2252      +/-   ##
==========================================
- Coverage   86.44%   86.43%   -<.01%
==========================================
  Files         106      106
  Lines        7793     7791       -2
==========================================
- Hits         6736     6734       -2
  Misses        794      794
  Partials      263      263

Impacted Files
Coverage Δ

internal/qerr/error_codes.go
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

@marten-seemann @lucas-clemente Does this fix #2204 ?

Yes, that's correct. Thank you.


Execution hangs when exceptions are thrown

When passing in a test filter that is improperly formed, Flank will throw an exception but will not terminate. When flank is run in CI, this causes unnecessary timeouts while the scripts are waiting for the command to finish.
Example error
java.lang.IllegalArgumentException: Invalid argument: com.package.Class#testMethod 	at ftl.filter.TestFilters.parseSingleFilter(TestFilters.kt:100) 	at ftl.filter.TestFilters.access$parseSingleFilter(TestFilters.kt:31) 	at ftl.filter.TestFilters$fromTestTargets$parsedFilters$2.invoke(TestFilters.kt:85) 	at ftl.filter.TestFilters$fromTestTargets$parsedFilters$2.invoke(TestFilters.kt:31) 	at kotlin.sequences.TransformingSequence$iterator$1.next(Sequences.kt:172) 	at kotlin.sequences.SequencesKt___SequencesKt.toCollection(_Sequences.kt:702) 	at kotlin.sequences.SequencesKt___SequencesKt.toMutableList(_Sequences.kt:732) 	at kotlin.sequences.SequencesKt___SequencesKt.toList(_Sequences.kt:723) 	at ftl.filter.TestFilters.fromTestTargets(TestFilters.kt:86) 	at ftl.args.AndroidTestShard.getTestMethods(AndroidTestShard.kt:32) 	at ftl.args.AndroidTestShard.getTestShardChunks(AndroidTestShard.kt:25) 	at ftl.run.AndroidTestRunner$runTests$2.invokeSuspend(AndroidTestRunner.kt:38) 	at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33) 	at kotlinx.coroutines.ResumeModeKt.resumeUninterceptedMode(ResumeMode.kt:45) 	at kotlinx.coroutines.internal.ScopeCoroutine.afterCompletionInternal(Scopes.kt:32) 	at kotlinx.coroutines.JobSupport.completeStateFinalization(JobSupport.kt:311) 	at kotlinx.coroutines.JobSupport.tryFinalizeFinishingState(JobSupport.kt:237) 	at kotlinx.coroutines.JobSupport.tryMakeCompletingSlowPath(JobSupport.kt:851) 	at kotlinx.coroutines.JobSupport.tryMakeCompleting(JobSupport.kt:813) 	at kotlinx.coroutines.JobSupport.makeCompletingOnce$kotlinx_coroutines_core(JobSupport.kt:789) 	at kotlinx.coroutines.AbstractCoroutine.resumeWith(AbstractCoroutine.kt:111) 	at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:46) 	at kotlinx.coroutines.DispatchedTask.run(Dispatched.kt:241) 	at kotlinx.coroutines.EventLoopImplBase.processNextEvent(EventLoop.kt:116) 	at kotlinx.coroutines.BlockingCoroutine.joinBlocking(Builders.kt:79) 	at kotlinx.coroutines.BuildersKt__BuildersKt.runBlocking(Builders.kt:54) 	at kotlinx.coroutines.BuildersKt.runBlocking(Unknown Source) 	at kotlinx.coroutines.BuildersKt__BuildersKt.runBlocking$default(Builders.kt:36) 	at kotlinx.coroutines.BuildersKt.runBlocking$default(Unknown Source) 	at ftl.cli.firebase.test.android.AndroidRunCommand.run(AndroidRunCommand.kt:49) 	at picocli.CommandLine.executeUserObject(CommandLine.java:1687) 	at picocli.CommandLine.access$900(CommandLine.java:146) 	at picocli.CommandLine$RunLast.handle(CommandLine.java:2059) 	at picocli.CommandLine$RunLast.handle(CommandLine.java:2026) 	at picocli.CommandLine$AbstractParseResultHandler.execute(CommandLine.java:1893) 	at picocli.CommandLine.execute(CommandLine.java:1822) 	at ftl.Main$Companion.main(Main.kt:41) 	at ftl.Main.main(Main.kt)

@RainNapper Thanks for reporting this bug, have you encountered similar issue with other cases?

Brought up some more details in slack message here: https://firebase-community.slack.com/archives/C72V6UW8M/p1583276521008200
Copy pasting here:
I filed https://github.com/Flank/flank/issues/649
It might be tangentially related to https://github.com/Flank/flank/issues/532
But I think I narrowed it down to couple things:

In Main.kt there's CommandLine(Main()).execute(*args). For non-zero exit statuses, I believe it should be replaced with exitProcess(CommandLine(Main()).execute(*args)) . It returns an exit code of 1, but the process never terminates. I can't figure out how to build the .jar locally, but I repro'd in Android Studio via Debug.kt throwing exceptions.
picocli.CommandLine#execute(...) doesn't catch Throwable. It only catches Exception. In my luckily dysfunctional local env, running Debug.kt would throw a VerifyError . Because Error is not caught, excute(...)  never even returns.
Regarding the picocli issue, I was able to "fix" the issue in Flank by just wrapping the AndroidRunCommand#run with a try { ... } catch (t: Throwable) { throw RuntimeException(t) } so it got caught by picocli's filters. This is obviously not an ideal fix, but it's worth noting as how I "confirmed" it was the issue with picocli.

If you're curious, here's the first couple lines of the VerifyError , but unrelated to this bug.
Exception Details:
  Location:
    com/fasterxml/jackson/databind/deser/std/StdDeserializer._parseDate(Lcom/fasterxml/jackson/core/JsonParser;Lcom/fasterxml/jackson/databind/DeserializationContext;)Ljava/util/Date; @77: astore
  Reason:
    Type 'com/fasterxml/jackson/core/JsonParseException' (current frame, stack[0]) is not assignable to 'com/fasterxml/jackson/core/exc/StreamReadException' (stack map, stack[0])
  Current Frame:
    bci: @69
    flags: { }
    locals: { 'com/fasterxml/jackson/databind/deser/std/StdDeserializer', 'com/fasterxml/jackson/core/JsonParser', 'com/fasterxml/jackson/databind/DeserializationContext' }
    stack: { 'com/fasterxml/jackson/core/JsonParseException' }```

Hello, @bootstraponline raised a ticket on the picocli issue tracker and asked me to help with investigation on this issue, mentioning that "It seems like the root cause is due to picocli". I'm not convinced yet that this is the case, but I'm happy to help.
Looking at the source code for Main.kt, I don't see anything unusual.
First, @RainNapper's point (1):

It (the CommandLine.execute method) returns an exit code of 1, but the process never terminates.

Does Flank, or kotlinx.coroutines spin off any non-daemon threads? That would explain the JVM not exiting even though the CommandLine.execute method, and subsequently the Main.main method returned and the main thread exited.
If that is the case, and the desired behaviour is the exit the JVM, I support @RainNapper's suggestion to explicitly call System.exit when the CommandLine.execute method returns.
The non-daemon thread may also explain @RainNapper's point (2):

picocli.CommandLine#execute(...) doesn't catch Throwable. It only catches Exception. In my luckily dysfunctional local env, running Debug.kt would throw a VerifyError . Because Error is not caught, excute(...) never even returns.

It is true that picocli only catches Exceptions and not Errors. But this does not mean that the CommandLine.execute method does not return, it means that the Error is percolated all the way up the stack to the Main.main method, never being caught, and killing the main thread. Usually that would mean the JVM would exit, but if a non-daemon thread is still running, that thread will keep the JVM going even after the main thread died.
Putting those together, perhaps the Main.kt's main method should be changed to the below:
// Main.kt
companion object {
    @JvmStatic
    fun main(args: Array<String>) {
        try {
            exitProcess(CommandLine(Main()).execute(*args))
        } (catch t: Throwable) {
            t.printStackTrace()
            exitProcess(picocli.CommandLine.ExitCode.SOFTWARE)
        }
    }
}

That looks good to me. The only thing I did before concluding that CommandLine.execute doesn't execute is put a print statement after it and verify it didn't print. I had (incorrectly) assumed that if Main.main caught the exception, then the everything would exit. But I am not familiar with the JVM / coroutine interactions that would create the same result.
What you said sounds like a good explanation to me, and thus the changes look good too.
Thanks for the input! Very much appreciated.

@remkop Thank you for the insight! That makes sense.

Glad I was able to help. Remember: your picocli GitHub :stars: stars are the only thing that fill the yawning void in my soul! :wink:

Additionally, assuming that my hypothesis holds water and the problem is caused by non-daemon threads preventing the JVM from exiting, if Flank creates those threads, Flank could also consider making those threads daemon threads. (I would still recommend the above changes in case something else creates a non-daemon thread.)

Hey, I managed to find root cause of this issue.
TLDR; it's not picocli but bugsnag :)
While debugging I noticed that flank has non-daemon thread (as @remkop predicted). Digging deeper lead me to this line in FtlConstants
val bugsnag = Bugsnag(if (useMock) null else (...) )

Bugsnag uses its own thread pool to handle sending statistics, reports etc. This executor is not closed properly and in this case prevents jvm from shutting down. I verified this by closing bugsnag manually by simple call
bugsnag.close()

and in all cases flank terminated successfully. When I tracked down it is bugsnag I found there is already issue posted Bugsnag#151
I tried proposed solution but I think simple wrapping with exitProcess(...) is enough (verified)

@pawelpasterz Thanks for figuring this out!

@pawelpasterz The proposed solution covers against the case where some component in the call stack starts a non-daemon thread, and then throws an Error that kills the main thread.
I'm not sure if any of the components you are currently testing against show this behaviour (I assumed they did, since @RainNapper brought it up). Even if there are currently no such components, it may still be worth protecting against them. :-)

@remkop got your point, I'll change my fix then. Thanks for pointing this out


feat: add Hashicorp Vault secrets backend

What this PR changes/adds
Add Hashicorp Vault backend for storing secrets.
Why it does that
To be able to deploy the EDC without any cloud dependencies.
Further notes
List other areas of code that have changed but are not necessarily linked to the main feature. This could be method
signature changes, package declarations, bugs that were encountered and were fixed inline, etc.
Linked Issue(s)
Closes #1026
Checklist

[x] added appropriate tests?
[x] performed checkstyle check locally?
[x] added/updated copyright headers?
[x] documented public classes/methods?
[x] added/updated relevant documentation?
[x] added relevant details to the changelog? (skip with label no-changelog)
[x] formatted title correctly? (take a look at the CONTRIBUTING and styleguide for details)

@ndr-brt @paullatzelsperger @juliapampus Hey, I'd appreciate some input in the following matter:
Downstream, we used Testcontainers for the integration test of this component. I've mostly adapted that to use services in github actions, as per your standards upstream. However, I am having some issues debugging this. I can run the actions locally with ACT, but I'm unable to find the issue there, as the test output is very sparse, even on -v. Do you have an idea how best to handle this?

@bcronin90 not entirely sure what ACT refers to, but I usually run the tests locally in IntelliJ using docker containers.
Moving to services from there is normally pretty straight-forward.

@bcronin90 not entirely sure what ACT refers to, but I usually run the tests locally in IntelliJ using docker containers. Moving to services from there is normally pretty straight-forward.

It's a similar approach: https://github.com/nektos/act
Basically runs the action locally. I can set up the necessary containers manually, but I was worried that might not be representative.

@bcronin90 to run a local instance of vault to be used for integration tests you can do (taken from the documentation)
 docker run  -e 'VAULT_DEV_ROOT_TOKEN_ID=myroot' -p "8200:8200" vault

(on the docs there are some details about a --cap-add=IPC_LOCK parameter, but on my machine it worked also without)
http://localhost:8200 would be the url and myroot the token to be configured to connect
theoretically, the github service would need to be configured like this:
    services:
      hashicorp-vault:
        image: vault
        ports:
          - 8200:8200
        env:
          VAULT_DEV_ROOT_TOKEN_ID: myroot

There is another problem, I noticed that, the OkHttpClient injection in the vault creates a cyclic dependency between the CoreServicesExtension, this could be a bug inside the injection loading, as in fact there would be no cyclic dependency, so my advice would be, momentarily, to have everything running, to avoid injecting the OkHttpClient and creating one from scratch leaving a TODO to be fixed later on.

@bcronin90 to run a local instance of vault to be used for integration tests you can do (taken from the documentation)
 docker run  -e 'VAULT_DEV_ROOT_TOKEN_ID=myroot' -p "8200:8200" vault

(on the docs there are some details about a --cap-add=IPC_LOCK parameter, but on my machine it worked also without)
http://localhost:8200 would be the url and myroot the token to be configured to connect
theoretically, the github service would need to be configured like this:
    services:
      hashicorp-vault:
        image: vault
        ports:
          - 8200:8200
        env:
          VAULT_DEV_ROOT_TOKEN_ID: myroot

There is another problem, I noticed that, the OkHttpClient injection in the vault creates a cyclic dependency between the CoreServicesExtension, this could be a bug inside the injection loading, as in fact there would be no cyclic dependency, so my advice would be, momentarily, to have everything running, to avoid injecting the OkHttpClient and creating one from scratch leaving a TODO to be fixed later on.

Running a vault and interfacing it with the tests works fine. And the command I put into the github action definition is identical to that even. But doing it by hand works, while the github action fails. I'm not quite sure why. Currently it seems there is some dependency problem, but I'm not 100% sure yet.

Codecov Report

Merging #1534 (1b63b0b) into main (d63aef8) will decrease coverage by 0.39%.
The diff coverage is 38.93%.

@@            Coverage Diff             @@
##             main    #1534      +/-   ##
==========================================
- Coverage   67.37%   66.98%   -0.40%
==========================================
  Files         737      749      +12
  Lines       16194    16420     +226
  Branches     1058     1073      +15
==========================================
+ Hits        10911    10999      +88
- Misses       4803     4932     +129
- Partials      480      489       +9

Impacted Files
Coverage Δ

...se/dataspaceconnector/common/security/PemUtil.java
0.00% <0.00%> (ø)

...ashicorpVaultCreateEntryRequestPayloadOptions.java
0.00% <0.00%> (ø)

...ty/hashicorpvault/HashicorpVaultEntryMetadata.java
0.00% <0.00%> (ø)

...curity/hashicorpvault/HashicorpVaultExtension.java
0.00% <0.00%> (ø)

...VaultGetEntryResponsePayloadGetVaultEntryData.java
0.00% <0.00%> (ø)

...ault/HashicorpVaultCreateEntryResponsePayload.java
16.66% <16.66%> (ø)

...rpvault/HashicorpVaultGetEntryResponsePayload.java
16.66% <16.66%> (ø)

...y/hashicorpvault/HashicorpCertificateResolver.java
56.25% <56.25%> (ø)

.../security/hashicorpvault/HashicorpVaultClient.java
63.07% <63.07%> (ø)

...r/core/security/hashicorpvault/HashicorpVault.java
77.77% <77.77%> (ø)

... and 2 more

Continue to review full report at Codecov.

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

@paullatzelsperger I don't really agree on removing the Hashicorp prefix from the class names, if we'd apply this to the rest of the project it will make all the project more hard to read and to go through (imagine having 10 Client classes...).
Personally I would keep it as it is for all the other components.

@paullatzelsperger I don't really agree on removing the Hashicorp prefix from the class names, if we'd apply this to the rest of the project it will make all the project more hard to read and to go through (imagine having 10 Client classes...).
Personally I would keep it as it is for all the other components.

I was mostly referring to the ...Payload dto classes, because there it made the class name very long and unwieldy, and since they're internal/package private classes I didn't think it too much of a problem, but we can definitely re-add that prefix to Client and Config

Let's get this merged! :champagne:


Update metalmatze's company

:partying_face: :ice_cube:

[ ] I added CHANGELOG entry for this change.
[x] Change is not relevant to the end user.

Changes
Quit a job and start a new one.
Verification
I signed a contract. :smile:
/cc @brancz

I guess we need to update a couple of other maintainers' company as well. cc @yeya24 @onprem


Add README Section Usage

wdyt @jmelot? I'm glad to put in a PR.

Also seems like a good idea, please go for it (and/or let me know if you want me to take any of these cleanup tasks on)!


decentralized ?

Is Sovereign decentralized .....Can Sovereign run on clients with no need for a central server ?
Thanks

We are aiming for that in our initial release. We still have a long way to go.


[BETA] Install Issue, howdy/dlib-data in wrong  folder?

Installed and configured howdy beta branch, all commands such as sudo work great!
Lockscreen auth fails with Unknown Error 1, I narrowed it down to maybe a permissions issue? (lock screen doesn't run howdy as root) except that Howdy 2.6.1 works perfectly.
So I follow the suggested fix here and realized /lib/security/howdy/dlib-data doesn't exist.
dlib-data however does exist in /etc/howdy/
Install log: https://pastebin.com/WFZCqHn4
auth.log: https://pastebin.com/Kt6cwvuV
Thanks in advance for any help and for creating howdy!

I've searched and found relevant #307 but I have no issues with DNS.
Linux distribution (if applicable):
PopOS 22.04
Howdy version (sudo howdy version):
Howdy 3.0.0 BETA

That's because you're very much on the bleeding edge, #649 got merged and moved Howdy folders around so it can be used on specific distros that don't allow writing to /lib. /etc/howdy is the correct place for the dlib files now, so you can try the fix there!


Update aws-java-sdk to 1.12.772

About this PR
📦 Updates com.amazonaws:aws-java-sdk from 1.12.761 to 1.12.772
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.amazonaws", artifactId = "aws-java-sdk" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.amazonaws", artifactId = "aws-java-sdk" }
}]

labels: test-library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #1034.


Issue: GacUtil should use /f option to force overwrite of existing

Currently /f is not passed to gacutil, so in the case of an updated assembly with identical assembly version to a previous build in the GAC, the new assembly will not be installed in the GAC.  This is not an issue in cases where the previous assembly is uninstalled first.  It mostly affects shared assemblies.
This work item was migrated from CodePlex
CodePlex work item ID: '10146'
Assigned to: 'tfabraham'
Vote count: '1'

[UnknownUser@5/30/2013]
Resolved with changeset 73064.


check the services of a bluetooth device

Description:
This adds a check to find the services of a bluetooth device. By doing this, the component actually connects to a bluetooth device. If there are no services found, the device isn't in range.
This solves the issue that paired devices are always found as 'home'. At least, that's on Windows, I'm not sure if this is also the case on other OS's.
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>

lgtm :+1:

This PR seems to have gone stale. Closing it. You can reopen it when you're ready to finish it.


#RGB color for specific app

Please add #RGB color setting for specific app. For example: use System color (#000000) for all apps, total commander and qBittorrent use custom color

There's an API to set titlebar color, but it only works on build 22000+ and doesn't work when Mica is enabled.

Building off this; is there a way to disable system colour highlighting when Acrylic is the option? For example below, choosing Light or Dark as the titlebar colour adds a strong tint to the Acrylic effect, is there any way to set this to None and just have acrylic be influenced by the colours behind it, rather than also adding a tint?

is there any way to set this to None and just have acrylic be influenced by the colours behind it, rather than also adding a tint?

This question is not related to this issue, it's off-topic. Also MFE doesn't render anything, it just asks Desktop Window Manager to do stuff. So there's no way. You may create an another project to hook in to apps or DWM or even patch DWM to achieve what you want, but it's not MFE's goal.

is there any way to set this to None and just have acrylic be influenced by the colours behind it, rather than also adding a tint?

This question is not related to this issue, it's off-topic. Also MFE doesn't render anything, it just asks Desktop Window Manager to do stuff. So there's no way. You may create an another project to hook in to apps or DWM or even patch DWM to achieve what you want, but it's not MFE's goal.

I was just asking because you literally mentioned in the comment above me that an API existed to set titlebar colour. All g tho thanks for the response

you literally mentioned in the comment above me that an API existed to set titlebar colour

It just sets solid color for titlebar, it doesn't work when using acrylic or mica.

no worries thanks for the info!


added stardust per hour

i was playing with config and i saw that there was no stardust per hour so i added it
i don't know how to edit more files at once so i made 3

👍

thanks this was my fist ever python script


[fuzz] Simplify macros used by single-instruction generator

This removes the multiple macros used previously to describe the
WebAssembly instruction signatures and replaces them with a single
one--inst!.

Looks great to me!


[backport] Fix cupyx.scipy.sparse._base tests for SciPy 1.11

Backport of #7905 by @asi1024

Superseded by #7918


[unifi] Rename port names for better traefik integration

Description of the change
I changed the names of the ports for the captive portal from the format captive-http(s) to the format http(s)-captive.
Benefits
Traefik automatically talks to backends via https when the port name starts with https. [1]
Otherwise you have to add the annotation traefik.ingress.kubernetes.io/service.serversscheme: https to the service.
[1] https://doc.traefik.io/traefik/routing/providers/kubernetes-ingress/#communication-between-traefik-and-pods
Possible drawbacks
Some people might have configured external loadbalancers on the old name and therefore might experience problems when upgrading to a version with this change.
Applicable issues
none
Additional information
I don't know if this would bump the Chart version to 2.0.0 ?
Checklist

[ ] Chart version bumped in Chart.yaml according to semver.
[x ] Title of the PR starts with chart name (e.g. [home-assistant])
[ ] (optional) Chart is using our common library as a dependency.
[ ] (optional) Variables are documented in the README.md

see as this could be considered a breaking change, please bump the major chart version in Chart.yaml

@cbergmann sorry one more change, you might have to rebase on-top of master.

thanks!

no problem. Glad I could help.

@all-contributors add @cbergmann for code


Bugfixes to Event Form

Description
Fixes for two bugs in add-event-form.

UserForm was not submitting data to backend. Fix: Added DataService.create() request to handleClick in UserForm.js.
MongoDB error when event data is submitted to backend, recurringDates field is required. Fix: Removed required: true from recurringDates in Event model schema.

Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings

Looks good. Can you create an issue describing what you found and tag @InstinctDev?


ValExprImpl more efficient opportunity?

Could Sum and Product be made more effective using the same approach as described on
https://wiki.haskell.org/Smart_constructors#Runtime_Optimisation_:_smart_constructors?

Another opportunity is equality of two sums. E.g. x+y == x + z...
Move one sum to the otherwise to get 0 == y - z.... And the problem is simplified....


spder can not run

Description
What steps will reproduce the problem?

i install anaconda navigator1.8.1
update some libs
launch spyder  display information

What is the expected output? What do you see instead?
Please provide any additional information below
File "d:\ProgramData\Anaconda3\lib\site-packages\spyder\plugins_init_.py", line 511, in
toggled=lambda checked: self.toggle_view(checked),
File "d:\ProgramData\Anaconda3\lib\site-packages\spyder\plugins\ipythonconsole.py", line 725, in toggle_view
self.create_new_client(give_focus=False)
File "d:\ProgramData\Anaconda3\lib\site-packages\spyder\plugins\ipythonconsole.py", line 1033, in create_new_client
self.connect_client_to_kernel(client)
File "d:\ProgramData\Anaconda3\lib\site-packages\spyder\plugins\ipythonconsole.py", line 1068, in connect_client_to_kernel
kc.start_channels(shell=True, iopub=True)
File "d:\ProgramData\Anaconda3\lib\site-packages\qtconsole\kernel_mixins.py", line 43, in start_channels
super(QtKernelClientMixin, self).start_channels(*args, **kw)
File "d:\ProgramData\Anaconda3\lib\site-packages\jupyter_client\threaded.py", line 212, in start_channels
self.shell_channel._inspect = self._check_kernel_info_reply
File "d:\ProgramData\Anaconda3\lib\site-packages\jupyter_client\client.py", line 143, in shell_channel
socket, self.session, self.ioloop
File "d:\ProgramData\Anaconda3\lib\site-packages\jupyter_client\threaded.py", line 205, in ioloop
self._ioloop = ioloop.IOLoop(

Please open the Anaconda prompt and run
conda install tornado=4.5.3

to fix this problem.

Duplicate of #6627 and its brethren.


triage accepted doesnt stale

/assign @knative-sandbox/technical-oversight-committee

fat fingered the dropdown


Remove simulator hypervisor plugin

Combined build running here: https://beta-jenkins.mcc.schubergphilis.com/job/cosmic/job/0002-tracking-repo-branch-build/130/

@miguelaferreira I couldn't find a single reference to the entire cosmic-core/agent-simulator/tomcatconf/ directory or the commands-simulator.properties.in file.
Removing seems fine by me.

ping @bheuvel
Build is green, see: https://beta-jenkins.mcc.schubergphilis.com/job/cosmic/job/0002-tracking-repo-branch-build/130/


Fix early morning preference for service booking

NEEDS REBASE after merging https://github.com/nhsalpha/nhs_prototype_kit/pull/170
First, a little refactor: factor out getServiceFromSlug as it's repeated code
Then enable the early morning appointments page for service appointments (by adding the service_slug URL parameter in routes.js) and make the following resulting fixes:

Tweak the appointment times so it actually makes sense when you click early appointments.
Actually pass in address for each appointment in every page, not just next-available-appointment
Actually hide filters for the early morning appointments page according to the service filters

click early morning appointment

Ah, I've just noticed the practice address is showing up when it shouldn't be. Closing until it's fixed.

:+1: Thanks for the super clear commits


[add-resource] New Theme and Skeleton

I would like to add my new theme and skeleton to the Grav Repository.
Here are the project details:

Skeleton: https://github.com/muuvmuuv/grav-skeleton-flaten
Theme: https://github.com/muuvmuuv/grav-theme-flaten

Added to GPM, sorry for the delay!


Fix typos in the WebSocket "order" command

@yankuu why this is closed?

I deleted the temporary repo backing this PR while cleaning up my "Repositories" list.