Not following redirects

I have some image URLs that respond with 302 or 301 like this one:
http://imageproxy.niiknow.org/rx/http://images.ingaiasites.com.br/dmtBYQdKfjyypH9Ho5VU1mbsJDoxJt5Y.jpg
It would be great if image-proxy could follow redirects.

Very common for images hosted in S3 to have redirects.

Actually, this was a consideration in the development of server.conf.  I purposely disable redirect for security reason:  https://github.com/niiknow/docker-nginx-image-proxy/blob/master/files/etc/nginx/sites-enabled/server.conf#L48
Additionally, image_filter + intercept_errors (you get that empty pixel - since anything above 300 is consider error) + follow redirect is kind of tricky to do https://github.com/niiknow/docker-nginx-image-proxy/blob/master/files/etc/nginx/sites-enabled/server.conf#L66 http://nginx.org/en/docs/http/ngx_http_proxy_module.html#proxy_intercept_errors
Though it's not impossible.  You can setup another server/listener on port, let say 81, and have that work like a CDN proxy redirect.   Let's called it backend that will perform the redirect and caching of source object.  Then just proxy_pass to http://backend/$clean_uri$is_args$args

Ok, I see. I gave it a try for a couple of hours to implement what you suggested but I'm no expert in nginx configuration, so no success on my end. Could you provide me a more clear example of the correct configuration for doing this? Couldn't we just set a @handle_redirect route for 301 and 302 and do a simple rewrite of the url?

I've just ported my image server from thumbor to this nginx module and I'm loving it. Much faster and more stable. But, thumbor used to follow the redirects and I really don't want to go back to it just because of this issue...

Couldn't we just set a @handle_redirect route for 301 and 302 and do a simple rewrite of the url?

Unfortunately, no.  The image_filter module will give you a 415 error because it did not get an actual image data from remote server.  This is complexity when using with lua openresty to handle nginx body data.
Unfortunately, I won't be able to setup an example until later tonight as I'm currently at work.

I see. No problem, I'll wait for your example then. Thanks!

I experienced the same.  I model the service after this module https://github.com/willnorris/imageproxy for easy conversion on my side.  I run about 5-8 servers haproxied in a rancher dockers environment.  It was great at first but it couldn't handle the load and get pretty unstable to the point of often crashing with out of memory (OOM).    Now, my docker/cattles has been running stable for a while/without breaking a sweat.
I do more complicated transformation with the cheap $10 legacy wp Cloudinary service plan.

Did you use in-memory caching with imageproxy? Did you figure out what caused the memory leaks?

Nope, I actually use the s3 caching method - https://github.com/willnorris/imageproxy/issues/58
I have no idea why it start to perform badly.  I'm not a Go programmer so I wasn't about to troubleshoot.   My requirement is simple.  I just need simple resize and I needed something really light weight and perform wells.   It must also be backward compatible with imageproxy.

Here you go: http://imageproxy.niiknow.org/rx/200/http://images.ingaiasites.com.br/dmtBYQdKfjyypH9Ho5VU1mbsJDoxJt5Y.jpg?cb=123
It's not pretty.  This is your gist:  https://gist.github.com/noogen/d28c560bb236bae6765c6440c66765dc
Launch the docker with SERVER_CONF=https://gist.githubusercontent.com/noogen/d28c560bb236bae6765c6440c66765dc/raw/943275e7fa66187ba952f2d6ff6b3af1f32471ec/image-proxy-test.conf
I recommend you make your own gist or store it somewhere secure URL.  Explain:
The conf is base off the current server.conf and my CDN conf here: https://github.com/niiknow/docker-nginx-image-proxy/blob/master/files/etc/nginx/cdn.conf
Since the cdn conf (port 81) is already doing the cache.  There is no need to cache on port 80, so I remove some/a couple of the caching directives.  Additionally, I was running into some 403 issue from remote servers because I didn't pass in User-Agent so that's why you have this line: https://gist.github.com/noogen/d28c560bb236bae6765c6440c66765dc#file-image-proxy-test-conf-L20

Nevermind, after taking some time off to think about my response above, the issue was the "User-Agent" after all.  So I added: https://github.com/niiknow/docker-nginx-image-proxy/blob/master/files/etc/nginx/sites-enabled/server.conf#L49
I tried the @handle_redirect first before doing the port 81 route but that didn't work.  Now the @handle_redirect is working correctly.

It's almost perfect. I identified an issue where it works the first time when there's cache miss, but fails when there's a second request (cache hit), with these headers:
X-ImageProxy-Code:500
X-ImageProxy-Debug:http://?w=400&h=300&q=96&r=0&e=0&cmd=resize&ofmt=

Notice the URL is actually missing.

So, since I also have a CDN in front of my server, after removing all the proxy_cache_* directives from your latest server.conf, it's working perfectly!!
You're the man!!

I can close this issue now, but let me know if you want to try to fix this issue and still keep the proxy_cache directives.
My final working conf is here: https://gist.github.com/demetriusnunes/796f2f14bffaf1725366d07e6f4be2e7

I'll have to look into what is causing that 500 as I do not see it in my testing.  It might be a different issue because it can't be the cache.  I actually do not use redirect and like to keep the proxy_cache directive.  It's only a 10m cache.

I just did this:
➜  ~ curl -I http://imageproxy.niiknow.org/rx/http://images.ingaiasites.com.br/dmtBYQdKfjyypH9Ho5VU1mbsJDoxJt5Y.jpg
HTTP/1.1 200 OK
Server: nginx
Date: Wed, 16 Aug 2017 22:15:34 GMT
Content-Type: image/jpeg
Content-Length: 84836
Connection: keep-alive
Keep-Alive: timeout=60
Cache-Control: max-age=86400
Accept-Ranges: bytes
Expires: Thu, 17 Aug 2017 22:15:34 GMT
X-ImageProxy-Cache: MISS
X-ImageProxy-Debug: http://ingaiasites-images.s3.amazonaws.com/dmtBYQdKfjyypH9Ho5VU1mbsJDoxJt5Y.jpg?w=-&h=-&q=96&r=0&e=0&cmd=resize&ofmt=
Cache-Control: public

➜  ~ curl -I http://imageproxy.niiknow.org/rx/http://images.ingaiasites.com.br/dmtBYQdKfjyypH9Ho5VU1mbsJDoxJt5Y.jpg
HTTP/1.1 200 OK
Server: nginx
Date: Wed, 16 Aug 2017 22:15:40 GMT
Content-Type: image/gif
Content-Length: 43
Connection: keep-alive
Keep-Alive: timeout=60
Last-Modified: Mon, 28 Sep 1970 06:00:00 GMT
X-ImageProxy-Code: 500
X-ImageProxy-Debug: http://?w=-&h=-&q=96&r=0&e=0&cmd=resize&ofmt=

You didn't pass in a resize in your rx.

Sorry, same result:
➜  ~ curl -I http://imageproxy.niiknow.org/rx/350x300/http://images.ingaiasites.com.br/dmtBYQdKfjyypH9Ho5VU1mbsJDoxJt5Y.jpg
HTTP/1.1 200 OK
Server: nginx
Date: Wed, 16 Aug 2017 22:17:35 GMT
Content-Type: image/jpeg
Content-Length: 32910
Connection: keep-alive
Keep-Alive: timeout=60
Cache-Control: max-age=86400
Accept-Ranges: bytes
Expires: Thu, 17 Aug 2017 22:17:34 GMT
X-ImageProxy-Cache: MISS
X-ImageProxy-Debug: http://ingaiasites-images.s3.amazonaws.com/dmtBYQdKfjyypH9Ho5VU1mbsJDoxJt5Y.jpg?w=350&h=300&q=96&r=0&e=0&cmd=resize&ofmt=
Cache-Control: public

➜  ~ curl -I http://imageproxy.niiknow.org/rx/350x300/http://images.ingaiasites.com.br/dmtBYQdKfjyypH9Ho5VU1mbsJDoxJt5Y.jpg
HTTP/1.1 200 OK
Server: nginx
Date: Wed, 16 Aug 2017 22:17:37 GMT
Content-Type: image/gif
Content-Length: 43
Connection: keep-alive
Keep-Alive: timeout=60
Last-Modified: Mon, 28 Sep 1970 06:00:00 GMT
X-ImageProxy-Code: 500
X-ImageProxy-Debug: http://?w=350&h=300&q=96&r=0&e=0&cmd=resize&ofmt=

ic, your curl bypass the cache and so it's using server side cache.  I couldn't reproduce because I've been testing on browser.
The problem was because it was serving cache for the old URL and not the redirected URL.   This is fixed.

Wow, awesome! Thanks a lot! 👍 🎆 😀

Not sure if this is still an issue.  I updated the attach server.conf with the suggested change above.  I'm going to close this.


Improved Location Field

Issue
Closes #60
What is changed / added

User is able to drop a pin on the map and have location field filled.
User is able to click on location field and change location if he so desires.

Screenshots

Firestore Data

The code looks good, but I realized we had a previous code where we hard-coded the collection to put the events in firestore.
https://github.com/GDSCUTM-CommunityProjects/ActNow/blob/5fc27a2cb12638bf7bbf936edd048d374cb0b2bb/src/app/lib/pages/map_page/add_event.dart#L66
Let's make this dynamic

made the change


Add web search to PowerToys Run

Having PowerToys Run search the web if it cannot find a file would be a brilliant way to quickly search the web.
It would also be good if:

The web search functionality could be toggled in the Settings app
You could change the search engine to anything you wanted, say (sorry Microsoft) using Google instead of Bing....

I came here to say this! However, I think specifying a web search would be better, rather than having just a toggle in the settings. Say if I wanted to search, I would start the command with "web:" followed by the search string. This would prevent unnecessary web searches, and having to figure out if the user actually is trying to search the web.

I completely agree, however would also suggest shortening the “prefix” if
you like, to something along the lines of “w https://www.google.com”, the W
being the prefix. Because I think by the time you’ve typed “web:” it’s
probably quicker using the Windows Start Menu search :)
On Sat, 30 May 2020 at 21:17, Rob Taylor<EMAIL_ADDRESS>wrote:

I came here to say this! However, I think specifying a web search would be
better, rather than having just a toggle in the settings. Say if I wanted
to search, I would start the command with "web:" followed by the search
string. This would prevent unnecessary web searches, and having to figure
out if the user actually is trying to search the web.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/3900#issuecomment-636379753,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APBILINHQM7FUW6FD3XCTZLRUFSWFANCNFSM4NOWBSEQ
.

--
This message was composed on a mobile device, please excuse my brevity.
Where necessary, I may also send a follow up email following my return to a
location where I have access to a Desktop or Laptop machine confirming
further required details. Thank you.

Duplicate of https://github.com/microsoft/PowerToys/issues/3245


Variable Names in "Update Parameters" Process Refer to DLZ

Describe the bug
In the "Update Parameters" process, many of the script input parameters and variables have names that refer to a Data Landing Zone when they actually control the Data Management Zone that is being created.

