recolour idea info page

recolor idea info page
it should match with team info page
its color should match with team info page
team info page : /views/Team-Builder-Dashboard/team-info-page/team-info-page.html

Hello, can you please assign this issue to me

@Nikhil-IITG
i can assign only one issue to one person
to have less chance of conflicts
please complete the previous one first
if this issue is still open you can ping me.

Hey @Naresh-chandanbatve I would like to contribute here, could you assign it to me please.

Ok @Yash-Ainapure
Go for it

Heyy @Naresh-chandanbatve  , I've been investigating the styling issue on the idea-info page, and after a thorough examination, I discovered that the deployed code differs from the one in the repository. The repository code seems to be correct, while the deployed version has some styling issues.
I spent a significant amount of time reaching this conclusion and have attached screenshots as proof of my observations. I believe updating the deployed code will solve this issue.
After inspecting the code which is deployed :-

After inspecting the code which is runned locally :-

I hope this observations will solve the issue and get me the points of Codepeak event
Thank you.


Delayed Market Open on Holidays

Currently, the market-hours-database.json includes a section for the 'holidays' of an exchange.  The format of the holiday is expected to be "M/d/yyyy" and represents a binary decision about whether an exchange trades on a particular UTC date. This is problematic because holidays can sometimes mean a delayed UTC opening for a particular security. For example, the FXCM holidays for 2017 indicates that on 12/25/2017, trading begins at 18:00 EST for some securities. I do not believe that Lean can handle this situation.
I believe the fix would be for holidays to be represented as something similar to a MarketHoursSegment which includes information about the start and end times of trading for a day.

One thing, the dates in that file are time zone specific to the exchange.

This issue is not reproducible with an algorithm because there are no files from days like Christmas. Maybe those days are cut off during processing. (cc @Jay-Jay-D) and, consequently, we need to include them first.


chore: windows dev support

Since Windows doesn't support providing env variables using command line - we should use cross-env package

Hey! Thanks for contributing. It seems this change causes the build check to fail.

Hi!
Error message from workflow https://github.com/balancer/frontend-v3/actions/runs/10524841452/job/29166397708?pr=993

Error: No projectId found. Every dApp must now provide a WalletConnect Cloud projectId to enable WalletConnect v2 https://www.rainbowkit.com/docs/installation#configure

pnpm build locally works fine
Moreover, event tiny PRs (e.g. https://github.com/balancer/frontend-v3/pull/992) have the same checks errors. Thus my changes are not the reason of fail...

Thanks, yeah just realised that. Something must have changed. Looking into it.


esp_wifi_set_protocol can only be set to WIFI_PROTOCOL_11B (IDFGH-8584)

Board
ESP32
Device Description
DevKit v3
Hardware Configuration
No
Version
v2.0.5
IDE Name
1.8.19
Operating System
Windows 7
Flash frequency
80MHz
PSRAM enabled
yes
Upload speed
115200
Description
I discovered that the protocol can only be set  to const uint8_t protocol = WIFI_PROTOCOL_11B | WIFI_PROTOCOL_11G || WIFI_PROTOCOL_11N; OR const uint8_t protocol = WIFI_PROTOCOL_11B OR const uint8_t protocol = WIFI_PROTOCOL_11B | 0 | 0; anything else fails.
It doesn't look the esp32 supports anything else than the very old protocol that only gives me a connection speed of 10Mbs. Any explanation on that?
Sketch
#include "esp_wifi.h"
#include "esp_log.h"

esp_err_t err;

wifi_init_config_t cfg = WIFI_INIT_CONFIG_DEFAULT();
	err = esp_wifi_init(&cfg);
	if (err != ESP_OK) {
		Serial.println(err);
		Serial.println("Could not set wifi power mode!");
		return false;
	}

	err = esp_wifi_set_mode(WIFI_MODE_STA);
	if (err != ESP_OK) {
		Serial.println(err);
		Serial.println("Could not set sta!");
		return false;
	}

	//const uint8_t protocol = WIFI_PROTOCOL_11G;
	//const uint8_t protocol = WIFI_PROTOCOL_11B | 0 | 0;
	const uint8_t protocol = WIFI_PROTOCOL_11B;

	err = esp_wifi_set_protocol(WIFI_IF_STA, protocol);
	if (err != ESP_OK) {
		//258 = ESP_ERR_INVALID_ARG
		Serial.println(err);
		Serial.println("Could not set protocol!");
		return false;
	}

	uint8_t getprotocol;
	//esp_err_t err;
	err = esp_wifi_get_protocol(WIFI_IF_STA, &getprotocol);
	if (err != ESP_OK) {
		Serial.println("Could not get protocol!");
		//log_e("Could not get protocol! %d", err);
		return false;
	}
	if (getprotocol & WIFI_PROTOCOL_11N) {
		Serial.println("WiFi_Protocol_11n");
	}
	if (getprotocol & WIFI_PROTOCOL_11G) {
		Serial.println("WiFi_Protocol_11g");
	}
	if (getprotocol & WIFI_PROTOCOL_11B) {//worst, but dificuld to set else
		Serial.println("WiFi_Protocol_11b");
	}

Debug Message
No

Other Steps to Reproduce
No response
I have checked existing issues, online documentation and the Troubleshooting Guide

[X] I confirm I have checked existing issues, online documentation and Troubleshooting guide.

@podaen  Does this can be closed also？Is same to #9949 ?

No, Because the protocol is not been set by ESP Arduino and therefore not showed in the log. Still it is some important information about the network.

@VojtaBartoska Do you agrea?

@VojtechBartoska I have checked the Arduino project, I didn't find a interface for user to call esp_wifi_set_protocol() to set the 11B/G/N phy mode, maybe need to add one.
@podaen If the user doesn't change the phy mode, the default phy mode is 11bgn,  the log you provided doesn't include the phy mode info.

To set the n protocol we need to set bgn and because it has been set after the start of the WiFi I think it can change over time depending on the connectivity of the device(s).

Yes，you need to set bgn for 11n protocol. And the STA will change the negotiated protocol according to the connected AP. Take a look at : https://github.com/espressif/esp-idf/issues/9949#issuecomment-1288513162

Q: The bandwide is restored from nvs, so will it stay WIFI_BW_HT20 when it had some power issues at startup?

Yes

I have implement WiFiProv and it connects eminently, because my credentials where allready been flashed in to the nvs. Now I need to entirely nvs_flash_erase() to see how the provisioning works. It would be useful if there was an esp_err_t esp_wifi_erase(void) in the esp_wifi.h like the restore where the credentials are located.

Can find WiFi multi in IDF...  esp_wifi.h api's can not be used as pointer because of the storage in the nvs. Would in not be better to adapted the storage of the wifi settings to make it more suitable for other uses like get, erase and multi wifi.
Q: For what is the manager.h for wifi_provisioning be used for? Can't find any documentation about that.

I found it. It was in the same documentation of provisioning
https://docs.espressif.com/projects/esp-idf/en/latest/esp32/api-reference/provisioning/wifi_provisioning.html#_CPPv418wifi_prov_mgr_init22wifi_prov_mgr_config_t

Hi @podaen, excuse delayed reply, I was on vacation.
Is there anything else what is not clear?

This topic seems finished.


Make RFI ID sequential and continuous

As an analyst
I want the RFI IDs to follow a sequential and continuous numbering system
So that it makes sense to everyone
Description:
the RFI ID is currently not sequential and continuous as it skips numbers randomly
This ticket is to can make the automation follow a sequential ID
Acceptance Criteria:

[ ] The RFI ID continues to auto-populate but now follow a sequential and continuous numbering system

for this one application it repeated the RFI ID:

Sure! I'll get on that


Update index.md: Add link to more information

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

The article is a stub, so it would be much better if you made a proper article out of it.

Thank you for opening this pull request.
This is a standard message notifying you that we’ve reviewed your pull request and have decided not to merge it. We would welcome future pull requests from you.
Reason: We do not accept PRs which only add links when there are already valid reference links for the article.  As the previous user said, you should be adding content and not just links.  Please create a new PR with original content to be considered for merging with the repo.
Thank you and happy coding.


More Options button is not Responsive in 742*830

Description
More Options button is not Responsive in 742*830
Could you Please assign me the Task  with Hacktoberfest tag
Screenshots

Hello SauravMukherjee, can you please assign this bug to me, I have idea about responsiveness , please let me know ?


Scripted Mission Trigger On Start Check

Many scripted missions spawn the player immediately on start.
If clients aren't able to enter the scene before the host does, they might not have a chance to spawn in with the host.
This delays both on spawn triggers and checkpoint triggers until all players are loaded into the scene

Oh cool! I trust the logic works OK, but maybe the static fields should be reset before this trigger stuff happens as I think ready will never be false again after it's set atm?
Also could you set the build ID to INDEV in the Plugin.cs before this gets merged TY :)


🐞 [Bug]: Error: message signature verification failed: could not verify signature.

What happened?
After updating TF Grid version 3.14, some farmers report that they are getting an error: message signature verification failed: could not verify signature.

which network/s did you face the problem on?
Main
Twin ID/s
946
Version
No response
Node ID/s
635
Farm ID/s
3617
Contract ID/s
No response
Relevant log output
Error: message signature verification failed: could not verify signature.

What is the version of farmerbot?

@rawdaGastan  The user is on the v0.15.0, the latest one.

v0.15.10 is the latest one

@rawdaGastan  Thank you for the update.

We've seen a few cases so far where updating the version resolved this issue. Some users have yet to update or yet to report back, but it seems that the older version is indeed the issue.


Add support for UUIDs

This commit implements the FromValue trait for the Uuid type.
It is quite convenient to do that because it reduces the boilerplate when writing SQL queries.
It's an optional feature so, no new dependencies are introduced by default.
I understand that UUIDs are not an official SQL type, but since traits can only be implemented in either the crate the trait or the struct was defined in, this feature has to be implemented in either the uuid or the mysql crate. And doing that in the uuid crate would obviously be wrong.

Thanks. I will look into it.
Could you please leave a that message in #36 :

I license past and future contributions under the dual MIT/Apache-2.0 license, allowing licensees to chose either at their option


build: use repo-metadata to generate README

Yes!  Let's fix the quickstart and add that in a followup PR though


Distinguishing text/binary messages

