Is onpagenavigate supported for RaiseCifClientEvent?

I tried to implement the onpagenavigate event in the USD as it was described in this article:
https://community.dynamics.com/365/customerservice/b/unifiedservicedeskblog/posts/channel-integration-in-unified-service-desk.
But here I found for RaiseCifClientEvent:
Supported client-side events are as follows:
OnClickToAct
SendKbArticleEvent

Please can you clarify if onpagenavigate is supported?

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

ID: d8d70403-4536-faea-17bb-83a1c75a8087
Version Independent ID: 19aded45-36cc-f23f-83f0-6a8c4037582e
Content: Channel Integration Framework (CIF) (Hosted Control)
Content Source: ce/unified-service-desk/channel-integration-framework-hosted-control.md
Service: dynamics-365-customerservice
GitHub Login: @kabala123
Microsoft Alias: kabala

@StephanBuSu : Thanks for reaching out. I've raised your concern with the internal teams. I will update as soon as I've an update. Thanks for your understanding.

@StephanBuSu : Yes, onpagenavigate is supported, and I will update the documentation. Hope this helps. Closing the issue for now.


bitbucket 4.5.2 server url slightly different

With the latest stable packages (2.8.3) I am seeing a bitbucket url embedded like this:
https://user@bitbucket.company.co.uk:8443/projects/san/repos/superlib/raw/SuperLib/SuperLib/SuperMan.cs?at=e0777d44f95cbaf7210dcbd7f316a9f1306eb275

We are running bitbucket server 4.5.2 which seems to want a slightly different url:
https://user@bitbucket.company.co.uk:8443/projects/SAN/repos/superlib/browse/SuperLib/SuperLib/SuperMan.cs?at=7e935c37d1af209cc4912a7d437e44f5e43a4dc4&raw

Would there be a workaround for this?

Yep, just set the SourceLinkUrl yourself.

Super, thanks :-)


[Feature Request]

Checklist

[X] This issue contains only one feature request. I will open one issue for every feature I want to request.
[X] I checked, but didn't find any duplicates (open OR closed) of this issue in the repo.
[X] This feature is merely an UI/UX update.
[X] This feature is not going to conflict with many of the existing features/options.

Is your feature request related to a problem? Please describe.
Have to add multiple widgets to my homescreen as i have a lot of goals
Describe the solution you'd like
Stacking or scrolling widget stack would be a great way to implement multiple widgets at once...
How will you/everyone benefit from this feature?
It will ease the hassle of creating multiple iteration of the same widget for different goals into a singular well-defined structure.
Additional context
Hey currently the widget implementation is limited and i think there is room for improvements. When choosing the goal to be displayed on the widget llet the user be able to add multiple goals. The user will be able to scroll through all his goals from a single widget instead of relying on many single goal widgets

Seems interesting, if you can show any reference for stacking/scrolling widgets it'll be helpful to understand how it should look like, thanks :)

There is a beautiful app called Hurry that provides this functionality with it's widgets. And the stacking of multiple widgets in Niagara launcher may also be something that is highly functional yet simple... Check out both of them and I assure that you will see them adding functionality to your app...

Thanks, I'll have a look


Identify/adapt NGSI-LD compliant Context Broker

Fiware GE and NGSI-LD compliant - 'Scorpio' has been identified for Context Broker. Scorpio broker is microservice compliant and is implemented in Spring Boot. Scorpio broker has all the NGSI-LD APIs implemented in its microservices - Entity Manager, Subscription Manager, Context Registry Manager, History Manager. The services exchange data over Apache Kafka. Persistence support is provided by Postgres. Scorpio is compliant with microservices patterns using the Spring cloud framework - Eureka for Service Discovery, Zuul for API Gateway, etc. Scorpio is licensed under BSD-3-Clause.


Terminology improvement --- call it "Reduce" rather than "Diff"

v0.3.6 I used a word "diff", "differ"
https://github.com/kazurayam/materialstore/tree/main/src/main/groovy/com/kazurayam/materialstore/differ
I should rename them to "reduce", "reducer".
Also see

"materialize", "map", "reduce", "report"

"reduce a pair of Materials into a single Material"
Diff stage is a variation of Reduce stage

In #123 I will do

materialize Excel files files in the store.
map a *.xlsx file into a *.csv file
reduce a pair of source *.csv files derived from *.xlsx into a *.html file which includes the text-diff information of CSV files.
compile a report of the projects of "reduce" process.

In #124 I will do the similar for PDF files.

in v0.4.3, the package com.kazurayam.materialstore.differ is moved to com.kazurayam.materialstore.reduce.differ


(wild) Idea: continuous mode

Just a wild (maybe useless) idea: would it be possible to add a 'continuous mode'? This would be a mode where VDebug would automatically restart to listen (with a large enough timeout) after a request has ended. That would allow one to setup some breakpoints, start the debugger in Vim, and click through the php project until the page is reached which will trigger the breakpoints.
Now most of the times I set my breakpoints, start clicking around to reach the correct page, and before the final click need to remember to, at that moment, start the debugger in vim before continuing.
(obviously this is most useful if you have configured XDebug to automatically open a debug session on the server)

Hi, might I suggest closing this ? I thought this mode didn't exist yet (since the issue is still open) and just stumbled upon on it in the documentation by luck, just like you said here @joonty.

I think this problem (blocking Vim UI while listening) is not related to the current issue (continuous mode) and can be moved to another issue.

Anyone reading this that cares for the solution, the option is
let g:vdebug_options['continuous_mode'] = 1
which you might have to combo with:
let g:vdebug_options['break_on_open'] = 0
let g:vdebug_options['continuous_mode'] = 1

Thanks by the way, this mode is working great :+1:

Hi @Dudemullet, thanks for raising this.
While there is a continuous mode in version 1, it doesn't quite fit the brief of the original request, as it blocks the user after a debugging session has ended, until the next one starts. In version 2 I have it working so that it doesn't block, but there are still some things that need to be done to get it production ready. So I said it's not related in a previous comment, but it sort of is related! :P
The reason I left this open was so I could notify everyone in the thread when I get version 2 out. That will happen at some point, promise!

Has continuous mode been added to the new version? (I saw the issue was closed)

@baptx yes, when the previous request is finished and another hits vdebug it will start debugging immediatly


feat: jump to neareast position in preview from cursor

resolve #626

I might finish on the server side, while I have no idea how to handle the client side for jump and debugging it.
I tried to console.log in the ws.ts file, while I don't know where the result will be logged to.

Could you add a sample document in https://github.com/Myriad-Dreamin/tinymist/tree/main/editors/vscode/e2e-workspaces/simple-docs to help me test the improved functionality?


adding documentations for wfc.exe

Summary
Describe your changes here.
Fixes #Issue_Number (if available)

@gewarren  @blowdart  Is there any other edit needed for this PR?


Automated cherry pick of #9021: fix(region): auto sync cloudaccount resources

Cherry pick of #9021 on release/3.4.
#9021: fix(region): auto sync cloudaccount resources

Codecov Report

Merging #9031 (43f5c84) into release/3.4 (fc16c37) will decrease coverage by 1.96%.
The diff coverage is 1.78%.

@@              Coverage Diff               @@
##           release/3.4   #9031      +/-   ##
==============================================
- Coverage         6.19%   4.22%   -1.97%
==============================================
  Files              761     454     -307
  Lines           118405   83868   -34537
==============================================
- Hits              7330    3545    -3785
+ Misses          110412   80019   -30393
+ Partials           663     304     -359

Flag
Coverage Δ

aFlag
4.22% <1.78%> (-1.97%)
:arrow_down:

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

Impacted Files
Coverage Δ

pkg/cloudcommon/notifyclient/notify.go
1.78% <0.00%> (-0.03%)
:arrow_down:

pkg/compute/models/cloudaccounts.go
0.48% <0.00%> (-0.01%)
:arrow_down:

pkg/compute/models/networks.go
0.47% <0.00%> (+<0.01%)
:arrow_up:

pkg/compute/models/storagecachedimages.go
0.00% <0.00%> (ø)

pkg/compute/models/vpcresource.go
0.00% <0.00%> (ø)

pkg/compute/models/wires.go
1.24% <0.00%> (-0.01%)
:arrow_down:

pkg/hostman/hostinfo/hostovn.go
27.45% <0.00%> (-2.03%)
:arrow_down:

pkg/monitor/alerting/eval_context.go
38.14% <ø> (ø)

pkg/monitor/alerting/notifier.go
0.00% <0.00%> (ø)

pkg/multicloud/esxi/manager.go
0.72% <0.00%> (-0.01%)
:arrow_down:

... and 306 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 91dd66b...43f5c84. Read the comment docs.


fix: theme override

any themes defined in the app is overridden in controls

using the new changes the colors defined by media_kit are still the same but we dont override the user themes

linux tests failed, should i merge or not?


spoofed-favicon.badssl.com

Hey, @lgarron?  I'm working on a new domain that's pretty neat:

Still need to do other browsers.  If you know where in the Chromium source tree that I could find the iconography for the lock icon, that would be a huge help.  :)
Thanks!

(both for Chrome and Chrome Mobile, as they seem to use different favicons)

Try securityState* from https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit/Source/devtools/front_end/Images/&q=securitystatesecure&sq=package:chromium&type=cs and

The actual new toolbar seems to use ToolbarModelImpl::GetVectorIcon, but I would use securityState* from DevTools.
For the (raster) desktop icon, try use GetIcon() above to search for the file name, e.g. omnibox_https_valid.png
If you want a faviconny one, steal the one from https://https.cio.gov/ ; I made it for @konklone.

Also, didn't we learn a long time ago not to put a favicon in the toolbar like that? :-P

Awesome!  Thank you!

by adapting the border of the Chrome icon to be sized correctly for a favicon.

(Also, to include a 32x32 size for Retina.)

And yes, you would think so.
The Wheel of Time turns, and ages come and pass, leaving memories that become legend. Legends fade to myth, and even myth is long forgotten when the Age that gave it birth comes again. In one Age, called the third age by some, an Age yet to come, an age long pass, a wind rose in the Mountains of Security Iconography. The wind was not the beginning. There are neither beginnings or endings to the turning of the Wheel of Time. But it was a beginning.

All of this has happened before and all of this will happen again.

You guys created your own vector format?  SVG not good enough or something?
Anyways, I've got the Chrome Mobile icon without too much trouble, but I'm having trouble finding the Chrome Desktop icon anywhere besides in that .icon file.

Oh wait, never mind.  I found it.  :)

Closing this out, although it needs to be added to the current home page.


Recommend a different VPS instead of Vultr

vultr is bad

https://www.reddit.com/r/legaladvice/comments/572be2/charged_200_for_no_reason_on_vultrcom/
https://hostadvice.com/hosting-company/vultr-reviews/#user-reviews
https://www.trustpilot.com/review/vultr.com

