Wrong class used in turbo streams

TurboBundle::STREAM_FORMAT doesn't exist, that static variable was moved to TurboStreamResponse::STREAM_FORMAT.
Also the stream didn't work for me unless I changed:
$this->render('task/success.stream.html.twig', ['task' => $task]); to $this->render('task/success.stream.html.twig', ['task' => $task], new TurboStreamResponse());

Q
A

Bug fix?
yes/no

New feature?
yes/no

Tickets
Fix #...

License
MIT

Hi!
Sorry about this. The documentation IS correct but not for the currently-released version. Let me be more clear:

v2.0  (current version) uses TurboStreamResponse (as you've seen)
v2.1 (unreleased) uses what you see in the documentation

What we need in the docs (and should have added when this change was originally made) was a versionadded:: 2.1 tag that explained the change and how it worked before. For an example, see https://github.com/symfony/ux/blob/2.x/src/LiveComponent/src/Resources/doc/index.rst#L59
Would you be able to revert your changes by add a nice versionadded note explaining things?
Thanks!

I added a brief message about the previous version using a different class, and returning a streaming response. If you can't use it, that's alright. I never knew contributing to the docs was so easy and smooth.

Thanks @htmlshaman1!


Setting up API Routes in back-end and connect them to front-end

Set up API routes in back-end
Connected the API route to front-end to get the API data
Add new components to front-end to display the data
fixed front-end sizing issue to be compatible to different screen sizes.

Notes:

I deleted MySQL as it was not working in my end, someone needs to add it again into back-end file artist.js

Al

Outside of the credentials comment, everything is looking great! The way you set up the server API makes a lot of sense to me and I will try to replicate it for other tasks. Also, looking at how you setup the front end and extracted the data from the server API call makes me think that we might not need a database implementation after all. We can talk more about the strategy for this in our meeting tonight.


Android immersive mode

Hi,
Not sure if this has been thought of and discarded, but Gdx is generally used for games and one typically does not want the "navigation bar" on the screen for pads or phones that do not have physical buttons. So I personally always add one line in the AndroidLauncher (middle line added):
		AndroidApplicationConfiguration configuration = new AndroidApplicationConfiguration();
		configuration.useImmersiveMode = true;  //Adding this!
		initialize(new GameStarter(), configuration);

Very minor thing, but this tool is so great that it only is minor things left to possibly tweak.

It'll be true by default in libGDX 1.11.1, but I don't see why not set it to true today in Liftoff.

Good idea, and if it's going to be true by default, it should be safe now, I would think. I'm adding this now.


Restore Chromium tests

In 3135af2511cd3dfa8e3874244c46054550c563e2, Chromium tests were removed from release deps with comment:

This is failing because it exceeds the hydra-queue-runner size limit.

I thought, possibly wrongly, that #12794 would fix this. If it did, should we restore the tests? If it did not, what needs to be done so that we can restore the tests?

cc @edolstra @aszlig

Yes, I think it should be fine to re-enable.

Done


[AMQP Python] Pure Python AMQP SDK Promotion

[x] demo to the team
[ ] blog post, work with lily/james), check azure-sdk-blog issue
[ ] #23094

[ ] get a list of EH users who previous have issues with uamqp
[ ] get a list of EH users who are interested in trying out new things
[ ] reach out to them!

When will this be ready to move forward?


Add support for Sqreen badges

At the request of some of our customers, I've added support for a badge to show whether a website is protected by Sqreen or not.

This PR is a bit premature, will revisit later. Apologies!


lxd/instance: Don't use RawOperation

Closes #8653
Signed-off-by: Stéphane Graber<EMAIL_ADDRESS>

@tomponline good to go


Shouldn't be possible to search without locations

This input:

will produce this:

We talked about it with @FilipMessa and for now we will at least add some default values like Praha (or Brno?) -> Anywhere. So the case when inputs are empty shouldn't happen.

Some form validation could be useful, but since we are going to use the placepicker eventually, I guess it's not so necessary?

In current app, you get an alert if you try to search without inputing anything.

@tbergq And does it work for iOS simulator and web? Because I haven't seen this alert.

@tbergq We could do something like that, yes.
@jvaclavik That's because that's the Kiwi.com app, not Margarita 😉


Terminal output is not well arranged

VSCode Version: Code 1.11.2 (6eaebe3b9c70406d67c97779468c324a7a95db0e, 2017-04-13T08:03:11.395Z)
OS Version: Windows_NT ia32 10.0.15063
Extensions:

Extension
Author
Version

python
donjayamanne
0.6.3

gitlens
eamodio
3.4.5

matlab
Gimly81
0.5.0

Steps to Reproduce:

