Change bounds of StereoPannerNode 'pan' to ± 1.0

The min and max values for the pan of a StereoPannerNode are -1 and 1 respectively. The input in popup.html returns a value from -4 to 4, which was then divided by 5 in popup.js to create a normalized value passed to the AudioContext created in background.js. This gives the pan an effective range of ± 0.8, rather than the actual min-max values.
When a user wants audio from a tab to be output in only one channel, only an 80% pan can then be obtained. Changing the denominator to 4 in popup.js allows for a complete pan (min-max of ± 1.0 instead of ± 0.8).

I missed this pr until now somehow, Thanks for catching that!

hi your extension is not working on dell PCs


Allow Order / Payment / Delivery State to be changed WITHOUT sending an email

Description
The fact, that in Shopware 6 it's required for an order / delivery / payment state to be connected to an email template is quite annoying. Right now you cannot change the state of an order without sending an email. I can not imagine that this is the desired behaviour of all shop admins that use shopware. Please allow us to change a state without sending an email to the customer.

I'd like to add one more thing. The fact that each order state change triggers an email is highly confusing for the customer. Especially when an order / payment / delivery state is being reopened.
Image, that the state of an order / payment/ delivery needs to be reopened. In many cases, this state is only an intermediate state, to be able to reach a new state, that was previously unreachable.
It is absolutely not desirable that the customer get's two emails in this case.

Is possible since some minor updates :)


Influence of the "conv_kernel_size" within the proposed Nystrom Attention

Congrats on your great work!
I am verifying your method on vision tasks and have a small concern on the influence of the "conv_kernel_size" of the 2D group-convolution in your task and I find that you choose relatively large numbers such as 35.
In vision tasks, applying the convolution with such a large kernel size is typically for ensuring a larger receptive field. Considering the proposed Nystrom attention already has the capability to model the long-range context following the original Multi-Head Attention.  In summary, I am a little bit confused about the motivation of such a design.
Another important concern is that: should we set the num_landmarks equal to the feature map width as the image feature maps are of grid structure?
It would be great if you could share your advice on the influence of this parameter!
https://github.com/mlpen/Nystromformer/blob/effde255e6b38282840d0b4b620002579b32e4a2/code/attention_nystrom.py#L23-L28
https://github.com/mlpen/Nystromformer/blob/2bcc280c8cc3ab834e0c5ead2520a5872adbe348/LRA/code/lra_config.py#L46-L52

@PkuRainBow, Thanks for your interest. There are several deadlines recently.  Sorry for the late reply.
The basic idea for this design, (a), it helps for faster training; (b), it helps capture the local details without requiring many landmarks on language modeling tasks. For vision task, we found that the local details are not that important comparing to language modeling tasks. I recommend you reduce the kernel size to a small number or without using it. For example, I directly run a trained model T2t-Vit-t-14 on ImageNet without retraining for inference by replacing the self-attention part in T2T module by Nystromformer with the kernel size = 0 and num_landmarks = 64 and it works pretty well, 78% top-1 accuracy. For Performer, it is 73.7%. For Linformer, it is 65.3%.
With respect to setting num_landmarks, it may depend on your tasks. When you use more landmarks, it will be more accurate to approximate standard self-attention. Based on my experience, num_landmarks = 64 works well for image classification. If you want higher accuracy, you can try to increase num_landmarks, e.g.128.

@yyxiongzju Thanks for your detailed explanation!
According to your comments, we guess that your method will be promising on the vision transformer tasks and we are wondering whether you have tried to retrain your method with DeiT or T2T-Vit by replacing all the MHSA with the Nystrom scheme.
In fact, I have tried to do such a change by replacing all the MHSA with the Nystrom scheme but find the loss becomes NAN at the early stage. It would be great if you could share with me your comments!

@PkuRainBow, I did not retrain DeiT or T2T-ViT with the Nystrom scheme. But I did try it on object detection. It works well.
I shared the code of using T2T-ViT with the Nystrom scheme by replacing all the MHSA.
I saw the NAN issue in the original T2T-ViT github repo. Can you run the code I shared to see if it works well?

@yyxiongzju Thanks for your reply! I will try the shared code soon.


Refactored exoplanets into modular plotting functions

The exoplanets example looked great, but I kept wanting to be able to see plots on their own, which was difficult when they were all bound together into the gal_map function. I've split it into separate plotting functions for each of the items in the overlay, which makes it possible to see how everything fits together and also makes it much easier to e.g. change the format into a layout instead of an overlay if desired. This also let me put all the Panel-dashboard stuff at the very end, cleanly separating the underlying data-manipulation code from the widgets and layouts.
I also vectorized the deltav function by rewriting it in terms of numpy calls. Previously it was using Pandas .apply for vectorization, but now it should be faster and simpler to use, working either with apply or simply being passed the entire dataframe.
Remaining issues:

[ ] Chained pn.bind issue? show_candidates only works the first time, to enable showing them; it doesn't let them be turned off afterwards.
[ ] Some of the cells could use more explanation -- what are we supposed to see? How do we interpret it?
[ ] I added a title to the final dashboard, but it should probably also have some explanatory text

Can we wrap the dashboard in a template at the end? I don't think we should be adding further examples that don't use templates at this point.

Now requires fixes on holoviews master and holoviews branch visible_update; shouldn't be published until those changes are available.


Cannot set property 'hrtime' of undefined

Hello,
After migrating from lolex 1.4.0 to 1.5.0, my tests fails with this exception:
     TypeError: Cannot set property 'hrtime' of undefined
      at uninstall (node_modules/lolex/src/lolex-src.js:360:39)
      at Object.clock.uninstall (node_modules/lolex/src/lolex-src.js:629:13)
      at bus.close.then.then (test/scheduling/testScheduler.js:64:57)

I'm using nodejs 6.2.0.
My tests use lolex this way:
Setup:
clock = lolex.install(context)

Tear down:
clock.uninstall()

lolex-src.js 1.5.0 seems to assume that the context object has a key named process.
Any idea regarding this ? Does this mean that compatibility was broken somehow ?

@rogierschouten Could this be related to #40 ?

You are right, the breaking code was added by this merge.
Doesn't this means that 1.5.0 breaks backward compatibility and should be tagged as 2.0.0 (a migration note can also probably help)?
Or perhaps the code should be modified to not rely on target.process?

@gautaz This breaking change was unintentional. Needs a bugfix.

looking into it now

Issued PR #62 for this. I could not reproduce the issue in any way also not in a browser. However, the code was clearly not airtight so I think this would fix it.

@gautaz Could you please provide a test case that is runnable in Node? I could then squash that together with #62 if that indeed fixes it.

Here it is: #65

@rogierschouten Mind seeing if @gautaz test breaks for you? If so, then you have your reproducible test case. You could then merge that into #62 (just remove the built source).

@fatso83 done

Fixed. Will release a new version asap.


Support https://status.zendesk.com

Summary
Status pages doesn't support https://status.zendesk.com. It would be good if it would.

Closing. Tracking the broader issue in #45.


Nest Issue with Config Entry

Home Assistant release with the issue: 0.72
Last working Home Assistant release (if known): 0.71
Operating environment (Hass.io/Docker/Windows/etc.): RPi3 AiO install w/ venv
Component/platform: nest
Description of problem: The nest component is not working now that it is set up as a config entry
Getting the following errors on start now. Looks to be related to a translation file?
018-06-25 11:37:19 ERROR (MainThread) [homeassistant.setup] Setup failed for nest: Component did not return boolean if setup was successful. Disabling component.
2018-06-25 11:37:27 ERROR (MainThread) [homeassistant.core] Timer got out of sync. Resetting
2018-06-25 11:37:29 ERROR (MainThread) [homeassistant.core] Error doing job: Task exception was never retrieved
Traceback (most recent call last):
  File "/usr/lib/python3.5/asyncio/tasks.py", line 239, in _step
    result = coro.send(None)
  File "/srv/hass/hass_venv/lib/python3.5/site-packages/homeassistant/components/frontend/__init__.py", line 493, in send_translations
    resources = await async_get_translations(hass, msg['language'])
  File "/srv/hass/hass_venv/lib/python3.5/site-packages/homeassistant/helpers/translation.py", line 121, in async_get_translations
    resources = await async_get_component_resources(hass, language)
  File "/srv/hass/hass_venv/lib/python3.5/site-packages/homeassistant/helpers/translation.py", line 100, in async_get_component_resources
    hass, component, language)
  File "/srv/hass/hass_venv/lib/python3.5/site-packages/homeassistant/helpers/translation.py", line 41, in component_translation_file
    component_path = path.dirname(module.__file__)
AttributeError: 'NoneType' object has no attribute '__file__'

Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
Set up as a config entry, no longer defined in configuration.yaml
Traceback (if applicable): see above
Additional information:

Maybe not, Nest setup failed reported at 11:37:19, translation issue reported 10 seconds later.
Could you enable debug logging for homeassistant.setup with other nest related logger?
# configuration.yaml
logger:
  default: warning
  logs:
    homeassistant.components.nest: debug
    homeassistant.setup: debug
    nest: debug
    sseclient: debug

Oh, by the way, whatever reason you got impression of

no longer defined in configuration.yaml

is wrong, you still need at least client_id and client_secret in your configuration.yaml

Interesting. I did a sudo systemctl restart home-assistant instead of using the Restart button in the configuration panel and its working now (with client_id and client_secret defined in configuration.yaml). Thanks!


Fixed pnpm path

Instead of the path always being set as /home/aelpxy it is now /home/$USER

LGTM.


Add Codecov

  - name: Codecov
    uses<EMAIL_ADDRESS>
Checklist:

[ ] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the master branch of freeCodeCamp.
[ ] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #XXXXX

Thanks a lot for your pull request. However, we are closing this at this time.


Feat(Progress): add progress component