This is the same for all repos. This script does not really get exposed to users and they are not really using this scripts elsewhere. For maintennace resons, I would like to use keep the same script across all repos. Changing this can make things more confusing and therefore can make things more error prone.


use which to search nu location, add icon too.

I tried out the vscode extension today and it gave error. so I looked at the code and found out the nu installation detection logic is manually searching through a hard coded list of directores, but my setup happens to be not in any of the presumed locations. so I modified the code to use which to search in ${env:PATH}, plus these additional locations, and also simplfied the code a little bit during the process.
also, I don't like the generic "terminal" icon, and I find there's already a svg icon file in the assets folder so I just point the terminal profile icon path to that svg file.

I think this is an excellent idea. Thanks for adding this. I just had a few notes above. Appreciate your help!

ok. I think we're good here if we can just get the ci green. thanks for your help and explanations! I wrote this mess, with some help, but I'm no good at javascript/typescript stuff. :)

oops, look like the linter doesn't like an empty function body. replaced with an empty value. should fix it now.

Thanks!


Improved security and refactor decodeMultipart. Closes #23

This removes a spurious BufferedReader and should also prevent memory exhaustion DoS by malformed multipart/form-data POST.

Coverage decreased (-0.22%) to 74.56% when pulling 3cbc1769b317f45ad257f33bd648ac97b595f84b on better-decode-multipart into a3905156708ffb95723a6c89d86088194f7d48b1 on master.

@ritchieGitHub , could you briefly review this commit? This should slightly simplify and secure multipart decoding, but it's still a pretty complicated part of NanoHTTPD.

Coverage decreased (-0.27%) to 74.51% when pulling f1409a7efe92b6ad022d4b6accc01e7b4fa4a95b on better-decode-multipart into a3905156708ffb95723a6c89d86088194f7d48b1 on master.

Coverage decreased (-0.27%) to 74.51% when pulling 5e6d76fe9c6bf67e7cdeb6e310952af639614030 on better-decode-multipart into a3905156708ffb95723a6c89d86088194f7d48b1 on master.

Coverage decreased (-0.32%) to 74.46% when pulling 731ecb0ecb91cc31d3f12a657c4a34b40fa18f3f on better-decode-multipart into a3905156708ffb95723a6c89d86088194f7d48b1 on master.

It looks good but I think the code looks much better/compacter using regular expressions, just commited it, you can throw my commit out if you like. For this kind of "String" parsing is where regex has it's power.

Yes, much better. In fact, the whole codebase could use a little more  regexp and a little less StringTokenizer since it's not Java 1.1 anymore.

As a little extra tweak, how about moving the static pattern variables a little closer to the function (i.e. inside HTTPSession), since they are only used in decodeMultipartFormData?

As a side note: I somewhat dislike the fact that every request is currently written on disk and memory mapped. I'd guess the overwhelming majority of HTTPD requests are smaller that the buffer used to write them in a file (512 bytes), so it would be nice to skip the disk in those cases. Unfortunately, there doesn't seem to be any implementations of ByteBuffer besides MappedByteBuffer.

Coverage decreased (-0.55%) to 74.23% when pulling 6625f70d53d26ca7b0c039479ed6120d72f2bcb5 on better-decode-multipart into a3905156708ffb95723a6c89d86088194f7d48b1 on master.

Coverage decreased (-0.54%) to 74.25% when pulling 770aaf0c6567d8b62a6eef18c8f0359208427c77 on better-decode-multipart into a3905156708ffb95723a6c89d86088194f7d48b1 on master.

Java restricts statics inside inner classes, there only "real" constants are legal (pattern is not allowed) That's why it's in the parent ....

Oh, I see. Any more thoughts on this branch or shall we merge it?

I would say merge it!


Add (missing) parameter 'drawLineToCenter' to 'drawSolidCircle(...)'

Describe your changes
Add parameter drawLineToCenter to drawSolidCircle:

Issue ticket number and link
Checklist before requesting a review
For each PR

[ ] Add Copyright if it missed:
- "Copyright (c) 2019-present Axmol Engine contributors (see AUTHORS.md)."

[x] I have performed a self-review of my code.
Optional:

[ ] I have checked readme and add important infos to this PR.
[ ] I have added/adapted some tests too.

For core/new feature PR

[ ] I have checked readme and add important infos to this PR.
[ ] I have added thorough tests.

Add drawSolidTriangle(...)

@halx99
Please merge it.


[FEATURE]: Pre & Post Save Signals

Hello Drizzle maintainers,
Firstly, I'd like to take a moment to appreciate all the effort that goes into maintaining and improving Drizzle. It's an incredibly valuable resource and it has made development a lot easier for me.
Summary
I would like to suggest a feature for the library: Pre & Post Save Signals, similar to the signals mechanism found in Flask or Django, or the middleware concept in Prisma.
Problem
In the current version of Drizzle, there is no in-built mechanism for triggering actions before or after a database query. Although we can manually code this logic around the Drizzle function calls in our application, it introduces additional complexity, scattered code and it’s also more error-prone. Furthermore, some tasks naturally belong to the ORM layer, such as data validation, logging, or automated reactions to data changes.
I do acknowledge that Drizzle already provides a comprehensive logging feature, which is highly versatile and configurable, allowing us to write logs anywhere desired - disk, API, etc. However, the feature I am proposing is conceptually different. While logging captures and records the state of the system at a given point of time, pre and post save signals would enable automated, direct response to these state changes, beyond just recording them.
Suggested solution
By introducing Pre & Post Save Signals, users of Drizzle would have a way to hook into the lifecycle of a model instance, allowing them to define tasks that should be performed before or after saving a record.
For example, a common use case could be sending a welcome email to a new user after a new row is created in the User table. With the suggested mechanism, one could define a 'post save' signal on the User model that automatically sends an email when a new instance of User is saved to the database.
Benefit to the Drizzle users
Such a feature would enable developers to manage side-effects more effectively by linking them directly to model lifecycle events. This would improve readability, maintainability, and reduce the likelihood of errors as changes in state can be handled at the moment they occur. It provides a clean and intuitive solution to manage a range of common tasks such as sending notifications, logging, data validation, etc.
I hope you'll consider this feature request for inclusion in a future version of Drizzle. I believe it could add a great deal of flexibility and improve the developer experience significantly.
References

Flask Signals
Django Signals
Prisma Middleware

We need this feature too. We are migrating from TypeORM so something like the TypeORM listeners/subscribers would be ideal:
https://orkhan.gitbook.io/typeorm/docs/listeners-and-subscribers

Related https://github.com/drizzle-team/drizzle-orm/issues/956 https://github.com/drizzle-team/drizzle-orm/issues/843

Would love to have this. Prisma's middleware applies to all requests. Not very good separation of concern and unnecessarily inefficient. It doesn't have to be fully implemented as signals in Django. Simple pre and post hooks that are definable with the model would work too.

We need this feature too. We are migrating from TypeORM so something like the TypeORM listeners/subscribers would be ideal:
https://orkhan.gitbook.io/typeorm/docs/listeners-and-subscribers

@ivanjd I'm curious what are your reasons for abandoning TypeORM?
@

The functionality that developers to extend their drizzle client like prisma client extension.
It would improve the developer experience way better.
Scenarios that I have met:

Hashing the password column before saving it into a database.
adding a new property to query results at runtime and type-level.
e.g. to use casl ability it requires to __caslSubjectType__ property.
Implement soft delete

From what I understand the team is looking into implementing middleware https://github.com/drizzle-team/drizzle-orm/discussions/1513#discussioncomment-7585593

We currently wrap drizzle in a base repository https://gist.github.com/cayter/49d5c256a885d90c399ca6c1eca19f51 which it will trigger the before and after hooks. Note that we only made it working with postgres.

Will it be supported officially in the future? This is probably the only reason why I can't use Drizzle ORM


Better composition for fragments

This PR aim to improve developer UX when composing fragments.
The sql interpolator is modified to remove optimize more the composition of Void codecs.

Void ~ Void -> Void
A    ~ Void -> A
Void ~ B    -> B
A    ~ B    -> A ~ B

@vbergeron Mind fixing the merge conflicts on this one? Also, we should think about how to handle source compatibility here. Now that we have skunk.feature.legacyCommandSyntax, perhaps we should leave Void (and Unit?) values in place when legacy syntax is enabled.


It looks quite strange in Django admin.. How can I make it follow admin system styling?

August 6th, any updates?
Have same problem. It doesn't recognize bootstrap styles.

@ahmedetefy Can you add screenshots to what you're exactly referring to?

@ahmedetefy @karen-oneplanetops @niccolomineo @ivlevdenis
Can you try using my fork and see if it works for you?
https://github.com/abogushov/django-admin-json-editor/pull/35

@ahmedetefy @karen-oneplanetops @niccolomineo @ivlevdenis
Can you try using my fork and see if it works for you?
https://github.com/abogushov/django-admin-json-editor/pull/35

@ahmedetefy @karen-oneplanetops @niccolomineo @ivlevdenis
Can you try using my fork and see if it works for you?
#35

Looks fine to me


release artifacts for release v0.0.7

Brings in ACK runtime v0.14.1
Signed-off-by: Jay Pipes<EMAIL_ADDRESS>By submitting this pull request, I confirm that my contribution is made under
the terms of the Apache 2.0 license.

/lgtm


Check that load and critical load data have same time step

Describe the bug
SAM UI checks added in https://github.com/NREL/SAM/pull/1390 display UI error messages when critical load and load data have different time steps, but SSC does not prevent simulation from running under this condition.
To Reproduce

Using a build of SAM from the Patch branch that includes https://github.com/NREL/SAM/pull/1390, open the SAM file and load the 15-minute weather file from here.
Go to the Grid Outage and Electric Load pages and see the error messages.
Run a simulation and note no simulation message.

On Data Tables tab of Results page, note that "Critical load in this timestep" and "Lifetime electricity load" are of correct length: Hourly critical load data is converted to 15-minute data.
Critical load data 60-minute input:

Resulting output, with 60-minute critical load data converted to 15-minute time steps:

Expected behavior
I would expect a simulation message if time steps do not match, but perhaps the automatic conversion is the behavior we intended?

@cpaulgilman I created an SSC branch that should fix this (ssc_1015_errors_for_load_mismatch), however when using the test file above the errors are not triggered. The issue seems to be that while load_user_data has 15 minute data, load is still being passed to SSC as an 8760:

Do you have time to debug the callbacks and equations to see why that might be happening? I'll create a pull request once we get the expected behavior with the test file.

Yes, I'll look into the SAM UI side of things.

Callbacks should be fixed in https://github.com/NREL/SAM/tree/load_ui_checks.


[Request] Support GitHub Dark

GitHub Dark style theme makes all buttons of GitHub becoming not flat. Hope Github-Mod can flatten them.
For screenshot, you can see a sample in #15 , here is another one:

After looking into it I think it would be difficult to try to override another userstyle and maintain it. I would recommend modifying GitHub Dark instead if you want to prevent the buttons from being changed.