ctrl+`
ls
the text is not arranged well.

Same condition ato ver. 1.12.1

This was fine before, and it makes the integrated terminal hardly usable. I have this issue since the 1.12 update.

Duplicate https://github.com/Microsoft/vscode/issues/19665, this is an issue with the latest Windows. You can try enabling the legacy console as recommended in https://github.com/Microsoft/vscode/issues/19665#issuecomment-294625978


using a web-component (which is implemented with react) inside the Layout component causes an exception

The error message i get is:
Warning: setState(...): Can only update a mounted or mounting component. This usually means you called setState() on an unmounted component. This is a no-op. Please check the code for the undefined component.

caused by the div inserted in the layout (with css class 'mdl-layout__inner-container'), which does not have a data-reactid attribute.

Can you share en example of code with the error?

Closing this for now... Please provide more information


[14.0] prettier-xml: A little bit too invasive

I had issue with a report : https://travis-ci.com/github/OCA/event/jobs/406980086#L369
prettier-xml is adding carriage return to xml values. In that case, this is annoying as Odoo does not trim, so when installing, the reference to the report is not found.
Values shouldn't be modified.

Another similar case:
https://github.com/OCA/community-data-files/pull/99#discussion_r510832164
It seems this needs to put the prettier xml as strict.
cc @Yajo @sbidoul

Fix in #35.


Hands on

I'm getting the below message when i try to access the Manager App and Server Status on localhost:8080...
HTTP Status 404 – Not Found
Type Status Report
Message The requested resource [/manager/html] is not available
Description The origin server did not find a current representation for the target resource or is not willing to disclose that one exists.
Apache Tomcat/9.0.64
What should i do to access the Manager App, Server Status and Host Manager on localhost:8080 ?

Hi @Frazer-Nyambe , our team has noticed the issues you've filed and suggests joining a future Tech Call if you can (the next meeting is July 20 at 1pm EST). We may be able to provide better support over Zoom.

Alright thank you. Surely I'll join.
On Tue, Jul 19, 2022, 21:21 Islandora Foundation Community <
@.***> wrote:

Hi @Frazer-Nyambe https://github.com/Frazer-Nyambe , our team has
noticed the issues you've filed and suggests joining a future Tech Call
https://github.com/Islandora/islandora-community/wiki/Tech-Call-Meetings-2022
if you can (the next meeting is July 20 at 1pm EST
https://github.com/Islandora/islandora-community/wiki/July-20%2C-2022).
We may be able to provide better support over Zoom.
—
Reply to this email directly, view it on GitHub
https://github.com/Islandora/documentation/issues/2132#issuecomment-1189464810,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AV33MNHLRVDHZUAD7V7WSGDVU354BANCNFSM53NQWVGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
_
_
CONFIDENTIAL NOTICE: This message (and its attachments) may contain
confidential information. Any unauthorized use or disclosure is prohibited.
If you have received this communication in error, please notify us
immediately by responding to this email and then delete it from your
system.

FYI, I spoke to Frazer by email and confirmed that he is using the "manual" installation process. I will work with him by email and/or Slack to try another installation approach if possible like ISLE or the newer "demo install" (ISLE + Portainer)


Design UDS Segmentation API

Description
As a developer, I would like to separate segmentation duty to separate entity, so I do not have to repeat the code for different Transport Interfaces that handles the same bus.
As a developer, I would like to have common interface for segmentation process for all buses, so I could use it in the same way for all buses.
As a developer, I would like to have possibility to perform segmentation (or reverse operation) of a diagnostic message without transmitting it, so I would be able to make simple verification separate from the rest of the implrementation and reuse it in the code multiple times.
Test Strategy
Unit Tests:

100% code coverage for common code for all segmentation classes (if there is any common code)

Acceptance Criteria

Abstract class for segmentation duty is created.
Segmentation class provides following functionalities:

UDS message is divided into UDS packets according to proper standard (e.g. ISO 15765-2 for CAN) - list of standards.
UDS packets are collected in one message.
Get number of packets required to build a message after initial frame (SF or FF).
Check whether given packet is a valid following packet (CF).

Concrete classes would be able to handle segmentation for the entire bus (exception: different version of bus might have separate implementation e.g. CAN and CAN-FD).

It would be nice to have user documentation for this feature.


soc: arm: microchip: Add save and restore for additional blocks (option power optimization)

Extend save and restore for additional peripherals if device driver power management is not supported.
Change introduces a macro disabled by default that allows to evaluate power consumption to prioritize optimizations are required for each SoC HW block

Closing this since seems MCHP better approach planned for MEC17xx SoC entry/exit


Remove Non-existent filterSidemenu Config Option

Summary
Remove non-existent site.config option 'filterSideMenu'. This option was removed in v6.0.0 release.

FYI: the screenshot updates are from https://github.com/cerner/terra-toolkit/pull/371 release


Line numbers off when using sassline

I'm using sassline to add baseline typings to my site, it works fine with prism but when adding line numbers they are not lining up properly:

I'm using Prism 1.15 with online the line-numbers plugin.
See plunkr for more details:
https://plnkr.co/edit/x2dG7ZEA2r4RGK0Uxp3K?p=preview

The problem seems to be the .typeset code rule in style.min.css which sets font-size: 1.06667rem.
But I don't know why line number has a problem with that.

Looks like it's a combination of these 2 lines:
code {
    @include fontsize(theta, all);

    line-height: 1rem;
  }

Only when I remove both of those it lines up again nicely.

Maybe I have found out the reason. But PrismJS team is working for v2, so I can't send a PR for this.
Spans for codes have "display: inline" by default, while spans for line numbers will be set "display:block" on purpose. When codes have "line-height: >1" set, codes will be higher than line numbers. See,
            real display
                 |
                 v

codes:           |-fontSize-|--lineHeight--|

line numbers:    |--lineHeight--|--lineHeight--|
                   |-fontSize-|

Solutions can be,

set negative "top:-x" for line numbers. (need tricky calculation)
change spans for codes to be "display:block", too. (big risks)
change spans for line numbers to be "display:inline". (html structure updates, i.e. other way to achieve text-align: right)


Send \x08 instead of \x7f when pressing backspace

By default, hterm sends the ASCII Delete character (\x7f) when hitting the backspace key. This is incompatible with the behavior of the nrf_cli library which only reacts on the backspace key (\x08). The result is that typos when entering stuff in the web-cli cannot be deleted from the command line. With the command below, we instruct hterm to actually emit the \x08 character when backspace is typed.

Was hoping someone had the same issue. Started using and couldn't backspace. Should merge

See also #7
how does this pull help, I don't know, I opened my solution.


Debugger Pretty view not correctly displaying Kotlin SDK events

Describe the bug
The debugger in my organization's Segment account doesn't seem to be correctly displaying information when I use the Kotlin SDK.
To Reproduce
Steps to reproduce the behavior:

Set up the SDK very minimally in order to send straightforward screen and/or track events.
Use buildJsonObject to add certain key/value pairs via the put() method.
Trigger the events within the app
See the events come into the debugger
See that they're not displayed in the same way that they are when using the "legacy" Segment SDK (https://github.com/segmentio/analytics-android)

Expected behavior
Expecting the debugger's Pretty view to display the data like it does 👇🏽  when I use the https://github.com/segmentio/analytics-android SDK.

Screenshots
Kotlin SDK usage

https://github.com/segmentio/analytics-android usage

Platform (please complete the following information):

Library Version in use: https://github.com/segmentio/analytics-kotlin 1.7.0
Platform being tested: Android
Integrations in use: None

Additional context

if you switch to Raw instead of Pretty, you will see the full payload in pretty format. this is the same in both analytics-android and analytics-kotlin, since the sdk has no control over to the format displaying the web page.

came back from your the another issue you created. seems like you know how to view the actual json payload through Raw. so I'm confused what you are asking in this one. the first image you uploaded seems to be a duplication of the last one. please correct the image if it's mistaken or provide more details regarding to what you are expecting the content of Pretty. feel free to reopen the issue.

the first image you uploaded seems to be a duplication of the last one

Yes, that was on purpose.
I guess I was thrown off by the Pretty view having raw curl info with -d and everything. I'm not sure why the Pretty view now shows anonymousId, receivedAt, and other specific info such as the writeKey. It's difficult to read what properties were attached to the event.
If I use the legacy non-Kotlin SDK and use Pretty view, the putValue() key/value pairs are cleanly shown. The pairs are the only info that's shown besides the event's name.

feel free to reopen the issue

I would if I could. Doesn't seem that this repo's setup allows me to re-open the issue.

@martymiller @langsmith These are issues in the Segment debugger and not the libraries themselves.  We've opened a ticket with the appropriate team internally.  If we get info on an ETA for fix, we'll report back here.

Ok, I'd opened the tickets here because I thought it might have been an SDK issue related to how it was sending events to Segment's servers. Thanks for the update.

@langsmith no problem, thanks for reporting it!  We had no idea. :(   Now it can get fixed.


[MINOR] Fix google style guide address

What changes were proposed in this pull request?
This PR update  google style guide address to https://google.github.io/styleguide/javaguide.html.
Why are the changes needed?
https://google-styleguide.googlecode.com/svn-history/r130/trunk/javaguide.html 404:

Does this PR introduce any user-facing change?
No
How was this patch tested?

Test build #115238 has started for PR 26865 at commit 8913a64.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/20047/
Test PASSed.

Test build #115238 has finished for PR 26865 at commit 8913a64.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/115238/
Test PASSed.

Merged to master


bugfix: last used brain key should be loaded by default in similarity sort

What changes are proposed in this pull request?
Fixed the regression that last used brain key was not loaded by default.
Updated the unique identifier of dataset when we store the key in local storage
How is this patch tested? If it is not, please explain why.
https://github.com/voxel51/fiftyone/assets/17770824/339f86af-4135-4f3c-b432-a07f97bebb37
Release Notes
Is this a user-facing change that should be mentioned in the release notes?

[x] No. You can skip the rest of this section.
[ ] Yes. Give a description of this change to be included in the release
notes for FiftyOne users.

(Details in 1-2 sentences. You can just refer to another PR with a description
if this PR is part of a larger change.)
What areas of FiftyOne does this PR affect?

[x] App: FiftyOne application changes
[ ] Build: Build and test infrastructure changes
[ ] Core: Core fiftyone Python library changes
[ ] Documentation: FiftyOne documentation changes
[ ] Other

Thanks for adding the immutable key!


Flaky test: IPyWidget Script Manager: "before all" hook for "Returns the right base Url"

https://github.com/microsoft/vscode-jupyter/runs/8238528902?check_suite_focus=true

Looks like the test failed due to something else.
No logs, very weird, the error renderer is not loading, i think there could be some other flaky ness here, more like infrastructure.

No longer flaky


[EPIC] Clean up deprecated code/implementation

Is your feature request related to a problem? Please describe.
For 1.3, we are working on stabilizing the exposed APIs based on the new version of CRDs, but at the same time, we found some deprecated codes/implementations we should revisit to clean up like deprecated v1alpha1 CRDs.
This is an EPIC to cover related issues to better understand the coverage.
Describe the solution you'd like
Identify the items related to deprecated items cleanup, especially some unsupported functions before 1.1.

https://github.com/longhorn/longhorn-manager/pull/1136

Describe alternatives you've considered
N/A
Additional context
N/A

cc @joshimoo @shuo-wu @derekbit @jenting


Remove baseurl from raw API calls

[pr-server]
View this change on PR Server

[pr-server]
View this change on PR Server

Wowowooww I didn't know u can just leave that out


ci: use latest branch instead tags

What does this PR do?
Use a branch instead of a tag to avoid the RSS issues with tags/releases in GitHub

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Build Cause: Pull request #1771 opened

Start Time: 2021-04-19T08:37:08.323+0000

Duration: 53 min 13 sec

Commit: 8d0502cf41fbda6818824353fa00094f1239fb6d

Test stats :test_tube:

Test
Results

Failed
0

Passed
1985

Skipped
19

Total
2004

Trends :test_tube:

:green_heart: Flaky test report
Tests succeeded.

Expand to view the summary

Test stats :test_tube:

Test
Results

Failed
0

Passed
1985

Skipped
19

Total
2004

I'll test this PR with https://github.com/elastic/apm-agent-java/pull/1771/commits/2310c66d4111cc561283b8819e5bcb8dd1eef0ba then I'll revert it

Tests
When branch does not exist

As I just mentioned in the related robots issue, we should also make sure that this stable branch do not get built in the main multi-branch pipeline.


Write a Ruby program to find the sum of all numbers in an array

Description
Write a Ruby program to find the sum of all numbers in an array
Input  : [1, 2, 3, 4, 5]
Output : 15

How to contribute

Save the solution in program/FindTheSumOfAllNumbersInAnArray.rb file
Add FindTheSumOfAllNumbersInAnArray.rb file in find-the-sum-of-all-numbers-in-an-array folder

!assign


pulumi-equinix-metal is broken

What happened?
pulumi-equinix-metal is not receiving updates. A couple of things:

It needs to be added to Zapier
It needs custom mapping because the TF provider is called terraform-provider-metal (add a breaking test, change the code, and if it passes, you're good)
The provider is deprecated, so there may not be that many more updates.

Steps to reproduce
n/a
Expected Behavior
n/a
Actual Behavior
n/a
Output of pulumi about
No response
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

The provider is soon to be deprecated and has been replaced with an equinix-owned provider, so this is not worth doing. Closing.


Missing addition in .bash_profile

File: machine/completion.md, CC @gbarr01
Hey guys, total newbie here for anything command line related. Battled with getting docker CLI completion to work, I believe there's a missing piece — the addition of
if [ -f $(brew --prefix)/etc/bash_completion ]; then
. $(brew --prefix)/etc/bash_completion
fi

to .bash_profile as explained on the CLI completion page for docker compose: https://docs.docker.com/compose/completion/
Added this and it worked.

Closing this ticket due to its age, and the impending refactor. If you think this is in error, feel free to reopen. Thanks!


STOR-1586: Rebase v1.29.3

This PR uses git merge -s ours strategy for creating the rebase.
https://github.com/kubernetes-sigs/azurefile-csi-driver/compare/v1.29.3...gnufied:azure-file-csi-driver:rebase-v1.29.3

/retest

/test images

/retest

Failures in nfs-csi looks unrelated.
/retest

/test e2e-azure-file-nfs-csi
/test e2e-azure-file-csi-extended

/test e2e-azure-file-csi-extended

QE is working on pre-merge test.

@gnufied For other rebases I amended the .snyk file to a single commit - the one that adds openshift files: UPSTREAM: <carry>: Add OpenShift files
Wouldn't it be better to do it here too and reduce the amount of carries?

@RomanBednar sure.  I can do that.

/retest

/retest-required

I hit the similar issue in pre-merge testing, the azure-file-csi-driver image build failed sometimes, and I got a successful job after several trying.

/retest required

Manually tested the customer-facing feature looks good, and I launched e2e test manually, it got passed.
/label qe-approved

/retest-required

/retest-required

/retest
after https://github.com/openshift/release/pull/48370

/lgtm

/label docs-approved
/label px-approved


Prettier error: Parsing error: Adjacent JSX elements must be wrapped in an enclosing tag. Did you want a JSX fragment <>...</>?

Before You File a Bug Report Please Confirm You Have Done The Following...

[X] I have tried restarting my IDE and the issue persists.
[X] I have updated to the latest version of the packages.

What version of ESLint are you using?
8.23.1
What version of eslint-plugin-svelte are you using?
2.8.0
What did you do?

Configuration
module.exports = {
  parserOptions: {
    parser: "@typescript-eslint/parser",
    project: "./tsconfig.json",
    sourceType: "script",
    extraFileExtensions: [".svelte"],
  },
  env: {
    browser: true,
    node: false,
  },
  plugins: ["@typescript-eslint"],
  extends: [
    "eslint:recommended",
    "plugin:@typescript-eslint/recommended",
    "plugin:@typescript-eslint/recommended-requiring-type-checking",
    "plugin:svelte/recommended",
    "plugin:prettier/recommended",
    "plugin:svelte/prettier",
  ],
  overrides: [
    {
      files: ["*.svelte"],
      parser: "svelte-eslint-parser",
      parserOptions: {
        parser: "@typescript-eslint/parser",
      },
    },
  ],
};

<h2>
  {$_("steps." + step + ".header")}
</h2>

<script lang="ts">
  import {_} from "svelte-i18n";

  export let step: string;
</script>

<style lang="scss">
  @import "@material/typography/mdc-typography";
</style>

What did you expect to happen?
No prettier error
What actually happened?
I got Parsing error: Adjacent JSX elements must be wrapped in an enclosing tag. Did you want a JSX fragment <>...</>? error from eslint-plugin-prettier
Link to Minimal Reproducible Example
I can't get eslint-plugin-prettier running in your playground. For a (non-minimal) repro, see https://github.com/skaut/shared-drive-mover/pull/1077/commits/563b5a9e1df759736a319c85d8428baeede05ba6
Additional comments
No response

Did you try prettier-plugin-svelte?
And I don't think that's a minimal reproduction, can you please try to provide another smaller one?

Please install prettier-plugin-svelte first.

Oh, thank you very much! Would you maybe consider adding such a note to the docs - in the place where you talk about the preset plugin:svelte/prettier? I somehow though by using that, everything would be taken care for... Thanks!

Oh, thank you very much! Would you maybe consider adding such a note to the docs - in the place where you talk about the preset plugin:svelte/prettier? I somehow though by using that, everything would be taken care for... Thanks!

@marekdedic
We do have a list item mentions:

plugin:svelte/prettier... Turn off rules that may conflict with Prettier (prettier-plugin-svelte).

But I agree we can document it better, please feel free to draft a PR for it from your view!

Hey, thanks for this. For anyone searching through the issues, this solved two other parsing issues I faced:
<script lang="ts" context="module">
// anything typescript gives squiggly red lines
</script>

and
import type { Readable } from "svelte/store"; // squiggly red lines

Since prettier-plugin-svelte comes by default with the official vscode svelte extension, the plugin:svelte/prettier rule should indeed maybe get a special mention in the docs?

Hi 👋🏻 I'm having this issue but the above comments that suggest how to resolve it do not appear to work.
This is a fresh vite svelte-ts template, you can find it here: https://github.com/bbugh/error-eslint-plugin-svelte-prettier
Just clone and run pnpm lint and you'll see:
/app/svelte/first-app/src/App.svelte
  6:2  error  Parsing error: Adjacent JSX elements must be wrapped in an enclosing tag. Did you want a JSX fragment <>...</>?  prettier/prettier

/app/svelte/first-app/src/lib/Counter.svelte
  2:7  error  Type number trivially inferred from a number literal, remove type annotation                                     @typescript-eslint/no-inferrable-types
  8:2  error  Parsing error: Adjacent JSX elements must be wrapped in an enclosing tag. Did you want a JSX fragment <>...</>?  prettier/prettier

/app/svelte/first-app/src/main.ts
  1:8   error  Replace `'./app.css'` with `"./app.css";`        prettier/prettier
  2:17  error  Replace `'./App.svelte'` with `"./App.svelte";`  prettier/prettier
  5:35  error  Replace `'app')` with `"app"),`                  prettier/prettier
  6:3   error  Insert `;`                                       prettier/prettier
  8:19  error  Insert `;`                                       prettier/prettier

✖ 8 problems (8 errors, 0 warnings)
  6 errors and 0 warnings potentially fixable with the `--fix` option.

As of 4.2.0 eslint-plugin-prettier supports svelte natively, so it should no longer be necessary to install prettier-plugin-svelte, but I did anyway as part of testing, and it still gives this error.
  "devDependencies": {
    "@sveltejs/vite-plugin-svelte": "^1.0.2",
    "@tsconfig/svelte": "^3.0.0",
    "@typescript-eslint/eslint-plugin": "^5.38.1",
    "@typescript-eslint/parser": "^5.38.1",
    "eslint": "^8.24.0",
    "eslint-config-prettier": "^8.5.0",
    "eslint-plugin-prettier": "^4.2.1",
    "eslint-plugin-svelte": "^2.9.0",
    "prettier": "^2.7.1",
    "prettier-plugin-svelte": "^2.7.1",
    "svelte": "^3.49.0",
    "svelte-check": "^2.8.1",
    "svelte-preprocess": "^4.10.7",
    "tslib": "^2.4.0",
    "typescript": "^4.6.4",
    "vite": "^3.1.0"
  }

/app/src/App.svelte
  6:2  error  Parsing error: Adjacent JSX elements must be wrapped in an enclosing tag. Did you want a JSX fragment <>...</>?  prettier/prettier

// .eslintrc.cjs
module.exports = {
  parser: "@typescript-eslint/parser",
  parserOptions: {
    project: "./tsconfig.json",
    extraFileExtensions: [".svelte"]
  },
  plugins: ["@typescript-eslint", "prettier"],
  extends: [
    "eslint:recommended",
    "plugin:@typescript-eslint/recommended",
    "plugin:@typescript-eslint/recommended-requiring-type-checking",
    "plugin:svelte/recommended",
    "plugin:prettier/recommended",
    "plugin:svelte/prettier"
  ],
  overrides: [
    {
      files: ["*.svelte"],
      parser: "svelte-eslint-parser",
      parserOptions: {
        parser: "@typescript-eslint/parser"
      },
    },
  ]
}

prettier-plugin-svelte doesn't seem to work well with pnpm, so I think you need to do a workaround.
https://github.com/sveltejs/prettier-plugin-svelte/issues/155

What the holy hell?! I would have never guessed that. I used pnpm all the time and haven't had an issue, but it looks like it's common. I guess I haven't used it in the circumstances for having a prettier plugin...?!
Thank you for the info (and this great package)!


Suggestion: Automated UI testing on Travis CI

The Node-RED project has automated UI testing to prevent bugs related to UI. Currently, the UI developers need to execute the UI test on their PC using grunt test-ui command after executing the install command, install-ui-test-dependencies.sh manually. I think that it is useful for contributors to execute the UI testing on Travis CI automatically when they submit their pull requests and commits.
To realize it on .travis.yml setting file, there are the following two methods.

Method 1: Install Google Chrome on Travis environment, and then execute UI testing

Pros

Short execution time because of local testing process on Travis

Cons

Testing for Google Chrome only

Method 2: Execute UI testing using BrowserStack from Travis

Pros

Test for both Firefox and Google Chrome
Ability to check UI behaviors from screen-captured video on BrowserStack

Cons

Long execution time due to network response between Travis and BrowserStack servers, etc.
BrowserStack approval as OSS to use it free

How about this suggestion? I can contribute the code whichever.

I think you are right we should be running the UI tests regularly.
I would suggest we start with Method 1 and see how we go. I think it is pretty rare that we get any sort of difference in behaviour between Chrome/FF - so I don't feel the need to jump straight to testing a full matrix of browsers with BrowserStack.
At the moment we run the existing tests against 4 different versions of Node - 8, 10, 12, 14. I don't think it is necessary to run the UI tests against all of those. I suggest picking one of them for the UI tests (which may as well be Node 14).

@knolleary Thank you for your comment. I will try to create travis.yml for method 1 tomorrow.

Closing as we have now enabled the UI tests in Node 10


Strange behavior with string interpolation

This is the smallest step that reproduce one bug. One step ahead is to insert a variable k in the middle assigned "/>" which trigger weird xml exceptions (although it is in a string)
Steps

Create a build.sbt in an empty folder
Inside project/build.properties, add sbt.version=0.13.9
Add the following content to it:

lazy val strangeBehavior= TaskKey[Unit]("strangeBehavior", "Perform strange behavior")

strangeBehavior:= {
    val t = s""
    t
}

Type in the console sbt

Expected
The sbt console opens.
Actual
build.sbt:5: error: value
t is not a member of String
possible cause: maybe a semicolon is missing before `value t'?
    t
    ^