@LukeSmithxyz, which other VPS did you add?


Fix Repeater clone button hit collapsible event (bubbling effect from header)

This PR fixes bug on Repeater clone button that still hit header event on collapsible

Thanks


Ignore events by default unless it's a supported event by non-api user

There was a bug where SG events weren't being ignored by default, the default should be to ignore, otherwise user API events were causing a recursion of updates (and failing to do so)

Thank you for highlighting the issue. It appears that the API events were encountering errors during user identification.
Could you confirm please that this was returning false even the event was emited from API?
https://github.com/fabiaserra/ayon-shotgrid/blob/c891fbbc9cc45046ae1fea3503c6b7498a04f7fd/services/leecher/leecher/listener.py#L271

It appears that the API events were encountering errors during user identification.

Not sure I understand what you mean.

Could you confirm please that this was returning false even the event was emited from API? https://github.com/fabiaserra/ayon-shotgrid/blob/c891fbbc9cc45046ae1fea3503c6b7498a04f7fd/services/leecher/leecher/listener.py#L271

Yeah it returned False but if you look at the code, anything that's not catching the first conditional or the type is one of the supported_event_types it would not be ignored when it should be?
                    if (
                        event["event_type"].endswith("_Change")
                        and event["attribute_name"].replace("sg_", "")
                        not in self.custom_sg_attribs
                    ):
                        # events related to custom attributes changes
                        # check if event was caused by api user
                        ignore_event = self._is_api_user_event(event)
                    elif event["event_type"] in supported_event_types:
                        # events related to changes in entities we track
                        # check if event was caused by api user
                        ignore_event = self._is_api_user_event(event)


Fixed problem with --min-ram when using google-batch provider

I tried switching from using pipeline API to the google-batch provider for a batch job that we've run regularly for years. However, when submitting it as a job to the batch API, the job consistently was getting killed, apparently by the OOM killer.
The job was submitted with --min-ram 10 and when looking at the cloud console, I can see it allocated an appropriate machine type, but the memory reported is 1.95.

Looking at the code I noticed that compute_resources was not being set, and according to the batch api docs there are per-task resource memory and cpu caps which default to 2GB and 2 cpus if not specified otherwise.
This PR adds a change which uses the values from job_resources to populate the compute_resource field on the job submission.
It seems to work for me, and I thought it'd be useful to merge it into the mainline for anyone else that might encounter this. (Especially as the pipeline API is being shutdown and the Batch API is what google recommends migrating to)
Please let me know if there are additional changes that you would like in order to get this merged in.

First and foremost, thank you so much for the PR @pgm!
So I had raised this issue with the Batch API about two months ago, and the conclusion was that it was a UI issue.
The expectation is that when you specify --min-ram 10, dsub will specify a custom machine that has at least 10 GB of RAM. That is what the custom-2-10240 machine type indicates.
So now we get to your screenshot. Despite the machine having 10GB of RAM, the Google Cloud UI says tasks are capped at 2GB RAM and 2 CPU cores. I asked the Batch API team about this, and they said that the per task resource requirements are treated as intention, which Batch uses to calculate how many tasks could fit into a VM. But tasks are free to use all resources once they are the VM. I even tested it myself in this comment and verified that the task had all the memory available despite the UI saying otherwise.
So all that being said, are you noticing with this change that your job is no longer killed from OOM? If so, this would be a very recently new behaviour of the Batch API that I'd like to bring up with them.

Ah, I'll close this PR because it appears you're right.
I commented out my change and tried it the job submission without compute_resource being set, and it still worked fine.
Looking back on the sequence of events that led me to make this PR, I think the issue may have been that I was originally testing with an old version of dsub. With an older version, it definitely failed with signs of memory exhaustion.
Perhaps when I tried with the newest version, I didn't actually confirm that the job was still getting the same out of memory condition but instead only assumed it was when I saw the memory wasn't increasing when looking at the google console view of the tasks.
Regardless, seems like the newest version is working fine without my change.
Sorry for the confusion.

Appreciate the response @pgm . You're not the first one to be confused by what the UI is showing there, and this is good feedback for that team.


Deprecate safe_level of ERB.new in Ruby 2.6

Summary
The interface of ERB.new will change from Ruby 2.6.

Add :trim_mode and :eoutvar keyword arguments to ERB.new.
Now non-keyword arguments other than first one are softly deprecated
and will be removed when Ruby 2.5 becomes EOL. [Feature #14256]

https://github.com/ruby/ruby/blob/2311087/NEWS#stdlib-updates-outstanding-ones-only
Details
This PR uses ERB.instance_method(:initialize).parameters.assoc(:key) to switch ERB.new interface. Because Cucumber multiple Ruby versions, it need to use the appropriate interface. This approach is built into Ruby.
https://github.com/ruby/ruby/commit/3406c5d

@koic Thanks for the PR! I just had one quick question I was wondering if you could answer before I merge it.

This looks good, but is there a real benefit to using ERB.instance_method(:initialize).parameters.assoc(:key) rather than just RUBY_VERSION >= '2.6'? I understand it's in the official ruby repo, but I'm not sure the complexity is otherwise justified?

AFAIK, comparison using Ruby version number seems to tend to be less favored. If Ruby version is easier to understand as Cucumber, I will replace ERB.instance_method(:initialize).parameters.assoc(:key) with RUBY_VERSION >= '2.6'. So, of course there is no problem with this.

Thanks, @koic. In the interest of keeping things simple, and assuming there's no disadvantages to doing so, I think it would be best if we use the RUBY_VERSION comparison method. Thanks once again for your work on this.
@danascheider, do you still have questions regarding this PR?

I get it! I updated this PR using RUBY_VERSION. Would you please review again?

I've merged the master branch into this, and have added an (optional) build for Ruby 2.6 into the CircleCI config.
Everything seems OK for Ruby MRI on Linux, but JRuby and Windows (AppVeyor) appear to be having issues. AppVeyor's issues appear to be due to differences between the usage of LF and CRLF on Linux and Windows.
I don't have time to look into this right now, so I'm happy for someone else to take over in the meantime, if you would like. :)

Hi @koic,
Thanks for your making your first contribution to Cucumber, and welcome to the Cucumber committers team! You can now push directly to this repo and all other repos under the cucumber organization! 🍾
In return for this generous offer we hope you will:

✅ Continue to use branches and pull requests. When someone on the core team approves a pull request (yours or someone else's), you're welcome to merge it yourself.
💚 Commit to setting a good example by following and upholding our code of conduct in your interactions with other collaborators and users.
💬 Join the community Slack channel to meet the rest of the team and make yourself at home.
ℹ Don't feel obliged to help, just do what you can if you have the time and the energy.
🙋 Ask if you need anything. We're looking for feedback about how to make the project more welcoming, so please tell us!

On behalf of the Cucumber core team,
Aslak Hellesøy
Creator of Cucumber

Thank you for inviting me to the Cucumber committers team. I'm very honored!


fsspec-xrootd fails pip-check as version 0.0.0 when coming from conda?

Solution to issue cannot be found in the documentation.

[X] I checked the documentation.

Issue
As in title, for some reason when trying to do conda builds of coffea pip check reports fsspec-xrootd as 0.0.0.

Installed packages
tornado:             6.3.3-py311h459d7ec_1       conda-forge
    tqdm:                4.66.1-pyhd8ed1ab_0         conda-forge
    typing-extensions:   4.9.0-hd8ed1ab_0            conda-forge
    typing_extensions:   4.9.0-pyha770c72_0          conda-forge
    tzdata:              2023c-h71feb2d_0            conda-forge
    ucx:                 1.15.0-h75e419f_2           conda-forge
    uhi:                 0.4.0-pyhd8ed1ab_0          conda-forge
    uproot:              5.2.0-ha770c72_0            conda-forge
    uproot-base:         5.2.0-pyhd8ed1ab_0          conda-forge
    urllib3:             2.1.0-pyhd8ed1ab_0          conda-forge
    wheel:               0.42.0-pyhd8ed1ab_0         conda-forge
    xorg-libxau:         1.0.11-hd590300_0           conda-forge
    xorg-libxdmcp:       1.1.3-h7f98852_0            conda-forge
    xrootd:              5.6.4-py311hab2d546_0       conda-forge
    xxhash:              0.8.2-hd590300_0            conda-forge
    xyzservices:         2023.10.1-pyhd8ed1ab_0      conda-forge
    xz:                  5.2.6-h166bdaf_0            conda-forge
    yaml:                0.2.5-h7f98852_2            conda-forge
    zict:                3.0.0-pyhd8ed1ab_0          conda-forge
    zipp:                3.17.0-pyhd8ed1ab_0         conda-forge
    zlib:                1.2.13-hd590300_5           conda-forge
    zstandard:           0.22.0-py311haa97af0_0      conda-forge
    zstd:                1.5.5-hfc55251_0            conda-forge

Environment info
Whatever's in the conda build system containers.

@lobis I think you may be on the hook for figuring this one out because I have no idea. :-)


Keep series name in mean on SeriesGroupBy

Due to a new pandas release, test_split_apply_combine_on_series in test_dataframe.py was failing on a mean of a SeriesGroupBy. The name of the series was being dropped in the computation. This fixes it, but there may be a better way.

this is the right way to do this :+1:


SAP HANA XS Advanced, Creating a Node.js Module

Tutorial URL: https://developers.sap.com/tutorials/xsa-xsjs-xsodata.html
Step 6: Execute the xsjs module.
Logs
Application is starting
Running module /HANA_APP/core_xsjs failed. See console logs for more details.
Service provisioning for module: '/core_xsjs'
Service instance 'XSA_DEV-e1i4v3gg18804q2p-HANA_APP-UAA-service' for resource 'XSA_DEV-e1i4v3gg18804q2p-HANA_APP-UAA-service' already exists, no operation will be performed.
Service instance 'XSA_DEV-e1i4v3gg18804q2p-HANA_APP-hdi_db' for resource 'XSA_DEV-e1i4v3gg18804q2p-HANA_APP-hdi_db' already exists, no operation will be performed.
[INFO] Injecting source code into builder...
[INFO] Source code injection finished
[INFO] ------------------------------------------------------------------------
npm ERR! code ETARGET
npm ERR! notarget No matching version found for<EMAIL_ADDRESS>npm ERR! notarget In most cases you or one of your dependencies are requesting
npm ERR! notarget a package version that doesn't exist.
npm ERR! notarget
npm ERR! notarget It was specified as a dependency of 'core_xsjs'
npm ERR! notarget
npm ERR! A complete log of this run can be found in:
npm ERR!     /hana/shared/HXE/xs/controller_data/executionagent/executionroot/52934674-5df4-4ac6-a048-689179ec1339/app/META-INF/cache/node10.17/.npm/_logs/2020-09-04T05_03_13_456Z-debug.log

