loadMoreAction fires prematurely

In my app I have many scrollable divs nested. This causes the loadMoreAction to fire prematurely from the component. Even when defining the scrollable attribute on the component.
I lack experience to really test en provide a working solution for everyone. By changing the way the offset is calculated when position is other than relative fixed my issue.
For relative ( original ):
var selfOffset  = this.$().offset().top;
var scrollableBottom = scrollable.height() + scrollable.scrollTop();

For fixed / absolute:
var selfOffset  = this.$().position().top;
var scrollableBottom = scrollable.height();

ember-infinity/addon/components/infinity-loader.js
Original:
_checkIfInView() {
  var selfOffset       = this.$().offset().top;
  var scrollable       = this.get("scrollable");
  var scrollableBottom = scrollable.height() + scrollable.scrollTop();

  var inView = selfOffset < scrollableBottom;

  if (inView && !this.get('developmentMode')) {
    this.sendAction('loadMoreAction');
  }
},

Edited ( works for me ):
_checkIfInView() {

  var scrollable = this.get("scrollable");
  var selfOffset;

  var scrollableBottom;

  if(scrollable.css('position') == 'relative') {

    selfOffset       = this.$().offset().top;

    scrollableBottom = scrollable.height() + scrollable.scrollTop();

  } else {

    selfOffset = this.$().position().top;

    scrollableBottom = scrollable.height();

  }

  var inView = selfOffset < scrollableBottom;

  if (inView && !this.get('developmentMode')) {

    this.sendAction('loadMoreAction');

  }

},

Innesting.  I think catering for every use case is outside of the scope of this library.  However I think maybe we should make this hook public, so that people who want to control how this check works can easily override it.
@daniel-de-wit would u be down to submit a PR to make this public, and document your use case in the README?

Thanks for your reply, I was on vacation so didn't see your message..
I believe there must be a proper way we can detect if the component is on screen. There are some jQuery plugins that work with nested scrollable divs, I think. Maybe I can extract a good solution for every use case since we just want it to fire when visible to the user on screen.
I definitely like your idea of making the hook public.
It might take me a little while to fix this as I have never actually commited something to an open-source project. But I will try to do it :)
Thanks again for making this wonderfull addon :+1:

I wonder if we can use this somehow?
https://github.com/poteto/ember-in-viewport-demo
On Tue, 1 Sep 2015 at 15:50 Daniel de Wit<EMAIL_ADDRESS>wrote:

Thanks for your reply, I was on vacation so didn't see your message..
I believe there must be a proper way we can detect if the component is on
screen. There are some jQuery plugins that work with nested scrollable
divs, I think. Maybe I can extract a good solution for every use case since
we just want it to fire when visible to the user on screen.
I definitely like your idea of making the hook public.
It might take me a little while to fix this as I have never actually
commited something to an open-source project. But I will try to do it :)
Thanks again for making this wonderfull addon [image: :+1:]
—
Reply to this email directly or view it on GitHub
https://github.com/hhff/ember-infinity/issues/82#issuecomment-136729124.

Sorry i mean this: https://github.com/dockyard/ember-in-viewport

Yeah!  We totally can...
wanna have a go at it?  I can help with the OSS stuff :+1:

Yes sounds like a fun challenge ;)
I wonder if the ember-in-viewport supports nested scrollable divs. I'll go fiddle with it.
I'll keep you posted.

awesome @daniel-de-wit !!!

Totally awesome, I noticed the premature loading behaviour as well but
this way the user never gets to see the loading message.
If this is implemented, we should also look into supporting an offset in
order to maintain such behaviour.

pretty sure ember-in-viewport supports offsets out of the box :+1:

jah it does
export default Ember.Component.extend(InViewportMixin, {
  viewportOptionsOverride: Ember.on('didInsertElement', function() {
    Ember.setProperties(this, {
      viewportUseRAF            : true,
      viewportSpy               : false,
      viewportScrollSensitivity : 1,
      viewportRefreshRate       : 150,
      viewportTolerance: {
        top    : 50,
        bottom : 50,
        left   : 20,
        right  : 20
      }
    });
  })
});

Cool :).
2015-09-01 16:16 GMT+02:00 Hugh Francis<EMAIL_ADDRESS>
jah it does
export default Ember.Component.extend(InViewportMixin, {
viewportOptionsOverride: Ember.on('didInsertElement', function() {
Ember.setProperties(this, {
viewportUseRAF            : true,
viewportSpy               : false,
viewportScrollSensitivity : 1,
viewportRefreshRate       : 150,
viewportTolerance: {
top    : 50,
bottom : 50,
left   : 20,
right  : 20
}
});
})
});
—
Reply to this email directly or view it on GitHub
https://github.com/hhff/ember-infinity/issues/82#issuecomment-136735620.

I also was having an issue where the loading component was not triggering the loadMoreAction when it came into view. I also tried the InViewportMixin and it worked for me! Here is the code that I used to achieve the results that I needed (note that I start loading 1500px before the loader comes into the viewport):
import Ember from 'ember';
import InfinityLoaderComponent from 'ember-infinity/components/infinity-loader';
import InViewportMixin from 'ember-in-viewport';

export default InfinityLoaderComponent.extend(InViewportMixin, {
  viewportOptionsOverride: Ember.on('didInsertElement', function() {
    Ember.setProperties(this, {
      viewportSpy: true,
      viewportTolerance: {
        bottom: 1500,
      }
    });
  }),

  didEnterViewport() {
    this.sendAction('loadMoreAction')
  },

});


This change eliminates the use of Range in the appendEscapeString met…

…hod as this imposes somewhere between a 12-18% performance penalty. There are several causes for this, but the main factor is the additional object creation caused by initializing a range object. This penalty can be avoided by using forEach directly on the input string.
Mailing List thread:
https://groups.google.com/forum/#!topic/liftweb/80Ewp4FPpqg

Thanks for the contribution. I'll do my own performance analysis later this week or this weekend and follow up accordingly.

My results using Lift 3.1.0 final:
[info] Result "liftbenchmark.LiftJsonBenchmark.serializeCaseClass":
[info]   546865.574 ±(99.9%) 37286.201 ops/s [Average]
[info]   (min, avg, max) = (491776.825, 546865.574, 576590.936), stdev = 24662.513
[info]   CI (99.9%): [509579.373, 584151.775] (assumes normal distribution)
[info]
[info]
[info] # Run complete. Total time: 00:00:20
[info]
[info] Benchmark                              Mode  Cnt       Score       Error  Units
[info] LiftJsonBenchmark.serializeCaseClass  thrpt   10  546865.574 ± 37286.201  ops/s

My results using this branch:
...forthcoming...

My results using Lift 3.1.0 final and Scala 2.12.3:
[info] Result "liftbenchmark.LiftJsonBenchmark.serializeCaseClass":
[info]   568927.308 ±(99.9%) 21357.497 ops/s [Average]
[info]   (min, avg, max) = (543322.990, 568927.308, 582870.346), stdev = 14126.662
[info]   CI (99.9%): [547569.811, 590284.805] (assumes normal distribution)
[info]
[info]
[info] # Run complete. Total time: 00:00:20
[info]
[info] Benchmark                              Mode  Cnt       Score       Error  Units
[info] LiftJsonBenchmark.serializeCaseClass  thrpt   10  568927.308 ± 21357.497  ops/s

My results using this branch and Scala 2.12.3:
[info] Result "liftbenchmark.LiftJsonBenchmark.serializeCaseClass":
[info]   646115.037 ±(99.9%) 25888.509 ops/s [Average]
[info]   (min, avg, max) = (608841.283, 646115.037, 672208.066), stdev = 17123.646
[info]   CI (99.9%): [620226.528, 672003.545] (assumes normal distribution)
[info]
[info]
[info] # Run complete. Total time: 00:00:20
[info]
[info] Benchmark                              Mode  Cnt       Score       Error  Units
[info] LiftJsonBenchmark.serializeCaseClass  thrpt   10  646115.037 ± 25888.509  ops/s


rustfmt: 0.8 -> 0.8.1

Update the rustfmt package from version 0.8 to version 0.8.1.
Additionally, update the rustRegistry to a more recent version to
provide the new rustfmt version's dependencies.
I have tested these changes per nixpkgs manual section 11.1 ("Making
patches").

Thanks!


Custom Icons

Thank U for component!
Is there some api or documentation to use icons?
For example I creating hieararchy Filesystem via web app! A need icons for folders, and need icons for my entities

Hi! Thanks!
Yes, you're able to customize a node. See example
To customize the arrow you need to use css styles. VueJS is not supporting deeper named scopes, that's why we cannot use named scopes for each part of node.

Maybe this need add to documentation ??
Could be some items with with different icons not with the same ??

We already have example: doc. It seemed to me that it would be understandable. But it is not. I will add more examples to the documentation.
Yes, I have plans to add abillity to filter tree. It will be done within 1-2 days.

If u plan add filter functionality !
A have a few ideas how filter tree!
If u interested let me know !

Yes, I'm open to any ideas. I've raised a new issue for this. Welcome :)

We are now June 2020 ...
Just want to thank you for this awesome component.
Kind regards,
Johnny
Belgium


Should our main README.md be in English?

Although most of our documentation is in Norwegian, our Github and code language is English. Should we therefore consider translating the README.md and CONTRIBUTING.md documents to English?

I like having technical documentation in English, but:

the working language of pretty much 100 % of the design system consumers today is Norwegian
the primary design system documentation language is Norwegian
due to the above we allow issues written in Norwegian

Maybe the core documents such as the root README and CONTRIBUTING should stick to Norwegian for now?

I my opinion, I think we should be consistent and use the same language all over. On a positive note, if we go for English might get more people to contribute.

