fit for class which enhanced by CGLIB

fit for class which enhanced by CGLIB,for example,some class annotated with '@Service' in Spring Framework

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


VM: Introduce volatile.last_state.vsock_id key

Set volatile.last_state.vsock_id on each VM start (if VM's DB ID has changed or if key is missing).
This will enable lxd-agent connections to still work after a running VM is recovered after removing its DB records (using future recovery tool) which will cause the instance DB record's ID to change.

Testsuite passed

@stgraber on 2nd thoughts I wonder if just recording the last start db Id to volatile may be of more general use than the vsock ID. I know we have the uuid too, but might be useful for other things in the future, and we can then derive the vsock id from that. What do you think?

@tomponline we've had issues in the past where when dealing with nested virtual machines you can get vhostid collisions.
To deal with that, I've had in the back of my mind that we should move the vsockid allocation over to LXD, then pass it as an fd to QEMU. This should let us use port 0 and have the kernel find an unused one avoiding this issue.
Doing that would have required the addition of a volatile key, so I was definitely fine with you introducing it for another reason ;)
I guess if we do end up doing that vsockid change we may just end up renaming the key from volatile.last_state.vsock_id to just volatile.vsock_id. Probably should have done it now but I didn't think of it when reviewing ;)

@stgraber I can change that this evening so it won't be in main for long.


vec: get a metric with labels that are arbitrarily ordered

Get a metric with labels that are arbitrarily ordered.
/cc @syepes

what is this PR for?

These is a bug in get_meric_with().
If we create a MetricsVec with unsorted labels, like &["b", "a"],
then get_metric_with(&labels!{"b" => "2", "a" => "1",}),
it will return a metric that labels are b => 1, a=> 2, because &labels!{"b" => "2", "a" => "1",} are wrongly sorted to &["1", "2"] in get_meric_with().

Please add test.

@overvenus Thanks for the fix, it looks like it has resolved my issue.

LGTM
PTAL @disksing

LGTM


@conda-forge-admin, please add user @weiji14