Both 3.1.2 and 3.1.3 of the library is available online: https://www.npmjs.com/package/@sap/xsjs-test
Perhaps your system is an older release of HANA Express and doesn't have an upstream link configured or no connection to the internet.  Then you might not have this newer version of the library that is required.  We would recommend that you configure your system with an upstream link to the public npm repository.  That way you can always pull the latest version from the internet.    Do you know if the system you are doing this tutorial on has the upstream link configured?

Thank you for your feedback. We haven't heard from you in a while, so we are closing the issue.
If you still have questions, feel free to reopen the issue.


Install Error

gyp info it worked if it ends with ok
gyp info using<EMAIL_ADDRESS>gyp info using<EMAIL_ADDRESS>| darwin | x64
gyp http GET https://atom.io/download/electron/v1.3.13/iojs-v1.3.13.tar.gz
gyp WARN install got an error, rolling back install
gyp ERR! install error
gyp ERR! stack Error: connect ETIMEDOUT <IP_ADDRESS>:443
gyp ERR! stack     at Object.exports._errnoException (util.js:870:11)
gyp ERR! stack     at exports._exceptionWithHostPort (util.js:893:20)
gyp ERR! stack     at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1061:14)
gyp ERR! System Darwin 15.6.0
gyp ERR! command "/Applications/Atom.app/Contents/Resources/app/apm/bin/node" "/Applications/Atom.app/Contents/Resources/app/apm/node_modules/node-gyp/bin/node-gyp.js" "install" "--runtime=electron" "--target=1.3.13" "--dist-url=https://atom.io/download/electron" "--arch=x64" "--ensure"
gyp ERR! cwd /Users/lidoujun/.atom
gyp ERR! node -v v4.4.5
gyp ERR! node-gyp -v v3.4.0
gyp ERR! not ok

the same to u, how to fix it..

Hi @ls69knight
How are you installing the package?
That's seems like a problem with something else.

you can use Offline installation to install those package.....

Closing for lack of activity, Open again if the issue persist with the latest release.
And yes, seems like a network problem.

Just a note, I was having this as well and now its gone.  Perhaps it really is a network issue, but i'd suspect some silliness in node on osx.  Really no idea why, though.
Error: gyp info it worked if it ends with ok
gyp info using<EMAIL_ADDRESS>gyp info using<EMAIL_ADDRESS>| darwin | x64
gyp http GET https://atom.io/download/electron/v1.7.5/iojs-v1.7.5.tar.gz
gyp WARN install got an error, rolling back install
gyp ERR! configure error
gyp ERR! stack Error: connect ETIMEDOUT <IP_ADDRESS>:443
gyp ERR! stack     at Object.exports._errnoException (util.js:1050:11)
gyp ERR! stack     at exports._exceptionWithHostPort (util.js:1073:20)
gyp ERR! stack     at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1093:14)
gyp ERR! System Darwin 16.7.0
gyp ERR! command "/opt/local/bin/node" "/Users/andrew/dev/st/desktop-gen-electron/src/app/node_modules/.bin/node-gyp" "rebuild" "--target=1.7.5" "--arch=x64" "--dist-url=https://atom.io/download/electron" "--build-from-source"
gyp ERR! cwd /Users/andrew/dev/st/desktop-gen-electron/src/app/node_modules/sharp
gyp ERR! node -v v7.10.1
gyp ERR! node-gyp -v v3.6.2
gyp ERR! not ok

That host maps to s3-1-w.amazonaws.com, and going to that url gives you the message that all access has been disabled.  After that I was able to complete the rebuild!
Anyway, try going to that url, maybe it'll help someone in the future.


Support for Ada

I would like to support Ada as a language.

I've heard many great things about Ada. This is awesome :)
I added a small repo to help with this initial exploration work: https://github.com/StateSmith/new-language-kickoff
Looking forward to it!

Thanks. I appreciate the work you've put into this project and your enthusiasm. I am very impressed with the simulation demos and I think you're off to a great start. Unfortunately, as I dug deeper into the actual generated code, I'm not sure this is the right tool for my needs.
First, the code relies on nullable references. It's a major source of bugs and should be used sparingly, especially in something as critical as a state machine.
Second, the generator produces a lot of unnecessary code. It generates a function for every state, making the code very difficult to follow. It also generates code for outputting states as strings, which is already built-in in both C# and JavaScript.
Finally, the entire state machine is far better represented using a transition table. It's more succinct, readable, and easier to maintain. I read somewhere you decided against it because you believe it's memory inefficient. This is a common misconception. Modern compilers do an excellent job of optimizing data structures, but it requires some expertise to fully exploit them.
I'm not sure whether this is a good fit for my needs.

If you'd like, you can write your own fully custom code generator. You can
hook it in without changing StateSmith if you are using a .csx file. Uses
dependency injection. Much of StateSmith can be customized from a CSX file.
Your code wouldn't need to transpile. You receive a state machine graph
from StateSmith and then the rest is up to you.
Then you get the benefit of diagram parsing and state machine graph
transformations.
There are settings to disable the "to string" functions if you want.
I'm personally not worried about null references in the generated code
because of the thorough specification test coverage. Maybe you could run
into an issue if you didn't respect concurrency.
There is no perfect Algorithm for every need. There are always trade offs.
The current algorithm works well for every need I've come across, but could
always be more optimized for specific needs/desires.
I'd be happy to support you if you wanted to write a transition table
algorithm if you like. It might not be too hard if the state machine is
flattened first (remove hierarchy).
The down side of larger tables is for architectures like AVR that can't
store data in flash without a bunch of PROGMEM work arounds.
On Fri., Aug. 9, 2024, 4:02 p.m. Joebeazelman, @.***>
wrote:

Thanks. I appreciate the work you've put into this project and your
enthusiasm. I am very impressed with the simulation demos and I think
you're off to a great start. Unfortunately, as I dug deeper into the actual
generated code, I'm not sure this is the right tool for my needs.
First, the code relies on nullable references. It's a major source of bugs
and should be used sparingly, especially in something as critical as a
state machine.
Second, the generator produces a lot of unnecessary code. It generates a
function for every state, making the code very difficult to follow. It also
generates code for outputting states as strings, which is already built-in
in both C# and JavaScript.
Finally, the entire state machine is far better represented using a
transition table. It's more succinct, readable, and easier to maintain. I
read somewhere you decided against it because you believe it's memory
inefficient. This is a common misconception. Modern compilers do an
excellent job of optimizing data structures, but it requires some expertise
to fully exploit them.
I'm not sure whether this is a good fit for my needs.
—
Reply to this email directly, view it on GitHub
https://github.com/StateSmith/StateSmith/issues/377#issuecomment-2278751905,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACC4XDNQWGAC3ZZHHNV6ITZQUU5NAVCNFSM6AAAAABMH6DZCGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENZYG42TCOJQGU
.
You are receiving this because you commented.Message ID:
@.***>

Sounds great! Fortunately, you made the generator very flexible.
The real value for me is the plantuml parsing and the state machine simulation, which doesn't have to be generated in the target language.
Presuming I go with the CSX approach, what are the next steps? Continue with hand translating the sample?

On a side note, I strongly encourage you to explore some of the benefits of using modern C++ over C. Not for its OOP constructs, but for its optimizations.
A state machine transition table can be optimized away to almost nothing in C++. If you think about it, the table or its values never change at runtime, giving the compiler enormous leverage. For instance, statements like print transitions[OnState], where the table and its values are declared constants, are substituted with print OffState at compile time, eliminating the transitions table entirely.

Are you thinking of creating a new algorithm for StateSmith that uses a
table and can be transpiled to other languages?
Or a custom ada code generator for StateSmith? No transpiling.
Or something completely custom just for your use case?
On Fri., Aug. 9, 2024, 9:12 p.m. Joebeazelman, @.***>
wrote:

Sounds great! Fortunately, you made the generator very flexible.
The real value for me is the plantuml parsing and the state machine
simulation, which doesn't have to be generated in the target language.
Presuming I go with the CSX approach, what are the next steps? Continue
with hand translating the sample?
On a side note, I strongly encourage you to explore some of the benefits
of using modern C++ over C. Not for its OOP constructs, but for its
optimizations.
A state machine transition table can be optimized away to almost nothing
in C++. If you think about it, the table or its values never change at
runtime, giving the compiler enormous leverage. For instance, statements
like print transitions[OnState], where the table and its values are
declared constants, are substituted with print OffState at compile
time, eliminating the transitions table entirely.
—
Reply to this email directly, view it on GitHub
https://github.com/StateSmith/StateSmith/issues/377#issuecomment-2278940830,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACC4XAQBRI65QEOPL456V3ZQVZG7AVCNFSM6AAAAABMH6DZCGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENZYHE2DAOBTGA
.
You are receiving this because you commented.Message ID:
@.***>

I'm planning  on doing the first one. Adding a new language with its own algorithm.  I've been wanting to add less general purpose algorithms, but most of my time has been on ease of use improvements lately.
Because we are looking at a new algorithm that will be applied to all supported languages, can we start prototyping it in C? It would make things a lot easier for me.
We can still use the new language kickoff repo, but focus on the algorithm and use C first.
I'm keen to see your example in more detail. I don't quite understand how the transition table will work with guards, actions, entry/exit and other features. It looks like it maybe requires a transition every event? Let's make something small that can be run with the same test-harness.c.
Have you considered a constant table of function pointers. It could have "nop()" like functions instead of nulls for when a state doesn't care about an event.

state
event: off
event: dim
event: inc

OFF
unused_event()
unused_event()
off_inc_handler()

ON1
on1_off_handler()
on1_dim_handler()
on1_inc_handler()

ON2
…
…
…

Awesome!
The problem I see with many code generators is the kitchen sink design approach where a lot of complex and overly generalized code is generated to handle developer use case. It almost always backfires, often resulting in inefficient, inflexible, and hard to comprehend. It’s precisely the reason why some of the most popular developer tools tend to be simple, hence the popularity of StarUML and PlantUML.
Consequently, the generated code should be as simple as possible. A state machine is fundamentally tracks state, taking an event as an input and returning the next state as an output. How it’s used should be left to the developer, however, how it is implemented should be left to the generator.
The best approach generate the most common implementation schemes with some configuration specified in the diagram, or in a separate file. Where this tool adds value is constructing the transitions in code, which is tedious and error-prone to do manually.
Consequently, I see three implementation levels:

Minimum level: A state machine with a transition table of state and event values. Substates are flattened. Client writes their own dispatcher function. Roughly the code I submitted.
Medium level: A state machine with a generated dispatcher function.
Maximum: A medium level state machine using a transition table of function pointers.

You can isolate the dispatch function in a separate file so the transition tables can be regenerated whenever the diagram is modified with perhaps a warning if it requires modification to dispatch function. You could refer the client to samples of how each one can be used.
The guards, actions, entry and exit functions could be stubbed in as comments, but only if they are specified in the actual diagram, otherwise it adds excessive noise to the code.
How do you think?  What's my next step from here? I am proficient in C#, C, and a little JavaScript. I'll follow up with a maximum level sample using Ada.

