Feature request: actual custom key generation

I see that this feature has been requested at least twice but got entrenched in a particular use case (#318 that just disables the body and #444 which goes on to discuss and fix Content-Length issues.
My particular use case is that I want to remove an argument from the query just for the cache key, the request should go to the upstream server unchanged. However, because the argument that I want removed only identifies the client, and because all data is public, I'd like to cache a response no matter the client argument is, because it will be the same exact response.
Also, I am not interested in the method, protocol and host and port as part of the key. For me, just the uri_path and the query (without a specific parameter) is required.
Any chance to be able to get a way to add a custom key?
I should also mention that I am using the cache-handler caddy plugin.
Thank you.

Hey, @teodorescuserban I'm actually implementing it in the core.

The PR is ready @teodorescuserban can you try it?

@darkweak, thank you for your PR!
Unfortunately, I am not able to build caddy with the specific commit:
59.74 2024/05/13 08:24:28 [INFO] exec (timeout=0s): /usr/local/go/bin/go build -o /usr/bin/caddy -ldflags -w -s -trimpath -tags nobadger
125.3 # github.com/darkweak/souin/plugins/caddy
125.3 /go/pkg/mod/github.com/darkweak/souin/plugins/caddy@v0.0.0-20240512124841-f4b3aefa97bb/configuration.go:389:11: ck.DisableScheme undefined (type configurationtypes.Key has no field or method DisableScheme)
125.3 /go/pkg/mod/github.com/darkweak/souin/plugins/caddy@v0.0.0-20240512124841-f4b3aefa97bb/configuration.go:391:11: ck.Template undefined (type configurationtypes.Key has no field or method Template)
125.3 /go/pkg/mod/github.com/darkweak/souin/plugins/caddy@v0.0.0-20240512124841-f4b3aefa97bb/configuration.go:393:11: ck.Hash undefined (type configurationtypes.Key has no field or method Hash)
125.3 /go/pkg/mod/github.com/darkweak/souin/plugins/caddy@v0.0.0-20240512124841-f4b3aefa97bb/configuration.go:477:18: config_key.DisableScheme undefined (type configurationtypes.Key has no field or method DisableScheme)
125.3 /go/pkg/mod/github.com/darkweak/souin/plugins/caddy@v0.0.0-20240512124841-f4b3aefa97bb/configuration.go:479:18: config_key.Template undefined (type configurationtypes.Key has no field or method Template)
125.3 /go/pkg/mod/github.com/darkweak/souin/plugins/caddy@v0.0.0-20240512124841-f4b3aefa97bb/configuration.go:481:18: config_key.Hash undefined (type configurationtypes.Key has no field or method Hash)
125.3 /go/pkg/mod/github.com/darkweak/souin/plugins/caddy@v0.0.0-20240512124841-f4b3aefa97bb/httpcache.go:192:108: dc.Key.DisableScheme undefined (type configurationtypes.Key has no field or method DisableScheme)
125.3 /go/pkg/mod/github.com/darkweak/souin/plugins/caddy@v0.0.0-20240512124841-f4b3aefa97bb/httpcache.go:192:133: dc.Key.Hash undefined (type configurationtypes.Key has no field or method Hash)
125.3 /go/pkg/mod/github.com/darkweak/souin/plugins/caddy@v0.0.0-20240512124841-f4b3aefa97bb/httpcache.go:192:192: dc.Key.Template undefined (type configurationtypes.Key has no field or method Template)
125.6 2024/05/13 08:25:34 [INFO] Skipping cleanup as requested; leaving folder intact: /tmp/buildenv_2024-05-13-0823.1014515775
125.6 2024/05/13 08:25:34 [FATAL] exit status 1
------
Dockerfile:13
--------------------
  12 |
  13 | >>> RUN CGO_ENABLED=0 GOARCH=${TARGETARCH} GOOS=${TARGETOS} \
  14 | >>>     xcaddy build \
  15 | >>>     --with github.com/darkweak/souin/plugins/caddy@f4b3aefa97bbdc8b97776854c3a0bf92172f161a
  16 |
--------------------
ERROR: failed to solve: process "/bin/sh -c CGO_ENABLED=0 GOARCH=${TARGETARCH} GOOS=${TARGETOS}     xcaddy build     --with github.com/darkweak/souin/plugins/caddy@f4b3aefa97bbdc8b97776854c3a0bf92172f161a" did not complete successfully: exit code: 1

I realised that was not the last commit in the PR; the last one doesn't work as well
139.0 # github.com/darkweak/souin/plugins/caddy
139.0 /go/pkg/mod/github.com/darkweak/souin/plugins/caddy@v0.0.0-20240512154727-aa3a9480609c/configuration.go:389:11: ck.DisableScheme undefined (type configurationtypes.Key has no field or method DisableScheme)
139.0 /go/pkg/mod/github.com/darkweak/souin/plugins/caddy@v0.0.0-20240512154727-aa3a9480609c/configuration.go:391:11: ck.Template undefined (type configurationtypes.Key has no field or method Template)
139.0 /go/pkg/mod/github.com/darkweak/souin/plugins/caddy@v0.0.0-20240512154727-aa3a9480609c/configuration.go:393:11: ck.Hash undefined (type configurationtypes.Key has no field or method Hash)
139.0 /go/pkg/mod/github.com/darkweak/souin/plugins/caddy@v0.0.0-20240512154727-aa3a9480609c/configuration.go:477:18: config_key.DisableScheme undefined (type configurationtypes.Key has no field or method DisableScheme)
139.0 /go/pkg/mod/github.com/darkweak/souin/plugins/caddy@v0.0.0-20240512154727-aa3a9480609c/configuration.go:479:18: config_key.Template undefined (type configurationtypes.Key has no field or method Template)
139.0 /go/pkg/mod/github.com/darkweak/souin/plugins/caddy@v0.0.0-20240512154727-aa3a9480609c/configuration.go:481:18: config_key.Hash undefined (type configurationtypes.Key has no field or method Hash)
139.0 /go/pkg/mod/github.com/darkweak/souin/plugins/caddy@v0.0.0-20240512154727-aa3a9480609c/httpcache.go:192:108: dc.Key.DisableScheme undefined (type configurationtypes.Key has no field or method DisableScheme)
139.0 /go/pkg/mod/github.com/darkweak/souin/plugins/caddy@v0.0.0-20240512154727-aa3a9480609c/httpcache.go:192:133: dc.Key.Hash undefined (type configurationtypes.Key has no field or method Hash)
139.0 /go/pkg/mod/github.com/darkweak/souin/plugins/caddy@v0.0.0-20240512154727-aa3a9480609c/httpcache.go:192:192: dc.Key.Template undefined (type configurationtypes.Key has no field or method Template)

xcaddy build --with github.com/darkweak/souin/plugins/caddy@aa3a9480609c4a6c7c1d79815b119fd214ac0dda --with github.com/darkweak/souin@aa3a9480609c4a6c7c1d79815b119fd214ac0dda still doesn't work?

Alright, the build does work. My bad, @darkweak.
Will test it.

OK, I tested it. It seems to work well. However, in my use case I would need to be able to feed a custom variable in the key template. This doesn't seem to work properly.
This is my Caddyfile:
{
	order cache before rewrite

	debug
	log {
		output file /logs/main.log
		format json {
			time_format iso8601
		}
	}
	local_certs
	cache {
		default_cache_control public
		api {
			basepath "/boop"
			souin
		}
	}
}

:80 {
	header Content-Type "text/html; charset=utf-8"
	respond "Nothing here... Yet??????"
}

http://test.example.local {
	map {http.request.uri.query} {cacheKey} {
		~^(.*)(?:[?&]some_id=[^&]*)$ "?${1}"
		~^(some_id=[^&]*&?)(.*)$ "?${2}"
		~^([^&]*)(&some_id=[^&]*)(&?.*)$ "?${1}${3}"
		~^$ ""
		~^(.+)$ "?${1}"
	}
	header Custom-Cache-Key {http.request.uri.path}{cacheKey}
	header Content-Type "text/html; charset=utf-8"
	respond "Nothing here... Yet??????"
	cache {
		key {
			template KEY-{http.request.uri.path}-{http.request.uri.query}-{cacheKey}
		}
	}
}

Doing a curl -IX GET "test.example.local/gg/dd?aa=11&rr=22" will show the expected value of the custom variable (in the Custom-Cache-Key header) but it wont show at all in the Cache-Status header:
~> curl -IX GET "test.example.local/gg/dd?aa=11&rr=22"
HTTP/1.1 200 OK
Cache-Control: public
Cache-Status: Souin; fwd=uri-miss; stored; key=KEY-/gg/dd-aa=11&rr=22-
Content-Length: 25
Content-Type: text/html; charset=utf-8
Custom-Cache-Key: /gg/dd?aa=11&rr=22
Server: Caddy
Date: Mon, 13 May 2024 10:16:10 GMT

Any ideas, @darkweak ?

I mean, this particular issue can be closed by you PR, I am wondering where I should open a new issue about the custom value though :)

@teodorescuserban It should work with the commit 4e350ccccf7c0e507e44ba0fd34eef324a41dff8
xcaddy build --with github.com/darkweak/souin/plugins/caddy@4e350ccccf7c0e507e44ba0fd34eef324a41dff8 --with github.com/darkweak/souin@4e350ccccf7c0e507e44ba0fd34eef324a41dff8

@darkweak it works like a charm. erm, actually, id does work very well ;)
Thank you for your quick response!
This ticket can now be closed imho (along with PR merge)
When we should expect this feature to be released in a new version?


when I complie with c++ ，is error

cpp_media_server/net/webrtc/support_rtc_info.cpp:68:1: sorry, unimplemented: non-trivial designated initializers not supported
};
^
cpp_media_server/net/webrtc/support_rtc_info.cpp:68:1: sorry, unimplemented: non-trivial designated initializers not supported
cpp_media_server/net/webrtc/support_rtc_info.cpp:68:1: sorry, unimplemented: non-trivial designated initializers not supported
cpp_media_server/net/webrtc/support_rtc_info.cpp:68:1: sorry, unimplemented: non-trivial designated initializers not supported
[ 70%] Building CXX object CMakeFiles/cpp_media_server.dir/net/httpflv/httpflv_server.cpp.o
[ 71%] Building CXX object CMakeFiles/cpp_media_server.dir/net/httpflv/httpflv_writer.cpp.o

change  gcc 9.4 , everything is ok

是的，推荐gcc8.4版本以上来编译。


change Paul to BaM

@scopatz this should do the work :)

done @scopatz

Thanks!

don't forget to fix the TWoFCS repo too :)

@scopatz sorry I miss that comment, did you do it ?

yeah I did!
On Thu, Jul 20, 2017, 18:03 Mouginot Baptiste<EMAIL_ADDRESS>wrote:

@scopatz https://github.com/scopatz sorry I miss that comment, did you
do it ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ergs/rg-db-twofcs/pull/8#issuecomment-316843027, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AATkKUFSqzlsq4B01DOVtVNn7QIWIiGWks5sP865gaJpZM4OeduJ
.

--
Asst. Prof. Anthony Scopatz
Nuclear Engineering Program
Mechanical Engineering Dept.
University of South Carolina
<EMAIL_ADDRESS>Office:<PHONE_NUMBER>
Cell:<PHONE_NUMBER>
Office: Horizon 1, #011 (ground floor, not first)
Book a meeting with me at https://scopatz.youcanbook.me/
Open up an issue: https://github.com/scopatz/me/issues
Check my calendar
https://www.google.com/calendar/embed?src=scopatz%40gmail.com