Summary
The progress component with the options to be circular or linear was added
Task

https://dd360.atlassian.net/browse/CD-1072

Affected sections

exports.json
package.json
src/components/Progress/Progress.test.tsx
src/components/Progress/Progress.tsx
src/components/Progress/ProgressCircle.tsx
src/components/Progress/ProgressLine.tsx
src/components/Progress/index.ts
src/components/Progress/progress.css
src/components/index.ts
src/stories/Progress.stories.tsx

How did you test this change?

Manually
src/components/Progress/Progress.test.tsx

It looks very good! 👍


make DEFAULT_HANDLER an undefined symbol

fixes #58
I found out that this fixes #58 while writing the embedonomicon

@crawford could you give this a spin? I tested with the example code you posted in #58 but perhaps you should try this out with some more complex program.
cc @pftbest you may want to include this in msp430-rt

@japaric Sorry, but looks like this fix doesn't work for me. Are you sure it fixed the issue on your side? I still see that EXCEPTIONS array is wrong.

@japaric Gah! I didn't see your note since GitHub doesn't always send me email alerts... I tried this out on my application (rustc 1.27.0-nightly-7360d6dd6 with LLD) and it seems to work. It looks like @pftbest's patch might be more complete though.

lld may not have this issue, it is probably GNU ld specific.

@pftbest Uh oh. I'm not seeing the original issue anymore (even with GNU ld).

@pftbest I'm seeing the same behavior as you for the example code, but not for my application. My application seems to link properly. I wonder if it is because I don't have a direct dependency on cortex-m-rt.

Subsumed by #69


Replication

@sougou @enisoc @aaijazi (only 1 reviewer needed)
Adding 'vtctl VtGateExecute...' commands (only 2 of them for now) so we have a command line vtgate client. Will add all of them over time as needed.
That allows me to prove custom sharding works (with VtGateExecuteShard command), and finish that integration test.

LGTM

Is the new custom_sharding.py ready to be added to Makefile and test/config.json?


Roll Engine from 7f00e2f6c53c to 5ee903f51701 (2 revisions)

https://github.com/flutter/engine/compare/7f00e2f6c53c...5ee903f51701
2021-01-07<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from 5jKxFxRiQ... to qedaMC5nT... (flutter/engine#23484)
2021-01-07<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from de_114mvQ... to cLR0CUb-e... (flutter/engine#23482)
Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from 5jKxFxRiQQBZ to qedaMC5nTQFw
fuchsia/sdk/core/mac-amd64 from de_114mvQcHR to cLR0CUb-eK5h
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md

Commit queue failed; closing this roll.

Commit queue failed; closing this roll.


Compatibility neovim, open up a new line following the current line and add text there, delete current line

open neovim
open a file
go to any line
press o to open a new line following the current line
Moves the line down, hides the line number.
EDIT: Happens if you delete the current line as well

@denismaxim0v - I just merged a fix for this, could you please check?

@denismaxim0v - I just merged a fix for this, could you please check?

@denismaxim0v - I just merged a fix for this, could you please check?

All is perfect

@denismaxim0v - I just merged a fix for this, could you please check?

All is perfect


make ensureAROOperatorRunningDesiredVersion a condition

A race condition is present in the current code, this PR
will fix it my modifying the existing action to a condition
with a timeout of 20 mins.
Earlier PR: https://github.com/Azure/ARO-RP/pull/1882
A race condition between the JSON patch applying to the deployment, the m.aroDeploymentReady function returning true immediately as the older revision is still ready and running, and then m.ensureAROOperatorRunningDesiredVersion fetching the older deployment and seeing the version doesn't match.
Which issue this PR addresses:

Fixes
What this PR does / why we need it:

Test plan for issue:

Is there any documentation that needs to be updated for this PR?

Yet to test the changes, will share the results soon.

the same exact error message is seen here too: https://github.com/Azure/ARO-RP/pull/2318

I tested the changes., the code doesn't exit directly and waits for a timeout of 20 mins while retrying.

Written unit tests for the functions and made suggested changes. cc @bennerv

Ack, made suggested change and testing the PR now, will update results. Thank you.

Ack, made suggested change and testing the PR now, will update results. Thank you.

Manual Testing is done for the below scenarios.,
Created a dev cluster via local-RP.

ARO_IMAGE is set, but the image doesn't have a tag, PUCM succeeds and looks for desiredVersion == "latest"
ARO_IMAGE modified and the image has a tag, PUCM succeeds and checks for desiredVersion == "imageTag"
desiredVersion is hard coded to a random string (say unknown), PUCM fails at step ensureAROOperatorRunningDesiredVersion-a-condition after a timeout of 5 mins.

/azp run ci

Manual Testing is done for the below scenarios.,
I created a dev cluster via local-RP.

ARO_IMAGE is set, but the image doesn't have a tag, PUCM succeeds and looks for desiredVersion == "latest"
ARO_IMAGE modified and the image has a tag, PUCM succeeds and checks for desiredVersion == "imageTag"
desiredVersion is hardcoded to a random string (say unknown), PUCM fails at step ensureAROOperatorRunningDesiredVersion-a-condition after a timeout of 5 mins.
OperatorVersion is passed via Payload, PUCM succeeds and looks for desiredVersion == "OperatorVersion">
'{ "Properties": { "OperatorVersion": "v20220729.01" } }'

/azp run ci


feat: ink, ink sepolia

Changes

added metadata and icon for ink and inkSepolia
required because BX hasn't adopted the icons.badgeURL property yet

@welps Had to add this fallback icon until we fix the remote badge support on BX. I noticed the current badge URL from networks is also 404ing; do we need to amend that in CMS or will it get ingested here?

@welps Had to add this fallback icon until we fix the remote badge support on BX. I noticed the current badge URL from networks is also 404ing; do we need to amend that in CMS or will it get ingested here?

@DanielSinclair The badges in networks is v. different and is closer to what the app has in https://github.com/rainbow-me/rainbow/commit/583ef49e19c6eb72e872bca74761d55774687074#diff-82f3808901445061dc870baaa09ef3dbd1faf368f0b360cd87870dec4d259161


Local name cannot be null

Hi there
I get following exception using 'com.microsoft.ews-java-api:ews-java-api:2.0-SNAPSHOT' and Java 8:
at com.sun.xml.internal.stream.writers.XMLStreamWriterImpl.writeAttribute(XMLStreamWriterImpl.java:619)
at microsoft.exchange.webservices.data.WSSecurityBasedCredentials.emitExtraSoapHeaderNamespaceAliases(WSSecurityBasedCredentials.java:153)```

Your function emitExtraSoapHeaderNamespaceAliases actually does really call the function writeAttribute with null for the localName parameter.

  ```@Override
  protected void emitExtraSoapHeaderNamespaceAliases(XMLStreamWriter writer)
      throws XMLStreamException {
    writer.writeAttribute(
        "xmlns",
        EwsUtilities.WSSecuritySecExtNamespacePrefix,
        null,
        EwsUtilities.WSSecuritySecExtNamespace);
    writer.writeAttribute(
        "xmlns",
        EwsUtilities.WSAddressingNamespacePrefix,
        null,
        EwsUtilities.WSAddressingNamespace);
  }

@mibto I made a PR with a possible fix for your problem. Maybe you can have a short test (since I currently don't have an integration test for it)?


Change AccountDataSettings and UpsertAccountDataSettingsDto entities

Changes

Adds dataTypeId to AccountDataSetting. Otherwise, the clients cannot know the AccountDataType referenced by the AccountDataSetting.
Changes UpsertAccountDataSettingDto.id to UpsertAccountDataSettingDto.dataTypeId. The id property is referencing an AccountDataType instance, not an AccountDataSetting instance, so naming it just id could be confusing for clients.
Both AccountDataSetting.name and AccountDataSetting.description were also removed as these point to an AccountDataType instance, so again they can confuse. The name and description properties are already available for clients on the AccountDataType, so there's no need to include them on the AccountDataSetting.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

3 of 5 (60.0%)  changed or added relevant lines in 3 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.04%) to 48.274%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/datasources/accounts/accounts.datasource.ts
0
1
0.0%

src/routes/accounts/entities/account-data-setting.entity.ts
2
3
66.67%

Files with Coverage Reduction
New Missed Lines
%

src/routes/accounts/entities/account-data-setting.entity.ts
1
66.67%

Totals

Change from base Build<PHONE_NUMBER>:
0.04%

Covered Lines:
4282

Relevant Lines:
7143

💛  - Coveralls


Footer styling issue in BlazorUI demo in small sizes

this issue is resolved now, so I'm closing it.


Table Outlines?

I have been using TPPDF in an older project for ever with version 1.2.1. I am able to use outlines on my tables as well as other styles have a nice looking for.
I have started another project now and using the latest version, but can't for the life of me get table outlines to work. Is anybody else having this issue? I have made a super simple table and can change all other styles.

Hi @MatthewMerritt thanks for opening up this issue.
Please use the bug report issue template and provide me with the following details:

What did you do?
What did you expect to happen?
What happened instead?
TPPDF Environment

TPPDF version
Xcode version
Swift version

Demo Code

If you are able to show a screenshot what you expect vs what you got, that would be very helpful too. Thank you

Closed due to inactivity


update serde_derive_internal and test rename_all_fields

update serde_derive_internal and test rename_all_fields
rebased on @siefkenj's fork

Sorry it took so long!

Thanks!


Docs reorganization!

Don't merge yet!
Creating a PR for comments. Please add your thoughts.

My biggest problem with these changes is the "time to first snippet" is increased in exchange for organization and readability. I think perhaps the root README could include screenshots / simple snippet just for marketing purposes.

This change LGTM.  I am not worried about the time to first snippet change.
Only concern is a lack of description for the features on the front page.  Not sure why anyone would click on 'Database' in the table of contents since the concept of Database UI is not so obvious.

@abehaskins what's the status of this PR?

@abehaskins Any updates on this?

@abehaskins there's nearly no way you can resolve these merge conflicts in any sane way, things have moved a lot since you opened this.  I am gonna close this for repo health reasons, but I would still love to reorganize the docs if you want to.


feat(SPV-736) adding admin contacts methods

Pull Request Checklist

[x] 📖 I created my PR using provided  : CODE_STANDARDS
[x] 📖 I have read the short Code of Conduct: CODE_OF_CONDUCT
[x] 🏠 I tested my changes locally.
[x] ✅ I have provided tests for my changes.
[x] 📝 I have used conventional commits.
[ ] 📗 I have updated any related documentation.
[x] 💾 PR was issued based on the Github or Jira issue.

testing 🧪


GEODE-2569: Avoid unhelpful log message via a null check.

Since the default for "ack-severe-alert-threshold"
is zero, there is a code path where the target member
is null at the point of logging. So check for null.

Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[x] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[x] Is your initial contribution a single, squashed commit?

[x] Does gradlew build run cleanly?

[n/a] Have you written or updated unit tests to verify your changes?

[n/a] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

Someone else had addressed the same problem in a different file by just skipping the log message when it would use null, so I chose consistency.


MM-18659 Improve documentation for outgoing proxy and link previews

Adds more documentation on how to configure an outgoing proxy including answers to some questions we ran into when configuring the proxy for a few customers. I also added a link from the Enable Link Preview setting to the proxy documentation
Ticket Link
https://mattermost.atlassian.net/browse/MM-18659

Removing the request for review from @icelander since I discussed the changes with him offline


diff.hide fix

diff.hide in the yaml config updated to use "(un)modified" instead of "(un)changed"

I knew there was one more spot I missed... Thanks!


Joker syntax

Maybe should we drop * syntax? It's rarely useful, we can retrieve all nodes easily (even with nut, with , separators). So it seems to take useless bytes and performance...

Fixed with bea1ee2c50


chore: Update badges for username change

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add 'neovim-plugin' as topic to the GitHub repository

Add 'neovim-plugin' to Github repo for Dotfyle plugin search integration as seen in Neovimconf 2024

Hey @rib9985 really appreciate!


The archive may contain identical file names with different capitalization - database\cities\GBR/CON.json): Failed to open stream: Invalid argument

This is  a bug(windows).

Prerequisites

[x] Are you running the latest version?
[x] Are you reporting to the correct repository?
[x] Did you check the documentation?
[x] Did you perform a cursory search?

Description
Has any of you got this error on composer update:
 - Installing laravel-enso/addresses (4.6.17): Extracting archive
    Install of laravel-enso/addresses failed
  - Installing symfony/var-exporter (v5.3.7): Extracting archive
  - Installing symfony/cache-contracts (v2.4.0): Extracting archive
  - Installing symfony/cache (v5.3.7): Extracting archive
  - Installing squizlabs/php_codesniffer (3.6.0): Extracting archive
  - Installing slevomat/coding-standard (7.0.15): Extracting archive
  - Installing symfony/stopwatch (v5.3.4): Extracting archive
  - Installing symfony/filesystem (v5.3.4): Extracting archive
  - Installing seld/phar-utils (1.1.2): Extracting archive
  - Installing seld/jsonlint (1.8.3): Extracting archive
    0 [>---------------------------]

  [RuntimeException]
  The archive may contain identical file names with different capitalization (which fails on case insensitive filesystems): ZipArchive::extractTo(c:\myproject/vendor/composer/267b1069/laravel-enso-addresses-86d7466\
  database\cities\GBR/CON.json): Failed to open stream: Invalid argument

  [ErrorException]
  ZipArchive::extractTo(c:\myproject/vendor/composer/267b1069/laravel-enso-addresses-86d7466\database\cities\GBR/CON.json): Failed to open stream: Invalid argument

This happens on windows.
I've tried composer clearcache. Not working.

Update:
I've tried composer update from MINGW64 and got a warning (yellow), instead of red [RuntimeException]
Loading composer repositories with package information
Updating dependencies
Nothing to modify in lock file
Installing dependencies from lock file (including require-dev)
Package operations: 1 install, 0 updates, 0 removals
  - Installing laravel-enso/addresses (4.6.17): Extracting archive
    The archive may contain identical file names with different capitalization (which fails on case insensitive filesystems): ZipArchive::extractTo(C:\myproject/vendor/composer/e7344ef3/laravel-enso-addresses-86d7466\database\cities\GBR/CON.json): Failed to open stream: Invalid argument
    Unzip with ZipArchive class failed, falling back to unzip command
Package fzaninotto/faker is abandoned, you should avoid using it. No replacement was suggested.
Generating optimized autoload files
> Illuminate\Foundation\ComposerScripts::postAutoloadDump
> @php artisan package:discover --ansi
....

Update:
On git add I get this:
error: open("vendor/laravel-enso/addresses/database/cities/GBR/CON.json"): No such file or directory
error: unable to index file 'vendor/laravel-enso/addresses/database/cities/GBR/CON.json'
fatal: adding files failed

Actual Windows issue:
https://superuser.com/questions/86999/why-cant-i-name-a-folder-or-file-con-in-windows/844514#:~:text=CON is a reserved name,%2C NUL %2C LPT1 and others.&text=To delete the folder%2C you,else it won't work.&text=My advice though is to just use a different name.
CON is a reserved name in Windows. So are PRN , AUX , NUL , LPT1 and others. To delete the folder, you have to reference it the same way, or else it won't work. My advice though is to just use a different name.
So... @GITmanuela if possible to change this filename to something more "Windows-friendly" will be much appreciated for all our MS users:)

