Please build vespaengine/vespa-dev-centos-stream8 as multi-arch, since aarch64 packages are available

Is your feature request related to a problem? Please describe.
Development on M1 mac is quite slow; the stream8 variant has both amd and arm variants of the rpms (x86_64/aarch64), and docker buildx supports creating multi-arch images.
E.g., this works on my mac to push to local registry, which I can then pull and work with:
$ docker buildx build --platform linux/arm64,linux/amd64 -f Dockerfile.stream8 -t localhost:5001/vespaengine/vespa-dev-centos-stream8 --build-arg VESPA_VERSION=7.571.25 . --push

Describe the solution you'd like
The result I would like (which influences solution) is to have https://hub.docker.com/r/vespaengine/vespa-dev-centos-stream8 be multi-arch so that pulling from M1 mac gets a native aarch64 docker image.
Describe alternatives you've considered
To unblock myself, I've just built from the Dockerfile directly as above (but slimmed down to the following), but pulling the pre-built, official image from docker hub would be nicer and easier for others.
$ docker build -f Dockerfile.stream8 -t vespaengine/vespa-dev-centos-stream8 --build-arg VESPA_VERSION=7.571.25 .

Additional context
Add any other context or screenshots about the feature request here.

The vespaengine/vespa-dev-centos-stream8:latest is now available as multi-arch:
$ docker manifest inspect vespaengine/vespa-dev-centos-stream8:latest
{
   "schemaVersion": 2,
   "mediaType": "application/vnd.docker.distribution.manifest.list.v2+json",
   "manifests": [
      {
         "mediaType": "application/vnd.docker.distribution.manifest.v2+json",
         "size": 1800,
         "digest": "sha256:527321e3d92af84aadac6c5a51a82f3dae0fac9a7f34dfc6655b25b720543f72",
         "platform": {
            "architecture": "amd64",
            "os": "linux"
         }
      },
      {
         "mediaType": "application/vnd.docker.distribution.manifest.v2+json",
         "size": 1800,
         "digest": "sha256:45ff3acab69d229d0d97e5a93af48117f2165f63a4acec55e9074ba73fe7dd54",
         "platform": {
            "architecture": "arm64",
            "os": "linux"
         }
      }
   ]
}

Please note that the support of CentOS Stream 8 is not available before Vespa 8 which is planned released this summer and these images can be used for preview and development.


Wifi not available on battery ?

Hello,
first of all thank you for making the OpenCO2 sensor.
Quick question related to the wifi, as it is only available while on power and not on battery.
Is this because of energy consumption ?
Also, while on power the temperature is messed up and not accurate anymore, probably by the heat generated by the charging battery.
Do you plan to allow wifi on battery in the future ?

Hey @YannDanthu  thanks for reaching out.
The current Wi-Fi implementation is only active if power is connected. You are correct this is for energy saving reasons. There are no hardware limitations that limit the activation on Battery, but a good energy saving implementation like ESP-NOW would be recommended.
There is one esphome implementation by Yourigh available with some limitations (no LED / Menu / limited Fonts)
https://github.com/Yourigh/OpenCO2_Sensor_esphome/tree/main
Other then that my main focus until now was not on enabling WiFi on Battery. This is because the use case was to use the OpenCO2 Sensor on Battery while being on the go and plugging it back in when coming Home, automatically enabling WiFi.
While charging elevated temperatures are expected because of the used TP4065 which lowers the 5V USB Voltage to the current Battery voltage via dissipating heat. Charging from empty to full should be completed after less then 1:50 h.
Hope this helps,
David

thanks for the answer.
One last question, I didn't dig into it yet but would it be easy to publish the metrics to an endpoint from the CO2Sensor (instead of pulling from another device)


Add update in enrollment

Add method to update a enrollment.
Add tests to create and update a enrollment.

Maybe update README?
There is a few examples to use the gem.

A use case inside the Readme would be really nice 👍


Add pnpify --sdk vscode to renovate postUpgradeTasks conf

See: https://docs.renovatebot.com/configuration-options/#postupgradetasks

Seems like this may only be possible via self hosted instances of renovate ??

Tuns out you can run the renovate command on your machine locally quite easily so this is doable


libwacom: 1.1 -> 1.3

Motivation for this change
Updating libwacom from 1.1 to 1.4, for updated tablet definitions.  Added doxygen and cmake to get compilation to work.
I am opening this PR in advance of actually seeing if it breaks anything, because many pkgs depend on this.  This also seems to have no maintainer set, and I'm a bit uncomfortable adding myself as a maintainer for such a core package.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

With the number of rebuilds, this should go against staging, not master; mind rebasing and re-targeting the PR?

@worldofpeace should we include this into the scope of freedesktop team?

Changes look okay:

https://github.com/linuxwacom/libwacom/releases/tag/libwacom-1.3
https://github.com/linuxwacom/libwacom/releases/tag/libwacom-1.2

@worldofpeace should we include this into the scope of freedesktop team?

I believe any member of the freedesktop team should be very familiar/capable of overseeing libwacom. Like, I'd do the same for libinput at least. Though it does make me question ownership/maintainer-ship a bit deeper here.


[Parent] 0.19.0 Release content for blog, social media and slack

What we need for the blog and/or social media and Slack channels:

[x] Migration guide documentation for migration to the breaking release (content created for https://github.com/kedro-org/kedro/issues/1887 such as a migration guide in the release notes). <-- covered separately as a docs ticket
[x] Blog post about "What's new in 0.19.0" which will cover how to migrate at high level (pointing to further info) plus high level features <--to publish on 15/01/24
[x] Blog post specifically about add-on flow in 0.19, since this is a big change. Tracked by https://github.com/kedro-org/kedro-devrel/issues/116

We will also need a range of graphics (built using the marketing template) and QR codes.
Not yet considered: Will there be any website content updates?

I think this is done as there are other tickets tracking the separate activities above. Closing.


Make ZetaClient logging and metrics clear, informative, and easy to read for monitoring

Creating a new EPIC to make this one focused on code structure and testing: https://github.com/zeta-chain/node/issues/1611
This EPIC aims at improve the general logging and metrics of the ZetaClient
### Tasks
- [ ] https://github.com/zeta-chain/node/issues/1869
- [ ] https://github.com/zeta-chain/node/issues/1622
- [ ] https://github.com/zeta-chain/node/issues/1495
- [ ] https://github.com/zeta-chain/node/issues/2031

Improvement Examples here - https://github.com/zeta-chain/node/issues/1902


fix: updated notation artifact type to application/vnd.cncf.notary.signature

This PR changes the notation artifact type from application/vnd.cncf.notary.v2.signature to application/vnd.cncf.notary.signature.

Codecov Report

Merging #231 (af47ad8) into main (180ad99) will not change coverage.
 Consider uploading reports for the commit 146ffbb to get more accurate results

@@           Coverage Diff           @@
##             main     #231   +/-   ##
=======================================
  Coverage   73.55%   73.55%
=======================================
  Files          23       23
  Lines        1732     1732
=======================================
  Hits         1274     1274
  Misses        366      366
  Partials       92       92

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Switch the groupid from org.brianmckenna.wartremover to org.wartremover

As the title says, this PR switches the groupid and all package paths to the use org.wartremover. This is the one big breaking change that needs to be made before the 1.0.0 release.

This comes as a big relief to me personally since I always mistype Brian's name when suppressing warts :P


Host_WindowLoadTimeout

I saw on reddit AgentRG is working this, but since I don't have reddit I thought I'd make an issue. I'm on a 13 in 2020 MacBook with an Intel i5 running macOS Ventura 13.0. I ran the installer and received the Host_WindowLoadTimeout error. Here is my host.developer.log file and my terminal output. Let me know if there's any way I can help. Apologies if the formatting is bad, I haven't really used GitHub before.
terminalOutput.txt

I exist in both places. Yea, I am fully aware of this error, as is everyone who works on this project. The only 100% reliable fix is installing the game through Steam, but for that I have to assume everyone has a Steam account. So for now, I'm trying other options that does not include Steam.

Quick note, thanks for the logs, but I suggest removing the host log from your comment. Machine ID isn't redacted.

Thanks!

Is there anyway I can contribute to this project?

The project is open source, so anything you think can be improved on would be welcome.

Yea, I heard that it works through Steam, but I go by the principal to involve the user as least as possible. If I have them install Steam, I have to assume they have a Steam account.

I've gotten this working on linux at least by using this bottles config: https://www.reddit.com/r/linux_gaming/comments/16pw37r/comment/k38xukv/?utm_source=share&utm_medium=web2x&context=3. Will leave this here in case a dependency/override might be helpful for fixing this issue. I still intermittently get a timeout error, but this at least gets it to launch most of the time.
I wrote this for the Jagex Launcher, and it might work here because they use the same installer technology.

@nmlynch94 Thanks for sharing this, I'll take a look at this when time permits.


Advertise announced prefixes ONLY to certain peers

I would just like to make a request that Pathvector be able to pick and choose which peers that advertised routes go to. This is the one feature thats missing that is preventing me from utilizing pathvector.

90efe33


[BD-46] fix: fixed Card border-radius and color

Description

fixed Card border-radius and color.

Issue: https://github.com/openedx/paragon/issues/2549
Deploy Preview
Include a direct link to your changes in this PR's deploy preview here (e.g., a specific component page).
Merge Checklist

[ ] If your update includes visual changes, have they been reviewed by a designer? Send them a link to the Netlify deploy preview, if applicable.
[x] Does your change adhere to the documented style conventions?
[ ] Do any prop types have missing descriptions in the Props API tables in the documentation site (check deploy preview)?
[x] Were your changes tested using all available themes (see theme switcher in the header of the deploy preview, under the "Settings" icon)?
[ ] Were your changes tested in the example app?
[ ] Is there adequate test coverage for your changes?
[ ] Consider whether this change needs to reviewed/QA'ed for accessibility (a11y). If so, please add wittjeff and adamstankiewicz as reviewers on this PR.

Post-merge Checklist

[ ] Verify your changes were released to NPM at the expected version.
[ ] If you'd like, share your contribution in #show-and-tell.
[ ] 🎉 🙌 Celebrate! Thanks for your contribution.

Thanks for the pull request, @PKulkoRaccoonGang!
When this pull request is ready, tag your edX technical lead.

@PKulkoRaccoonGang 🎉 Your pull request was merged!  Please take a moment  to answer a two question survey so we can improve your experience in the future.

:tada: This PR is included in version 21.1.8 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 22.0.0-alpha.6 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add more video info to Model classes.

Stuff like duration, tags, etc. Basically, parse the JSON from the -j switch output.

Closed with c3deceb


Move all cookies (delete, get) from STDcheck to FlexibleMink

Various methods were added to STDcheck's WebContext instead of adding them to FlexibleMink. This is one of many PRs that will move these methods so that all projects using FlexibleMink can utilize them.

Testing file changes were not found.
Paging @johnnie502 to review the back-end work in this pull request!
Paging @PrinceMegahit to review the testing work in this pull request!


Urho3D

SteamDB app page links to a few games using this
https://steamdb.info/app/375220/
https://steamdb.info/app/1203310/
https://steamdb.info/app/1213450/
https://steamdb.info/app/927840/
https://steamdb.info/app/793620/
https://steamdb.info/app/450360/
https://steamdb.info/app/694610/
Brief explanation of the change
Add Urho3D engine. Find the shader Urho2D.glsl.

Pull this.

After running changed rules in this pull request (this bot only tests rules.ini changes):
Changed technologies: Engine.Urho
These apps now match: (8)<PHONE_NUMBER>10<PHONE_NUMBER>30<PHONE_NUMBER>310 1204550 1213450
Please confirm that all these changed apps are correct.

not used by enough apps


[move-only] When adding implicit liveness uses for class field/global accesses, do not use the terminator, use the end access.

Previously, when doing this I could just use the terminator both in cases of
inout and for class field/global accesses... but after some recent changes to
field sensitive pruned liveness, this seems to no longer work. So just do the
right thing and use the field access.
The specific bug was that we would introduce a destroy_addr along one of the
paths where the value wasn't defined resulting in a dominance error.
I added a SIL test that shows this fixes the issue, a swift test, and also an
Interpreter test that validates the correctness.
rdar://111659649
(cherry picked from commit 29ca77778314736ff224c70fee3412cd0276965a)

Just preparing the testing. Going to do the CCC later.

@swift-ci test

but after some recent changes to field sensitive pruned liveness, this seems to no longer work.

I tried the new @assignableButNotConsumableEndAccessImplicitLifetimeTest test case with the compiler before my recent changes to FieldSensitivePrunedLiveness (git checkout 606eef619f2d2bc6b22e1f625461cbfe779617ca +  ./utils/update-checkout --scheme=main --match-timestamp) and see this verification failure
SIL verification failed: instruction isn't dominated by its operand: properlyDominates(valueI, I)
Verifying instruction:
     %7 = begin_access [modify] [dynamic] %6 : $*NonTrivialStruct // users: %10, %5, %8, %12
->   destroy_addr %7 : $*NonTrivialStruct         // id: %5
In function:
// assignableButNotConsumableEndAccessImplicitLifetimeTest

@nate-chandler Ok, I could be wrong. My assumption was that it was something related to maximizing lifetime or field sensitive pruned liveness. Regardless, this needs to be fixed.

@swift-ci test

As part of testing this, I discovered another issue in DI that is tied a little to this, so I am going to CCC them together.

@swift-ci test macOS platform


Memory / free memory is displayed correctly now

I removed swap from the output of free command and changed two lines to display the right results. There were no memory information at my device before. It's an universal solution.

This actually just reverts #95, which is bad IMO (see http://www.linuxatemyram.com/)
What is the output of 'free' on your distribution?

pizero $ ~  LC_ALL=C free
free              total        used        free      shared  buff/cache   available
Mem:         442804       26876      325396         388       90532      401092
Swap:             0           0           0

The output on a Pi2 running Raspbian (Wheezy):
pi@raspberrypi2 ~ $ LC_ALL=C free
             total       used       free     shared    buffers     cached
Mem:        949328     900484      48844          0     437944     412264
-/+ buffers/cache:      50276     899052
Swap:       102396          4     102392

If u find a way to support both formats (and using values without/with buffer/cache), I'm happy to merge it!

Well, I'm thinking of a naive solution. Before free ... is executed we need to execute uname -a to check the system version. If Arch Linux is detected just parse the data differently. It's a strange solution so I will think about a better one.


Out of bounds index in examples/wlan/src/mapper.rs

In mapper.rs:201, self.index may be an out of bounds index on the POLARITY array.
thread 'smol-10' panicked at src/mapper.rs:115:25:
index out of bounds: the len is 127 but the index is 127
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Reproducing
// in loopback.rs
rt.spawn_background(async move {
    loop {
        Timer::after(Duration::from_secs_f32(0.8)).await;
        // generate a payload with random size
        let payload = vec![0x41; rand::thread_rng().gen_range(0..wlan::MAX_PAYLOAD_SIZE)];
        println!("payload len: {}", payload.len());
        handle.call(0, 0, Pmt::Blob(payload)).await.unwrap();
    }
});

Is this a bug in mapper.rs? Or is there an additional precondition I must meet when generating my payload in loopback.rs?

Thanks for opening the issue. Looks like it was missing a modulo operation for indexing the pilot polarity array.


accessdev running slow

resolution_fixed   |    by<EMAIL_ADDRESS>
I've noticed that accessdev is running very slowly today and taking ages to do simple operations.
This could be due to memory: free tells me that it's used 7504 out of 7870 MB, and there is also no swap space available.  Although without including buffers/cache this is only 2958 MB so maybe that isn't the problem.
Otherwise, could it be the nfs filesystem is just not handling having more people accessing it at the same time?

Issue migrated from trac:70 at 2024-01-31 17:58:57 +1100

<EMAIL_ADDRESS>changed status from assigned to closed

<EMAIL_ADDRESS>changed owner from `` to cma900

<EMAIL_ADDRESS>commented

I note chris's comment in the nci ticket [nf.nci.org.au #67377]
[cma900 - Fri Jan 31 08:37:19 2014]:
Hi guys:
The server that provides home directories on "accessdev" is currently being impacted by I/O delays on it's virtual disks. This would certainly affect the time taken to login via ssh, plus also svn depending on where your working files are.  Moreover, "access-svn" uses the same storage infrastructure as "os-home" so it might be impacted too.
Regards,
Chris

<EMAIL_ADDRESS>changed status from new to assigned

<EMAIL_ADDRESS>removed milestone (was accessdev v1.0)

<EMAIL_ADDRESS>commented

This appears to be OK now.

<EMAIL_ADDRESS>changed resolution from `` to fixed

<EMAIL_ADDRESS>commented

Milestone accessdev v1.0 deleted


Enable use of be_computed_by_function in Opal

Mspec’s use of String#<< in BeComputedByFunctionMatcher makes it
impossible to use  be_computed_by_function in Opal because Opal does
not support mutable strings. Fortunately String#<< is completely
unnecessary in this case, and the same effect can be accomplished by
string concatenation using String#+.

I need this to run https://github.com/opal/rubyspec/blob/master/core/kernel/sprintf_spec.rb#L94 as part of my on-going Kernel#format rewrite...

The same PR should be sent to the upstream mspec repo too, @brixen already told me this kind of changes is welcome.

@elia pardon my ignorance, what's the "right" way to go about doing that? I mean without me forking the upstream mspec repo, making the change again, and submitting another PR?

@elia, also, I have no idea how to update Opal to make it use the latest mspec (i.e. with this change applied). Is there anything I need to do in opal/opal and/or opal/rubyspec to make sure CI runs with this change to mspec? Sorry, I still need a little "hand-holding" to get things done :)

Well, yeah, just checked and we're using the official mspec repo: https://github.com/opal/opal/blob/master/Gemfile#L27
This fork was intended as the place where you can push and then send the PR to rubyspec/mspec, but first needs to be updated.

@elia looks like you made some changes to mspec that have not been contributed upstream as well :) https://github.com/rubyspec/mspec/compare/master...opal:master

@elia, if so, are those changes really doing anything for us? I mean if Opal is using the official mspec repo, then commits to opal/mspec have no effect whatsoever, right?

IIRC I issued a couple PRs to allow Opal to run with the bin/mspec command, so they're not fundamental as we have custom runners for mspec anyway.
I forked in the Opal org just to signal they were kinda "official" PRs and also for those moments where we had a pending PR that was blocking the build.

@elia ok, here is what I did: I created a new branch in opal/mspec, cherry-picked my original commit 0c4333736b418db5e85e4480d8dc4b150df18171 into this new branch, and sent a PR off of this new branch to rubyspec/mspec. Does that make sense?

It makes perfect sense. Thanks for correctly interpreting my cryptic advice 😁
Also feel free to update Opal Gemfile to point to our fork instead of rubyspec/mspec if you need it to move forward.

Thanks @elia! I will update the Opal gemfile as a last resort if my mspec PR does not get merged by the time I'm ready with my Kernel#format magnum opus (hehe), but I'd rather see us working with rubyspec/mspec. All of the changes in opal/mspec fork are your commits - these would have to be pushed upstream or left behind for our fork to be even with upstream, but that's a call only you can make.

Time's the limit as always and tempus fugit ;)
I'll move those changes out of master and as time permits split them in smaller PRs so that they can be discussed and hopefully merged singularly.

@elia LOL, I completely understand :)


question about MOS_pred of synthetic speech

what's the difference of MOS_pred getted by nisqa.tar and nisqa_tts.tar? why the score getted by nisqa.tar is higher than that by nisqa_tts.tar?

nisqa_tts.tar is only thought to be applied on synthesized speech / text-to-speech samples, the corresponding paper can be found here: https://www.isca-speech.org/archive/Interspeech_2020/abstracts/2382.html
nisqa.tar is thought to be applied on transmitted speech, such as from audio calls. The paper can be found here: https://www.isca-archive.org/interspeech_2021/mittag21_interspeech.pdf


Tutorial button hidden on login screens

On all touch devices, the tutorial button is hidden and the entire UI is shifted upwards on login/create user screens. See here:

This was the case for existing users. The issue can also be replicated for a new user if the login screen is refreshed once.

I'm working on a fix.

It's not a bug, it's an expected feature on touch devices.
On real device (tablets/smartphone), half of the screen is used by a virtual keyboard. So we need to scroll up.

I don't think shifting the UI elements is required as the real devices shift the entire UI upwards automatically to make space for the virtual keyboard. See here:

And you can clearly see that the help button is mostly hidden here.
The solution I proposed in #698 would fix it.

Behavior on a mobile web browser is not the same than in a Cordova Webview app.
Did you test your change in the Android app?

I haven't done that yet. May I know how can I test the changes in the Android app?

