Create Social Media Preview Image

there is no social media preview image

https://socialsharepreview.com/?url=https://genesis.eden.eoscommunity.org/

Do you know where I can get this image? Or do we have to generate it?

We would need to generate these. nothing too complicated , maybe johanna or edwin can assist


A new subcategory under Observability and Analysis

We want to discuss a potential new category under Observability and Analysis.  We (StormForge) are currently listed under the Monitoring section, for lack of a better place to go.  The functionality of the platform, though, is much more accurately described as "Performance Testing and Application Optimization", and there is not really an appropriate subcategory under any category for experimenting on your apps and reacting to the results.
The O&A category feels closest aligned, because this is working on running apps, not working at the build or provision level.  So, I'd like to propose a subcategory named "Performance, Testing, & Optimization", under the Observability and Analysis category, and I believe there are others under the Monitoring umbrella that would also fit better under that heading.

I'm open to it, can you outline the new category and who would be there?

Okay, it's been a while, but we're getting back to this now.
The biggest defining characteristic of this category would be: products and projects that are focused on analyzing application or infrastructure behavior, then taking an active role in tuning various resources for desired outcomes, whether that's availability, performance, infra resilience, cost, or whatever other characteristics are desired.  It's really focused on the reactions being made from the observations being performed.
To that end, I think the originally proposed name is actually a bit misleading (for example, calling out "Testing" originally could easily include test suites, but that doesn't really fit the reactive nature that I think defines this).  So, we'd like to propose simply "Resource Optimization" as the title, with the idea that the resources may not be related to performance/applications/etc, and may be of a scope that we haven't considered yet.
As for who could be in this category, we don't necessarily want to speak for others, but Turbonomic, Replex, and AppNeta are easy fits.  Depending on how they feel about the scale of monitoring vs reactions from that monitoring, CloudHealth could be a good fit as well, and maaaaybe even Cloudwatch, though they seem more focused on the monitoring side.  There's also others who are not on this landscape yet, such as Cast.ai, who would now have a better place to land.
What are your thoughts?

see https://github.com/cncf/landscape/pull/3789


Add documentation for Text Matching

Could you provide a screen copy of the settings page to show how text matching could be used ?
How to:

Replace RCR by Road Crash Rescue ?
Highlight ALERT or FIRECALL ?
Use regex ?

Thank you 😀

Hi @pyrog
You can't do any replacing in the current implementation, only labeling on hover.
Highlighting of the word FIRECALL is in the default configuration, i highlight alert in my config as below.

Regex is a painful beast but very powerful once you know how to use it.
I use regex for highlighting street directory addresses
See the below regex string and the text it matches.

Did this help at all?

Here's my text matching settings if it helps:

Not using any regex there, but can see highlight vs text replacing.
@s3m1s0n1c is working on some documentation - may be worth adding to the list of things that need words

Thank you 😀


make filter arrow disabled instead of disappearing

Currently on the first and last page, the arrow for the end won't render at all.  This causes the entire page list to shift, which can confuse the user.

It's an easy change, but are you sure disabled is preferable to disappeared? I'll make the change and we can play around with that.

Mainly, I think we should just avoid the shift that causes the number to jump around.  If you want it to disappear but keep its spacing so the numbers don't move, that's fine also


Add find_additional_modules option to support partial bundling with externals

Disclaimer - WIP and untested!
This fairly chunky branch contains code that should allow us to search for rules matching modules when also bundling Workers.
Hey! 👋 This should be good to go now, see https://github.com/cloudflare/workers-sdk/pull/3726#issuecomment-1684225777 for details.

Confirming that this works and it's making the new lazy loading implementation of the workers nuxt adapter work 😃 🚀 🎉
See https://nuxt-split-workers-test.devdash.workers.dev/ (source)

Is there context on why we added this but didn't put in dev docs?

Is there context on why we added this but didn't put in dev docs?

An oversight on my part, or at least I got pulled onto other stuff and it fell off my radar; on top of that it was mainly designed for advanced users who kind of know the code already. But I think it is worth documenting this now.

https://github.com/cloudflare/cloudflare-docs/pull/17524


Fix/update docs

this PR makes two main changes:

it moves the examples related to data retrieval to its own file, outside of "Creating a conference", and adds a full example script for retrieving ICLR 19 metadata
it adds a submodule to openreview-py that can be called from the command line

I think the first change is necessary, but we don't have to do the second if people don't think this is a good way of distributing this kind of thing.
I personally think it will be useful to keep conference-specific data retrieval scripts somewhere, instead of trying to generalize the examples too much. There's also a good argument to be made that conference-specific retrieval scripts should be kept in the openreview-scripts repo, but including it with openreview-py means that users can call it directly from the command line.

I like the refactoring.
I think the script should go in the openreview-scripts repo. The library shouldn't distribute scripts. We need to put more effort in writing better examples and/or document the builder that contain function to retrieve papers, decisions, etc.

I can move the script to openreview-scripts.
Here's a question: should we remove the old scripts? I personally think that we should.

do you want to get rid of all the previous scripts?

That's what I was thinking, but I'm open to other suggestions.
My thought was that they would be mostly out of date, and not very useful to anyone.

Was there a reason to merge your own PR?

just trying to clean up. Mohit had approved and it seemed like your only issue was the script, which I removed...

We shouldn't approve our own PR unless there is an emergency and nobody can review it before sending it to live.
I was also a reviewer of this PR and I haven't approved it yet.

Sorry about that, my mistake. Should I revert it so that you can give it another look?


[HOLD for payment 2023-09-20] [$500] MacOS: Context Menu does not translate to Spanish

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

Action Performed:

Launch Expensify MacOS app
Set Profile > Preferences > Language > Spanish > Close
In Report Composer, right click

Expected Result:
ContextMenu options are translated to Spanish like MenuItems in ToolBar
Actual Result:
ContextMenu options are not translated to Spanish
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: v1.3.55-7
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/115492554/8ae9ee5d-a043-4334-8fb5-5dd83438ca9d
https://github.com/Expensify/App/assets/115492554/a052bad7-c875-44ac-ba7a-b68e8f91b838
Expensify/Expensify Issue URL:
Issue reported by: @ygshbht
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1691379420264869
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01780b67747fbfff83
Upwork Job ID:<PHONE_NUMBER>411463680
Last Price Increase: 2023-09-05

Proposal from @ygshbht
Proposal
Please re-state the problem that we are trying to solve in this issue.
The issue is that on the Expensify MacOS app, the ContextMenu options are not being translated to Spanish, even when the user has set the language preference to Spanish.
What is the root cause of that problem?
The current setup for the ContextMenu in the application does not handle translations. Specifically, the labels for various options in the ContextMenu (like cut, paste, and copy) aren't being translated based on the user's language preference.
https://github.com/Expensify/App/blob/21146e534492fc21811b5bac0718e0f1e8ffbfd4/desktop/main.js#L42-L57
What changes do you think we should make in order to solve the problem?
To address this issue, the ContextMenu function can be enhanced to support translations. Specifically, the labels for the default menu options can be provided directly using the labels option, like so:
labels: {
    cut: Localize.translate(preferredLocale, 'desktopApplicationMenu.cut'),
    paste: Localize.translate(preferredLocale, 'desktopApplicationMenu.paste'),
    copy: Localize.translate(preferredLocale, 'desktopApplicationMenu.copy'),
    /** Rest of the desired translations */
}

Additionally, for custom menu items that we're manually appending, the label can be set directly within the MenuItem:
append: (defaultActions, parameters, browserWindow) => [
        new MenuItem({
            // Only enable the menu item for Editable context which supports paste
            visible: parameters.isEditable && parameters.editFlags.canPaste,
            role: 'pasteAndMatchStyle',
            accelerator: DESKTOP_SHORTCUT_ACCELERATOR.PASTE_AND_MATCH_STYLE,
            label: Localize.translate(preferredLocale, 'desktopApplicationMenu.pasteAndMatchStyle'),
        }),
        new MenuItem({
            label: Localize.translate(preferredLocale, 'desktopApplicationMenu.pasteAsPlainText'),
            visible: parameters.isEditable && parameters.editFlags.canPaste && clipboard.readText().length > 0,
            accelerator: DESKTOP_SHORTCUT_ACCELERATOR.PASTE_AS_PLAIN_TEXT,
            click: () => pasteAsPlainText(browserWindow),
        }),
    ],

 Please note this is the bulk of solution, minor modifications such as listening to language change events and recreating the menu also need to be done to ensure the app functions like in the video below

Thanks!
Solution video:
https://github.com/Expensify/App/assets/65986357/990eba5f-370d-488e-b874-4b511879fa18

I had to brew on this one. I think it's a valid issue to fix, but it's small and non-urgent. For that reason I want to propose we dock the payments to be as follows for this one-off:
Bug reporting bonus: $50
Contributor fix: $250
C+ proposal/PR review: $250
I'm stating these now before I assign the External label so that folks know the prices before they invest too much time.

@michaelhaxhiu At the very least, I think the bug reporting price is fixed

@situchan when you are ready we have a proposal from @ygshbht ready for your review above 👀 .

bump @situchan

@ygshbht keep in mind there's no urgency bonus due to the merge freeze, but you can still get your PR ready to go
more context here

@michaelhaxhiu I'd say it should apply if the PR was ready to go and held by us

@Julesssss what do you mean? Are you saying the urgency bonus would still apply to this job/PR presently?

@michaelhaxhiu it's explained here that we do still have the urgency bonus during the freeze. I hope that helps 🙂

@ygshbht thanks for demo video. Proposal looks good.
🎀 👀 🎀 C+ reviewed

I agree that the proposal looks good. Thanks! Although we have a merge freeze you can create the PR and we'll get it reviewed. We can merge it once the freeze is over.

@michaelhaxhiu why did you assign him already? Isn't that my job? Anyways all good. @ygshbht please put up a review soon!

@neil-marcellini Sure!

@neil-marcellini mybad, DM'ed you and the assigned job is on the Engineer (you in this case).

@situchan The PR is ready for review! If you want to add more translations to the other lables that electron-context-menu provides (please see options argument and start reading from showLearnSpelling https://www.npmjs.com/package/electron-context-menu) please let me know

@situchan The PR is ready for review! If you want to add more translations to the other lables that electron-context-menu provides (please see options argument and start reading from showLearnSpelling https://www.npmjs.com/package/electron-context-menu) please let me know

I think they're out of scope. These are all app uses:

Ok

Bump for payment @michaelhaxhiu

Payment summary:

Bug reporter: $250 @ygshbht
Contributor: $500 @ygshbht
C+: @situchan $500 @situchan

@situchan do you get paid via NewDot or Upwork?

Does this not get 50% bonus?

I am basing my price point on Vit's slack comment here:
https://expensify.slack.com/archives/C01GTK53T8Q/p1693407966629239?thread_ts=1693347701.566439&cid=C01GTK53T8Q

Good question, i think to make this fair consider that if the PR is non-urgent ie non related to the above topics, you dont have to rush as it would most likely not be merged in the time for the urgency bonus. I believe if it’s urgent pr we make external the engineer/ bugzero person will clealy clarify that

This was assigned Sept 5, and merged sept 11 (delay likely due to feature freeze). But I get that there was some mixed guidance on this GH issue about bonuses and so this is totally fair to ask @ygshbht.
If you & situ or @neil-marcellini feel strongly, I am obviously open to offering the bonus. I just was trying to fairly apply the rule in slack across the board. As you can imagine this gets hairy for a BZ to discern when we start making one-off exceptions from the rule of thumb.

@situchan do you get paid via NewDot or Upwork?
upwork
Example of bonus applied: https://github.com/Expensify/App/issues/25889#issuecomment-1728889333

I approved the proposal on September 6th, the PR was raised the same day, C+ approved on the 8th, I approved and merged on the 12th. So it was good to merge within 3 days, but I was slow to review. I think we should give the bonuses @michaelhaxhiu.

Bug reporter: $250 @ygshbht
Contributor: $750 @ygshbht (50% bonus)
C+: @situchan $750 @situchan (50% bonus)

All are paid. @situchan can you please complete the BZ checklist so that we can close this?

No regression. This is for desktop only, Spanish only, context menu only which is super rare care for the user. I think this is enough reason why we don't wanna add regression test step.

Cool, I agree that this was just a candid oversight. Closing!


Add direct-from-marketing.yml issue template

Add direct-from-marketing template for marketing-approved pieces
Description & motivation

To-do before merge

Prerelease docs
If this change is related to functionality in a prerelease version of dbt (delete if not applicable):

[ ] I've added versioning components, as described in "Versioning Docs"
[ ] I've added a note to the prerelease version's Migration Guide

Checklist
If you added new pages (delete if not applicable):

[ ] The page has been added to website/sidebars.js
[ ] The new page has a unique filename

If you removed existing pages (delete if not applicable):

[ ] The page has been removed from website/sidebars.js
[ ] An entry has been added to _redirects

Hey @gwenwindflower can I get your review on this plz 🙏


improve search speed with ripgrep

Check for existing issues

[X] Completed

Describe the feature
VSCode uses ripgrep for code search and its speed is amazing.
Ref: https://github.com/microsoft/vscode-ripgrep
In a repo with the same keyword search, Zed is 10x slower than VSCode.
If applicable, add mockups / screenshots to help present your vision of the feature
No response

@osiewicz repo is https://github.com/microsoft/vscode, example keyword: service

Yup, this indeed lags quite noticeably.
It looks like VsCode has a limit on the # of search results. Therefore, in case of service query we report more results than VsCode, resulting in slower search.
That is not to say that this is not a valid report though - keyword query in VsCode repository does not hit VsCode's limit and we are still noticeably behind. We started investigating the slowness and when result streaming is turned off (== instead of seeing search results come in we update them once at the end), we see a noticeable improvement in keyword query - the time to process it goes down from ~6 s on my machine (M1 Max) to ~2s.


Fix SSL tests scripts with recent OpenSSL server with Diffie-Hellman

compat.sh and ssl-opt.sh don't work with a modern OpenSSL (e.g. the system one on Ubuntu 20.04) because they force the use of a small DH prime. We do this for compatibility with ancient versions of OpenSSL which we still use for legacy features. Only do it for those ancient versions, not for modern versions. This way you can run the SSL test scripts with the system version of OpenSSL on a modern system (barring other issues which I haven't checked).
Needs backports: to all supported branches. (3.0 may not need -dhparam at all, if it removes the use of OPENSSL_LEGACY altogether.)

barring other issues which I haven't checked)

ECDH (non-ephemeral) cipher suites fail with OpenSSL from Ubuntu 18.04 or 20.04. This PR doesn't make it worse, so the ECDH failure is out of scope.

Actually the pr-merge job on Jenkins passed but just failed to report, so this is all good.


⚠ Bofrost.fr has degraded performance

In 5f2571b, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1560 ms

Resolved: Bofrost.fr performance has improved in 8a95aed after 22 minutes.


[Bug]: Can't start a browser via 'launch' method without it erroring [Failed to launch the browser process! undefined]

Minimal, reproducible example
import puppeteer from 'puppeteer';
puppeteer.launch({ headless: true, args: ['--no-sandbox'] });

Error string
Error: Failed to launch the browser process! Running as root without --no-sandbox is not supported. See https://crbug.com/638180.
Bug behavior

[ ] Flaky
[ ] PDF

Background
No response
Expectation
I am expecting the code to run a browser like normal.
Reality
It just errors when initializing the launch method.
Puppeteer configuration file (if used)
No response
Puppeteer version
22.0.0
Node version
21.6.0
Package manager
npm
Package manager version
10.2.4
Operating system
Linux

I am unable to reproduce: perhaps try with the additional argument '--disable-setuid-sandbox'? In general, we do not recommend running without sandbox or as a root, please file a crbug.com/new for the headless component if --no-sandbox is not working correctly.

--disable-setuid-sandbox

yea, using this flag still comes out with the same error.


Remove unused commented-out code

As part of #2058 and #2154 we discovered few places where we have a commented-out code.
I.e. https://github.com/PowerShell/PowerShell/blob/master/src/Microsoft.WSMan.Management/CredSSP.cs#L59
We should revisit this code and possible remove.

Also, https://github.com/PowerShell/PowerShell/blob/7a89b33f8b0de97f592f65354ea435df7f163302/src/Microsoft.PowerShell.PackageManagement/Utility/HostApiExtensions.cs#L85 is not apparently used anywhere, including ISE (which is different codebase, but has internal visibility)

More code cleanup not needed for 6.0.0.

Can I work on this?

Hi @mi-hol
Yes, I definitely did. The issue is #7935
https://github.com/PowerShell/PowerShell/pull/7935.
Thanks
Vijay
On Mon., 31 Dec. 2018, 00:54 mi-hol<EMAIL_ADDRESS>wrote:

@vpondala https://github.com/vpondala did you create a pull request for
your changes?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/PowerShell/PowerShell/issues/2171#issuecomment-450581280,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABYhME_Ell4CG5JC8zFgecbGDeT5kb0Nks5u-RLdgaJpZM4J0IFe
.


Make artifacts available on Jitpack

Hi,
Thanks you for contributing this, it helps a lot.
Could you think that you can make the artifacts available from JitPack ?
Right now JitPack requires authentication to downlad these binaries meaning there should be some configuration to do on you side in the JitPack administration.
To be honest, GitHub packages is very painfull to use (when it works).
I understand that publishing to Maven Central is a long task, but it should be pretty straightforward to make it available from JitPack and is a very decent alternative to Central when it comes to tooling.

Hi, @ledoyen. I've finally reimplemented the Kotlin source scanning (Dokka replaced with Kotlin compiler) and deployed the new version (1.0.0) to Maven Central.

Many thanks !


Add option to disable DevTools for standalone VM

We should provide a --disable-devtools flag to allow for users to prevent DevTools from being served by DDS as some tools / IDEs currently serve their own instances.

I'm assuming that we're also not going to be starting devtools if we're passing --disable-dart-dev too? Or do we need to add a new flag to the flutter tool bootstrap and other parts of the tool?

Yes, that's the case. Also, DevTools will only start if --enable-vm-service or --observe is provided, so even if --disable-dart-dev isn't provided (aside: I'd like to drop that flag at some point since it won't impact startup time for basic runs like dart foo.dart) DevTools won't spawn unless the user explicitly enables the service.

i believe the concern was less to do with startup time and more to do with making sure programming errors in the tool are surfaced correctly. for example dart program.dart where the program is missing should not print out a warning about there not being a program.dart command

i believe the concern was less to do with startup time and more to do with making sure programming errors in the tool are surfaced correctly. for example dart program.dart where the program is missing should not print out a warning about there not being a program.dart command

Oh, well if that's the case, that shouldn't be an issue anymore:
$ dart bad.dart
Error when reading 'bad.dart': No such file or directory.

cool!


[FLINK-28861][table] Make UID generation behavior configurable and plan-only by default

What is the purpose of the change
This PR fixes two critical bugs in the stream operator UID generation of Table API.

Not deterministic UID:
Due to changes for FLIP-190, every operator generated by the planner gets a UID assigned in master. However, the UID is based on a static counter that might return different results depending on the environment. Thus, UIDs are not deterministic and make stateful restores impossible e.g. when going from 1.15.0 -> 1.15.1.

This PR restores the old pre-1.15 behavior for regular Table API. It only adds UIDs if the operator has been created from a compiled plan. A compiled plan makes the UIDs static and thus deterministic.

UID format limits migrations:
The current UID is not future-proof for migrations. The ExecNode version should not be in the UID, otherwise, operator migration won't be possible once plan migration is executed. See the FLIP-190 example that drops a version in the plan, once operator migration has been performed. Given that the plan feature is marked as @Experimental, this change should still be possible without providing backwards compatibility. The second commit will not be backported to 1.15 as it would make upgrades within a patch version impossible. Breaking the compatibility across minor versions should be fine as we don't provide state compatibility for SQL yet. A new config option allows to restore the UID format of 1.15 and allows for more flexibility around UIDs in general.

Brief change log

Introduce config options for when to add UID and how the format looks like
Propagate information through the planner whether compilation is the source of the translation

Verifying this change
This change added tests and can be verified as follows:

TransformationsTest

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): yes
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? JavaDocs

I found a nicer approach and will push an update soon.

Thanks for the review @zentol. Next to the necessity to pass around the isCompile flag, I found another issue due to the fact that transformations can be caches in a transient variable. In the end, I found a way nicer solution by just moving the flag a call higher in the planning phase. The code looks much cleaner now, I also added an additional test. Please have another look.


DCL is "jumping" - should be 0 ?! - JKBMS

Describe the bug
In the utils.py (file is attatched) I set the DCCM current settings to "0A" when cell voltage reached 2.90V.
The "SOC limitation" is disabled (False)
Now I have the problem that when this limit is reached and one cell is <= 2.90V, the discharge current starts jumping. And instead of switching the discharge off, the MP2-GX begins with charging the battery, then goes to "idle", then to "discharge" and so on.
While this happens there are many, many "Low battery: Alarm" messages from the VE.Bus System 275.
All the MP2-GX settings which could cause this are below this voltage levels and should not trigger any alarm.
The DVVC is switched ON.
Maybe I did somenthing wrong in the serialbattery config file or I missunderstand something.
Expected behavior
I would expect that the current goes down to 0A when the voltage limit of <= 2.9V is reached. I don´t expect that the battery is reloaded with AC-grid power in the middle of the night when there is no sun.
Screenshots
If applicable, add screenshots to help explain your problem.
VenusOS (please complete the following information):

Device type: MP2-GX
Firmware Version: 3.00 ~14, (v498)
Serialbattery version 0.14.2

Battery/BMS

BMS/Battery type: JKBMS / CALB LiFePo4 cells
Cells: 16
Interface: RS485 -->USB (MP2-GX)

Additional context
MP2 & ESS settings; VRM graphs: all settings.pdf
utils.py: utils.txt
Alarm log:
VRM DCL graph:

We'll have to improve the algorithm by adding a release voltage as well.
What is happening is that as soon as no more current is drawn the cell values starts to recover  and the BMS release
(the charger should only kick in when you are below the Min Soc value you have set in the ESS settings)

Hey Louis,
thank you very much for your reply and for your great work here.
Maybe I can set the "sustain voltage" in ESS a little bit higher than the smaltest "DCCM_CV" cell voltage x16. Maybe this prevents the cells from discharging before the 0A voltage is reached and activates the sustain voltage like set in ESS.
Can you tell me if all the voltages which I set in the ESS (with VE Configure) are still counting, when your driver is installed?
I wonder which of the ESS settings have a higher prioroty then the settings in the utils.py?!
Thanks again for your help an great work. I wish you a happy new year.
Best regards
Alex

The Limit values you set in DVCC always override everything.
If those are not set then the CVL, CCL, DCL that you see under the Parameters section of your battery in the GX is derived from the settings in utils.py and will apply.

Hey Louis,
I will try this and increase the "sustain voltage" - but this is not a setting in the DVCC, this is a "normal" setting in the VE configure. I attach a screenshot.
Best regards
Alex

Hey Louis,
unfortunately it didn't work to reduce the "Sustain-Voltage" value to 48V in the ESS assistant of the MP2.

In the DVCC section I cant set anything which could help me here. There is no setting for a minimal voltage, only in the ESS assistant.

The battery voltage is still dropping to the value which is set in the utils.py (2.9V) and then it begins to jump between charge-discharge-idle and so on.
I there something more I could do? Is this am JKBMS problem (or MP2-GX) only?
Thank you very much for your help!
Alex

Hey Louis,
I will try this and increase the "sustain voltage" - but this is not a setting in the DVCC, this is a "normal" setting in the VE configure. I attach a screenshot.
Best regards Alex

Just for information.
What helped me here was reducing the Cut-off voltages in the ESS asisstant (there are 4 voltages for different currents) - when the battery voltage dropped to this cut-off voltage, the discharging will stop and the "sustain voltage" will jump in until the battery voltage rises up over the "above-cut-off" value.
Important!!! - When the ESS asisstant is activated, all the 3 low-voltage voltage settings under the "Inverter" tab are completely ignored by the MP2.
Best regards
Alex

@MisterX1000 do I understand correctly, that your issue was solved? If yes, then I will add this to the FAQ.

@MisterX1000 do I understand correctly, that your issue was solved? If yes, then I will add this to the FAQ.

Hey mr-manuel,
for me the issue is solved. I think there are 2 ways to prevent this jumipng:

Current setting >0 (but I did not test this)
Reducing the ESS "Cut-off voltage" like I mentioned above

Thank you for your great support here!


Data exported is different

Hi, as described in issue number #48 , I have a problem with data export in .xlsx (.txt work fine).
I have a table with 3 columns: "First Data", "Second Data", "Third Data". When I try to export table, in the .xlsx file only the Third Column contains wrong data (all cell in this column contain "04/11/2017  23:00:00" instead of dd/mm/yyyy data).
I'm using these sources:

https://cdnjs.cloudflare.com/ajax/libs/xlsx/0.10.2/xlsx.core.min.js
https://cdnjs.cloudflare.com/ajax/libs/blob-polyfill/1.0.20150320/Blob.js"
https://cdnjs.cloudflare.com/ajax/libs/FileSaver.js/1.3.3/FileSaver.js"
https://cdnjs.cloudflare.com/ajax/libs/TableExport/4.0.3/js/tableexport.min.js
https://www.google-analytics.com/analytics.js

And before body end tag I have this:

I don't understand how to fix it. Any help would be appreciated.

You can add the tableexport-date class to any  or  to force a date type in Excel.
e.g.
4/11/1975

What are the original values inside the HTML before calling TableExport?

@rinoa96 – I have fixed this issue and published the patch in v4.0.5

You can add the tableexport-date class to any  or  to force a date type in Excel.
e.g.
4/11/1975

What are the original values inside the HTML before calling TableExport?

I had a similat issue of any table data containing 2 hyphens "-" would be automatically interpreted as a Date
e.g. Marka_WEB-generic-001
would be exported as a date 31-01-2001
Your solution worked in my case too except what i added was the string class :
 Marka_WEB-generic-001


ExtensionArray in Dask

In pandas, building an ExtensionArray is straight forward.  When writing them in Dask, do you have to implement anything to handle the chunking?
The blog posts makes it sound simple: https://blog.dask.org/2019/01/22/dask-extension-arrays
But I wanted to ensure that was the case.

Dask needs the extension array to be registered with make_array_nonempty and make_scalar. Please see https://docs.dask.org/en/latest/dataframe-extend.html#extension-arrays for more details and an example

@achapkowski does that documentation clarify how to register third-party extension arrays with dask.dataframe? Is it safe to close this issue?

I think we're good, let us know if things can be clarified though @achapkowski.

So if I read this right, I just need to create a throw away method that creates an empty Series essentially?

I don't think it's a Series. I think you register methods for creating a scalar and an array.


Use local OneAgent as in opentelemetry-metric-go

Looking at https://github.com/dynatrace-oss/opentelemetry-metric-go it says:
// No options are required. By default, the exporter will attempt
  // to export to the local OneAgent endpoint.
  // If no OneAgent is running, or if you wish to export directly
  // to your Dynatrace cluster, APIToken and URL are required.

Why is this exporter different?
Can the traces be picked up by the local OneAgent?

Hi there.
The package you linked is used for metrics only. This package can be used for OpenTelemetry-compatible traces. The traces will not be picked up by the local OneAgent. A Dynatrace endpoint must be configured, the traces will be exported automatically there.


Friend notes

As per #1877
ChatInputManager
Added max characters functionality
Friend Notes
Added a new plugin that allows adding/editing notes on friends in the friends list.

128 character long notes
Automagically update on name changes (Providing there is only one change)
Automagically delete notes for people who are removed

Amazing!

Automagically update on name changes (Providing there is only one change)

I'm so glad you already thought about this, great job!


Added a missing toRedis() -conversion for RedisHash.setIfNotExist

RedisHash!T did not compile when type T used a custom string serialization.

Thanks!


Create #rhiot IRC channel

Created and promoted via [1].
[1] http://henryk-konsek.blogspot.com/2015/10/socialize-join-rhiot-community.html


Logo should link to Home

@pkarw I've added this functionality in https://github.com/DivanteLtd/vue-storefront/pull/176


Use the full version for the Go toolchain

This works around https://github.com/golang/go/issues/65568.

FYI @schr3gl3j, this was discovered during ORT analysis.

@mreise added you as reviewer

Thx


solution: WebSockets client for API

Codecov Report

Merging #5 into master will decrease coverage by 5.01%.
The diff coverage is 76.94%.

@@             Coverage Diff              @@
##             master       #5      +/-   ##
============================================
- Coverage     93.16%   88.14%   -5.02%
- Complexity      339      421      +82
============================================
  Files            48       57       +9
  Lines           936     1316     +380
  Branches        108      179      +71
============================================
+ Hits            872     1160     +288
- Misses           40       93      +53
- Partials         24       63      +39

Impacted Files
Coverage Δ
Complexity Δ

...in/java/io/emeraldpay/pjc/client/RpcException.java
100.00% <ø> (ø)
4.00 <0.00> (?)

...main/java/io/emeraldpay/pjc/client/RpcRequest.java
52.63% <52.63%> (ø)
5.00 <5.00> (?)

...ain/java/io/emeraldpay/pjc/client/RpcResponse.java
54.54% <54.54%> (ø)
7.00 <7.00> (?)

...ava/io/emeraldpay/pjc/client/RpcResponseError.java
57.14% <57.14%> (ø)
4.00 <4.00> (?)

...a/io/emeraldpay/pjc/clientws/PolkadotWsClient.java
75.94% <75.94%> (ø)
17.00 <17.00> (?)

...ava/io/emeraldpay/pjc/clientws/DecodeResponse.java
76.63% <76.63%> (ø)
24.00 <24.00> (?)

...java/io/emeraldpay/pjc/clientws/MessageBuffer.java
83.33% <83.33%> (ø)
4.00 <4.00> (?)

.../java/io/emeraldpay/pjc/clientws/Subscription.java
85.71% <85.71%> (ø)
10.00 <10.00> (?)

...io/emeraldpay/pjc/clientrpc/PolkadotRpcClient.java
68.35% <100.00%> (-5.96%)
10.00 <0.00> (-5.00)

...in/java/io/emeraldpay/pjc/clientws/WsResponse.java
100.00% <100.00%> (ø)
9.00 <9.00> (?)

... and 10 more

Continue to review full report at Codecov.

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


Consistency checks for custom filters in Mistral vs StackStorm (ActionChains)

https://github.com/StackStorm/st2mistral/pull/30 copies a large number of the existing custom filters that currently exist in st2 and makes them available in Mistral as well. To limit interdependency, these were fully re-implemented in st2mistral, rather than rely on st2common in this repo.
However, this means that any changes will have to be implemented in both places, to ensure a consistent user experience (users that become familiar with a filter in one area will come to expect that it will be available in the other, which is reasonable, and the reason for https://github.com/StackStorm/st2mistral/pull/30). It would be nice if we could figure out a way - probably during st2 integration test time - to ensure that the custom filters available in each place are equivalent.
I added the "under discussion" label because the right answer may just be "it's not worth the effort", which I admit, it may not be (https://github.com/StackStorm/st2docs/pull/552 updates teh docs with a notice to keep the two in sync manually for the time being, and this may be good enough) - but it's good to have the discussion.

@Mierdin would it make sense to have a new package (maybe called st2jinja) where the Jinja filter code could live? I'm not sure if this is even technically possible due to dependencies.

Addressed at https://github.com/StackStorm/st2/pull/3565 and https://github.com/StackStorm/st2mistral/pull/30.


callback arguments

please specify in the README what are the arguments being returned to the callback function.
https://github.com/desandro/imagesloaded/blob/master/README.md

Callback arguments are documented under Events. Is there something in particular you are looking for?

it is not obvious at all that the same parameters as the events are also in the callback.
I would suggest to re-write it like so:
// selector string
imagesLoaded( '#container', function(instance) {...});

But, this is also extremely weird that the this scope is the exact same as the instance parameter, which makes the parameter redundant. Nowhere in the README does the scope of the callback function is mentioned btw.

I'm not a fan of the callback pattern as it isn't clear when the callback is supposed to be triggered to the developer. Use Events instead.


Infinite Scrolling?

Is it possible to implement infinite scrolling?  I've used a standard MUI Table component and added a Waypoint control to trigger a GET of more data.  I don't see a way to accomplish this with this component.
Is it possible to do in another way?

perhaps maybe a preRender function, or a way to inject an object?  Here's a stripped down version of my other code.  I'm new to this, so maybe I overlooked something simple.

Id
Other Column

{dataset.map(({id}, i, item) => (
<Fragment key={id, otherData}>
{/* Waypoint is used for infinite scrolling, when reached, fetch more data */}
{i === dataset.length - 10 && (
<Waypoint onEnter={() => {
const newData = this.getMoreRows(50);
this.setState({
dataset: [...dataset, ...newData]
});
}}
/>)
}

 {id}
 {otherdata}

Can you create a codepen https://codepen.io/ example with what you're trying to do? It should be possible to implement something like infinite scrolling in your table, but I'm not personally familiar with the Waypoint library.

Let me know if this works for you:
https://codesandbox.io/s/52xz76v5wn

It looks like you're nesting the Waypoint inside of the table body, but you don't have an easy way to do the same with mui-datatables. See if you can place it outside the table and still have the events and callbacks fire so you can update the state.
If not, you can try nesting in the footer with customFooter, the toolbar with customToolbar, or finally with a column via customBodyRender.

I don't think it is possible to place it outside of the table.  The point of waypoint is that is fires an event when the user scrolls to that element, so it would have to be in the table itself.
Do you have any other suggestions to make your component support infinite scrolling?  I'm not tied to waypoint, it is the infinite scrolling functionality that I would like to implement, regardless of how.

Did you try placing it in the footer, toolbar, or body via the options I suggested above?

My apologies.  I'm a bit new to react, so part of my question is "do you think I'm implementing this properly"?  I will try these, but thought maybe you had a better option in mind rather than using waypoint.  Thank you.

Sorry for the late reply, I got pulled off to something else for a bit...  Yes, I was able to use waypoint within customBodyRender and it works perfectly.  Thank you for the help!

Glad to hear it! Would you be able to update your codepen with a working example, in case this question comes up again?

Not the cleanest code, but here's the updated codepen.  Thanks again for your help!
https://52xz76v5wn.codesandbox.io/

Hi,
Any update on this one ? I am looking for an infinite scroll example. Can someone help me ?

This method no longer works in v3 - has anyone got a solution? Thank you!

I think this may be a codesandbox issue. I initially had trouble with it working up to version 2.6.3 of mui-datatables and failing in version 2.6.4. However, this sandbox - which uses the latest version of the table - appears to work fine in Firefox:
https://codesandbox.io/s/mui-table-infinite-scroll-s05b1?file=/src/messagemanager.js
It won't load in Chrome, but I have a feeling something else is going on. If you can try it on your own box I'd recommend that. If it fails let me know.

Thanks for the reply @patorjk
If you change responsive to "standard" then it stops working (both in codesandbox and locally). That was n.b. at the end of my last comment - sorry if that didn't make sense.

Looks like you just need to add the tableBodyHeight parameter (otherwise in standard mode the table will use the height of the parent):
https://codesandbox.io/s/mui-table-infinite-scroll-flumc?file=/src/messagemanager.js

Interesting... in the previous version, you didn't need to specify the height of the table. This means that you just needed to scroll the page. Now, for this to work, you need to have the table scroll independently of the page which is a bit annoying. Let me know if there is a solution to this. Thank you for your help!

The previous responsive modes will still work, you just get a deprecation notice in the console. The tableBodyHeight was separated out from the responsive parameter because previously there was no easy way to set the table height (ex: for "scrollMaxHeight" it was hard coded at 499px). You can see the styling used for the various modes here: https://github.com/gregnb/mui-datatables/blob/master/src/MUIDataTable.js#L1713
tableBodyHeight or tableBodyMaxHeight can be any CSS string (ex: '500px', '100%', 'auto', 'none', etc).

Gyes, I struggled with React Waypoint, but faced with problem onEnter fires always, so data uploading constantly without specific scroll point.
To avoid this problem you can just use  VisibilitySensor from "react-visibility-sensor";
`customBodyRender: (value, tableMeta, updateValue) => {
const {filteredData} = this.state;
const rowIndex = tableMeta.rowIndex;
                if (rowIndex === filteredData.length - 1) {
                    return (
                        <VisibilitySensor onChange={this.onChange}>
                            <div>{value}</div>
                        </VisibilitySensor>
                    );
                } else {
                    return <Fragment>{value}</Fragment>;
                }
            }`

`onChange = async (isVisible) => {
    const {state: {filteredData}} = this;

    if (isVisible) {
        const data = await this.getNewData(
            filteredData.length
        );
        this.setState({
            filteredData: [...filteredData, ...data],
        });
    }
}

`


can this be used with angular material instead of bootstrap?

I noticed that the collapse does not work without bootstrap. (Correct me if I am wrong). My question is can I still use this without bootstrap?

found out that the class .hidden of bootstrap is hiding the collapsed nodes. i simply added a .hidden class to my css and i have no need for bootstrap.
closing since i found the solution.


Fixes help desc. spo site set. Closes #1768

Fixes bug - Updated  help description for parameter sharingCapability. Closes 1768
This is PR which fixes the help description which was mentioned wrong for the parameter sharingCapability

Linked Issue

Closes #1768

Coverage remained the same at 100.0% when pulling 87f61c7917bf5639eeb84d390f91f9d952bd35da on arjunumenon:1768-fix-helpdesc-spositeset into 1cd587b6b285e144566152ad641ceed21081c5a3 on pnp:master.

@waldekmastykarz  - There has been an issue in my branch due to the project restructure and my branch was behaving differently after the rebase from the master.
It also had a the faulty commit which was mentioned in THIS REPLY which i deleted and forced-push to the branch. Somehow that had made the PR to be closed.
I have modified the changes required for this issue and have updated in the branch. Do you want to me create another PR for the same. The changes are made in the same branch which is referenced here. Can you advise on that please.
I am sorry since this has caused an issue across.
PS : I have also re-opened the PR

Perfect! Appreciate your help 👏

Merged manually. Thanks again! 👏


Not able to view counter component in "index.razor"

I followed the getting started tutorial and added the counter component <Couter /> tag inside the index.razor page but I am not able to see the Counter component in the browser.
Do I need to do any changes in the default web app template that is created by Blazor template extension in the VS2019?

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

ID: c07e57dc-526f-9479-4695-de7d8f06eb9a
Version Independent ID: 8226ed72-69b2-3c93-a931-218bd11cf9c8
Content: Get started with Blazor
Content Source: aspnetcore/blazor/get-started.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

Hello @jsowndarrajan ... Check your spelling ... it should be ...
<Counter />


Refactor flexmeasures.js (page-specific code to modules)

In ui.static.js.flexmeasures.js, we have lots of code about the general behaviour of the FlexMeasures UI.
We also have specific code, e.g. for the control page (lines 212 to 287 as of now) or the analytics page (at the bottom).
These should go into modules which are only loaded by these pages.
This reduces what each page loads, but even more than that, it increases readability of our code.

Also, some code is not needed at the moment, as we changed what pages are being used (i.e. analytics is out). This issue should do the cleanup there, or comment the loading of that code out.


Fixed the PDT-->PST time change tests. They should pass until atleast 2029

Pull Request Test Coverage Report for Build 5550

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.06%) to 42.411%

Totals

Change from base Build 5544:
0.06%

Covered Lines:
14214

Relevant Lines:
33515

💛  - Coveralls


[v6] Test OSV data to vers conversion

Once the CVE schema branch is merged, we need to test the vers conversion logic against the OSV data to reduce any precision loss in transformation.
https://github.com/AppThreat/vulnerability-db/tree/feature/store5
Test scenarios

[ ] If the type is maven, namespace should not be NULL
[ ] If the type is nuget or golang, namespace should be NULL
[ ] If the type is cargo, gem, pypi, npm or composer, namespace can be NULL
[ ] If the name starts with github.com, type must be go
[ ] If the namespace starts with github.com. type can be generic or swift but not go
[ ] All values must be lowercase, except for nuget type

Queries to test each condition:

SELECT COUNT(*) FROM main.cve_data WHERE type == 'maven' and namespace IS NULL
SELECT COUNT(*) FROM main.cve_data WHERE (type == 'nuget' OR type == 'golang') and namespace IS NOT NULL
SELECT COUNT(*) FROM main.cve_data WHERE (type == 'nuget' OR type == 'golang') AND namespace IS NOT NULL
SELECT COUNT(*) FROM main.cve_data WHERE name LIKE 'github.com%' AND type != 'go'
No test is needed as NULL is allowed as a value...?
 Unfortunately, that was not the case for any of them.

@cerrussell are you building the database with --cache --only-osv?

@prabhu I built with --cache. If I do --cache-os, it takes an hour to build.

@prabhu I will try osv only...

This is good. Will take a look at the issues tomorrow and get them to pass at least for osv.

@prabhu I will try osv only...

Please use the latest master

@cerrussell 4 is fine since it has a typo. Must be != 'golang'

6 is also fine since the source_data json has some camel cases and upper cases which cannot be changed.


[tool_crash] StateError: Bad state: Failed to start Dart Development Service

Command
flutter run

Steps to Reproduce

...
...
...

Logs
StateError: Bad state: Failed to start Dart Development Service
#0      DartDevelopmentService.startDartDevelopmentService (package:flutter_tools/src/base/dds.dart:114:9)
<asynchronous suspension>
#1      FlutterDevice.connect.<anonymous closure> (package:flutter_tools/src/resident_runner.dart:295:11)
<asynchronous suspension>

[✓] Flutter (Channel beta, 3.27.0-0.1.pre, on macOS 15.1 24B5046f darwin-arm64, locale en-US)
    • Flutter version 3.27.0-0.1.pre on channel beta at /Users/artemis/development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 2e2c358c9b (25 hours ago), 2024-10-22 11:02:13 -0400
    • Engine revision af0f0d559c
    • Dart version 3.6.0 (build 3.6.0-334.3.beta)
    • DevTools version 2.40.1

[✓] Android toolchain - develop for Android devices (Android SDK version 35.0.0)
    • Android SDK at /Users/artemis/Library/Android/sdk
    • Platform android-35, build-tools 35.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 16.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 16A242d
    • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2023.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)

[✓] VS Code (version 1.94.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.98.0

[✓] Connected device (4 available)
    • iPhone 15 (mobile)              • DDF815E8-05BE-4767-9E9B-7D705A8D0C86 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-17-2
      (simulator)
    • macOS (desktop)                 • macos                                • darwin-arm64   • macOS 15.1 24B5046f darwin-arm64
    • Mac Designed for iPad (desktop) • mac-designed-for-ipad                • darwin         • macOS 15.1 24B5046f darwin-arm64
    • Chrome (web)                    • chrome                               • web-javascript • Google Chrome 130.0.6723.60
    ! Error: Browsing on the local area network for Artemis. Ensure the device is unlocked and attached with a cable or associated with the
      same local area network as this Mac.
      The device must be opted into Developer Mode to connect wirelessly. (code -27)

[✓] Network resources
    • All expected network resources are available.

• No issues found!

Flutter Application Metadata
Type: app
Version: 38.0.0
Material: true
Android X: false
Module: false
Plugin: false
Android package: null
iOS bundle identifier: null
Creation channel: beta
Creation framework version: 2e2c358c9b14765c90343af9df11e12c5dfc3e6c
Plugins
applovin_max-4.0.1
audio_session-0.1.21
cloud_firestore-5.4.4
cloud_firestore_web-4.3.2
easy_audience_network-0.0.7
file_selector_linux-0.9.3
file_selector_macos-0.9.4+2
file_selector_windows-0.9.3+3
firebase_analytics-11.3.3
firebase_analytics_web-0.5.10+2
firebase_auth-5.3.1
firebase_auth_web-5.13.2
firebase_core-3.6.0
firebase_core_web-2.18.1
firebase_dynamic_links-6.0.8
firebase_storage-12.3.4
firebase_storage_web-3.10.3
flutter_native_splash-2.4.2
flutter_plugin_android_lifecycle-2.0.23
fluttertoast-8.2.8
geocoding-3.0.0
geocoding_android-3.3.1
geocoding_ios-3.0.1
geolocator-13.0.1
geolocator_android-4.6.1
geolocator_apple-2.3.7
geolocator_web-4.1.1
geolocator_windows-0.2.3
google_mobile_ads-5.2.0
image_cropper-8.0.2
image_cropper_for_web-6.0.2
image_picker-1.1.2
image_picker_android-0.8.12+16
image_picker_for_web-3.0.6
image_picker_ios-0.8.12+1
image_picker_linux-0.2.1+1
image_picker_macos-0.2.1+1
image_picker_windows-0.2.1+1
just_audio-0.9.41
just_audio_web-0.4.13
path_provider-2.1.4
path_provider_android-2.2.12
path_provider_foundation-2.4.0
path_provider_linux-2.2.1
path_provider_windows-2.3.0
permission_handler-11.3.1
permission_handler_android-12.0.13
permission_handler_apple-9.4.5
permission_handler_html-0.1.3+2
permission_handler_windows-0.2.1
razorpay_flutter-1.3.7
share_plus-10.1.0
shared_preferences-2.3.2
shared_preferences_android-2.3.3
shared_preferences_foundation-2.5.3
shared_preferences_linux-2.4.1
shared_preferences_web-2.4.2
shared_preferences_windows-2.4.1
sqflite-2.4.0
sqflite_android-2.4.0
sqflite_darwin-2.4.1-1
unity_ads_plugin-0.3.21
url_launcher-6.3.1
url_launcher_android-6.3.12
url_launcher_ios-6.3.1
url_launcher_linux-3.2.0
url_launcher_macos-3.2.1
url_launcher_web-2.3.3
url_launcher_windows-3.1.3
video_player-2.9.2
video_player_android-2.7.13
video_player_avfoundation-2.6.2
video_player_web-2.3.2
webview_flutter-4.9.0
webview_flutter_android-3.16.9
webview_flutter_wkwebview-3.16.0

Hi @aadityakushwaha,
How frequently do you see this error? Did it only happen one time? Do you see this issue on your Flutter project or even on a newly created project?
Can we get the full output of flutter run -v that reproduces the issue so that we can properly investigate this?
Thank you


Setting new Authentication Realm appears to affect existing Workflow Manager configuration.

Our Consuming Farm has web applications configured/registered to a Workflow Manager.  Upon setting a Realm value, as instructed here, existing custom workflows used in sites on this farm began failing.  For example, when attempting to view a workflow properties on a list, an error would be generated.  The ULS logs would report errors similar to the following:
"The caller does not have the necessary permissions required for this operation. Permissions granted: None. Required permissions: ReadScope."
To fix this issue, one needs to re-register any one site that was initially registered during the first configuration of Workflow Manager, using the Powershell cmdlet Register-SPWorkflowService.
If this is a normal occurrence when modifying Realm values, I would recommend a warning or reminder of this effect (and possibly others) included in this documentation.
Thank you.

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

ID: 96585065-aad9-9205-e602-a7426824031e
Version Independent ID: 7362519a-9a74-5b56-8f49-2bd5c5429fcd
Content: Configure server-to-server authentication between publishing and consuming farms
Content Source: SharePoint/SharePointServer/administration/configure-server-to-server-authentication-in-sharepoint.md
Service: unspecified
GitHub Login: @bentoncity
Microsoft Alias: stevhord

Tom, have researched this issue with product team, added an Important note with links to the PS cmdlet and additional links for more information. I've republished this topic.


CUPS not sending credentials to IPP printer

Describe the bug
I am trying to set up cups so that I can print to my school's print queue. In order to do this, I need to authenticate with the IPP print queue for each job. To do this, I have both added my credentials to the URL (e.g. http://user:pass@server:port/path/to/printer, and also set AuthInfoRequired so that CUPS requests credentials from the app sending the print job.
Whenever I send a job, the application correctly asks for credentials and I can see in the CUPS dashboard that the job is being processed. However, after a few seconds the job halts with the message "Held for authentication". It seems that CUPS isn't sending the credentials at all, which I verified using wireshark (capture attached).
To Reproduce
Steps to reproduce the behavior:

Add IPP printer that requires authentication
Attempt to send a job to printer
Job gets "held for authentication"

Expected behavior
I send a job to the printer, and it is successfully added to the school's print queue system.
System Information:

OS and its version: Arch Linux
Application: Any
CUPS version: 2.4.2

Additional context
Wireshark capture: http://0x0.st/H8RR.pcapng (my device is <IP_ADDRESS> and the printer is <IP_ADDRESS>)

OK, so using the "http" scheme isn't supported for this - that was just for IPP/1.0 - and putting the username and password in the device URI is not recommended (for obvious reasons).
We'll need a debug error_log file (run cupsctl --debug-logging and then try printing) to further debug this issue.

Removed the credentials from the URI, and switched to ipp while manually specifying port 80. Same thing happens: client submits job, credentials are requested from client, credentials are sent, job still ends up held for authentication.
[
cups_error.log
](url)

@rhysperry111 thank you for the logs!
The Microsoft IIS/10.0 server you are connecting to expects Kerberos authentication:
D [04/Jun/2023:19:21:47 +0100] [Job 91] Getting supported attributes...
D [04/Jun/2023:19:21:48 +0100] [Job 91] password_cb(prompt=\"Password for rhysperry on <IP_ADDRESS>? \", http=0x556406856710, method=\"POST\", resource=\"/printers/PrintQueue/.printer\", password_tries=0x7ffe3f84a164(0)), password=0x7ffe3f852fcb
D [04/Jun/2023:19:21:48 +0100] [Job 91] Get-Printer-Attributes: cups-authentication-canceled (Unauthorized)
D [04/Jun/2023:19:21:48 +0100] [Job 91] Get-Printer-Attributes returned cups-authentication-canceled.
D [04/Jun/2023:19:21:48 +0100] [Job 91] ATTR: auth-info-required=negotiate

the last message goes from this code:
      else if (ipp_status == IPP_STATUS_ERROR_FORBIDDEN ||
               ipp_status == IPP_STATUS_ERROR_CUPS_AUTHENTICATION_CANCELED)
      {
        const char *www_auth = httpGetField(http, HTTP_FIELD_WWW_AUTHENTICATE);
                                        /* WWW-Authenticate field value */

        if (!strncmp(www_auth, "Negotiate", 9))
          auth_info_required = "negotiate";
        else if (www_auth[0])
          auth_info_required = "username,password";

        fprintf(stderr, "ATTR: auth-info-required=%s\n", auth_info_required);

where you can see it gets the data from http structure, which represents HTTP connection with the destination and currently it contains HTTP header fields from the response - HTTP header WWW Authenticate from the response contains negotiate, which means Kerberos for CUPS.
You will have to set your local queue to use Kerberos and get the ticket, or reconfigure Microsoft server (if possible - but I haven't seen such configuration yet) to accept username,password auth.


add command line flags for specifying images for the machine-approver, cluster-autoscaler, and capi manager instance

This PR add command line flags for specifying the default image utilized for the machine-approver, cluster-autoscaler, and capi manager instance. The current architecture that Red Hat is using is one version of these components across all downstream clusters the operator manages. In addition: the hypershift-operator controls the versions of the CRDs these resources so it makes sense at least for now that these images correspond with the hypershift-operator versus getting versioned on a per cluster basis. Solves: https://github.com/openshift/hypershift/issues/1078

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes # https://github.com/openshift/hypershift/issues/1078
Checklist

[x] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/unhold

/hold
to revisit getting this from the payload instead.

Machine approver
49dd2dc8f511cdee7a846a0a6c49ca0caefeb902 (origin/release-4.10) Merge pull request #152 from JoelSpeed/upi-large-scale-renewal
7baac2367d3d38692a62b749486496b62f955a97 Account for large scale simultaneous renewal on UPI clusters
2153a9f834089c5617315955e71482271c0225e5 Merge pull request #149 from damdo/k8s-1.23
3d6d8d51bde08da34454d51cbc2912f4fa4f9b91 Update test version of kubebuilder-tools
f16e62a57d831feb2311411c4d9153b5a7cb3f28 Update ClusterOperator manifest
a5635ca18f6585ab90c6a595510f0cf65a0580d6 Update dependencies to K8s 1.23
dbddaad217043c6e491b3d6784163c5390a21dfd Merge pull request #140 from elmiko/golang1.17
a94ee294b3e84dfdcbbd136b4e216424f043480e update for golang 1.17
a7e41a26f0a716b806ac68bf0987816e2c776077 add BUILD_IMAGE to makefile
dcb4bdd190af23038896014ea151a2c6660f8a40 Merge pull request #145 from damdo/owners-tidying
76576be8fea9102273b56fec8ed0196ec645c2cc OWNERS: remove old peeps, add new
0b073bbc6bcd883f379986bf60d152c0fe8f066e Merge pull request #144 from JoelSpeed/replace-cert-gen
4c2f44a0a67bd88bf2817995cf96d89d404554f5 Replace certificates with generated certificactes
1e6ea3e8852385ceaec368978416f6795c3eee30 Merge pull request #138 from elmiko/add-desc-to-alert
10c0b070c6fb8ea2f969f45750fa3c57b9a74ef4 add summary and description to alerts
0a64d91c6d5cc3bc7dbf9940c7b57f12ac3a8a75 Merge pull request #137 from JoelSpeed/egress-ip-checks
ebf0dfc8b133ecbdcfb6c43a89f6e813b0741fe6 Merge pull request #135 from JoelSpeed/ensure-metrics
0128ba685c85abf6fab0a56555e2ac9670529912 Allow fallback to serving cert renewal accounting for egress IPs on SDN
7b7289a9e6419b9a5ce60cba22b60b9d8ddf3ed5 Merge pull request #133 from hanqiuzh/add-leader-election
a135d505be74a625d8519866c278ac783ae31d36 add leader election options
eb8ab3ac8ec489872e01854628fdcd42861141ee Ensure pending CSR count is valid post approval
4595f8ad6c4ef2685d2fea62069c1f89a8c49273 Merge pull request #132 from hanqiuzh/add-clusteroperator-arg
88d160e36acc1501bcd223781ad710be4a6dba41 Merge pull request #134 from s-urbaniak/pod-security
bfc8167ea01b7c27cf7da946d5b1776600d81f73 add commandline flag for status controller
d3fb5d1b9db39d67b0f8fbec5e90952f356f05c6 podsecurity: enforce privileged for openshift-cluster-machine-approver namespace
e3c5b539effa85db7081f284d909cc85bdec9f8d Merge pull request #131 from hanqiuzh/add-management-namespace
91ce1da6822c65c9fddfcd12892eaa170a6714ca add machine-namespace
df14739891e58f6964ddb3fea3427f2ec853c734 Merge pull request #129 from JoelSpeed/ignore-approved-csr
aa1a99d02626c0d10ff0c1701935de0095aed955 Prevent error loop when a CSR is queued and then approved externally
965c2ad2098ab49fd84eade67c1f46b1dbd0d568 Merge pull request #114 from alexander-demichev/capimachines
f93db04a19be4c236936f2d5e54ce040a2dde980 Fix clients setup
1cd5eb0378c98d067d45e72058678b606c212581 Revendor dependencies
0ab08aeb65d9e1744b32107f485863a9dc86c745 Add separate clients for machine and nodes
8dee0cc86b4cd8e216f57c75b77341443ec11170 Use unstructured machines
678445115118fe2135b75c3abddb48b58efd9727 Add helper library for handling unstructured machines

Autoscaler
➜  autoscaler git:(master) git log --pretty=oneline openshift/release-4.9..openshift/release-4.10 | grep UPSTREAM
3f7eafa865986f1684a64641034f23b5f39c3905 UPSTREAM: <drop>: revert defaultCAPIGroup constant
63cc4f9ff8ed4f2fc45824c4b9f6eba719eaab88 UPSTREAM: <drop>: Ensure ClusterAPI DeleteNodes accounts for out of band changes scale
148847f9abe02fd3cf49bc2f8dacf29cfb292467 UPSTREAM: <carry>: Updating atomic-openshift-cluster-autoscaler images to be consistent with ART Reconciling with https://github.com/openshift/ocp-build-data/tree/44bc7dde7a1abf6a25bf16d7d05e98599a9783e1/images/atomic-openshift-cluster-autoscaler.yml
4e6bf4adadf34462d2b2b95cb4520a46246dece3 UPSTREAM: <carry>: Fallback to CAPI annotations
39495109e5331bd86b7e6ceb17d75bf794456477 UPSTREAM: <carry>: add clusterapi build tags to dockerfiles
4d89639da2048c39e2903c3bd0688442165ace97 UPSTREAM: <carry>: Updating vertical-pod-autoscaler images to be consistent with ART Reconciling with https://github.com/openshift/ocp-build-data/tree/3dea8426481b269476342fc0a631273f32ce691e/images/vertical-pod-autoscaler.yml
a6af8fcf2220b9fb1c9cfe23481e44d188bba6da UPSTREAM: <carry>: Implement scale from zero
1ea5d7cf44369fa129da8b00ae788ee22cc96bd5 UPSTREAM: <carry>: Rename FailureMessage to ErrorMessage
25d528d845c5bc36ace35ef2b14c08d254b5aa7a UPSTREAM: <carry>: Revert "Adding config for event filtering"
e6402ed40218ab53cda76dcf2208bb02fe4f1c5b UPSTREAM: <carry>: Set default annotations, and cluster name label to use machine.openshift.io
3233a6997a5c1d78802c8c25a7a14cc0a3da205a UPSTREAM: <carry>: configure repository for OpenShift releases
1e02cef854fbeaa48d8463f89ede5382485bc214 UPSTREAM: <carry>: Updating vertical-pod-autoscaler images to be consistent with ART Reconciling with https://github.com/openshift/ocp-build-data/tree/5b89f5b601508a0bcc0399fd3f34b7aa2e86e90e/images/vertical-pod-autoscaler.yml
6aff3fb291b2bdfa739510b9084a998ffd399cd9 UPSTREAM: <carry>: Updating atomic-openshift-cluster-autoscaler images to be consistent with ART Reconciling with https://github.com/openshift/ocp-build-data/tree/d78e07522b3c9a920b0e2f5f44dd574d1e7f5091/images/atomic-openshift-cluster-autoscaler.yml
958f48ad9196d67f761a755d7dbf3f982946018b UPSTREAM: <carry>: Fallback to CAPI annotations
b3c489bad968de1eea517218d09b35161935c002 UPSTREAM: <carry>: add clusterapi build tags to dockerfiles
3d1019c9c90def83247782551da3891567ab33a7 UPSTREAM: <carry>: Updating vertical-pod-autoscaler images to be consistent with ART Reconciling with https://github.com/openshift/ocp-build-data/tree/3dea8426481b269476342fc0a631273f32ce691e/images/vertical-pod-autoscaler.yml
2631b46d41f8869675890ceb2e545f1833f388ad UPSTREAM: <carry>: Updating atomic-openshift-cluster-autoscaler images to be consistent with ART Reconciling with https://github.com/openshift/ocp-build-data/tree/3dea8426481b269476342fc0a631273f32ce691e/images/atomic-openshift-cluster-autoscaler.yml
7c5d27c127019497ab5c6c0298b236e593d381aa UPSTREAM: <carry>: Updating atomic-openshift-cluster-autoscaler images to be consistent with ART Reconciling with https://github.com/openshift/ocp-build-data/tree/44bc7dde7a1abf6a25bf16d7d05e98599a9783e1/images/atomic-openshift-cluster-autoscaler.yml
➜  autoscaler git:(master)

Based on the above I think we only need https://github.com/openshift/kubernetes-autoscaler/pull/226


Add a "Simple Participant Existence" check

Without all the SMP fuzz.

Radio button with "SMK / SML"
Input field for PI scheme, with predefined value iso6523-actorid-upis
Input field for PI value

Result says "yes/no" and provides links to "more details" showing to existing Participant Info page as well as a "Search Peppol Directory" link

Part of the next deployment


media-downloader: requires mutagen module to embed thumbnail

Describe the bug
media-downloader cannot embed thumbnail.
Steps To Reproduce
Steps to reproduce the behavior:

install media-downloader
enter URL: https://yewtu.be/watch?v=5YWRREgi8dA for example
download option: -f bestaudio -x --embed-thumbnail

Expected behavior
Video/audio can be embedded with thumbnail.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
ERROR: Postprocessing: module mutagen was not found. Please install using `python -m pip install mutagen`
[media-downloader] Download Failed(ErrorCode=1)

Notify maintainers
@zendo
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.

system: "x86_64-linux"
host os: Linux 6.6.4, NixOS, 24.05 (Uakari), 24.05pre556224.2c7f3c0fb7c0
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.18.1
channels(hervyqa): ""
channels(root): "nixos"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

Add a :+1: reaction to issues you find important.

https://github.com/mhogomchungu/media-downloader/issues/334#issuecomment-1783834762
bad for nixos.

Could you elaborate @zendo? What is the fix?


Increase build number

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Wait proxsuite-nlp release

@conda-forge-admin, please restart ci

@conda-forge-admin, please restart ci


feat(betterer-coverage-percentage): initial commit

initial commit for betterer-coverage-percentage

Reopened here: https://github.com/spaceribs/spaceribs/pull/267


Suggestion: option to use as link shortner

We can add an option which will redirect to the url pasted in pastebin. I have no idea how to implement it (sorry for that, i have no prior knowledge). But i found some of projects of url shortner based on cloudflare workers that may help:
https://github.com/dewanshchaudhari/url-shortner
https://github.com/hashirahmad/cloudflareWorkersUrlShortner
Thanks for this pastebin.

This feature has already been implemented. For example if you have a paste that can be accessed by shz.al/abcd, then shz.al/u/abcd will redirect you the the URL in the paste.


LINQ support status?

Hey @cskardon,
would you mind sharing the current status and/or perspective plans on LINQ support in Neo4jClient?

Hello! In what respect? As in LinqToNeo4j?

Exactly!

OK, well - I have no plans to write a LinqToNeo/Cypher driver, I'd happily look at pulls, but in terms of time to do it - well - there just is none!!!

Good to know. Thanks!


Add missing coerce-to-number-list code path in iOS

Change-Id: If7713cbb2432b5e5084e351d55e0564cf1953a0a

@daki7711 This fixes the issue that prevented the RL demo app from working on iOS.


Updated pyi file for point in time API

Description
Updated pyi file for point in time API
Issues Resolved
Closes #375
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #378 (8239d56) into main (83be2cd) will increase coverage by 1.28%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #378      +/-   ##
==========================================
+ Coverage   70.72%   72.01%   +1.28%
==========================================
  Files          77       77
  Lines        7157     7190      +33
==========================================
+ Hits         5062     5178     +116
+ Misses       2095     2012      -83

see 29 files with indirect coverage changes

Can we automate this or find a way not to get out of sync/catch that we're out of sync?

@dblock, I will think about it. And also client generation might help us identifying differences or mistakes.

@dblock any plans to cut and publish a tagged release that includes this change? v2.2.0 is the latest one and doesn’t include, unfortunately.

We release fairly often. Add your +1 to https://github.com/opensearch-project/opensearch-py/issues/432.


3.3.7 move to gitlab

try to update upstream url
ref https://github.com/mesonbuild/eigen/issues/7

I also tripped over this issue when trying to add rxdock to the meson corpus test.
Please apply something like this, since http://bitbucket.org/eigen/eigen/get/3.3.7.tar.bz2 is now 404, so the current wrap is no use.

thanks, basic tests work 👍


Merge #3191, #3196 and #3209

Let's make sure these merge cleanly

Merge #3191, #3196, and #3209
Closes #3191
Closes #3196
Closes #3214
Closes #3209

Good job Dan, we are getting close now 🎉
>       split_dict = pipenv.utils.split_file(pipfile_dict)
E       AttributeError: 'module' object has no attribute 'split_file'

Literally no idea why the new thing is failing....

Hmm maybe I left those in version control on the other packages. Otherwise everything passed so I’m cutting a release in a few mins

That's because the two files are missing, which are inconsistent with the versions specified in vendors.txt


remove download video view MA-1054

remove the download view

@ehmadzubair please review, the downloads view should be gone now.

:+1:


I search only first country and show me all translations also

This is a bug, yeah? 0.o
I recently used this repository to translate my records of location's addresses and I liked it very much, but I have a problem with get only one translation and all translations of location put the output.
See the code below:

     * @param $placeId
     * @param $language
     * @return \Illuminate\Http\JsonResponse|string
     */
    public function show($placeId, $language = null)
    {
        /** @var Location $location */
        $location = Location::where(['place_code' => $placeId])->first();

        if (!empty($language)) {
            if ($location->hasTranslation($language)) {
                $translation = $location->getTranslation($language);
                $address = $translation->getAttribute('address');

                $location->address_components = json_decode($address);
            }
        }

        return $location;
    }

The code is based of Feature list --> Available methods.
https://github.com/dimsav/laravel-translatable#available-methods
But here, I follow my especific route /api/restricted/locations/<placeId>/<language>
And the screen sees more details:

The address object also appears in response, but this is wrong object because is in English, but the translation searched is in portuguese.
The Location model:

class Location extends Model
{
    use Translatable;

    protected $table = "locations";

    public $translatedAttributes = ['address'];
    public $translationModel = 'App\Models\LocationTranslation';

    /**
     * The attributes that are mass assignable.
     *
     * @var array
     */
    protected $fillable = [
        'place_code', 'address_components', 'geometry', 'types'
    ];

...

Something else to configurate and I'm not know yet?

My workaround is Laravel eager-loading but this is not solved because all the location are loading before relationship is loaded with with().
I think in a solution but until not be able this.
I'm thinking of when I'm going to list all the locations and all of your 20 translations are loaded together. Multiply that by 10k users.

Do you have this in your config?
'use_property_fallback' => true,'

Yes it is but I'd not like to override the original text.
I'd like to bring from the database the original information + a specific translation.

Do you set the language from the url anywhere as the language to use for translations!? I don't see it and by default the App locale is en and this is used. So try to do an App::setLocale($language);

Thanks @Gummibeer and @dimsav for your help!
Now I solved using setLocale before returned the location to the API.
This way I can perform a much simpler search than the one I was performing and set the language of the application shortly before returning the data through the API.


Changes 'yes' under manage persons tab to red

https://uhawaii.atlassian.net/jira/software/c/projects/GROUPINGS/boards/18?modal=detail&selectedIssue=GROUPINGS-1520&assignee=6352ef57e14026a7397ea425
[1520]
List of squashed commits

Changes 'yes' under manage persons tab to red

Test Checklist

[ X ] Exhibits Clear Code Structure:
[ X ] Project Unit Tests Passed:
[ X ] Project Jasmine Tests Passed:
[ X ] Executes Expected Functionality:
[ X ] Tested For Incorrect/Unexpected Inputs:

There is a bootstrap class "text-danger" that we can use instead of the style attribute. Could you also correct the formatting of this PR? You can look at another person's PRs and press the edit button to see their text formatting. The ticket number and URL should be combined as a link with the ticket number as the text. The X's in the Test Checklist should be checkboxes. Thanks!

Closing duplicate PR.


zooming camera when change videosize

I create
var session = VCSimpleSession(videoSize: CGSize(width: 426, height: 240), frameRate: 30, bitrate: 500000, useInterfaceOrientation: false)
but when i change
session.videoSize = CGSize(width: 1280, height: 720)
i get preview zooming

I am also facing same issue did you find any solution. Please help me.Thanks in advance


imagemin dependency

Error encountered
When running hexo generate
ERROR Plugin load failed: hexo-optimize
Error: Cannot find module 'imagemin'

_config.yml was the same as README indicated.
# https://github.com/next-theme/hexo-optimize#readme
filter_optimize:
  enable: true
  # remove the surrounding comments in each of the bundled files
  remove_comments: false
  css:
    # minify all css files
    minify: true
    # bundle loaded css files into one
    bundle: true
    # use a script block to load css elements dynamically
    delivery: true
    # make specific css content inline into the html page
    #   - only support the full path
    #   - default is ['css/main.css']
    inlines:
    excludes:
  js:
    # minify all js files
    minify: true
    # bundle loaded js files into one
    bundle: true
    excludes:
  # set the priority of this plugin,
  # lower means it will be executed first, default of Hexo is 10
  priority: 12

Solution
I installed these dependencies manually and it succeeded.
npm install imagemin imagemin-mozjpeg imagemin-pngquant imagemin-gifsicle imagemin-jpegtran imagemin-optipng imagemin-svgo

And I need to install autoconf system-wide since imagemin-gifsicle failed to fetch pre-built binary and I had to compile it.
Should we add imagemin and plugins to dependencies?
My package.json : package.zip

Thanks for your feedback. A new version has been released to fix this issue: https://www.npmjs.com/package/hexo-optimize/v/2.5.1


Externalize struct names

It would be nice to have a configuration file for struct names used by ZAPD to write out assets in C, like SkeletonHeader, CamData, ...
This would make documentation easier by not having to first wait for ZAPD to be updated and merged into decomp repos

So, the external names would override the internal ones in case they exist? Or it would be mandatory to extern every struct name?

I was thinking "externalize" as in "no hardcoded struct name at all in source code"

So, each time a new resource would need to be added to ZAPD both repos would need to add the corresponding entry to their config file without fallback?

I guess? I don't see how that would be an issue since ZAPD updates are merged in each repo anyway, just edit whatever needs editing in the config at the same time

Because Jenkins would be mad in this repo if the config file is missing in the other repo.


MySQL 5.6 is not supported as an workflow archive

Checklist:

[x] I've included the version.

[x] I've included reproduction steps.

[ ] I've included the workflow YAML.

Not necessary. Any workflow will trigger this.

[x] I've included the logs.

What happened:
I'm trying to activate workflow archives. I've created a sql container, added a Secret to store credentials and fired it up. Now the workflow-controller pod crashes with this error (see below).
What you expected to happen:
The workflow should be archived normally.
How to reproduce it (as minimally and precisely as possible):
Use the mysql:5.6 docker container, out of the box, setting the env vars: MYSQL_USER, MYSQL_PASSWORD, MYSQL_DATABASE and MYSQL_ROOT_PASSWORD. Create a service that points to the pod, and finally, reference this in the workflow-controller-configmap, using mysql as an archive store.
Anything else we need to know?:
PostgreSQL works fine. Probably MySQL > 5.x works fine as well. I didn't test it though.
The MySQL syntax is wrong. It tries to:
alter table argo_archived_workflows rename column id to uid
but it should be:
alter table argo_archived_workflows change column id uid varchar(128)
(I'm not sure about the number within varchar() but it needs to be specified)
Environment:

Argo version:

$ argo version
argo: v2.7.0-rc2
  BuildDate: 2020-03-23T17:54:38Z
  GitCommit: 240d7ad9298c60a69d4ce056e3d83ef9283a83ec
  GitTreeState: clean
  GitTag: v2.7.0-rc2
  GoVersion: go1.13.4
  Compiler: gc
  Platform: linux/amd64

Kubernetes version :

$ kubectl version -o yaml
clientVersion:
  buildDate: "2019-06-19T16:40:16Z"
  compiler: gc
  gitCommit: e8462b5b5dc2584fdcd18e6bcfe9f1e4d970a529
  gitTreeState: clean
  gitVersion: v1.15.0
  goVersion: go1.12.5
  major: "1"
  minor: "15"
  platform: linux/amd64
serverVersion:
  buildDate: "2019-04-08T17:02:58Z"
  compiler: gc
  gitCommit: b7394102d6ef778017f2ca4046abbaa23b88c290
  gitTreeState: clean
  gitVersion: v1.14.1
  goVersion: go1.12.1
  major: "1"
  minor: "14"
  platform: linux/amd64

Logs
time="2020-04-06T07:24:03Z" level=info msg="Migrating database schema" clusterName=default dbType=mysql
time="2020-04-06T07:24:03Z" level=info msg="applying database change" change="alter table argo_archived_workflows rename column id to uid" changeSchemaVersion=10
time="2020-04-06T07:24:03Z" level=fatal msg="Failed to update config: Error 1064: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'column id to uid' at line 1"

Message from the maintainers:
If you are impacted by this bug please add a 👍 reaction to this issue! We often sort issues this way to know what to prioritize.

This could probably be best fixed by someone with MySQL knowledge.
Steps:

Edit the manifests/quick-start/mysql/mysql-deployment.yaml and change the image version.
Run make start pf to update the test manifests.
Examine the logs of the workflow controller (e.g. kubectl logs -l app=workflow-controller)
Edit persist/sqldb/migrate.go and fix the errors you find.

@lkeijser - Unfortunately the minimal Mysql version we support is 5.7.8, because column type json was not supported until that version.


Add support for Night Owl Light

Hello
First, a wonderful implementation of the original theme.
Is there a plan to also implement Night Owl Light?
As i frequently switch between Light/Dark Mode.

i'd love to have the light version 🙌

Quick update:
Now that the theme can be configured using Lua, I have the base implementation I need to work on something like this:
require('night-owl').setup({
  -- ...
  light_mode = true,
})

I'm currently trying to organize the color palette of the theme based on the original VS Code implementation by Sarah, to which I've added a few tweaks for what I consider better consistency. Once I have that, I'm going to work on getting all the right mappings form dark to light mode, before refactoring the implementation to easily swap the hex values used.
Please leave a thumbs up if you're interested in having this option available! 👍🏼

the colorscheme is wonderful, looking forward to lightmode. maybe instead of a setup option it would be nice to use the background option of neovim?


Error parsing boolean value. Path '', line 1, position 1.

I'm attempting to deploy a Bicep file that I don't have issues deploying locally
I keep hitting Error parsing boolean value. Path '', line 1, position 1.
    - name: Bicep Hyperscale
      uses<EMAIL_ADDRESS>      with:
        azPSVersion: '3.5.0'
        inlineScript: |
          Set-Location ${{ github.workspace }}/script/
          New-AzResourceGroupDeployment -ResourceGroupName "test-rg" -TemplateFile my.bicep -Mode Incremental

Seeing parameter parsing issues with azPSVersion: 'latest' also for Azure Deployments with Powershell.

Since the path parameter is not accepting, you might need to try with putting " "(double-quotes) for my.bicep so that your command looks like
 New-AzResourceGroupDeployment -ResourceGroupName "test-rg" -TemplateFile "my.bicep" -Mode Incremental also make sure you give a valid working path of TemplateFile.
You should add a step for azure/login@v1 before you execute az-pwsh commands as a prerequisite. Make sure you set enable-AzPSSession: true in azure/login

Have a look at the detailed example . Let us know if it works.

still hitting the issue with quotes added and I have enable-AzPSSession: true
Maybe related to https://github.com/Azure/bicep/discussions/3140

@dkirrane Good to hear. It is also recommended to use azPSVersion: "latest" or explicitly provide the latest version in our workflow unless you have any version preference in specific. Usually any bugs in the previous versions are covered in later versions. Closing this issue as this  page deals with only issues regarding azure/powershell action.


Badge added to readme.md

Badge added for Code Climate

placement of badge is excellent, team unanimously agreed that this pull request does not necessitate a template


Auto logout after some time and/or session expire?

Instead of calling a.init() to login, i'm doing the following:
a.playerInfo.provider = provider;
a.playerInfo.accessToken = accessToken;
a.playerInfo.apiEndpoint = apiEndpoint;

where provider, accessToken, and apiEndpoint are variables saved in a filesystem. And then I use a.setLocation to set location.
This helps restarting node without having to keep logging in (might trigger a soft ban). This works fine for a few hours, and then I start to get 'No result' error from Pokemon GO server, regardless of the api call. I'm guessing the session expired or something? Any tips regarding this?

I solved this by checking for "No result" inside my heartbeat function and logging back in.
if(err) {
    console.log(err);
    if(err === "No result") {
        clearInterval(start);
        init();
    }
    return;
}


Fix #2442

Fixes #2442
This adds a diff customizer that ignores changes to parameters that only change the apply_method and not the value for the aws_db_parameter_group resource. To make this work, the change also needs to modify the set element hashing function to identify parameters that differ only by apply_method as identical.
As a side-effect of the set hashing change, upgrading stacks to the newer version of the provider with this change will show a reordering update diff on the ParameterGroup resource.

Wanted to get feedback on this before adding some tests, but this seems to pass a local variation of tests for me with this change, this is one way to get this suggestion implemented:
applyMethod should basically be ignored for purposes of diffs

LGTM, should we call out the change somewhere in the RDS docs? It might be slightly unexpected for a user that they can not modify JUST the apply_method - that'd now yield no diff, right?

That is true we need to edit that note out of the docs.

I'm also a bit surprised that marking the field as Computed now decided that it will always be populated in the TypeScript type projection. So this needs to re-generate schema, and makes an optional field required. I think it's benign in this case but it is a strange bridge behavior.

LGTM. I would second @VenelinMartinov in mentioning the new behavior in the docs.

Adding a docs edit, then should be ready to go. One moment.


onDraw method for watermark

Hi
I'm trying to put watermark in the pdf view with a simply code :
PDFView pdfView= (PDFView) findViewById(R.id.pdfView);
        pdfView.fromAsset("test.pdf").onDraw(new OnDrawListener() {
            public void onLayerDrawn(Canvas canvas, float pageWidth, float pageHeight, int displayedPage) {
 }

but I don't really understand how it's works
Thanks for your time

EDIT:
my onLayerDrawn  is never call

if you are loading the PDF correctly and can view it and move it, then the layers are being redrawn are they not ?
Just use a canvas.drawBitmap( within the onLayerDrawn

put your code in the onlayerdrawn and tell it to draw on the canvas at a set location

How can I add a digital signature in the pdf loaded from url


Buffer Contents Bleed into Autocomplete Docs

Issue
The first level of the auto-complete-panel has a solid opaque background, making it nice and readable.
The second level technically has a background, but it has no blur and is almost entirely transparent, making the contained text and overlayed text visually merge, making it very had to read.
This screenshot explains it better than I can with words:

I have no clue what the UI components are actually called, otherwise I'd try to fix it myself and create a pull request with the adjusted catppuccin-blur.json
This applies to all variants.
Slightly off topic: The full-screen terminal-panel (activated via the "Zoom In" button) has a similar issue. It has no blur and is so transparent that it becomes hard to use. In the case of the terminal this mostly applies to the variants other than Latte.
Context
Zed 0.166.1
Operating System: Nobara Linux 40
KDE Plasma Version: 6.2.3
KDE Frameworks Version: 6.8.0
Qt Version: 6.7.2
Kernel Version: 6.11.9-200.fsync.fc40.x86_64 (64-bit)
Graphics Platform: Wayland

Was not able to replicate bug on my setup, running Zed Preview on a Mac

I was able to reproduce the terminal, although on me setup the zoom terminal just has some opacity, not completely transparent:
https://github.com/user-attachments/assets/b15b7167-2dfb-4f1e-948e-c73d8248588e

While I am able to reproduce the issue on Mac on stable, I downloaded the current preview and there the secondary autocomplete panel is fixed for me too. Same on Linux with the most recent preview build. Weird, since nothing in the release notes mentions anything regarding these components, but I guess maybe a side-effect of another fix. :shrug: So I suppose this should be resolved automatically with the v0.167.X release.
And regarding the terminal: yes, that is what it looks like, the effect is more pronounced in "the variants other than Latte.". I have been using a terminal buffer instead as a workaround which is good enough for me. I just wanted to mention it in case the see-through background for the terminal when it overlaps a buffer was an oversight and not an intentional decision.
Thank you!


Update package metadata

I know it's common to shy away from "production ready" tags these days, but I think Django might be ready to drop its pre-alpha status 😌
From what I can tell this was set several years back and just hasn't been touched.

Here's the list of relevant tags: https://pypi.org/classifiers/

On the main branch we're working on the next Django release, so it's always in the Pre-Alpha stage. Development status is changed in stable branches when alpha, beta, release candidate, etc. are released.

Right, that makes sense. Thanks!


ledgerutil: Add "verify" command

Type of change

New feature

Description
We would like to propose a new command in Ledger Utility that checks the integrity of a single ledger. The existing commands in Ledger Utility aim at checking the ledger when any difference is found by comparing two ledgers. The purpose of the command proposed is to complement these commands by finding any defect in ledger files in a single peer that could be caused by potential software bugs or disk errors.
This patch adds a new command, “verify,” to Ledger Utility, which performs integrity checks for a specified ledger. Currently, the checks for the hash values in the headers of the blocks are implemented.
A use case assumed is to run the command locally in a peer periodically, which can be utilized as one of the health checks for the peer.
Additional details
This patch contains a sample ledger that is tweaked to have a hash value error (the hash value for Block 0 is modified not to match the block payload). Using the ledger, the command proposed can detect the error as follows:
$ ./build/bin/ledgerutil verify ./internal/ledgerutil/testdata/sample_bad_ledger -o /tmp/ledgerutil-verify-result-bad
...
Successfully executed verify tool. Some error(s) are found.
$ cat /tmp/ledgerutil-verify-result-bad/mychannel_verification_result/blocks.json
[
{"number":0,"valid":false,"errors":["DataHash mismatch"]}
,
{"number":1,"valid":false,"errors":["PreviousHash mismatch"]}
]

@denyeart Thank you for the review! I have fixed the patch according to your suggestions.

@manish-sethi Could you review as well? Anything else you think should be in a ledger 'verify' utility?

@shimos One of the new unit tests failed.

@manish-sethi Thank you for the comment and review. I have added a logic to determine the first block available and a unit test using a ledger bootstrapped from a snapshot.
@denyeart Thank you for letting me know. I have fixed the failed unit test.

@denyeart Thank you for trying out and the suggestion. I've addressed your point.

We'll also need to add the documentation, do you want to do that in this PR or a next one?

Yes, I'm happy to work on the documentation and submit another PR.

@shimos Ok, I'll merge this PR and you can work on the doc in a next PR. Thanks!

@Mergifyio backport release-2.5


Streamline destructuring

This PR cleans up some of our destructuring transform to be more consistent, along with a few additional changes:

Converges flattening of binding pattern and assignment patterns to follow the same code paths.
Adds to NodeArray a transformFlags property which consists of the transform flags of each node in the array without exclusions. This allows us to query the transform flags of each node in a node array without the need to traverse the array.
Consolidates a few TransformFlags enum members to free up some space
Added ContainsObjectRest/ContainsObjectSpread to allow us to more accurately detect esnext features.
Changes the type check for Object rest assignment to allow any valid lref via a call to checkReferenceExpression to align with the spec proposal.
Fixes a few cases where we unnecessarily emit the rval for a DestructuringAssignment that goes unused.

// cc: @sandersn, @mhegazy


Update tutorial-windows-vm-access-sql.md

Error handling

@barclayn
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@verargulla - see @barclayn's comment https://github.com/MicrosoftDocs/azure-docs/pull/111762#pullrequestreview-1535796341 . thanks