Additional comment:
Hoping to get some newer versions of deepspeed (e.g. at #27) into conda-forge.

Thanks @hadim!


Missing climatological support

The recently new climatological kwarg should also to be supported for the following coordinate @classmethods:

Coord.from_coord, see here
DimCoord.from_regular, see here

Ping @trexfeathers


rosbag_uploader: 1.0.0-3 in 'kinetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository rosbag_uploader to 1.0.0-3:

upstream repository: https://github.com/aws-robotics/rosbag-uploader-ros1.git
release repository: https://github.com/aws-gbp/rosbag_uploader-release.git
distro file: kinetic/distribution.yaml
bloom version: 0.9.7
previous version for package: 1.0.0-2

Specified wrong mapping of s3_common ROS package to Ubuntu APT package, needed by the s3_file_uploader ROS package.


refactor(p2wdb): Refactor matchErrorMsg()

PR related to this trello task --> https://trello.com/c/pSEGQQO6

:tada: This PR is included in version 1.11.0 :tada:
Your semantic-release bot :package::rocket:


[Subscription UI] Subscription details maim page

Changes in this Pull Request:

Subscription main details page.

Mandatory checks:

[x] Branch is up to date (latest changes were fetched from the upstream branch before creating this PR)
[x] Changes were tested manually
[ ] Java code changes are unit tested
[ ] Webtests are successful

To be checked by the reviewer:

[ ] Clean Java code and unit tested changes
[ ] UI customizablity is preserved: No hard-coded styling and URL references in JSF views

To be checked by the pull request owner:

[ ] .MASTER/job/BES_MASTER_IT/
[x] .MASTER/job/OSCM_WS_Tests_INTERNAL/
[x] .MASTER/job/OSCM_WS_Tests_OIDC/
[ ] .MASTER/job/OSCM_Integration_Tests_INTERNAL/
[ ] .MASTER/job/OSCM_Integration_Tests_OIDC/

Cross repository and core changes:

[ ] Interdependent changes have been communicated to the team (if applicable)
[ ] This pull request includes high risk modifications or core changes (e.g API, datamodel, authentication...). Mandatory reviewers: @goebell or @kowalczyka.

OSCM Build References:
N/A
Screenshot (if applicable):

This change is

oscm-portal/WebContent/marketplace/playground/account/subscriptionUdaDetails.xhtml, line 2 at r1 (raw file):

<!-- Copyright FUJITSU LIMITED 2018-->
<!-- TODO #851 -->

What is this TODO about?

oscm-portal/WebContent/marketplace/playground/account/subscriptionUdaDetails.xhtml, line 1 at r1 (raw file):

<!-- Copyright FUJITSU LIMITED 2018-->

Update Copyright.

oscm-portal/WebContent/marketplace/playground/subscriptions/paymentInclude.xhtml, line 2 at r1 (raw file):

-->

TODO?

oscm-portal/WebContent/marketplace/playground/subscriptions/priceModelDetails.xhtml, line 17 at r1 (raw file):

    <h1>
      <h:outputText style="float:left;" value="#{msg['marketplace.services.overview']}" />
    </h1>

Please avoid inline styles where possible.


addOns Property Missing in angular-cli.json

Please provide us with the following information:

OS?

Windows 7, 8 or 10. Linux (which distribution). Mac OSX (Yosemite? El Capitan?)

Versions.

Please run ng --version. If there's nothing outputted, please run in a Terminal: node --version and paste the result here:

Repro steps.

Was this an app that wasn't created using the CLI? What change did you do on your code? etc.

The log given by the failure.

Normally this include a stack trace and some more information.

Mention any other details that might be useful.

Thanks! We'll be in touch soon.

This is intentional.
See: https://github.com/angular/angular-cli/pull/4052

My angular-cli project is using font-awesome along with Bootstrap 4.  The angular-cli setup instructions include defining the fonts path inside the "addOns" property.  Since angular-cli.json no longer has a "addOns" property by default should the property be added back?  Should I just add the "addOns" property in my project?  Or should I define the font-awesome fonts path inside a different angular-cli.json property?

The instructions (https://github.com/angular/angular-cli#global-library-installation) reference apps not addons.  The addons property is reserved for future use.

Closing as @clydin has answered.

Respectfully I think that I need a more specific answer since there are multiple properties under the "apps" property inside angular-cli.json
Here is what I want to do.  Have the angular-cli inject both the font-awesome.css and the font-awesome file path that is used for font-awesome icons.  I have included the font-awesome.min.css with no problem using the "styles" property under the "apps" property.
"styles": [   "../node_modules/bootstrap/dist/css/bootstrap.min.css",   "../node_modules/font-awesome/css/font-awesome.min.css",   "app/css/styles.css" ],
Help me understand where I need to define the font-awesome fonts file path since the "adons" property no longer exists.  I understand that the suggestion is to define the value under the "apps" property.  Help me understand on what property under the "apps" property.
Here is the old way that I was referencing the font-awesome fonts file path using the "addons" property
  "addons": [     "../node_modules/font-awesome/fonts/*.+(otf|eot|svg|ttf|woff|woff2)"   ],

The 'addons' field is simply a placeholder.  It provides no functionality.
Adding the stylesheet to 'styles' is all that is needed.
See: https://github.com/angular/angular-cli/blob/master/docs/documentation/stories/include-font-awesome.md

@clydin Are you saying the reference to the font files doesn't need to be int he config file at all?  @brookhutchinson  is asking you about where the addons include of font files should end up in the 'apps' structure.  He's not asking about the styles.


Update release-notes.md

@tiannaward - you seem to have access to the private repos and should be submitting your prs there. thanks.
@nitinnms

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

NOTE that the link text "Here" does not follow Microsoft style and guidelines for the learn platform. See https://styleguides.azurewebsites.net/StyleGuide/Read?id=2696&topicid=28111
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


[SSR] y.template is not a function

Hey! I'm using solid-toast with Astro and I'm getting y.template is not a function when using it.
It seems like a common problem if the library isn't SSR compatible.
IIRC it's recommended to ship Solid libraries with preserved JSX so the bundler can decide which build (client or server) to use.
Any chance to get this implemented? Maybe bundling with rollup-preset-solid?

This would be great, thanks! :)

@ardeora I've just created a PR #7 which solves this. I tried it locally with solid-start and it works without errors. Maybe the double quotes in the rollup.config.js need to be replaced with single quotes. I would've done it but my auto-format settings in VSCode made it impossible since you don't have any prettier config in the repo. 😂

Hey Marvin! I have merged your changes and it seems like these changes allow us to work with SSR frameworks although you would still need to set ssr.noExternal: ['solid-toast'] for now. I think this behaviour is expected. We can also revisit this later as Astro and solid-start mature more. I will close this issue for now and mark it as complete


OIDC Session Manamement Impl

Added this feature with https://github.com/wso2-extensions/identity-inbound-auth-oauth/pull/5


How to : Denoise Audio

It is not clear from the documentation and the codebase how I can denoise (if possible) a noised waveform. Such as in : "https://diffwave-demo.github.io", section 4 (Denoising steps in the reverse process of DiffWave).

I recommend reading the DiffWave paper for details on this model.


fix_#2110

942190 regular suspected error
change "\wiif\s*?()" -> "\Wiif\s*?(.+?)"
add a test case for 942190

@franbuehler Please check my PR, thank you!

Hi @NiceYouKnow
Thanks a lot for this PR!! And great you also added a test!
I just realized that the original regex was \wiif\s*?\( and not \wiif\s*?(). I didn't look this up correctly in the sourcefile of the generated regex: https://github.com/coreruleset/coreruleset/blob/v3.4/dev/util/regexp-assemble/regexp-942190.data. The closing bracket belongs to the outer or (...|...).
For long and complicated regexes we have a source file and we generate them by running ./util/regexp-assemble/regexp-assemble.pl util/regexp-assemble/regexp-942190.data.
But I think because a closing bracket ) has to follow an opening bracket ( we can still leave this "new" regex. It's only a means against false positives.
Could you please also update the source file of the regex: https://github.com/coreruleset/coreruleset/blob/v3.4/dev/util/regexp-assemble/regexp-942190.data in this PR here?
I generated the regex and it's exactly the one you added in the rule file rules/REQUEST-942-APPLICATION-ATTACK-SQLI.conf.
So just adding the changed regex to the sourcefile would be enough.
And then we just have a little lint problem: https://github.com/coreruleset/coreruleset/pull/2112/checks?check_run_id=2745587522#step:5:12. Could you please add a new line?
Thank you again for this contribution. It's very welcome!

Okay, I will notify you after the revision is completed based on your opinion.

Thank you for this contribution @NiceYouKnow.
@franbuehler : We're ready to merge, aren't we?

@dune73 : yes we are ready to merge this one.

Meeting decision June (https://github.com/coreruleset/coreruleset/issues/2074#issuecomment-856173925):
@theseion wants to review this PR.


Rotate the network around a central node

Hi, is it possible to make the graph constantly move without stopping? At the moment, it keeps reaching equilibrium after dragging the nodes around in interaction mode. I want the visualization to rotate continuously (while maintaining horizontal labels). Thanks!

I'm interested, I have a main node

You could play with mass of nodes and their parents,and work on the minVelocity + stabilisation interation number to achieve this.


fix(docker) nextjs unable to connect to db

To connect to the nextjs-monorepo-example-db container from the nextjs-app container both containers need to be in the same docker network. Otherwise, you get a connection error.
The service name main-db as the hostname when they are in the same network.

Nice ! Thank you


Profiles moniker flag take full-node moniker as default value

Bug description
If we try to save a profile with the desmos tx profiles save transaction, without specifying the --moniker flag, the module will not use its default [do-not-modify] one. Instead, he will use the moniker of the full node.
Steps to reproduce
desmos tx profiles save <dtag>...
check the tx info before sending it.
Expected behavior
It should take the default flag and not the full node moniker.

I've tried to solve it locally by replacing moniker flag name with monikerz and it worked.
Any thoughts on this? @RiccardoM

@bragaz I think that probably what we should do in this case is renaming the Moniker field to Username all together. This should clear up a lot of things and make it easier for users to understand what is it. Would also like to know @kwunyeung's opinion

@bragaz @RiccardoM If we have to change this, I prefer avoiding naming it username as it may create confusion. I prefer calling it nickname, pen-name, stage-name.

@bragaz @RiccardoM If we have to change this, I prefer avoiding naming it username as it may create confusion. I prefer calling it nickname, pen-name, stage-name.

nickname would be perfect IMO. We can go for that

Then nickname be 🚀


Post release fixes

Minor CI fixes following the release.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (3203bfb) 91.60% compared to head (cbdeda7) 91.60%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #830   +/-   ##
=======================================
  Coverage   91.60%   91.60%
=======================================
  Files         168      168
  Lines        6929     6929
=======================================
  Hits         6347     6347
  Misses        582      582

Flag
Coverage Δ

unittests
91.60% <ø> (ø)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Some css declarations (colors) appear twice

Version
@nuxtjs/tailwindcss:
nuxt:
Reproduction Link

https://codesandbox.io/s/stoic-panka-vf2kk8?file=/components/TButton.vue
Steps to reproduce
inspect the button:

What is Expected?
css background/color declarations should only appear once:
background-color: rgb(59 130 246 / var(--tw-bg-opacity));

What is actually happening?
they appear twice:
background-color: rgb(59 130 246 / 1);
background-color: rgb(59 130 246 / var(--tw-bg-opacity));

That's the fallback for browser that don't support variable inside rgb function

That's the fallback for browser that don't support variable inside rgb function

Thanks for providing input on this. I'm curious why this is happening with the module and not vanilla tailwindcss..

That's the fallback for browser that don't support variable inside rgb function

which browsers are we talking about?
I think it shouldn't differ from tailwinds default behavior.

This is probably your PostCSS/Autoprefixer config.


chore: ci test using --frozen-lockfile

Summary
Intercepting pnpm-lock.yaml to prevent updates in advance.

Related issue (if exists)

Types of changes

[ ] Docs change / Dependency upgrade
[ ] Bug fix
[ ] New feature / Improvement
[ ] Refactoring
[ ] Breaking change

Checklist

[ ] I have added changeset via pnpm run changeset.
[ ] I have added tests to cover my changes.

It’s a default for pnpm, so we don’t have to set it explicitly. Check this out: https://github.com/pnpm/pnpm/pull/2378


Build for 64bit google

This app only has 32-bit [x86] native libraries. Beginning August 1, 2019 Google Play store requires that all apps that include native libraries must provide 64-bit versions. For more information, visit https://g.co/64-bit-requirement
Affected Modules: trustwalletcore

@trustwalletvn can you stop using Trust Wallet branding/trademark?

ok. no problem

@trustwalletvn can you stop using Trust Wallet branding/trademark?

ok
No problem

WTF you report my website ?

@trustwalletvn can you stop using Trust Wallet branding/trademark?

https://github.com/trustwallet/wallet-core/issues/1589


[cast-opt] Change optimize{Swift,ObjC}to{ObjC,Swift} to use SILDynami…

…cCastInst.
I also debrided dead variables that resulted from eliminating the trampoline
argument code in optimizeBridgedCasts().

Andy, take a look at optimizeBridgedFunctions... its really clean now since we do not need to marshall any data.

@swift-ci smoke test and merge

@swift-ci smoke test linux platform

@swift-ci clean smoke test linux platform


License

Hey!
We'd like to feature this code in a blog post, but we are unable to do this because the code does not contain a license. Would you be willing to add a license?
More info: https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/licensing-a-repository
Luke

Hey @hakluke 👋
Am onto adding a MIT License 🙌

Done ✅

Thank you!


Adding experimental function 'mapRow'

Added 'mapRow' (found in ResultSetMapper.kt) and its supporting enums, functions.
I wanted to start a conversation about whether or not something like this could be useful, as I have already been using it a bit in my endeavors using lite-for-jdbc.
This 'mapRow' function reduces the boilerplate of manual mapping code required and allows for a quick turnaround for simpler use cases when result sets closely match their expected return Kotlin types. Please see the initial Kdoc on the function, as well as the unit tests (ResultSetMapperTest.kt).
I believe this is still inline with the heart/goal of lite-for-jdbc as this does not perform any sql generation magic.
As stated this is experimental as some liberties are being made on Timestamps whereas this mapping tool currently will convert all Timestamp and Time values to java.time.Instant as Timestamp is not an appropriate type to pass around outside of the relational/SQL boundary. There may be an issue with the Calendar on Timestamp, see the extension function readValue in _ResultSet.kt

I closed it due to lack of interest.


Remove MetaData test flakiness

: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 #1694 opened

Start Time: 2021-03-11T11:03:23.042+0000

Duration: 51 min 52 sec

Commit: e7dbba1e6d635fa62d2b4c0639225bc2d951af1f

Test stats :test_tube:

Test
Results

Failed
0

Passed
1849

Skipped
14

Total
1863

Trends :test_tube:

:green_heart: Flaky test report
Tests succeeded.

Expand to view the summary

Test stats :test_tube:

Test
Results

Failed
0

Passed
1849

Skipped
14

Total
1863


Dynamic loading of tools and custom tools

Feature request
Maybe I didn't get the idea: Some tools can be loaded like 'google-search' other tools can't be loaded like duckduckgo-search by the load_tools: langchain/agents/load_tools.py
E.g. specified in _EXTRA_OPTIONAL_TOOLS.
As for custom tools they can't be loaded at all it seems in that way. It would be great to register custom tools so that they can be loaded by load_tools as well.
I've done something like this for the moment, so I have Class-Names for my own tools and the lower-case would be standard:
        standard_tools = []
        for toolStr in tool_names:
            if toolStr[0].islower():
                standard_tools.append(toolStr)
        self.tools = load_tools(standard_tools, **tool_kwargs)
        for toolStr in tool_names:
            if toolStr[0].isupper():
                toolClass = globals()[toolStr]
                toolObject = toolClass()
                self.tools.append(copy.deepcopy(toolObject))

Motivation
When assigning agents a set of tools it's very handy to do it through a list.
Your contribution
I'm not familiar enough with the architecture of langchain so I'd better not. But I'm happy to test it.

having this feature would be a nice addtion


Feature/2421/link orcid

#2421
I'm sure it's missing something but I could use some feedback

It looks good to me, I think a test is all that is left. Not sure how easy that is though, are there tests for other tokens?

The only tests in TokenResourceIT are for the google and github logins, none for other linked accounts. I'm guessing this is because it would be too difficult to test?

@emlys You could probably mock some responses with Hoverfly, but I haven't thought deeply about it.

@emlys You could probably mock some responses with Hoverfly, but I haven't thought deeply about it.

This is mostly boilerplate code and given that we don't do this with the other account links, I'm inclined to say, and I can't believe I'm saying this, that it's OK without tests. I feel the gain would be too little for too much work.
If we still feel it's necessary to have some tests, I would suggest writing unit tests for the guts of addOrcidToken, using Mockito.

... I'm inclined to say, and I can't believe I'm saying this, that it's OK without tests.

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 4

See the complete overview on Codacy


Dev administration provider

Provides district and community, loaded from Backend.

I took the opportunity and added missing docs on all services and providers


bower pitfall

Maybe a bit anoying, but:
We were working with ngTabele and bower for some time now. Started a release 0.8.3.
Now - after updating the project ngTable wasn´t working anymore.
From scanning the issues here I couild see, that bower is not supported anymore (beginning with what version?).
It took some time to figure this out, because 'bower info ng-table' lists versions up to 3.0.1.
bower is delivering something with version 3.0.1, but I have no idea, what to do with this.
Could´t there be a hint in the 'bower install' output?
Which version of ng-table is the latest, supporting bower? (need some time to swich to npm)
Thanks,
Uli

Last supported vs for bower is 1.0.0.
Didn't realise that bower would continue to report every github release as a valid vs for bower... but then thinking about it that does makes sense.
Not sure if bower can be "told" about which version it should go up to. If not then it's a bit rubbish as this is totally confusing.

Thanks.
It might be usefull to completely unregister the package from the bower registry. 1.0.0 seems to be old enough and you are forcing me to switch to npm ;-)

I took care of it. To install latest build just type:
bower install ng-table-bundle --save
Fork is available here: https://github.com/Greenek/bower-ng-table

Why bower is no longer supported?

Hi,
I have created a repo for the bower support.
would it work for you?
we can maintain the updates
https://github.com/imana97/ng-table
bower ng-table-bower


Publish v0.8.4 to npm

0.8.3 doesn't contain setCategory for iOS, so i had to include this package in my package.json by using a recent commit ID.
Also please consider making a change log, it's very helpful to know what's changed in each version.
Thanks for a great package!

yes, that'll be a great help

Ping. Really should publish a version. There are changes over 6 months old which haven't been published.

@rawrmaan @jamesreggio Agreed. I've just joined the project and will release a new npm version as soon as I have the access I need. I've asked @zmxv for access to do so.

0.9.0 already released on npm thanks to @benvium


Adding severity as an attribute to an issue

Gitea version (or commit ref): new functionality
Git version: n/a
Operating system: n/a
Database (use [x]): n/a

Description
Would be nice to have an additional attribute "severity" for each issue. Severity types should be configurable like the labels are at the moment (customizing own severity types) and assign it to an issue.

What would be a use case that isn't covered by using existing labels functionality to indicate issue severity?

I think we need a perior according labels feature but not a new field.

What would be a use case that isn't covered by using existing labels functionality to indicate issue severity?

Basically, a severity is a one to one relations hip and labels might be added as many as available. Furthermore, an own field would allow to set a default value, which is not possible with tags as well.


Permission failed in flutter apk realease

My apk denied because i have using permission in my camera but
i already include it on my AndroidManifest.xml

why this happen ?

maybe you need to ask it yourself in dart code ( in my react native knowledge , so I think flutter is the same )

Hi @huntz20
This platform is not meant for assistance on personal code. Please see https://flutter.dev/community for resources and asking questions like this, you may also get some help if you post it on Stack Overflow. Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it
Thank you


Adopt eager workflow start to improve perf[Temporal]

https://temporal.io/blog/improving-latency-with-eager-workflow-start

Just realized that will also help with then versioning optimization. The error would be much lower


extension is not starting

Issue Type: Bug
extension marketplace has no extension shown, something is written <error while fetching . XHR failed >
pls help me to solve my issue
VS Code version: Code 1.67.2 (c3511e6c69bb39013c4a4b7b9566ec1ca73fc4d5, 2022-05-17T18:15:57.229Z)
OS version: Windows_NT ia32 6.1.7601
Restricted Mode: No

Hi @kd1318
Thanks for reaching out. If you're using a VPN client or some local firewall, please turn these off and retry. If the error is still persisting, it will help us to investigate, if you can share the network log ( command: Developer: Open Logs Folder).

Closing the issue as there have been no updates for a week. If the problem persists you can contact us with the requested network log.


Better build failed output

Raising on build failed shows a stacktrace, which isn't helpful.

Looks good to me.

you should probably do process.exit(1) or something, right?

Indeed. Thanks @dickeyxxx


Blank error when connection string connection fails

Open connection dialog
Select Connection String connection type
Enter connection string that will fail (such as server=skflds)

A blank error appears instead of the actual error message

When you click on "Copy Details", you'd get "UserName cannot be null or empty when using SqlLogin authentication" in text copied.. issue seems to be in rendering the text on dialogbox.


Background scan for midi server. Any interest?

I have added a background scan in BLEMidiClient. The idea is to connect immediately on turning on the instrument. Is there any interest? I have no experience with uploading on github.

Hi, sorry for replying late. yes i'm interested ! to upload on github, you have to make a fork of the library on github, you make your modifications, then you make a pull request.
or else you can send me a patch via e-mail if you want, or put it here as an attached file.
thank you !


Self-bot does not seem to work

Describe the bug
Self-bot doesn't receive and execute any commands.
To Reproduce
Just try running any command with self-bot.
What did you expected to happen
Running commands.
What actually happened
Nothing.
Desktop (please complete the following information):
Win10 LTSC x64 (I think it's OS-independent though)
C-REAL version 2.4.0 (latest)
What I can add.
Discord... Oh, I've visited the  SO question about this and saw your comments, so you surely have known about this issue. By the way, submitting issue is still great idea, isn't it? :)

The problem was another selfbot nerf issue -- discord won't maintain selfbots anymore, so we have to fix it.
Right now, I haven't pushed out a fix yet.
Just like you saw in the SO question: Yes I'm aware of the selfbot problem.

Indeed, this is a good question

Why the issue is closed if it's not fixed yet?

It is closed because we dont support the selfbot anymore sorry for not clarifying.


Finalise PD Milestone for Communication

Background
On #31  we did a first iteration using the Communication competency as an example of allocating milestones by identifying what is good and what isn't. We need to finalise this after all coursework has been brought into the test coursework planner.
This ticket is dependent on #41
Acceptance criteria

Add the milestones for all Communication/English Language coursework (Emerging - Level 1, Developing - Level 2, Established)
Evaluate if changes are needed to the coursework and raise other tickets for it.

Owner: @LaraHuzjan @kfklein15
Stakeholder: @SallyMcGrath
Approver: @kfklein15

@LaraHuzjan and I did this today


Still being actively developed?

Nice work on the initial work here :) I was considering writing a Seti UI theme and found your WIP. I was wondering if you're planning on continuing to develop it.
Some suggestions:

Usage of the theme without building? There could be a convenient way to install the theme without checkout. If a checked in version is supplied, the readme could be updated to point this out.
Document latest version of Chrome the theme works with.
Publish to Chrome Web Store

Thanks! I've been fairly strapped for time lately so I haven't been able to keep up developing this as I would like. Hopefully I can make some time soon to really dig my heels back into it.
I'll definitely make it easier to install the theme, and get it up on the chrome web store as soon as I can. Initially I hadn't built it for the store because I wanted to make sure I did as thorough an inventory of dev tools as I could. I believe there were a few parts I had yet to style appropriately, and some parts of dev tools (cubic bezier editor in particular) that I was waiting on coming out before I tried to really finish off the core.
One of my frustrations in building this was that I couldn't match a lot of the syntax due to limitations by devtools (codemirror?) syntax classes. I'll try to do an inventory of everything and ensure my styles work on both canary and stable (and list the current version I've tested with). Once I do that I'll create a build that I can post as a release, and perhaps get on the chrome store.

Thanks for the detailed (and fast!) reply :) If it helps at all I've been using https://github.com/kenwheeler/brogrammer-devtools pretty happily lately. It doesn't cover all the edge cases, but does seem to capture a lot of the sources panel syntax highlighting well. I have yet to see any extensions specifically style some of our newer UI (web animation player, bezier editor) well, so that could be fun!

Oh neat! I'll definitely check that one out and see if it can help me find anything I'm missing. Thanks!


Add driver.max_packet_size to config.yml

Add configuration driver.max_packet_size to config.yml and document this variable in Troubleshooting.md.

Large events may lead to this message
ERROR io.debezium.pipeline.ErrorHandler - Producer failure
io.debezium.DebeziumException: log event entry exceeded max_allowed_packet; Increase max_allowed_packet on master; the first event 'binary.085408' at 388427867, the last event read from '/var/lib/mysql/logs/binary.085408' at 126, the last byte read from '/var/lib/mysql/logs/binary.085408' at 388427886. Error code: 1236; SQLSTATE: HY000.
	at io.debezium.connector.mysql.MySqlStreamingChangeEventSource.wrap(MySqlStreamingChangeEventSource.java:1088)
	at io.debezium.connector.mysql.MySqlStreamingChangeEventSource$ReaderThreadLifecycleListener.onCommunicationFailure(MySqlStreamingChangeEventSource.java:1133)
	at com.github.shyiko.mysql.binlog.BinaryLogClient.listenForEventPackets(BinaryLogClient.java:1096)
	at com.github.shyiko.mysql.binlog.BinaryLogClient.connect(BinaryLogClient.java:648)
	at com.github.shyiko.mysql.binlog.BinaryLogClient$7.run(BinaryLogClient.java:949)
	at java.base/java.lang.Thread.run(Thread.java:833)
Caused by: com.github.shyiko.mysql.binlog.network.ServerException: log event entry exceeded max_allowed_packet; Increase max_allowed_packet on master; the first event 'binary.085408' at 388427867, the last event read from '/var/lib/mysql/logs/binary.085408' at 126, the last byte read from '/var/lib/mysql/logs/binary.085408' at 388427886.
	at com.github.shyiko.mysql.binlog.BinaryLogClient.listenForEventPackets(BinaryLogClient.java:1060)
	... 3 more

It is also recommended to set max_allowed_packet on the primary.


not all default apps were uninstalled

[x] Disney +
[x] Spotify
[x] Mixed Reality Portal
[ ] 3D Viewer
[x] Feedback Hub
[x] One Note For Windows 10
[x] Paint 3D
[x] Skype
[x] Xbox Live

Get-AppxPackage -AllUsers


Python and Tensorboard version?

Hi, I have some incompatibility problems, would you mind sharing what python version and tensorboard version you were using while you made the latest release (few days ago)?

2.4.0 and 2.5.0

Thank you. My problem was actually with some modification I have done to my python itself.
Without opening another issue, can I ask you @Lornatang if you obtained your results after training for 512 epochs (P-epochs) and 128 epochs (G-epochs)? I found that training for only 100 and 25 will give me an average PSNR of aprox 25 for Set14, however, the result for the butterfly won't look as good as yours (in README). Even with the high PSNR, the results from the test set won't show a very appealing look. Most probably needs more training from my side
I'm just wondering if 512 and 128 was chosen after different tests, or just a number which should work.

In addition, without making a pull request, in train.py L148 and L174, I believe based on your code that you should have 'sr.png' instead of 'hr.png'. Otherwise, the next line after these 2 will crash, as sr.png is not existent.

The SRGAN training process is unstable in the first stage. You should wait for it to be fully trained. In the second stage, take the highest SSIM Epoch as the final model.

sr.png is generated after each Epoch, it is generated by lr.png, and image evaluation is performed with hr.png

The SRGAN training process is unstable in the first stage. You should wait for it to be fully trained. In the second stage, take the highest SSIM Epoch as the final model.

By first stage, you mean training the psnr and second stace being the gan training, right? So for the best result "wait for it to be fully trained" you mean to wait the 512 epochs before Gan ?

sr.png is generated after each Epoch, it is generated by lr.png, and image evaluation is performed with hr.png

Sorry, maybe I said it wrong. The path you set is wrong in L148 and L174. You are overwriting the hr.png

yeah

yeah

Thank you.
1 more note. In test.py you are linking to the path of LR images instead of ground truth L121

Thanks. It fixed.


Double order from Medusa not blocked by the plugin

When you already received the notification order from medusa and this one didn't received a success response, you will have an other notification.
We should block the second or other notifications from medusa. We can block it in checking the id of the order if it is the same we should delete this notification or not to read it.
Don't forget to response to medusa that we already have this notification.

Release


Add current row as argument to callback

This small path adds current row as a last argument to the callback function for field. So now it is posible to do some extra checks and show the value depending of other data in the row.
Example:
Row:
{
  transaction_id: '123456',
  amount: 100,
  direction: 'in'
}

Callback:
showAmount (value, direction, row) {
  return (row.direction == direction)
    ? value
    : '';
}

Fields:
{
  name: 'amount',
  title: 'Income',
  callback: 'showAmount|in'
},
{
  name: 'amount',
  title: 'Outgo',
  callback: 'showAmount|out'
}

The row added as last argument for not breaking BC

Thanks for the PR. But this can be achieve using __slot.  This might seems convenient but the callback is strictly for formatting the column's value.

Sorry, I missed __slots functionality in the documentation. Thank you for pointing me.


About complex navmesh update

I wish we could have an overload API of NavMeshBuilder.UpdateNavMeshDataAsync to accept multiple bounding box
Because sometimes I want to update navmesh in complex shape, L shape or T shape for example, and I don't want to build the whole area. It should be better to allow us to compose many bounding box in any shape and only update what we need
Also I experience that the NavMeshBuilder.UpdateNavMeshDataAsync must input a bounding box that completely cover the voxel size and it is tedious process to check. So I which I could have some helper function that will return the expanded the bounding box that cover the whole voxel it touch. Or maybe have an argument bool includeIntersectVoxel that will do it internally
I also guessing that internally this API actually access voxels directly, so maybe we should just have api that accept voxel filter function? So we could update navmesh with any shape that really changed in the scene, which might not in the shape of AABB box (diagon line for example)
Is it possible ?

Hi! Please note that more teams in Unity are interested in improvement requests like this one and they can only track and act on them if the ideas are submitted through the Product Roadmap portal.
Some of the changes that you've mentioned are possible but not trivial. They have to be planned carefully and I can't say right now when we'll be able to do that.

maybe we should just have api that accept voxel filter function? So we could update navmesh with any shape that really changed in the scene, which might not in the shape of AABB box

We don't plan to expose in the API anything related to voxel manipulation.

NavMeshBuilder.UpdateNavMeshDataAsync must input a bounding box that completely cover the voxel size and it is tedious process to check.

Can you tell me please in which situation it is not easy to specify the proper bounding box?

Can you tell me please in which situation it is not easy to specify the proper bounding box?

When we have update a small area, smaller than the voxel size, such as adding some small item that can block or add navigation into the scene (wall / door / bridge)
I have naively try to use the boundingbox of the object to update the scene directly and just found out that the bound is too narrow or inbetween the voxel, so it don't update anything or just update partially
To scaling programmatically is doable but it just tedious, I which I could just throw in a bound I have and make it update every intersect voxel, instead of completely covered voxel

We don't plan to expose in the API anything related to voxel manipulation.

Not for manipulate the internal voxel, but just a filter to specify which voxel need to be updated in UpdateNavMeshDataAsync. I mean, instead of specified a bound that must cover all voxel in the bound, we could write our own culling function
/// original
Bounds localBound = new Bounds(Vector3.zero,new Vector3(100,100,100));
NavMeshBuilder.UpdateNavMeshDataAsync(data,buildSettings,sources,localBound);

/// filtering style api
NavMeshBuilder.UpdateNavMeshDataAsync(data,buildSettings,sources,(Bounds voxel) => {
    if(voxel.max.x > 50 || voxel.max.y > 50 || voxel.max.z > 50)
        return false;
    if(voxel.min.x < -50 || voxel.min.y > -50 || voxel.max.z < -50)
        return false;

    return true; // filter the same set of voxels as above
});

NavMeshBuilder.UpdateNavMeshDataAsync(data,buildSettings,sources,(Bounds voxel) => {
    var center = voxel.center - offset;
    return math.abs(center.x) < 50 && math.abs(center.z) < 50; // filter all voxel in 100 width/depth of horizontal plane
});

NavMeshBuilder.UpdateNavMeshDataAsync(data,buildSettings,sources,(Bounds voxel) => {
    return voxel.center.magnitude < 30; // filter in the sphere of radius 30
});

Platform Roadmap portal

Already add some comment to it

Platform Roadmap portal

Already add some comment to it. But something was better written as issue in github, with code sample and other system
Really, I think that portal should be made as github repo instead, actually the C# reimplementation you mention in the portal should be opensource in github repo and we could contribute or experiment with it, and the suggestion and planning could be made as an issue. The criticality could be reaction

👍 for nice to have
❤ for important
🚀 for critical


Inclusive Design page cards missing hyperlink and description

The line spacing and formatting on the inclusive-palettes card is gone. The implement card description is also not appearing.

Here's the page - https://main--design-website--adobe.hlx.live/toolkit/inclusive-design/
Plus word docs:
Indexhttps://adobe.sharepoint.com/:w:/r/sites/Adobe.design/Shared Documents/website/toolkit/inclusive-design/[index.docx](https://adobe.sharepoint.com/:w:/r/sites/Adobe.design/Shared Documents/website/toolkit/inclusive-design/index.docx?d=w31e3922e0d42451e9476014835cdf3cb&csf=1&web=1&e=2Kiijg)?d=w31e3922e0d42451e9476014835cdf3cb&csf=1&web=1&e=2Kiijg
implementhttps://adobe.sharepoint.com/:w:/r/sites/Adobe.design/Shared Documents/website/toolkit/inclusive-design/[implement.docx](https://adobe.sharepoint.com/:w:/r/sites/Adobe.design/Shared Documents/website/toolkit/inclusive-design/implement.docx?d=w0d043443f0e54516bea04584499efc32&csf=1&web=1&e=NBfAI2)?d=w0d043443f0e54516bea04584499efc32&csf=1&web=1&e=NBfAI2
paletteshttps://adobe.sharepoint.com/:w:/r/sites/Adobe.design/Shared Documents/website/toolkit/inclusive-design/[inclusive-palettes.docx](https://adobe.sharepoint.com/:w:/r/sites/Adobe.design/Shared Documents/website/toolkit/inclusive-design/inclusive-palettes.docx?d=w8f9a3b36440d42cd9fa055368aa77f17&csf=1&web=1&e=ubwfmN)?d=w8f9a3b36440d42cd9fa055368aa77f17&csf=1&web=1&e=ubwfmN


Update jsoniter-scala-core to 2.14.1

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

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.github.plokhotnyuk.jsoniter-scala", artifactId = "jsoniter-scala-core" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "com.github.plokhotnyuk.jsoniter-scala", artifactId = "jsoniter-scala-core" }
}]

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