[error] Type error in expression
Project loading failed: (r)etry, (q)uit, (l)ast, or (i)gnore?

This a known Scala 2.10 bug - https://issues.scala-lang.org/browse/SI-7919 (also see https://issues.scala-lang.org/browse/SI-9027 aka #1666). You can reproduce this without sbt:
scala> :paste
// Entering paste mode (ctrl-D to finish)

object A {
  val a = s""
  1
}

// Exiting paste mode, now interpreting.

<console>:3: error: ';' expected but integer literal found.
         1
         ^

The workaround would be to put the semicolon:
scala> :paste
// Entering paste mode (ctrl-D to finish)

object A {
  val t = s"";
  t
}

// Exiting paste mode, now interpreting.

<console>:9: warning: a pure expression does nothing in statement position; you may be omitting necessary parentheses
         t
         ^
defined module A


Fixed README badge

Fixed README badge address

Changes Unknown when pulling c722610fd16801e67c15093ceeb7057aec16f2b3 on fixed-coveralls-mikko into ** on master**.

LGTM.

Changes Unknown when pulling 88049b4681a8f2d88d521ae4287f072386c3520b on fixed-coveralls-mikko into ** on master**.


Return richer information about accounts

At the moment, the tool makes the id, username, first_name, last_name fields available for each account that is successfully found.
Telethon returns several more fields associated with a User, as detailed in their api.
Some of these fields (fake, scam, premium,verified, etc...) look like they could be of interest to researchers. This may need some research, as it is unclear how widely these fields are used and if they are likely to be accurate.
It would be good if useful fields could be identified and returned to the tool user.

I would really love to work on this issue! Please assign it to me

Hi @BDFL669 👋 Thanks for reaching out - we'd really welcome a pull request on this issue - I'm happy to assign it to you for now. If you don't have time to work on it, just let me know and I'll unassign you later!
Also, if you're interested in being part of the wider Bellingcat community, you can join our Bellingcat Discord too.

I'm happy to help! Lemme get to work

Looking at the other issues here, this might solve #9 too, though finding a banned number for testing might be difficult.

Hi @BDFL669, just wanted to check in and see how you were getting on. Have you found anything of interest about the fields?

hi @GalenReich, sorry for the late response. I've been playing around with the app and most of the entries are not relevant to our usecase. However, I've added new fields I think you might find useful.

@BDFL669 said:

hi @GalenReich, sorry for the late response. I've been playing around with the app and most of the entries are not relevant to our usecase. However, I've added new fields I think you might find useful.