I'm closing this for now, since there isn't a lot of people putting forward their opinions. I'm cool with it staying Norwegian - but the mixing of languages is a bit of a... downer.


[QUESTION] Error during build

Please provide the environment you discovered this bug in.
package.json
{
"name": "@analogjs3/source",
"version": "0.0.0",
"private": true,
"license": "MIT",
"scripts": {},
"dependencies": {
"@analogjs/content": "^0.2.0",
"@analogjs/router": "^0.2.0",
"@angular/animations": "^17.2.0",
"@angular/common": "^17.2.0",
"@angular/compiler": "^17.2.0",
"@angular/core": "^17.2.0",
"@angular/forms": "^17.2.0",
"@angular/platform-browser": "^17.2.0",
"@angular/platform-browser-dynamic": "^17.2.0",
"@angular/platform-server": "^17.2.0",
"@angular/router": "^17.2.0",
"dotenv": "^16.4.5",
"front-matter": "^4.0.2",
"marked": "^5.0.2",
"marked-gfm-heading-id": "^3.1.0",
"marked-highlight": "^2.0.1",
"mermaid": "^10.2.4",
"prismjs": "^1.29.0",
"rxjs": "~7.5.6",
"tslib": "^2.4.0",
"zone.js": "~0.14.0"
},
"devDependencies": {
"@analogjs/platform": "^0.2.0",
"@analogjs/vite-plugin-angular": "^0.2.0",
"@analogjs/vite-plugin-nitro": "^0.2.45",
"@angular-devkit/build-angular": "^17.2.0",
"@angular/compiler-cli": "^17.2.0",
"@nx/angular": "^18.0.0",
"@nx/devkit": "^18.0.0",
"@nx/eslint": "^18.0.0",
"@nx/js": "18.0.8",
"@nx/vite": "^18.0.0",
"@nx/workspace": "18.0.8",
"autoprefixer": "^10.4.0",
"jsdom": "^22.1.0",
"nx": "18.0.8",
"postcss": "^8.4.5",
"tailwindcss": "^3.0.2",
"typescript": "~5.3.0",
"vite": "~5.0.0",
"vite-tsconfig-paths": "^4.2.0",
"vitest": "^1.0.4"
}
}
Which area/package is the issue in?
Don't know / other
Description
Hi, I have a question about the build. When I use the command npx nx run frontend:build I get this error:

Does anyone know what could be the cause?
Please provide the exception or error you saw
No response
Other information
No response
I would be willing to submit a PR to fix this issue

[ ] Yes
[ ] No

What's in your vite.config.ts?

What's in your vite.config.ts?

///
import analog from "@analogjs/platform";
import { defineConfig, Plugin, splitVendorChunkPlugin } from "vite";
import { nxViteTsPaths } from<EMAIL_ADDRESS>import nitro from "@analogjs/vite-plugin-nitro";
// https://vitejs.dev/config/
export default defineConfig(({ mode }) => {
return {
root: __dirname,
publicDir: "src/public",
resolve: {
mainFields: ["module"],
},
build: {
outDir: "../../dist/apps/frontend/client",
reportCompressedSize: true,
commonjsOptions: { transformMixedEsModules: true },
target: ["es2020"],
},
plugins: [analog(), nxViteTsPaths(), splitVendorChunkPlugin(), nitro()],
test: {
globals: true,
environment: "jsdom",
setupFiles: ["src/test-setup.ts"],
include: ["**/*.spec.ts"],
reporters: ["default"],
cache: {
dir: ../../node_modules/.vitest,
},
},
define: {
"import.meta.vitest": mode !== "production",
},
};
});

The nitro plugin doesn't need to be there, its already included with the analog plugin

Here is repo: , Even without nitro() im getting error:

I cloned your repo, removed nitro, and everything built fine for me, but I'm not on Windows. Can you try it with WSL also?


Cosmetic Bug: Need to underline all content links for WP standard

For some reason, all the content links aren't underlined which won't pass 1.4.1 as Koji mentioned.

This issue is captured by https://github.com/cagov/cannabis.ca.gov/issues/132

Work in progress, will push it to dev in next minor release


Return Boolean needs clarification

What does "Returns a Boolean indicating characters in cells within the specified range." mean?
Running this function on a sheet with matches returns true.
Running this function on a sheet with no matches returns true.
Running this function on a lock sheet displays an alert saying this function can't run on a lock sheet, and still returns true.
What does the Boolean represent?

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

ID: 887c0d0a-9936-e2b4-98dd-88d9fdcd3818
Version Independent ID: 382a45f8-d13d-3efa-45cf-b431d28a2409
Content: Range.Replace method (Excel)
Content Source: api/Excel.Range.Replace.md
Product: excel
GitHub Login: @o365devx
Microsoft Alias: o365devx

@AlpsDevABE Thanks for your feedback. We'll look into updating the documentation. In the meantime, try the VBA developer community on Stack Overflow (https://stackoverflow.com/questions/tagged/vba) where you can get help from actual developers. If you find a bug in our documentation, please let us know! Thanks!


Update minimum npgsql to 4.1.5

Changes: https://github.com/npgsql/npgsql/milestone/70?closed=1
Most importantly, this fixes the long-running problem of combining NodaTime with BCL DateTime types.

Note: The custom type mappings in the nodatime package should be able to be binned once 4.1.6 is released, as 4.1.5 missed a mapping that the event stream requires. I'll create a PR whenever it's available.

This change is already added to 4.x so cherry picking to merge from 3.x is not required.


Cleanup

I just run the tool pep8 against all the files.
The most annoying part was to limit lines at 80 columns

Much of the value of PEP-8 is to stop people arguing about inconsequential formatting rules, and get on with writing good, consistently formatted code.
I suggest that we adopt PEP-8 for many things , but not for line length. I think 100 or even 120 columns is best.
We do that work because most people work using 16:9 widescreen.

Try running pep8 with --max-line-length=120  :)

Try running pep8 with --max-line-length=120
I wish I knew that a few hours ago :)
Do you think I should drop and redo the changes or it is fine anyway?
Definitely from now on I will use this flag!

It is fine as it is. ;)
We'll just make an agreement that every time we visit a file, we pep8 it with 120 line length.

Created coding guidelines suggesting to use pep8 with 120 line lenght: https://github.com/Mindwerks/worldengine/wiki/Coding-guidelines
Thanks!


fix: wasm provider errors

Since #344 we can no longer compile to wasm due to what appears to be a bug in alloy.
alloy issue for tracking: alloy-rs/alloy#1147

fixed in #403
Need to figure out a better retry strategy now though as we had to disable retries.


Exposed password to repository

Hello there !
We exposed one of our environment test user password to our repository with exported insomnia collection
https://github.com/catenax-ng/tx-traceability-foss
it is already in one of the past commits, neverthen less we should change password for exposed user.
exposed password is for following client id - "sa-cl6-cx-10".
What is the process in that situation. is reseting password for that user enough ?
Best regards

@the-tatanka @Essenbreis can you please take a look?

The Secret must be revoked!
Additionally, the secret can be removed from the commit history. A rebase is necessary for this. From my point of view this is a clear recommendation, but not a must.

Depending on how critical the secret is and how long it was published, you should also check your access logs. An attacker could have accessed your systems.

We did not notice any suspicious activity in our test environments. neverthen less we need this password to be changed so we could update it in our vault so our integration tests and other related processes are working with new secure password.

secret revoked

secret revoked

@the-tatanka we found this issue last week and, as a result, a new client secret for the client 'sa-cl6-cx-10' was generated in Keycloak (TRACEFOSS-1524).
As you aren't a Keycloak admin, what did you mean here when stating 'secret revoked'?

I assumed that the honorable colleague @ds-ext-sceronik  had taken care of that or set that in motion.
Apparently not.
The "secret revoked" was noted for me as a reminder, as I thought the issue was settled.
Thanks for pointing this out, please revoke secret.

I assumed that the honorable colleague @ds-ext-sceronik had taken care of that or set that in motion. Apparently not. The "secret revoked" was noted for me as a reminder, as I thought the issue was settled.
Thanks for pointing this out, please revoke secret.

The secret has been regenerated and, as a result, revoked last.
Thanks for clarifying! I just wanted to make sure, that there is awareness about the necessity to revoke it in Keycloak.


Edit context menu appears only for utf-8 files in project explorer

Description of your problem
I created a new project from an existing folder. Some .py-files in the Project explorer had an "Edit" context menu, others only an "Open" context menu. Drag and drop into the editor would also not work.
A short look in the sources revealed that in explorer.py line 261:
def create_file_manage_actions
only utf encoded files are accepted as editable.
I have no problem with this behavior but it should be easy to find in the documentation.
What steps will reproduce the problem?

Create a .py file with ANSI encoding and try the context menu

Versions and main components

Spyder Version: 3.0.1
Python Version: 3.5.2
Operating system: Win 10

Dependencies
Please go to the menu entry Help > Optional Dependencies (or
Help > Dependencies), press the button Copy to clipboard
and paste the contents below:
jedi >=0.8.1     :  0.9.0 (OK)
matplotlib >=1.0 :  1.5.3 (OK)
nbconvert >=4.0  :  4.2.0 (OK)
numpy >=1.7      :  1.11.2 (OK)
pandas >=0.13.1  :  0.19.0 (OK)
pep8 >=0.6       :  1.7.0 (OK)
pyflakes >=0.6.0 :  1.3.0 (OK)
pygments >=2.0   :  2.1.3 (OK)
pylint >=0.25    :  1.5.4 (OK)
qtconsole >=4.2.0:  4.2.1 (OK)
rope >=0.9.4     :  0.9.4-1 (OK)
sphinx >=0.6.6   :  1.3.1 (OK)
sympy >=0.7.3    :  1.0 (OK)

An easy one for you @rlaverde :-)