Superseded by #958.

Superseded by #960.


Startmenu Shortcut Support

PR for issue #678

This looks cool. What do you think about grouping the shortcuts under a start menu folder? Maybe called "Apps" or something generic. Just so we don't overwrite the user's other shortcuts accidentally.
Also can you check over the code style for createShortcut so it matches the other code? Lower-case, underscored names etc. Everything else looks good.

Yeah, I was wondering about something like that myself. I was thinking something more specific to Scoop like Scoop Apps for Scoop Applications. Reason being I know a few places that group corporate apps under a Start menu folder called Apps. Grouping them under a single folder would make more sense.

Cool, let's try it out!

@lukesampson Any reason why not put the shortcuts into the start menu directly instead of using the Scoop apps folder? Would make them feel more "native" if you know what I mean, taking scoop out of the shortcut making them feel like they were installed "normally".

Or maybe set the folder name as a config option:
scoop config startmenu_folder “Scoop Apps”

and default to “” for new installs?

Yes, that might be nicer installing directly to the start menu by default but making it configurable.

This would address the Windows 10 issue in https://github.com/lukesampson/scoop/issues/2124


nsurlerrordomain code=-1004 error

Bug description:

Hi everyone. I am using this wonderful library and it gives error I attached below only at ios. I checked documentations but couldn't find the reason why this error occurs. Are there any body experienced this? (Note: android works without no problem)
Screenshots/Videos:

Environment:

OS: ios
OS version: all
react-native version:
react-native-webview version:

@ozergul did you found any solution for this?

I logged onError and saw that an URL while redirection gives 500 error. So react-native-webview gives this -1004 error if some server gives 500.

I am having a similar issue. Unfortunately for me, the App Store Review team is having this issue, which I am not able to reproduce at all in simulation or on physical iPhone.
@ozergul , you are saying that the problem comes from the server giving an error code 500, and you didn't have to change anything on the App side to get it to work? But how could it be working for Android and not iOS? (the server is the same, no?)
@nolife08021  did you find a solution to the problem?

@tingofurro our problem was related with third party payment processor. They have had very simple bug on their backends (related with useragent)
That's why app was broken on Ios but not Android
I would prefer adding some logs to backend to understand why the web page broken for App Store Review team
Or see my above message. Log onError with sentry or something else and you will see exactly which page is broken

For future lost souls, it was in fact a DNS problem, where Apple's testing environment does not follow Dynamic DNS (no-ip or ddns.net).

hi @tingofurro thanks! do you have by any chance the source of where apple states this about their testing environment not following up Dynamic DNS?
and what's the solution for that case?

hey @ednjv, I never found the information in their documentation, I just tested it empirically.
I changed my domain name to point to a static IP directly, and when I resubmitted with no other change, I got through to the next step (which were other problems), but eventually got the app accepted!
I hope this helps :)


[feat] provide direction when creating relationship

In most of our scenarios, we build cypher-dsl statements programatically with runtime evolving schemas, meaning that the input to our cypher-dsl builder generally contains a triplet for relationships like (nodeToExpand, otherNodeLabel, direction) which leads to have very often the following logic in the cypher-dsl builder :
var node = Cypher.node(nodeToExpand.label()).named("n");
var otherNode = Cypher.node(otherNodeLabel).named("o");
var rel = direction.equals(Direction.OUTGOING)
   ? node.relationshipTo(otherNode)
   : node.relationshipFrom(otherNode);

It would be great if as a user I could simply pass a direction, something like :
node.relationshipTo(otherNode, "TYPE", Direction.INCOMING)

It's not like @jexp did request this already when we have been designing the builder ;)


Grommet Docs is Broken

Notified by https://twitter.com/BaseDefined/status/1348482693093339136
Users are unable to search and the code examples are not working.

might be related as well

might be related as well

Ahh, so that's why it's been acting up :thinking: good find, thanks! Will subscribe myself to this issue -- if any help is needed, please reach out :-)

Ahh, so that's why it's been acting up :thinking: good find, thanks! Will subscribe myself to this issue -- if any help is needed, please reach out :-)

Is it possible that this broke the app? I remember it working fine before that. I'll test this tomorrow.

Is it possible that this broke the app? I remember it working fine before that. I'll test this tomorrow.


Improve warning messages for no_#_allowed_in_word to include client and topic

Environment

VerneMQ Version: 1.9.2 Official Docker Alpine image

In logs I see messages.
10:58:56.563 [warning] session stopped abnormally due to '{error,unexpected_message,{error,'no_#_allowed_in_word'}}'
10:58:57.535 [warning] stopped connected session, due to unexpected frame type {error,'no_#_allowed_in_word'}

But from such log message I can not find out which client provided bad topic and which topic caused such error. I do not know if that was our client or somebody trying to hack. If that is our client we need to know what topic it tried to subscribe so that we can go to that client and check logs there also later to fix code.
I am not sure if enabling debug mode will show more here. I do not want to restart whole broker as all of my 6k clients will  reconnect and this problem does not always shows up immediately after restart.
Also there are other validation errors so all of them could include such information.

@vytasmk thanks, yes, we could get the subscriberID from state, and log it: https://github.com/vernemq/vernemq/blob/7e8963fd645a22b4d2e0b8ffce94227ecb31ba21/apps/vmq_server/src/vmq_mqtt_fsm.erl#L479


Attempted to fork repository for personal use

Sorry, this is not a bug with the module, but a general question.
I tried forking the repository, editing it, and then yarn adding it to my personal project. However, when importing 'react-d3-graph' I get "Module not found" error. I believe this is because the build files are not in the 'react-d3-graph' repo since everything else looks correct. So then I tried running "npm run build" in the forked 'react-d3-graph' repo to generate the build files, but the "script does not exist". Any suggestions? How do I build this repo?

how did you solve this?


Broken CSS and JS

What steps will reproduce the problem?
1. Enable mod_pagespeed

What is the expected output? What do you see instead?
The page breaks. CSS is missing.
[Warning] Invalid CSS property declaration at: * (explore, line 12)
[Error] SyntaxError: Unexpected token '<'
    (anonym funksjon) (404, line 1)
[Error] ReferenceError: Can't find variable: mod_pagespeed_zDPdHCVuFD
    global code (explore, line 17)
[Error] ReferenceError: Can't find variable: mod_pagespeed_iQpYc3y0WS
    global code (explore, line 19)
[Error] ReferenceError: Can't find variable: mod_pagespeed_CEsjKphoff
    global code (explore, line 20)
[Error] ReferenceError: Can't find variable: mod_pagespeed_BsJQbncASZ
    global code (explore, line 21)

Disabling filters rewrite_javascript, and rewrite_images seems to fix it.

What version of the product are you using (please check X-Mod-Pagespeed
header)?
X-Mod-Pagespeed <IP_ADDRESS>-4307

On what operating system?
Ubuntu 12.04 3.13.0-34-generic

Which version of Apache?
Apache/2.4.10 (Ubuntu)

Which MPM?

URL of broken page:
http://cloudaud.io

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 9 Nov 2014 at 9:50

