nixos/plymouth: use bgrt theme

Motivation for this change
Closes https://github.com/NixOS/nixpkgs/pull/84158.
In a followup I'd like to pick up or encourage @puckipedia towards https://github.com/NixOS/nixpkgs/pull/88789. LUKS+ZFS is a tentative goal to having plymouth working entirely as expected.
This pull request focuses on getting "FlickerFree Boot" with the BGRT theme, an initiative from Fedora https://fedoraproject.org/wiki/Changes/FlickerFreeBoot, that now (as usual because RedHat values) other projects can benefit from. Most recently (IIRC), Ubuntu is using the master branch of plymouth. Whether plymouth is ready for release is debated, but if you look at this pull request certain polishing upstream is needed, but if you want it we certainly can use it.
I tested this pull request on a machine with using i915 and it worked pretty well, nothing seemed weird.
Here's an actual example of how the bootsplash should look: https://ic.pics.livejournal.com/hansdegoede/13347631/433/433_900.png, with a vendor image, a logo, and spinner.
If you want to see examples in other situations the mockup is useful: https://gitlab.gnome.org/Teams/Design/os-mockups/-/blob/master/boot/boot-progress.png, but they changed the shutdown to be the same as the startup.
Todo's:

[x] Need a better logo available

~We need something monochromatic, the current image we have looks too big and draws too much attention to itself.
Mentioned this in https://github.com/NixOS/nixos-artwork/pull/58.~

[x] Switch to the white version of the nixos logo https://github.com/NixOS/nixpkgs/pull/114276

[x] Merge https://github.com/NixOS/nixos-artwork/pull/59 and update nixos-artwork

[x] We need to sync the GDM logo with this logo change for a smooth transition https://github.com/NixOS/nixpkgs/pull/114387

[ ] Testing for AMD+Nvidia users would be helpful but isn't required

[x] logo doesn't show up at shutdown
In initrd we copy these logo's, but on the actual system we
would need to use environment.etc. However, when I try to do this I get ln permission denied errors.  It's also rather confusing that in initrd everything is in $out/share,  but on the running system it's /etc. ~This might require patching all theme files ImageDir to /run/current-system, and using runCommand to get an installable derivation with the logos?
It seems the source code attempts to change directory to the location of the image files, so no XDG_DATA_DIRS.~
Why doesn't plymouth use the logo path it's configured to use?

Questions:
In pull request the following nix code is added:
boot.initrd.availableKernelModules = [ "i915" ];

also part of the work done by Fedora to get fastboot enabled for Intel users.
This sort of thing is a part of nixos-hardware, but I think ideally nixos really should have some base profiles that nixos-generate-hardware detects and adds like https://github.com/NixOS/nixos-hardware/blob/0cb5491af9da8d6f16540ffc4db24e2361852e46/flake.nix#L77
Perhaps in the release note nixos-hardware can be mentioned or just the configuration for intel users if they're not already using it.
Thanks
@eadwu and @andersk were motivation to finally fix this
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
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 114000 at feb0443e run on aarch64-linux 1

1 package marked as broken and skipped:

xmonad_log_applet

26 packages built:

adapta-gtk-theme
chrome-gnome-shell
gnome3.gdm
gnome3.gnome-applets
gnome3.gnome-flashback
gnome3.gnome-initial-setup
gnome3.gnome-panel
gnome3.gnome-session (gnome3.gnome_session)
gnome3.gnome-shell (gnome3.gnome_shell)
gnome3.gnome-terminal (gnome3.gnome_terminal)
gnome3.gnome-tweaks (gnome3.gnome-tweak-tool)
gnome3.pomodoro
gnomeExtensions.easyScreenCast
gnomeExtensions.gsconnect
libsForQt5.breeze-plymouth (libsForQt515.breeze-plymouth ,plasma5Packages.breeze-plymouth)
libsForQt512.breeze-plymouth
libsForQt514.breeze-plymouth
lightdm
lightdm-enso-os-greeter
lightdm-mini-greeter
lightdm-tiny-greeter
lightdm_gtk_greeter
lightdm_qt
pantheon.elementary-greeter
pantheon.elementary-session-settings
plymouth

1 suggestion:

warning: unclear-gpl
gpl2 is a deprecated license, check if project uses gpl2Plus or gpl2Only and change meta.license accordingly.
Near pkgs/os-specific/linux/plymouth/default.nix:115:5:
    |
115 |     license = licenses.gpl2;
    |     ^

AMD GPU
https://youtu.be/88boyDEVKDs
I still see stage1 console output.

AMD GPU
https://youtu.be/88boyDEVKDs
I still see stage1 console output.

We still see stage-1 because plymouth can't start that early because we don't have https://github.com/NixOS/nixpkgs/pull/74842#issue-347785052

Yes, the splash screen disappeared, but also it's loading time. It seem a lot faster now.
The "NixOS" you see shortly at the start is from GRUB. So it boots in BIOS mode.
Is what you see in the video the expected behavior except not showing the start splash?
I also have an Nvidia card and will test that too.
Let me know if i can help with testing anything else.

Yes, the splash screen disappeared, but also it's loading time. It seem a lot faster now.
The "NixOS" you see shortly at the start is from GRUB. So it boots in BIOS mode.
Is what you see in the video the expected behavior except not showing the start splash?
I also have an Nvidia card and will test that too.
Let me know if i can help with testing anything else.

It is expected behavior, BGRT is a UEFI thing. Nvidia in UEFI would also be helpful.

The "NixOS" you see shortly at the start is from GRUB. So it boots in BIOS mode.

I'm not sure I follow, (sorry it is a bit tangentially off-topic) GRUB can be used, and is used, both with UEFI and Legacy boot.

As part of fedora's documentation, they list the planned changes they had to make to GRUB, with a user FAQ.
So it should be perfectly usable with GRUB, if we bring in the required patches (assuming they are not in upstream, since development upstream is not quick).

@worldofpeace am I understanding right that this has only been verified and tested with systemd-boot?
Note that it is not an issue if it is the case. But it is something to know.

As part of fedora's documentation, they list the planned changes they had to make to GRUB, with a user FAQ.
So it should be perfectly usable with GRUB, if we bring in the required patches (assuming they are not in upstream, since development upstream is not quick).
@worldofpeace am I understanding right that this has only been verified and tested with systemd-boot?
Note that it is not an issue if it is the case. But it is something to know.

Ah, I should have mentioned. I use GRUB. Ahh, I've never actually noticed https://hansdegoede.livejournal.com/19081.html, but it's good that you mention this.
It seems particularly difficult to pick through what Fedora does with grub2 https://src.fedoraproject.org/rpms/grub2/tree/rawhide... They have a 301 patch series :grin:  !
A quick find and I spot two patches https://src.fedoraproject.org/rpms/grub2/blob/rawhide/f/0129-Add-auto-hide-menu-support.patch https://src.fedoraproject.org/rpms/grub2/blob/rawhide/f/0170-grub.d-Split-out-boot-success-reset-from-menu-auto-h.patch, so it appears that they have this patched in. It appears from other threads that they have their GRUB done up specifically and there's kernel parameters like the log level etc. that also are being configured.
We should probably split this off as some other tangible sub tasks. I also did just peruse the grub ml archive because I was wondering if they sent these patches upstream, but I'm not very good at that :cat:

All my required todo's are now done :sparkles:
More review and testing is welcome, but I believe it's pretty much "complete".

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/what-are-your-goals-for-21-05/11559/7

Planning to merge this in a few days. Has been running totally fine on my machine.

Do I only need this to test with nvidia?
{
  boot.plymouth.enable = true;
  initrd.availableKernelModules = [ "nvidia" ];
}

I don't see plymouth while booting. I'm using systemd-boot.

I just tested on my laptop (i915). As expected (I think), the splash is only visible after I input the LUKS password and when I shutdown.
It's pretty :) Great work!
I wonder if plymouth could be visible while selecting the nix generation with systemd-boot.

the splash is only visible after I input the LUKS password and when I shutdown.

Yep, once we have https://github.com/NixOS/nixpkgs/pull/88789 we'll have that through plymouth also.

Oh, I was able to test with nvidia proprietary with:
{
    initrd.availableKernelModules = [ "nvidia" "nvidia_modeset" "nvidia_uvm" "nvidia_drm" ];

    kernelParams = [
      "nvidia-drm.modeset=1"
      "quiet"
      "splash"
    ];
}

I had some console text, then plymouth was visible, a black screen and then gdm.
Do you want me to open a new issue for that? and maybe wait for the PR to be in unstable.

Do you want me to open a new issue for that? and maybe wait for the PR to be in unstable.

"If you have an AMD or Nvidia GPU driving your screen, then this is normal. The graphics drivers for AMD and Nvidia GPUs reset the hardware when loading, this will cause the display to temporarily go black. There is nothing which can be done about this."

https://hansdegoede.livejournal.com/20632.html

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/what-are-your-goals-for-21-05/11559/18


Manually update the RESEARCH AND DATA NEEDS

This should probably come at the end after we've sorted everything else out, just reminding ourselves to remember to do it. I think there's no way this section can every be automated because it reflects the current thinking of the humans involved, not model results.

I made an initial stab, but didn't end up changing much as most of all the good stuff there still stands.  Anything else to add?

I prob won't think of anything...

Haven't read Aaron's text yet, but I'm sure I'm fine with it. However, let's check in about it on the phone Monday (tomorrow) if y'all are up for a 10am call.

Yep, talk then

Chris and I are good for 10am call.


Add pipe_text command

pipe_text is a simple demonstration of how one can pipe the contents of the selection(s) or active buffer to the "stdin" of an external shell command, and replace those selections with the "stdout" returned from said program.
This is useful for things like formatting XML, JSON etc. when the reindent command can't be used (i.e. because all the text is on one line etc.) and without having to wait for some slow Python program to parse the text, or manually doing a find and replace...
This relies on a Python method which was added to the standard library in Python 3.5, so one needs to be using a ST 4xxx build, and to drop the plugin into the User package, or otherwise have the appropriate .python-version file in the package so that the Python 3.8 plugin host will be used.

thanks for the reviews!


Configure Story Map