Hi -
Thanks for the work on this library. It's certainly the easiest-to-consume C++ WS library I've come across.
One thing that I don't see is a way to determine whether an incoming message is binary or text. The _dispatchBinary method gets the opcode, but then just eats it without telling the callable. It seems like it would make sense to pass the type on as well.
Or maybe I'm missing something in the way things are supposed to work.

Hi @jpek42, sorry for taking awhile to get back to this issue. This is a limitation of the API: The information about text or binary can be inspected based on which callback was called. I.e., two callbacks need to be registered, one for text and one for binary.

Thanks for the response. I guess it wasn't clear what those 2 methods were (dispatch, dispatchBinary). Are they just establishing callbacks? So there's no need to call them within the loop?

Hi @jpek42, the original issue you raise is accurate: this is a limitation of the API, and also unexpected/confusing.
The easiest solution would be: make sure that your service ever only sends one or the other (text or binary), and never mixes them.
The more better solution would be to fix the API. I will see if I can do that this weekend, or pull requests are also welcome. Thanks for reporting this issue.


Patch methods are not working

Trying to make a PATCH request to update a Location results in the following error:
ERROR OcpiResponseBody com.fasterxml.jackson.databind.exc.MismatchedInputException: Cannot construct instance of `com.izivia.ocpi.toolkit.modules.locations.domain.LocationPartial` (although at least one Creator exists): no String-argument constructor/factory method to deserialize from String value ('{
"name": "Interparking Gent Zuid",
"last_updated": "2019-06-24T12:39:09Z"
}')
 at [Source: (String)""{\n\"name\": \"Interparking Gent Zuid\",\n\"last_updated\": \"2019-06-24T12:39:09Z\"\n}""; line: 1, column: 1]

I think that it's because the toolkit is trying to create a LocationPartial that have all the fields as optional, but without a default value, so is requiring all the fields to be present in the JSON input.

Thank you for your report, we have been using EVSE patches in our plateform without any issues.  I will take a look into it

Maybe I'm doing something wrong, but I can't get your test to work :thinking:
package com.izivia.ocpi.toolkit.common

import com.izivia.ocpi.toolkit.modules.locations.domain.LocationPartial
import org.junit.jupiter.api.Test
import strikt.api.expectThat

class MapperTest {

    @Test
    fun shouldProperlyMap() {
        expectThat(
            mapper.readValue<LocationPartial>(
                """{
                "name": "Interparking Gent Zuid",
                "last_updated": "2019-06-24T12:39:09Z"
                }
            """.trimIndent()
            )
        ).isA<LocationPartial>()

        expectThat(
            mapper.readValue<LocationPartial>(
                "{\n\"name\": \"Interparking Gent Zuid\",\n\"last_updated\": \"2019-06-24T12:39:09Z\"\n}"
            )
        ).isA<LocationPartial>()
    }
}

is generating the following error:
e: file:///home/.../ocpi-toolkit/ocpi-toolkit-2.2.1/src/test/kotlin/com/izivia/ocpi/toolkit/common/MapperTest.kt:22:20 None of the following functions can be called with the arguments supplied:
public final fun <T : Any!> readValue(p0: JsonParser!, p1: ResolvedType!): TypeVariable(T)! defined in com.fasterxml.jackson.databind.ObjectMapper
public open fun <T : Any!> readValue(p0: JsonParser!, p1: TypeReference<TypeVariable(T)!>!): TypeVariable(T)! defined in com.fasterxml.jackson.databind.ObjectMapper
public open fun <T : Any!> readValue(p0: JsonParser!, p1: JavaType!): TypeVariable(T)! defined in com.fasterxml.jackson.databind.ObjectMapper
public open fun <T : Any!> readValue(p0: JsonParser!, p1: Class<TypeVariable(T)!>!): TypeVariable(T)! defined in com.fasterxml.jackson.databind.ObjectMapper
public open fun <T : Any!> readValue(p0: DataInput!, p1: JavaType!): TypeVariable(T)! defined in com.fasterxml.jackson.databind.ObjectMapper
public open fun <T : Any!> readValue(p0: DataInput!, p1: Class<TypeVariable(T)!>!): TypeVariable(T)! defined in com.fasterxml.jackson.databind.ObjectMapper
public open fun <T : Any!> readValue(p0: File!, p1: TypeReference<TypeVariable(T)!>!): TypeVariable(T)! defined in com.fasterxml.jackson.databind.ObjectMapper
public open fun <T : Any!> readValue(p0: File!, p1: JavaType!): TypeVariable(T)! defined in com.fasterxml.jackson.databind.ObjectMapper
...

Forget it. In my test I was JSON encoding a JSON encoded string before doing the real call to the server :man_facepalming:
Sorry for that, and thanks for your time.


v2.6.0 shows unreleased in version call

Bug Report

Version call shows unreleased
Steps to Reproduce

!sau v

Expected behavior

Screenshots

Well... drat. :D

This issue is resolved in version 2.6.1


[Error] Fetching error

Cannot read property 'url' of undefined & getHostNode

Somehow it was fixed, probably it wasn't even issue related to this project.


Allows runtime override of query and mutation consistency levels.

What this PR does:
Enables users to supply a list of consistency levels via the following system properties:

stargate.graphql.mutation_consistency_levels
stargate.graphql.query_consistency_levels

These values are comma separated, and if any given value is invalid it is skipped.
LOCAL_QUORUM is automatically added if it is omitted as it's the default consistency level.
If this approach looks sound I can add some documentation around the new parameters, and maybe add them to starctl.
Which issue(s) this PR fixes:
Fixes #729
Checklist

[x] Changes manually tested
[x] Automated Tests added/updated
[ ] Documentation added/updated

/gcbrun

@tub have you by chance completed the DataStax CLA process? (You may have already if you've contributed to other OSS projects sponsored by DataStax.) Our apologies for not having this step listed in our PR checklist, we'll work on fixing that.

@tub have you by chance completed the DataStax CLA process? (You may have already if you've contributed to other OSS projects sponsored by DataStax.) Our apologies for not having this step listed in our PR checklist, we'll work on fixing that.

Hey @jeffreyscarpenter, just running it past our legal folks & will get back to you.

@tub any update on this?

@tub any update on this?

Sorry for the delay, got it signed off, CLA is complete :)

/gcbrun

@olim7t Can you provide a review here?

@tub this needs to be rebased on the latest master, some tests are failing because branch is not up-to-date.. Old issues with some tests..

/gcbrun


Microk8s ha-cluster Add-on Causing Request Timeouts

Been running microk8s on two Raspberry PIs (Raspberry PI 4)  for over a year and it generally always has been running great. However I have recently started experiencing random timeouts when making http requests to services within the cluster.
Thinking there might be an issue with the ingress controller I decided to configure a NodePort service to one of the deployments to see if the timeouts also occur when calling into the NodePort service directly and, it turns out that making requests to the NodePort service directly is also resulting in random timeouts...
One thing I noticed has changed recently is that the cluster is now using calico (before it wasn't - was using flanneld I think) for networking and when I do microk8s status I see that the add-on ha-cluster is enabled? I always thought this add-on was disabled by default as it is only useful if running on a cluster with three nodes at least?
So thinking that the ha-cluster add-on or calico is what may be causing the problem I decided to do microk8s disable ha-cluster.
After doing this my two node cluster became separated and now the two nodes are no longer aware of each other. Also, the master node now shows no resources at all when doing kubectl get all --all-namespaces. So effectively I am now needing to reconstruct everything and put the cluster back together again :o/
Anyway, now that my master node no longer has ha-cluster add-on enabled, I redeployed a pod to the k8s cluster and recreated the NodePort service and now the http requests are responding swiftly again and no request timeouts occur any more.
I have reconstructed the whole cluster by rejoining the second node, redeployed all resources and can confirm that request timeouts are no longer occuring. i.e. with ha-cluster disabled.
So my questions are:

What is the current status of ha-cluster?
Why was ha-cluster add-on enabled on my cluster? Can other add-ons maybe enable ha-cluster add-on? I ask this because one add-on I added recently was metallb so wondering if it was metallb add-on that enabled ha-cluster???
Has anyone else experienced this kind of issue with ha-cluster add-on enabled?
How can I avoid ha-cluster being enabled again in future?

Thanks in advance,
Pm

Apologies for the experience you are having.

What is the current status of ha-cluster?

Ha-cluster was the default from v 1.19 (i think), prior to that its using etcd + flannel.

Why was ha-cluster add-on enabled on my cluster?
I don't recall there is an explicit enabling of HA when a cluster is not in HA prior to an update.  I know there's been a lot of care to maintain the current configuration of the system when it is being upgraded.

Can other add-ons maybe enable ha-cluster add-on? I ask this because one add-on I added recently was metallb so wondering if it was metallb add-on that enabled ha-cluster???

Certainly no addon will enable HA implicitly.
Has anyone else experienced this kind of issue with ha-cluster add-on enabled?

How can I avoid ha-cluster being enabled again in future?
Enabling HA is an explicit action.

Btw may i know what microk8s channel are you in?  If you are using stable and as far as i can tell, you are running a long lasting cluster, the recommendation is to use a specific channel.
For example 1.23/stable.
Using stable will implicitly upgrade your cluster to the latest released version of kubernetes.  These versions may have breaking changes.
@neoaggelos or @ktsakalozos feel free to add in anything else i missed.

Thank you for reporting the issue and sorry for the not great experience with Calico.
What operating system are you using? I think for Calico you might also need to install the linux-modules-extra-$(uname -r) package.
Also, are there any warnings in the output of microk8s inspect?

@balchua, when the issue started happening I was running either 1.21/stable or 1.22/stable. I then upgraded to 1.23/stable hoping it would help and restarted the nodes but still, the random timeouts would still occur.
@neoaggelos, when the timeouts started occurring, I did a microk8s inspect but no obvious warnings or errors showed up. The only thing that stood out was that ha-cluster was enabled when doing a microk8s status.
To offer more detail, when the issue started I began by unpicking various things one at a time:

Disabled metallb - timeouts would still occur.
Added NodePort services direct to a couple of services (i.e. to bypass Ingress Controller and therefore rule out NGINX misconfig as a possible problem) -  timeouts would still occur.
Disabled ha-cluster - Caused whole cluster resources to disappear and worker node to leave the cluster (Not sure why all this happened). At this point I was forced to re-build the cluster and redeploy all resources.
Once cluster was up and running again (This time ha-cluster not enabled - i.e. flanneld in use instead), timeouts no longer occurred.

When the timeouts were occurring, I deployed a simple pod app to test which consisted of only CMD python -m http.server 8080 service and a 1.3MB moon image just to rule out s/w bugs in the full-fledged services but this simple test pod was also periodically timing out.
The test (run in microk8s master node):
wget http://<node-ip>:<node-port>/moon.jpg --read-timeout=5
Would basically fail (timeout) between 0-4 times and on last attempt would successfully fetch the image rapidly (in 0.2s or less).
Extending the read timeout to e.g. 2 mins (read-timeout=120) again it would fail between 0-4 times and on last attempt fetch the image rapidly (in 0.2s or less).
So it seemed to me like it was a connection issue more than a bandwidth issue. i.e. when it could connect then it would then service the request promptly/successfully.
At all times the two node hosts (Raspberry PI 4s) were showing normal healthy CPU/memory/disk/network use.
Thanks again,
PM


v0.45.0 - credentials.source upbound  -  cannot initialize the no-fork async external client

What happened?
When using providers with versions higher than v0.45.0, an error occurs for all managed resources when utilizing credentials.source upbound - wonder if the issue is related to this PR: https://github.com/upbound/provider-aws/pull/1002
v0.45.0 - is working
v0.46.0 - not tested
v0.46.1 - is not working
v0.47.0 - is not working
apiVersion: aws.upbound.io/v1beta1
kind: ProviderConfig
metadata:
  name: default
spec:
  credentials:
    source: Upbound
    upbound:
      webIdentity:
        roleARN: arn:aws:iam::123456789101:role/test

message: |-
      connect failed: cannot initialize the no-fork async external client: cannot get terraform setup: could not configure no-fork AWS client: failed to configure the provider: [{0 configuring Terraform AWS Provider: no valid credential sources for Terraform AWS Provider found.

      Please see https://registry.terraform.io/providers/hashicorp/aws
      for more information about providing credentials.

      AWS Error: failed to refresh cached credentials, no EC2 IMDS role found, operation error ec2imds: GetMetadata, http response error StatusCode: 404, request to EC2 IMDS failed
        []}]

NAME                                                                                HEALTHY   REVISION   IMAGE                                                                       STATE      DEP-FOUND   DEP-INSTALLED   AGE
providerrevision.pkg.crossplane.io/upbound-provider-aws-ec2-cfeb0cd0f1d2                 True      2          xpkg.upbound.io/upbound/provider-aws-ec2:v0.47.0                            Active     1           1               39m
providerrevision.pkg.crossplane.io/upbound-provider-aws-eks-641a096d79d8                 True      2          xpkg.upbound.io/upbound/provider-aws-eks:v0.47.0                            Active     1           1               39m
providerrevision.pkg.crossplane.io/upbound-provider-aws-iam-438eac423037                 True      2          xpkg.upbound.io/upbound/provider-aws-iam:v0.47.0                            Active     1           1               39m
providerrevision.pkg.crossplane.io/upbound-provider-aws-rds-58f96aa9fc4b                 True      2          xpkg.upbound.io/upbound/provider-aws-rds:v0.47.0                            Active     1           1               39m
providerrevision.pkg.crossplane.io/upbound-provider-aws-s3-1e7325b432db                  True      2          xpkg.upbound.io/upbound/provider-aws-s3:v0.47.0                             Active     1           1               39m
providerrevision.pkg.crossplane.io/upbound-provider-aws-ssm-1eba1515ce00                 True      2          xpkg.upbound.io/upbound/provider-aws-ssm:v0.47.0                            Active     1           1               39m
providerrevision.pkg.crossplane.io/upbound-provider-family-aws-ddac5a22918f              True      2          xpkg.upbound.io/upbound/provider-family-aws:v0.47.0                         Active                                 39m

How can we reproduce it?

What environment did it happen in?

Crossplane Version:
Provider Version:
Kubernetes Version:
Kubernetes Distribution:

Looks like the root cause of the issue is that we are using a map[string]any for the value of the assume_role_with_web_identity configuration key whereas the schema declares it as a list.


Fix active coolers not being recognized when run on the OpenJ9 VM

This was caused by comparing fluid name strings with ==, which only works if they share the same memory location

Cheers! Fixed a similar issue for the MSR coolant heaters too ;)


Grid Request

Grid bag request 1: It would be helpful to toggle into grid bag mode from an icon on the default bag as well.  This helps with sorting a bag for a vendor so other non-grid players can still items.
Grid bag request 2: Can the closed state of a bag be saved?  For example, everything opens in the grid the first time it’s opened, then if a certain bag (like a vendor or a sorted chest) is toggled to the default bag view, it stays in that view until toggled back into the grid.

PR - https://github.com/bittiez/ClassicUO/pull/15

So as for number 1, I like the idea, however I don't want to modify CUO's classic gump's, I'm trying to keep this fairly close to original CUO as possible
And as for number 2 I also like the idea of saving this, but the issue lies in that it would require the client to basically save the state of every container you open which would eventually be an issue.. I suppose you could possible add a date to each saved contaienr state and after xx of days remove it from the saved data, I will keep this in mind but won't promise anything yet

As an alternative I'm working on player vendors opening both grid and regular containers vendor-packs

Vendors now open both grid and regular containers(as an alternative)


Data used in ML

Can you upload the data used on your research as .csv ?

Dataset is private. Can't share!


Using Github URLs are dependencies causes deployment failures with misleading error messages.

Describe the bug
Github URLs is an officially supported way of declaring dependencies. ASK Toolkit fails to deploy with Github URL dependencies though.
To Reproduce

Open package.json.
Add a dependency like this: "ask-sdk-test": "s-maheshbabu/ask-sdk-test",
Commit the change.
Try to deploy.
The deployment will fail.

Expected behavior
There are two issues to be fixed.

The error messages are utterly useless and misleading. A better error message would 'Sorry, something went wrong'.
Direct Github URL dependencies should work.

Screen Shots
On VSCode, the error looks like this -

On Dev Console, the error looks like this -

Hey @s-maheshbabu , thanks for creating the issue. The issue regarding supports for GitHub URLs seems to be specific to the Alexa-Hosted skills service itself. We raised a ticket with the internal service team and will keep you posted regarding that.
As for the error messages, this is what we get from the GetSkillStatus API that we use, to figure out if the skill deployment went ahead and if the skill build is successful. We are talking with the corresponding service team to enhance the API, to provide better error messages. Sorry for the inconvenience.

@s-maheshbabu thanks for being patient. We checked with the Alexa-hosted skills service team regarding the Github URL support and got to know that there service cannot handle this currently and there are plans on supporting this in future. I will update this issue whenever such support comes out.
With regards to the error messages, Alexa-hosted skills team knows about this issue and is working on providing better status messages in the API calls. Sorry for the inconvenience.

Responding to prevent closure.

I think issues shouldn't be closed automatically simply because there are no responses. Please create a different bucket for issues that you know are going to take time or remove the auto-closure setting.

I think issues shouldn't be closed automatically simply because there are no responses. Please create a different bucket for issues that you know are going to take time or remove the auto-closure setting.

@s-maheshbabu sorry about this.
In the commit here you will find that a rule has been added to exempt issues marked - waiting: external dependency are exempted from the stale rule. The issue is the space in the label and this issue has been addressed in the V3 version of actions/stale - https://github.com/actions/stale/issues/98. The reason we have not received this update is because we are using the V1 version. I am going to create a PR to update this and that would get looked at next week. In the mean time I am reopening this issue. Thanks for replying to this thread.

@s-maheshbabu sorry about this.
In the commit here you will find that a rule has been added to exempt issues marked - waiting: external dependency are exempted from the stale rule. The issue is the space in the label and this issue has been addressed in the V3 version of actions/stale - https://github.com/actions/stale/issues/98. The reason we have not received this update is because we are using the V1 version. I am going to create a PR to update this and that would get looked at next week. In the mean time I am reopening this issue. Thanks for replying to this thread.


MicrosoftPowerBIMgmt.Profile source location is not valid

I'm trying to install the MicrosoftPowerBIMgmt module. I run the following command:
Install-Module -Name MicrosoftPowerBIMgmt -Scope CurrentUser
I received the following message:
WARNING: Source Location 'https://www.powershellgallery.com/api/v2/package/MicrosoftPowerBIMgmt.Profile/1.0.326' is not  valid. PackageManagement\Install-Package : Package 'MicrosoftPowerBIMgmt.Profile' failed to download.
Any ideas whats happening?

Try updating NuGet and PowerShellGet libraries by following these instructions - https://docs.microsoft.com/en-us/powershell/gallery/installing-psget
If that doesn't work, there might be an issue with your PSGallery Source Location or an issue with either PowerShellGet or PowerShell Gallery. You might have to open an issue at https://github.com/powershell/powershellget to get this resolved. If the issue isn't getting resolved, open a support ticket to gain more traction.

Closing as this is external to this repo and an issue in PSGallery


Analytics request to (defunct?) engagio.com always returns 404

We have a script in public/index.html that sends requests to web-analytics.engagio.com and always gets a 404. Is it still valid?
Console error:
ei_track_all_packed.js:1 GET https://web-analytics.engagio.com/api/stat?page_url=https%3A%2F%2Fdocs.influxdata.com%2F&page_title=InfluxData%20Documentation&track_type=page&action=ei_view&category=ei_page_tracking&client_id=&account_id=e9b11baeb0d0dda444c3105e7cf0877955e378ca&method=post&callback=EI.api._callbacks.s18511115 net::ERR_ABORTED 404
Relevant URLs
All https://docs.influxdata.com URLs.

We installed a few marketing tools a while ago that I'm not sure we're using any more.
@ChrisChurilo Are we still using Engagio and Marketo?


Update README.md

[ ] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the master branch of freeCodeCamp.
[ ] None of my changes are plagiarized from another source without proper attribution.
[ ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Trivial and not user-facing, I suggest closing.

Hi,
Thanks for this pull request (PR).
Unfortunately, we are marking closing this PR invalid, as we do not believe it adds a significant improvement.
Thank you


hyperGeometricU - is it possible to get -Infinity, Infinity as a result?

Example:
  hypergeometricU(3.0, 1.0, 0.0);

Should return Infinity
Can we have some special values for this case or a specialized exception?
Same for "ArithmeticException" with "Division by zero" text?

I suppose this would be in general a useful idea, having some kind of subclasses of ArithmeticException to indicate what happened (infinity, indeterminate, ...). Especially if a CAS system is to catch the exception and use it as a symbolic value somehow.
However there's a lot of places that would need to be changed (surely not just hypergeometricU) so it would be quite a lot of work. Also presumably some places would need a bit more logic to throw either "infinity" or "indeterminate" and not just the generic ArithmeticException, for example.

I suppose this would be in general a useful idea, having some kind of subclasses of ArithmeticException to indicate what happened (infinity, indeterminate, ...). Especially if a CAS system is to catch the exception and use it as a symbolic value somehow.

Does it make sense to use "checked exceptions" for these cases?

can OverflowException become a "checked exception", so that the user is forced to handle it?

As noted in #38, it might be quite non-trivial to get the actual function value in the exception, because an ArithmeticException (or OverflowException) can be thrown anywhere in the middle of the evaluation of the algorithm for computing the function value, and it may not be the same value that would be the ultimate value of the function.
For example if your function is computed as 1/(1/x) then the first 1/x might throw +∞ but then the value of the function would be simply zero (i.e. 1/∞) and it shouldn't throw any exception at all.
Or for example xy is computed with exp(y * log(x)) and if y * log(x) overflows and throws an OverflowException but the real part of y * log(x) would be negative then the exp() would underflow to zero. But instead the function just throws OverflowException.


High CPU usage by a python process spanned by vscode

I get high CPU usage (100%), by a python process spanned by vscode. This happens in a private repo that I cannot share, even when everything is idle.
Relevant process info obtained via ps aux
PATH_TO_MY_ENVIRONMENT/bin/python ~/.vscode/extensions/ms-python.python-2020.4.74986/pythonFiles/pyvsc-run-isolated.py ~/.vscode/extensions/ms-python.python-2020.4.74986/pythonFiles/completion.py

Environment data

VS Code version:

Version: 1.44.2
Commit: ff915844119ce9485abfe8aa9076ec76b5300ddd
Date: 2020-04-16T17:07:18.473Z (1 wk ago)
Electron: 7.1.11
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Darwin x64 19.4.0

Extension version (available under the Extensions sidebar): 2020.4.74986
OS and version: macOS 10.15.4
Python version (& distribution if applicable, e.g. Anaconda): 3.7.7 64bit Anaconda
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): conda
Relevant/affected Python packages and their versions: No answer
Relevant/affected Python-related VS Code extensions and their versions: No answer
Jedi or Language Server?: python.jediEnabled=True
Value of the python.languageServer setting: Microsoft

Potential duplicate of #2149.
Setting python.jediEnabled=False appears to solve the issue.


Validate number with commas

Versions:
VueJs: 2.1.10
Vee-Validate: ^2.0.0-rc.6
Description:
We are working on a site that allows users to make offers for a property such as £9350000. I need to automatically add commas to the number so it looks like £9,350,000.
But if I do that I can't validate the number with Vee-Validate, as it doesn't recognise that as a number.
As a work around I have added a hidden field that has the number without the commas and can validate it correctly as a number.
Any suggestions on how this could be achieved without this worked around. How can I visually show the commas but not have the input changed.
Steps To Reproduce:

                £
                <input name="make_offer" class="make-offer-input" type="text" v-model="messageText">
                <input name="offer_validate" v-validate="'required|between:1,10000000'" type="number" v-model="offerValidate" class="hide">

{{ errors.first('offer_validate') }}

And
        watch: {
            messageText: function(newValue) {
                var copyValue = newValue;
                const offerResult = copyValue.replace(/\,/g,'');
                Vue.nextTick(() => this.offerValidate = offerResult);
                const result = newValue.replace(/^0+/, '').replace(/\D/g, "").replace(/\B(?=(\d{3})+(?!\d))/g, ",");
                Vue.nextTick(() => this.messageText = result);
            },
        },

You can create your custom rule instead of using the built in ones, your custom rule would typically allow numbers and commas.
http://vee-validate.logaretm.com/rules#custom-rules

The number only needs to have commas visually, do you have any recommendations for vue.js input masks that might work well with vee-validate.


Feature Request:

I just discovered Grommunio Desktop, and I like it. I switch out between Chrome & Firefox often for work, so it's nice to have it as a separate desktop window. Kudos!
I log into two different Grommunio server accounts. This works great, but both 'servers' are stacked in one tab along the top.
Is there any setting to have the servers side by side, in a row (each server in its own tab along the top row?)
There is space to the right of the stacked servers tab that could be utilized I think.
If this is not possible currently, I would like to make this a feature request!
Thank you!

Default settings in the login are desirable in the DeskAPP. This is already possible in the WebAPP via a browser. Could a "remember password" function be set up? Or something similar that makes it easier for the user to log in?


error when using my project

E/EventLogger: internalError [0.67, loadError]
com.google.android.exoplayer2.source.UnrecognizedInputFormatException: None of the available extractors (MatroskaExtractor, FragmentedMp4Extractor, Mp4Extractor, Mp3Extractor, AdtsExtractor, Ac3Extractor, TsExtractor, OggExtractor, PsExtractor, WavExtractor) could read the stream.
at com.google.android.exoplayer2.source.ExtractorMediaPeriod$ExtractorHolder.selectExtractor(ExtractorMediaPeriod.java:713)
at com.google.android.exoplayer2.source.ExtractorMediaPeriod$ExtractingLoadable.load(ExtractorMediaPeriod.java:636)
at com.google.android.exoplayer2.upstream.Loader$LoadTask.run(Loader.java:295)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1133)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:607)
at java.lang.Thread.run(Thread.java:761)
08-20 14:48:39.194 13800-14077/com.softsion.livebit E/ExoPlayerImplInternal: Source error.
com.google.android.exoplayer2.source.UnrecognizedInputFormatException: None of the available extractors (MatroskaExtractor, FragmentedMp4Extractor, Mp4Extractor, Mp3Extractor, AdtsExtractor, Ac3Extractor, TsExtractor, OggExtractor, PsExtractor, WavExtractor) could read the stream.
at com.google.android.exoplayer2.source.ExtractorMediaPeriod$ExtractorHolder.selectExtractor(ExtractorMediaPeriod.java:713)
at com.google.android.exoplayer2.source.ExtractorMediaPeriod$ExtractingLoadable.load(ExtractorMediaPeriod.java:636)
at com.google.android.exoplayer2.upstream.Loader$LoadTask.run(Loader.java:295)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1133)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:607)
at java.lang.Thread.run(Thread.java:761)
08-20 14:48:39.197 13800-13800/com.softsion.livebit E/EventLogger: playerFailed [0.67]
com.google.android.exoplayer2.ExoPlaybackException
at com.google.android.exoplayer2.ExoPlayerImplInternal.handleMessage(ExoPlayerImplInternal.java:345)
at android.os.Handler.dispatchMessage(Handler.java:98)
at android.os.Looper.loop(Looper.java:154)
at android.os.HandlerThread.run(HandlerThread.java:61)
at com.google.android.exoplayer2.util.PriorityHandlerThread.run(PriorityHandlerThread.java:40)
Caused by: com.google.android.exoplayer2.source.UnrecognizedInputFormatException: None of the available extractors (MatroskaExtractor, FragmentedMp4Extractor, Mp4Extractor, Mp3Extractor, AdtsExtractor, Ac3Extractor, TsExtractor, OggExtractor, PsExtractor, WavExtractor) could read the stream.
at com.google.android.exoplayer2.source.ExtractorMediaPeriod$ExtractorHolder.selectExtractor(ExtractorMediaPeriod.java:713)
at com.google.android.exoplayer2.source.ExtractorMediaPeriod$ExtractingLoadable.load(ExtractorMediaPeriod.java:636)
at com.google.android.exoplayer2.upstream.Loader$LoadTask.run(Loader.java:295)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1133)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:607)
at java.lang.Thread.run(Thread.java:761)