Recently page speed has been implemented on our website-
there is a functionality for reset password on my website that triggers otp to mobile. After getting otp page redirects to reset password page. Afetr entering otp and clicking on continue i am getting the following error :
clientlibs.js.pagespeed.jm.P4bq0aKKBT.js:8 Uncaught TypeError: Illegal invocation
at K (clientlibs.js.pagespeed.jm.P4bq0aKKBT.js:8)
at a.fn.init.text (clientlibs.js.pagespeed.jm.P4bq0aKKBT.js:8)
at Object.success (resetpassword.js.pagespeed.jm.b8U5GDu-kU.js:1)
at i (clientlibs.js.pagespeed.jm.P4bq0aKKBT.js:8)
at Object.fireWith [as resolveWith] (clientlibs.js.pagespeed.jm.P4bq0aKKBT.js:8)
at z (clientlibs.js.pagespeed.jm.P4bq0aKKBT.js:8)
at XMLHttpRequest. (clientlibs.js.pagespeed.jm.P4bq0aKKBT.js:8)
K @ clientlibs.js.pagespeed.jm.P4bq0aKKBT.js:8
text @ clientlibs.js.pagespeed.jm.P4bq0aKKBT.js:8
success @ resetpassword.js.pagespeed.jm.b8U5GDu-kU.js:1
i @ clientlibs.js.pagespeed.jm.P4bq0aKKBT.js:8
fireWith @ clientlibs.js.pagespeed.jm.P4bq0aKKBT.js:8
z @ clientlibs.js.pagespeed.jm.P4bq0aKKBT.js:8
(anonymous) @ clientlibs.js.pagespeed.jm.P4bq0aKKBT.js:8
XMLHttpRequest.send (async)
send @ clientlibs.js.pagespeed.jm.P4bq0aKKBT.js:8
ajax @ clientlibs.js.pagespeed.jm.P4bq0aKKBT.js:8
submitResetPassRequest @ resetpassword.js.pagespeed.jm.b8U5GDu-kU.js:1
(anonymous) @ resetpassword.js.pagespeed.jm.b8U5GDu-kU.js:1
dispatch @ clientlibs.js.pagespeed.jm.P4bq0aKKBT.js:8
r.handle @ clientlibs.js.pagespeed.jm.P4bq0aKKBT.js:8

@Ranjit-Kr You could try disabling the rewrite_javascript to see if that resolves the issue.
https://www.modpagespeed.com/doc/config_filters
Note that this issue is old and has been closed a long time ago -- I think it's best to move further discussion to https://groups.google.com/d/topic/mod-pagespeed-discuss


[CL][e2e]: CreateConcentratedPosition

CreatePosition
rename to CreateConcentratedPosition

Closes: #4055
What is the purpose of the change
This PR adds a function - CreateConcentratedPosition - that creates a concentrated position
Brief Changelog
CreateConcentratedPosition added to commands.go
Testing and Verifying
Testing blocked before #4038 is merged
Documentation and Release Note
no

Wdyt about pointing this PR to ruslan/e2e-cl instead of main?

@p0mvn Current positions:


build(typescript): upgrade to Typescript v4.9.x

In order to upgrade the frontend projects to Angular v16 we need to first
migrate the compiler to v4.9.x
Source: https://angular.io/guide/versions

This will be fixed by another issue because of cascading dependencies: https://github.com/hyperledger/cacti/pull/1475


Support for draining node during pod termination

Please describe your use case / problem.
We have a legacy very-stateful application (tomcat/java), which needs sticky sessions. When we deploy new versions of our applications, or a scale down event happens, we need to stop sending new connections to a server, while sending bound sessions to the old server. Please note: this is not referring to in-flight requests, we're needing the active tomcat sessions to expire, which normally takes about an hour.
We have this working currently in Kubernetes with HAProxy-Ingress by setting its drain-support flag, which drains a pod when it transitions to Terminating, but keeps existing sessions attached to that pod. We then have a preStop hook which blocks the shutdown until users have finished their sessions.
When looking at Ambassador to fulfill this ingress role, we see the pod being removed from routing as soon as it transitions to Terminating. This was tested with a simple application returning the hostname of the pod:

Getting two sessions against different pods - Pod A and Pod B
Scale the deployment down to 1 - Pod A moved to Terminating, Pod B still active
Session immediately switched over to the remaining pod - that is, Pod A session is dropped

Describe the solution you'd like
We'd love for ambassador to drain a pod when it is in Terminating, and keep existing connections alive, whilst routing new connections to other active Pods.
Additional context
Looking at the Kubernetes docs, it looks like they are dropping the pods from the Service registry, as soon as they pod moves to terminating. As per: https://kubernetes.io/docs/concepts/workloads/pods/pod/#termination-of-pods
3. Pod shows up as “Terminating” when listed in client commands

4. (simultaneous with 3) When the Kubelet sees that a Pod has been marked as terminating because the time in 2 has been set, it begins the pod shutdown process.
...
5. (simultaneous with 3) Pod is removed from endpoints list for service, and are no longer considered part of the set of running pods for replication controllers. Pods that shutdown slowly cannot continue to serve traffic as load balancers (like the service proxy) remove them from their rotations.

Test Details
Ambassador version: 0.60.1

Ambassador Service Configuration
getambassador.io/config: |
      ---
      apiVersion: ambassador/v1
      kind: KubernetesEndpointResolver
      name: my-resolver
      ---
      apiVersion: ambassador/v1
      kind:  Module
      name:  ambassador
      config:
        resolver: my-resolver
        load_balancer:
          policy: round_robin

Ambassador Target Service Configuration:
apiVersion: v1
kind: Service
metadata:
  name: sticky
  labels:
    app: sticky
  annotations:
    getambassador.io/config: |
      ---
      apiVersion: ambassador/v1
      kind:  Mapping
      name:  sticky_mapping
      prefix: /sticky/
      service: sticky
      resolver: my-resolver
      load_balancer:
        policy: maglev
        cookie:
          name: sticky-cookie
          ttl: 300s
spec:
  ports:
  - name: http
    port: 80
  selector:
    app: sticky
---
apiVersion: apps/v1
kind: Deployment
metadata:
  name: sticky
spec:
  replicas: 2
  selector:
    matchLabels:
      app: sticky
  template:
    metadata:
      annotations:
        sidecar.istio.io/inject: "false"
      labels:
        app: sticky
    spec:
      terminationGracePeriodSeconds: 120
      containers:
      - name: hello
        image: nginxdemos/hello
        lifecycle:
          preStop:
            exec:
              command:
              - /bin/sleep
              - '60'
        ports:
        - containerPort: 80

Related issue for nginx ingress controller: https://github.com/nginxinc/kubernetes-ingress/issues/267

Somewhat related use-case:
Could someone answer the following question regarding the use of ring_hash load-balancing with a header? If we’re using a header (ex: user-id) to create session affinity between a specific user and a pod, is the affinity immediately cleared once that pod is downed by k8s (in case of a failure or new deployment)?
We’re using multiple long-lived gRPC connections for each user (Chat app) and have the idea of using session affinity to ensure any connections for a specific user end up on the same pod. When set up in this fashion, it would allow us to have those server / pod instances reroute data to the appropriate pod in which is affinitzed to a specific user by setting the user-id header to the “destination” user and essentially calling their own route through Ambassador.
It seems like its worth a try, but I want to ensure that if a pod is downed, all of the user-id header values affinitized to said pod get evicted and the next incoming request with that header is forwarded and affinitized to a new pod. (edited)
Aaron Triplett [11:45 PM]
Looks like I found the answer in an envoy issue: https://github.com/envoyproxy/envoy/issues/2819
Long-lived connections would stay connected but new requests would hit a new pod. Split-brain issue, and likely not a good idea until Envoy has connection termination support.

I've also opened this on the envoy side: https://github.com/envoyproxy/envoy/issues/6730

Wanted to note this needs to happen on any host change, scaling up, scaling down, and pod termination or failure.

I'm estimating this at 5 story points because it involves Envoy and Kubernetes and Ambassador. Honestly fitting this into parc-güell may be a bit aggressive.

I'm currently working on this, and this is at a state where the terminating pods have made their way to Envoy - but indicating Envoy that this connection is being drained is proving to be a problem.
I've opened an issue in Envoy regarding this at https://github.com/envoyproxy/envoy/issues/7218
Also, from Slack -
I've tried the following approaches -

Whenever a pod goes into terminating, Ambassador captures it and puts the terminating pod's IP right back there with the endpoints with -
1. setting `load_balancing_weight: 1` (allowed values are 1-128) - but this does not works as expected! The new requests are not routed to this IP - but the older sticky sessions do not make their way to the terminating pod. I _thought_ this was working but scaling the pods to more than 2 busted this myth. I'm surprised as this is the implementation used by haproxy-ingress - but envoy behaves differently.

2. setting `'health_status': 'DRAINING'` for the endpoint (See https://www.envoyproxy.io/docs/envoy/latest/api-v2/api/v2/core/health_check.proto#envoy-api-enum-core-healthstatus). This makes envoy believe that the endpoint is unhealthy so no requests are routed to this, but again, the older sticky sessions do not persist.

I've tested with websocket connections as well as normal ones, but no luck. Now there could be something wrong with my testing as well. In subsequent `curl` requests from the command line, `curl` does not re-use the same connection again - which I think is fine as long as the sticky conditions are met i.e. the right cookie is set. Is the right?

Also, I've noticed that when a pod with a preStop hook preventing it to die goes into terminating state, the connection between host and the pod remains intact even though Ambassador (and hence Envoy) removes the host from the endpoint list - which makes me think that this issue is for subsequent sticky connections than for single persistent connections.

Any leads/pointers appreciated.

not stale

not stale

not stale

not stale

not stale

not stale

Hi, Would this issue include other active connections? Like draining TCP connections

Contour already has this and their implementation seems pretty solid https://github.com/projectcontour/contour/issues/145

Any updates on this one? I see Contour has added a shutdown manager sidecar for handling clean shutdowns in https://github.com/projectcontour/contour/pull/2227/files . I tried to to add a preStop hook for draining connections and failing envoy healthchecks, but 8877/ambassador/v0/check_ready was still reporting 200 OK, while 8001/ready was reporting 503s (as it should). I also tried setting the health checks in 8001/ready, but it seems Envoy's admin port does not bind to <IP_ADDRESS>.

We have a patch we'll put up soon for this in legacy mode, but non-legacy is looking a bit tricky, because dgroup makes assumptions - in particular that there are no actions accessible to clients of dgroup during graceful shutdowns: https://github.com/datawire/dlib/blob/master/dgroup/group.go#L294
One possible way is to permit fallible user supplied calls to be called-back-to by the graceful shutdown logic. But also dgroup explicitly doesn't support dependencies (https://github.com/datawire/dlib/blob/master/dgroup/group.go#L19) I note ambex that similarly needs to manage TERM logic and doesn't use dgroup.
Another possibility would be a config option that disables signal handling for TERM, allowing the TERM logic to be written outside of dgroup, rather than the all-or-nothing signal handling present today.
Some guidance on this would be needed before we start investing in coding.

@rbtcollins
Yeah, I've been growing increasingly dissatisfied with dgroup's signal
handling. Not so much as an excuse, but as explanation: It's more or
less exactly what the amb-sidecar part of AES needed for signal
handling, since dgroup was originally factored out from AES.
That said, I'm not sure I follow what you need dgroup to do that it
won't let you.  The way to have graceful shutdown code would be have a
goroutine that just blocks on <-ctx.Done() before doing anything:
grp.Go("drainer", func(ctx context.Context) error {
	<-ctx.Done() // wait for graceful shutdown to start
	ctx = dcontext.HardContext(ctx)

	// graceful shutdown code goes here

})

That said, if you're still unsure, but you've got a working patch for
legacy mode, I'd say to go ahead and just PR that to release/v1.14
(legacy mode is "gone" in master and release/v2.0).  From there,
I'd happily do/assist in porting it over to non-legacy mode and v2.

Ah, thanks for the hint. Yes, we'll get the legacy patch up in a day or two , just sorting attribution stuff internally and validating the most recent forward port - its currently against master, since all the code is still there as of late last week, but we'll see :)