Hi guys i have problems to configure my story map when i try to open it in the browser it is not working.
I already tryed many tutorials but it is still not working.
Here is my code:

	<meta name="apple-mobile-web-app-capable" content="yes">
	<meta name="viewport" content="width=device-width, initial-scale=1.0, maximum-scale=1.0, user-scalable=0">

	<link type="image/ico" rel="shortcut icon" href="//resources.esri.com/favicon.ico">
	<link type="image/ico" rel="icon"  href="//resources.esri.com/favicon.ico">

	<!--
		To correctly reference your Map Tour in search engine:
		 - create and fill out extensively an ArcGIS.com item that link to your final application
		 - edit the following four tags as well as the upper title tag on line 4
	-->
	<meta name="description" content="This story map was created with the Esri Map Tour application in ArcGIS Online.">

	<!-- Facebook sharing -->
	<meta property="og:type" content="article"/>
	<meta property="og:title" content="Story Map Tour"/>
	<meta property="og:description" content="This story map was created with the Esri Map Tour application in ArcGIS Online."/>
	<meta property="og:image" content="resources/icons/esri-globe.png"/>

	<!--
		This template is released under the Apache License V2.0 by Esri http://www.esri.com/
		Checkout the project repository on GitHub to access source code, latest revision, developer documentation, FAQ and tips
		https://github.com/Esri/map-tour-storytelling-template-js
	-->

	<script type="text/javascript">
		//----------------------------------------------------
		//     Template configuration (ignored on Portal)
		//----------------------------------------------------
		configOptions = {
			// Enter either a web map ID or a web application ID
			// - Using a Map Tour application ID created through the interactive builder in ArcGIS.com allow to reuse
			//   the settings like the header configuration without having to edit any configuration here
			// - Web map and application can be dynamically specified through url parameters (webmap and appid)
			//   if the authorizedOwners property is specified (see below)
			// - If an application id is specified, the webmap configuration isn't used
			webmap: "",
			appid: "10129a28a5644de18c2a1559df60612e",
			// Optional list of authorized application or web map owners
			// Has to be specified when using appid or webmap url parameters (e.g. ["user1"], ["user1", "user2"])
			authorizedOwners: ["Betzen"],
			// Select the application layout between "three-panel" and "integrated"
			layout: "three-panel",
			// The application header title, if no title is specified, the webmap's title is used
			title: "FH Map",
			// The application header subtitle, if not specified the ArcGIS.com web map's summary is used
			subtitle: "Students internships",
			// Optionally force a web map layer to be used as the tour points data source (can be overridden through URL)
			// Otherwise the upper visible point layer is used (excluding map notes)
			sourceLayerTitle: "Points",
			// Use the first data record as an introduction instead of a point (can be overridden through URL)
			firstRecordAsIntro: true,
			// Optional zoom level to be applied for the story points following introduction (can be overridden through URL) (e.g: 0 to 16 for Gray Canvas, 0 to 19 for Street Map, -1 to disable)
			zoomLevel: -1
		};
		// Optionally sharing and proxy URLs can be configured in app/maptour-config.js. This is only required
		//  when the webmap is not hosted on ArcGIS Online or a Portal for ArcGIS instance and the template isn't deployed as /home/MapTour/ or /apps/MapTour/.
		// Optionally Bing Maps key, Geometry and Geocode service's URLs can be configured in app/commonConfig.js. This is only required
		//  if the Organization or Portal for ArcGIS instance default configuration has to be overwritten. If the application is deployed
		//  on Portal for ArcGIS as /home/MapTour/ or /apps/MapTour/, that configuration is available in ../commonConfig.js
	</script>

Would be awesome if someone of you could help me
Thank you

Duplicate #42


Fix #1256: duplicated Content-Type header in SageMaker Runtime

As I described in #1256, setting rusoto_sagemaker_runtime::InvokeEndpointInput.content_type produces two content-type headers which is not correct.

Just that one change and this is good to go.  😄


Upgrade Firebase

The old version of firebase depends on the library of the closed JCenter, which makes it unable to compile

Had to revert this since it broke the build. JCenter isn't down yet so it sould be fine for a while. Weirdly it worked locally but I have had this same build issue before when trying to update these libraries.

Maybe there are some build caches to build successfully?
Howard

From: jobobby04 @.>
Sent: Thursday, November 3, 2022 1:14:52 AM
To: jobobby04/TachiyomiSY @.>
Cc: Howard Wu @.>; Author @.>
Subject: Re: [jobobby04/TachiyomiSY] Upgrade Firebase (PR #715)
Had to revert this since it broke the build. JCenter isn't down yet so it sould be fine for a while. Weirdly it worked locally but I have had this same build issue before when trying to update these libraries.
—
Reply to this email directly, view it on GitHubhttps://github.com/jobobby04/TachiyomiSY/pull/715#issuecomment-1300948161, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJRNWK33G3HLT2363LDPM4DWGKOQZANCNFSM6AAAAAARSYB2HE.
You are receiving this because you authored the thread.Message ID: @.***>

I deleted all build caches in the preview repo and tried again but it didn't work, so no idea


Landing page changes for V1 to V2 migration - changes for Project Login & CLA Manager Login links on LFX Header

wireframe for V1/V2 redirection from landing page.
https://wireframepro.mockflow.com/view/platform-style-guide#/page/Dfcc3e0be0361ae66dbf30f238a7814ad
User will click on Project Login & CLA Manager Login hyperlinks and that will open the dialog box which will give the user an option to choose between V1 and V2.
If you already know that the user is coming from V1 or V2 you can put that logic and not show this option. Wherever you don't know what version the user is in, you should show this option.

Note: This ticket has a dependency on https://jira.linuxfoundation.org/browse/IAM-1310 as there is a need of click event support on the LFX Header hyperlinks.

@amolsontakke3576  also do the following:
Change the option "LFX Version 1" to "EasyCLA Version 1" and
change the option  "LFX Version 2" to "EasyCLA Version 2".

@amolsontakke3576  also do the following:
Change the option "LFX Version 1" to "EasyCLA Version 1" and
change the option  "LFX Version 2" to "EasyCLA Version 2".

Issue is tested on dev/staging site & found below results

Dialog box open which will give the user an option to choose between V1 and V2 when user click on Project Login & CLA Manager Login hyperlinks.
"LFX Version 1" text is properly updated to "EasyCLA Version 1"
"LFX Version 2" text is properly updated to "EasyCLA Version 2".

Refer video clip for more details
https://images.zenhubusercontent.com/194341141/44644110-46d7-44e9-becf-02b2ade17fc2/2492.mp4

Issue is tested on dev/staging site & found below results

Dialog box open which will give the user an option to choose between V1 and V2 when user click on Project Login & CLA Manager Login hyperlinks.
"LFX Version 1" text is properly updated to "EasyCLA Version 1"
"LFX Version 2" text is properly updated to "EasyCLA Version 2".

Refer video clip for more details
https://images.zenhubusercontent.com/194341141/44644110-46d7-44e9-becf-02b2ade17fc2/2492.mp4

Issue #2492 is opened for the case where user comes from GitHub & based on the parameter, user should directly be taken to corresponding version console if the user clicks on Proceed or the links on LFX Header. So, marking this as QA Done.

Issue #2492 is opened for the case where user comes from GitHub & based on the parameter, user should directly be taken to corresponding version console if the user clicks on Proceed or the links on LFX Header. So, marking this as QA Done.

This one is deployed to PROD as well.

This one is deployed to PROD as well.


OOM killed

Description:
watch some HTTPRoute that has some error. maybe the service does not exist. the EG has been killed due to  Reconcile them.
Logs:

IMO, should set RequeueAfter to requeue

I did not reproduce it, can you provide the steps to reproduce it ? @qicz

Tried that and just HTTPRoute reported BackendNotFound, the eg works still well

Tried that and just HTTPRoute reported BackendNotFound, the eg works still well

this report too often and there are more invalid HTTPRoute, the EG has been killed due to Reconcile them.

@qicz I'm facing same error here.
But My usage is setting about ~1300 HTTPRoute CR with about ~20 Gateway with mergeGateway=true.

May be it's not non-exists backends cause eg oom, but the count of gateway api crs cause eg oom, I'm facing that deployment envoy-gateway pod eats too many memory.

default eg memory limit is 1g, you can change this to unlimited, but the problem is still the problem.

@qicz in your logs, can you please paste the entire log showing the namespace and name of service, along with kubectl info on the service as well the httproute that is linking to it ?

@qicz in your logs, can you please paste the entire log showing the namespace and name of service, along with kubectl info on the service as well the httproute that is linking to it ?

@arkodg sorry reply slowly. the namespace and service are from my company app, so they have been cleared by me.

@qicz I'm facing same error here. But My usage is setting about ~1300 HTTPRoute CR with about ~20 Gateway with mergeGateway=true.
May be it's not non-exists backends cause eg oom, but the count of gateway api crs cause eg oom, I'm facing that deployment envoy-gateway pod eats too many memory.  default eg memory limit is 1g, you can change this to unlimited, but the problem is still the problem.

in my case, there are only ~30 HTTPRoute.  but can not set the memory to unlimited, it is bad for the Kubernetes cluster

in my case, there are only ~30 HTTPRoute. but can not set the memory to unlimited, it is bad for the Kubernetes cluster

No need to be unlimited, but some thing larger for routes is enough.
But as always, it must be some thing wrong with oom here.

@qicz @zzjin can you outline steps to reproduce the problem, from this chat its hard to understand what the trigger is

The analysis concludes that the OOM problem is that there are many secrets and the MEM limit is not set properly.

suggestion: using protobuf connect to Kubernetes to optimize the mem. xref #1596

@qicz @zzjin can you outline steps to reproduce the problem, from this chat its hard to understand what the trigger is

The analysis concludes that the OOM problem is that there are many secrets and the MEM limit is not set properly.

May be that's the problem, our cluster we have about ~3000 ingress with https,witch means about ~3000 secrets.

@qicz can you share mem stats of EG before & after https://github.com/envoyproxy/gateway/pull/1596 ?

closing due to no response, please reopen if you hit this issue again

Hi @arkodg, I've hit the same issue.
It seems like the envoy gateway is creating infinite HTTPRoutes for the HTTP01 challenge, while the challenge is not satisfied. My (unproven) theory is that it is provisioning the HTTPRoute resource with generate_name instead of using a predictable name, and this causes an infinite reconciliation loop.

thanks for debugging this one @miguelvr , cross linking the cert-manager issue here https://github.com/cert-manager/cert-manager/issues/7176
@envoyproxy/gateway-maintainers should we consider something like envoy's overload manager  where we stop reconciling more resources (flag this in a GatewayClass status) in case we hit some specified memory threshold ?


Should we add ngi-nix/ngipkgs to nixos search?

It looks to be as simple as adding it here https://github.com/NixOS/nixos-search/blob/main/flakes/manual.toml and it looks like there are some other ngi-nix org repos there already.
Perhaps this could have helped us avoid https://github.com/ngi-nix/summer-of-nix/blob/a640251153b4d20c1aa3c05ba95be077d635dede/2023/report.md?plain=1#L53-L54

I'm not sure if it would really have helped avoid the duplicated work, because it was simply an oversight by me that could have been avoided by more diligent review of sources.
I'm also not sure the flakes search actually works. At least I can't find anything with it.
I hope that https://github.com/ngi-nix/ngipkgs/issues/166 would at least partially address the problem at least partially automated.


test: two small tweaks to the tests

I keep finding small improvements, I hope the stream of PRs isn't a nuisance.
Merge checklist:
Check off if complete or not applicable:

[ ] Changelog record added
[ ] Documentation updated (not only docstrings)
[ ] Fixup commits are squashed away
[ ] Unit tests added/updated
[ ] Manual testing instructions provided
[ ] Noted any: Concerns, dependencies, deadlines, tickets

What prompted the changes to the hardcoding test?

@timmc-edx Are you still waiting on something from me?

Yeah, not sure if you saw this comment:

What prompted the changes to the hardcoding test?

I'm wondering what the failure mode was that prompted this (whether observed or hypothetical).

If you mean counting the right and wrong lines, I was curious about how many lines there were, and figured there should be something more than "no bad lines."  There should also be some good lines.

Got it, makes sense.


Hide Android Notification

I have looked for RN background library, I think this is the best. But I don't want to show push notification on android, does someone can explain it?

@agunbuhori, I am sorry, but the notification is mandatory in Android for running background notifications.


Feature: Expose bhe_saml_sp_key and bhe_saml_sp_cert via ENV variables

Feature Description
Expose bhe_saml_sp_key and bhe_saml_sp_cert via ENV variables, so that they can be managed via docker secrets or AWS SecretsManager or equivalent for other platforms.
Are you intending to implement this feature?
no
Current Behavior
bhe_saml_sp_key and bhe_saml_sp_cert must be set in the config-file. Which means that you have to put a private key in plaintext in a static config file, which might be managed via git. This is considered bad practice (at least in our organization), as secrets should only be available at runtime.
Desired Behavior
bhe_saml_sp_key and bhe_saml_sp_cert can be controlled via ENV variables, so that they can be provided only at container runtime and can be stored via the platforms secrets management facility.
Use Case
The features increases the security of bhe_saml_sp_key, makes management of BH easier and makes BH more compliant with security best practices.
Additional Information
First mentioned in Issue 83.

Any news on this? :)


