issues.gradle.org has incorrect instructions

If you go to issues.gradle.org, it shows you this:

Searching for "gradle issue tracker" on google led me there. Trying to create a "bug" post on discuss.gradle.org was not possible. I suppose this is because that has been deprecated in favor of github issues.
The banner on that page should be updated to reflect this change.
Also, if possible, robots.txt should be updated so google doesn't lead people to that site.

Thanks for reporting! I changed the banner.

Thanks @bmuschko. The "Introduction" section is fixed. The banner on the top still needs fixing.

@rahulsom Done. Hadn't noticed that before.

:+1: :-1:


Use another service than httpbin.org for tests

Create a fake local service for executing http request during unit tests.
httpbin.org is great but sometimes the network is not fast enough or maybe the requests are getting throttled by httpbin because I use it too much.
Anyway, the default 2sec timeout for each tests in karma is not enough. Augmenting that timeout is not really a solution.

Use sinonjs to stub the axios instances used in the tests

Find a way to proxy axios import in src/index.js
proxyquire only works for node.js 😢
Tried rewiremock without success.
Maybe try inject-loader

Actually maybe just try to use axios-mock-adapter instead of trying to replace complete dependencies.


Bug: shop=car in area with service=dealer, tyres, parts, ...  show road area outline.

The wiki page  allow to use "service=*" tag to specifiy car shop services like dealing, tyres, parts, repair, ...
See also this wiki page .

Oh yeah, I changed this in #3535 (see that ticket for more context)...
We couldn't use the service=* tag for vehicle services because the values are already overwhelmingly used for highway and railway stuff.  This makes it impossible to suggest good values to a user wanting to tag vehicle services for a car shop.  (see taginfo output below - you can see that service=* used for car shops is relatively rare).

Instead we are supporting multikey tags like service:vehicle:*=yes/no
(This is similar to, and inspired by, service:bicycle:*=yes/no, which bicycle shops use)

If you are a wiki editor, please feel free to update the wiki to reflect the new tagging - thanks!


Support Weights and Improve Time Complexity of Prediction Methods

Description of Changes

[x] Wrote at least one-line docstrings (for any new functions)
[x] Added test(s) covering the changes (if testable)

Lot of changes:

Implements per-player weights and/or partial play support.
Support a new balance flag to skew tail end skill ratings in high uncertainty matches. Complements beta parameter.
Add descriptions to README
Update dependencies
Reduce time complexity of all predict methods to $O(n^2)$.

Issue(s) Resolved

Fixes #29
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under openskill.py's MIT license.

I certify the above statement is true and correct: @vivekjoshy

oh snap!! weights!? :clap:


Handshake spec

MongoDB has requested that we implement the handshake spec for the driver so that they can identify users of the driver who are connected to Atlas. This is important so that the company can get some actual numbers on who is using the driver and whether it's worth investing more in it. This pull request implements the desired feature.
In addition to Kevin, I'm also asking @pmeredit to take a look at this just to get another pair of MongoDB eyes on the code. (It doesn't seem to allow me to add him as a reviewer because he's apparently not in the mongodb-labs organization, hence my @-mentioning him inline).