I can't reproduce this. @tsadowski, could you send us a simple file that's failing for you?

Don't worry about it, I found how to reproduce this error.


Group routes

What's this PR do?
This pull request enables v3/group API routes in Northstar 🎉
All the business logic was already copied over from Rogue, so this pull request just worked on getting the tests for these routes working again! ✅
How should this be reviewed?
👀
Relevant tickets
References Pivotal #176851327.
Checklist

[x] This PR has been added to the relevant Pivotal card.
[ ] Documentation added for new features/changed endpoints.
[ ] Added appropriate feature/unit tests.
[ ] If new attributes were added to users, there is a corresponding PR to surface these in Aurora.
[ ] If new attributes were added to users, then the data team already knows about these changes.

Thanks @aaronschachter for the 👀 !


Chat shows Sign In even though I'm signed in to GitHub

Type: Bug
Recently started seeing Copilot Chat window showing Sign In button even though I'm signed in

The Output shows GitHub login failed

Extension version: 0.16.1
VS Code version: Code 1.90.2 (5437499feb04f7a586f677b155b039bc2b3669eb, 2024-06-18T22:34:26.404Z)
OS version: Windows_NT x64 10.0.22631
Modes:

System Info

Item
Value

CPUs
AMD Ryzen 9 5950X 16-Core Processor             (32 x 3394)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
63.92GB (18.86GB free)

Process Argv
--folder-uri file:///c%3A/Users/viktordikov/OneDrive%20-%20Microsoft/Documents/Docs --crash-reporter-id 20eadb33-7708-475b-ab95-b7740165afae

Screen Reader
no

VM
0%

A/B Experiments
vsliv368:30146709
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscod805:30301674
binariesv615:30325510
vsaa593:30376534
py29gd2263:31024239
c4g48928:30535728
azure-dev_surveyone:30548225
a9j8j154:30646983
962ge761:30959799
pythongtdpath:30769146
welcomedialog:30910333
pythonnoceb:30805159
asynctok:30898717
pythonregdiag2:30936856
pythonmypyd1:30879173
h48ei257:31000450
pythontbext0:30879054
accentitlementst:30995554
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302
cppperfnew:31000557
dsvsc020:30976470
pythonait:31006305
jchc7451:31067544
chatpanelc:31048052
dsvsc021:30996838
0ee40948:31013168
pythoncenvpt:31062603
a69g1124:31058053
dvdeprecation:31068756
dwnewjupyter:31046869
2f103344:31071589
impr_priority:31082723

It may have something to do with  multiple github accounts ,mine worked after i set a preferred account under extension settings

I have the same issue but nothing is helping.

I signed in but yet is keep failing with this outpot
I keep having this issue.
2024-07-02 11:27:24.272 [error] [auth] Retrying auth 2024-07-02 11:27:57.618 [error] [auth] The operation was aborted.

2024-07-02 11:35:32.376 [error] [default] Error sending telemetry AbortError: The operation was aborted.
at fetch (/home/yadinbk/.vscode-server/extensions/github.copilot-1.208.963/node_modules/@adobe/helix-fetch/src/fetch/index.js:96:13)
at processTicksAndRejections (node:internal/process/task_queues:95:5)
at cachingFetch (/home/yadinbk/.vscode-server/extensions/github.copilot-1.208.963/node_modules/@adobe/helix-fetch/src/fetch/index.js:288:16)
at Dq.fetch (/home/yadinbk/.vscode-server/extensions/github.copilot-1.208.963/lib/src/network/helix.ts:88:22) {
type: 'aborted',
_name: 'AbortError'
}
2024-07-02 11:35:46.017 [error] [auth] The operation was aborted.
2024-07-02 11:35:47.714 [error] [auth] Retrying auth
2024-07-02 11:36:43.297 [error] [auth] The operation was aborted.
2024-07-02 11:38:11.617 [error] [auth] Retrying auth
2024-07-02 11:38:45.398 [error] [auth] The operation was aborted.
2024-07-02 11:38:53.789 [error] [auth] Retrying auth
2024-07-02 11:40:29.488 [error] [auth] The operation was aborted.

It may have something to do with multiple github accounts ,mine worked after i set a preferred account under extension settings

Ah yep that worked once I set the github.copilot.preferredAccount to the one that has the Copilot subscription it worked.

mine even shows Im loggedin:
2024-08-21 13:18:38.603 [info] [certificates] Removed 3 expired certificates
2024-08-21 13:18:40.053 [info] [fetcher] Using Helix fetcher, Electron fetcher is not available.
2024-08-21 13:18:40.053 [info] [code-referencing] Public code references are enabled.
2024-08-21 13:18:40.170 [info] [auth] Sucessfully authenticated
2024-08-21 13:19:59.903 [info] [fetcher] Using Helix fetcher, Electron fetcher is not available.
2024-08-21 13:19:59.903 [info] [code-referencing] Public code references are enabled.
2024-08-21 13:19:59.910 [info] [fetcher] Using Helix fetcher, Electron fetcher is not available.
2024-08-21 13:19:59.910 [info] [code-referencing] Public code references are enabled.

But still shows me a sign in button

This is a duplicate of https://github.com/microsoft/vscode-copilot-release/issues/1444#issuecomment-2299464817
It's a bug and I have a fix for this Insiders & Pre-release of Copilot Chat.


bump the addon framework to v0.10.0

Summary
The controller which adds ownerReference to managedclusteraddon is moved from addon-framework to addon-manager. So, the integration test might have failed with addon-framework v0.10.0+, Add for the future. We will use the addon-manager to manage the lifecycle of the addon agent.
Related issue(s)
reference: https://github.com/stolostron/multicluster-global-hub/issues/1112
Fixes #
Tests

[ ] Unit/function tests have been added and incorporated into make unit-tests.
[ ] Integration tests have been added and incorporated into make integration-test.
[ ] E2E tests have been added and incorporated into make e2e-test-all.
[ ] List other manual tests you have done.

/lgtm


Batch normalizatino call

Hello people ,
I'm trying to implement a batch normalization function on an input of dimension { batch_size, features, height, width } what i want is normalizing images for each filter , in  c++ code the function is :
c_output(N, N, FIn, BATCH_SIZE);    c_mean(FIn);    c_variance(FIn);

for (int z = 0; z < FIn; ++z)
    {c_mean(z)=0;
for (int n = 0; n < BATCH_SIZE; ++n)
    for (int y = 0; y < N; ++y)
	for (int x = 0; x < N; ++x)		    c_mean(z) +=input(x, y, z, n)/(BATCH_SIZE*N*N) ; }
for (int z = 0; z < FIn; ++z)
    {c_variance(z)=0;
for (int n = 0; n < BATCH_SIZE; ++n)
    for (int y = 0; y < N; ++y)
	for (int x = 0; x < N; ++x)		    c_variance(z) +=(input(x, y, z, n)-c_mean(z))*(input(x, y, z, n)-c_mean(z))/(BATCH_SIZE*N*N) ; }

for (int n = 0; n < BATCH_SIZE; ++n)
for (int z = 0; z < FIn; ++z)
   for (int y = 0; y < N; ++y)
	for (int x = 0; x < N; ++x)
	    {resultfile <<(input(x, y, z, n)-c_mean(z))/sqrt(c_variance(z)); c_output(x, y, z, n)=(input(x, y, z, n)-c_mean(z))/sqrt(c_variance(z)); }

how could i have the same call using mkldnn primitive ?
Thank you

Thank you so much


Executor thread safe guarantee

Are these public functions thread safe?

enqueueRequest
awaitResponses
cancelRequest
shutdown
getLatestIterationStats
getNumResponsesReady

What happens if you call cancelRequest on a request that has already been canceled (or completed) (calling awaitResponses at the same time).

Calling cancelRequest with a requestId that already has already completed will return right away and a log message should get printed saying that the provided requestId has already terminated. Calling cancelRequest with a requestId that's in flight, and for which another thread is waiting for the responses using awaitResponses should cause awaitResponses to eventually return a response with result.isFinal = true.

Calling cancelRequest with a requestId that already has already completed will return right away and a log message should get printed saying that the provided requestId has already terminated. Calling cancelRequest with a requestId that's in flight, and for which another thread is waiting for the responses using awaitResponses should cause awaitResponses to eventually return a response with result.isFinal = true.

Thanks!

@pcastonguay same question, is enqueueRequest thread safe. What if I have 2 or more threads calling enqueueRequest concurrently?


Update README to document the new package layout and how to use it

The markdown spec allows numeric lists to be specified with a 1. at the start of each line. I generally prefer this, since it allows us to add items to the middle of the list without having to commit a change for every following item. IntelliJ-based IDEs typically warn about this, since they can automatically update the numbers. Most people will be viewing a README via GitHub's interface or an IDE that should recognize markdown.


update non-core docs to use nautobot.apps.jobs module

ref: #4460
What's Changed
Updated the App Developer Guide and Job Developer Guide to use references from nautobot.apps.jobs instead of their core locations. This will allow for a stable API for Apps/Jobs while the core developers can continue to iterate.

I've made the recommended changes as well as some of what you pointed out in #4460. Since I've opened the discussion item around the Jobs API I've gone ahead and removed the TODO items from this PR.

Thanks for catching those items @glennmatthews, I've updated it.


Override of spring consul config not working?

I have properties in my consul key value store that are loaded properly by my spring boot app.  However, overriding a Consul property by specifying a value on the command line does not seem to work.  Here are the details:  My consul data source contains a property "a.b.c" with value "foo".   Spring boot reads this value correctly.  However, when I run my app like so:
java -jar api.jar --a.b.c="bar"
I am expecting to see the value "bar". Instead, I still see "foo".
Notice that if the property "x.y.z" is set in bootstrap.yml to "hello", and overridden on the command line like so:
java -jar api.jar --x.y.z="world"
the value of "x.y.z" is correctly reported as "world".
Is property overriding supported for properties in Consul key value store?