Container Name field

Is there a field/way to get the container name from either the id or the attribute of the container itself? I can't seem to find a field for the name.

Docker::Container.get(id).info["Name"].


Fix image reference by pointing to quay.io

image is set to quay.io/multicloudlab/multicloud-operators-placementrule
Signed-off-by: Richard Su<EMAIL_ADDRESS>

Hi @rwsu
Thanks for your PR. After reviewing, we prefer to remain the image: REPLACE_IMAGE as what it is.  This is to align with the operator SDK guidance about how to create and deploy an app-operator.
https://github.com/operator-framework/operator-sdk#create-and-deploy-an-app-operator
As shown there, the REPLACE_IMAGE is there until the app operator installation, in which it is replaced with the real app image in quay.io.
# Update the operator manifest to use the built image name (if you are performing these steps on OSX, see note below)
$ sed -i 's|REPLACE_IMAGE|quay.io/<username>/app-operator|g' deploy/operator.yaml


Settings.json keeps reseting

Settings.json keeps reseting to default even if i start a new one from scratch.
note:
I used the listed here as my "template"
Commit hash:
Version: 2.0.6
Commit hash: 60846f5
Compiled on June 24, 2020
OS: MacOS Catalina
Terminal: iTerm2

I have the same problem on Windows and Linux

The behavior of settings.json was changed in 2.0.5 to be similar to bindings.json in that only modifications from the default are stored. Any default settings are automatically removed from the config.


Fix #668 strip lang in code fences sent to Slack

Just had the chance to implement #668. handleSlack appears to be converting Slack messages to the central config.Message format, so I hope I've found the right place!

Thanks!


Guid time

@jbaxleyiii @benwiley does this fix the sync?

@johnthepink until we get norman and apollos-rock up and running no telling. BUT it look like it should!


Add README for projects that are currently without (fishtornado, 28_gltf)

It seems like fishtornado(xr) and 28_gltf projects currently don't have a README. It might be useful to add, particularly to document additional command line arguments such as the ones added in  https://github.com/google/bigwheels/pull/153

Can I get assigned on this Task

Sure thanks! 😊


Questions about the several datasets provided in your code

Hello, could you please send me an introduction to the VIDEO and Wikipedia datasets provided in your code? If you are using the datasets from previous papers, could you please send me that article? Thank you very much.

Pls check the https://github.com/kang205/SASRec repo and https://cseweb.ucsd.edu/~jmcauley/datasets.html by yourself(good luck), to be clear, I re-implemented the model, ported it from tf to pytorch, instead of implementing the model and published the paper. BTW, may I know why u need these datasets, and why could u find it before?

Thank you for your answer. I will investigate the commonly used dataset in the recommended sequence, and the VIDEO dataset in SASrec's job should be Amazon Games. Thank you again for your answer.
?
@.***

------------------ 原始邮件 ------------------
发件人:                                                                                "pmixer/SASRec.pytorch"                                                                                @.>;
发送时间: 2023年7月24日(星期一) 上午10:00
@.>;
<EMAIL_ADDRESS>主题: Re: [pmixer/SASRec.pytorch] Questions about the several datasets provided in your code (Issue #37)
Pls check the https://github.com/kang205/SASRec repo and https://cseweb.ucsd.edu/~jmcauley/datasets.html by yourself(good luck), to be clear, I re-implemented the model, ported it from tf to pytorch, instead of implementing the model and published the paper. BTW, may I know why u need these datasets, and why could u find it before?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Can't use CTCBeamSearchDecoder in c++, LINK ERROR occur,BUG in CTCBeamSearchDecoder 's source code

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):NO
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10 Pro 1803
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:N/A
TensorFlow installed from (source or binary): from source
TensorFlow version (use command below): r1.4  and r1.10
Python version: 3.6
Bazel version (if compiling from source): cmake 3.12.2
GCC/Compiler version (if compiling from source): VS 2015
CUDA/cuDNN version: N/A
GPU model and memory: N/A
Exact command to reproduce:N/A

Describe the problem
I want to use CTCBeamSearchDecoder , but there is a link error when I call CTCBeamSearchDecoder  constructor  function(CTCBeamSearchDecoder<> decoder(num_classes, 10 * top_paths, &default_scorer);).  And I comment the constructor  function, the error disappear.  I test the r1.4 tensorflow.lib complied by myself with cmake and r1.10 download from github . So I think there might be a bug in CTCBeamSearchDecoder  's source code.
Source code / logs
This is the test code which is from ctc_beam_search_test.cc. The code is very simple.
#include "third_party/eigen3/Eigen/Core"
//#include "tensorflow/core/platform/env.h"
#include "tensorflow/core/util/ctc/ctc_beam_search.h"
#include <vector>
void test()
{
	using tensorflow::ctc::CTCBeamSearchDecoder;
	using tensorflow::ctc::CTCDecoder;
	const int batch_size = 1;
	const int timesteps = 5;
	const int top_paths = 3;
	const int num_classes = 6;

	// Plain decoder using hibernating beam search algorithm.
	CTCBeamSearchDecoder<>::DefaultBeamScorer default_scorer;
        //COMMENT AND NO ERROR
	CTCBeamSearchDecoder<> decoder(num_classes, 10 * top_paths, &default_scorer);

	// Dictionary decoder, allowing only two dictionary words : {3}, {3, 1}.
	//DictionaryBeamScorer dictionary_scorer;
	//CTCBeamSearchDecoder<HistoryBeamState> dictionary_decoder(num_classes, top_paths, &dictionary_scorer);

	// Raw data containers (arrays of floats, ints, etc.).
	int sequence_lengths[batch_size] = { timesteps };
	float input_data_mat[timesteps][batch_size][num_classes] = {
		{ { 0, 0.6, 0, 0.4, 0, 0 } },
		{ { 0, 0.5, 0, 0.5, 0, 0 } },
		{ { 0, 0.4, 0, 0.6, 0, 0 } },
		{ { 0, 0.4, 0, 0.6, 0, 0 } },
		{ { 0, 0.4, 0, 0.6, 0, 0 } } };

	// The CTCDecoder works with log-probs.
	for (int t = 0; t < timesteps; ++t) {
		for (int b = 0; b < batch_size; ++b) {
			for (int c = 0; c < num_classes; ++c) {
				input_data_mat[t][b][c] = std::log(input_data_mat[t][b][c]);
			}
		}
	}

	// Plain output, without any additional scoring.
	std::vector<CTCDecoder::Output> expected_output = {
		{ { 1, 3 },{ 1, 3, 1 },{ 3, 1, 3 } },
	};

	// Dictionary outputs: preference for dictionary candidates. The
	// second-candidate is there, despite it not being a dictionary word, due to
	// stronger probability in the input to the decoder.
	std::vector<CTCDecoder::Output> expected_dict_output = {
		{ { 3 },{ 1, 3 },{ 3, 1 } },
	};

	// Convert data containers to the format accepted by the decoder, simply
	// mapping the memory from the container to an Eigen::ArrayXi,::MatrixXf,
	// using Eigen::Map.
	Eigen::Map<const Eigen::ArrayXi> seq_len(&sequence_lengths[0], batch_size);
	std::vector<Eigen::Map<const Eigen::MatrixXf>> inputs;
	inputs.reserve(timesteps);
	for (int t = 0; t < timesteps; ++t) {
		inputs.emplace_back(&input_data_mat[t][0][0], batch_size, num_classes);
	}

	// Prepare containers for output and scores.
	std::vector<CTCDecoder::Output> outputs(top_paths);
	for (CTCDecoder::Output& output : outputs) {
		output.resize(batch_size);
	}
	float score[batch_size][top_paths] = { { 0.0 } };
	Eigen::Map<Eigen::MatrixXf> scores(&score[0][0], batch_size, top_paths);
}
int main()
{
	test();
	return 0;
}

This is complier 's error log.
error LNK2001 unresolved external symbol  "void __cdecl tensorflow::internal::MakeCheckOpValueString<unsigned char>(class std::basic_ostream<char,struct std::char_traits<char> > *,unsigned char const &)" (??$MakeCheckOpValueString@E@internal@tensorflow@@YAXPEAV?$basic_ostream@DU?$char_traits@D@std@@@std@@AEBE@Z)	test1	C:\Users\46099\Desktop\testtf\test1\test1\main.obj	1

Nagging Assignee @ebrevdo: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ebrevdo: It has been 29 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Is this still an issue?
On Sat, Nov 10, 2018, 10:48 AM Alfred Sorten Wolf<EMAIL_ADDRESS>wrote:

Nagging Assignee @ebrevdo https://github.com/ebrevdo: It has been 29
days with no activity and this issue has an assignee. Please update the
label and/or status accordingly.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/22894#issuecomment-437608991,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABtimyMckf_pCXq8fjajAdDS2dENYJ5sks5utx9ygaJpZM4XXZTK
.

Is this still an issue?
…
On Sat, Nov 10, 2018, 10:48 AM Alfred Sorten Wolf @.*** wrote: Nagging Assignee @ebrevdo https://github.com/ebrevdo: It has been 29 days with no activity and this issue has an assignee. Please update the label and/or status accordingly. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#22894 (comment)>, or mute the thread https://github.com/notifications/unsubscribe-auth/ABtimyMckf_pCXq8fjajAdDS2dENYJ5sks5utx9ygaJpZM4XXZTK .

Yes, it is still an issue. But I  manually  copy  MakeCheckOpValueString function from logging.cc
and it works.
namespace tensorflow {
   namespace internal {
   	template <>
   	void MakeCheckOpValueString(std::ostream* os, const unsigned char& v) {
   		if (v >= 32 && v <= 126) {
   			(*os) << "'" << v << "'";
   		}
   		else {
   			(*os) << "unsigned char value " << static_cast<unsigned short>(v);
   		}
   	}
   }
}

Nagging Assignee @ebrevdo: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

@yjcn,
Looks like this wouldn't be issue in the latest TensorFlow releases as the mentioned file was updated. Kindly find the reference
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/core/util/ctc/ctc_beam_search_test.cc
Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The Tensorflow team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Thank you!


clean: remove models from dbt-project.yml (no longer needed)

@jitoquinto @kanelouise Now that we have FINALLY completed our quest of only delivering materialized views to partners (not tables), the time has come to remove these options from dbt-project.yml, so that we may never again stray down that dark path.
Also clarified the commentary surrounding the special model types for google_ads and removed one that we don't actually need. 🧹 ✨ 🧹 ✨ 🧹 ✨

Before we do this, can we confirm that we definitely don't want tables anymore (see bug here: https://techallies.atlassian.net/browse/DE-362?atlOrigin=eyJpIjoiY2I2YjY2NGQyZTA0NGEyZWJkMmNmMmFhNzI3ZmQxNzciLCJwIjoiaiJ9)