Hello, composer update should fix the issue


Fixes #5 - block now delivers complex stream

@cjam

Conflicts are turd-o. Closing and retrying.

Okay @cjam it's ready for you.


[Bug]: 404 page not found

Error Message and Logs
This issue still exists https://github.com/coollabsio/coolify/issues/3528
I have pointed all to the same IP.
Autogenerated *.sslip.io works, my coolify.example.com means the main portal domain works but any other domain I assign to other services throws 404 error.
Steps to Reproduce

Add fqdn to a service

Example Repository URL
Templates: Dashy, portainer, File Browser
Coolify Version
v4.0.0-beta.373
Are you using Coolify Cloud?
No (self-hosted)
Operating System and Version (self-hosted)
Ubuntu 24.04
Additional Information
No response

Coolify UI was not meant to work on subdomains I guess, so it needs the @ of the root domain to point at least once and then restart the proxy. It's possible to cache it in the proxy server.
I have tried many permutations and combinations, and finally, this worked.


Add more content on configurations

Context
See https://github.com/gradle/dotorg-docs/issues/179
The terms "configuration" and "configurations" is among the top 20 user-based search terms. This change adds more content on certain aspects around configurations.

Cross-reference user guide from DSL guide/Javadocs. (Those links may easily break in the future. There's no automation around verifying the existence or pass in a specific version of the user guide.)
More references to API from user guide
Explain the concept of configuration inheritance
More cross-references to the basic "configuration" information
Additional use cases in "Working with Dependencies" chapter. More complex code samples use the concrete types of the API to give a better hint to the referenced classes.

I also like the cross-referencing from the javadocs. Can we create a follow up issue to establish some automating link testing in the future?

https://github.com/gradle/dotorg-docs/issues/186


Object Detector training loss = NaN

Hello, can you please suggest reasons that would make the value of the loss to start becoming NaN?
>>> object_detector = tc.object_detector.create(test_data_392, annotations='annotations',feature='image',max_iterations=500,verbose=True)
2018-04-26 00:50:15  Training   1/500  Loss  6.347
2018-04-26 00:50:30  Training   2/500  Loss  6.344
2018-04-26 00:50:44  Training   3/500  Loss  6.265
2018-04-26 00:50:56  Training   4/500  Loss    nan
2018-04-26 00:51:08  Training   5/500  Loss    nan
2018-04-26 00:51:19  Training   6/500  Loss    nan
2018-04-26 00:51:35  Training   7/500  Loss    nan
2018-04-26 00:51:46  Training   8/500  Loss    nan
2018-04-26 00:51:58  Training   9/500  Loss    nan
2018-04-26 00:52:09  Training  10/500  Loss    nan
2018-04-26 00:52:20  Training  11/500  Loss    nan
2018-04-26 00:52:31  Training  12/500  Loss    nan

@gustavla I'm guessing the loss should never be nan. Seems like a bug?

I have capped the same annotations to 43 and nan is not appearing:
>>> object_detector = tc.object_detector.create(test_data_43)
Using 'image' as feature column
Using 'annotations' as annotations column
2018-04-26 10:45:50  Training    1/1000  Loss  6.893
2018-04-26 10:46:01  Training    2/1000  Loss  6.828
2018-04-26 10:46:13  Training    3/1000  Loss  6.749
2018-04-26 10:46:24  Training    4/1000  Loss  6.648
2018-04-26 10:46:36  Training    5/1000  Loss  6.589
2018-04-26 10:46:48  Training    6/1000  Loss  6.472
2018-04-26 10:47:00  Training    7/1000  Loss  6.441
2018-04-26 10:47:11  Training    8/1000  Loss  6.374
2018-04-26 10:47:23  Training    9/1000  Loss  6.326
2018-04-26 10:47:34  Training   10/1000  Loss  6.244
2018-04-26 10:47:46  Training   11/1000  Loss  6.221
2018-04-26 10:47:57  Training   12/1000  Loss  6.206
2018-04-26 10:48:09  Training   13/1000  Loss  6.229

@andremontenegrof Can you share your dataset by any chance. It seems like the divergence is a property of your dataset and we'd like to be able to trace it down.

@srikris can you please download it from: https://www.dropbox.com/sh/v8659d7e2unwa35/AADg_4hr5hMM8fxdvu6MTDoMa?dl=0

Hey @srikris please let me know if you have trouble downloading the dataset. Thanks!

@gustavla @srikris Sorry, I just found out I had some negative heights and widths.
I fixed that and the gradient is not exploding anymore.
I assumed some behaviour in the tool I used for annotation and this was the result :S

@gustavla We should consider skipping data with bad bounding boxes.

Yes, we should definitely handle this better (I thought we did, but I don't see it anywhere in the code). Thanks for reporting this @andremontenegrof! I'll work on a fix.

Hey! I believe raising exception is better than silently skipping. For example, it is always great if we have the program telling us that the data in row 142 is invalid.

@andremontenegrof Perhaps an eye-catching warning would be the way to go? Since this is a recoverable issue and the user has potentially been training for hundreds of iterations (or even more), then an exception caused by a single bad sample could be really frustrating.
For this to be effective though, we should probably have a mechanism that tracks warnings and then re-reports them or at least notifies the user to scroll up to read them once training completes. It doesn't matter how eye-catching it is if the user leaves the computer to let it run and it produces enough valid output after the warning to completely miss it.

@znation I like that idea!

Indeed an exception would only make sense in the beginning. To present a set of warnings at the end is also an elegant solution. Thank you!


Insanely slow dev experience

Vscode is getting that slow that I can't work anymore. Meaning I am experiencing a typing delay of +1.0 seconds. The more you type the bigger the delay becomes.
It's the Code Helper Renderer if I look at the activity monitor that runs crazy.
Does this issue occur when all extensions are disabled?: Yes. I suspected that the Vim-plugin would be the problem, but with every plugin disabled the problem still exists.

VS Code Version:  1.57.1 I also downgraded to version 1.56.2 but same bug.
 I am working on a next.js app and create-react-app with typescript.
I just did a test and wrote some lines in a .txt file. Which works fine.
Did another test with using vim and .mdx which also works fine.
So I get the impression that something is going on with tsserver.js an server.js. Any ideas? Even the Settings lag, if I am inside a typescript/javscript project.
Steps to Reproduce:
I honestly don't know how to reproduce it.
Any other tips I can try?

Does this reproduce in the latest VS Code insiders build with all extensions disabled?
Also does it happen in all projects or just one in particular?

Hey,
I didn't try to fix it anymore but I had the problem with all extensions disabled. What I think was the problem, the temperature here in Austria got to high (+30C) and the macbook couldn't handle it anymore. Which is a bit wired as it is a mac pro 2.6GH 6-core intel core i7 with 32 gig ram.
The problem is kinda gone if it's not to warm. 🙄

Were other apps similarly slow when this was happening?

No.

If you are still seeing the issue, please collect the TS Server log:

Set "typescript.tsserver.log": "verbose",
Restart VS Code and reproduce the problem
In VS Code, run the TypeScript: Open TS Server log command
This should open a large log file called tsserver.log

⚠Warning: The TypeScript log may include information from your workspace, including file paths and source code. If you have any concerns about posting this publicly on Github, just let me know and we can arrange something else. On our side, we only use these logs to investigate issues like this


Misc Changes

Cherry pick
Doc update

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: bpradipt
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [bpradipt]

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

@bpradipt: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

pull-ocp4-upi-powervs-terraform-validate
647d0d4402914cc43c13ac0ed11e76df3b04545c
link
/test pull-ocp4-upi-powervs-terraform-validate

Full PR test history. Your PR dashboard.

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

@yussufsh any idea why this flaky error:
Connecting to github.com (<IP_ADDRESS>:443)
Connecting to github-production-release-asset-2e65be.s3.amazonaws.com (<IP_ADDRESS>:443)
saving to '/root/.local/share/terraform/plugins/registry.terraform.io/terraform-providers/ignition/terraform-provider-ignition_2.1.0_linux_amd64.zip'
terraform-provider-i   0% |                                | 16360  0:08:24 ETA
terraform-provider-i 100% |********************************| 7966k  0:00:00 ETA
'/root/.local/share/terraform/plugins/registry.terraform.io/terraform-providers/ignition/terraform-provider-ignition_2.1.0_linux_amd64.zip' saved
Initializing modules...
- install in modules/5_install
- nodes in modules/4_nodes
- prepare in modules/1_prepare
There are some problems with the configuration, described below.
The Terraform configuration must be valid before initialization so that
Terraform can determine which modules and providers need to be installed.
Error: Invalid reference from destroy provisioner
  on modules/1_prepare/prepare.tf line 93, in resource "ibm_pi_instance" "bastion":
  93:             user        = var.rhel_username
Destroy-time provisioners and their connection configurations may only
reference attributes of the related resource, via 'self', 'count.index', or
'each.key'.
References to other resources during the destroy phase can cause dependency
cycles and interact poorly with create_before_destroy.
Error: Invalid reference from destroy provisioner
  on modules/1_prepare/prepare.tf line 95, in resource "ibm_pi_instance" "bastion":
  95:             private_key = var.private_key
Destroy-time provisioners and their connection configurations may only
reference attributes of the related resource, via 'self', 'count.index', or
'each.key'.
References to other resources during the destroy phase can cause dependency
cycles and interact poorly with create_before_destroy.
Error: Invalid reference from destroy provisioner
  on modules/1_prepare/prepare.tf line 96, in resource "ibm_pi_instance" "bastion":
  96:             agent       = var.ssh_agent
Destroy-time provisioners and their connection configurations may only
reference attributes of the related resource, via 'self', 'count.index', or
'each.key'.
References to other resources during the destroy phase can cause dependency
cycles and interact poorly with create_before_destroy.

@yussufsh any idea why this flaky error:

TF 13 with master branch
TF 12 with release-4.5
I always do rm -rf .terraform to reconfigure TF plugins. Or best would be to have different directories for master and release-4.5.
No other option here since TF versions are not backported.

@mkumatag I am not so expert in PROW jobs yet. If possible can you do TF 12 for release-4.5 branch and what we have now will only run on master and other future branches.

@mkumatag I am not so expert in PROW jobs yet. If possible can you do TF 12 for release-4.5 branch and what we have now will only run on master and other future branches.

got it.! fix it..

/lgtm

/cc @Prajyot-Parab

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: bpradipt, Prajyot-Parab
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Prajyot-Parab,bpradipt]

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

/cc @yussufsh

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: bpradipt, Prajyot-Parab, yussufsh
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Prajyot-Parab,bpradipt,yussufsh]

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: bpradipt, Prajyot-Parab, yussufsh
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Prajyot-Parab,bpradipt,yussufsh]

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