A lot of the legacy-mode code is still on master, but a lot of it has been deleted (in particular: cmd/watt and pkg/supervisor).

@LukeShu the problem is that dexec.CommandContext kills off things when the graceful shutdown starts. Which is the opposite of graceful when dealing with longer lived sockets in a subprocess.
We could perhaps write a layered Context.Background() :
    group.Go("envoy", func(ctx context.Context) error {
        child_ctx,cancel  := context.WithCancel(Context.Background())
        go runEnvoy(ctx, envoyHUP)
        <- ctx.Done()
        ctx = dcontext.HardContext(ctx)
        ... graceful drain of envoy
       cancel()
    })

This might also need the same done for ambex, to ensure that envoy doesn't get configuration errors during draining.
Thoughts?

Righto, PR up at #3926

@LukeShu ping [on the PR and the golang thoughts here]

Any updates on this one? We were seeing 5XXs during pod restarts which we were able to fix by calling localhost:8001/drain_listeners?graceful on a preStop lifecycle, but we prefer to have this handled/fixed by ambassador. We are using the AMBASSADOR_LEGACY_MODE.

Hey @marianafranco and thread, apologies for the delay, the team will get back to you on this shortly!

@marianafranco you could use our patch from #3926 for legacy mode. We haven't got a patch for non-legacy together yet.

@rbtcollins Thanks for this fix! I tested it in one of our staging environments and it's working (no 5XXs during pod restarts/termination) when AMBASSADOR_LEGACY_MODE=true is set.
Unfortunately, I was mistaken and we are not using the AMBASSADOR_LEGACY_MODE today. Do you know what would be the downside of move back to the AMBASSADOR_LEGACY_MODE vs the default one?

@rbtcollins Thanks for this fix! I tested it in one of our staging environments and it's working (no 5XXs during pod restarts/termination) when AMBASSADOR_LEGACY_MODE=true is set.
Unfortunately, I was mistaken and we are not using the AMBASSADOR_LEGACY_MODE in production today. Do you know what would be the downside of move back to the AMBASSADOR_LEGACY_MODE vs the default one (we are using v1.14.2)?

I don't know - needless to say we're using legacy mode just fine :).

Hi, any updates if this is supported on the non-legacy versions?
If not, curious to understand if switching to LEGACY mode will have any consequences.

Hi @mohitreddy1996 , the docs indicate that AMBASSADOR_LEGACY_MODE is not recommended when performance is critical.  Closing as it appears the original issue has been addressed.

@cindymullins-dw I'm not sure this has been addressed…
The patch @rbtcollins shared is for legacy mode only, and as you point out legacy mode is not recommended if one needs high performance.


[core]Remove created_from_snapshot field from BranchesTable

Purpose
in #3934 ,we removed create branch from snapshots, maybe we also should remove created_from_snapshot field from BranchesTable.
Linked issue: close #xxx

Tests

API and Format

Documentation

It is possible that the branches tag does not exist but the snapshot exists?

It is possible that the branches tag does not exist but the snapshot exists?

Yes, if you create empty branch then insert some data to this branch, or you create branch from tag then delete branch's tag. In above two case snapshot will exists,tag will not exists. In this case created_from_tag will be null.
I do not think we should remain created_from_snapshot,because we only support create_branch from tag. Snapshot will expire quickly, it's hard for us to know whether the earliest snapshot of the branch is from the main branch.

created_from_snapshot can be the origin of the tag?

created_from_snapshot can be the origin of the tag?

That's okay, I'll modify it.

With the current architecture, we cannot accurately obtain the origin snapshot of the branch. Maybe we should open a discussion thread to discuss this issue.

With the current architecture, we cannot accurately obtain the origin snapshot of the branch. Maybe we should open a discussion thread to discuss this issue.

I create a issue #4166

Can we just delete created_from_tag and created_from_snapshot first?

Can we just delete created_from_tag and created_from_snapshot first?

I think it's ok if we can't get the exact tag and snapshot.I'll change it.


UI design

Decide the pages and components we'll use.

[ ] decide on colors
[ ] mock mobile pages
[ ] mock desktop pages

[ ] About Page
[ ] Account Page
[ ] Browse Page
[ ] Favorites Page (possible component)
[ ] Message Page (possible component)
[ ] Post Detail Page
[ ] Sign Page


Unable to remove cookie on https domain

when the cookie is set on the server side, it has httpOnly set to false, secure to false.
I can access the cookie in the SPA, but unable to delete it(set it to past).

Having Same Issue

What i do for now, is deleting it on the server side..

doesn't work for me either. from SO
function clearCookie(name) {
  let domain = document.domain
  let path = '/'
  document.cookie = name + '=; expires=' + +(new Date()) + '; domain=' + domain + '; path=' + path
}


UnicodeEncodeError: 'latin-1' codec can't encode characters in position 28-33: ordinal not in range(256)

Traceback (most recent call last):
  File "/app/.venv/lib/python3.10/site-packages/uvicorn/protocols/http/httptools_impl.py", line 426, in run_asgi
    result = await app(  # type: ignore[func-returns-value]
  File "/app/.venv/lib/python3.10/site-packages/uvicorn/middleware/proxy_headers.py", line 84, in __call__
    return await self.app(scope, receive, send)
  File "/app/.venv/lib/python3.10/site-packages/fastapi/applications.py", line 1106, in __call__
    await super().__call__(scope, receive, send)
  File "/app/.venv/lib/python3.10/site-packages/starlette/applications.py", line 122, in __call__
    await self.middleware_stack(scope, receive, send)
  File "/app/.venv/lib/python3.10/site-packages/starlette/middleware/errors.py", line 184, in __call__
    raise exc
  File "/app/.venv/lib/python3.10/site-packages/starlette/middleware/errors.py", line 162, in __call__
    await self.app(scope, receive, _send)
  File "/app/.venv/lib/python3.10/site-packages/starlette/middleware/exceptions.py", line 79, in __call__
    raise exc
  File "/app/.venv/lib/python3.10/site-packages/starlette/middleware/exceptions.py", line 68, in __call__
    await self.app(scope, receive, sender)
  File "/app/.venv/lib/python3.10/site-packages/fastapi/middleware/asyncexitstack.py", line 20, in __call__
    raise e
  File "/app/.venv/lib/python3.10/site-packages/fastapi/middleware/asyncexitstack.py", line 17, in __call__
    await self.app(scope, receive, send)
  File "/app/.venv/lib/python3.10/site-packages/starlette/routing.py", line 718, in __call__
    await route.handle(scope, receive, send)
  File "/app/.venv/lib/python3.10/site-packages/starlette/routing.py", line 276, in handle
    await self.app(scope, receive, send)
  File "/app/.venv/lib/python3.10/site-packages/starlette/routing.py", line 66, in app
    response = await func(request)
  File "/app/.venv/lib/python3.10/site-packages/fastapi/routing.py", line 274, in app
    raw_response = await run_endpoint_function(
  File "/app/.venv/lib/python3.10/site-packages/fastapi/routing.py", line 191, in run_endpoint_function
    return await dependant.call(**values)
  File "/app/app/webservice.py", line 78, in asr
    return StreamingResponse(
  File "/app/.venv/lib/python3.10/site-packages/starlette/responses.py", line 246, in __init__
    self.init_headers(headers)
  File "/app/.venv/lib/python3.10/site-packages/starlette/responses.py", line 73, in init_headers
    raw_headers = [
  File "/app/.venv/lib/python3.10/site-packages/starlette/responses.py", line 74, in <listcomp>
    (k.lower().encode("latin-1"), v.encode("latin-1"))
UnicodeEncodeError: 'latin-1' codec can't encode characters in position 28-33: ordinal not in range(256)

Hey, how did you solve the problem?

Hey, how did you solve the problem?

Changed the data format to mp3


Fix Stock FFT for CUDA 12.2 on Perlmutter

Relying on the compiler intrinsic macro __AVX__ leads to compile errors when including the public heFFTe header in CUDA code of WarpX of the form:
/usr/lib64/gcc/x86_64-suse-linux/12/include/avx512fp16intrin.h(38): error: vector_size attribute requires an arithmetic or enum type
  typedef __half __v8hf __attribute__ ((__vector_size__ (16)));
                                        ^
...
/usr/lib64/gcc/x86_64-suse-linux/12/include/avx512fp16intrin.h(62): error: more than one conversion function from "__half" to "<error-type>" applies:
            function "__half::operator __half_raw() const" (declared at line 309 of /opt/nvidia/hpc_sdk/Linux_x86_64/23.9/cuda/12.2/include/cuda_fp16.hpp)
            function "__half::operator float() const" (declared at line 337 of /opt/nvidia/hpc_sdk/Linux_x86_64/23.9/cuda/12.2/include/cuda_fp16.hpp)
            function "__half::operator signed char() const" (declared at line 436 of /opt/nvidia/hpc_sdk/Linux_x86_64/23.9/cuda/12.2/include/cuda_fp16.hpp)
            function "__half::operator unsigned char() const" (declared at line 443 of /opt/nvidia/hpc_sdk/Linux_x86_64/23.9/cuda/12.2/include/cuda_fp16.hpp)
            function "__half::operator char() const" (declared at line 451 of /opt/nvidia/hpc_sdk/Linux_x86_64/23.9/cuda/12.2/include/cuda_fp16.hpp)
            function "__half::operator short() const" (declared at line 476 of /opt/nvidia/hpc_sdk/Linux_x86_64/23.9/cuda/12.2/include/cuda_fp16.hpp)
            function "__half::operator unsigned short() const" (declared at line 483 of /opt/nvidia/hpc_sdk/Linux_x86_64/23.9/cuda/12.2/include/cuda_fp16.hpp)
            function "__half::operator int() const" (declared at line 490 of /opt/nvidia/hpc_sdk/Linux_x86_64/23.9/cuda/12.2/include/cuda_fp16.hpp)
            function "__half::operator unsigned int() const" (declared at line 497 of /opt/nvidia/hpc_sdk/Linux_x86_64/23.9/cuda/12.2/include/cuda_fp16.hpp)
            function "__half::operator long() const" (declared at line 504 of /opt/nvidia/hpc_sdk/Linux_x86_64/23.9/cuda/12.2/include/cuda_fp16.hpp)
            function "__half::operator unsigned long() const" (declared at line 528 of /opt/nvidia/hpc_sdk/Linux_x86_64/23.9/cuda/12.2/include/cuda_fp16.hpp)
            function "__half::operator long long() const" (declared at line 554 of /opt/nvidia/hpc_sdk/Linux_x86_64/23.9/cuda/12.2/include/cuda_fp16.hpp)
            function "__half::operator unsigned long long() const" (declared at line 561 of /opt/nvidia/hpc_sdk/Linux_x86_64/23.9/cuda/12.2/include/cuda_fp16.hpp)
            function "__half::operator bool() const" (declared at line 593 of /opt/nvidia/hpc_sdk/Linux_x86_64/23.9/cuda/12.2/include/cuda_fp16.hpp)
    return __extension__ (__m128h)(__v8hf){ __A0, __A1, __A2, __A3,
                                            ^
...

Since heFFTe has explicit control for AVX via macros, we can rely on those. Most GPU users will disable AVX support anyway when building for GPU.
System details
Perlmutter (NERSC) HPE machine with:

GCC 12.3.0
NVCC 12.2.91

Currently Loaded Modules:
  1) libfabric/<IP_ADDRESS>                      4) gcc-native/12.3          7) gpu/1.0                  10) cray-libsci/23.12.5 (math)  13) craype-x86-milan      (cpe)  16) cmake/3.24.3                (buildtools)
  2) craype-network-ofi                      5) perftools-base/23.12.0   8) cray-dsmml/0.2.2         11) PrgEnv-gnu/8.5.0    (cpe)   14) craype-accel-nvidia80        17) cray-hdf5-parallel/<IP_ADDRESS> (io)
  3) xpmem/2.6.2-2.5_2.38__gd067c3f.shasta   6) cpe/23.12                9) cray-mpich/8.1.28 (mpi)  12) craype/2.7.30       (c)     15) cudatoolkit/12.2      (g)    18) cray-python/3.11.5          (dev)