Checking: '{Phone Number Removed}' ...Done
{
  "{Phone number removed}": {
    "id": {ID removed},
    "username": "{Username removed}" ,
    "first name": "{Name removed}" ,
    "last name" : null,
    "fake": false,
    "scam": null,
    "verified": false,
    "premium": false,
    "mutual contact": false,
    "user status" : null,
    "bot": false,
    "bot chat history": false,
    "restricted": false,
    "restriction_reason": [ ]
  }
}

Great stuff! Just wanted to check in! I'm going to delete your last message so that the image with the phone number and user information is not preserved, but have quoted it above - it looks good👍

Awesome, thank you so much! Would you like me to submit a PR with the
latest changes?
On Fri, 1 Mar 2024, 12:47 Galen Reich, @.***> wrote:

@BDFL669 https://github.com/BDFL669 said:
hi @GalenReich https://github.com/GalenReich, sorry for the late
response. I've been playing around with the app and most of the entries are
not relevant to our usecase. However, I've added new fields I think you
might find useful.
Checking: '{Phone Number Removed}' ...Done
{
"{Phone number removed}": {
"id": {ID removed},
"username": "{Username removed}" ,
"first name": "{Name removed}" ,
"last name" : null,
"fake": false,
"scam": null,
"verified": false,
"premium": false,
"mutual contact": false,
"user status" : null,
"bot": false,
"bot chat history": false,
"restricted": false,
"restriction_reason": [ ]
}
}
Great stuff! Just wanted to check in! I'm going to delete your last
message so that the image with the phone number and user information is not
preserved, but have quoted it above - it looks good👍
—
Reply to this email directly, view it on GitHub
https://github.com/bellingcat/telegram-phone-number-checker/issues/16#issuecomment-1972858155,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOZBHUWTG2EZRXN54N6HILTYWBFCBAVCNFSM6AAAAABDBXQNI2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZSHA2TQMJVGU
.
You are receiving this because you were mentioned.Message ID:
@.***>

Feel free to open a PR or a Draft PR - that would be good - the main thing will be to understand to how common those fields are. For example, does Telegram ever set the scam field?
It might be good to include status.was_online and have some parsing to make it human readable

Thanks for the tip!
On Fri, 1 Mar 2024, 14:25 Galen Reich, @.***> wrote:

Feel free to open a PR or a Draft PR - that would be good - the main thing
will be to understand to how common those fields are. For example, does
Telegram ever set the scam field?
It might be good to include status.was_online and have some parsing to
make it human readable
—
Reply to this email directly, view it on GitHub
https://github.com/bellingcat/telegram-phone-number-checker/issues/16#issuecomment-1973014250,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOZBHUQZUKJCQ3UGT6RZOXLYWBQRNAVCNFSM6AAAAABDBXQNI2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZTGAYTIMRVGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Greetings! The API seldom provides information for the scam parameter so
I've had to remove it. Moreover, After parsing status.was_online data, I
assume this covers the user status field so I've also removed this one as
well (telegram rarely propagates this field as well). Here is a screenshot
of my output after redacting any revealing details about my telegram
account. If the results are satisfactory I shall proceed to open a PR
[image: image.png]
On Fri, 1 Mar 2024 at 18:15, Anvi Licious @.***> wrote:

Thanks for the tip!
On Fri, 1 Mar 2024, 14:25 Galen Reich, @.***> wrote:

Feel free to open a PR or a Draft PR - that would be good - the main
thing will be to understand to how common those fields are. For example,
does Telegram ever set the scam field?
It might be good to include status.was_online and have some parsing to
make it human readable
—
Reply to this email directly, view it on GitHub
https://github.com/bellingcat/telegram-phone-number-checker/issues/16#issuecomment-1973014250,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOZBHUQZUKJCQ3UGT6RZOXLYWBQRNAVCNFSM6AAAAABDBXQNI2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZTGAYTIMRVGA
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

here is the screenshot:

Greetings! The API seldom provides information for the scam parameter so
I've had to remove it. Moreover, After parsing status.was_online data, I
assume this covers the user status field so I've also removed this one as
well (telegram rarely propagates this field as well). Here is a screenshot
of my output after redacting any revealing details about my telegram
account. If the results are satisfactory I shall proceed to open a PR

Please open a PR! ⭐


Change redirect url with code

I have the following problem.
What I want is that of a user press a link to /baskets/save to redirect him to login and then later on to /baskets/save again.
In /baskets/save:
if(!$this->Auth->user()) {
$this->request->session()->write('goto', ['controller' => 'Baskets', 'action' => 'save']);
return $this->redirect('/users/login');
}
and in users/login:
if($goto && count($goto) > 1)
$this->Auth->redirectUrl(array_merge($goto, ['plugin' => false]));
but then I get an error:
Error: A route matching "array ( 'controller' => 'Users', 'action' => 'login', 'plugin' => 'ADmad/HybridAuth', '_ext' => NULL, )" could not be found.
The error occurs in HybridAuthController.php on line 50:
if ($user) {
$this->Auth->setUser($user);
return $this->redirect($this->Auth->redirectUrl());    <------------- here
}
What am I doing wrong?

Use hauth_return_to option to set the redirection url you want.

how can I access it and change it programmatically? I thought you only set it once in AuthComponent initialization

Hmm nvm, trying to change the config after authenticator instance has already been created won't work anyway. You need to figure out yourself how/where the plugin get's added to your redirect URL.
Why do you need to manipulate the redirect URL yourself anyway?  Returning back to the URL that was called before login is default behavior.

Because I want to redirect to specific actions depending on when the user redirected to login page.
Without the plugin I could do it before. I believe it is a common case scenario.


Test against Python 3.4.

As discussed in #264.

This should be all good pending the green tick...

So this looks good, as the tests pass, but we haven't restored anything such as use of six.u() (although use of @python_2_unicode_compatible from django.utils.encoding would be better.)
It may be that we can just get away without this which will make life easier.
@camilonova: Would it be possible to release v3.0.3 with the restored support if you're happy with this?

Sure.


opencv_contrib functionality not working anymore

In Version 3 in include_opencv.h you added #undef ENABLED_CONTRIB, resulting all opencv contrib methods not working anymore (e.g. face detection, BackgroundMOG). What is the reason to disable it?

The OpenCvSharp NuGet package bundles the opencv_world310.dll from the OpenCV official package. It does not contain contrib features, so OpenCvSharp's contrib wrapper is disabled at the moment. :persevere:
I want to eventually provide a custom opencv_world DLL and implement contrib wrapper. :muscle:

I was able to compile all the contrib files however contrib_world.dll will not compile according to the opencv bug report I filed.  If they ever get this fixed it would be perfect to have both the world dlls.  I just can't figure out how to make cmake work to compile into contrib_world.dll.

Thanks guys, understood. So I guess I'll go with making a custom build for now to get those features back (module_face is a must).
@shimat Sounds cool. Let me know if I can help. Having a common solution would be nice.
@fdncred Saw your bug report. Also looked into the corresponding CMake files, turns out contrib_world apparently has been intentionally removed in 3.1, so my guess would be it won't come back.

@thomasbielagk If you get the custom build working I'd like to see it if possible.  I'd love to have a contrib_world too!

Got it working.
First I built opencv + opencv_contrib.
cmake -DOPENCV_EXTRA_MODULES_PATH=....\opencv_contrib\modules\ ..
MSBuild OpenCV.sln /p:Configuration=Release
MSBuild INSTALL.vcxproj /p:Configuration=Release
Then go to OpenCvSharp:
In the project OpenCvSharpExtern, replace the preset directories for INCLUDE / LIBS
(bot point to a prebuild opencv) with the directories where you just built opencv, something like
\build\install\include
\build\install\x86\vc14\lib
In "include_opencv.h" remove the lines
#define ENABLED_CONTRIB
#undef ENABLED_CONTRIB
Finally add a preprocessor definition ENABLED_CONTRIB
Compiling this (Release mode), still gave me errors in face.h (face_createXXXRecognizer).
Replace the clone(...) calls with clone(...).
Now it should compile fine.
Using the new OpenCvSharpExtern.dll together with the required OpenCV dlls
(\build\install\x86\vc14\bin) should do fine, I can now run e.g.BackgroundSubtractorMOG.
Let me know if that works for you, If required I can also create and share a fork of the repo.

@thomasbielagk Are you creating a contrib_world.dll or are you just using the opencv_world.dll with all the contrib modules compiled in?  If opencv_world, what is the size of your dll?  I've been down this path before but i got stuck with OpenCvSharpExtern not loading.