Before we do this, can we confirm that we definitely don't want tables anymore (see bug here: https://techallies.atlassian.net/browse/DE-362?atlOrigin=eyJpIjoiY2I2YjY2NGQyZTA0NGEyZWJkMmNmMmFhNzI3ZmQxNzciLCJwIjoiaiJ9)

We just spent several months working through tech debt tickets to deliver matviews instead of tables. The goal of our entire dbt setup is to perform any deduping (etc) logic so that the content of the _base tables exactly match what we deliver to partners in materialized views, which should just be SELECT * statements (unless we care about omitting _airbyte_* fields). I feel pretty strongly that we should keep this methodology consistent across all of our partner syncs, even if it requires additional transformation steps to deliver the _base tables the way we need them.

bumping this for 👀 @kanelouise @jitoquinto


Implement deferred highlighting for all completion styles

This is implemented in Vertico as follows: https://github.com/minad/vertico/blob/ffe34e5960e6d932f7ede9aa38be4aa194feeb8d/vertico.el#L232-L258
See discussion here https://github.com/raxod502/selectrum/issues/535#issuecomment-817303072.
If deferred highlighting is implemented one could remove the variable orderless-skip-highlighting from Orderless.

Note that generalized deferred highlighting is also necessary if one defines custom orderless styles, e.g., orderless+initialism https://github.com/oantolin/orderless/#defining-custom-orderless-styles. Then it won't work to set the highlighting function to orderless-highlight-matches, which does not take the completion category into account.


Rename Card className from c-card to c-Card

Resolves https://github.com/helpscout/blue/issues/66

Coverage remained the same at 100.0% when pulling 26b4374a2a3505a5b71906076b7ad217b08203f6 on card-resolve-classname into 970cb0331361440698b14e8d14f80f24284b0630 on master.


Add driver for HID 5021 CL

Add the following driver into Smart Card Connector: https://github.com/NC-DEV-Ahmet/driver-5021CL

The following landed PRs were preparation work for this effort: #1191 and #1192.

Other PRs that have landed:

Prepare our port of PC/SC-Lite for supporting the second driver: #1194, #1201, #1202, #1203, #1204.
Import, compile and enable the HID 5021 CL driver: #1199, #1214, #1215, #1221, #1222, #1223.


Factory as part of vm and stdlib

Adds Factory with Line and SetLine
close #87

Coverage increased (+0.2%) to 94.842% when pulling cdc46a906a4ad783c000472cced521913f9c8bee on factory into 8878d13a63a4a1ee73ceee0710ddaebeb003bbe1 on main.

coverage: 94.842% (+0.2%) from 94.621%
when pulling cdc46a906a4ad783c000472cced521913f9c8bee on factory
into 8878d13a63a4a1ee73ceee0710ddaebeb003bbe1 on main.

coverage: 94.842% (+0.2%) from 94.621%
when pulling cdc46a906a4ad783c000472cced521913f9c8bee on factory
into 8878d13a63a4a1ee73ceee0710ddaebeb003bbe1 on main.


Save returns objects different from what they are into the database

Hi,
I have a very strange behavior when I save an object, and that I apply a beforeSave trigger on it.
We suppose we have an object with an attribute equal to "100". We also have a "beforeSave" trigger that modifies this value to "50", if some conditions are right.
Now, when I call object.save(), the "beforeSave" is triggered, the value into the database is "50", but the value returned by the "save" operation is still "100".
How is it possible that the "save" operation returns an object with attributes different from what they are into the database?
Thank you.

Maybe it is related to #653

Have you tried this with the latest version of parse-server? There were some updates to the beforeSave triggers recently.

Sorry, I forgot to mention that I already use the last version of parse-server.

I am seeing the same issue. Anytime a value is changed in a beforeSave trigger it is not set on the object passed as a result of the save promise. However the change is set correctly in the database. Here is a minimal example:
Parse.Cloud.beforeSave('Person', function(req, res) {
    res.object.set('name', 'Anton');
    res.success();
});

// Expected behavior is that Anton is printed both times, but William is printed at first
Parse.Cloud.define('changeName', function (req, res) {
    var obj = new Parse.Object('Person');
    obj.save({name: 'William'}).then(function(obj) {
        console.log('From save method: ' + obj.get('name')); // From save method: William
        return new Parse.Query('Person').get(obj.id);
    }).then(function(obj) {
        console.log('From database: ' + obj.get('name')); // From database: Anton
        res.success();
    });
});

Verified this by creating a test inside specs/ParseAPI.spec.js:
  it('beforeSave change propagates through the save response', (done) => {
    Parse.Cloud.beforeSave('ChangingObject', function(request, response) {
      request.object.set('foo', 'baz');
      response.success();
    });
    let obj = new Parse.Object('ChangingObject');
    obj.save({ foo: 'bar' }).then((objAgain) => {
      expect(objAgain.get('foo')).toEqual('baz');
      Parse.Cloud._removeHook("Triggers", "beforeSave", "ChangingObject");
      done();
    }, (e) => {
      Parse.Cloud._removeHook("Triggers", "beforeSave", "ChangingObject");
      fail('Should not have failed to save.');
      done();
    });
  });

Working on a solution now, will submit a PR.

We'll fix this with #865 :smile:

After upgrading to parse-server 2.1.5, it works again. Thanks for the fix.


Readme.md Overhaul

Added Table-of-Contents
Removed issue tracking section
Added Maven installation section
Added License section

Resolved #311


[Suggestion] Set up a Gitter.im room for easier discussion

I use Gitter a lot for my own repos, and it's been adopted by a lot of large OSS projects as an alternative to IRC. The Gitter room could be kept as dev-focused discussion, while the subreddit would remain the primary point of contact with DIM's users.
Would simply require @kyleshay adding the repo on Gitter and adding a badge to the project's README.

We've setup a place on Slack for the conversation.

how can we have people use our slack board? do we need to approve them, or can they just join the discussion?
destinyitemmanager.slack.com

That's one of the things I really like about Gitter - you don't have to worry about inviting people and such. I'm good with either though.

:+1: for Gitter.  Slack just seems to be too heavy-handed for a project like this.

It seems like Gitter has only one room for chat around the repo.  Is there support for rooms to have side conversations?  What is the discover-ability for those rooms for new people just joining the conversation?

Gitter is intended to emulate an IRC room, only tightly integrated to a Github repo, so there is no support for project-level side conversations. You can have private chats with individual users in the room though, and you can set up your own channels for topic-specific chats.

Thanks.

gitter created


"Unexpected error" when trying to configure a PVE host

Checklist

[X] I have updated to the latest available Proxmox VE Custom Integration version, see the latest version here.

Describe the issue you are experiencing
When trying to add my Proxmox VE server to HGA using this integration and all proviced instructions, I get an 'Unexpected error' in the add integration dialog.

There's nothing relevant in the logs; enabling the debug logging for proxmoxve did not change anything.
In which version of Home Assistant Core do you have the problem?
core-2024.5.5
What version of Proxmox VE Custom Integration has the issue?
3.4.2
What version of Proxmox VE do you have the problem?
8.2.2
Diagnostics information
No response
Additional information
No response

In the host you must only enter the IP address, without https or forward slashes (/).


Same addresses for different versions of contracts

Good afternoon!
We are using Kernel SW and validators extensively, and are happy with updates frequency (v2.0, v2.1...).
The latest (v2.4) release has updated ECDSAValidator and SessionKeyValidator, however, their addresses (specified at the bottom of the README.md) were left unchanged.
Could you specify the addresses of updated validator modules in the readme, please?

for ECDSAValidator and SessionKeyValidator, the code should have not been changed but pushed the wrong code by mistake.
So the address of the ECDSA validator and session key validator is not changed actually

Thanks for the response!

... the code should have not been changed but pushed the wrong code by mistake

Do you mean the code is not production-ready yet?

Actually, it is production-ready, sorry if there was a confusion
Status for the each of the validators

SessionKeyValidator has been used with 0xB8E3c4bEaACAd06f6092793012DA4a8cB23D6123 since last november and you should be using this if you are using kernel 2.x, so we made this to show up in all versions to not make things confusing

ECDSAValidator this has been left unchanged, so you can use the version identified on the readme.md

the reason why i said " the code should have not been changed but pushed the wrong code by mistake." is that some commits that i tried while building v2.4 has change the interface in IKernelValidator, but i reverted it back to what it used to be before i published v2.4. So if you check the code, it has not been changed at all(for ECDSAValidator)
So tl;dr, the addresses shown on the readme is the latest address


pop_pools, region_pools, and country_pools do not remove when last block is removed

Confirmation

[X] My issue isn't already found on the issue tracker.
[X] I have replicated my issue using the latest version of the provider and it is still present.

Terraform and Cloudflare provider version
Terraform v1.5.3
on darwin_arm64

provider registry.terraform.io/cloudflare/cloudflare v4.11.0

Affected resource(s)
cloudflare_load_balancer
Terraform configuration files
resource "cloudflare_load_balancer" "pop-cf-test" {
  name        = "my-test-lb.cf.my-test-domain.com"
  description = "my-test-lb"
  default_pool_ids = [
    cloudflare_load_balancer_pool.failover_pool_geo.id,
  ]
  fallback_pool_id = cloudflare_load_balancer_pool.fallback_pool_geo.id
  enabled          = true
  proxied          = true
  steering_policy  = "geo"
  zone_id          = data.cloudflare_zone.my_test_domain.id
}

# Resource applied
resource "cloudflare_load_balancer" "pop-cf-test" {
  name        = "my-test-lb.cf.my-test-domain.com"
  description = "my-test-lb"
  default_pool_ids = [
    cloudflare_load_balancer_pool.failover_pool_geo.id,
  ]
  fallback_pool_id = cloudflare_load_balancer_pool.fallback_pool_geo.id
  enabled          = true
  proxied          = true
  steering_policy  = "geo"
  zone_id          = data.cloudflare_zone.my_test_domain.id

  pop_pools {
    pool_ids = [
      cloudflare_load_balancer_pool.pool.id,
      cloudflare_load_balancer_pool.failover_pool_geo.id,
    ]
    pop = "FSD"
  }

  region_pools {
    pool_ids = [
      cloudflare_load_balancer_pool.pool.id,
      cloudflare_load_balancer_pool.failover_pool_geo.id,
    ]
    region = "EEU"
  }
}

Link to debug output
https://gist.github.com/troymjones/815f8f633abf9296c949dc5dc254803b
Panic output
No response
Expected output
Terraform will perform the following actions:
cloudflare_load_balancer.pop-cf-test will be updated in-place
~ resource "cloudflare_load_balancer" "pop-cf-test" {
id               = "<zone_id>"
name             = "my-test-lb.cf.my-test-domain.com"
# (11 unchanged attributes hidden)
  - pop_pools {
      - pool_ids = [
          - "<pool_id>",
          - "<pool_id>",
        ] -> null
      - pop      = "FSD" -> null
    }

    # (1 unchanged block hidden)
}

Plan: 0 to add, 1 to change, 0 to destroy.
Actual output
No changes. Your infrastructure matches the configuration.
Terraform has compared your real infrastructure against your configuration
and found no differences, so no changes are needed.
Steps to reproduce

Create and apply load balancer resource with pop_pools, region_pools and/or country_pools blocks
Remove all of the remaining pop_pools, region_pools and/or country_pools blocks
Run terraform plan and/or terraform apply
No changes are shown in the plan or apply