Add keyboard disabler

This adds a new feature to the TestButler API that allows users to disable the software keyboard during tests. Often when running UI tests, entering text can cause the keyboard to rise up. This is a major source of flakiness, so TestButler allows you to disable it.
Also made some minor changes to the spell check disabler.

Fixes #41

This is awesome!


regarding security

hi,
I am fascinated by your framework and support. as an organisation, you all support cake PHP.  I am a beginner to the backend and I see using the plugins ease coding stress.
My question is how secure is plugins? or auto-generated codes. if I use auto-generate code or plugin will my e-commerce secure?
this is very important for me. please answer. for example as an authentication plugin given by the organisation I believe it has to be secure. whereas other PHP frameworks mentioning authentication developed by this person rather than the organisation.
will wait for reply

This is an awesome list, not a forum/chat.

@Jeewakak try
https://discourse.cakephp.org/
https://www.reddit.com/r/cakephp/


Add ll alias back to 00-wlinux.sh

Somehow this got lost amongst many commits and much development. Now here it is back :woman_shrugging:

Removed bashrc for now as it wasn't being sourced and the only used code was placed in profile.d/00-wlinux.sh anyway. Think it's probably safer to include any future changes in 00-wlinux.sh from now


'MultiOutlet' Device not working

I have a smart power strip that is having a communication error of some sort. It appears that the device is not discoverable. I have configured it with and without "ip". Here are the details.
Device:
CE Smart Home - Smart Home Power Strip
6 Controls in Ucomen Home app (Main Power Button + 5 Controlled Outlets)
IP Address is statically assigned at router
(ping:  address is reachable only when device is plugged in)
Environment:
HOOBS Version 3.1.20
Homebridge Version 0.4.50
Node Version 10.18.0
Tuya Lan Version 1.4.0 (published 9/15/2019)
Configuration:
{
"platform": "TuyaLan",
"devices": [
{
"name": "Power Strip",
"type": "MultiOutlet",
"manufacturer": "CE Smart Home",
"model": "Smart Wifi Power Strip",
"id": "XXXXXXXXXXXXXXXXXXX",
"key": "XXXXXXXXXXXXXXX",
"outletCount": 5,
"ip": "192.168.XXX.XXX"
}
]
}
Homebridge Log...
[Homebridge] [1/19/2020, 4:36:44 PM] [TuyaLan] Starting discovery...
[Homebridge] [TuyaAccessory] Discovery started.
[Homebridge] [1/19/2020, 4:37:44 PM] [TuyaLan] Failed to discover Power Strip (XXXXXXXXXXXXXXXXXXX) in time but will connect via 192.168.XXX.XXX.
[Homebridge] [TuyaAccessory] Socket had a problem and will reconnect to Power Strip (EHOSTUNREACH)
[Homebridge] [TuyaAccessory] Closed connection with Power Strip
[Homebridge] [1/19/2020, 4:37:52 PM] [TuyaLan] Connected to Power Strip
[Homebridge] [TuyaAccessory] Socket had a problem and will reconnect to Power Strip (Error: ERR_CONNECTION_TIMED_OUT)
[Homebridge] [TuyaAccessory] Closed connection with Power Strip
[Homebridge] [TuyaAccessory] Socket had a problem and will reconnect to Power Strip (Error: ERR_PING_TIMED_OUT)
[Homebridge] [TuyaAccessory] Closed connection with Power Strip

If I turn on/off any power button on the device (using Ucomen Home app)...
[Homebridge] [TuyaAccessory] Odd message from Power Strip with command 8: { cmd: 8, data: 'Áî\u0017\u0000Ð4½\u0006dÂgÆcs6\u0003ÆÙ1·ÇÊü\u0010ê}\u001c\u0017Y' }
	(data value is different for every event, even if I press the same button)

I'm getting the same thing but with the TESSAN Outdoor Wifi Outlet. I also had to specify the IP manually. Using the Ucomen Home app I was able to get the correct ID and Key and the device does show on HomeKit and no longer says "Not Responding". But it still doesn't work and eventually the bad connections will cause homebridge to run out of memory and quit.
My config looks identical to the OP.
Here is my error.
[TuyaAccessory] Odd message from Outdoor Lights with command 8: { cmd: 8, data: 'Iù47/ýßcWö]ËBÌh\u000e' }


Feature/neba 25

Implementation of #25

Coverage decreased (-0.14%) when pulling 665d5409686ded0c51d88158d2eb179da2540aa0 on feature/neba-25 into 64697f68321999823ba98eb694e885182a0e36ed on develop.

Coverage decreased (-0.14%) to 90.56% when pulling 06aa18ff3109e8f5169377abefe1d306a375dad6 on feature/neba-25 into 6ebb1c1fcab1f93e3ef4366475ffe06fcc0d8aa3 on develop.


[BACKLOG-21753] Fixes issue with Cassandra Output step and column fam…

…ily terminology

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -Dsurefire.runOrder=alphabetical -B -fn -f 'pom.xml' -pl 'core' -P '!assemblies' -amd clean install && mvn -B -f 'pom.xml' -pl 'core' -P '!assemblies' -amd site

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'core' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  core/src/main/java/org/pentaho/di/trans/steps/cassandraoutput/CassandraOutputMeta.java

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes: <0.1% (click to expand)
org.pentaho.di.trans.steps.cassandraoutput.CassandraOutputMeta

Complexity Change: -0.7% :small_red_triangle_down:
Instruction Change: -0.4% :small_red_triangle_down:
Line Change: -0.6% :small_red_triangle_down:


Fix Siamese example

Broadcast -1 into all elements that equal 0 in original label.

Cherry-picked into digits-5.0.

Bug 200257814


Reduce memory consumption with command line option

Hi @fd0! Is just finished watching the video from your recent talk that is linked in the blog and really came to like restic and the philosophy behind the tool and the development process.
At one point in the talk it is mentioned that restic currently allocates around 300MB of RAM for reading multiple files and is therefore not suited to run on, e.g., a Raspberry Pi. I wondered whether you would consider adding a command line option for manually reducing that number at runtime to allow for restic to be used on low-memory ARM devices. Next to the ubiquitous RasPi I was thinking of backing up Android and Sailfish devices.

Hey, thanks for your interest in restic. The number of around 300MiB of ram that I mentioned in the talk is used for two things (mainly):

calculating scrypt(password): the constants for scrypt are hard-coded right now, and there's #17 to add code that automatically figures out how hard scrypt should be for the current system
having multiple buffers of blobs/packs to backup in memory: This is tightly coupled with the maximum concurrency that is allowed. For this I'd also like to have auto-tuning code, but a command line option is also likely.

To summarize my points above: It's planned and will be implemented at some point in time.

Sounds good to me. I'd like to keep this issue open to track that feature, then.

Agreed.

I'd be interested in this topic also from a server perspective, so not necessarily low memory but "still not enough". :) For example I've got a mail server with ~950GB of data (scanned 64173 directories, 2147728 files in 2:41) and with 2GB of RAM I ran into OOM with restic (running under ionice -c 3 nice -n 19). Are there any known numbers for calculating memory needs for restic (per file/per GB/....)? (I ran into OOM with restic in all kind of different VMs, known those needs upfront would make life easier, and if there'd be any way how to reduce memory consumption that would be great.)
Thanks!

My experience is around 4GB of RAM for 1.5TB repo size (for backup). prune takes even more (~9-10GB of RAM).
Memory usage depends mostly not on data size on particular machine, but mostly on repository size. So it is impossible to backup 100KB file on machine with 2GB of RAM (no swap) if restic repository itself is ~1TB.

Here:

Build:restic 0.8.3 (v0.8.3-11-gda77f4a2)
compiled with go1.9.4 on linux/amd64

Dimensions (but I actually believe that the majority of those files - a lot of repository clones, metadata caches, build directories, etc. - are actually excluded and those numbers reported by restic reflect the total amount):118775 directories
829113 files
129.220 GiB

Memory usage: around 3.8G per daily run

Is there any progress on this? Love restic, but my backups don't complete anymore due to memory exhaustion (1.5TB repository, uses ~23G RAM, b2 backend). Afaik, currently the concurrency can only be modified by modifying the source itself (https://github.com/restic/restic/issues/979#issuecomment-374359647), but this isn't really maintainable.

Has anything changed in terms of memory usage requirements?

I am also running into this issue. 2GB system 250GB restic repo. Running:
ionice -c 3 nice -n 19 ~/local/bin/restic -r /mnt/restic -p rk check --read-data-subset 3/7
(also using 4/14 to try to reduce memory use from 3/7) takes 1.7G. It can take 10 or 12 hours to complete the command because the poor system is swapping its brains out. Also the system is pretty much useless for anything else because of the swapping.
I like restic but this isn't sustainable.

Can someone clarify what the actual suggestion in this issue is? I mean, if the memory consumption/requirements can be made smaller, why should we not just do that and instead have a command line switch to enable using less memory?

I assume that allocating less space, if possible, would mean the process is slower, so that would warrant not doing this by default?

There have been improvements with respect to memory usage already and some others are in the pipeline.
In principle, I'm along the argumentation of @rawtaz that reducing memory usage will allow restic to be usable on a much wider field of devices.
That said, there are of course possibilites to further tune down memory usage, especially trading for speed. I started working on index-related possibilies in #2794

A proper solution for this will most likely use an on-disk repository index or allow clients to only load parts of the index as discussed in #1988.

We're trying to use Velero with restic and it's pretty much unusable for us. The pod gets killed as it consumes more than our limit of 6 GiB. We've got a lot of small files. How in the world can it use so much memory still?

How in the world can it use so much memory still?

We've got a lot of small files.

Sorry it doesn't work for you with the current setup!

It is by now possible to somewhat adjust the restic memory usage by limiting the number of used CPU cores, the number of backend connections and the GOGC environment variable. Other problems like the index size should be tracked in #1988, so I'm closing this issue.


Added sensors API

Hello!
Found your cool lib for Telldus Live. Sensors API was missing so I've added it. Included simple test. Just add credentials to system environment variables. Contact me if anything is unclear :-).

Good stuff! Thank you :)


Bug: can't install dotenv

@0dm I believe you experienced this issue, can you please clarify how you resolved it?

I opened a new terminal 😅

Let's document the workaround in the README troubleshooting.


Support mongo object literals in agg.pipeline

Fixes #2771

After the many planner test changes for arrayLits, this 1 test change is a bit disappointing. That may change though after #2681 gets addressed.


Start to Refactor Commissioning process

This PR is a start to refactor the Controller side commission process.
It basically splits out the commissioning logic from one methods into a class and converts it into steps as described in matter specs. I added step infos very verbose into code comments too.
The process is still not complete, but better as before (one step after the other).
It also enhnces the process for Thread and Wifi credential commissioning (and it works!). I did not yet enhanced the tests but it works in reality. testing we need to see how we improve this later.
The ControllerNode reference implementation also has the data as CLI parameter.