@sarawutl , please give the stream url and let me try.

it working in demo project, but error in my project

rtmp://<IP_ADDRESS>:1935/tan/e3qH1If0JtZ6CiL3rvBB

I'm copy package liveVideoPlayer to my project and run LiveVideoPlayerActivity not working

If it is working on demo project but not working on your project, then please find out the differences between demo project and your project. You must have missing somethings.

Closing this issue @sarawutl , please let me know if you have any question


📘 Transformation Catalog

Finished Transformation Catalog view modal and done some component enhancement.

Also, you can already add the "Add to canvas" button with an empty onClick in this issue. Adding the block to the canvas will be part of #87

This is issue is resolved by using selectedTransformationIndex === null since using falsey will also trigger 0 value which is an index. also the null value is intended to indicate that no item have been selected.


perf: copy entire file before unpacking

Description
This PR introduce a small performance improvement for streaming tar.bz2 files.
Based on the benchmarks, downloading file into SpooledTempFile and then un-compressing is faster than download & extract at the same time.
Benchmarks for python tar.bz2:
Without changes
Running benches/extract.rs (/Users/graf/projects/oss/rattler-1/target/release/deps/extract-f947b197b5ad3a13)
Gnuplot not found, using plotters backend
extract tar             time:   [1.8692 s 1.8893 s 1.9125 s]
                        change: [-1.4149% +0.5238% +2.4181%] (p = 0.61 > 0.05)
                        No change in performance detected.