Add the norm step to WIT

We add the Normalise WIT result example section from: https://github.com/GeoscienceAustralia/dea-notebooks/blob/witbook/Scientific_workflows/Wetlands_Insight_Tool/WITBook.ipynb to the formal WIT processing when we stack PQ files to CSV files.

The attach image is the test running result (the updated WIT CSV file).


Text and icon alignment inconsistent for mdi & images

Hi, I have recently updated from 1.1.6 to 1.5.1 and noticed this issue. The alignment of the text and the icons in a tile is different if MDI icons are used and different if custom images are used.

The config blocks in both cases are identical.
Am I doing something wrong?

Whoops, sorry about that!
Can you give me a fragment of your config.json so I can replicate it?

Here's the output source for the example with the image...

and here's the output source for the with the MDI icon...

Can you give me a fragment of your config.json so I can replicate it?

Here's the config block for the service that uses a custom image...
      {
        "name": "Short Name",
        "uri": "http://service.url",
        "description": "Short description",
        "icon": "/Custom/mariadb.png",
        "iconColor": "#FFF",
        "iconBG": "#555",
        "iconBubble": true
      },

And here's one for a service that uses MDI icon...
      {
        "name": "Another Service",
        "uri": "http://different.url",
        "description": "Similar description",
        "icon": "mdi-certificate",
        "iconColor": "#FFF",
        "iconBG": "#555",
        "iconBubble": true
      }

Dashboard icons behave similar to custom image icons. I believe there is an issue with the layout with MDI icons.

Fixed in v1.5.2

The fix works. Thanks.


Add ability to upload inline PDFs in blog

feature request that enables authors to upload inline PDFs in Sanity IO

Indeed me and @rstycedar spoke already but I'm posting here for transparency: ideally this can be a generic "file" of known file types, and can just be inline text rather than a text block. I believe sanity refers to this as 'annotation' https://www.sanity.io/docs/configuration#057f677d86d9

@Atmosfearful @0xAeterno we can also add a displayAsButton option to the schema if we wanted to let authors choose to display the annotations as a button


Add python 3.11 to testing

Codecov Report

Merging #286 (187b5db) into main (f626fb5) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main      #286   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           12        12
  Lines          411       411
=========================================
  Hits           411       411

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Some questions about reproducing the results in the paper

Hello author! ~ I really like your work, fantastic job, I would like to try to perform a reproduction of the results in the paper, such as the dfew dataset, can you explain how to do this, such as how to run it? What parameters should be modified, I see that the input image size in the code is 160, but the dfew dataset Clip is 224*224, what kind of preprocessing is done to the data? Thank you very much for reading ~ looking forward to your reply!

Hi, thanks for your interest in our work!
For fine-tuning on DFEW, you can check the running script here. The hyperparameters in the script shall be the same as that given in the paper. You can also modify them (e.g., batch size and learning rate) as you want and you might achieve better results as we do not heavily tune them.
For facial image size, since the original size of cropped face provided by DFEW is 256256 (not 224224), we resize it to 160*160 before patch embedding. More augmentations can be found in the dataset code.

Okay! I understand! Thank you very much~ :D

@sunlicai
I have some small problems with the model, can you please answer them, it won't take much of your time.