Additional factoids
This seems to be an issue with the fact the key does not exist and therefore the blank/null value is never set. See code here: https://github.com/cloudflare/terraform-provider-cloudflare/blob/master/internal/sdkv2provider/resource_cloudflare_load_balancer.go#L184-L190
Datadog has a similar provider with assertions under datadog_synthetics_test resource. By setting it as an empty object before checking for the key, it gets set to a blank value if the key does not exist. See code here: https://github.com/DataDog/terraform-provider-datadog/blob/master/datadog/resource_datadog_synthetics_test_.go#L1458
References
No response

can you please provide the entirety of the debug output? i'm not seeing where the issue is here.

Here's the full debug output, but there's a lot in there that isn't relevant. https://gist.github.com/troymjones/00ba9c87e5c1f9c260473d32ca6e1f18
The original gist above only shows the relevant portion.
Did you see the example given as well as the expected output?
To summarize - when a load balancer has pop_pools set, they correctly apply. Then on an update, when all the pop_pools blocks are removed, the terraform does not show any changes when it should in fact remove the pop_pools and show an update to the load balancer. Instead it shows no changes and the apply makes no updates. This is the same situation with country_pools and region_pools.


test: what if LRO completes immediately

This added test covers the situation described in https://github.com/googleapis/nodejs-dialogflow/issues/450 (and turns out it actually works as expected).

@alexander-fenster shoot :laughing: one of those times where you wish the unit test would fail.


Unable to export to path "" as it doesn't exist

#5 but the path is configured already. path set to public_path(). even when getWorkingFilepath the result is null. BTW i'm using laravel 5.2

Hi there,
So what have you set in your configuration file? For path? Is the path writable and readable by the web user?

Just seen the issue myself on Windows, I've pushed a fix up to the develop branch if you want to give it a go.

The fix has been pushed to current versions v2.4.0/1.3.0.


IoTCOM errors from eclipse

@Mohannadcse  When I try to run the code on eclipse following the steps (IoTCOM_Behavioral Rule Extractor ) . i found the attached errors . I need to solve this problems.

I don't recall I had such error. But make sure you're using java 1.7. Your Java compiler setting should be something like this

Thank you Dr.Mohannd for your fast reply . I will try to run the code with Java 1.7


Local region, keyID and secret field added at connect form

Thank you very much for contributing! I'll publish it soon.


Using regular expressions in the title substitution