Bug fix in thre3d_reprs/reders.py

Hi, I wanted to report a bug.
I ran the train_sh_based_voxel_grid_with_posed_images.py script and found this error:
sample_aabb_bound_uniform_points_on_rays() got an unexpected keyword argument 'linear_disparity_sampling'.
It looks like at line 71 of thre3d_reprs/reders.py the kwarg linear_disparity_sampling is being passed to the sample_aabb_bound_uniform_points_on_rays even if it does not accept it.
I removed that line and the code runs flawlessly.

Thank you very much for this, we have deleted the line in mention from our repo.


[BugRecord] Kernel build failed: linux/netfilter/xt_connmark.h not found

Problem Summary
I initially pushed the source kernel code from my Windows 11 PC to my repository by extracting a tar package of the code and uploading it. However, when I cloned the repository on my Ubuntu machine and tried to build the kernel, I encountered the following error:
net/netfilter/xt_connmark.c:28:41: fatal error: linux/netfilter/xt_connmark.h: No such file or directory
compilation terminated.
  LD      net/netfilter/nf_nat.o
scripts/Makefile.build:258: recipe for target 'net/netfilter/xt_connmark.o' failed
make[2]: *** [net/netfilter/xt_connmark.o] Error 1
make[2]: *** Waiting for unfinished jobs....
  CC      net/netfilter/ipvs/ip_vs_nfct.o
  CC      net/netfilter/ipvs/ip_vs_ftp.o
net/netfilter/xt_hl.c:19:42: fatal error: linux/netfilter_ipv4/ipt_ttl.h: No such file or directory
compilation terminated.

Solution Investigation
After several attempts to resolve this issue, I traced the problem back to a behavior specific to Windows. When I extracted the source code on my Windows machine, Windows automatically removed the xt_connmark.h file. This happened because Windows couldn't differentiate between xt_connmark.h and xt_CONNMARK.h due to case-insensitive file system handling. As a result, Windows discarded one of the files, leading to the compilation error.
Conclusion
The solution is to never unzip the code on a Windows machine. Instead, I need to extract the source code on a Linux machine (like Ubuntu), push it to my remote repository, and clone it again on Ubuntu. This avoids the case-sensitivity issue in Windows and allows the kernel to build properly.

repo: https://github.com/quinnwencn/tqimx6q-bsp


File direct download icon

Download files directly via download button in the listing view.
Discussed in https://github.com/spencerwooo/onedrive-vercel-index/discussions/49

Originally posted by cgmark101 August 15, 2021
Hi man, thanks for this awesome tool. Is there a way to direct download a video without redirecting to the embeded player?

Yes, but the button show direct url onedrive for download multimedia. I like can download multimedia with links of indexator, not from onedrive.


UrlGeneratorServiceProvider in Silex 2.0 dev

I'm testing SIlex version 2.0 dev.
I can see that UrlGeneratorServiceProvider not exist in this version. What alternative can I use, for example in my Twig templates for using path() function ?
Thanks

If you have the RoutingServiceProvider and TwigServiceProvider enabled it should work.

My error was related with a bad named route in bind calling. Now it works
thanks


add method to get gsp and dno automatically

Pull Request
Description

tidy up
automatically get gsp and dno, this is done by storing gsp and dno data in the code.