my dataset is not balanced and I want to use a different loss function (e.g. focal loss) do I just need to change criterion to the new loss function and turn off mixup_fn (if I don't turn it off it will give me an error)
does this code support regression task, my dataset needs to perform (0,1) regression task, if so how to modify it?
Looking forward to your reply!

Translated with www.DeepL.com/Translator (free version)

@sunlicai
I have some small problems with the model, can you please answer them, it won't take much of your time.

my dataset is not balanced and I want to use a different loss function (e.g. focal loss) do I just need to change criterion to the new loss function and turn off mixup_fn (if I don't turn it off it will give me an error)
does this code support regression task, my dataset needs to perform (0,1) regression task, if so how to modify it?
Looking forward to your reply!

Hi, my answer is as follows:

Yes!
Currently, the code does not support regression task. However, I believe that the modification is simple, i.e., you just use your own regression loss (e.g., mse or l1) as in question (1).

Best,
Licai


Signing errors on macOS pod bundle resources on Xcode 14 "Signing for "x" requires a development team."

macOS version of https://github.com/flutter/flutter/issues/111475.

Since I upgraded to Xcode 14, my Flutter project doesn't run because of Signing issues of iOS pods
Error:
Could not build the precompiled application for the device.
Error (Xcode): Signing for "DKPhotoGallery-DKPhotoGallery" requires a development team. Select a development team in the Signing & Capabilities editor.
{path to project folder}/ios/Pods/Pods.xcodeproj

Error (Xcode): Signing for "DKImagePickerController-DKImagePickerController" requires a development team. Select a development team in the Signing & Capabilities editor.
{path to project folder}/ios/Pods/Pods.xcodeproj

Error (Xcode): Signing for "TOCropViewController-TOCropViewControllerBundle" requires a development team. Select a development team in the Signing & Capabilities editor.
{path to project folder}/ios/Pods/Pods.xcodeproj

My project's dependencies (to reproduce the problem, the error is from one of these dependencies and the corresponding iOS Pod)
flutter doctor -v:
[✓] Flutter (Channel stable, 3.3.1, on macOS 12.6 21G115 darwin-arm, locale en-NL)

Need to find an example plugin that reproduces this, as google_sign_in is not supported on macOS.

@cbracken this will cause macOS apps not to build if any dependencies contain a resource bundle.  https://github.com/flutter/flutter/issues/111475 iOS variant was a customer: crowd.

The iOS fix was https://github.com/flutter/flutter/pull/111714.  The same change needs to be made to https://github.com/flutter/flutter/blob/1b583a837d692e73488a8ea6b8ab696d1e75bc41/packages/flutter_tools/bin/podhelper.rb#L96  I would have just done this myself in the same PR but there's no good existing macOS tests for this area so it would have taken much longer, and I wanted to get the iOS change out ASAP.

Thanks; picking this up now on the macOS side.

Great news is this was fixed by @jmagman in https://github.com/flutter/flutter/pull/82048 1.5 years ago!


ESBuild failures related to farmhash on firebase-admin v12.1.0

Environment

Operating System version: Mac OS Ventura (13.x) on Intel
Firebase SDK version: firebase-admin v12.1.0)
Firebase Product: firebase-admin for node
Node.js version: 20.5.1
NPM version: 9.8.0

Problem
Steps to reproduce:
Starting work on a colleague's project, I run npm install. We've pinned firebase-admin to ^12.0.0; this installs 12.1.0. This works correctly.
When I build my SvelteKit project - which will eventually use ESBuild - I get the same errors as reported in this StackOverflow post - "can't resolve build/Debug/farmhash.node".
When I pin firebase-admin to 12.0.0 (specific version): no such error. So the StackOverflow post, filed a day or two ago, seems to be related to something in the 12.1.0 changes.
Beyond this, I'm a bit lost, but managed to find evidence of someone else with the same issue, and now have a workaround - pinning to an old verison. There seem to have been other farmhash issues that Doug Stevneson has mentioned.

Update: farmhash needs to be specified as an ESBuild external. In the case of svelte-adapter-appengine, that was an extra configuration step I hadn't configured, is all.

@infovore can you talk a little bit more about what you did? I reverted to 12.0.0, but I'm hoping this doesn't prevent me from upgrading indefinitely

@balloman of course - I added farmhash to the externals section of my esbuild configuration.
Vite - in optimizeDeps.externals - and other build tools should have similar configuration options.

Did anyone made it work with @nx/esbuild ? Adding "external": [ "farmhash" ] doesn't work..

I'm also facing this with Next.js and had to downgrade to 12.0.0 in order to build as well.


Add a section about multiple file tests.

Personally, I chased my tail for a while because this was missing. There are a few raised issues about this, so I really think this will help others get started with the library quicker.

Thanks for this. You're absolutely right that this is something that is often missed - and something needs to go in the tutorial about it.
I had a slightly different view on how to present it, though - so I've actually written this instead:
https://github.com/philsquared/Catch/blob/master/docs/tutorial.md#scaling-up
Thanks for the prompt.

@philsquared perhaps: s/real world tests/real-world tests/ ?

yep - good call - thanks


add systemd hardening options - #26

I took options standard for Ubuntu 20.04 and Centos/RHEL as current ones.
That should not be blocking on older releases, just some warning of unsupported options.
Commented out options for more recent (ProtectProc)

Any feedback here?
would be a good addition to https://github.com/Sysinternals/SysmonForLinux/pull/125
integrated in my ansible role since and used fine in my setups
https://github.com/juju4/ansible-sysmon/blob/main/templates/systemd-hardening.conf.j2

Sorry for the delay here. I'll spend some time next couple of days testing this out and let you know.


BUG - unhandled control characters in VT reports prevents report generation/copying

Probably requires the same filter as you used for #109
I'll send a query to reproduce.

fixed - it was assuming type string for all input


[Days with QSOs] Added stats for weekdays

Thanks to @DanielBausset for the idea. Adapted to Wavelog.

Wow, you guys are fast ! 😄

@DanielBausset thanks. I liked your idea and wanted it in Wavelog. So credit where credit is due!
(I originally implemented days with QSOs and streaks in Cloudlog).


Redundant tool tips (tipsy and native) appear in IE when live:true

When using tipsy 1.0.0a with live:true (and default options otherwise), the first time the mouse hovers over the trigger element, IE8 shows both the tipsy tooltip and the native tooltip. The second time the mouse hovers, this doesn't happen -- only the tipsy tooltip appears.
This doesn't happen when 'live' is false, probably because tipsy moves the title attribute out of the way well before the hover event happens, before IE has a chance to render it. A workaround is to use some attribute besides 'title', although that can cause pages not to validate.

The above fix is working for me, too.

@ sarachandlee  Thanks its works for me, too


Invalid Component definition

I'm using your loader with vue 2.3.4 and after runing webpack and checking it in the browser it says Invalid Component definition but when i check the .svg it shows for example:
module.exports = {template: '<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 20 20"><path d="M6 2l2-2h4l2 2h4v2H2V2h4zM3 6h14l-1 14H4L3 6zm5 2v10h1V8H8zm3 0v10h1V8h-1z"/></svg>'};

I don't know why it isn't working.

Could you attach your svg file?

@santigarcor Could you attach your svg file?

@visualfanatic
<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 20 20" class="icon is-small is-filled-light"><path d="M3.94 6.5L2.22 3.64l1.42-1.42L6.5 3.94c.52-.3 1.1-.54 1.7-.7L9 0h2l.8 3.24c.6.16 1.18.4 1.7.7l2.86-1.72 1.42 1.42-1.72 2.86c.3.52.54 1.1.7 1.7L20 9v2l-3.24.8c-.16.6-.4 1.18-.7 1.7l1.72 2.86-1.42 1.42-2.86-1.72c-.52.3-1.1.54-1.7.7L11 20H9l-.8-3.24c-.6-.16-1.18-.4-1.7-.7l-2.86 1.72-1.42-1.42 1.72-2.86c-.3-.52-.54-1.1-.7-1.7L0 11V9l3.24-.8c.16-.6.4-1.18.7-1.7zM10 13a3 3 0 1 0 0-6 3 3 0 0 0 0 6z"/></svg>

Weird, could you try updating to v0.2?

same problem here..

https://github.com/visualfanatic/vue-svg-loader/issues/1#issuecomment-268308578 solved it for me.

@sijakret, @havgry, @sterlini-fueled: do you still have this problem? Let me know, so we can find a solution 👍

For me it was definitely just a conflict between file-loader and vue-svg-loader on the matched file types in my webpack configuration (see my reference above). I haven't had any issues since then.
Perhaps it would be a good idea to add a reminder in the readme to check for conflicts with other existing rules.

i think it was the same for me - thanks!
haven't had any issues recently, a mention of potential conflicts with other loaders is for sure not a bad idea!


Is Mistral-7B v0.2 supported?

System Info
A100 GPUs (40GB)
Who can help?
@byshiue
Information

[ ] The official example scripts
[ ] My own modified scripts

Tasks

[ ] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction
n/a
Expected behavior
n/a
actual behavior
n/a
additional notes
n/a

Yes, the tensorrt-llm already supported the  Mistral-7B v0.2 even we don't clarify it.
You may have a try with latest code.


Search

Note: Faceted Search is not in 7.5. We'll be using a combined search index for now.

Closing. This was implemented in #24


Why custom header are used in ApplicationInsightsInterceptor

Hey @VladimirRybalko ,
Since I'm using your module in our app, I've noticed that our sever received a lot more of OPTION requests.  I didn't know exactly where it could come from until I read in some documentation that custom headers could trigger such requests.  I remember that I add to add 'x-ms-request-root-id', 'x-ms-request-id' in our web.config file since they're used in this lib which make me think that these headers could be the reason why we get more OPTION requests now.
I suppose these headers are useful and I don't know why.  Can you explain me why they're used and a piece of documentation?
Thanks!
Quentin.

Hey, @n0is
The both headers are correlation headers that are used by Microsoft to link different telemetry requests.
You can find more information here


React-native 074 pod install error

Description
I'm getting this error when i do pod install:
[!] The following Swift pods cannot yet be integrated as static libraries:
The Swift pod lottie-react-native depends upon RCT-Folly, which does not define modules. To opt into those targets generating module maps (which is necessary to import them from Swift when building as static libraries), you may set use_modular_headers! globally in your Podfile, or specify :modular_headers => true for particular dependencies.
if i add the use_modular_headers! at the top of the Podfile i'm getting all these errors on Xcode

if i add the use_frameworks! :linkage => :static, please look at the image attached

with the use_frameworks! :linkage => :static i was able to do pod install, but when compiling the project i've got the error at the image
This is my PodFile:
`def node_require(script)  # Resolve script with node to allow for hoisting
require Pod::Executable.execute_command('node', ['-p',
"require.resolve(
'#{script}',
{paths: [process.argv[1]]},
)", dir]).strip
end
node_require('react-native/scripts/react_native_pods.rb')
node_require('react-native-permissions/scripts/setup.rb')
platform :ios, min_ios_version_supported
prepare_react_native_project!
setup_permissions([
'AppTrackingTransparency',
'Bluetooth',
'Calendars',
'CalendarsWriteOnly',
'Camera',
'Contacts',
'FaceID',
'LocationAccuracy',
'LocationAlways',
'LocationWhenInUse',
'MediaLibrary',
'Microphone',
'Motion',
'Notifications',
'PhotoLibrary',
'PhotoLibraryAddOnly',
'Reminders',
'Siri',
'SpeechRecognition',
'StoreKit',
])
React Native Maps dependencies
rn_maps_path = '../node_modules/react-native-maps'
pod 'react-native-google-maps', :path => rn_maps_path
pod 'Firebase', :modular_headers => true
pod 'FirebaseCore', :modular_headers => true
pod 'FirebaseCoreInternal', :modular_headers => true
pod 'GoogleUtilities', :modular_headers => true
linkage = ENV['USE_FRAMEWORKS']
if linkage != nil
Pod::UI.puts "Configuring Pod with #{linkage}ally linked Frameworks".green
use_frameworks! :linkage => linkage.to_sym
$RNFirebaseAsStaticFramework = true
end
target 'myapp' do
config = use_native_modules!
use_frameworks! :linkage => :static
use_react_native!(
:path => config[:reactNativePath],
# An absolute path to your application root.
:app_path => "#{Pod::Config.instance.installation_root}/.."
)
target 'myappTests' do
inherit! :complete
# Pods for testing
end
post_install do |installer|
# https://github.com/facebook/react-native/blob/main/packages/react-native/scripts/react_native_pods.rb#L197-L202
react_native_post_install(
installer,
config[:reactNativePath],
:mac_catalyst_enabled => false,
# :ccache_enabled => true
)
end
end
`
Steps to reproduce
Create a project with react-native 0.74 CLI
install lottie
open ios folder and pod install
Ps: cant add lottie at the snack

Snack or a link to a repository
https://snack.expo.dev/?platform=ios
Lottie React Native version
6.7.2
React Native version
0.74.0
Platforms
iOS
Workflow
React Native
Architecture
Paper (Old Architecture)
Build type
None
Device
None
Acknowledgements
Yes

https://snack.expo.dev/?platform=ios

Hi @rodrigodiasf1984
Your reproducible link is incorrect. Please provide the correct one, and we will take a look. Otherwise the issue will be closed automatically after some time.

Hi @TheRogue76, i cant add the last version of the lottie on the snack, have you saw the error on the attached image?

Hi @TheRogue76, i cant add the last version of the lottie on the snack, have you saw the error on the attached image?

An image and a version are not really helpful when it comes to these things. I am missing a lot here, this is just a snippet of a section of your setup. I need a reproducible to be able to check things out.

@TheRogue76  I've just create a repo, can you please take a look at?
https://github.com/rodrigodiasf1984/rn074Test

run pod install
build the project on Xcode
the error shows up

@rodrigodiasf1984 That's not the error your reproducible is giving me:

i get this error while doing pod install on react native 0.74.1 :
`
[!] Error installing lottie-ios
[!] /usr/bin/git clone https://github.com/airbnb/lottie-ios.git /var/folders/10/q1z46hyj75dbvn03g6p2k1b00000gn/T/d20240506-35168-abvjti --template= --single-branch --depth 1 --branch 4.4.1
Cloning into '/var/folders/10/q1z46hyj75dbvn03g6p2k1b00000gn/T/d20240506-35168-abvjti'...
error: RPC failed; curl 92 HTTP/2 stream 5 was not closed cleanly: CANCEL (err 8)
error: 3529 bytes of body are still expected
fetch-pack: unexpected disconnect while reading sideband packet
fatal: early EOF
fatal: fetch-pack: invalid index-pack output
`
my podfile :
require File.join(File.dirname(node --print "require.resolve('expo/package.json')"`), "scripts/autolinking")
Resolve react_native_pods.rb with node to allow for hoisting
require Pod::Executable.execute_command('node', ['-p',
'require.resolve(
"react-native/scripts/react_native_pods.rb",
{paths: [process.argv[1]]},
)', dir]).strip
platform :ios, min_ios_version_supported
prepare_react_native_project!
use_frameworks! :linkage => :static
$RNFirebaseAsStaticFramework = true
linkage = ENV['USE_FRAMEWORKS']
if linkage != nil
Pod::UI.puts "Configuring Pod with #{linkage}ally linked Frameworks".green
use_frameworks! :linkage => linkage.to_sym
end
target 'Soroosh' do
use_expo_modules!
post_integrate do |installer|
begin
expo_patch_react_imports!(installer)
rescue => e
Pod::UI.warn e
end
end
config = use_native_modules!
use_react_native!(
:path => config[:reactNativePath],
# An absolute path to your application root.
:app_path => "#{Pod::Config.instance.installation_root}/.."
)
target 'SorooshTests' do
inherit! :complete
# Pods for testing
end
post_install do |installer|
# https://github.com/facebook/react-native/blob/main/packages/react-native/scripts/react_native_pods.rb#L197-L202
react_native_post_install(
installer,
config[:reactNativePath],
:mac_catalyst_enabled => false,
# :ccache_enabled => true
)
end
end
`

Your Podfile is not configured properly and it has nothing to do with lottie-react-native. @rodrigodiasf1984
You can apply this patch for the fix on your reproducible:
diff --git a/ios/Podfile b/ios/Podfile
index d75e5d1..77e26a8 100644
--- a/ios/Podfile
+++ b/ios/Podfile
@@ -35,15 +35,6 @@ setup_permissions([
   'StoreKit',
 ])

-# React Native Maps dependencies
-
-rn_maps_path = '../node_modules/react-native-maps'
-pod 'react-native-google-maps', :path => rn_maps_path
-
-pod 'Firebase', :modular_headers => true
-pod 'FirebaseCore', :modular_headers => true
-pod 'FirebaseCoreInternal', :modular_headers => true
-pod 'GoogleUtilities', :modular_headers => true

 linkage = ENV['USE_FRAMEWORKS']
 if linkage != nil
@@ -54,13 +45,23 @@ end

 target 'rn074andLottie' do
   config = use_native_modules!
-  use_frameworks! :linkage => :static
   use_react_native!(
     :path => config[:reactNativePath],
     # An absolute path to your application root.
     :app_path => "#{Pod::Config.instance.installation_root}/.."
   )

+  # React Native Maps dependencies
+
+  rn_maps_path = '../node_modules/react-native-maps'
+  pod 'react-native-google-maps', :path => rn_maps_path
+
+  pod 'Firebase', :modular_headers => true
+  pod 'FirebaseCore', :modular_headers => true
+  pod 'FirebaseCoreInternal', :modular_headers => true
+  pod 'GoogleUtilities', :modular_headers => true
+
+
   target 'rn074andLottieTests' do
     inherit! :complete
     # Pods for testing

Then you'll need to run this in your ios folder:
USE_FRAMEWORKS=static bundle exec pod install

I am gonna close this issue.


feat: reset to default settings

Add a reset default settings option.  Of increasing usefulness as we've been adding, and may continue to add more settings in future.

I perceive that icon as "undo my last change," not "undo all changes, ever." I could, therefore, click it and get effective data loss without intending...

Is there an alternative icon you'd propose - https://primer.style/foundations/icons?
Would there be a different location you'd prefer?
I figured the hover text was enough 🤷♂

I'd argue more against the feature, than the icon itself. Why would we want this?
(But if we really want this, I'd say actual text aligned to the right saying "Reset to defaults *currentUndoIconHere*")

I'd argue more against the feature, than the icon itself. Why would we want this?

I was a little unclear in my PR.  I think this could be helpful for users who have ignored new settings, or, tinkered with settings and want to revert back to the out-of-the-box / default combination of settings that we recommend.

(But if we really want this, I'd say actual text aligned to the right saying "Reset to defaults currentUndoIconHere")

How about this 😎 - update pushed to branch

@setchy took the liberty to push to this branch the option I think is the most viable. Hope that's okay.
Let me know what you think!

Works for me

I think it is a bit more clear right out of the gate and it's only visible if you're looking for it. Which, for a "deep" setting, makes sense!


Fix Broken Link - Pull request template

Describe your changes :

Worked on fixing the broken link on PR template

Type of change :

[x] Bug fix
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation

Frontend Preview (Screenshots) :
For frontend related change, please link screenshots of your changes preview! Optional for backend related changes.

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers

Failing tests not related to this PR.
Merging it


How to make Postgres "SELECT DISTINCT ON" query using gorm sql builder?

Your Question
Is it possible build "SELECT DISTINCT ON" query using sql builder?

The document you expected this should be explained
https://gorm.io/docs/query.html#Distinct

Expected answer

Sample table example:
CREATE TABLE browsers (
    name character varying,
    version integer
);

INSERT INTO browsers VALUES ('chrome', 68);
INSERT INTO browsers VALUES ('chrome', 69);
INSERT INTO browsers VALUES ('chrome', 70);
INSERT INTO browsers VALUES ('firefox', 40);
INSERT INTO browsers VALUES ('firefox', 41);

I want to build the query below using sql builder
SELECT DISTINCT ON (name)
  name,
  version
FROM
  browser
ORDER BY
  name,
  version DESC

Expected output (browser name and latest version):
|name   |version|
|-------|-------|
|chrome |71     |
|firefox|41     |

Sorry, Distinct api not support it, you can use Select with raw query.
DB.Select("DISTINCT ON (name) name", "version").Find()


[CI] Downgrade python to 3.9 for torch on MacOS

Following a hint at https://github.com/langchain-ai/langchain/issues/2991
Fixes #10 (I hope)

Guess not


Handle redeem_swap error in case the invoice is known

This PR attempts to fix the issue in the following scenario:

user starts a swap on device A
user pays onchain amount and it confirms, but for one reason or another the invoice cannot be paid (for example, the amount was below min_allowed_deposit)
user restores on device B
there, the ongoing swap sees a confirmed tx and attempts to create an invoice, but fails because GL cannot create another invoice with the same preimage

The main change brought by the PR is in redeem_swap:

if no invoice is known, try to create it on GL
if invoice creation fails with InvoicePreimageAlreadyExists, try to retrieve it from DB

Fixes #710

The PR is marked as draft, because it doesn't yet solve the problem.
The issue is this invoice doesn't exist on device B. It exists on GL, but we only sync() settled invoices (see here and here). The not-yet-paid invoice doesn't therefore reach the DB.
On device B, calling in_progress_swap will currently result in redeem_swap failing with
"last_redeem_error": "Preimage already known, but payment not in DB"

IMO the solution is to move bolt11 from the (local) swaps_info to the (sync) swaps. The downside is this is intrusive. Still looking for a simpler solution.

One option would be to just call list_invoices with a payment_hash filter to retrieve it from GL.

The PR is ready for review.
I added a few other fixes and smaller refactorings. I separated them in individual commits, for easier review.
I tested both the success and failure cases, both were handled correctly.


Add a password policy for IAM users

This PR:

Creates a password policy for IAM users

This forces:

30 day rotation periods
minimum 12-character passwords
lowercase and uppercase characters, numbers, symbols
prevents reusing any of the 5 previously used passwords

It also allows a user to change their own password without enforcing an administrator reset.

Generally I'm not a fan of forcing password resets but if this is purely for the admin users and we are all using a password manager it shouldn't be an issue. I thought we had some password guidance somewhere but I can't find it now :/ have you seen the AWS baseline requirements - https://ministryofjustice.github.io/security-guidance/baseline-aws-accounts/#identity--access-management
We'll need to have MFA as well but can be done in a later PR.

@davidkelliott If we do decide to use traditional IAM users as well as or rather than federated IAM users, I thought this would be good to have in place before it gets forgotten about or overlooked in the future, regardless of if it's actually used at all.
I'll align this with the password standard, thanks for letting me know!
As MFA involves an IAM policy and is a bit more involved, I thought I'd separate that out into a separate PR 👍

Posting the link to the guidance you found here in case I lose it again - https://intranet.justice.gov.uk/guidance/security/it-computer-security/ict-security-policy-framework/password-standard/


Add call-for-proposal copy for 2022

Add a section on giving talks
Add call for proposal link to CFP section

Based on the netlify deploy preview, the font font sizes looked a bit weird. Unsure if that is intentional

Based on the netlify deploy preview, the font font sizes looked a bit weird. Unsure if that is intentional

We can fix that separately

Spacing and font-size fixed in #176


Test that os/exec correctly handles duplicate environment variables

Builds on top of #16.
What
In #3 a user reported problems when conduit set duplicate environment variables. We traced this to a problem in os/exec that was fixed in Go 1.9. In #9 we planned to work around this, but decided to disable Go 1.8 instead.
As well as telling people to use Go 1.9 or greater in the readme, this adds a test that os/exec is behaving correctly. It then disables Go 1.8 in Travis builds, as the test fails and should fail in such an environment.
How to review
Code review should suffice. The tests failed on Go 1.8 only after adding the new test: https://travis-ci.org/alphagov/paas-cf-conduit/builds/305550179.
If merging this, close #3 as fixed and close #9 as replaced by this other direction.
Who can review
Not @46bit.

I managed to reproduce the 1.8 fail and 1.9 pass. 👍


publish esm

refs: https://blog.sindresorhus.com/get-ready-for-esm-aa53530b3f77
i'm happy to help if you want this way. ✋

This is a super popular package, so I plan to wait a little bit so I don't have to deal with so many support related issues. I'll probably do this in May next year.

It's now May, and I just ran into the nasty require is not defined and tracked my way back to this issue.

Hi @sindresorhus I really appreciate your work and many thanks for this package. Are there plans to offer esm export?
As you said its a very popular package and having ESM will help tools to optimize and reduce the bundle sizes


translated english to simplified Chinese

Woho - awesome,

Can you please run yarn lintfix

And add something like this to nuxt.config.js under modules['i18n'].locales:

...
          {
            name: 'Chinese',
            code: 'zh-Hans',
            iso: 'zh-Hans',
            file: 'zh-Hans.js',
            dir: 'ltr'
          },
...

If you now run yarn and then yarn dev and open the web-browser on http://localhost:3000 you will find your language in the dropdown-list.
See that it works at expected.

Woho - awesome,

Can you please run yarn lintfix
And add something like this to nuxt.config.js under modules['i18n'].locales:

...
          {
            name: 'Chinese',
            code: 'zh-Hans',
            iso: 'zh-Hans',
            file: 'zh-Hans.js',
            dir: 'ltr'
          },
...

If you now run yarn and then yarn dev and open the web-browser on http://localhost:3000 you will find your language in the dropdown-list.
See that it works at expected.

Okay, will do that

Hey, I've done the 2 tasks:
>yarn lintfix
>add the lang info to nuxt.config.js
But when I try to run yarn dev it is continuously giving me errors.

Looks good to me :-) (but I don't know the language)
Thanks
Deployed here: https://bigfive-test.com/zh-Hans


Drain operations should not complete until the new nexus target is connected

Use a vector of drained volumes and the start timestamp of the drain operation so that marking a node as drained can be delayed until no shutdown nexuses remain on that node.

@chriswldenyer please squash the commits before merging

bors try

bors try

bors merge

bors merge

bors merge


CI: Bump Rust to 1.83.0

coverage: 61.022%. remained the same
when pulling a3b36dff54e0ec8fbff19c2d7ef77a7243ca802c on feature/ci-bump-rust
into b4e0e37d3a27e16c1a6e8cd5301345573180c171 on master.


Added hash input order option when importing users

Implements #313.
This PR includes the implementation of a new hash.inputOrder option for the importUsers() auth methods, with the relevant unit and integration tests and type definitions. See the issue for more information and a usage example.
Note that all unit tests pass on my local machine and all integration tests except admin.auth importUsers() successfully imports users with MD5 to Firebase Auth w/ explicit password-first hash pass as well. It is my understanding from the --hash-input-order documentation that it should apply to the MD5 algorithm as well. However, this integration test fails although the request being made to the Firebase Auth backend looks correct to me. It would be great if someone from the Firebase Auth team took a look into what is going wrong with that test.

Sorry for the delay @jwngr. The API review for your proposed Node.js changes is under way.


expose the navigator route as a service extension

Something like:

[ ] get the current value for the route
[ ] set a new value for the route
[ ] enable and disable receiving events on route changes

@Hixie, I very briefly looked at the flutter sources, but didn't see how to get this info.

This would be if IntelliJ wants to display it.

Yes, we would like to display this. When we discussed this previously the main issue that was brought up was that there was no concept of a single navigator.

On cupertino (and indeed in any app, but it's common for cupertino apps) there can be multiple navigators. How would you want to handle that?


feat: :lipstick: move the generation accordion closer to the top

What type of PR is this? (check all applicable)

[x] Refactor
[ ] Feature
[ ] Bug Fix
[ ] Optimization
[ ] Documentation Update
[ ] Community Node Submission

Have you discussed this change with the InvokeAI team?

[ ] Yes
[x] No, because I feel like it can be just as easily discussed here.

[ ] Yes
[x] No

Description
This PR moves the model and concepts accordion up one. Why? Because the model should be the very next setting adjusted after the prompting.
Hell, if I had it my way the very first thing selected should be the model, and loRas should be in the middle. But I feel like this is the next best way to do it without too many compromises.
Why do I believe the model should be selected first? Hierarchy. A setting's effects should bubble it's way down to it's lesser relevant settings. Given the model influences how one would approach prompting, it would ideally be the first thing selected and take position 1 in hierarchy. Then prompting, then image settings, then concepts and so on and so forth.
The recent UI updates to invoke, while improving on many fronts of the UI as a whole have also compromised the hierarchy, leaving me and others juggling up and down the list of settings whereas it was not this way before 3.6.
Related Tickets & Documents

Related Issue #
Closes #

QA Instructions, Screenshots, Recordings

Merge Plan

Added/updated tests?

[ ] Yes
[x] No : Shouldn't effect tests

[optional] Are there any post deployment tasks we need to perform?

Noted this in discord, but placing here as well --
I think this is a broader problem that can be solved with custom-optionals panel ordering, vs changing it for all. I don't think the "Model first" hierarchy is a universal workflow - especially for folks who have one standby generalist model (a lot of the 'average' userbase)


Colourblind-friendly Hunger and Thirst Icons

About the PR

Changes the icons for the three stages of both Hunger and Thrist to be more visually distinct from their other stages and to no longer rely on colour coding.
Creates a flashing animated sprite for Starvation and Parched states, making them much more noticable with the intention to draw attention towards managing these states.
Related: https://github.com/space-wizards/space-station-14/issues/18032#issuecomment-1636675201
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

First iteration

Second iteration:

Changelog

:cl: Keiku

tweak: Nanotrasen employees' bodily needs are now less ambigious and ignorable. Icons for Hunger and Thirst states have been modified.

definitely much better spritework and colorblind-friendly icons are always good, im not sure the thirst ones make a lot of sense to me though

definitely much better spritework and colorblind-friendly icons are always good, im not sure the thirst ones make a lot of sense to me though

Thank you!
I'm not entirely confident on how to convey increasing levels of Thirst, so it's something I will iterate on further.

I think the thirst one is a clear enough indicator personally, definitely a big improvement over the prior one.
Color-blind icons are always a nice to have, we really need more accessibility features.

Thirst icons have been changed and the screenshots have been updated. I'm much happier with this second iteration.
How do the new ones look?

that's really should ve toggleable

yeah i like the new ones a lot more

These are great, but the starving one seems a bit complicated to me and the yellow outline looks a bit of weird (the shape I mean)

Thank you for your contribution!


1minute bar range issue

Hi, thanks for putting this nice feature together. In an attempt to verify the result, I noticed that the function get_bars in download.py use time filter 09:29 to 15:59, i think (only based on observation), the minute bar in IQFeed means the bar from previous defined seconds up to that time. In our case, 09:30 bar means data from 09:29 to 09:30 and 15:59 means bar from 15:58 to 15:59. The drawback of it is that if we aggregate the minute bars to get daily open and close, the numbers are not accurate. I used the following (TICKER: EWZ) as a test, here is two lines I get from the file:
DateTime,Open,High,Low,Close,Volume
20161104 093000,35.55,35.55,35.55,35.55,138622
20161104 093100,35.53,35.545,35.39,35.41,678595
20161104 155900,35.355,35.37,35.34,35.345,18997559
if we check google or yahoo for 20161104 open and close, i have the following:
Date	                Open	High	        Low	        Close	Volume
Nov 4, 2016	35.53	36.11	35.18	35.36	20,544,162
You can see that the Open is what we have on<PHONE_NUMBER>3100 and the Close is not matched.
I believe simply change the filter time to 0930 and 1600 will solve this problem.

@boyuhou:
Thanks for reporting this issue. IIRC I needed to specify the datetimes as 9:29 and 15:59 to get correct daily results.
Currently I do not have IQFeed access therefore I cannot validate your assumption directly.
I do have access realtime & historical data through other provider (IB).
If you have IQFeed access I'd suggest that you download yesterday's prices for SPY using both the original (9:29 & 15:59) and your proposed (9:30 & 16:00) timestamps, I load it from IB then we cross-check.
Is this sounds reasonable to you?


Adapt solver-service for OCaml-CI and OCaml-multicore-CI

This is the steps.

[x] Upgrading solver-service to ocaml-ci one.

the solver in solver-service is from ocaml-ci solver at some point. The last code of ocaml-ci's solver don't change much.

[x] Adapt the protocole of solver-service for more opam-repository.
[ ] Split schema.capnp file to 2 two file solve.capnp and build.capnp
[x] Connect ocaml-multicore-ci using the protocol from solver-service-api library (vendored).
[ ] Connect ocaml-ci same as ocaml-multicore-ci but with some modification to adapt the protocole.

[ ] Extend solver requests to include caching hints, similar to build jobs, so that repeated solves re-use the same machine.
[ ] Return current solver machine (toxis) to general ocluster useage.


Dependabot/bundler/rubocop jekyll tw 0.13.0

I'm opening this PR in the hopes of helping move along a new release of github-metadata.
This has the same version bump of rubocop-jekyll as #251 but I also went ahead and let rubocop autocorrect the formatting issues it found.

Just curious, why is appveyor used for CI? It seems like it has something to do with testing for Windows, but GitHub Actions builds for Windows also?

Hello @dunkmann00,
Thank you for submitting this PR. However, I am out of context here. That means the title + description of this pull request is inadequate. Anyone stumbling across this pull request (including maintainers, at a future date) is going to scratch their heads wondering about the purpose / aim of this PR. (No, assuming context from included commit messages isn't the right way.)
On the contrary to suggested fixes from RuboCop, resorting to ENV.fetch isn't ideal here because the changed lines here all default to nil which is already the implicit result of these lines prior to being changed. If you were to follow the link(s) given in the CI report for PR #251, you shall understand the reason why RuboCop chose to suggest change(s). Regardless, I'd like you to revert use of ENV.fetch and explicitly disable the cop in .rubocop.yml. That said, let us benchmark the two situations for clarity first.
The reasons why AppVeyor is still around when GH Actions exist:

Notice how AppVeyor has just one commit check status as against the many for GHA jobs? Notice how it's now slightly inconvenient  when wishing to access reports for just Ubuntu-based tests (Hint: visual noise).
Perhaps, the above point is just my problem, i.e. subjective. But, let us actually compare two  passing test reports for a given Ruby version, say Ruby 2.7. Notice (expand step Run script/cibuild) how while the Ubuntu-based report reads as expected, the Windows-based test hasn't actually run the underlying test-suite?!
Notice how the passing check for Windows-based tests led to the false assumption that the GH Action workflow was set up correctly and has run without errors, whereas AppVeyor has attempted to run the test-suite but failed due to version mismatch in appveyor.yml.

Hello @ashmaroli! My apologies on the title. I had used 'Checkout with GitHub Desktop' to get dependabot's commit and in the process of doing that I never really gave the title any attention (the current title was just auto generated). I'll fix that.
I did look at Rubocop's explanation, the change to ENV.fetch seemed unnecessary but I figured I'd let it make the auto change and then go back and correct anything as requested.

Notice (expand step Run script/cibuild) how while the Ubuntu-based report reads as expected, the Windows-based test hasn't actually run the underlying test-suite?!

😆 I didn't realize it didn't actually test anything! Well that makes perfect sense then. Would you like me to change anything in appveyor.yml to run Ruby 2.7 instead 2.6?

Would you like me to change anything in appveyor.yml to run Ruby 2.7 instead 2.6?

@dunkmann00 Actually, getting AppVeyor to run Ruby 2.7 is a bit tricky. I'll handle it via a separate PR and will ping you after its merged.

@dunkmann00 AppVeyor has been reconfigured on the main branch. You may incorporate those changes into the branch here by running the following in your local terminal:
git pull upstream main   # 'upstream' refers to the base repo in Jekyll org.

@ashmaroli Okay I incorporated your changes and updated the other rubocop issues. Let me know if you are okay with the current changes or if you would like any of them reverted.

Current changes look fine to me. However, there's a warning from RuboCop about incorrect cop namespace in .rubocop.yml. The current PR is the ideal place to fix the cop name. Please rename and move directive to appropriate location in .rubocop.yml (alphabetical order)

Should be all set now.

Thanks @dunkmann00
@jekyllbot: merge +dev

Thanks @ashmaroli! Any idea when you'll be able to release a new version?


version(4.2.7): Bug Fixes, DB change

DB SYSTEM CHANGED

DB cooldown
Array prefix
autoTyping (gcommands option)
autoDefer (button#edit option) [boolean]
Log event
msg#awaitButtons/createButtonCollector
Embed links fix
name is undefined (userRequiredRoles)
language per guild

^ Problem with structures has been fixed


GAM-    ERROR: 500: Backend Error - backendError

I have created a script to update 100 groups.  This same script has worked before but now I am getting the following error:
Temp error backendError. Backing off 16 seconds...attempt 5/10
Temp error backendError. Backing off 32 seconds...attempt 6/10
Temp error backendError. Backing off 60 seconds...attempt 7/10
Temp error backendError. Backing off 60 seconds...attempt 8/10
Temp error backendError. Backing off 60 seconds...attempt 9/10
Temp error backendError. Backing off 60 seconds...attempt 10/10
ERROR: 500: Backend Error - backendError
Temp error backendError. Backing off 16 seconds...attempt 5/10
Temp error backendError. Backing off 32 seconds...attempt 6/10
Temp error backendError. Backing off 60 seconds...attempt 7/10
Temp error backendError. Backing off 60 seconds...attempt 8/10
Temp error backendError. Backing off 60 seconds...attempt 9/10
Temp error backendError. Backing off 60 seconds...attempt 10/10
ERROR: 500: Backend Error - backendError
Part of the output txt file, does say "updated group xxx"
then I get the above error.
Running GAM 362
If anyone could help, would appreciate it.  I don't want to change the group settings manually.  I have these 100 plus another 150

This is generally a temp issue on Google's end, not a GAM bug. Closing. Re-open if you can consistently reproduce this issue.


Recreate GithubBridge to support separate content repos in self-hosted projects

Summary

This adds a GithubBridge in order to better support separate content repositories for self-hosted projects
This will also help provide a path to better support reindexing for self-hosted projects (i.e. keeping the data layer updated when changes are pushed directly via git, rather than made through the Tina UI)

This PR draws heavily from the original GithubBridge that previously existed - i.e. this historical file that was removed in this commit, so full credit to whoever wrote the original implementation!
Background

See the excellent information @logan-anderson provided in this discussion thread: https://github.com/tinacms/tinacms/discussions/3589#discussioncomment-4960323
And the further details in this issue: https://github.com/tinacms/tinacms/issues/3609, including this video overview of the key components to consider: https://www.loom.com/share/c7d1c4f5bda04ec3a1dacff2a03efac1
Follows on from the updates in this PR: #3830

@coreyaus many thanks for putting this together, it will help us a lot as we're currently struggling with this topic.

Thanks for the quick review @logan-anderson and great to hear it'll be useful @Perni1984!
That makes sense about moving this to its own package to avoid the @octokit/rest dependency for users who don't need it - I'm definitely happy to give that a go and should hopefully be able to take a look at it this weekend 👍 I'll post an update when there's a revised version you can look at, or if I have any questions along the way.
Cheers gents, and have a great day!

Thanks for the early feedback and guidance on this PR. The github-bridge worked well but the main issue I ran into when I created it a couple of months ago was that handling the sync to GitHub via the put function in the github-bridge made the onPut and onDelete handlers in the database.ts file redundant. When I initially ran the tinacms build from this PR locally in my test project I saw 2 commits for each saved change - the commit with the changes made by the github-bridge and an empty commit made via the onPut handler in the database.ts file.
To be honest I thought I'd written at the time to get feedback on how to handle that behaviour but obviously dreamt that so apologies I didn't provide an update earlier! I've been following some of the recent draft PRs related to self-hosting improvements (e.g. #4095 and #4106). It looks like those will address that point above and also add a bunch of powerful features so I will close this PR now, given it will be superseded when #4106 is merged to main
Let us know if there's any useful way we can contribute to that current work on self-hosted improvements (e.g. testing, docs, etc). In the meantime l'll give the latest code on the feat/self-hosted-improvements branch a spin locally - looks like that's possible by installing version 0.0.0-20230901181730 of the core packages.
Happy Friday, and thanks for all the epic work you're doing!

Hey all - just a quick follow up that now that the latest self-hosting features have been officially launched I've created a new PR around this same concept of re-introducing the GitHubBridge to support self-hosted repositories with separate content repositories. See the full details in PR #4384. Happy Friday!


metrics CI job is failing, blocking issue merging

Description of problem
Some PRs, like for example https://github.com/kata-containers/runtime/pull/2891, are blocked due to the CI failing. The jenkins-metrics-ubuntu-18-04 job is failing, and this is not directly related to these CRs.
Example of failure: http://jenkins.katacontainers.io/job/kata-metrics-runtime-ubuntu-18-04-PR/2069/console.
14:26:49 Report Summary:
14:26:49 +-----+----------------------+-------+--------+--------+-------+--------+--------+-------+------+-----+
14:26:49 | P/F |         NAME         |  FLR  |  MEAN  |  CEIL  |  GAP  |  MIN   |  MAX   |  RNG  | COV  | ITS |
14:26:49 +-----+----------------------+-------+--------+--------+-------+--------+--------+-------+------+-----+
14:26:49 | P   | boot-times           | 95.0% | 101.8% | 105.0% | 10.0% | 94.6%  | 117.6% | 24.3% | 4.5% |  20 |
14:26:49 | *F* | memory-footprint     | 95.0% | 106.4% | 105.0% | 10.0% | 106.4% | 106.4% | 0.0%  | 0.0% |   1 |
14:26:49 | P   | memory-footprint-ksm | 95.0% | 100.2% | 105.0% | 10.0% | 100.2% | 100.2% | 0.0%  | 0.0% |   1 |
14:26:49 | P   | blogbench            | 90.0% | 103.4% | 110.0% | 20.0% | 103.4% | 103.4% | 0.0%  | 0.0% |   1 |
14:26:49 | P   | blogbench            | 90.0% | 103.5% | 110.0% | 20.0% | 103.5% | 103.5% | 0.0%  | 0.0% |   1 |
14:26:49 +-----+----------------------+-------+--------+--------+-------+--------+--------+-------+------+-----+
14:26:49 Fails: 1, Passes 4

Expected result
CI metric repaired, or root cause for it failing repaired, so as to not block independent PRs.
Actual result
PRs that have no direct impact on that metric are blocked.
Further information
I have searched for any issue containing "metrics" and did not find any.

@GabyCT is working on this - issue https://github.com/kata-containers/tests/issues/2839

@GabyCT I assigned this one to you, since @devimc pointed out you were working on it. Please feel free to close as a duplicate if you think it's the same thing. Thanks.

@jodh-intel I propose to add a new temporary flag, "blocked-by-metrics", to flag PRs that are blocked by this.

It looks like https://github.com/kata-containers/packaging/commit/3d2bb461eab80fe5a1f61ea96ced8c27451fb7e7 introduced the regression

To increase memory limits PR https://github.com/kata-containers/tests/pull/2840

The PR got merged this issue should be resolved now

@c3d We will need to rerun the metrics job for the pending PRs with /test-ubuntu-qemu-metrics command.


Fix for bug when attempting to Cursor Slice an empty result set

When no results are returned by an underlying data source (e.g. empty list/table, etc) IEnumerableCursorPagingCustomExtensions.SliceAsCursorPage() creates a new CursorPageSlice and passes null for the results parameter. This causes an ArgumentException to be thrown by the CursorPageSlice constructor.
This bug was discovered when testing a resolver over a postgresql table I had just created but contained no rows.
The GraphQL query should return an empty node collection and totalCount = 0.
Relevant line for the bug:
https://github.com/cajuncoding/GraphQL.RepoDB/blob/52dabefdeec5f6d29e4f97c3a927bc05426ac31c/GraphQL.PreProcessingExtensions/Paging/CursorPaging/IEnumerableCursorPagingCustomExtensions.cs#L45
This can be reproduced by modifying Tests.StarWarsCharacterResolver.CreateCharacters to return an empty list and running the TestParamsContextCursorPagingOnlyFirst test.
This PR modifies behavior this by:

Avoid throwing ArgumentException in CursorPageSlice to match the way Offset pagination handles empty results.
Swapped from creating a new List to Enumerable.Empty() to avoid un-needed allocation.

Thanks for the details, I’ll take a look at the info provided this week!

Ok, Merged, pulled, and re-ran tests.... new test coverage is passing as expected (was definitely failing before).

I'll push a minor update to Nuget soon!


Where is the dot file?

I installed the plugin in Obsidian and Graphviz on Windows 10 but I have no clue where to find the DOT file.

It's under below directory under my Win 10.
C:\Program Files\Graphviz\bin

It's under below directory under my Win 10.
C:\Program Files\Graphviz\bin

I get this error from the codeblock: "C:\Program Files\Graphviz\bin -Tpng,C:\Users\dga\AppData\Local\Temp\tmp-19612-MefSc0JsUdfh" failed, Error: spawn C:\Program Files\Graphviz\bin ENOENT

@scriptor-dga

Please check if your dot is under same file
You should use the full path of the dot executable file, not the directory it's in, e.g. "C:\Program Files\Graphviz\bin\dot"


Update next_url in page_listing_buttons within wagtailadmin_tags.py

When using the search bar, the listing buttons get updated with the wagtailadmin_explore_results path. So if the move action button was used, after a successful move, I'll be redirect the the IndexResultsView rather than the IndexView.

Purpose: To not break the view on a success redirect.
Fixes #...
The next_url in page_listing_buttons template tag that might retrieve the IndexResultsView path rather than the IndexView path. Not sure if a replace is the proper way to do this, you're welcomed to change it.
Please check the following:

[x] Do the tests still pass?[^1]
[x] Does the code comply with the style guide?

[x] Run make lint from the Wagtail root.

[ ] For Python changes: Have you added tests to cover the new/fixed behaviour?

Please describe additional details for testing this change.
Go to the page explorer, type in a title in the search bar, hover over a the page listing button that receives next_url (like "Copy"),  and see if the next_path in the URL ends with /results/.
[^1]: Development Testing

Would it be ok if you raised this as a big first, with steps to reproduce and a screenshot if possible.
It will help the team triage this and maybe recommend a different approach to solving it.
I'm not sure hard coding a specific response path is the right way to go here.

I think #11170 may fix the same issue.

Closing in favour of the more generic approach https://github.com/wagtail/wagtail/pull/11191
@fadedsky thank you so much for alerting us to this problem and raising a really clear bug also.


MacOSX kernel panic

Your pull request seems good, but must be checked by a human maintainer for the following reason(s):

This contains potentially hazardous tags that need manual review.

Please be patient until a maintainer has time to look over your contribution.
This is an automated response, beep boop.
if you think there is a mistake, contact a maintainer

36ca4870e3cd18226b16b040ad1f94ab27d8a00d

Line 106 has "Yuo" instead of "You". I suggest you go through this one last time and make sure your spelling is correct. Thanks for your submission

Your pull request seems good, but must be checked by a human maintainer for the following reason(s):

This contains potentially hazardous tags that need manual review.

Please be patient until a maintainer has time to look over your contribution.
This is an automated response, beep boop.
if you think there is a mistake, contact a maintainer

fc44ffeba3fa7af4e9fa3a34c0de24acccaaa9bb


ngx_mruby supports ngx_devel_kit on git-subtree

ref: https://github.com/matsumoto-r/ngx_mruby/tree/master/dependence/ngx_devel_kit

👍  Looks ok. It's convenience to reduce build time. I like this changes.


Editing file from web browser keeps inserting new lines

Issue Type: Bug

Open a folder
Add a .py file to the folder and open the file.
Start a Live Share session
Join that session from a web browser.
From the web browser, begin typing. Do not press enter.

Expected
No line feeds are inserted and the caret remains in place
Actual
The web editing experience is frequently disturbed by the insertion of a line feed at the caret position. The caret is moved to the new line. This happens very frequently, sometimes while typing a single word. Check out my animated gif. Understand that of all the line feeds you see, the only time I pressed ENTER is the time the caret properly moved to the next line indented.

Extension version: 1.0.5449
VS Code version: Code 1.65.2 (c722ca6c7eed3d7987c0d5c3df5c45f6b15e77d1, 2022-03-10T14:33:55.248Z)
OS version: Windows_NT x64 10.0.22000
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8650U CPU @ 1.90GHz (8 x 2112)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.92GB (2.61GB free)

Process Argv
--crash-reporter-id 2257e870-20d1-47a1-95b1-3b48761b245c

Screen Reader
no

VM
0%

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492:30256859
vstes627:30244334
pythonvspyl392:30443607
pythontb:30283811
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscoreces:30445986
pythondataviewer:30285071
vscod805cf:30301675
pythonvspyt200:30340761
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
vsc1dst:30438360
pythonvs932:30410667
wslgetstarted:30449410
vsclayoutctrt:30451275
dsvsc009:30452663
pythonvspyt640:30450904
vscscmwlcmc:30438804
vscgsvidc:30447482
pynewfile477:30450038

Interesting - it looks like for this issue, the contents for both participants are the same. However, they're not the contents you intended (no unintended line feeds). Is that fair to say?
Also it doesn't seem like you did anything special. So does this consistently reproduce?

Yes on all counts.
I tried it twice

OK, at least that'll make it easier to fix. We'll get on it ASAP. Thanks for the reports!

I wonder if it depends on my VS Code's setting that prefers files to end with a newline character. VS Code alone only applies this policy when I save the file. But I wonder if during buffer sync it applies it as well and ends up moving the caret in the process.

Maybe... because I just tried the above steps and it did not reproduce for me. Let me try enabling that setting.

This is the files.insertFinalNewline setting?

Re-watching your GIF, it does seem like the file is being saved, which would cause that setting to insert a new line. I'm guessing you weren't intentionally saving it? So maybe the web guest is auto-saving, which triggers a save on the host, which triggers the insertion of the new line?

Ah yes, I think that's exactly what's happening here. By default, the web client will auto-save after a delay.
So this unbelievably is not actually a bug 😂. It's just a bad intersection of settings + modifying the last line of a file.
Perhaps we should ignore auto-saves from the guest though...

I'm glad you can repro.
I wouldn't expect an explicit save on the VS Code client to move the caret on the web client any more than an auto-save from the web client would. So in addition to any change to auto-save behavior, can we fix the buffer sync to not move the caret when a newline is added by the VS Code client?

This doesn't repro any more.


Feature/multi index

Implement multi-text searching using wildcards.
Possibly going to nuke search performance but it's basically useless in current form anyway.
Rather than debate performance, suggest we implement and see if it has any noticeable impact on performance of the application, which can be remedied fairly easily by removing the index at a later date.
Also amended the startup script so the index gets added to when the docker container starts.

It looks good to me and I agree that we can run with it, and it ends up being a performance issue later on we can deal with it.
The mongo init.js doesn't work for the test database. I will try to restructure this on my PR.


Hello

[Enter feedback here]

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

ID: d3e2b8c0-6652-53b8-ff40-37473c359b7e
Version Independent ID: 5be7927d-cf08-7438-ecfd-554a6928d122
Content: Upgrade your Azure account - Microsoft Cost Management
Content Source: articles/cost-management-billing/manage/upgrade-azure-subscription.md
Service: cost-management-billing
Sub-service: billing
GitHub Login: @bandersmsft
Microsoft Alias: banders

@sali762 Thank you for reaching out to us. Please note that this channel (the Azure Docs repository) is only for driving improvements related to the Azure documentation. It would be great if you can share more details about your ask, so that we can assist you accordingly.

@sali762 We haven't heard back from you. If you have any feedback or need any help, please share more details about the same.


GOTR Tiles

Here are tiles I made for Guardians of the Rift
[{"regionId":14484,"regionX":28,"regionY":15,"z":0,"color":"#BB00FF00"},{"regionId":14484,"regionX":27,"regionY":15,"z":0,"color":"#BB00FF00"},{"regionId":14484,"regionX":27,"regionY":14,"z":0,"color":"#BB00FF00"},{"regionId":14484,"regionX":28,"regionY":14,"z":0,"color":"#BB00FF00"},{"regionId":14484,"regionX":6,"regionY":26,"z":0,"color":"#BB00FF00","label":"Mine"},{"regionId":14484,"regionX":21,"regionY":16,"z":0,"color":"#BB00FF00","label":"Mine"},{"regionId":14484,"regionX":9,"regionY":31,"z":0,"color":"#BB00FF00","label":"Portal"},{"regionId":14484,"regionX":24,"regionY":31,"z":0,"color":"#BB00FF00","label":"Cell"},{"regionId":14484,"regionX":25,"regionY":15,"z":0,"color":"#BB00FF00","label":"Runes"},{"regionId":14484,"regionX":38,"regionY":31,"z":0,"color":"#BB00FF00","label":"Cell"},{"regionId":14484,"regionX":19,"regionY":21,"z":0,"color":"#BB00FF00","label":"SW"},{"regionId":14484,"regionX":43,"regionY":31,"z":0,"color":"#BB00FF00","label":"East"},{"regionId":14484,"regionX":31,"regionY":30,"z":0,"color":"#BB00FF00","label":"Power Up"},{"regionId":14484,"regionX":35,"regionY":16,"z":0,"color":"#BB00FF00"},{"regionId":14484,"regionX":34,"regionY":16,"z":0,"color":"#BB00FF00"},{"regionId":14484,"regionX":50,"regionY":31,"z":0,"color":"#BB00FF00","label":"Climb"},{"regionId":14484,"regionX":52,"regionY":31,"z":0,"color":"#BB00FF00","label":"Climb"},{"regionId":14484,"regionX":31,"regionY":15,"z":0,"color":"#BB00FF00","label":"South"},{"regionId":14484,"regionX":56,"regionY":26,"z":0,"color":"#BB00FF00","label":"Mine 160~"},{"regionId":14484,"regionX":6,"regionY":37,"z":0,"color":"#BB00FF00","label":"Mine"},{"regionId":14484,"regionX":43,"regionY":21,"z":0,"color":"#BB00FF00","label":"SE"}]

For the "Mine 160~" tile, does it scale with mining level? If so it may not be the best idea to keep that label

No it doesn't, I should probable just make it "mine"

Just to add what the tiles look like:
https://gyazo.com/63f7deab47d82cd838ce1dfb6b4900be

I'm thinking about adding the tiles for each Runecrafting altar as well, but I am not sure that would qualify at GOTR tiles since they would show even if not doing GOTR. So, I left them out.


Cached the URL used when the app is launched from the homescreen

Without this URL, I can't launch the app in offline mode

Thanks for the PR and sorry for the delay, I've just made a number of changes to the serviceworker that make this redundant now.... but it was the correct solution ;)


Bug gifts

:tada: This PR is included in version 9.7.7 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Per input/output transaction hash jets

The tx-hash and *-hash jets for Elements transactions generally hash parts of the transaction across all input and or outputs. This lets you do something like CTV and APO. However, for covenants you typically want to hash all the data that corresponds to a specific input or specific output.

input-hash : TWO^32 ⊢ S(TWO^256) Similar to inputs-hash but only hashing the data of one input.
output-hash : TWO^32 ⊢ S(TWO^256) Similar to outputs-hash but only hashing the data of one output.
issuance-hash : TWO^32 ⊢ S(TWO^256) Similar to issuances-hash but only hashing the data of one input.
input-utxo-hash : TWO^32 ⊢ S(TWO^256) Similar to inputs-utxos-hash but only hashing the data of one input.

This PR refactors some of the existing jet specifications changing their CMRs.

@apoelstra This is ready for your review.

LGTM.
Two observations:

When hashing issuance data we also cover the rangeproof/surjectionproof of the issuance even though this is arguably witness data. Unsure if this is the right choice.
When hashing inputs we don't hash the nonce of the spent output. I think this is correct, but in any case we have no choice, because Elements does not keep track of the nonce in the utxoset. (This is an old elements bug.)

Looks like primitiveEnumJet.inc is not updated
++++ diff primitiveEnumJet.inc /nix/store/571z6kp50kw0x4v2x5vsgg3qk4qqp4jl-source/C/primitive/elements/primitiveEnumJet.inc
174a175
> INPUT_HASH,
183a185
> INPUT_UTXO_HASH,
201a204
> ISSUANCE_HASH,
329a333
> OUTPUT_HASH,

Should be fixed now.


Add Coverage

Adds coverage checks to the CI, and a badge to the README.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@ea72d37). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #11   +/-   ##
=======================================
  Coverage        ?   96.34%
=======================================
  Files           ?        6
  Lines           ?      219
  Branches        ?        0
=======================================
  Hits            ?      211
  Misses          ?        8
  Partials        ?        0

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 ea72d37...3a04f27. Read the comment docs.


Jobs issue on reset

On server reset, all jobs with levels >100 are reset back to 100 while not affected any levels gained below this value.
We recently changed the max level for each job from 100 to 200.
https://pastebin.com/qNLLEjnd - generalConfig file.

We've tried that and it fixed the issue.


fix #6863, update scripts/utils/minify.js to fix source maps for ES modules.

Fixes issue #6863
This commit addresses an issue with the missing ".min" part in the source map URLs, which leads to broken links for .jsm files. The problem is that the URLs are not correctly formed, causing the files to point to non-existing locations.
In this code change, we have rectified this problem by ensuring that the source map URLs now include the ".min" part. This adjustment ensures that the links to .jsm files are correctly directed to the appropriate URLs, resolving the problem of broken links.
With this fix, the source map URLs will now be accurately generated, and the .jsm files will successfully link to their corresponding source maps, resolving the previously broken links.

Merged, thank you!


HTML/CCS

#HTML
##User Story
Creating a portfolio of work I can use to showcase my skills and talents to employers looking to fill a part-time or full-time position.
##Acceptance Criteria
When the page is loaded the page presents my name, a recent photo or avatar, and links to sections about me, my work, and how to contact me.
When one of the links in the navigation is clicked, the UI scrolls to the corresponding section.
When viewing the section about my work, the section contains titled images of my applications.
When application's image is larger in size than the others.
When images of the applications are clicked, the user is taken to that deployed application.
Responsive layout when the page is resized or viewed on various screens and devices.

Page presents my name, a recent photo or avatar.
When one of the links in the navigation is clicked, the UI scrolls to the corresponding section.
My applications contains titled images
Application's image is larger in size than the others.
When images of the applications are clicked, the user is taken to that deployed application.
Responsive layout when the page is resized or viewed on various screens and devices.


Documentation issues

The site with the documentation stopped working.
http://docs.nestjs.com

+1

@alex-chuev @kattsushi use this one for now  https://kamilmysliwiec.gitbooks.io/nest/content

Why don't the author put a index.html on docs.nestjs.com and redirect to the working address

@zxsoft because docs are now under heavy development and stay tune :+1:

Now it is clear. So the cats were the cause of the problem with the documentation... :smile:

well, that's it @alex-chuev :smile: enjoy v4!

Documentation page https://docs.nestjs.com/controllers
have the navigation bar cut off in the middle

it is caused by attribute max-height: 350px;

when you disable it it works as expected


feat(olm): add support for openshift 4.16

Targeted for the 1.1.0 release, but I think this could be added at any point. This commit adds FBC support for OpenShift 4.16.
Fixes: https://issues.redhat.com/browse/SECURESIGN-886
Please note: I do not know how to test this, so it has not been tested.

Converting to draft until we determine which image/tag to use

Closing this as the FBC resources have moved


WIP: directly use Metallib instance

dont use unexposed api and directly use MetalLibrary
Metal.framework will auto cache shader IR on disk for you at location:
$(getconf DARWIN_USER_CACHE_DIR)/com.python.python/com.apple.metal
todo:

[x] include benchmark info with/without cache
[ ] investigate why MTLLibrary is being created twice

benchmarks:
Initial run with no cache:

Subsequent run with cache:

Generating the MTLLibrary is the most expensive part of compilation in Metal

As I said in #2226 this resolves the issues exporting metal compiled shader code on my m1 air on macOS 14.1.1.

@nullhook I see this branch has diverged from master so can't run any examples from it, getting ModuleNotFoundError: No module named 'tinygrad.runtime.interpreted'
Was the function signature issue in compile_metal() the only thing preventing this from being merged? I can't run anything on Metal with master right now so trying to revive this branch

yes I think the compile_metal() does not follow the str->bytes signature as @wozeparrot mentioned. I am also not able to run anything on metal for M1 Sonoma 14.1.1
Trying to patch this with master on my local, but no luck in passing the tests yet.

Can someone give me ssh into a machine that doesn't work?

not sure whether this is still being worked on, but for those still facing this issue I had to update this this line: self.lib = compiler.__wrapped__(self.prg) if getenv("DISABLE_COMPILER_CACHE") else compiler(self.prg) in device.py instead of ops.py on my local as the branch has diverged since the original pr. I am on M1 Sonoma 14.2

@tylercosgrove wat did u update?

@tylercosgrove wat did u update?

self.lib = compiler.__wrapped__(self.prg) if getenv("DISABLE_COMPILER_CACHE") else compiler(self.prg)
becomes
self.lib = compiler.__wrapped__(self.prg) if getenv("DISABLE_COMPILER_CACHE") or getenv("METAL") or Device.DEFAULT == "METAL" or getenv("METAL_XCODE") else compiler(self.prg)
This is what the pr is for, but this line change is now in device.py instead of ops.py. The changes in ops_metal.py stay the same. This could still be problematic in regards to the compile_metal signature, but this is just in case anyone had a problem as a result of the branches diverging (when device moved to device.py in #2466 ).

So this can't be merged because it violates the spec, and it's now out of date so I'm going to close. But there's a bounty for fixing this bug in a proper way, if it's still affecting people.


[Bug]: navigateTo params 0 doesn't work.

Is there an existing issue for this?

[X] I have searched the existing issues

Description
navigateTo("Page2", {id: 0}) doesn't add id to the URL. (Any variable with value 0 doesn't get passed)
Workaround: call toString() in the variable
navigateTo("Page2", {id: "0"})
Steps To Reproduce

Call navigateTo("Page2", {id: 0}) on click.
Click the button and notice URL is not appended with id=0.

This is the same when using any variables, navigateTo("Page2", {id: myVariable}).
Public Sample App
No response
Version
Cloud - 1.6.16

Would like to add that I have this issue as well.  This makes the navigateTo function much less useful, as I can't pass Table.selectedRowIndex (or really anything with an index).  I'm going to use storeValue as a less-optimal workaround unless there's a better option available.

Assuming 10% of users who update widget properties with navigateTo() face this issue
Stats

Stat
Values

Reach
250

Effort (months)
0.5


New ingest

New ingestion code and ingest runner. Cleaner code, and gets the alerts even when delayed past midnight.

Tested that SIGTERM stops ingest+runner and ingest cleanly with no processes left over
Tested that removing lockfile pauses processing, the runners just sleep until lockfile is back
Docopt for arguments to ingest_runner and ingest

Looks OK.
Does this fix the multiple alert ingest issue?

Yes. By default it has the regex to take all alerts. And also arguments to get a specific topic or nid if thats what you want.

Sorry, I was asking if this fixes #56 or not.

It should fix 56. Certainly we are no longer starting and stopping the kafka consumer every 2 minutes. I want to leave it run over the weekend without touching anything, then hope that the various numbers that should be the same actually are the same.

OK. In that case we should probably hold off on merging the PR until Monday.

The counting is accurate


Added .NET Framework 4.6 as first-class citizen

Fixes #3817 - added .NET Framework 4.6 (net46) Target Framework to switch from CallContext to AsyncLocal for contexts
Note that locally there are a few unit tests failing locally. I didn't think to run them before making my changes, so I'll try to circle back around to see if they were already broken locally, but this is the majority of the work either way.
For 4.5, 4.6, and .NET Standard 2.0 the following tests are broken locally:
NLog.UnitTests.LoggerTests.MixedStructuredEventsConfigTest(parseMessageTemplates: null, param1: "0", param2: "@Client")
NLog.UnitTests.LoggerTests.MixedStructuredEventsConfigTest(parseMessageTemplates: null, param1: "1", param2: "@Client")

For 4.5 and 4.6 the following test is also broken locally:
NLog.UnitTests.Internal.AppDomainPartialTrustTests.MediumTrustWithExternalClass

Okay, I checked the tests and the same tests are failing locally before my changes, so I think this commit is passing all checks that were passing before.

Think net46 should be added to the test-collection:
https://github.com/NLog/NLog/blob/dev/run-tests.ps1
But as repeated before, then it would be nice to remove other builds before adding net46. See also #2871

Also think the failing builds are caused by the new build-pipeline for VS2019.

Yes removed that for now.
@TheXenocide are there now with the PR features for .NET 4.6 that aren't in  the .NET 4.5 build

If the features are the same, we could skip the .NET 4.6 tests for now.

Features should be the same, if NET4_6 defines have been applied properly. That is what tests confirms.
The important change between NET4_5 and NET4_6 is the use of AsyncLocal for NDLC and MDLC. Meaning NET4_6 works just like NET4_5 and NDLC + MDLC will no longer travel between AppDomains (good or bad).

Meaning NET4_6 works just like NET4_5 and NDLC + MDLC will no longer travel between AppDomains (good or bad).

Is this a breaking behavior change?

I'm guessing it is breaking behavior change. But remoting is a thing of the past. So I'm for the change going forward. But I'm also for removing NET4_0 :)

Yes ok, but this is for NLog 5 then

That's correct, it doesn't change the NLog-specific feature profile, but it does remove cross-AppDomain support from NDLC/MDLC by switching to AsyncLocal. It's worth noting that the precompiler symbols and conditions were already setup for that particular feature swap, I didn't change that when I turned the target on, though I did have to add NET4_6 to other conditions to keep other features. It may be worth considering if those other features are needed anymore or not, but I didn't go to that extent since this feature very specifically had conditions around it, whereas other features didn't seem to indicate one way or the other. As is, this pull request assumes all 4.5 features stay, but not all of them exist in .NET Standard 2.0.
As for the concept of the feature difference more generally, unfortunately CallContext was not designed for ambient context, it's just what most of us have been using it for; it was designed specifically for Remoting and using it in a new world of async/await and Task can result in scenarios where ExecutionContext flow is polluted with CallContext data in a way that cannot be easily reclaimed. In my case this caused issues with MSTest v2 because the isolated parent domain of the test adapter could not load NLog types, but some portion of the CallContext required it to. In my code I am disposing all contexts I create and even tried clearing the MDLC and NDLC contexts during AssemblyCleanup, but unfortunately nothing I could get at prevented MSTest from trying to load NLog in the parent domain. Each test assembly runs in its own AppDomain and it explicitly intends to isolate itself from assemblies required by the test project (which is one of the reasons it's using AppDomains in the first place).
In honesty they could probably better isolate tests by controlling ExecutionContext flow in a way that prevents this type of pollution, but I think they probably also don't expect people to leave uncleaned logical thread affinitives around. We've had problems in MSTest in the past with our own uses of CallContext causing AppDomainDisposedException as well (even though all of our uses were applied via using (SomeDisposableContext) { ... }. It seems like not all TestMethods run in the same context and sometimes new empty contexts are introduced as well. Switching our stuff to AsyncLocal allowed us to keep all of our ambient context functionality without much compromise and we basically never need our contexts to flow across AppDomain boundaries (even though we actually do have an application that creates multiple domains). As an interesting side note, when using NLog in said multi-domain application we already had to go out of our way to configure it differently since each domain had different instances of the file targets (we now use the AppDomain Id in the filename).

Created  #4223 where Travis-Build also works

Created  #4223 where Travis-Build also works


[State Sync] Add auto-termination of blocked streams, garbage collection and unit tests.

Motivation
This PR extends the data streaming service with: (i) auto-termination when data streams become blocked and are unable to make progress (i.e., data refetches continuously fail); (ii) garbage collection for long lived streams (i.e., reducing internal notification storage); and (iii) additional unit tests to verify termination and garbage collection functionality.
To achieve this, the PR offers the following commits:

Add auto terminate functionality for blocked streams. These are streams that are unable to make progress after a specified number of data refetches.
Add unit tests for blocked streams and auto-termination.
Add garbage collection support for long lived streams and a unit test to verify this behaviour.
Add new unit tests to verify stream termination and creation.

Have you read the Contributing Guidelines on pull requests?
Yes.
Test Plan
New unit tests have been added.
Related PRs
None, but this PR relates to: https://github.com/diem/diem/issues/8906

This PR has been migrated to https://github.com/diem/diem/pull/9614 after merge conflicts.


docker: Upgrade engine to latest stable release

This will hopefully fix a problem with docker exec hanging in the
Kubernetes rewrite.

Yep, it passes the network and mean integration tests. Don't know if it'll fix the exec bug though since it's a bit hard to repro.


eval config options

Codecov Report

Merging #320 into experimental will increase coverage by 0.05%.
The diff coverage is 100%.

@@               Coverage Diff                @@
##           experimental     #320      +/-   ##
================================================
+ Coverage         83.37%   83.43%   +0.05%
================================================
  Files                24       24
  Lines              1131     1135       +4
================================================
+ Hits                943      947       +4
  Misses              188      188

Impacted Files
Coverage Δ

vent/api/actions.py
98.85% <100%> (+0.05%)
: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 afcc9c2...dfe5375. Read the comment docs.


xcarchive and xcexport actions don't seem to sign ipa correctly

Tried moving over to xcarchive and xcexport to create builds after ipa started triggering watchkit warnings on upload.  The builds I generated did not seem to be signed correctly with the deployment identity created using actions produce, cert, sigh, update_project_code_signing, xcarchive, xcexport.  Got the following error from itunes connect.
ERROR ITMS-90035: "Invalid Signature. A sealed resource is missing or invalid. The binary at path [Framework.app/Framework] contains an invalid signature. Make sure you have signed your application with a distribution certificate, not an ad hoc certificate or a development certificate. Verify that the code signing settings in Xcode are correct at the target level (which override any values at the project level). Additionally, make sure the bundle you are uploading was built using a Release target in Xcode, not a Simulator target. If you are certain your code signing settings are correct, choose "Clean All" in Xcode, delete the "build" directory in the Finder, and rebuild your release target. For more information, please consult https://developer.apple.com/library/ios/documentation/Security/Conceptual/CodeSigningGuide/Introduction/Introduction.html"
Never managed to get this working despite setting the correct signing identiy in xcode and running again.  Ended up downgrading fastlane and shenzhen and now it's working again.

+1 to this.  I'm using xcarchive and xcexport to create ipa and send to iTC with deliver.
But in my case I haven't got any errors - everything seems green but no one can install an app from iTC TestFlight.
Seems like signing issue to me.

+1

Looks like this is caused because xcodebuild exporting from the CLI doesn't archive in the archived-expanded-entitlements.xcent file that Xcode itself does.
I'm looking at solutions but a build action running this seems to do a better job of not touching the package.
/usr/bin/xcrun -sdk iphoneos PackageApplication -v /Path/To/Archive/name.xcarchive/Products/Applications/name.app -o /Path/To/Archive/DiplomatStaples.ipa

nice catch @robertjpayne .
I've found this interesting too: http://stackoverflow.com/questions/23682083/xcodebuild-not-copying-file-from-app
The winning answer cleared up my mind a little bit.

As you know I've been working on gym which uses the latest API to build and sign your app. Please give it a try by passing a valid provisioning profile and let me know if that works for you.

In case someone is interested I'm currently implementing the xcrun export command on my gym fork.
It's currently under test with my projects but in case you want to give it a spin: https://github.com/fabiomassimo/gym

I'm gonna go ahead and close this since now there's the awesome gym!


when GFToggle in Expanded(Row),whatever value(true/false)，display is  disable

Describe the bug
when GFToggle in Expanded(Row),whatever value(true/false)，display is  disable
To Reproduce
Row(
                children: [
                  const Text('Show'),
                  SizedBox(
                    width: UIState.padding,
                  ),
                  GFToggle(
                    value: true,
                    onChanged: (bool? checked) {
                      if (checked != null) {
                        // to do
                      }
                    },
                  )
                ],
)

Expected behavior
none
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: mac os

Correction: The cause of this issue should be an issue with the animation and state logic of GFToggle, rather than a layout issue.
Reproduction steps: Create a Statefull Widget, then create a Boolean field in the state class with an initial value set to false. Then, in a network request or a Future then, set the State field to true. You will find that GFToggle is still in the off state

Unfortunately this library is no longer active, will be looking for other solutions


[BUG]: Ubuntu, CMake, .so file is built but no python library.

Problem description
This is my CMakeLists.txt that downloads Nanobind and build the main.cpp example file with the documentation sourcecode. Why does cmake output NanoTest.cpython-311-x86_64-linux-gnu.so but not the .pyd file?

This is my sourcecode:
cmake_minimum_required(VERSION 3.5)
project(TestProject)
include(ExternalProject)
option(DOWNLOAD_DEPENDECIES ON)
option(BUILD OFF)

set(CMAKE_CXX_STANDARD 14)

if(DOWNLOAD_DEPENDECIES)
    ExternalProject_Add(Nanobind
    GIT_REPOSITORY https://github.com/wjakob/nanobind.git
    GIT_TAG master
    PREFIX ${CMAKE_BINARY_DIR}/nanobind
    CONFIGURE_COMMAND ""
    BUILD_COMMAND ""
    INSTALL_COMMAND ""
    LOG_DOWNLOAD ON
    )
endif()

if(BUILD)

    find_package(Python 3.9 COMPONENTS Interpreter Development.Module REQUIRED)

    if (NOT CMAKE_BUILD_TYPE AND NOT CMAKE_CONFIGURATION_TYPES)
        set(CMAKE_BUILD_TYPE Release CACHE STRING "Choose the type of build." FORCE)
        set_property(CACHE CMAKE_BUILD_TYPE PROPERTY STRINGS "Debug" "Release" "MinSizeRel" "RelWithDebInfo")
    endif()

    include(${CMAKE_BINARY_DIR}/nanobind/src/Nanobind/cmake/nanobind-config.cmake)
    nanobind_add_module(NanoTest libShapeOp/examples/main.cpp)

endif()

I run it like this:
#!/bin/bash

mkdir -p build
cd build
cmake -B . -S .. -DDOWNLOAD_DEPENDECIES=ON -DBUILD=OFF -G "Unix Makefiles" && make
cmake -B . -S .. -DDOWNLOAD_DEPENDECIES=OFF -DBUILD=ON -G "Unix Makefiles" && make

Reproducible example code
No response

".so"(on linux) and ".pyd"(on windows) files are basically the same. Have you tried import NanoTest?

@a603032070  Yes I tried, no module found.
I changed binding name to my_ext to be inline with the example. I checked on my system I have python 3.11.
I also tried to give a path directly to the .so file:
import sys
sys.path.append('/home/petras/brg/2_code/ShapeOp/build/')
import my_ext

Still the same error.
I removed any 3rd party libraries and just left nano, could you check my repository:
https://github.com/petrasvestartas/ShapeOp

@a603032070 Yes I tried, no module found. I changed binding name to my_ext to be inline with the example. I checked on my system I have python 3.11.
I also tried to give a path directly to the .so file:
import sys
sys.path.append('/home/petras/brg/2_code/ShapeOp/build/')
import my_ext

Still the same error.
I removed any 3rd party libraries and just left nano, could you check my repository: https://github.com/petrasvestartas/ShapeOp

I've checked your repo. It works well. Maybe something wrong about the path?

@a603032070
The issue is in python version. In my base environment with python 3.11 it works.
But with my conda environment where python is 3.9 it does not.
How you can specify in CMakelists which python version the library is built for?

I don't really know. Maybe it's about this line in Cmakelists.
find_package(Python 3.9 COMPONENTS Interpreter Development.Module REQUIRED)
https://cmake.org/cmake/help/latest/module/FindPython.html

ExternalProject_Add is not a supported way of dealing with the nanobind dependency, I don't use it and cannot provide support for it.
I also suspect you are working partly on windows and on Linux (WSL?), hence the confusion about .pyd and .so files which are for different operating systems.
Anyways, these are all not bugs bug issues with your setup. Please don't open tickets here unless you are certain to have found a bug in nanobind.