Problem to solve
Some titles contain strings that are not desirable in the final markdown link. The title substitution currently solves this problem by specifying strings that get removed.
The problem is that the current functionality only allows static strings. We can't remove dynamically generated parts of the title.
Example
I use the extension to copy Issue links from GitLab. CMD+SHIFT+E copies
[Automated CHANGELOG entries creation (#198) · Issues · GitLab.org / gitlab-vscode-extension · GitLab](https://gitlab.com/gitlab-org/gitlab-vscode-extension/-/issues/198)

The current substitution can remove  · Issues · GitLab.org / gitlab-vscode-extension · GitLab but it can't handle the (#198) bit.
Proposal
Allow user to specify a regular expression. A simple solution might be to recognise strings that are wrapped in /exp/ and not escape those. That shouldn't introduce breaking change (unless someone was escaping titles with /<something>/ in them.
Additional details
The logic that uses regex-escape to convert the title substitution to a regular expression: https://github.com/0x6b/copy-selection-as-markdown/blob/master/packages/core/src/copy.js#L68

@viktomas: Appreciate your time for this enhancement request! Your suggestion makes sense for me. Let me consider how I can implement this.

@0x6b Would you like me to try and implement this enhancement? How does the "prposal" in the description sound?

@viktomas sorry for delay on my end. Of course you can try to implement the feature! Proposal looks good to me. (If we'd like to avoid breaking change completely, we can introduce new dedicate textarea for regex substitution.)

@viktomas v0.21.0, which includes your PR #80, is just published. Thanks much for your contribution!


Implement team viability measure

Cooperstein, Jessica Nicole. 2017. “Initial Development of a Team Viability Measure.” Edited by Suzanne T. Bell. Master of Arts, DePaul University. https://via.library.depaul.edu/cgi/viewcontent.cgi?article=1211&context=csh_etd.

[ ] Implement as a surveyJS survey
[ ] Document in mdx
[ ] Add as an exit step

@markwhiting - when you have used this survey in the past, have you used all 20 questions in the paper, or have you used a subset?

I used the 14 question version in appendix E of the paper, which is also here → https://github.com/StanfordHCI/bang/blob/master/txt/midsurvey-q.txt
One of my students showed that we could collapse this into one question, but we didn't have enough data to validate so we have not published that.

How did you go about figuring out which was the one question to ask? Not sure how one goes about doing this sort of thing. My ideal would probably be about 3-5 questions, I think, if we knew which ones.
Do you collapse the questions into a single measure, or use them as independent measures?

We collapsed into a single measure, as is standard with a Likert of this type. Let me see if I can get any helpful insight from the student who worked on the simplification bit.

My thoughtful student helped me realize that we'd actually included some initial analysis on this scale in a past paper → see pp 12–13 here for a few paragraphs on this.
We started a validation process but it's not done yet. Let me know if someone here wants to take that on as a reasonably straightforward paper and I loop in the appropriate people.

@sky1113 - lets implement the 14-item for our experiment this week.
@markwhiting It would be great to do the full validation - let's outline it as a project for the fall.

Yep, sounds great! I'd say it's a rather good independent study project.

Adding an issue for tasks for the fall, I expect this will become its own project eventually: https://github.com/Watts-Lab/deliberation-project/issues/38

Blocked on #28

Lets split this into sub-issues:

[ ] Write aggregation function for team viability survey
[ ] Identify short version of team viability survey to use temporarily


Can't use version 2.3.3

I couldn't fetch the version 2.3.3. Also pod update SwiftyInsta didn't work. What could be the problem?
pod 'SwiftyInsta', '~> 2.3.3' doesn't work. The error is
[!] CocoaPods could not find compatible versions for pod "SwiftyInsta":
  In Podfile:
    SwiftyInsta (~> 2.3.3)

None of your spec sources contain a spec satisfying the dependency: `SwiftyInsta (~> 2.3.3)`.

You have either:
 * out-of-date source repos which you can update with `pod repo update` or with `pod install --repo-update`.
 * mistyped the name or version.
 * not added the source repo that hosts the Podspec to your Podfile.

Hello,
Try again, and make sure to update your repo.


containers: add container groups parents rollups

@miq-bot add_label providers/containers, metrics, enhancement
cc @blomquisg @chessbyte

@miq-bot add_label wip

@miq-bot remove_label wip

@simon3z what's the status of this PR? The dashboard PRs depend on it iiic

@simon3z what's the status of this PR? The dashboard PRs depend on it iiic

@abonas the dashboard should depend on #5587 (merged).

Collaborator
simon3z commented 38 minutes ago
@simon3z what's the status of this PR? The dashboard PRs depend on it iiic
@abonas the dashboard should depend on #5587 (merged).

@zeari

@abonas the dashboard should depend on #5587 (merged).

got it, thanks

@simon3z this looks good!  can you add screenshots for some of the UI updates?  I'm fine if you'd rather just add screenshots to #5713


format values

return Viessmann values as float without the unit string

Currently the vcontrold values are returned as string to iobroker. This makes it hard to process the values any further.
On a side note:
It would be great to have a way to set "role" and "unit" in the Gets section of the admin panel. These could be added as properties to the states.

Thanks! It works great.


Add ability to get current "User-Agent" for browser session. Related to #639

Proposed changes
Add "User-Agent" header when downloading file. Related to #639
Checklist

[x] Checkstyle and unit tests pass locally with my changes by running gradle check chrome htmlunit command
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation (if appropriate)

Coverage increased (+0.03%) to 64.34% when pulling ee488d3e3f8a6c638f97667b533d4d2b066c6050 on rosolko:user_agent into 5a8d489ac6ee0401506f86cedea1fb03795b0c04 on codeborne:master.

Codecov Report

Merging #657 into master will increase coverage by 0.06%.
The diff coverage is 50%.

@@             Coverage Diff              @@
##             master     #657      +/-   ##
============================================
+ Coverage     60.49%   60.55%   +0.06%
- Complexity      771      772       +1
============================================
  Files           148      148
  Lines          2749     2751       +2
  Branches        270      270
============================================
+ Hits           1663     1666       +3
  Misses          981      981
+ Partials        105      104       -1

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/com/codeborne/selenide/Selenide.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...rne/selenide/impl/DownloadFileWithHttpRequest.java
79.72% <100%> (+0.27%)
15 <0> (ø)
:arrow_down:

...e/selenide/impl/WebDriverThreadLocalContainer.java
81.88% <0%> (+1.44%)
31% <0%> (+1%)
: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 5a8d489...ee488d3. Read the comment docs.

Coverage increased (+0.06%) to 64.377% when pulling 607eef0650669278a16292bb5a8bb5a47aa08af3 on rosolko:user_agent into 5a8d489ac6ee0401506f86cedea1fb03795b0c04 on codeborne:master.

I don't know why reason, but codecov/patch won't pass :(

@rosolko relax and don't care :)


fix: ramp slider (#829)

Adding back ramp-slider and removing color-slider.
Description
Addresses issue #829

added back ramp-slider
removed color-slider from slider page that is now it's own component under /docs/colorslider.html

How and where has this been tested?
URL: http://localhost:3000/docs/slider.html
Chrome Version 85.0.4183.102 (Official Build) (64-bit) on MacOS
Screenshots

To-do list

[x] If my change impacts other components, I have tested to make sure they don't break.
[x] If my change impacts documentation, I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.

[x] This pull request is ready to merge.

better re-base on PR #923


ignore state change - not writable

Hi, i posted this one before, but i keep on getting these errors..
When checking the device through the daikin app then it's available and controllable, so it's not offline or anything.
Any ideas?

Thanks!

I also get these:

The weird thing is that if i disable en re-enable the MQTT subsciption, than it works right away...

Ääähhmm ... the two adapters should not have anything to do with each other.
ALso these single lines do not help me that much. Please enable Debug log for theinstance and post full bedug level logs and tell when which issue or situation happened

PS: "not writeble" means that on the last polling of data by the adapter the device was reported as offline by the daikin cloud. Maybe this already changed as you openerd the app but the adapter did not recognized that because the data were not polled so far and it would chnage back once this is done?

Thanks Ingo. I’ll enable debugging and keep an eye on whats happening. The strange thing is that nothing works until I disable and enable the mqtt subscription. Not sure what the link between the two can be though.Op 14 nov. 2022 om 23:02 heeft Ingo Fischer @.***> het volgende geschreven:
PS: "not writeble" means that on the last polling of data by the adapter the device was reported as offline by the daikin cloud. Maybe this already changed as you openerd the app but the adapter did not recognized that because the data were not polled so far and it would chnage back once this is done?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


I lose my xp and transaction in linea ecosystem

hello I got mistake and submit my POH passed metamask wallet as secandry and set my particle network wallet as primary as you know particle wallet do not have seed phrase and private key thats why i can not import that to metamask for doing  POH so please give help me to shift my primary and secondary wallet.
because is not fairly for a mistake I lose my all transaction and xp in linea
with a simple compare you can see Jupiter in solana ecosystem after their airdrop published a form and told if your wallet hacked you can create a new wallet and give your airdrop in a new wallet this is respect to their community but here in linea I cant shift my secondary wallet to primary and I will lose my xp and transaction I think  this is not fairly.
I request from all of linea  developer team  find a solution to receive my xp.
Thank you

Have you able to find solution to your lost transaction and XP in linea?

Have you able to find solution to your lost transaction and XP in linea?

i think just the Linea team can help me and let me shift my wallet between  primary wallet and  secondary wallet in https://linea.build/activations thats why i can get my Lxp


Hotfix for #52 - missing 'end'!

Apologies - I missed the end to my if block.
I did not bump the cookbook version on my branch for #52 when testing, so Berkshelf did not actually use my change when validating.
I've included a bump to 0.9.1 here.
Resolves #53.

No worries -- I probably should have tested further!


Question regarding multiple mail adresses per scan

Does wpwatcher support multiple email addresses per scan?
Example: 'http://test.com': '1<EMAIL_ADDRESS>?
Regards!

Hello @joberget ,
Check out the new version of the script at https://github.com/tristanlatr/WPWatcher .
The new version uses a config file and you can define multiple addresses :)


Build error when publishing

Description
I've installed Nerdpack, set up profile and generated uuid.
But when I type nr1 nerdpack:publish it all seems fine in this window, but back in the server terminal I see the following error:
ERROR in ./tmp/a355d06d-21ef-44eb-89e0-3ce37309af61-styles.1596193331921.scss
See more below.
It seems to be running but when I connect to a GitHub repo I get Oops, something went wrong message without any further info.
Found and loaded 1 nr1.json files on GitHub Integration (a355d06d-21ef-44eb-89e0-3ce37309af61) Nerdpack.

Nerdpack:
 ✔  GitHub Integration (a355d06d-21ef-44eb-89e0-3ce37309af61) nr1.json

Nerdlets:
 ✔  github nerdlets/github-about/nr1.json

🛠  Built artifact files for:
 ⁎  a355d06d-21ef-44eb-89e0-3ce37309af61--github built ✔

 ✔  Nerdpack built successfully!

Publishing Nerdpack GitHub Integration (a355d06d-21ef-44eb-89e0-3ce37309af61)
 ✔  Nerdpack published successfully!
 ✔  Deployed a355d06d-21ef-44eb-89e0-3ce37309af61 version 0.6.3 to the STABLE channel.

Steps to Reproduce
nr1 nerdpack:clone -r https://github.com/newrelic/nr1-github.git
cd nr1-github/
nr1 nerdpack:serve

Other terminal
nr1 nerdpack:publish

Expected Behaviour
Publish nerdpack without error
Relevant Logs / Console output
🛠  Built artifact files for:
 ⁎  a355d06d-21ef-44eb-89e0-3ce37309af61--github built ✔

 ✔  Nerdpack built successfully!
Hash: ec7f7e306b1e1ff3a570
Version: webpack 4.42.1
Child
    Hash: ec7f7e306b1e1ff3a570
    Time: 24ms
    Built at: 07/31/2020 12:03:31 PM
                                                  Asset      Size                                        Chunks                   Chunk Names
        a355d06d-21ef-44eb-89e0-3ce37309af61--styles.js  5.32 KiB  a355d06d-21ef-44eb-89e0-3ce37309af61--styles  [emitted]        a355d06d-21ef-44eb-89e0-3ce37309af61--styles
    a355d06d-21ef-44eb-89e0-3ce37309af61--styles.js.map  3.72 KiB  a355d06d-21ef-44eb-89e0-3ce37309af61--styles  [emitted] [dev]  a355d06d-21ef-44eb-89e0-3ce37309af61--styles
    Entrypoint a355d06d-21ef-44eb-89e0-3ce37309af61--styles = a355d06d-21ef-44eb-89e0-3ce37309af61--styles.js a355d06d-21ef-44eb-89e0-3ce37309af61--styles.js.map
    [./tmp/a355d06d-21ef-44eb-89e0-3ce37309af61-styles.1596193331921.scss] 1.05 KiB {a355d06d-21ef-44eb-89e0-3ce37309af61--styles} [built] [failed] [1 error]

    ERROR in ./tmp/a355d06d-21ef-44eb-89e0-3ce37309af61-styles.1596193331921.scss
    Module build failed (from /usr/lib64/newrelic-cli/node_modules/mini-css-extract-plugin/dist/loader.js):
    ModuleBuildError: Module build failed (from /usr/lib64/newrelic-cli/node_modules/fast-sass-loader/lib/index.js):
    Error: ENOENT: no such file or directory, open '/home/zfejer/nr1-github/tmp/a355d06d-21ef-44eb-89e0-3ce37309af61-styles.1596193331921.scss'
        at runLoaders (/usr/lib64/newrelic-cli/node_modules/webpack/lib/NormalModule.js:316:20)
        at /usr/lib64/newrelic-cli/node_modules/loader-runner/lib/LoaderRunner.js:367:11
        at Array.<anonymous> (/usr/lib64/newrelic-cli/node_modules/loader-runner/lib/LoaderRunner.js:203:19)
        at Storage.finished (/usr/lib64/newrelic-cli/node_modules/enhanced-resolve/lib/CachedInputFileSystem.js:55:16)
        at ReadFileContext.provider (/usr/lib64/newrelic-cli/node_modules/enhanced-resolve/lib/CachedInputFileSystem.js:91:9)
        at ReadFileContext.callback (/usr/lib64/newrelic-cli/node_modules/graceful-fs/graceful-fs.js:115:16)
        at FSReqWrap.readFileAfterOpen [as oncomplete] (fs.js:237:13)
    Child mini-css-extract-plugin ../../../usr/lib64/newrelic-cli/node_modules/css-loader/dist/cjs.js??ref--5-1!../../../usr/lib64/newrelic-cli/node_modules/postcss-loader/src/index.js??postcss!../../../usr/lib64/newrelic-cli/node_modules/fast-sass-loader/lib/index.js!tmp/a355d06d-21ef-44eb-89e0-3ce37309af61-styles.1596193331921.scss:
        Entrypoint mini-css-extract-plugin = *
        [../../../usr/lib64/newrelic-cli/node_modules/css-loader/dist/cjs.js?!../../../usr/lib64/newrelic-cli/node_modules/postcss-loader/src/index.js?!../../../usr/lib64/newrelic-cli/node_modules/fast-sass-loader/lib/index.js!./tmp/a355d06d-21ef-44eb-89e0-3ce37309af61-styles.1596193331921.scss] /usr/lib64/newrelic-cli/node_modules/css-loader/dist/cjs.js??ref--5-1!/usr/lib64/newrelic-cli/node_modules/postcss-loader/src??postcss!/usr/lib64/newrelic-cli/node_modules/fast-sass-loader/lib!./tmp/a355d06d-21ef-44eb-89e0-3ce37309af61-styles.1596193331921.scss 255 bytes {mini-css-extract-plugin} [built] [failed] [1 error]

        ERROR in ./tmp/a355d06d-21ef-44eb-89e0-3ce37309af61-styles.1596193331921.scss (/usr/lib64/newrelic-cli/node_modules/css-loader/dist/cjs.js??ref--5-1!/usr/lib64/newrelic-cli/node_modules/postcss-loader/src??postcss!/usr/lib64/newrelic-cli/node_modules/fast-sass-loader/lib!./tmp/a355d06d-21ef-44eb-89e0-3ce37309af61-styles.1596193331921.scss)
        Module build failed (from /usr/lib64/newrelic-cli/node_modules/fast-sass-loader/lib/index.js):
        Error: ENOENT: no such file or directory, open '/home/zfejer/nr1-github/tmp/a355d06d-21ef-44eb-89e0-3ce37309af61-styles.1596193331921.scss'

Your Environment

NR1 CLI version used: @datanerd/nr1/1.22.2 linux-x64 node-v10.16.3
Browser name and version: Firefox v78.0.2
Operating System and version: Windows 10, WSL2 Amazon linux 2

Additional context

@ZsoltPath Sorry for the delay in response! I am able to reproduce this, but I'm not sure why it's happening. I'm going to pull in some help from some colleagues who are better equipped to address this issue. Thank you for taking the time to log this issue.

tldr: Try publishing the nerdpack without serving it.
@ZsoltPath It looks like the issue here is that you're trying to publish the nerdpack while also serving the nerdpack. nr1 nerdpack:serve is running a local webhost on files in /dist, when you execute nr1 nerdpack:publish it deletes everything in /dist in order to generate the artifact it needs to upload to publish it. So the errors about "file not found" are correct, b/c nr1 nerdpack:publish just deleted them all. These two process shouldn't happen at the same time, that's the source of your trouble.
Give this a go and let us know how it works out for you.


feat: mobile support

Mobile environment support for the Dart SDK.

Added conditional imports for detecting Dart/Flutter environments.
The znn_sdk_dart library must work out-of-the-box on both Dart and Flutter environments.

It seems that the getApplicationSupportDirectory() function is the reason the path related functions have been refactored and changed to be async. To maintain compatibility with existing clients and since the getApplicationSupportDirectory() function also causes the dependency to the Futter SDK, should we consider an alternative approach?
What if the default path could optionally be provided to the SDK, so that the public interface of the paths can remain the same?
Something like this (compared to master branch):
diff --git a/lib/src/global.dart b/lib/src/global.dart
index ea9472a..3da1af8 100644
--- a/lib/src/global.dart
+++ b/lib/src/global.dart
@@ -8,6 +8,11 @@ export 'package:logging/logging.dart' show Level;
 const znnSdkVersion = '0.0.7';
 const znnRootDirectory = 'znn';

+final _noDefaultDataPathException = ZnnSdkException(
+    '''No default data path set. An application support directory has to be '''
+    '''provided for the ZNN SDK when using the Android or iOS platforms.''');
+String? _defaultDataPath;
+
 class ZnnPaths {
   Directory main;
   Directory wallet;
@@ -25,17 +30,23 @@ class ZnnPaths {
 ZnnPaths _getDefaultPaths() {
   late Directory main;
   if (Platform.isLinux) {
-    main = Directory(
-        path.join(Platform.environment['HOME']!, '.$znnRootDirectory'));
+    main = Directory(path.join(
+        _defaultDataPath ?? Platform.environment['HOME']!,
+        '.$znnRootDirectory'));
   } else if (Platform.isMacOS) {
-    main = Directory(
-        path.join(Platform.environment['HOME']!, 'Library', znnRootDirectory));
+    main = Directory(path.join(
+        _defaultDataPath ?? path.join(Platform.environment['HOME']!, 'Library'),
+        znnRootDirectory));
   } else if (Platform.isWindows) {
-    main = Directory(
-        path.join(Platform.environment['AppData']!, znnRootDirectory));
+    main = Directory(path.join(
+        _defaultDataPath ?? Platform.environment['AppData']!,
+        znnRootDirectory));
+  } else if (Platform.isAndroid || Platform.isIOS) {
+    if (_defaultDataPath == null) throw _noDefaultDataPathException;
+    main = Directory(_defaultDataPath!);
   } else {
-    main =
-        Directory(path.join(Platform.environment['HOME']!, znnRootDirectory));
+    main = Directory(path.join(
+        _defaultDataPath ?? Platform.environment['HOME']!, znnRootDirectory));
   }
   return ZnnPaths(
       main: main,
@@ -84,6 +95,10 @@ int getChainIdentifier() {
   return chainId;
 }

+void setDefaultDataPath(String path) {
+  _defaultDataPath = path;
+}
+
 final logger = Logger('ZNN-SDK');

 class ZnnSdkException implements Exception {

@vilkris4 please check this commit. I assume that if you are on a mobile device, you have a Flutter SDK dependency.

@drblazer21 The issue I see is that it only works on both platforms if the Flutter SDK is used. When using the plain Dart SDK it's not able to resolve the dependencies and compile (see my first comment). Unless I'm missing something?

Please change the Dart version requirement in the README.md file.

@vilkris4 you cannot compile using only the Dart SDK because of the path_provider dependency. However, you can run the app without a Flutter runtime perfectly fine. This was the whole point of the conditional imports. The purpose of the Dart SDK is to work simultaneously on desktop and mobile environments.
@KingGorrin I've fixed the library loading paths for desktop. I've also incremented the SDK version.

This PR and other PRs merge into the develop, a release branch will eventually be merged into main.
It is recommended to update the version in the release branch.

It is exactly at the start of a release branch that the upcoming release gets assigned a version number—not any earlier. Up until that moment, the develop branch reflected changes for the “next release”, but it is unclear whether that “next release” will eventually become 0.3 or 1.0, until the release branch is started. That decision is made on the start of the release branch and is carried out by the project’s rules on version number bumping.

See https://nvie.com/posts/a-successful-git-branching-model/ for more info.

@vilkris4 you cannot compile using only the Dart SDK because of the path_provider dependency. However, you can run a znn_sdk_dart based app without a Flutter runtime perfectly fine - for example a CLI app. This was the whole point of the conditional imports. The purpose of the Dart SDK is to work simultaneously on desktop and mobile environments.

Please try to compile the lib in the cli and syrius. I've updated both the znn_ledger_dart and znn_cli_dart to use the mobile-support branch of znn_sdk_dart and used flutter pub get to resolve the dependencies, but cannot get past the following error when compiling.
/pub.dev/dcli-3.4.0/lib/src/util/wait_for_ex.dart:38:17: Error: Method not found: 'waitFor'.
value = cli.waitFor<T>(wrapped);
                ^^^^^^^
Error: AOT compilation failed
Bad state: Generating AOT kernel dill failed!
make: *** [Makefile:9: windows] Error 64

I'm using the following to compile the cli.
flutter pub get
dart compile exe znn-cli.dart -o build\znn-cli.exe

@vilkris4 you cannot compile using only the Dart SDK because of the path_provider dependency. However, you can run a znn_sdk_dart based app without a Flutter runtime perfectly fine - for example a CLI app.

This to me still seems backwards to require the Flutter SDK, when it's not necessary in non-mobile environments, and the only functionality it does provide is the convenience function for the application support directory. Is that really worth the dependency to the Flutter SDK and changing the public interfaces? The name of this repository suggests this is a pure Dart SDK.
In addition, changing the public interfaces breaks backwards compatibility with all other applications that use this SDK and they will have to be updated.
Any thoughts on the alternative solution I suggested?

After some considerations I support @vilkris4 proposed changes. It will eliminate the unnecessary Flutter SDK dependency, the interface changes and make it useable for both mobile and desktop platforms by removing the platforms specifics from the SDK.
Please comment on the alternative solution suggested by @vilkris4


bump-formula-pr fails for Formula using :homebrew_curl

brew config output
➜  ~ brew config
HOMEBREW_VERSION: 3.5.2-152-gac454d6
ORIGIN: https://github.com/Homebrew/brew
HEAD: ac454d65632c333ec016799bdcff657547738eb6
Last commit: 2 hours ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: 2e3d51340f2f9b47d680f656e712fbee77cbcf79
Core tap last commit: 2 hours ago
Core tap branch: master
HOMEBREW_PREFIX: /opt/homebrew
HOMEBREW_CASK_OPTS: []
HOMEBREW_GITHUB_API_TOKEN: set
HOMEBREW_MAKE_JOBS: 10
Homebrew Ruby: 2.6.8 => /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/bin/ruby
CPU: 10-core 64-bit arm_firestorm_icestorm
Clang: 13.1.6 build 1316
Git: 2.36.1 => /opt/homebrew/bin/git
Curl: 7.79.1 => /usr/bin/curl
macOS: 12.4-arm64
CLT: <IP_ADDRESS>.1.1651278267
Xcode: 13.4.1
Rosetta 2: false

brew doctor output
➜  ~ brew doctor
Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry or file an issue; just ignore this. Thanks!

Warning: Some installed kegs have no formulae!
This means they were either deleted or installed manually.
You should find replacements for the following formulae:
  szip

Verification

[X] I ran brew update and am still able to reproduce my issue.
[X] I have resolved all warnings from brew doctor and that did not fix my problem.

What were you trying to do (and why)?
brew bump-formula-pr speex --version=1.2.1
What happened (include all command output)?
➜  ~ brew bump-formula-pr speex --version=1.2.1
==> Downloading https://downloads.xiph.org/releases/speex/speex-1.2.1.tar.gz
Already downloaded: /Users/owine/Library/Caches/Homebrew/downloads/a78055588bce593bfa45327769802fcc954269169f3d564b825dacb98dc71cc6--speex-1.2.1.tar.gz
Warning: Cannot verify integrity of 'a78055588bce593bfa45327769802fcc954269169f3d564b825dacb98dc71cc6--speex-1.2.1.tar.gz'.
No checksum was provided for this resource.
For your reference, the checksum is:
  sha256 "4b44d4f2b38a370a2d98a78329fefc56a0cf93d1c1be70029217baae6628feea"
==> replace / +mirror "https:\/\/ftp\.osuosl\.org\/pub\/xiph\/releases\/speex\/s
==> replace /https:\/\/downloads\.xiph\.org\/releases\/speex\/speex\-1\.2\.0\.ta
==> replace "eaae8af0ac742dc7d542c9439ac72f1f385ce838392dc849cae4536af9210094" w
==> replace /^( +)(url "https:\/\/downloads\.xiph\.org\/releases\/speex\/speex\-
Error: inreplace failed
/opt/homebrew/Library/Taps/homebrew/homebrew-core/Formula/speex.rb:
  expected replacement of /^( +)(url "https:\/\/downloads\.xiph\.org\/releases\/speex\/speex\-1\.2\.1\.tar\.gz"\n)/m with "\\1\\2\\1mirror \"https://ftp.osuosl.org/pub/xiph/releases/speex/speex-1.2.1.tar.gz\"\n"

What did you expect to happen?
Text replacement appears fail because the regex is expecting a new line after the URL but in this formula, using: :homebrew_curl is specified causing the failure.
Step-by-step reproduction instructions (by running brew commands)
`brew bump-formula-pr speex --version=1.2.1`

Thanks for bringing this to our attention. I've opened a PR (#13474) with a simple fix that will handle url calls with a trailing hash on the same line (e.g., ..., using: :homebrew_curl). This should work for all the existing instances in homebrew/core (like speex) but we may need to revise it for a more comprehensive solution in the future (e.g., if we encounter a multiline url call).


I dont want emails . I want to extract phone numbers only

Hey The library is awesome and build very well. I want to implement phone numbers not email . There should be an optional argument for both contact phone number and contact email.

Also it would be nice if I could get only contact with email addresses. Nice library btw :-)

You should add:
contactPicker.putExtra(ContactPickerActivity.CP_EXTRA_SELECTION, CP_SELECTION);  contactPicker.putExtra(ContactPickerActivity.CP_EXTRA_SELECTION_ARGS, CP_SELECTION_ARGS);
and
 public static final String CP_SELECTION =  "(" + ContactsContract.Data.MIMETYPE + "=?)"; public static final String [] CP_SELECTION_ARGS = new String[] {             ContactsContract.CommonDataKinds.Phone.CONTENT_ITEM_TYPE     };

Here is the solution.  For emails only please refer the closed issue  #1
    public void launchContactPicker(View view) {
    Intent contactPicker = new Intent(this, ContactPickerActivity.class);

    //phone numbers only
    String customSelection = "(" + ContactsContract.Data.MIMETYPE + "=? )";
    String [] customArgs = new String [] {ContactsContract.CommonDataKinds.Phone.CONTENT_ITEM_TYPE};
    contactPicker.putExtra(ContactPickerActivity.CP_EXTRA_SELECTION,customSelection);
    contactPicker.putExtra(ContactPickerActivity.CP_EXTRA_SELECTION_ARGS, customArgs);
    contactPicker.putExtra(ContactPickerActivity.CP_EXTRA_HAS_CUSTOM_SELECTION_ARGS, true);

    startActivityForResult(contactPicker, CONTACT_PICKER_REQUEST);}


Add option to make shapekeys cleanup optional when merging armatures

This makes cleaning up shapekeys optional when merging armatures. CATS does this every time you use the merge armatures tool, but it ignores common empty visemes used for the VRC avatar descriptor in Unity.
In my example, the shapekeys VRC.v_sil and VRC.v_PP were removed by CATS because they were empty, causing LipSync not to work.
This PR adds the option to skip the cleanup and keep those visemes or other important shapekeys that should not be removed but must remain empty.

Duplicate of https://github.com/absolute-quantum/cats-blender-plugin/commit/56470ec1c457b2a5430b2dc449360960ab4e0e7f


Block-side port arrays

Partial resolution for #20, adds block-side port arrays with:

Ability to define and export port arrays in blocks in the frontend
Block side port arrays can have undefined elements (useful for an abstract block, is an error if this shows up in the final design), or defined elements.
Port array elements can be defined statically as part of a block, or 'dynamically' (including reactive to incoming connections) as part of a generator. Array elements can also be defined by propagation (exports).

A future PR will complete #20 with these features:

Port array to port array connections (between blocks at the same level, as opposed to cross-hierarchy export operations)

A work-in-progress PR will use this feature to add an abstract microcontroller block (#23) and refactor examples to use that. That may also modify the frontend interface for port arrays.

TODO: get rid of required-ports, which is hackish and used by the current microcontroller generator.

Implementation details:

IR changes

Add separate array-export and array-connect constraints, to differentiate from the single element export and connect constraints
For Port Arrays, differentiate between undefined and empty with a contains element.
Add Array-valued ValueExpr
Change ALLOCATE reserved-param (LocalStep) to an allocate step with optional suggested name
Add ALLOCATED reserved-param (LocalStep) which returns the requested connections to a port-array as an array-of-string

ALLOCATED propagates root-inwards, and is defined by the top-most-level connections

Add ELEMENTS reserved-param (LocalStep), which returns the elements of a port-array as an array-of-string

ELEMENTS propagates leaf-outwards, and is defined by statically defined ports of an array
This is more of an internal thing and is not exposed in the frontend

Minor refactorings

Clean up generator to get rid of function name and other legacy cruft. (as part of #95)

Major refactoring of compiler to make this all work

Block and links now expand as part of their container. This elaborates their ports (so IsConnected can be known).

Exception: port arrays have delayed elaboration, until ELEMENTS is defined (which can be dependent on ALLOCATED).
All link-side port arrays are treated as dynamically sized by incoming connections, so this defines a directed ALLOCATED -> ELEMENTS assignment.
Their contents are processed separately

Per the above, the generator task is integrated with the rest of the block task. The generator's block stub includes the port definition (excluding arrays), and the implementation includes the internal blocks / links / constraints.
IsConnected handling is now (more) centralized, enabled by the above change.
Add separate lowering of array-connect constraints (expands into leaf-level connect constraints - and defines ALLOCATED) and ALLOCATE references (which assigns them a concrete index)
Additional stylistic changes

Add a single-write hash map data structure that builds in check for overwriting or multiple writes

Add compiler error check for undefined port-arrays (such as might be seen on an abstract block, distinct from empty port arrays)
Add compiler check pass for bad references to ports in connects and exports.

These do not check parameter references, since connectivity to the link is needed to resolve steps like CONNECTED_LINK.

Frontend changes

Add frontend support for Port Arrays: append_elt (from the inside) and allocate(suggestedName) (from the outside)
Add Vector.allocated() as a way to get incoming connections (including suggested-names) as a string-array

Add generator support for array-valued ValueExpr

Minor refactorings

Eliminate intermediary ParamVariableBinding that wasn't doing anything useful
Split out GeneratorBlock into its own file

Not sure what's going on with automated tests, it seems to think Builder.get_curr_block() doesn't exist?


Add block failure reasons into failed_blocks table during indexing

Our FailedBlocks model looks like this currently:
https://github.com/DefiantLabs/cosmos-indexer/blob/6997c0ca80400e3dc0b44122b3286fad72964e88/db/models.go#L18-L23
Its lacking a failure reason, making it hard to determine quickly what happened.
We could either:

Have an Enum of known failure reasons and store the int - this would potentially save data but not be as extensible
Store a string error message - this would increase data size but be more generalized

i vote option 2


replace while(true) with !Thread.currentThread().isInterrupted()

Could you please replace
replace while(true)

with
!Thread.currentThread().isInterrupted()

Hi,
Feel free to send a pull request


CFGenerator' object has no attribute 'enable_vae_slicing | HELP

this only happens when you type 'consi' in easyfunction. I tested it in different environments.
2024-12-24T10:02:32.462496 - !!! Exception during processing !!! 'CFGenerator' object has no attribute 'enable_vae_slicing' 2024-12-24T10:02:32.463152 - Traceback (most recent call last):   File "/teamspace/studios/this_studio/ComfyUI/execution.py", line 328, in execute     output_data, output_ui, has_subgraph = get_output_data(obj, input_data_all, execution_block_cb=execution_block_cb, pre_execute_cb=pre_execute_cb)   File "/teamspace/studios/this_studio/ComfyUI/execution.py", line 203, in get_output_data     return_values = _map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True, execution_block_cb=execution_block_cb, pre_execute_cb=pre_execute_cb)   File "/teamspace/studios/this_studio/ComfyUI/custom_nodes/ComfyUI-0246/utils.py", line 353, in new_func     res_value = old_func(*final_args, **kwargs)   File "/teamspace/studios/this_studio/ComfyUI/execution.py", line 174, in _map_node_over_list     process_inputs(input_dict, i)   File "/teamspace/studios/this_studio/ComfyUI/execution.py", line 163, in process_inputs     results.append(getattr(obj, func)(**inputs))   File "/teamspace/studios/this_studio/ComfyUI/custom_nodes/ComfyUI_StoryDiffusion/Storydiffusion_node.py", line 1938, in story_sampler     pipe.enable_vae_slicing() AttributeError: 'CFGenerator' object has no attribute 'enable_vae_slicing'

Yes, this error is a logical error. When using ComfyUI classic mode, 'consi' mode should be invalidated. I fix it.

2024-12-25T21:56:09.807162 - Traceback (most recent call last):   File "/teamspace/studios/this_studio/ComfyUI/execution.py", line 328, in execute     output_data, output_ui, has_subgraph = get_output_data(obj, input_data_all, execution_block_cb=execution_block_cb, pre_execute_cb=pre_execute_cb)   File "/teamspace/studios/this_studio/ComfyUI/execution.py", line 203, in get_output_data     return_values = _map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True, execution_block_cb=execution_block_cb, pre_execute_cb=pre_execute_cb)   File "/teamspace/studios/this_studio/ComfyUI/custom_nodes/ComfyUI-0246/utils.py", line 353, in new_func     res_value = old_func(*final_args, **kwargs)   File "/teamspace/studios/this_studio/ComfyUI/execution.py", line 174, in _map_node_over_list     process_inputs(input_dict, i)   File "/teamspace/studios/this_studio/ComfyUI/execution.py", line 163, in process_inputs     results.append(getattr(obj, func)(**inputs))   File "/teamspace/studios/this_studio/ComfyUI/custom_nodes/ComfyUI_StoryDiffusion/Storydiffusion_node.py", line 1408, in story_model_loader     raise "Use comfyUI calss processing don't support consistory mode,please checke readme how to use consistory mode." TypeError: exceptions must derive from BaseException
i couldn't find anything in the readme explaining how to use consistory mode other than typing 'consi' in easy function. is there anything else? sorry for the inconvenience

do not link any  other comfyUI node,just like this:


Sort imports in exported notebooks using isort

Edvart already uses isort to sort imports in the code the library. The imports in the exported are not nicely sorted, which isort would help with.
isort would need to be moved from development dependencies to main dependencies, but that shouldn't cause any issues since isort is a small library with no dependencies.

The sorting would be much easier with #95 -- non-imports among imports would be removed.


Switch to using feature branches and a protected master

Relocated from https://github.com/open-forcefield-group/smarty/pull/22#issuecomment-227913804
--
Perhaps we want to move to using feature branches of the main repo to ensure tests are run in the future?
There is a way to do this safely by preventing pushes to the master.

We've already switched to using branches. I just adjusted the settings by protecting the master branch and requiring tests to pass (except for admins) before merging to master. So, I'll close this.


Document#toJSON({getters: true}) performance

I have just benchmarked our performance problem - a query was running almost 3000 ms. The evil is Document#toJSON. It takes up to 70-80% of time from starting the query until getting plain json on large docs.
I was reading the code, It seems like the biggest impact comes from Document#get because it triggers every property getter on the document?
Probably you know a way how to optimize that, if not user need to be warned in the documentation.
Best,
Oleg

was this ever resolved? I am experiencing the exact same problem. I'm spending ~15ms on toJSON

Nope, no progress was made. Can you provide an example of the code that you're seeing bad performance with?

For some background, I'm running on a single, Heroku dyno, talking to a free-tier MongoLab instance. New Relic is reporting that I spend a long time in my route code. See example screenshot. Most of my routes (not just the attached) look like this - the majority of the time spent in the route itself.

I was doing something like res.json(200, { users: _.invoke(users, 'toAdminJSON') }); where toAdminJSON(); invokes this.toJSON(); (as well as some hash munging) By avoiding a call to toJSON(); - and instead using a handwritten function to accomplish the same thing for my specific model - I was able to shave a few seconds off that route, which, by the way, is returning about ~100 users. The route used to take ~5 seconds, now it takes ~3.
By using my hand-written toJSON, I've shaved off a few hundred milliseconds in a few other routes. I haven't looked into it much further. I will say that my documents are very large. About ~25kB per document on average.

Could you show me a few example users? Would be helpful to be able to run this myself.
Also may be a case of your route sending too many users - if it takes 5 seconds for toJSON() to convert the users, it must be taking quite a while for your client to receive all that data.


Fix documentation CI for PRs from forks and separate pytest and regression tests in Github actions

What the title says.

Codecov Report

Merging #73 (a80bfe7) into main (69dfbbf) will decrease coverage by 0.29%.
The diff coverage is n/a.

:exclamation: Current head a80bfe7 differs from pull request most recent head 9dee6c7. Consider uploading reports for the commit 9dee6c7 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main      #73      +/-   ##
==========================================
- Coverage   92.01%   91.72%   -0.29%
==========================================
  Files          76       86      +10
  Lines       14003    15271    +1268
==========================================
+ Hits        12885    14008    +1123
- Misses       1118     1263     +145

Flag
Coverage Δ

lint_rules
97.35% <ø> (?)

loki
91.97% <ø> (-0.04%)
:arrow_down:

transformations
86.82% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 20 files with indirect coverage changes


Event RecordLine only contains the line up to the first EOL

If I read a CSV with quoted strings that contain EOL characters, the line is correctly read into values, but the events only receive the first part of the line.
void Main()
{
    var data = new[] { new[] { "First Record First Value", $"First Record Second{Environment.NewLine}Value" }, new[] { "Second Record First Value", "Second Record Second Value"} };
    var csv = String.Join(Environment.NewLine, data.Select(d => $"{String.Join(",", d.Select(i => $"\"{i}\""))}"));
    var fh = new FileHelperAsyncEngine<Record>();
    using (fh.BeginReadString(csv))
    {
        foreach (var r in fh)
        {
            Console.WriteLine($"|{r.BeforeRecordLine}', '{r.AfterRecordLine}|");
            Console.WriteLine($"*{String.Join(",", r.Values.Select(v => $"'{v}'"))}*");
        }
    }
}

// Define other methods and classes here
[DelimitedRecord(",")]
private class Record : INotifyRead
{
    [FieldQuoted('"')]
    public string[] Values;

    [FieldHidden]
    public string BeforeRecordLine;

    [FieldHidden]
    public string AfterRecordLine;

    public void BeforeRead(BeforeReadEventArgs e)
    {
        BeforeRecordLine = e.RecordLine;
    }

    public void AfterRead(AfterReadEventArgs e)
    {
        AfterRecordLine = e.RecordLine;
    }
}

The above code snippet results in the following output:
|"First Record First Value","First Record Second', '"First Record First Value","First Record Second|
*'First Record First Value','First Record Second
Value'*
|"Second Record First Value","Second Record Second Value"', '"Second Record First Value","Second Record Second Value"|
*'Second Record First Value','Second Record Second Value'*

It seems like the RecordLine property doesn't get appended to if the field requires more input from the data.

I also ran into this problem as well.  I really need to be able to read in all the values for the record when it's a multiline field as sometimes I'm fixing characters like " to "" in my incoming data.


Error: Cannot find module 'bluebird'

Hi,
I've installed<EMAIL_ADDRESS>as a dependency of my project but after running the npm prune --production command to remove the devDeps I get the following error:
➜  myproject git:(main) node dist/index.js
internal/modules/cjs/loader.js:888
  throw err;
  ^

Error: Cannot find module 'bluebird'
Require stack:
- /Users/wilk/Projects/myproject/node_modules/dataloader-sequelize/lib/index.js

Then, if I install bluebird as a dependency of the project, it works fine.
Shouldn't be defined inside dataloader-sequelize itself?

bluebird used to ship with sequelize i think.
the bluebird dependency should probably be removed from this library, node.js promises are just fine now.

Yep, you're right.
I created a PR for that: https://github.com/mickhansen/dataloader-sequelize/pull/124


Links inside Flash are not colored

Describe the bug
A Link component inside of a Flash component should be styled to match the variant of Flash.
To Reproduce
Steps to reproduce the behavior:

Nest a Link inside of a Flash
Link is colored blurple, not variant color

https://codesandbox.io/embed/flash-link-color-bug-bs3us
Expected behavior
Links should be same color as Flash variant.
Screenshots

Rimble UI Version

0.10.0 (latest)

@MikeLockz I don't think this is a valid bug. Can you verify?

Correct, not a bug. Docs were wrong and have already been updated.


Removing application from the library manifest

The library project should not contain an application element as there is no application included. This causes issues with the manifest merger when using this library with an app that has an app_name defined in the application element.
The other workaround is to use tools:replace="android:label".
Manifest merger failed : Attribute application@label value=(XYZ) from AndroidManifest.xml:31:9-35 is also present at [com.roughike:bottom-bar:1.1.7] AndroidManifest.xml:11:18-50 value=(@string/app_name).
Suggestion: add 'tools:replace="android:label"' to <application> element at AndroidManifest.xml:27:5-219:19 to override.

Thanks for pointing out and contributing!