closes https://github.com/orgs/project-chip/projects/11/views/1?pane=issue&itemId=34438128
closes https://github.com/orgs/project-chip/projects/11/views/1?pane=issue&itemId=23088427

@lauckhart Please have a final look ( should work best on commit level). Adjusted and enhanced as discussed and tested again

Looks good


Addition of optional parameters for publisher and register

What is the context of this PR?
Addition of two new optional parameters i.e. eq_id and form types to help with publisher and register.
In relation to card:

https://trello.com/c/4m9Dnqh3/1322-supply-url-in-parameter-for-launcher-url-s-m

How to review
Check that it is valid JSON

Should we add descriptions for each of these two new elements?

Need re-basing and squashing


Trigger a slash command.

Description
We have a series of slash commands that were being used in development, and now that we are pushing to production, we'd like to remove them, but still be able to trigger their functions somehow. Is this possible from within the Bolt framework?
What type of issue is this? (place an x in one of the [ ])

[ ] bug
[ ] enhancement (feature request)
[x] question
[ ] documentation related
[ ] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[x ] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x ] I've read and agree to the Code of Conduct.
[x ] I've searched for any related issues and avoided creating a duplicate issue.

Hi,
If I understand you correctly you need to trigger a Bolt function outside of Bolt context, so can just use a standard Express endpoint :
expressApp.get('/my-specific-endpoint', async ({ body },res) => {
	res.sendStatus(200);
	DoSomething();
});

ah thanks.. I think this is what I meant!

Just in case you can also leave your command where they are and check the user that typed it (against some configuration file for example) to decide if it's ok to execute them.


Documentation parts not created with Lambda Proxy integration

Hello,
I used the plugin as the README, and it created methodResponses and models. But the plugin did not created the Documentation parts, the methodRequest, queryParams, headers, etc
I'm using Lambda Proxy integration
Could someone help me?
yaml.txt

I will create a test environment and have a look into this after christmas.

This could be because of the same bug discussed in #9. I will merge the PR soon.
@fssantana I will inform you when the fix is published. Maybe this solves your problem, too.

@fssantana Just try to update to the newest version of this plugin. Maybe that will fix your issue. Can you give me an update if that helped? :)

I believe I have the same issue. The section in custom/documentation and models/description do not yield any output in the generated documentation.

Hi,
If resource.Properties.MethodResponses = [] then this line is always true => MethodResponses aren't created.
Could you please check this?
Thank you.

Hello there,
I am also not having Documentation generated for the AWS Resources, Methods, etc. if the API Endpoint is configured as a Lambda_Proxy.  I have the latest version of this library.  Is there any progress with this bug?
Thanks!

I haven't had the time to look into that. But when I have some time I will investigate that behaviour. Thanks for all the support from you 👍

@brettcohen, @hoangleanhtu, @fredoche
Can you test the commit in the PR #18 with your project?
Maybe #17 is just a duplicate of this issue.

I'm using v0.5.7 and I'm not seeing any documentation either.
custom:
  documentation:
    summary: My REST API
    description: My description
functions:
  getthing:
    handler: handler.getthing
    events:
      - http:
          method: get
          path: thing
          cors: true
          documentation:
            summary: Get things
            description: Get things, optionally filtered by queryParams
            queryParams:
              - name: doodad
                description: only return things for the specified doodad
            requestHeaders:
              - name: Authorization
                description: "Needs to contain 'Bearer [JWT]'"

So I have been debugging this issue as I am experiencing it too. According to https://github.com/9cookies/serverless-aws-documentation/blob/master/src/index.js#L60 you need to also specify customVars.documentation.version

Yeah, this apparently got lost when writing the documentation. I will add it there.
I would love to add some object hash library in there so you don't need to define the version yourself. But in addition make it also possible to define the version yourself.

Thanks very much for the random version @tchock Just started a new project at work so this all comes at a perfect time for me :+1:

@tchock for some reason the generated documentation is still different between lambda and lambda proxy functions (using the latest version of the library, 0.6.0).
For example, I have this lambda-proxy snippet:
functions:
  hello:
    handler: handler.hello
    events:
      - http:
          path: /
          method: GET
          documentation:
            summary: 'Root Endpoint Summary'
            description: 'Root Endpoint Description'
            queryParams:
              -
                name: "hello"
                description: "Who do you want to greet? Add ?hello=world"
            methodResponses:
              -
                statusCode: '200'
                description: "yo!!"
                responseHeaders:
                  -
                    name: 'x-superheader'
                    description: 'Content-Type of the response'
                responseBody:
                  description: "Successful operation."
                responseModels:
                  "application/json": "HelloWorldResponse"

In the documentation on the API Gateway, the Response Code, Response Headers, Response Body and HelloWorldResponse model as well as the Query Parameters do show up ( http://nvgl.it/Jlkd) , but when I look at the Method Execution, it does not list the query parameters: http://nvgl.it/fX47
Any idea how to solve this?

Same problem as you @nielsgl
Using v0.7.0


The PageColumns component displays a bug on my computer, with two lines missing at the top and two extra lines at the bottom.

Environment
The PageColumns component displays a bug on my computer, with two lines missing at the top and two extra lines at the bottom.
Version
last version
Reproduction

Description
The PageColumns component displays a bug on my computer, with two lines missing at the top and two extra lines at the bottom.
Additional context
No response
Logs
No response

This is actually a Safari bug, have you tried to wrap each item with break-inside-avoid like this?
<template>
      <UPageColumns>
        <div v-for="(testimonial, index) in page.testimonials.items" :key="index" class="break-inside-avoid">
          <ULandingTestimonial v-bind="testimonial" />
        </div>
      </UPageColumns>
</template>

testimonial, index) in page.tes

The code copied from the pro saas project includes break-inside-avoid by default, but there is still this bug

Yeah it's not a perfect fix, I haven't found a better way yet. I assume that if you remove it it's even worse right?

Yeah it's not a perfect fix, I haven't found a better way yet. I assume that if you remove it it's even worse right?

Yes, it was worse after removal. Just ignore it for now, thank you


Allow deployManagedBeta to work with private GitHub repos

CumulusCI 2's cci task run install_beta now works with private Github repositories via the github service configured in the cci keychain with the command cci project connect_github.


Nested object in fragment unexpectedly not optional

Describe the bug
I have an entity that has optional entities as fields, and this is reflected in the type generated for that entity. However if I use a fragment to query, then the type generated for the fragment loses the ? and no longer matches.
Unless I'm misunderstanding something, I would expect a fragment that fully specifies all the fields of an entity to exactly match the type generated for the field itself.
To Reproduce
Steps to reproduce the behavior:

My GraphQL schema:

type Address {
  street: String!
  suburb: String!
}

type User {
  id: String!
  address: Address # <-- optional
}

type Query {
  user(id: String!): User
}

schema {
  query: Query
}

My GraphQL operations:

fragment AddressFields on Address {
  street
  suburb
}

fragment UserFields on User {
  id
  address {
    ...AddressFields
  }
}

query getUser($id: String!) {
  user(id: $id) {
    ...UserFields
  }
}

My codegen.yml config file:

schema: "schema.graphql"
documents: "operations.graphql"
generates:
  types.ts:
    - typescript
    - typescript-operations:
        preResolveTypes: true
        dedupeOperationSuffix: true
    - typescript-react-apollo:
        reactApolloVersion: 2
        withHooks: true
        withHOC: false
        withComponent: false

Minimal example that will complain because of type mismatching:

import { User, UserFieldsFragment } from "./types";

const userMapper = (user: User): UserFieldsFragment => {
  return user;
};

tsc says:
error TS2322: Type 'User' is not assignable to type 'UserFieldsFragment'.
Property 'address' is optional in type 'User' but required in type 'UserFieldsFragment'.

Expected behavior
For UserFieldsFragment to declare address as ?
Environment:

OS: MacOS Mojave
@graphql-codegen/...: 1.9.1
NodeJS: 11.13.0

Running into this one as well. Any clue where the regression was? Happy to help fix this up.

I think it's an issue with preResolveTypes, can you please try to set it to false and let us know if it works?

preResolveTypes: false has the same issue, just with a little more output
example.ts:4:3 - error TS2322: Type 'User' is not assignable to type '{ __typename?: "User"; } & Pick<User, "id"> & { address: { __typename?: "Address"; } & { __typename?: "Address"; } & Pick<Address, "street" | "suburb">; }'.
  Type 'User' is not assignable to type '{ address: { __typename?: "Address"; } & { __typename?: "Address"; } & Pick<Address, "street" | "suburb">; }'.
    Property 'address' is optional in type 'User' but required in type '{ address: { __typename?: "Address"; } & { __typename?: "Address"; } & Pick<Address, "street" | "suburb">; }'.

4   return user;
    ~~~~~~~~~~~~

Ok I checked that, and it actually makes sense.
The type User is being transformed into:
    export type User = {
       __typename?: 'User',
      id: Scalars['String'],
      address?: Maybe<Address>,
    };

And it represents the way the type could be structured in your API implementation. it means that it can be there, or not (marked as ?), and it could be null as well (marked as Maybe).
In GraphQL operations, if you ask for a field - it will be there - there is no such thing optional, so it might have a value, or it will be null. That's why the generated operation types doesn't use optionals.
Also, trying to convert the generated TypeScript User into UserFragment might be wrong, because User is a representation of all fields, and UserFragment is only part of it (selection set).
If you wish to have more "similar" types, you can set avoidOptionals: true and it will make sure that you generated User type will not use ? to mark nullable types, and it will use only Maybe - which is similar to what operations are generating. But still - note that this might not work because of selection sets. You client-side implementation should use only the generated UserFragment only. The User type is there only to be a base type.


Update javascript.rst

This is a minor change to the docs to fix #4650

Thanks, but documentation fixes should always be made against the current stable release branch.

Apologies, I wasn't aware of that. I will open another PR.

tip: create a separate branch. that way you could have just rebased, force-pushed and updated this PR's target branch instead of having to open a new one :)


IK preview does not work

Godot version
Godot_v4.0-rc1_mono_win64.zip
System information
Windows 10
Issue description
in editor can not preview the IK of skeletonIK3D,
worked in December betas
Steps to reproduce
add skeletonIK3D node
setup
click play IK
click target node3D
IK preview mode no longer active so  can not see results of moving it
Minimal reproduction project
N/A

Although SkeletonIK3D is not working right now, I also encountered this problem

Can confirm, switching from SkeletonIK3D to any other node results IK solver stop playing (in editor).

@tamagos @dszhblx ive slapped a quickfix as editor tool, attach this to your SkeletonIK node and click "playing" checkbox in properties
@tool
extends SkeletonIK3D

@export var playing: bool = false

func _process(delta):
	if not Engine.is_editor_hint():
		return

	if self.playing and not self.is_running():
		self.start()

	if not self.playing and self.is_running():
		self.stop()

@Calinou Since you guys decided that Godot 4.0 includes SkeletonIK3D which works fine, this little bug should also be fixed in Godot 4.0.Thanks!