How Has This Been Tested?

added test

run locally

[x] Yes

Checklist:

[x] My code follows OCF's coding style guidelines
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have checked my code and corrected any misspellings

hi @peterdudfield, the tests have all passed for me, but when I run the code locally and put nothing into any of the fields,
I get this error:


Cannot always select item when clicking on it

I have a seemingly random problem - when I click on an item SOMETIMES it selects and sometimes it does not. This is without searching or typeing anything - just using the dropdown.
I have 14 items in my dropdown:

Note: This is a even worse on ipad and mobile devices - you literally cannot use the control

I am having the same issue. I have to use the enter button in order to make it work. It's making my users think that the website is unresponsive.

I'm now looking at other controls - it essentially means this control is unusable unfortuantely.

The last thing you do is try using event.stopImmediatePropagation() in your select method. However this means you need to pass in the $event object from the select event to make the effect work in browsers with no global event object, such as Firefox.
As I said, I am having the same problem, so I'm really just grasping at straws here.

Out of interest, are you using "encapsulation: ViewEncapsulation.None  // Enable dynamic HTML styles" or standard text values?

I noticed that the demo used an inline CSS style whereas mine uses an external stylesheet. I removed this completely and now it seems to work.

I'm using standard text values. And the idea of using the inline CSS might help me out.

Hmmm that's interesting that you are using standard test. But yes, try and inline any styles used by that component and see if it makes a difference. It seems to have helped in my case. I can't find any other dropdown component that lets you do styled templating

Ok, I found some more bugs - on ipad the scroll bar is not usable and my workaround above does not always work. I'm going to have to find a more supported component.

I'm seeing what I think is the same issue.  Clicking on the text in the dropdown rarely calls my (select) method.  clicking in whitespace outside of the text in the dropdown (but still in the item) calls my select method.  Any ideas?

@timmarks I encounter exactly the same problem. I tried to play with z-index it still not work.

Hi,
I've look up your issue and I find a trick.
Add in your css .dropdown-item > div {      pointer-events: none;    }
It'll ignore the div inside the a tag and the click event work great.

In addition to @khylias answer, if you can't open the dropdown list too, just add .ui-select-match-text to the selectors of the above css rule.

I faced the same issue. How I solved:
timeArray = new Array();
var temp = 00:00; //loop for pushing every 30 mins for 24 hours 00:00 to 23:30
this.timeArray.push(''+temp+'');

This is the html part:
<ng-select [items]="timeArray"  [multiple]=false placeholder="Start Time"></ng-select>
When I push in the array the list which I have to show, I put the content in a span with pointer events none, this way I could fix it in mozilla.

The suggested solution with '+temp+' breaks UI experience on EDGE =)


Separate build and validateAndBuild method in DataSourceMetadata

Description

Separate build and validateAndBuild method in DataSourceMetadata to let internally build special DataSourceMetadata.
The validation doesn't allow datasource name starting from _(underscore)

Issues Resolved
n/a
Check List

[x] New functionality includes testing.

[x] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Separate build and validateAndBuild method in DataSourceMetadata to let internally build special DataSourceMetadata.

Is it feasible to apply different set of validation rules on another DataSourceMetadata implementation?

Separate build and validateAndBuild method in DataSourceMetadata to let internally build special DataSourceMetadata.

Is it feasible to apply different set of validation rules on another DataSourceMetadata implementation? (With or without this PR)

Any additional validation can be done when datasource is created.

What is the use case that is driving this change?

What is the use case that is driving this change?

When try to instantiate DataSourceMetadata with _xxxxx (starting from underscore), it was failing and we couldn't instantiate DataSourceMetadata with such special value.

What is the use case that is driving this change?

When try to instantiate DataSourceMetadata with _xxxxx (starting from underscore), it was failing and we couldn't instantiate DataSourceMetadata with such special value.

Is this for testing?


Pipeline Producer overflowed...

As an experiment, I tried running the beta UR ROS2 driver on an Nvidia Xavier AGX (ARM processor), both in a docker container as well as standalone. In both cases, I get a runtime warning that "no realtime capabilities found", and then once the external node on the robot starts up, I get an alternating " Pipeline producer overflowed!" from line 412 of Universal_Robots_Client_Library/include/ur_client_library/comm/pipeline.h followed by an attempt to reconnect, and then the overflow and so on. Note, I have also seen the overflow error on my Linux machine, but it is much rarer.
I originally posted it as a  URCLientLibrary issue #59, and got this response from Felix, so I am filing this issue:

This is indeed a problem with the ROS2 driver.
The client library stores packages received from the robot inside a queue which has to be emptied (consumed) by a second party. If the queue fills up without being emptied, the mentioned warning is printed.
It is the driver's job to empty the queue, which should happen here. I'll close this, as it looks like a ROS2 driver issue. Please open an issue inside the ROS2 driver repo linking to this issue. If any news concerning the library come up hunting this down, feel free to comment or reopen this one.

I encountered this problem recently while fixing the GPIOs. Here we wait for the response of the client library in a different thread so we can return the correct success flag to the service call. In the old version occurred some errors which lead to an endless waiting which in turn prevented the pipeline from being cleared. Could you try it again with the current version and verify that the error still occurs @bblumberg

I successfully ran the ROS2 driver in a docker container on my nvidia xavier agx.  It seems like the issue is still present, though.  For example, if I bring up the driver in a terminal, ctrl z and then run it in the background, I get the unrecoverable overflow error.  By contrast, when I brought it up in one terminal, and then switched to another terminal to run the demo it all worked. Clearly, the latter way of bringing up the demo is the preferred way, but it seems like the system should be robust enough to deal with the interruption caused by the first way of running the system. I am also using a realtime kernel now which might be helping as well.

Please check the latest main now when  #281 is merged.

Closing this since there have been changes tackling this and it hasn't shown up recently.


Optional dependencies for data downloaders

Relates to #106 and #102

Adds some more detail to the user guide's data processor page and docstrings for two data downloaders on them needing optional dependencies (or more specifically, non PyPI compatible dependencies for get_station_data).
Adds a check into get_earthenv_auxiliary_data that rioxarray is installed before downloading the data.
For good measure, adds the sphinx todo exension for displaying todo directives in docstrings.

This is great, thanks @davidwilby! I wasn't aware of importlib.util.find_spec. With the get-station-data import check I just tried to catch an ImportError, but importlib is probably more principled.
One thing before LGTM, I realised the user will bump into this error in the Data Requirements notebook before the DataProcessor one. Would you mind moving your note to that page? For example, just after the Data Sources paragraph.
Also left a comment on the paragraph you added that needs resolving.

This is great, thanks @davidwilby! I wasn't aware of importlib.util.find_spec. With the get-station-data import check I just tried to catch an ImportError, but importlib is probably more principled.

That would be my usual approach but my thinking here was that since deepsensor doesn't import rioxarray directly (only via xarray), it would avoid a sort of needless import. But we end up just importing importlib instead, maybe this is just fancier :shrug:
Arguably, using try/except and catching ImportError is more usual and possibly more readable. But I'd rather import a standard library module and barely use it than import a third-party dependency and not use it at all, since it will be more liable to possible instability/weirdness in the future.

One thing before LGTM, I realised the user will bump into this error in the Data Requirements notebook before the DataProcessor one. Would you mind moving your note to that page? For example, just after the Data Sources paragraph.

Absolutely, will do!

Also left a comment on the paragraph you added that needs resolving.

No problem, agreed.

LGTM, thanks very much for these doc improvements @davidwilby!


Vulnerability roundup 48 (release-18.03)

Scanned nixos/release-combined.nix @ 862fb52. Filtered out previously reported CVEs. May contain false positives.
nmap-7.70 (search, files)

[x] CVE-2018-15173

tetex-3.0 (search, files)

[ ] CVE-2005-3624
[ ] CVE-2005-3625
[ ] CVE-2005-3626

Cc: @joepie91, @phanimahesh, @the-kenny, @7c6f434c, @k0001, @peterhoeg, @nh2, @LnL7, @grahamc, @adisbladis, @fpletz, @vcunat, @andir, @sphalerite
Contact @ckauhaus for any questions.

master is #48051

See my comment about nmap: https://github.com/NixOS/nixpkgs/issues/48051#issuecomment-428499646


temp 2

https://github.com/hyper63/journal/blob/tillathehun0/ideas-id-id-migration-12/ideas/04-id->_id-migration.md


Getting  token occasionaly in output

Hello team,
Thanks for building this wonderful open-source project. I sometimes notice that output is returned with  tokens.
I got this when translating an Malayalam article to English.

The results are not deterministic always as well. Sometimes it happens with  tokens and sometimes it doesn't work. How can I get deterministic results always?

Hey @kurianbenoy
Can you share details about the hyperparameters used for decoding along with commands?

Hey @jaygala24 ,
This was the code I used for inference and as you can see below in model generate, I used hyperparameters such as min_length, max_length, num_beams, num_return_sequences. Is there any parameter to ensure that I get deterministic output always?
def batch_translate(input_sentences, src_lang, tgt_lang, model, tokenizer, ip, device):
    import torch
    BATCH_SIZE = 16

    translations = []
    for i in range(0, len(input_sentences), BATCH_SIZE):
        batch = input_sentences[i : i + BATCH_SIZE]

        # Preprocess the batch and extract entity mappings
        batch = ip.preprocess_batch(batch, src_lang=src_lang, tgt_lang=tgt_lang)

        # Tokenize the batch and generate input encodings
        inputs = tokenizer(
            batch,
            src=True,
            truncation=True,
            padding="longest",
            return_tensors="pt",
            return_attention_mask=True,
        ).to(device)

        # Generate translations using the model
        with torch.no_grad():
            generated_tokens = model.generate(
                **inputs,
                use_cache=True,
                min_length=0,
                max_length=256,
                num_beams=5,
                num_return_sequences=1,
            )

        # Decode the generated tokens into text
        generated_tokens = tokenizer.batch_decode(generated_tokens.detach().cpu().tolist(), src=False)

        # Postprocess the translations, including entity replacement
        translations += ip.postprocess_batch(generated_tokens, lang=tgt_lang)

        del inputs
        torch.cuda.empty_cache()

    return translations