with spooled file 5 mb
     Running benches/extract.rs (/Users/graf/projects/oss/rattler-1/target/release/deps/extract-f947b197b5ad3a13)
Gnuplot not found, using plotters backend
extract tar             time:   [1.8519 s 1.8676 s 1.8811 s]
                        change: [-2.5692% -1.5904% -0.5665%] (p = 0.01 < 0.05)
                        Change within noise threshold.

We can see a small 1% improvement. Similar improvement can be observed also for bat.tar.bz2
I've also added a small webserver that will serve files from the memory, so it can be used in benchmarks.

This basically means we are no longer downloading and extracting in parallel. I wonder if that makes a difference with a slower connection.

Do you think it would be possible to implement the server in rust instead? That would make the whole thing more standalone.

Look for StaticFileServer (from top of mind) we already have something similar.

I think my idea would be that we ideally keep parallel downloading and extraction, but if downloading is much faster than extraction, that we store the remaining data in a (spooled) temporary file, so that download can progress at maximum speed.

I think what could be also interesting is to try out async_compression (uv uses this). uv also controls the BufReader size more, and uses a 128Kb buffer (124 * 1024). I think we end up using a default BufReader which has a 8Kb buffer which might lead to less optimal download speeds.


install error on Powershell 4

When ImportExcel installed on Powershell 4, I got the following error:

Thanks for reporting and submitting the fixes


Add Joyent to LF-Edge lanscape

Add logo and metadata for Joyent

https://landscape.cncf.io/logos/joyent-member.svg
Full entry to leverage for PR (website, Crunchbase) = https://landscape.cncf.io/format=card-mode&grouping=no&selected=joyent-member


Starter workflow for creating AKS cluster and deploying containerized app

Adding an example workflow that uses this action to accomplish creating a cluster, and deploying a containerized app. I imagine there may be more examples here in the future, so I started the trend of naming your workflow yaml, and then creating an md that can explain the finer details of the workflow if needed. Free free to change this approach if you can think of a cleaner way to do this.

looks great @Marcus-Hines -- maybe make it in an additional sub-folder though? in case we have more examples later.  So something like examples/starter


can't change twitter account, after modifying it on twitter

Hi,
i'm not able to change my twitter account within keybase. It shows, it isn't available anymore, but it doesn't give me the option to change it.
Keybase GUI Version: 1.0.20-20170329160043+6d18a6a

We actually just had a server outage, so that might explain why.  That's fixed now, mind giving it another try?

Too late, I had reset the account and re-did the steps. cheers


Public file-lines constructor

Related to https://github.com/rust-lang-nursery/rls/pull/346

Thanks!


VRController: Check also for gamepad being undefined.

Fixes #9

I also fixed that in VRController here: https://github.com/stewdio/THREE.VRController/commit/0102e0b1447a193b392f445e83c9c3043f68071d
It’s weird that Chrome and Firefox will populate the Gamepads array so each element either contains a Gamepad instance or a null, but Edge will actually leave unused slots undefined. But now I know—and knowing is half the battle ;)


remove shortkeys from shortkey_to_tool_map_

this should fix #880

@hdeeken, if you get a chance please try this out w.r.t. your issue #880. I'm gong to go ahead and merge this since I'm pretty sure it addresses the problem.


ManipHandler fixes when using programmatically, and two playmode manipulation tests!

Overview
A few maniphandler fixes, and adding a few simple tests for ManipHandler!

Expose all serialized fields as properties so they can be set programmatically (we had missed a few)
ManipHandler check for null before invoking events.
Initialize events handlers so that events like hover, manipstart can be accessed programmatically.

Also, adding a couple playmode test changes to actually test manip handler!

Add ManipHandler test for instantiating programmatically, and seeing basic hover events.
Fix bug that was preventing multiple playmode tests from running sequentially. CleanupScene() would create a new scene every run, however in PlayMode this would cause MRTK to not be initialized. Instead, let the PlayMode tests instantiate the scenes (they do this by default, it seems).

Fixes

Fixes: #4179
Fixes: #4173

Verification

I verified this by running all EditMode and PlayMode tests. Also verified by testing the repro case in #4173.
Also tested the HandInteractionExample scene in editor.
Verified that code compiles for UWP

/azp run mrtk_pr

It looks like the mrtk_PR pipeline doesn't do play mode tests, so my theory that it should be run in PRs didn't pan out. @MenelvagorMilsom do you know if there were specific things that made it so that we weren't able to do the play mode tests in CI?

/azp run mrtk_pr

It looks like the mrtk_PR pipeline doesn't do play mode tests, so my theory that it should be run in PRs didn't pan out. @MenelvagorMilsom do you know if there were specific things that made it so that we weren't able to do the play mode tests in CI?

Okay hopefully this will work:
https://github.com/Microsoft/MixedRealityToolkit-Unity/pull/4185

It looks like the mrtk_PR pipeline doesn't do play mode tests, so my theory that it should be run in PRs didn't pan out. @MenelvagorMilsom do you know if there were specific things that made it so that we weren't able to do the play mode tests in CI?

Okay hopefully this will work:
#4185

Ah, that one didn't work because of some other issues, anyways, let's just move forward with this PR and I'll handle the build part.

/azp run mrtk_pr

It looks like the mrtk_PR pipeline doesn't do play mode tests, so my theory that it should be run in PRs didn't pan out. @MenelvagorMilsom do you know if there were specific things that made it so that we weren't able to do the play mode tests in CI?

Sorry, I experimented with this before, but there were issues with the build machines. It looks like you've hit a different issue in #4185. We should definitely look at getting these tests running on CI and for PR validation.


Merge to live

OPS Build status updates of commit 839de36:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

office-365-management-api/office-365-management-activity-api-schema.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Would need more space between title and images in cards

Not really sure if it's possible or not (or if one of the feature requests already entered is specifically about that) but would it be possible to add space between the title and the first picture cards (see picture, it feels like it should have the same sapce like I'm poiting with the arrow... Does it make sense?

Which browser are you using?


Refactor/add test

Codecov Report

Merging #18 (ab0fded) into main (adeb868) will increase coverage by 30.46%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##             main      #18       +/-   ##
===========================================
+ Coverage   62.19%   92.65%   +30.46%
===========================================
  Files          16       16
  Lines         484      490        +6
===========================================
+ Hits          301      454      +153
+ Misses        183       36      -147

Impacted Files
Coverage Δ

lib/js_dependency.rb
67.21% <100.00%> (+34.42%)
:arrow_up:

lib/js_dependency/extractor/extract_script_tag.rb
100.00% <100.00%> (ø)

lib/js_dependency/mermaid/nodes_link.rb
100.00% <100.00%> (+35.29%)
:arrow_up:

lib/js_dependency/mermaid/root.rb
100.00% <100.00%> (+60.00%)
:arrow_up:

lib/js_dependency/mermaid/target_pathname.rb
100.00% <0.00%> (+42.10%)
:arrow_up:

lib/js_dependency/source_analysis/leave.rb
100.00% <0.00%> (+50.00%)
:arrow_up:

lib/js_dependency/replace_path_alias.rb
100.00% <0.00%> (+55.55%)
:arrow_up:

... and 3 more

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


Accommodate NomadSecrets store to resolve Nextflow config-secrets

Actually, I must add a clarification here since as of now we have two functional scenarios

Setup
LocalStore
NomadStore

Config vars
✅
🔴

Job vars
✅
✅

sun-nomadlab
✅
✅

✅ Scenario-1
Config vars => LocalSecret store
Job vars    => LocalSecret store
✅ Scenario-2
Config vars => LocalSecret store
Job vars    => NomadSecret store
🔴  Scenario-3
Config vars => NomadSecret store
Job vars    => NomadSecret store
Conclusion
@jagedn , please correct me if I'm wrong - in scenario-3 we can't use the NomadSecret store for config vars due to the current design decisions on the Nextflow side, however we can support scenario-1 and scenario-2 just fine.
Originally posted by @abhi18av in https://github.com/nextflow-io/nf-nomad/pull/75#discussion_r1694712898

I think I founded an easy way to have also in config vars
obviously only case we'll not able to use secrets will be in the own plugin configuration, i.e.,
client {
        address = secrets.MY_HOST
    }

Related #81  #79

@jagedn , the #81 should address scenario-3 described here right? https://github.com/nextflow-io/nf-nomad/issues/76#issue-2434723428

@jagedn , the #81 should address scenario-3 described here right? #76 (comment)

yes, I think with this PR we cover all use cases


Feature/yury/s2s3 merge

This pull request has two updates.

Update to CICE parameter in AGCM.ru. @zhaobin74 could you review this update?
Update to gcm_run.j script to enable it to copy checkpoints to restarts for MAPL monthly collections. @atrayano @lltakacs could you review this update and let me know if this is acceptable or should it be done different way?

@lltakacs Larry, if you cast @amdasilva, Arlindo will receive a notification. I think that "_chk" suffix is coded into MAPL, @atrayano could you clarify this?


About FA icon

You can actually avoid using icons.css file if you replace icon-snowflake with icon-snowflake-o in code so Font Awesome icon from Winter CMS core will be used - https://fontawesome.com/v4.7/icon/snowflake-o

https://github.com/skripteria/wn-snowflake-plugin/blob/01d770ec731c727b25e000543431af5245845cc8/Plugin.php#L81
https://github.com/skripteria/wn-snowflake-plugin/blob/01d770ec731c727b25e000543431af5245845cc8/Plugin.php#L107
https://github.com/skripteria/wn-snowflake-plugin/blob/59518fcd2f48c323934a4246c4b4485ba83d2e67/components/SfPage.php#L19

Currently I left this on hold because probably https://github.com/skripteria/wn-snowflake-plugin/pull/8 should be merged or rejected first.

oohps, I must have overlooked the icon-snowflake-o when I checked the core.
Of course this is much better, glad you found it.


Request object mutability issue

Laravel Version: 5.1 and 5.4 (both has this issue)
PHP Version: 7
Database Driver & Version: 5.7 mysql

Description:
Hard to modify request. I wanted to intercept any request having some value of callback and continue via redirect headers. Function fullUrl() of a modified request returns original value which is unexpected.
Steps To Reproduce:
Try out this middleware.
namespace App\Http\Middleware;
use Closure;
class Test
{
    public function handle($request, Closure $next, $guard = null)
    {
        // so request is http://laravel.com?foo=bar&callback=123
        if ($callback = $request->get('callback')) {
            $request->query->remove('callback');
            $qp = $request->query->all();
            // an array as expected! count($qp) == 1
            $requestAsIf = $request->duplicate($qp);
            // duplicate all but set new queryBag

            $stillTheRequestOriginalUrl = $requestAsIf->fullUrl();
            // would expect http://laravel.com?foo=bar, right?!
            $resp = $next($request);
            // ..
            #! $resp->headers->set('Location:' , 'foo');
            #! $resp->withCookie('callback' , $callback, 4200);
            return $resp;
        }
        return $next($request);
    }
}

fullUrl() builds the URL query string using server's QUERY_STRING and that's why $request->query->remove('callback') won't take effect on the URL.
I don't think there's a way to mutate the request URL.
Anyways this all happens inside the Symphony HTTP component and Laravel is based on that, so I'd say this is not a laravel issue, maybe try the forums for some ideas on how to achieve what you need.

See this issue on the symfony repo.

All clear. I see, Symfony wouldn't admit.

Closing this issue then :)


Confusing error message