This issue still persist in 4.0😢 but hopefully this gets fixed on the next update 🤞

@lyuma

I'm surprised this regressed since I don't think there's been any work done on the IK node, but I am looking into issues related to SkeletonIK3D so I'll include this in my testing.

In case you're still looking at SkeletonIK3D issues, I just had a brief look into this.
void SkeletonIK3DEditorPlugin::edit(Object *p_object) {
	if (p_object != skeleton_ik) {
		if (skeleton_ik) {
			play_btn->set_pressed(false);
			_play();
		}
	}

If the edit method is called on the SkeletonIK editor plugin with a nullptr as the object, the SkeletonIK is disabled.
If this particular code is removed, the issue goes away - however this hasn't been touched in years. So it almost feels like this was originally the intended behaviour?
It does look like there was some recent work done in editor_node.cpp. In this particular case, the method is called by the hide_unused_editor method:
void EditorNode::hide_unused_editors(const Object *p_editing_owner) {
	if (p_editing_owner) {
		const ObjectID id = p_editing_owner->get_instance_id();
		for (EditorPlugin *plugin : active_plugins[id]) {
			_plugin_over_edit(plugin, nullptr);  // <-- this is it
		}
		active_plugins.erase(id);

I'm not sure if its as simple as removing the first part of the edit method on the SkeletonIK editor plugin - that seems to fix the issue, but I'm not sure if it would have unintended side effects.

Here's another workaround you can use today (kind of the Blender style approach, like how you'd do pose mode while editing a mesh)

basically select two nodes, and then you get to keep the toolbar item for the SkeletonIK3D and it keeps updating.
We could theoretically mention this in the documentation. unfortunately it's not so obvious. I'll think what to do.

SkeletonIK3D woks in 4.1 beta3，but this issuestill persist
@lyuma

any updates, its hard to recommend a engine when feature regression are ignored

@tamagos Please don't bump issues to make demands like this. You already clarified that the issue is still valid a week ago. Most Godot contributors are volunteers and work on their free time, and that includes all contributors knowledgeable about IK.

@akien-mga
ok but I did not bump it a week ago, I have not posted since I created this 6 months ago

My bad, I misread the handle of the user who bumped it last.

I was able to manipulate my Marker3D positions while IK was updating by pressing the relevant hotkey. 'Q', 'W', 'E', 'R'
Hope that helps someone.

@tamagos @dszhblx ive slapped a quickfix as editor tool, attach this to your SkeletonIK node and click "playing" checkbox in properties
@tool
extends SkeletonIK3D

@export var playing: bool = false

func _process(delta):
	if not Engine.is_editor_hint():
		return

	if self.playing and not self.is_running():
		self.start()

	if not self.playing and self.is_running():
		self.stop()

This script no longer keeps IK running when the IK node is deselected. Not sure if it ever did. Sadly makes it no better than the Play IK button.

Why does it have to be so difficult? Press and hold CTRL and do the other things? Why cant you press play, make checkbox to set it active? Still persists in godot 4.1.1

@andzejsp The pr should hopefully be merged soon so this won't be an issue anymore.
https://github.com/godotengine/godot/pull/82391
Not sure if it will be cherry-picked for 4.1

@andzejsp The pr should hopefully be merged soon so this won't be an issue anymore.
#82391
Not sure if it will be cherry-picked for 4.1

i hope its fixed in 4.2 or the next stable 4.xx version. No problems. Just difficult to follow tutorials and the feature you see in older versions is missing in the newer ones. Keep up the good work! #DEATH TO UNITY

@andzejsp I edited your last comment to remove a statement which clear broke the Godot Code of Conduct. Please keep such things out of Godot community platforms, this is not welcome.

confirming IK preview works correctly on:
Godot_v4.2-dev6_mono_win64

@tamagos
Could you share a basic skeleton3DIK demo that show the IK preview works

SkeletonIK3D "play IK" doesn't stay in played state when selecting other nodes,

Issue still persists in Godot 4.3_rc2 (at least in mono version)
Reproduce: Set a Skeleton3DIK, put a Node3D target, play IK (works)
Save, close scene, reopen
Now it wont play anymore in editor.


add jdk16 and remove jdk15

Close #13
Signed-off-by: Sophia Guo<EMAIL_ADDRESS>

https://github.com/sophia-guo/runaqaTest/runs/1824344982?check_suite_focus=true looks good.


Add missing testcase for Integer to Roman exercise

45 is expected to be "XLV" not "VL"
need to only handle the 9th factor on base 10 factor

Codecov Report

Merging #10 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #10   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          18     18
  Lines         307    308    +1
  Branches       55     55
=====================================
+ Hits          307    308    +1

Impacted Files
Coverage Δ

src/integer-roman-conversion.js
100% <100%> (ø)
: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 eed6ccb...a3cc1b8. Read the comment docs.


Add :charset and :collation start_link options

Ref: #74

@josevalim this is a similar approach to mariaex [1] although we additionally pass collation to the query.
An alternative approach would be to use after_connect for this, if user had it, we'd adjust it with "SET NAMES" query:
Keyword.put(opts, :after_connect, fn conn ->
  set_names!(conn, opts[:charset], opts[:collation])
  opts[:after_connect] && opts[:after_connect].(conn)
end)

using this approach we could even leave the driver as is, and instead make this change in the ecto adapter. Though, I think it's fine to have this in the driver because then we can talk a little bit about charset and collation in the docs as I do on the PR.
WDYT?
[1] https://github.com/xerions/mariaex/blob/v0.9.1/lib/mariaex/protocol.ex#L222

I would that on the driver as is because it has been shown to be surprising.

Thanks, merging as is.


Bumped TAG to the latest release

Bumped the isle-buildkit images to the latest release.

Can merge after https://github.com/Islandora-Devops/isle-buildkit/actions/runs/3388978322 completes.


Review and publish new meeting

As City Staff,
I want to review and publish a new meeting on Civinomics,
so that I can get structured public feedback on open items and prevent getting input on closed items.

Acceptance criteria:

You can publish a new meeting with some both open and closed agenda items.

Acceptance tests:

Navigate to a draft meeting
Close one of the items to public feed back.
Publish the meeting.
Use the Civinomics interface to notify at least one other person that the meeting has been published.
Navigate to an item that you marked as closed and make sure that you can't comment or vote on the item.

Publish/Unpublish works but does not give visual feedback to the user of the state change.


Trigger animation when div is visible

I have a little paper plane and want to satrt its animation qhen the div enters de viewport. This is what i have in the css:
@keyframes example {
0%   {left:0%; top:75%;}
25%  {left:25%; top:75%;}
50%  {left:75%; top:75%;}
100% {left:100%; top:75%;}
}
.target[data-emergence=visible] {
animation-name: example;
animation-duration: 5s
}
Do i have to attach this to the div or to the image i want to animate?

Hey @FlatAutumn, I'd need to see more of the code to determine how you want to render the animation, can you provide some of the HTML—are you also able to confirm that the script is set up correctly and changing the state to visible when through dev tools?

Dont worry, i figured it out. It was a little mistake i didnt see xD
Thanks anyways :)


does reference free mode work?

Hi just curious, since the feature is implemented in the code but seems not mentioned in the paper.
Does it work?
Is it worse or better than using an out of domain SFT model (like those finetuend with alpaca)?
Thank you!

reference-free mode is equivalent to using the uniform policy as the reference model (i.e., one that assigns the same probability to all responses), since then the reference model log prob cancels for the chosen and rejected for all pairs.
In my experience, naively using reference-free DPO works worse than using a real reference model. However, it's possible this is due to response length (autoregressive models will tend to assign lower probability to longer sequences). I'd be curious to see what happens if you do reference-free DPO, but make sure the chosen and rejected responses are the same length!


FileNotFoundError: [Errno 2] No such file or directory: '/home/liu/ZZB/DiffIR-master/DiffIR-inpainting/models/lpips_models/vgg.pth'

Hello, thank you very much for your contribution! I encountered this issue during debugging and am unsure how to resolve it. I kindly request your guidance. Wishing you a joyful life and successful endeavors!
[main][CRITICAL] - Training failed due to [Errno 2] No such file or directory: '/home/liu/ZZB/DiffIR-master/DiffIR-inpainting/models/lpips_models/vgg.pth':
Traceback (most recent call last):
File "bin/train.py", line 50, in main
training_model = make_training_model(config)
File "/home/liu/ZZB/DiffIR-master/DiffIR-inpainting/saicinpainting/training/trainers/init.py", line 25, in make_training_model
return cls(config, **kwargs)
File "/home/liu/ZZB/DiffIR-master/DiffIR-inpainting/saicinpainting/training/trainers/defaultS1.py", line 32, in init
super().init(*args, **kwargs)
File "/home/liu/ZZB/DiffIR-master/DiffIR-inpainting/saicinpainting/training/trainers/baseS1.py", line 79, in init
self.val_evaluator = make_evaluator(**self.config.evaluator)
File "/home/liu/ZZB/DiffIR-master/DiffIR-inpainting/saicinpainting/evaluation/init.py", line 16, in make_evaluator
metrics['lpips'] = LPIPSScore()
File "/home/liu/ZZB/DiffIR-master/DiffIR-inpainting/saicinpainting/evaluation/losses/base_loss.py", line 109, in init
self.score = PerceptualLoss(model=model, net=net, model_path=model_path,
File "/home/liu/ZZB/DiffIR-master/DiffIR-inpainting/saicinpainting/evaluation/losses/lpips.py", line 26, in init
self.model.initialize(model=model, net=net, use_gpu=use_gpu, colorspace=colorspace,
File "/home/liu/ZZB/DiffIR-master/DiffIR-inpainting/saicinpainting/evaluation/losses/lpips.py", line 294, in initialize
self.net.load_state_dict(torch.load(model_path, **kw), strict=False)
File "/home/liu/anaconda3/envs/DIFFIR/lib/python3.8/site-packages/torch/serialization.py", line 986, in load
with _open_file_like(f, 'rb') as opened_file:
File "/home/liu/anaconda3/envs/DIFFIR/lib/python3.8/site-packages/torch/serialization.py", line 435, in _open_file_like
return _open_file(name_or_buffer, mode)
File "/home/liu/anaconda3/envs/DIFFIR/lib/python3.8/site-packages/torch/serialization.py", line 416, in init
super().init(open(name, mode))
FileNotFoundError: [Errno 2] No such file or directory: '/home/liu/ZZB/DiffIR-master/DiffIR-inpainting/models/lpips_models/vgg.pth'

The VGG model should automatically download when running. Please check your network connection.

Thank you very much for your answer, it has been resolved.


Create loader for Open Images data on AWS RDS database

Decide on a mechanism to import the following data from the CC S3 bucket:
s3://cc-openledger-sources/openimages/
to the RDS Postgres database spun up in the Open Ledger Elastic Beanstalk environment.
It'll probably be something something "container commands"?
http://docs.aws.amazon.com/elasticbeanstalk/latest/dg/customize-containers-ec2.html#linux-container-commands

Another approach could be to spin up a Worker Environment (https://aws.amazon.com/blogs/aws/background-task-handling-for-aws-elastic-beanstalk/) and queue up image ids to process, but I think that's more appropriate for recurring tasks like getting updated information about images via APIs, or otherwise batch processing them (like generating perceptual hashes).

Can now call arbitrary loaders on a specialized EC2 instance, closing this.


Resize multiform on update

The default size for the multiform area is not specified. When we switch from one visualization to another the scaling of SVG is not correct.
The width of SVG is small to render the boxplot.

@bikramkawan @katush, is this still an issue?

This is fixed.


Input value not set

Trying out the basic functionality of the plugin and following the instructions however the input field seems to be blank for me and click on a date of any kind doesn't update it.
Looking in the using devtools it seems that no value attribute is set on the input field.

Code?  Are you setting the selected prop in the DatePicker?

Yep, I copied and pasted the code from the github example, I'm relatively new to React and to plugins but this particular one will solve the exact problem I have so keen to use it.

Right, can you paste your app code here?  Hard to debug without full context on what's going on =P

`var React = require('react');
var DatePicker = require('react-datepicker');
var moment = require('moment');
var Application = React.createClass({
displayName: 'Example',
getInitialState: function() {
    return {
        startDate: moment()
    };
},

handleChange: function(date) {
    this.setState({
        startDate: date
    });
},

render: function() {
    return <DatePicker
        selected={this.state.startDate}
        onChange={this.handleChange} />;
}

});
module.export = Application;`
moment and react have both been installed via NPM as well as the date-picker itself

Not sure what I've done but It seems to be working fine now. Thanks for your quick replies, it's a great plugin.

Cool, glad you were able to get it working!


aarch64 fix for big endian types

NEON intrinsics are not supported on aarch64 big endian targets. Change to use fall back types for big endian aarch64.
tested on aarch64_be-unknown-linux-gnu_ilp32 and aarch64_be-unknown-linux-gnu

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 87.46%. Comparing base (ea28664) to head (c5525e3).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1786   +/-   ##
=======================================
  Coverage   87.46%   87.46%
=======================================
  Files          16       16
  Lines        6021     6021
=======================================
  Hits         5266     5266
  Misses        755      755

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Thanks for this! A few small things:

Can you rebase on main and force-push this as a single commit?

Is there a citation for the fact that NEON intrinsics aren't supported on big-endian targets? If you could add a comment with a citation in the source code, that'd be good.
Can you add an include section here and use matrix includes to add the following two build combinations. Please add a comment that references this PR and explains why we test on these combinations.

toolchain: nightly / target: aarch64_be-unknown-linux-gnu_ilp32 / features: --all-features / crate: zerocopy
toolchain: nightly / target: aarch64_be-unknown-linux-gnu / features: --all-features / crate: zerocopy

@joshlf  fixed the stuff that you requested,
this is the issue you are looking for as evidence https://github.com/rust-lang/stdarch/issues/1484
I don't think the ilp32 version tests anything different so I skipped including that

it's failing because 'aarch64_be-unknown-linux-gnu' is not provided by rustup, only rustc
to test should do something like:
cargo build --target=aarch64_be-unknown-linux-gnu -Zbuild-std

it's failing because 'aarch64_be-unknown-linux-gnu' is not provided by rustup, only rustc to test should do something like:
cargo build --target=aarch64_be-unknown-linux-gnu -Zbuild-std

Ah gotcha. I confirmed locally that this works:
$ ./cargo.sh +nightly build --target=aarch64_be-unknown-linux-gnu -Zbuild-std --features simd

Can you add a new build target (under kani and before check_fmt) that runs that command? That can replace the other changes to .github/workflows/ci.yml.

Unfortunately I think that the build failure you're currently seeing is going to require you to compute the correct nightly to install based on the contents of Cargo.toml. You can extract that via ./cargo.sh --version nightly and then tell the GitHub action to install that particular nightly.
Here's how we do that in the build matrix - here it should be much simpler because you only need to support a single toolchain and none of the Miri stuff is relevant:
https://github.com/google/zerocopy/blob/a9f09d7afc526a3a54a7a37b991eca0f22358950/.github/workflows/ci.yml#L264-L321

@joshlf not sure what dependency is missing

@joshlf not sure what dependency is missing

You need to edit this line:
https://github.com/google/zerocopy/blob/a9f09d7afc526a3a54a7a37b991eca0f22358950/.github/workflows/ci.yml#L707
...to include check_be_aarch64

Thanks again for your work on this!

Please fix this by fixing the intrinsics upstream. Fixing this in every single crate is not less effort than fixing it in stdarch: https://github.com/rust-lang/stdarch/issues/1484#issuecomment-1942934003

@workingjubilee is the lack of support in zerocopy breaking you? I'm hesitant for us to own a fix in stdarch unless there are users who actively need this feature.

@joshlf Just to be clear, you're fine. If you want to accept this fix, that's okay. But I just don't think that fixing zerocopy, memchr, half, etc., in a fix that might be best-off reverted once stdarch is fixed, is better than fixing stdarch.

Okay gotcha, that makes senes.


Guides on how to trigger tasks from web instead of command line

I am new to this great tool and I am still learning. My guess is that is possible to start deployment tasks from a web UI ( nginx / apache + php-fpm), but not very straightforward. Hope someone can shed some light on this. Thanks

How about <?php shell_exec('dep deploy'); ?>?
https://www.php.net/manual/en/function.shell-exec.php

How about <?php shell_exec('dep deploy'); ?>?
https://www.php.net/manual/en/function.shell-exec.php

Yeah, I had it work using that technique. I almost forget to update thread... I will post the detailed how-to later. Let's close the issue for now. Thanks


[#5032] add url validator for geoconnex task

Resolves #5032
The update_from_geoconnex_task was making calls to the Geoconnex API even when the GeospatialRelation was a "custom" relation (in which case the feature is not included in the API).
This PR modifies the task so that API calls are only made for GeospatialRelations that have a valid Geoconnex URL

Pull Request Checklist:

[x] Positive Test Case Written by Dev

[ ] Automated Testing

[ ] Sufficient User and Developer Documentation

[ ] Passing Jenkins Build

[ ] Peer Code review and approval

Positive Test Case

Create a resource and add a CUSTOM geospatial relation by typing it in and hitting enter:

Open a shell and run the update_from_geoconnex_task
The task no longer fails with   File "/hydroshare/hs_core/hydroshare/utils.py", line 462, in get_jsonld_from_geoconnex  id = relative_id.split("/")[1] IndexError: list index out of range

METRIC
VALUE

https://sonarqube.cuahsi-workstation.com:9000/dashboard?id=hydroshare-5033


blank builtin images

The images that have builtin code rather than an image, are going blank with the new grafana. Not sure how to describe them, but those with a fill option.
I'm just getting a blank bounding rectangle (filled with whatever color I selected). For instance this is how it worked before

After the current grafana

I'm using 0.9.1 of the plugin, and happens with both safari and firefox. Thanks for the plugin, it was great before 8 and now let us do some neat stuff

Hi,
I notice something similar in grafana v8.1.2, but not yet in v8.1.1. I looks like a regression.
v8.1.1 on one PC:

v8.1.2 on another PC (the V in the voltage sensor and the arrow in the current sensor are missing):

Kind regards,
Dennis

I had the same issue but with Kubernetes shapes. It works fine with Grafana 8.1.1, but not with 8.1.2

Hi,
I had a look at the browser console just after grafana 8.1.2 loaded by dashboard and noticed a number of failed requests for defaultGraph.drawio, it answers "Plugin file access forbidden", I guess this is the recent change in 8.1.2.

I noticed in the filesystem that the file is there with slightly different path (including static) and accessible to grafana:
$ ls -lsa /var/lib/grafana/plugins/grafana-flowcharting/dist/static
total 324
4 drwxr-xr-x 2 grafana grafana   4096 Dec  6  2020 .
4 drwxrwxrwx 7 grafana grafana   4096 Dec  6  2020 ..
8 -rw-r--r-- 1 grafana grafana   5217 Dec  8  2020 defaultGraph.csv
84 -rw-r--r-- 1 grafana grafana  83260 Dec  8  2020 defaultGraph.drawio
224 -rw-r--r-- 1 grafana grafana 229289 Dec  8  2020 shapes.txt
Kind regards,
Dennis

#313 - are using 8.1.2 version of grafana, its workign before just last week the issue happen.

Hi,
A relatively easy workaround for now would be to revert back to grafana 8.1.1 until this gets fixed:
sudo apt install grafana=8.1.1. With this, the shapes become visible again.
Kind regards,
Dennis

Hi Dennis,
Thanks for the update. i am using Grafana cloud free version. how can i revert back the version?
Best regards,
Murad

Hi Murad,
I'm using the free version on embedded systems and PCs using the apt repository. When you installed the package separately), you can probably just download and re-install the earlier version.
Kind regards,
Dennis

Hi Dennis,
Thank you for the input. will try that until they found the root cause and resolve it. By the way, if we downgraded the version, do you think all the previous dashboards, data source setting, etc, will be gone? just wondering before downgraded it.
Best regards,
Murad

Hi Murad,
For minor downgrades this is probably safe as the structure of the grafana database in /var/lib/grafana/grafana.db remains largely the same. It never hurts to make a backup copy of the database before the downgrade, you can also re-use the database and other /var/lib/grafana folders on other machines to quickly restore the same dashboards on another machine ....
Kind regards,
Dennis

Thank you Dennis for the input and suggestion. Appreciate it. Hopefully the developer can resolve this issue soon.
Best regards,
Murad


Update session_api.rst

fix grammar

cherry-picked, thanks!

No problem - what does cherry-picked mean, though?

https://git-scm.com/docs/git-cherry-pick

ah interesting! thanks!


build on fedora22: dependency on /usr/bin/file

When building on Fedora 22 without /usr/bin/file installed, the build succeeds but stage1.aci does not contain all the required dynamic libraries. A closer look to the middle of the build log reveals the following:
cd systemd_build/installed && find -type f | xargs file  | grep ELF | cut -f1 -d: | LD_LIBRARY_PATH=usr/lib xargs ldd | grep -v ^\\. | grep '/' | sed  -e 's/^[[:space:]]*//' -e 's/.*=> //' -e 's/ (0x[0-9a-f]*)$//' | grep -v ^[^/] | sort -u > ../../host_deps.txt.tmp
xargs: file: No such file or directory
ldd: missing file arguments
Try `ldd --help' for more information.
mv host_deps.txt.tmp host_deps.txt
echo mkdir -p "\${ROOT}" > install.tmp
echo cp -af ../usr_from_src/systemd_build/installed/. \"\${ROOT}\" >> install.tmp
cat host_deps.txt | while read dep; do echo install -D \"${dep}\" \"\${ROOT}/${dep}\"; done >> install.tmp
echo ln -sf src \"\${ROOT}/flavor\" >> install.tmp
echo "echo v220 > \"\${ROOT}/systemd-version\"" >> install.tmp
mv install.tmp install
cp -f install ../aggregate/install.d/00usr
touch usr.done

The build should be stopped when /usr/bin/file is not installed.
@krnowak: something to check in the autoconf-based build #908?

usr_from_coreos has checks like this here fwiw

Hi the fix for this issue will be part of the new autotools port, the commit is in my local branch here:
https://github.com/endocode/rkt/commit/f924ad0f65723c45d5ae227c738d9ae2894fd54e

A v2 for the fix was posted, should be the final version.
Link:
https://github.com/endocode/rkt/commits/tixxdz/autotools-no-dbus-08-07-2015-version2

Fixed via #1121


Stuck on "waiting_close_channels"

I force closed all the channels I had on 3/30 and they have been stuck on "waiting_close_channels" ever since. I need to know how to finalize these closes and withdraw all funds associated with my lnd wallet back to mainnet.
By "all funds" I mean anywhere I might have sats such as my "lncli walletbalance" as well as "lncli listunspent".

Did you try restarting? Does it say that it's broadcasting the transactions in the logs when starting the node?

I tried restarting and it did not fix the problem. This is what lnd shows upon restart:
https://i.ibb.co/f9gRSXy/Problem.png

Did you try force closing channels that are in waiting close?

Maybe, whats the command for that?

lncli closechannel txid vout --force

Is there an easy way to do it for all of them at once? If not, where do I find the value for "txid"? Is it the "local_txid"?

It will be the funding txid, like id:number, id is tx id and number is the vout
I don't think it can be done for all of them at once

I'm not sure if this is correct but I used the "channel_point" to fill in the values for the command:
lncli closechannel 40f957cc2163513aca4353e759ac694d1a9a218afafa6289dedc9c8905cd8aa9 0 --force
That returned this:
[lncli] rpc error: code = Unknown desc = server is still in the process of starting
If I'm not getting the values from the right place then I need help on where to find them because I'm a massive noob. Fortunately, once this is resolved I won't be on this repository with noob questions anymore because I'm running a myNode now.

Did you just start the lnd or it has been running a long time?

I restarted it 20 minutes ago when I saw your initial response. Should I wait a while and try again? But again, prior to restart, this thing had been running for like 5 days and the channels have been stuck on waiting close.

Maybe try again in a bit

Okay I'm going to try again tomorrow and then give you an update

Just now tried. It throws the same error message unfortunately @alexbosworth

So the server is still in the process of starting for 24 hours? Anything happening in the log file?

Do you want the feed of bitcoind or lnd? It appears that they are both up to date though.

In case you want to force close all at once manually I helped someone with an umbrel with the same problem (in case your lnd starts correctly):
sudo docker exec lnd lncli pendingchannels | jq -r '.waiting_close_channels[].channel.channel_point' |awk -F: ' {print "sudo docker exec lnd lncli closechannel --force --funding_txid "$1" --output_index "$2}' | bash

for other setups just omit sudo docker exec in front of the commands

Appears to be a duplicate of https://github.com/lightningnetwork/lnd/issues/4819: as is, if you try to do a cop-op close, then abort, the only option after that is to force close.


Bug 1788140: bump(*): vendor update

openshift/library-go@6c7d7cc3: staticpod: improve node not ready degraded message to include reason and message
openshift/library-go@0b9c208d: build-machinery: Add human readable messages to go mod verify-deps

/lgtm

/bugzilla refresh


Incorrect language key

Minecraft version & Mod Version:
Minecraft 1.12.2 and Cyclic 1.17.2
Single player or Server:
both
Describe problem (what you were doing / what happened):
The language key "item.storage_bag.pickup" should be "item.storage_bag.togglepickup"
Crash log (if any):
None

1.17.3 https://minecraft.curseforge.com/projects/cyclic/files/2625191


Web development

What does this PR do?
Add resource(s) | Remove resource(s) | Add info | Improve repo

Web-Development readme added to courses
Algorithm and Datastrcuture book added to books

For resources
Web Development

Odin project
FreeCodeCamp
Codecademy
W3Schools
MDN Web Docs
CSS-Tricks
CSS Reference
CSS Zen Garden
CSS Grid Garden
Flexbox Froggy
JavaScript30

Data Structure and Algorithm

Groking Algorithms

Description

Added courses for web development
Added books for Data Structures and Algorithms

Why is this valuable (or not)?

These are one of the best resources available for the topic

How do we know it's really free?

You can check it on the website

For book lists, is it a book? For course lists, is it a course? etc.

web development  is a course list of websites

Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

please use the repo's lists, not your own


Enhance deployment manifest of component-reconcilers to support different image-tags per component-reconciler

Description
It has to be possible to use different container images per component-reconciler to gain more flexibility in the rollout approach. It has to be possible to release a new reconciler version but to use this new released container only for a particular component reconciler.
AC:

Per component reconciler can a different image-tag be used (configurable in see values.yaml)
The bump-reconciler CI job has to be updated to support this new configuration approach (a bump has to update the version of the mothership reconciler and all component reconcilers)

Reasons
Make deployments more flexible to allow the upgrade of single component reconcilers if needed.
Attachments

Reconciler has moved to the new architecture, this issue will not be worked on.


Bump version to 0.4.0 in main.go

Recent tag seems to have bumped version without bumping it in the source.

oops


GDScript 2.0: using add_autoload_singleton in plugin let the editor in a invalid changed state

Godot version
v4.0.beta4.official [e6751549c]
System information
Windows 10
Issue description
I use add_autoload_singleton in my plugin to register global events.
After activating the plugin the Editor shows a (*) symbol on the window bar.

When now try to close the editor a dialog is shown to save modified resources, but i have no scenes/scripts changed:

When pressed the  saven and close button, the editor is not closed and shows an error in the console.
editor/editor_data.cpp:598 - Index p_idx = -2 is out of bounds (edited_scene.size() = 1).
When now deactivate the plugin the next error is shown.
 Trying to remove non-existent autoload.
Steps to reproduce
Use the attached project.

activate the plugin
check the windows bar
try to close the editor or switch to project overview

Minimal reproduction project
AutoLoadTest.zip

Interesting how ProjectSettings.save() does not seem to fix this error. I would've assumed it would remove the unsaved indicator.

Hi, any update here? This bug blocks me from saving my project

I think the problem is that add/remove_autoload_singleton create undo_redo actions assuming the user is toggling them in Project Settings GUI. The script methods were recently added for convenience, and the C++ methods still need modification to skip undo_redo when called this way.
Source links:
https://github.com/godotengine/godot/blob/eaf306e0b15e4283883f06bf903b05711a4dbfe8/editor/editor_plugin.cpp#L417
https://github.com/godotengine/godot/blob/b5f3ac522e16b2d899604e2d0788a86c66476e78/editor/editor_autoload_settings.cpp#L763

There is a workaround for this issue. Instead of adding the autoload in _enter_tree(), add it in _enable_plugin(). Then disable it in _disable_plugin() instead of _exit_tree().
The autoload will stay there & never be removed as long as the plugin isn't disabled. It will not leave the project in an "unsaved" state.

Thanks @neth392, I will try that out for a while.
IF that's the solution, there's a documentation issue.

Indeed there is an open issue suggesting this change: https://github.com/godotengine/godot-docs/issues/9571 (and various people weighing in for and against it)
The primary documentation currently says to use _enter_tree(): https://docs.godotengine.org/en/stable/tutorials/plugins/editor/making_plugins.html#registering-autoloads-singletons-in-plugins
Most plugins seem to use _enter_tree() or require you to manually add/remove the autoload due to this problem.
There are many related issues and proposals, mostly piecemeal "fixes". Seems like there's a major need for a comprehensive solution.

@Hilderin you've done a deep dive into this aspect of Godot... got any ideas?

Indeed, the best practice should be to use _enable_plugin and _disable_plugin to add and remove autoloads.
The problem with _enter_tree and _exit_tree is that they are executed everytime the plugin is added or removed in the scene and that includes when the editor is starting and closing. That's why your project is marked modified when you open the project.
The methods  _enable_plugin and _disable_plugin are executed only once when the plugin is enabled or disabled by the user.
The methods _enter_tree and _exit_tree should be used for everything that needs to be executed on every editor startup/closing, for example adding buttons, docks, add the new types, etc...
Note: the error Index p_idx = -2 is out of bounds was an error in the scene saving that was fixed since.
I made a PR in the documentation to make to appropriate modifications: https://github.com/godotengine/godot-docs/pull/9979


Thank you!

Hey,
Just wanted to thank you for react router! I use it in all my react applications and very happy with it.
I have read criticism about v4 but the new api looks a lot cleaner and simpler to use. Very glad we have it now.
Keep up the great work. :+1:  Thanks again!

Thanks!


Chore: added more npm cheat sheet

What does this PR do?

Add content(s)

Description

Added more npm cheat sheet

Screenshots or Links

If applicable, add screenshots or a modified UI link to help understand your contribution.

Additional context

Add any other context about your PR.

Checklist:

[x] Read our contributing guidelines.
[ ] Search for duplicates.
[ ] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

@crescentpartha Kindly review my PR


fix(list): scss for lines="none" on ios

Short description of what this resolves:
Fixes an issue where list items in ios mode still had lines with lines="none" set.
Changes proposed in this pull request:

Don't apply this scss rule that sets border width, if lines="none".

Ionic Version: 4.x
Fixes: #14769

Thank you for the PR!


Python: api update for 0.4.0

** This should land Monday the 17th **
Chroma is upgrading from 0.3.29 to 0.4.0. 0.4.0 is easier to build, more durable, faster, smaller, and more extensible. This comes with a few changes:

A simplified and improved client setup. Instead of having to remember weird settings, users can just do EphemeralClient, PersistentClient or HttpClient (the underlying direct Client implementation is also still accessible)

We migrated data stores away from duckdb and clickhouse. This changes the api for the PersistentClient that used to reference chroma_db_impl="duckdb+parquet". Now we simply set is_persistent=true. is_persistent is set for you to true if you use PersistentClient.

Because we migrated away from duckdb and clickhouse - this also means that users need to migrate their data into the new layout and schema. Chroma is committed to providing extension notification and tooling around any schema and data migrations (for example - this PR!).

After upgrading to 0.4.0 - if users try to access their data that was stored in the previous regime, the system will throw an Exception and instruct them how to use the migration assistant to migrate their data. The migration assitant is a pip installable CLI: pip install chroma_migrate. And is runnable by calling chroma_migrate
Please reference the readme at chroma-core/chroma-migrate to see a full write-up of our philosophy on migrations as well as more details about this particular migration.
Please direct any users facing issues upgrading to our Discord channel called #get-help. We have also created a email listserv to notify developers directly in the future about breaking changes.
TODO

[x] Migrated any duckdb+parquet strings to the new format
[ ] Notified users about the breaking change (this PR, other suggestions?)
[ ] Move pypi target away from feature-branch and to 0.4.0 after 0.4.0 is released.

@microsoft-github-policy-service agree

@awharrison-28 happy to work on this - how can I run integration tests? I expected them to run in the test runner here "20 successful checks".

poetry run pytest .\tests\integration\connectors\memory\test_chroma.py from the semantic-kernel/python directory

The test runner for PRs only runs tests\unit...
tests\integration can take almost 2 hours to run and many of them hit external services.

@awharrison-28 thanks! ill just run the integration test for chroma then 👍


New update

Can someone update scriphookv.net because it mods won’t work it says scriphookv not supported

This is SHVDN, not ScriptHookV.


Update to vn0.6.12

Updates to use the vn0.6.12 release

Hi! This is the friendly automated conda-forge-linting service.
I was trying to look for recipes to lint for you, but it appears we have a merge conflict.
Please try to merge or rebase with the base branch to resolve this conflict.
Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug.


[Bug]: Invalid import path for DeepEvalCallbackHandler

Bug Description
When turning on the deepeval integration with set_global_handler("deepeval"), i receive the error:
ImportError: DeepEvalCallbackHandler is not installed. Please install it using `pip install llama-index-callbacks-deepeval

despite the package being istalled.
Version
0.10.17
Steps to Reproduce

add set_global_handler("deepeval") to one of your project files

Relevant Logs/Tracbacks
No response

i was having a hard time getting my env set up to run/write tests, however i believe this would solve the issue (its untested)

Fixed + merged+ published a new version


Add secp256k1_new.

This change is

crates/blockifier/src/execution/syscalls/syscalls_test.rs line 176 at r1 (raw file):
Previously, elintul (Elin) wrote…
Here too.

nothing is alphabetized here.
the first test is
test_storage_read_write
and the second is
test_call_contract

crates/blockifier/src/execution/syscalls/syscalls_test.rs line 176 at r1 (raw file):
Previously, ilyalesokhin-starkware wrote…
nothing is alphabetized here.
the first test is
test_storage_read_write
and the second is
test_call_contract

I think it is done now.

crates/blockifier/src/execution/syscalls/mod.rs line 497 at r2 (raw file):

}

pub fn secp256k1_new(

based on
https://github.com/starkware-libs/cairo/blob/cce494fd766d5c57478db067d4c98d77597810b8/crates/cairo-lang-runner/src/casm_run/mod.rs#L1061

crates/blockifier/src/execution/syscalls/mod.rs line 467 at r2 (raw file):

}

// Secp256k1 new syscall.

split into:
https://reviewable.io/reviews/starkware-libs/blockifier/678
Code quote:
// Secp256k1 new syscall.

crates/blockifier/src/execution/syscalls/mod.rs line 467 at r2 (raw file):
Previously, ilyalesokhin-starkware wrote…
split into:
https://reviewable.io/reviews/starkware-libs/blockifier/678

merged.

crates/blockifier/feature_contracts/cairo1/test_contract.cairo line 198 at r4 (raw file):
Previously, noaov1 (Noa Oved) wrote…
Can you please explain why it should be none?

0, 1 is not on the curve and
(0, don't care) is interpreted as None.

crates/blockifier/src/abi/constants.rs line 40 at r3 (raw file):
Previously, noaov1 (Noa Oved) wrote…
Can you please explain what this is?

thats the value we've decided on.

crates/blockifier/src/abi/constants.rs line 40 at r3 (raw file):
Previously, ilyalesokhin-starkware wrote…
thats the value we've decided on.

for the cost os a using a single range check unit.

crates/blockifier/src/execution/syscalls/secp.rs line 35 at r5 (raw file):
Previously, elintul (Elin) wrote…
WDYM by ID, BTW? Simply its creation order? Perhaps elaborate shortly.

I'll just say the co-pilot figured it out on its own.

crates/blockifier/src/abi/constants.rs line 40 at r3 (raw file):
Previously, noaov1 (Noa Oved) wrote…
WDYM? I wonder if it fits better under // OS gas costs.

I has to do with cairo costs, not os costs.
I think it makes sense to put it next to STEP_GAS_COST or maybe next to other builtin costs.


feat: Add http metrics - counter and duration DEV-2473

Pull Request Checklist
Task Description/Number

Issue Number: NO-TICKET
Adds a counter for requests:
# TYPE http_request_count counter
# HELP http_request_count
http_request_count{method="GET",code="200",path="/admin/projects/iri/__IRI__",} 1.0<PHONE_NUMBER>909
http_request_count{method="POST",code="200",path="/v2/searchextended/count",} 36.0<PHONE_NUMBER>909

Adds a histogram for request durations:
# TYPE http_request_duration histogram
# HELP http_request_duration
http_request_duration_bucket{time_unit="millis",method="POST",code="200",le="10.0",path="/v2/searchextended/count",} 36.0<PHONE_NUMBER>909
http_request_duration_bucket{time_unit="millis",method="POST",code="200",le="100.0",path="/v2/searchextended/count",} 36.0<PHONE_NUMBER>909
http_request_duration_bucket{time_unit="millis",method="POST",code="200",le="1000.0",path="/v2/searchextended/count",} 36.0<PHONE_NUMBER>909
http_request_duration_bucket{time_unit="millis",method="POST",code="200",le="10000.0",path="/v2/searchextended/count",} 36.0<PHONE_NUMBER>909
http_request_duration_bucket{time_unit="millis",code="200",path="/v2/searchextended/count",method="POST",le="100000.0",} 36.0<PHONE_NUMBER>909
http_request_duration_bucket{time_unit="millis",le="+Inf",method="POST",code="200",path="/v2/searchextended/count",} 36.0<PHONE_NUMBER>909

Basic Requirements
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] fix: represents bug fixes
[ ] refactor: represents production code refactoring
[x] feat: represents a new feature
[ ] docs: documentation changes (no production code change)
[ ] chore: maintenance tasks (no production code change)
[ ] test: all about tests: adding, refactoring tests (no production code change)
[ ] other... Please describe:

Does this PR introduce a breaking change?

[ ] Yes
[x] No
[ ] Maybe (not 100% sure => check with FE)

Does this PR change client-test-data?

[ ] Yes (don't forget to update the JS-LIB team about the change)
[x] No

IMHO this approach does not really make sense. This PR was simply meant to be a quick fix for easily adding some http metrics. However, it is clear that the chosen approach is suboptimal and hard to extend.
It makes more sense to use Route specific metrics like https://github.com/RustedBones/akka-http-metrics for Akka. However this would require to label each individual endpoint.
Since we are moving away from Akka Http this is too much work. Honestly, I was only interested in a single route and it is much easier to add a dedicated Metric for this route than trying to retrofit http metrics for all endpoints.
Closing this PR without followup.


[WS-3148] Service Accounts configuration for cms and proxy

Could you add tests for automountServiceAccountToken parameter?
cms.serviceAccount.automountServiceAccountToken
proxy.serviceAccount.automountServiceAccountToken

Could you add tests for automountServiceAccountToken parameter? cms.serviceAccount.automountServiceAccountToken proxy.serviceAccount.automountServiceAccountToken

I tried to avoid testing simple setters instructions in tests and focus on places where a property indicates some relation between resources (e.g. Pod vs ServiceAccount). Adding more tests is tempting, however the trade-off is that future changes e.g. adding/removing simple property would require changes in tests.


pro.jsonlint.com has a DNS error

Hi.
pro.jsonlint.com has a DNS error, it worked up until a day or two ago!

Yes, we really messed up a registrar switch. The old registrar (Name.com) name servers started redirecting to a domain monetization service, and there was some sort of issue with the pre-setup of DNS at NameCheap (looking into it with them). And, we simply didn't setup pro.jsonint.com. Our fault. It should be OK in a few moments. Thank you for letting us know!


Unit of measurement: why is there a duplicate fieldset for unit definition under Standard Unit tab in legacy GUI? (The field links to UoM anyway)

Consult the standard…


Make sure that the return value of wrapX() is stable

Follow-up on #3397.

+1, thanks

Thanks for the review! Will merge when Travis is happy.


(WIP) new meta service

@corylanou and I have been pairing on this.  This PR is what we have so far and feedback is welcomed.

New meta package in services/meta where meta.Client, meta.Service, and meta.cache will live
meta.Service has been roughed out
meta.Service creates an HTTP handler and a meta.store
Any GET request that comes into the HTTP handler is handled by serveCache and is treated as a long poll request for a cache update.
HTTP client passes in its current cache index, handler compares it to the current index, and sends the cache update if clientIndex < currentIndex that the store has.  If client's index is current, the request blocks until there is an update to send or until leadership is lost.
Any POST request will be handled by serveExec, which is currently stubbed out.  These requests will cause the cache to be updated, the current index to be incremented, and all waiting client cache requests to receive an update.

It's also fair to note that there is code in the handler.go that will likely get removed, etc.  I would mostly take a look at how we are doing the long polling for now, most everything else is either mock out code or has't been put under any real thought yet.

@dgnorton This can be closed right?

Closed without merging because work moved to another branch.


Playgrounds; Import AudioKitUI

I was getting errors running Hello World in Playgrounds ("Can't lookup symbols, etc.")
Not sure why, but importing AudioKitUI fixed it.
AudioKit is awesome! Thanks a bunch.

Oh thanks, we'll address this in 4.4.1

I fixed a bunch of the playgrounds on develop.

I'm not convinced this issue is resolved. Maybe I'm doing something wrong, but I get No such module 'AudioKitUI' in the AKPhoneView.swift file. I checked out the develop branch.

Same error: No such module 'AudioKitUI' on everything in AudioKit. Tried develop, cleaning rebuilding etc. with no difference. Is this an Xcode 10 issue?

@funct7 @MWAudioAI For Playgrounds in [repo]/Playgrounds/ you need to use the terminal to build the frameworks first.
See Building universal frameworks from scratch


Virtual: User is allowed to enter the current password as new password in the change passord section

Steps to follow:
a) Click on the link Change Password
b) Enter the current password in Old Password text box
c) Enter the current password in New Password text box
d) Enter the current password in Confirm New Password text box
e) Click on "Submit"
f) Observe the message displayed on the screen
Actual Result: Password Changed
Your password has been changed successfully! is displayed
Expected Result: User should be indicated that the old and new passwords are same and no password change is done/can be done
Priority: 2

Milestone - Features backlog


[mesheryctl] Fix for #1651: Allow users to pass in custom meshery server url instead of using the hardcoded localhost

@leecalcote one of the two suggestion you had on improving the cmd

The Issue page is  #1651

Reference on mesheryctlconfig - https://github.com/layer5io/meshery/issues/1083


Add ability to specify callback thread

Current coverage is 87.50% (diff: 100%)

Merging #27 into master will increase coverage by 1.01%

@@             master        #27   diff @@
==========================================
  Files             7          7
  Lines            74         80     +6
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits             64         70     +6
  Misses           10         10
  Partials          0          0

Powered by Codecov. Last update 85ad16f...e47c2aa