One of my goals with StateSmith is to provide a tool to make it easier to
code generate state machines. We don't need to all reinvent the wheel from
scratch.
Your suggestion is fundamentally different than where StateSmith is
currently heading, but I think that's cool. It gives people more options
which is always good. We just need to document the different algorithms to
help users. Because the approach is fundamentally different and likely
won't have the same feature set, I plan to help support you as I'm able,
but you'd be the primary driver of the new algorithm. Sound good?
I think a helpful next step is to manually implement the state machine that
satisfies the c test harness linked above.
In a couple days, I'll put together an example for how you can hook into
StateSmith to add a new algorithm.
You can also explore some of the more advanced examples in the StateSmith
examples repo.
On Sat., Aug. 10, 2024, 2:25 p.m. Joebeazelman, @.***>
wrote:

Awesome!
The problem I see with many code generators is the kitchen sink design
approach where a lot of complex and overly generalized code is generated to
handle developer use case. It almost always backfires, often resulting in
inefficient, inflexible, and hard to comprehend. It’s precisely the reason
why some of the most popular developer tools tend to be simple, hence the
popularity of StarUML and PlantUML.
Consequently, the generated code should be as simple as possible. A state
machine is fundamentally tracks state, taking an event as an input and
returning the next state as an output. How it’s used should be left to the
developer, however, how it is implemented should be left to the generator.
The best approach generate the most common implementation schemes with
some configuration specified in the diagram, or in a separate file. Where
this tool adds value is constructing the transitions in code, which is
tedious and error-prone to do manually.
Consequently, I see three implementation levels:

Minimum level: A state machine with a transition table of state and
event values. Substates are flattened. Client writes their own dispatcher
function. Roughly the code I submitted.
Medium level: A state machine with a generated dispatcher function.
Maximum: A medium level state machine using a transition table of
function pointers.

You can isolate the dispatch function in a separate file so the transition
tables can be regenerated whenever the diagram is modified with perhaps a
warning if it requires modification to dispatch function. You could refer
the client to samples of how each one can be used.
The guards, actions, entry and exit functions could be stubbed in as
comments, but only if they are specified in the actual diagram, otherwise
it adds excessive noise to the code.
How do you think? What's my next step from here? I am proficient in C#, C,
and a little JavaScript. I'll follow up with a maximum level sample using
Ada.
—
Reply to this email directly, view it on GitHub
https://github.com/StateSmith/StateSmith/issues/377#issuecomment-2282253325,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACC4XCOCA5VPGA76P2Q52LZQZSMDAVCNFSM6AAAAABMH6DZCGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBSGI2TGMZSGU
.
You are receiving this because you commented.Message ID:
@.***>

This example is particularly relevant: https://github.com/StateSmith/StateSmith-examples/tree/main/visitor-graph-1
It doesn't show how to hook in custom code generation, but it does show how you can get and "visit" the state machine graph.
Also, if you are new to debugging C# scripts (.csx), check out the resources here: https://github.com/StateSmith/StateSmith/wiki/csx

@Joebeazelman how's it going?
I recently added support for a new algorithm Balanced2 (the new default) that uses switch statements instead of function pointers. I know you mentioned that nullable function pointers are a concern for you.
I also added support for Java and Python. Java was relatively easy because it is really close to C#. Python was more difficult. For both of them, I decided to only support the Balanced2 algorithm for now.
Anyway, it reminded me that this work can be pretty challenging and thought I would see how it was going.

I've decided to go with a manual approach. It's really not worth the time and effort to customize a tool for my purposes. I was hoping I could find something I could easily modify and get running.

Makes sense. The work is non-trivial. I'm going to close the issue for now. We can re-open if there is interest in the future.


Adding Karen Language

Hey team,
Our partner who would like the language installed sent along this link that they used.  I don't know if this helps or not.

Any status update?

We're still working on getting the web font to display properly in the tool. We will know more and be able to give you a status update later today.

Sounds good to me.  I'm OOO for a few days after today so I just wanted to make sure I was touching base on all our projects.  Thanks!

Updated. Karen is now one of the language selection options on the tool, as well as a configurable embed option.

Awesome, thanks!

I see Karen on the drop down, but not on the customizable embed page, am I missing something?


fix Aircraft

Aircraft was not working :)

I've cherry-picked both your Aircraft fix and Angel's Logistics support, and I've listed you in the changelog accordingly. Thanks for your contributions!


Website stuck on first purchase

The first purchase on the two brands does not work when clicked

That's a msg and not a button

The website freezes when clicked


Deprecate ec bn2point

Checklist

[x] documentation is added or updated
[ ] tests are added or updated

Replaced another instance of EC_POINT_point2bn() with EC_POINT_point2buf().

Updated based on comments..

Does this fix #10366?

Yes it has a commit message that states this.. Thanks for checking..

rebased..

ping

This pull request is ready to merge

Thanks. Merged to master.


Symbol not found: OBJC_CLASS$_Reachability: ReachabilitySwift causes symbol conflict with Reachability

Describe the bug
Amplify (through AmplifyPlugins/AWSAPIPlugin) depends on @ashleymills  ReachabilitySwift. This library introduces a conflict with the Objective-C Reachability library used by many other packages as it's author has openly chosen to use a conflicting name and publicly refused to fix the issue (https://github.com/ashleymills/Reachability.swift/issues/348).
As such, by depending on libraries that introduce a naming conflict AWS Amplify is breaking builds through requiring an essentially broken dependency the author openly refuses to fix. The most obvious symptom is a symbol resolution conflict that results in a compiler warning about multiple targets, that at run time, causes as "Symbol not found: OBJC_CLASS$_Reachability" error at app start for any app that includes Amplify and any other library that itself directly or indirectly relies on the original Reachability.
Consider dropping this dep and finding an alternative way to replicate Reachability's functions in Swift and this problem goes away.
To Reproduce
Steps to reproduce the behavior:

Create an iOS project that includes AWS Amplify for iOS and any other library that requires Reachability (the objective C version).
Try to build on iOS 14 target on Xcode 12

Expected behavior
Compiles without specific warning to this, and runs, no problem.
Environment(please complete the following information):

Amplify Framework Version: 1.1.1
Dependency Manager: Cocoapods
Swift Version : 5.3

Device Information (please complete the following information):

Device: Sim
iOS Version: ios 14
Specific to simulators: /.*/

Hi @24601
Thanks for reporting this. We'll add this to our backlog and consider options, and update this issue as we have any information.

A preliminary decision (pending an engineering spike) has been made to write a thin layer to replace ReachabilitySwift.  We will update the ticket when we know more about when this work can be started.

@24601 ,
I'm trying to reproduce this issue, so that we can be sure we fix this issue correctly.
Can you please suggest a library which "requires Reachability (the objective C version)", so that i can reproduce this issue on my machine?
Thanks

Hello Mr. Woo,
Thanks for looking into this and moving this issue out of the library! I
know that PusherSwift is one that many people have mentioned on
StackOverflow, and it might even have been responsible for ours as a
transient dependency, but let me go back in our SCM history and get the
branch of our app that had this issue and I'll build you a sample repo
demonstrating the issue if you'd like!
Thanks!
Basit
On Tue, Dec 8, 2020 at 12:31 PM John Woo<EMAIL_ADDRESS>wrote:

@24601 https://github.com/24601 ,
I'm trying to reproduce this issue, so that we can be sure we fix this
issue correctly.
Can you please suggest a library which "requires Reachability (the
objective C version)", so that i can reproduce this issue on my machine?
Thanks
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/aws-amplify/amplify-ios/issues/797#issuecomment-740904525,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAI2QV2MWWKC3DOZ2OF23BDSTZ5JHANCNFSM4SBHYG5A
.