I am not creating a world DLL at all. As you said opencv now only includes few of the extension modules into world, so I decided to go with single DLLs (don't want to change opencv makefiles)
Building opencv via "cmake -DOPENCV_EXTRA_MODULES_PATH=....\opencv_contrib\modules\ .."

Sorry about the confusing clone-line, didn't realize I had to escape the template so it doesn't look like an HTML tag :smiley:
What I meant is
clone(...)
to
clone<FaceRecognizer>(...)
This way the method signature remains unchanged, but your way may work as well.

@thomasbielagk Did you change OpenCvSharp to load these DLLs instead of loading opencv_world.dll?
NativeMethods.cs only wants to load one opencv dll.
private static readonly string[] OpenCVDllNames =
{
    "opencv_world",
};

I would like to see your code for OpenCvSharpExtern (and perhaps others) if it's not too much trouble. I can't get mine to load but that could be because I'm trying to load opencv_world as displayed above.

Created a fork:
https://github.com/thomasbielagk/opencvsharp
Visual Studio 2015, build OpenCvSharpExtern (Release, x86).
Good point about the loading, I assumed the DLLs would be loaded automagically (which appears to be the case, as I can see it working).

@thomasbielagk Ok.  I see what you're doing.    That is how they are auto-magically loading.
The only problem with that route is that the collateral DLLs need to be in the path where the exe can find them. Usually that means the same directory.
If they were in the referenced in the NativeMethods class then you could keep the collateral DLLs in the dll\x64 dll\x86 folders and the exe could find them.
Another problem that perhaps @shimat can help with is that since most of the contrib functions are not used anymore things like SURF just blow up because they haven't been tested.  Some things may work just find while others need implementation modifications.
Thanks @thomasbielagk for showing one method of getting some of the contrib functions working in OpenCvSharp!

@shimat
Any updates about this issue?

I'm sorry for the late reply :bow:
I released OpenCvSharpExtern.dll that is built with opencv_contrib static library. 🎉

FaceRecognizer.Create*FaceRecognizer does not seem to work on OpenCvSharp3-AnyCPU v <IP_ADDRESS>70110.  EntryPointNotFoundException still happens.

(I think I should write this comment on the other thread, but I wrote here.)
I could not find any documents about compiling the latest source code, which are based on .NET Core (.xproj) and cmake.
And the build system will change again.
".NET Core Tools Switch to MSBuild"
https://www.infoq.com/news/2016/11/netcore-tools-msbuild
https://www.infoq.com/jp/news/2016/12/netcore-tools-msbuild
Comments in this thread contain various important view points related to the ground design of OpenCvSharp, I think.
Thank you for your consideration.

(日本語で失礼します)
有益な情報をありがとうございます。記事の通りdotnet migrateしてみましたら、うまくいったようですがVisual Studio 2015では読み込めないようです。2017RC移行ですね。
もし2015でビルドされるなら、こちらをお試しください。
https://www.microsoft.com/net/core#windowsvs2015

ご紹介頂いた内容試してみます。
早速のご返事どうもありがとうございました。
取り急ぎ御礼まで。

@matstumpf Haven't tried to compile to Xamarin Android. Will look at it at home. But you don't need my fork or custom code anymore, shimat has brought back support for the external stuff (using static lib). Just clone the repo.
Is Xamarin a fix requirement? I guess Java-Bindings might be easier to work with, like these: https://github.com/bytedeco/javacpp-presets


missing coverage in fault filter

@rshriram I noticed the following line is missing coverage in the fault filter. Do you mind adding a test for that case? It's pretty important to make sure the timer gets correctly disabled, etc.
https://github.com/lyft/envoy/blob/master/source/common/http/filter/fault_filter.cc#L121

Correct me if I am wrong. I am not testing that function directly. However,
see the following line
https://github.com/lyft/envoy/blob/master/test/common/http/filter/fault_filter_test.cc#L85
that checks if timers are disabled after use.
And here is an example test case:
https://github.com/lyft/envoy/blob/master/test/common/http/filter/fault_filter_test.cc#L274
WDYT?
Also, is there a tool that provides line by line code coverage stats?
On Tue, Jan 10, 2017 at 10:44 PM Matt Klein<EMAIL_ADDRESS>wrote:

@rshriram https://github.com/rshriram I noticed the following line is
missing coverage in the fault filter. Do you mind adding a test for that
case? It's pretty important to make sure the timer gets correctly disabled,
etc.
https://github.com/lyft/envoy/blob/master/source/common/http/filter/fault_filter.cc#L121
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/lyft/envoy/issues/342, or mute the thread
https://github.com/notifications/unsubscribe-auth/AH0qd7XJfn_U5x8XxxeU4tnu8fA0MBacks5rRFAugaJpZM4LgKii
.

--
~shriram

You need to have a test where you invoke the stream reset callback like this:
https://github.com/lyft/envoy/blob/master/test/common/http/filter/buffer_filter_test.cc#L110
This will insure that things are cleaned up properly in the reset case.
You can do a full coverage build like so:
https://github.com/lyft/envoy/blob/master/ci/do_ci.sh#L18
After that runs there will be a lot of HTML files including coverage.html which show you exactly what is covered.


Error thrown on product detail page when any of the first items in each variant is not available for sale

For example, it's happening for this url: https://demo.vercel.store/product/acme-t-shirt
If variants are not supplied in the url via query params, then the template will select the first combination available for sale. This will eventually call router.replace, which can cause a re-render if the first option of any variant isn't available for sale (ie. out of stock, etc.).

Officially fixed in #1117.


feature size of a8a dataset may be wrong

Hi, in the file sparse_classifier.py, feature size is assigned to 124 for a8a dataset.
flags.DEFINE_integer("feature_size", 124, "Number of feature size")

However, I think the feature size should be 123 according to this website.
Am I right?

Thanks @ethan-funny .
You are right the feature size of a8a dataset is 123 which is from 1 to 123. However, TensorFlow needs to define the array to contain all the feature ids which is from 1 to 123. The array is 0-based and we need to initialize with the size of 124 to support the id of 123.
Otherwise, you will get this error if you change the size from 124 to 123.

Thanks @tobegit3hub .


GetServiceInfoListAsync not working on a 6.2.274 cluster

GetServiceInfoListAsync fails on the newest version (6.2.274) of a cluster (not sure if it worked on older versions)
The reason it fails is because when parsing the serviceInfo, the response it reads the first property and expects it to be "ServiceKind" but the response it got is something like the following:
{"ContinuationToken":"", "Items":[{"Id":"devApp~SomeActorService", "ServiceKind":"Stateful","...},...]}

It's a know issue andit will be fixed in next update of Service Fabric 6.2.* and 6.3 release

This should work with clusters with version >=6.2.183


Multiple mapping to same class error

I've got a problem mapping multiple classes to the same class when they belong to the same package. For example:
Class A map to Class C
Class B map to Class C
Class A and B are in the same package.
The problems seems to be with the generated Transformer class (which is created by package) and that add 2 entries with the same key:

addMapper("com.example.model.C", new com.example.viewmodel.A());
addMapper("com.example.model.C", new com.example.viewmodel.B());

As a result when I try to transform class C to class A it works.
But when I try to transform class C to class B it give me a cast error: class A connot be cast to class B.

So class A and class B are identical ?

No, class A and B are differents as I use them in differents views and contains differents properties of class C

IMHO, you have a bad smell in your approach, for me, you should not mix different objects with the same middle object, I mean, if you do that you're breaking you separation between objects and you're coupling those.


Not working with Bitbucket Cloud

Hi,
this resource isnt working for me with Bitbucket Cloud as you get the PRs with git ls-remote ( https://github.com/laurentverbruggen/concourse-bitbucket-pullrequest-resource/blob/master/assets/check#L56 ) but that isnt listing PRs.
I guess you are using it with Bitbucket Server (self-hosted) and there its working ?
Best !

@philicious did you ever get this to work?  I am trying now and it doesn't see any of my PR's

@fewknow kinda.  I used this resource now productively https://github.com/zarplata/concourse-git-bitbucket-pr-resource . I added support for Bitbucket Cloud and it also supports self-hosted Bitbucket Server

Yes, this resource was developed for Bitbucket Server. Since this repository is no longer actively maintained I will close this since I won't be fixed.

@laurentverbruggen maybe you could add to README that its only supposed to work with self-hosted Bitbucket Server and not with Cloud so people dont get confused or waste time when actually working with Bitbucket Cloud and finding your resource

yes, you are right. I'll add it.


Unable to link your Google Assistant for Alexa Skill. Please try again later.

I've followed the process and no matter what I do, when I try to link my google account; it'll ask me for my account, tell me to confirm that I accept the access that it's needing...then it gives me this error message.
"Unable to link your Google Assistant for Alexa Skill. Please try again later."
I have 'tried later' a couple times over the last few days and nothing ever changes.
Any ideas?

No idea, I think this may not have anything to do with this device manager


CVAT doesnot delete old zip files that were prepared for "export dataset"

My actions before raising this issue

[ x] Read/searched the docs
[x ] Searched past issues

I have to export datasets frequently. Hence, the zip files for download consume a large portion of space on the disk. Eventuallly docker duns out of space because of these zip files that are currently not needed.
Steps to Reproduce (for bugs)

Create CVAT image, allocate minimal disk space to the image
Create a task that occupies 1/4 of the allowed disk space
Perform a random markup
Export the dataset as "CVAT for images"
Repeat the export step several times (until you get the message "CVAT ran out of space")

Expected Behaviour

CVAT should delete old zip files after user downloaded the exported set
Current Behaviour

Apparently, CVAT keeps old zip files, hence it runs out of space due to obsolete zip files with exported data
Possible Solution

Delete obsolete zip files once user has downloaded them.
OR provide a command to cleanup obsolete zip files.
Context

I have to ask our system administrator to increase the size of the disk, available for the image. We will run out of total disk memory soon.
Your Environment

I have to ask our system admin to collect this data. Briefly, this is Windows server. We use older version:
Server version: 1.1b0, Core version: 3.3.1, Canvas version: 2.0.2, UI version: 1.6.7

Git hash commit (git log -1):
Docker version docker version (e.g. Docker 17.0.05):
Are you using Docker Swarm or Kubernetes?
Operating System and version (e.g. Linux, Windows, MacOS):
Code example or link to GitHub repo or gist to reproduce problem:
Other diagnostic information / logs:
Logs from `cvat` container

Hi,
Currently, the policy is the following: the exported datasets are cached for the time specified here (10 hours by default). The export parameters (job/task/project, format name, save_images) define the file name in the cache. The same file name should be reused for the same export parameters, meaning the file will be reused after the task is updated. There are temporary directories being created during exporting, but they are removed after exporting.
Could you please check if the export parameters are the same in your case?

the exported datasets are cached for the time specified (10 hours by default)
This solved my issue. Thank you.


[Enhancement] Support adjust cachesize automatically

What type of PR is this：

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

The BE memory is used by many logics, such as queries, ingestions, compactions, pagecache, and so on. pagecache is enabled by default, and the user can adjust the maximum amount of memory. Furthermore, we would like to have a pagecache self-adjustment policy, so that when memory pressure is low, pagecache can buffer more data, and when memory pressure is high, Pagecache can buffer less data. pagecache can actively evict some data to ensure the running of BE.
This PR automatically resize the Pagecache by monitoring the current memory water level. Below is a test result with auto-tuning cachesize enabled. This test specifies a urgent water level of 30% and a high water level of 20%. The memory usage fluctuates between urgent and high water levels.

If auto-tuning cachesize is turned off, memory usage is not constrained.

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

run starrocks_be_unittest

run starrocks_admit_test


Github text says Distance matrix

In the pulldown menu this is now called Heatmap. Github still has it as distance matrix. Took me a while to find it.

Good catch. It's fixed.
Just FYI, you can edit these typos by going to the Wiki and pressing the Edit button.

Good to know- will do it myself if I find any more!

From: Tony Boyles<EMAIL_ADDRESS>Date: November 7, 2017 at 3:09:54 PM EST
To: CDCgov/MicrobeTRACE<EMAIL_ADDRESS>Cc: Shankar, Anupama (CDC/OID/NCHHSTP<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [CDCgov/MicrobeTRACE] Github text says Distance matrix (#115)
Good catch. It's fixedhttps://github.com/CDCgov/MicrobeTRACE/wiki.
Just FYI, you can edit these typos by going to the Wikihttps://github.com/CDCgov/MicrobeTRACE/wiki and pressing the Edit buttonhttps://github.com/CDCgov/MicrobeTRACE/wiki/Home/_edit.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/CDCgov/MicrobeTRACE/issues/115#issuecomment-342606505, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Af4Ld0DMd368GDlTaOYEjJ9lql94UsJ5ks5s0LjngaJpZM4QVQip.


feat: add enumtype selection in the playground for typescript

Description

Add support for selecting enumType in playground
Related issue(s)

Fixes #1162

@jonaslagoni PTAL

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4732

Relevant Lines:
4991

💛  - Coveralls

/rtm

@all-contributors please add @prayutsu for code

If you are hungry for more @prayutsu we need to add all the features of TypeScript to the playground, I just have not had the time to add the issues yet.
We are talking about the documented features here: https://github.com/asyncapi/modelina/blob/master/docs/languages/TypeScript.md
You don't have to do all, but feel free to jump into some of them 🚀


Extract list of people allowed to review from MAINTAINERS

There is nice https://lgtm.co/ project that provide quite similar functionality but instead of configuration file it uses MAINTAINERS file inside repository. It would be nice addition to Homu.

Interesting. Is the MAINTAINERS format widely accepted?

I think it is quite popular and it could be just an addition as a fallback if there is no configuration file.

Actually, it is already possible to look up the GitHub collaborator list using the GitHub API. The reasons Homu didn't do that were:

You may want to give a user the r+ permission, but not a direct access to the repository. In that case you can only add the user as a reviewer, but not as a GitHub collaborator.
It consumes one additional GitHub API request, which counts against the rate limit.

Being able to modify MAINTAINERS means being able to add arbitrary reviewers. In the current approach, only GitHub collaborators can add new reviewers, so they can be thought as admins. However, the suggested method grant that permission to any reviewers. I guess this is OK in general, but I'm curious if this workflow may be useful to some people.

For the heck of it I threw together https://github.com/ashcrow/maintainers which could be helpful here.
"Python3 library for parsing MAINTAINERS files. It can parse the non toml formats listed at lgtm."


Update Kitchen config

This PR updates test kitchen to use the latest stable version of Chef and adds CentOS version 7.3 as box.

Thanks!


Created ssh command shell script to allow ssh from container.

Part of the solution for https://github.com/samsung-cnct/kraken/issues/109

@leahnp sure thing i will have to document it, probably be best with a help option. However its meant to run in the kraken-lib docker container, and a way to allow for using ssh to run a command remotely, so the idea is you would run kraken in a bastion, or your local machine, which ever you prefer.

@alejandroEsc would it be better to not accept an ssh_config file at all but rather accept a cluster name, optionally accept an output location (for nonstandard kraken output locations) and then we can find the ssh_config file?  this way we're encapsulating the ssh_config work and we can accept the short name as specified in the ssh_config file to determine which node to ssh to.
that might be a bit much but I think would be well in-line with letting kraken control infra and then asking kraken to help connect to that infra.

@coffeepac  Hmm, thats one option, my concern there is that users may want to share the ssh_config file, but yeah true, maybe thats not expected behavior, or they should reproduce the directory structure.

I was thinking if users are sharing the config they  probably won't be using the ssh bundled with kraken.  That user group probably knows what they are doing and don't need a helper script.

@coffeepac so after thinking about this a bit more, i think the best thing is not to make assumptions at this level about where the ssh_config is and probably let it happen a level above, mostly because it would be difficult to pick out where the config file exists in order to find that ssh_config file.

@coffeepac oh sorry about that. I mean kraken (cli) will make that determination, based on input config file etc..

Note: bridge code, for kraken (CLI)


Reset to factory settings

I made a change to the wordpress URL in settings and now my site is busted. Is there a quick way to reset the wordpress blog to factory settings?
Thanks!

This might address the "busted" state.

@ryanburnette just ran it and it did not work. I see that there is a mysql instance tied to it, anyway I can access that?

Try changing the settings in wp-config and push to heroku.
If u really need to u can do some database tools I think I used heidisql. Then u go heroku jawsdb to get all the login details.just rmb to flush cache after changes. And always backup!!


Update MagneticConnector.md

Updated for the new 0.1.5 version

It seems GitHub felt it pertinent to keep all the copies of the "Remove the remarks" comments it hid from me due to the change you commited halfway through me writing this review (and my computer crashing...). You can just dismiss them all, and remove Remarks.

Is this everything? When you've made all the changes you want to, re-request a review from the top right.

Ok, I think every thing requested is done.

Thanks for contributing this so quickly!

Yes, as soon as I've see the update on Steam, I knew I could make some cartridge experiments, but didn't see the changes in this doc.


Do not merge - Incomplete optimization

Just an optimization that might turn out to be useless. Do not merge.

I can't say for certain without looking at the output, but I would expect the compiler to optimize that. If you're using clang the --emit-llvm flag can be interesting reading.

I am almost sure it does not. I can't imagine it's intelligent enough to decide that it does not need to construct a new object. It's a logic-level thing.

For clang at least, once it reaches the optimizer, the constructor is just a function. And so if you inline the constructor, it becomes a load of moves, which can be canceled, and some zero adds, which are easily removed.

Yes, zero adds are removed, but 3 brand new x,y,z's are needlessly allocated.

Its a stack allocation. And most optimizers are capable of removing unused stack allocations, in order to deal with register promotions.

Interesting. What about copying?
In the old version, Whenever we do CurrentCell->m_Location + vector(...), We create 1 new vector which houses the result (and 1 new vector which is the right hand side of the +).

Its the same thing, once its inlined its just a bunch of moves.

Yes but is it smart enough to optimize a bunch of needless but intertwined copy moves?
I guess we should look at the output.

Yep, reading the llvm IR can be interesting when optimizing.

I double checked this. Because of the way register machines work, on x86 with clang 3.6 at least, they both lower to assembly that is very similar, but the current version results in an extra stack allocation. This means it has slightly worse cache usage, but executes the exact same number/sequence of instructions.

register machines work

Who knew tills could be so advanced?

As apposed to stack, accumulator, or belt computers.

"The current version", as in the Master version?

Actually by current version I meant a slightly modified version of master, that has the control flow changes, and takes the parameter by value. This was so I could actually look at the changes caused by the objects only.

How could the non-master version produce MORE stack allocations?

So, in conclusion. Is this PR is useless? -with the exception of the else if change-

master produces more, but its not really worth worrying about. ANd yes the PR is useless unless this function shows up in a profiler.


[Citadel] Update tutorials

Use Doxygen code snippet and image
Add example code
Update tutorial titles and content

Will forward port this to Dome and main if approved.

@chapulina  I addressed your tutorial comments in this PR. Feel free to take a look if you have time .

@chapulina  I addressed your tutorial comments in this PR. Feel free to take a look if you have time .

Can we add a CMakeLists file to examples/simple_tpe_plugin so that it's possible to compile it?

I added an example CMakeList but don't think it's working correctly yet. I'm still not sure how exactly CMake links different libraries so it'd take some time to figure that out. I think it'd be preferable to merge the tutorial part of the this PR first, and take the CMake part out for later improvement. What do you think?


Develop new integer histogram API

See Slack discussion and proposal and release goals in Notion for context. This issue is only about implementing the backend API, not the related frontend work.

[x] Add a task list here and socialize it (share in Slack or setup a meeting to discuss).
[x] Pinpoint the implementation plan (see proposal).
[x] Port D3.js tickIncrement, nice procedures along with unit tests.
[x] Change Histogram API to support 2 methods.
[x] Implement Diagnostic method using ported D3.js procedures.

@pjain1 : Can you help with this one?

Closed by https://github.com/rilldata/rill-developer/pull/1719


Known Problems

LAST UPDATE: 17.08.2023
I apologize, but currently i do not have time for this because of understaffing and way to much work in my normal job.
When that changes i will work on it again (hopefully just a few weeks...)

[ ] Every start of the Container Netdata sees a new Node (no persistence of /etc/netdata)
[ ] If the container is started without a config, subsequent starts fail.
[ ] Claim Node without Room
[x] no fixed Netdata version set
[ ] GH Actions sometimes sets the latest tag for DEV Image
DEV Version: it's where i test things and most of the time it won't work. Use at your own Risk...
That said, i dont think you can actually damage anything...

Hey @CryoRig, I'd just like to mention that I started my own version of the Netdata add-on:
https://github.com/felipecrs/netdata-hass-addon
All your known issues are addressed there, as far as I know.


Update default.env

Adding a comment to tell the '$' sign is forbidden in RCONWEB_API_SECRET

#453 allows this now, but we might want to update .env and the README to add some more warnings in general because $ can cause Docker to try to apply parameter expansion to environment variables


Upgrade versions

$subject

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/7297530073

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/7297530073
Status: failure


Avoid pretty-printing of JSON objects

I'd like to use Cereal to output JSON data in newline-separated record format (of course this semi-standard convention has a webpage: http://jsonlines.org/).
Right now, Cereal always outputs newlines while pretty-printing the JSON object, with no way to switch them off. This messes up my format.
What's the best way to get rid of the pretty-printing?

At the moment, no real way to do so, but take a look at this PR https://github.com/USCiLab/cereal/pull/146 for how to do it.

You can simply replace rapidjson::PrettyWriter by rapidjson::Writer in the file cereal/archives/json.hpp. You then also need to comment out the call to itsWriter.SetIndent, since rapidjson::Writer has no SetIndent. That's all.

I'm in favor of this feature, too. It would be nice if cereal provides a way for my application to dump the minified JSON in the logs and show pretty JSON on the UI.

I also want this feature. Is there a way to avoid the unnecessary new lines in the outpout?

This would be very handy... why not add another option for minimised json?

Seriously, this needs to be a configurable option, without needing to modify source code.  Really only need two parameters, indent size (int, default 4) and end of line (string, default "\n").  Or just provide a single on/off.  I'm using this library for a messaging protocol in an embedded system with constrained resources, and every byte counts.

I agree we need an easy obvious way to avoid pretty printing: Consider making pretty printing special and non-formatting the default!

Alors? Milestone?

Similar vain, need the most compact json for client/server communication.

FWIW, I had a patch here: https://github.com/USCiLab/cereal/pull/146
that provided for the ability to disable pretty-printing for JSON.  It got held up at the time because @AzothAmmo was evaluating a newer RapidJSON and then I stopped using Cereal for anything production so it got dropped by me as well.
Someone could likely rebase that and get this functionality.

Has this been implemented yet? If so how to generate this compact output?


fix: when redirects is set to 0 we do not want curl to follow location (FOLLOWLOCATION = 0)

What has been implemented?
We have a use case, where we want to make sure we are redirecting with a 301. This is because we are redirecting http to https (but also non-apex to apex -> www.taxdown.es to taxdown.es) in all these cases we want to make sure it is a 301 so that we don't get penalized for web crawlers at duplicating content.
With the current implementation and FOLLOWLOCATION = 1, redirects are being followed and the actual status code is 200 always. Putting it in another way it is impossible to receive a 301 if FOLLOWLOCATION = 1.
However, it is desired behaviour to follow redirects by default, so I left it enabled and only disabled when maxRedirects is set to 0.
Closes #127 (Although it was incorrectly closed)
Steps to verify

With default configuration and url http://taxdown.es and expectedStatusCodes: [200]

It should success

With maxRedirects > 0 and url http://taxdown.es and expectedStatusCodes: [200]

It should success

With maxRedirects = 0 and url http://taxdown.es and expectedStatusCodes: [301]

It should success

With maxRedirects = 0 and url http://taxdown.es and expectedStatusCodes: [200]

It should fail

CLA
I have read the CLA Document and I hereby sign the CLA

I have read the CLA Document and I hereby sign the CLA

Looks good, thanks!


bump connection

The connection library is no longer supported and is considered unsafe to use. Many libraries that used connection in the past have migrated to the fork crypton-connection, and none of these will be buildable together with HaskellNet-SSL if it remains on connection. Instead, the dependency should be change to crypton-connection.
See issue #35 and PR #33.


Please bring back Synapse support

For a large customer, experimentation with Synapse is taking place. The team I am a part of is one of those teams. It has been decided to use Spark Notebooks in Python (PySpark) to convert large data processes that currently exist on-premise. In this large project, people encounter the annoying limitations of working in a browser screen compared to working in an IDE on a daily basis.
The limitations/bugs are:
• There is no syntax check while typing, so you can happily type syntactically incorrect code and only find out that it is incorrect during execution.
• It is not possible to automatically apply formatting to adhere to a certain coding style, such as PEP-8, which would improve consistency now that multiple people are working on this project.
• Because we are now working in a browser, it can happen that the session expires (or something similar) and someone unexpectedly loses their work. This happens frequently, causing people to commit nervously in between, resulting in a heavily polluted git history.
• Another issue is that sometimes opening a notebook results in a view with only empty cells. The content is not displayed. It seems to be some kind of cache-miss in the product or something similar. In any case, that notebook cannot be worked on until it is committed. This is not pleasant because we have to commit blindly.
What we would like is:
• Good support for a code editor, such as VSCode, that can be connected to the Spark environment in Synapse, so that we can:
o Write code in a representative environment (i.e., the actual PySpark engine in the relevant pool in Synapse)
o Write controlled code in clear steps (commits)
o Have good support from Python tools, such as linting and syntax checking and formatting
The only reference I have been able to find for a possible solution is this extension, but the Synapse support was removed from it two versions ago: https://marketplace.visualstudio.com/items/mshdinsight.azure-hdinsight/changelog
If this extension were to work again, it could very well provide what we are looking for, but at the moment there is no way to check as older versions of this extension are no longer available for download.

Bumping this. Why was this functionality removed?

Any update on this issue?

Would be great to have this back. Any plans to reintroduce this feature?

For a large customer, experimentation with Synapse is taking place. The team I am a part of is one of those teams. It has been decided to use Spark Notebooks in Python (PySpark) to convert large data processes that currently exist on-premise. In this large project, people encounter the annoying limitations of working in a browser screen compared to working in an IDE on a daily basis.
The limitations/bugs are:
• There is no syntax check while typing, so you can happily type syntactically incorrect code and only find out that it is incorrect during execution.
• It is not possible to automatically apply formatting to adhere to a certain coding style, such as PEP-8, which would improve consistency now that multiple people are working on this project.
• Because we are now working in a browser, it can happen that the session expires (or something similar) and someone unexpectedly loses their work. This happens frequently, causing people to commit nervously in between, resulting in a heavily polluted git history.
• Another issue is that sometimes opening a notebook results in a view with only empty cells. The content is not displayed. It seems to be some kind of cache-miss in the product or something similar. In any case, that notebook cannot be worked on until it is committed. This is not pleasant because we have to commit blindly.
What we would like is:
• Good support for a code editor, such as VSCode, that can be connected to the Spark environment in Synapse, so that we can:
o Write code in a representative environment (i.e., the actual PySpark engine in the relevant pool in Synapse)
o Write controlled code in clear steps (commits)
o Have good support from Python tools, such as linting and syntax checking and formatting
The only reference I have been able to find for a possible solution is this extension, but the Synapse support was removed from it two versions ago: https://marketplace.visualstudio.com/items/mshdinsight.azure-hdinsight/changelog
If this extension were to work again, it could very well provide what we are looking for, but at the moment there is no way to check as older versions of this extension are no longer available for download.

Did you manage to find any work arounds on this?

We have not, alas.


td: fail without retry on all 4xx errors

Currently we're retrying forever on 422. We could continue blacklisting status codes but that seems error prone.

@frsyuki @komamitsu PTAL

👍

👍


[SPARK-20396][SQL][PySpark][FOLLOW-UP] groupby().apply() with pandas udf

What changes were proposed in this pull request?
This is a follow-up of #18732.
This pr introduces @pandas_grouped_udf decorator for grouped vectorized UDF instead of reusing @pandas_udf decorator.
How was this patch tested?
Exisiting tests.

Test build #82790 has started for PR 19505 at commit 4d2bd95.

Test build #82790 has finished for PR 19505 at commit 4d2bd95.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82790/
Test FAILed.

Test build #82793 has started for PR 19505 at commit f096870.

Test build #82793 has finished for PR 19505 at commit f096870.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class PythonUdfType(object):

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82793/
Test PASSed.

Test build #82803 has started for PR 19505 at commit 10512a6.

Test build #82805 has started for PR 19505 at commit 789e642.

Test build #82810 has started for PR 19505 at commit 122a7bc.

Test build #82811 has started for PR 19505 at commit fdafb35.

Test build #82803 has finished for PR 19505 at commit 10512a6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82803/
Test PASSed.

Test build #82805 has finished for PR 19505 at commit 789e642.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82805/
Test PASSed.

Test build #82813 has started for PR 19505 at commit 7332969.

Test build #82810 has finished for PR 19505 at commit 122a7bc.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82810/
Test PASSed.

Test build #82811 has finished for PR 19505 at commit fdafb35.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82811/
Test PASSed.

Test build #82813 has finished for PR 19505 at commit 7332969.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82813/
Test PASSed.

Test build #82831 has started for PR 19505 at commit 85f250d.

Test build #82831 has finished for PR 19505 at commit 85f250d.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82831/
Test FAILed.

retest this please

Test build #82835 has started for PR 19505 at commit 85f250d.

Change itself LGTM if we are okay to go separating this.

Test build #82835 has finished for PR 19505 at commit 85f250d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82835/
Test PASSed.

Test build #82843 has started for PR 19505 at commit 1ef25c3.

Test build #82843 has finished for PR 19505 at commit 1ef25c3.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82843/
Test PASSed.

So, looks we are good to go?

I'd mark this pr as [WIP] for now because we don't reach consensus on API changes. Thanks.

I meant to ask if others agree with the current change as I could not see the ongoing discussion at that time.

Here is a summary of the current proposal during some offline disuccsion:
I. Use only pandas_udf
The main issues with this approach as a few people comment out is that it is hard to know what the udf does without look at the implementation.
For instance, for a udf:
@pandas_udf(DoubleType())
def foo(v):
      ...

It's hard to tell whether this function is a reduction that returns a scalar double, or a transform function that returns a pd.Series of double.
This is less than ideal because:

The user of the udf cannot tell which functions this udf can be used with. i.e, can this be used with groupby().apply() or withColumn or groupby().agg()?
Catalyst cannot do validation at planning phase, i.e., it cannot throw exception if user passes a transformation function rather than aggregation function to groupby().agg()

II. Use different decorators. i,e, pandas_udf (or pandas_scalar_udf), pandas_grouped_udf, pandas_udaf
The idea of this approach is to use pandas_grouped_udf for all group udfs, and pandas_scalar_udf for scalar pandas udfs that gets used with "withColumn". This helps with distinguish between some scalar udf and group udfs. However, this approach doesn't help to distinguish among group udfs. For instance, the group transform and group aggregation examples above.
III. Use pandas_udf decorate and a function type enum for "one-step" vectorized udf and pandas_udaf for multi-step aggregation function
This approach uses a function type enum to describe what the udf does. Here are the proposed function types:

transform
A pd.Series(s) -> pd.Series transformation that is independent of the grouping. This is the existing scalar pandas udf.
group_transform
A pd.Series(s) -> pd.Series transformation that is dependent of the grouping. e.g.

@pandas_udf(DoubleType(), GROUP_TRANSFORM):
def foo(v):
      return (v - v.mean()) / v.std()

group_aggregate:
A pd.Series(s) -> scalar function, e.g.

@pandas_udf(DoubleType(), GROUP_AGGREGATE):
def foo(v):
      return v.mean()

group_map (maybe a better name):
This defines a pd.DataFrame -> pd.DataFrame transformation. This is the current groupby().apply() udf

These types also works with window functions because window functions are either (1) group_transform (rank) or (2) group_aggregate (first, last)
I am in favor of (3). What do you guys think?

@cloud-fan asked:
"
what's the difference between transform and group_transform? Seems we don't need to care about it both in usage and implementation.
"
My answer is:
transform defines a transformation that doesn't reply on grouping semantics: for instance, this is a wrong udf definition:
@pandas_udf(DoubleType(), TRANSFORM):
def foo(v):
return (v - v.mean() / v.std())
because the transformation is replying some kind of "grouping semantics", otherwise v.mean() and v.std() has no meaning for arbitrary grouping.
Also, catalyst should throw exception for the code example below:
@pandas_udf(DoubleType(), GROUP_TRANSFORM):
def foo(v):
      return (v - v.mean()) / v.std()

# Should throw exception here, it should only take `transform` not `group_transform` type
df = df.withColumn(foo(df.v))

Btw, I think the scope of this change is more than just a follow-up. Should we create another JIRA for it?

@icexelloss The summary and the proposal 3 looks great. To prevent confusing, can you also put the usage of each function type in proposal 3? E.g., group_map is for groupby().apply(), transform is for withColumn, etc? Thanks.

+1 for separate JIRA to clarify the proposal and +0 for 3. out of those three, too.

The group_transform udfs looks a bit weird to me. @icexelloss Can you explain the use case of it?

@viirya @cloud-fan I updated my original summary. I think it answers group_transform question. I also added more example to each type.
@HyukjinKwon @viirya I agree we can move this to a separate Jira and merge current PR of @ueshin. Maybe I can open another PR with just the proposal design doc? Not sure what's the best way is.

@ueshin Maybe close this PR?

Sure, I'd close this.
@icexelloss Of course you can open a separate JIRA and another PR. Thanks!


go-dupfinder deletes the original file

Describe the bug
A clear and concise description of what the bug is.
I tested go-dupefinder on Ubuntu and it deletes the original file
To Reproduce
Steps to reproduce the behavior:

Copy and paste a document in a folder
In terminal go-dupfinder find /path/to/some/directory
press y to see duplicates: y
go-dupfinder shows me the original file eg file.pdf instead of file(1).pdf
delete duplicate: y
deletes the original file.pdf file
file cannot be found in the trash bin!

Expected behavior
A duplicate finder that uses hash and time to figure out what the duplicate is, and if a file has been changed internally.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

Ubuntu 22.04
Konsole

Sorry to hear that from you. I know about this. It keeps only one unique file and deletes others but does not think about the filename. It has been 2 years and I lost motivation and the time to work on this project. I developed it because I had thousands of music files on a hard disk and in some directories, there were duplicates. so I decided to clean them up and free up spaces.
But after some time, I saw that no one was gonna use this project, so I abandoned it. there are a lot of alternatives out there. one of them is fdupes. Also this list of alternatives.
Thank you for using this tool. but do have not the time to maintain it anymore.
Again, thanks for using dupfinder. Now consider using other alternatives.


The Problem of Allen Query Result Error

As shown in the figure, mydata.png is my original data, myquery.png is my query interval, result.png is my query result, and my query statement is: ./ query_hint_m.exec -v -m 20 -o all -q CONTAINS -r 10 samples/mydata.dat samples/myquery.qry. Theoretically, there should be three results: 20140928 20180717
20110627 20180717
20140811 20180814, but only one item is displayed in resul.png

Hello.
The code offers two workload options for presenting the results; either counting them or XOR'ing their ids, see the "Workloads" section of the README file.
Based on your screenshot you are using the second, "Total result [XOR]". Under this, "1" doesn't mean 1 result but that the XOR of the result ids is 1. Please activate the WORKLOAD_COUNT flag in def_global.h and run again your query. You should get 3 as the total result.


Ast transformer help

I have got issue with this custom ast transformer https://github.com/wessberg/DI-compiler. Could you show me hioow it use with ts-jest (i tried it with another test runner ava and it works great). Thanks in advance.

I think the problem is because ts-jest doesn't pass Program into that transformer. That transformer requires TypeScript Program to work.
I will convert this into a feature request.

I think the problem is because ts-jest doesn't pass Program into that transformer. That transformer requires TypeScript Program to work.
I will convert this into a feature request.

Close via https://github.com/kulshekhar/ts-jest/commit/387964faed14ce24d2cf8170a04eee244d69b8b9
Unfortunately that is a breaking change so it will be available in v27

Close via https://github.com/kulshekhar/ts-jest/commit/387964faed14ce24d2cf8170a04eee244d69b8b9
Unfortunately that is a breaking change so it will be available in v27


Processing of message 'textDocument/completion' failed with error

Describe the bug
The error below appears very frequently while programming.
[Error - 11:46:12] Please report this issue to https://github.com/microsoft/al/issues including information on how to reproduce it, if possible.
Processing of message 'textDocument/completion' failed with error: 'Node span: 0-1161, position: 1259 (Parameter 'position')'
Details:
System.ArgumentOutOfRangeException: Node span: 0-1161, position: 1259 (Parameter 'position')
   at Microsoft.Dynamics.Nav.CodeAnalysis.SyntaxNode.FindToken(Int32 position, Boolean findInsideTrivia) in D:\a\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\Syntax\SyntaxNode.cs:line 1002
   at Microsoft.Dynamics.Nav.CodeAnalysis.SyntaxNode.FindToken(Int32 position, Func`2 findInsideTrivia) in D:\a\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\Syntax\SyntaxNode.cs:line 1035
   at Microsoft.Dynamics.Nav.CodeAnalysis.SyntaxNode.FindToken(Int32 position, Boolean findInsideTrivia) in D:\a\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\Syntax\SyntaxNode.cs:line 1002
   at Microsoft.Dynamics.Nav.CodeAnalysis.Syntax.SyntaxTreeExtensions.IsEntirelyWithinStringLiteral(SyntaxTree syntaxTree, Int32 position, CancellationToken cancellationToken) in D:\a\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\Syntax\SyntaxTreeExtensions.cs:line 515
   at Microsoft.Dynamics.Nav.CodeAnalysis.Workspaces.Completion.MemberSyntaxContext.CreateContext(Workspace workspace, SemanticModel semanticModel, Int32 position, CancellationToken cancellationToken) in D:\a\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis.Workspaces\Completion\ContextQuery\MemberSyntaxContext.cs:line 159
   at Microsoft.Dynamics.Nav.CodeAnalysis.Workspaces.Completion.CompletionProvider.CreateMemberContextAsync(Document document, Int32 position, CancellationToken cancellationToken) in D:\a\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis.Workspaces\Completion\CompletionProvider.cs:line 83
   at Microsoft.Dynamics.Nav.CodeAnalysis.Workspaces.Completion.AbstractSymbolCompletionProvider.GetPerContextSymbols(Document document, Int32 position, OptionSet options, IEnumerable`1 relatedDocuments, Boolean preselect, CancellationToken cancellationToken) in D:\a\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis.Workspaces\Completion\AbstractSymbolCompletionProvider.cs:line 248
   at Microsoft.Dynamics.Nav.CodeAnalysis.Workspaces.Completion.AbstractSymbolCompletionProvider.GetItemsWorkerAsync(Document document, AbstractSyntaxContext context, TextSpan span, OptionSet options, Boolean preselect, CancellationToken cancellationToken) in D:\a\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis.Workspaces\Completion\AbstractSymbolCompletionProvider.cs:line 206
   at Microsoft.Dynamics.Nav.CodeAnalysis.Workspaces.Completion.AbstractSymbolCompletionProvider.ProvideCompletionsAsync(CompletionContext context, AbstractSyntaxContext memberSyntaxContext) in D:\a\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis.Workspaces\Completion\AbstractSymbolCompletionProvider.cs:line 181
   at Microsoft.Dynamics.Nav.CodeAnalysis.Workspaces.Completion.CompletionServiceWithProviders.GetContextAsync(CompletionProvider provider, Document document, Int32 position, CompletionTrigger triggerInfo, OptionSet options, Nullable`1 defaultSpan, CancellationToken cancellationToken) in D:\a\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis.Workspaces\Completion\CompletionServiceWithProviders.cs:line 373
   at Microsoft.Dynamics.Nav.CodeAnalysis.Workspaces.Completion.CompletionServiceWithProviders.GetCompletionsAsync(Document document, Int32 caretPosition, CompletionTrigger trigger, ImmutableHashSet`1 roles, OptionSet options, CancellationToken cancellationToken) in D:\a\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis.Workspaces\Completion\CompletionServiceWithProviders.cs:line 183
   at Microsoft.Dynamics.Nav.EditorServices.Protocol.LanguageServer.CompletionRequestHandler.HandleAsync(TextDocumentPositionParams request, Int32 requestId, CancellationToken cancellationToken) in D:\a\1\s\source\Prod\Microsoft.Dynamics.Nav.EditorServices.Protocol\LanguageServer\CompletionRequestHandler.cs:line 45
   at Microsoft.Dynamics.Nav.EditorServices.Protocol.MessageProtocol.RequestHandlerBase`1.HandleAsync(JToken requestContents, Int32 requestId, CancellationToken cancellationToken) in D:\a\1\s\source\Prod\Microsoft.Dynamics.Nav.EditorServices.Protocol\MessageProtocol\RequestHandlerBase.cs:line 54
   at Microsoft.Dynamics.Nav.EditorServices.Protocol.RequestRegistry.Process(Message message) in D:\a\1\s\source\Prod\Microsoft.Dynamics.Nav.EditorServices.Protocol\Endpoints\RequestRegistry.cs:line 80

To Reproduce
I do not know how to exactly reproduce the problem. It seems to start happening when there is a syntax error (a column missing for instance) and then you continue to type without fixing the error.
But, even if the code compile, once the error appeared, it is triggered every time the intellisense is supposed to open.
The only way I found to get rid of the problem is to restart VS Code.
Expected behavior
No error, add a try/catch :)
Screenshots
./.
5. Versions:

AL Language: 5.0.254558

Hi @nicolassaleron,
Thanks for reporting the issue, but if you don't mind could you please provide a more detailed way of producing the error.
The best thing for us is if you could provide a small  self-contained example that produces the error so we can easily and quickly reproduce it.


GUACAMOLE-1099: add Docker secrets support for LDAP properties

Added _FILE for:

LDAP_SEARCH_BIND_DN
LDAP_SEARCH_BIND_PASSWORD
LDAP_USER_BASE_DN
LDAP_CONFIG_BASE_DN
LDAP_GROUP_BASE_DN

Added SSL certificate support via LDAP_SSL_CERT_FILE (eg /run/secrets/ldap_cert) with keytool import for the file.  It could probably use an exists check but it works :)

@ss10sb: Thank you for the contribution, appreciate it.  Per our standard procedures, all pull requests need to be associated with a JIRA issue, and tagged with that.  Please see:
http://guacamole.apache.org/open-source/

GUACAMOLE-1099

@ss10sb A handful of requests for changes, here - any chance you can polish this up so we can move forward with this? Also, there's a conflict, now, that needs to be resolved.

@ss10sb A handful of requests for changes, here - any chance you can polish this up so we can move forward with this? Also, there's a conflict, now, that needs to be resolved.

We're no longer using Guacamole + Docker. Sorry!