I'm still learning all of this. I've used Obsidian for a while but not in power-user mode, and hadn't tried Canvas either.
I made a couple nodes based on the "2. writing to nodes" sample, I'm sure I'll figure out my mistakes eventually but just wanted to share that the error message,

Error:
Error calling LLM:
[] is too short - 'messages'

Is pretty cryptic. It tells me that something is missing somewhere. Is it possible to say more here?

Thanks for giving cannoli a try! This error message is actually coming from the openai api, I believe it means that no messages were sent. It is pretty cryptic, I can see about catching this and displaying better error message.
Empty call nodes will not add any messages to the messages array, so if there are no messages coming from other nodes, nothing will be sent to the openai api, resulting in this error.
Can you share a screenshot of the cannoli that's producing this?

You can print a log of all the messages sent to openai using a logging arrow, which is an orange arrow pointing from the call node to a content node (purple).
In the case of the error above, this likely wouldn't work because the error would happen in the call node itself.


proposal: x/sys/windows: add func windows.DisconnectNamedPipe(handle Handle) (err error)

Proposal Details
This proposal is for the DisconnectNamedPipe function I need for my code, but is still missing here.

+1
Adding a trivial function to x/sys doesn't need to go through the proposal process. Thanks.

Thanks for the info. As this is my first contribution, the process is still very new to me, and it is not yet clear when something is trivial or not.
After submitting the change, I realized I made a type in the capitalization of the function, so I also got the chance to try an amend.
I have a list of additional functions I would like to add, so not needing a proposal saves me a lot of typing.
I also do not yet understand how to respond in https://go-review.googlesource.com/ to comments, so for now I’ll respond by email.
Clearly, I still have a lot to learn ...
From: Quim Muntal @.>
Sent: Sunday, 10 March 2024 15:14
To: golang/go @.>
Cc: Peter Hoogendijk @.>; Author @.>
Subject: Re: [golang/go] proposal: x/sys/windows: add func windows.DisconnectNamedPipe(handle Handle) (err error) (Issue #66234)
+1
Adding a trivial function to x/sys doesn't need to go through the proposal process. Thanks.
—
Reply to this email directly, view it on GitHub https://github.com/golang/go/issues/66234#issuecomment-1987247379 , or unsubscribe https://github.com/notifications/unsubscribe-auth/BDYPXZSG5IJMQSRS2PTBQ7TYXRTCRAVCNFSM6AAAAABEO7FKEKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBXGI2DOMZXHE .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/BDYPXZTWNWWJXHJ6OYCQ2UTYXRTCRA5CNFSM6AAAAABEO7FKEKWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTWOL6RG.gif Message ID: @.*** @.***> >


Docs missing graphql-endpoint!

https://github.com/facebook/create-react-app/blob/next/packages/react-scripts/template/README.md#adding-graphql-files
The example above only shows how to query not where to query from meaning you need to have a URL to fetch from.

We need an HttpLink that knows how to fetch data from it

Hi! This sounds more like a general purpose question; we were not trying to give an example on how to use Apollo & friends.
You can find excellent community resources here: https://reactjs.org/community/support.html
Thanks!


Command line option /stopable is incorrectly spelled

It should be /stoppable.

Fixed via https://github.com/xplicit/HyperFastCgi/commit/3da3cee9410fee34c360ef2096a25eb38742a319


Refer people to The Turing Way

Update the main page and contribution guidelines to refer people to The Turing Way. Add a warning that many topics in the guide are outdated.
Related to #286

Great PR! The Contributing.md additions are perfect IMHO. See above my comments on the README.md.


Add troubleshooting documentation for CAPV session issue

Issue #, if available:
Description of changes:
Add troubleshooting documentation for capv session issue.
Testing (if applicable):
Documentation added/planned (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve
/cherry-pick release-0.20
/cherry-pick release-0.19


Is it possible to turn off Speed test and leave latancy running?

I've been in an 8 moth battle with my ISP over poor internet connectivity, drop outs and High latency.  They have been to my home 3 times. Replaced the cables, splitters and the modem.  They have replaced the line from the pole to the house, but the issues continue.  I evenly had to file a Complaint with the Board of Public Utilities.
Last night the representative I spoke with claimed that I had a tremendous amount of DL/UL traffic.  I asked them if they Cap traffic and they claimed they did not.  Looking at my traffic report, 49% of the traffic is coming from the PiHole/Monitor server, with 37% of it being the Ookla.  This is of course speed test.
I can see in the config.yml where I can set network monitoring to false.  But how do I turn off only the speed test portion?   I'd like to reduce the traffic utilization to prove that I'm not causing my own issues.

I don't have an option that only does the ping/latency tests, at least not with the current configuration. You could set the monitoring speedtest interval much higher (like 24h so it only runs once a day): https://github.com/geerlingguy/internet-pi/blob/master/example.config.yml#L14

You could also just remove the speedtest container from the docker-compose file (you would also need to remove the link to speedtest in the prometheus section), stop the speedtest container with docker stop <container-id>, and re-run docker-compose up -d in the internet-monitoring sub-folder.

Came here looking for the same thing.  I recently implemented this to monitor some ISP issues, only to find that my ~1Gb down speeds combined with the Internet-Pi was heavily taxing my Xfinity data cap.  This was confirmed while I was out of town after a new billing cycle started and I could clearly see the usage occurring and stop it by disabling the Internet-Pi remotely.  I absolutely want to continue monitoring this, but need to modify the frequency of the speedtest.  It seems silly this is something I'm having to track in the 21st century.   Can't someone make a handy Raspberry-Pi dashboard for that too!


Extraction failed

https://www.ruv.is/frett/2021/06/08/sigur-islands-a-englandi-sa-ovaentasti

@awolad sorry, it looks like the extraction algorithm in this lib only supports English as well.


[Bug]: Some error in join, seems relate to #927

Describe the bug
This is the result of query 13 from TPC-H.
Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "/home/luke/codes/python/narwhals/tpch/execute/q13.py", line 9, in <module>
    print(q13.query(fn(customer), fn(orders)))
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/luke/codes/python/narwhals/narwhals/translate.py", line 717, in wrapper
    result = func(*args, **kwargs)
             ^^^^^^^^^^^^^^^^^^^^^
  File "/home/luke/codes/python/narwhals/tpch/queries/q13.py", line 12, in query
    customer_ds.join(orders, left_on="c_custkey", right_on="o_custkey", how="left")
  File "/home/luke/codes/python/narwhals/narwhals/dataframe.py", line 1925, in join
    return super().join(other, how=how, left_on=left_on, right_on=right_on, on=on)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/luke/codes/python/narwhals/narwhals/dataframe.py", line 217, in join
    self._compliant_frame.join(
  File "/home/luke/codes/python/narwhals/narwhals/_pandas_like/dataframe.py", line 495, in join
    return self._from_native_frame(result_native.drop(columns=extra))
                                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/luke/codes/python/narwhals/.venv/lib/python3.12/site-packages/pandas/core/frame.py", line 5581, in drop
    return super().drop(
           ^^^^^^^^^^^^^
  File "/home/luke/codes/python/narwhals/.venv/lib/python3.12/site-packages/pandas/core/generic.py", line 4788, in drop
    obj = obj._drop_axis(labels, axis, level=level, errors=errors)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/luke/codes/python/narwhals/.venv/lib/python3.12/site-packages/pandas/core/generic.py", line 4830, in _drop_axis
    new_axis = axis.drop(labels, errors=errors)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/luke/codes/python/narwhals/.venv/lib/python3.12/site-packages/pandas/core/indexes/base.py", line 7070, in drop
    raise KeyError(f"{labels[mask].tolist()} not found in axis")
KeyError: "['o'] not found in axis

Steps or code to reproduce the bug
TPCH-Q13 can be reproduced,
and similarly, Q16, Q17, and Q18 might also be affected, although not necessarily for the same reason.
Expected results
Return as expected.
Actual results
Error see above.
Please run narwhals.show_version() and enter the output below.
System:
    python: 3.12.4 (main, Jul 25 2024, 22:42:01) [Clang 18.1.8 ]
executable: /home/luke/codes/python/narwhals/.venv/bin/python
   machine: Linux-<IP_ADDRESS>-microsoft-standard-WSL2-x86_64-with-glibc2.35

Python dependencies:
     narwhals: 1.6.3
       pandas: 2.2.2
       polars: 1.6.0
         cudf:
        modin:
      pyarrow: 17.0.0
        numpy: 2.1.0

Relevant log output
No response

thanks @luke396 for the report
I can't reproduce this
(polars-api-compat-dev) marcogorelli@DESKTOP-U8OKFP3:~/polars-api-compat-dev/tpch$ python -m execute.q13
    c_count  custdist
2         0     50004
8        10      6668
12        9      6563
20       11      6004
9         8      5890
14       12      5600
6        13      5029
13       19      4805
1         7      4680
3        18      4531
7        20      4507
17       14      4473
19       15      4463
18       17      4445
5        16      4410
15       21      4168
10       22      3742
0         6      3273
11       23      3189
22       24      2700
23       25      2090
21        5      1957
26       26      1653
16       27      1177
4         4      1010
24       28       901
28       29       564
31        3       408
27       30       378
29       31       242
30       32       133
35        2       128
32       33        72
34       34        52
36       35        32
25       36        20
33        1        20
38       37         8
41       38         4
37       41         3
40       40         3
39       39         1
shape: (42, 2)
┌─────────┬──────────┐
│ c_count ┆ custdist │
│ ---     ┆ ---      │
│ u32     ┆ u32      │
╞═════════╪══════════╡
│ 0       ┆ 50004    │
│ 10      ┆ 6668     │
│ 9       ┆ 6563     │
│ 11      ┆ 6004     │
│ 8       ┆ 5890     │
│ …       ┆ …        │
│ 37      ┆ 8        │
│ 38      ┆ 4        │
│ 41      ┆ 3        │
│ 40      ┆ 3        │
│ 39      ┆ 1        │
└─────────┴──────────┘

Are you running this from your fork of Narwhals or from the latest install?

I can replicate it. Issue seems to be the removal of
if isinstance(left_on, str):
    left_on = [left_on]
if isinstance(right_on, str):
    right_on = [right_on]

which then result in "zipping" characters instead of column names in
for left_key, right_key in zip(left_on, right_on):
    ...

interesting thanks - how are you running it? latest commit? latest pip version? i'm confused as to why it's not showing up for me

Latest commit via the editable install

ok i just recreated my env and I can reproduce it, thanks both! not sure what had gone wrong before
ok looks like the issue is that our tests all had single-letter variable names and so this was never caught 🤦 sorry about that, i should've picked up on it during review, fix incoming

fixed by #930
thanks a tonne both for reporting and triaging it so quickly 🙏


No context of what DAO I’m in

Issue 6 https://github.com/near-daos/astro-ui/issues/91

not able to reproduce


[Localization] Configure file associations to languagues setting not localized

VSCode Version: 0.10.12-alpha --locale=fr (or any locale)
OS Version: Windows 10

Steps to Reproduce:

Launch VS Code from command prompt with --locale=fr
Open workspace settings file
Scroll to line 123

Actual:
Non-localized setting for Configure file associations

Expected:
Localized setting text. It is bracketed in --locale=pseudo with double vowels.

@vsccarl please test with PSEUDO, those strings are not localized yet.

@bpasero I saw before that in PSEUDO mode, bracketed and double vowel strings should be translated.
Here it is in PSEDUO:


PAL caching subtitle streams issue

I might be wrong on how PAL is supposed to behave, but I believe how PAL caches streams to ensure that it does not need to update future episodes again when users play the episodes again might be causing some issues when switching back and forth between languages.
Example flow of reproducing the issue
(you can pick any two different languages to test this out)

Start PAL container fresh with no cache and empty plex.cache.default_streams.
Open show's first episode, setting subtitles language to Arabic
a. This will cache this episode's stream to dict key and value /library/metadata/25911: (155623, 155627).
b. All future episodes are set to be Arabic. Thanks PAL!
Actually...nvm I want the show subtitles to be in Japanese.
Open show's first episode again, setting subtitles language to Japanese
a. PAL will check here, is the episode's stream the same as the cached one? No it's not since it's now Japanese and a different value; /library/metadata/25911: (155623, 155641), so PAL continues to change the show's subtitles!
b. ❗ PAL will NOT cache this new set Japanese stream for this episode here. Because we use the method setdefault, which if a key is already set in a dict, will not override it's value. Since we already set this episode key in step 2a., then the cache will not be overridden, and the cached value for this episode stream is still Arabic; /library/metadata/25911: (155623, 155627).
c. Even though it didn't cache to Japanese, PAL code continues and all future episodes subtitles are set to be Japanese. Thanks PAL!
Actually...nvm again! I want the show subtitles to be in Arabic again.
Open show's first episode, setting subtitles language back to Arabic
a.  ❗ Again, PAL will check here, is the episode's stream the same as the cached one? Well, since we never updated the cache to be Japanese, the cached value is still Arabic; /library/metadata/25911: (155623, 155627), and it thinks the stream is unchanged, and returns early!
b. ❗PAL will NOT set any of the future episodes subtitles language to Arabic.

This essentially means since the cache has the langauge Arabic stream value stuck in it for this episode and it cannot be updated, we can't switch the show's subtitles back to Arabic, unless we recreate the container and wipe the cache.
Proposed fix:
I think the issue is when we cache the episode's stream value here. We use setdefault instead of a method that should always override the cached value like:
replace this line with:
plex.cache.default_streams[item.key] = pair_id

cc @JourneyOver

Hmm this bug seems to only be an issue when the update_strategy is set specifically to next instead of all as I did a test running through your example flow above with my config file which had it set to all and I wasn't experiencing this issue, so decided to do the test again with it set to next and ran straight into the issue of it not switching the subtitle language back.
I'll do a quick test with the proposed fix to see if it affects the update_strategy any when switching between the two different settings and if it doesn't then I'll push up the changes so you can test and see if everything is working correctly on your end and then we can call this fixed or not depending on things.

Have pushed up a new build (which should be done building here shortly) with the proposed change, I did a small test beforehand and it didn't seem to cause any adverse affects, but feel free to test and confirm/deny if everything is working okay on your end.
I've also gone ahead and allowed issues over on my repository now as well.

@JourneyOver Good point I forgot to mention I was in next strategy! I'll test the both modes as well.
Thanks as always for these quick fixes 💯

It's all good :P thankfully I remembered that some of my settings are different than the default settings, so I decided to check it out with default settings which for sure ran into the issue.
And it's not a problem, gotta thank you for the proposed code changes as well as it makes getting these fixes out quickly :)


Revert "Fix #286 - Linked List: change pop and shift to return option"

Reverts exercism/xfsharp#289

Thanks @ErikSchierboom!


select files during commit process

Closes #8

@mpetojevic looks good, but I have two issues in the branch:

When I access a submission, which has a manualgrade folder (aka I already pulled the submission), I receive the following error: Cannot read properties of undefined (reading 'id')
Relevant Jupyterhub logs:

    Traceback (most recent call last):
      File "/home/fjaeger/miniconda3/envs/migration/lib/python3.10/site-packages/tornado/web.py", line 1786, in _execute
        result = await result
      File "/home/fjaeger/miniconda3/envs/migration/lib/python3.10/site-packages/jupyter_server/services/contents/handlers.py", line 121, in get
        model = await ensure_async(
      File "/home/fjaeger/miniconda3/envs/migration/lib/python3.10/site-packages/jupyter_core/utils/__init__.py", line 182, in ensure_async
        result = await obj
      File "/home/fjaeger/miniconda3/envs/migration/lib/python3.10/site-packages/jupyter_server/services/contents/filemanager.py", line 877, in get
        raise web.HTTPError(404, "No such file or directory: %s" % path)
    tornado.web.HTTPError: HTTP 404: Not Found (No such file or directory: lectures/lect1/manualgrade/5/31)

My versions:
grader-service                    0.4.1
jupyterhub                        4.0.2
jupyterlab                        4.0.10

The info dialog title for the commit feature is not representative and the last sentence is missing a dot:


[release-1.15] Manual cherry-pick of #40717

Please provide a description of this PR:
Fixes https://github.com/istio/istio/issues/40731
#40717 was a dependabot update updating the helm dependency for a security issue. This change is simply a go get<EMAIL_ADDRESS>and a go mod tidy.

@ericvn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio_release-1.15
01598319976389028c6b5f44d333dd7ddff5a750
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ericvn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio_release-1.15
01598319976389028c6b5f44d333dd7ddff5a750
link
true
/test unit-tests-arm64

gencheck_istio_release-1.15
01598319976389028c6b5f44d333dd7ddff5a750
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Additional info on state changes of buttons

DigitalIO and Navigators
This little improvement enables you to see which buttons was pressed in your callback function.
Therefore you need only one callback function for all your buttons.
e.g.:
from baxter_interface.digital_io import DigitalIO
def updateState(*args):
    print "button pressed", args
button = DigitalIO("left_shoulder_button")
button.state_changed.connect(updateState)

Camera
The current implementation raises an exception, if the camera is currently closed.  I think it is useful in the camera controller to check, if the camera is opened or closed. Also in the past the service call to the camera list yielded all 3 cameras, which is not the case any more. Therefore I recommend to change the condition of the if in the init

@sven-bock you've got a whole wad of master commits in here. I see why: you're using the HumaRobotics master branch. You've got a couple options to fix this, either

Make a new branch off of your HumaRobotics master branch, do a git interactive rebase of that branch including of your 3 commits onto RethinkRobotics/baxter_interface-development and push that up as a branch in your repo, then submit a pull request from that branch.

The simpler to explain and execute, (but not the cleanest in terms of git history) would be make a clean branch off of RethinkRobotics/baxter_interface-development and do a git cherry-pick of your three commits onto that branch. Then perform a pull request off that branch.

Sorry for the extra work. Let me know if you need any additional guidance on cleaning up your branch. I promise, it's not usually this hard to submit a contribution :)

superseded by #62


Fire an event when new location url has changed

I have a use case where I would like to be notified when a location change has occurred so I can proceed with some computation based on the new url/pathname.
I tried to use event listeners on window.location for 'hashchange' or 'popstate' but I did not get any events. I noticed that there is a 'getCurrentView' function available but this is not event.
Ideally I would like to either add an event listener on the  SimplrRouter._instance to be notified about the new view or maybe a custom event is fired from the window object so its available without import.
I see that we can add a middleware to the instance but from what I understand those are meant to add functionality to the router functionality and not to get notified when the location has changed.
Any ideas how I can get notified about the new location url ?

Heya,
There actuall is some events that get fired as navigation happens but looking at it, they are quite outdated.
I'll take a look at them tonight and try to time them better and provide more information.
Are there other cases you would want to listen to than "url changed" ?

Basically I have a list of links and I am trying to figure out which link to style as selected based on the currently loaded view.
The dispatcher sendEvent function is it called after history is updated ?

Currently it's called after it is called.
I'm thinking of adding a "simplr-router-history-updated" -event that gets dispatched AFTER the history has been updated. You could then plug into that

That sounds great. I just changed and vaadin router is doing something similar. It fires an custom event 'vaadin-router-location-changed' and you can attach a window event listener from any web component.

Yeah that's exactly what I was planning on doing. I'll try to get it pushed out today. I'll inform you about it as I publish it.
If some other events come to mind that might be of use, let me know.
In the mean time... You can also get the current view data using the API of simplr router
import { getCurrentView } from "@simplr-wc/router";

getCurrentView();

The latest Release candidate (rc9) will contain an event called simplr-router-navigated which triggers in window scope when a navigation has finished and updated the history of the page.
You can read more about it here https://simplr.github.io/simplr-router/docs/api/events/

This works as expected. Thanks for the releasing so fast 👍


Add new app: exotic clocks

I have read the CLA Document and I hereby sign the CLA

@matslina Just wandering if I should remove the python file for a review or is it in progress?

yeah that'd be good. in general, the review bot doesn't like subdirectories, and since (I believe) the graphics are all in the starlark file they don't really need to be in here either.

sorry for the delay, I somehow kept missing this one

No problem!


Problem with the aspect ratio of the images by resize_image

Hi,
Issue:
The images on my website's sidebar are all with a wrong aspect ratio after resizing. The only fix for this is to disable rewrite_images but it is one of the best speed bumps, so I am looking how I can fix the aspect ratio.
Screenshots:
resize_image disabled: http://prntscr.com/8lcfdj
resize_image enabled: http://prntscr.com/8lcfux
System:
I am using mod_pagespeed which is running on Apache server trough the Nginx module.
OS: CentOS 6.4 (64 bit)
Nginx pagespeed: ngx_pagespeed-release-<IP_ADDRESS>-beta
Apache pagespeed: mod_pagespeed-<IP_ADDRESS>-0.x86_64
Other versions: https://supplementswatch.com/info.php
Rules:
pagespeed RewriteLevel CoreFilters;
pagespeed EnableFilters inline_google_font_css;
pagespeed EnableFilters remove_comments;
pagespeed EnableFilters extend_cache;
pagespeed DisableFilters insert_image_dimensions;

Anyone?

Looking at the html I see:
<img itemprop="image" class="widget_review_screenshot_img photo" src="..." alt="Bulking Stack" width="320" height="180">

The image itself is 680x300.
There's also css with:
media="all"
img, video, object {
    ...
    height: auto!important;
    ...
}

That height directive ends up in context at setting the image height to 66px, and then the browser maintains the 680x300 ratio of the source image and sets the width to 149.9px.
The problem is PageSpeed doesn't understand your whole css setup, and instead just goes with the width="320" height="180" in the html.  But 320x180 is 16:9 while 680x300 is closer to 16:7, so this gets the aspect ratio wrong.
To fix this, you could either remove the width and height tags from the image, which the browser is just ignoring, you could turn off the resize_images filter, or you could turn on the resize_rendered_image_dimensions filter that uses beaconing to handle css problems like yours.
More doc: https://developers.google.com/speed/pagespeed/module/filter-image-optimize#resize_images

Thanks


Issue using image cache for new agents

I'm trying to reuse a previously built docker image so new agents don't need to build the image from scratch every time (docs), but that doesn't seem to be working as expected.
It seems like the image is being pulled, and the docker-compose file is being modified with a cache_from that should make it use the image, but it always builds the image again:

Here's the config I have:
steps:
  - label: ":docker: Build image"
    plugins:
      - docker-compose#v4.13.0:
          cli-version: 2
          build: rails
          image-repository: us-east4-docker.pkg.dev/myrepo/core-buildkite
          cache-from:
            - rails:us-east4-docker.pkg.dev/myrepo/core-buildkite:${BUILDKITE_BRANCH}
            - rails:us-east4-docker.pkg.dev/myrepo/core-buildkite:latest

  - wait

  - label: ":docker: Push to final repository"
    plugins:
      - docker-compose#v4.13.0:
          cli-version: 2
          push:
            - rails:us-east4-docker.pkg.dev/myrepo/core-buildkite:${BUILDKITE_BRANCH}
            - rails:us-east4-docker.pkg.dev/myrepo/core-buildkite:latest

  - wait

  - name: "Test image cache"
    plugins:
      - docker-compose#v4.13.0:
          run: rails
          cli-version: 2
    command: echo 'it works'

I can see the image is being correctly pushed to the remote registry, and it's also pulled as expected.
I can even see that when I'm pushing this image, all the layers already exist, so I'm not sure why they are not being used when I'm building the image:

Am I missing something?

I was missing the BUILDKIT_INLINE_CACHE build arg:
steps:
  - label: ":docker: Build image"
    plugins:
      - docker-compose#v4.13.0:
          cli-version: 2
          build: rails
          image-repository: us-east4-docker.pkg.dev/myrepo/core-buildkite
          args:
            - BUILDKIT_INLINE_CACHE=1 # <-- This
          cache-from:
            - rails:us-east4-docker.pkg.dev/myrepo/core-buildkite:${BUILDKITE_BRANCH}
            - rails:us-east4-docker.pkg.dev/myrepo/core-buildkite:latest


Flag for multiple args nests array inside array

Trying to write a script that takes multiple flags in a couple places, as well as multiple arguments.
Usage:
  script [options] [--ruleset FILE]... [RULE]...
  script [options] [--skip-rules CODE] [RULE]...

Command Options:
  --repository NAME       Name of repository to run rules against.
  -r, --ruleset FILE      Include ruleset found in FILE.
  -s, --skip-rules CODES  Comma separated list of rules to be skipped.

However, when I run
script --ruleset file/008.rb --ruleset file/009.rb --ruleset file/010.rb --skip-rules 30,40,50
it produces the following result:
--ruleset"=>
  [["file/008.rb",
    "file/009.rb",
    "file/010.rb"]],
 "--skip-rules"=>[["30,40,50"]],

I don't see a reason why those arrays would be nested, but it doesn't make sense to me and seems like a bug.

@ThatGerber This indeed looks like a bug. I suspect #26 may fix it.

Looks like this should be fixed by 0.6.0 on rubygems


Add snippet-bot config

Needs an org admin to enable on repo. Install link: https://github.com/apps/snippet-bot


Fix release script GCS bucket for the release.yaml

Fixes #595
Changes

Submitter Checklist
These are the criteria that every PR should meet, please check them off as you review them:

[ ] Has Docs included if any changes are user facing
[ ] Has Tests included if any functionality added or changed
[ ] Tested your changes locally (if this is a code change)
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including functionality, content, code)
[x] Has a kind label. You can add a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[ ] Release notes block below has been updated with any user-facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[ ] Release notes contain the string "action required" if the change requires additional action from users switching to the new release

Release Notes
NONE

@enarha This should fix the #595
/release-note-none

/test pull-tekton-results-integration-tests

/approve

/lgtm


function will returns false if the username doesnot start with an letter

Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

The change was done to check first character, any character not a-z then it will return false


Use scurl instead of wget and plain curl.

https://www.whonix.org/wiki/Secure_Downloads
The link refers to problems with wget and how to improve curl. This does not work with self signed certificates or any http website, including onion services.
curl --tlsv1.3 --proto =https --location --remote-name

alias scurl="curl --tlsv1.3 --proto =https --location"

Ocurrences
https://github.com/rootzoll/raspiblitz/search?q=wget
https://github.com/rootzoll/raspiblitz/search?q=curl

Sounds worth a deeper look ... but kicking it up to v1.8.1 for now because of prio.

Funny that keybase.io uses TLS 1.2, which breaks the above. (was trying to import rootz keys)
curl --tlsv1.2 --proto =https https://keybase.io/rootzoll/pgp_keys.asc works
from curl --help all
 -1, --tlsv1         Use TLSv1.0 or greater
     --tlsv1.0       Use TLSv1.0 or greater
     --tlsv1.1       Use TLSv1.1 or greater
     --tlsv1.2       Use TLSv1.2 or greater
     --tlsv1.3       Use TLSv1.3 or greater

--tlsv1.2 will be the default, if encountering with any trouble, even blitz does not support 1.1 https://github.com/rootzoll/raspiblitz/blob/a0fd288e553678d0816d682e968a2eea0d165a9b/home.admin/assets/nginx/snippets/ssl-params.conf#L7
but maybe use 1.1?


Support TF SavedModel checkpoints

Most likely depends on https://github.com/r-three/git-theta/issues/163

I'm close to getting this working, the main issue is when smudging.
https://github.com/blester125/git-theta/pull/new/feat/extra-info
It also depends on what we want to happen if they change model code which changes the saved model, should that trigger a diff?


Newest apk can't load files

The newest apk (Initial Installer + Droid Drip) can't load the online data but older apk versions can.

So you're trying to play online here and it isn't working? Just to clarify.

Oh I see what weirdness is going on here. For some reason android hates showing the 100% completion for the game file download, and I have no idea why. Literally when I go into the code and cap the update message to only show from 0% to 99%, it works, so uh, I guess that's what I'm going to do.

I'll go ahead and edit the current release with a hotfix for this issue.

So you're trying to play online here and it isn't working? Just to clarify.

Sry for the late answer, I was trying to update for offline play. I will leave feedback here once I tried the fixed version.

I'll go ahead and edit the current release with a hotfix for this issue.

I can confirm that the fix worked


Fitting - minor bugs and possible tweaks

Provide a general description of the issue or problem.
Testing the fitting for a variety of NuSTAR sources and finding similar results to XSPEC . But also finding a number of inconsistencies, minor bugs and things that need tweaked:

For default plotting y-axis label is wrong - not a "Count Spectrum [cts/s/keV]" as units count/s/keV. Just change label to units, i.e. count/s/keV?
For default plotting, add option to turn off annotate plot with fit results, i.e. _plot_params in https://github.com/sunpy/sunxspex/blob/818ae4f7f47db1c3049f02a57a2aaf8ea2eac963/sunxspex/sunxspex_fitting/fitter.py#L3005
Can easily extract spectrum and fits for own manual plotting but:

resid=spec.plotting_info['spectrum1']['residuals'] doesn't return the residuals, but a modified version of them for the internal plotting - every element repeated so need to do spec.plotting_info['spectrum1']['residuals'][::2].
Naming of things a bit inconsistent particularly in terms of plurals, i.e. counts, count_rates, count_channel_error, count_rate_errors.
Also have count_rate in loaded_spec_data but  count_rates in plotting_info.
What is called count_rates (and count_rate) is actually count/s/keV - probably call it flux though flux usually count/s/keV/cm2?
It would help if each had a descriptor saving what they are, or include actual units?

Not clear the units/scaling of the fit parameters. So add descriptor or units when returning something like spec.params["T1_spectrum1"]?

Also confusing fit uses different thermal params from actual model thermal.thermal_emission, i.e. MK and 1e26 cm^-3 against keV and cm^-3. Is this something to do with fit optimisation?

Need a simpler (and more) examples, just doing basic fit but also show how to extract fit params and other stuff.

i think these are really good points--refactor should make things clearer.
keeping units attached to the model parameters might be the best way to do that but also might slow things down.


Add favorite locations

Hi j2rong,
I really love FakeLocation is a neat and really useful tool. However there is one thing your app is missing: a feature to search locations instead of having to pin locations zooming in a map OR a feature to save places as favorites and being able to use them later. If you could add one of these features, you would save users a lot of time.
Please consider adding this.
Thanks for your hard work!

Nice idea!

<EMAIL_ADDRESS>


chore(deps): updates packngo to v0.30.0

updates packngo to v0.30.0
adds Rescue to packngo mock DeviceService interface

Codecov Report
Patch coverage has no change and project coverage change: -0.11 :warning:

Comparison is base (b23f9ea) 60.97% compared to head (219776f) 60.87%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #339      +/-   ##
==========================================
- Coverage   60.97%   60.87%   -0.11%
==========================================
  Files          90       90
  Lines       17942    17942
==========================================
- Hits        10941    10923      -18
- Misses       6698     6713      +15
- Partials      303      306       +3

see 3 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.