@wooj2 - we were using graphql_flutter which depended on connectivity which used ReachabilitySwift, but connectivity has moved away from including ReachabilitySwift for this same reason, so the actual test case from our code won't help much with the new version, but pinning to a backlevel version should still replicate it.
BUT, I would still be happy to build you one (that is likely more straightforward as it won't involve the Flutter -> iOS portion) that is just native iOS Swift with PusherSwift as an example

Hi @24601 ,
I've made a draft PR effectively forking Reachability.swift to AmplifyReachability, but before moving forward with this, I'd feel more comfortable if I was able to reproduce the linker error you are seeing.
When you refer to the "Objective-C Reachability" library, which library are you referring to?

Hi @24601 ,
I believe I've been able to reproduce this issue by creating a sample flutter application with connectivity: ^2.0.2 as suggested by @Rayv1 located here, so there's no need for you to build me one.
Just to tie up loose ends, I believe I found the reachability library you were referring to, which seems to be: https://cocoapods.org/pods/Reachability, which looks very similar to the Apple Reachability sample code located here: https://developer.apple.com/library/archive/samplecode/Reachability/Introduction/Intro.html#//apple_ref/doc/uid/DTS40007324

Code changes have been pushed to the main branch.  I've applied the pending release tag as this code change is scheduled to be in our next release.

The fix is released in 1.5.2, please take a look https://github.com/aws-amplify/amplify-ios/releases/tag/1.5.2


Ignore XFS mountpoints in CageFS skeleton

CloudLinux CageFS mountpoints can sometimes cause problems with the XFS scanner actor. An example:
2023-07-05 19:39:39.111 DEBUG    PID: 11761 leapp.workflow.FactsCollection.xfs_info_scanner: External command has started: ['/usr/sbin/xfs_info', '/usr/share/cagefs-skeleton/var/www/cgi-bin\\040(deleted)']
2023-07-05 19:39:39.117 DEBUG    PID: 11761 leapp.workflow.FactsCollection.xfs_info_scanner: xfs_info: /usr/share/cagefs-skeleton/var/www/cgi-bin\040(deleted) is not a mounted XFS filesystem

CageFS mountpoints causing problems with the xfs_info utility can be safely skipped over instead of halting the actor and the upgrade process.

Your review was submitted on a merged pull request

Meh


Onyx Leaf 2 strings

Strings from the Onyx Leaf 2 device

Sorry for the duplicate (https://github.com/Hagb/decryptBooxUpdateUpx/pull/46).

Tried the keys from #46 on the latest firmware available for the Leaf 2, and it did not work. However, this set did. Based on that, you might want to re-open this PR.

@CrazyCoder  Thanks for your strings!

Tried the keys from #46 on the latest firmware available for the Leaf 2, and it did not work. However, this set did. Based on that, you might want to re-open this PR.

@FreeTheTech101 It seems that there are two models called "Leaf 2". The one in #46 has "MODEL": "Leaf2_P", and the one in this PR has "MODEL": "Leaf2".

Can confirm these are what I got on my Leaf 2

Leaf2_P is likely the China Mainland variant. (https://github.com/Hagb/decryptBooxUpdateUpx/pull/46#issuecomment-1403700386)

Thanks for your contribution! The PR is merged now.


 IF YOU DO NOT FOLLOW THE FORMAT OR MAKE A GENERIC TOKEN REQUEST YOUR ISSUE WILL BE DELETED WITHOUT COMMENT
YOUR JUSTLIST MUST FOLLOW THE JSON SPECIFICATION
https://github.com/justswaporg/justlists/example.justlists.ts
Checklist

[x] I understand that this is not the place to request a token listing.
[x] I have tested that my JustList is compatible by pasting the URL into the add a list UI at justswap.org.
[x] I understand that filing an issue or adding liquidity does not guarantee addition to the justlists website.

Please provide the following information for your token.
JustList URL must be HTTPS.
JustList URL:
JustList Name:
Link to the official homepage of the JustList manager:

JustList URL must be HTTPS.
JustList URL:https://rtc-free-coin.com/
JustList Name:treadecoin
Link to the official homepage of the JustList manager

JustList URL must be HTTPS.
JustList URL:https://rtc-free-coin.com/
JustList Name:treadecoin
Link to the official homepage of the JustList manager


Bootable JAR Logging Updates

The following issues are fixed in this PR:

https://issues.redhat.com/browse/WFCORE-5115
https://issues.redhat.com/browse/WFCORE-5109

WFCORE-5115
Use a hard-coded boot-config.properties to load system properties in the bootable JAR entry point. This properties file is generated by the wildfly-jar-maven-plugin. It's used at boot to configure properties required by the log manager during initialization.
WFCORE-4895
Minor follow up to the bootable JAR module.
WFCORE-5109
Ensure the configurator is attached to the root logger. This means the logging subsystem will not reconfigure resources that do not need to be reconfigured.

I'm placing a temporary hold as we need to ensure the default relative-to directories are available. More than likely only the jboss.server.log.dir is used, but we should ensure they're all set as system properties for the log manager to use.

Core - Full Integration Build 9893 outcome was UNKNOWN using a merge of 4b38e25e835de0b5a411db53f412475555d9d686
Summary: Canceled (Tests passed: 2400, ignored: 14; exit code 143 (Step: Build & test full (Maven)) (new)) Build time: 00:40:10

Core - Full Integration Build 9944 outcome was UNKNOWN using a merge of 4b38e25e835de0b5a411db53f412475555d9d686
Summary: Canceled (Tests passed: 2263, ignored: 41; exit code 143 (Step: Build & test full (Maven)) (new)) Build time: 00:40:10

Core - Full Integration Build 9894 outcome was UNKNOWN using a merge of 4b38e25e835de0b5a411db53f412475555d9d686
Summary: Canceled (Tests passed: 1288, ignored: 14; exit code 143 (Step: Build & test full (Maven)) (new)) Build time: 00:37:23

Core - Full Integration Build 9945 outcome was UNKNOWN using a merge of 4b38e25e835de0b5a411db53f412475555d9d686
Summary: Canceled (Tests passed: 2354, ignored: 41; exit code 143 (Step: Build & test full (Maven)) (new)) Build time: 00:37:25

Core - Full Integration Build 9946 outcome was UNKNOWN using a merge of 4b38e25e835de0b5a411db53f412475555d9d686
Summary: Canceled (Tests passed: 2226, ignored: 41; exit code 143 (Step: Build & test full (Maven)) (new)) Build time: 00:29:27

Core - Full Integration Build 9895 outcome was UNKNOWN using a merge of 4b38e25e835de0b5a411db53f412475555d9d686
Summary: Canceled (Tests passed: 1947, ignored: 14; exit code 143 (Step: Build & test full (Maven)) (new)) Build time: 00:29:28

Approved, @jamezp really awesome changes, thank-you!

@jamezp thanks


fix spelling mistakes

fix spelling mistakes

Hi @hwdef. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

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

/assign @geeknoid

/ok-to-test

@hwdef: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_istio
2cd8f5be5e0e5706d0401af6484799908b40cd98
link
/test gencheck_istio

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

/test gencheck_istio

@hwdef: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_istio
2cd8f5be5e0e5706d0401af6484799908b40cd98
link
/test gencheck_istio

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

Could you run BUILD_WITH_CONTAINER=1 make gen? which should help to update the binary blobs in mixer folder.

@bianpengyuan
Thanks for your suggestion.
It helped me a lot.

/retest


Image option disappears from the menu for user when I add ImageResize in extensions

Hey! I was using this with the starterkit package available.
Here is how my configs look like:

And when I remove ImageResize from extensions options in the configurations:

Hello, @0x0elliot
I checked it in my code sandbox, but i can't find view like your captured is floating box like 'No results found' or 'Image' according to '/' command. 🤔
It has to do with Markdown package, not StarterKit. So I think it would be easier to check if you showed more detailed examples or package dependencies.
Thanks for your interest in our repository. 🤗


Add support to migrations in folders

Hi guys,
the PR fixes a bug found here: https://github.com/spatie/laravel-multitenancy/issues/238
Thanks for your work.

Thanks!

❤


Rewrite all localizations and use secure URLs

This PR lets the transform function rewrite all Wikipedia localizations and uses the secure (https) URL variant. Relevant tests are included and existing tests have been updated to use https URLs.

Yikes, it's about time I fixed this. Thanks for the PR, you were right about the tests!


Add ability to select node filter using config options

Closes: #77

Please modify one of e2e tests to use first node filter. Also I suggest to change name from first to first-live or first-alive, to make it clear that we won't choose node that is considered unreachable from ipcontroller point of view,


Improve performance of domain update

Previously the loop would always query the db even for none existing accounts
Now cache previously looked up accounts and skip db queries for empty accounts

Loop time previously: 3.000699043273926 sec
Loop time now: 0.2798025608062744 sec

This drastically increases the loading time of the dashboard for us with around 1000 dns zones.
Maybe you could take a look at it @jbe-dw

Nice one. You meant "decreases" I guess :p
I'm sorry we are not as fast as we intended for merging, but we should move on this month.

Yes, that is what I meant. ^^

@AdvanticGmbH The numbers are very impressive! I look at this soon.
Thank you for the contribution!


SliverAppBar with ShrinkWrap Patch

This better emulates the behavior of the standard Viewport class, which, after painting all children before its center child, paints its center child and all subsequent children in reverse order. This fixes a bug causing children of a shrink-wrapping CustomScrollView to paint over a SliverAppBar.
Description
This PR simply reverses childrenInPaintOrder for the RenderShrinkWrappingViewport class. This solves any issues where slivers in a CustomScrollView would paint over a SliverAppBar whenever the scroll view had shrink-wrapping enabled.
Related Issues
#28197 Resolved by this PR.
#36002 Partly resolved by this PR. I did not modify scrolling behavior.
#52976 Partly resolved by this PR.
Tests
I added the following tests:

"Viewport childrenInPaintOrder control test" (rendering/viewport_test.dart)

Tests both RenderViewport and RenderShrinkWrappingViewport.
Checks if childrenInPaintOrder is in reverse order of a list of three children.

Does NOT test how specifying a center child modifies that order.

Checks if childrenInPaintOrder is in reverse order of childrenInHitTestOrder.

This rule is specified in the documentation of RenderViewportBase.

Checklist
Before you create this PR, confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.

Hey @Piinks! :)
Could you please give insight on the release date on stable branch for this patch ?
I need it asap on a big prod app 😳
Since the issue is in the the rendering layer, I can't just copy paste the patch 😅

Hey @Piinks! :)
Could you please give insight on the release date on stable branch for this patch ?
I need it asap on a big prod app 😳
Since the issue is in the the rendering layer, I can't just copy paste the patch 😅

Hi @BazinC I do not know the date that this change will arrive on stable.
You can read in our wiki about our release process and channels, maybe building your app on beta or dev can work in the meantime.
https://github.com/flutter/flutter/wiki/Flutter-build-release-channels
Our wiki also has some guidance on identifying where a fix is, like this one, so this may be helpful as well: https://github.com/flutter/flutter/wiki/Where's-my-Commit%3F

Hi @BazinC I do not know the date that this change will arrive on stable.
You can read in our wiki about our release process and channels, maybe building your app on beta or dev can work in the meantime.
https://github.com/flutter/flutter/wiki/Flutter-build-release-channels
Our wiki also has some guidance on identifying where a fix is, like this one, so this may be helpful as well: https://github.com/flutter/flutter/wiki/Where's-my-Commit%3F

Thanks for your answer @Piinks!
But we would not want to build our app on other branch than stable.
It's ok, I managed to copy/paste the code from this patch and create a PatchedCustomScrollView using it.
I don't like this kind of workaround but that's all I got for now.
👍 Hopefully Flutter is an open source framework
👎 Fortunately  Flutter code has migrated to null-safety, so it is no longer easy to copy/paste code in a non-null-safety project😅
I hope the fix will be released asap :)

Thanks for your answer @Piinks!
But we would not want to build our app on other branch than stable.
It's ok, I managed to copy/paste the code from this patch and create a PatchedCustomScrollView using it.
I don't like this kind of workaround but that's all I got for now.
👍 Hopefully Flutter is an open source framework
👎 Fortunately  Flutter code has migrated to null-safety, so it is no longer easy to copy/paste code in a non-null-safety project😅
I hope the fix will be released asap :)


Type argument cannot be inferred from usage

Hi
I'm trying to create a simple select query that looks like this (I'm using kysely in a Next.js project):
export const database = new Kysely<DB>({
    dialect: new PostgresDialect({
        pool: new Pool({
            host: process.env.DATABASE_HOST,
            port: process.env.DATABASE_PORT ? parseInt(process.env.DATABASE_PORT) : 5432,
            database: process.env.DATABASE_NAME,
            user: process.env.DATABASE_USER,
            password: process.env.DATABASE_PASSWORD,
        }),
    }),
});

const run = async () => {
    await database.selectFrom("table").selectAll().execute();
}

run();

The query works perfectly fine but for some reason the database.selectFrom("table") throws the following typescript error:
Type argument cannot be inferred from usage
Typescript version: 5.0.2
{
    "compilerOptions": {
        "target": "es5",
        "lib": ["dom", "dom.iterable", "esnext"],
        "allowJs": true,
        "skipLibCheck": true,
        "strict": false,
        "forceConsistentCasingInFileNames": true,
        "noEmit": true,
        "esModuleInterop": true,
        "module": "esnext",
        "moduleResolution": "node",
        "resolveJsonModule": true,
        "isolatedModules": true,
        "jsx": "preserve",
        "incremental": true,
        "paths": {
            "@/*": ["./src/*"]
        }
    },
    "ts-node": {
        "compilerOptions": {
            "module": "commonjs"
        }
    },
    "include": ["next-env.d.ts", "**/*.ts", "**/*.tsx"],
    "exclude": ["node_modules"]
}