@kurianbenoy
This is straight from the example script and doesn't help us debug the issue.
Please provide the following basic details to proceed further.

Input text (plain text format)
Translation generated by the model (plain text format)
Model that is being used and details about quantization.

The outputs are indeed deterministic if you don't do sampling, you can try re-running the example any number of times, you will always get the same outputs.
Also, looking at your image, it seems you are trying to translate a large chunk of text using batch_translate.
Note that IndicTrans2 is trained with a max_seq_len of 256 tokens and it is not surprising that for such long passages, the outputs would obviously be suboptimal.
In such cases, the right way to do translations is to use any sentence splitter of your choice, segment the paragraph into constituent batch of sentences, use batch translate on the batch of sentences and rejoin the translated batch of sentences to once again reconstruct the paragraph.
You can find an example script that does this here.

Hello @PranjalChitale, I will try to use sentence splitter of my choice and pass results as chunks of sentence.
I am doubtful about the deterministic nature of translation output you suggested. Please check the outputs of input text I give, and why I say that way.

Please provide the following basic details to proceed further.

Input text (plain text format)

This article is from a leading news provider in Kerala. PFA article:
https://gist.github.com/kurianbenoy/9b12b76f7aebab3b2c9e684999f8bc05

Translation generated by the model (plain text format)

Model that is being used and details about quantization.

indic_en_ckpt_dir = "ai4bharat/indictrans2-indic-en-1B"  # ai4bharat/indictrans2-en-indic-dist-200M
    qconfig = None
    model = AutoModelForSeq2SeqLM.from_pretrained(
        indic_en_ckpt_dir,
        trust_remote_code=True,
        low_cpu_mem_usage=True,
        quantization_config=qconfig,
        revision="74a49d37c8bfd517d2cc04db8cd1baf8cef1fe7a"
    )

Hi @kurianbenoy
I was able to translate the article you shared in this issue correctly when fragmented at the sentence level. I would like to point out that the IndicTrans2 models are trained with sentence-level translation pairs so it is likely that the model may produce unexpected translations when passing the entire paragraph as the input. I have modified the example.py (link) script to also support paragraph-level translations in the latest commit.
Regarding the deterministic nature of the translations:
I am able to reproduce the translations with the same article as input for multiple runs. You can refer to the translation in output.txt attached to this reply. I suspect that you are using sampling-based decoding which might possibly lead to non-deterministic outputs.
output.txt
One difference I notice in your model loading is the use of the revision argument. Please try once with removing the following argument.

Hi @jaygala24, thanks for updating the example.py so it supports paragraph-level translations.

One difference I notice in your model loading is the use of the revision argument. Please try once with removing the following argument.

I can try that as well. Yet I do that for stability, so it is pointed to correct git commit in huggingface folder. If you update the model repository in huggingface, with revision argument it won't be updated. While without revision argument, it takes latest commit from model repository.

Yes, I know that revision points out to the correct git commit in huggingface folder. However, I would like to inform you that we won't be releasing any updates to the IndicTrans2 checkpoints on Huggingface.
I suspect that one of the following reasons might be behind the occurrence of <unk> tokens in the translations:

Issues with the fragmentation of the paragraph into sentences
Use of sampling-based decoding instead of deterministic decoding methods
Use of models with quantization

It looks like the issue has been resolved based on your recent response, so I'll close this issue for now. Feel free to re-open the issues in case of any further queries. Thank you!


Craft 4 Support

I would love to see this plugin come to Craft 4. Are their any plans to update for Craft 4?

Just to chime in, we've added this to our Craft 3 migration project, as people are keen to see this plugin on Craft 4.
If you're no longer developing this plugin, we'd be happy to continue development at Verbb (including updating to Craft 4) - but just let us know if you've got plans to continue things and update to Craft 4! @moacode

Hey @engram-design, thanks for this. Unfortunately I'm not really part of the Craft world these days due to do a change in job (😢) - Mainly Laravel, React or blockchain nowadays.
I have good intentions to port it but the reality is that I probably won't get to it any time soon.
If you would like to take it over and continue development then that is absolutely fine by me—Let me know what you need.
Apologies @richhayler for not responding to your comment in a timely manner.

@moacode Completely understand! If you want to transfer this repository to verbb or myself engram-design (you might have to add me as a collaborator first, then promote to admin), we'd be happy to continue development!
And you would be more than welcome to take this back in the future if you'd like to continue on with this. We're just trying to give people a hand where we can, but not looking to rip plugins away from people forcibly! 🙂

Just wanted to add my vote for a craft 4 upgrade. I really love this plugin. But it holds me back for upgrading. I do understand that you're not really a part of the craft community anymore. But maybe you can find a solution with an extra contributor? Best of luck on your change of job!

Still keen to convert this over, just waiting for @moacode to get back to me about transferring the repository.
Otherwise, might have to fork it and start another repo, it'd just be good to keep the issues and commit history for the plugin.

Hey @engram-design, sorry for the delay. I tried to transfer it to Verbb but was unable to. I've added you as a collaborator 😄

@moacode No problem! I've just accepted and I think you can promote me to an admin. Transferring directly to Verbb is only possible if it's from another organisation account I believe, so transferring to me personally should do the tricky. Or inviting me as a collaborator, and promoting to admin.
Sorry for the hassle!

@moacode Just wanted to check if you're able to transfer this repo to engram-design and I can take it from here?

+1 for getting this upgraded to Craft 4

@moacode Any luck getting me setup as an admin on the repo to kick off the transfer?

Hey @engram-design,
I had trouble when I attempted this last, but I just tried again and a request should have gone through to you now 😄

Thanks @moacode that’s sorted it!

Added in 3.0.0.
Do note however that the plugin has changed to verbb/prism.
To migrate, run the following:
php craft plugin/uninstall craft-prism-syntax-highlighting
composer require verbb/prism && composer remove thejoshsmith/craft-prism-syntax-highlighting
php craft plugin/install prism

The new plugin will migrate your fields across.


Create helpers to render components

Rendering simple components such as Primer::HeadingComponent is verbose and not user friendly.
e.g:
<%= render Primer::HeadingComponent.new(tag: :h1) { "My Heading" } %>

We should provide helpers to simplify the API, something like:
<%= primer_heading(:h1, "My Heading") %>

Related https://github.com/primer/view_components/pull/58