Looks like the repo isn't active on Travis anymore, so we don't have automated tests :(

Hmm, I'll check with IT. Good catch!

Travis is back online!

All passing; ready for review

Unfortunately this relies on most people using 3.4+ and also upgrading their drivers, but not much we can do there 🤷

Upgrading the driver is definitely a necessity, but the server version isn't a huge deal since this only becomes useful for connections to Atlas, where almost all if not all of the clusters are running on 3.4+.

Ok, nothing really major here other than it might make sense to at some point add a VersionAtLeast(version: &[u32]) -> bool function that compares semantic versions.  I'm not as familiar with how many times you need such functionality in a driver, in the BI-Connector we need it everywhere because of what agg pipeline functions are available in which versions.

Ok, nothing really major here other than it might make sense to at some point add a version_at_least(version: &[u32]) -> bool function that compares semantic versions.

I've added a macro that skips tests given a minimum version number to meet and used it for the handshake test. If we move to drivers-evergreen-tools/mongo-orchestration, we'd be able to do some macro magic to only define the test based on the environmental variable with the MongoDB version, but for now I think that's overkill.

Also, does Atlas even support servers older than 3.4?

I don't think so, but it's not a huge deal either way. If there are users using this driver on Atlas, I'm not super worried that they'd be hesitant to upgrade past 3.2 even if it's not required.

Waiting for a final Travis run after rebasing, and then I'll merge!
On a tangential note, it looks like 0.3.8 was never published to crates.io. Since it was already tagged and we don't want to mess with history, we shouldn't add this to 0.3.8. Should I just release the version with this as 0.3.9 (and if so, should we still publish 0.3.8)?

yep, let's release both versions 👍

Hmm, actually, it looks like we didn't actually tag 0.3.8, so I'll just include this in that release.


Allow to use berry again only if capture failed

Short Description:
I'm seeing the bot was throwing berry again after the ball had completely missed the pokemon:

[pokemon_appeared] A wild Aerodactyl appeared! [CP 793] [NCP 0.37] [Potential 0.49] [A/D/S 6/1/15]
[threw_berry] Threw a Razz Berry! Catch rate with Pokeball is now: 24.01
[threw_pokeball] Nice Curveball throw! Used Greatball, with chance 34.53 (40 left)
[pokemon_capture_failed] Pokeball thrown to Aerodactyl missed.. trying again!
[threw_berry] Threw a Razz Berry! Catch rate with Pokeball is now: 36.01

However, that is not allowed in the game. One can only use berries again after capture had failed.

oh, big catch!

Great catch, I was about to work on this! This fixed #4290. 👍

@alexyaoyang  Thanks all the same 👍


Add step to bin/setup for git submodules

Hey @manuelmeurer,
I believe we need git submodules installed before we can call bundle install. This fixes #49.
Please check it out.
Thanks!

Great, thank you @etagwerker! 👍


Discussion on Probing Resistance of this Project

I am not a specialist on Censorship Circumvention, so if I'm wrong, I'd be glad if I could be corrected.
I've quickly scanned the code, so this is just a TLS tunnel set up to forward signal SNIs traffic as-is.
But what if the censor actively probe the suspected proxies? For example, connect and then use real signal app traffic and non-signal app traffic? The difference can be detected. And if a proxy can be detected, it can be blocked.
Hopefully we can come up with some new ideas to overcome this.

One crude idea is to use preshared tokens and fallback mechanism. In this case, we can hide a preshared key within SNI. For example, we can preshare a key 538fd09a066a, and then mix this with original SNIs, where cdn.signal.org could become 538fd09a066a.cdn.signal.org. We then only accept whitelisted SNIs mixed with our preshared key, and deny all other SNIs including the original one cdn.signal.org. This way without knowing the preshared key, the censor is unable to active probe any longer.

Also there's a weak temporal feature in this project, where the certificates tend to be newly signed. If possible, relay hosts should try to reuse existing certificates rather than brand new ones, which can be far more suspicious to the censors.

Besides, for large portions of TLS traffic, there would be an ALPN in the Client Hello frame. If this proxy runs TLS over TLS, then the ALPN would possibly not be commonly seen ones like h2 and http/1.1. Instead, the application tend not to send this extension, which creates a passive feature of this proxy.

I'd recommend against using SNI or anything else that happens before TLS is established as a way to trigger the connection to Signal.  Anything sent in plaintext before TLS can be replayed to uncover the nature of the proxy.
Maybe buffer the first 1024 bytes of data the proxy receives after TLS is established and if the first N bytes match a proxy-specific key/password then initiate the connection to Signal and strip off the N key bytes and begin proxying with the remaining bytes once TLS to Signal is established.  Otherwise connect and forward the buffered bytes to either a local default install of nginx that can serve a default index.html or to a legitimate site that can respond as usual.  The Signal app would need to know both the host name of the proxy and the key which would in theory be distributed to users as a pair.
In theory this allows existing popular sites to run the proxy without affecting their normal traffic and also allows double proxies running in firewalled countries to evade detection longer.

@benlivengood
Maybe I am not stating it clearly, but this project is a TLS in TLS tunnel.
The SNI trick happens inside TLS connection.
But anyway, thanks for your precious comment!

package main

import (
	"crypto/tls"
	"fmt"
	"log"
	"net"
	"os"
	"time"
)

func send(addr, server, sni string) int {
	c0, e := net.Dial("tcp", addr)
	if e != nil {
		log.Fatal(e)
	}

	c1 := tls.Client(c0, &tls.Config{
		ServerName:         server,
		InsecureSkipVerify: true,
	})

	c2 := tls.Client(c1, &tls.Config{
		ServerName:         sni,
		InsecureSkipVerify: true,
	})
	c2.SetDeadline(time.Now().Add(2 * time.Minute))
	s := fmt.Sprintf("GET / HTTP/1.1\r\nHost: %s\r\nUser-Agent: curl/7.68.0\r\n\r\n", sni)
	//b := make([]byte, 4096)
	l, _ := c2.Write([]byte(s))
	log.Println(l)
	if e != nil {
		return 0
	}
	log.Printf("%s->%s->%s\n", addr, server, sni)
	return l
}
func main() {
	if len(os.Args) != 3 {
		log.Fatalln("usage: main.exe server_name addr_port")
	}
	server := os.Args[1]
	addr := os.Args[2]
	l1 := send(addr, server, "updates.signal.org")
	l2 := send(addr, server, "telegram.org")
	if l1 != 0 && l2 == 0 {
		log.Fatalln("It is a Signal TLS Proxy")
	}
	log.Println("It is not a Signal TLS Proxy")
}

By the way, DPI (Deep Packet Inspection) techniques can detect TLS in TLS traffics easily (there are lots of papers available, if you do a search). Although not all that TLS in TLS traffic is our lovely proxy, you can still risk being caught using a proxy when censorship is there.
DPI might be expensive, but code like @studentmain is not. The current probing resistance is really a problem.

Here is an example run of the PoC from @studentmain:
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256

C:\src\signalTlsPoc>main.exe
2021/02/05 03:40:15 usage: main.exe server_name addr_port

C:\src\signalTlsPoc>main.exe your.proxy.server.example.com your.server.ip.address:443
2021/02/05 03:40:34 69
2021/02/05 03:40:34 your.server.ip.address:443->your.proxy.server.example.com->updates.signal.org
2021/02/05 03:40:35 0
2021/02/05 03:40:35 your.server.ip.address:443->your.proxy.server.example.com->telegram.org
2021/02/05 03:40:35 It is a Signal TLS Proxy

C:\src\signalTlsPoc>main.exe baidu.com <IP_ADDRESS>:443
2021/02/05 03:43:09 0
2021/02/05 03:43:09 <IP_ADDRESS>:443->baidu.com->updates.signal.org
2021/02/05 03:43:10 0
2021/02/05 03:43:10 <IP_ADDRESS>:443->baidu.com->telegram.org
2021/02/05 03:43:10 It is not a Signal TLS Proxy

C:\src\signalTlsPoc>

(modified to protect my personal information)
-----BEGIN PGP SIGNATURE-----

iQIzBAEBCAAdFiEEdbyjO018JNZhjd2SqnhRnCCMh0IFAmAcT0AACgkQqnhRnCCM
h0Lx6RAAlx6C6R7tZROlj8I+1uu2Owde9EjOA3BTYw7aC8pr4cd2LuTkmAY07TuZ
mbQRueUIKOXx2ZlTPRL1yOAW60nePC5SRmlOkNssafWzM+8zyqfHdRe64OX0ZA5V
KPgSVHOz2JKfh650oEqlaHb/6N/Nj++617TcHmmYoy/xsiDy/fD7IoifD8bT5N3+
4znZ2Sr2+IirKxqvpVDl5AwRBR5EfXUI54c0OjKB8Sq+VPlhAmAcj/eZgSz7A4rV
EWSJjwbN1fp7g2C4jNtjh0Z3jyvvA9yGNThylcMPbXcrc1a9sOzhbCgGTtF1413n
tXSjy3vRBEnwvlVs78ItQljCZV1S+jwpXlkjFPGAIYyRRQbvaQ4Biaki/4g3ofVu
nfqEnc5m4vSsde2r5GLBqxSkbsKuZjGBbd1KRE+5LZPvkVrA5wP400tirThnvNU9
G7S5Noz/8Npwazag0beDlio45e8sBafTBRJy/xuq6HVGdFkPa1C7EUMuDxVVnPdg
mOgyqrjXju+4y/pl39U6JmDt6/ZSn34/1EFVchRWS5Kq0gPDGdjvrjI88aOtqhRH
tT8F6dU8rwRa8tjMHv3gNVTQY+jQZIi4SHNB54DRc1K0AKzIkAOgD/ROhcU4LkXq
vudKRTfhbtL6l8FIo8WCzGVONK1dY+1alKXKc8cVJNrye/hm5t4=
=H1r/
-----END PGP SIGNATURE-----

Hey everyone! Thanks for the interest in this. We normally don't use GH issues for this type of discussion, though, and prefer to have that happen over on the Signal community forum. Please check out https://community.signalusers.org/ if you want to get involved. Thanks!


Bug: OpenID Address Claim Does Not Conform to OpenID Standard

In OpenID token, Address should be a json object not an array of string. This causes more rigorous openID client libraries to reject responses from Casdoor.
Failed to parse payload JSON: Error(\"invalid length 0, expected struct AddressClaim with 6 elements\", line: 1, column: 1578) at line 8 column 1)

See OpenID Connect 1.0 Spec: https://openid.net/specs/openid-connect-core-1_0.html#AddressClaim

I use https://docs.rs/openidconnect/latest/openidconnect/ openidconnect library in rust.

Quote from https://openid.net/specs/openid-connect-core-1_0.html#AddressClaim
The Address Claim represents a physical mailing address. Implementations MAY return only a subset of the fields of an address, depending upon the information available and the End-User's privacy preferences. For example, the country and region might be returned without returning more fine-grained address information.
Implementations MAY return just the full address as a single string in the formatted sub-field, or they MAY return just the individual component fields using the other sub-fields, or they MAY return both. If both variants are returned, they SHOULD be describing the same address, with the formatted address indicating how the component fields are combined.

formatted: Full mailing address, formatted for display or use on a mailing label. This field MAY contain multiple lines, separated by newlines. Newlines can be represented either as a carriage return/line feed pair ("\r\n") or as a single line feed character ("\n").
street_address: Full street address component, which MAY include house number, street name, Post Office Box, and multi-line extended street address information. This field MAY contain multiple lines, separated by newlines. Newlines can be represented either as a carriage return/line feed pair ("\r\n") or as a single line feed character ("\n").
locality: City or locality component.
region: State, province, prefecture, or region component.
postal_code: Zip code or postal code component.
country: Country name component.

Note that opened-connect-core advice the possibility of returning "just the full address as a single string in the formatted sub-field". That is, it is also acceptable to include only the formatted field without including the other subfields such as street_address and so on.

Therefore, in short, I would consider the previous reverted {formatted: "some address string"} as a valid compliance solution to AddressClaim.

@Congyuwang the previous PR was reverted because it has broken all our SDK code. All applications will fail. Can you make a PR to make a perfect fix?

Seems to me those SDKs need to make changes accordingly.

@gtn1024

I think a figured a solution. That is to implement access_token and openid_token separately. So you can have your SDKs still use the old access_token format, while improving your compliance to OpenID protocol.


middleware for redirects

If a request.is_intercooler() and the response contains a Location header, it needs to be replaced with an empty HttpResponse with an X-IC-Redirect header (because XHR has to follow actual redirects, we can't just add the header to the existing HttpResponseRedirect)

Fixed by b7d912d6b5c6c6e139d254b0065712c7c7636404


Devilbox UI footer overlaps embedded tools

Devilbox: 0.11
OS: Ubuntu 16.04
docker: 17.06.0-ce
docker-compose1.13.0

Adminer (and maybe other embedded tools) overlapped by footer. Unable to see table list or controls for pagination and additional command on bottom.

@ilsenem as this is a browser issue, could you please provide your browser name and version.

Chrome and FF latest versions - 100%. IE and Safari - unable to test.
This issue appears only on Adminer as far as i can see.

@ilsenem could you check out branch DVL-012 and see if it is working as expected.

Nope, still there. =(
Maybe there is a good solution to just open Adminer in new tab outside of Devilbox UI like you do with phpMyAdmin?

Sorry, ticket got auto-closed by merging the PR.
The above merged PR introduced some minor enhancements, but does not yet fully resolve the issue.
I will probably go with your solution to separate Adminer from the devilbox intranet.


Stage 6 -  Referee Submits Reference (Old 119)

In gitlab by @A-Lawrence on Aug 31, 2015, 01:09
A referee submits their reference.
It would be nice for referees to have a list of applications to write references for on the dashboard of the VT page.
Once reference has been submitted, an email should be sent to them confirming the reference is saved and thanking them for their time.
An email should also be sent to community manager telling them that a reference requires review.

In gitlab by @A-Lawrence on Aug 31, 2015, 01:15
mentioned in issue #158

In gitlab by @A-Lawrence on Aug 31, 2015, 01:15
mentioned in issue #159

In gitlab by @A-Lawrence on Dec 20, 2015, 18:45
Removed ~98469 label

In gitlab by @A-Lawrence on Jan 11, 2016, 01:05
Removed ~98476 label

In gitlab by @A-Lawrence on Jun 19, 2016, 17:33
Removed ~98471 ~98472 ~137454 labels

In gitlab by @A-Lawrence on Aug 20, 2016, 12:12
Changed title: Stage {-7 --} Referee Submits Reference (Old 119) → Stage {+6 - +} Referee Submits Reference (Old 119)

In gitlab by @A-Lawrence on Aug 20, 2016, 19:34
Removed ~160393 label

In gitlab by @A-Lawrence on Aug 20, 2016, 20:17
Removed ~98471 label


Toggling the Table using a UI Toggle

I cant seem to toggle the table and its components on and off using Unity's UI Toggle.
Unsure if its an issue on my end of unity or the assets end, thought id message here to see.

There is a toggle for the table's UI included with it, what do you mean by "Unity's UI Toggle" ? look for toggling the canvas if you want something different than the table's included toggle.

Then that's not related to the table, your UI toggle is just not working. Look up how to make a simple UI toggle in unity, there are tons of tutorial for that.

Reava is correct, as long as you are referencing the parent GameObject properly, there should be no issues assuming your toggle logic is setup correctly. Closing this as it's unrelated to our prefab specifically (I hope you end up figuring out the issue on your end!)

I was sure that it was the assets end, but you're right I found the issue on my end.


Proposal for Claim's values validation

Hi,
Checking th docs we found the methods Claims() and ClaimsAll() to ensure an endpoint is accessible only if the current user has the specified claims.
It works well, but in some scenarios we also need to check one of the claim's value and it does not seem possible with the current 'Claims' helpers.
Currently, we achieve that with Preprocessors (or middlewares, helpers, or whatever).
What would you think of adding new helpers (maybe "ClaimsValue()" and "ClaimsValueAll()") that allows to specify a claim AND a value to check ? The challenge would be to allow to check a static value or dynamically at runtime (via callback actions maybe ?)
If you find this relevant, I can try a PR to suit this need.
Regards,
Romain.

in v<IP_ADDRESS>-beta you can now do stuff like this:
public override void Configure()
{
    Get("test");
    Policy(b => b
        .RequireUserName("Administrator")
        .RequireClaim("UserId", "001") //checking claim value
        .RequireAssertion(ctx =>
        {
            return ctx.User.ClaimValue("Email") ==<EMAIL_ADDRESS>        }));
}

Nice ! I'll give it a try, thanks ;)


Style guide

I have been trying to follow this style guide recently, and it has served me pretty well. I like it because it has rationales for most of its points, and most of them are very reasonable.
But there are a few exceptions that I would like to make:

Two-space indent instead of three-space indent
Hard line limit at 160 characters. As per discussion on Discord, there is no hard line limit, though try to keep everything under 120 characters. If something goes a bit over, it's not the end of the world.
Omitting parenthesis for functions that take a unique table argument on a single line is allowed:
foo{bar=true, lorem="ipsum"}
We are not using a top-level src directory.

I'd like to recommend [this guide(https://chris.beams.io/posts/git-commit/)] for git commit messages.

I like 120-160 characters as a line limit, if for no other reason than readability on Github with side-by-side diffs. Your screen may have room for 200 characters when writing the code; it probably doesn't have room for 400 characters when reviewing somebody else's PR.

I don't usually enforce a strict line length limit, but I just measured the editor layout i'm using and the main pane is close to 100 apparently, with a smaller (<80) pane next to it. If I full-screen a single pane i get out to about 190 or so.

You should not omit parenthesis for functions that take a unique table argument on a single line. You may do so for table arguments that span several lines.

I'm gonna disagree with this one too, since table arguments are so common in factorio's api and we're all used to it already.

I agree, I overlooked that one. I'll add it to the list.

flib uses stylua now, so there's no need for manually upkeeping style any longer.


Use response_mode in authorize_uri if the option is defined

This resolves: https://github.com/m0n9oose/omniauth_openid_connect/issues/29
Some OpenID servers are configured to format the request to the client application as a POST request (form_post). This PR allows you to actually use an unused configuration option.

@januszm could you please test current master branch one more time before i push this update to the rubygems. This would help a lot to make sure this release is stable.

@m0n9oose I tested the master branch today, works fine on a dev server. It's not a production, user-facing system yet though.


Identifier "PhantomCamera2D" not declared in the current scope

Issue description
I'm getting the error "Parse Error: Identifier "PhantomCamera2D" not declared in the current scope." intermittently when hitting play from the Godot editor. This has been happening more frequently it seems like, especially after updating to the latest version. Sometimes it runs, other times it fails.
When the error occurs, it points to this line (348):
## Called when a [param PhantomCamera] is added to the scene.[br]
## [b]Note:[/b] This can only be called internally from a
## [param PhantomCamera] node.
func pcam_added_to_scene(pcam: Node) -> void:
    if is_instance_of(pcam, PhantomCamera2D) or is_instance_of(pcam, PhantomCamera3D): <-- This line
        _pcam_list.append(pcam)

        if not pcam.tween_on_load:
            pcam.set_has_tweened(self, true) # Skips its tween if it has the highest priority on load

        _find_pcam_with_highest_priority()

    else:
        printerr("This function should only be called from PhantomCamera scripts")

Steps to reproduce
I'm really unsure as it seems to happen randomly upon hitting play. I'm not even using PhantomCamera2D in my game.
(Optional) Minimal reproduction project
No response

If you have updated the addon from version 0.6.x ⇾ 0.7 within a project, then it's entirely possible that there's some lingering code in one of the hidden Godot directories that hasn't properly updated the name change of phantom_camera_2D to phantom_camera_2d — lowercased “d” at the end. Creating a new project and transferring over the res:// content might solve it then.
If the above doesn't help, would you be able to share a Minimal Reproduction Project (MRP) where that issue happens?
Mainly so I can see what setup and logic could potentially trigger this.
Haven't come across it myself, and there's not much I can do until I can replicate and verify that a fix would work.

Hello,
This is an issue for me as well-- but, in my case at least, I've narrowed the issue down to the use of GDScript's class_name within a Mono build of Godot.
I've seen Nathan Hoad's users (the Godot Dialgue Manger addon) mention this issue as well, though, as far as I know, no one has more information as to a solution or cause, or has created a reproducible project.
A workaround to avoid getting an error is to click build, then play.

I no longer encounter the error I described after having upgraded into 4.3rc1 (I think around beta1 is when my issue was resolved, I don't know which PR was relevant).

It was resolved in, if my reading is correct, Godot 4.3 Beta 3 in this PR.
Has been a long-standing issue with removing or changing classes in GDScript.

Closing the issue as Godot 4.3 should be addressing, or at least mitigating, this issue.
If it's still being encountered after updating to that Godot version, or later, please do reopen it.


Timestamp on logging

Hi everyone.
We use the library for 4 months now and we are very satisfied with it. We came up with some logs on the console and we needed a timestamp for every log. The issue is that on chrome devtools you may enable the timestamp but when you save the logs as a .log file the timestamp is not saved. Is there a way that we can globally enable a timestamp on the library to prefix it when the library writes to console? Is there another issue already addressing this or something similar?
Thank you!

Yes, you can do that, but it's not built into the library itself. The message-prefix plugin should do what you're looking for though: https://github.com/NatLibFi/loglevel-message-prefix.
This will have the side effect of messing up the stack traces in your developer console (they'll appear to come from inside that plugin, not directly from where you call log.error()). It's sadly not possible to both run code on demand when a logging method is called, e.g. to print the timestamp, and to also avoid that, which is why it's not available by default. There's some more detail in the README. You could look at only enabling the plugin when you know you need to save the logs though, if that's a problem for you.
Hope that helps! Note that I'm not the developer of the plugin, so you'll have to file issues over there if you have questions on exactly how it works. I'll close this for now, but happy to reopen it if you need a different approach.

Unfortunately the use of the plugin produces some errors on the command line after the npm start script of our project.
This is the error
ERROR in ./~/json-loader!./~/loglevel-message-prefix/resources/parameters-schema.json
Module build failed: SyntaxError: Unexpected token m
at Object.parse (native)
at Object.module.exports (C:\Users\andreasi\My Projects\Logan\node_modules\json-loader\index.js:7:48)
@ ./~/loglevel-message-prefix/lib/main.js 38:4-44:15
so we cannot use it for now. I'm going to address a new issue on the plugins github page.
Thank you for your time


Add describe_log_streams and filter_log_events to the CloudWatch module

Feature

Feature

Detail

Add describe_log_streams to CloudWatch module for getting log streams' details from specified log group.
Add filter_log_events to CloudWatch module for returning filtered log events from specified log group's log streams.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: 839e790704edd67a980280a81c19bbaec153480e
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: 8f8697204cc551c5eaf73810df1db10ff07e9f7a
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: 792f6e821c7129bf1c88a30de21f3a2d79d5b4ec
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

Hi @KhueNgocDang looks like we are getting a test failure on the change.
tests/test_cloudwatch.py::test_describe_log_streams_and_filter_log_events

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: 4a153221f970d918576db2ac8fa76d1e1ca0b4ff
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@malachi-constant
The test for commit id: https://github.com/aws/aws-sdk-pandas/commit/4a153221f970d918576db2ac8fa76d1e1ca0b4ff failed due to lack of policy logs:filterLogEvents in the CodeBuild's service role. Would you please add the policy to the service role?

@malachi-constant The test for commit id: 4a15322 failed due to lack of policy logs:filterLogEvents in the CodeBuild's service role. Would you please add the policy to the service role?

Hey, I've added the permission to the policy and re-ran the CodeBuild.

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: 4a153221f970d918576db2ac8fa76d1e1ca0b4ff
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: a88a3a2380bab70be92f9421f1243285e3a0588d
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: ae5ff239fd78ed2447292c2a2f810098d9703642
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: b618f6700036f911dfd31d5984023dba930ccf81
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: 5e7b568428bb558b6f5dade58e05632e7f0420d6
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: 2c33c90ba08cb385b30cb3bda0ca7913ef33ad95
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@malachi-constant @LeonLuttenberger
Thank you for helping me with this.
I encountered a bug in CloudWatch's 'test_query' while debugging the pull request in https://github.com/aws/aws-sdk-pandas/commit/5e7b568428bb558b6f5dade58e05632e7f0420d6. This did not happen to the other commits.

@malachi-constant @LeonLuttenberger Thank you for helping me with this.
I encountered a bug in CloudWatch's 'test_query' while debugging the pull request in 5e7b568. This did not happen to the other commits.

Hey,
Yes, that's a transient error in our testing suite. It's not consistent, and re-running the job usually gets rid of it. We haven't gotten around to addressing the root cause of the error yet sadly.
But, the good news is that it's not an issue with your code :)
Cheers,
Leon

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: f5838043cb36e01061bc88bed19019992dcf7495
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


feat: Holder Sign Presentation API

closes #323
Signed-off-by: Rolson Quadras<EMAIL_ADDRESS>

BDD test to follow.

@troyronda @rolsonquadras do we need new mode https://github.com/trustbloc/edge-service/blob/master/cmd/vc-rest/startcmd/start.go#L150

@troyronda @rolsonquadras do we need new mode https://github.com/trustbloc/edge-service/blob/master/cmd/vc-rest/startcmd/start.go#L150

Yes @fqutishat . I'm adding it as part of BDD test (next PR).


Command of './absolute build_client' cannot be terminated by CTRL + C

When execute './absolute build_client' , webpack is running in watch-mode.
In this case , if user would make it stop by using CTRL + C , it doesn't work currently.

@hyungheo, I'm closing this issue.


short tooth root

Add:
short tooth root
DEF: decreased length of the part of a tooth that is implanted in the gum; the root is located below the neck of the tooth, covered by cementum rather than enamel, and attached by the periodontal ligament to the alveolar bone
SYN: short dental root
SYN: short tooth roots
SYN: short dental roots
SYN: decreased length of dental root
SYN: decreased length of tooth root
HP:0006336

Added:
short tooth root MP:0030497


Testimonials structures are not adjusted properly when the web[age is enlarged or minimized.

What happened?
The names are not shown for some of the boxes i.e. the lower portion got cut.

How can we reproduce this bug?
This bug is produced whenever the webpage is enlarged.
Desktop Information (Optional)
Browser: Microsoft edge
Urgency (Optional)
Medium priority

I can solve this issue. Please assign it to me under GSSOC'23.

Hey, someone already working on the same issue


Updated Readme

Readme is updated according to guidelines.

There's a pull request template, pls use it, thank you

Can you tell me where can I find it?

Can you tell me where can I find it?

You saw some demo text while you were creating this pull request, right?
That's a template

Ohh that one! Okay sorry from my end I will update and make a new PR.

[DOCS] add readme file #7
🛠 Fixes Issue
Closes #7
👨💻 Changes proposed(required)

Added separate sections for getting started, prerequisites
Added logo of smart search on the readme.
Restructured the readme file for better understanding.

✔ Check List (Check all the applicable boxes)(required)

[ x] My code follows the code style of this project.
[ x] This PR does not contain plagiarized content.
[ x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers
Some more modifications are possible.

[DOCS] add readme file #7
🛠 Fixes Issue
Closes #7
👨💻 Changes proposed(required)
* Added separate sections for getting started, prerequisites

* Added logo of smart search on the readme.

* Restructured the readme file for better understanding.

✔ Check List (Check all the applicable boxes)(required)
* [ x] My code follows the code style of this project.

* [ x] This PR does not contain plagiarized content.

* [ x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers
Some more modifications are possible.

don't do this, you can edit the orignal message

Sorry again ! Updated my orginal comment please check it. Thank you :)

Thank you Dunsin for being patient with the changes ! Have a nice day :)

Thank you Dunsin for being patient with the changes ! Have a nice day :)

Yeah! You are welcome, tho you kind of scattered a lot of things 😭 which I have to now fix 😅
but yeah when contributing to a readme, try to look at what you have done in the preview section so you know what to fix and make look better


Add how-to-ask tag

Why include product? Wouldn’t it be better if it’s something else (for app/base/drive) or not an option?

https://github.com/deta/deta-discord-bot/pull/5/files#diff-c945fd50368c33b9052c604dc7204ffd63f5d06fc8f3728c5027b0c7a3b7893aR22

Why include product? Wouldn’t it be better if it’s something else (for app/base/drive) or not an option?

I copied the posting guidelines from the help-me channel and edited them. Product already means Space/Drive/Base/SDK/BrowserExt/etc, I can probably make it easier to understand.


Add flag for tuning in migraphx-driver

Adds --tune flag in migraphx-driver to enable MIOpen tuning.
Changes C/C++ and Python API to also support this.
~It does this by setting the environment flag MIOPEN_FIND_ENFORCE internally when compiling instead of relying on users.~
It does this by setting exhaustive-search flag in MIOpen's Find API.

Adds --tune flag in migraphx-driver to enable MIOpen tuning.

I think this flag should be called --exhaustive-search as it is more descriptive of what it is doing.

It does this by setting the environment flag MIOPEN_FIND_ENFORCE internally when compiling instead of relying on users.

We should not be setting the env variable. MIOpen caches the env variables so this can be wrong if it does two compiles with different settings in the same process.
We should be using MIOpen's API. For Find 1.0, they have a exhaustiveSearch flag that could be passed to enable tuning. For find 2.0, I believe there is the miopenSetFindOptionTuning to enable tuning in MIOpen similar to exhaustiveSearch.

I think this flag should be called --exhaustive-search as it is more descriptive of what it is doing.

I find --tune more intuitive for the users to understand. Also In the future, we may want to use this flag for rocBLAS/CK/MLIR tuning.

I find --tune more intuitive for the users to understand.

Also, we already do some tuning even without this flag. This flags enables a more comprehensive set of tunings. If you want the tune word then maybe this flag should be --exhaustive-tuning?

I find --tune more intuitive for the users to understand.

Also, we already do some tuning even without this flag. This flags enables a more comprehensive set of tunings. If you want the tune word then maybe this flag should be --exhaustive-tuning?

renamed to --exhaustive-tune

Add test case for the CI coverage. Overall looks good though

Modified API tests to set tuning flag.


GI: remove fullstack-core/fullstack-readiness-api/gradle.properties so that it will not override the version to 0.1.0-SNAPSHOT

:tada: This issue has been resolved in version 0.18.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 0.18.0 :tada:
The release is available on:

npm package (@release-0.18.x dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


ConditionalOptionalArrayResolver pass in preprocess

Introduce a new pass in preprocessing that aims to reduce the well define case of optional hinted arrays. E.g.
def program(array: Optional[]):
  if array is None:
    ...
  else:
    ...

program(None, ...)

See #1209 for full background
The PR adds a new pass and fix the optional flag on sdfg.arrays.
Test test_optional_array_inference_via_parse in tests/python_frontend/optional_array_test.py

Fixing test it seem my original logic is flawed. Type hinting an array as Optional doesn't tell us anything more than "we should check at runtime". The way parsing is setup we skip looking at the actual given argument if any hinting is setup (e.g. argtypes build by _get_type_annotations, especially line 577)
So best we can do when we have a type hint is "maybe this will go away, we have to wait for call time to know". When no type hint is given, then we can infer because we are looking at given args, but isn't that more a side effect? Having the system behaves differently when type hinted or not seems dubious at best.

Fixing test it seem my original logic is flawed. Type hinting an array as Optional doesn't tell us anything more than "we should check at runtime". The way parsing is setup we skip looking at the actual given argument if any hinting is setup (e.g. argtypes build by _get_type_annotations, especially line 577)
So best we can do when we have a type hint is "maybe this will go away, we have to wait for call time to know". When no type hint is given, then we can infer because we are looking at given args, but isn't that more a side effect? Having the system behaves differently when type hinted or not seems dubious at best.
I can write the optional inference by actually resolving the given argument but that means making the SDFG dependent on the way it's called. E.g.
@dace.program
def meh( A: Optional[dace.Float64[2]]):
 if A is None:
  ...

 meh(A) <- one SDFG
 meh(None) <- another SDFG

Isn't that problematic in the wider usage of DaCe?

I think that this is fine. In fact, @tbennun isn't the purpose of the closure to make separate SDFGs only when the arguments change in a significant way?

Seems fair, I will write the code that go down that path and we can reason on it.

Just make sure that if type hints were given, it’s AOT compilable

TL;DR: to make sure AOT based on type hints only is supported, no inference can be done, including in the OptionalArrayInference which might be a bug.
Going deeper on this, clearly the AOT & JIT clashes.
My understanding is as follows.
Given

@dace.program
def program(hinted_a: Optional[dace.float64[2]], b):
   if hinted_a is None:
      ...
   if b is None:
      ...

This should be compilable with

program.to_sdfg(A, B)
program.to_sdfg()

When down in parsing, I can't reason with given_args because of 2. Therefore I can only read the hint for the optional flag. This means that Optional=True is not an information we can infer on, since at call time we can have either an array or None in the case of 2 without triggering recompilation.
Happy to dive in this with you guys whenever you have time

The only option left is to restrict this pass to nested calls made with clear None type e.g.

def nested(A. B):
   ..

@dace.program
def program(A, B):
   nested(A, None) <- we can infer here when doing program.to_sdfg()

The only option left is to restrict this pass to nested calls made with clear None type e.g.
def nested(A. B):
   ..

@dace.program
def program(A, B):
   nested(A, None) <- we can infer here when doing program.to_sdfg()

but we don't have the information in preprocess if this is the top-level or a nested call

The underlying issue is that we just don't support the redefinition of arrays and views (the latter is the issue in your case). An experimental SDFG feature supports redefinition, and I am considering using it to support redefining views as an experiment. However, I don't have an ETA because I will probably need to spend a day on that, and it may be hard to commit before the SC deadline. I will update you.


Playback with subtitle enabled stalls at period without text AdapatationSet

Have you read the Tutorials?
Yes
Have you read the FAQ and checked for duplicate open issues?
Yes
What version of Shaka Player are you using?
v3.0.7
Please ask your question
Hi,
We found that Shaka Player will enter stall state in following scenario:

Play a live stream with multiple periods and enable subtitle
In the live stream, some periods have "text/vtt" AdapatationSet, and some periods don't have
When playback from period with "text/vtt" AdapatationSet to period with no "text/vtt" AdapatationSet, playback stall occur after playing for a few seconds since entering new period

We found that the playback stall is caused by a wait logic in streaming engine:
https://github.com/google/shaka-player/blob/master/lib/media/streaming_engine.js#L1032
The text stream stopped updating as there are no new segments in new period
However, audio and video tried to wait the text stream to catch up
The audio and video stream stopped downloading next segments and caused playback stall
There is a filter to ignore some embedded text, but this does not filter the "text/vtt" AdapatationSet
https://github.com/google/shaka-player/blob/master/lib/media/streaming_engine.js#L1020
Is this playback stall expected?
Are all the periods expected to have "text/vtt" AdapatationSet once a period contains a "text/vtt" AdapatationSet?
Thanks.

Found a duplicate issue: https://github.com/google/shaka-player/issues/2957


PyQt5 apps do not use system display preferences on Linux

Issue:
Python apps on Linux that use PyQt5 from conda-forge do not appear to inherit the display preferences set by the operating system.  Instead they use some other ("default"?) settings, which make them look different from other programs on the user's computer.  This is annoying for users of these apps.
The problem can be seen in apps such as Spyder.  Here is a screenshot of Spyder running in a conda environment.

Here is a systemwide install of Spyder (via apt-get) installed on the same system:

You can see that the menus styles, button styles, fonts, etc. are not the same.  The latter is using the settings set in the OS preferences, while the former isn't.  Those examples are from Linux mint, but various reports of the problem can be found around the internet (e.g., here, here, here and here).  Many of these suggest that the problem is that because the conda pyqt is disconnected somehow from the system pyqt, it's not getting the themes right.
Is there any way to allow installing various versions of PyQt5 in separate conda environments, while still having those versions automatically obtain and honor systemwide appearance preferences?
As far as I can tell the problem exists for all Python apps using PyQt5 from conda.  It appears to be present with pyqt in anaconda defaults channel as well, but I decided to raise an issue here in hopes some wise maintainers may have ideas about how to fix the issue.  At the least, the problem does exist for PyQt5 obtained from this feedstock, so the issue is applicable here.  Likewise I'm unsure whether this is a PyQt issue or a Qt issue but figured I'd raise it here to start.  (A related issue was recently raised in the qt feedstock.)  Please let me know if there's a better place to raise the issue.

I don't care about this matter at all. And 'fixing' it would be extremely complicated. If you want to try to 'fix' it go ahead.

In all likelyhood any attempt to fix this would create more problems than it could ever solve (non-native look is not a serious problem and not worth spending time on, IMHO).
To fix it you would need to subvert our Qt libs to read system Qt config which would work to some extent while those configs are somewhat compatible, but as soon as one Qt version is updated to make the configs incompatible, everything will break.
If you want to use a Qt that integrates better with your system, I would suggest using your system Qt.

Thanks @mingwandroid .  I realize it's not a showstopper bug but it's irritating for end users of GUI apps that are installed via conda.  One of the biggest advantages of Qt is the ability to get native-looking apps across different platforms.
What exactly is "system Qt config"?  I've been trying to find information about that but had no luck.  Is there any way that a third-party lib could be written to act as a shim to read config from system Qt (from within the conda environment) and pass it to the conda Qt?  Is this info stored in a config file anywhere?

"Qt config" isn't a proper noun here. I mean the files and content of those files that control the look and feel of Qt. They are scattered and ephemeral.
Each distro tweaks aspects of Qt, particularly around trying to make them integrate cohesively into their 'Desktop Experience'. Sometimes this involves config files and sometimes it involves patching Qt's source code (less frequently these days).
So these things are not a part of our Qt, nor can they be because they are different on each distro and they are often achieved via hacks that we should not entertain.
We don't have the luxury of being able to make our Qt work like your distro, because then it's not going to work like that on your friend's linux distro if it is different to yours.
I would honestly say that the risk of breakage from any attempt to make Qt read GUI settings from any system location (particularly since there's little likelihood the versions are even in sync) would make one of the already most difficult to maintain packages in our ecosystem tangibly worse, more difficult to maintain and more bug-prone as you are dynamically changing how our Qt behaves according to local conditions (system Qt) we are not in control of.

FWIW, to my knowledge, spyder from defaults used to have the intended behavior on Ubuntu 20.04.
However, I've configured my system to seriously prefer conda-forge so it is difficult for me to recreate.
conda has done what I used to think was impossible before. It is isn't inconceivable that we get this to work nicely :D.

i think the main issue here is that conda-forge's Qt doesn't come with support for Gtk (you can pass a flag to Qt's build system for that). We took care of that with @mingwandroid in my old Anaconda days (and it seems it's still in place, at least according to @hmaarrfk).


super.call(), super.call_deferred(), etc... do not call methods of super class

Godot version
4.0.3-stable and later
System information
Windows 11, NVIDIA GeForce GTX 1650
Issue description
super.call(), super.call_deferred(), super.call_thread_safe(), super.call_deferred_thread_group() do not call methods of super class but call extended class's. Is this an expected behavior?
Steps to reproduce
Please try the following code:
base.gd:
class_name Base
extends Node

func function():
	print("base")

extends.gd:
extends Base

func _ready():
	function() # extends
	super.function() # base
	super.call("function") # extends
	super.call_deferred("function") # extends
	super.call_thread_safe("function") # extends
	super.call_deferred_thread_group("function") # extends

func function():
	print("extends")

Minimal reproduction project
N/A

These methods call the most derived type, and it wouldn't be possible to change this I'd say, because it is how inheritance is implemented

You can't override native non-virtual methods, so super is not needed here, super.call_deferred("function") is the same as call_deferred("function"), it doesn't mean "call the function of the base class deferred".
Try super.function.call_deferred() instead (call_deferred() is a Callable method).

Internally all that call_deferred does is use call, so behavior would be equivalent
Try with a lambda:
var fun = func():
    super.function()
fun.call_deferred()


[Feature Request]: Top loading bar animation for page transitions

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
Adding a top-loading bar animation can significantly enhance user experience by indicating when a page is loading. This is particularly useful for pages that involve server-side rendering or client-side rendering with preprocessing, where changing the URL path might take some time without any visual feedback. Implementing a top-loading animation bar will ensure smoother page transitions and provide users with clear indicators of loading activity, thereby improving overall user experience.
Use Case
For better page transition and user experience
Benefits
Benefits:

Clear indication of page loading status
Enhanced perceived performance
Smoother page transitions

Add ScreenShots
https://github.com/ashutosh-rath02/git-re/assets/65452005/9bbf6d69-f23d-45cc-9ed8-aceecec03274
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

Thank you for creating this issue! 🎉 We'll look into it as soon as possible. In the meantime, please make sure to provide all the necessary details and context. If you have any questions or additional information, feel free to add them here. Your contributions are highly appreciated! 😊
You can also check our CONTRIBUTING.md for guidelines on contributing to this project.

Are you asking for a progress bar @dhairyathedev ?

Are you asking for a progress bar @dhairyathedev ?

Yes, but on top of the page.

How about making a progress loader at the middle of the screen? This loader can be used as a template for all such loading processes?

Okay Ashutosh
I suggest, for page transitions, a top loader is ideal as it doesn't hide any content and smoothly shows the progress. Using a middle loader would require additional components to manage loading states, potentially leading to performance issues and poor user experience as changing states very frequently. If you prefer, we can use a middle loader where data is fetched from the client end, not the server end, to avoid these complications.
One example is GitHub itself using the top loading animation,
https://github.com/ashutosh-rath02/git-re/assets/65452005/95578f83-2ffe-4b45-9bbf-20c97047023a
If you still want to add the middle loader, I'll need to create a custom hook for it, and I'll need to do some research to make that happen.

Performance is one of our concerns too. It's taking too much time to load the resume after entering the username. You can give this a try.

Okay, I will try.
Also, can you assign this issue to me?

Hello @dhairyathedev! Your issue #264 has been closed. Thank you for your contribution!


Nextcloud Backup Fails

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

This issue is not new, but has been closed a few different times by timeout.
Describe the bug
opnsense/core#4232 seems to be what I'm facing. Also reported in opnsense/core#4337

{"url":"https:\/\/nextcloud.mydomain.org\/ocs\/v1.php\/cloud\/user","content_type":null,"http_code":0,"header_size":0,"request_size":0,"filetime":-1,"ssl_verify_result":10,"redirect_count":0,"total_time":0.016409,"namelookup_time":0.00349,"connect_time":0.004077,"pretransfer_time":0,"size_upload":0,"size_download":0,"speed_download":0,"speed_upload":0,"download_content_length":-1,"upload_content_length":-1,"starttransfer_time":0,"redirect_time":0,"redirect_url":"","primary_ip":"<IP_ADDRESS>","certinfo":[],"primary_port":443,"local_ip":"<IP_ADDRESS>","local_port":1448,"http_version":0,"protocol":2,"ssl_verifyresult":0,"scheme":"HTTPS","appconnect_time_us":0,"connect_time_us":4077,"namelookup_time_us":3490,"pretransfer_time_us":0,"redirect_time_us":0,"starttransfer_time_us":0,"total_time_us":16409}

Then...
Cannot get real username

To Reproduce
Steps to reproduce the behavior:

Go to Configuration > Backups
Configure Nextcloud Backup:

Test backup
See error

Expected behavior
Backup configuration should be saved on Nextcloud
Environment
OPNsense 21.7.3_3-amd64
FreeBSD 12.1-RELEASE-p20-HBSD
OpenSSL 1.1.1l 24 Aug 2021

You have an TLS Error (verify code 10 - certificate expired).
This is not a bug. This is intentional as it should not connect to a server with an invalid certificate. Fix your webserver.

I can confirm that it was a user error on my part. I was trying to backup to the wrong nextcloud path (that didn’t exist).

I think I have a similar issue (though I don't appear to have the TLS issue). I tried adding the directory in advance too and no luck. How did you structure your path?

This is my path, and the IT directory is on the root of my Nextcloud Files.


Multus not working with Deployment file

Hi, I'm read and exampled with your source. I use Deployment file to create pod. But my pod run with nothing additional interface.
This is my pod and it's interfaces

deployment file

dmesg of node

You need to add annotation in 'template.metadata' section of Deployment, not metadata of Deployment because multus parse annotation in Pod, not in Deployment.

Hi @s1061123 , Thank you for watching my issue. I tested with your suggestion. I see annotation on pod but don't see interface. As below:

Thank you, @s1061123 . My problem is solved.


Describing an overridden but incompatible method signature in an ambient class

From SO: http://stackoverflow.com/questions/34087631
/** Represents an iterable collection of Pnp device objects. */
abstract class PnpObjectCollection extends Array {
    /** Returns the iterator for iteration over the items in the collection. */
    first(): Windows.Foundation.Collections.IIterator<Windows.Devices.Enumeration.Pnp.PnpObject>;
    /** Returns the PnpObject located at the specified index. */
    getAt(index: number): Windows.Devices.Enumeration.Pnp.PnpObject;
    /** Retrieves multiple elements in a single pass through the iterator. */
    getMany(startIndex: number): { /** Provides the destination for the result. Size the initial array size as a "capacity" in order to specify how many results should be retrieved. */ items: Windows.Devices.Enumeration.Pnp.PnpObject; /** The number of items retrieved. */ returnValue: number; };
    /** Retrieves the index of the specified item. */
    indexOf(value: Windows.Devices.Enumeration.Pnp.PnpObject): { /** The index of the item to find, if found. */ index: number; /** True if an item with the specified value was found; otherwise, False. */ returnValue: boolean; };
    /** Returns the number of items in the collection. */
    size: number;
}

This is the current generated code by expressing UWP PnpObjectCollection. PnpObjectCollection.prototype.__proto__ === Array is true here so I added extends Array but as you see the signature of indexOf method conflicts.
TS2415	Class 'PnpObjectCollection' incorrectly extends base class 'any[]'.
  Types of property 'indexOf' are incompatible.
    Type '(value: PnpObject) => { index: number; returnValue: boolean; }' is not assignable to type '(searchElement: any, fromIndex?: number) => number'.
      Type '{ index: number; returnValue: boolean; }' is not assignable to type 'number'.

30+ classes in entire d.ts file have this problem, so how can I express this class without errors? I cannot add indexOf(value: T, fromIndex: number): number line because calling indexof(value, 0) returns an object, not a number.

Current winrt.d.ts in DefinitelyTyped have manually added ES5 compatible Array methods on IVectorView interface but that will:

force the class declarations to have all the interface members copied
make them lack ES6 methods (adding ES6 ones will break ES5 compatibility)

I've answered on http://stackoverflow.com/a/34299582/4386952.
In general, it seems strange that the library authors disregarded the implementation - but that's the thing. We require all derived signatures to be compatible in an ambient class even if a library author is doing shenanigans, and we should discuss whether that should be the case.

@DanielRosenwasser It helps me a lot, thanks! :+1:
My thought:
abstract class PnpObjectCollection extends Array<PnpObject> {
    /* suppress signature confliction error */
    override indexOf(value: Windows.Devices.Enumeration.Pnp.PnpObject): { index: number; returnValue: boolean; };
}
let collection: PnpObjectCollection;
let ary: Array<PnpObject>
ary = collection; // error
collection = ary; // error

Answer on SO is generally correct, as well as the observation that you don't really need to extend Array in an ambient class except to save typing (which is sort of a red herring since the type is nonsubstitutable anyway so you should be checking each method individually)

@RyanCavanaugh I think I need to extend Array as the class have Array in its prototype chain and the declaration should not require users to manually add those new ES6 methods, no?


change: react favicon to t-picker favicon

Fix : #https://github.com/Binary-Cumates/T-Picker-Frontend/issues/13

@jesus7110 pr is completed. by uttkarsh.


SSLv3 Downgrade Error

I was testing the script and got the following error.
TLSv1.0  downgrade
Sending handshake failure

Exception happened during processing of request from ('<IP_ADDRESS>', 58339)
Traceback (most recent call last):
  File "/usr/lib/python3.6/socketserver.py", line 317, in _handle_request_noblock
    self.process_request(request, client_address)
  File "/usr/lib/python3.6/socketserver.py", line 348, in process_request
    self.finish_request(request, client_address)
  File "/usr/lib/python3.6/socketserver.py", line 361, in finish_request
    self.RequestHandlerClass(request, client_address, self)
  File "/usr/lib/python3.6/socketserver.py", line 696, in __init__
    self.handle()
  File "poodle-exploit.py", line 203, in handle
    ssl_header = self.request.recv(5)
ConnectionResetError: [Errno 104] Connection reset by peer

Exception happened during processing of request from ('<IP_ADDRESS>', 38795)
Traceback (most recent call last):
  File "/usr/lib/python3.6/socketserver.py", line 317, in _handle_request_noblock
    self.process_request(request, client_address)
  File "/usr/lib/python3.6/socketserver.py", line 348, in process_request
    self.finish_request(request, client_address)
  File "/usr/lib/python3.6/socketserver.py", line 361, in finish_request
    self.RequestHandlerClass(request, client_address, self)
  File "/usr/lib/python3.6/socketserver.py", line 696, in __init__
    self.handle()
  File "poodle-exploit.py", line 211, in handle
    traffic.protocol_current = traffic.protocol_all[version][0]
KeyError: 29295

Exception happened during processing of request from ('<IP_ADDRESS>', 38795)
Traceback (most recent call last):
File "/usr/lib/python3.6/socketserver.py", line 317, in _handle_request_noblock
self.process_request(request, client_address)
File "/usr/lib/python3.6/socketserver.py", line 348, in process_request
self.finish_request(request, client_address)
File "/usr/lib/python3.6/socketserver.py", line 361, in finish_request
self.RequestHandlerClass(request, client_address, self)
File "/usr/lib/python3.6/socketserver.py", line 696, in init
self.handle()
File "poodle-exploit.py", line 211, in handle
traffic.protocol_current = traffic.protocol_all[version][0]
KeyError: 29295

Hello,
Which client did you use ? (firefox, chrome, opera, ie)

Okay, I think you try on Safari ! I find a bug and will push it at the end of the day

I was using Android device 4.4.2

can you try with code pushed on dev branch ?


Fix optimistic lock

Pipeline along with watch is not sufficient to provide unique task to workers, since its an optimistic lock and doesn't guarantee. That means its possible that more than one worker gets the same task.
To avoid above problem, a special lock and release functions are added to achieve mutex functionality. Lock is implemented using key value, key->lockname and value->workerID
Lock is also automatically released when it times out, in case the worker crashes or dead.
changes:

acquire_lock
release lock
get

Good catch! thanks a lot


KeepassXC persist not working

KeepassXC has keepassxc.ini set as a persisted file. However, when upgrading, the file is replaced with the new version and all settings are back to default.

From which version did you upgrade?
I upgraded just now from version 2.2.1 to 2.2.2 and it worked like it should.

I upgraded the same version. The same thing happened the last time I upgraded, although I'm not sure when the persist was added in relation to the last release.
While attempting to recreate this issue, I've run into some new problems that may or may not be related.
I just tried uninstalling keepassxc and deleting the persist/keepassxc directory. After installing keepassxc again, I now see a directory under persist called keepass.ini' with nothing in it, instead of the actual keepassxc.inifile. I also see a linked directory in the actual currentkeepassxcdirectory with the same name. It looks like keepassxc does not create the configuration file until after the first run. When I launch the program I seeerror accessing config file. After deleting current\keepass.ini` and launching the program again, the keepass.ini file is created.
So it looks like if the persisted file is not found at install time, scoop is actually creating a directory with the same name.
I'm using the latest version of scoop.

I hope this fixes the problem: https://github.com/lukesampson/scoop/commit/c15d8e2393112ed595c250bda04b9fc16b1b9970
Scoop now tries to detect if it's a file or a directory before just creating a new empty directory.

I tried uninstall and installing again - this time I received the following message. This was with nothing in the persist directory.
Persisting keepassxc.ini
The system cannot find the file specified.

The second test I attempted was to uninstall, manually edit the current version to a lower number then install again. I received the same message as above. I then manually copied over keepassxc.ini to the current folder, updated the current version and ran scoop update keepassxc. This resulted in the same error again. The persist/keepassxc directory is still empty and the keepassxc/current directory does not contain keepassxc.ini although I can still see it in the previous version folder.
So it looks like your update prevented a bad file from being created. However, it did not correct the issue where files not created until first launch are not being persisted.
It looks like scoop runs the persist operation at install time but does not check again at update or uninstall time when the file is actually needed.

Do you mean you lowered the version by changing the directory name? If so, this will break the symlink to the current directory and scoop is unable to find the file.
There is no real way to downgrade a version.
But for you could try to replicate the update behavior like so:

scoop uninstall keepassxc
delete keepassxc.ini from perist directory
scoop install<EMAIL_ADDRESS>change the content of the ini file
scoop install<EMAIL_ADDRESS>validate the content of the ini file

This keepassxc installation can not be updated with scoop update keepassxc, so you have to reinstall it normally again.
The logic for persistence can be found in the persist_data() function.

No,
I lowered the version by editing the keepassxc.json file in scoop/buckets/extras and doing a scoop install. Then I reverted the manifest back to the current version and did a scoop update.

The ini file is missing in the 2.2.2 release: https://github.com/keepassxreboot/keepassxc/issues/1134

It should now work as intended.


[Enhancement]: Enhance Game Brick Buster

Do you want to have the enhancement of existing game ? 😀 Describe yourself..
Game enhancement
Enhance ui ux of game, add more functionality, improve bricks and ball speed
Describe the solution you'd like
Solution steps
I would like to update css and js file of the game
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @ananyag309 !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

Hey @ananyag309 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Introduction: Fixes missing version increase

Minor documentation fix.

Actually, @cstuder can you update README.md too please?

It seems we have more places:
Search "bootstrap-4" (8 hits in 4 files)
  E:\Downloads\Internet Downloads\bootstrap-main\bootstrap-main\.github\CONTRIBUTING.md (2 hits)
	Line 22:   Overflow ([`bootstrap-4`](https://stackoverflow.com/questions/tagged/bootstrap-4) tag),
	Line 22:   Overflow ([`bootstrap-4`](https://stackoverflow.com/questions/tagged/bootstrap-4) tag),
  E:\Downloads\Internet Downloads\bootstrap-main\bootstrap-main\.github\SUPPORT.md (2 hits)
	Line 11: - Ask and explore Stack Overflow with the [`bootstrap-4`](https://stackoverflow.com/questions/tagged/bootstrap-4) tag.
	Line 11: - Ask and explore Stack Overflow with the [`bootstrap-4`](https://stackoverflow.com/questions/tagged/bootstrap-4) tag.
  E:\Downloads\Internet Downloads\bootstrap-main\bootstrap-main\site\content\docs\5.0\getting-started\introduction.md (2 hits)
	Line 125: - Implementation help may be found at Stack Overflow (tagged [`bootstrap-4`](https://stackoverflow.com/questions/tagged/bootstrap-4)).
	Line 125: - Implementation help may be found at Stack Overflow (tagged [`bootstrap-4`](https://stackoverflow.com/questions/tagged/bootstrap-4)).
  E:\Downloads\Internet Downloads\bootstrap-main\bootstrap-main\README.md (2 hits)
	Line 164: - Implementation help may be found at Stack Overflow (tagged [`bootstrap-4`](https://stackoverflow.com/questions/tagged/bootstrap-4)).
	Line 164: - Implementation help may be found at Stack Overflow (tagged [`bootstrap-4`](https://stackoverflow.com/questions/tagged/bootstrap-4)).

Will do, didn't think to check other files too.

@XhmikosR Done. Please re-review.


[warning]Unexpected input 'time', valid inputs are ['']

Using the suggested syntax from the README this action works great (as expected), but GitHub Actions logs always reports a warning:
##[warning]Unexpected input 'time', valid inputs are ['']
Looking around online, it appears this may be a new behavior from Github Actions while it's in beta.
ref: https://github.com/samuelmeuli/lint-action/issues/53
Excerpt from 17 hours ago as of this writing: "I've just gotten the same thing pop up, I was testing this last week and wasn't hitting it, all of a sudden it has appeared."
Not sure what the fix is yet, but it seems likely Github Actions beta will announce a way to specify expected input values, probably resolving this.

Same problem here:

This action needs an action.yaml and input metadata

Still getting the issue
Warning: Unexpected input(s) 'time', valid inputs are ['entryPoint', 'args']


Allow user to specify download directory

This change allows the user to specify the download directory in the config file. For example, we can set the download directory to ~/Music/.
If a download directory is not specified, the playlist will default to downloading the playlist to the spodl directory.

Looks good now :)


OutOfOrder migration detection options

Is there a mechanism for hooking into the out-of-order migration detection? instead of ignoring OoO migrations or executing them (with the outOfOrder=true flag) I'd like to have an exception thrown / halt the application at startup. Alternatively, I'd like to execute an arbitrary callback.

I don't think we have a mechanism for hooking into that.
If outOfOrder is set to false (which is the default), Flyway will fail when running a migrate.
What's your need for the failure to happen at startup? Could you tell us more about your requirements?

For the time being you can use Java-based callbacks by extending BaseCallback and hooking into beforeEachMigrate
This gives you access to the MigrationInfo for the next migration. You can use the version from this compared against the current version of your DB to determine if a migration is being executed out of order
For now we will be closing until further interest

Sorry, I had some issues with my Github account.
I should have mentioned the project is still using Flyway 4.2.  As of that version, if OutOfOrder is disabled, and I introduce an out-of-order migration, it just seems to be ignored. No exceptions are thrown and the app starts up normally.
(note: I have ValidateOnMigrate disabled, but as I understand it, that just compares the checksums of available/executed migrations)
I tried adding a beforeEachMigrate Java callback, but that doesn't seem to get called in the context of out-of-order migrations assuming OutOfOrder is disabled. It is only called for normal, in-order migrations, or out-of-order migrations when OutOfOrder is enabled.
Ultimately, the goal is just to detect out-of-order migrations being introduced while OutOfOrder mode is disabled and throw an exception.

created new issue here so this one can remain closed:
https://github.com/flyway/flyway/issues/3353


feat(orderBy): Stable sort the input

Stable sort the input array

This indirectly fixes #12405

Does this have a performance implications?

Does this have a performance implications?

This only adds a natural comparator with the original element position, so
this adds some cycles but much smaller than even one $parse evaluation

This only adds a natural comparator with the original element position, so
this adds some cycles but much smaller than even one $parse evaluation

It might still be a good idea to look at the impact this has on the bigtable benchmarks, and put some numbers in the commit message and issue --- so that there's a bit more confidence there. Can we do that?

Run order_by benchpress several times with and without the patch
With the patch
mean: 152.74ms ± 22%
(stddev 33.61)
Without the patch
mean: 156.36ms ± 32%
(stddev 53.76)
This PR does not make the code any faster (as the numbers look like they claim), but do show that the variations between runs is much bigger than whatever performance penalty this patch adds

Can you add to the commit message that it fixes #12405? Otherwise LGTM


DemoModal renders twice

When you open the demo modal the markup renders twice in the dom. This means that if you use a form within the modal you will have duplicate id's and causes accessibility issues with the form.

Okay this should be fixed now! Thanks for the heads-up!


examples in chinese

What's up with examples?   They are not visible in code view. When downloaded they are not visible in geany editor and in Windows notepad shows Chinese characters!

Hi, the examples are in the tokenised BBC BASIC format so are not readable with a text editor. This folder is included as-is as a stipulation of the license with R.T.Russell. They will load in BBC BASIC fine though.


Reset config does not really reset config

Add a configurable mod.
Reset config to create .conf
Hand edit config.
Reset config.  Config is not reset.

Fixed in commit ae668118c9d2f2c4934a1edc1c505848aa74a3f2 and release with 1.0.


Binding several views to a Listener with multiple methods in Butterknife

Hey, it seems like it was once possible to bind a method to multiple views at once. Like I have three EditViews and each one should listen on text change events. It looks like it is not possible to determine which EditView was changed.
A good example is this issue: http://stackoverflow.com/questions/40963739/binding-several-views-to-a-listener-with-multiple-methods-in-butterknife

There's a test for this: https://github.com/JakeWharton/butterknife/blob/fc10f1674a08162ad7cf8c762c11a748c20c25d8/butterknife/src/test/java/butterknife/OnClickTest.java#L118. If you think it's broken please provide a failing test case or full sample.


[sentry-native] Build error when building x64-windows on arm64 host

Host Environment

OS: Windows 11 on Apple M1 (Parallels)
Compiler: 17.3 Preview

To Reproduce
Steps to reproduce the behavior:

Ensure you have an Arm64 device with Windows 11.
Uninstall any prior versions of Visual Studio 2022 from your Arm64 device.
Download and install Visual Studio 2022 17.3 Preview 2 with arm64 and x64 build tools.
Install sentry-native for x64 ./vcpkg install sentry-native:x64-windows

Failure logs
FAILED: crashpad_build/util/CMakeFiles/crashpad_util.dir/misc/capture_context_win.asm.obj ^M
ml64 -DCRASHPAD_LSS_SOURCE_EMBEDDED -DCRASHPAD_ZLIB_SOURCE_SYSTEM -DNOMINMAX -DUNICODE -DWIN32_LEAN_AND_MEAN -DZLIB_CONST -D_CRT_SECURE_NO_WARNINGS -D_HAS_EXCEPTIONS=0 -D_UNICODE -IC:\Users\glebignatev\vcpkg\buildtrees\sentry-native\src\y-native-0-bf71db22ea.clean\external\crashpad -IC:\Users\glebignatev\vcpkg\buildtrees\sentry-native\src\y-native-0-bf71db22ea.clean\external\crashpad\third_party\mini_chromium\mini_chromium -IC:\Users\glebignatev\vcpkg\buildtrees\sentry-native\src\y-native-0-bf71db22ea.clean\external\crashpad\compat\win -IC:\Users\glebignatev\vcpkg\buildtrees\sentry-native\src\y-native-0-bf71db22ea.clean\external\crashpad\third_party\mini_chromium -IC:\Users\glebignatev\vcpkg\installed\x64-windows\include /wd4201 -c -Fo crashpad_build\util\CMakeFiles\crashpad_util.dir\misc\capture_context_win.asm.obj C:\Users\glebignatev\vcpkg\buildtrees\sentry-native\src\y-native-0-bf71db22ea.clean\external\crashpad\util\misc\capture_context_win.asm^M
CreateProcess failed: The system cannot find the file specified.

ninja: fatal: ReadFile: The handle is invalid.

install-x64-windows-dbg-out.log
install-x64-windows-dbg-err.log
Additional contenxt
I've successfully built sentry-native manually with cmake in x64 Native Tools Command Prompt for VS 2022 Preview.

Please try to use below command:
./vcpkg install sentry-native:x64-windows --host-triplet=arm64-windows

Thanks for posting this issue. Please reopen this issue if this is still a problem for you.


feat(tutorial): complete step 4, Boeblingen

Closes #
{{short description}}
Changelog
New

{{new thing}}

Changed

{{change thing}}

Removed

{{removed thing}}

@FSoprano Looks fantastic 🔝! I've recorded your completion of step 4.


ImportError

I have installed nflgames using pip on python 2.7. If I try and run just
import nflgames I get this error:
DLL load failed: %1 is not a valid Win32 application
Any help would be appreciated.
Thanks

I think this an issue with your path  Go into the parent directory of nflgame.  This should be 'site-packages'.  Then try your python command line from there.  If it loads the module then your path is not defined.  I'm not a windows guy but I think this is how you add it.  http://stackoverflow.com/questions/6318156/adding-python-path-on-windows-7


use SemVer

Please plan on using Semantic Versioning to keep versioning sane

Agreed

Agreed. The question is if we should treat all the current API additions as a jump of the major version or the minor version. Because I'd merge the minor and patch part into one, to keep the version short.

I'd treat them as minor bumps, until the API could be considered stable


Add more load hooks to Rails/ActiveSupportOnLoad

Mostly from Rails 7.1, except for active_record_sqlite3adapter which has been around for longer but wasn't documented until recently.
Reference list: https://guides.rubyonrails.org/engines.html#available-load-hooks
Rails PRs:

ActiveRecord::TestFixtures/active_record_fixtures: https://github.com/rails/rails/pull/47690
ActiveModel::Model/active_model: https://github.com/rails/rails/pull/48698
ActionText::EncryptedRichText/action_text_encrypted_rich_text: https://github.com/rails/rails/pull/47737 7.1
ActiveRecord::ConnectionAdapters::PostgreSQLAdapter/active_record_postgresqladapter: https://github.com/rails/rails/pull/44601
ActiveRecord::ConnectionAdapters::Mysql2Adapter/active_record_mysql2adapter: https://github.com/rails/rails/pull/48012
ActiveRecord::ConnectionAdapters::TrilogyAdapter/active_record_trilogyadapter: https://github.com/rails/rails/pull/48175

For active_record_sqlite3adapter:
Functionality introduced with https://github.com/rails/rails/pull/29699, docs only added with https://github.com/rails/rails/pull/48012

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.
[ ] If this is a new cop, consider making a corresponding update to the Rails Style Guide.

Thanks!


Stylesheet - loading order problem

Describe the bug
Layout was forced before the page was fully loaded. If stylesheets are not yet loaded this may cause a flash of unstyled content.
To Reproduce
When switching sites to fast from one to another this error is in the console
Expected behavior
No error message
Additional context
Seems to a problem when one site is loading and then switching to another one

With change to calculation on server solved


Remove version: foxglove.studio version 1.27.0

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/foxglove/studio/releases/download/v1.27.0/foxglove-studio-1.27.0-win-arm64.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 720)


tools: Move most logic from detect-pr-changes.sh to ReleaseManager

This should make it much easier to tweak, e.g. to handle different TFMs being involved.

(It looks like our comparison tools don't spot constants - I've raised b/330680714 for that.)

(Obviously I'll remove the second commit before merging.)

(It looks like our comparison tools don't spot constants - I've raised b/330680714 for that.)

Nice, I just closed an issue I had created for looking into that.


[WIP] Edit Checkout Shipping Components

fix the name of a foundation shipping action
(cherry picked from commit d3aee3136a4c3c40b7d7fbb4ed6c3093bad4ef12)
PR Checklist
Please check if your PR fulfills the following requirements:

[X] The commit message follows our guidelines: https://github.com/graycoreio/daffodil/blob/master/CONTRIBUTING.md#commit
[X] 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?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[X] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?
[ ] Yes
[X] No

Other information

Lgtm


Where is A/B test, html editor, etc  in new marketing email designer

Hi, after last update there are big changes in marketing email designer. You should also update documentation (instructions).
For example it is unclear how can swith to html email editor. Where you can find A /B test, etc.
Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: eb43cf90-2fa4-8cf5-ee42-b04990b31b08
Version Independent ID: 74ecb3a3-f792-5536-0424-dbe27d0d6741
Content: Email A/B testing (Dynamics 365 Marketing)
Content Source: ce/marketing/email-a-b-testing.md
Service: dynamics-365-marketing
GitHub Login: @alfergus
Microsoft Alias: alfergus

Hi -- we are currently in the process of updating our documentation to reflect the new email editor.
In the meantime, you can switch to the HTML editor by selecting the code toggle in the upper right:

The Early Access editor does not currently have A/B test send functionality, so you will need to use the old editor for that for the time being.

It's 28 months later, and this functionality does not appear in the RTM design editor. Please remove this article until the feature is available.


App stucks on "Retrieving deployment details..." after drag&drop deployment.

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

App Id
d2outlzd2gbtvh
Region
eu-central-1
Amplify Console feature
Manual deploys
Describe the bug
Drag&drop deployment works well right after creating new app but stucks forever on message "Retrieving deployment details..." after some number of deployments
Expected behavior
Successfully completed deployment
Reproduction steps
In my case I just create an app then update it about 10 times by drag&dropping zip-build (or folder) then it stuck
Build Settings
No response
Additional information
No response

Hi @zuhaib10 👋 , thanks for reaching out to us.
Please refer to this FAQ which outlines some potential reasons why manual deployments using the drag and drop method can fail or get stuck in the Amplify Console.
To unblock your current deployments, please run the stop-job CLI command using the AWS CLI for each frozen deployment:
aws amplify stop-job --app-id {VALUE} --branch-name {VALUE} --job-id {VALUE}
This CLI command will cancel the pending jobs and will enable you to retry a new manual deployment. Once this is done, can you then try to upload your artifact zip file to a S3 bucket and deploy it using this method? Please let us know how it goes.

@zuhaib10 These are the statuses associated to a build job: https://docs.aws.amazon.com/amplify/latest/APIReference/API_JobSummary.html#amplify-Type-JobSummary-status
After you run the stop-job CLI command: aws amplify stop-job --app-id {VALUE} --branch-name {VALUE} --job-id {VALUE}
You can then run get-job CLI command to confirm if the job status has changed to CANCELLED: aws amplify get-job --app-id {VALUE} --branch-name {VALUE} --job-id {VALUE}

@Jay2113 in the get-job CLI command it does show  status = CANCELLED


merge Radhika to main

Codecov Report

Merging #46 (b2919cd) into main (11fb531) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #46   +/-   ##
=======================================
  Coverage   96.15%   96.15%
=======================================
  Files           1        1
  Lines          26       26
=======================================
  Hits           25       25
  Misses          1        1

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 11fb531...b2919cd. Read the comment docs.


Issue when running interact script

Hi there,
I am trying to run the interact script and am getting the following behaviour:
(logenv) C:\Users\Harry\source\repos\HuggingFace>python interact.py
2021-06-27 23:37:29.687306: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library cudart64_101.dll
INFO:interact.py:Namespace(dataset_cache='./dataset_cache', dataset_path='', device='cpu', max_history=2, max_length=20, min_length=1, model='openai-gpt', model_checkpoint='', no_sample=False, seed=0, temperature=0.7, top_k=0, top_p=0.9)
INFO:C:\Users\Harry\source\repos\HuggingFace\utils.py:extracting archive file C:\Users\Harry/.cache\huggingface\transformers\2f5114b5eb72f9515802779c42c1b289bebdb1cfc8ce94c653237518eb530b34.75f2a4fe69178ff43138117a977e107a5fc7d402603a0825a296b531f246b3f2 to temp dir C:\Users\Harry\AppData\Local\Temp\tmpd181v50y
INFO:interact.py:Get pretrained model and tokenizer
ftfy or spacy is not installed using BERT BasicTokenizer instead of SpaCy & ftfy.
Some weights of the model checkpoint at C:\Users\Harry\AppData\Local\Temp\tmpd181v50y were not used when initializing OpenAIGPTLMHeadModel: ['multiple_choice_head.summary.bias', 'multiple_choice_head.summary.weight']
- This IS expected if you are initializing OpenAIGPTLMHeadModel from the checkpoint of a model trained on another task or with another architecture (e.g. initializing a BertForSequenceClassification model from a BertForPreTraining model).
- This IS NOT expected if you are initializing OpenAIGPTLMHeadModel from the checkpoint of a model that you expect to be exactly identical (initializing a BertForSequenceClassification model from a BertForSequenceClassification model).
INFO:interact.py:Sample a personality
INFO:C:\Users\Harry\source\repos\HuggingFace\utils.py:Load tokenized dataset from cache at ./dataset_cache_OpenAIGPTTokenizer
INFO:interact.py:Selected personality: i never have had alcohol in my life. i have a girlfriend of 7 years. i watch every football game at alabama. i currently suffer from social anxiety. i'm a geology major at alabama university.
>>> hey there
Traceback (most recent call last):
  File "interact.py", line 154, in <module>
    run()
  File "interact.py", line 146, in run
    out_ids = sample_sequence(personality, history, tokenizer, model, args)
  File "interact.py", line 72, in sample_sequence
    logits = logits[0, -1, :] / args.temperature
  File "C:\Users\Harry\Anaconda3\envs\logenv\lib\site-packages\transformers\file_utils.py", line 1808, in __getitem__
    return self.to_tuple()[k]
TypeError: tuple indices must be integers or slices, not tuple

(logenv) C:\Users\Harry\source\repos\HuggingFace>

Any ideas on a fix?
Thanks

Fix was to downgrade transformers version to that indicated in readme


Remove Fetch API

Replace Fetch with XHR

Hi, can you please tell what's the reason to remove Fetch? And what's the reason to replace it with XHR, and not something like Axios?

Hi, can you please tell what's the reason to remove Fetch? And what's the reason to replace it with XHR, and not something like Axios?

Fetch does not work in older browsers and requires polyfilling, while XHR is old enough and works everywhere. By rewriting Fetch to XHR, we will get rid of polyfills and will load less javascript code.
Axios is overkill here. Why do we need to load extra 4.4kb of javascript for a couple of requests?

I understand your point, but XHR isn't available in the nodejs environment.
How about using a fetch polyfill?

I understand your point, but XHR isn't available in the nodejs environment.
How about using a fetch polyfill?

As far as I know, there is neither fetch nor XHR in node.js. You will have to load the polyfill anyway.
But if we replace fetch with XHR, we will help the user of the client JS. Let's save them from polyfills.

Ohh ok thank you; so for replacing fetch with XHR, we'll have to update the tests as well because they're using fetch mocks for testing, correct?

Yes, there is a lot of work. Do you want to help? 😊

Already working on it :)

Do we need to update the tests though? Tests are supposed to test the functionality of the system, by replacing fetch with XHR, we aren't changing any functionality of the system so I think we can keep the existing test suite.

Here's the progress:
https://github.com/yusufkhan07/nanogram.js/commits/feature/xhr-migration
I'll make a pull request soon, let me know if something can be improved :)

Running npm run build:all is fine but when I run npm run build, all the docs are updated and your git username is replaced with mine. Is this the desired behaviour, or shall I either commit the dist files generated with npm run build:all or should I just excluded the docs folder from the commit?

Running npm run build:all is fine but when I run npm run build, all the docs are updated and your git username is replaced with mine. Is this the desired behaviour, or shall I either commit the dist files generated with npm run build:all or should I just excluded the docs folder from the commit?

Better to exclude. I will rebuild myself


Add batch processing to parseJSONL

WIP
Solved the biggest contributor to blocking on controller init. 2nd biggest contributor loadData not yet modified.
Intends to fix #21
Could also explore the use of web workers as an alternative solution to the batching solution used below. Pros and cons of each strategy should be considered.

I think we would be better off multi threading grafer. This approach adds too much complexity to the code and I have the feeling that debugging is going to be even harder,

I think we would be better off multi threading grafer. This approach adds too much complexity to the code and I have the feeling that debugging is going to be even harder,

Yup I agree, I have been investigating how to put together a framework which will allow multithreading using web workers.

Yup I agree, I have been investigating how to put together a framework which will allow multithreading using web workers.


Can't get the Emote Example to work

Is there any example yet how to get the emotes to work for example in a RichTextBox?
I would like to use them but the example (API Tester) does not work for me. :/
Cheers

In a richtextbox? That can be tricky. There are two routes. One is that you can try to build a custom control that allows you to insert images in. This can be quite difficult if you don't have experience with custom controls in .NET. The second route is much easier using WPF. My friend SkyHuk is working on a twitch WPF client that has managed to insert badges and emotes in his form. You can see his project here: https://github.com/SkyHuk/Purple

Ah nice. WPF is exactly what I need. I'll take a look how he managed to get this done. 👍
Thanks (as always) for your help.

Will close this :)


Zuchka/add vale action

This PR adds a very lightweight version of the Vale Github Action to the docs repo. A few things to note:

this is just an initial setup. It will only run about four tests. Three tests come from the built-in Vale package and those check for pretty uncontroversial things: spelling errors, repeated words, stuff like that. The other one is a readability test that will output a readability score for the each page. I see this as a small foundation for building out our own house style
I've also started a 'Vocab' section. It's like an allow-list of words the spell-checker won't flag. This is also the beginning of our customizations and it's totally open to revisions
the GH action is informative only and totally non-blocking. It doesn't stop anything from getting merged--it only provides helpful feedback.
I included a small section in the readme on how you can set up the repo and Vale to run your own tests locally.

I'm excited to start building this out!

might be best to wait on this until Melissa merges her overhaul...

SO COOOOL.  Agree on maybe waiting to merge until mine goes.. or maybe only merge the base of the Vale set up and not the fixes?  or maybe it doesn't matter since I haven't made all of the big organization changes yet and could probably merge main into mine with minor conflict. I'm fine either way and excited to get this in

SO COOOOL. Agree on maybe waiting to merge until mine goes.. or maybe only merge the base of the Vale set up and not the fixes? or maybe it doesn't matter since I haven't made all of the big organization changes yet and could probably merge main into mine with minor conflict. I'm fine either way and excited to get this in

@melissa-hale agree! I will merge the GH action itself but not any of the edits. Once your overhaul is in we can re-assess :)


[ButtonGroup] Determine first, last and middle buttons to support different elements with correct styling

[x] I have followed (at least) the PR section of the contributing guide.

Based on point 2 of https://github.com/mui/material-ui/issues/29514#issuecomment-962924834.
Closes #31210
Closes #29514
Decided to tackle this after reviewing PR #38403.
Before: https://codesandbox.io/s/lively-http-3w24nw?file=/src/App.tsx
After: https://codesandbox.io/s/dreamy-mestorf-7pmr63

@ZeeshanTamboli Please don't ping the core group for reviews.

This is a PR for the ButtonGroup, see the owner and maintainers of it at https://www.notion.so/mui-org/Components-Inputs-ecd48c9afe354bda8ecda3e96abc8c3c.

Can this PR be reviewed further?

@ZeeshanTamboli

Can this PR be reviewed further?

The logic looks good to me, but I agree with @michaldudak points:

Using classes seems to be the way to implement something like this in Material UI. Data attributes would be unexpected for users.
With this in mind, we should add tests to ensure the classes are applied correctly. The regression test won't cover all cases. For example, imagine we implement this using certain CSS classes and later refactor to rename them. If the refactoring is done well, the regression test will continue to work, but the classes might have changed, which might break the apps of users relying on them.

@DiegoAndai I made the changes, switching from data attributes to class names. I've also relocated the styles from the MuiButtonGroup-grouped className. This change may appear substantial in the code diff. We avoided targeting the first, middle, and last class names within the grouped classname to prevent increasing specificity, making it easier for users to override styles using the theme's styleOverrides or the sx prop. Previously, we applied styles using :first-of-type and :last-of-type selectors only to buttons with the MuiButtonGroup-grouped classname so it had to be inside the grouped class. This is no longer the case.
Additionally, I've included unit tests. So, if I understand correctly, these tests are added to ensure that we don't unintentionally introduce breaking changes when modifying the classes, is that right?


How to access an MDX component's children as markdown?

Question
Description
I'd like to build a custom MDX component I'd like to use like this:
import SpecialBox from 'mycomponents';
Inside an .mdx file
<SpecialBox>
Markdown content here. I'd like to be able to use [links](tourls)
</SpecialBox>

Is there an example of how to do that?  I'd like to set an enclosing container (maybe a styled div), but then use the same component that renders docz's markdown to render the children.

Thanks.

Hey @godmar,
Thanks for reporting this issue,
Inside the mdx file, the block :
<SpecialBox>
Markdown content here. I'd like to be able to use [links](tourls)
</SpecialBox>

is parsed as JSX so it won't render SpecialBox's children as markdown.
You could alter the behavior of SpecialBox to parse and render markdown but I'm not sure that's what you want.

Thanks for the reply. I've been using a string literal:
<SpecialBox>
{`
Markdown content here.
`}

could you share a pointer how I would then process it in a way that uses MDX builtin parser/renderer?

Ok, just in case someone else finds this.  The answer is to leave an empty line before and after the content, like so:
<SpecialBox>

Markdown content here

</SpecialBox>

I discovered this from the carbon theme documentation....

Awesome ! Thanks for reporting back with the answer 👍


zphysics: ConvexHullShape: Some shape-exclusive methods exposed

Since no other derived classes of Shape had yet been exposed, I did this in the way that I thought best fit the pattern laid out in the rest of zphysics.
If this isn't the way we want to go about exposing derived shape classes (in cases like these where the derived class has added functionality that isn't just an overridden virtual method from Shape), then I'd be happy to redo this in whatever way you might prefer.
As usual, I've tested these functions by using them all in my own project, but that's the extent of it.

Looks good to me.
cc @michal-z


Fix HTML in gists

Following #70, Gists containing HTML have an issue parsing the script that adds their copy-to-clipboard button.

Possible fix:
Simply rename *.html to *.txt.