You need to call select or selectAll. Calling selectFrom without any selections is not supported.

I changed strict: false into strict: true (It was left there for some reason :D). I also changed the target: "ES5" into target: "ES2021" but it made no difference.

You need to have strict: true in compilerOptions. Also targetting ES5 makes no sense. You need to use ES2015 or newer.

I changed strict: false into strict: true (It was left there for some reason :D). I also changed the target: "ES5" into target: "ES2021" but it made no difference.

Ok.

You need to have strict: true in compilerOptions. Also targetting ES5 makes no sense. You need to use ES2015 or newer.

I changed strict: false into strict: true (It was left there for some reason :D). I also changed the target: "ES5" into target: "ES2021" but it made no difference.

Can you provide a repository that reproduces this issue?

Closing due to staleness.


Adding tw`` to an element for the first time causes the extension to become unresponsive

See attached video (and CPU dump), basically, I'm forced to reload the window any time I try to add a new tw=`` tag to an element inside of a .tsx file.

Create a new element with the css prop, such as <span css={tw`m-0`}>
Autocomplete doesn't load, and attempting to apply all fixes hangs, forcing a window reload.
If you cancel the pending fix, save anyway, reload the window, and resume editing the new element, autocomplete becomes available and things function normally.

Does this appear to be related to this extension?
CPU dump: Arthie.vscode-tailwindcssinjs-unresponsive.cpuprofile.txt
Repro .MOV: Repro Steps.mov.zip
My extension list:
andrewmcodes.tailwindcss-extension-pack-1.0.0/
arthie.vscode-tailwindcssinjs-0.1.3/
austenc.tailwind-docs-1.3.0/
bradlc.vscode-tailwindcss-0.4.3/
christian-kohler.npm-intellisense-1.3.0/
christian-kohler.path-intellisense-2.2.1/
codeandstuff.package-json-upgrade-1.2.0/
coenraads.bracket-pair-colorizer-1.0.61/
coenraads.bracket-pair-colorizer-2-0.2.0/
davidanson.vscode-markdownlint-0.36.3/
dbaeumer.vscode-eslint-2.1.8/
deepscan.vscode-deepscan-1.10.2/
eamodio.gitlens-10.2.2/
ecmel.vscode-html-css-0.2.3/
editorconfig.editorconfig-0.15.1/
eg2.vscode-npm-script-0.3.13/
esbenp.prettier-vscode-5.5.0/
fallenmax.styled-components-extractor-0.0.6/
github.vscode-pull-request-github-0.19.0/
heybourn.headwind-1.7.0/
iocave.customize-ui-0.1.47/
iocave.monkey-patch-0.1.10/
jpoissonnier.vscode-styled-components-0.0.30/
kettanaito.nako-0.1.2/
luyizhi.vscode-graphql-0.0.4/
macieklad.tailwind-sass-syntax-1.2.1/
muhajirdev.tailwind-styled-snippets-0.2.1/
muhajirframe.tailwind-styled-snippets-0.2.1/
oderwat.indent-rainbow-7.4.0/
prisma.vscode-graphql-0.3.1/
rbbit.typescript-hero-3.0.0/
redhat.vscode-yaml-0.10.1/
richie5um2.vscode-sort-json-1.18.0/
ritwickdey.liveserver-5.6.1/
shardulm94.trailing-spaces-0.3.1/
steoates.autoimport-1.5.3/
stylelint.vscode-stylelint-0.85.0/
tht13.html-preview-vscode-0.2.5/
visualstudioexptteam.vscodeintellicode-1.2.10/
wallabyjs.quokka-vscode-1.0.315/
wwm.better-align-1.1.6/
yummygum.city-lights-icon-vsc-1.1.3/
yummygum.city-lights-theme-1.1.8/

Hello Tim, thanks for the excellent report! I was able to reproduce the issue. Downgrading Typescript from v4.x.x back to v3.x.x in the extension solved the issue.
Please try updating the extension to the latest release (v0.1.4).

@Arthie fyi your latest commit ('solving' this issue) introduces a very unpleasant UX change that took me hours to track down. See below:

To get rid of it, I had to disable the plugin. I can imagine other people encountering the same issue as me so it might be good publishing a new version reverting this change for now.

@mugoosse Thank you for taking the time to find this issue. So sorry for the inconvenience! v0.1.6 fixes both issues!

Thanks @Arthie  :)


InsertStream/UpsertStream

Hi, I upgraded the old tool to the new repo with release2.1.4. When trying to run the tool to import records from json to comsos db insert/upsert is working while insert stream and upsert stream are not doing any import(the records count still the same). I am using the CLI command and not the settings file.
Here is the command args for the tool:
string source = "json";
string sink = "cosmos-nosql";

string sourceSettingJson = "https://raw.githubusercontent.com/azure-samples/cosmos-db-migration-sample-data/main/nosql-data.json";

string targetDbConnnectionString = "AccountEndpoint=https://localhost:8081/;blabla";
string targetDatabaseName = "Database";
string targetCollectionName = "Prod8";

int retry = 2;
int batchSize = 100;

string cmdArgs =
             $"--source {source} --sink {sink} " +
             $"--SourceSettings:FilePath={sourceSettingJson} " +
             $"--SinkSettings:ConnectionString=\"{targetDbConnnectionString}\" "+
             $"--SinkSettings:Database={targetDatabaseName} " +
             $"--SinkSettings:Container={targetCollectionName} " +
             $"--SinkSettings:PartitionKeyPath=\"/partitionKey\" " +
             $"--SinkSettings:MaxRetryCount= {retry}" +
             $"--SinkSettings:BatchSize={batchSize} " +
             $"--SinkSettings:WriteMode=InsertStream";

The tool is not failing but no documents are imported when using streams, what could be wrong with this setup if the insert and upsert are working? Are the streams logic ready to use?

The -Stream modes can be slightly more performant but also less compatible. Depending on network specifics, ConnectionMode=Direct and WriteMode=InsertStream should be the fastest, but may also fail to transmit data. At the opposite end, ConnectionMode=Gateway and WriteMode=Upsert should be the most compatible set of options but also the least performant.
Outside of actual transfer performance, Stream modes also require different JSON serialization during transfer so there may be an issue specific to your data that is only causing problems with that serialization.


Bug - Exceptions occrued and hierarchy is hidden if the game object name is same as the style prefix.

Current Unity version used
I'm met on 2023.2.7.
Describe the bug
An exception occurred and objects in the hierarchy were hidden, below the game object that has the problem.
To Reproduce (If applicable)
Rename the any game object, "-", "=" or "+".
Screenshots
I have already fixed this on my fork, so no.
Additional context
Here is my fixed code.
// # HierarchyInfo.cs