closing because of https://github.com/primer/view_components/pull/260
For the moment, we want to experiment only with primer(:octiconand primer(:heading and see how it feels


Universal Surround Audio

Is your feature request related to a problem? Please describe.
Surround audio incompatible with current audio receiver. Would like to keep more modern surround formats as well as a new compatible one.
Describe the solution you'd like
Similar to Universal Audio function to generate a stereo track, but this would generate surround track if there wasn't one in the selected codec.
Describe alternatives you've considered
I've worked on this some my own in #1681 and as far as I can figure out this would require main source code changes (I don't think I can override with existing options using validate custom function)

How would this be different than the base functionality that already handles the stereo tracks?

I have different acceptable codecs for stereo or surround.
Stereo: aac, ac3
Surround: ac3
I have currently found no way to handle all these options, generating only 1 new AC3 surround track (if one does not exist) and keep all original audio tracks. Universal Audio seems like it does exactly that but just for stereo.

The audio copy original option is what you're looking I think
Does exactly what you're describing
Define your stereo codecs in universal
Define your surround codecs in regular audio options
Turn on copy original if you always want the original track preserved

I don't want additional stereo tracks if none already exist. Just to preserve the existing stereo tracks.
autoprocess.ini
[Audio]
codec = ac3
languages =
default-language =
include-original-language = True
first-stream-of-language = False
channel-bitrate = 128
variable-bitrate = 0
max-bitrate = 0
max-channels = 0
filter =
profile =
force-filter = False
sample-rates =
sample-format =
copy-original = True
aac-adtstoasc = False
ignored-dispositions =
force-default = False
unique-dispositions = False
stream-codec-combinations =

[Universal Audio]
codec = aac, ac3
channel-bitrate = 128
variable-bitrate = 0
first-stream-only = False
filter =
profile =
force-filter = False

Test cases:
Input
        "audio": [
            {
                "index": 1,
                "codec": "aac",
                "bitrate": 224000,
                "channels": 6,
                "samplerate": 48000,
                "language": "eng",
                "disposition": "+default-dub-original-comment-lyrics-karaoke-forced-hearing_impaired-visual_impaired-captions"
            }
        ],

Output - includes Universal Audio, naturally, but not one I want.
        "audio": [
            {
                "map": 1,
                "codec": "ac3",
                "channels": 6,
                "bitrate": 768,
                "profile": null,
                "quality": 0,
                "filter": null,
                "samplerate": null,
                "sampleformat": "",
                "language": "eng",
                "disposition": "+default-dub-original-comment-lyrics-karaoke-forced-hearing_impaired-visual_impaired-captions",
                "bsf": null,
                "debug": "audio",
                "title": "5.1 Channel"
            },
            {
                "map": 1,
                "codec": "copy",
                "bitrate": 224.0,
                "channels": 6,
                "language": "eng",
                "disposition": "-default-dub-original-comment-lyrics-karaoke-forced-hearing_impaired-visual_impaired-captions",
                "debug": "audio-copy-original",
                "title": "5.1 Channel"
            },
            {
                "map": 1,
                "codec": "aac",
                "channels": 2,
                "bitrate": 256,
                "quality": 0,
                "profile": null,
                "samplerate": null,
                "sampleformat": "",
                "filter": null,
                "language": "eng",
                "disposition": "-default-dub-original-comment-lyrics-karaoke-forced-hearing_impaired-visual_impaired-captions",
                "debug": "universal-audio",
                "title": "Stereo"
            }
        ],

Input
        "audio": [
            {
                "index": 1,
                "codec": "ac3",
                "bitrate": 256000,
                "channels": 6,
                "samplerate": 48000,
                "language": "eng",
                "disposition": "+default-dub-original-comment-lyrics-karaoke-forced-hearing_impaired-visual_impaired-captions"
            },
            {
                "index": 2,
                "codec": "aac",
                "bitrate": 128001,
                "channels": 2,
                "samplerate": 48000,
                "language": "eng",
                "disposition": "-default-dub-original-comment-lyrics-karaoke-forced-hearing_impaired-visual_impaired-captions"
            }
        ],

Output - Creates UA where it doesn't seem to be needed.
        "audio": [
            {
                "map": 1,
                "codec": "copy",
                "channels": 6,
                "bitrate": 768,
                "profile": null,
                "quality": 0,
                "filter": null,
                "samplerate": null,
                "sampleformat": "",
                "language": "eng",
                "disposition": "+default-dub-original-comment-lyrics-karaoke-forced-hearing_impaired-visual_impaired-captions",
                "bsf": null,
                "debug": "audio",
                "title": ""
            },
            {
                "map": 1,
                "codec": "aac",
                "channels": 2,
                "bitrate": 256,
                "quality": 0,
                "profile": null,
                "samplerate": null,
                "sampleformat": "",
                "filter": null,
                "language": "eng",
                "disposition": "-default-dub-original-comment-lyrics-karaoke-forced-hearing_impaired-visual_impaired-captions",
                "debug": "universal-audio",
                "title": ""
            },
            {
                "map": 2,
                "codec": "copy",
                "channels": 2,
                "bitrate": 256,
                "profile": null,
                "quality": 0,
                "filter": null,
                "samplerate": null,
                "sampleformat": "",
                "language": "eng",
                "disposition": "-default-dub-original-comment-lyrics-karaoke-forced-hearing_impaired-visual_impaired-captions",
                "bsf": null,
                "debug": "universal-audio",
                "title": ""
            }
        ],

So it seems like you just want to adjust your codec pool just based on the number of channels, this should be pretty easy with a custom function
skipStream is run at the start of processing each stream, so you can adjust your settings in that function
def skipStream(mp, stream, info, path, tagdata):
    if stream.type == "audio":
        if stream.audio_channels == 2:
            mp.settings.acodec = ['aac', 'ac3']
        else:
            mp.settings.acodec = ['ac3']
        self.log.debug("Modified pool of audio codecs is %s." % (mp.settings.acodec))
    return False

Thank you for the help. I based my code on your example and it was working well until I ran into some multi language files. I changed my audio language options to only keep english and I noticed that skipStream runs before filtering out languages. I'm checking all the audio tracks and filtering the languages now in the validate method. So I can reject changing files for cases I'm not handling yet. Seems to do what I want for what I've tested so far.
def validation(mp, info, path, tagdata):
    mp.log.info("Initiating custom validation method.")
    audiotracks = info.audio
    if len(audiotracks) == 1:
        #TODO: more stereo audio codecs that are compatible with NMT. vorbis?
        if audiotracks[0].audio_channels == 2:
            mp.settings.acodec = ["aac","ac3"]
            mp.log.info("Single stereo audio track. Modified pool of audio codecs is %s." % (mp.settings.acodec))
        return True
    else:
        mp.log.info("Video has more than 1 audio stream.")
        AC3SurroundFound = False
        acodecs = []
        validTrackCount = 0
        awl = mp.settings.awl
        for audio in audiotracks:
            if mp.validLanguage(audio.metadata['language'], awl, []):
                mp.log.info("Audio Track: %s %s %s channels" % (audio.metadata['language'], audio.codec, audio.audio_channels))
                validTrackCount += 1
                if audio.codec not in acodecs:
                    acodecs.append(audio.codec)
                if audio.codec == 'ac3' and audio.audio_channels == 6:
                    AC3SurroundFound = True

        mp.log.info("Valid audio track count is %i." % (validTrackCount))
        if AC3SurroundFound:
            #keep all audio, let SMA process as normal
            mp.settings.acodec = acodecs
            mp.log.info("AC3 5.1 found, keeping all audio. Modified pool of audio codecs is %s." % (mp.settings.acodec))
        return AC3SurroundFound or validTrackCount == 1

Thanks for sharing, closing this out


About dnscrypt.org

Hi.
I don't know if here is the best place to ask this.
In FreeBSD we fetch dnscrypt-proxy 1.9.5 through https://download.dnscrypt.org/dnscrypt-proxy/ .
This site will be available again?
Regards.

I don't own that domain any more, and I don't know who does. So, probably not. It might even be eventually used to spread malware.
But version 2 will be released in a couple days :)

Ok, thanks for the information.


Add Output Variable ArtifactBuildNumber

Question, Bug, or Feature?
Type: Feature
Enter Task Name: DownloadBuildArtifacts
https://github.com/microsoft/azure-pipelines-tasks/commits/master/Tasks/DownloadBuildArtifactsV0
Issue Description
There is already an Output Variable called BuildNumber. Unfortunately it does not contain the BuildNumber, but the BuildId. Requests to change the name from BuildNumber to BuildId have been denied in the past with regards to backward compatibility.
Hence, there should be a new Output Variable that contains the real BuildNumber.
The same applies to the DownloadPipelineArtifact task.

Any update?


Renew c api

Rewrite the c-api, with all improvements we get from a newer Rust version and more insight into soundness of unwinding across ffi.

On suggestion of a few people, the c-api will live in a separate crate that is not published. This makes it easy to control what we bindgen, it ensures the interface of the main crate is a superset. If anything it makes it far easier to move back should the need arise.


"mounted()" block "event" registration for "watch()" but not for "updated()"

https://github.com/vuejs/vue/issues/4827 as a refrence that some lifecycle hooks DO have the event available while not for others.
export default {
    props:['hasErrors'],
    components: {
        'FormErrors': require('./Errors.vue')
    },
    data() {
        return {
            input: ''
        }
    },
    mounted () { // get the hidden inputs value
        if (this.$el.type == 'hidden') {
            this.input = this.$el.value
        }
    },
    watch: { // send data to the central storage to be picked up by the parent
        input(val, oldVal) {
            storage.data.userInfo[event.target.name] = val;
        }
    },
}

error Cannot read property 'target' of undefined

without mounted()
https://cl.ly/3V3p3U2L2I3j/Screen Recording 2017-02-01 at 07.59 PM.gif
with
https://cl.ly/3H3p3j3H3n2J/Screen Recording 2017-02-01 at 08.01 PM.gif
fiddle
https://jsfiddle.net/svz0hb8j/2/
now if we replaced watch() with updated() the error is gone
updated() {
    storage.data.userInfo[event.target.name] = this.input;
},

don't know why this happens specially that both pretty much does the same thing. https://vuejs.org/v2/api/#updated

If you remove the mounted callback, you never update input, therefore your watcher is never called, hence, you never see the error.
event doesn't exist in your watcher because you're not declaring it as a parameter for the function, but even if you did, Vue doesn't pass it to watchers. Watchers are triggered on changed data on the view model. Those changes aren't necessarily originated from DOM (input events), so it doesn't make sense to pass event to watchers.

if u checked the first gif, u will see that i don't need to pass event as param to the watcher because every time the input or whatever the watcher is hooked to change its value, it automatically fires an event which we simply catch it inside the watch() same as updated().
the only reason am using mounted() is to get a pre-init value of a hidden input, also mounted() doesnt care whether the DOM was changed or not, hence we have to use either watch() or updated() to catch those changes if any.
however as i stated in the main ticket, calling event inside updated() works just fine and doesn't triggers any errors, while watch() does & if you need another gif as a prov i can happily create it.

I have no idea where the event is coming from in your code...

@ctf0, brother, event won't just magically exist if you don't declare it somewhere. In this case, it would be as a param since window.event isn't a thing (except for IE, but never mind that). There is no global event identifier you can use here.
There is no event fired that is "caught" by watch. watch is a callback invoked by the VM when whatever it is watching changes. In this case, you are watching input. The only place you are changing its value is in your mounted callback by doing this.input = this.$el.value (this is the reason your watcher is called). That statement does not produce any new DOM event that could be sent to your watcher.
Now, if you comment out your mounted callback, there is nothing in your code that is changing input. This means your watcher will never be called. This is the reason you never see the error.
Regarding not seeing the error when using updated. You are not doing anything in your code that would cause a DOM update. Your updated callback is therefore never called. This is the reason you never see the error.
I hope this makes it clearer for you.

okay then, i will make a small demo video to clear things up and make it easier to understand why i opened those 2 tickets.

Closing since I don't see anything actionable here, neither a bug or a feature request. Please discuss elsewhere so that only fully reproducible bugs or properly elaborated feature requests are opened as issues.

@yyx990803 all i was asking for is little more time to present the issue more clearly and prov that there is a bug and its not related to what everyone is describing.

@ctf0 You can take all the time you need, and when your reproduction is ready, please open a new issue.

@yyx990803 @hector-humberto @LinusBorg @HerringtonDarkholme https://drive.google.com/open?id=0B1q9LR33Y0lNZXR5TGVMTWlkN28
and here is the code to test with if u want
https://github.com/ctf0/vuebug