Sure. I suggest you to use the Sugarizer APK builder.

I'm encountering an error while building the apk
FAILURE: Build failed with an exception.

* What went wrong:
Gradle build daemon disappeared unexpectedly (it may have been killed or may have crashed)

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.
Error: /sugarizer-cordova/platforms/android/gradlew: Command failed with exit code 1 Error output:
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
The message received from the daemon indicates that the daemon has disappeared.
Build request sent: Build{id=937565a5-5dae-424d-a577-b878e2a59ae6.1, currentDir=/sugarizer-cordova}
Attempting to read last messages from the daemon log...
Daemon pid: 1310
  log file: /root/.gradle/daemon/3.3/daemon-1310.out.log
----- Last  20 lines from daemon log file - daemon-1310.out.log -----
:incrementalDebugJavaCompilationSafeguard
:compileDebugJavaWithJavac
:compileDebugJavaWithJavac - is not incremental (e.g. outputs have changed, no previous execution, etc.).
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
:compileDebugNdk UP-TO-DATE
:compileDebugSources
:mergeDebugShaders
:compileDebugShaders
:generateDebugAssets
:mergeDebugAssets
:transformClassesWithDexForDebug
:mergeDebugJniLibFolders
:transformNative_libsWithMergeJniLibsForDebug
:processDebugJavaRes UP-TO-DATE
:transformResourcesWithMergeJavaResForDebug
:validateSigningDebug
:packageDebug
----- End of the daemon log -----

Hmmm. Is it the result of Sugarizer APK builder?
The docker container is very huge and take memory. It could be the cause of the unexpected crash of Gradle. Try to allocate more resource to docker.

Thanks @llaske, allocating more memory worked. I checked the APK and I agree that the UI needs to be scrolled up as you mentioned. My major concern still surrounds the disappearance of tutorial icon on such devices.
My current solution #698 removed the UI scroll, but I've updated the PR with my latest commit restoring the UI behavior and at the same time fixing the tutorial icon disappearance. Please review.

Fixed in https://github.com/llaske/sugarizer/pull/698

Revert back the fix in https://github.com/llaske/sugarizer/commit/4866d97fae314d5770e509e7a4bbe3350ed8325e because it don't work on iOS: the help button is no longer visible.
BTW https://github.com/llaske/sugarizer/pull/740 solve the issue too because the screen size is reduced so the help button become visible.

That's right, this shouldn't be a problem anymore on Android. Would look for a solution similar to #740 for iOS.


Fix for issue #82.

The actual fix is in the Azure SDK for Java. This commit removes the maven central dependency on the Azure SDK and instead uses a custom build of the SDK with the jars included in this repo. The changes made to the Azure SDK are available in https://github.com/avranju/azure-sdk-for-java/commit/0fa83ae864b55151453dd3fb99352493bb17a9e1.

The problem in the Azure SDK for Java is documented in this issue: https://github.com/Azure/azure-sdk-for-java/issues/458.

LGTM


Testing for too many should test last: 4, not last: 3

Pretty self explanatory, testing for last: 3 is the same as the exactly right test underneath it.

Sorry for the delay, thanks!


Break main Connect The Dots into several ones

To allow for additional devices, Cloud services and scenarios to be added, we need to break the main solution into several ones. One for the Raspberry Pi + Arduino code, one for the Cloud Deploy tool, one for the Web site.
The Wiki will have to be adapted as well and the Step-by-Step document broke into several pages.
Once we have this in place, it will be easier to add devices, and then add  services and scenarios.

Addressed in PR #21


Access denied, login session expired

The following message appears in the iob log:
homeconnect.0 | 2024-09-09 17:52:00.131 | error | {"error":"authorization_pending","error_description":"The authorization request is still pending as the end-user hasn't yet completed the user interaction steps"}
homeconnect.0 | 2024-09-09 17:52:00.130 | error | https://api.home-connect.com/security/oauth/device_verify?user_code=WQVR-9H9I
homeconnect.0 | 2024-09-09 17:52:00.128 | error | Bitte überprüfe Benutzername und Passwort oder besuche diese Seite für manuelle Anmeldung:
homeconnect.0 | 2024-09-09 17:52:00.127 | error | Please check username and password or visit this site for manually login:
homeconnect.0 | 2024-09-09 17:52:00.126 | error | AxiosError: Request failed with status code 400

After I follow the link https://api.home-connect.com/security/oauth/device_verify?user_code=WQVR-9H9I and enter my e-mail, the error message appears:
{
  "error": "access_denied",
  "error_description": "login session expired"
}

I finally found the error. The issue was that my account was very old and not linked to the new SingleKey login. So, I had to register at singlekey-id.com with my email and confirm the login in the smartphone app. After that, everything worked smoothly.

issue can be closed!


Very slow on large pages

When trying to access very large pages (e.g. huge GitHub diffs), this addon can slow down the page processing so much that Firefox warns about it, and the page feels extremely sluggy (and it spends like this for a long time, even on a system with a AMD Ryzen 5 7600X 6-Core Processor). It would be useful to either skip processing of pages which are too large, or exclude pages based upon user-definable URL regexes.

Thanks for bringing this up, that's super interesting. Streetpass uses a mutationobserver to detect dom changes to parse for verification links, and I kind of assumed that mutation observers were performant/off the main thread. Happy to go back to it and see if there's a less intensive detection system I can use

This occurs when opening "large diffs not previously rendered" sections, and the problem is that it doesn't appear to scale even nearly as well as the browser rendering itself, probably because the browser only renders stuff which is roughly on-screen, and skips everything not currently in view as much as possible... A fix could be skipping processing of very large pages or I suppose asking the user if they want to process the page instead of basically instantly hanging the page processing

after thinking about this for a day I think I have a way to do this in a more performant way without setting up a mutation observer -- will try fixing soon. thank you for bringing this up it was interesting to see that this DOM api isn't as fast as I thought it was!


Nuget Package Restore failing: NU1108 Cycle error

I cannot restore veldrid to a fresh net core project via nuget. Tried within Visual studios and the CLI. Gives cycle error.

Caused by the .csproj file being named Veldrid... I am dunce. Will close.


chore: specify packageManager in package.json

This makes corepack automatically pick the correct version of pnpm to use.
The pnpm GitHub action also supports this field.

Thanks!


deprecate model_name

Closes #315
Replaces calls to unstructured that had model_name with hi_res_model_name. This PR should NOT be merged until after https://github.com/Unstructured-IO/unstructured/pull/2289 is merged in the unstructured repo.

Feels good to be rid of that hack! I guess we can sit on this until 0.11.6 is published, and then bump unstructured in this pr.

The related PR in unstructured has been merged:
https://github.com/Unstructured-IO/unstructured/pull/2289

@awalker4, when I inspect the call to partition from within general.pipeline_api(), it is not the recently updated partition in unstructured that has hi_res_model_name as a parameter. I created a new virtual env and reinstalled, but I'm still seeing the same thing. I saw that you updated the unstructured requirement and see unstructured[local-inference]==0.11.6 in base.txt, so I'm wondering if you might know what else might be happening.

@awalker4, when I inspect the call to partition from within general.pipeline_api(), it is not the recently updated partition in unstructured that has hi_res_model_name as a parameter. I created a new virtual env and reinstalled, but I'm still seeing the same thing. I saw that you updated the unstructured requirement and see unstructured[local-inference]==0.11.6 in base.txt, so I'm wondering if you might know what else might be happening.

@qued, would you have any insights into this? I'm trying to figure out why the failing test is returning 200 when it should be returning 400 given an invalid hi_res_model_name


Add support to show a specific view (tab on top of the lovelace interface)

Would it be possible to make a sensor to force fullykiosk to show a specific page ('view' from lovelace interface).
For example: I'm using now custom:state-switch lovelace cards to show/display cards.
When someone rings the doorbell, I show the camera for 1 minuit, then it disappears. Problem with this: the camera is shown in a picture entity, I prefer to have it full screen. To solve this I thought to have the camera in a view and force fullykiosk, based on a switch (helper / input_boolean).