The consul config is given the highest priority by default.  You can set spring.cloud.config.override-none=true and it will make external config the lowest priority. You can set spring.cloud.config.override-system-properties=false which will put them below system properties, which will also make cli args higher priority.

Thanks @spencergibb our team spent a while trying to figure out how to get this working before stumbling on this issue. Would you accept a PR to add the above to the docs? I would also add that these properties need to be set in Consul. I had originally tried setting them in bootstrap.properties


⚠ Bofrost.fr has degraded performance

In 0b1faeb, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1368 ms

Resolved: Bofrost.fr performance has improved in 6bbbb64 after 13 minutes.


PyCharm installer mangles the console output

When installing within a Python project (I used Misago) in PyCharm, the console used for the installer CLI has messy and unreadable text breaks:

:tada: This issue has been resolved in version 0.36.3 :tada:
The release is available on:

v0.36.3
GitHub release

Your semantic-release bot :package::rocket:

@jansorg this issue from June looks identical to the issue we discovered recently. Do you have any idea why it would have regressed?

@brikelly could you link the other issue, please?
This issue is about garbled output, the only other issue I know was about a json parsing error in the pycharm console.

The other issue is this one: https://github.com/getappmap/appmap-js/issues/1308


Deduplicate identical environments in a pipeline.

This is especially useful when external transforms from the same environment
are used multiple times.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @chamikaramj

Run Python PreCommit

Run Python_PVR_Flink PreCommit

Container build failed pulling licenses.

Run Python_PVR_Flink PreCommit


Fix OS test condition and RedHat ansible distribution name

Description:
Task with name "- name: Compare host OS with supported matrix" is currently being skipped most of the time. This is because "skip_os_test" is undefined and because "ansible_distribution_version | int not in os_versions" is failing as well. The transformation to "INT" means that only the major version is kept (E.g.: RedHat 8.6 >> 8). Since "8" is never present in "os_versions" (which contains 8.2, 8.5, 8.6, etc.), then this check is always skipped. According to the documentation, it is supposed to be the opposite, meaning that it's always executed, except if "-i skip_os_test=true" is added as extra var.
Also, the correct ansible distribution for RedHat is "RedHat" and not "RHEL". This causes the "supported_os" to not find anything in case the target OS is RedHat, c.f. https://docs.ansible.com/ansible/latest/playbook_guide/playbooks_conditionals.html#ansible-facts-distribution.
Solution:
This PR includes the switch from "RHEL" to "RedHat" in the group_vars/all.yml file as well as the fix of the conditions so that the check is always executed and gives the expected result, meaning:

failure if distribution/version doesn't match what is defined in the group_vars/all.yml file
skipping if distribution/version matches OR if skip_os_test=true

Please review

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Morgan Patou seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Works for me with this alternative definition. I quickly tested the different possibilities (supported, unsupported, without and with false/true for the skip_os_test), no problems spotted.

There are a few test failures that expose that we missed updating the supported os matrix due to the bug you just fixed.
Rocky 8.6 and 8.7 should be added for sure, not sure about CentOS 7.7 that is an old version and I suppose we need to bump the AMI we are testing against.
up to you if you want to try to tackle also this last issues in order to obtain a green build, or just put this PR on hold and I can try to take a look soon and raise another PR to fix those tests (internally tracked as OPSEXP-2047).

If needed I can modify the group_vars/all.yml to include Rocky and CentOS versions mentioned but I do not know if you have any other steps to perform to include a new OS? Do you have some guidelines with the list of all steps? I would also need some details on which ACS versions are supported on these 2 OS, so I can update the doc as well. Currently CentOS 7.7 is marked as supported for all ACS versions from 6.2 to 7.3, including Community as well.

If needed I can modify the group_vars/all.yml to include Rocky and CentOS versions mentioned but I do not know if you have any other steps to perform to include a new OS?

You should just really need to add Rocky 8.6, Rocky 8.7, CentOS 7.7
docs should be fine https://github.com/Alfresco/alfresco-ansible-deployment/blob/master/docs/README.md#versioning
thanks again!


debug

Similar to https://github.com/crate/crate/pull/13536
https://wacklig.pipifein.dev/github/crate/crate/case/io.crate.integrationtests.MetadataTrackerITest/test_schema_changes_of_subscribed_table_is_replicated_and_new_data_is_synced
https://wacklig.pipifein.dev/github/crate/crate/case/io.crate.integrationtests.MetadataTrackerITest/test_schema_changes_of_multiple_tables_are_replicated
It's possible to reproduce it locally as well with 500 repetitions. It got better with 35/40 (started to see at least one added column) but in order to pass in multiple iterations, had to set 50.

Failed runs on this PR before increasing assertBusy timeout:
https://github.com/crate/crate/actions/runs/6544421167/job/17770850029?pr=14902
https://jenkins.crate.io/job/CrateDB/job/crate_on_pr/10723/testReport/junit/io.crate.integrationtests/MetadataTrackerITest/___/

the gap in duration between successful run and failed run are quite high. Are you sure this is hitting the timeout too early?

the gap in duration between successful run and failed run are quite high. Are you sure this is hitting the timeout too early?

You mean that if in average 5 seconds is enough to replicate all schema changes, then we need to figure out why have some spikes?
I will have another look. So far, increased timeout helps which means mechanism itself is working but sometimes has a huge delay. I will look what could be the root cause of that occasional replication speed drops.

I will have another look. So far, increased timeout helps which means mechanism itself is working but sometimes has a huge delay

In that case we could also merge the timeout increase to get the CI stable and continue investigating the huge delay in

In that case we could also merge the timeout increase to get the CI stable and continue investigating the huge delay

It passed locally but on CI even 50 seconds doesn't work. We cannot merge it, there is some issue with schema changes being not visible, will investigate further.


5.9: [DCE] Destroys of live values are live.

As is done for end_borrows, add a dependency from a destroy_values to its operand so that it is not eliminated if its operand isn't.

@swift-ci please test

@swift-ci please test


Add contributing guidelines

As discussed in #118
Closes #118

:+1:

Where is the rule about github stars ??

IMO those aren't important.


Roll Flutter Engine from 202506d686e3 to 9384df4425fd (2 revisions)