Sorry, I don't use composer and don't want to install it only for a reproduction that should be reproducable in a fiddle on jsfiddle.net (the fiddle from your original post does not reproduce the problem)
I also have no idea where in thsat huge folder structure the component(s) in question are to be found.
That being said, I can say the following:
window.event generally
That event variable that use use is non-standard. Not all browsers support it, among those is Firefox.
https://developer.mozilla.org/en-US/docs/Web/API/Window/event
So it's generally a very bad idea to rely on this, unless you know none of your users will ever use Firefox.
Using this in Vue lifecycle events, watch etc.
This variable will only be available in any of those, if those methods are run synchronously after some event was fired in the DOM, presumably be the user.
Since some of those are run asynchronously, so this variable should be gone in most cases by the time they are called. it simply doesn't exist anymore.
In mounted, it may be there if the component was rendered & mounted because of a click. If it was another reason (some async code like setTimeout, or even the initial page render) there simply is no event because no event happened.
Honestly, I'm a bit flabbergasted that it is there in the watch in the beginning.
This whole approach of yours is very brittle and will certainly bring you a lot more pain.
Now about your video
As far as I could follow (didn't take the time to search the matching code in this repository), updated() only has this event because it is fired sychronously after the component has been updated and is in the DOM(!) - and the insertion into the DOM triggered the blur event.
So the fact that the event is not there is not because somehow update "has it" and  mounted doesn'T have it, and it's not a bug in Vue - it's there "by accident".

@LinusBorg appreciate the thoughtful explanation, just a couple of points

u r correct about not getting the same result from the fiddle, however it only prove the watch event undefined error.

if u r still interested, here is the link to the js files only https://github.com/ctf0/vuebug/tree/master/resources/assets/js

regarding the event, which i originally got the idea from (https://vuejs.org/v2/guide/components.html#Form-Input-Components-using-Custom-Events)  and thought that its being registered by vue.
yes i may have misinterpreted this as a bug but there is definitely something unclear of why we are seeing this behavior.

i believe the event is available in updated(), watch & even in computed because all of them gets triggered when the property gets changed, which in our case was the @input.

at last, the reason i made this is because i wanted to keep the components stateless.
so i was thinking maybe i can use $children inside the parent component to extract everything i want from the child components but didnt know how to exactly use it 😢  or if it even offer such ability.
so in ur opinion what would be the best way to achieve that & how we can treat an input with pre-ini value same as we do with others that use v-model

To be clear, the event you are accessing has absolutely nothing to do with Vue, nor any of the lifecycle hooks or watch callbacks. Variables don't magically appear in JavaScript functions - the only special keywords available in every function are this and arguments. You just happened to be able to access event because it is exposed on window as a global variable. It's browser quirk that only exists in some browsers: they set window.event when a native DOM event is triggered. But this has nothing to do with Vue: Vue's custom event system and the browser's DOM event system are two separate concepts.
I suggest taking this discussion elsewhere since it's really about understanding JavaScript and browser event systems.


[Bug] tidb 从checkpoint 恢复后，不会消费任务失败时间产生的binlog数据

Search before asking

[X] I searched in the issues and found nothing similar.

Flink version
1.14.3
Flink CDC version
3.0.0
Database and its version
5.7.25-TiDB-v5.2.2
Minimal reproduce step
1.set restart config
    // 设置从最近的检查点恢复         env.setRestartStrategy(RestartStrategies.fixedDelayRestart(                 3, // 最大尝试重启次数                 Time.seconds(60) // 重启的时间间隔         ));
2.kill taskmanager
3.update db data
4.start taskmanager
5.view data print (do data print)
What did you expect to see?
print data change between in task faild
What did you see instead?
fix bug
Anything else?
No response
Are you willing to submit a PR?

[X] I'm willing to submit a PR!

Considering collaboration with developers around the world, please re-create your issue in English on Apache Jira under project Flink with component tag Flink CDC. Thank you!


Endpoint routing chooses fallback route due to conflict

This is a follow-up from a Stack Overflow question, reduced to a minimum repro, so you don’t actually have to go through the question there.
Let’s take the following route configuration on an otherwise empty ASP.NET Core 2.2 project:
app.UseMvc(routes =>
{
    routes.MapRoute("SettingsRoute", "{controller}/{action=Default}", null, new
    {
        controller = "Settings"
    });

    routes.MapRoute("NestedRoute", "Settings/{controller}/{action}", null, new
    {
        controller = "Categories"
    });

    routes.MapRoute("FallbackRoute", "{*url}", new
    {
        action = "Gone",
        controller = "Default"
    });
});

Along with three very simple controllers:
public class CategoriesController : Controller
{
    public IActionResult Add() => Content("Category added");
}

public class DefaultController : Controller
{
    public IActionResult Gone() => Content("Gone");
}

public class SettingsController : Controller
{
    public IActionResult Default() => Content("Settings");
}

As you can see, there are three route templates which should match the following routes. These work:

/Settings/Default → SettingsController.Default
/Settings/Categories/Add → CategoriesController.Add
/Foo → DefaultController.Gone

However, since the SettingsRoute has a default action of Default specified, /Settings should also trigger the SettingsController.Default action. Instead, endpoint routing picks the FallbackRoute here and goes to DefaultController.Gone.
If you remove the FallbackRoute or the NestedRoute, the /Settings path does work again correctly, so it appears to be a side effect of these three route definitions.
I have been debugging through the route matcher now to figure out why this happens and for some reason the DfaMatcher finds two candidates for the /Settings path, in this order:

Endpoint: DefaultController.Gone, Score: 0
Endpoint: SettingsController.Default, Score: 1

Since the matcher then iterates through these candidates and appears to pick the first one that matches the route, it decides to use the DefaultController.Gone route, although the SettingsController.Default one would be the better choice.
During initialization, the MvcEndpointDataSource calculates the following five endpoints:

CategoriesController.Add – Order 1, RoutePattern: Settings/Categories/Add
DefaultController.Gone – Order: 1, RoutePattern: {*url}
SettingsController.Default – Order: 1, RoutePattern: Settings/{action=Default}
SettingsController.Default – Order: 2, RoutePattern: Settings
SettingsController.Default – Order: 3, RoutePattern: Settings/Default

I was thinking that maybe this order was determining which candidate came first, so I moved the DefaultController declaration after the SettingsController (in a usual situation where each class is in its own type, the name would have to be changed since the file name order decides this). Sure enough, that moved the DefaultController.Gone endpoint to the end, but that didn’t actually change the candidate order.
So I am a bit at loss now what else could determine the candidate order. I haven’t digged deep enough into the matcher to tell.
I do realize that the endpoint routing is known to be broken in certain situations in ASP.NET Core 2.2. And for what it’s worth, this issue does not appear to exist in the current 3.0 preview (phew! 😅). So it’s maybe not worth to dig too deep into this, but I would still like to understand and see if we can’t figure out a workaround (one that does not change the route templates).

Thanks for contacting us, @poke.
@rynowak can you please look into this? Thanks!

Thanks for the thorough writeup.
This has been fixed now in 3.0 - I tried out your example and it works the way that you described.
The smoking gun for 2.2 is this part DefaultController.Gone – Order: 1, RoutePattern: {*url} should have order: 3 and SettingsController.Default – Order: 3, RoutePattern: Settings/Default should have order: 2.
We tried to layer MVC's conventional routing concept on top of existing routing constructs in 2.2 and it's definitely a failed experiment. For 3.0 we built in some of MVC's ideas in a more first-class way in routing, and the result has been way better.
2.2 was a learning experience in how obtuse and complicated some users' routing setups are - and I don't mean that as a bad thing. My learning is to try and define systems with clearer and simpler rules in the future - I don't think there are too many people that have all of the rules of how routing works in their head - however if it works, users will use it 👍

If you're still trying to find a solution for this, I can dig in a bit, but I think there are some dragons here. If you're able to remove {controller} from some of these routes and hardcode the controller you want as a default value that will probably help - but I'm not sure if thats something can do or not.

Closing due to lack of response. Ping if you still need help with something.

@rynowak Sorry for the lack of responses. I missed this within my notifications.
Thanks for the explanations, I was already suspecting that this was just a problem with 2.2's attempt at endpoint routing. Routing can really be a lot more complex than one would think initially. But I think it's great that ASP.NET Core offers such flexibility there (assuming it works at least 😝). And I'm glad you got this all working so much better in 3.0.

If you're still trying to find a solution for this, I can dig in a bit, but I think there are some dragons here.

No worries, I was mostly looking for an explanation here with the additional option that I might have missed something easy. Since that's not the case, I can totally live with this. After all, it wasn't my problem to begin with 😉 but the OP from the SO question seems to be satisfied with the answer I gave too, so workarounds are probably in place now anyway.
Thanks for your time! 😊


[WIP] Email templating

Added email template main header footer
Added nodemailer-juice to compile inline css for nodemailer transporter

Preview v1

Added button-link partials
Trimmed header brand icon

Updated request password email content


SwipeView dont open programmatically

SwipeView dont open programmatically.

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

ID: 02d932c9-7483-c314-b1c6-3850778cd6dc
Version Independent ID: 8ab4f230-a8ff-5164-d377-b0b12cac5f09
Content: Xamarin.Forms SwipeView - Xamarin
Content Source: docs/xamarin-forms/user-interface/swipeview.md
Product: xamarin
Technology: xamarin-forms
GitHub Login: @davidbritch
Microsoft Alias: dabritch

Hi @edmlbox
It does in Xamarin.Forms 4.6.


fix group_concat be crash in vectorized engine

Proposed changes
Issue Number: close #7288
Problem summary
The key points to reproduce the problem:

create table with string NOT NULL column;
call left join with group_concat;

The reason is because:

aggregate function framework pick combined null function by arguments type;
as the input argument is NOT NULL, so the origin group_concat is selected;
as a result, we need to consider insert_result_into with input column = ColumnNullabe(ColumnString);

Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[ ] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[ ] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[ ] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

https://github.com/apache/doris/pull/11700 already fix this case from FE side.
Make sure input column type is always nullabe.