This should be possible already with the load_url service. The different lovelace tabs look to update the URL in the browser (for example https:///lovelace/default_view or https:///lovelace/rooms) so in theory you'd jsut need that URL and add an action to the automations to call the load_url service for that.


Replace marked

Since marked has not been updated for 7 months it's reasonable to take a look at more active alternatives (e.g. markdown-it) the question is - would it work with nunjucks-markdown?

Yes, we definitely have to replace it. I will look later for possible candidates. So far markdown-it looks good.
It should work with nunjucks-markdown, since the last one only parses Nunjucks templates and determinates which exactly portion of content should be passed to markdown formatter and returned back, nothing more.


Add Proxy Pattern Two way Binding and validator Sample

Related Issue
Add Two way binding sample by proxy pattern sample #1247
Closes: # #1247
Describe the changes you've made
I have updated script.js and add an HTML and an image to project
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
It is simple and hasn't got any thing unusual
Checklist:

[ *] My code follows the style guidelines of this project.
[*] I have performed a self-review of my own code.
[*] I have commented my code, particularly in hard-to-understand areas.
[] I have made corresponding changes to the documentation.
[*] New and existing unit tests pass locally with my changes.

Screenshots

@mghanatabady Looks Good. Few Changes

When adding I don't see any changes in the UI, was this way the code was done? It would be great if Age can also been seen.
In script.js its png not PNGdue to this image not loading.
It would be great if you could add some css, If don't know css don't worry, leave it Fix(1,2)

I would like to work on this issue, will you assign me ?

@mghanatabady Looks Good. Few Changes

When adding I don't see any changes in the UI, was this way the code was done? It would be great if Age can also been seen.
In script.js its png not PNGdue to this image not loading.
It would be great if you could add some css, If don't know css don't worry, leave it Fix(1,2)
Properly link the issue in description too
Its like Closes #PRNO

1- I'm working on it
2- I'm changing the image file
3- It's a good idea but I think so this example  should be simple in UI and the code reader can focus only on pure code

Congratulations on merging of first PR. :tada:
Keep Contributing.

Congratulations on merging of first PR. 🎉
Keep Contributing.

Thank you Indeed, Ok I will.


textView(_:shouldChangeTextIn:replacementString:) not been called

Hi szweier,
I have added SZMentionsListener in my code. I also implemented textView(_:shouldChangeTextIn:replacementString:) in my controller. I realised that if I paste string from external app into the textview that has implemented this library. My callback not getting called. I have stepped into the library code.
else if text.count > 1 {
            //Pasting
            if let editedMention = mentions.mentionBeingEdited(atRange: range) {
                clearMention(editedMention)
            }

            textView.delegate = nil
            if let mutableAttributedString = textView.attributedText.mutableCopy() as? NSMutableAttributedString {
                mutableAttributedString.replaceCharacters(in: range, with: NSAttributedString(string: text))
                mutableAttributedString.apply(defaultTextAttributes, range: NSRange(location: range.location, length: text.count))
                mentionsTextView.selectedRange = NSRange(location: 0, length: 0)
                textView.attributedText = mutableAttributedString
                mentionsTextView.selectedRange = NSRange(location: range.location + text.count, length: 0)
            }

            mentions.adjustMentions(forTextChangeAtRange: range, text: text)
            textView.delegate = self

            return false
        }
        _ = delegate?.textView?(textView, shouldChangeTextIn: range, replacementText: text)

The delegate callback will not get called if it fulfilling the text pasting action. However, I need to do some adjustment to my UITextView (To resize the UI) if received the callback.
Is there any way I can get my delegate get called in the event of pasting text?

@wonderhero please check out this PR and let me know if it solves your problem. https://github.com/szweier/SZMentionsSwift/pull/66

@szweier I have tested the code and tried to step into the implementation. The defer statement still not getting called. Would you mind to have a check on this again?

@wonderhero are other delegate methods being called? I find it strange that the defer wouldn’t be hit. Any chance you can push a sample app to a public repo that demonstrates this problem?

@szweier yes, other delegate methods working well. Only textView(_:shouldChangeTextIn:replacementString:) not been called. Let me try setup a sample app for this.

@wonderhero I just placed the following code into the ExampleViewController
    func textView(_ textView: UITextView, shouldChangeTextIn range: NSRange, replacementText text: String) -> Bool {
        print("called")
        return true
    }

And it appears to be getting called (print statement is shown in console). It is worth noting that, due to the way the listener works, what you return from this method is ignored.

@szweier I have setup a sample project to test this. Would you mind to check it out? https://github.com/wonderhero/SampleMentionApp
If I paste a text into the textview. The print text won't get executed.
func textView(_ textView: UITextView, shouldChangeTextIn range: NSRange,
                  replacementText text: String) -> Bool {
        print("TextView change text in")
        return true
    }

@szweier You are right, I have tested it. The listener is working. It is my bad.

In my case I forgot to initialize listener with viewController delegate
public init(         mentionsTextView: UITextView,         **delegate: UITextViewDelegate? = nil,**         ...


docs: curl request

PR review comment

Closed, because the file will be moved in https://github.com/meshcloud/meshcloud-docs/pull/529


[nit] oops... s/MACKREL_PLUGIN_WORKDIR/MACKEREL_PLUGIN_WORKDIR/g

ref: https://github.com/mackerelio/go-mackerel-plugin-helper/pull/21


[Description] A more attractive READ.ME

🤯 I might change it a bit, in order to keep the doc "icy" and "elegant"


Add command line arguments

What are command line arguments? Are they like functions or extra information?

When you start a program from a terminal, you can pass additional values (as strings) to the program, sometimes called arguments. In Python, these are accessible from the sys.argv property

You may also consider using https://docs.python.org/3/library/argparse.html or adding https://click.palletsprojects.com/en/8.1.x/ as a library to handle command line arguments.


Reviewers update: Adding EdDev as a code reviewer

What this PR does / why we need it:
Eddy's has a big history of reviews in the project.
His reviews are very thorough and helpful.
Eddy also contributed high quality code to the project.
I propose him as a code reviewer.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

/lgtm

/lgtm

/lgtm

/lgtm

/approve

/approve


Minor grammar and spelling fixes

Seem like needed corrections. Any reason this was closed, @roeelandesman ?

Hey there! Was going to dedicate more time to do a thorough passthrough. I can re-open here and add edits :)

Ok, no problem. The more review and polishing you're willing to provide the better. We thank you!


Add createNamespace PBC setting

This PBC setting will tell the controller to create the namespace on the target cluster if it doesn't exist. This is disabled by default because the controller does not impersonate the user applying the package and will always have namespace creation permissions.
Fixes #601
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Will this require an upgrade to the Controller, or since it's PBC it should be self update when the chart does?

Will this require an upgrade to the Controller, or since it's PBC it should be self update when the chart does?

Not sure what you mean by this. It will be a feature only available if you upgrade the controller and the CRDs. New controllers will work with existing PBCs.


Build a vppagent Firewall NSE

Intro
The purpose of this issue is to build a firewall NSE using the vppagent.
The intended end state is roughly this:
Advertise a service, with labels (called secure-intranet-connectivity for now, label app=firewall).
When a new connection comes in to the Firewall (hereafter called the incoming connection), open a new connection, also to secure-intranet-connectivity Network Service, with sourceLabels app-firewall (hereafter called the outgoing connection).
Connect the incoming connection to the vppagent.  Connect the outgoing connection to the vppagent.  Cross connect them.  Apply a simple stateful ACL to the ingress of the incoming connection.
Here's the fun trick.  The src_ip that comes back from the return of the outgoing connection in its Connection context, don't assign that to the interface for the outgoing connection, instead pass it back as part of the ConnectionContext in response to the incoming connection.
So sequence looks roughly like this:

Call RegisterNSE with network service name "secure-intranet-connectivity" and labels "app=firewall".
When a Request(Connect) comes into the Firewall (incoming connection)
Before responding the Firewall should call Request(Connect) for a network service "secure-interanet-connectivity" with sourceLabels "app=firewall"
When the Firewall gets the Connection back in response to its Request, create an interface for that connection (outgoing connection).  Do not apply the "src_ip" from the ConnectionContext to that interface.
Instead, copy the ConnectionContext from the outgoing connection to the ConnectionContext for the response to the incoming connection.
Create the interface for the incoming connection.  Don't apply any IP to that either.
Create an ACL for ingress on the incoming connection.
Cross connect the interfaces for incoming and outgoing connections
Return for the Request(Connection) for the incoming connection.

Effectively, the Firewall is a transparent firewall between the two network services.

Notes on the ACL API for vppagent
vppagent has an ACL API.  It can apply L2-L4 ACLs to ingress or egress on an interface.
I would recommend doing an L4 ACL (like only allowing port 80 to pass).  Please note, you can make this stateful using the REFLECT ACL action.

Comments on setting things like Advertises NS Name/Labels, and outgoing NS Name/Labels
Even though we talk about these both being "secure-intranet-connectivity" and "app=firewall", you will find that allowing them to be specified by ENV variables in the Pod spec will bear fruit... because at first you will only have the icmp-responder to connect to with the outgoing connection.

Comments on how to connect various things
vppagent-icmp-nse shows a good example of how to build a vppagent NSE.  Do mind the trick with the ConnectionContexts mentioned in the intro.
Similarly
vppagent-nsc provides a good example of how to build a vppagent based NSC.
Since this example involves a single executable that is both an NSE and an NSC, both are relavent.
Here is an example of building cross connects.  Note that they are directional.  To Cross connect iface0 and iface1, you need to calls:
iface0 -> iface1
and
iface1 -> iface0.


default token algorithm is HS256

Yup my bad. I had the intention of updating it to 512 but decided against it and forgot to change the docs. Thank you!


6.28 OP code anyone?

Any op code?

{
"C2S_ActionRequest": "0x038f",
"C2S_ActionRequestGroundTargeted": "0x0199",
"S2C_ActionEffect01": "0x0130",
"S2C_ActionEffect08": "0x01df",
"S2C_ActionEffect16": "0x0202",
"S2C_ActionEffect24": "0x00bd",
"S2C_ActionEffect32": "0x014c",
"S2C_ActorCast": "0x0398",
"S2C_ActorControl": "0x02d3",
"S2C_ActorControlSelf": "0x029b",
"Server_IpRange": "<IP_ADDRESS>/24, <IP_ADDRESS>/24, <IP_ADDRESS>/24, <IP_ADDRESS>/24, <IP_ADDRESS>/24, <IP_ADDRESS>/24",
"Server_PortRange": "1025-65535"
}

Sorry for my ignorance,what are OPcodes ?

OP codes are what make XIVAlexander work. XIVAlexander should auto-update them every new patch but sometimes you may have to copy+paste them from someone here if XIVAlexander doesn't do so.


DNN-8294. Fixed TabController unit tests. Changes to TabController to…

… work with unit tests

Code review done.


Topic/storage pull request: add AttachVolume to controller & scheduler

This pull request adds support to the ciao-controller for the AttachVolume command from the block API. The scheduler was also modified to route the attach volume ssntp command to the correct agent.

Coverage decreased (-0.1%) to 64.425% when pulling f77eb76ef6a10880893e630cfeb79835f9a1e03f on kaccardi:topic/storage-pull-request into d3bfa02ba4b2c58660245f57608e59875940e3a7 on 01org:kaccardi/topic/storage.


Bundle the test suite with the pip installer

The setuptools API now supports a test_suite capability.
We can investigate how this can be used for bundling the test suite into the pip installer.

Sounds interesting, although the test requires some load of data in ParaVision format to convert. I guess, this should be downloaded with pip at the moment of the installation to make it work.
An external benchmarking, as discussed offline with @neurolabusc, also to compare other converters, and to have a dataset for tests, can be an optimal solution.

re "this should be downloaded with pip at the moment of the installation"
I wouldn't recommend doing that. It unnecessarily creates a dependency between your pip installation and the data download. For instance imagine the data download goes wrong, and the pip installation just fails.
It can be configured to download the data when the test command is run.

Agree. My preference is for the external benchmarking.
re "It can be configured to download the data when the test command is run."
This can be done already  downloading the repo... Is there any benefit in linking it to the pip installation?

@SebastianoF "This can be done already downloading the repo" is not clear to me. Is the data included as part of a repo?

I've had another look, and noticed the test_data folder. If with "This can be done already downloading the repo" you mean that folder, it's possible to bundle it as part of the pip installer as well. See the package_data keyword.
The benefit of package_data is that the user doesn't have to go through a separate data download. package_data is suitable for small datasets that do not make the pip installer fat. It "installs" data together with the software and you can later on fetch that data from within Python without any input from the user such as the absolute file path to the data.
Also please note the information about the test_suite keyword in the description of this issue. (I've never used this before, but) I reckon this can be used for bundling your tests provided in the test directory into the pip installer. If you think this is useful, I'm happy to help set it up.

Given the heavy data weights possibly involved, and being purposes of this repo the conversion it may be better to opt for future external benchmarking. Will reopen and update when/if available.
Thanks for the help!!


[MESSAGE] modifies prometheus response

This PR is for:

Message

modifies prometheus response
Checklist

[ ] Added unit test.

Backport to branches

main

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


v0.0.22

Added basic GeoTIFFs
Added basic TIFFs
Added base Image ABC
Added logging
Reformatted code
Started Adding Elevation Data
Created C Package Loader

I'm the only one working on this right now.


If you resubmit a job, will it purge files that have been deleted?

I followed link to github from the Sharepoint Migration tool. I wondered when re-submitting a job, will it purge files that have been deleted?

Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.

We are the supplier.. FYI, when you re-submit a job, changes in either the source or destination are retained (last modified is retained), whereas deleted items in the source are not purged in the destination. I could not see any documentation regarding this.


Undefined variable 'errorGp' in tutorial.m

In line 124 of tutorial.m, there is an error "Undefined function or variable 'errorGP'." How to fix that?

@adamian can you take a quick look?
On Wed, Apr 6, 2016 at 1:20 PM, 董胤蓬<EMAIL_ADDRESS>wrote:

In line 124 of tutorial.m, there is an error "Undefined function or
variable 'errorGP'." How to fix that?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/SheffieldML/deepGP/issues/3

Woops, that line (and the errorRecGP) shouldn't be there. Either comment them out or pull the changed I just did.
Thanks for pointing this out.


Backport 2.16: Add missing brackets around macro parameters

This is a straightforward backport of https://github.com/ARMmbed/mbedtls/pull/2092 .
I made a full backport, not just a backport of the one user-facing macros that was actually broken (MBEDTLS_X509_ID_FLAG), because this is a very low-risk change and keeping the code aligned will make future backports easier.

CI failure is known-flaky timing selftest.


cd to the working directory of another active session

When I cd to a certain directory, I often want to cd to the same directory in another session (and have both running simultaneously). It is nice to have the working directories of other sessions marked specially in the location mode.
For instance, we can put a * in front of the working directories of active sessions, and having * in the filter restricts the candidates to those directories.

Stray thought on this: Since all sessions are talking to the same daemon process anyway, why not use this fact to implement a sort of message bus allowing the shells to communicate between themselves on a more general level? The current working directory is only one of many possible pieces of data that one might want to share between sessions, after all.

It is nice to have the working directories of other sessions marked specially in the location mode.

I don't see how this is useful other than very narrow scenarios. If I have ten elvish sessions on a particular system, all of which have a different CWD, how useful is it for the location mode to mark all ten directories as being a CWD in another session? What happens to the location mode display if I terminate one of those sessions (e.g., via [ctrl-D])? Why not just copy/paste the output of pwd?
This seems to me a lot like the use case for merging global command history with local history via the edit:history:fast-forward command so that a session can "see" the commands recently run in other sessions. In this case the global history would contain the most recent directory change in any session. Merging global history into local history would expose the global "most recent cd destination" as the first entry shown by location mode so that simply pressing [enter] would select it. Merging of the global "most recent cd" could be done implicitly on every invocation of location mode rather than requiring an explicit edit:history:fast-forward.
Whether implicitly merging the global CWD is a good idea is not obvious. Especially since it is not obvious that we should, or even can, automatically synchronize with the global CWD state without also syncing command history. At least without using a message bus/queue as proposed by @hanche. The problem with that solution is that whether or not we use OS specific features (e.g., man 7 mq_overview on Linux) it requires a lot of additional baggage.

Just to be clear on this, I am not really in favour of the message bus proposal. As @krader1961 says, it does require quite a lot of baggage, though likely less than more general solutions intended for more general purposes. But I think I do favour it over sharing CWD information, being a more general mechanism but not necessarily much more complex. (You'd have to be careful to keep it as simple as possible.)

LIke @hanche I am ambivalent about including support for an elvish message bus. On the one hand it might enable interesting features. On the other hand it adds a lot of baggage that can't be justified solely to support sessions sharing CWD changes.
Important: I would like to see the most recent CWD change by any other elvish session appear as the first entry in the location mode of a particular elvish session (regardless of whether the change is done using location mode or an explicit cd command). That would make it easier to do an explicit cd /a/long/path in one session and quickly switch to it in a different session without resorting to copy/paste or explicitly syncing command history via edit:history:fast-forward. That's something I do with some regularity. What I don't think is workable, or particularly useful, is the idea of making the CWD of every other elvish session visible in the location mode of a particular elvish session.
P.S., Note that this feature requires converting relative paths (e.g., cd .) into absolute paths when exposed to other elvish sessions.

I would like to see the most recent CWD change by any other elvish session appear as the first entry in the location mode of a particular elvish session

Indeed, considering recently CWD changes is much simpler than considering CWD of active sessions.
Instead of treating the most recent CWD change specifically, there is a more general solution: in location mode, support ordering the directory list purely by recency. This can be part of a general mechanism of supporting custom ordering in listings - so not just location mode, but also history listing mode, completion mode, and the individual columns of navigation mode.

... in location mode, support ordering the directory list purely by recency.

That solution works for me assuming I don't need to run edit:history:fast-forward. That is, I want CWD changes to be immediately visible in the location mode of other shells but do not want commands run in a shell to be immediately visible in the other shells.


Adds python_workers compatibility flag

This is necessary in order to allow upload for non-cloudflare users. This is safe because EWC has custom logic for blocking the upload of bundles containing Python modules, so only certain users will be allowed to upload.

Thanks!


Add a database reconciler

Description
In this PR I've added the option to apply database intents. The intents are reported to our cloud and then being applied there.
Thoughts
Maybe we should release it behind a feature flag? We also might want to create a tutorial on how to use it and add an e2e test.
Testing
I've added a test suite to test the new database reconciler and added few tests. Let me know if there are more tests I can add.
To manually test this feature, setup a postgres database and apply database intents, e.g.:
apiVersion: k8s.otterize.com/v1alpha2
kind: ClientIntents
metadata:
  name: client
spec:
  service:
    name: client
  calls:
    - name: test-int2
      type: database
      databaseResources:
        - table: jordan
          operations: [ ALL ]

[x] This change adds test coverage for new/changed/fixed functionality

Pushed a round - added support for: deletion with the finalizer jazz, multiple intents, and a feature flag.

Pushed the PR fixes:

Handling multiple intents
Fixed the tests, also test for adding a finalizer


Minor change PT-BR

Related to: https://github.com/DistributedCollective/Sovryn-frontend/pull/1543

Merged via #1550


git branch name collumn

if you have several folders with same project but different branches

now available in latest build, reads .git/HEAD file
https://github.com/unitycoder/UnityLauncher/releases


Adds comprehensive tests for data augmentor

Codecov Report

Merging #145 (37ae122) into main (904b6da) will increase coverage by 0.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #145      +/-   ##
==========================================
+ Coverage   83.99%   84.07%   +0.08%
==========================================
  Files          67       67
  Lines        2305     2273      -32
==========================================
- Hits         1936     1911      -25
+ Misses        369      362       -7

Impacted Files
Coverage Δ

ezflow/modules/models/recurrent.py
47.61% <0.00%> (-2.39%)
:arrow_down:

ezflow/modules/blocks.py
64.70% <0.00%> (-0.82%)
:arrow_down:

ezflow/similarity/learnable_cost.py
85.13% <0.00%> (-0.58%)
:arrow_down:

ezflow/decoder/iterative/recurrent_lookup.py
77.22% <0.00%> (-0.45%)
:arrow_down:

ezflow/modules/dap.py
88.46% <0.00%> (-0.43%)
:arrow_down:

ezflow/functional/criterion/multiscale.py
79.16% <0.00%> (-0.43%)
:arrow_down:

ezflow/similarity/correlation/layer.py
94.73% <0.00%> (-0.27%)
:arrow_down:

ezflow/encoder/residual.py
89.47% <0.00%> (-0.16%)
:arrow_down:

ezflow/functional/criterion/sequence.py
96.00% <0.00%> (-0.16%)
:arrow_down:

ezflow/similarity/correlation/pairwise.py
97.77% <0.00%> (-0.14%)
:arrow_down:

... and 12 more


invalid PostScript font name

Hello,
I tried using bdf2ttf to transform the Tamzen font into TTF, but when I double-click on the resulting TTF file under Windows 7, an error dialog appears saying "Windows Font Viewer: The requested file ... is not a valid font file.".  Similarly, I'm getting an error message when opening the resulting TTF file in FontForge under Linux: :disappointed:

In the above screenshot, that "|0003|" Unicode glyph at the end of the font name is suspicious. :suspect:
Steps to reproduce the bug
$ mkdir -p ~/.fonts/tamzen-font

$ git clone https://github.com/sunaku/tamzen-font ~/.fonts/tamzen-font

$ cd ~/.fonts/tamzen-font

$ git checkout bdf2ttf

$ cat ttf/TamzenForPowerline.ini
Fontname = TamzenForPowerline
Version = 1.11.5
Copyright = Copyright
Trademark = Trademark

$ bdf2ttf ttf/TamzenForPowerline-regular.ttf ttf/TamzenForPowerline.ini bdf/TamzenForPowerline*r.bdf
Input files:
  bdf/TamzenForPowerline5x9r.bdf
  bdf/TamzenForPowerline6x12r.bdf
  bdf/TamzenForPowerline7x13r.bdf
  bdf/TamzenForPowerline7x14r.bdf
  bdf/TamzenForPowerline8x15r.bdf
  bdf/TamzenForPowerline8x16r.bdf
  bdf/TamzenForPowerline10x20r.bdf

$ bdf2ttf -b ttf/TamzenForPowerline-bold.ttf ttf/TamzenForPowerline.ini bdf/TamzenForPowerline*b.bdf
Input files:
  bdf/TamzenForPowerline5x9b.bdf
  bdf/TamzenForPowerline6x12b.bdf
  bdf/TamzenForPowerline7x13b.bdf
  bdf/TamzenForPowerline7x14b.bdf
  bdf/TamzenForPowerline8x15b.bdf
  bdf/TamzenForPowerline8x16b.bdf
  bdf/TamzenForPowerline10x20b.bdf

$ fontforge ttf/TamzenForPowerline-regular.ttf
... now observe the error shown in the screenshot above ...

Thanks for your consideration. :sweat_smile:

I can't reproduce your problem, but get font files attached.  Please check this.
TamzenForPowerline.zip
It seems correct font files for me.
I have compile bdf2ttf by MSVC10's 32bit compiler on WIndows 10,
run and check result TTF files on same machine.

Thanks! :+1: Your generated fonts worked successfully in Linux FontForge and Windows 7. :100:
I should have mentioned that I compiled and ran bdf2ttf under Linux, using make gcc target.
So there appears to be some functional difference between the results of MSVC and GCC. :bug:

For your reference, here is my GCC information and compilation output:
$ gcc --version
gcc (GCC) 4.9.3
Copyright (C) 2015 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

$ make gcc-clean
make -f compile/Make_gcc.mak clean
make[1]: Entering directory '/home/sunny/lab/bdf2ttf'
rm -f ./src/bdf.o ./src/bdf2ttf.o ./src/main.o ./src/rcfile.o ./src/table.o ./src/ucsconv.o
make[1]: Leaving directory '/home/sunny/lab/bdf2ttf'

$ make gcc
make -f compile/Make_gcc.mak
make[1]: Entering directory '/home/sunny/lab/bdf2ttf'
cc    -c -o src/bdf.o src/bdf.c
g++      -c -o src/bdf2ttf.o src/bdf2ttf.cpp
src/bdf2ttf.cpp:71:1: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
 };
 ^
src/bdf2ttf.cpp:71:1: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
src/bdf2ttf.cpp:71:1: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
src/bdf2ttf.cpp:71:1: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
src/bdf2ttf.cpp:71:1: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
src/bdf2ttf.cpp:71:1: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
src/bdf2ttf.cpp:71:1: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
src/bdf2ttf.cpp:71:1: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
src/bdf2ttf.cpp:71:1: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
src/bdf2ttf.cpp:71:1: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
src/bdf2ttf.cpp:71:1: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
src/bdf2ttf.cpp:71:1: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
src/bdf2ttf.cpp:71:1: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
src/bdf2ttf.cpp:71:1: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
src/bdf2ttf.cpp:71:1: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
src/bdf2ttf.cpp:71:1: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
src/bdf2ttf.cpp:71:1: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
src/bdf2ttf.cpp:71:1: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
src/bdf2ttf.cpp:71:1: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
src/bdf2ttf.cpp:90:30: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
 static char* STYLE_REGULAR = "Regular";
                              ^
src/bdf2ttf.cpp:91:28: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
 static char* STYLE_BOLD  = "Bold";
                            ^
src/bdf2ttf.cpp:92:29: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
 static char* STYLE_ITALIC = "Italic";
                             ^
src/bdf2ttf.cpp:93:33: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
 static char* STYLE_BOLDITALIC = "Bold Italic";
                                 ^
In file included from src/bdf2ttf.cpp:11:0:
src/version.h:12:19: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
 #define COPYRIGHT "(C) 2003 KaoriYa";
                   ^
src/bdf2ttf.cpp:95:21: note: in expansion of macro 'COPYRIGHT'
 char *g_copyright = COPYRIGHT;
                     ^
src/version.h:13:18: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
 #define VERSION  "2.0"
                  ^
src/bdf2ttf.cpp:100:20: note: in expansion of macro 'VERSION'
 char *g_version  = VERSION;
                    ^
src/version.h:14:19: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
 #define TRADEMARK "KaoriYa"
                   ^
src/bdf2ttf.cpp:102:21: note: in expansion of macro 'TRADEMARK'
 char *g_trademark = TRADEMARK;
                     ^
src/bdf2ttf.cpp: In function 'void generate_OS2_PREP(bdf2_t*)':
src/bdf2ttf.cpp:831:35: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
     os2->array::addString(VENDORID); // achVendID
                                   ^
cc    -c -o src/main.o src/main.c
cc    -c -o src/rcfile.o src/rcfile.c
g++      -c -o src/table.o src/table.cpp
cc    -c -o src/ucsconv.o src/ucsconv.c
g++ -o bdf2ttf  src/bdf.o src/bdf2ttf.o src/main.o src/rcfile.o src/table.o src/ucsconv.o
make[1]: Leaving directory '/home/sunny/lab/bdf2ttf'

There seems be bad assumption of bdf2ttf.
bdf2ttf consider unsigned long is 32bit.  But most of gcc treat it as 64bit.
It will cause some problems.

It would be better that use size aware types uint32_t or so.
But I don't have time to do that.

I have fixed for gcc.
@sunaku please try again with recent version.

Thanks @koron! :bow:  I tried the fix but the problem still occurs. :disappointed: I will try to debug it further. :construction_worker:

I tried rebuilding bdf2ttf for 32-bit architecture (my system is 64-bit):
$ make gcc CFLAGS=-m32 LDFLAGS=-m32
make -f compile/Make_gcc.mak
make[1]: Entering directory '/home/sunny/lab/bdf2ttf'
cc -m32   -c -o src/bdf.o src/bdf.c
g++  -m32    -c -o src/bdf2ttf.o src/bdf2ttf.cpp
cc -m32   -c -o src/main.o src/main.c
cc -m32   -c -o src/rcfile.o src/rcfile.c
g++  -m32    -c -o src/table.o src/table.cpp
cc -m32   -c -o src/ucsconv.o src/ucsconv.c
g++ -o bdf2ttf  src/bdf.o src/bdf2ttf.o src/main.o src/rcfile.o src/table.o src/ucsconv.o -m32
make[1]: Leaving directory '/home/sunny/lab/bdf2ttf'

I still get the "Postscript font name" error in FontForge :disappointed: but at least the embedded bitmap fonts appear correctly now :relieved: (previously FontForge rendered them as empty, red boxes). :construction_worker: Next idea...

I still get the "Postscript font name" error in FontForge

I won'f fix about this.
Because it is not in bdf2ttf scope.
But welcome to receive patches.

:sweat_smile: Fair enough, thanks for your help! :bow:


Add /stage page

i.e.:

Current status: stage page basically all done except for the "Summary Metrics…" table.
Load-testing and possible pagination concerns remain.


Not updating archive?

Hey guys,
I have some problems with adding files to an archive. I use chokidar to watch a folder as follows:
const watcher = chokidar.watch(filePath, {
    ignored: /(^|[\/\\])\../, // ignore dotfiles
    persistent: true,
    ignoreInitial: true,
});

watcher.on('add', async (addedPath: string) => {
    fs.readFile(filePath, async (err, data) => {
        await archive.writeFile(path.basename(addedPath), data);
        console.log('Wrote ${path.basename(addedPath)}, ${addedPath} to the archive');
        console.log(`Current archive contents: ${await archive.readdir('/')}`);
    });
});

Which works as expected: It adds the file to the archive and lists it in archive.readdir('/').
However, when using the following code to watch for updates, it does not register any updates:
const { Hyperdrive } = sdk;
const archive = Hyperdrive(key, {
    sparse: false,
    sparseMetadata: false,
});
// This registers the initial content of the archive correctly
let files = await archive.readdir('/');
// Just updates do not work
archive.on('update', async () => {
      console.log('archive updated');
});

This is wrapped in a async click listener in a react electron app.
Am I missing something? Is it the async method?

Hey, I think there might be an issue with the update event in hyperdrive right now where the update event isn't firing. https://github.com/hypercore-protocol/hyperdrive/issues/284
I've been temporarily working around it by listening to archive.metadata.on('remote-update', () => {}) which fires sometimes when there aren't updates too, but is better than nothing. 😅

Hey, just figured it out thanks to Mafintosh.
Instead of using the update event, you should use archive.watch('/', () => {}) to watch for changes. 😁


Debug Renderer

Debug Renderer

Move typescript typing generation into CMake - supports CMake modifiable IDL file
Conditional inclusion of debug-renderer IDL into build and typings
THREE.JS - CSS3D Renderer (used for text display in Debug Renderer)

Example Three.JS DebugRendererJS implementation
Demo using Debug Renderer, using widget to show flag usages

The example Renderer demo is currently set to check for the Jolt.DebugRendererJS class, so the page will do nothing on non-distribution builds and the github-pages site, so I did not include it on the index.html

@PhoenixIllusion Thanks a ton again for all your hard work!


Improve max tokens count

Codecov Report

Merging #66 (e5dcf49) into master (00f3f4a) will increase coverage by 0.27%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #66      +/-   ##
==========================================
+ Coverage   86.18%   86.45%   +0.27%
==========================================
  Files          34       34
  Lines        2070     2090      +20
  Branches      305      307       +2
==========================================
+ Hits         1784     1807      +23
+ Misses        286      283       -3

Impacted Files
Coverage Δ

contracts/PowerIndexPool.sol
94.52% <100.00%> (+2.21%)
:arrow_up:

contracts/PowerIndexPoolController.sol
48.38% <100.00%> (+32.59%)
:arrow_up:

contracts/balancer-core/BPool.sol
98.61% <0.00%> (+0.55%)
:arrow_up:

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 00f3f4a...e5dcf49. Read the comment docs.


What role does the Coulomb force play in the potential function, and why is it significant for Na₄Cl₄ clusters?

The Coulomb force is the dominant force that determines the potential energy of Na₄Cl₄ clusters, and it plays a key role in the stability and shape of the cluster. The Coulomb force is responsible for the arrangement of ions in the cluster, which in turn determines the cluster's potential energy. The stability of the cluster is determined by the difference between its potential energy and the potential energy of its separated state.

Thank you @Mosaddeq107 for making it clear.


Support for App Store Server Notifications V2

Do you plan to add support for App Store Server Notifications V2, since V1 is deprecated?

This library supports V2 notifications, not V1

When I check the code, I saw this:
def request_test_notification(self) -> SendTestNotificationResponse:
        """
        Ask App Store Server Notifications to send a test notification to your server.
        https://developer.apple.com/documentation/appstoreserverapi/request_a_test_notification

        :return: A response that contains the test notification token.
        :throws APIException: If a response was returned indicating the request could not be processed
        """
        return self._make_request("/inApps/v1/notifications/test", "POST", {}, None, SendTestNotificationResponse)

its using the /v1 in all occurrences where notification is mentioned, why is that?

That's the App Store Server API version you're referencing, not the server notification version.

Each endpoint in the App Store Server API is given a version in case the endpoint is changed in the future. For this endpoint https://developer.apple.com/documentation/appstoreserverapi/request_a_test_notification, the current endpoint version is V1. For example, Get Refund History https://developer.apple.com/documentation/appstoreserverapi/get_refund_history currently has a V2 as this was updated to include a new parameter, revision, unlike the now deprecated V1. This versioning, however, is per endpoint and not correlated with App Store Server Notification version.

I understand now, thank you for the explanation!


Few extra milliseconds taken while navigating through sidebar

Describe the bug
Hi Team, So I was going through Liveblocks.io and I figured out that when I am navigating from one section to another its taking unusual amount of time.
I just wanted to know whether it's intended or it's a small lag.
If you guys can clarify then i could work on this issue.
☺
To Reproduce
Steps to reproduce the behavior:

Open App
Make account
Navigate through sidebar

Expected behavior
The intended page should load up in a flash.
Illustrations
https://github.com/liveblocks/liveblocks/assets/55043383/4667a45e-9fe4-45a9-807d-992888195ed2
Environment (please complete the following information):

Liveblocks packages and packages version [e.g<EMAIL_ADDRESS><EMAIL_ADDRESS>Browser's version [e.g. chrome v99.0.4]

Yeah, although the issue seems to go away now, since I have been using @liveblocks actively.
Thanks @adigau

Hi @Neon-20, I wanted to provide you with an update on the performance issue you reported. Last week, we addressed several performance concerns on the dashboard. Please note that the recent improvements have been applied to specific routes, so you might still experience some delays when navigating other routes. We have taken note of these remaining issues and will focus on resolving them as soon as we can.
Thank you for bringing this to our attention. Have a great week!


SauceLabs microsoftedge 12 is not available

We are seeing this is all CI builds at the moment. It looks like SauceLabs has removed this environment for the time being.
Error: [init({"name":"grunt-test:Thu, 13 Oct 2016 12:59:18 GMT","record-video":true,"record-screenshots":true,"tunnel-identifier":"SauceTunnel:Thu, 13 Oct 2016 12:59:18 GMT","browserName":"microsoftedge","version":"20.10240","platform":"Windows 10"})] The environment you requested was unavailable.

ah, they do that. It's annoying, I don't know why they remove browsers, especially IE/Edge variants
https://saucelabs.com/platforms#windows8-10
We will need to remove that browser, and add the new Edge version.

@triblondon Is there a contact at SauceLabs we can speak to about this?

I haven't been in touch with them for some time.  I'd suggest just going through the support ticket system.


PostgreSQL PropertiesToExcludeOnUpdate not implemented?

Hi - love this library! Question, is PropertiesToExcludeOnUpdate not implemented for PostgreSQL ?

That's right, currently it works only for SqlServer.

Love this library as well ❤
Any ETA on this?

I couldn't wait so I cloned the repo and wrote a single line that makes the trick.
Perhaps what I did is naive and wrong. But it's working.
I think it's worth taking a look at
Here you can see the changes
https://github.com/borisdj/EFCore.BulkExtensions/compare/master...MatiKloster:PropertiesToExcludeOnUpdatePostgres?expand=1

Thx for contrib. Fix is Merged, and test passes. Will be published with next v.

When will it be published?

NuGet v.6.5.1


Reduce indirection in treeshakeable client

We're now in a position to cut out the middleman. This makes things somewhat simpler, and makes go-to-definition work much better

(there's a case to be made that the code should just live in navigation.js in the first place instead of being re-exported from client.js, but we can explore that later. for now i'm being incremental)


Building tests for a binary crate in a virtual workspace doesn't work.

I'm working on this repository: https://github.com/purestake/moonbeam
It contains several crates. Some of them have tests, and some don't (yet). Most of our crates are libraries, but one (moonbeam which lives in the node directory) is a binary application. The moonbeam binary depends directly or indirectly on the library crates in the repo as well as several external dependencies. The root Cargo.toml is a virtal workspace.
Problem

The issue is with building the tests in the binary crate moonbeam. I expect cargo test to build all tests in the virtual workspace including moonbeam and all the libraries. I expect cargo test -p moonbeam to build just the tests in the moonbeam crate. However, when I run cargo test -p moonbeam I get a build error that I do not get when I just run cargo test
Steps

I've created a CI job to demonstrate this clearly and you can see it's output here https://github.com/PureStake/moonbeam/runs/2343542484 Notice I ran several cargo commands that build correctly before the cargo test -p moonbeam that fails.
You can also reproduce yourself.

git checkout https://github.com/purestake/moonbeam
cd moonbeam
cargo test (this will take a while, but the build will ultimately succeed)
cargo test -p moonbeam to see the failure

This is caused by using different features.  The features selected in a workspace depend on which packages are being built.  In this particular case, rand 0.5.6 is being built with the i128_support and rand_core 0.4.2 is built with alloc and std. Rand doesn't seem to like that combination.
You can inspect the features with cargo tree.  In the root of the workspace:
cargo tree -p rand:0.5.6 -f '{p} {f}'
rand v0.5.6 alloc,cloudabi,fuchsia-cprng,i128_support,libc,std,winapi
├── libc v0.2.91 align,default,extra_traits,std
└── rand_core v0.3.1 alloc,std
    └── rand_core v0.4.2 alloc,std

and then in the node directory:
cargo tree -p rand:0.5.6 -f '{p} {f}'
rand v0.5.6 i128_support
└── rand_core v0.3.1
    └── rand_core v0.4.2 alloc,std

If you want to see what is enabling the features, you can use the -i flag with -e features.  For example:
cargo tree -i rand:0.5.6 -f '{p} {f}' -e features shows how the features are set on rand in the workspace.
cargo tree -i rand:0.5.6 -f '{p} {f}' -e features -p moonbeam shows how the features are set on rand when building just moonbeam.

Thank you very much for the help and detailed explanation. It seems this is not a cargo bug so much as a learning opportunity for me, so I'll close this for now. (Might be back with more questions though).
cc @girazoki


Added accessibility features to mwc-textfield

As a follow-up of #1215.
The purpose of this PR is to make mwc-textfield more accessible, basically following https://github.com/material-components/material-components-web/tree/master/packages/mdc-textfield/helper-text#accessibility.

https://github.com/material-components/material-components-web-components/commit/b020c1fa8dd29d0c21e8758b022b65fe7fcc0a20 makes test suite work on my machine. Test engine would not start without.

https://github.com/material-components/material-components-web-components/commit/c3c45c17237c54abcefd7a505c7ab5b62746a932 is added to make use of chai a11y aXe testing lib (https://open-wc.org/testing/testing-chai-a11y-axe.html). The way it is embedded now is pretty ugly. Open to any better suggestions on how to hook this to the current testing library (I am not a all  used to typescript, so bare with me)

main work done under https://github.com/material-components/material-components-web-components/commit/0693e313ccece935d524efe7db87974cbcf476fe. Tell me what needs to change before I commit similar work to textarea.

Test failing on https://github.com/material-components/material-components-web-components/pull/1852/commits/0693e313ccece935d524efe7db87974cbcf476fe certainly because I used pnpm and not npm locally. I wish not to change this.

Thanks for the PR, and sorry for the delay on commenting!
We discussed this at our triage meeting last week, and our consensus is to hold off on implementing #1215 right now for a few reasons that are not immediately obvious:

Text field a11y must meet strict requirements internally and sync up to its internal implementation. While it may be possible to go back and forth on this PR right now to get it synced up, there are several outstanding bugs for the internal text field's a11y (some of which are being closed in a fixit this week), so it would quickly become out of sync. We'd like to find a better approach for keeping them in sync first or wait until the a11y story is more stable.
Design has stated that non-decorative actionable icons should display a ripple, since they are their own touch targets. Rather than adding more complex logic, we wish to implement this in MWC (and upstream in MDC) by using icon buttons. This would eliminate many of the requirements in the text field icon foundation and #1215 around icon a11y since that is built in to icon buttons already.
Changes to our build and test system are very difficult to bring in internally, and would require much more refactoring on this PR and effort that we don't think is beneficial right now for the short term given our current priorities.
The text field container element (which holds the icons) is expecting a refactor this quarter, which would make much of the above high effort work lost or redone.

We really appreciate the contribution, but given these reasons and our need to carefully upcoming work, we'd like to wait before addressing this outstanding bug.

Thanks a lot for the detailed answer, truly appreciated !
Beyond the specifics of this PR, I think it illustrates a series of concerns I was not sure where/how to state. So, I'll do it here, not without thanking the devs who are putting tremendous amount of quality in the code base.

It is difficult for me (us ?) to know where the library is heading and what is happening behind the scene. I am de-facto left with  proxy indicators at hands (like pace of development, type of PR being merged, milestones being pushed back) to guess what is happening. For the time being, they leave me intrigued.

Would it make sense to post some type of high-level outline of medium term vision / problems encountered / area of work / what to expect in next releases ... It would be interesting to know a bit more about thinking behind theming options, envisioned long-term relationship with MDC foundation... It would also be kind of reassuring. For instance, discussion in #1121 was helpful.

I understand the rationale behind "This repo is for extending the base layer with Web Components, and is not intended to be a complete replacement of MDC" (#223). I hope this is not on the expenses of loosing lit-element clarity and expressiveness and investing a large portion of the efforts just keeping in sync with upstream.

This is partially a consequence of 2 and potentially a costly one: it's hard to contribute! Except for very minor changes, it is often not clear where to intervene as the dependencies/multi-plexing with the base layer forces to digest multiple layers of overhead.

I came to select this library because of the pleasant experience playing with / extending / contributing to other lit-based components (also because of very good experience with using Polymer elements). It wish it could be more of the same here.
As a consequence I end up, like in the old times, with a series of fragile override for doing things the library does not yet care of.
I will now be closing this PR, as - again - reasons for not incorporating it makes sense, and open a couple follow-up issues (#1873, #1874).


Register IStorageService for Sloth.Jobs.CyberSource.BankReconcile

Forgot to register this dependency.
Job-specific appsettings.json may need to be updated for Storage settings in both test and prod.

@sprucely The one is this job is transient. Should they match?
https://github.com/ucdavis/Sloth/blob/master/Sloth.Jobs.Kfs.ScrubberUpload/Program.cs#L83


Fix parallel-e2e.sh to work on my macbook (bash v3.2)

My bash doesn't like "|&", so changing this back to the old gross way.

@zmerlynn

Stupid OS X. L"G"TM.

(And thanks for the fix. I didn't even realize this was such a new bash-ism.)


Install TagBot as a GitHub Action

You're receiving this pull request because the now-deprecated Julia TagBot GitHub App is installed for this repository. This pull request installs TagBot as a GitHub Action.
If this PR does not belong in this repository for any reason, feel free to close it.
See this Discourse thread for more details and to ask questions.
Please do not comment on this pull request, because it is unlikely that I (@christopher-dG) will see it.

Codecov Report

Merging #30 into master will increase coverage by 0.06%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master     #30      +/-   ##
=========================================
+ Coverage   80.74%   80.8%   +0.06%
=========================================
  Files          12      12
  Lines         296     297       +1
=========================================
+ Hits          239     240       +1
  Misses         57      57

Impacted Files
Coverage Δ

src/gmphd.jl
100% <0%> (ø)
:arrow_up:

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 d6e3f8d...a509925. Read the comment docs.

This should be done the new way without chron


[clang-format] Format all covered files

This should cover moving to the lowrisc toolchain's clang-format.

Let me know if you'd prefer to have these formatted if/when they materially change. This grouping together does make for more cumbersome rollback...


CAMEL-12969 : Map based Service Usage counting to remove memory leak

This showed comparable performance to using the service cache and thread in my previous PR.  The leak does not occur since the Queue has been replaced with a map of AtomicLongs to count usage.

Thanks for the PR. I am fixing the checkstyle and then backporting it


Bug: parse out version string properly in scripts

a version variable assignment like this: version = '0.1.1'  // package version, changed from 3.4.0 so it doesn't match cutadapt caused a problem, it should be 0.1.1 instead of 0.1.1'  // package version, changed from 3.4.0 so it doesn

addressed by #64


Only deploy products with changed files

This PR improves the workflow process by using the paths-filter workflow to conditionally build jobs only if a path has changed related to that product. It's WIP because not every product has been added yet—cc @adamschwartz for review before I keep hacking on it.

Wow this is so cool. Does it work? (Have you tested on a test repo?)

Wow this is so cool. Does it work? (Have you tested on a test repo?)

yep, it does! I just removed the draft status 👍
@adamschwartz gave me the OK to merge this so I'm going to do that shortly!


Add RunBash : Seamlessly Run Bash Scripts and Linux Binaries on Windows from Explorer, Cmd ,and PowerShell

What is RunBash?
RunBash is a handy utility that allows you to run Bash scripts and Linux binaries directly from your Windows system. It integrates seamlessly with both Windows Explorer and the Command Prompt, providing a versatile and efficient way to execute your scripts and binaries without needing a separate terminal or extra steps.
Key Features:

Direct Execution: Run your Bash scripts and Linux binaries directly from Windows Explorer or the Command Prompt. No need to open a separate terminal.
Linux Command Integration: Easily link and manage Linux commands within your Windows environment.
Context Menu Integration: Add options to the right-click context menu in Explorer, making it easy to execute scripts or commands from any directory.
Customizable SourceCode: Add any code you want to the main batch file (\ProgramData\RunBash\RunBash.bat) to adjust the execution to your needs.
Customizable Execution: Control output, error handling, and execution behavior with various parameters.
Root/Admin Access: Option to run scripts with root or admin privileges, providing the flexibility to handle system-level tasks.
Error and Output Handling: Fine-tune what outputs and errors are displayed or hidden, making debugging easier.

Why Use RunBash?
RunBash bridges the gap between Windows and Linux environments, allowing you to leverage the power of Bash and Linux tools without leaving your Windows workspace. Whether you're a developer needing to run cross-platform scripts or a power user looking to streamline your workflow, RunBash offers a robust solution, and get you out the headacke of changing every path in the arguments from windows based to Linux based.
Getting Started
To get started with RunBash, you can check out the repository on GitHub: benzaria/runbash.

Clone the Repo: git clone https://github.com/benzaria/RunBash.git
Run the Setup: Execute setup.bat to install and configure RunBash.
Start Using It: You can now run Bash scripts or Linux binaries directly from Explorer or the Command Prompt!

Thank you for considering RunBash for inclusion in the Awesome Windows list! 🚀

Just to inform you, this repo is dead, but I'm working on a remake here https://github.com/0PandaDEV/awesome-windows


refactor: remove auto pr opener assign action

What this PR does:

Remove the action responsible for auto-assigning the PR opener to the PR.
We used it in the past for easier project management (e.g., filtering in the project board), but since we no longer have projects, it is unnecessary.

@mergifyio refresh

@mergifyio refresh


Add support for sw-precache to the build

Unless there's a good reason not to. If there is a good reason not to, I'd like to try to address it in the library.
As it stands, the SW implementation will end up trying to re-fetch all the static resources each time the version is bumped, and there's no cache-busting URL parameter to protect against stale resources being returned from the HTTP cache during the install phase.
Depending on what the HTTP cache expiration is for the static resources, you'll either end up making unnecessary network requests (if your cache expiration is very short) or risk serving stale content from your service worker's fetch handler (if your cache expiration is longer).
sw-precache will version each resource automatically and only re-fetch those that change, appending a cache-busting URL parameter.

Good question!. The initial reason for not using sw-precache for this project was attempting to keep the architecture vanilla and not as heavily reliant on build-time tooling. That said, you're completely correct about the re-fetching issue and we could be a lot smarter with out approach to cache busting.
If we opt for sw-precache here, I think we should make using it either a core or optional part of the appshell + content documentation we ship so that we're consistent with how we present the approach. Any interest in a PR that adds in sw-precache here? :)

sw-precache support is probably the next item we'll want to tackle. I think once we've landed support for it, @robdodson (Polymer and c/o) should have a rough reference point they can follow for their progressive app.

:+1: I'd imagine that getting the dynamicUrlToDependencies settings correct and using navigateFallback might be the trickiest parts of migrating over to sw-precache, so please ping me if the documentation is unclear.
https://github.com/GoogleChrome/sw-precache/blob/app-shell-demo/app-shell-demo/gulpfile.babel.js#L92 might help as a reference.

We now have an implementation of this in master. I've already reviewed, but it might be useful to get @jeffposnick's eyes on it too in case there's further improvements we can make.

https://github.com/addyosmani/app-shell/blob/deabdc15701dfebb64d8c9158a45d00f5e97a013/gulp-tasks/service-worker.js#L30 gets a :+1: to me. Apologies that there were some rough spots getting the initial integration set up.
My only comment is that swPrecache.write() returns a promise now, so you can do away with the cb and just use the promise as the return value for the gulp task. Not that it changes anything functionally, but it "looks" nicer.

PR including the promise change in the works.


Removed duplicate Participate method

Removed duplicate Participate method from FileGrainStorage class

Docs Build status updates of commit b02de88:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

src/docs/tutorials_and_samples/custom_grain_storage.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Support InfluxDB v2 Auth (ie: InfluxDB Cloud)

It would be amazing if Unpoller support token based authorization for InfluxDB Cloud. According to the documentation for the InfluxDB Cloud (https://docs.influxdata.com/influxdb/cloud/reference/cli/influx/v1/auth/create/) v1 username and password authorization mechanisms isn't support on InfluxDB Cloud
This represents a feature request to update the authorization mechanism to support token based authorization.

Already should work with the auth_token option for influx 2+

Thanks, I'll have to take a look at the code then, since it appears its not documented.


Minimize the code to review when updating the API server based on a new OpenAPI document

Related to https://github.com/Sage-Bionetworks/research-benchmarking-technology/issues/7
@thomasyu888 Could you give a try to adding an OpenAPI Generator template to this microservice to attempt to make its update easier?

Putting PR #96 aside for now.


Issues to be addressed

For information purposes, here's a quick rundown of the things I've discovered and that I'll try to work on in the coming days. Please let me know if there's any related input from anyone's side.

API Keys are not being used in this library, yet are required for most endpoints
There's no sufficient way to determine the "success" state of a call made to FusionAuth API. Other FusionAuth client libraries (i checked PHP and Python) attribute "success" to a response code 200-299 from FusionAuth API. Neither does this library pass through the HTTP status code.
CreateUser as an example for many similar occurencies: the first parameter (userId) is considered optional. If omitted, the FusionAuth API will create its own user ID. Yet, as is, this library does not support this optional behaviour.
Recently, the FusionAuth API has changed significantly, e.g. for creating tenants. It probably makes sense to introduce a Tenant struct (like the existing User struct), and maybe others.

I'm gonna try to address these things and possibly others in upcoming PRs.

Thanks @markschmid
Lots of good points. If you would like to contribute, the client is built using a template, and we can easily build the full domain using a template as well. For someone who knows Go, this should be fairly straight forward.
See
https://github.com/FusionAuth/fusionauth-client-builder
Here is the Go template:
https://github.com/FusionAuth/fusionauth-client-builder/blob/master/src/main/client/go.client.ftl
The .NET Core example is building domain and the client, see for example:

netcore.client.ftl
netcore.domain.ftl

The recent change (https://github.com/FusionAuth/fusionauth-go-client/commit/bdb6660c58bcbcda4fe8630017bad08b4610757b) has addressed bullet number 4.
Time to close this issue, thanks.


hyperlink with query arguments corrupts workbook

Found quite nasty bug.
If you include xml unsafe characters into url, resulted workbook is not readable,
this is especially true if you have query params where & is a separator.
This is due to relation target stored in the xml attribute.
following chars must be xml encoded:
"
'
<
>
&

@maksimlikharev I've just tried to reproduce this and I'm having trouble...
The following (unlikely) hyperlink saves fine and opens in Excel:
    ws.getCell('A1').value = {
      text:'Somewhere with query params',
      hyperlink: 'www.somewhere.com?a=1&b=2&c=<>&d="\'"'
    };

Could you post some example code that causes your issue?

This is what we do.
exceljs: 0.3.0
One important detail I forgot to mention, this is streaming mode.
let rowData = {};

rowData['url'] = { text: 'text', hyperlink: 'http://localhost/getsome?sss=1&bbb=2&ccc=<>' }

this.workbook.getWorksheet(1).addRow(rowData);

row.commit();


fix: [Docs Bug 🐞 report]: #3167

Description
This pull request addresses a documentation bug. The changes ensure that the documentation now accurately reflects the functionality and usage of the affected components.
I have followed the contribution guidelines.
I have tested the changes to ensure they resolve the issue.
Please review the updated documentation and let me know if any additional improvements are needed.
screenshots are added below:-

Related issue(s)

closing this because of no response


Sub repo libnvram being public should have https url in .gitmodules

Since nvram is public and uses public sub repo references should be made using https.
Moving nvram to another mirror breaks since sub-repo libnvram requires ssh keys,

Fixed in: 164b1d276cfc75ce1fda546f1d06b4ebeb7027a6
nvram v2.1.1