Kudos
Thanks to @aeriforme and @Haavaan for reporting this. First seen in https://github.com/ECP-WarpX/WarpX/pull/4937

attn @mkstoyanov

I'll test and merge some time tomorrow. Thanks for fixing this.

Fully understand, thanks a lot 😊 have a nice evening!

Thank you for merging! :pray:


Update to v0.6.0

Update version in Makefile to v0.6.0
Print commit id in addition to version
Only log for specific commands starting with init and server

/ok-to-test
/lgtm


Fix the submap transformation in the RViz plugin.

Removes the SceneManagerListener and instead computes the transform
when a new submap list is received. The second part of the transform
is also updated when a new texture is received.
Also simplifies the drawing code a bit by using a triangle strip.

LGTM


Storage attributes

In progress Support the following Storage features:

[x] Support generation on file copy
[ ] Support Bucket attributes
[ ] Support File attributes
[ ] Support File metadata

[Closes #89, Closes #91, Closes #92]

Closing as new pull requests will be created for this. e.g. #281


Delete record with ID == GUID doesn't work right

Create record with this:
        id = (String.IsNullOrEmpty((String)input.ID) ? Guid.NewGuid().ToString() : (String)input.ID);
        col.Insert(new TTS { ID = id, TotalRequests = 1, Text = (String)input.Text });

Everything works great if I pass in something like 103, 101, etc - including deletes.
If I leave that blank and let it auto-assign a string Guid value, everything works fine EXCEPT delete.
When I run this code:
if (col.Delete(s))   (where s is the value from above)
I get a TRUE response, but then when I run:
col.FindAll() - I still get back that record EXCEPT that Text has been blanked out:
    private class TTS
    {
        public String ID { get; set; }
        public String Text { get; set; }
        public Int64 TotalRequests { get; set; }
    }

Here is what I'm using to get the collection in all cases:
                var col = db.GetCollection<TTS>((String)input.IVR_Application);

If I remove the dashes, it also works fine.
id = (String.IsNullOrEmpty((String)input.ID) ? Guid.NewGuid().ToString().Replace("-", String.Empty) : (String)input.ID);

Please ignore my shoddy code.  Everything works great, I was actually falling out after the delete and creating a dummy record.  Nothing to see here folks, move along.


External Image from API

I wanted to use this library but I cant make it work using external url (images) coming from an API.

From #74, it seems like this isn't possible ☹ :

This is not possible, unfortunately. Since this is only a webpack loader, it can only handle local files (you can also not import remote js files with webpack). If you are in charge of the API, you can optimize the images in the API (e.g. on upload). If you are using a 3rd party API and they haven't optimized their images, you could also use an image optimization proxy in front of their images.


fixes metrics designer workspace layout

Distributes space better between measures and dimensions with the following behavior:

if both tables fit in the available space, they take up the amount of space they need to fit their contents
if both tables together exceed the available height, and one takes up less than 50% of the space, the bigger one shrinks to accommodate the small one
if the tables together exceed the available height and both take up more than 50% of the available space, both should have max height 50%

The build is failing and I am getting this error when I run npm run build locally.

[vite:load-fallback] Could not load /Users/marissagorlick/Documents/rilldata/rill-developer/src/lib/components/table-editable/Table.svelte (imported by src/lib/components/table-editable/PinnableTable.svelte): ENOENT: no such file or directory, open '/Users/marissagorlick/Documents/rilldata/rill-developer/src/lib/components/table-editable/Table.svelte'
> Could not load /Users/marissagorlick/Documents/rilldata/rill-developer/src/lib/components/table-editable/Table.svelte (imported by src/lib/components/table-editable/PinnableTable.svelte): ENOENT: no such file or directory, open '/Users/marissagorlick/Documents/rilldata/rill-developer/src/lib/components/table-editable/Table.svelte'

The build is failing CI and I am getting this error when I run npm run build locally.

Dang, not sure what happened there :-( I'll fix it and update.

I haven't seen the features yet, but from the description I am also wondering why we decided not to implement the draggable bar we have on other surfaces so users can resize to what makes sense for them. Is that part of this PR?

This is just a first step to make the default space distribution at least reasonable (hamilton and dhiraj noted that it was hard to use, and I agree), and it puts in place the ground work for the dragger. I can add that in this PR if you want, or put this in and follow up with the dragger later.


Fix incorrect Etcd lock ticker initialization

Etcd TTLs are in seconds, without this fix the
ticker goes off uncontrollably every millisecond,
flooding etcd with put requests
Signed-off-by: Mariusz Borsa<EMAIL_ADDRESS>

@wrotki nice catch, this slipped through the last PR. LGTM when the comment above is addressed.

Here you go. Thought that this extra nanosecond could prevent NewTicker from panicking when the TTL somehow drops below one second, but in this case it is not a real possibility.

Thanks @wrotki! LGTM
Would be perfect if you can squash your commits into one though before merging :+1:

Pushed the squashed change


fix PDF ts links

Reported by Ben on Slack: in the current PDF, the links are dead (because they are relative links). This PR changes the build script to turn them into absolute links in-between building the HTML and PDF versions.
This isn't "clean" or "robust" by any means, but I can't think of any other way to solve this.

I used a different approach in https://github.com/digital-asset/daml/pull/14225. Since we already have per-target conf.py, each with its rst_prolog, I thought we could define the typescript links there, with an absolute target in the pdf config. I think that's a bit cleaner since it keeps the logic in the daml repo.
This also fixes the links in docs/source/app-dev/bindings-ts/index.rst, which iiuc this PR doesn't.
A drawback of my approach is that the fix would only land with the next release, but I hope that's okay given the other considerations.

Thanks @akrmn !


Can't run a project

I have installed all the components and packages.
After init docker-compose up my website on port 5000 return '{"error":"Not Found"}'.
When i try to get info for example ':5000/api/proMatches' it return empty array.
how can i get all info correctly?
thank you

I read your blog and understood everything i want.
Thank you.

there will be no data in your local instance until you run some of the ingestion services, e.g. scanner.
If you want to request parses of specific matches, use the /request endopint to send parse requests to your local instance.
You need to manually start services besides web (see README)

Thanks for your answer.
On more question. If we would have more than "1200 calls per minute" you'll block our calls? Or we can increase their number?

Calls above the limit will be blocked, if you need more we sometimes work with users directly to raise the limits.

I see.
Btw we need only tournaments mathes. Can we get list of tournaments from your api with their matches?
I don't see it on your website.
Thank you

You can get a list of the tournament IDs then make additional queries for each ID.
Or you can use the data explorer to create a custom query that fetches matches and their tournament IDs.


Downloadbutton h5p element

User Story
As a researcher I want insights to the usage of a h5p learning element.
Problem: Currently there is no option to download the h5p log activities.
Acceptance criteria

[ ] Possible approaches have been evaluated (Own plugin to extend report functionality, extend h5p default plugin)
[ ] Moodle UI Includes Download button
[ ] Download is csv or similar
[ ] Analog to feedback activity download

Definition of Ready

[x] User Story is small enough for Sprint
[x] User Story is clearly understandable for everyone involved
[ ] User Story effort is estimated
[ ] User Story has acceptance criteria
[ ] User Story has added value for the product or development
[ ] User Story origin is known (stakeholder)
[ ] User Story is assigned to Release
[ ] Backend and Frontend are cloned from GitHub and Running locally
[ ] Basic knowledge of the backend and frontend architecture
[ ] Activity diagram (or sequence diagram) screenshot in issue

Definition of Done

[ ] All acceptance criteria are met
[ ] The implementation of the User Story has been successfully accepted by the Product Owner
[ ] The implementation is in GitHub
[ ] A code review has been performed (or created in pairing program)
[ ] Code conventions are followed
[ ] Unit test coverage must be greater than 90%
[ ] All tests must have passed
[ ] Technical documentation created (Feature documented in GitHub Wiki)
[ ] UML class diagrams and component diagrams created and up-to-date
[ ] Documentation in realization created (source code)
[ ] Changelog created (Clear description in merge request)
[ ] There are no known bugs

Comment
https://trello.com/c/0vsUYJsC/318-moodle-plugin-andere-webserver-ansprechen

Hey team! Please add your planning poker estimate with Zenhub @DimitriB01 @Platura


Bug in  use-response-body (#a06576)

Not all Excon exception messages have .response method.
This causes the following exception:
undefined method `response' for #<Excon::Errors::Timeout: read timeout reached>

Bug was introduced in: a06576566ba5bcc0a30104077f2ed1f312ec784d

Would make sense that a timeout doesn't have a response.  Will get a patch version out for that immediately.

Fixed in v1.21.2, the other errors should all have a response attached.


feat(prefer-immutable-types): allow overriding options based on where the type is declared

Fix #000
Proposed Changes

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#804  👈
#803
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @RebeccaStevens and the rest of your teammates on  Graphite

We probably want to bump the minimum ts version up to 4.7


Render commit messages as labels

Currently, commit messages are allowed via -m and stored, but not rendered in any way. Render these as labels after the branch/tag labels.

Just for information, I have updated your script in my fork to render the commit labels: https://github.com/fthomasfr/grawkit/blob/roadmap/tests/explanation.svg

That looks awesome! I'll look to incorporate these changes in once I've got some time on my hands.


Getting error on - pnpm dev

Hi, I am trying to run project locally and getting this error -

I would be thankful for any help. Thanks!!

It looks like you haven't configured your environment variables correctly or pulled them down locally from Vercel. Specifically the SHOPIFY_STORE_DOMAIN environment variable.


 I've started with the File menu, but if everything goes well I'll continue with other components like the Import popup or the top bar itself.
Anyways, this time I've changed only 2 things:

Adjusted divider on fileMenu
Replaced the big dot symbol we were using for selected selected workbooks with a check icon

Let me know what you think!
D

Hi @dg-ac ,
There is WIP commit in this PR, maybe that shouldn't be here?
Also once you make your changes make sure to run on the webapp folder:
npm run check

If that fails that means there is a linter error. You can normally fix that by running:
npm run check-write

That will not write a check, regrettably. Will check for lint issues in your code and fix them if possible.

Closing this PR in favour of #154


Use it for another Game

Hello , i would try to use your scripts for another Game called APEX can you pls tell me how i can Start this ?
what things i need to delet ?

retrain the model, then you can go.


chore: fix some comments

Problem
Summary of changes
fix some comments
Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

[ ] Do not forget to reformat commit message to not include the above checklist

Thanks~

friendly ping~


Update overview.md

For the sake of clarity, following should be clearly stated in this page. Currently, to find this out, one has to dig out the doc and already be aware of "policy layering", then google it then to find it out.
"If you have multiple policies, cumulative most restrictive permission will be applied. A single Deny policy trumps all Allowed policies within the within scope hierarchy. For example, if you have a Deny policy in Tenant Management Group that prevents creation of virtual networks but its subordinate management groups and subscriptions have explicit Allowed policy for virtual networks, you won't be able to create any virtual network."

Thanks for the contribution. I'm not going to merge the change but I created an internal backlog work item to review your suggestion for a future update.

Do not merge
#please-close


[WIP] fix PPO

PR type

[x] Bug Fix
[ ] New Feature
[ ] Document Updates
[ ] More Models or Datasets Support

PR information
Write the detail information belongs to this PR.
Experiment results
Paste your experiment result here(if needed).

Is it still in WIP status, or can it be reviewed now

Is it still in WIP status, or can it be reviewed now

It's still under testing now


Replace non-numeric characters in phone numbers with url encoding

PULL REQUEST
Summary
This PR changes the way that we generate the list of phone numbers to search for by replacing non-numeric characters, e.g. + and -, with their url encoded equivalents. This allows us to query for multiple phone numbers via eHealth Exchange, which eHealth devs recommend.
Related Issue
Fixes #167
Additional Information
We may look into more robust URL encoding for other fields, e.g., first names with spaces.
Checklist

[X] Descriptive Pull Request title
[X] Link to relevant issues
[ ] Provide necessary context for design reviewers
[ ] Update documentation

Nice find @robertandremitchell! I added your suggestion (had to use encodeURIComponent) to get it to work on the full list. I'm going to follow up this ticket with a more comprehensive look at how we can do encoding for all input fields since I know first name also can have this issue if you have two given names with a space, e.g., 'Jacob Jingleheimer`


Feature: Add support for NS1

As the title indicates, this is a feature request for adding support for NS1 via a new provider.
The new provider has already been written and is ready for review. Please let me know if this request is accepted and I'll open a new PR for the contribution.

My team could make use of this. Cool stuff.

@mburtless please create a PR. I'm happy to review it 👍

Kicked the tires on this. Works like a charm. Really appreciate it @mburtless 👍

No worries, thanks for testing it out!
I'll get the PR open as soon as I'm added to NS1's CLA


Column of type select_multiple doesn't work with AjaxTables

On the demo, I'm trying to create this column on articles
[
   // n-n relationship (with pivot table)
   'label' => "Tags", // Table column heading
   'type' => "select_multiple",
   'name' => 'tags', // the method that defines the relationship in your Model
   'entity' => 'tags', // the method that defines the relationship in your Model
   'attribute' => "name", // foreign key attribute that is shown to user
   'model' => "Backpack\NewsCRUD\app\Models\Tag", // foreign key model
]

works well on a normal table, but, when you enable ajaxTable it doesn't work.
SQLSTATE[42S22]: Column not found: 1054 Unknown column 'tags' in 'field list' (SQL: select `date` as `date`, `status` as `status`, `title` as `title`, `featured` as `featured`, `category_id` as `category_id`, `tags` as `tags`, `id` as `id` from `articles` order by `date` asc limit 25 offset 0)

Hi @Alexgmin ,
Thank you for the bug, I can confirm that n-n relationship columns don't seem to work on the AjaxDataTable. I'll look for a solution and get back to you.
Cheers!

@tabacitu I've had a dig around, and it seems the issue is occurring because the columns passed to the DataTable are not columns, so it breaks.
I've added a potential fix to prevent this - however the filter can be adjusted if there is a better way to find out if its a relational column
https://github.com/Laravel-Backpack/CRUD/pull/139

Hi @Alexgmin ,
You can use "select_multiple" columns with Ajax Datatables. A composer update should fix it for you.
Cheers!


[Full Video]** Pierre Feuille Ciseaux Robe Jaune X Pierre Feuille Ciseaux En Anglais

Full Video ⤵⤵⤵Pierre Feuille Ciseaux Robe Jaune X Pierre Feuille Ciseaux En Anglais
Watch Full HD Private Leaked Video Video Pierre Feuille Ciseaux Sur Robe Jaune Twitter Leaked Online
🔴 ➤► WATCH ✅👉 ONLINE VIDEO HERE
🔴 ➤► CLICK ✅👉HERE TO DOWLOAD VIDEO
Vidéo complète pierre feuille ciseaux robe jaune x , video pierre feuille ciseaux robe jaune , pierre feuille ciseaux robe jaune , pierre feuille ciseaux sur x idéo complète pierre feuille ciseaux robe jaune x , video pierre feuille ciseaux robe jaune , pierre feuille ciseaux robe jaune , pier
The viral video of Video Pierre Feuille Ciseaux Sur Robe Jaune Twitter emerging online has amassed tremendous views across fan accounts and social media sites with one video clip. The viral video circulating recently shows Video Pierre Feuille Ciseaux Sur Robe Jaune Twitter in an unexpected and hilarious moment while in public running errands.
Numerous fans have rushed to watch the viral video and share the clip amongst followers featuring Video Pierre Feuille Ciseaux Sur Robe Jaune Twitter caught in an amusing and awkward situation. While surprising, the authentic and candid video showing Video Pierre Feuille Ciseaux Sur Robe Jaune Twitter handling a real life blooper so genuinely has earned praise from viewers. Nonetheless, fans watch the current viral video of Video Pierre Feuille Ciseaux Sur Robe Jaune Twitter that emerged online with delight and clamor for what the celebrity icon’s reaction to the widely spread viral video will be.
The fan video showing Video Pierre Feuille Ciseaux Sur Robe Jaune Twitter first emerged on Twitter on Monday, posted by an amused fan who claimed to have captured the silly incident on their phone camera while out and about. The viral Clip of Video Pierre Feuille Ciseaux Sur Robe Jaune Twitter had garnered over 2 million views by Tuesday morning as more users on Instagram, Facebook and YouTube began uploading and sharing the embarrassing yet entertaining video.

Discussion over the viral Video Pierre Feuille Ciseaux Sur Robe Jaune Twitter video showing the star candidly dealing with sudden wardrobe issues dominated pop culture chatter online. Fans watched and rewatched the short video, praising Video Pierre Feuille Ciseaux Sur Robe Jaune Twitter for taking the malfunction with grace and humor even as cameras captured the celebrity video now flooding timelines. Some fans have scrutinized the viral clip, trying to discern what exactly led to the star appearing in such a silly viral video, seemingly unaware they were being filmed.
The video shows Video Pierre Feuille Ciseaux Sur Robe Jaune Twitter suddenly facing trouble with their outfit coming undone while casually walking about and running errands. Despite the embarrassment most would feel at having a wardrobe malfunction publicly, viral footage shows Video Pierre Feuille Ciseaux Sur Robe Jaune Twitter simply laughing the incident off themselves. It is this down-to-earth reaction that has earned Video Pierre Feuille Ciseaux Sur Robe Jaune Twitter such positive responses rather than ridicule after surprised fans rushed online to see the video and share the celebrity blooper seen around the internet.
While reps for the star have not commented directly on the leaked embarrassing video, Video Pierre Feuille Ciseaux Sur Robe Jaune Twitter will likely have to address the viral clip rapidly spreading online featuring quite the comedic celebrity wardrobe malfunction. Fans eagerly

Viral
### 📺📱Watch👉Click to Access the Link (Viral Video)
### 📺📱Watch👉Click to Access the Link (Viral Video)

Viral
### 📺📱Watch👉Click to Access the Link (Viral Video)
### 📺📱Watch👉Click to Access the Link (Viral Video)

Viral
### 📺📱Watch👉Click to Access the Link (Viral Video)
### 📺📱Watch👉Click to Access the Link (Viral Video)

Viral
### 📺📱Watch👉Click to Access the Link (Viral Video)
### 📺📱Watch👉Click to Access the Link (Viral Video)

Viral
### 📺📱Watch👉Click to Access the Link (Viral Video)
### 📺📱Watch👉Click to Access the Link (Viral Video)

➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭

Viral
### 📺📱Watch👉Click to Access the Link (Viral Video)
### 📺📱Watch👉Click to Access the Link (Viral Video)

Viral
### 📺📱Watch👉Click to Access the Link (Viral Video)
### 📺📱Watch👉Click to Access the Link (Viral Video)

Viral
### 📺📱Watch👉Click to Access the Link (Viral Video)
### 📺📱Watch👉Click to Access the Link (Viral Video)

Viral
### 📺📱Watch👉Click to Access the Link (Viral Video)
### 📺📱Watch👉Click to Access the Link (Viral Video)

Viral
### 📺📱Watch👉Click to Access the Link (Viral Video)
### 📺📱Watch👉Click to Access the Link (Viral Video)

➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭
➭

Viral
### 📺📱Watch👉Click to Access the Link (Viral Video)
### 📺📱Watch👉Click to Access the Link (Viral Video)

### 📺📱Watch👉Click to Access the Link (Viral Video)
### 📺📱Watch👉Click to Access the Link (Viral Video)

### 📺📱Watch👉Click to Access the Link (Viral Video)
### 📺📱Watch👉Click to Access the Link (Viral Video)

### 📺📱Watch👉Click to Access the Link (Viral Video)
### 📺📱Watch👉Click to Access the Link (Viral Video)

### 📺📱Watch👉Click to Access the Link (Viral Video)
### 📺📱Watch👉Click to Access the Link (Viral Video)

### 📺📱Watch👉Click to Access the Link (Viral Video)
### 📺📱Watch👉Click to Access the Link (Viral Video)

.
.
.
.
.
.


Configure with a file

Read configuration from ~/.config/gogh/config.yml
introduce: https://github.com/motemen/ghq/issues/21

GOGH_GHE_HOST can holds multiple host, but GOGH_GITHUB_TOKEN cannot.
It's inconsistent.

for v1
https://github.com/kyoh86/gogh/tree/v1