https://github.com/flutter/engine/compare/202506d686e3...9384df4425fd
2024-11-22<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from zhFzwYCH-N_wasTnM... to D5CBHuB2c-v3Zai-c... (flutter/engine#56757)
2024-11-22<EMAIL_ADDRESS>Roll Dart SDK from 141291fd570d to 8b65a7a628e2 (1 revision) (flutter/engine#56755)
Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from zhFzwYCH-N_w to D5CBHuB2c-v3
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC aaclarke<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Reason for revert:
tree is red on " Linux_build_test flutter_gallery__transition_perf_hybrid" after this PR


Add docker environment

Looks good to me. Thank you!
In the Dockerfile, let's just make the dockerhub as the default option and comment the dockerproxy for those for who need it.

Looks good to me! In the Dockerfile, let's just make the dockerhub as the default option and comment the dockerproxy for those for who need it.

OK, my network needs a proxy to work.

Perfect. What I meant was to set the regular as default and leave the one with the proxy commented (with a comment explaining for those who need it)

Perfect. What I meant was to set the regular as default and leave the one with the proxy commented (with a comment explaining for those who need it)

Your suggestion is good, but it only works in my area.


Remove Vue from the project

Currently our UI relies on the reactive framework Vue. While there are benefits to this, I've decided to remove it from the codebase for the following reasons:

A lot of the functionality of the UI can be implemented in a much simpler manner without the need for a reactive framework.
Vue adds a considerably large amount of code to the final bundle.
Currently numerous shims are required for Vue to work along with additional build variables.
Components are more complicated than they need to be.
Vue currently conflicts with Bun and requires a hack-fix in the build pipeline.

Are you against the idea of using another framework in place of Vue? I've been using this one at work and it's pretty good: https://lit.dev/

That's interesting, I hadn't heard of that one before but I'll give it a look, I'm currently using Vue for a number of projects at work, so this might be a fit there. As for wow.export, this issue is stale and was created when we were making large refactors on the codebase, which we're not doing anymore.
As much as there are some pain points with Vue (or more-so with how we've used it in this project), we're not going to commit the time to make a refactor like this when it's not causing any major issues with the functionality of the software and it's not blocking any outstanding issues.
Going to close this issue as out-of-date.

That makes sense. I quite like Vue actually, but I've mainly used v3 with SFC and composition API, so I probably got a much better experience. The only reason I chose Lit instead at work was because it was an existing standard server-side rendered HTML page using basic standalone scripts doing manual DOM manipulation with jQuery, no build process for front-end, etc, basically a very old-style style application, so integrating Vue was not really feasible. I needed to implement some complex components though, and jQuery wouldn't really cut it (and I'm not about to seriously use it in 2024), so I found this Lit guy and used it to great success. I really recommend it if you need to incrementally add complex components to an existing project.


Mysql. The server requested authentication method unknown to the client [caching_sha2_password]

Info:

Docker version: 17.12.0-ce, build c97c6d6
Laradock commit: 13f55ef7967989337ed3538af3a8164268eeb623
System info: Win
System info disto/version: 10

Issue:
New fresh laradock installation. All default settings. Driver is shared.
Except:
DATA_SAVE_PATH=../../data

Can't connect to the mysql from phpMyAdmin.

#2054 - The server requested authentication method unknown to the client
mysqli_real_connect(): The server requested authentication method unknown to the client [caching_sha2_password]
mysqli_real_connect(): (HY000/2054): The server requested authentication method unknown to the client
But I can connect to mysql inside mysql container with standard root/root credentials.
I tried to delete all containers and images. Several times downloaded laradock. Tried to reinstall Docker. No success.

Expected behavior:
Can connect to mysql with phpMyAdmin

Logs
$ docker logs laradock_mysql_1
2018-02-23T13:22:10.682788Z 0 [Warning] [MY-011070] Disabling symbolic links using --skip-symbolic-links (or equivalent) is the default. Consider not using this option as it is deprecated and will be removed in a future release.
2018-02-23T13:22:10.682938Z 0 [Warning] [MY-010915] 'NO_ZERO_DATE', 'NO_ZERO_IN_DATE' and 'ERROR_FOR_DIVISION_BY_ZERO' sql modes should be used with strict mode. They will be merged with strict mode in a future release.
2018-02-23T13:22:10.695441Z 0 [System] [MY-010116] /usr/sbin/mysqld (mysqld 8.0.4-rc-log) starting as process 1 ...
mbind: Operation not permitted
mbind: Operation not permitted
2018-02-23T13:22:11.537504Z 0 [Warning] [MY-010068] CA certificate ca.pem is self signed.
2018-02-23T13:22:11.542367Z 0 [Warning] [MY-000000] Insecure configuration for --pid-file: Location '/var/run/mysqld' in the path is accessible to all OS users. Consider choosing a different directory.
2018-02-23T13:22:11.549449Z 0 [Warning] [MY-010315] 'user' entry 'mysql.infoschema@localhost' ignored in --skip-name-resolve mode.
2018-02-23T13:22:11.549556Z 0 [Warning] [MY-010315] 'user' entry 'mysql.session@localhost' ignored in --skip-name-resolve mode.
2018-02-23T13:22:11.549594Z 0 [Warning] [MY-010315] 'user' entry 'mysql.sys@localhost' ignored in --skip-name-resolve mode.
2018-02-23T13:22:11.549629Z 0 [Warning] [MY-010315] 'user' entry 'root@localhost' ignored in --skip-name-resolve mode.
2018-02-23T13:22:11.549676Z 0 [Warning] [MY-010323] 'db' entry 'performance_schema mysql.session@localhost' ignored in --skip-name-resolve mode.
2018-02-23T13:22:11.549727Z 0 [Warning] [MY-010323] 'db' entry 'sys mysql.sys@localhost' ignored in --skip-name-resolve mode.
2018-02-23T13:22:11.549779Z 0 [Warning] [MY-010311] 'proxies_priv' entry '@ root@localhost' ignored in --skip-name-resolve mode.
2018-02-23T13:22:11.565161Z 0 [Warning] [MY-010330] 'tables_priv' entry 'user mysql.session@localhost' ignored in --skip-name-resolve mode.
2018-02-23T13:22:11.565260Z 0 [Warning] [MY-010330] 'tables_priv' entry 'sys_config mysql.sys@localhost' ignored in --skip-name-resolve mode.
2018-02-23T13:22:11.570621Z 0 [System] [MY-010931] /usr/sbin/mysqld: ready for connections. Version: '8.0.4-rc-log'  socket: '/var/run/mysqld/mysqld.sock'  port: 3306  MySQL Community Server (GPL).

Root cause
https://mysqlserverteam.com/mysql-8-0-4-new-default-authentication-plugin-caching_sha2_password/
Corrective action
Downgrade mysql version

Don't uninstall MySQL latest version. Rather follow these steps https://gist.github.com/benlinton/d24471729ed6c2ace731
Basically you need to unlink latest installed mysql, install older MSQL version and link it.

If you want to use mysql 8 with native password (the old way) you can follow the following steps:
Add this to my.cnf in laradock\mysql\my.cnf under [mysqld]:
default-authentication-plugin=mysql_native_password
Clear the data-folder (will destroy your data), rebuild the mysql container and it should work.

alter user 'username'@'localhost' identified with mysql_native_password by 'password'; would fix it.

alter user 'username'@'localhost' identified with mysql_native_password by 'password';
Bravoo. This is the solution

Hi Mr. Ailionx and Mr. Najamalvi,
I'm a student in a country which don't speak English. I'm very happy when reading your guide but I don't understand what steps I should do. Please tell me more details. Thank so much !!!

Don't uninstall MySQL 8.  Sign into mysql using a terminal program:
mysql -u root -p
my_password
alter user 'username'@'localhost' identified with mysql_native_password by 'password';
This does the trick.

alter user 'username'@'localhost' identified with mysql_native_password by 'password'; would fix it.

work for me, thank you!

It also work for me. Many many thanks for you. Have a nice day!!!
On Sat, Oct 27, 2018 at 9:55 AM William Alexander<EMAIL_ADDRESS>wrote:

alter user 'username'@'localhost' identified with mysql_native_password
by 'password'; would fix it.
work for me, thank you!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/laradock/laradock/issues/1390#issuecomment-433585330,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AqZUP-QX0NTNn9J98RuhesSqjAuiuGnWks5uo8s6gaJpZM4SQ5gP
.

added these lines in docker-compose.yml fixed my issue:
    mysql:
        image: mysql:8
        command: ['--character-set-server=utf8mb4', '--collation-server=utf8mb4_unicode_ci','--default-authentication-plugin=mysql_native_password']

You do not have to uninstall MySQL 8.x. You just need to go to your MySQL installer, reconfigure your server, and select Legacy Authentication Method (which retains MySQL 5.x compatibility). I figured this out through trial and error after my dsn in my PDO didn't allow me to connect.
https://i.imgur.com/u4giuUM.png

alter user 'username'@'localhost' identified with mysql_native_password by 'password'; would fix it.

you saved my life.
I changed default auth plugin to mysql native password in /etc/my.cnf and then used your command.
my problem solved

if external/remote use replace 'localhost' to '%'

added these lines in docker-compose.yml fixed my issue:
    mysql:
        image: mysql:8
        command: ['--character-set-server=utf8mb4', '--collation-server=utf8mb4_unicode_ci','--default-authentication-plugin=mysql_native_password']

You just saved my life @zacksleo, Thanks. Installing latest drupal 8.6.4 with MySQL 8.0.13 both via docker.

If you want to use mysql 8 with native password (the old way) you can follow the following steps:
Add this to my.cnf in laradock\mysql\my.cnf under [mysqld]:
default-authentication-plugin=mysql_native_password
Clear the data-folder (will destroy your data), rebuild the mysql container and it should work.

Just in case someone else is using laradock too, by "clear the data-folder" I guess is:
   $ rm -rf ~/.laradock/data

just remove damn mysql and use mariadb 10.x as main database, it works fine.

ERROR 1396 (HY000): Operation ALTER USER failed for 'username'@'localhost'
I am getting this error when I write this :

alter user 'username'@'localhost' identified with mysql_native_password by 'password';

alter user 'username'@'localhost' identified with mysql_native_password by 'password'; This one helps me. Thank you!

mysql> ALTER USER 'root'@'localhost' IDENTIFIED WITH mysql_native_password BY 'rootpassword';
login through command line, it works after that

Access denied for user 'root'@'localhost' (using password: YES)
i have already used ALTER USER 'root'@'localhost' IDENTIFIED WITH mysql_native_password BY 'rootpassword';
what to do now??

If you want to use mysql 8 with native password (the old way) you can follow the following steps:
Add this to my.cnf in laradock\mysql\my.cnf under [mysqld]:
default-authentication-plugin=mysql_native_password
Clear the data-folder (will destroy your data), rebuild the mysql container and it should work.

Just in case someone else is using laradock too, by "clear the data-folder" I guess is:
   $ rm -rf ~/.laradock/data

Thanks so much.

Basic PDO user with a possible solution. You do not have to uninstall MySQL 8.x or even do the above (maybe... I'm not using Laradock). You just need to go to your MySQL installer, reconfigure your server, and select Legacy Authentication Method (which retains MySQL 5.x compatibility). I figured this out through trial and error after my dsn in my PDO didn't allow me to connect.
Finding the Installer in Windows
Click 'Reconfigure'
Select the bottom option

Worked fine! Thank you

If you want to use mysql 8 with native password (the old way) you can follow the following steps:
Add this to my.cnf in laradock\mysql\my.cnf under [mysqld]:
default-authentication-plugin=mysql_native_password
Clear the data-folder (will destroy your data), rebuild the mysql container and it should work.

Just in case someone else is using laradock too, by "clear the data-folder" I guess is:
   $ rm -rf ~/.laradock/data

This is working, but to clarify the things a bit:

if you don't find the .laradock folder and you are on Windows look in the C:\Users\YourProfileName, it should be there, on Linux try the logged user's folder (root if you logging in with it)
build with docker-compose build --no-cache mysql and restart containers (down & up)
don't forget to login in phpmyadmin with mysql/root/root or whatever you have in your .env file (mysql/default/secret)

Don't uninstall MySQL 8. Sign into mysql using a terminal program:
mysql -u root -p
my_password
alter user 'username'@'localhost' identified with mysql_native_password by 'password';
This does the trick.

well done! thanks

if you use mysql latest version in docker container, you can type this command into docker-compose.yaml
` mysql:
image: mysql:latest
command: --default-authentication-plugin=mysql_native_password
ports:
  - 3306:3306`

or use oldest version mysql.

ALTER USER 'root'@'localhost' IDENTIFIED WITH mysql_native_password BY 'root';
ALTER USER 'root'@'%' IDENTIFIED WITH mysql_native_password BY 'root';
ALTER USER 'default'@'%' IDENTIFIED WITH mysql_native_password BY 'secret';

source: https://stackoverflow.com/questions/52364415/php-with-mysql-8-0-error-the-server-requested-authentication-method-unknown-to

This worked for me on CentosOS.
ALTER USER 'root'@'localhost' IDENTIFIED WITH mysql_native_password BY 'root';
ALTER USER 'root'@'%' IDENTIFIED WITH mysql_native_password BY 'root';
ALTER USER 'default'@'%' IDENTIFIED WITH mysql_native_password BY 'secret';

source: https://stackoverflow.com/questions/52364415/php-with-mysql-8-0-error-the-server-requested-authentication-method-unknown-to

This works very well for me :D

Its work for me !!!
Don't uninstall MySQL 8. Sign into mysql using a terminal program:
mysql -u root -p
my_password
alter user 'username'@'localhost' identified with mysql_native_password by 'password';

alter user 'username'@'localhost' identified with mysql_native_password by 'password'; would fix it.

This worked for me. Thank you

Basic PDO user with a possible solution. You do not have to uninstall MySQL 8.x or even do the above (maybe... I'm not using Laradock). You just need to go to your MySQL installer, reconfigure your server, and select Legacy Authentication Method (which retains MySQL 5.x compatibility). I figured this out through trial and error after my dsn in my PDO didn't allow me to connect.
Finding the Installer in Windows
Click 'Reconfigure'
Select the bottom option

This really helps me fix my problem! Thanks for sharing!!!

For Mac users, just go to System Preference, MySql panel, initialize Database, if use "strong password encryption" is selected choose "Use Legacy Password encryption", insert the root password and then click ok. It will require your pc password. Thats all.

or You can follow this video if you want or reset mysql password
https://www.youtube.com/watch?v=T5wg7ekMje0

alter user 'username'@'localhost' identified with mysql_native_password by 'password'; would fix it.

That works for me as well!! Thanks brow!

if you are facing "caching_sha2_password" error during connectivity of your mysql database using python ,
go through this
https://www.youtube.com/watch?v=YNq-EuQEJos

You can also fix this issue using phpmyadmin:
navigate to users->
change password->
enter your new password
make sure to select "Native MYSQL authentication" on the drop down "password hashing" menu

alter user 'username'@'localhost' identified with mysql_native_password by 'password'; would fix it.

Amazing😄

Add following line under [mysqld] of your mysql configuration file, so called my.cnf
default-authentication-plugin=mysql_native_password

alter user 'username'@'localhost' identified with mysql_native_password by 'password';

thank you...it worked for me

It's work for me too !!
Thank you

Root cause
https://mysqlserverteam.com/mysql-8-0-4-new-default-authentication-plugin-caching_sha2_password/
Corrective action
Downgrade mysql version

Thanks to tell the root of the problem :)

Don't uninstall MySQL 8. Sign into mysql using a terminal program:
mysql -u root -p
my_password
alter user 'username'@'localhost' identified with mysql_native_password by 'password';
This does the trick.

Thankyou so much <3

Changing Auth Method for User didn't worked for me, but changing standard Auth method of MySQL 8 - did.
Add lines:
[mysqld]
default-authentication-plugin=mysql_native_password
At the end of:
/etc/mysql/my.cnf

if you are using Mysql 8.0.+ on Mac,
Uninstall Mysql
re-install it and choose 'legacy password'
problem solved for me.

This worked for me https://stackoverflow.com/questions/49194719/authentication-plugin-caching-sha2-password-cannot-be-loaded

in mysql 8.4
command: [
      '--character-set-server=utf8mb4',
      '--mysql-native-password=ON',
      '--collation-server=utf8mb4_unicode_ci'
    ]


Update chunking for knowledge documents

Currently we have a fixed chunking (every N words/tokens), this has issues with hanging sentences, table and code. We want a better way to do this chunking, something which is context aware. Would be good to add this to our list of todos

Demonstrated new chunking methods in replace of RecursiveCharacterTextSplitter()
Notable updates:

Used an efficient library to detect file type.
Applied document-specific test splitter from Langchain in replace of original naive version.
Made heuristics changes to markdown file, especially using regex to trim markdown tables in attempt to fit in the whole table with limited context window.
For updated chunk_document() function, see Chunking_Demo.ipynb on chunking with server_ctx_size=4096, chunk_word_count=1024). Granite 7b has 4k context windows.

Pull Request: https://github.com/instructlab/sdg/pull/45

resolved by #65


Can I use it as python liblary?

What exactly would you like to use as a python library? Maybe you want to look at the models converted to pyTorch that have been integrated into the huggingface transformers library?


Swagger from file

Hi,
This is a proposal to decode a special line in a docstring swagger_from_file: path.
I think it could be useful sometimes to take the yaml swagger spec out of the docstring:

if you have other things to say in the docstring,
if it's easier to edit yaml code from a .yml file.

In the pull request you'll find besides flask_swagger.py a modified example and README.md
Please criticize as needed.
Thanks.

Otherwise looks good and thanks for contributing!

Thanks for your review.
You're right, it's better to leave this off by default.
I've just committed the change.


chore: Refactor feature flag service to remove the dependency from tenant service

Description
We are facing the cyclical dependecy issue with tenant service and feature flag service on EE codebase. This is required to force update feature flag whenever the tenant license is updated. This PR updates the implementation to use tenantId reference in feature flag service to fix the above mentioned issue.
Type of change

Chore (housekeeping or task changes that don't impact user perception)

How Has This Been Tested?

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

[x] Manual
[ ] JUnit

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

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

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

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

/ok-to-test

/ok-to-test

/ok-to-test


Added UDP tests

There is one baseline test, and the rest involve outputting
fragmented UDP. Both IPv4 and IPv6 fragmented output are tested.
Fragmented input is also tested.
Both vlan and non-vlan tests are provided (with equal coverage).

LGTM.


wip

Description
Changes proposed in this pull request:

...
...
...

Related issue(s)

/hold

/test pre-kyma-environment-broker

/test pre-kyma-environment-broker

/test pre-kyma-environment-broker

/test pre-kyma-environment-broker

/test pre-kyma-environment-broker

/test pre-kyma-environment-broker

/test pre-kyma-environment-broker

/unhold


render_preview is not respecting the default_preview_layout

I've set config.view_component.default_preview_layout = "component_preview" in my development.rb and it's correctly been used when I open the previews (through Lookbook).
Now I'm starting to write tests on my components and I've discovered that there is the render_preview which allows to use the previews. But when I try to render any component it gets rendered with the application  layout instead of component_preview, which in my case causes errors as there are some methods there that are not available at the component level.
If I explicitly set the layout 'component_preview' in the component itself it works, but I think that given the global config it should work with that one as well.

Your tests will be run with Rails.env set to "test", so config/environments/development.rb won't be run and thus this setting won't take effect. You can move that setting into config/application.rb, and optionally limit the environments in which it'll take effect:
config.view_component.default_preview_layout = "component_preview" if Rails.env.in? %w[development production]


State diagram failed to render \n in transition labels compared to plantuml

Given
stateDiagram

[*] --> S1
S1 --> S2: long line using html
shouldn't work
S1 --> S3: long line using escape\nshould work

renders

but with plantuml given
@startuml

[*] --> S1
S1 --> S2: long line using html
shouldn't work
S1 --> S3: long line using escape\nshould work

@enduml

renders

Kept as a bug since, I believe, the original intention was to utilize plantuml's syntax.

I found that
 works instead of

I will make  and  work even though they won't in plantUML. This is they work in sequence diagrams so it will be consistent in that sense.
New lines there are a bit trickier... will need to think about those, they will mess upp the grammar a bit as this:

[*] --> S1
S1 --> S2: long line using html
shouldn't work
S1 --> S3: long line using escape\nshould work

is really the same as the code below when mermaid receives it. Should work will be poercieved as two new states.
[*] --> S1
S1 --> S2: long line using html
shouldn't work
S1 --> S3: long line using escape
should work

Then you are missing an encoding step between the raw text and by the time mermaid gets it.
Makes me wonder if there's a security issue if there's a form of automatic interpretation of \n. What else is unintentionally interpreted?

PlantUML automatically interprets \n. (the escape character, not the literal ␤)

In this case it will be up to the integrator to handle the escaping but then br, br/ and \n could be supported.

Part of release 8.4.1


feat: add best practices check-deprecated-apis policy in CEL expressions

Related Issue(s)
Partially addresses #891

Description
PR to convert check-deprecated-apis policy in the best-practices folder to CEL

Checklist

[] I have read the policy contribution guidelines.
[] I have added test manifests and resources covering both positive and negative tests that prove this policy works as intended.
[] I have added the artifacthub-pkg.yml file and have verified it is complete and correct.

@MariamFahmy98 We probably have to wait for this issue (https://github.com/kyverno/kyverno/issues/10313) to be resolved before merging this PR. This policy won't function properly as of now (Comment link for the details: https://github.com/kyverno/policies/pull/1042#discussion_r1647560658)


SVGGeometryElement  is missing methods:  isPointInStroke, isPointInFill

see:
https://developer.mozilla.org/en-US/docs/Web/API/SVGGeometryElement/isPointInFill
https://developer.mozilla.org/en-US/docs/Web/API/SVGGeometryElement/isPointInStroke
Typescript: 4.3.2

mhmmm...
in the IDL it seems correct:
https://github.com/microsoft/TypeScript-DOM-lib-generator/blob/3a6fa3b6707dd65eb28c2b3b66e866a654ddcfbd/inputfiles/idl/SVG - Basic Data Types and Interfaces.widl#L36

I found the issue:
SVGpathElement has a wrong inhertiance chain
This is from lib.dom.d.ts

This is the expected chain:

so SVGPathElement should extend SVGGeometryElement and not SVGGraphicsElement

https://developer.mozilla.org/en-US/docs/Web/API/SVGPathElement

To expedite the triage process, we need everyone to follow the issue template and instructions.
When you clicked "Create New Issue", the issue form was pre-populated with a template and some instructions. We need you to read those instructions completely, follow them, and then fill in all the fields in that template.
We are not able to assist with issues that don't follow the template instructions as they represent a significantly larger amount of work compared to issues which are correctly specified. Thank you for understanding.


feat(helm-chart): remove helm-chart support

Resolve #108

@danhuawang @xunliu would you like to review this PR ?

@unknowntpo I will review this PR today, Thanks.

Do we still need the following code in init/spark/init.sh ?
# inject host information
sed -i 's/__GRAVITINO_HOST_IP__/'"$GRAVITINO_HOST_IP"'/g' /opt/spark/conf/spark-defaults.conf
sed -i 's/__HIVE_HOST_IP__/'"$HIVE_HOST_IP"'/g' /opt/spark/conf/spark-defaults.conf

Shall we remove the placeholder  "MYSQL_HOST_IP","HIVE_HOST_IP"  in init/gravitino/gravitino.conf ?  If removed, set env var  steps in init/gravitino/init.sh is also no longer needed.

Do we still need the following code in init/spark/init.sh ?
# inject host information
sed -i 's/__GRAVITINO_HOST_IP__/'"$GRAVITINO_HOST_IP"'/g' /opt/spark/conf/spark-defaults.conf
sed -i 's/__HIVE_HOST_IP__/'"$HIVE_HOST_IP"'/g' /opt/spark/conf/spark-defaults.conf

ok, removed.

Shall we remove the placeholder "MYSQL_HOST_IP","HIVE_HOST_IP" in init/gravitino/gravitino.conf ? If removed, set env var steps in init/gravitino/init.sh is also no longer needed.

ok, removed.

LGTM.
@xunliu


Login - There is No phone number is +15005550006 at the error when typed 2 digits and continue

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Issue found when executing PR https://github.com/Expensify/App/pull/8511
Action Performed:

Go to staging.new.expensify.com
Type 2 digits and continue
Verify that the phone number is +15005550006 at the error

Expected Result:
There is phone number is +15005550006 at the error when typed 2 digits and continue
Actual Result:
There is No phone number is +15005550006 at the error when typed 2 digits and continue
Workaround:
Unknown
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: No
Email or phone of affected tester (no customers): none
Logs: https://stackoverflow.com/c/expensify/questions/4856
https://user-images.githubusercontent.com/93399543/183114396-82f697dd-c4a5-4f2c-ab43-abf5406b917f.mp4
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

@kbecciv the video evidence for this is for a different issue? I cannot reproduce otherwise

@johnmlee101 Sorry about that, updated correct video.

Okay to clarify the issue is that the error message doesn't pop up for invalid numbers, not that "No phone number is +15005550006"

I think this is caused by https://github.com/Expensify/App/commit/cd4081848a849e9b9821509e86a21477057660d1 cc @yuwenmemon

Okay looked into this for a bit, but a lot has changed (and is changing) with regards to error handling and Onyx so I'm just going to revert that previous PR to get the error handling right

Reverted the offending PR


Add LiveTestBase class to the test framework

Fixes: https://github.com/Azure/azure-sdk-for-net/issues/15699

/azp run net - extensions


README/'Self' Validation Expressions - logical error in 3rd example

"* def isValidMonth = function(m) { return m >= 0 && m <= 12 }"
0 will be a valid number of month in this example

closing as wontfix. please be a little more clear next time

Hello,
sorry for the bad description.
you have an example in readme.md, near the line 2542:
And functions work as well ! You can imagine how you could evolve a nice
set of utilities that validate all your domain objects.
* def date = { month: 3 }
* def isValidMonth = function(m) { return m >= 0 && m <= 12 }
* match date == { month: '#? isValidMonth(_)' }

this statement is not perfect:
return m >= 0
( should be return m >= 1 or return m > 0 )
It's not a bug, or issue, but... a remark, and if you want, correct it :)
Thanks:
révész
On Tue, May 10, 2022 at 10:13 AM Peter Thomas @.***>
wrote:

closing as wontfix. please be a little more clear next time
—
Reply to this email directly, view it on GitHub
https://github.com/karatelabs/karate/issues/2006#issuecomment-1122072121,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIO3WSMGXRSUSFP47PUAM7LVJILCZANCNFSM5VQWI3BQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

thanks, made a commit/edit that will be on the site when we make the next release

I know it's just a trifle and doesn't affect the operation, but i think,
the precision required it.
have a nice day
révész
On Tue, May 10, 2022 at 12:31 PM Peter Thomas @.***>
wrote:

thanks, made a commit/edit that will be on the site when we make the next
release
—
Reply to this email directly, view it on GitHub
https://github.com/karatelabs/karate/issues/2006#issuecomment-1122212206,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIO3WSOK6I5LZYSM3BEXSQLVJI3GXANCNFSM5VQWI3BQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

@zumi000 just so you know, it saves a lot of time if you submit a pull-request for these kinds of things


When minecraft:empty is specified, it is output that datapack.env.dependsOnVanilla does not exist even though it is true

:tada: This issue has been resolved in version 3.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


pull request for e2e test at 11/11/2016 03:26:09

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Added notification of early failure

The fail() API of BootstrapManager was not called when the BS failed early (for example, when no  no BootstrapConfig exists.
(This could be useful for logging or reporting.)

Integrated in master (commits 387bee8bca180333b22782ed6b9ae74730524e7c,18d3271a61df6e8d2e171aae3f9f5d14b0eb9c53)


Missing module/folders in Label Only Jupyter Notebook

I'm trying to run the jupyter notebook label_only_membership_inference.ipynb, but I'm unable to the first cell and third cell. Seems that the models folder is not included in the current repo. Unsure if this is a problem on my end or the repo.
Thanks in advance,
Anand
First cell:
import torch
from torch import nn
import numpy as np
from models.mnist import Net
Third cell:
state_dict = torch.load("models/mnist_cnn.pt")
model = Net()
model.load_state_dict(state_dict)

Hi @anandseshadri1998 Thank you very much for using ART! I think you are right, this notebook does currently not provide a model to run, we'll try to add a model in the future. A short term solution for you would be to train a MNIST classification model in PyTorch and use it to run the notebook.
In case you create your own model, please let me know if you would like to contribute it to ART and fix this issue.


chore: add $attrs to vue2

chore: add $attrs to vue2

:hammer_and_wrench: Type Of Change

[ ] Fix
[ ] Feature
[x] Refactoring
[ ] Documentation

:book: Description
Attempting to put all storybook props on $attrs in vue 2 repo. For some reason in vue 2 you have to bind argTypes directly to props instead of just using args. Not sure exactly why this is, maybe a difference in how reactivity works in vue 2 vs vue 3.
Did this with a regex and then fixed a bunch of abnormal cases manually so it's definitely possible there are problems. I have not tested well yet. Going to rely on the visual tests first and go from there.
:bulb: Context
So we don't have to always update mundane things between vue 2 and vue 3

Alright, lookin good now. Going to get this in because it's quite conflicty.


Process crawler technical requrements and make architecture decision

https://docs.google.com/document/d/1RJG4Dg35DgT-6HLjLTmGrawzI3U2Kq6HBgESA2mtX_A/edit#
Process this document. Make sure - it is clear for you. You are welcome to ask questions in comments to google doc.
As result of this issue must be architecture decision posted to another google doc - so we can discuss it.

@vkuchyn Doc.

@vaa25 thanks, will review soon.

@vaa25 see my comments and process them. Need minor improvement

@vkuchyn answered to you there

@vaa25 shall I review again? It is not clear with uniq key for event - only problem declaration

@AndriyBaibak 4 hours

@vaa25 nice job. thanks!


Mention auto-complete

Closes #529
Support mention auto-complete in markdown input fields. Closely follows the suggestions in #529, which are largely inspired by GitHub's experience.

When @ is typed, show a list of top stackers for the day to pick from (5)
Use the text immediately following @ to search for similarly named users to pick from
Use up/down arrow to navigate between search options
Use Tab/Enter to pick the current selection
Use escape to close the dropdown
Selected user is inserted into the textarea input, and your cursor position is set to the end of the mention
Suggestions do not show up if there's non-whitespace characters immediately preceding @ e.g. x@
Suggestions are shown as you're deleting an existing mention, all the way back to @. Suggestions are based on the remaining mention text.
Suggestions appear below the caret position in the textarea, so it's easy to see but doesn't block your current text

Demo cc @ekzyis I believe this captures all of your input
Ignore the delay on searching users, that's just my slow machine :P
https://github.com/stackernews/stacker.news/assets/128755788/8883ccda-f542-4f26-9391-8da0cdd666bb

Thank you! This is going to be a big release!

YUGE 😄


Only run deletion reconciler on repo deleted events

Summary
The trigger for the reconciler was too greedy and would issue deletion events
on more entities than just repos. e.g. when a branch is deleted. This is not ideal.
Instead, this trims down the triggering of the event to just repo deletion events coming from GitHub.
Change Type
Mark the type of change your PR introduces:

[x] Bug fix (resolves an issue without affecting existing features)
[ ] Feature (adds new functionality without breaking changes)
[ ] Breaking change (may impact existing functionalities or require documentation updates)
[ ] Documentation (updates or additions to documentation)
[ ] Refactoring or test improvements (no bug fixes or new functionality)

Testing
Review Checklist:

[x] Reviewed my own code for quality and clarity.
[ ] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[ ] Included tests that validate the fix or feature.
[ ] Checked that related changes are merged.

coverage: 48.228% (+0.001%) from 48.227%
when pulling 01c471d7e935da5c43ff94679afcffb423e01901 on only-delete-repos-installations
into 350aa0f2f750c72fc67ec201f588fca6433dd46f on main.


Bucket Sort Added in Python

Description
Bucket sort, or bin sort, is a sorting algorithm that works by distributing the elements of an array into a number of buckets.
Fixes #991
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

Checklist:

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

@Ayush7614 @ravikr126 please review this...


Question on how to customize the className(s) of root and wrapper components

I have some complex styling for sizing components and need to set flex: none and flex-grow: 1 on the toolBarRoot and the wrapper around ContentEditable.
https://github.com/mdx-editor/editor/blob/5a101aee11ef4f0ed6b9b0efca50f832ff8bd49b/src/plugins/toolbar/index.tsx#L39C62-L39C62
It isn't clear to me how (or if) I can adjust them. From what I can tell, it isn't possible. Is the best course of action here to customize the @mdxeditor/editor/style.css style?

I assume you want to achieve something like this: https://codesandbox.io/s/sweet-cloud-pfs7sx?file=/src/styles.css.

You are correct! I've cheated and just added some custom css around the compiled class names.
._rootContentEditableWrapper_16b3d_991 {
    flex-grow: 1;
}

._toolbarRoot_16b3d_141 {
    flex: none;
}

Not knowing the code my first impression is that I would have to fork and add a custom hook to the toolbar plugin (and probably something similar for the wrapper, haven't really looked at that part of the code yet)

My brain clearly wasn't working this morning. I looked at your example and didn't even notice the flex-editor class. I've gone that route for now, incredibly easy now that you point out the obvious solution.
Thanks for creating and publishing this project of yours. I'm a big fan of the editor, it allowed me to strip out quill from the project.


Confirm password; Recent Files; Readable version number

Hello,
I just tested <IP_ADDRESS>, and have three suggestions:

When encrypting a new file, require the user to re-type the password, to prevent errors
Add a File > Recent Files items
In the Help > About, the infos are hard to read (blue text on a dark grey background → Black text on white background)

Cheers,

I understand your 3 suggestions. The requirement to re-type the passwords when encrypting a new file is a good and easy precaution to take. Adding a 'Recent Files' item in the File menu gives users quick access to previously used files without having to search.And finally, changing the colors of the text in the Help > About page will make it easier for
users to read the information.
I will add these improvements to the next version.


add Giraffe.ViewEngine.StrongName

This PR adds Giraffe.ViewEngine.StrongName as a strong named assembly signed with the key in src/Griaffe.ViewEngine.StrongName/key.snk.
See the discussion in #23. Build and release workflows would have to be adopted for also publishing this package.
Usage is equal to Giraffe.ViewEngine other than referencing another nuget package:

I don't know much about signing assemblies, but normally things are signed with a private key. I suspect this key shouldn't be here.
I'm thinking maybe we could store the key as a secret and create a github action to upload to nuget.

https://learn.microsoft.com/en-us/dotnet/standard/assembly/strong-named

If you are an open-source developer and you want the identity benefits of a strong-named assembly for better compatibility with .NET Framework, consider checking in the private key associated with an assembly to your source control system.

Signing an assembly is not a security feature, so I don't think keeping the key private is necessary when even the official docs recommend commiting it in source control.

also the existing actions fail in the pack step

I did not touch the original Giraffe.ViewEngine.fsproj in this PR, so I suspect there is a general issue with how the project is packed now that there is more than one project. The version string for example is coming from -p:PackageVersion=$GITHUB_RUN_ID which i am pretty sure i had no influence on in any of the code added.


Problem with generating model from Visual Paradigm when classes are part of package

Hi Guys.
I've prepared model diagram with VP. I tried to generate model  but I got info that no classes are changed and nothing was generated. After few hours of digging I've figured that problem was caused by package that contained all classes. Generator found one object in tree root and it was not class.

I'll test it tonight and, hopefuly, fix it right after. Thanks for reporting this!

That message appears when you already generate your model and nothing changed since then.
Try the option "-f" to force jhipster-uml to re-generate  your entities.

I think it's not related with force option. There was nothing generated in .jhipster folder and i got this message. After removing package from my model, classes were properly generated.

I'll try to see how to ignore the packages in the parser later today so that you can test it tomorrow I hope.

The work needed to bypass packages is a bit more complicated than what I had in mind. Packages are logical layers and anyone can encapsulate a lot of packages before extracting classes... I'll need to come up with a working prototype before closing this thread.


bot.loadSession no longer behaving properly in 3.10.1

Bot Info

SDK Platform: Node.js
SDK Version: 3.10.1
Active Channels: Facebook
Deployment Environment: Elastic Beanstalk

Issue Description
bot.loadSession(address, done) is breaking on the latest release. When the function completes, the handler (done) is called twice - one with the session object, and one where it is set to null. Calling session.send() doesn't seem to work even in the instance where the session object exists. I have verified that the address exists and is valid.
Looking at the latest commits, it appears that loadSession has been refactored to call loadSessionWithOptionalDispatch(address, false, done), to incorporate the feature of optional dispatching.  However, shouldn't it call loadSessionWithOptionalDispatch(address, true, done) instead, since loadSession used to dispatch by default?
Reverting to 3.9.1 fixed the issue.

exactly the same for me

I was able to reproduce the problem and have created a PR that switches the 2nd parameter to true, which has UniversalBot.loadSession() returning to its original behavior.
PR that changed behavior: #3396

I'm having a similar problem, although I don't see the callback being called twice. In my case, it gets called with the following error -- TypeError: Cannot read property 'callstack' of undefined
This was working fine in 3.9.1.

resolved in<EMAIL_ADDRESS>out now


certificate addition for requests + interface changes for notifyLogin

I didn't understand why there's a "AccountsNotifyLoginParamsSiteUID"
Each call to notifyLogin has to have siteUID which mandatory.
I just added the sessionExpiration as optional.
I added an "if" for the certificate read to prevent multiple IO in case there's a use of the same object

The certificate is now bundled by default into the library.


Package Requested

[X] I would like someone else to build the manifest.
[ ] I would like help so I can submit the manifest.
[X] I have performed a search and couldn't find this package.
[X] I have checked that the installer for this package is not a standalone or portable executable or a .zip file
[ ] I think there is a new version available and I have provided the URL.

Please provide the following information

Publisher: Guillaume FOUET (aka Ozone)
Publisher Url: http://www.visipics.info/index.php
Package Name: VisiPics
Description: Quick & Easy Duplicate Images Cleaner
Package Version: 1.31
Installer URL: https://www.fosshub.com/VisiPics.html?dwl=VisiPics-1-31.exe
Current Package Identifier (for new versions): None

There's something wierd with the Installer URL. When I create the manifest of this package, I met the following errors:
manifests  master ≡ λ wingetcreate.exe new

 InstallerUrls 值为:
  https://www.fosshub.com/VisiPics.html?dwl=VisiPics-1-31.exe

正在下载和分析: https://www.fosshub.com/VisiPics.html?dwl=VisiPics-1-31.exe...
无法从 [https://www.fosshub.com/VisiPics.html?dwl=VisiPics-1-31.exe] 中分析数据包

The "actual" download url of this package looks like this (Got from Download Panel in Microsoft Edge):

https://download.fosshub.com/Protected/expiretime=1644620952;badurl=aHR0cHM6Ly93d3cuZm9zc2h1Yi5jb20vVmlzaVBpY3MuaHRtbA==/9d1bd67c5d275c117d4777bf69372695190da83d7753c50c064646a73392a4f0/5b8d472959eee027c3d78953/5b8d474959eee027c3d7895c/VisiPics-1-31.exe

I can't handle this situation myself, so I create this issue (package request). I'm willing to submit the manifests myself when I find the "stable" URL of this package.

We're not allowed to use Fosshub as its against their TOS.

We're not allowed to use Fosshub as its against their TOS.

Oops?! It's mentioned in Abuse - Section B. and Exceptions Part in TOS.
Is it possible to talk with FossHub and adding winget.exe to FossHub's "Exception" List?

I believe they don't want bots to constantly access and download from their server as they probably monitor usage and stuff
Feel free to ask them but you'll probably get denied

I believe they don't want bots to constantly access and download from their server as they probably monitor usage and stuff

Okay, I'll installed it manually 😂
Thanks for your help. This issue can be closed.


Please confirm these before moving forward

[X] I have searched for my issue and not found a work-in-progress/duplicate/resolved issue.
[X] I have not been informed if the issue is resolved in a preview version of the winget client.

Category of the issue
Other
Brief description of your issue
Package has wrong version (as of this posting), should be 2.5.2, and has wrong installer hash.
Steps to reproduce
winget upgrade --all
Actual behavior
Tries to upgrade installed (latest) version 2.5.2, fails due to wrong installer hash.
Expected behavior
No upgrade.
Environment
Windows Package Manager v1.6.2771
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.22621.2428
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.21.2771.0

Winget Directories
-----------------------------------------------------------------------------------------------------------------------
Logs                               %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\Diag…
User Settings                      %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\sett…
Portable Links Directory (User)    %LOCALAPPDATA%\Microsoft\WinGet\Links
Portable Links Directory (Machine) C:\Program Files\WinGet\Links
Portable Package Root (User)       %LOCALAPPDATA%\Microsoft\WinGet\Packages
Portable Package Root              C:\Program Files\WinGet\Packages
Portable Package Root (x86)        C:\Program Files (x86)\WinGet\Packages
Installer Downloads                %USERPROFILE%\Downloads

Links
---------------------------------------------------------------------------
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget
Windows Store Terms https://www.microsoft.com/en-us/storedocs/terms-of-sale

Admin Setting                             State
--------------------------------------------------
LocalManifestFiles                        Disabled
BypassCertificatePinningForMicrosoftStore Disabled
InstallerHashOverride                     Disabled
LocalArchiveMalwareScanOverride           Disabled

Screenshots and Logs
No response

Working on #125986 by @ottnorml .

Thanks for the fix! Closing this.