private Rect GetLabelRect(Rect rect, string name, Settings settings)
{
    GUIStyle labelStyle = null;
    GUIContent labelGUI = new GUIContent();

    bool hasStyle = settings.styleData.TryGetStyleFromPrefix (name, out HierarchyStyle style);

    if (hasStyle)
    {
		int len = style.prefix.Length;
		if (len < name.Length)
		{
			labelStyle = style.style;
			name = style.FormatString(name.Substring(len + 1, name.Length - len - 1));
		}
		else
			labelStyle = EditorStyles.label;
    }
    else
    {
        labelStyle = EditorStyles.label;
    }

Hey there braces, cheers for this appreciate it.
If you'd like, can always do a PR for this, and I can accept it if you'd like if you want credit on this more 🙏
Otherwise I'll add this fix in myself, and still make sure to credit you in the version release info

Hi, okay I will try PR!

@WooshiiDev
I actually had no experience with pull requests.
So I looked into it, and it seemed difficult to submit a pull request because I had modified my forked repository quite a bit from the folder hierarchy. So I would be happy if you could fix it.
I just now learned for the first time it will show up like this if put an Issue url in the commit message,
Forget about the first code, as I think my latest modification is the most suitable.
Cheers!


[tech] FastifyDeprecation warning

Describe the bug
The API displays the following warning:
[FSTDEP017] FastifyDeprecation: You are accessing the deprecated "request.routerPath" property. Use "request.routeOptions.url" instead. Property "req.routerPath" will be removed in `fastify@5`.
To Reproduce
Steps to reproduce the behavior:

Run the carbonmark API
curl the /projects endpoint
see error in console

PR submitted here: https://github.com/denbon05/fastify-lcache/pull/32

Nice fix @biwano Do you think we should switch to @fastify/caching as the 'official' solution? They provide a simplistic cache which should serve well for our purposes (only caching a few dozen vintages or a few hundred projects at a time)
https://www.npmjs.com/package/@fastify/caching
  fastify.cache.set('hello', {hello: 'world'}, 10000, (err) => {
    if (err) return reply.send(err)
    reply.send({hello: 'world'})
  })

Migration is quick as it's only used for simple get/set invocations in 1 file
https://github.com/KlimaDAO/klimadao/blob/staging/carbonmark-api/src/utils/helpers/utils.ts#L144

Nice fix @biwano Do you think we should switch to @fastify/caching as it is maintained by the official fastify team and not a community plugin?
They provide a simplistic cache which should serve well for our purposes (only caching a few dozen vintages or a few hundred projects at a time)
https://www.npmjs.com/package/@fastify/caching
  fastify.cache.set('hello', { hello: 'world' }, 10000)

Migration is quick as it's only used for simple get/set invocations in 1 file
https://github.com/KlimaDAO/klimadao/blob/staging/carbonmark-api/src/utils/helpers/utils.ts#L144

Hello @Atmosfearful
Yes, it is a good opportunity to switch to the official package.


Multi-Select component

It would be nice if fuelux would have a multi-select component too.
Pillbox is nice (but more for tags), but often something like an improved multi-select is required in forms.
One possible candidate based on jQuery might be:
https://github.com/dio-el-claire/jquery-multiselect
with the live example: http://www.std42.ru/jquery-multiselect/
Thanks in advance.

This looks like a grouping of an  with a checkbox group.


Update flags

The flags haven't kept up with new features. They should be made current. They are useful if you just want to redeploy and by hitting the up arrow.

https://github.com/mhkeller/kestrel-cli/commit/3ca5433f0c1f72e1af04edcfeea89c379d0ae89c
https://github.com/mhkeller/kestrel-cli/commit/f7687fd96250d62b105dbd850761e2c7d8796b19
Flags also appear in the preview of promzard now, which is awesome


Make the Explore Cluster Section adjustable

On master-head commit id: 8c014ef

Navigate to Dashboard UI
On the left pane, can we make the Explore Cluster Section adjustable? As I have a large number of clusters -- 7
And to list/view them all at a time, instead of scrolling in that section, can we make it adjustable, so that user can maybe drag and make that section bigger to list all the clusters?

Added a new preference to allow the number of clusters shows to be configured.

On master-head commit id: 5d49b92

For issues around it logged a separate bug - https://github.com/rancher/dashboard/issues/3150


Clarify the challenge of word replacements in UIs in specific domains

Some of the terms on our list may have special meaning in other domains, for example:

"parent/child" has a meaning in healthcare but in finances it might be "parent/dependent". So a recommendation about the term "parent/child" shouldn't apply to the UI in those cases.

We should clarify that our terms should be considered with regards to UI, but we cannot know all the special cases, so a domain expert should be consulted when making UI language change considerations.

Hi @quaid, I'd like to work on this.
How do I get started??


why forcing using APCU?

I'm wondering why you are forcing people to use apcu?
There is lot cache system and this part can be configured. So except i you start documenting that this is mandatory and forcing for a real good reason, I think you should remove it.
thanks

We're not forcing it.

when it's a requirement in composer and you need to use composer to install it... yes your are forcing

when it's a requirement in composer and you need to use composer to install it... yes your are forcing

No we're not. You can just remove it from that file. It's designed to be editied.

Sorry, but for me it's a stupid way to act, It's just slow down adoption and installation.... but if you think it's a good way to act ...ok

I disagree. Cachet has had hundreds of installations. APCU is a perfect, light-weight caching system which is ideal for Cachet. Like @GrahamCampbell said, you can easily change it within the composer.json file.


monitoring connectivity between peers

Fix https://github.com/coreos/etcd/issues/2980
Fix https://github.com/coreos/etcd/issues/1440

pretty nice feature! I have played with it a little and it looks great.

lgtm


Selection Cell: new pattern (final name TBC)

Final name TBC.
Single select:
Proposal: button with aria-current

Multi-select:
Proposal: button with aria-pressed

Started MIND pattern documentation here: https://ebay.gitbook.io/mindpatterns/input/toggle-button-group
Still need to add working examples

Other potential names are:

Content Switcher
View Switcher

Created: https://ebay.gitbook.io/mindpatterns/input/toggle-button and https://ebay.gitbook.io/mindpatterns/input/toggle-button-group


Converting a fraction of one BTC to USD seems not supported

Converting a fraction of 1BTC to USD got same results as the one of converting 1BTC.
For instance, the following conversions and results were performed on 2023-07-05:
ex := exchange.New("BTC")
result, err := ex.ConvertTo("USD", 0.000330) //  30147.44. Not correct. Expected result should be around 9.95 (USD).
result, err := ex.ConvertTo("USD", 1)             // 30147.44. Correct.
result, err := ex.ConvertTo("USD", 1.5)          // 46074.39. Correct.

What are the proper method and snippet to convert a fraction of one BTC to USD?

hey sorry about the problem, im trying to find the source and it seems to be a problem in the API, can you confirm this?
https://api.exchangerate.host/convert?from=BTC&to=USD&amount=0.000330
i think it doesn't support this low of a number which in the case of crypto makes it unusable :/

Thanks for the response. I ran the above URL to convert 0.00033BTC to USD and confirmed returning null result. But I tried other fractions and found that  converting any fraction greater than or equal 0.015 BTC worked. But converting fraction smaller than 0.015 BTC returned null result.

oh thats a bummer, sadly I'm not in control over that API just this client.
Here is the repo for the API it may have more info or you can make an issue l there
https://github.com/Formicka/exchangerate.host


[patch] Target a particular tag of the actions repo

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
6

Relevant Lines:
6

💛  - Coveralls


verifier should set errorreturn code for inconsistent nodes [phab:T177]

Created by pludemann at 2016-10-18 17:10:49:
The Kythe entries verifier outputs a message Two facts about a node differed in value:, but doesn't set the return code to non-zero. It would be nice if the return code were non-zero so that I could automate some tests.
The message Could not verify all goals. The furthest we reached was: does set a non-zero return code. (It also stops at that point, which is a bit annoying, but I suppose makes a certain amount of sense.)

pludemann wrote at 2016-11-25 20:02:52:
It appears that the verifier now gives a non-zero return code for two facts differing. The code didn't seem to have changed around where the error message was generated, so perhaps there was a change in how the error code is propagated?

salguarnieri wrote at 2016-12-05 19:03:30:
Since your last comment seems to indicate that this is working as intended, I'm closing the ticket.


Add documentation for Azure Support

This issue covers the step by step guide and is merged.  There is a separate issue for the (only) Azure guide.


Do not display the "Reserve DOI" button for resources with a publishing organization without an affiliation with DataCite.

Functionality should be hidden if:

[x] There is no organizations associated with DataCite
[x] Current user has no registration rights
[x] Current publishing organization is not set or does not have DataCite association

@MattBlissett @CecSve did I miss anything?

Sorry that I am late in following up. Just to make sure I understand.
Would the functionality be hidden if (keep in mind the original question was coming from a publisher on a cloud IPT:

one or more organizations associated with the IPT has a DataCite account
the user has registration rights
the user (wrongly) choose the publishing organization with the DataCite association

@ymgan does this sound like the scenario you were highlighting, please?

Users without registration rights won't see the DOI button.

Yes exactly as you highlighted @CecSve , thank you!! I can share the screenshot if it is clearer! The scenario I hope that this bug fix could prevent is: Reserved DOI button should be hidden from organization without DataCite account.

The Italian National Antarctic Museum does not have an associated DataCite account, so the "Resolve DOI" button should be hidden in the screenshot below.

Ok thank you both - that seems to be resolved in the next version then.

The 3.0.2 already deployed to cloud.gbif.org


CUDA support

Whisper.cpp now supports CUDA. While it's really annoying to setup CUDA and its drivers, it should give a nice performance boost for Windows and Linux PCs with Nvidia cards.
I want to make it optional. Probably best way to do that is to compile one .dll with CUDA support and one without. I would need to figure out how to switch between them in runtime.
Also it will probably be smart to add fallback to CPU when no CUDA available.

This would be awesome. Any updates on this enhancement? What would be the use case for switching during runtime.

Any updates on this enhancement?

I don't have PC with CUDA supported GPU. But in theory, you would need to just recompile dlls with this flag and replace it in plugin folder.

What would be the use case for switching during runtime.

I want to make CUDA optional, because this is a very niche use case. Most people will use CPU inference. If someone without installed CUDA will try to run CUDA compiled dlls - it won't fallback to CPU inference and may crash (I guess?).
Therefore I need to figure out how to ship plugin with CPU inference and optional CUDA inference. This is what I mean by switch in runtime, because Unity usually load all dlls at start.

Got it that makes sense. Yeah I might try and recompile the dlls with the -j flag and see what happens in Unity.

Done, though no fallback to CPU yet. Hope whisper.cpp gang will add it in future.

I have pytorch installed, do I need to install CUDA separately?
On the other note, I am little bit struggling to enable CUDA. I read your guidance, unfortunately, the instructions are not clear to me. Can you enlighten me more?

I have pytorch installed, do I need to install CUDA separately?

You would need to install CUDA Toolkit 12.2.0 (download it here). Select your platform and follow instructions. Pytorch isn't relevant.

On the other note, I am little bit struggling to enable CUDA. I read your guidance, unfortunately, the instructions are not clear to me. Can you enlighten me more?

It should be very straightforward. Open your Unity project and find Project Settings window. Next select Whisper category and click enable CUDA.


output folder naming doesn't account for daylight saving and is now 1 hour out

something like this should work
>>> import pytz
>>> import datetime
>>> bst = pytz.timezone('Europe/London')
>>> now = datetime.datetime.now()
>>> bst.localize(now).strftime("%y%m%d_%H%M")
'230503_1445'


Set reporter._show_progress_info = False

This works around https://github.com/browsertron/pytest-parallel/issues/46.
It also seems to be good in general, since this also mangles the output
(and appears to be off (far behind in percentage) for when it appears etc).

@kevlened
Would be great to get this merged for CI to become green again (for the other PRs).
Note that pytest master has the workaround in pytest already (https://github.com/pytest-dev/pytest/pull/6261), so testing with that would also be an option - maybe also good in general to catch issues early?

Thanks for all the PRs. Monkey-patching isn't very maintainable, so I appreciate the support.
re: pytest-dev/pytest#6261 - I'll get these PRs in first


ENHANCEMENT: has_extension() should allow injector overrides

DataObject::has_extension() allows subclasses, but not overrides.
MyObj::has_extension(Versioned::class) // true
MyObj::has_extension(VersionedSubclass::class) // true
SilverStripe\Core\Injector\Injector:
  SilverStripe\Versioned\Versioned:
    class: MyProject\MyVersionedSubclass

MyObj::has_extension(VersionedSubclass::class) // false

Have fixed the tests and added the strict check.

Can't figure out what's wrong with the phpcs test. That comes back clean in my local copy (same version), and the file itself looks fine. The changes it's suggesting would break the formatting. Any suggestions would be appreciated.


✨ add fallbackLocale & includeLocaleIds

Description
DO NOT MERGE TILL https://github.com/Shopify/quilt/pull/1188 is merged
Fixes https://github.com/Shopify/quilt/issues/1170
Some more details here: https://github.com/Shopify/quilt/pull/1173#issuecomment-556905273

Type of change

[ ]  Patch: Bug/ Documentation fix (non-breaking change which fixes an issue or adds documentation)
[x]  Minor: New feature (non-breaking change which adds functionality)
[ ]  Major: Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

[ ] I have added a changelog entry, prefixed by the type of change noted above

🎩
Using this branch as consumer:
https://github.com/Shopify/web-proving-ground/pull/40

in quilt yarn tophat react-i18n ../../web-proving-ground

run dev up && dev s

see dev mode using the original babel plugin

check build/server/main.js and see if (["fr"].indexOf(locale) < 0) { in the translation function of CustomerList

run yarn build

see prod mode using the index generation

check /build/server/main.js and see _translations__WEBPACK_IMPORTED_MODULE_0__[/* default */ "a"].indexOf(locale) < 0 in the translation function of CustomerList

accidentally close, reopen https://github.com/Shopify/quilt/pull/1191


Changing material parameters does not update material for all tiles

When changing the parameters of a 3D Tileset's material, tiles won't update to reflect these changes uniformly.
Here's a visualization - a single simple material was assigned to this tileset, and the color of the material was changed three times. Only some tiles reflect the changes. It seems like only new tiles will change, while tiles that are already loaded do not update.

Note that using the "Refresh Tileset" button does not resolve this.
Steps to reproduce:

Create a new material using the Universal Render Pipeline/Lit shader.
Assign the new material to a 3D Tileset, like Cesium World Terrain.
Select the material and change the Base Color to a different color.
Move the camera.

Side note - At first glance, I'd assume this is happening because each tile uses its own instance of the material, and I don't think material instances phone home to the original material in the same way that Unreal materials do. Possibly, the best solution here would be to use MaterialPropertyBlocks for all 3D Tileset materials in order to update all instances at once.


[BUG] host_config.json writes out to the current directory, not where the release is installed

Describe the bug
Before mix releases, the host_config.json file would write to the directory where the release is installed.  This results in JSON configuration files strewn about in the filesystem. This is especially frustrating with git since these files can alter host behavior and shouldn't be committed alongside other changes.
To Reproduce
Steps to reproduce the behavior:

Start host
See host_config.json

Expected behavior
I expect the host_config.json file to write out where the host release is unpacked, or with a configurable environment variable (perhaps an optional variable to avoid writing the file at all would be useful too)
Environment (please complete the following information)

OS: [e.g. Linux, MacOS, Windows] Linux/Macos, not tested on Windows
Arch: [e.g. x86_64, aarch64] aarch64
wasmCloud Version: [e.g. 0.51.0] 0.57.0+

Hi, I was checking out the good first issues to get familiar with the codebase. Can I take hold of this one?
After a quick look, I think the culprit is line 225 in host_core.ex, it seems like there is a double write_json. One in the current directory and one for .wash/host_config.json
  defp write_config(config) do
    write_json(config, @host_config_file)   <--- This must be writing in $(pwd)

    case System.user_home() do
      nil ->
        Logger.warn(
          "Can't write ~/.wash/#{@host_config_file}: could not determine user's home directory."
        )

      h ->
        write_json(config, Path.join([h, "/.wash/", @host_config_file]))
    end
  end


FIRInstallationsAuthTokenResult - add installationID property

Draft change for API review

Decided not to implement (see go/fis-sdk-api-fid-in-token-result for details).


Statsbeat do not count failed request when throttle

@hectorhdzg here is breeze's definition of these 2 response codes:
 **429 - Too many requests**
  The client send too many telemetries to the endpoint within in the given time interval so it is throttled

* **439 - Too many requests and refresh cache**
  The client is throttled at one point so frequently that the endpoint would send it 439 status code saying that it is throttled and should not retry for a long time.
Then, the client would drop the current data from its buffers.

@hectorhdzg let's chat tomorrow if we want to send two count to breeze.. daily quota exceeded vs. monthly quota exceeded.


Csprojes are not indexed properly [source.dot.net]

I've noticed significant differences between csprojes in source.dot.net and in the dotnet repositories.
Some examples:
https://source.dot.net/#Microsoft.AspNetCore.Authentication.JwtBearer/Microsoft.AspNetCore.Authentication.JwtBearer.csproj
https://source.dot.net/#System.Linq/System.Linq.csproj
All cspject still have netcoreapp2.1 as TFM.

I think this is by design. The .csproj files are artificially cleaned up and generated to ensure the indexing process can run in isolation from the actual build. @alexperovich can confirm, he wrote source.dot.net.

Yes, this is by design. The projects and build system in all the dotnet repos are different enough and don't always have synchronized versions of dependencies that the indexer can't load all of them in the same process. I generate csprojs based on the csc commands found in the binlog to ensure that the compiler gets all the proper inputs for the indexing process.


Remove unnecessary text at gyazo element.

fix #105
it is  success to remove textContent from script and style tag.
But it's not enough at google's result page...

It seems difficult to remove meaningless words from captured HTML textContent.

idea to remove invisible text
Array.from(target.querySelectorAll('*')).forEach((elm) => {
  if (elm.tagName === 'script' || elm.tagName === 'style') {
    return elm.remove()
  }
  if (getComputedStyle(elm).display === 'none' || getComputedStyle(elm).visibility === 'hidden')
    return elm.remove()
})

idea to remove invisible text

it sounds good!! i'm trying!

fixed point

Mark up all hidden DOM nodes before generate image desc.
Restore marked classes after generate image desc.

LGTM


[UI] apply final design to home page

ceremony view is not yet ready. leave that away for now

@brenzi where can I find Didiers design (I only have https://www.figma.com/proto/R4O8yy8PyUMgYbS57SCdLw/Encointer?node-id=1%3A15&starting-point-node-id=1%3A7
)

Please mention here the issues that you close, like #302


kapp public API should not hide error types

What steps did you take:
As a codebase which uses the public kapp go API, we call the public RecordedApp.UsedGVs() which in turn calls the private RecordedApp.meta() which in turn uses the k8s API to fetch the config map:
https://github.com/vmware-tanzu/carvel-kapp/blob/3486b0b61553e4cd1ebfc2fed3f95d81c1f65d10/pkg/kapp/app/recorded_app.go#L281-L287
So the code neatly checks if the error is a NotFound error, but rather than wrapping the actual error (which would not lose information), it returns a new error which loses this information, which is returned up the chain to the response to the public UsedGVs() call.
What happened:
In our code, (before digging into the kapp source) I tried to do:
if errors.IsNotFound(err) { ... }

which will happily work with wrapped errors. But because the returned error is not a wrapped NotFound, it obviously does not detect it.
What did you expect:
To be able to use errors.IsNotFound() on the returned error. Instead, we need to do some string matching:
if strings.Contains(err.Error(), fmt.Sprintf("configmaps %q not found", appName)) { ... }

Anything else you would like to add:
Guessing this code was just written before the go errors api was released, so it may be similar throughout, but should be simple enough to change to use %w instead of %s in those error strings.

Hi @absoludity,
Thanks for raising this. We will be putting in our prioritised backlog and will try to address it ASAP. In the meanwhile, if you are willing to raise a PR for this we will be happy to review it on priority.


Custom locale corrupts matrixdynamic contents.

I have noticed that setting custom locale deletes all the contents from certain matrixdynamic. Please take a look at this plunk.
Notice that it restores previous answers from var data.
On lines 216 and 217 there is a locale customization:
//model.locale = "es";
//model.render();
If you uncomment those and re-run the survey, you'll see the contents of the first matrixdynamic disappear.

@Kuhax As a workaround, please set rowCount from 0 to 2 and it will start to work:
http://plnkr.co/edit/PXgfBDT4mzHvhapDxlbs?p=preview
I found a unit test ("Matrixdynamic adjust rowCount on setting the value") that adjust increase the rowCount on setting value when necessary, for some reason it doesn't work with different locale. I will try to find the reason.
Thank you,
Andrew

@Kuhax  I have found the reason why it doesn't work and fix it.
Thank you,
Andrew


[4876] Fix incorrectly awarded trainees

Context
Data migration to fix the states of some trainees that were accidentally awarded. We synced their state with DQT but accidentally picked up their EYTS records. These trainees are awarded EYTS and not QTS. So setting them back to trn_received.
Changes proposed in this pull request

Add data migration

Guidance to review

I ran this locally but updating trainees to have these TRNs. It ran ok and they updated successfully.
Will test in production data

Important business

[ ] Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?
[ ] Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?

NB: Please notify the #twd_data_insights team and ask for a review if new fields are being added to analytics.yml

 Revert QTS-awarded trainees back to 'in training'


run/h2c: missing repo directory in cd command

Hi!
I might find a wrong command in the blog.
I think cd ./run/h2c below should be cd ./golang-samples/run/h2c.
git clone https://github.com/GoogleCloudPlatform/golang-samples.git
cd ./run/h2c
gcloud beta run deploy http2-test --use-http2 --source=.

(https://cloud.google.com/blog/products/serverless/cloud-run-gets-websockets-http-2-and-grpc-bidirectional-streams)
I am not sure this issue is a proper place to report blog issues.

Thanks for fixing the doc. Is the command in the blog https://cloud.google.com/blog/products/serverless/cloud-run-gets-websockets-http-2-and-grpc-bidirectional-streams fixing also? I meant to report the command in the blog in the issue.

Thanks for fixing the doc. Is the command in the blog https://cloud.google.com/blog/products/serverless/cloud-run-gets-websockets-http-2-and-grpc-bidirectional-streams fixing also? I meant to report the command in the blog in the issue.


Support for elasticsearch pipeline

This pull request is to add support for elasticsearch pipelines.
Associated issues

#53
add a pipeline for elasticsearch #731
Ingest node #607

Implementation
In this build there is a new class named PipelineConfig. This class will validate if each pipeline configured exists and expose a method to get the corresponding pipeline to the given namespace using fnmatch.
To improve the performance of the PipelineConfig.get_pipeline method, I've added a simple cache to keep the corresponding pipelines of each namespace.
Set elasticsearch dependency from 5.0.0 to 5.4.0
setup.py:
extras_require={
  ....
  'elastic5': ['elasticsearch>=5.4.0,<6.0.0']
},

Since elastic2_doc_manager is using the helper functions of elasticsearch, the only way to guarantee the pipeline feature, without extra validation or refactoring, is to force the elasticsearch driver version to be above or equal to 5.4.0.
If the driver is bellow 5.4.0 the pipeline property in the document action will be ignored.
elasticsearch 5.4.0 changelog
Configuration example
{
  "docManager": "elastic2_doc_manager",
  "targetURL": "localhost:9200",
  "bulkSize": 1000,
  "uniqueKey": "_id",
  "args": {
    "pipelines": {
      "app.user": "custom-pipeline-1",
      "app.*": "custom-pipeline-2"
    }
  }
}

I share the need for this feature, any news on the merge?

Hey, any idea when this pipeline support will be merged?

Hi,
Nice work! I am trying to use it however, I do not understand where should I put the config you mention
{
 "docManager": "elastic2_doc_manager",
 "targetURL": "localhost:9200",
 "bulkSize": 1000,
 "uniqueKey": "_id",
 "args": {
   "pipelines": {
     "app.user": "custom-pipeline-1",
     "app.*": "custom-pipeline-2"
   }
 }
}

Thankyou

ok....got it "-c config.json"

Apologies for letting this linger for so long. Please feel free to resolve the conflicts and resubmit.


Ensures auth_data[provider] gets deleted from document

Before that PR, the auth_data{provider} would stay in the DB as  = null, this proposed PR actually performs and $unset operation on the column to ensure a better consistency.

@Scoup if you could try this branch and let me know if the bug keeps occurring

Current coverage is 86.15%

Merging #1081 into master will decrease coverage by -5.33% as of 4c21940

@@            master   #1081   diff @@
======================================
  Files           75      76     +1
  Stmts         4546    5120   +574
  Branches       910    1111   +201
  Methods          0       0
======================================
+ Hit           4159    4411   +252
- Partial         11      13     +2
- Missed         376     696   +320

Review entire Coverage Diff as of 4c21940

Powered by Codecov. Updated on successful CI builds.


Displaying list of events (dots) on dates (using Parse online datastore)

I'm getting a list of objects from Parse database, which contains int day, int month and int year. How can I use those three data to show dots in the calendar?

@myatthiha You can use CalendarDay.from() method. You can do this:
ArrayList dates = new ArrayList<>();
for (ParseDateObject date : list) {
dates.add(CalendarDay.from(date.year, date.month, date.day));
}
widget.addDecorator(new EventDecorator(Color.RED, dates));
Closing this issue, feel free to reopen if you want more info.